'Rolling' object has no attribute 'cov' in dask.dataframe

Hi,
I wanted to compute some rolling covariance matrix in a large dataset, which is quiet computationally expensive. But I am facing an issue using dask.dataframe. Here is a reproducing code:
import numpy as np
import pandas as pd
import dask.dataframe as dd

d = 20
n = 1000

df = pd.DataFrame(np.random.random((n, d)))
ddf = dd.from_pandas(df, npartitions=3)
cov = df.rolling(100).cov()
dcov = ddf.rolling(100).cov()

Output:
pandas version: 0.23.4
dask version: 0.19.2
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-24-a4ca6b7745d7> in <module>()
     13 ddf = dd.from_pandas(df, npartitions=3)
     14 cov = df.rolling(100).cov()
---> 15 dcov = ddf.rolling(100).cov()

AttributeError: 'Rolling' object has no attribute 'cov'

Regards,
Nazih.

Thank you for the excellent issue @NazBen .  I'm curious why this wasn't implemented originally.  I think that most of the rolling algorithms are pretty consistent, so I'm not sure why this would have been missed.
Would you be interested in submitting a small pull request that adds cov?
You can probably add the implementation here:
https://github.com/dask/dask/blob/5c23342850c7cebdd638d34ee55b6a9d0a4b8fac/dask/dataframe/rolling.py#L251-L261
And add tests here:
https://github.com/dask/dask/blob/5c23342850c7cebdd638d34ee55b6a9d0a4b8fac/dask/dataframe/tests/test_rolling.py#L96-L99

I looked into this bug a little but and a couple of things stood out to me.

Compared to all of the other methods like mean, sum, etc., cov has a 2-D output whereas the other methods have a 1-D output.

As the window size increases, the dask dataframe's shape will increasingly mismatch the pandas dataframe's shape, due to some artifact from #1 I guess. I have a couple examples describing it below, perhaps this could help someone else solve this issue,

Using the cov method, the difference in dataframe shapes is a function of npartitions and window_size:
N=10
df = pd.DataFrame({'a': np.random.randn(N).cumsum(),
                   'b': np.random.randint(100, size=(N,)),
                   'c': np.random.randint(100, size=(N,)),
                   'd': np.random.randint(100, size=(N,)),
                   'e': np.random.randint(100, size=(N,))})
npartitions = 2
ddf = dd.from_pandas(df, npartitions)

window_size = 5
a = df.rolling(window_size).cov()
b = ddf.rolling(window_size).cov().compute()
print(a.shape, '?=', b.shape)
# (50, 5) ?= (66, 5)

dask_diff = 2**npartitions * (window_size-1)
print(dask_diff)  # == 16
shape_diff = b.shape[0] - a.shape[0]
print(shape_diff) # == 16

Using any other method with a 1-D output like sum(), mean(), etc., the shapes will match.
N=10
df = pd.DataFrame({'a': np.random.randn(N).cumsum(),
                   'b': np.random.randint(100, size=(N,)),
                   'c': np.random.randint(100, size=(N,)),
                   'd': np.random.randint(100, size=(N,)),
                   'e': np.random.randint(100, size=(N,))})
npartitions = 2
ddf = dd.from_pandas(df, npartitions)

window_size = 5
a = df.rolling(window_size).mean()
b = ddf.rolling(window_size).mean().compute()
print(a.shape, '?=', b.shape)
# (10, 5) ?= (10, 5)

dask_diff = 2**npartitions * (window_size-1)
print(dask_diff)  # == 16
shape_diff = b.shape[0] - a.shape[0]
print(shape_diff) # == 0

@mrocklin Can you guide me a little about this issue. I think i can work on it.

@HSR05 where are you stuck at? Does the summary from https://github.com/dask/dask/issues/4053#issuecomment-466738711 make sense? Do you need help writing tests, or implementing the fix?

@TomAugspurger I just started looking into it again and it is making sense to me. I just need a little guidance in implementing the fix. Can you guide me?

Closed by #5154. Thanks all!


Inflector issue

[ErrorException]
 The "Doctrine\Common\Inflector\Inflector::tableize" method is deprecated and will be dropped in doctrine/inflector 2.0. Please update to the new Inflector API.
in /app/vendor/doctrine/inflector/lib/Doctrine/Common/Inflector/Inflector.php:80

Exception Trace:
 Spiral\Boot\ExceptionHandler::handleError() at /app/vendor/doctrine/inflector/lib/Doctrine/Common/Inflector/Inflector.php:80
 Spiral\Boot\ExceptionHandler::handleError() at n/a:n/a
 trigger_error() at /app/vendor/doctrine/inflector/lib/Doctrine/Common/Inflector/Inflector.php:80
 Doctrine\Common\Inflector\Inflector::tableize() at /app/vendor/cycle/annotated/src/Entities.php:191
 Cycle\Annotated\Entities->tableName() at /app/vendor/cycle/annotated/src/Entities.php:106
 Cycle\Annotated\Entities->run() at /app/vendor/cycle/schema-builder/src/Compiler.php:63
 Cycle\Schema\Compiler->compile() at /app/vendor/spiral/framework/src/Bootloader/Cycle/SchemaBootloader.php:113
 Spiral\Bootloader\Cycle\SchemaBootloader->schema() at n/a:n/a
 ReflectionMethod->invokeArgs() at /app/vendor/spiral/core/src/Container.php:506
 Spiral\Core\Container->evaluateBinding() at /app/vendor/spiral/core/src/Container.php:166
 Spiral\Core\Container->make() at /app/vendor/spiral/core/src/Container.php:133
 Spiral\Core\Container->get() at /app/vendor/spiral/prototype/src/Bootloader/PrototypeBootloader.php:170
 Spiral\Prototype\Bootloader\PrototypeBootloader->initCycle() at /app/vendor/spiral/prototype/src/Bootloader/PrototypeBootloader.php:112
 Spiral\Prototype\Bootloader\PrototypeBootloader->boot() at n/a:n/a
 ReflectionMethod->invokeArgs() at /app/vendor/spiral/boot/src/BootloadManager.php:124
 Spiral\Boot\BootloadManager->initBootloader() at /app/vendor/spiral/boot/src/BootloadManager.php:98
 Spiral\Boot\BootloadManager->boot() at /app/vendor/spiral/boot/src/BootloadManager.php:66
 Spiral\Boot\BootloadManager->Spiral\Boot\{closure}() at /app/vendor/spiral/core/src/Container.php:282
 Spiral\Core\Container->runScope() at /app/vendor/spiral/boot/src/BootloadManager.php:67
 Spiral\Boot\BootloadManager->bootload() at /app/vendor/spiral/framework/src/Framework/Kernel.php:35
 Spiral\Framework\Kernel->bootstrap() at /app/vendor/spiral/boot/src/AbstractKernel.php:142
 Spiral\Boot\AbstractKernel::Spiral\Boot\{closure}() at /app/vendor/spiral/core/src/ContainerScope.php:50
 Spiral\Core\ContainerScope::runScope() at /app/vendor/spiral/core/src/Container.php:279
 Spiral\Core\Container->runScope() at /app/vendor/spiral/boot/src/AbstractKernel.php:143
 Spiral\Boot\AbstractKernel::init() at /app/app.php:18

Did you try to composer update. It has already been patched, thx to Doctrine for BC.

I am using latest version
{
            "name": "cycle/annotated",
            "version": "v2.0.5",
            "source": {
                "type": "git",
                "url": "https://github.com/cycle/annotated.git",
                "reference": "bdb0183d5cb22f766178300ab8454ffdfa994674"
            },

Seems not all classes updated, this one still uses deprecated methods that trigger this error.
https://github.com/cycle/annotated/blob/master/src/Entities.php#L191

Got it.

Check v2.0.6

Works, good jobs, thanks


Introduce the Float field type

Introduce the Float field type and suppress the resource not found error on loading on-disk streams/measures.
Related to https://github.com/apache/skywalking/issues/8940

Why should we suppress the resource not found error?

Why should we suppress the resource not found error?

A stream or measure might not link to any index rules. Considering a measure without index rules, throwing such an error would mess up the initialization. In the OAP scenario, the installation procedure will be interrupted when loading existing measures from the disk. Then the OAP server will crash down.

Why should we suppress the resource not found error?

A stream or measure might not link to any index rules. Considering a measure without index rules, throwing such an error would mess up the initialization. In the OAP scenario, the installation procedure will be interrupted when loading existing measures from the disk. Then the OAP server will crash down.

Do you mean due to last time bug? I don't think we intend to have no rule.

I don't think we intend to have no rule.

Service-level metrics don't need any index rules. All searchable tags belong to the entity(SeriesID).

Do you mean due to last time bug?

It's from an improvement more than a bug. We don't create the indexRuleBinding when there is no indexRule. But the measure initialization procedure is based on the assumption that there is an indexRuleBinding at least. That is why an indexRuleBinding not found error is raised.


Deduplication

The Backup agent FAQ specifies that it works with servers using Windows Server 2012+ deduplication feature.
Does VM backup using the Backup extension also support deduplication?

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

ID: 88beaa58-adf5-28ef-5510-01ace6923eef
Version Independent ID: 02b2de6f-ba0a-d9dc-fd4f-c0c4d3a46107
Content: Azure VM Backup FAQ
Content Source: articles/backup/backup-azure-vm-backup-faq.md
Service: backup
GitHub Login: @trinadhk
Microsoft Alias: trinadhk

Thank you @SadiqhAhmed-MSFT. It might be useful to add this information to the VM Backup FAQ, especially as it differs from the Backup agent specification. I'll leave the thread closing to your discrection.

@dcbrown16 Thank you! We will share your valuable feedback with the content developer and the Product Group to make necessary changes in the document.
We will now proceed to close this issue. Please feel free to share your ideas, suggestions and feedbacks by tagging me in your reply and I shall reopen the issue to continue the discussion.

@SadiqhAhmed-MSFT sorry, I need to reconfirm here.
If you are using deduplication on an Azure VM, is VM backup using the extension still guaranteed as far as backup and restore of files, VM state etc.? Or does using deduplication completely break the backup?

@dcbrown16 Yes, you would be able to backup and restore files. For more clarity see the FAQ


feat: schema library support

This is split from https://github.com/googleapis/nodejs-pubsub/pull/1262 because that one was getting too big. This PR covers only the actual library support, and test changes that happened around making that happen.

There are unit tests, but not system-tests yet. I have tested the functionality by hand using samples in another PR, but I am hoping to do the system-tests in a separate PR to try to keep it smaller.


add a image with summary for showing  in tensorboard

Hi,
for add a image to graph for showing in tensorboard , i used below code. but the tensorflow error that :"TypeError: Parameter to MergeFrom() must be instance of same class: expected Summary got Tensor. for field Event.summary"
the code :
<<  writer =tf.summary.FileWriter('C:\Program Files\python3.5\logs',graph=tf.get_default_graph())
<v
<<tf.Tensor 'Slice_2:0' shape=(1, 152, 138, 1) dtype=float32>
<<imgsumarry=tf.summary.image('image', v, 3 ,collections=None)
<imgsumarry
<<tf.Tensor 'image_2:0' shape=() dtype=string>
<< writer.add_summary(imgsumarry)
full error:
Traceback (most recent call last):
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\google\protobuf\internal\python_message.py", line 516, in init
copy.MergeFrom(new_val)
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\google\protobuf\internal\python_message.py", line 1208, in MergeFrom
'expected %s got %s.' % (cls.name, msg.class.name))
TypeError: Parameter to MergeFrom() must be instance of same class: expected Summary got Tensor.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "<pyshell#579>", line 1, in
writer.add_summary(imgsumarry)
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\tensorflow\python\summary\writer\writer.py", line 113, in add_summary
event = event_pb2.Event(summary=summary)
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\google\protobuf\internal\python_message.py", line 518, in init
_ReraiseTypeErrorWithFieldName(message_descriptor.name, field_name)
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\google\protobuf\internal\python_message.py", line 446, in _ReraiseTypeErrorWithFieldName
six.reraise(type(exc), exc, sys.exc_info()[2])
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\six.py", line 685, in reraise
raise value.with_traceback(tb)
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\google\protobuf\internal\python_message.py", line 516, in init
copy.MergeFrom(new_val)
File "C:\Program Files\python3.5\python-3.5.3.amd64\lib\site-packages\google\protobuf\internal\python_message.py", line 1208, in MergeFrom
'expected %s got %s.' % (cls.name, msg.class.name))
TypeError: Parameter to MergeFrom() must be instance of same class: expected Summary got Tensor. for field Event.summary

