pandas DataFrame containing columns with datetime dtype doesn't display with qgrid 1.3.1 in JupyterLab 2.1.2

Environment

Operating System: Ubuntu 18.04.4
Python Version: Python 3.7.6
How did you install Qgrid: pip
Python packages:

# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                      1_llvm    conda-forge
attrs                     19.3.0                     py_0    conda-forge
backcall                  0.1.0                      py_0    conda-forge
bleach                    3.1.5              pyh9f0ad1d_0    conda-forge
brotlipy                  0.7.0           py37h8f50634_1000    conda-forge
ca-certificates           2<IP_ADDRESS>           hecc5488_0    conda-forge
certifi                   2<IP_ADDRESS>       py37hc8dfbb8_0    conda-forge
cffi                      1.14.0           py37hd463f26_0    conda-forge
chardet                   3.0.4           py37hc8dfbb8_1006    conda-forge
cryptography              2.9.2            py37hb09aad4_0    conda-forge
decorator                 4.4.2                      py_0    conda-forge
defusedxml                0.6.0                      py_0    conda-forge
entrypoints               0.3             py37hc8dfbb8_1001    conda-forge
icu                       67.1                 he1b5a44_0    conda-forge
idna                      2.9                        py_1    conda-forge
importlib-metadata        1.6.0            py37hc8dfbb8_0    conda-forge
importlib_metadata        1.6.0                         0    conda-forge
ipykernel                 5.2.1            py37h43977f1_0    conda-forge
ipython                   7.14.0           py37hc8dfbb8_0    conda-forge
ipython_genutils          0.2.0                      py_1    conda-forge
ipywidgets                7.5.1                    pypi_0    pypi
jedi                      0.17.0           py37hc8dfbb8_0    conda-forge
jinja2                    2.11.2             pyh9f0ad1d_0    conda-forge
json5                     0.9.0                      py_0    conda-forge
jsonschema                3.2.0            py37hc8dfbb8_1    conda-forge
jupyter_client            6.1.3                      py_0    conda-forge
jupyter_core              4.6.3            py37hc8dfbb8_1    conda-forge
jupyterlab                2.1.2                      py_0    conda-forge
jupyterlab_server         1.1.3                      py_0    conda-forge
ld_impl_linux-64          2.34                 h53a641e_0    conda-forge
libblas                   3.8.0               16_openblas    conda-forge
libcblas                  3.8.0               16_openblas    conda-forge
libffi                    3.2.1             he1b5a44_1007    conda-forge
libgcc-ng                 9.2.0                h24d8f2e_2    conda-forge
libgfortran-ng            7.3.0                hdf63c60_5    conda-forge
liblapack                 3.8.0               16_openblas    conda-forge
libopenblas               0.3.9                h5ec1e0e_0    conda-forge
libsodium                 1.0.17               h516909a_0    conda-forge
libstdcxx-ng              9.2.0                hdf63c60_2    conda-forge
libuv                     1.34.0               h516909a_0    conda-forge
llvm-openmp               10.0.0               hc9558a2_0    conda-forge
markupsafe                1.1.1            py37h8f50634_1    conda-forge
mistune                   0.8.4           py37h8f50634_1001    conda-forge
nbconvert                 5.6.1            py37hc8dfbb8_1    conda-forge
nbformat                  5.0.6                      py_0    conda-forge
ncurses                   6.1               hf484d3e_1002    conda-forge
nodejs                    14.2.0               h568c755_1    conda-forge
notebook                  6.0.3            py37hc8dfbb8_0    conda-forge
numpy                     1.18.4           py37h8960a57_0    conda-forge
openssl                   1.1.1g               h516909a_0    conda-forge
packaging                 20.1                       py_0    conda-forge
pandas                    1.0.3            py37h0da4684_1    conda-forge
pandoc                    <IP_ADDRESS>                       0    conda-forge
pandocfilters             1.4.2                      py_1    conda-forge
parso                     0.7.0              pyh9f0ad1d_0    conda-forge
pexpect                   4.8.0            py37hc8dfbb8_1    conda-forge
pickleshare               0.7.5           py37hc8dfbb8_1001    conda-forge
pip                       20.1               pyh9f0ad1d_0    conda-forge
prometheus_client         0.7.1                      py_0    conda-forge
prompt-toolkit            3.0.5                      py_0    conda-forge
prompt_toolkit            3.0.5                         0    conda-forge
ptyprocess                0.6.0                   py_1001    conda-forge
pycparser                 2.20                       py_0    conda-forge
pygments                  2.6.1                      py_0    conda-forge
pyopenssl                 19.1.0                     py_1    conda-forge
pyparsing                 2.4.7              pyh9f0ad1d_0    conda-forge
pyrsistent                0.16.0           py37h8f50634_0    conda-forge
pysocks                   1.7.1            py37hc8dfbb8_1    conda-forge
python                    3.7.6           h8356626_5_cpython    conda-forge
python-dateutil           2.8.1                      py_0    conda-forge
python_abi                3.7                     1_cp37m    conda-forge
pytz                      2020.1             pyh9f0ad1d_0    conda-forge
pyzmq                     19.0.1           py37hac76be4_0    conda-forge
qgrid                     1.3.1                    pypi_0    pypi
readline                  8.0                  hf8c457e_0    conda-forge
requests                  2.23.0             pyh8c360ce_2    conda-forge
send2trash                1.5.0                      py_0    conda-forge
setuptools                46.1.3           py37hc8dfbb8_0    conda-forge
six                       1.14.0                     py_1    conda-forge
sqlite                    3.30.1               hcee41ef_0    conda-forge
terminado                 0.8.3            py37hc8dfbb8_1    conda-forge
testpath                  0.4.4                      py_0    conda-forge
tk                        8.6.10               hed695b0_0    conda-forge
tornado                   6.0.4            py37h8f50634_1    conda-forge
traitlets                 4.3.3            py37hc8dfbb8_1    conda-forge
urllib3                   1.25.9                     py_0    conda-forge
wcwidth                   0.1.9              pyh9f0ad1d_0    conda-forge
webencodings              0.5.1                      py_1    conda-forge
wheel                     0.34.2                     py_1    conda-forge
widgetsnbextension        3.5.1                    pypi_0    pypi
xz                        5.2.5                h516909a_0    conda-forge
zeromq                    4.3.2                he1b5a44_2    conda-forge
zipp                      3.1.0                      py_0    conda-forge
zlib                      1.2.11            h516909a_1006    conda-forge

Jupyter lab packages (if applicable): $ jupyter labextension list

        @jupyter-widgets/jupyterlab-manager v2.0.0  enabled  OK
        qgrid2 v1.1.3  enabled  OK

Description of Issue
Pandas DataFrames containing columns with datetime dtype are no longer displayed by qgrid in JupyterLab 2.1.2, i.e., nothing appears in the cell output, and no error message is displayed.
Reproduction Steps

After installing JupyterLab 2.1.2, qgrid 1.3.1, and the required JupyterLab extensions, attempt to run the following code in a new notebook:

import datetime
import pandas as pd
import qgrid

df = pd.DataFrame({'a': [datetime.datetime(2020,1,1,10,1), datetime.datetime(2020,1,1,11)]})
qgrid.show_grid(df)

Nothing will be displayed in the cell output.
If the DataFrame is modified to contain a mix of datetime and non-datetime
values, the DataFrame will be displayed normally.
Repeat the above with JupyterLab 2.1.0; the qgrid widget will be displayed normally.

What steps have you taken to resolve this already?
Revert to JupyterLab 2.1.0.

Thank you for posting this. That helped a lot!
With Python 3.7.0, here is what I did to get qgrid to work:
pip install qgrid==1.3.1
pip install jupyterlab==2.1.0
jupyter nbextension enable --py --sys-prefix qgrid
jupyter nbextension enable --py --sys-prefix widgetsnbextension
jupyter labextension install @jupyter-widgets/jupyterlab-manager
jupyter labextension install qgrid2
jupyter lab build

If you don't have nodejs (which is required for "labextension install" and "lab build" commands) on an old system, I found one that worked for me by installing Anaconda3.

The problem seems to have been resolved in JupyterLab 2.1.3.

I'm still getting the "Error displaying widget: model not found" after upgrading to Jupyter 2.1.5.
Here is what I ran:
pip install qgrid==1.3.1
pip install jupyterlab==2.1.5
jupyter nbextension enable --py --sys-prefix qgrid
jupyter nbextension enable --py --sys-prefix widgetsnbextension
jupyter labextension install @jupyter-widgets/jupyterlab-manager
jupyter labextension install qgrid2
jupyter lab build

@drlelong Not sure why it isn't working for you - when I tried running jupyterlab from within a new conda environment created as follows, the widget was displayed properly:
conda create -n jupyterlab-2.1.5 jupyterlab=2.1.5
conda activate jupyterlab-2.1.5
conda install qgrid nodejs
jupyter labextension install @jupyter-widgets/jupyterlab-manager qgrid2
jupyter lab build

After adjusting my environment to reinstall qgrid from conda (which results in v1.1.1) and reinstalling the qgrid2 labextension, I'm getting the following error when trying to execute
jupyter                   1.0.0                    py37_7
jupyter_client            5.3.4                    py37_0
jupyter_console           6.1.0                      py_0
jupyter_core              4.6.1                    py37_0
jupyterlab                2.1.5                    pypi_0    pypi
jupyterlab_server         1.1.5                      py_0
nodejs                    10.13.0              he6710b0_0
qgrid                     1.1.1                    py37_2

Not sure why you are experiencing import problems; qgrid.show_grid seems to exist in version 1.1.1. Alternately, try installing 1.3.1 from conda-forge.


Delete of Bulk Export Job fails

