UDP message not arriving on service on Azure

I am currently exploring SFM and as part of that I have a mesh service that opens port 90 and receives a string via UDP and then echo's it back to the sender.  This scenario runs on my local machine where the sender and the SFM containerized service are on the same machine but doesn't work when I try that same sender to the same service hosted in Azure SFM.
I have a couple services as part of the application and the HTPP ones are able to be receive and respond to an external request but the UDP one doesn't do anything.   It should log that it received a message in the docker log but it only has the messages that I added to indicate that the service started.
The UdpClient in the service is binding to <IP_ADDRESS>:90 so should theoretically pick up the message.
Is there a reason why the service works on my local machine but is prevented from working in Azure with the same configuration?
Cheers,
Clint

Are you using a gateway? You could try replacing your image with one of the samples to make sure the configurations are otherwise correct.

I just used the most recent VS project template to create the application.  There is a gateway.yaml so I would assume there is some kind of gateway involved but the Azure Portal experience for mesh applications is pretty weak so I am not too sure, though the query via the "az mesh gateway..." ( trying to remember the actual command) seems to indicate that there one similar to what is described on my local dev cluster.
To which sample are you referring?  The ones that I have seen are 7 months old, don't use gateways and use ingressConfig which is no longer being used despite the documentation still vaguely referencing it.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Matt Trower<EMAIL_ADDRESS>Sent: Sunday, July 7, 2019 9:29:57 PM
To: Azure/service-fabric-mesh-preview
Cc: clintsinger; Author
Subject: Re: [Azure/service-fabric-mesh-preview] UDP message not arriving on service on Azure (#374)
Are you using a gateway? You could try replacing your image with one of the samples to make sure the configurations are otherwise correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/service-fabric-mesh-preview/issues/374?email_source=notifications&email_token=AALB6CCGM32EZHMDNMU5JYDP6K7ELA5CNFSM4H6W4FD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZL52EY#issuecomment-509074707, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AALB6CBLVBGYGWX3DP6YGIDP6K7ELANCNFSM4H6W4FDQ.

I realized we aren't able to support UDP traffic currently with how things are configured.
@arturenault  this is an interesting scenario to handle with gateways.  You can't rely on Azure LB heartbeats to help with any routing decisions.


value of getTime() reset every second?

When adding something like material.baseColor = vec4(sin(getTime()), 1.0, 0.0, 1.0); to a custom material fragment shader, it seems like the value getTime() returns starts from 0 every second. Is this expected behaviour?
It is written in the docs that resets happen, but the way it is right now doesn't allow to use this value for animations () or color fades at all.

The docs were not precise enough. getTime() is indeed the current fraction of a second.

Where can I then get a continuous time from alternatively, to implement animations like this
flag?

getTime() should work fine for this, you can simply multiply it by a constant (* 4.0 for instance) to get longer animation cycles.

The question is more like, if the current behaviour is there to stay, or is there a chance to get a longer time span through the getTime() call.

Understandable. How about an additional call, for example getContinuousTime() ;)

We've been talking about it adding something like this yes. But it's not more useful for the case of a flag :)

Would be good to get this feature. The flag was just an example.

I'm in a similar situation. Tried to make a deformation shader with material.worldPosition += vec4(0.1,0,0, 0) * sin(frameUniforms.time * 3.14); and that's the only way I get a smooth cycle. Now I would like to slow down this deformation effect. :)

Note that nothing prevents you to supply your own time as material parameter.

The issue is being fixed in Filament: https://github.com/google/filament/pull/601
Once this gets in, Sceneform will have to integrate.

Excellent, thank you.
Is there a way to know which version of fillament is used in a Sceneform version?

We use tags based on Sceneform versions and we create a GitHub release for every code drop for Sceneform. See https://github.com/google/filament/releases (sceneform1.7-pr1 for instance).

I see. Was just wondering as I didn't remember seeing this mentioned anywhere on the Sceneform side

This is still happening on Sceneform 1.11.0

getTime() should work fine for this, you can simply multiply it by a constant (* 4.0 for instance) to get longer animation cycles.

And this doesn't fix the issue. Time resets each second so thats the maximum duration an animation can have. Multiplying just makes the animation faster or slower but still with the one second constraint.

You should now be able to use:
**getUserTime()**                   | float4   |  Current time in seconds: `time`, `(double)time - time`, `0`, `0`

Unlike getTime(), getUserTime() doesn't get reset every second. See https://github.com/google/filament/pull/601 for more details.


Enhancements

Exported the buildSelectStatement() functionality for reuse (moved function from one file to the other),
Removed unused code

LGTM


Add the test for SBT factory and SBT.

Add the test for SBT factory and SBT.

This issue is done.


renamed tips file name to use dashes

Docs Build status updates of commit 12bfe9a:
:clock10: Full build: the average full build time is 7 min(s) 57 sec(s), this is based on the last 778 build(s) for this repository.

Docs Build status updates of commit 12bfe9a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/use-cli-effectively.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Call for action: test the new wheels

We have been working on setting up a new infrastructure for building wheels based on cibuildwheel and vcpkg (https://github.com/geopandas/pyogrio/issues/48, https://github.com/geopandas/pyogrio/pull/49, https://github.com/geopandas/pyogrio/issues/54 and others), including wheels for windows! pyogrio 0.4.0a1 has now been released with wheels for MacOS, Linux, Windows on PyPI (x86_64 architectures only, sorry!).
It would be very welcome to test those wheels! You can do this by installing the prerelease with pip:
pip install --pre pyogrio

(includes GDAL 3.4.2 in the wheels)
and then test with reading a file of your own:
import pyogrio
pyogrio.read_dataframe("path/to/file")

or by running the tests of the installed package, for which you need some extra dependencies:
pip install pytest pandas pyproj shapely
pip install --no-deps geopandas

pytest --pyargs pyogrio

If you have a chance to test this version, let us know if you run into issues with installation or read_dataframe() / write_dataframe().
Any feedback (also positive, that it is working for you on your system) is very welcome.

Tried on Windows 10, Python 3.10.2 64bit, fresh venv with pip updated & pytest installed afterwards
Then "pip install --pre pyogrio"
"pytest --pyargs pyogrio" failed ("message pandas not installed")
Installing geopandas failed
gpt) C:\temp>python -m pip install geopandas
Collecting geopandas
Using cached geopandas-0.10.2-py2.py3-none-any.whl (1.0 MB)
Collecting fiona>=1.8
Downloading Fiona-1.8.21.tar.gz (1.0 MB)
---------------------------------------- 1.0/1.0 MB 9.5 MB/s eta 0:00:00
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [1 lines of output]
A GDAL API version must be specified. Provide a path to gdal-config using a GDAL_CONFIG environment variable or use a GDAL_VERSION environment variable.
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Part 2: Then pip installed pandas and tried "pip install --pre pyogrio" again.
Failed: See attached full output.
=========================== short test summary info ===========================
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_vsi_handling_read_functions
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_path_absolute
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_path_relative
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_uri_local_file
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_zip_path - Impo...
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_detect_zip_path
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_url - ImportErr...
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_url_with_zip - ...
FAILED gpt/lib/site-packages/pyogrio/tests/test_path.py::test_uri_s3 - Import...
FAILED gpt/lib/site-packages/pyogrio/tests/test_raw_io.py::test_read_autodetect_driver[.fgb]
FAILED gpt/lib/site-packages/pyogrio/tests/test_raw_io.py::test_read_autodetect_driver[.geojson]
FAILED gpt/lib/site-packages/pyogrio/tests/test_raw_io.py::test_read_autodetect_driver[.geojsonl]
FAILED gpt/lib/site-packages/pyogrio/tests/test_raw_io.py::test_read_autodetect_driver[.geojsons]
FAILED gpt/lib/site-packages/pyogrio/tests/test_raw_io.py::test_read_autodetect_driver[.gpkg]
FAILED gpt/lib/site-packages/pyogrio/tests/test_raw_io.py::test_read_autodetect_driver[.json]
=========== 15 failed, 80 passed, 150 skipped, 3 warnings in 4.61s ============
output.txt
If you provide a test dataframe to import, I could try if it would import.

OK, tried this (Windows10, 64bit, Python 3.10.2, fresh venv)
python -m pip install pip -U
pip install --pre pyogrio
pip install pytest pandas pyproj shapely
pip install --no-deps geopandas
Works with only some warnings - output attached.
output2.txt
Happy to test reading a dataframe if you provide a link to an example file to read

Followed all the instructions. However, when it comes to the pytest --pyargs pyogrio, it returns an import error which says that the GDAL DLL could not be found.  It must be on the system PATH. What could have gone wrong?

Thank you @dtjunkie ; I think this is exactly the case I was running into trying to get Windows-specific tests to run correctly on CI before we moved the Windows build to use VCPKG.  However, those have all been working since using VCPKG.
Could you please try these lines from our Windows CI wheel test?
python -c "import pyogrio; print(f'GDAL version: {pyogrio.__gdal_version__}\nGEOS version: {pyogrio.__gdal_geos_version__}')"
python -m pytest --pyargs pyogrio.tests -v

I think we're bumping into an issue with not correctly resolving the GDAL DLL when running pytest directly.

Sure, no problem. I'll get back to you with the result.

I've already try those lines. The first line returns the same error. Furthermore, the second line returns an error message stating that a module or package was not found.

@dtjunkie thanks for further testing!  Your attempts are valuable, I don't think any of the core devs here have a Windows machine to easily test on, so we're reliant on our Windows CI tests, which don't appear representative in this case.
What version of Windows are you using?
I see that our CI tests are run using Bash on Windows (rather than Powershell / Cmd)?  Is that something you can test if you didn't already so so above?  (I don't know how to run Bash vs Powershell / Cmd, only that it is available as an option on our CI)
I see that you are using a Conda-based environment - is this a clean environment just for pyogrio, or do you have other things installed?  Can you share results of conda list?

I'll be checking on it tomorrow. It's already late evening here in Manila and time for me to take a rest.

I'm currently using Windows 11 v. 21H2 OS Build 22000.708