You need to eval the imgsumarry tensor to get a string first.

Many Thanks


Publish updated version to NPM

NPM still has a version of this library with an Engine restriction:
http://search.npmjs.org/#/ipv6
This has been fixed in master, but the published version on NPM still contains the engine restriction, which prevents NPM from installing it, even though it works in Node v0.6

Thanks for letting me know, I'll update this right now. :)

Alright, all pushed--I also updated the 'stable' tag to 1.1.0.


Update Monolingual.app to v1.7.1

Thank you for the contribution. It was merged directly as ffaa271f31a5c3799ceaca00c302cfa3d657bba7 to keep commit history cleaner. Your contribution is still credited to you.


Robot Framework and websockets integration results a RuntimeError exception

I need to use websockets lib for test creation with the help of Robot Framework application. RF can import standard python class and use it's methods as test steps/actions. The issue is during test execution, I receive [ ERROR ] Task was destroyed but it is pending!, RuntimeError: Event loop is closed. These errors and exceptions do not affect test execution but they waste my test logs and reports.
The simplified python class which I import in test:
class WebsocketSubscription:

    #library can be used only in the context of a test case
    ROBOT_LIBRARY_SCOPE = 'TEST'

    def __init__(self, adr):
        self.adr = adr
        self._ws = None
        self
        self._loop = asyncio.new_event_loop()
        asyncio.set_event_loop(self._loop)
        self._loop.run_until_complete(self._ws_connect())

    async def _ws_connect(self):
        self._ws = await websockets.connect(self.adr)

    async def _ws_close(self):
        await self._ws.close()

    def ws_close(self):
        self._loop.run_until_complete(self._ws_close())
        self._loop.close()

Robot framework test case:
*** Settings ***
Library		../../libs/Tester.py

*** Test Cases ***
Import Library		Tester.WebsocketSubscription	ws://[en53:6ab7:383:2::5]:8000/
Ws Close

Test execution log:
==============================================================================
Test
==============================================================================
Test       							      | PASS |
------------------------------------------------------------------------------
[ ERROR ] Task was destroyed but it is pending!
task: <Task pending coro=<WebSocketCommonProtocol.transfer_data() done, defined at /home/dena/.local/lib/python3.7/site-packages/websockets/protocol.py:818> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7fbb63af12d0>()]> cb=[<TaskWakeupMethWrapper object at 0x7fbb63af1390>()]>
[ ERROR ] Task was destroyed but it is pending!
task: <Task pending coro=<WebSocketCommonProtocol.keepalive_ping() done, defined at /home/dena/.local/lib/python3.7/site-packages/websockets/protocol.py:1103> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7fbb63af1350>()]>>
[ ERROR ] Task was destroyed but it is pending!
task: <Task pending coro=<WebSocketCommonProtocol.close_connection() done, defined at /home/dena/.local/lib/python3.7/site-packages/websockets/protocol.py:1153> wait_for=<Task pending coro=<WebSocketCommonProtocol.transfer_data() done, defined at /home/dena/.local/lib/python3.7/site-packages/websockets/protocol.py:818> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7fbb63af12d0>()]> cb=[<TaskWakeupMethWrapper object at 0x7fbb63af1390>()]>>
Exception ignored in: <coroutine object WebSocketCommonProtocol.close_connection at 0x7fbb63b3c0e0>
Traceback (most recent call last):
  File "/home/dena/.local/lib/python3.7/site-packages/websockets/protocol.py", line 1204, in close_connection
    self.transport.close()
  File "/usr/lib/python3.7/asyncio/selector_events.py", line 653, in close
    self._loop.call_soon(self._call_connection_lost, None)
  File "/usr/lib/python3.7/asyncio/base_events.py", line 683, in call_soon
    self._check_closed()
  File "/usr/lib/python3.7/asyncio/base_events.py", line 475, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
Test                                                                 | PASS |
1 critical test, 1 passed, 0 failed
1 test total, 1 passed, 0 failed

Some observations:

Removing Ws Close (actual WebsocketSubscription class method) from test case and leaving only Import Library... (which creates WebsocketSubscription class instance) will disable these errors.
At first I got the same errors on class constructor execution (Import Library...) but asyncio.set_event_loop(self._loop) desabled them

Could somebody please help me to identify reason of the issue?

Corresponding issue report on Robot Framework page
https://github.com/robotframework/robotframework/issues/3581

Can you provide the definition of self._ws_close()? It looks like it doesn't do what it should.

@aaugustin sorry, I've missed that, original post is updated. Actually removing closure of socket and leaving only self._loop.close() results the same type of error.

If you run the loop once (or a few times) before closing it, does that help?
    def ws_close(self):
        self._loop.run_until_complete(self._ws_close())
        # Run loop once to let websockets clean up
        self._loop.call_soon(self.loop.stop)
        self._loop.run_forever()
        self._loop.close()

If you run the loop once (or a few times) before closing it, does that help?
    def ws_close(self):
        self._loop.run_until_complete(self._ws_close())
        # Run loop once to let websockets clean up
        self._loop.call_soon(self.loop.stop)
        self._loop.run_forever()
        self._loop.close()

I've put the same code but it gives the same errors:
Task was destroyed but it is pending!
task: <Task pending coro=<WebSocketCommonProtocol.keepalive_ping() running at /home/developer/.local/lib/python3.7/site-packages/websockets/protocol.py:1121> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7fc46f55b750>()]>>
Task was destroyed but it is pending!
task: <Task pending coro=<WebSocketCommonProtocol.transfer_data() running at /home/developer/.local/lib/python3.7/site-packages/websockets/protocol.py:827> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7fc46f55b6d0>()]> cb=[<TaskWakeupMethWrapper object at 0x7fc46f55b790>()]>
Task was destroyed but it is pending!
task: <Task pending coro=<WebSocketCommonProtocol.close_connection() running at /home/developer/.local/lib/python3.7/site-packages/websockets/protocol.py:1169> wait_for=<Task pending coro=<WebSocketCommonProtocol.transfer_data() running at /home/developer/.local/lib/python3.7/site-packages/websockets/protocol.py:827> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7fc46f55b6d0>()]> cb=[<TaskWakeupMethWrapper object at 0x7fc46f55b790>()]>>
Exception ignored in: <coroutine object WebSocketCommonProtocol.close_connection at 0x7fc46f5528c0>
Traceback (most recent call last):
  File "/home/developer/.local/lib/python3.7/site-packages/websockets/protocol.py", line 1204, in close_connection
    self.transport.close()
  File "/usr/lib/python3.7/asyncio/selector_events.py", line 653, in close
    self._loop.call_soon(self._call_connection_lost, None)
  File "/usr/lib/python3.7/asyncio/base_events.py", line 683, in call_soon
    self._check_closed()
  File "/usr/lib/python3.7/asyncio/base_events.py", line 475, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed

I've rechecked the issue by executing python script:
ws = WebSocketSubscription("ws://[en53:6ab7:383:2::5]:8000/")
ws.close()

and the output is clean, so issue is definitely in chain Robot framework - websockets - asyncio

If this works:
ws = WebSocketSubscription("ws://[en53:6ab7:383:2::5]:8000/")
ws.close()

then I believe this rules out websockets and the code provided in your initial message as the cause of the problem? (assuming close here is the same thing as ws_close in your initial post) For this reason I'm going to close the issue.
Are you sure that close() is actually called?
If you can provide a reproducible example without involving Robot framework, I'm happy to look at it. However, I'm not willing to spend time fighting Robot framework, which I'm not familiar with.


Bug: floating point error

Recently we observed a bug in mapcoder library which could be reproduced by providing a string '40822.schol' as a mapcode to decodeMapcodeToLatLonUtf8.
The problem located here:
mapcoder.c:1453
    divy = SMART_DIV(m);
    if (divy == 1) {
        divx = X_SIDE[prelen];
        divy = Y_SIDE[prelen];
    } else {
        divx = (NC[prelen] / divy);
    }

divy can be zero that causes division by zero exception. We workarounded the problem by changing 'if' condition from == 1 to <= 1, but probably better fix could be made.
[...]

Fixed in 2.5.2.


Fixed bug: type 'Null' is not a subtype of type 'List' in type cast

Anyone can use the null-safety branch while waiting for it to be released
dependencies:
  # ...
  excel:
    git:
      url: https://github.com/chuyentt/excel.git
      ref: null-safety

Thanks for the pull-request friend.
Before this PR, #135 has already resolved it.
But I really appreciate it, you opening a PR.

how can I apply this fixed in my flutter project?

Same question

@miguelahc @YowFung , use this in pubspec.yaml
  excel:
    git:
      url: https://github.com/justkawal/excel.git
      ref: null-safety


[bazel] build with -c dbg flag

Bazel allows for control of compilation mode as described in their documentation:
"--compilation_mode (fastbuild|opt|dbg) (-c)

