Decoding KNeighboursClassifier model from Sklearn fails

I upgraded to version 0.96 after following this issue #147 and KNeibhoursClassifier model still doesn't decode. Here's what I tried-
    import json
    import jsonpickle
    import jsonpickle.ext.numpy as jsonpickle_numpy
    rfc = KNeighborsClassifier()
    rfc.fit(X,Y)

    jsonpickle_numpy.register_handlers()
    p = jsonpickle.pickler.Pickler()
    u = jsonpickle.unpickler.Unpickler()
    j = jsonpickle.json.dumps(p.flatten(rfc))
    json.dump(j, open(pickle_path, 'w'))
    j = json.load(open(pickle_path, 'r'))
    rfc = u.restore(jsonpickle.json.loads(j))

and the error-
TypeError: 'dict' object is not callable
What should be done ?

Thanks for the report, much appreciated.
What is X and Y in your example above?  I'm only a numpy dabbler, so this example looks incomplete to me, but the folks who wrote most of the numpy extension will hopefully see this issue and help out too.
Mentioning @EelcoHoogendoorn and @marscher since they're much more familiar with the numpy extension, and probably a lot smarter than me in all things numpy too ;-)
Instead of pasting the  problem code into this issue, can you please add your testcase to the tests/numpy_test.py file?  That'll make this process a million times easier, and it'll make it very clear what we need to fix this issue.  The easiest way to run the test suite is to run make test in a virtualenv that has requirements-dev.txt installed.
With the given example above, the calling sequence might have a buglet.  I'll try to walk through it here below.
This first part looks fairly boilerplate.  We could use the top-level jsonpickle.loads/dumps functions but this is using the lower-level API with picklers and unpicklers instead:
jsonpickle_numpy.register_handlers()
p = jsonpickle.pickler.Pickler()
u = jsonpickle.unpickler.Unpickler()

At this point we have a pickler and unpickler.  Next we create a json string j:
j = jsonpickle.json.dumps(p.flatten(rfc))

The following line looks like a buglet:
json.dump(j, open(pickle_path, 'w'))

The buglet above is we're using the json module to dump a string that's already json-encoded.  I think this string should be written to disk directly, either with a raw filehandle, or by using json.dump on the result of p.flatten() rather than the result of json.dumps(p.flatten(...)).  This line might explain part of the issue, but technically it actually should be okay because...
j = json.load(open(pickle_path, 'r'))

The line above is now undoing the double json encoding above.  Technically that should work, but it is kinda confusing.  Did you truly intend to double-json encode the result from jsonpickle?
That said, this should work because the double-encoded json string gets restored by the call to json.load(...) which is then handed back to jsonpickle.json.loads(j), but just wanted to mention the nested json encoding that's going on.
I suspect that we could simplify that part of the test case down to something simpler like this below:
json_str = jsonpickle.dumps(rfc)
rfc_clone = jsonpickle.loads(json_str)

I only mention this because it doesn't look like the raw dict is used in a way that needs explicit picklers and unpicklers, but perhaps they're used by other parts of the code that's not relevant to this issue.
Let me know about getting a complete test case (regarding the unknown X and Y variables above) and hopefully we can make the numpy support even more complete.
The best part about adding the test case to numpy_test.py is that this use case will be forever in our test suite.  That's a pretty strong guarantee that the use case will keep working in the future.

hi, which line is actually failing (please show us the stack trace). This error message is popping up, if there is an unpickle-able type involved. By not-pickleable I mean, there is no get/setstate, reduce etc. protocol plus it is not a primitive type. Since you have registered the numpy handlers, this should actually work.
When I used jsonpickle, I have written my own handler, so I can not really tell how good it is (eg. does it support exotic dtypes, like structured arrays etc.?).
Are you familiar with pdb? You can wrap your script in an try: .... except block and within except call: pdb.post_mortem() to gain insight which object is causing this TypeError.
If you need more info on this: https://docs.python.org/2/library/pdb.html#pdb.post_mortem

Here's the stacktrace
Traceback (most recent call last):
  File "D:/backup/PycharmProjects/test/ML/ferrero_work.py", line 115, in <module>
    rfc = u.restore(jsonpickle.json.loads(j))
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 138, in restore
    value = self._restore(obj)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 192, in _restore
    return restore(obj)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 304, in _restore_object
    return self._restore_object_instance(obj, cls)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 369, in _restore_object_instance
    return self._restore_object_instance_variables(obj, instance)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 431, in _restore_object_instance_variables
    instance = self._restore_state(obj, instance)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 436, in _restore_state
    state = self._restore(obj[tags.STATE])
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 192, in _restore
    return restore(obj)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 491, in _restore_dict
    data[k] = self._restore(v)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 192, in _restore
    return restore(obj)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 213, in _restore_reduce
    reduce_tuple = f, args, state, listitems, dictitems = map(self._restore, reduce_val)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 192, in _restore
    return restore(obj)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 476, in _restore_tuple
    return tuple([self._restore(v) for v in obj[tags.TUPLE]])
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 476, in <listcomp>
    return tuple([self._restore(v) for v in obj[tags.TUPLE]])
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 192, in _restore
    return restore(obj)
  File "C:\Users\mohit.badwal.NOTEBOOK546.000\Anaconda3\lib\site-packages\jsonpickle\unpickler.py", line 222, in _restore_reduce
    stage1 = f(*args)
TypeError: 'dict' object is not callable

I just hit that bug. I was able to narrow it down where it crashed. 100% reproducible.
import jsonpickle
from enum import Enum
class Bar:
class Buz(Enum):
WTF = 1
jsonpickle.decode(jsonpickle.encode(Bar.Buz.WTF))
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/init.py", line 155, in decode
return unpickler.decode(string, backend=backend, keys=keys, classes=classes)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 26, in decode
return context.restore(backend.decode(string), reset=reset, classes=classes)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 138, in restore
value = self._restore(obj)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 192, in _restore
return restore(obj)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 304, in _restore_object
return self._restore_object_instance(obj, cls)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 369, in _restore_object_instance
return self._restore_object_instance_variables(obj, instance)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 419, in _restore_object_instance_variables
instance = self._restore_from_dict(obj, instance)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 387, in _restore_from_dict
value = self._restore(v)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 192, in _restore
return restore(obj)
File "/.virtualenvs/pywiktionary/lib/python3.6/site-packages/jsonpickle/unpickler.py", line 222, in _restore_reduce
stage1 = f(*args)
TypeError: 'dict' object is not callable

I am facing same issue while doing something like this
json_kd = jsonpickle.encode(kdtree)
decoded = jsonpickle.decode(json_kd, classes=cKDTree)
and getting error as
TypeError: 'dict' object is not callable

I can reproduce dict object is not callable with the following code:
import jsonpickle
from enum import Enum

class WrapperClass(object):
    class myenum(Enum):
        Hello = 1

mydict = {
    WrapperClass.myenum.Hello: 'test'
}

print(mydict)
print(jsonpickle.dumps(mydict, keys=True))
print(jsonpickle.loads(jsonpickle.dumps(mydict, keys=True), keys=True))

I don't know if this is the same issue @mohitbadwal is noticing with numpy.

Hi, I am also facing same issue, interestingly if I switch to jsonpickle=0.7.2 everything works fine.