Describe the bug
[DELETE] fails when called on bulk export data job.
Environment
IBM FHIR 4.10.1
To Reproduce
Steps to reproduce the behavior:
method:[DELETE] uri:[https://fhirserver.justtest.in:9443/fhir-server/api/v4/$bulkdata-status?job=LqzauvqtHSmkpChVHo%2B1MQ]
Expected behavior
Delete should be successful.
Additional context
We are using IBM FHIR server with Inferno tool. There are different scenarios in it, and one among them is DELETE the bulk status job.
Please see attached logs.
logs.zip

Some comments are present at https://chat.fhir.org/#narrow/stream/212434-ibm/topic/Delete.20Job just in case you need some more information.

Related #2713
Posted this directly in conversation:
HI @Sagar Sarvankar basically your job is a STOPPING state with "batchStatus": "STOPPING",[\n]" it's looping over this status. When you run with trace.specification = *=fine you essentially bury the machine, and extend the amount of time it takes to process the batch job being stopped.
Could you retry with =info:- org.jboss.=FINE:io.openliberty.microprofile.=FINE:com.ibm.fhir.=FINE:javax.batch.=FINE:com.ibm.ws.jbatch.=FINE:com.ibm.ws.=INFO:com.ibm.jbatch.=FINE:com.ibm.ws.jbatch.rest.*=FINEST?
I also asked about OpenJDK 1.8 vs Adoptium/Semeru.


Call to a deprecated function in convert_sulisCam_copy_script.py

Encountered while installing a transfer from sea onto the shoreside server. This lowering included no SulisCam image files under images/SulisCam.
Processing Lowering J2-1583
Processing Lowering Record...
Processing Event Records...
Processing Aux Data Records...
Processing Framegrab copy script...
Processing SulisCam copy script...
/home/sealog/sealog-import-tools/convert_sulisCam_copy_script.py:88: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead
logger.warn("Nothing to return")
2024-08-12 18:02:33,940 - /home/sealog/sealog-import-tools/convert_sulisCam_copy_script.py - WARNING - Nothing to return

Was the lack of Sulis camera images expected or unexpected?

Hi Ryan,
Expected.
S
From: Ryan Govostes @.>
Date: Monday, August 12, 2024 at 3:00 PM
To: WHOIGit/ndsf-sealog-scripts @.>
Cc: Scott J Mccue @.>, Author @.>
Subject: Re: [WHOIGit/ndsf-sealog-scripts] Call to a deprecated function in convert_sulisCam_copy_script.py (Issue #10)
Was the lack of Sulis camera images expected or unexpected?
—
Reply to this email directly, view it on GitHubhttps://github.com/WHOIGit/ndsf-sealog-scripts/issues/10#issuecomment-2284710462, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANLQ32NKRIVTZLQDJIGIHQ3ZREA23AVCNFSM6AAAAABMMVXEA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBUG4YTANBWGI.
You are receiving this because you authored the thread.Message ID: @.***>


add array_join for multi-valued column

I try to support the array_join function similar to https://prestodb.io/docs/current/functions/array.html
array_join(x, delimiter, null_replacement) → varchar#
Concatenates the elements of the given array using the delimiter and an optional string to replace nulls.

Hey, I am also looking for the similarity of LISTAGG : https://stackoverflow.com/questions/43931929/aggregate-string-values-to-a-list
Or agg function for multi-value column that also return a combined list.

#14537 added the scalar function (ARRAY_TO_STRING) to join array into a string
#13146 added the LIST_AGG support

@Jackie-Jiang Thank you. Do we have LISTAGG documentation on Pinot's official website?


CAMEL-19109: Avoid blocking Vert.x event loop in vertx-websocket consumers

Looks I made one of the tests flakier than it was. I'll see if I can fix it. Otherwise I'll drop the Fix flaky tests commit from the PR and come back to it later.


Added DQN Agent

Added DQN Agent

Had a quick glance and this looks good :)
One small nit, I would remove the Powerful DQN directory so that we have the same directory structure as the nba-pipeline.

All in all, impressive work. I have added some minor changes that you can directly commit. I would recommend:

Adding comments to the code to aid readability.
Adding an architecture diagram to show the steps in the pipeline (will help you with that)

I would say some sort of animated gif on top of the diagram to show how this works might also be very cool, I can also help out with that.

All in all, impressive work. I have added some minor changes that you can directly commit. I would recommend:

Adding comments to the code to aid readability.
Adding an architecture diagram to show the steps in the pipeline (will help you with that)

I would say some sort of animated gif on top of the diagram to show how this works might also be very cool, I can also help out with that.

Would look awesome 💯


Implement K-DPM-2 Discrete scheduler

Hi @LaurentMazare,
this PR aims at integrating the K-DPM-2 Discrete Scheduler into this repository, solving the 5th task mentioned in #23.
This implementation ports all the features implemented in the official python version from huggingface.

Thanks!


Hello world redux

This change is

-review-needed
+(ready to merge)
+@robwise

Reviewed 12 of 12 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable

Coverage increased (+0.004%) to 99.298% when pulling e59d25b205e853ad0ce224fb04d40df285492dbd on hello-world-redux into a1fe599afcf1b134232a09d3a27a993b73d0f479 on master.


Catch exceptions in MultithreadedMolSupplier callbacks

Reference Issue
This is a followup to #7763 as discussed with Greg during the UGM hackathon.
What does this implement/fix? Explain your changes.

In next(), simply ignore any exceptions from nextCallback.
In reader(), if readCallback throws, log a warning and proceed using the unmodified record.
(writer() was already handling exceptions from writeCallback.)

Any other comments?
The multithreadedSupplierCatchTest sometimes hangs, and getting a stack trace, I see something involving com.apple.rosetta.exceptionserver ; I'm using an Intel build on an M1 Mac. Google suggest this may have something to do with System Integrity Protection (SIP) on the Mac. I suspect and hope that this is only be a problem for Intel builds on the Mac. I should note that I see the same random hangs with a build before applying my changes! (I was at commit 4c37b03071181007316c306d874f75aa9f5cb7cf ).
Questions

Should we also catch exceptions not derived from std::exception in reader()? We wouldn't have a "what" for those.
Refactoring idea: would make sense to define a couple of structs for the queue types so that we can use descriptive names to access the data members, instead of having to use std::get<0> et al.?

@i-tub sorry it took me so long to get back to this.


how to detect need?

How can I detect if the poly is needed?

You can check:
"ResizeObserver" in window


Remove support for Chef 11, update chef version to test on, use latest Fauxhai

:+1: LGTM

Still waiting on 💚

soon...

@sethvargo any chance you can nuke the gem cache in travis and kick this job off again. The error I'm seeing generally means there's bad cache.

@tas50 done

Can you kick those 3 failing builds to see if they just magically start working. If not I guess there's something real going on there.

I did @tas50

@sethvargo Travis still hates me. I'll try to carve a bit of time out for this post devopsdays

@sethvargo I just excluded the subscribes and notification features from <= 12.5.1 since the matcher changes broke testing on those releases. It's less than ideal and I'm open to a better solution, but this gets us a green built with testing of those features on 8 versions of Chef.


Added the 'spo term group add' command solving #598

Added the 'spo term group add' command solving #598

Coverage remained the same at 100.0% when pulling 574288822d628691fad02a43136889faf3ea20f2 on waldekmastykarz:spo-termgroup-add into 14cfca93437d863cb40c844396b6277114783064 on pnp:dev.

Merged manually


Question : how is it possible not displaying the label (for accessibility reasons) ?

Hello !
And thanks a lot for this awesome package !
Here is my code :
`    $dataToQrCodify = $this->getParameter('app.app.site.root.url.www');
// Create file if it does not exist
if (!file_exists($directory.$filename)) {
    $writer = new PngWriter();
    $qrCode = QrCode::create($dataToQrCodify)
        ->setEncoding(new Encoding('UTF-8'))
        ->setErrorCorrectionLevel(new ErrorCorrectionLevelHigh())
        ->setSize(300)
        ->setMargin(10)
        ->setForegroundColor(new Color(0, 0, 0))
        ->setBackgroundColor(new Color(255, 255, 255));
    $writer->write($qrCode)->saveToFile($directory.$filename);
}`

But the content of the variable $dataToQrCodify is displayed under the QRCode. How can i hide it (for accessibility reasons) ?
Thanks for your answer ;)
Screenshot :

Hi @Brice155 the text under the QR code is the label which can be set via Label::create($labelText). You are probably calling it from somewhere as QrCode::create($data) does not set this label.

Hi @endroid
Thanks for your answer...
Indeed, the picture was cached... after delete it, the label was not here anymore.
Sorry


no matching media type for  pixel format 59555932

Required Info

Camera Model
R200 (and SR300)

Firmware Version
<IP_ADDRESS>

Operating System & Version
Windows 10

Build System
VS2015

I'm getting this output as of late when trying to run the cpp_capture example upon startup (and any of my old projects utilizing the librealsense):
RealSense error calling rs_start_source(device:00000238A92AE9D0, source:VIDEO):
no matching media type for  pixel format 59555932
This happens after starting the RGB stream (depth stream alone seems to work) using whatever preset or format I can find. Nothing has changed except for obvious Windows 10 updates between now and the last time I used the camera. Any ideas?

The reported format 0x59555932 is (HEX for) 'YUY2'.
If the error is persistent with both your code and all the librealsense demos (did you try cpp-config-ui ?)
and additionally, the same camera worked correctly some time ago then I'd suggest to verify HW with Microsoft (3rd party) tool called GraphEdit:
GraphEdit allows to configure and stream standard UVC color data from R200. It can also usable to verify Resolutions/FPS modes supported.

Please check and update

This is what I'm seeing when trying to connect those pins.
I don't know anything about GraphEdit, but I did notice the "Video Camera Terminal" text atop your filter there doesn't exist for my RGB filter, but does for the Depth and Left/Right.

I've noticed OBS is better for messing around with Directshow properties and such, and since I noticed this weird texture when trying to Render in GraphEdit and in OBS, this is what I see when trying to see RGB:

Ok, after having tried updating drivers, restarting a lot, uninstalling and reinstalling the DCM, updating my graphics card drivers (I had installed a second GPU and figured it might have messed up my Directshow drivers since it seemed to be at a Directshow level), I finally just tried uninstalling only the RGB device in my device manager.

That worked for some reason. Even though I only tried uninstalling the R200 RGB, it seemed to fix my SR300 RGB as well, so... black magic.

Thanks @hinmanj !!!

I faced this exact same problem after I updated my machine to Windows 10 with Creator's Update. Solution by @hinmanj worked for me as well.

I faced the same problem. However the solution by @hinmanj did not work for me. Specifically, the demos did not work without the driver. Also after reconnecting R200 or scanning HW change in the device manager the same version of the driver re-installed.
Could you give me any idea to fix the problem?

The solution by @hinmanj works now for me.
I updated Windows again after Windows Creator's Update.

I am facing the same issue. I am trying to run the default librealsense demo (cpp-config-ui ) with R200 in Windows 10 with VS2017. I have issue with RGB camera. I noticed similar texture like @hinmanj with OBS. Following hinmanj steps - reinstalling DCM, updating GPU and rebooting system several times helped me. Now all three R200 capture cameras are looking good on OBS. Yet I am getting the same error "no matching media type for pixel format 59555932" when I run on VS.
I updated Windows like @MatsumotoJ  tried and yet no luck.
How to fix this problem ? Any help is highly appreciated.

I have no idea but

Hi Praattzz,
I have no idea but the problem happens every PC with the Creator's Update around me. Although the solution above worked in my case, it is a bit inconvenient.