The --compilation_mode option (often shortened to -c, especially -c opt) takes an argument of fastbuild, dbg or opt,
and affects various C/C++ code-generation options, such as the level of optimization and the completeness of debug
tables. Bazel uses a different output directory for each different compilation mode, so you can switch between modes
without needing to do a full rebuild every time.

fastbuild means build as fast as possible: generate minimal debugging information (-gmlt -Wl,-S), and don't optimize.
This is the default. Note: -DNDEBUG will not be set.
dbg means build with debugging enabled (-g), so that you can use gdb (or another debugger).
opt means build with optimization enabled and with assert() calls disabled (-O2 -DNDEBUG). Debugging information
will not be generated in opt mode unless you also pass --copt -g."

This should be changed from the default fastbuild to debug, but doing so appears to cause issues with our builds that should be debugged.

@cfrantz could you confirm this is still an issue? I think we decided that we didn't want to bother making the mask ROM build in -c dbg. I think there's a separate question about making fastbuild be -O1, because the code clang emits in -O0 is hot garbage. Clang allocates everything on the stack in -O0 and needs a mem2reg pass from LLVM to be halfway decent. If we don't want to build at -O1 I think -mllvm -mem2reg might do what we want...?

lowRISC/bazel-embedded#9 updated fastbuild to use -Os and -g


Payloads cleanup

Decouple messages payloads by name. Removed unnecessary cases (they can be viewed at payloadmap.js and used when needed). Added support for two parameters logTraffic and logData (they may be useful for better understanding of the communication).
Usage:
$ node app.js logTraffic => Logs the message name.
$ node app.js logData => Logs the payload data.
$ node app.js logData logTraffic => logs both.
[recommended] Use with chrome debugger:
$ npm install -g node-inspector
$node-debug app.js logData logTraffic

Magic numbers were replaced by constants.
Please adjust your code.

Ok done.

Merge conflict

No longer needed


Warn on ununsed directive

Fixes #354
Adds a new flag to enable the new behaviour. When the flag is enabled all the files are parsed for directives (otherwise warnings would only exist for files with at least one issue).  Once all linting is done all the ignore ranges are checked and new issues are created for any ignore range that was never matched to an issue.
The order of issue processing operations was changed to filter before sorting so that the new nolint issues are properly sorted.

Nice, thanks!


Bug fix in migraphx filename mangling

Summary of Changes
One-line fix for a bug in filename handling.  An incorrect extension was added to migraphx model file names that contain more than one period.
Draft PR because I haven't run the test scripts.

Can one of the admins verify this patch?

test this please

ok to test

retest this please

ok to test

ok to test

Build failed!

Check failed for VAI for a used port. The MIGraphX test passed.


python: Avoid relying on Collection supertype of QuerySet