There seem to be at least two separate issues in this thread (and both might be different from the OP's issue...).
For the examples provided by @ldesegur  and @edjubuh, the problem is that jsonpickle does not automatically handle nested class definitions, the class has to be given explicitly as an argument to classes when decoding. For the example that @kapilkd13 provided, the problem is that jsonpickle does not correctly handle builtin functions.
Here's another test case that shows the root of the first problem:
import jsonpickle

class Outer(object):
    class Inner(object):
        pass

print(jsonpickle.encode(Outer.Inner))

The output is {"py/type": "__main__.Inner"}>, showing that jsonpickle did not correctly encode the class name. This is because the importable_name function in jsonpickle.util assembles the name from the module name (__main__) and the class name (Inner), which does not include the name of the outer class. This is easily fixable for Python >=3.3 which provides an attribute __qualname__ to get the fully qualified name, but unfortunately needs complex codes that inspects the code file (see e.g. https://github.com/wbolster/qualname). For Python >=3.3, this is the necessary change:
diff --git a/jsonpickle/util.py b/jsonpickle/util.py
index 8566450..dff7dac 100644
--- a/jsonpickle/util.py
+++ b/jsonpickle/util.py
@@ -507,7 +507,7 @@ def importable_name(cls):
     True

     """
-    name = cls.__name__
+    name = cls.__qualname__
     module = translate_module_name(cls.__module__)
     return '%s.%s' % (module, name)

print(jsonpickle.encode(Outer.Inner)) then correctly prints {"py/type": "__main__.Outer.Inner"}.
However, this does not fix the decoding, because the decoding code assumes that only the last part of the "importable name" is a class name, and the rest is a module; in this case, it would therefore try to import the Inner class from the __main__.Outer module which obviously fails.
It might be cleaner to store the module and class information separately in a tuple, but the following change makes it simply try all options:
diff --git a/jsonpickle/unpickler.py b/jsonpickle/unpickler.py
index 283d54b..06864e9 100644
--- a/jsonpickle/unpickler.py
+++ b/jsonpickle/unpickler.py
@@ -606,13 +606,21 @@ def loadclass(module_and_name, classes=None):
         except KeyError:
             pass
     # Otherwise, load classes from globally-accessible imports
-    try:
-        module, name = module_and_name.rsplit('.', 1)
-        module = util.untranslate_module_name(module)
-        __import__(module)
-        return getattr(sys.modules[module], name)
-    except (AttributeError, ImportError, ValueError):
-        return None
+    names = module_and_name.split('.')
+    # First assume that everything up to the last dot is the module name,
+    # then try other splits to handle classes that are defined within
+    # classes
+    for up_to in range(len(names)-1, 0, -1):
+        module = util.untranslate_module_name('.'.join(names[:up_to]))
+        try:
+            __import__(module)
+            obj = sys.modules[module]
+            for class_name in names[up_to:]:
+                obj = getattr(obj, class_name)
+            return obj
+        except (AttributeError, ImportError, ValueError) as ex:
+            continue
+    return None

With these changes, my example provided above and the examples provided by @ldesegur and @edjubuh work fine.
For the second issue, the problem is relatively obvious, the output of this code:
import jsonpickle

print(jsonpickle.encode(dir))

is {"py/object": "__builtin__.builtin_function_or_method"} which is of course impossible to convert back to the dir function.
Here are the changes that seem to fix this issue:
diff --git a/jsonpickle/util.py b/jsonpickle/util.py
index dff7dac..ed6bd8d 100644
--- a/jsonpickle/util.py
+++ b/jsonpickle/util.py
@@ -110,7 +110,8 @@ def is_object(obj):
     False
     """
     return (isinstance(obj, object) and
-            not isinstance(obj, (type, types.FunctionType)))
+            not isinstance(obj, (type, types.FunctionType,
+                                 types.BuiltinFunctionType)))

 def is_primitive(obj):
@@ -277,7 +278,7 @@ def is_module_function(obj):
     """

     return (hasattr(obj, '__class__') and
-            isinstance(obj, types.FunctionType) and
+            isinstance(obj, (types.FunctionType, types.BuiltinFunctionType)) and
             hasattr(obj, '__module__') and
             hasattr(obj, '__name__') and
             obj.__name__ != '<lambda>')

The above example now prints {"py/function": "__builtin__.dir"} which can be converted back to dir with jsonpickle.decode.
I'd be happy to provide these changes as a pull request (adding/adapting the tests), but I'm unsure what to do about the fact that __qualname__ only exists in Python versions starting at 3.3. Also, is it a backward-compatibility issue if the output for nested classes is now different (e.g. test_can_serialize_inner_classes fails with my changes)?

This sounds great, yeah I'd totally merge these changes.
Regarding the __qualname__, my suggestion would be to check hasattr(cls, '__qualname__')) before using it, and when it's not available fallback to __name__ as before.  That way we're backwards compatible, and folks using newer stuff can benefit.


Unit Testing

TL;DR: CodeIgniter doesn't make it easy to do unit testing. All database code needs to get refactored.
PHPUnit is now installed in our project, but we can't make proper unit tests yet because we don't have a stub database. In order to make this work we're going to have to create a DatabaseAccess interface, then create a SQLDatabase object and a StubDatabase object that use that interface. We shouldn't be making calls directly to the database in the controllers anyway like I've seen in a few places.
Any comments or suggestions?

I've posted a sample set of unit tests using a stub database for the Users.php model. Still have to create an interface to link the real database object with the stub database object (CodeIgniter doesn't like interfaces). Hopefully you guys can use this as an example.

We've completed changed the way we do testing, ignore all above.


Account specific audio source and playback

Add support for specifying audio source and player per account.
The global config will be used by default.
Syntax:
;audio_source="module,device"
;audio_player="module,device"

Example with multiple accounts/devices:
<sip:user1@example.com>;audio_source=alsa,plughw:0,1
<sip:user2@example.com>;audio_source=alsa,plughw:0,2
<sip:user3@example.com>;audio_source=alsa,plughw:0,3

Please review and test.

Nice!
But will there also be account specific playback device added?

thanks for your feedback.
I added also audio_player per account, same syntax:
;audio_player="alsa,default"

This works fine now! Thank you!


Update Tor to v13.5.7

@yahiheb Because I made the code few weeks ago and <IP_ADDRESS> was not released at the time.
I will update again in this PR

Thanks @yahiheb


 Please admin, close this issue and ban this kind of shitty user. His comment don't deserve more than the time you take to click on ban.

Thanks for your review, I will try my best to make the script better at the next update. Love from your friendly developer!

bro you know u can increase the threads and multiple by yourself , idk whats wrong with you but i can actualy use 50K threads on this script .


byacc: 20170709 -> 20180510

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/byacc/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 20180510 with grep in /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510
directory tree listing: https://gist.github.com/328d2b1900cfb911e7567cabf95d80c3
du listing: https://gist.github.com/cc161fe31193017387ae25b64829aaf2

meta.description for byacc is: '"Berkeley YACC"'.

Success on x86_64-linux (full log)
Attempted: byacc
Partial log (click to expand)
/nix/store/jy9knxp7nmw80jkf932axrs1b4p9k4hi-coreutils-8.29/bin/install -c yacc /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510/bin/`echo yacc        | sed 's/$//'       |sed 's,x,x,'|sed 's/$//'`
/nix/store/jy9knxp7nmw80jkf932axrs1b4p9k4hi-coreutils-8.29/bin/install -c -m 644 ./yacc.1 /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510/share/man/man1/`echo yacc.1| sed 's/1$//'|sed 's,x,x,'|sed 's/$/1/'`
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510
shrinking /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510/bin/yacc
gzipping man pages under /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510/bin
patching script interpreter paths in /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510
checking for references to /build in /nix/store/z8i5f145zpz750hvcbjlncy8kqak8zv2-byacc-20180510...

Success on x86_64-darwin (full log)
Attempted: byacc
Partial log (click to expand)
mkdir -p /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510/bin
mkdir -p /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510/share/man/man1
/nix/store/5lkrw9dnsgy62qm1ampvww1c5n1pdm4b-coreutils-8.29/bin/install -c yacc /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510/bin/`echo yacc        | sed 's/$//'       |sed 's,x,x,'|sed 's/$//'`
/nix/store/5lkrw9dnsgy62qm1ampvww1c5n1pdm4b-coreutils-8.29/bin/install -c -m 644 ./yacc.1 /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510/share/man/man1/`echo yacc.1| sed 's/1$//'|sed 's,x,x,'|sed 's/$/1/'`
post-installation fixup
gzipping man pages under /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510/share/man/
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510/bin
patching script interpreter paths in /nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510
/nix/store/007a2iam04iav56ss8bsfjxb45x44rdm-byacc-20180510

Success on aarch64-linux (full log)
Attempted: byacc
Partial log (click to expand)
/nix/store/k6pzkhdivfd205wg942m98n198pklhg3-coreutils-8.29/bin/install -c -m 644 ./yacc.1 /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510/share/man/man1/`echo yacc.1| sed 's/1$//'|sed 's,x,x,'|sed 's/$/1/'`
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510
shrinking /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510/bin/yacc
gzipping man pages under /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510/share/man/
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510/bin
patching script interpreter paths in /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510
checking for references to /build in /nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510...
/nix/store/0f0zqi48rj7frjx5wdj1mr2i73y8pnm2-byacc-20180510


Linkedin share issue

https://github.com/brendt/rfc-vote/blob/98217934b0db4b2e8ed4824d3fe579a43a5c9062/resources/views/components/argument-card/share-links.blade.php#L42
i have fix this so many time and and code get overwrite IDK what happen
this is fix of this https://github.com/brendt/rfc-vote/pull/179

Looks like your PR for README https://github.com/brendt/rfc-vote/pull/204 had dirty history. It included a this change and then a revert of it and it was merged after #179 was merged.

You should always keep your fork up to date:

And when you create a PR, always do it from a new branch from main.
At this point you'll have to re-submit a new PR with the changes you did in #179

okay so let me re-submit then thanks @Plytas


backup missing wireguard/setupVars.conf

When migrating my pivpn settings to a new server the backup did(does) not contain all used settings.

pivpn -bk
fresh install of Rapberry PI OS lite (32 bit)
installed PiVPN via curl ... bash
restored files as described in https://docs.pivpn.io/wireguard/#migrating-pivpn-wireguard

Behaviour exhibited:

able to connect,  pivpn -c shows an active client,
client does not connect to the internet

Checked with pivpn -d, no change
Discovered that the pivpnNET (in wireguard/setupVars.conf) was using a different subnet from the original machine.
updated wireguard/setupVars.conf, ran pivpn -d which updated "Iptables MASQUERADE rule", vpn now works for existing clients
If the backup had contained the setupVars.conf and the guide included a couple of extra step frustrations could have been avoided :)

Pre-archive closing, more information here


DEBUG logging in the extension

Whenever debug is true, the extension should also collect logs. Where do we store it? Maybe in
local storage and can serve it to the user whenever asked for?
We should also log the URL visited on a floating action. Should we log this in the companion too?
If so, we must let the user know that we collect the URLs in the logs.

We also need to wire in the debugging parameter to the companion, and possibly the smart positioning script as well. TODOs are in the code.


Player-suggested changes

#839

Consider finding other usages of "view" (with grepwin or something similar) and replace them accordingly as well.

I did look for other usages, but decided to leave them because they were paired with things like "BlockView".

I did look for other usages, but decided to leave them because they were paired with things like "BlockView".


ParallaxBackground not showing in secondary viewports

Godot version
v4.1.dev3.official [a67d37f7c]
System information

OS: Windows 11
Godot version: v4.1.dev3.official [a67d37f7c]
Renderer: Vulkan
Videocard: GTX1060

Issue description
When adding an extra viewport for a split-screen game and two camera's the ParallaxBackground TileMap doesn't show on the second viewport (All other nodes do show). In the player code I use these lines to assign the camera to the right viewport:

func _ready():
	var viewport = get_node('/root/BugDemo/GridContainer/SubViewportContainer' + str(player) + '/SubViewport')
	var game = get_node('/root/BugDemo/GridContainer/SubViewportContainer1/SubViewport/Game')
	viewport.world_2d = game.get_viewport().world_2d
	$Camera2D.custom_viewport = viewport
	$Camera2D.make_current()

The ParallaxBackground TileMap only shows in the first viewport.
Steps to reproduce
The example project below demonstrates the issue.
Minimal reproduction project
parallaxissue.zip

Related #28222?

I did a little digging and found a possible issue, the ParallaxBackground is inherited from CanvasLayer, and it doesn't seem to pickup the right viewport. I made a little patch to make it inherit from Node2D instead and now it does show in all viewports, but the position is not correct and the set_layer(...) isn't available. I've also noticed that the canvas layer items have a different camera group name: "__cameras_c" instead of "_cameras" but just changing that didn't seem to work.
diff --git a/scene/2d/parallax_background.cpp b/scene/2d/parallax_background.cpp
index 3d57aaf6aa..a99ba2ce0f 100644
--- a/scene/2d/parallax_background.cpp
+++ b/scene/2d/parallax_background.cpp
@@ -35,7 +35,7 @@
 void ParallaxBackground::_notification(int p_what) {
        switch (p_what) {
                case NOTIFICATION_ENTER_TREE: {
-                       group_name = "__cameras_" + itos(get_viewport().get_id());
+                       group_name = "__cameras_" + itos(get_viewport_rid().get_id());
                        add_to_group(group_name);
                } break;

@@ -73,7 +73,7 @@ void ParallaxBackground::_update_scroll() {

        Vector2 scroll_ofs = base_offset + offset * base_scale;

-       Size2 vps = get_viewport_size();
+       Size2 vps = get_viewport_rect().size;

        scroll_ofs = -scroll_ofs;
        if (limit_begin.x < limit_end.x) {
@@ -186,5 +186,5 @@ void ParallaxBackground::_bind_methods() {
 }

 ParallaxBackground::ParallaxBackground() {
-       set_layer(-100); //behind all by default
+       // set_layer(-100); //behind all by default
 }
diff --git a/scene/2d/parallax_background.h b/scene/2d/parallax_background.h
index 70766ae7fa..b2ed69a9db 100644
--- a/scene/2d/parallax_background.h
+++ b/scene/2d/parallax_background.h
@@ -31,11 +31,10 @@
 #ifndef PARALLAX_BACKGROUND_H
 #define PARALLAX_BACKGROUND_H

-#include "scene/main/canvas_layer.h"
-
-class ParallaxBackground : public CanvasLayer {
-       GDCLASS(ParallaxBackground, CanvasLayer);
+#include "scene/2d/node_2d.h"

+class ParallaxBackground : public Node2D {
+       GDCLASS(ParallaxBackground, Node2D);
        Point2 offset;
        real_t scale = 1.0;
        Point2 base_offset;

I made a little patch to make it inherit from Node2D instead

This is not a solution, just to be clear

Thank you, can you please explain why it inherits from CanvasLayer?

Because it is a CanvasLayer, it's how it gets the functionality, what is your argument that it should be different?

I've replaced the ParallaxBackground with a CanvasLayer in the attached demo project and it also doesn't show on other viewports.

Then this bug isn't about ParallaxBackground but about CanvasLayer, and can't be fixed by changing the former

Because it is a CanvasLayer, it's how it gets the functionality, what is your argument that it should be different?

Which functionality? Inheriting from Node2D doesn't change any of it's functionality, it works well.

Then this bug isn't about ParallaxBackground but about CanvasLayer, and can't be fixed by changing the former

yes

Which functionality? Inheriting from Node2D doesn't change any of it's functionality, it works well.

That of being a canvas layer, it's supposed to be one, the fact that it seems to work doesn't mean it does, there's a lot of details you are probably missing. And yes it does change the behaviour. Why are you assuming someone made such a massive mistake and couldn't figure out they could just do this?

That of being a canvas layer, it's supposed to be one,
Please explain why and which details I'm missing.  Perhaps the name "ParallaxBackground" is a bit confusing, it can be a parallax foreground as well.
Why are you assuming someone made such a massive mistake and couldn't figure out they could just do this?
Why do you call it a massive mistake? ParallaxBackground is having this issue, because the underlying issue is in CanvasLayer. Whether ParallaxBackground should inherit from CanvasLayer is a separate discussion.

Listen, it is a CanvasLayer because it works as a layer, with specific behaviour, the canvas layer handles the relative movement, so it can't be a Node2D because that has transformation, making ParallaxBackground a Node2D would just not work right, that's why it isn't one. It has a lot of similarities around organizing and processing, a lot of details, if you study the code it will be more clear to you. Please, you haven't stumbled on some ridiculous error, ParallaxBackground is supposed to be a CanvasLayer, and you aren't understanding the details.

Why do you call it a massive mistake

Because it would be a massive mistake to make it the wrong kind of node
Have a nice day

On further consideration:
I don't think this is a bug at all, it's a limitation, the simple solution to your situation is to add a separate background to each player, just like you would if you had individual UI

Thanks for taking the time to elaborate on this. I'll be using a custom solution for parallax scrolling with multiple viewports.

I'll be using a custom solution for parallax scrolling with multiple viewports.

what's wrong with the established method that I linked?

This should probably be documented, but the fact that CanvasLayer specifies you can add a custom viewport, and that if not specified it will use the default one, should be an indication that it is restricted to a singular viewport, I'm not sure documentation needs to specify this, just like Camera doesn't specify it can only be used with one viewport

For camera this is obvious, but not obvious for Canvas Layer - it should be documented imho

But it explicitly states it uses the default viewport?

The label should be changed though, as this isn't a bug but a limitation, it only works with a single viewport and a single camera unless the code is fundamentally changed

PR open

On further consideration: I don't think this is a bug at all, it's a limitation, the simple solution to your situation is to add a separate background to each player, just like you would if you had individual UI

I would have to disagree. From the product standpoint, there isn't a line of reasoning that can justify ParallaxBackground being visible in one vport but disappear in another. CanvasLayer or not, as world_2d is the designed and documented way to share the world between multiple viewports, it is expected to handle all apects of that world.
The so called "established" method is in essense a hack, that relies on duplicating same nodes into multiple places on the tree and creating additional scene files that are otherwise not useful for anything. It may just do the job for a basic parallax demo with a single level, but in a real world case it would turn into a massive headache with all the extra scenes, with no way to position the background objects against the main plane, and then still it falls apart completely once camera zooming is added to the mix. As it stands, I am looking at removing the split screen, as these two features of the engine, it seems, are unusable together.

ParallaxBackground has been replaced with Parallax2D so this isn't critical to fix specifically for ParallaxBackground, it's a limitation due to how CanvasLayer is designed to work, and one of many aspects why it was replaced with Parallax2D


Problem loading start page

Hello, nice addon!
Just that in firefox when opening a new window (or starting a new session) the extension isn't loading correctly, message:
File not found
Firefox can’t find the file at moz-extension://4547a75f-a139-af4f-a708-7f6d70b92957/dist/index.html.
but afterwards one I create a new tab then it does work well. Just a heads up :)
thanks and cheers!

Hi @illustribe! Thank you for the note! This is due to a Firefox bug. The only workaround at the moment is to disable and then re-enable the extension.
I've reported the bug to Mozilla, but there isn't anything I can do on my end to fix it. Disabling and then re-enabling will get it back up and running for you though!

Ok great, all the best with that! :)


Version Fix

Hey, i made an Pull request #5 for updating the versions. I dont see any problems with netbox version 4.1.x

Hi!
First of all, sorry for the long wait. The new release v0.3.1 has been published. Thank you for your work and interest in the project.


[QUERY] What is the replacement for ServiceBus' ExtractActivity?

Query/Question
I'm in the process of upgrading code that has a dependency on Microsoft.Azure.ServiceBus (Track 1) to Azure.Messaging.ServiceBus (Track 2).
It seems that there is no equivalent for MessageDiagnosticsExtensions.ExtractActivity
What is it replaced with? Where can I find upgrade documentation? I've looked into https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/servicebus/Azure.Messaging.ServiceBus/MigrationGuide.md and can't find anything there.

Hi @yvesgoeleven.  Thank you for reaching out and we regret that you're experiencing difficulties.   To my knowledge, there is not yet an official replacement for extracting diagnostic activities.   Until one is available, the following approach can be used:
public static bool TryExtractDiagnosticId(ServiceBusMessage message, out string id)
{
    id = null;

    if (message.ApplicationProperties.TryGetValue("Diagnostic-Id", out var objectId) && objectId is string stringId)
    {
        id = stringId;
        return true;
    }

    return false;
}

@pakrym or @JoshLove-msft may have additional insight to offer on the plan for a diagnostics extension replacement.

@jsquire, the diagnostics/AppInsights story seems very fractured among several Microsoft groups/teams. Is there an "umbrella" issue to track the efforts to bring this all to a state where no manual workarounds such as the suggested one are needed?

@SeanFeldman:  If there is, I'm not aware of it.   @lmolkova may have more visibility and additional insights.

@yvesgoeleven - can you clarify why you would need to extract the diagnostic Id? Are you correlating it with another activity that you are creating in your app?

@JoshLove-msft Something similar. I need to manually tie the activity representing the inbound message to the activity representing the outbound message, as the default propagation of activities is broken by a disruptor pattern that sits in the processing pipeline.

@yvesgoeleven thanks for the context!  In addition to extracting Diagnostic-Id, make sure to create Activity and pass to ApplicationInsights StartOperation (or other API you're using).
@SeanFeldman I've generalized this issue here https://github.com/Azure/azure-sdk/issues/3319. If you know about any cases where it was needed/missed, please help us prioritize it.


Macro: Expand RNA builder when clicking on New Preset button

Steps to Reproduce

Open RNA tab
Click on 'Cancel' button
Collapse RNA builder
Click on New Preset button

Actual behavior
RNA builder stays collapsed
Expected behavior
RNA builder expands
Screenshots

Manual test added to automation table for Macromolecules

Tested.
https://github.com/epam/ketcher/assets/72072840/5f269684-6f4a-4da9-8f32-cb57a98c4110


list subparser: introduce a non-strict mode

this is analogous to the dump subparser.
Andreas Lauser<EMAIL_ADDRESS>on behalf of MBition GmbH.
Provider Information

@juleq: merge at will

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
6 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.1%) to 96.26%

Files with Coverage Reduction
New Missed Lines
%

cantools/logreader.py
1
97.17%

cantools/tester.py
1
93.88%

cantools/subparsers/monitor.py
2
84.16%

cantools/subparsers/plot.py
2
97.58%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
5560

Relevant Lines:
5776

💛  - Coveralls


provide user as a request extension

This provides the authenticated user, if there is one, as a request extension. Later applications may retrieve the user directly, e.g. via axum::extract::Extenion. This will fail in situations where there is not an authenticated user and therefore must be used only after authentication. For example, in routes protected by login_required!.

cc @seblj @arpankapoor
I'm a little hesitant about this because this makes it easier to misuse the crate. On the other hand, being able to annotate the tracing span with the user ID seems pretty handy.

Closing this as we'll prefer implement FromRequestParts for user types instead.


Insane memory useage when using loop filter

I saw memory usage skyrocketing when using loop video filter (#304):
loop=loop=-1:size=1:start=0
Not sure what is going on. Buffers are allocated and freed, looks normal to me, but something is wrong. Need to investigate with memory profiler.

As a side note i encountered a similar issue with regular playback as well, on w11.
Sometimes video would stop rendering while audio and subs would still go on. Memory would go up over time as video frames are buffered, as well as disk usage going up (probably to fetch said frames).
I guess you are on w11 as well?
I don't have a w10 machine anymore.

This seems to be because of timestamps. FFMpeg loop filter seems to actually loop timestamps as well.
Using your example someplace in the file I keep getting the same timestamp for the video sample
35201833
This will cause them to buffer very quickly and memory does go up spectacularly.


Renaming import with alias not work

I try to rename file: @/components/component.vue with action and the server returns the error: you cannot rename a module via a global import.
project tsconfig:
{
  "extends"<EMAIL_ADDRESS>  "compilerOptions": {
    "composite": true,
    "baseUrl": ".",
    "paths": {
      "@/*": ["./src/*"],
      "@/components/*": ["./src/components/*"]
    },
    "noEmit": true
  },
  "include": ["env.d.ts", "src/**/*", "src/**/*.vue"<EMAIL_ADDRESS>  "exclude": ["src/**/__tests__/*"]
}

../components/component.vue - renamed without error

Please check if this issue has been resolved in version 2.x. If not, feel free to create a new one.


update README.md

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release notes:

/assign @fabriziopandini

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release notes:

@TheAbhishekS Kindly mention the issue number which the PR fixes and edit the PR message for better readability. Try to look through other PRs for reference.

@TheAbhishekS What you changed is the commit message. That was okay. I am talking about the quoted text right here.

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release notes:

so u basically saying i should make another comment here and mention issue number and what file i changed

@TheAbhishekS No you can simply edit it and mention the issue number. No need to mention the files changed.

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # 34
Special notes for your reviewer:
Release notes:

What this PR does / why we need it:
there was a typo in line no.48  of README.md ,This change fix that typo
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # 34
Special notes for your reviewer:
Release notes:

/hold

Hi @TheAbhishekS,  do we still need this PR? If not, you could close it.

@cchengleo done and sorry for the inconvenience


[Bug] Inconsistencies between Fern Definition availability and x-fern-availability extension

Describe the bug
For the Fern Definition availability property to produce the Beta tag in Fern Docs, you must use in-development or pre-release. You cannot use beta.
For the x-fern-availability openapi extension to produce the Beta tag in Fern Docs, you must use in-development or beta. You cannot use pre-release.
Expected behavior
These should be consistent with each other.
Version
Fern Version: 0.18.4
Additional context
Note; The Fern component of Availability has a yet another set of options:
type: 'beta' | 'GA' | 'deprecated'
I find this less relevant though as that just echoes the tag that's created.

nice find of an inconsistency!  cc: @dsinghvi for awareness

Closing per tracking internally.


isochrone with turn costs enabled much slower

For routing with turn restrictions enabled we have a slow down of approx. 2x and so I would expect a maximum slow down of 2x (because the shortest path exploration is usually the fast part of the isochrone calculation). But it seems that the slow down factor is over 6x even for smaller time limits like 10min:
time curl 'localhost:8989/isochrone?point=52.550732,13.350878&profile=car&time_limit=600'
...
real	0m1,179s
user	0m0,000s
sys	0m0,018s

time curl 'localhost:8989/isochrone?point=52.550732,13.350878&profile=car_no_tc&time_limit=600'
...
real	0m7,464s
user	0m0,010s
sys	0m0,004s

Related to #1978 and #1458.

Strange. I cannot reproduce this on my local machine (4.5s vs. 2.8s). Closing for now.

Keep in mind that if turn costs are enabled the graph exploration is edge-based and so the shortest path tree is larger (contains edges that otherwise would be skipped as they point to already explored nodes). Maybe this could be an explanation?
See this: https://discuss.graphhopper.com/t/missing-edges-in-spt-response/5586/4

You could profile the execution using JFR if you're using Java 11 or higher.
JMC: https://adoptopenjdk.net/jmc.html

the resulting shape is only slightly different

Yeah but the shortest path trees could still be very different in size? In your example the number of visited nodes is very different (486191 for car vs 170438 for car_no_tc) so to me it seems like the real question is why in one case much less nodes are explored (could be due to better/worse LM heuristic?). Maybe MiniGraphUI could help here :)

Hmm. Seems elusive. Works fine on my machine.
2020-07-22 10:44:11.757 [dw-21 - GET /isochrone?point=48.839413,2.351074&profile=car_no_tc&time_limit=400] INFO  c.g.resources.IsochroneResource - took: 1.1765695, visited nodes:170247, consumed nodes:21612, {time_limit=[400], profile=[car_no_tc], point=[48.839413,2.351074]}
2020-07-22 10:44:19.128 [dw-20 - GET /isochrone?point=48.839413,2.351074&profile=car&time_limit=400] INFO  c.g.resources.IsochroneResource - took: 2.435008, visited nodes:432537, consumed nodes:36972, {time_limit=[400], profile=[car], point=[48.839413,2.351074]}

Am I doing this right?
  datareader.file: ile-de-france-latest.osm.pbf
  graph.flag_encoders: car|turn_costs=true
  graph.location: graphs/ile-de-france/graph
  profiles:
    - name: car_no_tc
      vehicle: car
      weighting: fastest
      turn_costs: false
    - name: car
      vehicle: car
      weighting: fastest
      turn_costs: true

The landmark generation probably depends on the size of the map (ile-de-france vs. france)? Which map did you exactly use @karussell? Can we use bayern-190101.osm.pbf or germany-190101.osm.pf maybe as these are the ones we use for benchmarks?

Isochrones aren't using Landmarks...

right... :)

Hmm. Seems elusive. Works fine on my machine. Despite lots of visited nodes.

Thanks for investigating this @michaz. I think I found the problem: we use u_turn_costs: 80 in the config and without that I get the times for car with tc that you get (and the factor is roughly the expected 2x).
Still, let me try to give you the exact same conditions:

config.yml https://gist.github.com/karussell/079f7d7dae8373d6136881b07e5afba3
commit d621386e1
JVM options: -Xmx5g -Xms5g -Ddw.graphhopper.datareader.file=ile-de-france-200721.osm.pbf (http://download.geofabrik.de/europe/france/ile-de-france-200721.osm.pbf)

Then I get:
2020-07-22 14:07:23.660 [dw-19 - GET /isochrone?point=48.839413,2.351074&profile=car&time_limit=400] INFO  c.g.resources.IsochroneResource - took: 5.3310757, visited nodes:480436, consumed nodes:62789, {time_limit=[400], profile=[car], point=[48.839413,2.351074]}
2020-07-22 14:07:26.463 [dw-20 - GET /isochrone?point=48.839413,2.351074&profile=car_no_tc&time_limit=400] INFO  c.g.resources.IsochroneResource - took: 1.0640897, visited nodes:170247, consumed nodes:35447, {time_limit=[400], profile=[car_no_tc], point=[48.839413,2.351074]}

The described scenario requires many queue update/remove operations, which neither zero nor infinite turn-costs do. ShortestPathTree needs to be optimized for this. Is much faster for me now, needs some more testing.


Shuffling Data

When you're using time series analysis and technical indicators, you can't shuffle your data. Real time series data is not shuffled. Also, shuffling the data leaks historical data, as the technical indicators are calculated based on historical data. Your first indicator should have been your very high accuracy (i.e. if you could accurately predict 85%+ accuracy against the stock market, you'd be a rich man).
This is a very similar issue with this research: https://www.reddit.com/r/algotrading/comments/cv83yh/overfitting

I checked the code and I retrieve training data and test data from loaded csv sequentially (historical data). Then I shuffle and split the training data to train and validate. Shuffling is an issue if you leak any future data into your training data which is AFAIK, I haven't done (or at least never intended, unless there is a bug). Can you point out relevant code section if you found a bug?

 Try running your train and prediction with all shuffle=True updated to shuffle=False.

Yes. Technical indicators are function of historical data, that's why people use technical indicators as features to learn from various patterns. "Leaking" is when you use future data in your training data which would be cheating. Looking past data is not an issue.


Remove six from requirements

It's obsolete now and isn't used anywhere.

Waiting for cont-build before accept


fix(pipeline): correct warning path for duplicate param name in pipeline tasks

fix #7650
This error message has been present since v0.48. https://github.com/tektoncd/pipeline/commit/fd09e499de9e549007c39782c2b441b5abf6f1e7

Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[ ] pre-commit Passed
[ ] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

/kind bug
Release Notes
fix: correct warning path for duplicate param name in pipeline tasks

/cc @lbernick

/kind bug

/ok-to-test

/approve
/cc @chitrangpatel @JeromeJu
/cherry-pick release-v0.56.x

/lgtm


[Bug] Update plugins on top bar crashes BD

Describe the Bug
When you go to update plugins on the blue top bar BD crashes and restarts regularly without BD.
To Reproduce

Have a plugin that needs updating
Click plugin name at the top blue bar
BD crashes and Discord starts normally

Expected Behavior
Plugins should be updated without a crash.
Screenshots
Attached Video
Discord Version
Stable
Additional Context
None
https://user-images.githubusercontent.com/22356378/125639430-29f1455c-b4c7-4228-ab81-71cd39037b17.mp4

Update ZeresPluginLibrary manually, use this link:
https://raw.githubusercontent.com/rauenzi/BDPluginLibrary/master/release/0PluginLibrary.plugin.js
Use CTRL + S to save the page as a .plugin.js and place it into the plugins folder.
Hope this helps!

Duplicate of #888

See above. Reinstalling BD once more also repairs it.


Feature: audio output via virtual microphone

Disclaimer:
Please go through existing feature requests to avoid creating duplicates. If you found a similar issue please post a comment there. If you are uncertain your issue is related to others then create the issue. Obvious ignorance of this rule might get you blocked from posting new issues. Please tick the box below to show that you have followed the guideline.

[x] I have checked for a similar issue and sure it hasn't been reported before.

 Same funtionallity here would be a nice feature.
User flow & interface:
Select eqMac's virtual microphone as output the settings. Then go to an app like zoom that accept microphone input. There you can select eqMac's virtual microphone as a input device. Now you recieve the audio from eqMac in zoom.

This is already a feature, I've done this before using the latest version. It's not present in the app's settings, but If you look in Sound preferences, you can see that eqMac is listed as an input

Oh yes you are right. That is perfect!

The input stream you can grab from the eqMac Input Device is the raw system audio, before EQ processing. Just FYI.
I do plan on adding an option of a Virtual Output where you can use any app to grab the post processed audio stream

Now with 1.4.0 the virtual microphone feature is no longer available. Please add a option to reactivate it.

Now with 1.4.0 the virtual microphone feature is no longer available. Please add a option to reactivate it.

Please try enabling the Beta Program in eqMac settings and downloading the latest pre-release version v 1.4.3. You should be able to grab the system audio with that version (still pre-processed). An option to grab pre or post processed audio is coming in v 1.5 0l.0

I'm doing some cleanup of the issues.
This has probably been fixed long time ago, however the post processed audio capture is still now out :( hoping to squeeze it in in the next update, but that deserves a separate issue.
I encourage people to try v1.7 by download it from the website: https://eqmac.app
Closing for now, any new issues with eqMac should be submitted through the website.
Thank you!


Make --color affect missing line number formatting

Fixes #213

Thanks for the great bug report and PR :)


DA-385: Migration Dialog

@teetangh going forward, pay attention to the following things:

Your code keeps storing unnecessary states. The more stateless your code is, the less prone to errors;
A single class for everything is a pretty bad code style. Read about separation of concerns and MVC;
This "addListeners" approach of yours is a good recipe for nullpointers. Add the listener where the component is created;
Code gets hard to read with everything happening at the same place.


Changing network interface discovery to use net.Interfaces

Adding the ability to specify an arbitrary interface to use. This also cleans up the IP discovery for an interface by using net.Interfaces instead of shell commands.

Thank you so much! great thing made and something i've learned 👍


Ignore gradle plugin in codecov

The code coverage reports from codecov about our gradle plugin aren't correct. For that reason it's better to ignore them so we don't get that noise in our PRs.
Following this documentation: https://docs.codecov.io/docs/ignoring-paths

Does filtering out the project in the root build.gradle.kts have the same effect?
There are already some subprojects that are ignored that way, might add well use the same method if possible.

Does filtering out the project in the root build.gradle.kts have the same effect?

Good point. The build.gradle.kts configures jacoco test report while codecov.yml configures the code coverage CI.
If we want to report code coverage for detekt-gradle-plugin separately, I believe modifying codecov.yml as shown in this PR would be a reasonable choice.

Does filtering out the project in the root build.gradle.kts have the same effect?

Good point. The build.gradle.kts configures jacoco test report while codecov.yml configures the code coverage CI.
If we want to report code coverage for detekt-gradle-plugin separately, I believe modifying codecov.yml as shown in this PR would be a reasonable choice.

Yes, this would be the preferred way.

:detekt-gradle-plugin:test task should also be skipped on the code coverage CI job which will speed it up. It doesn't currently use the build cache and the Gradle test task takes 12 minutes (only part of run time is in parallel with other tasks).
https://scans.gradle.com/s/dt73yepgj2gxm/timeline


How to inject the  at the top of the  in react-helmet v6 ?

Hi, I want to inject my social networks meta at the top of the head (og:title, ...), so far it keep being injected at the end.
How can I configure react-helmet to inject at the top?
Thanks a lot !

I was able to accomplish this by replacing "appendChild" with "prepend" within the Helmet.js file.
headElement.prepend(newChild)

Related: https://github.com/nfl/react-helmet/issues/514

headElement
Where do i find headElement.appendChild ?

I was able to accomplish this by replacing "appendChild" with "prepend" within the Helmet.js file.
headElement.prepend(newChild)

Where do I find headElement.appendChild ?

I was able to accomplish this by replacing "appendChild" with "prepend" within the Helmet.js file.
headElement.prepend(newChild)

I did this like in the image but no luck :(


[OpenSearch Playground] Enable autoscaling for the Playground site by customer request numbers

Is your feature request related to a problem?
We enabled auto scaling for EKS cluster node based on CPU/Memory with Up limit currently. We would like to enable auto scaling with customer request number or other performance metrics except for CPU/Memory. it will provide us more extendable scale set for the future.
What solution would you like?
EKS auto scaling based on the customer requests, need to leverage auto-scaling feature.
What alternatives have you considered?
No
Do you have any additional context?
No

We added auto scaling dynamic policy


Status update

0a5b58

We will be publishing the appropriate PR guidelines soon.
Please keep on contributing, thank you!


Effectively blacklist blacklisting disk-partition.com

disk-partition.com has been in and out of the blacklist repeatedly. It should not be blacklisted.
Add disk-partition.com to GlobalVars.valid_content so as to trigger the "too broad" check if someone attempts to blacklist it again.

Just a note here: That valid_content was intended for accidental blacklisting of common phrases (like a single dot - that means everything) since Rapid Reload bypasses CI and can cause potential mess. If there's need to prevent more stuff from being blacklisted, we should probably find a new approach (IMO it's a decent solution pro-tempore).


Current page not detected properly by menu

On Middleware Configuration site, the Configuration item is also active.

@stancl just to confirm, what are the complete URL paths of your 'Configuration' and 'Middleware Configuration' pages?

https://tenancy.samuelstancl.me/docs/v2/middleware-configuration/
https://github.com/stancl/tenancy-docs/blob/master/docs/navigation.php#L11-L27

@stancl thanks for your patience, this was tricky 😂
TL;DR - in the isActive helper on line 34 of your config.php, make the following change:
- return ends_with(trimPath($page->getPath()), trimPath($path));
+ return ends_with(trimPath($page->getPath()), trimPath($page->version() . '/' . $path));

What's happening here is that isActive assumes it's checking the entire path to the current page. In the documentation starter template, it is—we specify the complete path for every page in navigation.php:
    'Getting Started' => [
        'url' => 'docs/getting-started',
        'children' => [
            'Customizing Your Site' => 'docs/customizing-your-site',
            'Navigation' => 'docs/navigation',
            // ...

With this setup, isActive is going to check if docs/middleware-configuration ends with docs/configuration, which it doesn't, so everything works.
This involves a fair bit of repetition in path names, because we have to write out docs/ over and over. You've quite thoughtfully hoisted this up into top level v1 and v2 keys in your navigation, and you add the version back into each page's URL in the link helper, which means for every page you only have to specify the last part of the slug:
    'v1' => [
        'Getting Started' => [
            'url' => 'getting-started',
            'children' => [
                'Installation' => 'installation',
                'Storage Drivers' => 'storage-drivers',
                // ...

The default isActive helper, however, naively still thinks it's getting the complete path, so on your site it checks if middleware-configuration ends with configuration, which it does, and that's why you get two active menu items.
One solution to this of course would be adding the version back to every path in your navigation.php. But, since you've also got a handy version helper set up, it's quicker to just add the version back to the path dynamically right in the isActive helper.

Ah, thanks a lot for the detailed answer @bakerkretzmar :)


Lemmas with IsCommutative etc. arguments require type annotations

class IsCommutative (op : α → α → α) where
  comm : op x y = op y x

class LeftUnit (α : Type u) extends Mul α

class Monoid (α : Type u) extends LeftUnit α

class CommMonoid (α : Type u) extends LeftUnit α where
  mul_comm {x y : α} : x * y = y * x

instance [CommMonoid α] : IsCommutative (α := α) (· * ·) where
  comm := CommMonoid.mul_comm

instance [CommMonoid α] : Monoid α where

theorem foo [Monoid α] [IsCommutative (α := α) (· * ·)] {x y : α} :
    x * y = z ↔ y * x = z := by
  rw [IsCommutative.comm (op := (· * ·))]
  exact Iff.rfl

instance : CommMonoid Nat where
  mul_comm := Nat.mul_comm _ _

set_option trace.Meta.synthInstance true
variable (a b c : Nat) (h : a * b = c)

#check foo -- ok
#check foo.1 -- failed to synthesize instance IsCommutative (NOT stuck!)
#check foo.1 h -- failed to synthesize instance IsCommutative (NOT stuck!)
def bar := foo.1 h -- failed to synthesize instance IsCommutative (NOT stuck!)

The last three commands should all succeed.  Mathlib has a lot of lemmas like foo (usually due to CovariantClass, so e.g. le_neg, abs_pos, etc.).  (It's important that the instance [CommMonoid α] → Monoid α is not a subobject projection here, but this is super common in the algebraic hierarchy.)
What's happening here is that we ignore the stuck exception in nested TC problems.  So Lean thinks that the only IsCommutative instance does not apply because it cannot unify Monoid.toLeftUnit =?= CommMonoid.toLeftUnit while the type is still a metavariable.  And since we ignore the stuck exception, we never try again.  cc @Kha
A workaround is to specify the type manually (surprising syntax btw):
#check foo (α := Nat).1 -- ok
#check foo (α := Nat).1 h -- ok
def bar := foo (α:= Nat).1 h -- ok

For comparison, all is well without the IsCommutative argument:
theorem foo' [Monoid α] {x y : α} : x * y = z ↔ x * y = z := .rfl
#check foo' -- ok
#check foo'.1 -- ok
#check foo'.1 h -- ok
def bar' := foo'.1 h -- ok

I think we might have had this problem in lean 3 too, it was not uncommon to have to add a type annotation to make dot notation work.

I've clarified the title of the issue.  This is a different issue, and it's new in Lean 4 because we didn't have the stuck exception handling in Lean 3.  This issue also appears without dot-notation:
#check Iff.mp foo -- fails
#check Iff.mp foo h -- fails


GitHub action: cache Go mods and deb packages

GitHub action: cache deb packages install
Installing libbpf-dev used to take 20 seconds. Additionally, this causes
unnecessary network traffic to Ubuntu package repositories and PPA
repositories. This patch optimizes that by using the GitHub action
cache.
After this patch, checking the cache takes less than 1 second and
installing from the cache takes 1 second.
Technique inspired from:
https://stackoverflow.com/questions/59269850/caching-apt-packages-in-github-actions-workflow

GitHub action: cache Go modules
Using the technique from the documentation:
https://github.com/actions/cache/blob/main/examples.md#go---modules
This is useful for building the kubectl plugin ("make kubectl-gadget")
and the unit tests ("make test"): those are built directly on the Ubuntu
host.
Before this patch:

make kubectl-gadget: 2m 42s (test 1) and 2m 18s (test 2)
make test: 57s (test 1) and 48s (test 2)

After this patch, in case of cache hit:

restoring the cache: 8s
make kubectl-gadget 12s
make test: 3s

Note: this patch is complementary with the previous commit 93f2f767688e
("Dockerfile: faster builds") where we cache the Go modules in a Docker
layer. This was useful for building the gadget container image in Docker
("make gadget-container").

Done: rebased, commitmsg and PR description updated.


Don't try to access the network in Stocks test

Accessing the network created a race condition that would sometimes cause other
tests to fail.

LGTM.


fix dashboard content for expired patient that completed the question…

address this issue:
https://www.pivotaltracker.com/story/show/148875877
add logout button for dashboard display for expired patient that completed questionnaire

Hello @achen2401! Thanks for updating the PR.

In the file portal/models/intervention_strategies.py, following are the PEP8 issues :

Line 450:87: W291 trailing whitespace
Line 451:41: E127 continuation line over-indented for visual indent


facebookAuthentication Error

Describe the bug
The previous version of the code in the facebookAuthentication.dart file had multiple issues.
Prerequisites

I am running the latest version
I checked the documentation and found no answer
I checked to make sure that this issue has not already been filed
I'm reporting the issue to the correct repository (for multi-repository projects)

To Reproduce
Steps to reproduce the behavior:

Go to Friday Repo
Click on lib\services\facebookAuthentication.dart
See error

Expected behavior
Code without any errors successfully compiles and runs as expected.

Hello @avinashkranjan
Please assign me this issue
Thank you

Heyy @avinashkranjan
Please review my pull request #227
Thank you


The source distribution tarball is an empty package.

I'll try to send a PR to fix that ASAP. One can test that by trying to install the latest version from PyPI with pip. See https://github.com/conda-forge/oceanspy-feedstock/pull/9

Thank you @ocefpaf, I was just starting to look into that PR. I'll be waiting for the upcoming PR

Almost ready BTW, looking at the sizes of the sdist and the wheel one can see that the sdist has everything in the repo and the wheel is too small to contain the package. BTW, the sdist is quite big! In a time of Python on the web maybe we could reduce the size by excluding some extras that are not part of the package. (I'll add that to my PR.)

thank you again @ocefpaf !


use the CVO to create the operator config

The CVO supports create-only resources now, so we can use the CVO instead writing our own code a dozen times.
/assign @damemi

/lgtm
farewell old friend

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/test all

needs https://github.com/openshift/installer/pull/1561

/hold
missing logs

/retest

/retest


Initialize mReceiveMessage when starting a new DTLS session.

Current coverage is 64.73% (diff: 0.00%)

Merging #656 into master will decrease coverage by 0.14%

@@             master       #656   diff @@
==========================================
  Files            99         99
  Lines         14390      14391     +1
  Methods        1979       1979
  Messages          0          0
  Branches       1690       1690
==========================================
- Hits           9336       9316    -20
- Misses         4525       4546    +21
  Partials        529        529

Powered by Codecov. Last update 1e386cb...1cbec18


JBPM-4390: Exception stack trace overflows from Job Requests Details …

…dialog

@nmirasch Do you have the right jira ID? JBPM-4390 has title "Looking at Dependency clean-up and management for the kie-remote modules" ...

@cristianonicolai @jhrcek Thanks both, you are right, the commit title was wrong. I have changed it, and have fixed the  test. Can you take another look?


Fixed: Created CODE_OF_CONDUCT.md

Fixed #50
Added CODE_OF_CONDUCT to the Repo.
Importance of CODE_OF_CONDUCT - We propose adding a comprehensive Code of Conduct to our repository to ensure a safe, respectful, and inclusive environment for all contributors and users. This code will
serve as a guideline for behavior, promoting diversity, reducing conflicts,
and attracting a wider range of perspectives.
@Portponky  kindly review it and merged the PR. Thank You!

This just looks like a phishing attempt.


Hi,what is the label tool do you use?

I also want to know. can u tell me if you already know,here is my mail<EMAIL_ADDRESS>


[Question] may I convert result from data source returns to resource?

Is there an existing issue for this?

[X] I have searched the existing issues

What version of the Terraform provider are you using?
1.2.0
What version of the Terraform CLI are you using?
1.8.2
What type of issue are you facing
bug report
Describe the bug
Hi team,
Just as title, I want to copy / backup subaccount for redeploy or setup a new subaccount. So, I want to know if there is a convenient way to convert the data source returns to resources directly. So, I don't need to key in every setting.
Thanks
Expected Behavior
No response
Steps To Reproduce
No response
User's Role Collections
No response
Add screenshots to help explain your problem
No response
Additional context
No response

Van you please openn a question in the GitHub Discussion on this topic. This is neither a bug nor a Feature request


Exception Value: single positional indexer is out-of-bounds

Hello. I've got following error when I try to runserver first time:
IndexError at /
single positional indexer is out-of-bounds

I tried some debug and I have no data when I put weekly or monthly ==  True. Without param works fine. Should I configure something more before first runserver?
Function which generates error:
def percentage_trends():
    """[summary]: Returns percantage of trend, relative to week prior delta
    Returns:
        [dataframe] -- [percentage objects]
    """
    current = realtime_growth(weekly=True).iloc[-1]
    last_week = realtime_growth(weekly=True).iloc[-2]
    trends = round(number=((current - last_week)/last_week)*100, ndigits=1)

    rate_change = round(((current.Deaths/current.Confirmed)*100)-((last_week.Deaths / last_week.Confirmed)*100), ndigits=2)
    trends = trends.append(pd.Series(data=rate_change, index=['Death_rate']))

    return trends

@mateuszz0000 What python version are you running?

3.8.2

What operating system? Also, any errors installing the dependencies?
Another user, also had the same issue running on Python 3.8.2 -> #6

Found problem already. I will make PR for a moment :)

@mateuszz0000 Great! I'll be looking out for it.

@mateuszz0000 May you confirm the closing of this issue? and, #6 equally


Searching for twitter does not return the twitter package

URL: https://pub.dartlang.org/packages?q=twitter
When searching for twitter the package twitter is not returned on the first page of results. However, when searching for tweet the package twitter shows up as the 3rd result. It seems that the search algorithm should return the twitter package as the 1st result when searching for twitter.

What @sortie described is correct. However, the twitter package is now at the top again, because there was a bug in the score/threshold calculation, and it was pushed even further back. If the package  will be without maintenance further, it will get lower scores, and eventually it will fall off again.


ChromeDriver version isn't compatible with Chrome version

Having issue with YAML?

Having issue with Tasks?

Having issue with software on Hosted Agent?

Having generic issue with Azure-Pipelines/VSTS/TFS?

Have you tried troubleshooting?

Agent Version and Platform
windows-2019, 2.183.1
Azure DevOps Type and Version
https://lgre.visualstudio.com/
What's not working?
Selenium auto tests cannot open Chrome
Agent and Worker's Diagnostic Logs
TearDown : System.Reflection.TargetInvocationException : Exception has been thrown by the target of an invocation.
  ----> System.InvalidOperationException : session not created: This version of ChromeDriver only supports Chrome version 89
Current browser version is 88.0.4324.182 with binary path C:\Program Files\Google\Chrome\Application\chrome.exe (SessionNotCreated)

Hi @yesenin this issue doesn't seem to be related to agent - for tools like Chrome and ChromeDriver installed on hosted images please feel free to create an issue on https://github.com/actions/virtual-environments
Let me close this one here since this repo is mostly for pipelines agent.


Server to use external DB

Hi!
OpenTest Server uses on default files to store all data related to test execution (./db/actors, ./dbsessions). Is there already prepared integration with external data storage we could use?

The DB access module in OpenTest is written in a way that makes it possible to switch the persistence layer to other databases. We envisioned that, in the future, some teams might need to switch to a proper database to deal with scalability and performance issues, but we haven't seen such a need yet. Furthermore, once performance and scalability becomes a bottleneck, there are at least a couple of workarounds that can be applied before we need to change the persistence layer.
If you describe the problem you're trying to solve, I can try to come up with some recommendations.

We build from scratch docker image on which Server is deployed after every restart. Problem with this approach is that we lose all 'run-time' data after each restart, and results in the local folder are one of this. Additionally, we want to have a chance to do analytics on a test result data which is much easier to implement if all data is stored in DB.

I gave some thought to this and here's what I came up with. For reporting and analytics you would need a relational database, but OpenTest works best with a NoSQL document-oriented database. Furthermore, it's not ideal to use the persistence layer of OpenTest for analytics, because you'd have to update your analytics solution each time OpenTest makes an update to it's DB structure and there's no telling how often these types of changes will be necessary in the future. Because of this, I think the best option would be to extract the data from OpenTest via the web API and insert it into whatever database is best for the purpose you intend. We already have the pieces you need to do this:

The API to extract the necessary data from the server.
A notification mechanism, via WebSocket to get notified when a test session status changes.

Let me know if it sounds like something you want to do and I can try to put together some more information to describe how you'd go about it.

Thanks for your response and analyzes.
To load data in the existing object form to DB instead of the file seems natural for me.. but there is a good reason for not doing that as you pointing above. I think to separate our result base DB and OpenTest file as you suggesting. This independence may come handy at some point for many reasons.
Would be great if you could put more information on how to attack this.

I will give you a solution for this and probably even some working code around the second week of January.

Happy New Year!
That sounds great. Looking forward to checking this out then :)
I think to use reportportal.io, but POC on this is waiting in the queue.

All clear here, I have used your example project to save data to mysql DB

Great to hear you got your problem solved, and thanks for the update.

@galegor Hi. Did you end up setting up the integration with reportportal?


about the wider face data

hi
I follow your guide to start this project. I download the data WIDER face.but when i run the command:"No dataset given" , i find that the faster_rcnn_end2end.sh scripts only support only voc and coco data. and then i try to convert the wider face data to the voc style . but i meet a new question:assert (boxes[:, 2] >= boxes[:, 0]).all() .i try all i can do ,but it does not work.

The faster_rcnn_end2end.sh script does support the wider dataset. You might need to exactly follow the instructions in the readme file.

hi playerkk
I am a fresh bird,you said the faster_rcnn_end2end.sh script does support the wider dataset,but i open the faster_rcnn_end2end.sh. it said that :
case $DATASET in
pascal_voc)
TRAIN_IMDB="voc_2007_trainval"
TEST_IMDB="voc_2007_test"
PT_DIR="pascal_voc"
ITERS=70000
;;
coco)
# This is a very long and slow training schedule
# You can probably use fewer iterations and reduce the
# time to the LR drop (set in the solver to 350,000 iterations).
TRAIN_IMDB="coco_2014_train"
TEST_IMDB="coco_2014_minival"
PT_DIR="coco"
ITERS=490000
;;
*)
echo "No dataset given"
exit
;;
esac
i have no idea .i need your help,thank you

Are you talking about this file? https://github.com/playerkk/face-py-faster-rcnn/blob/master/experiments/scripts/faster_rcnn_end2end.sh

oh my god ......i use the rbgirshick faster rcnn  faster_rcnn_end2end.sh......many many many thanks.......

is there anything i should replace? thank you very much.

i clone your project . thank you very much........ : )

hi playerkk
I am sorry to bother you .I think i should run it first and then to analysis it.
AssertionError: Path does not exist: data/WIDER/train_face_train_annot.txt .
i think it has some problem with the file name .
image_set_file = self._name + 'face' + self._image_set + '_annot.txt'
I think it should not produce the name :train_face_train_annot.txt
I input the command:　**voole@zhx:~/face-py-faster-rcnn$ ./experiments/scripts/faster_rcnn_end2end.sh 0 ZF wider **
thanks for your help
zhanghongxing

i really really really really really need your help, if you dont have time ,i will be waiting for you.

I would strongly recommend you to read the instructions before running the cod. https://github.com/playerkk/face-py-faster-rcnn/blob/master/README.md

I am sorry to bother you again ,I have read the README for several times , http://blog.csdn.net/zhanghongxing007/article/details/56479206
I write down the details, I hope you can read for several minutes, thanks for your help.

You didn't do the step 4 of the section "prepare the training data".

first of all . thank you very much. that i use the folder WIDER/WIDER_train/ to Create symlinks. after that , it said that need:
AssertionError: Selective search data not found at: /home/voole/face-py-faster-rcnn/data/WIDER/train.mat
so i down the wider_face_split.zip from the WIDER FACE NET. and then i change the name wider_face_train.mat to train.mat
after that i meet a new problem:
NotImplementedError: Please use HDF reader for matlab v7.3 files
so i have to change the scipy  io to fit the v7.3 but................i cant find the loadmat function............

I tested the repository on another machine. I also had some errors of loading wider dataset. But different from yours. I've updated the source files. Please update the repository by doing "git pull origin master" and re-run the program as "./experiments/scripts/faster_rcnn_end2end.sh 0 VGG16 wider"
Sorry for the inconvenience.

hi playerkk
First of all ,thanks for your reply.  I am sorry to bother you again again again....but i have be here for one day.
Traceback (most recent call last):   File "./tools/train_net.py", line 106, in <module>     imdb, roidb = combined_roidb(args.imdb_name)   File "./tools/train_net.py", line 71, in combined_roidb     roidbs = [get_roidb(s) for s in imdb_names.split('+')]   File "./tools/train_net.py", line 68, in get_roidb     roidb = get_training_roidb(imdb)   File "/home/voole/face-py-faster-rcnn/tools/../lib/fast_rcnn/train.py", line 118, in get_training_roidb     imdb.append_flipped_images()   File "/home/voole/face-py-faster-rcnn/tools/../lib/datasets/imdb.py", line 106, in append_flipped_images     boxes = self.roidb[i]['boxes'].copy()   File "/home/voole/face-py-faster-rcnn/tools/../lib/datasets/imdb.py", line 67, in roidb     self._roidb = self.roidb_handler()   File "/home/voole/face-py-faster-rcnn/tools/../lib/datasets/face.py", line 227, in selective_search_roidb     ss_roidb = self._load_selective_search_roidb(gt_roidb)   File "/home/voole/face-py-faster-rcnn/tools/../lib/datasets/face.py", line 247, in _load_selective_search_roidb     'Selective search data not found at: {}'.format(filename) AssertionError: Selective search data not found at: /home/voole/face-py-faster-rcnn/data/WIDER/wider.mat
wider.mat? it means the wider_face_train.mat?
Look forward to your reply

It's weird. Since we are using RPN to generate proposals, there is no need to call the _load_selective_search_roidb function. Can you post your command to run the code?

hi playerkk
thanks for your reply. i  write down the detail at the : http://blog.csdn.net/zhanghongxing007/article/details/56479206
many thanks.

Sorry, I should have mentioned only VGG16 is currently supported.

Hi playerkk
yeah,i use the command:./experiments/scripts/faster_rcnn_end2end.sh 0 VGG16 wider. i use the VGG16 net.

i have the same quetion .the faster rcnn use the RPN net instead of the selective_search. I  called some old script for fast-rcnn ?

You are training a faster r-cnn. Do you have errors right now?

when i input the conmmand:  './experiments/scripts/faster_rcnn_end2end.sh 0 VGG16 wider'
it still point that : ' Selective search data not found at: /home/voole/face-py-faster-rcnn/data/WIDER/wider.mat'
i am sorry to bother you.

It seems case sensitive. Try vgg16 instead of VGG16. Also do "git pull origin master" to update the source files.

You might also need to delete *.pkl files in the cache folder.

hi man
i have solved this question~~~,because of the easydict, my version is 1.4 .even i config the yml. but easydict did not pass the parameter to the train_net.py. so ...... the solution is to update your easydict. thank you very much very much very much


Remove swedish

It was just quicker this way - the files have already been moved over:
https://github.com/dlang-tour/swedish/pull/1

@vladdeSV @nordlow @vild @jacob-carlborg I directly went ahead and did the migration, s.t. we have a clear breakpoint in time & as I assume it's easier if the migration is already done for you ;-)

Fine with me. Thanks for you work.


How to make responsive without breakpoints

I'm trying to make my swiper responsive without the use of breakpoints. I want the container itself to just shrink in width, bringing the navigation buttons along with it, while my slides maintain the same width and just get covered up as the window moves. Is this possible? By default, it maintains full width and brings up the horizontal scrollbar when the window is shrunk too much, and it messes with all the other elements of the page that used to shrink correctly.
I have loop set to true, slidesPerView is 'auto' and loopedSlides is set to 4. I thought enabling freeMode might do it but it hasn't helped. Any advice is appreciated. I'm a jr. dev and I still have a lot to learn, especially regarding CSS. If you need to see more of my code let me know.

https://github.com/surmon-china/surmon-china.github.io/blob/source/projects/vue-awesome-swiper/examples/12-slides-per-view-auto.vue
use slides-per-view: 'auto'
In order for it to work, you will have to define your slides fixed-width with CSS. Otherwise VAS won't be able to properly calculate positions of your slides. All of this is reflected in the example above. Just investigate it diligently


move version information to cmake file

I have thought about the open point from #33: commit hash and dirty state is not updated without explicit run of cmake.
Do you see anything that speaks against removing the include/lomse_version.h and having the versions only in a cmake file?
The version information can be created like all the other cmake variables at cmake runtime. If you like the idea, I will prepare the necessary code changes. If you don't like it, I will try to change the cmake file to rebuild version strings with every change that affects them (like making the code dirty with respect to the repository).

As I see this:

version information should be extracted from git
But when building from a source package or if git is not installed in the machine, some static version information should be available and must be used. Currently, this is the role of lomse_version.h.
The library should have methods to provide:

the version info extracted from git or blank/empty string if not available at build time
the version info from the static variables

All these objectives are currently achieved by having the static information in lomse_version.h. But I don't see any problems for moving these static values to CMake. But in any case, what is the gain? Why do you think it is important to remove this info from a header file?
I don't understand your second point: "change the cmake file to rebuild version strings with every change that affects them (like making the code dirty with respect to the repository)".  Current script (scripts/update-version.sh) could be used for it. By the way I have to modify it because it was designed for using git tags in the form "major.minor" or "major.minor.0" but not current tag "0.17.20". With current script next version could be for instance "0.17.2"

I second your analysis with a small objection to the first: If the version is extracted from git and it's given in some file, there may be a difference that should at least be warned upon. If it's only given by git, it's not available in a source package without a special release process for building the source package. And a special release process makes it impossible to just use tags as release packages :disappointed:
It's not important to remove the version information from a header file but if it's included in a cmake file it's not necessary to parse a header file to get it. So it's in the book of removing unnecessary code to reduce possible errors.
And to the second question: Imagine this workflow

checkout the code from the repo
run cmake
run make

all versions are what we need and expect
but then

make some changes in the code
run make

the version still is the same as the one from the fresh checkout. But we expect a dirty in the version string. This is only added if cmake is run after changing anything. The same if you change the version file. And at least this version changed can be easily  managed by having the version information in the cmake files, because every change in a cmake file will trigger a cmake run by only running the normal build process. I'm not sure about how to achieve the same for changes in the git information, but will try to find out.
And regarding the update-version script: Maybe this could be done as a (c)make target.

Ok, understood. Your solution is better, so go ahead with the changes if you like.
Thank you!


feat: add missing self call methods

Adds aurora-engine contract methods that are missing from the workspace. This PR only deals with the public view methods, others will be added separately.
The methods have already been defined in the SelfCall enum here, which currently has variants that are never used in the code.
Added methods

factory_update_address_version
refund_on_error
set_eth_connector_contract_data
set_paused_flags

Note
The SelfCall enum variants related to the eth-connector-specific methods (new_eth_connector, ft_resolve_transfer, finish_deposit) have been removed to reflect the changes in the NEP-141 split PR.
Please make sure that all the method input/output arguments match those on the engine contract. I tried my best to match them but I'm still lacking some familiarity to be absolutely sure of the changes.
Other changes
Local mocked test cases for these methods have been added. The existing ft_resolve_transfer mock-evm stub method has been fixed too.

@mandreyel Pls remove private methods and also update PR description.

@mrLSD, done, please review again.


not count characters with LEN function when ref cell is empty

i use LEN function referenced to a cell, but if the cell is empty not return 0,
example:

return

expected:

Similar to #378


Selecting child select parent

re #71063

use sample from https://github.com/Microsoft/vscode/issues/71063#issuecomment-476669886
start, child process 3 is selected
open drop down select child process 2
:bug: cluster is selected

Seems similar to what @weinand noted in https://github.com/Microsoft/vscode/issues/71183#issuecomment-476674224

/duplicate #71183


Fix usage of --destination-server flag when copying/moving storage volumes between remotes

This commit https://github.com/lxc/lxd/commit/0468e1caf403ddfb11963d58eadece1bb454e8f9 (see https://github.com/lxc/lxd/issues/11061) disallowed usage of --destination-server when the source and destination were different remotes. This was to prevent an error that occurs when the destination server is not clustered.
This PR introduces a few checks on both --target and --destination target to ensure correct usage:

--target is no longer allowed if the source server is not clustered.
If the source server is clustered and the source volume is in local storage, validate that the --target flag matches the location of the source volume.
--destination-target is no longer allowed if the destination server is not clustered.

Note, I'm not certain that #11061 is truly fixed by this PR.

Linter is failing with Checking that there are no trailing spaces in shell scripts... or at least looks like it is

If the source server is clustered and the source volume is in local storage, validate that the --target flag matches the location of the source volume.

@markylaing can you help me understand why this is needed?

Linter is failing with Checking that there are no trailing spaces in shell scripts... or at least looks like it is

Looks like it's actually failing here: https://github.com/lxc/lxd/actions/runs/4126701428/jobs/7128967115#step:6:493
Our usage of run-parts in the static-analysis Makefile target obfuscates the error. I'll create a separate PR to add an --exit-on-error flag.

Linter is failing with Checking that there are no trailing spaces in shell scripts... or at least looks like it is

Looks like it's actually failing here: https://github.com/lxc/lxd/actions/runs/4126701428/jobs/7128967115#step:6:493
Our usage of run-parts in the static-analysis Makefile target obfuscates the error. I'll create a separate PR to add an --exit-on-error flag.

Never mind I can see that you've already done this @stgraber

jenkins: test this please

LGTM, just waiting for tests...

jenkins: test this please

Testsuite passed

Looks like there is still a failure in the clustering storage test for LVM but I can't see how it is related to my change as it is backend agnostic. Is this a common error @tomponline?

Thats ok, its probably a race in the underlying storage driver.


fix(examples): map height and width were 0px for custom provider

Good job ! Thank you very much.
That's valuable contribution as there are that much to do...

OK

OK

Closed in https://github.com/Dominique92/ol-geocoder/releases/tag/v4.3.0


⚠ Bofrost.de has degraded performance

In 5601fa4, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2044 ms

Resolved: Bofrost.de performance has improved in 327d3dc.


Is there a way to export the heatmap from Keymap?

I'm intersted in trying alternative keyboard layouts, so I'd like to save the heatmap after I do a particular activity (coding or writing text, for example) and then compare the heatmap with my current layout to generate a report on which characters I have used most.

Hi @SJaffa ,
There is no way to do that currently, I think that would be a change to do in Keymapp (A button next to reset that says "export"?)
Which format would you expect? Something grokable programatically (csv, json, ...) or a "visual" screenshot, something like a pdf of some sort ?

Machine-readable please. At the moment I'm working from screenshots of the heatmap, but its tiresome to type it all out.

If I may add to what @SJaffa said: It would be also amazing if it were possible to see which key has been triggered. I.e., if I have a tap and a hold action on the same key, both would show up separate.


Updated nan version to support Node v4

can we get this merged in please?

Hello, @MobileSam and @SupremeTechnopriest.
nan has been updated in 399f22901dadecc8132aa86cc638c7a379bc61b2.
Although late, this issue is resolved.
Closing for now.


CleanTarget for AzureBlob File Copy

Hi there, I'm using vso - was wondering if there're any way to clean the target container/ or target folder before copying AzureBlob (not VM) File Copy? thanks a lot!

Currently this feature is not available, we are welcome for your contributions as well, if you can do it please send a PR. Otherwise we are adding to our backlog we will update you once it is done.

@wilsonng
We have this on our backlog. However it will be a while before we get to it


Build single distfile

That's really not working very well, quite some bugs here, any better suggestion is welcome
This is a config made from the one in yourlabs/facond.
The point is to compile our js into a single file with jquery and select2 in it, to bypass django's form media management system.

@luzfcb this loads the right language file, but loading the language file fails because of a JS error.
Any idea please ?

@jpic What is the correct way to build this?
I installed nodejs via https://pypi.python.org/pypi/nodeenv and i run npm install, but a got a lot of error messages

Normally you only need to run sudo npm install -g yarn; yarn install inside the repo

You don't need nodeenv, yarn install will populate the local directory's node_module directory.
If you want a crash course for npm: https://blog.yourlabs.org/post/171557820263/js-tdd-in-your-django-apps

Wouldn't there be a problem if the user includes another jQuery in addition to our bundle?
Maybe we should treat jQuery as external dependency: https://webpack.js.org/configuration/externals/ ?

BTW, what's the exact problem with Media in Django 2.0?

There's a report that having a datetime widget and a dal widget on the same admin form would break js for one of the two.
Also an RFC in progress for Django: https://github.com/django/django/pull/9743
On django-webpack-loader I'm suggesting a completely different approach, what i think it should look like in the future, and supported by adapters: https://github.com/ezhome/django-webpack-loader/issues/141

Fixed in 3.7.0.dev0


switch from opt-in to opt-out for analytics

Because the user isn't able to opt in.

i think we need to default to some amount of analytics. i am very open to letting users opt out. re: cookies - we should investigate the requirements here more closely.
Originally posted by @jbibla in https://github.com/cosmos/voyager/issues/1718#issuecomment-456448624

This is a prerequesit for removing the pref page

i am having second thoughts about this

I think GDPR requires opt-in


Visual Studio 17.1.0 and SDK 22000 Problem

Brief description of your issue
Hello!
Just installed MS VS 16.11.10 (with [x] SDK 19041) and MS VS 17.1.0 (with [x] SDK 22000). Both are fresh installation on a fresh Windows 11 22000.527.

Looks like it can't find SDK 22000 though it's installed and checked in the VS installer.
Steps to reproduce (optional)

install windows 11 22000.527
install ms vs 16.11.10 with [x] SDK 19041
install ms vs 17.1.0 with [x] SDK 22000

Expected behavior (optional)
I can build PH with no problems
Actual behavior (optional)
Custom build tool can't find the latest SDK 22000
Environment (optional)
The latest PH trunk revision from repo
Windows 11 22000.527

@dmex could you please help to investigate the problem? Thanks!

You need to execute \build\build_thirdparty.cmd and recompile the static library dependencies.

Thanks!
So the SDK 19041 is the right one for now?
I think the last time I built my own build of PH it was using SDK 22000. It was ~ 3 weeks ago. Di you change it in the latest commits?

So the SDK 19041 is the right one for now?

No.
The build tools use the Visual Studio API to show the latest installed SDK version... The latest Visual Studio update broke that API and it now returns identical version numbers for every Windows SDK installed and the build tools can't determine the latest SDK  because the versions are all identical.

Oh, thanks for the explanation! Another MS fault then :(


Documentation patch

Under custom article collections, at http://middlemanapp.com/basics/blogging/ the documentation should note that:
-- custom collections REQUIRE a template and that
-- the template must be stored in the /source directory.
I spent about three hours banging my head against this today.

As far as I work custom collections they always do require a template as below : they can be stored in a folder inside source.
    # Custom collections
    blog.custom_collections = {

        # Category
        category: {
            link:     "#{ path }{category}.html",
            template: "template/category.html"
        },

        # Author
        author: {
            link:     "#{ path }{author}.html",
            template: "template/author.html"
        }

    }


Modify default variables to install self-signed certificates by default

Recent changes to OMERO.py removing the ADH ciphers combined with increased security requirements on modern platforms means that an OMERO.server without self-signed certificates is unlikely to be able to function correctly.
This changes proposes to set the default value of the internal variable to true so that certificates are generated automatically on startup.
Note this is a backwards-incompatible change and should likely be included in a release increasing the major version number of this role.

Negative test

Comment out the definition of the variable from the used playbook (training-type of playbook, see https://github.com/openmicroscopy/management_tools/pull/1710)

  #omero_server_selfsigned_certificates: True

Delete /opt/omero and /OMERO/*, stop and disable server and web services, uninstall postgres and nginx.
Make sure the ome.omero_server role is installed (== without this PR)
Observe that playbook finishes successfully, but the server is actually not responding to clients, insight, web and cli all claim Ice.ConnectionRefusedException: localhost:4064 isn't running or similar (server not responding for UI clients).
omero admin diagnostics see below

Server:     icegridnode                    running
Server:     Blitz-0                        active (pid = 277466, enabled)
Server:     Indexer-0                      active (pid = 277488, enabled)
Server:     OMERO.Glacier2                 inactive (disabled)
Server:     OMERO.IceStorm                 active (pid = 277500, enabled)
Server:     Processor-0                    active (pid = 277499, enabled)
Server:     Tables-0                       active (pid = 277489, enabled)

Unfortunately, I am having the same (==fail) result with or without this PR. The fact that the defaults are changed does not seem to matter so much. The only thing that matters is whether or not the
omero_server_selfsigned_certificates: True
is defined in the playbook (== successful start and functioning of the server) or is not defined in the playbook (== "successful" start, but without Glacier2 service, clients do not connect.)
Is it maybe that the variable is crucial for other roles too ?
I can say that this PR is not harmful, i.e. I am able to start the OMERO server and use it with this PR.
But not sure how do I profit in my workflow with this -> still need to define the variable
omero_server_selfsigned_certificates: True
as well.

Maybe some clue could be inside the play record of the playbook
If the server does not work, then the outupt of a task (see below) is claiming -certificates -v
TASK [pwalczysko.omero_server : omero server | configuration 00-omero-server.omero] ***
--- before: /opt/omero/server/config/00-omero-server.omero
+++ after: /Users/pwalczysko/.ansible/tmp/ansible-local-85556vivnmgfq/tmpl0v9v3db/00-omero-server-omero.j2
@@ -26,4 +26,3 @@
 config set -- omero.jvmcfg.percent.pixeldata 20
 config set -- omero.server.nodedescriptors master:Blitz-0,Indexer-0,Processor-0,Storm,Tables-0

-certificates -v

changed: [<IP_ADDRESS>]

Whereas if it does work, the output of the same task is
TASK [pwalczysko.omero_server : omero server | configuration 00-omero-server.omero] ***
--- before: /opt/omero/server/config/00-omero-server.omero
+++ after: /Users/pwalczysko/.ansible/tmp/ansible-local-88093bz3js2_w/tmpqy1uruu7/00-omero-server-omero.j2
@@ -26,3 +26,4 @@
 config set -- omero.jvmcfg.percent.pixeldata 20
 config set -- omero.server.nodedescriptors master:Blitz-0,Indexer-0,Processor-0,Storm,Tables-0

+certificates -v

changed: [<IP_ADDRESS>]

Edit:
Deleted the previous text. The error was on my side, as I trusted too much that the galaxy install will perform as expected. The role was not published correctly, and the changes from this PR were not included, this is why I falsely concluded that this PR is not sufficient.
Sorry for the back and forth.

Actually I do think though that this PR should be rebased on master please @sbesson  ? (I tested the changes here in a setup where I cherry-picked them on top of master and then pushed a new role to ansible galaxy - this is why I got a bit entangled in that :)
I think that it would be good to have an additional OK from GHA here.

Merged the mainline including the Rocky9/Ubuntu 22.04 changes