This issue is open now in Realsense Forums. The link to the thread is below:
https://communities.intel.com/thread/117338
P.S: If anyone found a way to fix it apart from the above methods, please let other know the logic too.
Thanks in advance.


Doesn't work with SafeAreaView on Android only

Current behavior
I'm wrapping the TabView in a SafeAreaView and it doesn't seem to work on Android.

I tried repro-ing in the snack but it doesn't seem to be the same in the snack
Expected behavior
It should like how it does in iOS:

Reproduction
https://snack.expo.dev/auZcPPHTe
Platform

[X] Android
[X] iOS
[ ] Web
[ ] Windows
[ ] MacOS

Environment

package
version

react-native-tab-view
3.1.1

react-native-pager-view
5.4.9

react-native
0.64.3

expo
N/A

node
14.16.1

npm or yarn
1.22.10

Couldn't find version numbers for the following packages in the issue:

expo

Can you update the issue to include version numbers for those packages? The version numbers must match the format 1.2.3.
The versions mentioned in the issue for the following packages differ from the latest versions on npm:

react-native-pager-view (found: 5.4.9, latest: 5.4.15)
react-native (found: 0.64.3, latest: 0.68.1)

Can you verify that the issue still exists after upgrading to the latest versions of these packages?

Not upgrading because i currently use the expo-unimodules package and expo causes problems with my client database

As official React Native documentation says about SafeAreaView -  It is currently only applicable to iOS devices with iOS version 11 or later., so it isn't any problem with react-native-tab-view, to achieve your goal you can use e.g. StatusBar.


⚠ Bofrost.lu has degraded performance

In 571d3fc, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1635 ms

Resolved: Bofrost.lu performance has improved in e7b3211 after 7 minutes.


Azure Function Kafka Trigger

Hi
We have an Azure function with Kafka trigger running in Azure cloud. We are seeing some frequent failure occurring. Will it possible to guide us why these will be occurring and what can be done or troubleshoot these. We did check our broker and finding no errors logged there.
Kafka Trigger : https://github.com/Azure/azure-functions-kafka-extension
Broker : Confluent Kafka 5.5.1
Errors reported
Libkafka: [thrd:GroupCoordinator]: GroupCoordinator: xxx:9092: 1 request(s) timed out: disconnect (after 585436ms in state UP)
Libkafka: [thrd:xxxx:9092/bootstrap]: xxxx:9092/bootstrap: Connect to ipv4#<IP_ADDRESS>:9092 failed: Unknown error (after 21034ms in state CONNECT)
Libkafka: [thrd:xxxx:9092/2]: xxxxx:9092/2: Failed to resolve 'xxxxx:9092': No such host is known. (after 2768ms in state CONNECT)
Please let us know if any other information is required.
Thanks

[thrd:xxxx:9092/2]: xxxxx:9092/2: Failed to resolve 'xxxxx:9092': No such host is known. (after 2768ms in state CONNECT)

looks like it can't resolve the bootstrap server - seems like an issue outside the client.


Add JSON output option

Just like other Dash Tools we should have a --json option for output.

Make it so!


Migrating from encrypted to GitHub version

Firstly, thanks for opening up the door:)
I had no clue BB was going into open source so installed Version: 3.6.11 and tested for a few months and now paying pro license. For the record I do not mind the small monthly fee as it will help keep the project alive (i hope).
But I have issue:
I have Version: 3.6.11 installed, setup and working. How do I upgrade to the latest GitHub version without starting over?
Many thanks and cheers!

Hello ;
I thinks that you can follow normal update path ( uploading un-encoded files ).  Need a confirmation but...

You can find update instructions here: http://www.boxbilling.com/forum/boxbilling-support/faq-please-read-before-posting in second post.
Also keep it mind to turn off webcron if you were using it.

Thank you both for answers. I have to admit my eyesight is very poor and I missed the instruction to upgrade from 3 to 4.


Clone humble branch of tutorials

People frequently run into issues due to changed API between MoveIt Humble and main, e.g. https://github.com/ros-planning/moveit_task_constructor/pull/508.
Additionally to more clearly emphasizing, which tutorial to use (unfortunately, the corresponding combobox to select the version was removed at some point), the humble version should only clone the humble branch of the tutorials.

This was already handled in https://github.com/ros-planning/moveit2_tutorials/pull/813, will merge that and close this as duplicate.

You should better use the sphinx variable {DISTRO} instead of hardcoding those names. This will transfer more easily to other branches...

Please also consider re-introducing the version-combobox.


pull request for e2e test at 11/16/2016 13:56:07

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add support for other queue backends

Is it worth to add support for other backends? It could be achieved using  go interfaces.
We could start with redis, writing a code that is easy to use different queue vendors.

👍 We should probably improve the current one, maybe having some good tests, and then start to try out redis. But, since redis is easier to test, since we can run with docker, maybe we try out redis as the first adapter.

👍 Good idea! I just started to write some tests.

Awesome, once we have that in place, we should start nailing the whole end-to-end exp. (maybe integration/performance tests).

Working on it #7


InputField button generate a new notification after tapping on "SEND"

Hello,
first of all thank you for the great work on this package ! It made me save a lot of time and the result is beautiful!
I've a small issue with the inputField button. When I click on the send button, the actio callback is called just fine and the action and content retrieved correctly. However as soon as I click on the notification (testing on android for now) the notification get dismissed, but a new equivalent one is generate as soon as the first one is dismissed.
The new one is generated before any of the stream callback is called.
Here it is the notification payload generated trough FCM Admin sdk:
const payload = {
    data: {
      type: type,
      content: JSON.stringify({
        payload: {
          type: type,
        },
        id: notificationId,
        channelKey: 'team_updates',
        title: title,
        body: body,
        largeIcon: largeIcon,
        notificationLayout: 'Messaging',
        displayOnForeground: true,
        displayOnBackground: true,
        showWhen: true,
        autoCancel: true,
      }),
      // add a reply button only if the message was a text (spread operatior)
      ...(type == 'TEXT' && {
        actionButtons: JSON.stringify([
          {
            key: 'REPLY',
            label: 'REPLY',
            autoCancel: true,
            buttonType: 'InputField',
          },
        ]),
      }),
    },
  };

If I tap on the notification instead, it get dismissed alright.

I have now tested my code on three different devices but only getting this issue on my Samsung device.

Hello, somehow one answer I gave few days ago didn't get posted.
I'm indeed using a Samsung too.
Also the problem is coming from the parameter debug = true. Once you init awesome_notification without that, it worked fine for me. @rafaelsetragni I tried to understand where the problem is coming from but couldn't pinpoint something. So I'm not 100% sure either

I have now tested my code on three different devices but only getting this issue on my Samsung device.

This is very interesting!! I guess i know whats going on.
Since Android 9 (API 29), the remote input on notifications was modified to not be dismissible. Instead, a dialog box is displayed on screen, as bellow:

As soon this form appears, the notification became programaticaly undismissible. The only workarround is to update the notification and after some milisseconds to dismiss it, as described on the topic bellow:
https://stackoverflow.com/questions/54219914/cancel-notification-with-remoteinput-not-working/56867575#56867575
It seems that some Android devices, which modifies the S.O., also modifies this strange reply behavior. Because of that, instead of update the notification and after that dismiss it, the first one is successfuly dismissed, but the update creates a new notification that become lost.
So, i gonna need to rethink how to deal with these undismissible replies, that not only change the default behavior between Android versions, but between brands and devices.

Anyway, I encountered this issue only when the debug option is true.
On Tue, Sep 14, 2021, 16:31 rafaelsetragni @.***> wrote:

I have now tested my code on three different devices but only getting this
issue on my Samsung device.
This is very interesting!! I guess i know whats going on.
Since Android 9 (API 29), the remote input on notifications was modified
to not be dismissible. Instead, a dialog box is displayed on screen, as
bellow:
[image: image]
https://user-images.githubusercontent.com/40064496/133328343-171be546-784b-41be-8250-48ef51f0ff0f.png
As soon this form appears, the notification became programaticaly
undismissible. The only workarround is to update the notification and after
some milisseconds to dismiss it, as described on the topic bellow:
https://stackoverflow.com/questions/54219914/cancel-notification-with-remoteinput-not-working/56867575#56867575
It seems that some Android devices, which modifies the S.O., also modifies
this strange reply behavior. Because of that, instead of update the
notification and after that dismiss it, the first one is successfuly
dismissed, but the update creates a new notification that become lost.
So, i gonna need to rethink how to deal with these undismissible replies,
that not only change the default behavior between Android versions, but
between brands and devices.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rafaelsetragni/awesome_notifications/issues/255#issuecomment-919490883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADK6WKR7MTHBLJHQ7WVURETUB6WJHANCNFSM5CBZNJSA
.

Hi @ralphvmo in the action Listener your ReceivedAction has two keys:
buttonKeyInput ↔ String --> The input received for exemple from the edit text
buttonKeyPressed ↔ String --> The key you have set in your notification payload for the button being pressed
Source: https://pub.dev/documentation/awesome_notifications/latest/awesome_notifications/ReceivedAction-class.html

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


Support for multiple environment files?

This is a feature request, not a bug, so I'm not using the template. I searched the open/closed issues for mention of this, but couldn't find anything. If this request already exists, I apologize.
I would like for pipenv to support multiple environment files, a la Honcho and Foreman. For many workflows I'm aware of, it's beneficial to have a default .env file checked in to the repository, along with .env.local files on developer machines, with secrets and such. Based on my reading of the docs (and no further digging, sorry), pipenv doesn't seem to support this. Could it?

We sort of support this => https://docs.pipenv.org/advanced/#automatic-loading-of-env
PIPENV_DOTENV_LOCATION
There isn't much else we can do as far as I can tell, hope this helps!


nrf82: reel_board: USB unable to send fragmented packets through Control endpoint

When testing issue with wpanusb I noticed that after Host sends more then 64 bytes through Control endpoint communication stalls.
@mike-scott

Assigned to @pawelzadrozniak so he can take a look.

I cannot reproduce the problem. CPU hangs in radio IRQ handler after "iwpan phy $PHY set channel 0 26".

I cannot reproduce the problem. CPU hangs in radio IRQ handler after "iwpan phy $PHY set channel 0 26".

Do you mean Linux hangs? BTW: have you disabled Bluetooth? Looks like reel_board always include BT in every project ;)

Do you mean Linux hangs? BTW: have you disabled Bluetooth? Looks like reel_board always include BT in every project ;)

:smile: well, like other NRF based boards:
boards/arm/reel_board/reel_board_defconfig:24:CONFIG_BT_CTLR=y