QuerySet doesn’t implement __contains__, so it can’t be a subtype of Container or Collection (https://code.djangoproject.com/ticket/35154). This incorrect subtyping annotation was removed in https://github.com/typeddjango/django-stubs/pull/1925, so we need to stop relying on it before upgrading to django-stubs 5.

Preparation for #29721.

Merged, thanks @andersk!


New graphs - is useless(

http://prntscr.com/fy4g7x
sorry, but it sucks...
When I looked on it before, I saw the difference between the games. Which one is making good tail after discounts and which is not.
It was information about quality on game start, how fast it drops down,
And it was one of the most important metrics for me.  :(

This was not intentional as we lost some data. We will try restoring it this or next week.

Hi! Thanks a lot for a quick response. Ok, I thought that was some kind of optimization.
Отправлено с iPhone

20 июля 2017 г., в 18:36, Pavel Djundik<EMAIL_ADDRESS>написал(а):
This was not intentional as we lost some data. We will try restoring it this or next week.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


BinarySearchTree [Python]

Fixes #547
By submitting this pull request I confirm I've read and complied with the below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[x] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.

Hi! Thanks for showing interest in contributing to this repository.
Please make sure that you have ticked the points mentioned in PR description correctly.
Thanks again!

@avsingh999: Please mention the issue number and make sure that you are assigned to it. Thanks.

@avsingh999 Please add its entry to README

Please rebase. It's good to go @singhpratyush. Take a look

Hey! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost!

Automated rebase was successful!


Deprecate instanceId Config For Broker/Minion Specific Configs

fixes #9290
I felt it might be best to mark instanceId as deprecated and create a new config for Minion ID. Lmk your thoughts.
cc: @Jackie-Jiang

Codecov Report

Merging #9308 (782b038) into master (0adf5ef) will decrease coverage by 48.33%.
The diff coverage is 71.42%.

@@              Coverage Diff              @@
##             master    #9308       +/-   ##
=============================================
- Coverage     63.64%   15.31%   -48.34%
+ Complexity     4984      168     -4816
=============================================
  Files          1809     1820       +11
  Lines         96993    97346      +353
  Branches      14832    14907       +75
=============================================
- Hits          61735    14905    -46830
- Misses        30707    81310    +50603
+ Partials       4551     1131     -3420

Flag
Coverage Δ

unittests1
?

unittests2
15.31% <71.42%> (-0.09%)
:arrow_down:

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

Impacted Files
Coverage Δ

.../main/java/org/apache/pinot/minion/MinionConf.java
35.29% <0.00%> (-4.71%)
:arrow_down:

...va/org/apache/pinot/spi/utils/CommonConstants.java
0.00% <ø> (-27.70%)
:arrow_down:

...e/pinot/broker/broker/helix/BaseBrokerStarter.java
72.09% <100.00%> (+0.26%)
:arrow_up:

...src/main/java/org/apache/pinot/sql/FilterKind.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/java/org/apache/pinot/core/data/table/Key.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...in/java/org/apache/pinot/spi/utils/BytesUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/spi/utils/BooleanUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/data/table/Record.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/util/GroupByUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/BaseRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1206 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Currently controller and server is using pinot..instance.id as the key, I'd suggest making broker and minion following the same pattern to be consistent

@Jackie-Jiang : We might need to deprecate pinot.broker.id as well in that case. Is that okay?

Currently controller and server is using pinot..instance.id as the key, I'd suggest making broker and minion following the same pattern to be consistent

@Jackie-Jiang : We might need to deprecate pinot.broker.id as well in that case. Is that okay?

@ankitsultana Per the current logic, I think we can directly modify the CONFIG_OF_BROKER_ID to pinot.broker.instance.id without worrying about backward compatibility because the field is always set within the BaseBrokerStarter and then read again

The field is read in BrokerAdminApiApplication and BaseBrokerRequestHandler.
But you are right, I took a look and both of these classes get the value from the one set by BaseBrokerStarter. I'll update the PR.

@Jackie-Jiang : Updated. One of the integration tests is failing with a "ServerSegmentMissing" error. Not sure if it is related.


[Android] Nagging dialogue when "Google Location Accuracy" is turned off.

🐛 Bug Report
On a phone with "Google Location Accuracy" turned off, invoking any method to request location updates causes a system dialog to be displayed on the screen, over the app.

"To continue, turn on device location, which uses Google's location service" with the options "No, thanks" and "OK".
Pressing "No, thanks" will dismiss the dialog and location updates will work as normal, but any future calls for location updates triggers the same dialog over and over again with no apparent way to permanently dismiss
It appears to be triggered by this line of code: https://github.com/Baseflow/flutter-geolocator/blob/e60daab16c5568ee46ff3c8851b16167c3e412e9/geolocator_android/android/src/main/java/com/baseflow/geolocator/location/FusedLocationClient.java#L227
Some debugging shows that despite location permissions being granted, and the GPS services turned on, the addOnFailureListener is called with a ResolvableApiException insisting that a related, but not required, service be enabled.
Expected behavior
Users should not see this dialog at all, it's not related to the app's location permission, nor is it a requirement to receive location updates.
Reproduction steps
Navigate into the device settings, find the "Location Services" options for the phone and disable "Google Location Accuracy", while leaving the actual "Location" service enabled.
Reuqest COARSE_LOCATION and FINE_LOCATION in your manifest, install the app and manually grant the location permissions through the app's settings page.
Make any request for locations, eg:
Geolocator.getCurrentPosition(desiredAccuracy: LocationAccuracy.best)
Configuration
Version: 9.0.2
Platform:

[  ] :robot: Android

I'm having this problem. But more severe. getCurrentPosition() actually fails, because this package mistakes Location is turned off, if "Google Location Accuracy" is disabled on my Android 12 (API 31) test device, despite Location is turned on and Google Maps successfully getting position from GPS. (The same as your testing condition, except the result is a failure rather than nagging)
I'm just using the example project in this repo for testing. Version 9.0.2, Android API 31.

I am having the same issue indeed. At first glance all our Samsung test devices had this issue. But after turning the setting off. all our test devices had this issue

I'm also facing this issue

It appears to be triggered by this line of code:
https://github.com/Baseflow/flutter-geolocator/blob/e60daab16c5568ee46ff3c8851b16167c3e412e9/geolocator_android/android/src/main/java/com/baseflow/geolocator/location/FusedLocationClient.java#L227

@simtr thank you! I copied startPositionUpdates method to create this Google dialog in Kotlin.

any solution here?

I believe I got user reports for this issue. I think the behavior is that when "Improve Location Accuracy" is turned off, the pulgin may failed to get any location and raise this dialog.


Update microshift flow with secret and name update

/hold
Need to make sure microshift workflow is green. There is some issue wit workflow as it is not taking the pull secret from env.

Closing it from the fork and creating another PR from the branch.


[root] ProtocolError: Protocol error (Target.setDiscoverTargets): Target closed.

Error Details

Error ID: 1105
Error Type: root
Error Name: ProtocolError
Error Message: Protocol error (Target.setDiscoverTargets): Target closed.
Error Occured At: Tuesday, April 26, 2022, 4:15:23 PM Eastern Daylight Time

Error Stack Trace
ProtocolError: Protocol error (Target.setDiscoverTargets): Target closed.
    at /home/aoyansarkar/HealthScreeningBot/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:75:24
    at new Promise (<anonymous>)
    at Connection.send (/home/aoyansarkar/HealthScreeningBot/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:71:16)
    at Function.create (/home/aoyansarkar/HealthScreeningBot/node_modules/puppeteer/lib/cjs/puppeteer/common/Browser.js:121:26)
    at ChromeLauncher.launch (/home/aoyansarkar/HealthScreeningBot/node_modules/puppeteer/lib/cjs/puppeteer/node/Launcher.js:129:50)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async startupBrowser (/home/aoyansarkar/HealthScreeningBot/node_modules/@healthscreening/generate-screenshot/generateScreenshot.js:29:27)
    at async startup (/home/aoyansarkar/HealthScreeningBot/src/client.js:39:9)

Error Metadata
{}

Issue Details

Posted By: @PythonCoderAS
Posted At: Thursday, April 28, 2022, 10:26:57 AM Eastern Daylight Time

Theorized to be a faulty server.


Variable explorer: allow array editor to deal with arrays with more than 2 dimensions

From<EMAIL_ADDRESS>on 2009-10-21T18:10:47Z
To deal with multiple dimensions, I think the best way would be to display
a 2 dimensional page, with some control at the top allowing the user to
select which "slice" of the data is to be edited.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=93

From pierre.raybaut on 2010-01-03T11:06:27Z
Labels: -Type-Defect Type-Enhancement

From pierre.raybaut on 2010-01-03T11:06:07Z
Status: Accepted

From pierre.raybaut on 2010-10-17T12:58:41Z
Labels: -Milestone-Release2.0 Milestone-Release2.1

From pierre.raybaut on 2010-06-19T05:46:43Z
Summary: Variable explorer: allow array editor to deal with arrays with more than 2 dimensions
Labels: Milestone-Release2.0

From pierre.raybaut on 2010-01-03T11:33:06Z
Labels: -Priority-Medium Priority-Low

From pierre.raybaut on 2011-05-18T00:27:10Z
Status: ContributorNeeded

From pierre.raybaut on 2011-05-16T01:05:15Z
Labels: Cat-VariableExplorer

From pierre.raybaut on 2012-03-18T14:04:16Z
Labels: -Type-Enhancement Type-Enh

From pierre.raybaut on 2012-03-18T14:17:42Z
Status: HelpNeeded

From pierre.raybaut on 2011-06-25T05:25:05Z
Labels: -Milestone-Release2.1

From ccordoba12 on 2014-07-23T07:00:26Z
Status: Accepted
Owner<EMAIL_ADDRESS>Labels: MS-v2.3.1

From contrebasse on 2014-07-19T13:24:06Z
There is an opened PR to enable 3d arrays: https://bitbucket.org/spyder-ide/spyderlib/pull-request/38/support-for-3d-arrays-in-arrayeditor/diff

From ccordoba12 on 2014-07-23T07:35:22Z
This issue was closed by revision 6ed57f01520a .
Status: Fixed


Reduce the contention on DispatcherSingleActiveConsumer by using a co…

…mmon thread for frequence operations
Motivation
Ensure the synchronized sections of the DispatcherSingleActiveConsumer are all executed in same thread to avoid blocking IO threads.

retest this please


Go modules won't working. this is weired.

What version of Go are you using (go version)?

$ go version
go version go1.12.14 windows/amd64 and linux xenial i think it dosen't matter..

Does this issue reproduce with the latest release?
i didn't try yet
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

$ go mod init
$ go test -v
go: finding github.com/stretchr/testify/assert latest
go: finding go.mongodb.org/mongo-driver/mongo latest
go: finding go.mongodb.org/mongo-driver/bson/primitive latest
go: finding go.mongodb.org/mongo-driver/mongo/options latest
go: finding go.mongodb.org/mongo-driver/bson latest
go: finding github.com/xdg/scram latest
go: finding github.com/aws/aws-sdk-go/aws latest
go: finding github.com/aws/aws-sdk-go/aws/awserr latest
go: finding github.com/aws/aws-sdk-go/aws/session latest
go: finding github.com/aws/aws-sdk-go/aws/credentials latest
go: finding github.com/aws/aws-sdk-go/service/ses latest
go: finding github.com/aws/aws-sdk-go/service/s3 latest
go: finding github.com/aws/aws-sdk-go/service/s3/s3manager latest
go: finding github.com/google/go-cmp/cmp latest
go: finding github.com/google/go-cmp/cmp/cmpopts latest
go: finding github.com/Azure/azure-sdk-for-go/services/servicebus/mgmt/2017-04-01/servicebus latest
go: finding github.com/Azure/azure-sdk-for-go/services/eventhub/mgmt/2017-04-01/eventhub latest
go: finding github.com/Azure/go-autorest/autorest/azure latest
go: finding github.com/aws/aws-sdk-go v1.27.1
go: finding github.com/aws/aws-sdk-go/service latest
go: finding golang.org/x/sync/semaphore latest
go: finding golang.org/x/sync latest
go: downloading github.com/aws/aws-sdk-go v1.27.1
go: finding github.com/Azure/azure-sdk-for-go/services/servicebus/mgmt/2017-04-01 latest
go: finding github.com/Azure/azure-sdk-for-go/services/eventhub/mgmt/2017-04-01 latest
go: finding github.com/Azure/azure-sdk-for-go/services/servicebus/mgmt latest
go: finding github.com/Azure/azure-sdk-for-go/services/eventhub/mgmt latest
go: finding github.com/Azure/azure-sdk-for-go/services/servicebus latest
go: finding github.com/Azure/azure-sdk-for-go/services/eventhub latest
go: finding github.com/Azure/azure-sdk-for-go/services latest
go: finding github.com/Azure/azure-sdk-for-go v38.0.0+incompatible
go: downloading github.com/Azure/azure-sdk-for-go v38.0.0+incompatible
go: extracting github.com/aws/aws-sdk-go v1.27.1
go: extracting github.com/Azure/azure-sdk-for-go v38.0.0+incompatible
go: updates to go.mod needed, but contents have changed

What did you expect to see?
go.sum file is little weird..
this is fine in the local environment.
but when I try to use it in travis or docker compose the build keeps failing
because I can't find a particular repository.
Result
get "gopkg.in/yaml.v2": found meta tag get.metaImport{Prefix:"gopkg.in/yaml.v2", VCS:"git", RepoRoot:"https://gopkg.in/yaml.v2"} at https://gopkg.in/yaml.v2?go-get=1
go: finding gopkg.in/yaml.v2 v2.2.2
go<EMAIL_ADDRESS>git fetch -f origin refs/heads/*:refs/heads/* refs/tags/*:refs/tags/* in /home/travis/gopath/pkg/mod/cache/vcs/5b0453a6aac74c86b31f0a771f02325d031eaf6ec0f0503f1cc20852de3abb0a: exit status 128:
	remote: Repository not found.
	fatal: repository<EMAIL_ADDRESS>not found

the belogik/goes repository not found error
i didn't use this repository
but always bring me the repository by the go modules
here is my go.mod file
module github.com/{MYREPO}

go 1.12

require (
	github.com/Azure/azure-sdk-for-go v38.0.0+incompatible // indirect
	github.com/Azure/go-autorest/autorest v0.9.3 // indirect
	github.com/Azure/go-autorest/autorest/adal v0.8.1 // indirect
	github.com/Azure/go-autorest/autorest/to v0.3.0 // indirect
	github.com/Azure/go-autorest/autorest/validation v0.2.0 // indirect
	github.com/DataDog/zstd v1.4.4 // indirect
	github.com/appleboy/gin-jwt/v2 v2.6.2
	github.com/aws/aws-sdk-go v1.27.1 // indirect
	github.com/fortytw2/leaktest v1.3.0 // indirect
	github.com/gin-contrib/cors v1.3.0
	github.com/gin-gonic/gin v1.5.0
	github.com/go-stack/stack v1.8.0 // indirect
	github.com/google/go-cmp v0.3.1 // indirect
	github.com/gorilla/websocket v1.4.1
	github.com/stretchr/testify v1.4.0
	github.com/xdg/scram v0.0.0-20180814205039-7eeb5667e42c // indirect
	github.com/xdg/stringprep v1.0.0 // indirect
	go.mongodb.org/mongo-driver v1.2.0
	golang.org/x/sync v0.0.0-20190911185100-cd5d95a43a6e // indirect
	gopkg.in/go-playground/validator.v9 v9.31.0
	pack.ag/amqp v0.12.5
)

What did you see instead?

Is github.com/belogik/goes a private repo? If so, please see https://golang.org/doc/faq#git_https for additional information.
Also, keep in mind that the go.mod file doesn't necessarily contain all the module versions that your project depends on, usually it only contains the direct dependencies. Take a look at your go.sum file, maybe it's there. Or use go mod why github.com/belogik/goes to check the reason.

Thank you for your answering.
github.com/belogik/goes is not my repository it's just auto created by the go modules
and i get to type go mod why github.com/belogik/goes
below is the result
# github.com/belogik/goes
(main module does not need package github.com/belogik/goes)

main module does not need thie package but i can't remove that link in the go.mod file
it's just auto create

Try this: go mod graph | grep 'github.com/belogik/goes'

$ go mod graph | grep 'github.com/belogik/goes'
<EMAIL_ADDRESS><EMAIL_ADDRESS>$ go mod why github.com/astaxie/beego
# github.com/astaxie/beego
(main module does not need package github.com/astaxie/beego)
$ go mod graph | grep 'github.com/astaxie/beego'
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Keep digging until you find the one you depend on directly: go mod graph | 'github.com/appleboy/gofight/v2'

i found it github.com/appleboy/gin-jwt/v2
i was recongnized the go.sum file changed because of the import github.com/appleboy/gin-jwt/v2 changed without v2.
but i should use this repository(v2)
because they were recommended this way link

See this: https://github.com/appleboy/gin-jwt/issues/228

thank you @aofei i love you

@Kyocfe @aofei  Already fixed https://github.com/appleboy/gin-jwt/pull/229

thank you for your quick feedback :) i'll follow it


bug(al2): Remove empty string fallback on cached image list

Issue #, if available:
Description of changes:
follow up for https://github.com/awslabs/amazon-eks-ami/pull/1810. when there are no images to cache during worker installation, the blank variable img will be passed to the pull-image.sh script.
this PR short circuits when the loop variable is blank, and adds logging to help diagnose issues with the images in the future.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Testing Done
build with the fix alleviates the issue.

/ci build
+build cache_container_images=true

had to redo the approach since the linter feel strongly about quoting array expansions
/ci build
+build cache_container_images=true

Thanks for the quick fix! 🚀


”Izvoru Bârzii” are listed in the drop-down menu

Description:
"Izvoru Bârzii" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Izvoru Bârzii" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Izvoru Bârzii, MEHEDINȚI".


Getstorybook CRNA support

why
Currently getstorybook requires a few extra steps to get working with create-react-native-app (aka CRNA). They are documented (excellently) here:
https://medium.com/@inyono/using-react-native-storybook-with-create-react-native-app-471e531bb128
what
Proposal for streamlining the setup process:

[ ] Simplify as much as possible:

[ ] Consolidate storybook/index.ios.js and storybook/index.android.js into index.js
[ ] Detect CRNA app and print out additional (minimal) manual step

[ ] Update docs: discuss why manual step and why manual step is necessary

[ ] react-native/README.md
[ ] react-native/docs
[ ] https://storybooks.js.org

Props to @orta and @tmeasday for helping figure this out and for @inyono for the awesome post.

That would be so awesome :) Btw: https://github.com/serlo-org/serlo-abc has an updated version of the blog post that e.g. picks the real app entry on build. Feel free to ask if you need input or want help from me.

Is this done?


Added Fibonacci series without recursion in CPP

Why:
Closes [Fibonacci series without using recursion #285]
What's being changed:
Added Fibonacci series without recursion in CPP
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Hey @NeelPatel31
Thanks for your contributions to this project 😃
This PR will be reviewed and merged shortly.

Preview : https://codinasion--pr29-dev-mzwvnjuf.web.app/programme/fibonaci-series-without-using-recursion/
(expires Fri, 11 Mar 2022 03:09:29 GMT)

LIVE : https://codinasion.web.app/programme/fibonaci-series-without-using-recursion/


fix: address issue #7458

CATEGORY

[x] Bug Fix

SUMMARY
Addresses the issue described in #7458

right


Allow running getInitialProps again on the client, on the initial page load

Expected Behavior

Sometimes it's really useful to use the async getInitialProps method on the client before doing the first ReactDOM.render, even on the initial page load. For example this would allow things like using import() to dynamically populate a Redux store and pass it to the page.
Current Behavior

There's no way to run getInitialProps again on the initial page load.
PR and proposed implementation

Pages can expose a runInitialPropsAgain static method which should return true or false. runInitialPropsAgain receives the same context than getInitialProps including serverProps.
static async runInitialPropsAgain({ pathname, query, serverProps, ... }) {
    return true; // Add your own logic here.
}

When it returns true, pages can check if serverProps is set and define their custom logic. Returned props are merged with the server props.
static async getInitialProps({ serverProps }) {
    if (serverProps) {
      const extension1 = await import('./extensions/custom-code-1.js')
      const extension2 = await import('./extensions/custom-code-2.js')
      const store = createStore([extension1, extension2])
      return { store }
    }
  }

serverProps won't be set again on subsequent renders, when Router is used.

I'd love to know if you would merge something like this, and if so, what do I need to change. I'm also open to any other implementation ideas and discussion.
I've also added some tests but I may need more help with that. The code is working great on my projects but I can't get it to work on the tests. I'm not sure what I may be doing wrong.
And by the way, many thanks for this project. We are loving it, next is awesome! 🙌

Thanks for the PR.
But this is not something we'd like to see on the core.
I totally get the usecase and I think this is an edge case.
What I recommend is to do this with HOC.
You don't need the support from the core.

Hey @arunoda, thanks for checking this out.
The problem with HOCs is that they are synchronous. You can't wait for anything which is async and introduce it before the first ReactDOM.render. The only way to do so is with getInitialProps.
I understand this is not a common case but anyone who needs to be able to wait for async things to reconstruct their first DOM (and match the one from the server) is going to need this.

Until we get an alternative way to deal with asynchronous client initialization on next, we are going to maintain a fork and merge next changes from time to time.
For anyone interested, we have published it under @worona/next on npm. It has the same runInitialPropsAgain API explained in the OP of this PR.
For any problems please open an issue in our repo:
https://github.com/worona/next.js

Glad to know that there is such a PR. Sad to know that this PR is rejected.
As @luisherranz said, without change in core. There is no way to do async job for first render in browser side. With HoC, it can only simulate async in componentDidMount. But, it involves re-rendering of the page after async job is done.


Jargon on the MinWin page - What is ScratchDir?

Jargon on the MinWin page - What is ScratchDir?

Think of a ScratchDir as a temporary directory for management tasks. It's basically where temporary stuff for drivers, packages, and more is stored

@billytwotone, don't download that. It's an infostealer. @ChrisTitusTech, are we starting to see an influx of these messages or what?

What are you asking me not to download? What is an infostealer?.

What are you asking me not to download? What is an infostealer?.

You don't have to worry about that. Basically, some account (perhaps a scam or a hacked account) was promoting a "fix" which involved downloading something from MediaFire. I've seen that yesterday as well, and it was a Google Chrome infostealer. Thankfully it was taken down, but expect more trouble regarding these comments.
If you want to know more about infostealers, check out this Wikipedia article

This ghost guy is spamming it in #2635.

I am worried about some of the comments.
Is MinWin safe to use?

@billytwotone, yes, it's safe to use. You start with a base Microsoft image and then use it to debloat it. It doesn't introduce malware - it only debloats the image.
There's no need to worry

I pleased the spammers have gone away. They tend to spoils things.
What if they return?

This is fixed by #3066. /close


Override serverName

via https://github.com/getsentry/raven-java/pull/193

Rebased off master.
Suppressed unimportant checkstyle error (too many args in constructor, but it's a a log4j2 PluginFactory anyway)
Added serverName config for JUL.

Also, does this close #175, or was there goal there something different?

👍
I think this is sufficient to cover #175, since at a glance this appears to prevent any usage of the HostnameCache.

Squashed: https://github.com/getsentry/raven-java/commit/ec69aca36f8bcc4540b47d44f226c837bc78a187

I think the docs were forgotten. How does one use this?
/me goes to read the code, but really I should not have to do that.
I blame @tkaemming :-)

It should be the same as any other configuration, for example like dsn here you should be able to do serverName: https://github.com/getsentry/raven-java/blob/master/raven-logback/README.md#configuration
I'll see about adding better docs tomorrow.

https://github.com/getsentry/raven-java/pull/202


demo: script to set up vanilla multicluster env

Description:
Add a script to allow bookbuyer on cluster c1 to access
bookstore on cluster c2 using MCS API. The prerequisite is
that c1 and c2 are configured as k8s contexts on the local
machine.
Testing done:
Affected area:

Functional Area

Demo
[x]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?
No.

The demo is inspired by mcs-api

Is this a breaking change?

No

Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?

No

Codecov Report

Merging #4985 (af81711) into main (39b7ad5) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4985      +/-   ##
==========================================
+ Coverage   68.40%   68.43%   +0.03%
==========================================
  Files         218      217       -1
  Lines       15844    15818      -26
==========================================
- Hits        10838    10825      -13
+ Misses       4951     4938      -13
  Partials       55       55

Flag
Coverage Δ

unittests
68.43% <ø> (+0.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/certificate/manager.go
76.67% <0.00%> (-0.40%)
:arrow_down:

pkg/utils/mtls.go
100.00% <0.00%> (ø)

pkg/webhook/server.go
0.00% <0.00%> (ø)

pkg/envoy/eds/cluster_load_assignment.go

pkg/smi/health.go

pkg/envoy/eds/builder.go
76.92% <0.00%> (ø)

cmd/osm-controller/osm-controller.go
16.94% <0.00%> (+0.14%)
:arrow_up:

pkg/envoy/eds/response.go
72.34% <0.00%> (+10.27%)
:arrow_up:

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

FYI I executed the script with c1 and c2 deployed on same subnet without OSM installed, c1 bookbuyer cannot access c2 bookstore by ClusterIP. But c1 bookbuyer can access c2 bookstore by:

c2 bookstore endpoint's pod IP
OR c2 bookstore external IP (if exposed)

With OSM installed, c1 cannot access c2 services due to failed mTLS authn.

With OSM installed, c1 cannot access c2 services due to failed mTLS authn.

Is it possible to make them talk with OSM installed?
Maybe a readme could be helpful for things like: what the configuration of the two clusters was (kind/aks) and was there additional components installed (Submariner)?


fix: partition html fail with table without tbody

This PR resolves #1807

fix a bug where when a table tagged content does not contain tbody tag but thead tag for the rows the code fails
now when there is no tbody in a table section we try to look for thead isntead
when both are not found return empty table

I would treat a table with no body as an empty table. I don't think the header content should be extracted as table content.
What happens on SEC Form 3 is that either Table 1 or Table 2 or both can contain data. If a table is empty using the head content makes it hard to compare against a table with data. It should either have content or no content. So I would leave out the or tag_elem.find('thead').

that's a fair point but wouldn't it be better if partition doesn't drop contents. We could put this element as a different category or in table as html properly mark this is just headers. Downsteam application can then filter based on the need.

I would treat a table with no body as an empty table. I don't think the header content should be extracted as table content.
What happens on SEC Form 3 is that either Table 1 or Table 2 or both can contain data. If a table is empty using the head content makes it hard to compare against a table with data. It should either have content or no content. So I would leave out the or tag_elem.find('thead').

that's a fair point but wouldn't it be better if partition doesn't drop contents. We could put this element as a different category or in table as html properly mark this is just headers. Downsteam application can then filter based on the need.

On the topic of dropping content, don't you skip the thead in the case where the tbody is present? I don't think HTMLTable gets the headers in the happy flow


Update xpk.py

Remove server argument from proxy server arguments.
Fixes / Features

Without this change, the proxy server was not starting up for me. I'd get an unknown argument error.

Testing / Documentation
This unblocked me from running a workload.

Hey Sadi,
I am curious to know if any logic around this argument was recently changed in Pathways, as I did not face this issue.

Looking at the Build Test failure, you may need to create a PR from a branch instead of a fork. I provided you permissions to do this, just in case.

Since it's such a small change @RoshaniN aligned with me to incorporate this PR in #100 . I'll close this PR.


Remove simpleSuccess and simpleFailure from Dispatcher Interface

Confusing to users to have access to what is an internal Redux function

Merged in https://github.com/cashapp/misk-web/pull/641


Master

Updated the User Manual and Preferences for clarity and formatting purposes.

Good to merge. Thanks for correcting my poor english.

Not at all! Thank you for contributing to the open-source community, and for your work on this app! It is a pleasure to work with you.
P.S. Thanks for the heads-up on the Gradle 2.3.0-rc1 performance warning on the most recent commit. I'll be sure to avoid it for now and stick with the stable version!


Install failed withe go 1.19.2

Describe the bug
Hi team,
When I install sqls by
go install github.com/sqls-server/sqls@latest

I got below error
# runtime/internal/sys
/usr/local/go/src/runtime/internal/sys/consts.go:16:7: DefaultPhysPageSize redeclared in this block
        /usr/local/go/src/runtime/internal/sys/arch_amd64.go:10:2: other declaration of DefaultPhysPageSize
/usr/local/go/src/runtime/internal/sys/consts.go:20:7: PCQuantum redeclared in this block
        /usr/local/go/src/runtime/internal/sys/arch_amd64.go:11:2: other declaration of PCQuantum
/usr/local/go/src/runtime/internal/sys/consts.go:23:7: Int64Align redeclared in this block
        /usr/local/go/src/runtime/internal/sys/arch_amd64.go:12:2: other declaration of Int64Align
/usr/local/go/src/runtime/internal/sys/consts.go:30:7: MinFrameSize redeclared in this block
        /usr/local/go/src/runtime/internal/sys/arch_amd64.go:13:2: other declaration of MinFrameSize
/usr/local/go/src/runtime/internal/sys/stubs.go:16:7: StackGuardMultiplier redeclared in this block
        /usr/local/go/src/runtime/internal/sys/consts.go:13:7: other declaration of StackGuardMultiplier
# runtime/internal/atomic
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:18:6: Load redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:17:6: other declaration of Load
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:24:6: Loadp redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:23:6: other declaration of Loadp
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:30:6: Load64 redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:29:6: other declaration of Load64
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:36:6: LoadAcq redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:35:6: other declaration of LoadAcq
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:41:6: Xadd redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:52:6: other declaration of Xadd
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:44:6: Xadd64 redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:55:6: other declaration of Xadd64
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:47:6: Xadduintptr redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:58:6: other declaration of Xadduintptr
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:50:6: Xchg redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:61:6: other declaration of Xchg
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:53:6: Xchg64 redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:64:6: other declaration of Xchg64
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:56:6: Xchguintptr redeclared in this block
        /usr/local/go/src/runtime/internal/atomic/atomic_amd64.go:67:6: other declaration of Xchguintptr
/usr/local/go/src/runtime/internal/atomic/atomic_amd64x.go:56:6: too many errors
# math
/usr/local/go/src/math/acosh.go:43:6: Acosh defined in both Go and assembly
/usr/local/go/src/math/asin.go:20:6: Asin defined in both Go and assembly
/usr/local/go/src/math/asin.go:58:6: Acos defined in both Go and assembly
/usr/local/go/src/math/asinh.go:40:6: Asinh defined in both Go and assembly
/usr/local/go/src/math/atan.go:96:6: Atan defined in both Go and assembly
/usr/local/go/src/math/atan2.go:30:6: Atan2 defined in both Go and assembly
/usr/local/go/src/math/atanh.go:48:6: Atanh defined in both Go and assembly
/usr/local/go/src/math/cbrt.go:26:6: Cbrt defined in both Go and assembly
/usr/local/go/src/math/erf.go:189:6: Erf defined in both Go and assembly
/usr/local/go/src/math/erf.go:274:6: Erfc defined in both Go and assembly
/usr/local/go/src/math/erf.go:274:6: too many errors

My go version is 1.19.2
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Versions (please complete the following information):

OS Version: [linux/amd64]
sqls Version [latesr]

Additional context
Add any other context about the problem here.

After reinstall the latest go, I solve this problem.


Adds the ZoneForward resource

Thanks for your contribution :clinking_glasses:  - can you add this resource to the list of available resources in the README.md as well?

Sorry for not getting back to this.

This was pushed out in gem version 2.0.5.  Thanks again for your contributions!


TeamsAppPermissionPolicy

Is 'TeamsAppPermissionPolicy' available? I can't find any documentation on how to configure.

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

ID: a9f904e5-d015-70b0-b809-9b0b62931ab5
Version Independent ID: aa59fcc6-0a6d-c82f-6b5e-ffb85e08ea8a
Content: Manage Teams during the transition to the new Microsoft Teams admin center
Content Source: Teams/manage-teams-skypeforbusiness-admin-center.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@officedocsbot assign @scanum

@toddjohnson39 Thank you for reaching out.  I'll have an answer for you in the next couple of days.

@toddjohnson39 Please have a look at this article and let me know if you need any further assistance.
https://docs.microsoft.com/microsoftteams/enable-features-office-365.

@toddjohnson39 We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.

Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close

I reviewed that link and it doesn't talk about how to setup a TeamsAppPermissionPolicy. The comment on this page references this policy type, however there is no link provided that details how to implement. "Default apps and external apps can be enabled and disabled at the user level in TeamsAppPermissionPolicy. Additionally, apps can be blocked at the tenant level in org-wide app settings which overrides any user and tenant-level settings."

The configuration in the link provided would apply to all users, what I am trying to achieve is the ability to create custom policies for different user groups.

@toddjohnson39 Hope this one helps.  https://docs.microsoft.com/microsoftteams/teams-custom-app-policies-and-settings. Please let me know soonest if this link helps.

Thanks, This link doesn't discuss app permission policies, but led me to the correct article that does.
https://docs.microsoft.com/en-us/microsoftteams/teams-app-permission-policies
Thanks!
[https://docs.microsoft.com/en-us/media/logos/logo-ms-social.png]https://docs.microsoft.com/en-us/microsoftteams/teams-app-permission-policies
Manage app permission policies in Microsoft Teams | Microsoft Docshttps://docs.microsoft.com/en-us/microsoftteams/teams-app-permission-policies
Say, for example, you want to block all third-party apps and allow specific apps from Microsoft for the HR team in your organization. You would create a custom policy named HR App Permission Policy, set it to block and allow the apps that you want, and then assign it to users on the HR team.
docs.microsoft.com

From: Lindsay<EMAIL_ADDRESS>Sent: Thursday, May 2, 2019 10:18 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness
Cc: toddjohnson39; Mention
Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] TeamsAppPermissionPolicy (#2059)
@toddjohnson39https://github.com/toddjohnson39 Hope this one helps. https://docs.microsoft.com/microsoftteams/teams-custom-app-policies-and-settings. Please let me know soonest if this link helps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2059#issuecomment-488940334, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKKFLUVFWVJ3FRPFMXHO4SDPTPDKRANCNFSM4HGYJW3A.

@toddjohnson39 That's great news. Thanks for letting me know.


What tech should be used for the real-time sub link?

Sub task of #29
There are a few different ways we can connect the sub device to the backed:
Over Azure Hub:
Pro:

No extra work as we have todo the link anyway.

Cons:

May be slow ( need to test)
May be expensive (need to calculate)

Over Azure Device Streams
Pro:

Azure will handle securing the connection.
Can go directly to browser. (If device validate inputs.)

Con:

Will need to have dotnet installed on device driver.
May be slow.

Over web-socket with backed.
Pro:

Fastest
If backed do validation it will remove a jump.
Gives us a lot of control.

Con:

We need to handle our own security. Make sure that only authenticated devices can connect to backed.

This task is done when:

[ ] One of them have been chosen.
[ ] We have plans to mitigate the cons.
[ ] Plan for how to develop this feature.

Messages sent directly over IoT Hub reach minimum latencies of around 100ms, which is acceptable. However, the latency varies too much. This solution is therefore not reliable enough to be useful in this application.

We have decided to go with WebSockets to backend, which should yield the lowest latency.


How to proper close database? after colse statement still exists

im on
ubuntu 16.04
dart sdk 1.19
sqlite 0.4.0

I do
...
var db = new sqlite.Database("pathtosqlite")
...

I get a database connection and pass it through some functions
...
var x = await doStuff(db); (doing several db.query() calls)
db.close()
...

but then I always get an error saying
...
Warning: sqlite.Database.close(): 2 statements still open.
finished with exit code 139 (interrupted by signal 11: SIGSEGV)
...
any tips on what Im doing wrong?

You (and, of course, mostly an authors of this software) should learn how works the sqlite and how to properly use sqlite.
Your problem not in your code, the problem which you are experiencing only in that the authors of this software does not provided the correct way to use sqlite.
If they are professionals then I think they will be able to find the problem in a few minutes.

Thanks @mezoni for the stellar comment, this is super helpful.
@HannesRammer could you give more insight into what doStuff does? IIRC there will be issues if you get a stream of rows and do not reach the end of the stream (which takes care of disposing the underlying resources). You might need to get a StreamSubscription and cancel when you're done with it.


Container should render

I'm getting the following react warning:

 cannot appear as a child of
. Add a
,
 or
 to your code to match the DOM tree generated by the browser.

It seems the culprit is the  element in my case, see. https://github.com/resendlabs/react-email/blob/5e8e465f89162bc0a7f4beeb2fc6062d77ce0b97/packages/container/src/container.tsx#L24
 has the `tbody` element for example: https://github.com/resendlabs/react-email/blob/5e8e465f89162bc0a7f4beeb2fc6062d77ce0b97/packages/section/src/section.tsx#L28
Would you accept a PR adding it?

Can you share your code?

I am not at home at the moment. I will try to give you a reproducible example.

Here's a reproducible example: https://github.com/satazor/react-email-tr-inside-table-warn
This doesn't help within react-email's preview app because it uses renderToStaticMarkup which doesn't print these errors.
For some context, I'm creating my own preview app that allows previewing emails based on react-email + liquid. It's a Next.js app that doesn't use renderToStaticMarkup because HMR is near to impossible to do it with it. Instead, it renders the the email into a <template> tag. Then, I have a MutationObserver on the template node and updating an  srcDoc whenever the contents of the <template> changes. This way I got full HMR support: when the user edits the email, the email gets previewed instantly without a refresh.

@bukinoshita thanks for the fix. I leave here the Preview component code with the strategy I explained above. I ended up having to use a regular
 instead of <template> due to React issues.
import { renderLiquid } from '../../shared/render/index.js';
import { useEffect, useState } from 'react';
import { wrap } from 'lodash-es';
import ErrorScreen from '../../shared/react/error-screen/index.js';
import PropTypes from 'prop-types';

if (typeof window !== 'undefined') {
  // Ignore React warnings of  inside

  console.error = wrap(console.error, (error, message, tag, parent, ...rest) => {
    if (tag === '' && parent === 'div') {
      return;
    }

    error.call(console, message, tag, parent, ...rest);
  });
}

const useRender = (node, data) => {
  const [html, setHml] = useState();

  useEffect(() => {
    if (!node) {
      return;
    }

    const observer = new MutationObserver(() => setState(node.innerHTML));

    observer.observe(node, { attributes: true, characterData: true, childList: true, subtree: true });
    setState(node.innerHTML);

    return () => observer.disconnect();
  }, [node, data]);

  return html;
};

const emptyObject = {};

const Preview = ({ children }) => {
  const [node, setNode] = useState(null);
  const html = useRender(node);

  // To have full HMR support, `children` must be rendering regularly somehow. In this case, we render it on a `<div>`.
  // Then, by having a MutationObserver on the div node, we can be notified whenever it changes as part of a re-render.
  // When we get notified, we take it's HTML and render it using liquid to an iframe.
  return (
    <>

        {children}

      {html && }
    </>
  );
};

Preview.propTypes = {
  children: PropTypes.element.isRequired
};

export default Preview;

Could you release a new version of react-email with the fix? 🙏

To give some context: I'm creating my own preview app that allows previewing emails based on react-email + liquid....

Hi @satazor ! I'm currently try to build my email using react-email for Auth0 where it uses Liquid Engine for templating it.
Came across your issue here where you mentioned liquid + react-email and wondering, if you are doing the similar thing for me to know how exactly are we handling the props and variables?
Thanks !

Hi @kihonq I've inherited the project from @satazor so I can answer some of your questions.
Variables:
We are passing directly the variables to the HTML as a Liquid's keys: since we're first getting the html from a React component and only then using the parseAndRenderSync we can have things directly like :
<CustomComponent label="{{ user.first_name | capitalize }}" description="Plain text here" />

that will create a element with a similar node.innerHTML:

{{user.first_name | capitalize}} - Plain text here

and that can be just called with:
liquid.parseAndRenderSync(node.innerHTML, { customer: { first_name: "John" }}, { lenientIf: false, strictVariables: false });

and Liquid will do it's thing. This works even if any curious user tries something weird like having a first name %}.
Note: "Application" specific variables.
We are using an email provider that contains a bunch of reserved liquid variables such as unsubscribe_url. Since we don't want them to be transformed in our email previewer tool we need to register the tag.
Props
Sadly, no interaction between LiquidJS and JSX. I've been using "raw" LiquidJS for that.
<Liquid  code={`
  {% case event.genericAttribute.counter | strip %}
    {% when '0' %}`} />

  <ResponsiveBanner src={src['0']}  (...)  />

  <Liquid code="{% when '1' %}" />

  <ResponsiveBanner src={src['1']}  (...)  />

<Liquid code="{% endcase %}" />

the <Liquid> component is just this:
const Liquid = ({ code }) => ;

There's no magic here unfortunately: LiquidJs support a nice number of features such as for, switch - case, and capture.
I guess it should be possible to do something similar and create a conditional render component
const ConditionalRender = ({ condition, children }) => (
  <>
    <Liquid code={`{% if ${condition} %}`} />
    {children}
    <Liquid code="{% endif %}" />
  </>
);

and make use of PropTypes and other React helpers. But I didn't found it worth.

Hi @JoaoOliveiraRocha !
Thank you for the prompt response (unlike me)
This is more than enough to get me started !
Thank you for being an awesome member of open-source community 🥂


Update setup.md

some version of Ubuntu might not be able to install binaryen directly,  so I add a command using npm.

Please add a -g parameter to it. Don't wanna waste another pr. lol.

why npm? it builds just fine with cmake . && make as suggested in binaryen docs

Im using ubuntu in WSL2 and only version 91 binaryen was available when doing apt install. I tried doing apt update but had no luck. Finally, had to download the binaryen-version_102.tar file from releases and install it manually using these commands sudo dpkg -i binaryen_99-3_amd64.deb' followed by 'sudo apt-get install -f

This repo is moving to an archive, I opened a new issue to track this and make sure we update as needed here:
https://github.com/substrate-developer-hub/substrate-docs/issues/360


harlequin: init at v1.0.0

Description of changes
Adds the TUI DuckDB IDE "harlequin", see https://harlequin.sh/
This is a Python program which builds with Poetry, and some additional necessary overrides have been submitted to poetry2nix. Once the poetry2nix PR is merged, and it has been synched back into nixpkgs, I'll remove the draft status from this PR.
Feedback in the meantime is welcome.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Now just waiting for the updated poetry2nix overrides to get synced across to nixpkgs.

do new packages go in pkgs/by-name/ha/harlequin or is that a failed initiative?

Unsure, but happy to move if that's the case.

do new packages go in pkgs/by-name/ha/harlequin or is that a failed initiative?

Unsure, but happy to move if that's the case.

i'm new at this so have been going over the docs and https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#quick-start-to-adding-a-package which to use pkgs/by-name but looking over recent PRs it doesn't seem like that policy is tightly enforced.
change seems fine to me either way.

I've gone ahead and moved it, and also added the poetry2nix overrides here so that we're not blocked on a sync from the upstream repo (where they have already been added). So I think I can remove the draft status from this now.

Weird. I'm guessing the failure now is because for some reason poetry2nix doesn't work within the pkgs/by-name check. The attribute builds just fine for me.

apologies for suggesting the pkgs/by-name.
i pulled your branch and looking at the workflow i think these are the commands to reproduce.
nix-shell pkgs/test/nixpkgs-check-by-name/shell.nix --run bash <<"EOF"
channel=nixos-unstable
nixpkgs=$(nix-instantiate --find-file nixpkgs -I nixpkgs=channel:"$channel")
rev=$(<"$nixpkgs"/.git-revision)
nix-build -A tests.nixpkgs-check-by-name
result/bin/nixpkgs-check-by-name .
EOF

not sure where to go from there other than to move the package back to where it was initially.

Thanks @natsukium, this is very good feedback. I expect it will require lots more work, since there are several new python libraries I would need to include, but I did indeed notice that mkPoetryApplication was not really used elsewhere in nixpkgs, and the reasoning for this makes sense.

Switched back to a draft, and I don't know that I'm likely to work further on this. I spent a bit of time at the weekend trying to get the additional python modules packaged, but concluded it's not worth the effort for me relative to how much I want to use the software.


Clarify error message for PipelineRun alpha fields

Changes
This commit updates the error message when a user has specified
PipelineRun.spec.taskRunSpecs.stepOverrides without the alpha feature flag
enabled to clarify what they need to do to enable the field (and likewise
for sidecarOverrides). A similar error message is already used for TaskRun's
spec.stepOverrides. It also updates the error message for when a TaskRun's computeResources
or PipelineRun.TaskRunSpecs.computeResources are used without the alpha feature flag.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

n/a Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
NONE

/test pull-tekton-pipeline-build-tests

FYI @austinzhao-go


Use CSS instead of JS for the default night-mode toggle

💥 Proposal
Switch away from a JS night-mode toggle, as JS has to load first and some users go through a quick flash when visiting the website. I really don't know the implementation details on this one, but it would be much better if done with CSS.
Have you read the Contributing Guidelines on issues?
Yes

Are you talking about the deafult value right. Are you thinking in something like the prefer-color-scheme?

Hi @Indicardo
As far as I know, it's not possible to do this only with CSS, as the CSS can only read the (prefers-color-scheme: dark) media query, it cannot update this preference that is global for the computer/browser.
The user might prefer dark mode for a specific site, despite his default global preference being light mode.
We need to store the color scheme in some js storage (like local storage) so that it's possible for the user to switch from light to dark, despite the global (prefers-color-scheme: dark) staying constant over time.
I've not been able to find any css-only solution in the static ecosystem:
https://github.com/gaearon/overreacted.io/blob/master/src/html.js
https://github.com/insin/gatsby-plugin-dark-mode/blob/master/gatsby-ssr.js
https://github.com/wKovacs64/gatsby-plugin-use-dark-mode/blob/master/src/theme-hydration-script-tag.js
If you can link to a CSS only solution, I'm interested to read that, but I don't think it exists, so I'm going to close this issue for now.

Hi @slorber
I was indeed refering to prefers-color-scheme. I considered this to be CSS-only as it doesn't require any dark-mode switch and instead relies on user settings. Now that I think of it, it would be cool to have both a switch and prefers-color-scheme.
As of the quick flash, we are currently seeing one @ https://pgm.dev. The source code for the website is available right here,

Hi @Indicardo, that's weird, I also see the flash, this only happens on your site and I don't see anything that should be the cause. It does not happen on other Docusaurus sites afaik.
Is master deployed?
Have you modified something regarding the theme?
I'm going to look at it.

In fact, in the case of the site https://pgm.dev, this issue is due to the Rocket Loader from CloudFlare, if you disable it, then quick flash will no longer appear.

Thanks @lex111 , investigated this earlier and was not able to understand why the JS seems to hydrate after the images download completes ^^, never heard of this rocket-loader before.
All scripts (including the inline one for the theme) is replaced by <script type="12a2ef273e9b71f07d358b73-text/javascript">
After all assets loaded it puts back to text/javascript and executes the required theme script.

You may disable Rocket Loader for a specific script tag by putting an attribute data-cfasync="false"
Read more:
 How can I have Rocket Loader ignore specific JavaScripts?

Thanks @patryk
@lex111 @yangshun do you think we should add data-cfasync="false" to the inline code block handling the dark theme?
I'm not really fan of introducing proprietary CF markup. In the context of a static React SPA, using Rocket Loader might not be recommended due to delaying React hydratation

Not a fan too. Unless we have many users suffering from this.


[BUG] ${map[${host}]} run error using host ip

scp-hosts.sh
if worker in workersGroup
if [[ "${map[${host}]}" ]] && [[ "${dsDir}" -eq "conf" ]]; then
  sed -i ${txt} "s#worker.group.*#worker.group=${map[${host}]}#g" $workDir/../conf/worker.properties
fi

I'm using the latest dev branch, seems the issue has not been fixed, why closed it ?


Install steps broken

When I try to start-service sshd I get
Start-Service : Cannot find any service with service name 'sshd'.
At line:1 char:1
+ Start-Service sshd
+ ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (sshd:String) [Start-Service], ServiceCommandException
    + FullyQualifiedErrorId : NoServiceFoundForGivenName,Microsoft.PowerShell.Commands.StartServiceCommand

 Get-Service -Name *sshd*

Status   Name               DisplayName
------   ----               -----------
Stopped  SshdBroker         SshdBroker

It looks like it's installed according to powershell.
 Get-WindowsCapability -Online -Name openssh*

Name         : OpenSSH.Client~~~~<IP_ADDRESS>
State        : Installed
DisplayName  : OpenSSH Client
Description  : OpenSSH-based secure shell (SSH) client, for secure key management and access to remote machines.
DownloadSize : 1316207
InstallSize  : 5300763

Name         : OpenSSH.Server~~~~<IP_ADDRESS>
State        : Installed
DisplayName  : OpenSSH Server
Description  : OpenSSH-based secure shell (SSH) server, for secure key management and access from remote machines.
DownloadSize : 1291185
InstallSize  : 4947581

Windows and Powershell version.
$PSVersionTable.PSVersion

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      18362  145

[System.Environment]::OSVersion.Version

Major  Minor  Build  Revision
-----  -----  -----  --------
10     0      18363  0

It seems you have opened an issue ticket without any reference to a relevant documentation page. Please be aware that this site is primarily used for Microsoft documentation improvement suggestions and corrections, when there is incorrect information or a need for additional information to provide a working documentation page for IT professional workers using Microsoft Windows Server software. Without a Microsoft Windows Server documentation page reference, it is less likely that you will get a reply or follow-up from the MS Docs team members.
Have you tried looking up pages via web search to see if there are others who might have found answers to your issue question, like the following?

https://www.bing.com/search?q=start-service+sshd
https://github.com/PowerShell/Win32-OpenSSH/issues/907
https://social.technet.microsoft.com/Forums/windows/en-US/5347fb87-5b90-4cf0-a80f-12d759af53a4/having-trouble-getting-the-builtin-openssh-server-to-work?forum=win10itprogeneral

I have searched, it seems that something is broken in windows on this machine.  I'm going to pursue this with wdg.

Fair enough, feel free to close this issue ticket, unless there is a related Microsoft Docs Windows Server documentation page for which you want to suggest any improvements.

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


When opening the Dialog of material-UI, the screen will jump

Bug is related to

[ ] The Vanilla JavaScript version
[x] The React version
[ ] The Embla Carousel core (all versions)
[ ] Documentation website
[ ] Other

Embla Carousel version

v4.5.0

Describe the bug
When opening the Dialog of material-UI, the screen will jump
CodeSandbox
I forked from https://codesandbox.io/s/embla-carousel-default-react-88p7h

https://codesandbox.io/s/embla-carousel-default-react-report-ly70c?file=/src/js/EmblaCarousel.js

Steps to reproduce

Click the 'Click it first' button.
Click the 'open dialog' button.
Click on the background to close the dialog.
The screen will jump to the top.

Expected behavior

Don't jump to the top of the screen.

Additional context

Video

Hi @Dylan0916,
Thank you for creating this issue. This is happening because the Dialog of material-UI will restore focus on the last element that was focused before the modal opened. And you might think: Ok, but the last element that should have focus should be the button that opens the modal?
Why this is happening
Not in this case, because Embla Carousel calls event.preventDefault() on the mousedown and touchstart event which also prevents the button from receiving focus. And Embla has to to this, because it locks the intersecting scroll axis while you're dragging the carousel. For example, if you have a carousel that scrolls horizontally, along the X axis, it prevents the browser from scrolling the Y axis. So this is by design. Scrolling a carousel on mobile device that allows you to scroll the website at the same time have the most horrible UX I can think of.
Solving the problem
So I can think of two solutions. The first being:
<button
  style={{ position: "absolute" }}
  onClick={(event) => {
    if (embla.clickAllowed()) event.currentTarget.focus(); // Focus on this button when clicked and not dragged
    setOpen(true);
  }}
>
  open dialog
</button>

And the second solution:
<MuiDialog
  open={open}
  onClose={() => setOpen(false)}
  disableRestoreFocus={true} // Add this and the dialog won't restore the focus to the last focused element when it's closed
>
  123
</MuiDialog>

I hope this helps.
Cheers,
David

Hi @Dylan0916,
Thank you for creating this issue. This is happening because the Dialog of material-UI will restore focus on the last element that was focused before the modal opened. And you might think: Ok, but the last element that should have focus should be the button that I just clicked, the one that opens the modal?

Hi,
Thank you very much for your quick reply.
Your answer is very helpful to me,
I'll close this issue.
Best.

Hi @Dylan0916,
Thank you for creating this issue. This is happening because the Dialog of material-UI will restore focus on the last element that was focused before the modal opened. And you might think: Ok, but the last element that should have focus should be the button that I just clicked, the one that opens the modal?

Hi,
Thank you very much for your quick reply.
Your answer is very helpful to me,
I'll close this issue.
Best.


Add LICENSE

technically without a license, nobody else can use this code. It would be good to add one. If you're not familiar with licensing, this site is helpful: https://choosealicense.com/. The best ones for academic work IMO are MIT and Apache

Y, very true. Added a LICENSE now, thanks for bringing it up.

@CMargreitter @patronov Where do I find the licence?

I see it got deleted on 982b26dd6cfeb8aa84b6d7e4a8c2a7edde2bad36, but I assume it's still MIT license.

I just added the license. Actually with version 3.0 we are switching to Apache 2 hope this is not an obstacle.


Corrected git link location and added temperatureF

See response in #1 regarding temperatureF, associated discussion tracked in #7.
Thanks for pointing out the incorrect link, I've updated that


DNS fails on image pull: dial tcp: lookup index.docker.io

See error message below:
Get https://index.docker.io/v1/repositories/library/ubuntu/images: dial tcp: lookup index.docker.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:53: i/o timeout

This seems to be a problem with your DNS loading an older entry.
Is this on Mac or Windows?
Try:

Click on the Docker CLI button

Type in docker-machine ssh

echo "nameserver <IP_ADDRESS>" > /etc/resolv.conf

Hi there,
I'm having the same issue and when I follow your answer I get the following Error: Please specify a machine name. After adding, what I assume is, the machine name I get the following: Error: Host does not exist: hello-world. I was trying to add the basic offical hello-world container to test it out.
Any further help is appreciated.

If you run docker-machine ls it will show which you have available. That being said, mine is named dev (run docker-machine ssh dev).

Ah, thanks a bunch for that. Runs like a charm now.

I'm not sure if this is the/a cause or needs to be a separate issue:
my boot2docker.iso shows as:
Linux dev 4.0.7-boot2docker #1 SMP Wed Jul 15 00:01:41 UTC 2015 x86_64 GNU/Linux
I've not only had this DNS issue, but it keeps reoccurring every 12 hours.
I discovered that the dhcp configurations in this iso has some issues:
* when the dhcp reservation (which might be low) expires, the /etc/resolv.conf gets rewritten by udhcpc
* udhcpc uses  /usr/share/udhcpc/default.script
* default.script either clobbers and rewrites only the dhcp response, or ...
* you can override the DHCP list by creating a 'dns' environment variable entry in default.script as a whitelist seperated list of ips. However, default.script handles this case poorly and appends the configured nameservers every time the lease expires

the end result is that as configured (at least with the responses I get from my DHCP server) the VM might be working initially for people, then stops working because of a pause/resume machine event, network reconfiguration or just a DHCP lease timeout.
Since all of these events can trigger the problem and DHCP leases can vary widely between environments, it might be complicating efforts to find the root cause.
As this issue seems to come from upstream on the boot2docker.iso, my current workaround seems to be holding (bypassing the NAT DNS and forwarding to the host).
docker-machine stop dev
VBoxManage modifyvm dev --natdnsproxy1 off --natdnshostresolver1 off
docker-machine start dev

There are lots of ways to address this, not sure which makes the most sense. I think I'd like the equivalent of the resovlconf framework with head and tail entries.  Improving the udhcpc script to look for /etc/resolvconf/resolvconf.d  or pull settings from /etc/default seems like a possible path, however that still ends up requiring manually fiddling files on the virtual host machine.

I suppose it's almost pointless to waste much time on the TinyCore image and revisit this subject after Debian is further along.
My current persistence hack, after starting the vm (but from OSX command line):

Create docker machine boot patch script
$ cat ~/.docker/bootsync.sh
  #!/bin/sh
  sed -i '/if \[ -n \"\$dns\" \]/a     dns=\"$dns <IP_ADDRESS>\"' /usr/share/udhcpc/default.script
  #sed -i '/if \[ -n \"\$domain\" \]/a     domain=\"$domain dev\"' /usr/share/udhcpc/default.script

Inject boot script into persistent location on virtual host image
$ docker-machine scp ~/.docker/bootsync.sh dev:/mnt/sda1/var/lib/boot2docker/bootsync.sh

Execute script to patch current ephemeral default.script. Or do a docker-machine restart
$ docker-machine ssh dev sudo sh /mnt/sda1/var/lib/boot2docker/bootsynch.sh

Confirmed, once I overwrote the dns-crypt proxy settings in resolve.conf within the default VM with 'nameserver <IP_ADDRESS>' it works.

Is this still happening on the latest toolbox / Kitematic?

Thanks @mchiang0610 , your suggestion helped me

@mchiang0610 your fix is still relevant as of 2/7/1026.
It just helped me on a fresh install of Docker toolbox v1.10.0. My resolv.conf had seemingly valid settings but they gave same results as initial issue reports using just straight docker-machine ssh default and then docker pull commands.
Setting to Google's DNS as you suggest instantly fixed the issue.
Here's what the .conf looked like before the fix, note the first line. I am wondering if it is the root cause:
search ph.cox.net
nameserver 68.105.**.**

@r4j4h Unfortunately not all DNS are made equal - In your case it seems that your ISP's DNS isn't up to par. Using Google's DNS helps quite a bit (probably why they created the service in the first place)

The problem is not the DNS you have, it is the packet return size. Im guessing for <IP_ADDRESS> the return size is less than 512 bytes. So the UDP packet is forwarded from the host mini vm running on your machine to the host machine. When the DNS returns a packet size over 512 DNS will fall over to TCP, which it looks like the vm is not forwarding to your host machine and thus it cannot resolve.

@tylerhjones thanks for the detailed info.
The 192.168 network is created by VirtualBox, which may not forward the TCP packet.

The problem is not the DNS you have, it is the packet return size. Im guessing for <IP_ADDRESS> the return size is less than 512 bytes. So the UDP packet is forwarded from the vm running on your machine to the host machine. When the DNS returns a packet size over 512 DNS will fall over to TCP, which it looks like the vm is not forwarding to your host machine and thus it cannot resolve.

This was it for me - <IP_ADDRESS> -> <IP_ADDRESS>  (opendns)

Your docker host may be behind the proxy. Set the proxy in /etc/sysconfig/docker. That should fix the problem.

Hi all,
i had similar issues with occasionally occuring lookup outages. I've tracked the issue down to the libc library of the gitea docker image.
The image uses the libc.musl-x86_64.so.1 libc library ... which uses an unusual resolve mechanism. It
queries the nameservers in parallel (see docker-alpine). This behavior differs from the glibc resolver behavior. The glibc resolver uses always the first entry.
In my case i had a local dns server in the first line, followed by external dns servers. This leaded to resolving errors, because only the local dns server was able to resolve the ip address of the db container.
May that helps.

@tylerhjones thanks for your explanation.

When the DNS returns a packet size over 512 DNS will fall over to TCP, which it looks like the vm is not forwarding to your host machine and thus it cannot resolve.

Why  a packet size over 512 will fall over to TCP?

Same problem from KVM image using CentOS7
Using default tag: latest
Trying to pull repository docker.io/atlassian/jira-software ...
Get https://registry-1.docker.io/v2/: dial tcp: lookup registry-1.docker.io on 192.168.x.x:53: read udp 192.168.x.x:54328->192.168.x.x:53: i/o timeout```

why don't you use the default dns service of the system instead of guessing what the ip address is?


Posting Articles/Content using the API...

Hey is it currently possible to use this api to post content to STEEM in the same way there is a facebook api that can post content to the facebook newsfeed?

@BCwhoLikesPHP i am working on posting content and it will be available within few weeks

@davidkevork - Please let us know if it is available. Thank you!

Is it still not availabel? How to POST on steem with PHP?