Hi! I tried repeat the steps above using the new conda environment that I created just now. Here's the result when the final step was done.

Tried importing and accessing the file using pyogrio and it worked

My question though is whether geopandas can't be used for the time being?

@dtjunkie Thank you for further testing and reporting back!  It looks like everything worked properly in a clean conda environment, but not in one that already had a bunch of other things installed.  That might give us something to test further with in order to nail down the issue.

My question though is whether geopandas can't be used for the time being?

I don't know what you mean, can you clarify.  You can certainly use GeoPandas > 0.10 with Pyogrio.  It looks like both the last tests and your manual example worked properly; are you seeing a problem?

I tried using geopandas (gpd) along with pyogrio but when I tried to use the gpd's read_file function it returns an error stating that the function requires the fiona package which is absent in the --no-deped version of geopandas. Did I do something wrong?

I downloaded the following versions of GDAL and Fiona from this website: https://www.lfd.uci.edu/~gohlke/pythonlibs/
GDAL-3.4.3-cp310-cp310-win_amd64.whl
Fiona-1.8.21-cp310-cp310-win_amd64.whl
When I tested the geopandas' read_file function, it finally worked.

I was also able to get this running today using a fresh miniconda on windows:
conda create -n testing
conda activate testing
conda install -c conda-forge pip
pip install --pre pyogrio
pip install pytest pandas pyproj shapely
pip install --no-deps geopandas

pytest --pyargs pyogrio
python -c "import pyogrio, geopandas; print(pyogrio.read_dataframe(geopandas.datasets.get_path('nybb')))"

As noted above, geopandas 0.10 won't read files without fiona, but pyogrio worked just fine with this!