@finikorg

that always enabling Bluetooth, is this a bug?

Well, we do the same for all nRF-based boards. That said this might not be ideal, but in any case we would need to revisit all the boards, so this is outside the scope of this issue.

Is this correct that BT does not work with 802.15.4 and the board crashes when both are enabled?

Yes, this is correct since both access the radio and are not coordinated.


Ensure the last element of reduction in the throttle is emitted and use appropriate delay

This corrects and validates the case where the last element of a throttle is within the period of the throttle interval but no subsequent elements are produced after that terminal event.

@swift-ci please test


Integration with AWS Codebuild?

AWS recently announced Codebuild at Re:Invent this year, any plans on integrating it into Lambci for greater builder flexibility?
This would solve the problem of a 5 minute build limit, allow for more flexibility of build images without going to ECS, preserve statlessness, and allow provisioning more resources for a build.

@antsankov in what cases would you want to use LambCI if you're using CodeBuild?

Specifically use cases surrounding the infrastructure around builds:

Responding to Github webhooks.
Merge behavior - Amazon doesn't make it easy for you to customize merge and branch behavior.
Notifying subscribers of build events - Github badge, Slack bot, etc.
Cheaper than Codepipeline - This service does cost money per active pipeline, which can add up if you're maintaining a separate pipeline for forks of a repo.

Hmmm, interesting – so mostly cases where you don't want to use CodePipeline then? (I thought that supported GitHub?)
I'll leave this open – I kinda feel like most of deficiencies in CodeBuild/CodePipeline will just be fixed in time by AWS – but I guess it wouldn't be bad to explore this as an option for LambCI in the meantime.

Definitely. I'm fascinated by the promise of completely stateless CI, and am curious about how this project evolves in the future.

I thought about it too. The integration with PR/Slack/... Is indeed missing in code build.
I guess the(my) main pain points are 5min limit and 500mb disk space.
I thought about integrated with Function step(also from the last reinvent) or even poc GCE functions

Hi,
First, I wanted to say that LambCI is an interesting project, and love all the work that has gone into it so far.
Second, I think this is a good idea as well.
I might be missing something, and if I am please let me know, but I think you might be able to use CodeBuild almost like you would use an ECS task, except you don't need to manage an ECS pool, and you can choose an docker image as your base build image.
This is interesting to me for a couple of reasons.
Mainly, Many CI pipelines don't have first class support for Github PR's, and I'd like to have fine tune control over that flow. Like, I want to run static analysis, and unit-tests, but I don't want to run functional tests or deploy to an integration environment.
Regarding CodeBuild, I think you might be able to initiate a CodeBuild, and then subscribe to an SNS for when it is done.
I don't know if I could commit to doing this work, but if I am in the ballpark I wouldn't mind trying something.

I would also be interested in whether CodeBuild could replace ECS in LambCI

I might be missing something, and if I am please let me know, but I think you might be able to use CodeBuild almost like you would use an ECS task, except you don't need to manage an ECS pool, and you can choose an docker image as your base build image.

This. Yes.
I find it interesting that the StartBuild API can override what looks like every parameter of a CodeBuild project, so depending on the desired abstraction, a single CodeBuild Project could maybe be used for all lambci jobs, with the event details and the lambci config determining the params to send along to the StartBuild request.

Jenkins has a plugin to use CodeBuild as an agent:
https://docs.aws.amazon.com/codebuild/latest/userguide/jenkins-plugin.html
https://github.com/jenkinsci/aws-codebuild-plugin
That could serve as inspiration on how to do it, since I'm guessing the API invocation is kind-of similar.


fix(cdk/overlay): scrolling broken in some overlay sizes

When the positioning of an overlay is calculated, the overlay pane is
moved to the top right corner, and given an unbounded size. Based on the
calculated size, a position is attempted where the entire element can
fit next to the origin element without needing to scroll.
If an overlay pane is created with a size large enough that it needs
scrolling in any valid position (top/bottom/left/right) but small enough
that it can fit in the viewport without needing to scroll, the scrolling
of that pane will break. When the pane is moved to the top left corner
for size calculation, the pane will lose it's scroll position since it
no longer has a scrollbar.
This fix introduces a maximum constraint on the size of the overlay pane
during size calculation, where the maximum available vertical and
horizontal space is used as the constraint.
This is a quick approach and there are probably some things I've missed.
I haven't written any tests for this fix, and would like some help regarding
how to best write those tests, as well as fix two tests that broke because
of these changes.

Is there an example of the issue that this fix is addressing?

Yes, sorry should've added a stackblitz right away
https://stackblitz.com/edit/components-issue-yusmdo?file=src/app/example-component.html
Open the 75vh overlay, scroll a bit in it and then scroll the body. The scroll in the overlay will reset to the top.
Open the 125vh overlay, scroll a bit in it and then scroll the body. The scroll in the overlay will not reset.


Added docs for PackageManager::activatePackage and PackageManager::deactivatePackage

:memo: I noticed that the PackageManager::activatePackage function is used frequently in specs for packages (ex: 1 2 3), however is undocumented in the API docs.
This PR adds a new section Activating and deactivating packages to the PackageManager docs, with documentation for the activatePackage and deactivatePackage functions.

@nathansobo we comfortable making these public?

I feel weird about the API of activatePackage. I'm wondering if we should change the implementation to always activate the package, then use another method or an option internally to do the deferred version.

@nathansobo This seems to be a common point that confuses people when writing specs - see https://discuss.atom.io/t/cant-activate-package-in-specs/13672 and https://discuss.atom.io/t/can-you-force-the-activation-of-another-package/10885.
Right now there is Package#activateNow which I think immediately activates the package - https://github.com/atom/atom/blob/master/src/package.coffee#L156.

Have we decided to leave these private for now?

Yes, I think we should fix the API before making it public. @kuychaco maybe this is something you could look into and we could discuss.

We're considering evolving our API so I'm going to close this issue for now. Thanks @rameshvarun for your thought and effort!


About the large dataset and Unet Training

Thanks for your great work!
I have repeated the lora training，and got a same results in supplied prompt.  But I have some issues in large dataset and Unet Trainning.

Problem1: Overfitting
When I training the LoRA with 400 prompts, I find the reward overfitting to maximum easily in 4K steps.  Even I update the prompts to 20K, the same problem will happen.
So, is it work for large dataset ? How to train the ddpo in a large dataset ?
400 prompts

20K prompts

Problem2: Unet Trainning
I set "config.use_lora = False" in config/base.py to train Unet, but the reward change to zero within tens steps, and the sd model generate black images.

Thanks for your great work!
I have repeated the lora training，and got a same results in supplied prompt.  But I have some issues in large dataset and Unet Trainning.
1 . Overfitting
When I training the LoRA with 400 prompts, I find the reward overfitting to maximum(one) easily in 4K steps.  Even I update the prompts to 20K, the same problem will happen.
So, is it work for large dataset ? How to train the ddpo in the large dataset ?

Unet Trainning
I set "config.use_lora = False" in config/base.py to train Unet, but the reward change to zero within tens steps, and the sd model generate black images.

Same problem here. Training with LoRA disabled, the network starts to produce broken images after 1 epoch and gets worse as training continues.

Above are epochs 0 (before training starts), 1, and 2.

For such a large dataset, you may need some more advanced techniques, such as a prompt-dependent value function.
I would recommend a lower learning rate for full finetuning, such as 1e-5.

For such a large dataset, you may need some more advanced techniques, such as a prompt-dependent value function.
I would recommend a lower learning rate for full finetuning, such as 1e-5.

Thanks for your answers. I trans the reward function to bigG and the problem solved. Thanks!


API Test Cases

API Test Cases

@Stefano10 please write the test case take help from here:
https://github.com/FIWARE-Blockchain/tutorials.Step-by-Step


Make the DVM/Kind Browser tables sortable using js

Should be able to sort based on each of the columns, such as sort by Kind number or DVM name, total jobs, total sats earned, etc

simple approach is to use some jquery (or something similar) on the frontend


kdf.head(10) vs df.limit(10).toPandas()

I've got a PySpark DataFrame df.
df = ...
When I invoke:
df.limit(10).toPandas()
I get the result very quick.
When I invoke:
kdf = df.to_koalas()
kdf.head(10)
The results takes a very long time (if I even get it). Furthermore disk is filled until job get crashed due to "No space left on device".
Why? From my point of view both commands should do the same.
Environment:

JupyterHub
Spark 2.4.5
Delta Lake

The biggest difference between them are, Koalas assume the size of the data is large. Very large and does not fit in single note. So, it does distribute data, shuffle, and etc even for small datasets.
Can you share your full codes?

Of course:
PARTITION_SIZE = 1000
os.environ["PYSPARK_PYTHON"] = "/opt/conda/envs/Umsatzdatenkategorisierung/bin/python"