@ljwolf thanks a lot for testing!
One more thing, could you test doing a conda install jpeg libcurl libiconv libpng libtiff libwebp libxml2 zlib zstd  in that environment, and then run the test of pyogrio again? (that might somewhat mimick the environment above in which case the wheel didn't correctly load its bundled GDAL library)

Yep, just did. I still get 254 passes, 12 skips, and 23 warnings, and can read the file fine.
(testing) C:\Users\Levi John Wolf>conda list
# packages in environment at C:\Users\Levi John Wolf\miniconda3\envs\testing:
#
# Name                    Version                   Build  Channel
atomicwrites              1.4.0                    pypi_0    pypi
attrs                     21.4.0                   pypi_0    pypi
bzip2                     1.0.8                he774522_0
ca-certificates           2022.4.26            haa95532_0
certifi                   2<IP_ADDRESS>     py310haa95532_0
colorama                  0.4.5                    pypi_0    pypi
geopandas                 0.10.2                   pypi_0    pypi
iniconfig                 1.1.1                    pypi_0    pypi
jpeg                      9e                   h2bbff1b_0
libcurl                   7.82.0               h86230a5_0
libffi                    3.4.2                hd77b12b_4
libiconv                  1.16                 h2bbff1b_2
libpng                    1.6.37               h2a8f88b_0
libssh2                   1.10.0               hcd4344a_0
libtiff                   4.2.0                he0120a3_1
libxml2                   2.9.14               h0ad7f3c_0
lz4-c                     1.9.3                h2bbff1b_1
numpy                     1.23.0rc3                pypi_0    pypi
openssl                   1.1.1o               h2bbff1b_0
packaging                 21.3                     pypi_0    pypi
pandas                    1.4.2                    pypi_0    pypi
pip                       22.1.2                   pypi_0    pypi
pluggy                    1.0.0                    pypi_0    pypi
py                        1.11.0                   pypi_0    pypi
pyogrio                   0.4.0b1                  pypi_0    pypi
pyparsing                 3.0.9                    pypi_0    pypi
pyproj                    3.3.1                    pypi_0    pypi
pytest                    7.1.2                    pypi_0    pypi
python                    3.10.4               hbb2ffb3_0
python-dateutil           2.8.2                    pypi_0    pypi
pytz                      2022.1                   pypi_0    pypi
setuptools                61.2.0          py310haa95532_0
shapely                   1.8.2                    pypi_0    pypi
six                       1.16.0                   pypi_0    pypi
sqlite                    3.38.3               h2bbff1b_0
tk                        8.6.12               h2bbff1b_0
tomli                     2.0.1                    pypi_0    pypi
tzdata                    2022a                hda174b7_0
vc                        14.2                 h21ff451_1
vs2015_runtime            14.27.29016          h5e58377_2
wheel                     0.37.1             pyhd3eb1b0_0
wincertstore              0.2             py310haa95532_2
xz                        5.2.5                h8cc25b3_1
zlib                      1.2.12               h8cc25b3_2
zstd                      1.5.2                h19a0ad4_0

OK, thanks for testing. Then there was still something else going wrong in @dtjunkie's env, but it is good to know that having those packages that are bundled in the wheel also installed normally doesn't generally mess up the wheel.

I've also managed to install and test the wheels successfully on a couple of windows computers using pip inside a virtualenv with no issues.

tried on windows 10 in an virtual env,
tried on conda environment too
upgraded my pip and
computer is x64

it yielded these errors.
$ python -m venv "test_geopandas_wheel"
$ . test_geopandas_wheel/Scripts/activate
$ python -V
Python 3.7.0
$ pip -V
pip 22.1.2
$ pip install --pre pyogrio
Collecting pyogrio
  Using cached pyogrio-0.3.0.tar.gz (322 kB)
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'error'
  error: subprocess-exited-with-error

 Getting requirements to build wheel did not run successfully.
  exit code: 1

  [24 lines of output]
  Traceback (most recent call last):
    File ".\test_geopandas_wheel\lib\site-packages\pip\_vendor\pep517\in_process\_in_process.py", line 363, in <module>
      main()
    File ".\test_geopandas_wheel\lib\site-packages\pip\_vendor\pep517\in_process\_in_process.py", line 345, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
    File ".\test_geopandas_wheel\lib\site-packages\pip\_vendor\pep517\in_process\_in_process.py", line 130, in get_requires_for_build_wheel
      return hook(config_settings)
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\setuptools\build_meta.py", line 178, in get_requires_for_build_wheel
      config_settings, requirements=['wheel'])
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\setuptools\build_meta.py", line 159, in _get_build_requires
      self.run_setup()
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\setuptools\build_meta.py", line 282, in run_setup
      self).run_setup(setup_script=setup_script)
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\setuptools\build_meta.py", line 174, in run_setup
      exec(compile(code, __file__, 'exec'), locals())
    File "setup.py", line 131, in <module>
      compiler_directives={"language_level": "3"},
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\Cython\Build\Dependencies.py", line 986, in cythonize
      aliases=aliases)
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\Cython\Build\Dependencies.py", line 821, in create_extension_list
      for file in nonempty(sorted(extended_iglob(filepattern)), "'%s' doesn't match any files" % filepattern):
    File "C:\Users\<USERS>~1\AppData\Local\Temp\pip-build-env-6_g226y_\overlay\Lib\site-packages\Cython\Build\Dependencies.py", line 116, in nonempty
      raise ValueError(error_msg)
  ValueError: 'pyogrio/_err.pyx' doesn't match any files
  [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

Getting requirements to build wheel did not run successfully.
exit code: 1

See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.
(test_geopandas_wheel)

@sutanAE sorry for the issues!  Something is clearly not right with that installation call, as it is using the 0.3.0 source distribution instead of the currently available 0.4.0 wheels, and it is failing because it is trying to build 0.3.0 from source (not sure why it is using the 0.3.0 source distribution rather than 0.4.0, maybe a caching issue on your end?).
I do see that you are using Python 3.7, which is older than our oldest officially supported version of 3.8.  We don't have pre-built wheel files available for older versions of Python.  Can you please try again within a Python 3.8+ environment?
Also, now that 0.4.0 is officially released, you should not need to use the --pre flag; you should now be able to use pip install pyogrio to get it.  Alternatively, pip install pyogrio==0.4.0 may work to force installation of that specific version if you continue running into pip caching issues (on Python 3.8+).

@brendan-ward seems like yes my end, sorry about that. Tried with Python 3.8 and installation works just fine. I missed reading the initial documentation. Now still trying to read some shapefilees but still failing because of gdal version from my anaconda installattion, still debuging the case.

I'm back with python 3.8 and gdal 3.1.1. I'm using windows 11 with no administration privilege, and anaconda.
Installation, with no error:
(base) > conda create --name test-gdal
(base) > conda activate test-gdal
(test-gdal) > conda install -c conda-forge gdal=3.1.1
(test-gdal) > python -V
Python 3.8.13
(test-gdal) > conda list | grep gdal
# packages in environment at \.conda\envs\test-gdal:                                                                                gdal                      3.1.1            py38h3ba59e7_2    conda-forge                                                                                libgdal                   3.1.1                h98769a9_2    conda-forge

(test-gdal) > pip install --pre pyogrio
(test-gdal) > pip install pytest pandas pyproj shapely
(test-gdal) > pip install --no-deps geopandas

reading .shp with no error:
(test-gdal) > python
Python 3.8.13 | packaged by conda-forge | (default, Mar 25 2022, 05:59:45) [MSC v.1929 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import pyogrio
>>> pyogrio.read_dataframe("test.shp")

   field1   name       id                                           geometry
0      4      x      88025.0  LINESTRING (439227.827 110955.749, 439247.835 ...
1      4      x      87173.0  LINESTRING (430833.883 119926.692, 430884.813 ...
2      4      x      87175.0  LINESTRING (429978.613 122877.359, 429959.102 ...
3      4      x      87184.0  LINESTRING (429978.613 122877.359, 430000.709 ...
4      4      x      87219.0  LINESTRING (433308.737 113544.224, 433576.132 ...

the pytest result
(test_gdal) > pytest --pyargs pyogrio >> test_result.csv
(test_gdal) > type test_result.csv
...
=========================== short test summary info ===========================
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_write_dataframe_truly_mixed[.fgb]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_write_dataframe_truly_mixed[.geojson]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_write_dataframe_truly_mixed[.geojsonl]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_write_dataframe_truly_mixed[.gpkg]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_write_dataframe_truly_mixed_invalid
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_custom_crs_io[.fgb]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_custom_crs_io[.geojson]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_custom_crs_io[.geojsonl]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_custom_crs_io[.gpkg]
FAILED .conda/envs/test-gdal/lib/site-packages/pyogrio/tests/test_geopandas_io.py::test_custom_crs_io[.shp]
=========== 10 failed, 260 passed, 12 skipped, 5 warnings in 21.60s ===========

@sutanAE you should install everything using conda, rather than GDAL via conda and everything else via pip; that may produce incompatibilities.  However, if you are just using conda for the base python environment and installing everything into it via pip, that may be OK; in that case you don't need to install GDAL because it is already bundled into the pyogrio 0.4.0 wheel on PyPI.  (you also don't need --pre for pyogrio as 0.4.0 is now available).
Unfortunately, I'm also not able to reproduce the failed tests using your setup instructions on macOS 10.15.  Can you please attach the output of conda list?  And also perhaps pytest --pyargs pyogrio.tests.test_geopandas_io -v might provide a little more detailed info for us to go on (just need to paste in the more detailed errors for the failing tests).

@sutanAE thanks for the extra info - that is helpful!  The tweet promoting this was when we only had pre-release wheels available and we wanted extra testing.  :)
For the record, I'm not specfically using sudo on macOS.  From your logs, it looks like that might not be the issue; it is writing test outputs to subdirectories within C:/Users/SutanAE/AppData/Local/Temp/ which should be owned by your individual account and not run into permission issues.  Though I'm not a Windows user, so I can't say that for certain.
Instead, it looks like you are running into issues with shapely / GEOS writing the WKB version of geometries that we pass along to GDAL for writing to the output file:
self = <shapely.geos.WKBWriter object at 0x0000029AE36A03D0>
geom = <shapely.geometry.polygon.Polygon object at 0x0000029AE36A09A0>

    def write(self, geom):
        """Returns WKB byte string for geometry"""
        if geom is None or geom._geom is None:
            raise InvalidGeometryError("Null geometry supports no operations")
        size = c_size_t()
>       result = self._lgeos.GEOSWKBWriter_write(
            self._writer, geom._geom, pointer(size))
E       OSError: exception: access violation writing 0x0000029A00000000

From your dependency list, it looks like you have GEOS installed via conda-forge, and shapely installed via pip.  I think that shapely wheels ship with GEOS built in, but perhaps on Windows there is a chance that the wrong GEOS dll is getting loaded (perhaps the one included with GDAL via conda-forge?  No idea...)
However, I'd still suggest using either conda OR pip but not a mix of both; that might be contributing to the issues here.  Since this issue is focused specifically on testing the new python wheels instead of conda, use pip all the way through.
Try creating your new environment using virtualenv or your tool of choice (pipenv, poetry, etc) or possibly an empty conda environment.  E.g.,
python -m venv c:\path\to\myenv

(see the link above for activation instructions for Windows)
c:\path\to\myenv\Scripts\activate.bat

Then
pip install pyogrio shapely pygeos pytest pyproj pandas
pip install geopandas --no-deps
pytest --pyargs pyogrio.tests.test_geopandas_io -v

(note: I also suggest installing pygeos per the requirements)
If we're lucky, those tests will now pass!

Closing because wheels have now been generally tested for some time.


esp32s3 (maybe others) missing dedicated gpio?

I can't find the DEDICATED_GPIO in esp32s3 while in espressif docs (https://docs.espressif.com/projects/esp-idf/en/latest/esp32s3/api-reference/peripherals/dedic_gpio.html) this functionality is avaliable in that model.
It is available in esp32s2.
Is it missing for a reason?

Maybe I'm missing something, but as far as I can tell there are no actual registers for this peripheral? The header file linked in the documentation you shared isn't even in the soc component of ESP-IDF, which has been the case for all other peripherals as far as I can recall.
Is there something specific missing that you're looking for?

After searching some more, it seems like they provide that functionality but not using registers but rather through specialized cpu instructions:
https://www.esp32.com/viewtopic.php?t=27857
That probably means that esp-pacs is not the place to expect that to be, right?

Yeah from the sounds of that we will need some driver code in another package somewhere, whether that's esp-hal or somewhere else. The scope of the PACs is really just to provide register access and enumerate the peripheral interrupts, so not really anything to do here.
Thanks for opening the issue regardless, though!


fog:Rackspace uses servers#create, which only allows a key name

Hey! So I'm working on a PR that adds a Rax-specific example of this driver, and I've tripped over something. This driver uses servers#create which requires a named keypair already be created on the account.
Taking a survey of what other tools use to create nodes at Rackspace using fog, I found two different behaviors:

knife-rackspace uses passworded SSH with an optional ability to provide a key name, but you must create the key yourself (not unlike the requirement currently in chef-provisioning-fog)

kitchen-rackspace actually uses fog's other way to create Rackspace instances, servers#bootstrap, which actually uses SSH to get a public key on the server.

I also found that the fog Rackspace compute v1 provider had a similar bootstrap implementation.
It seems like the recommended 'fog best practice' way to bootstrap a Rackspace server using SSH keys is to use the bootstrap method, according to their docs. chef-provisioning-fog currently uses the create server method. I'm not sure this is an urgent fix, but I wanted to at least raise an issue so folks who try to do what I did are aware.

@martinb3 when you get some time in the near future mind syncing with me on this? I don't think this is still relevant, but I'd like to confirm.

Hey sure! I'm not sure which way is the right answer here, though. @mdarby do you have any thoughts about which is better?

I don't honestly; I'd be concerned about breaking existing libs.


Fix IO attributes and Z-1 PLL

The GW1NZ-1 has PLL power saving enabled by default. We turn it off.
And also we continue to select the formula for setting the OPENDRAIN flag, which, as it turned out, interferes with the processing of PULLMODE

let's wait for confirmation from Catherine?

I can confirm the mismatches with proprietary tools are gone. The netlist I had issues with is translated equally with Apicula and Gowin tools with this patch.


Namespace is not set correctly while debugging

Problem Description
Setting a breakpoint inside the function for debugging,
Looking at the value of the function's input parameter in variable explorer, it is not expected
What steps reproduce the problem?

What is the expected output? What do you see instead?
Normally, the value of a variable is not printed using "print", but viewed directly through variable explorer.
I think ,when debugging,the value of the variable "a" should be 5,not 10
code
def test(a):
    print('a:',a)
    print()#Set a breakpoint on this line
a = 10
test(5)

Versions

Spyder version: 4.1.1
Python version: 3.7.5
Qt version: 5.9.6
PyQt version: 5.9.2
Operating System name/version: win 10

Thanks for reporting. It seems the user namespace is shadowing the pdb namespace and that's why a is shown as 10 and not as 5.
@impact27, could you take a look at this one?

Fixed in https://github.com/spyder-ide/spyder-kernels/pull/212

Thanks for the quick response @impact27! Could you add a test for this? (I don't know if it's necessary to do it here or in spyder-kernels).

https://github.com/spyder-ide/spyder/pull/12117


Result systems

The only way to handle errors in systems right now is to result.unwrap() them. This is both less ergonomic and panic-ey. Ideally systems could optionally return an anyhow::Result<()>. It would also be great if developers could define their own error handlers. Maybe some devs want to print their errors whereas others want to panic.
I think there are two approaches we could adopt here:

Adapt legion to support return types
Add impls for system fns that return results and handle the Result inside of into_system()

I think option (2) is the least destructive / most friendly to upstream, but it means that only system fns can handle errors.
If this is implemented, it also makes sense to modify bevy libs to return error types instead of Options for common operations to improve erognomics.
fn some_system(a: Res<A>, b: Res<B>, x: Com<X>, y: Com<Y>) -> Result<()> {
  a.do_something_risky()?;
  // system logic here
  Ok(())
}

// inside into_system() impl for Fn(Res<A>, Com<X>) -> Result<()>
system_builder.with_resource::<ErrorHandler>()
// resources are (error_handler, a, b)
result = run_system(a, b, x, y);
error_handler.handle(result);

The biggest downside to implementing this I can think of is that this multiplies the number of system fn impls by 2 (which is already in the hundreds of impls). That would come at an estimated clean-compile time cost of 40 seconds on fast computers ... not ideal. The best way to mitigate that is to revert to non-flat system fn impls, which would then only require a single new impl:
Doing so would both remove the need for a macro for system fn impls and reduce clean compile times by 40 seconds (current) / 80 seconds (with Result impls). But its also not nearly as nice to look at / type
// pseudocode
impl IntoSystem for Fn(ResourceSet, View<'a>) -> Result<()> { /* impl here */ }

fn some_system((a, b): (Res<A>, Res<B>), (x, y): (Com<X>, Com<Y>)) -> Result<()> {
  a.do_something_risky()?;
  // system logic here
  Ok(())
}

What happens to the error returned by a system? What happens to systems after the one that produced the error, are they skipped? If not, what happens if several systems return errors?
In my opinion, if the error needs to be handled somewhere other than the system that produced it then a more direct way of transferring the necessary data would be more fitting (e.g. a channel).

Skipping errors is certainly not desirable (at least while developing). The "error handler" that i mentioned above would probably still panic by default. But it could also be configured to just print the error or write it to a file.
In order for such an error handler to work, we would likely need to be able to send them from the system execution thread to some receiver. As you mentioned, a channel would make a lot of sense here.

I was insinuating that for all of the systems in a game there's unlikely to be a good one-size-fits-all error recovery strategy. Systems related to different mechanisms of the game are bound to have different recovery requirements, users shouldn't have to lump them all together into a single handler. This isn't something that the engine should anticipate.
If this isn't about error recovery but strictly about log and/or panic - still not convinced that returning results is going to win any ergonomics points, versus providing convenience utilities to do that on site of the error. Like a logging plugin; one can be easily built, for example, on top of log and any of it's compatible libraries (I like fern).

If this isn't about error recovery but strictly about log and/or panic - still not convinced that returning results is going to win any ergonomics points, versus providing convenience utilities to do that on site of the error.

Being able to use ? in a system would be more ergonomic compared to destructing the result and calling a function, no?

For systems where ? is preferred for many early returns, I have so far used a separate function for the system's implementation. The system can then handle the error and/or send it to an error handler via an event.
fn fallible_system(state: ResMut<State>, query: Query<Component>) {
    if let Err(e) = fallible(state, query) {
        // Handle the error here and/or forward error as an event,
        // e.g. error_events.send(e);
    }
}

fn fallible(mut state: ResMut<State>, mut query: Query<Component>) -> Result<ok, error> {
    // use `?` and friends
}

Resolved by #1021 (although the error handling story could still be improved)


Unabled to extract existing text from locales.

Describe the bug

With version 2.10.x, while debugging the extension, I am sure the extension can get the replace recommendations prefectly.
When updated the fork repo and build the extension locally, with the version 2.3.3 activated, the extesion failed to load the replace recommendations.
Extension Version
v2.3.3
i18n Ally
Framework/i18n package you are using
vue-i18n
To Reproduce
Steps to reproduce the behavior:
Switch to the new version
Device Infomation

OS:macOS
Version:10.15.6
VS Code Version:1.48.0

Extension Log
Go to View -> Output -> i18n Ally, and paste the content below. You should mask any sensitive information
🈶 Activated, v2.3.3

――――――

💼 Workspace root changed to "/Users/apple/Documents/sourcecode/seeworld/iotweb2"
🌞 Enabled
🧩 Enabled frameworks: Vue
🧬 Enabled parsers: json

🚀 Initializing loader "/Users/apple/Documents/sourcecode/seeworld/iotweb2"
📂 Directory structure: file
🗃 Path Matcher Regex: /^(?<locale>[\w-_]+)\.(?<ext>json)$/

📂 Loading locales under /Users/apple/Documents/sourcecode/seeworld/iotweb2/static/lang
	📑 Loading (en) en.json [1597719664580.499]
	📑 Loading (zh-CN) zh-CN.json [1597717873920.1787]

👀 Watching change on /Users/apple/Documents/sourcecode/seeworld/iotweb2/static/lang
✅ Loading finished

Screenshots

I am trying to debug this one

Hi, I noticed this happens after the commit 2bd4855a4071370bcafdd44ba5951d64a364e9cf, the extract.ts file has a block of code deleted after the this commit. Dir:src/core/editor/extract.ts
CurrentFile.loader.keys.map(key => ({ label: key, description: CurrentFile.loader.getValueByKey(key, Config.displayLanguage, 30) }))
      .filter(labelDescription => labelDescription.description === text)
      .flatMap(t => Global.refactorTemplates(t.label, document.languageId))
      .map(t => ({
        command: Commands.replace_with,
        title: i18n.t('refactor.replace_with', t),
        arguments: [t],
      }))
      .forEach(c => commands.push(c))

this block was deleted after the 2bd485 commit. And which leads to a empty commands array. I looked into the update in extract.ts, and found no other changes.
I wound if you have changed the way to extract key in locales and provide code actions in a different way. Thank you very much.

Oh, I see. It's unified into a single command which is in your screenshot. Try with that, I will show you an option to replace it with existing if you have an exact match.

https://github.com/antfu/i18n-ally/pull/368#issuecomment-672789950

Checked. No replace command in quick pick item view. Only extract options are presented. Pretty sure I have the exact match. Cause when I use the old version it worked perfectly.

I see. It's get filtered out. Should be fixed in 2.3.4

Now it has an existed translation option but when I choose it, not thing pops out.

Thanks, fixed in 2.3.6.

Cool


Fixes code styling of new.html.subtemplate.

@smcgregor sir,
Hi!
Since I found all Level 1 task were done, I did this style fixing. Or sir you may assign me a task.

:+1:


MB-12516 Removed awardqueue

Jira ticket for this change
Summary
Removed unused pkg/awardqueue

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against f5a6786827e85cdcb0306ad1114f197f7f737cb2

Ticket to capture the calls/models that awardqueue uses...
https://dp3.atlassian.net/browse/MB-12601


Loading Mocketry with Core group loads tests of dependencies

I wanted to load the 'Core' group of Mocketry to limit the amount of code I load in some project but doing so still loads the tests of StateSpecs and Ghost.
Maybe the granularity of the groups could be reworked so that we can avoid to load the tests of the dependencies when we do not load the tests of Mocketry?

Hi @jecisc .
It is known bug of Metacello. But author does not consider it as a bug. See discussion here https://github.com/Metacello/metacello/issues/472.
You can check baselines of Mocketry and dependencies. They all clearly define that Core depends on Core of dependencies and should not include any tests.

Looking deeply at baselines I found my mistake. So it will be fixed in upcoming release.
(Not sure if it is related to my Metacello issue because I used simpler projects in that case).

Thanks a lot!
I added to my todo list to check the Metacello bug, but if it works without it it's even better since I don't have much time to check!

Hi, @dionisiydk
I can confirm that loading Mocketry core does not load the tests of StateSpecs and Ghost anymore.
Thank you!

(using v6.0.x)


Add python serde support for varopt sketches

The immediate result is for varopt, but having this working should be an important part of adding tuple support.
I'm least confident in the error recovery during deserialization. I think it's most likely that Python will throw an error which propagates through the c++ code. In this case I'm intercepting it to try to decrement reference counts for items we've already allocated but it's hard to tell if that's actually preventing a leak or not. Happy for suggestions on how to clean up that code.
Also, fixed a size bug in varopt.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
2196

Relevant Lines:
2378

💛  - Coveralls


Optional fields which were mandatory earlier creating failure with older service version

[ ] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[ ] Updates to module CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

Please add an entry to CHANGELOG.md as part of this fix.


RenderBox was not laid out: RenderRepaintBoundary#d49d2 relayoutBoundary=up1 NEEDS-PAINT 'package:flutter/src/rendering/box.dart':

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

@marqueetest
When opening an issue, please try to provide all or some of the information indicated in the template.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


Create email field

@richardhjtan added a simple regex validation. Not trying to promise a whole validation system. Can upgrade or remove as we go along.


linting part 2 -complete as of 5:12 pm 8-12-17

couldn't run the linter through the terminal on the front end files
BUT fixed linting on front end, added two exceptions (confusing arrow rule override and exception to camelcase), as well as a small refactor with redux form and renderField method.

I just did this pull to see if CI was catching client linting and tests, it is not.

I modified the rootReducer function to use a ternary operator to assign newState. @perfectlynormalbeast Let me know what you think 😸


Import shorten memtable switch latency #14

pls review f3a7b17f3551b2a0e52c1a430ea0449e28a79329

pls review

@ayulas , check sources fails on:
https://github.com/speedb-io/speedb/actions/runs/3168558728/jobs/5159839033#step:14:17
build_tools/check-sources.sh
plugin/speedb/memtable/hash_spd_rep.cc:using namespace stl_wrappers;
make: *** [Makefile:1221: check-sources] Error 1
^^^^ Do not use "using namespace"

QA passed on 63ee520058f53d406a3ac8ec46c673a44df09c38

Switch functionality was added to ticket https://github.com/speedb-io/speedb/issues/98 PR https://github.com/speedb-io/speedb/pull/188 https://github.com/speedb-io/speedb/commit/7e5e7bc04bb8813f4bea66dc6030074b4b2658c3


Fine-tune fingerprint if the final jar is shaded

The maven-shade-plugin can perform relocations, debloating, etc. The fingerprint should account for that.

"The ConstantPool and Methods are the core of a Java class, just as the code is for a native application [1]"
[1] H. Ba, H. Zhou, H. Qiao, Z. Wang, and J. Ren, ‘RIM4J: An Architecture for Language-Supported Runtime Measurement against Malicious Bytecode in Cloud Computing’, Symmetry, vol. 10, no. 7, Art. no. 7, Jul. 2018, doi: 10.3390/sym10070253.


Update readthedocs to use vcs

lbr_fri_ros2_stack uses vcs tools for the fri dependency. This dependency cannot be modeled as a submodule in readthedocs

done #10


FancyZones: remove dialog boxes filtering

Summary of the Pull Request
We added the dialog box styles to the "not zonable window" filter, to prevent Save As.. and similar dialogs to be assigned to a zone. This caused a lot of apps that use dialog styles to not work with FancyZones. This PR removes those dialog box styles, since the "window has no owner" rule makes the actual dialog boxes not zonable.

PR Checklist

[x] Applies to #970 #908, possibly also to #894, but I did not verify this.
[x] CLA signed

Validation Steps Performed
Manual

Did I understand correctly that this PR depends on https://github.com/microsoft/PowerToys/pull/1078 ?

No, those are two separate changes.

Tested with Notepads Save as dialog, and dialogs in https://github.com/Marginal/EDMarketConnector


Do WSL support ROCm（GPU computing of AMD)?

T use GPU RX580，i wnt to do march learning. AMD's  Open GPU Computing is only for linux now. so if i use WSL, can i useing ROCm and tensorflow-gpu?

Basically same as all-time favorite #829.


Unit conversions

UserMeasurementUnitMetric on all objects exposing metrics.
Added ISupportUserMeasurementUnits interface to make code in StravaService class more elegant

Noticed you dropped quite a few INPC's. Didn't open the the codebase yet, but I suppose you checked that they're not needed?

@bartlannoeye : correct! These aren't required any more. All the changes of units are now handled in the UserMeasurementUnitMetric class.
The properties of type UserMeasurementUnitMetric should only be set once on a particular object and should change during its lifetime (so no need for INPC there).


[SPARK-12113] [SQL] Add some timing metrics for blocking pipelines.

This patch adds timing metrics to portions of the execution. Row at a time pipelining means these
metrics can only be added at the end of blocking phases. The metrics add in this patch should be
interpreted as a event timeline where 0 means the beginning of the execution.
The metric is computed as when the last task of the blocking operator computes. This makes sense
if it is interpreted as a timeline and gives a measure of wall clock latency of that phase.
For example, in a plan with
Scan -> Agg -> Exchange -> Agg -> Project. There are two blocking phases: the end of the first
agg (when it's done the agg and not yet returned the results) and similar for the second agg.
This patch adds a timing to each that is the time since the beginning. For example it might
contain
Agg1: 10 seconds
Agg2: 11 seconds
This captures a timeline so it means that Scan + Agg1 took 10 seconds. Agg1 returning results +
exchange + agg2 took 1 second. We can post process this timeline to get the time spent entirely
in one pipeline.
This adds the metrics to Agg and Sort but we should add more in subsequent patches. The patch
also does not account of clock skew between the different machines. If this is a problem in
practice, we can adjust for that as well if ntp is not used.

I've attached two examples running different tpcds queries. For Q65 (the bigger plan), we can see some useful things. Looking at the chain of 3 aggregations in the top middle, the first took 36.8 seconds to complete. This includes all the time in the operators before. The next aggregation finished at 36.8 (meaning it is itself simple) and the third aggregation finished at 1 min.

Test build #47104 has started for PR 10116 at commit c386e56.

Test build #47104 has finished for PR 10116 at commit c386e56.

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

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

Merged build finished. Test FAILed.

@nongli ISTM this is good performance metrics for users. You do not keep working on it?
One question; why do you compute finish time from a plan start instead of accumulated duration time for each blocking stage?

@nongli ping

I don't have a lot of time to work on this. Do you have any interest in taking it over?
with regard to your question, I think there are many ways we can present this. I generally find the "user wall clock timeline" to be the easiest to think about.

@nongli okay and I'll do it.

@maropu can you take this over?


Add joint velocity limits task and switch to toml configuration files in tests

This PR adds a task to add joint velocity limits and switches the type of configuration files in tests from ini to toml.

The CI is failing because the task JointVelocityLimitsTask is not available in the latest version of blf on conda; I would merge this PR and then fix the CI.

The CI is failing because the task JointVelocityLimitsTask is not available in the latest version of blf on conda; I would merge this PR and then fix the CI.

This will be fixed by https://github.com/conda-forge/bipedal-locomotion-framework-feedstock/pull/81 in 2/3 hours.

The CI is failing because the task JointVelocityLimitsTask is not available in the latest version of blf on conda; I would merge this PR and then fix the CI.

This will be fixed by conda-forge/bipedal-locomotion-framework-feedstock#81 in 2/3 hours.

Great, thank you! Then I would wait for it!

Ok, I was a bit optimistic, the packages were uploaded ~1 hour ago: https://anaconda.org/conda-forge/bipedal-locomotion-framework/files, hopefully they should be CDN propagated now or in a few minutes.

Merging as the CI was successful!


Resource loaded without being associated with script class name

Tested versions
v4.3.dev2.official [352434668]
System information
Windows 10 Mobile
Issue description
I'm running into a strange error where attempting to load a custom resource with a class_name results in an object with none of the resources properties and which cannot be passed to any method which requires the class_name.
It seems to be related to having a resource that contains an array that preloads other resources.  In my original project, I was using custom resources as UIDs to reference game levels and items.  I wanted to create a class that I could use as an index of all rooms and items in my game which my game's save file loader could reference to turn string ids back into these resource UIDs.  However, this seems to be breaking things.
Steps to reproduce
Run the attached project.  Press the Test button.
Minimal reproduction project (MRP)
resource_load_test.zip

Possibly related:

#87019

I've reworked the code so that the lists just contain String names and I load() them later.  This works around the issue.  And I didn't realize this preload this was likely what was causing the issue until I was finishing up this bug report.  But Godot's inability to handle cyclic references makes any use of preload or PackedScene an obscure bug just waiting to happen.  It can be really hard to make sure you're not accidentally creating a cycle when you're using custom resources and your code starts to get large.

Yes, this looks pretty similar to that other error you referenced.

I got a similar error but even without preloading.
But otherwise the same situation as you have.
I have a Dictionary with object in it. And for some he does not detect what type of Resource it is, therefore I always get an error.
Invalid call. Nonexistent function 'xxx' in base 'Resource'.


Need masternode list fix Windows.GUI

In windows GUI qt exe

Need to fix auto-adding masternodes to list-conf from masternode.conf.
Need to fix multiple masternodes running.
Need to fix libpng errors.
Need to fix INVALID PORT ERROR from masternode.conf. See debug.log
Then recompile bithold-qt.exe

In the file masternode.conf there is added masternode.
After running this, no any masternodes in list-conf:
console command:
masternode list-conf
response:
{
}

So need to add masternode by button "Create...", every time, and start this masternode.
Need to fix this bug.
Also, I see an error in window, when wallet is closing.
I see many errors in debug.log:
GUI: libpng warning: iCCP: known incorrect sRGB profile GUI: libpng warning: iCCP: cHRM chunk does not match sRGB
GUI: QObject::connect: Cannot queue arguments of type 'QVector<int>' (Make sure 'QVector<int>' is registered using qRegisterMetaType().)
port error, but port was ben specified correctly:
Invalid port detected in masternode.conf: MasterNode domainname.com:15971 ... (must be 15971 for mainnet)
Masternode added to list only if this created again:
CActiveMasternode::Register() - Adding to masternode list service: IP:15971 - vin: CTxIn(COutPoint(4ed51f7def, 0), scriptSig=) CActiveMasternode::Register() - RelayElectionEntry vin = CTxIn(COutPoint(4ed51f7def, 0), scriptSig=) CActiveMasternode::EnableHotColdMasterNode() - Enabled! You may shut down the cold daemon.
Then, I can see my masternode in the masternode-list and by console command
masternode list-conf
command
masternode status
return status 2. Unable to connect by IP:port.
Also, I cann't see connections to the node. I did open port 15971, but this is still closed.
Test is failed:
CActiveMasternode::ManageStatus() - Checking inbound connection to 'MY.ACTUAL.IP.ADDRESS:15971' CActiveMasternode::ManageStatus() - not capable: Could not connect to MY.ACTUAL.IP.ADDRESS:15971
Need to fix this and recompile windows.GUI.QT.exe

Also, how to run multiple masternodes in one wallet on windows?
When I try to run two masternodes, I see 0 seconds for one from this, by using command
masternodelist full MY_IP:PORT
But for second masternode, time of running this, have the value greater than 0, in seconds...
My masternodes are disabled sometimes, and sometimes I see status EXPIRED,
by console command: masternodelist full MY_IP:PORT
So I need to do manually restart my masternodes... Very very bad...
I cann't try to using ubuntu-version, because I cann't install ubuntu to VirtualBox, there is errors on Windows...
Can you write instructions about compile wallet-qt.exe for windows?
For example, you can see this: https://github.com/username1565/signatum-source/blob/master/doc/build-msw.txt

How to start bithold-qt.exe on windows, in portable mode, with parameter -data-dir="./data/"?

How to start and config multiple masternodes? Is there any guide, like this:
https://github.com/HarvestMasternodecoin/Harvestcoin/wiki/MULTIPLE-MASTERNODES-SETUP-GUIDE
Can I specify a many masternodeprivkeys in bithold.conf, like a many "addnode notes"?

Is there any script to auto-check and autorestart ALL masternodes, when this have a status EXPIRED?
Can I do this using any python-script through JSON-RPC API, or ANY HTTP RPC or telnet?
I see by command [quote]masternodelist full MY_IP:PORT[/quote]
many my masternodes have a status expired.

How to add masternodes from masternode.conf to list-conf?
I have masternodes in masternode.conf, but console command:

masternode list-conf
give me an empty responce.
So need to add masternodes, by "Create..." button every time after each loading bithold-qt.exe
And in this case, masternodes adding to list-conf...
Is there any command or parameter to add masternodes in list-conf?

Why I see so many libpng errors in debug.log? How to fix this?

How to fix INVALID PORT ERROR for the notes of masternode.conf, in debug.log?
I did specify default port 15971, but I see this errors in debug.log

How to compile bithold-qt.exe? Is there any instructions??

Can you, devs, fix all this and recompile bithold-qt.exe for windows yourself?

Just need to config strat wallet to resolve this all.


PowerShell v6.2 preview 2 won't start on Windows 10

Downloaded and installed v6.2 preview 2 using the msi. It installed over preview 1 as expected.  PowerShell v6.1.1 is also installed.  No apparent errors during installation.
Attempting to start V6.2 preview 2 there is a flash of the console opening and then closing immediately.
Attempting to start from the command line - Windows PowerShell and CMD - the following error is displayed:
C:\Program Files\PowerShell\6-preview> pwsh.exe
Error:
  An assembly specified in the application dependencies manifest (pwsh.deps.json) was not found:
    package: 'System.Private.ServiceModel', version: '4.5.3'
    path: 'runtimes/win/lib/netstandard2.0/System.Private.ServiceModel.dll'

Windows build is 10.0.17134.407 - fully patched Windows 10 1803
The PowerShell v6.1.1 installation has two servicemodel dlls
PS>  ls  *.servicemodel.dll

    Directory: C:\Program Files\PowerShell\6

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----       19/07/2018     19:32        1889344 System.Private.ServiceModel.dll
-a----       19/07/2018     19:32          30784 System.ServiceModel.dll

BUT the v6.2 preview 2 folder only has one
PS> ls '*servicemodel.dll'

    Directory: C:\Program Files\PowerShell\6-preview

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----       19/07/2018     19:32          30784 System.ServiceModel.dll

Any help / ideas appreciated
UPDATE:
tried a repair from the MSI and it now seems to work. Very odd!
This issue can closed

I have the same issue with Powershell Core 6.2.0-preview.2 with same Windows version as above. Running the installer again and using "repair" seems to fix the problem.

I got to the habit of always to uninstall the preview version.
But, the Issue is valid only if you install the preview over an existing version.
If you first uninstall the preview and then install the latest version, there is no problem starting pwsh-preview.exe.
This is a bug!

Its a bug as further checking shows the dll is in the zip and the msi.  As to uninstall previous version first - the whole point is to make that unnecessary. With PowerShell core versions been released on a relatively frequent basis the install should be a simple process otherwise people won't bother

Duplicate of: https://github.com/PowerShell/PowerShell/issues/8289


Ensure that the cached layout mode is in sync

Fixes https://github.com/godotengine/godot/issues/71178.
I missed this in https://github.com/godotengine/godot/pull/70882. Now that the relationship between the two editor-only properties is validated more strictly, some operations require us to set the layout mode beforehand. This doesn't affect setting anchors directly, using handles or set_anchor, set_anchors_preset, etc. But it does affect the inspector and other parts of the editor UI which need a bit more work to keep in sync.
With this PR I fix two cases:

Controls now check for the node they've been parented to to update the layout mode, if necessary (without changing anything else, so user changes should be preserved).
Using the toolbar to change the anchors preset on controls currently in the "Position" mode works again (changing the preset with the inspector didn't suffer).

Thanks!


CCU Backup

After update to the latest version and reinstallating a CCU backup of a former version the communication to HM devices is not working anymore.
Any info what might be wrong ?

Please provide the output of pivccu-info

Solved it ... Took an old image from CCU2 for a CCU3 ;-)


validator: Check IP config in bond slaves.

When a bond interface is created, the slaves cannot have their IP
configuration enabled. In order to fix this problem, this patch
introduces a validation check to see whether the slave interfaces had
their IP enabled and if so, it raises an NmstateValueError.
Ref: https://bugzilla.redhat.com/1782674
This fixes #774
Signed-off-by: Elvira García Ruiz<EMAIL_ADDRESS>

Coverage remained the same at ?% when pulling 37beaa614302a4768846e9c1bb400259f562efbe on elvgarrui:bond_ip_problem into 33bf357b3c22b809267aa5a6a8726d97a28cad5e on nmstate:master.

Coverage remained the same at ?% when pulling 37beaa614302a4768846e9c1bb400259f562efbe on elvgarrui:bond_ip_problem into 33bf357b3c22b809267aa5a6a8726d97a28cad5e on nmstate:master.

Merged. Thanks.


Implement a did dependencies change that calls onEvent

It would be impossible for it to call the onEvent function, however, I can make it call another pre-defined function when dependencies change on the page widget.

Done


Make it feasible to use the component states via the C API without serializing

Using #[repr(C)] structs for certain data types could really improve the usability of the C API compared to parsing JSON.
I have very little ideas for what this could look like so I'm going to experiment a bit here. We will definitely need to add support for this in the bindings generator.
This should be labeled as low priority as this isn't too important but would definitely improve API usability.
General Idea:
#[repr(C)]
struct Data {
    x: u32,
    y: u32,
}

would become
struct Data {
    uint32_t x;
    uint32_t y;
};

The libc crate will probably be extremely useful here.

I don't think that's feasible. It's important that we can generate bindings for all the languages that we are currently generating bindings for. So this would mean that we need to figure out how to express C structs in all of those languages (which contain scripting languages like Python, Ruby, JavaScript and co. which will complicate this) and also we need to annotate everything with repr(C), which we can't even do for everything. Any HashMaps, Vecs and co. will make this basically impossible.
So what's much more likely is that we are staying OOP, not leak any implementation details and have methods for accessing all the data like we already do in the C API. This just needs to be done. However we will then run into another problem (indirectly) and that is that the C API is getting extremely big. To the point where it significantly hurts the web version of LSO. So we then likely also needs to figure out how to conditionally compile in subsets of the API, instead of the whole thing. But that needs to be understood by the binding generator then (it's not that hard to do this and we need it anyway for networking and co.).

Personally I don't think making everything in livesplit-core #[repr(C)] is the right approach here to begin with, and the C API having hundreds of getter functions isn't good either. A good solution would involve having the C API define its own boxed C structs and passing them around. We could then return a lot more data from a single function call then we'd be able to otherwise.
I experimented a bit with this outside of livesplit-core and it seems to work a lot better than I had imagined.

I've also started looking for the documentation we'll need to support this in ever language we currently generate bindings for.

[x] C/C++ - Native Support
[ ] C# - Supported but can't find documentation for it yet.
[x] Java (JNA) - Documented Here
[ ] Java (JNI)
[ ] Kotlin (JNI)
[ ] Swift
[x] Ruby - Documented Here
[x] Python - Documented Here
[ ] JavaScript (Node FFI)
[ ] JavaScript (Wasm)

You don't actually need to heap allocate structs at all on either side then. Although it certainly does make it easier for unsupported languages to handle them.

What I'm thinking a feasible way of going about this is to simply "serialize to a C struct". I.e. keep everything "repr(rust)" in the normal code, but similar to the json functions, we would have functions that spit out a raw C representation of the serialized state.

This is the solution I had in mind.

However I still am not sure if we want that at all tbh. We can do all of this via the opaque object oriented way that is already in use all over the API that works perfectly well across all the languages.

I ran into a roadblock working out a good design for the languages that arent C:
How should we handle this in languages where we wrap the opaque pointer instead of dealing with the pointer directly? The best solution I can think of would be automatically generating getter functions that dereference the pointer and get the appropriate data, but this will require changing how class handling works. (Said getter functions will be implemented in the target language rather than the rust side)

After a bit of experimentation and thought, I feel like automatically generating getters that access struct data directly for our OO style bindings is the best way to go about this.
These getter functions would be a part of the bindings that are generated rather than implemented on the rust side to avoid having to refactor how the bindings are done entirely.
I also have a general idea of how to best implement the getter functions as well for the datatypes we might want to be able to use:

Getting a string should copy the C string to a native string and return that.
Getting a primitive value should just return the primitive value.
Getting a pointer to another struct should construct a new ref object using that pointer and return that. There are some lifetime semantics to keep in mind here so we need to avoid using this if possible.

When I find time, I'll work on a proof of concept for this in our C# bindings


Remove incentivized channel & channel ID from relayer

mage build passes. Will regenerate the bindings in contracts/ in the Solidity PR. Will run E2E tests once Solidity code is removed.
The main changes here are follow-ons from #753:

Remove the incentivized channel.
Remove channelID.

Also removed now-unused code & simplified code that now only has a single path. eg. Events struct is now just BasicChannelEvent and we don't need a mapping from address to contract method.

LGTM, Simplifies the codebase a lot, Great! @doubledup

@claravanstaden The Go contract bindings are removed in #757 where we're removing the contracts. I'd actually forgotten to check that the incentivized channel bindings are removed there, so good catch! 🙏


CI test scripts should fail when any command fails

You're too quick! ;-) Indeed it does, and that was the intended result! @gvdongen pointed out that the cdk synth test in the Lambda/CDK samples has been quietly failing - I wanted to make sure that the workflow fails before merging the fix.


S3 PutBucketTagging moto bug if only 1 tag is supplied

As it turns out, if you only supply 1 tag in the s3.put_bucket_tagging() operation, the _bucket_tagging_from_xml() in S3'sresponses.py breaks because the data type is a dict and not a list apparently.
I'm going to submit a PR to fix very soon.

Fix in #1182


Koa not waiting on await

Having a issue where Koa is returning early and not waiting for any of my await functions to finish.
Node: 9.10
//root.js
module.exports = async (ctx, next) => {
   const test = new Promise(function(resolve, reject) {
    setTimeout(() => {
      return resolve('word');
    }, 1000);
  });

  ctx.body = await test
}

//index.js
app.use(async ctx => {
  switch(ctx.path) {
    case '/':
      await root(ctx);
      break;
  }
})

This always throws a 404. How can I get Koa to wait for the await function to finish?

Figured it out, I need to be returning the middle into Koa itself and check the route in module itself because of the way i set it up. Sorry to bother.

Ran into another similar issue with await Promise.all
//root.js
  let feedPromises = feeds.map(feed => {
      return parser.parseURL(feed); //return pending promise
  });

  let result;
  try {
    result = await Promise.all(feedPromises) // after this line koa returns
  } catch(e) {
    console.log(e);
  }

  ctx.body = result; // this is happening after the return

I'm going to to need a full example of what you're doing to pinpoint any issue. There's insufficient information for me to be helpful here. On a different note, general usage questions are better suited elsewhere and you might get help faster - unless you think this is a Koa bug in which case I'd ask you to provide a fully reproducible example for further investigation.

Sorry, provided a full example. I'm not sure if its a bug, the behavior doesn't seem like its inline with how the docs describe the behavior but I can definetly be wrong.
//index.js
const Koa = require('koa');
const app = new Koa();
const update = require('./routes/index.js');
app.use(index);
app.listen(3000, () => { console.log('listening on 3000') });

// routes/index.js
module.exports = async (ctx, next) => {
  let promise = () => {
    return new Promise(function(resolve, reject) {
      setTimeout(() => {
        return resolve('word');
      }, 10);
    });
  }

  result = await Promise.all([promise()])
 // result = await promise() //this and the above line are happening after koa has already returned

  ctx.body = result;
}

The only thing I see is that you're assigning to result without defining it. In strict mode this causes a ReferenceError, not sure how it plays out in normal mode. Anyway, I can't recreate your issue because I'm getting expected results (i.e. word).

You require routes/index.js as update, but you app.use it as index.

Ended up figuring it out. I was testing this code inside of a larger koa application. And in one of the other middleware files, I was not using await next(), when I updated my code everything worked properly. My bad.

Hello , I have similar issue :
app.use(async (ctx, next) => {
    const req = ctx.req.req || ctx.req;
    ctx.mySelf = await new Promise((resolve, reject) => {
        // req.on('data', (buffer) => {
        //     const value = buffer.toString('utf-8')
        //     resolve(JSON.parse(value))
        // })
        setTimeout(() => {
            resolve('test')
        }, 1000)
    })
    console.log(ctx.mySelf)
    await next()
})

It works , but If run the annotated code, the http request is always pending.
The console also has no output
interview.js
const interviewRouter = require('./routes/interview');
app.use(router.routes()).use(router.allowedMethods());
router.use('/interview', interviewRouter.routes(), interviewRouter.allowedMethods());
interviewRouter.post('/post', async (ctx) => {
    console.log(ctx.request.body)
    console.log(ctx.mySelf)
    ctx.body = {
        success: true
    }

})

Ohh. My fault... It doesn't matter of koa.

@Cmdrobot you probably just missed to return. Remember that Promise.all also is a promise.

@Cmdrobot , you can simplify your code, and show it. It's better way for get answer.

This code works very well,  and don't know what your question. @Cmdrobot
var app = new Koa();
var router = new Router();

function f1() {  // no need use async in here, async and await use by pairs
  return new Promise((resolve, reject) => {
      let message = {
          greeting: "Hello",
          planet: "World"
      }
      setTimeout(function(){  // why use setInterval?
          resolve(message)
      }, 3000);
  })
}

router.get('/test-async', async (ctx) => {
  ctx.response.body = await f1();
});

app
  .use(router.routes())
  .use(router.allowedMethods());

app.listen(3000);

Ended up figuring it out. I was testing this code inside of a larger koa application. And in one of the other middleware files, I was not using await next(), when I updated my code everything worked properly. My bad.

I spent many hours finding the issue and this made it clear ! thanks a lot @fedgrant

Ended up figuring it out. I was testing this code inside of a larger koa application. And in one of the other middleware files, I was not using await next(), when I updated my code everything worked properly. My bad.

You saved me, man. Thanks a lot @fedgrant

Ended up figuring it out. I was testing this code inside of a larger koa application. And in one of the other middleware files, I was not using await next(), when I updated my code everything worked properly. My bad.

Thanks! I spent many hours too for this too...


RPC /dump_consensus_state router is not working

Bug Report
After doing some chains upgrades, I experienced that chain's tendermint rpc endpoint wasn't working when querying  /dump_consensus_state path. However, when I query other paths (/status, /consensus_state), they work fine.
I mean that other route path( /status, /consensus_state) are working good

And dump_consensus_state handler is not working right now and the result is as follow

# curl -m 5 -sS localhost:26657/dump_consensus_state
curl: (28) Operation timed out after 5000 milliseconds with 0 bytes received

I found this issue when doing a chain upgrade for  Band(v2.5.2) and Cosmos(v10)
Setup
CometBFT version (use cometbft version or git rev-parse --verify HEAD if installed from source):
node command runtime flags:
Gaia(cosmos v10) uses cometbft core v0.34.29
Band(v2.5.2) uses cometbft core v0.34.28
(personally, I think the problem is being caused after v0.34.28)
How to reproduce it

Just call the https://<public-rpc-endpoint>/dump_consensus_state and https:///status
dump_consensus_state normal output and expriements
Actually, we can get json data like that if it works fine

Finally, i've been tested for about it with fixing some codes like bottom and then I got the problem was where to be caused by calling peer data.
so, If i modify some codes about calling peers, the dump_consensus_state router works fine!
https://github.com/cometbft/cometbft/blob/v0.34.28/rpc/core/consensus.go#L51
// DumpConsensusState dumps consensus state.
// UNSTABLE
// More: https://docs.cometbft.com/v0.34/rpc/#/Info/dump_consensus_state
func DumpConsensusState(ctx *rpctypes.Context) (*ctypes.ResultDumpConsensusState, error) {
	// Get Peer consensus states.
	peers := env.P2PPeers.Peers().List()
	peerStates := make([]ctypes.PeerStateInfo, len(peers))
	// for i, peer := range peers {
	// 	peerState, ok := peer.Get(types.PeerStateKey).(*cm.PeerState)
	// 	if !ok { // peer does not have a state yet
	// 		continue
	// 	}
	// 	peerStateJSON, err := peerState.ToJSON()
	// 	if err != nil {
	// 		return nil, err
	// 	}
	// 	peerStates[i] = ctypes.PeerStateInfo{
	// 		// Peer basic info.
	// 		NodeAddress: peer.SocketAddr().String(),
	// 		// Peer consensus state.
	// 		PeerState: peerStateJSON,
	// 	}
	// }
	// Get self round state.
	roundState, err := env.ConsensusState.GetRoundStateJSON()
	if err != nil {
		return nil, err
	}
	return &ctypes.ResultDumpConsensusState{
		RoundState: roundState,
		Peers:      peerStates}, nil
}

This is very likely caused by https://github.com/cometbft/cometbft/issues/863

and also, if someone do dump_consensus_state query path over and over again, I think the node become dead.

This should be fixed in  v0.34.29.

peerStateJSON, err := peerState.ToJSON()

In the original issue, this method was triggering in a recursive loop to produce the JSON encoded version of the peer state (a summary of the consensus state of every connected peer). The bug doesn't affect the consensus state (the state of the local consensus instance), that is why your workaround probably works.

Wait for a while. I'm checking other chains with v0.34.29

Thanks! @cason
In cometbft v0.34.29, the problem was resolved greatly, the dump_consensus_state query works fine.
I'll close this issue


Add support for lowercases DN elements

A follow up to https://github.com/cesanta/docker_auth/pull/228#issuecomment-389545187 comment:

FreeIPA seems to have types of DN elements lowercased

In this PR we transform DN attributes to upper case (as in most cases they are already uppercased, so it looks like it's more optimal then lowercasing) and compare to "CN".

merged, thanks!


hygieia-jenkins plugin

Hi,
Couple of questions. After the .hpi is uploaded in jenkins, instructions to enter integration token and Hygieia server URL were given, which i dint not understand. i added /api to my Hygiea dashboard URL and that didnt bring up any thing.

secondly what is the integration token and where can i get it from.
thanks

Currently the integration token not been implemented.
Once you have set the Hygieia API url set, "Apply" the config and then "Test Connection". If it says "Success", you can now create a job and publish events to Hygieia.

thank you once again for your responnse. This where i am stuck.. how do i set my API url.
apologies if I am being dumb!!


Warn when non default ciphers are removed because the engine rejects them.

When using nmap --script ssl-enum-ciphers -p port host
the ciphers proposed are often a small subset of the expect ones.
A warning helps to see what is rejected.

LGTM


Discuss pros/cons - yaml versus helm charts versus cdk8

Currently we are deploying yaml based definition of microservices. They are easy to create and use. Helm charts are more complex but they allow modifying the template, passing in HW information to even determine which image to download. CDK8 is yet another format that allows varying allocated resources. With Service mesh both yaml and helm charts work to trigger launch of services.
It would be good to pause and determine if we should support one or more formats of each for the sample microservices and GMC support connecting services in any format.

FYI: Helm (explicitly) does not support updating nor removing CRDs, for those one needs to apply/remove manifests manually any way.
If common configuration options are in e.g shared configMaps, instead of hard-coded to multiple pod specs, IMHO editing them is not much harder than using Helm.  IMHO more interesting question is which of those options could be updated at runtime / do OPEA services have config reloader like e.g. Prometheus services have?

Thank you @eero-t ! Will be leveraging your expertise in this space.


[Feature] Global Error Handler

Would be cool to have as part of the data an errorHandler that gets called on any HTTP error returned, globally.
As an example: https://github.com/zerqu/qingcheng/blob/master/lib/xhr.js#L67
Would be super handy...

Nevermind...  Vue.http.options.error is already a thing. Ignore me today. ;)
Thanks for the great work!

Maybe I'm mistaken. I've added Vue.http.options.error and it does trigger, but so does the success callback, even on a 500 error from the server.
https://github.com/vuejs/vue-resource/blob/master/src/http.js#L155
Maybe just technical assistance is needed here... ;)

Once we get a proper response intercepting pipeline, you'll be able to do this.

Closing as dupe of #5


Middle-click paste in terminal on non-Linux

This PR fixes no existing issue - but I can create a feature request if you want me to 😃 .
GUIs on UNIX-like systems using X11 (or Wayland) have two clipboards:

one that uses ctrl-c for copying and ctrl-v for pasting
one that copies selected text to the clipboard, and uses the middle-mouse button for pasting

If you are not on one of those platforms, the second one can be simulated by the application. It can be enabled in iterm2 on Mac OS, for example. The only limitation is that there is only one clipboard that is used for both types of copy/paste.
As vscode already supports half of it with the setting terminal.integrated.copyOnSelection, I thought it might make sense to also support the other half.
The configuration setting could be a boolean, but was made to look consistent to the existing terminal.integrated.rightClickBehavior.

This has come up before and we decided not to go with it because of the extra complexity it brings mainly around settings clutter. The good news is this month we're looking into making mouse input customizable which this would fit with perfectly, let's reconsider after https://github.com/microsoft/vscode/issues/3130, feel free to make an issue to track it.

Reopening for https://github.com/microsoft/vscode/issues/187365 to see how difficult it is to update this branch.


Uncaught SyntaxError: Unexpected token

Describe the issue:
Getting these errors in the browser console, when accessing the MyAccount.
Uncaught SyntaxError: Unexpected token ';' (at myaccount:46:29)
Uncaught SyntaxError: Unexpected token ';' (at myaccount?code=aaf7edf5-7a3f-31cb-b681-684631c5910c&session_state=3b07e47b70bfd87b736a4921b3f38c90bb3658fda44b5035f19df02a3efe2ddf._mTka7LU1YNjBwtGIB5McQ&state=sign-in-silently_request_0&scope=SYSTEM+openid+profile:46:29)

How to reproduce:
Access the MyAccount and observe the browser console.
Environment information

Product Version: IS 7.0 Alpha

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Clent.authenticate() in popup using nw.js

Hi,
I am creating a node-webkit app, and i would like to use this library to allow people to authenticate with Dropbox. Currently I have the auth flow working, but it has the unwanted effect of opening chrome to auth the client, before redirecting back to my app, whilst leaving the auth page open for the user to close at a later date. Ideally, i would have a function that returned the complete URL for authenticating to dropbox, which i could then open in a popup and close programatically once the auth has completed.
Any suggestions/examples?
Thanks

I think the best option would be to write a new auth driver. See https://github.com/dropbox/dropbox-js/blob/stable/src/auth_driver/cordova.coffee for the one that's likely to be the most similar. You'll just need to use different code to open and close the window and hook the navigation event.
If you want to just build the URL yourself, it's documented here: https://www.dropbox.com/developers/core/docs#oa2-authorize. Basically, you want https://www.dropbox.com/1/oauth2/authorize?response_type=token&client_id=<your app key>&redirect_uri=https://www.dropbox.com/1/oauth2/redirect_receiver, and then catch the redirect and extract the access token from the URL fragment.

Ok, will attempt that, could you help me with one other thing, namely accessing the getStateParam() function, I cannot get my client to read it, no matter what I do. Here is a snippet of the code, if it helps ;)
var dropbox = new Dbox.Client({ key: "***" });
dropbox.authDriver(new Dbox.AuthDriver.NodeServer({port: "7640"}));
$(function() {
dropbox.authenticate({interactive: false}, function(error, client) {
if (error) {
//return handleError(error);
}
if (dropbox.isAuthenticated()) {
Thanks.

I'm utterly confused. Why are you using the NodeServer auth driver? Per https://github.com/dropbox/dropbox-js/blob/stable/guides/builtin_drivers.md#dropboxauthdrivernodeserver:

Dropbox.AuthDriver.NodeServer
This driver is designed for use in the automated test suites of node.js applications. It completes the OAuth token authorization step by opening the Dropbox authorization page in a new browser window, and "catches" the OAuth redirection by setting up a small server using the https built-in node.js library.

And why are you trying to call getStateParam()? (Also, I don't see any call to it, and you haven't said what the issue is.)

Sorry about that, just realised I typed the wrong thing into the comment box ;) I was meaning to ask whether there was a hook in dropbox.authenticate that allowed me to override the open in browser feature by specifying a window.open instance. Anyway, not to worry, I have solved it by rewriting the NodeServer module a little so that it is compatible with node-webkit in the sense that it now displays a popup window that closes after successful authentication. What I was wondering is, as the nodeserver is meant for automated test suites, is there any method to get the access token, like client.accesstoken(), or   do I need to build that in as well?
Thanks :)

I'm a bit lost, sorry. I would think you just want something like the Cordova auth driver. I don't really know exactly the NodeServer auth driver does or how appropriate it is for what you're trying to do.

I would use the standard oauth with the receiver callback, but node-webkit doesn't expose URL's outside the application, thus it doesn't work. I will have a look at the auth feature of the cordova driver, and see what i can do ;)

I wrote an auth driver that seems to work for me using nw.js:
gui = global.window.nwDispatcher.requireNwGui()

class Dropbox.AuthDriver.NWAuthDriver extends Dropbox.AuthDriver.BrowserBase
  constructor: (options) ->
    super options

  url: ->
    'https://www.dropbox.com/1/oauth2/redirect_receiver'

  doAuthorize: (authUrl, stateParam, client, callback) ->
    popup = gui.Window.open authUrl, { focus: true }
    removed = false
    popup.on 'loaded', =>
        url = popup.window.location.href
        if @locationStateParam(url) is stateParam
            return if removed
            removed = true
            popup.close()

            callback Dropbox.Util.Oauth.queryParamsFromUrl(url)
            return
    popup.on 'closed', =>
        return if removed
        callback new Dropbox.AuthError('error=access_denied&error_description=User+closed+browser+window')
        return

I put that in a file called NWAuthDriver.coffee, compiled it to NWAuthDriver.js, and then just included it in a <script> tag under dropbox.js itself. Here's my main HTML:
<!doctype html>
<html>
<head>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/dropbox.js/0.10.2/dropbox.min.js"></script>
    <script src="./NWAuthDriver.js"></script>
</head>
<body>
    <script>
        var client = new Dropbox.Client({ key: '<REDACTED>' });
        client.authDriver(new Dropbox.AuthDriver.NWAuthDriver());

        client.authenticate(function (error) {
            console.log('Authenticated.');
            client.getAccountInfo(function (error, info) {
                console.log(JSON.stringify(info));
            });
        });
    </script>
</body>
</html>

This is my first time using nw.js, so apologies if I've done things in a strange way. :-) Let me know if this helps or what you come up with instead.

Thanks for spending the time to put that together, I must confess, I have never used coffee, so it is a bit odd for me also ;) it's 1am here, so will probably leave it for tonight, but will give it a go tomorrow and let you know how it goes :)

Here's the corresponding JavaScript in case it's easier to understand:
// Generated by CoffeeScript 1.8.0
(function() {
  var gui,
    __hasProp = {}.hasOwnProperty,
    __extends = function(child, parent) { for (var key in parent) { if (__hasProp.call(parent, key)) child[key] = parent[key]; } function ctor() { this.constructor = child; } ctor.prototype = parent.prototype; child.prototype = new ctor(); child.__super__ = parent.prototype; return child; };

  gui = global.window.nwDispatcher.requireNwGui();

  Dropbox.AuthDriver.NWAuthDriver = (function(_super) {
    __extends(NWAuthDriver, _super);

    function NWAuthDriver(options) {
      NWAuthDriver.__super__.constructor.call(this, options);
    }

    NWAuthDriver.prototype.url = function() {
      return 'https://www.dropbox.com/1/oauth2/redirect_receiver';
    };

    NWAuthDriver.prototype.doAuthorize = function(authUrl, stateParam, client, callback) {
      var popup, removed;
      popup = gui.Window.open(authUrl, {
        focus: true
      });
      removed = false;
      popup.on('loaded', (function(_this) {
        return function() {
          var url;
          url = popup.window.location.href;
          if (_this.locationStateParam(url) === stateParam) {
            if (removed) {
              return;
            }
            removed = true;
            popup.close();
            callback(Dropbox.Util.Oauth.queryParamsFromUrl(url));
          }
        };
      })(this));
      return popup.on('closed', (function(_this) {
        return function() {
          if (removed) {
            return;
          }
          callback(new Dropbox.AuthError('error=access_denied&error_description=User+closed+browser+window'));
        };
      })(this));
    };

    return NWAuthDriver;

  })(Dropbox.AuthDriver.BrowserBase);

}).call(this);

Actually, forget all that. The built-in Popup auth driver works fine once you whitelist your app scheme:
<!doctype html>
<html>
<head>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/dropbox.js/0.10.2/dropbox.min.js"></script>
</head>
<body>
    <script>
        var client = new Dropbox.Client({ key: 'tavlmnc2mpl6rtx' });
        client.authDriver(new Dropbox.AuthDriver.Popup({ receiverUrl: 'app://nwjs_test/oauth_receiver.html' }));

        // This is the only non-standard bit. It allows the redirect back to app://nwjs_test/oauth_receiver.html to work.
        require('nw.gui').App.addOriginAccessWhitelistEntry('https://www.dropbox.com/1/oauth2/authorize', 'app', 'nwjs_test', true);

        client.authenticate(function (error) {
            console.log('Authenticated.');
            client.getAccountInfo(function (error, info) {
                console.log(JSON.stringify(info));
            });
        });
    </script>
</body>
</html>

I couldn't get the Popup driver working, but the NWAuthDriver you made works a treat ;) There are a few small things that need tweaking, but nothing I shouldn't be able to sort out. Thanks for all your help so far ;)

I noticed that too. It looks like the code I copied from the Cordova plugin may not work at all. We should probably open a new issue for that.
I don't want to spend much time on fixing NWAuthDriver, since that code shouldn't be necessary at all. (Using the built-in Popup driver is much easier!) Maybe you can take another stab at using the Popup driver and let me know what issue you're running into?

will give the popup auth another go tomorrow ;)

Ok, I got the Popup working, you are right it is a much neater solution. Is there any scope for passing options to the window opened by the Popup Driver?

What kind of options? https://github.com/dropbox/dropbox-js/blob/stable/src/auth_driver/browser.coffee#L319 is the code... maybe you can find what you're looking for there or modify it to suit your needs.

For anyone interested, I've created working Electron auth driver, here's the source code. To make this work, in main app process, you should add:
require('app').openWindow = function(opts) {
    var BrowserWindow = require('browser-window');
    return new BrowserWindow(opts);
};


FIxed bug of Closing Search Bar

PR Description 📜
Fixes issue no. #102
Along with fixing the issue, I've added a little hover effect on the search icon.

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for review.

## Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

@Anshu3301 Please resolve conflict & don't forget to clone latest codes

Well I was checking the live website and noticed someone has already resolved the issue #102 ... so I think there's no need to update the code. Kindly close issue no. #102 .
Thank You!


community: update for compatibility with Meilisearch v1.8

Thank you for contributing to LangChain!

[x] PR title: "package: description"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..."  for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

[x] PR message: Delete this entire checklist and replace with

Description: Updates Meilisearch vectorstore for compatibility with v1.8. Adds ”showRankingScore”: true” in the search parameters and replaces _semanticScore field with  _rankingScore

[ ] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[x] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.

out of curiosity will this break compatibility with meilsearch < v1.8? if so should we document this somewhere and add a nice error message?

Hello,
With Meilisearch v1.8, the _semanticScore field is no longer returned in the search response. Its value is identical to _rankingScore, which can be obtained by setting "showRankingScore": true in the search parameters.
In Meilisearch v1.7, adding "showRankingScore": true in the search parameters will return both the _semanticScore and _rankingScore fields in the search response.
In conclusion, these changes should not be breaking for users working with Meilisearch v1.7.


👁🗨 Issue to add labels

closed by https://github.com/ministryofjustice/network-access-control-integration-tests/pull/45


Expose release information through REST API

As an administrator on CodeOcean
I would like to get an overview of the system components
in order to check which version is currently running as part of my application.

We should include a release identifier to expose it through a REST API. We might want to use a Git hash (as this makes tracking very easy) but if this is not feasible, we can also use something else (e.g. build time of the server). We might also investigate #26 as Sentry has some support for release information and is also required for proper error tracking.

Usage of the new API has been added to CodeOcean with https://github.com/openHPI/codeocean/commit/5ef7da14be804a2b046d402642ca93ad92595d20


Paginator not working properly when clicking the "last page" button (http://unosquare.github.io/tubular/#/Basic)

After clicking the "last page" button the data table starts doing several requests to the server. It seems that the data table is trying to get the last page but going one by one.

Hello Mr. PHP, I gonna take a look.

@kadosh can you check again please?