spark = SparkSession \
    .builder \
    .master("spark://...:7077) \
    .config("spark.driver.memory", "4g") \
    .config("spark.logConf", True) \
    .config("spark.cores.max", "8") \
    .config("spark.executor.cores", "2") \
    .config("spark.executor.memory", "10g") \
    .config("spark.sql.execution.arrow.enabled", "true") \
    .config("spark.sql.execution.arrow.fallback.enabled", "true") \
    .appName("Koalas Test") \
    .getOrCreate()
# Apache Spark 2.4.x has a known issue (SPARK-25003) that requires explicit activation
# of the extension and cloning of the session. This will unnecessary in Apache Spark 3.x.
if spark.sparkContext.version < "3.":
    spark.sparkContext._jvm.io.delta.sql.DeltaSparkSessionExtension() \
        .apply(spark._jsparkSession.extensions())
    spark = SparkSession(spark.sparkContext, spark._jsparkSession.cloneSession())

sc = spark.sparkContext
sc.setLogLevel("WARN")
df = spark.read.format("delta").load("/mytestdata/raw5") # ~ 20 Columns, some are numbers, some are 4 - 5 words.
df = df.where(df.partition <= 300)
rowcount = df.count()
print("RowCount: " + str(rowcount)) # 20623197
# df = df.repartition(10000) # number of partitions don't seem to have an impact on the error, tried it with 100, 1000 and 10000
print("Num Partitions: " + str(df.rdd.getNumPartitions())) # 162
dfk = df.to_koalas()
dfk.head(10)

Spark History UI:

Could you try to set default index to "distributed"?
I guess the default index might be the cause. Could you take a look at https://koalas.readthedocs.io/en/latest/user_guide/options.html#default-index-type and https://koalas.readthedocs.io/en/latest/user_guide/best_practices.html#specify-the-index-column-in-conversion-from-spark-dataframe-to-koalas-dataframe?

Thanks a lot @ueshin! That was it!


License about the Data

Hi,
Thank you so much for your impressive results! I saw that the data is also shared. The code license is Apache-2.0 license, is this license also applied to all the data or just the two datasets under the 'release' section?
best,
Lin Li

Hi Lin Li,
The Apache-2.0 license is only applied for the two datasets under the 'release' section.
You can click the links for other datasets to check their licenses.
Best,
Huahua


Question: Extending marked

Is it possible to extend markdown with custom tokens?
marked has the marked.use() method for this (see https://marked.js.org/using_pro#use) but I cound't find it exposed. Or maybe somehow through the options object (see https://marked.js.org/using_pro#extensions)?
And if not, how would one go about implementing this into svelte-markdown?

So, in the current implementation of this package we are only importing the Lexer from marked. Everything else gets tree-shaked out of the bundle. Skimming the docs it seems there's no way to extend it like this from just the Lexer. The options prop in <SvelteMarkdown /> is exactly the same one as Marked's to which you can pass a "tokenizer" or "renderer" property which might already do what you want? If it's not possible like that then I'd be open to add that option somehow.

I've tried, but there are a few problems.

Extending tokenizer and renderer using marked.use(options) works, but when passing the options object directly marked(src, options), it seems that the tokenizer and renderer functions get overwritten instead of being merged (this.tokenizer.space is not a function)
I am not sure if it is also necessary to modify the Lexer, because I am not skilled enough in this area. I tried to do it with just using a tokenizer and a renderer, but it didn't work out well. Could be just a lack of knowledge.

BTW what I am trying to do it to implement footnotes. [fn] footnote [/fn] should turn into [1] footnote
https://github.com/retani/marked-footnotes

Thanks for the repo! I'll be tinkering with it. From what I see you're using Marked directly. I'll get back to you as soon as possible.

Thanks for checking it out. I was using marked directly to see if I can extend it using just the options. The strangest bit is that it doesn't work without marked.use() at all.
I have found a workaround for my specific problem, but would be good to being able to extend it and knowing how. I realized that extending markdown is something for true regex experts.

Sorry for the delay. I'll look into this. That is really weird. I've been debating on whether I should migrate this to remark instead of marked due to its plugins (and because React Markdown uses remark and I'm trying to recreate a similar API), but sadly I have not even had time to experiment with it.

My experience with marked hasn't been great. remark's plugins are quite nice. That's basically all I know :)

Hey guys.. sorry to jump in in this conversation but I have a question regarding remark: Will that work client-side?
I'm asking that because I'm diving deep into a remote mdx/shortcodes solution to run client-side and it seems because unified rely on a lot of node apis and package, it won't work exactly like how marked works right now. 🤔

Svelte Markdown is inspired but React Markdown which uses remark behind the scenes and works client side. The reason I used marked was that I wanted a working version ASAP and remark was giving me some issues with rollup.

Hmm... gotcha.
Maybe I have to read more about how React Markdown does that... Are you open for contributions?
I'm trying to find a good solution to fetch "mdx" content from a CMS and render truly Svelte components without rely on {@html}.

@raulfdm I am definitely open to contributions. The only issue I see is that with mdx style syntax, Svelte components do need a build step. If you wanted to fetch it in the browser and process it there you'd need to bundle the Svelte compiler with your app which takes away most benefits of Svelte. In this scenario I'd usually just recommend mdsvex.

The problem I'm trying to solve is this one:

But I've encountered a lot of issues which I realized that's because of Vite SSR:

https://github.com/sveltejs/kit/issues/1961
https://github.com/sveltejs/kit/issues/1895

mdsvex is indeed a really good candidate but it's truly focused on local files .svx
Unless I walk through ALL posts via node script and generate those files, It won't work. Even importing the compile method, it would throw the same SSR error.

@retani @pablo-abc I've found a solution which enables the use of extensions (but costs an import of marked).
See #38 .

You can pass extensions to SvelteMarkdown components indirectly by first passing your extensions to marked.use() and then reusing the marked.defaults as the options-parameter for the component.
If you want to set other options in addition to the extensions, you may use marked.setOptions() and use the marked.defaults afterwards.
Example:
File App.svelte

<SvelteMarkdown {source} {options} {renderers} />

Imported file LatexMarkdown.svelte

LaTeX

Has this been tested? I'm trying to do something similar using marked-katex-extension, but I noticed you're passing the extension to marked.use(), where is this defined? This hasn't been imported in your example either.

So far I have this:

<svelte:head>
  <link rel="stylesheet"<EMAIL_ADDRESS>integrity="sha384-bYdxxUwYipFNohQlHt0bjN/LCpueqWz13HufFEV1SUatKs1cm4L6fFgCi1jT643X" crossorigin="anonymous" />
</svelte:head>

<Markdown {source} {options} />

But the LaTeX portions are not displayed, as shown below:

I'm not sure what I'm missing

Sorry for the back-to-back posting, but the following code seems to work:
File: App.svelte

<svelte:head>
  <link rel="stylesheet"<EMAIL_ADDRESS>integrity="sha384-bYdxxUwYipFNohQlHt0bjN/LCpueqWz13HufFEV1SUatKs1cm4L6fFgCi1jT643X" crossorigin="anonymous" />
</svelte:head>

<Markdown {source} {options} {renderers} />

File: KatexRenderer.svelte

<Katex>{text}</Katex>

Something about this feels a bit inefficient though, so if anyone has any ideas for improvement, I'd appreciate the feedback.


feat: Integrate ROCK in metadata-writer charm

Closes canonical/pipelines-rocks#66

Please rebase your branch, your tests should pass now that #433 is merged (:


Bypass dont work on megaup (NSFW)

Domain
https://megaup.net
Link
https://megaup.net
Version
Last
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
Anything

Please provide an example link

Example link: https://megaup.net/1TAzt/The_bot.png
but when i wait: https://download.megaup.net/?idurl=0lixLaV5L89JKamcHXUbeswkcyY8BAN%2BwO2I0DGIPkOSdArt%2Bnk0DNkgT1vgbt%2BolX%2FcKYgLNoasii3YQX0Oq4zGpDA0iZ%2BnmmQzQaGZYPk%3D&idfilename=The+bot.png&idfilesize=35.00+KB


Returning large number of rows in SimpleQueryHandler::do_query() is extremely slow

Hi there. I've been experimenting with pgwire. Using psql when returning a large number of rows (even with a single i32 column) from SimpleQueryHandler::do_query() the results are almost 10x slower than in postgres.

Example, returning 7.5M rows of a row containing a single i32 column in pgwire using TextDataRowEncoder takes approx 6.5 secs in psql
The same results from Postgres comes back in 600ms.

version : 0.7.0
OS: MacOS M1

Thank you for reporting. I haven't got chance to work on performance of pgwire. I will do some profile to find out the bottleneck. Contribution is welcomed if you are interested in this part.

In #94 I'm adding tcp_nodelay to client socket by default, according to my pgbench test the performance is now on par with postgresql.

Oh thats really cool.

I was looking at the again. It still seems pretty slow. When executing a query with alot of results (millions) on postgres directly it seems that some values are returned to the client before the query has finished executing (so it gives the impression that its quicker), however when using pgwire, all results are returned before any results are shown. Is this the case?
The implementation of streaming to the client I'm using is identical to the datafusion example.

Sorry for late response. I have been super busy these days. At greptime we haven't cover this part on postgres interface.
I'm going to check the code again but iirc we are using a stream based API to return results to client. It seems my datafusion example has some potential improvement that for a recordbatch, we don't need to add all results in the vector. This might be the reason you it's blocking for results. I will  find time to update the example.

I just improved performance of DataRowEncoder in #165 and it should have twice throughtput in some cases.


Leverage new SES integration with Eventbridge in SST Email

It would be nice if SST Email could take advantage of the new SES integration with Eventbridge to allow users to easily track specific events like Email Bounced, Email Clicked, Email Delivered, etc. for building some analytics, which you could possibly display in SST Console.

@thdxr maybe some ideas for the Event stuff


In my case, I'm using the local file system

In my case, I'm using the local file system
Originally posted by @DanielSeara in https://github.com/microsoft/PTVS/issues/7100#issuecomment-1198276590

it is a dup


Not Compatible with Docker 1.12

This error will come from the Jenkins Logs:
"Error in provisioning; template='DockerTemplate{configVersion=2, labelString='testslave', launcher=com.nirima.jenkins.plugins.docker.launcher.DockerComputerSSHLauncher@52deb96, remoteFsMapping='/home/jenkins', remoteFs='/home/jenkins', instanceCap=1, mode=EXCLUSIVE, retentionStrategy=com.nirima.jenkins.plugins.docker.strategy.DockerOnceRetentionStrategy@4e4e5b9e, numExecutors=1, dockerTemplateBase=DockerTemplateBase{image=jenkins_slave}, removeVolumes=false, pullStrategy=PULL_LATEST}' for cloud='Local Docker' com.github.dockerjava.api.BadRequestException: {"message":"starting container with HostConfig was deprecated since v1.10 and removed in v1.12"}"
It's due to an incompatibility in the Docker-Plugin.  Figured I'd consolidate issues here where it's clear what the root cause is.  I'll be working on testing fixes/updates based on the newer Docker Plugin, called "Yet Another Docker Plugin", very soon!

https://github.com/docker/docker/blob/master/CHANGELOG.md
what are the odds that 1.13 will just fix the issue?

I believe this is now fixed as I'm running it fine locally. So I'm going to close this issue with my next major push (soon tm, so much feedback to incorporate.)

@SalahAdDin The Jenkins2 folder now contains updates and should work with Docker 1.12 and Docker 1.13 out of the box.  I haven't updated the Tutorial stuff yet because it's tied to the legacy blog. That stuff will still only work with Legacy Docker installs.  Working on a new blog series to update everything but for now the repo has what you need in the Jenkins2 Folder.

@maxfields2000 It have the JNLP ıntegratıon also?


CH-2 tests work as expected, submit fails with HTTP 500

Seems quite common that tests work but submission fails.
output of hurl -v ....
* Options:
*     fail fast: true
*     follow redirect: false
*     insecure: false
*     max redirect: 50
*     retry: false
*     retry max count: 10
* Variables:
*     serviceUrl: https://solution-2-r65yr6qt.fermyon.app/
* ------------------------------------------------------------------------------
* Executing entry 1
*
* Cookie store:
*
* Request:
* POST https://advent-of-spin-backend-tlrr40nq.fermyon.app/c2
*
* Implicit content-type=application/json
*
* Request can be run with the following curl command:
* curl 'https://advent-of-spin-backend-tlrr40nq.fermyon.app/c2' -H 'Content-Type: application/json' --data $'{\n  "serviceUrl": "https://solution-2-r65yr6qt.fermyon.app/"\n}'
*
> POST /c2 HTTP/2
> Host: advent-of-spin-backend-tlrr40nq.fermyon.app
> accept: */*
> content-type: application/json
> user-agent: hurl/1.8.0
> content-length: 62
>
* Response: (received 89 bytes in 442 ms)
*
< HTTP/2 500
< date: Sat, 17 Dec 2022 13:46:14 GMT
< content-type: text/plain; charset=utf-8
< content-length: 89
<

I think the issue is with "/" at the end
hurl --variable serviceUrl="https://solution-2-r65yr6qt.fermyon.app" submit.hurl

will work
while
hurl --variable serviceUrl="https://solution-2-r65yr6qt.fermyon.app/" submit.hurl

will return error.

Thanks for pointing me to this @anishcr will check and report 🙏🏻

@ThorstenHans The -v on hurl gives us a curl command to replicate the test.
Running yours:
curl 'https://advent-of-spin-backend-tlrr40nq.fermyon.app/c2' -H 'Content-Type: application/json' --data $'{\n  "serviceUrl": "https://solution-2-r65yr6qt.fermyon.app/"\n}'

We get the error message that the Content-type header is incorrect.
Endpoint /hello didn't contain correct Content-Type header: "text/plain; charset=utf-8"!

Just looked into it. @anishcr was right. Removing the / at the end of the serviceUrl fixed it.


Use a version agnostic await macro

Tokio has a version of await! that takes both 0.1 and 0.3 futures. This is very convenient. It would make sense that macros like join! would use this version of await internally, so people can join futures of any version.
In general, it would be most convenient I think if libraries accept both versions of futures transparently as method parameters, and provide 2 methods when the return type is a future, so the client can choose which version they want.
I think that will help the ecosystem migrate.

0.1 futures are not fully compatible with the std::future execution model, they rely on extra global state that is not provided by it.
tokio-async-await gets around this by being only partially a std::future wrapper, it only allows using async intermediary layers as long as the executor and leaf futures are based on 0.1.
futures(0.3)::compat includes the extra overhead to make 0.1 futures truly std::future compatible, by wrapping them in a compatibility layer that injects the global state they rely on at runtime.
IMO transparently injecting the compatibility layer would not be a good thing long term. It introduces a non-obvious source of overhead, and seems like it would lead to issues with ecosystem fragmentation if library developers decide to stay with 0.1 because they have less pressure to update (while things like the await!/join! macro may be able to make the Future trait compatible across versions (until await! becomes a non-macro relatively soon),  APIs will be designed very differently now that they have the ability to use lifetimes like synchronous code).
As soon as std::future stabilizes I think libraries should start updating to it, using explicit compatibility layers where needed, then as their dependencies update they can easily remove the compatibility layers.

It doens't look like there's any animo for this, so I'll close it.


DHT

Investigate how DHT-based routing could work on top of gossip.

Dumping this here as might (or might not) be relevant: http://iptps03.cs.berkeley.edu/final-papers/kelips.pdf


map on popup framework7 v1 not working

Im develop a app but i use framework7 v1 but i use a popup when i need to load a map inside but its not working my code of popup es

https://github.com/mapsplugin/cordova-plugin-googlemaps#how-does-this-plugin-work-android-ios

thanks, i have changed the logic on my project i steel framework7 v1 but now i cant see any map
`

Cuenta

                Cerrar sesión

`

error on logcat
DexFile_isDexOptNeeded failed to open oat file '/data/dalvik-cache/arm/data@data@com.google.android.gms@app_chimera@m@00000052<EMAIL_ADDRESS>

That error does not comes from this plugin, or your authentication is failed.
If you ask me further question, please share your project files on github.

ok wait me some minutes

this is my project
rt2.zip

Please create a git repository, and put files on it.

https://github.com/dsemerida/mapframework7v1

:flushed: Are you kidding me, or you are git beginner(I don't think so)?

And did you read & understand the url I shared? (I don't think so)
This plugin displays the native map view UNDER browser view.
It means (1) map is not HTML DOM element, and (2) you can not set background color for any HTML elements that are displayed over the map.

  <--- here!

sorry

Then did you solve your problem?

Since there is no response for a while, I close this thread.


.Net 6 string interpolation for Int32 using a format specifier

Description
Some tests regarding the string interpolation for an int were failing after the project upgrade to net6.0
A minimal reproducible example of the Test would be
    [Fact]
    public void StringInterpolation()
    {
        const int utf32 = 129412;
        var result = $"\\x{{{utf32:x8}}}";
        Assert.Equal("\\x{0001f984}", result);
    }

The test was passing before the project upgrade and is now failing. The Assert output for the test:
  Assert.Equal() Failure
               ↓ (pos 3)
  Expected: \x{0001f984}
  Actual:   \x{x8}}
               ↑ (pos 3)

Configuration
MacOs with dotnet SDK 6.0.100 and target framework net6.0
Regression?
The test is passing if you target net5.0 . The workaround would be to use toString("x8") which works as expected.

This is related to previous discussion at https://github.com/dotnet/csharplang/discussions/4361.
In short, prior to .Net Core 3.0, both the compiler (when parsing interpolated strings) and the runtime (when parsing format strings passed to string.Format) interpreted }}} "incorrectly" and this is still the documented behavior (see https://github.com/dotnet/docs/issues/22156). In .Net Core 3.0, the runtime fixed this issue. The compiler didn't change, but it didn't matter much, because interpolated strings mostly compiled to string.Format, which meant only the runtime actually mattered. But with .Net 6.0, interpolated strings don't use string.Format anymore, and the compiler's (outdated) interpretation now makes a difference.
I think the compiler should be updated and make the same change that the runtime made in .Net Core 3.0.


Developer tmp to master

SSL Context trusting all certificates
SMP participant id structure modifications

Especially the "null" on error is no good - let if fail if something goes wrong pls

dont worry, its late, too much time on computer, I need review more than ever

haha. same here ;)

go go go


Users not able to buy PKOIN using credit card

Find https://github.com/pocketnetteam/documentation/issues/2
I tried to move this ticket from the repository documentation to pocketnet.core But found no way to do this from my end.

https://github.com/pocketnetteam/pocketnet.core/issues/431


add -merge command to Transform

Add ability to use as INPUT multiple splitted BAMs or SAMs separated by the comma and merge them to one RDD.
Example of INPUT: /path/to/1.bam,/path/to/2.bam,/path/to/3.bam,/path/to/4.bam

I'm sorry for old commits that were manualy merged by @fnothaft

Can one of the admins verify this patch?

So, the impetus for this is your work on #689 and #733, where you have BAM/SAM file pieces laying around without headers, right?
Is there any motivation to be able to do this with ADAM files as well?
Do the reads in the merged files retain any provenance info about which file they came from? Do we care?
I guess my concern is that this makes it seem like we support merging BAM files that are logically actually multiple files, not just partitions of one; given that this is only about the latter, it feels like we're shipping some implementation details if we don't have a story for how this works in the general case.
At the very least maybe we should document the context that this is meant to be used in, and the contexts where it wouldn't make sense or would be lossy?

@ryan-williams no no no, this is just a small tweak to load splitted bam/sam. In my case it were output files from http://ngsutils.org/modules/bamutils/split/

ah ok, cool. any reason to not allow it to work for e.g. ADAM files as well?

in any case, this lgtm but i'll let @fnothaft or someone else have a look before mergning

Add to whitelist

Jenkins, test this please.

Jenkins, test this please.

Seems like ./scripts/format-source is failing?

@antonstamov Can you run the ./scripts/format-source script in the ADAM repo and then push any changes in the source? That's the cause of the test failure -- it's failing on the format of your source.
Thanks.

@antonstamov I'd like to merge this PR as soon as possible. Can you run the following?
$ cd <adam repo>
$ ./scripts/format-source
$ git add .
$ git commit -m "Formatting source"
$ git push --force origin adam-merge-tool:adam-merge-tool

This will update this PR and fix the syntax errors. Tests are failing because of the format of the source.

@ryan-williams, @massie fixed, sorry for long response

@ryan-williams I think this is what you were looking for on #831, no?

@fnothaft now that #831 has been merged, do you know what the status of this PR should be?

I believe this has been superseded by #831. We can reopen this later if needed.


[WFLY-13834] Document the new file encoding parameter added to Elytron audit logging.

https://issues.redhat.com/browse/WFLY-13834
Documentation for https://issues.redhat.com/browse/EAP7-1557
Requires https://github.com/wildfly/wildfly-core/pull/4319

@ivassile Please file a WFLY for this and use that in the commit message and PR title. Thanks!

@ivassile as @OndrejKotek mentioned, I would also expect to add a brief note about encoding attribute under File audit log (it is in the same file just above ) and mentions other attributes that can be specified.

Corrected the document.


Logout

Hi, how can I remove state and logout user from this project?
I tried:
const signOut = async () => {
try {
await firebase.auth().signOut();
navigation.navigate("Login");
} catch (e) {
console.log(e);
}
}
but state wasn't removed

Hello,
have you found the solution?
I'm also looking for how to disconnect.

So I saw a stack overflow saying I should use redux, then came across a documentation saying redux is ok for state, but not Navigation state.
I wrote a logout function as:
   const onLogoutButtonPress = () => {         // navigation.navigate('Login');         firebase             .auth()             .signOut()             .then(() => {                 console.log("USER LOGGED IN !!!: " +  JSON.stringify(user))                 setUser(null)                 console.log("USER LOGGED OUT !!!: " +  JSON.stringify(user))                 // navigation.navigate('Login')}                 // navigation.navigate(                 //     'Home',                  //     {},                  //     NavigationActions.navigate({                  //         routeName: 'HomeScreen'                  //     })                 // )             })             .catch(error => {                 alert(error)             })     }
That logs out user, but doesnt re-render app.js which houses user auth logic.
I looked react navigation documentation, but to no avail. Maybe I missed something
Some sources say you can use NavigationActions , as navigation is not available for use as navigation.navigate('Login')}
That said I passed navigation as a prop and it was empty. Thats because navigation.navigate() is only available to navigate to pages within the stack.
In our case, HomeScreen is in one stack and Login + Registration is in another stack. So we cannot move up from stack to stack.
But I wonder if we put all of them in the same stack, then navigation.navigate() will work right and being a mobile app, the user wont be able to navigate to the page without a button provided by us ? ....hmmm this just came to me #rubberDuckTheory .. I will test and get back to you guys.
PS. I am not using redux for this, but might keep it in to manage other all round state bits.
If any one else gets a solution, do post #happyLockdownCoding
Mo

UPDATE:
So I did 2 things and I think it works.

add a copy of Login and Registration components in Logged in NavigationStack:

Add this logout function:

It seems to now be logging out for me and loads the component as the Login and Registration components are now in the stack. By the time it calls that component firebase .signOut() has already run
So in effect, you wont be logged in anymore
Try and let me know
Hope this helps
@mrcflorian Thank you for this awesome repo.  Will be good to hear your thoughts.

this should work for any future googlers. setting setUser to null triggers a rerender and thus you to automatically go to the login screen, so there is no need to redirect explicitly (unless you wanted to make a "logged out" screen).


#9 Improve test coverage for packages/wallet

Summary
This PR adds more unit tests for packages/wallet with the goal to increase the test coverage to about 75%.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.6%) to 72.57%

Totals

Change from base Build<PHONE_NUMBER>:
1.6%

Covered Lines:
2594

Relevant Lines:
3403

💛  - Coveralls

@brunomenezes I'm getting an error for the Docker Packaging step:
buildx failed with: ERROR: failed to solve: failed to push docker.io/cartesi/rollups-explorer:pr-12: failed to authorize: failed to fetch oauth token: unexpected status: 401 Unauthorized

Please check it out when you can.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 72.497%

Totals

Change from base Build<PHONE_NUMBER>:
1.5%

Covered Lines:
2580

Relevant Lines:
3389

💛  - Coveralls


Some language courses are counting double towards total BSE humanities count

Expanded from https://github.com/TigerPathApp/tigerpath/issues/357#issuecomment-662691932. cc: @pranavavva
It seems that whenever a language course also has another distribution requirement designation (for example SAN 300 / TRA 300 counts as LA, while also implicitly completing the language requirement by virtue of requiring SAN 107 as a prereq), it's being counted as double in the "Seven Total Courses" count for the BSE Humanities and Social Sciences requirement.
This is because the current structure counts the courses under each subcategory and then adds them up, which is only correct as long as no course falls under more than one category, which is why it falls apart here.
Two possible fixes come to mind:

The best way would probably be to just count them all together in one subrequirement. This would require a slight update to the requirements file format.
A simpler solution to implement, but which wouldn't be quite as clean, would be to just exclude language courses from counting in the other subcategories of the "Seven Total Courses" requirement.

The second one would probably be quite messy and wouldn't really solve the underlying issue, so it would probably be best to go with the first.

I'm looking at how to set up a dev environment locally... do I need to have a postgres database running locally? And I'm confused about the TIGERBOOK env vars in .env-example. The tigerbook API endpoint seems to down, unless I'm not using it correctly?

Hi @pranavavva. I sent you an email you with some more details.


Enable linting for all packages

lots to solve!
we may wish to exclude some files

Codecov Report

Merging #4561 into master will increase coverage by 0.00%.
The diff coverage is 47.76%.

@@           Coverage Diff           @@
##           master    #4561   +/-   ##
=======================================
  Coverage   58.59%   58.59%
=======================================
  Files         881      881
  Lines       29068    29067    -1
  Branches     4196     4196
=======================================
+ Hits        17032    17033    +1
+ Misses      12036    12034    -2


Update from 0.0.19 to 0.0.20 with Python 3.11, torch 2.2.1 and CUDA 12.1: DLL load failed while importing exllamav2_ext: The specified procedure could not be found.

I tried a clean install and still got this error:
 DLL load failed while importing exllamav2_ext: The specified procedure could not be found.
  File "python-3.11.8-amd64\Lib\site-packages\exllamav2\ext.py", line 19, in <module>
    import exllamav2_ext
  File "python-3.11.8-amd64\Lib\site-packages\exllamav2\fasttensors.py", line 6, in <module>
    from exllamav2.ext import exllamav2_ext as ext_c
  File "python-3.11.8-amd64\Lib\site-packages\exllamav2\config.py", line 3, in <module>
    from exllamav2.fasttensors import STFile
  File "python-3.11.8-amd64\Lib\site-packages\exllamav2\model.py", line 23, in <module>
    from exllamav2.config import ExLlamaV2Config
  File "python-3.11.8-amd64\Lib\site-packages\exllamav2\__init__.py", line 3, in <module>

I can see in the release notes it says "Wheels compiled for PyTorch 2.3.0": https://github.com/turboderp/exllamav2/releases but the requirements.txt says >=torch 2.2.0: https://github.com/turboderp/exllamav2/blob/master/requirements.txt.
Note that I am running on Windows, and with PythonNet (although I don't think it would be related, since it runs python311.dll anyway)
It worked with version 0.0.11, 0.0.12, 0.0.13.post1 and post2, 0.0.14, 0.0.15 and 0.0.19.
Let me know if there's some additional information I can provide. Upgrading PyTorch may be feasible, but since I also depend on multiple other packages it might wreak havoc on the dependencies tree :)
Note: The error seems to be the same as https://github.com/turboderp/exllamav2/issues/118 - which may point to a problem with torch 2.2.x support.

The reason I compiled against Torch 2.3 this time was that many people have upgraded and can no longer use wheels compiled against 2.2. I don't really know what to do about it unless someone can convince the PyTorch people to stop breaking the extension API/ABI with every new release.
You don't need 2.3 to run ExLlamaV2 if you're not installing it with a prebuilt wheel. In fact it likely works with Torch 2.0 and maybe 1.x as well. But I've mostly tested on 2.2 so I set that as a minimum req out of caution. Sadly I don't know of a good way to release wheels with individual requirements for each.
An option could be to add another dimension to the build matrix and have four times as many release versions to support the four most recent PyTorch versions, but I'm not sure that's a great road to go down either. :shrug:
But anyway, yes, you need PyTorch 2.3 to use the prebuilt wheels for v0.0.20.

Got it, thanks for the quick answer. I'll see what breaks if I update to torch 2.3 and worst case, wait for everyone to catch up. Thanks again for the great lib, I appreciate the work you do!

Well, I'm not out of the woods yet. With Torch 2.3.0:
[WinError 126] The specified module could not be found. Error loading "...\python-3.11.9-amd64\Lib\site-packages\torch\lib\shm.dll" or one of its dependencies.

The DLL file is there, but this might be on my end. I'll update this ticket when I figured it out for others :)

It's possible installing the CUDA toolkit solves it? The wheels are built with Torch 2.3.0 on a Windows instance, after all.

That's what some people have been saying on StackOverflow (see the pytorch issue, it was marked as high priority so we should know for use soon). If it's a new requirement for 2.3 so be it, but since it wasn't needed before I'm guessing we might just need an additional dependency or some files should be packaged with pytorch itself. I guess we'll know for sure soon (at least I hope so!).

The reason I compiled against Torch 2.3 this time was that many people have upgraded and can no longer use wheels compiled against 2.2. I don't really know what to do about it unless someone can convince the PyTorch people to stop breaking the extension API/ABI with every new release.
You don't need 2.3 to run ExLlamaV2 if you're not installing it with a prebuilt wheel. In fact it likely works with Torch 2.0 and maybe 1.x as well. But I've mostly tested on 2.2 so I set that as a minimum req out of caution. Sadly I don't know of a good way to release wheels with individual requirements for each.
An option could be to add another dimension to the build matrix and have four times as many release versions to support the four most recent PyTorch versions, but I'm not sure that's a great road to go down either. 🤷
But anyway, yes, you need PyTorch 2.3 to use the prebuilt wheels for v0.0.20.

afaik there aren't even any flash attention wheels for windows+pytorch2.3 yet, it seems really premature to updoot this soon.

At least https://github.com/pytorch/pytorch/issues/125109 was closed, so we can expect the next torch release to fix this part. In the meantime, I guess we'll have to be patient 🤷

Is there a workaround in the meantime?

I wasn't able to find one, I thought fiddling with the path would work, but it didn't. I did not spend a lot of time on this though.

I've reworked the build actions and added some Torch 2.2.0 Windows wheels to the release of v0.0.21. Hope that helps.

It seems to be working, thanks a lot, it does help!


Add Match to Result

When working with Results, you'll eventually need to eventually leave the Result context near the boundary and work on the value directly.
As it stands now, the only way to get the value is to use Do and a mutable variable.
Suggested feature is to add a method called Match that satisfies the following
var success = Result.Success<string, int>(4);
var failure = Result.Failure<string, int>("number cannot be less than 0");
string error = success.Match(s => s, i => String.Empty); // would yield string.empty
string error = failure.Match(s => s, i => String.Empty); // would yield "number cannot be less than 0"

PR #32 created


Packaging with Hunter: Finding LICENSE files to distribute with package

In cases where 3rdparty libraries are used, how does one apporach the issue of packaging their license as part of one's own library?
Ex.
hunter_add_package(Boost COMPONENTS system filesystem)
set(Boost_USE_STATIC_LIBS ON)
find_package(Boost REQUIRED system filesystem)
# Other things
install(FILES ${all_license_files} '.')

How do we include the Boost license as part of all_license_files?

You can use BOOST_ROOT variable to access installed Boost directory directly. But I see no LICENSE file there. What do you think about manually downloading it from boost.org?
file(DOWNLOAD "http://www.boost.org/LICENSE_1_0.txt" "/path/to/file")

Ok. I did not know about BOOST_ROOT.
I just wanted to make sure that Hunter can deal with all kinds of 3p products, even licenses that require forwarding a copy with the binary or headers.
Boost was just one example. :)

In some sense, it would be nice to have a BOOST_LICENSE, or in general:
${<PRODUCT>_LICENSE} variable to be able to include it if required.
For example:
if(<PRODUCT>_LICENSE)
  install(FILES ${<PRODUCT>_LICENSE ".")
endif()

I just wanted to make sure that Hunter can deal with all kinds of 3p products, even licenses that require forwarding a copy with the binary or headers

Yes, you can download anything you want. Just pick the scheme you need, see wiki.

In some sense, it would be nice to have a BOOST_LICENSE, or in general: ${<PRODUCT>_LICENSE} variable to be able to include it if required.

Well it's possible to add such variable but I don't know how to implement it in real life. So we must store the path to the license somewhere? Like:
hunter_download(PACKAGE_NAME Boost LICENSE "http://www.boost.org/LICENSE_1_0.txt")

Or try to find LICENSE file in top of the unpacked archive (?) and copy it to the root directory. Like <root>/licenses/Boost/LICENSE (?)

Most products have the license as part of the unpacked archive. That seemed to be the right approach to me, but I haven't thought about it a lot.
Many times the license file is called COPYING, or LICENSE, etc. This is decided on a per-product level. It would be nice if Hunter also defined the _LICENSE variable so products that needed redistribution and do so with Hunter support, rather than going around Hunter for just this one piece.

Most products have the license as part of the unpacked archive. That seemed to be the right approach to me, but I haven't thought about it a lot

Okay, let's try it: https://github.com/ruslo/hunter/issues/162

Implemented by: https://github.com/ruslo/hunter/pull/249


bazel: adapt cc_wraper.py to python3

Description:
In Arch and Clear Linux /usr/bin/python points to python3 causing build failures due to type mismatch in os.write(): string is used where bytestring is expected.
Explicitly convert string to bytestring.
Risk Level: low
Testing: unit tests
Release Notes: N/A
Documentation: N/A

@envoyproxy/maintainers as an aside, given the number of things that had trouble I think we should be more careful tweaking our python library in future

And thanks for picking up this fix, @rojkov :-)

/retest

@htuch I've updated the file's shebang. Or you want to enforce compatibility with both pythons?

This change breaks compiling on RHEL7. I do have python3 but in a non-standard place. After updating the shebang to point to python3 the compilation fails in spectacular ways. Will be interesting to see if this breaks those on Centos. /cc @htuch @rojkov

@moderation would switching to #!/usr/bin/env python3 work?


test: add authenticator test

add authenticator test for basic and session authenticator

link: #1457

Coverage increased (+0.3%) to 64.126% when pulling 6148fd3ca79a2ab1f6030d397824d65c7c675356 on test/authenticatorTest into cb2709bd04dbc60164dfae89d325ac91a0b335fd on master.


Test public API instead of internal API

What does this PR do?
Updates the unit tests so that they test the public LocationAuthorizationService and LocationUpdateService, rather than the internal LocationManager class.
Any background context you want to provide?
When I originally wrote these tests, I used the LocationManager directly. Since then, I realized that it would make more sense to test the public API instead.
Where should the reviewer start?

In order to support the tests, I modified the LocationAuthorizationService and LocationUpdateService to allow the underlying providers to be passed in (via an internal initializer).
The tests now use this internal initializer to pass in a LocationManager instance.

As you can see, the tests still interact with the LocationManager in order to create instances linked to the mock CLLocationManager instances.

Thanks @nonsensery!


Adding St. Jude Children's Research Hospital to the list

Proof of educational activities:
https://www.stjude.org/education-training/st-jude-graduate-school-of-biomedical-sciences.html?sc_icid=tr-mm-grad-school
https://www.stjude.org/education-training/advanced-training.html?sc_icid=tr-mm-advanced
https://www.stjude.org/research/departments/computational-biology.html

@luka-svet First stage of verification passed, official domain for St. Jude Children's Research Hospital is stjude.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@luka-svet Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please notice that the educational license is exclusively intended for educational purposes. If any employees of the Institute who are not students or teachers need the license, please kindly instruct them to email<EMAIL_ADDRESS>to request an academic discount for a paid license.


update dependencies 'react-helmet', 'rxjs', 'webpack'

Hi @steida ,
Updating some dependencies,

Remove redux-connect as it is not compatible with React Router v4
Update rxjs and webpack from rc to stable
Update react-helmet to v4 as no breaking changes https://github.com/nfl/react-helmet/commit/79403c704ec9e5d5a1fb485374df0ffcf94d6ba5
Don't update eslint-config-airbnb yet as this issue https://github.com/yannickcr/eslint-plugin-react/issues/1017 is breaking eslint gulp task

Thanks!

Thank you. Btw, you removed it but not added the previous. Also, can you explain why it is not compatible?

Oh I removed it since its not used anywhere also since in React Router v4 routing is done by real components, we can make async calls in componentWillMount() so I think there is no need for redux-connect for React Router. Its mentioned here https://github.com/ReactTraining/react-router/issues/3854 and here https://github.com/reactjs/react-router-redux/issues/454, also mentioned here https://github.com/ReactTraining/react-router/blob/v4/README.md There is not good example out there but the author mentions about some big changes in coming v5.

I think I have to update RN to 0.40 before. https://github.com/facebook/react-native/issues/6063#issuecomment-273707539


Credential Scope

This PR adds a scope to credentials that is used for endpoint resolution through the AWS::AuthCredentialScope builtin parameter. Similar to account ID, it is treated as completely optional and only set on credential fetchers if the value is present. A couple of properties are added to the endpoint builtin resolver and piped through to the ruleset resolver in order to raise a configuration warning for customers that still rely on legacy behavior.
The base branch is https://github.com/dlm6693/botocore/tree/account-id-endpoints since this PR is largely built on top of that branch and both will be merged at the same time. Once this is approved, account ID will be merged, this branch will be rebased with those changes and the base branch will be updated to https://github.com/boto/botocore/tree/develop/

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Adding missing Param zoneId in the ec2 describe-availability-zone

Adding missing Param zoneId in the describe-availability-zone and added tests against the same

Adding missing Param zoneId in the describe-availability-zone and added tests against the same
localstack/localstack#2263

@bblommers Added a small enhancement here.Please review it and merge.

Coverage remained the same at 94.285% when pulling 7a9cdd4fd24978ba127e0c41a558d2ef740e0975 on usmangani1:2263_Issue into 48288d8bb12a719c042b65fe445cd3f161e2112b on spulec:master.

Coverage remained the same at 94.285% when pulling 7a9cdd4 on usmangani1:2263_Issue into 48288d8 on spulec:master.

@bblommers  can you please help me get this merged to master?

Coverage remained the same at 94.285% when pulling 7a9cdd4 on usmangani1:2263_Issue into 48288d8 on spulec:master.

@bblommers can you please help me get this merged to master?

@bblommers  Can you please help me here?


Restoring open source registry garbage collection page

Deploy preview for docsdocker ready!
Built with commit 7363574f425fae1d9ed8bb1ca02759b092e64455
https://deploy-preview-7692--docsdocker.netlify.com


Copilot Chat: Update attributes on CopilotChat skills

Motivation and Context

The SK attribute format we were using on the Copilot Chat skills is being deprecated.
Description

This change updates the attributes to the new format and make the warnings go away.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

@gitri-ms Closing as this will need to be ported to the new repo.


Update entities due to API change

Seems like there were some changes in API. In particular endpoint of the list: https://api.realworld.io/api/articles?limit=10
So there is the error while validation article schema: missing body prop..
I've tried to find correct doc like swagger or whatever to ensure DTO change, but I wasn't able to find it to exact api.realworld.io. And those docs that I've found say that body is expected to be.
I guess there might be some bug in api, so we can wait and check. Or maybe it was done on purpose..

Hey, good catch. That's why we use contracts. According to this caution
GET /api/articles
GET /api/articles/feed
have been updated and do no longer return the body field.
@Dodje would you like to open a PR and fix issue? I believe that deleting the body field in the schemas will solve the issue.
P. S. It looks like that realworld-docs is in the process of updating right now. @geromegrignon Do we have a centralized place where we can view all upcoming updates?

Yeah, I guess it will be possible for me to create a PR. But I believe there will be some other changes than deleting body prop.
It still exists in single article dto, so we should separate Article and ArticleListItem. I'll check it again more carefully today later or tomorrow.

Hello, indeed, as unused for the purpose of the UI I recently removed it from the public API responses as a solution to reduce the server traffic (as I had to move to a paid one).
I'll double check the docs to ensure there is no mention left, I already removed it from the open API file.

Currently there is no other plans to make changes due to the nature of the project: most implémentations are not maintained, making it hard to make breaking changes.
It might change in the future but I'm focusing on a spin-off focused on Angular for a more complex (and so more RealWorld) application as a way to get a playground I'll be be able to update with ease unlike this one.
I plan to add a proper documentation for anyone willing to create their own spin-off, like a one for React.

@geromegrignon Sounds good, thank you for additional details.

@yurisldk Hi again :)
I went through the project today and found some other changes like "author.bio" -> optional for list item and nullable for single article.
At first I thought that It might be a good idea to fully separate DTO's and logic on the frontend. But it seems that there gonna be really lots of changes like creating/separating features for favorite-feed-article/favorite or create middle model for both at once, query mutations changes and so on.
The simplest idea is to set optional/nullable props, but It can help only in the short-term..
What do you think?

The first part is completed.
Need some time to meditate on ReactQuery docs :) Haven't worked with this tech yet

@geromegrignon I hope it's okay to ask you here since we've already begun the discussion.
I went throw the docs and I found some issues with it:

Methods POST /api/articles/:slug/favorite and DELETE /api/articles/:slug/favorite should be reviewed since the separation of article DTOs. There is still expected Article in return, but now we have to operate with different DTOs for the same action and we can favorite/unfavorite Article Preview.

The Article DTO link return 404.

@yurisldk I also have some questions to you if I may :)
I've found some schema named FavoriteArticleDtoSchema and I believe it shouldn't exist.
We work with exact Article anyway no need to create yet another contract.
If we need to separate api methods in shared layer which might be the case (negotiable/debatable) we can work with existing ArticleDTO as it is the same on the backend side.

I've posted another commit 37d245c76bb253e3d86ef910b871d75d68ed76d2 with favorite/unfavorite button changes.
Seems working fine except the thing that when I favorire the article or artice preview, there is a huge ping before data actually updates on the server. Maybe some cache issues...


Fix Parameter name

closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint-PowerShell/issues/251

Docs Build status updates of commit 380b99e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-server-ps/sharepoint-server/Get-SPSite.md
:white_check_mark:Succeeded
View (sharepoint-server-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@JohanFreelancer9

@dariomws @tseward

@tiburd Please merge this PR. Thanks


Close start page (instead of untitled = true) on project open and document open

I have both close start page on project open and show start page instead of untitled set to true. When I open a project, the start page remains open (as explained in the settings help - since no other document is open). After that I choose a document in the project panel and open it, I think at this moment the start page tab should close. How do you think guys?

It's like this so that we don't end up reopening the startpage multiple times if you open one file and then close it. Also these options can contradict each other so we don't easily change the startpage logic as we may end up breaking it in another case. :)

All right sounds reasonable


Values upto a certain step

Is there a way I can get access to the values up to a certain step?
Say I have 4 steps: A, B, C, and D.
A has 2 values (name and age)
B has 2 value (companyName and designation)
C has 3 value (street, zipcode and country)
D is the final step
I want to gain access to the values before step C that is (name, age, companyName, designation) at C. Is there any way I can get that?

Pretty sure you can use the wizard context for that.

Oh yes!! Thank you very much. I actually missed that part in the Usage section.

@supremeboy18 @Hopp3r But, how do you access the wizard context from a step. I can access it from the form wrapper, but I don't know how to get access to it from inside the step.