Failed to get HTTP platform singleton instance. Error: 27

env: centos7  python3.9  pip install azure-cognitiveservices-speech
python quickstart.py
Traceback (most recent call last):
File "/home/ecs-user/py_excel_course_extract/text-to-speech/quickstart.py", line 16, in
speech_synthesizer = speechsdk.SpeechSynthesizer(speech_config=speech_config)
File "/usr/local/lib/python3.9/site-packages/azure/cognitiveservices/speech/speech.py", line 2225, in init
_call_hr_fn(fn=_sdk_lib.synthesizer_create_speech_synthesizer_from_config, *[
File "/usr/local/lib/python3.9/site-packages/azure/cognitiveservices/speech/interop.py", line 62, in _call_hr_fn
_raise_if_failed(hr)
File "/usr/local/lib/python3.9/site-packages/azure/cognitiveservices/speech/interop.py", line 55, in _raise_if_failed
__try_get_error(_spx_handle(hr))
File "/usr/local/lib/python3.9/site-packages/azure/cognitiveservices/speech/interop.py", line 50, in __try_get_error
raise RuntimeError(message)
RuntimeError: Exception with error code:
[CALL STACK BEGIN]
/usr/local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x86086) [0x7fb3ee486086]
/usr/local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(synthesizer_create_speech_synthesizer_from_config+0x274) [0x7fb3ee5d3952]
/lib64/libffi.so.8(+0x78d6) [0x7fb3ef3728d6]
/lib64/libffi.so.8(+0x4556) [0x7fb3ef36f556]
/usr/lib64/python3.9/lib-dynload/_ctypes.cpython-39-x86_64-linux-gnu.so(+0x8e5d) [0x7fb3eee83e5d]
/usr/lib64/python3.9/lib-dynload/_ctypes.cpython-39-x86_64-linux-gnu.so(+0x8742) [0x7fb3eee83742]
/lib64/libpython3.9.so.1.0(_PyObject_Call+0x78) [0x7fb3ef125498]
/lib64/libpython3.9.so.1.0(_PyEval_EvalFrameDefault+0x4026) [0x7fb3ef113c66]
/lib64/libpython3.9.so.1.0(+0x10ec85) [0x7fb3ef10ec85]
/lib64/libpython3.9.so.1.0(_PyFunction_Vectorcall+0xe5) [0x7fb3ef11c975]
/lib64/libpython3.9.so.1.0(PyVectorcall_Call+0x95) [0x7fb3ef1255a5]
/lib64/libpython3.9.so.1.0(_PyEval_EvalFrameDefault+0x4026) [0x7fb3ef113c66]
/lib64/libpython3.9.so.1.0(+0x10ec85) [0x7fb3ef10ec85]
/lib64/libpython3.9.so.1.0(_PyFunction_Vectorcall+0xe5) [0x7fb3ef11c975]
/lib64/libpython3.9.so.1.0(_PyObject_FastCallDictTstate+0xda) [0x7fb3ef117f0a]
/lib64/libpython3.9.so.1.0(+0x123987) [0x7fb3ef123987]
/lib64/libpython3.9.so.1.0(+0x1186b3) [0x7fb3ef1186b3]
/lib64/libpython3.9.so.1.0(_PyObject_MakeTpCall+0x14c) [0x7fb3ef1184fc]
[CALL STACK END]
Failed to get HTTP platform singleton instance. Error: 27

I've been getting the same issue.
This is most likely a package upgrade that broke something. Locally, my scripts are working, but when installing in a fresh docker file this starts happening (I know, I should be freezing my packages...)
I'm trying to discover what package is causing this with no success.
What's frustrating is that this happens AFTER the connection start is accepted...

I've upgraded all python packages locally, and still the issue only happens on the docker image.
This causes me to suspect this is related to libSSL 1 being installed in the image (due to #2204).
It also seems that libSSL 3 support was FINALLY (!!!!) added to the latest release, and there are updated platform requirements:
(https://learn.microsoft.com/en-us/azure/ai-services/speech-service/releasenotes?tabs=speech-sdk#speech-sdk-1380-2024-june-release)
This seems like a very likely culprit
I'll try to rebuild the image without libSSL 1 and report

Well, I've tried removing the installation for SSL 1.1.1, which was previously required due to #2204, and installed all platform requirements as stated here).
 Error: 27
INFO:     <IP_ADDRESS>:52956 - "POST /test-tts HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/opt/conda/lib/python3.10/site-packages/uvicorn/protocols/http/httptools_impl.py", line 399, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
  File "/opt/conda/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 70, in __call__
    return await self.app(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/fastapi/applications.py", line 1054, in __call__
    await super().__call__(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/applications.py", line 123, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/middleware/errors.py", line 186, in __call__
    raise exc
  File "/opt/conda/lib/python3.10/site-packages/starlette/middleware/errors.py", line 164, in __call__
    await self.app(scope, receive, _send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/middleware/cors.py", line 93, in __call__
    await self.simple_response(scope, receive, send, request_headers=headers)
  File "/opt/conda/lib/python3.10/site-packages/starlette/middleware/cors.py", line 148, in simple_response
    await self.app(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 65, in __call__
    await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
    raise exc
  File "/opt/conda/lib/python3.10/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
    await app(scope, receive, sender)
  File "/opt/conda/lib/python3.10/site-packages/starlette/routing.py", line 756, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/routing.py", line 776, in app
    await route.handle(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/routing.py", line 297, in handle
    await self.app(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/routing.py", line 77, in app
    await wrap_app_handling_exceptions(app, request)(scope, receive, send)
  File "/opt/conda/lib/python3.10/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
    raise exc
  File "/opt/conda/lib/python3.10/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
    await app(scope, receive, sender)
  File "/opt/conda/lib/python3.10/site-packages/starlette/routing.py", line 72, in app
    response = await func(request)
  File "/opt/conda/lib/python3.10/site-packages/fastapi/routing.py", line 278, in app
    raw_response = await run_endpoint_function(
  File "/opt/conda/lib/python3.10/site-packages/fastapi/routing.py", line 193, in run_endpoint_function
    return await run_in_threadpool(dependant.call, **values)
  File "/opt/conda/lib/python3.10/site-packages/starlette/concurrency.py", line 42, in run_in_threadpool
    return await anyio.to_thread.run_sync(func, *args)
  File "/opt/conda/lib/python3.10/site-packages/anyio/to_thread.py", line 56, in run_sync
    return await get_async_backend().run_sync_in_worker_thread(
  File "/opt/conda/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 2177, in run_sync_in_worker_thread
    return await future
  File "/opt/conda/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 859, in run
    result = context.run(func, *args)
  File "/hiredge/apps/streaming-server/streaming_server/app.py", line 78, in test_tts
    wav_data = _get_tts_result(tts_request.language, tts_request.text)
  File "/hiredge/apps/streaming-server/streaming_server/app.py", line 68, in _get_tts_result
    tts_generator = AzureTTSGenerator(language, "int-id")
  File "/hiredge/apps/streaming-server/streaming_server/tts/azure_tts_generation.py", line 34, in __init__
    self.speech_synthesizer = speechsdk.SpeechSynthesizer(speech_config=self.speech_config, audio_config=None)
  File "/opt/conda/lib/python3.10/site-packages/azure/cognitiveservices/speech/speech.py", line 2225, in __init__
    _call_hr_fn(fn=_sdk_lib.synthesizer_create_speech_synthesizer_from_config, *[
  File "/opt/conda/lib/python3.10/site-packages/azure/cognitiveservices/speech/interop.py", line 62, in _call_hr_fn
    _raise_if_failed(hr)
  File "/opt/conda/lib/python3.10/site-packages/azure/cognitiveservices/speech/interop.py", line 55, in _raise_if_failed
    __try_get_error(_spx_handle(hr))
  File "/opt/conda/lib/python3.10/site-packages/azure/cognitiveservices/speech/interop.py", line 50, in __try_get_error
    raise RuntimeError(message)
RuntimeError: Exception with error code:
[CALL STACK BEGIN]

/opt/conda/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x86086) [0x7fe0a9350086]
/opt/conda/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(synthesizer_create_speech_synthesizer_from_config+0x274) [0x7fe0a949d952]
/opt/conda/lib/python3.10/lib-dynload/../../libffi.so.8(+0x6a4a) [0x7fe0c6868a4a]
/opt/conda/lib/python3.10/lib-dynload/../../libffi.so.8(+0x5fea) [0x7fe0c6867fea]
/opt/conda/lib/python3.10/lib-dynload/_ctypes.cpython-310-x86_64-linux-gnu.so(+0x12461) [0x7fe0c6880461]
/opt/conda/lib/python3.10/lib-dynload/_ctypes.cpython-310-x86_64-linux-gnu.so(+0x86eb) [0x7fe0c68766eb]
/opt/conda/bin/python(PyObject_Call+0x207) [0x565216f47067]
/opt/conda/bin/python(_PyEval_EvalFrameDefault+0x2d83) [0x565216f2d2b3]
/opt/conda/bin/python(_PyFunction_Vectorcall+0x6c) [0x565216f3aa2c]
/opt/conda/bin/python(PyObject_Call+0xbc) [0x565216f46f1c]
/opt/conda/bin/python(_PyEval_EvalFrameDefault+0x2d83) [0x565216f2d2b3]
/opt/conda/bin/python(_PyFunction_Vectorcall+0x6c) [0x565216f3aa2c]
/opt/conda/bin/python(_PyObject_FastCallDictTstate+0x187) [0x565216f33007]
/opt/conda/bin/python(+0x14e0c1) [0x565216f440c1]
/opt/conda/bin/python(_PyObject_MakeTpCall+0x27b) [0x565216f33a7b]
/opt/conda/bin/python(_PyEval_EvalFrameDefault+0x5723) [0x565216f2fc53]
/opt/conda/bin/python(_PyObject_FastCallDictTstate+0xd0) [0x565216f32f50]
/opt/conda/bin/python(+0x14e0c1) [0x565216f440c1]
[CALL STACK END]

Failed to get HTTP platform singleton instance. Error: 27

This is very frustrating. Not only is this a regression, the error message is completely useless, and only triggers at the last possible moment, when an actual call is made, instead of at initialization time (or even package installation time!). The worst type of error.
For now, the only solution that worked for me was pinning the version to 1.37:
azure-cognitiveservices-speech==1.37.0

I hope the maintainers respond, instead of letting this issue auto-close as is way too common in this repo...

replace azure-cognitiveservices-speech==1.37.0
python quickstart.py
Traceback (most recent call last):
File "/home/ecs-user/py_excel_course_extract/text-to-speech/quickstart.py", line 16, in
speech_synthesizer = speechsdk.SpeechSynthesizer(speech_config=speech_config)
File "/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/speech.py", line 2223, in init
_call_hr_fn(fn=_sdk_lib.synthesizer_create_speech_synthesizer_from_config, *[
File "/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/interop.py", line 62, in _call_hr_fn
_raise_if_failed(hr)
File "/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/interop.py", line 55, in _raise_if_failed
__try_get_error(_spx_handle(hr))
File "/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/interop.py", line 50, in __try_get_error
raise RuntimeError(message)
RuntimeError: Exception with error code:
[CALL STACK BEGIN]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1e3f11) [0x7f90bbbe3f11]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x20d2e7) [0x7f90bbc0d2e7]
/lib64/libc.so.6(+0x8ee18) [0x7f90bc48ee18]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x20e41c) [0x7f90bbc0e41c]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1b445f) [0x7f90bbbb445f]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1b3d15) [0x7f90bbbb3d15]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1eaca5) [0x7f90bbbeaca5]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1dbdc0) [0x7f90bbbdbdc0]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1d5d8e) [0x7f90bbbd5d8e]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0xdc8bd) [0x7f90bbadc8bd]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1eaca5) [0x7f90bbbeaca5]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1ccb4f) [0x7f90bbbccb4f]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x21b9a6) [0x7f90bbc1b9a6]
/home/ecs-user/.local/lib/python3.9/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(synthesizer_create_speech_synthesizer_from_config+0xf3) [0x7f90bbabf169]
/lib64/libffi.so.8(+0x78d6) [0x7f90bc2998d6]
/lib64/libffi.so.8(+0x4556) [0x7f90bc296556]
/usr/lib64/python3.9/lib-dynload/_ctypes.cpython-39-x86_64-linux-gnu.so(+0x8e5d) [0x7f90bc2e8e5d]
[CALL STACK END]
Runtime error: Failed to initialize platform (azure-c-shared). Error: 2176

This is a separate issue, with a different solution - see #2048

Same issue with the C# SDK and the new version of the package.
The previous version 1.37 works with openssl 1.1 but NOT with 3 (as expected, error thrown: platform dependency not met)
The new version 1.38 fails with openssl 1.1 (platform dependency not met) and ALSO fails with openssl openssl 3 (with the error of this thread).
Arch Linux with all most-recent packages.
Right now the only option is to go back to 1.37 and continue to install openssl 1.1.

replace azure-cognitiveservices-speech==1.37.0
/home/ubuntu/.conda/envs/py310/bin/python /home/ubuntu/project/news_reporter/tools/azure_tts.py
Traceback (most recent call last):
File "/home/ubuntu/project/news_reporter/tools/azure_tts.py", line 92, in
speech_synthesizer = speechsdk.SpeechSynthesizer(speech_config=speech_config, audio_config=audio_config)
File "/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/speech.py", line 2223, in init
_call_hr_fn(fn=_sdk_lib.synthesizer_create_speech_synthesizer_from_config, *[
File "/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/interop.py", line 62, in _call_hr_fn
_raise_if_failed(hr)
File "/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/interop.py", line 55, in _raise_if_failed
__try_get_error(_spx_handle(hr))
File "/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/interop.py", line 50, in __try_get_error
raise RuntimeError(message)
RuntimeError: Exception with error code:
[CALL STACK BEGIN]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1ec00c) [0xeaa6037dc00c]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x20c5e4) [0xeaa6037fc5e4]
/lib/aarch64-linux-gnu/libc.so.6(+0x825d4) [0xeaa6042325d4]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x20d564) [0xeaa6037fd564]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1bc394) [0xeaa6037ac394]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1b9ec0) [0xeaa6037a9ec0]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1f2d0c) [0xeaa6037e2d0c]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x2066d0) [0xeaa6037f66d0]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1dbc88) [0xeaa6037cbc88]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1094dc) [0xeaa6036f94dc]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1f2d0c) [0xeaa6037e2d0c]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1b9a0c) [0xeaa6037a9a0c]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x225290) [0xeaa603815290]
/home/ubuntu/.conda/envs/py310/lib/python3.10/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(synthesizer_create_speech_synthesizer_from_config+0x10c) [0xeaa6036b21a4]
/home/ubuntu/.conda/envs/py310/lib/python3.10/lib-dynload/../../libffi.so.8(+0xc050) [0xeaa603dcc050]
/home/ubuntu/.conda/envs/py310/lib/python3.10/lib-dynload/../../libffi.so.8(+0x9580) [0xeaa603dc9580]
/home/ubuntu/.conda/envs/py310/lib/python3.10/lib-dynload/_ctypes.cpython-310-aarch64-linux-gnu.so(+0x144f8) [0xeaa603e044f8]
[CALL STACK END]
Runtime error: Failed to initialize platform (azure-c-shared). Error: 2176

I've been struggling with this as well. I ended up switching to using the REST API which worked perfectly for my use case. Actually Claude Sonnet 3.5 switched the implementation without any errors on the first attempt when I fed in this autogenerated porting guide along with my azure-cognitiveservices-speech based implementation: https://www.perplexity.ai/page/azure-text-to-speech-rest-api-mw5a5jOfS8ySs5UjlkWssA

Same here on Fedora 39. With azure speech sdk 1.38 it fails. Downgrading to 1.37 fixes the issue (because I also have openssl 1.1 installed). I suppose it is again the OpenSSL version as described here:
https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2048#issuecomment-2161849523
Unfortunately OpenSSL 3.x support is initially limited to 3.0.x which is used in Debian 12 and Ubuntu 22.04 LTS

So the issue is probably the fact that CentOS Stream and Fedora use OpenSSL 3.1 or 3.2 :man_shrugging:

The Speech SDK 1.40.0 release is now available and fixes the detection of OpenSSL 3 not working on Linux arm64.
It also removes the internal limit of support for OpenSSL 3.0.x versions only. However, Speech SDK releases are only verified using specific "officially supported" Linux distributions (= Debian and Ubuntu) where the highest OpenSSL version is currently 3.0.x (which is an LTS branch with the longest support at the moment).
Note that since the 1.38.0 release, all Speech SDK packages include the following 2 new library files that are required for the OpenSSL version detection to work:
libpal_azure_c_shared.so
libpal_azure_c_shared_openssl3.so

If your project is set up so that SDK libraries from the release packages are copied to a custom location, make sure that these libpal libraries are copied as well, otherwise you will get errors like in the title. All Speech SDK libraries must reside in the same directory.
So please

check that your Linux environment meets the Speech SDK platform requirements
re-test with the Speech SDK 1.40.0 release
and if you still have issues then open a new bug report with full details.


Is it possible to generate assetMap.json in development?

I'd like to know use a hash of all known assets within a helper to decide if an asset is in my project's public folder, or if it's on S3. Is it possible to get the assetMap generated in development?

Yes, set generateAssetMap to true


More friendly aliases?

🚀 Feature request
I think fp-ts is an awesome library but the name of the functions is not always:

the most friendly for newcomers to fp
in line with the JS ecosystem

Current Behavior
either.chain

Desired Behavior
// More in line with ecosystem:
either.flatMap

// More friendly to newcomers (e.g. see rust API's)
either.andThen
// The above is also more in line with
either.orElse

Suggested Solution
Add a bunch of aliases to make it more beginner friendly and potentially deprecate the old ones (and remove in v3) or ship both
Who does this impact? Who is this for?
This will help with beginners and spreading adoption of fp-ts. In my team, we love fp-ts but we fear it can make the codebase harder to understand, especially because the names are very academic / haskelly and are not self-evident in what they do.
Describe alternatives you've considered

Create own package with aliases
Write better documentation

If you are open to it, I'm happy to help in picking better names. I just gave one example but I think there are others.

chain is very fundamental to the library and I think any benefit in renaming it would be outweighed by the cost of deprecation to everyone who's already written code against that function name.
Function aliases would in a sense fragment the ecosystem and I'm not sure it makes things any less confusing; the inevitable question that follows from a beginner is "what's the difference between these?".
Additionally, the adherence in some cases (though not with chain) to Haskell idioms can be a win for anyone coming from a functional language or anyone going to or otherwise learning a functional language.
Sorry to be so negative - how to make the library beginner-friendly is certainly something that deserves attention, and I've never liked this naming choice either. Just my two cents on this specifically.

I think those are very fair points!
I'm open to other alternatives as well, where e.g. in the documentation we add "common alternatives" so if you hover / if you check out the documentation, you also see other names you might be more used to to help understand what a function does. This would also help with transitioning to other languages.
I get your point about more fp languages but on the other hand, every language / library seems to use different names, so I'm not sure if that's 100% fair. I think one of the reasons rust doesn't seem to have a problem with map and andThen for their result type, is because it's just understandable as to what the function does. While with chain and bind, it's not as clear (IMO)

Some way to find chain by searching for "flatMap" for example in the docs would be handy, likewise as you've said a "this is also sometimes known by" line. :+1:

What could be really cool is on top of that to have typescript definitions for flatMap, andThen, ... and just make them obsolete / deprecated and not actually implement them. And then just in their docs mention "use chain instead" or something. That way, you can even in your editor type what you are used to and get reminded what the actual name should be.

Btw. is there any list of these aliases and what they do?
For example whats the difference between T.fromIO and T.fromIOK etc.
There are many similar examples and for me as a beginner it is really hard to decide which one to use. I see examples and the code works but I don't understand why

https://github.com/gcanti/fp-ts/pull/1849


How-can-i-sign-a-string-using-rsa-and-sha1-with-AL- BC Cloud

I am working with a task where we need to sign data by using private key received from Azure Vault.
as per these specifications defined

we are running in this issue

Here we are able to do it by following this blog:
https://www.c-sharpcorner.com/article/how-to-access-azure-key-vault-secrets-through-rest-api-using-postman/
after having this key in system we are trying to use Codeunit cryptography or RSA function signdata.
The Private key which we are receiving is in Base64 format.
CryptoGraphy.txt
https://community.dynamics.com/business/f/dynamics-365-business-central-forum/380150/using-cryptography-management
We have been through this also as installing a utility for Cloud does not seems valid option.
Attaching file with Code:Tried with both Cryptography Codeunit and RSA Codeunit
trigger OnRun()
var
SIGNATURERECORD: Record "Signature Key";
CryptoMgmt: Codeunit "Cryptography Management";
rsa: Codeunit RSACryptoServiceProvider;
Algo: Enum "Hash Algorithm";
stream: InStream;
outstream: OutStream;
OUTSTREAM1: OutStream;
TempBLOB: Codeunit "Temp Blob";
SecretKey: Text;
hASHVALUE: Text;
begin
SecretKey := 'MIICXgIBAAKBgQDIYh3tSlmpU7rxPepe+c20abLXiYpI1NfxlKh9OThouEble9ir3ymDq4H/OuKsw6yZ21QFYvuE1U7vrwqECb1lmOY4EqdcsmxgKvoZVOcHgB3Bj4DiSl6yGbOE6wwoJbUXlO39eKBRYGBGdg1POEr2ivDPph0XvSk+y+7sZ3ZF1QIDAQABAoGAYtkL3EdmDYJxYJnjaLCfydSuVWbS4XSlhdWIfCVi7z2gM4oxay9t4jcTrJ0WvpTUKx2nx/R5EKDNDK1zaYkIdRs+ASDSI4UuJ9XDLjB3ei9HnNP83ZzSoZ3LYHGNvuqM8ZSk9dJmb7+wuJocScOOnh8LI2GEjYd48SrPxQpXiEECQQDoO5304nUH1ZAuGWel5AER0F9Zg2XEA8carN3AQ4fjX+dKCSgWlYxjMtwjXpN0FL0ohTbvmvJgO3E9XLgdvQYFAkEA3OQObQYw5RokYHg7gmq+20+nyWm2gdbxfG7FyfIfpKCWoLIGMLcTprXr6Xgap/aTvPuuBknmXM5wtqPAxU/5kQJBAJDA6FpF9ZCmi7JQX4zFq+0Apw5SmtItUKPN8ILxFPijRLWfM3+wQ8tpXlIll1+ZtbZWIsIDIOTe6FzJOHJA6kkCQQDOpozYcnJL46kkl1cVK8hiZKMUS1HzJqPFRdzEkUAJRDOnIuWnrZ801/1c7oABoy2UH+EvAPQC3x+Tvp8Z+TgBAkEAySwRR2zFCcQFUYje26G60tyzcd1yKDBau3OOyxC/h1emBWoJtz+VC2nuA3gxBgwSq3sH2jJ2+WEHtolGLz3sCQ==';
// SecretKey := '' + SecretKey + '';//'AQAB';
SIGNATURERECORD.DeleteAll();
SIGNATURERECORD.Init();
SIGNATURERECORD."Key Index" := 1;
SIGNATURERECORD.FromXmlString(SecretKey);
SIGNATURERECORD.Insert();
CryptoMgmt.SignData('FAC11', SIGNATURERECORD, Algo::SHA1, outstream);
SIGNATURERECORD.Delete();
outstream.WriteText(hASHVALUE);
    //RSA Codeunit
    SecretKey := 'MIICXgIBAAKBgQDIYh3tSlmpU7rxPepe+c20abLXiYpI1NfxlKh9OThouEble9ir3ymDq4H/OuKsw6yZ21QFYvuE1U7vrwqECb1lmOY4EqdcsmxgKvoZVOcHgB3Bj4DiSl6yGbOE6wwoJbUXlO39eKBRYGBGdg1POEr2ivDPph0XvSk+y+7sZ3ZF1QIDAQABAoGAYtkL3EdmDYJxYJnjaLCfydSuVWbS4XSlhdWIfCVi7z2gM4oxay9t4jcTrJ0WvpTUKx2nx/R5EKDNDK1zaYkIdRs+ASDSI4UuJ9XDLjB3ei9HnNP83ZzSoZ3LYHGNvuqM8ZSk9dJmb7+wuJocScOOnh8LI2GEjYd48SrPxQpXiEECQQDoO5304nUH1ZAuGWel5AER0F9Zg2XEA8carN3AQ4fjX+dKCSgWlYxjMtwjXpN0FL0ohTbvmvJgO3E9XLgdvQYFAkEA3OQObQYw5RokYHg7gmq+20+nyWm2gdbxfG7FyfIfpKCWoLIGMLcTprXr6Xgap/aTvPuuBknmXM5wtqPAxU/5kQJBAJDA6FpF9ZCmi7JQX4zFq+0Apw5SmtItUKPN8ILxFPijRLWfM3+wQ8tpXlIll1+ZtbZWIsIDIOTe6FzJOHJA6kkCQQDOpozYcnJL46kkl1cVK8hiZKMUS1HzJqPFRdzEkUAJRDOnIuWnrZ801/1c7oABoy2UH+EvAPQC3x+Tvp8Z+TgBAkEAySwRR2zFCcQFUYje26G60tyzcd1yKDBau3OOyxC/h1emBWoJtz+VC2nuA3gxBgwSq3sH2jJ2+WEHtolGLz3sCQ==';

    Clear(outstream);
    TempBLOB.CreateOutStream(OUTSTREAM1, TextEncoding::UTF8);
    OUTSTREAM1.WriteText('FAC11');
    TempBLOB.CreateInStream(stream);
    rsa.SignData(SecretKey, stream, Algo::SHA1, outstream);
    outstream.WriteText(hASHVALUE);
    Message(hASHVALUE);
end;

This repository no longer handles general questions and is only dedicated to bug reports for the latest preview version of the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


Can I edit the scenes?

Thank you for your great work!
Can I edit the scenes ,like moving the cars' position or rotation and so on.
I'd like to hear your advice!
Thank you so much!

oh i see the #4 and i will close this


HBASE-24887 Remove Row.compareTo

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 26s
root in the patch failed.

-1 :x:
compile
0m 25s
hbase-client in the patch failed.

-0 :warning:
javac
0m 25s
hbase-client in the patch failed.

-1 :x:
shadedjars
2m 56s
patch has 22 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 24s
hbase-client in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 25s
hbase-client in the patch failed.

21m 22s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2262

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 37880c39a04f 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c81ef7368e

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-client.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-client.txt

shadedjars
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/patch-shadedjars.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-client.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/testReport/

Max. process+thread count
79 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 25s
master passed

+1 :green_heart:
checkstyle
0m 27s
master passed

+1 :green_heart:
spotbugs
0m 58s
master passed

_ Patch Compile Tests _

-1 :x:
mvninstall
1m  1s
root in the patch failed.

-0 :warning:
checkstyle
0m 25s
hbase-client: The patch generated 1 new + 33 unchanged - 1 fixed = 34 total (was 34)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
hadoopcheck
1m 13s
The patch causes 16 errors with Hadoop v3.1.2.

-1 :x:
hadoopcheck
2m 27s
The patch causes 16 errors with Hadoop v3.2.1.

-1 :x:
spotbugs
0m 20s
hbase-client in the patch failed.

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 25s
root in the patch failed.

-1 :x:
compile
0m 25s
hbase-client in the patch failed.

-0 :warning:
javac
0m 25s
hbase-client in the patch failed.

-1 :x:
shadedjars
3m  1s
patch has 16 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 26s
hbase-client in the patch failed.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 28s
the patch passed

+1 :green_heart:
compile
0m 28s
the patch passed

+1 :green_heart:
javac
0m 28s
the patch passed

+1 :green_heart:
shadedjars
6m 25s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 24s
hbase-client in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 17s
hbase-client in the patch passed.

26m 56s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2262

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux f85d87566688 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c81ef7368e

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/3/testReport/

Max. process+thread count
197 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/3/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 29s
the patch passed

+1 :green_heart:
compile
0m 28s
the patch passed

+1 :green_heart:
javac
0m 28s
the patch passed

+1 :green_heart:
shadedjars
6m 20s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 24s
hbase-client in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 22s
hbase-client in the patch passed.

27m 15s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2262

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux ca58b4e14e96 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c81ef7368e

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/4/testReport/

Max. process+thread count
190 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/4/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  7s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

+1 :green_heart:
shadedjars
5m 45s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 26s
hbase-client in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m  8s
hbase-client in the patch passed.

26m 34s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2262

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux b2bb35a0aeb0 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c81ef7368e

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/5/testReport/

Max. process+thread count
278 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/5/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 38s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 29s
the patch passed

+1 :green_heart:
compile
0m 28s
the patch passed

+1 :green_heart:
javac
0m 28s
the patch passed

+1 :green_heart:
shadedjars
6m 19s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 25s
hbase-client in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
1m 19s
hbase-client in the patch passed.

26m 52s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2262

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d97fff8f8d44 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c81ef7368e

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/testReport/

Max. process+thread count
194 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 52s
master passed

+1 :green_heart:
checkstyle
0m 30s
master passed

+1 :green_heart:
spotbugs
1m  6s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 28s
hbase-client: The patch generated 0 new + 35 unchanged - 1 fixed = 35 total (was 36)

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
11m  7s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m  7s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

29m 36s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2262

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 431075ebaf14 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c81ef7368e

Max. process+thread count
94 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2262/6/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Please fix the checkstyle issue?

Done.

Thanks @Joseph295 . Will merge this after some time if there are no more review comments.


Some questions on DAB-DETR

Hi there,
Thanks for the great repo and the nice compilation of so many power DETR methods. But when I look into the code, I have some problems understanding some of them. Hope to get some advice here.
One is
https://github.com/IDEA-Research/detrex/blob/75318d139157ea2dbf0b3100a3dc3623d9740878/projects/dab_detr/modeling/dab_transformer.py#L208-L220
where the first layer is indicated by is_first_layer. Only in the first decoder transformer layer, the cross_attn's query_content would add query_pos. I wonder why it is the case? I think query_pos is already added to query in self_attn before cross_attn in the first layer? What if I remove the addition of query_pos in the first cross_attn?
Another one is
https://github.com/IDEA-Research/detrex/blob/75318d139157ea2dbf0b3100a3dc3623d9740878/projects/dab_detr/modeling/dab_transformer.py#L223-L231
where reference_boxes for the next layer is a detached version of new_reference_boxes. What's the idea here of detach()? Does removing it affect the performance a lot?
The third one is about initialization:
https://github.com/IDEA-Research/detrex/blob/75318d139157ea2dbf0b3100a3dc3623d9740878/projects/dab_detr/modeling/dab_detr.py#L129-L130
What's the idea of initializing the last fc layer in MLP as all 0?
Sorry for so many questions proposed at a time. It's totally ok if there's no 'standard' answer. Just any advice would be very much appreciated if it can boost the understanding of this great model.
Thanks in advance!

Thanks for your questions.

It is borrowed from Conditional DETR. As the 1st layer's query_content are all-zero vectors, we add query_pos to the query_content to distinguish different queries.
It is borrowed from Deformable DETR. The detach operations help to stable the training.
it is borrowed from Deformable DETR. The 0 init results in a better result.

Got them. Thanks!


Thank you so much.

Lem makes it possible to develop software using Common Lisp once you install it on your computer. You don't need any tidy setting on Emacs. You don't need to install any other plugin. Lem is the best editor for developers using Common Lisp.

You really made my days warm.
Thank you so much for this project and your faculty.
Hope you always healthy and successful
Happy hacking.
PS: sorry for writing here, I do not find your email address nor your blog, I hope I can write you an email soon.

@cxxxr
I deeply appreciate your project. Think of me as a resource. The code I submit is always a suggestion. If your ideas are different, please let me know.
私はあなたのプロジェクトに深く感謝します。 私をリソースと考えてください。 私が提出するコードは、常に提案です。 あなたのアイデアが異なる場合は、私に知らせてください。

@stacksmith  Japanese translation of "Think of me as a resource" sounds like you are kind a material we(he?) can consume. a bit scary...
Anyway, Think of me as an interface for you to him. :)
What I like about lem is less ambitious than climacs,it provide us a joy of hacking  and it takes trivial patch!
I hope lem become richer to be a choice for people who stop using emacs.
Any contributions are literally welcomed.

@snmsts - got it.
Using the same Lisp to program the editor makes a lot more sense.

This isn't issue to do something. so close.


log_batch_ranking_metrics error

Description
You can repro using this notebook:
https://github.com/whylabs/whylogs/blob/mainline/python/examples/experimental/Writing_Ranking_Performance_Metrics_to_WhyLabs.ipynb
Hits an error in versions of whylogs after 1.3.21:
target_order = np.array(row_dict[target_column]).argsort()[::-1]
     99         dcg_vals = [
--> 100             (rel / math.log(i + 2, 2)) for i, rel in enumerate(np.array(row_dict[target_column])[predicted_order][:k])
    101         ]
    102         idcg_vals = [

IndexError: index 1 is out of bounds for axis 0 with size 1

whylogs 1.3.21 does not hit this.

[x] I have reviewed the Guidelines for Contributing and the Code of Conduct.

This was fixed since whylogs 1.3.26


Print PDF issue in Internet Explorer

I was unable to print full PDF page in Internet Explorer 11, Right half of the page was broken and i was getting blank page after every page. I have fixed that issue now we can print full page without any break in the right side of the page and no more blank pages.

thanks !

Your Welcome.


seems that python3.10 not include all sb3_contrib yet

from sb3_contrib import RecurrentPPO
ImportError: cannot import name 'RecurrentPPO' from 'sb3_contrib' (/usr/local/lib/python3.10/dist-packages/sb3_contrib/init.py)
then
cat /usr/local/lib/python3.10/dist-packages/sb3_contrib/init.py
import os
from sb3_contrib.qrdqn import QRDQN
from sb3_contrib.tqc import TQC
Read version from file
version_file = os.path.join(os.path.dirname(file), "version.txt")
with open(version_file, "r") as file_handler:
version = file_handler.read().strip()
---so only QRDQN and TQC now ?

i mean when simply pip install sb3-contrib in google colab,  then we'll got a packages that only has QRDQN and TQC now , but use pip install git+https://github.com/Stable-Baselines-Team/stable-baselines3-contrib, then all policies and algorithm are ok.

i mean when simply pip install sb3-contrib in google colab, then we'll got a packages that only has QRDQN and TQC now , but use pip install git+https://github.com/Stable-Baselines-Team/stable-baselines3-contrib, then all policies and algorithm are ok.

For SB3 contrib 1.x, doing:
!pip install setuptools==65.5.0 pip==21  # gym 0.21 installation is broken with more recent versions
!pip install sb3_contrib

installs sb3 contrib 1.8.0 that has all algorithms describe in the doc.
If you want gymnasium support (or don't want to downgrade setuptools/pip), you can either install sb3 contrib 2.x or use the master version as you wrote:
!pip install "sb3_contrib>=2.0.0a9"


Hello!About the track speed evaluation

Hello!During the test, I set time stamps before and after tracking a single frame by referring to the time module, but it is not as fast as described in the paper. Isn't the evaluation of tracking time calculated in this way?

Run 100 times and take the average.
https://github.com/LitingLin/SwinTrack/blob/4b88bcff6f09d31bc0929a7b3d8dbad4ef8a6e44/core/run/profiler/efficiency_assessor.py#L85


tab complete on UNC path strips first backslash

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:02:52.799Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

PowerShell
ms-vscode
0.10.1

Steps to Reproduce:

start a UNCH path - when the file name appears click on it
the first \ will get stripped off the path.
example output: \<IP_ADDRESS>\Data\vt2_Run-Build-Package.ps1

It's very likely an issue with Powershell extension, please file issues there.


Trouble building @kobalte/core in forked repo

Describe the bug
I recently forked Kobalte (https://github.com/jsimonrichard/kobalte), but I can't build the core package on machine (or any other package depending on @kobalte/tests) without changing the dependency to "file:../tests." This fixes it on my machine, but breaks the CI build.
It appears that @kobalte/tests is a private package on NPM. Would you consider making that package public so that people outside of your organization can build @kobalte/core?
Or is this a new regression (I noticed there are already quite a few forks, nobody has raised this issue yet as far as I can tell)?

@kobalte/tests is a local monorepo package (not on npm), make sure you're using pnpm@8 as a package manager.

It's working now, thank you. I wasn't using pnpm


[API] Container environment field doesnt properly support valueless entries

The docker api/cli allows for env vars like this: -e FOO (as opposed to -e FOO="").

The rancher api does not allow for this functionality. The closest we can get is a variable with a blank string as a value.
Its worth noting that a valueless environment variable does not show up inside a container when you run the printenv command.

The valueless -e FOO takes the current value of $FOO from the host you're running docker run on.  So what would that mean to us... take the value of a variable on the host we happen to have scheduled the container on to?

@cjellick @vincent99 is this fixed and can be closed? do you still need something answered @vincent99?

I believe it's still relevant, we can talk about what valueless keys should do.


Alerts redesign follow-up fix.

This fixes the rendering when using the ui_report.message feature.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

This is failing casper tests.

Yep, it's been fixed.

@brockwhittaker did you forget to push your fix?  Travis is failing the Casper tests.

Travis has (now) passed.

@brockwhittaker can you rebase?

Merged, thanks @brockwhittaker!


Add shell unit testing harness

We have a bunch of shell scripts in the deis project, and no unit (or integration) tests for them. There are many shell unit testing frameworks and even ones that have mock/stub functionality. Let's pull one of them in and provide a model for shell script tests.
I know it may sound weird to test your shell scripts with more shell scripts, but take for example #4023. Wouldn't it have been cool to have a way to test all those conditions during the development of that script? Additionally, it would help someone in the future verify that future changes have not broken existing behavior.

:+1: for more tests

Also see parallel discussion happening in https://github.com/deis/rigger/issues/14.

Good stuff here, but this isn't going to be a development priority as Deis v1 LTS is focused on stability and Deis v2 has minimized the use of shell scripts.


fixed formatting  a file:// URI

add the two forward slashes even when the authority part of the URI is empty
do this only for "file" scheme, to not break other schemes like mailto: or tel:
see RFC 3986 for details
fixes #8665

Thank you so much for the pull request @vividos! I noticed this is your first pull request and I wanted to say welcome to the Cesium community!
The Pull Request Guidelines is a handy reference for making sure your PR gets accepted quickly, so make sure to skim that.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Changes to third party files were made.

Looks like a file in one of our ThirdParty folders (ThirdParty/, Source/ThirdParty/) has been added or modified. Please verify that it has a section in LICENSE.md and that its license information is up to date with this new version.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

I already signed the CLA, hope the PR updates itself...

I rebased the changes on current master branch

@OmarShehata can you take a look at this? Thanks!

Rebased on latest master

So I wasn't actually able to verify that this fixes the Cesium ion credit image when building with Cordova and running on my Pixel 3.
I set my Cesium base url to window.CESIUM_BASE_URL = 'file:///android_asset/www/CesiumUnminified/';. I confirmed the new code is running by verifying that window.alert(new Cesium.Uri('file:///android_asset/').toString()) produced file:///. Regardless of whether the fix was applied I saw:

The Cesium ion credit at the bottom always had a blank src
The imagery base layer picker at the top right always had a src of file:///android_asset/www/CesiumUnminified/Widgets/Images/ImageryProviders/bingAerial.png.

@vividos did you have a different setup than what I described here?
I'm not sure why the Cesium ion credit has a blank src, but this fix to Uri.toString seems to produce the correct behavior from my reading of RFC 3986 - that authority is optional, but if we always appended // regardless of whether authority was present or not, that would produce incorrect behavior for urls with a  mailto scheme.
This is also consistent with the behavior in URI.js. Here's how they do it:
https://github.com/medialize/URI.js/blob/73c10412ce97b760d1cb143080bee25e99d12f5b/src/URI.js#L670-L700

Omar, I didn't have a chance to test the fix in my own app. I just tried to come up with a fix for my Sandcastle example in the original issue #8665. Is there a minified version of Cesium.js from this PR? Then I could drop this into my app and also test. My guess is that it would behave like your Cordova test, though.
About other URIs: I thought about this and also read RFC 3986. With this PR, the mailto: and other URIs should be unaffected by the fix, and only "file" URIs are adjusted (notice the extra "if" in the else part).
About fixing the image: In my app it seems this is the only instance where local resources couldn't be resolved. I have no problems referencing other images (as you also suggested) and referencing the Service Workers, which I guess also use the CESIUM_BASE_URL. From reading https://github.com/CesiumGS/cesium/blob/cbc09093d5f27b701925ec06766d8b2eceef6f73/Source/Scene/CreditDisplay.js#L490 I couldn't think of a way how the resulting image would be empty...

I just tested with my Android app, an unminified Cesium 1.67 version, my patched URI.toString() and some console.log() lines that the logo variable in CreditDisplay.js line 490 (see above) is correctly formatted. So the fix is valid.
The icon doesn't show up, though. Investigating with more console.log calls...

I think I found the reason why the <img> tag has no src attribute:
https://github.com/CesiumGS/cesium/blob/master/Source/Core/Credit.js#L93
This line tries to sanitize the Credit's _html property, which is accessed from this line:
https://github.com/CesiumGS/cesium/blob/master/Source/Scene/CreditDisplay.js#L39
Unfortunately I don't know why it's working in the browser, but not in the Android WebView...

@OmarShehata I can confirm that replacing this line fixes showing the image on Android:
https://github.com/CesiumGS/cesium/blob/5a4a66a2e79cb5dcc6a00ed1b7efc05c01d96306/Source/Core/Credit.js#L93
(replaced with: var html = this._html)
I also tested on Windows Firefox, and the img tag is also replaced there when accessing element. My guess is that at this time the Ion credit is already added to the DOM and the DOM elements are not replaced by the desktop browser.
What should we do with this PR? I would like to have it in 1.68, even when it doesn't fix #8665 yet. Thanks!

It's definitely DOMpurify, it strips out all src attributes that don't contain http or https links. I think I will solve the original ticket by manually adding the Ion credit string without a full path. CreditDisplay should recognize it as Ion credit, not showing the other Cesium credit, and DOMpurify will not sanitize a relative link.

@vividos thank you so much for investigating and resolving this! Can you please add or link to your conclusions here in the original issue https://github.com/CesiumGS/cesium/issues/8665 ? So someone can either suggest or implement a fix based on your findings. There may be other areas in the code that don't correctly handle links without http or https.

For what to do with this PR, @mramato the behavior for parsing URIs with file:/// (three slashes, no authority or hostname) is incorrect in master, and is fixed in this PR. Sandcastle with test case here https://github.com/CesiumGS/cesium/issues/8665#issue-576350228.
I'm not aware of a way to fix this without modifying the third party code, since ThirdParty/Uri.js incorrectly skips the // when the authority and hostname are blank. I would merge this after unit tests are added for it but wanted to get a second opinion here.

Omar, I will comment in the initial issue. Thanks!

@OmarShehata I also rebased this PR on top of master, but it isn't strictly necessary to fix #8665, now that I have PR #8758 in place. What should we do with this PR?

I'm closing this PR, as my initial issue #8665 is fixed.


refactor: add FS Router

Requires vaadin/hilla#2214 to be released

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: Artur-:white_check_mark: taefi:white_check_mark: tarekoraby:white_check_mark: cromoteca:white_check_mark: Lodin:x: platoshaYou have signed the CLA already but the status is still pending? Let us recheck it.


new extension nanoid

Description
New extension to generate nanoid's.
Screencast
Video: https://share.cleanshot.com/ra57LS
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

The faker extension you mention cannot generate nanoid's afaik.

alright 👌
I thought UUID is kinda the same thing.


is_converstation_view() returns wrong value after switching before reload

Seems like the globals[17][4][1] doesn't update the conversation view flag until after reload

I'm not intimately comfortable in this part of the codebase, but from what I can see, api.tracker.globals (and api.tracker.globals[17] too) doesn't really ever get updated except on reload.
I've tried just fiddling in the browser (without gmail.js) loaded, and from what I can tell, this flag is only updated by Gmail itself on reload. And there's nothing we can do about that.
Given how changing this preference is something people do once and never again, I'm fine with this short-coming.

What I did find while looking into this, was this odd one out: https://github.com/KartikTalwar/gmail.js/blob/master/src/gmail.js#L313
Everywhere in the codebase we refer to our instance-variable api.tracker.globals, but here we access GLOBALS directly. To me that looks like a bug worth fixing. Anyone have any opinions on this matter?
cc: @keven425 @KartikTalwar

@josteink Makes sense.
Btw seems like the GLOBALS direct access will cause NPE in popup window.

Makes sense to me too. Don't believe there's any particular reason - we can use api.tracker.globals instead

Thanks for the feedback. I've fixed the potential NPE in https://github.com/KartikTalwar/gmail.js/commit/8f45519f3639f812b3e8503b3f0a6cc897bd3f49.
In the meantime, I think this issue is not something which is worthwhile trying to find a work-around for. I vote close, unless ofcourse @patrickriordan has some really good arguments for why we should keep it open.
Patrick: We are all ears :)

Looks like get.displayed_email_data is also affected by conversation view not updating, it probably won't give the right data back after updating conversation before reloading. Best argument i got haha.
I'm fine with closing it though. Low priority for me as well.

Great. Closing this one then.


Composer: Control selection using setSelection

@hannojg

Details
Instead of controlling the selection the traditional way (passing selection prop) which is always render dependent and is so buggy on RN especially for Android. We are going to control the selection directly using an independent and render free method which is setSelection for Native and setSelectionRange for Web. We we will be using our setSelection library for that.
Fixed Issues
$ https://github.com/Expensify/App/issues/12854
PROPOSAL: https://github.com/Expensify/App/issues/12854#issuecomment-1506213114
Tests

Open App
Go to any chat
Type message Hi there
Move the cursor to be between Hi and there
Type :wave:
Verify the 👋 emoji got inserted correctly between Hi and there
Verify the cursor position is after the emoji
Verify the cursor did not jump around

[ ] Verify that no errors appear in the JS console

Offline tests
n/a
QA Steps

Open App
Go to any chat
Type message Hi there
Move the cursor to be between Hi and there
Type :wave:
Verify the 👋 emoji got inserted correctly between Hi and there
Verify the cursor position is after the emoji
Verify the cursor did not jump around

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://user-images.githubusercontent.com/16493223/233198531-42e5701c-5ecd-40c3-ad54-ddac1274c8e6.mp4

Mobile Web - Chrome

https://user-images.githubusercontent.com/16493223/233200013-c0f2b716-6b06-46eb-bf5c-4e0dc9d09afc.mp4

Mobile Web - Safari

https://user-images.githubusercontent.com/16493223/233198480-b6f15e31-a484-4a1c-abe4-60b37940d0ec.mp4

Desktop

https://user-images.githubusercontent.com/16493223/233197377-ea9c4bd5-cdcd-4f6a-ad2d-a37c998f767f.mp4

iOS

https://user-images.githubusercontent.com/16493223/233197348-adb21625-b6c4-4dac-9017-b888cbb329ac.mp4

Android

https://user-images.githubusercontent.com/16493223/233197332-b1f7437a-34d7-42e5-bab2-3aa2608c55e4.mp4

@MariaHCD @mollfpr One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

I would highly recommend to run the tests from this PR, as it should cover most edge cases.
Here is a copy of it:

Regression tests:
Test plan
Helper list to confirm you ran every test

[ ] Selection is not reset after selecting an emoji

[ ] Android
[ ] iOS
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Broken emojis are rendered when picked from the picker in a series

[ ] Android
[ ] iOS
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Cursor position is broken

[ ] Android
[ ] iOS
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Adding emoji via EmojiPicker in replacement for multiple lines creates a delay before the composer height is calculated.

[ ] Android
[ ] iOS
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Selected Emoji is placed in the wrong position.

[ ] Android
[ ] iOS
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Pasting image does not send the attached text message.

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] There is a delay before Composer expands the height on the web.

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Cursor position is not focused after picking a emoji in Composer.

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Content flashing while pasting.

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Edit Message: Copy-pasting is slow.

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Cursor jumping or moving positions after picking emoji.

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

[ ] Compose text is not cleared after sending the message

[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

All platforms, make sure for web to run on every browser (Safari, Chrome, Firefox, Edge):
Selection is not reset after selecting an emoji

Add multiple message on the composer.
selected multiple lines.
Open the emoji picker and choose an emoji.
selection remained.

https://user-images.githubusercontent.com/24370807/195184513-5d525c74-5db4-4fba-9b79-529ab49abe04.mp4
Broken emojis are rendered when picked from the picker in a series

Add a bunch of text lines in the composer.
Move the cursor to some middle position in the para.
Now pick an emoji from the emoji picker.
Try to pick the same emoji a few more times from the picker.

https://user-images.githubusercontent.com/24370807/195188451-c5566cad-bb70-4fd5-b4a4-4a52ca669358.mp4
Cursor position is broken

Steps

Add a bunch of lines to the composer on web.
Place your cursor somewhere in the middle.
Go to any other page and copy some strings. (e.g. Copy step 2 in clipboard)
Paste in the composer.
Cursor is not at the end of pasted string.

https://user-images.githubusercontent.com/24370807/195190228-f67f08b0-0971-4a49-ab05-07cd077afa53.mp4

Steps

Steps.

Open the app on the web.
Copy long text from some where (only text no markup).
Paste it on the composer.
Cursor is at the start of the pasted text.

Expected: it should be at the end of the pasted text.
Adding emoji via EmojiPicker in replacement for multiple lines creates a delay before the composer height is calculated.

Add multi-line message in composer.
Selects the multi-line text.
Add an emoji from the picker.
Observe that there is a delay before the composer height is adjusted correctly.

https://user-images.githubusercontent.com/24370807/196530115-2f2dede4-99d9-4074-89e9-19894113fce9.mp4
Selected Emoji is placed in the wrong position.

open the app.
Add three lines of message in composer.
Select some text from the middle line.
Open Emoji Picker and select an emoji.
Notice emoji is placed correctly and the selected text is gone.
Now reopen the EmojiPicker.
Select another emoji.
Notice that new emojis are added to the end of the line (not the previous cursor position).
Repeat from 4 to 6 to observe the same pattern.

[Web]
Pasting image does not send the attached text message.

Type some message in the composer.
Go to  image  and copy the image.
Back to the app and focus after the typed text in the composer.
Press  CTRL+V  to paste the image.
Click send on the opened attachment modal.
Observe the image is sent but the text remained in the composer and Send button is disabled.

Expected: Pressing send on the attachment modal should first send the text and then the image. (See staging for correct behaviour).
There is a delay before Composer expands the height on the web.

Open the app on the web.
type text quickly.
Use Shift+Enter to enter a couple of lines.

https://user-images.githubusercontent.com/24370807/202203404-5c0ad148-2cfe-4ae6-9073-3bfff1333a86.mp4
Cursor position is not focused after picking a emoji in Composer.

Paste the following content in composer.

ASDSADSA
DA😆 🤫 😙
SD
ASDSA
D
ASDAS
DSA
DSA
DAS
D
AS
D

😋 🤫 😋 😋 😋
SADSA
D
ASD
SA
D
SADS
AD
ASDAS
DAS

Now place the cursor after the first three emojies.
Scroll the composer content below so that cursor is no more visible but composer remains focused.
open emoji Picker.
select an emoji.
Notice that cursor position is not revealed.

https://user-images.githubusercontent.com/24370807/210625230-096ab82a-2882-4143-b441-cbc6a02b9a1a.mp4
Content flashing while pasting

Paste the following content in composer.

ASDSADSA
DA😆 🤫 😙
SD
ASDSA
D
ASDAS
DSA
DSA
DAS
D
AS
D

😋 🤫 😋 😋 😋
SADSA
D
ASD
SA
D
SADS
AD
ASDAS
DAS

Now use  Cmd + A  to select the whole content in the composer.
Paste the same copied content to replace.
Notice that there is a flashing of content. (Not present on staging).

https://user-images.githubusercontent.com/24370807/210626606-0dc1ab40-031c-4226-bfbd-7df7ddc3a692.mp4
Edit Message: Copy-pasting is slow.

Edit an old send message.
Copy following to clipboard.

ASDSADSA
DA😆 🤫 😙
SD
ASDSA
D
ASDAS
DSA
DSA
DAS
D
AS
D

😋 🤫 😋 😋 😋
SADSA
D
ASD
SA
D
SADS
AD
ASDAS
DASASDSADSA
DA😆 🤫 😙
SD
ASDSA
D
ASDAS
DSA
DSA
DAS
D
AS
D

😋 🤫 😋 😋 😋
SADSA
D
ASD
SA
D
SADS
AD
ASDAS
DAS

Replace the content on the edit message input with the copied content.
Select everything on the edit composer.
Paste via shortcuts.
Notice that it takes a few secs to replace.

https://user-images.githubusercontent.com/24370807/211291302-a3910abf-893a-4fbb-ae71-f4dcb8b92172.mp4
Cursor jumping or moving positions after picking emoji.

Edit a message or draft a long message on the composer. (Use the content from the previous report).
Place the cursor either before the center or after the center line.
Now pick an emoji.
Notice that the cursor does not remain in the same position even though the previous cursor position was in the view.

https://user-images.githubusercontent.com/24370807/211293351-868f78f0-7b2c-489e-a2b7-db20c2f708ce.mp4
Compose text is not cleared after sending the message
https://user-images.githubusercontent.com/22219519/214962291-e6cfc11f-c451-4c6e-84cc-ba556c92cc25.mov

@hannojg That's a LIST. By the time we get to finish testing RN would have fixed the bug and Safari would be awesome :joy:.

@hannojg

I am by now quite experiences with the weird bugs that can happen when using SetSelection

Can you please highlight some of those? or you are referring to those in the list?

Also there are some merge conflicts @s77rt

Thanks @mountiny! I’ll start testing after the conflict resolved.

Resolved conflicts. Should be ready for test/review.

I am just loosely going through a few items of the list, @parasharrajat will probably do a full audit.
What's following is just a few bugs I have noticed
iOS: Enter multiline text, set the cursor to the top position, scroll down, insert emoji. Bug: text input doesn't scroll up
https://user-images.githubusercontent.com/16821682/234298060-9ce35447-b6e0-49b6-aa02-2d424dc77898.mp4

iOS: Inserting an emoji by using the : moves the text input's content:
https://user-images.githubusercontent.com/16821682/234300673-9ad86c11-d633-426a-90bb-93e1f7ce2cf4.mp4

Android: Selection is not reset after inserting an emoji
https://user-images.githubusercontent.com/16821682/234301738-951c8a6d-6047-4c3c-b008-b0074314c6bd.mov

@hannojg Those bugs does not occur with traditional selection prop right? If so then we should probably fix the controlled selection bug instead

@mollfpr will you have time to go through the testing?
Whats the plan @s77rt regarding what Hanno pointed out?

@mountiny I was hoping that setSelection will magically work regression-free :sweat_smile:. I have no idea how to fix the reported bugs on E/App, it's probably something that must escalates to RN. And if it's the case we should just fix the buggy selection prop. That bug is about 2-3 years so it may not be that straightforward.
@hannojg Do you have some comments on those bugs? Or what would you suggest in this case?

I see, I guess if we can clearly define some problem/ solution for that buggy selection issue, we could get some bigger job running 8k+ and someone will have to fix this in RN eventually :D

Selection is not reset after selecting an emoji
❌ Android
✅ iOS
✅ mWeb/Chrome
✅ mWeb/Safari
✅ Safari
✅ Chrome
✅ Firefox
✅ Desktop
Broken emojis are rendered when picked from the picker in a series
✅ Android
✅ iOS
✅ mWeb/Chrome
✅ mWeb/Safari
✅ Safari
✅ Chrome
✅ Firefox
✅ Desktop
Cursor position is broken
✅ Android
✅ iOS
✅ mWeb/Chrome
✅ mWeb/Safari
✅ Safari
✅ Chrome
✅ Firefox
✅ Desktop
Adding emoji via EmojiPicker in replacement for multiple lines creates a delay before the composer height is calculated.
✅ Android
✅ iOS
✅ mWeb/Chrome
✅ mWeb/Safari
✅ Safari
✅ Chrome
✅ Firefox
✅ Desktop
Selected Emoji is placed in the wrong position.

[ ] Android
[ ] iOS
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
Pasting image does not send the attached text message.
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
There is a delay before Composer expands the height on the web.
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
Cursor position is not focused after picking a emoji in Composer.
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
Content flashing while pasting.
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
Edit Message: Copy-pasting is slow.
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
Cursor jumping or moving positions after picking emoji.
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop
Compose text is not cleared after sending the message
[ ] mWeb
[ ] Safari
[ ] Chrome
[ ] Firefox
[ ] Desktop

Issue when executing Adding emoji via EmojiPicker in replacement for multiple lines creates a delay before the composer height is calculated.
Firefox - Adding emoji from the picker adding a bottom space inside the input.

Focus on the input and open the emoji picker
Select any emoji
Observed the input bottom

Video
https://user-images.githubusercontent.com/25520267/234471178-2ed7be4c-e2b0-43e6-b5b8-a24431e49c42.mov

Issue when executing Selected Emoji is placed in the wrong position.
Android - The selection selects the previous letter and will select the previous text

Type multiline text e.g.:

abcdefg abcdefg
abcdefg abcdefg

Select the letter cde from the second line
Open emoji picker and select an emoji
Notice that the selection highlights the letter b, and the emoji added
Do step 3 again
Notice that the letter b is removed, and now the selection highlights the previous a
Do step 3 again
Notice that the letter a is removed, and the selection highlights space in the first line
Do step 3 again
Notice that the selection randomly highlights the letter of the second word from the first line with the emoji added

https://user-images.githubusercontent.com/25520267/234497252-62cd86da-26c4-41c0-884a-698f21238ada.mp4

@s77rt
https://user-images.githubusercontent.com/25520267/234594339-a9fcfbbe-0cda-453d-b41a-a25df9cfa4ea.mp4

@parasharrajat Yes sadly we don't have a clear solution. I believe the other issue is only for Android. I think we should do what @mountiny proposed, raise a new clear P/S statement in an attempt to fix the buggy selection prop on RN for both Android and iOS.

There is not that many fails, but lets go with fixing this upstream. Anyone is willing to take this on? I will close this PR after

@mountiny I will but can I get a clarification on the plan, is it:

Create 2 E/App issues (one for Android and other for iOS)
Wait for proposals
Fix the bugs

Or we will be taking this to RN first?

Hey, Hanno here from Margelo 👋 I am here to help find a solid fix for this issue, also in hindsight of my other PR https://github.com/Expensify/App/pull/11684.
I am ready to make a proposal in some appropriate place, that will handle all the issues discussed.
I just want to note that the proposal likely won't be a upstream fix, as I figured that some of the problems are simply not fixable upstream (there exists no solution that is working generically enough, to suite all use cases with inputs in react native). My proposal would very likely include creating an own native component for TextInput on android (which will extend/inherit the existing react native's text input).
I just want to point out that if you set as a hard requirement that it should be fixed upstream, you are very likely to deadlock these issues, as -again - its not really possible to fix it upstream to serve all our use cases.

Alright, I think we could go deep haha but what I suggest then @hannojg is to create a P/S in the #expensify-open-source channel to explain what problems we are solving (this should be quite straightforward we have real bugs right here haha) and then in the threads also please dont forget to mention why its not fixable upstream ideally with backing of some exisitng tries from other people. I am sure some people will come and ask "are you sure" so adding osmething like this will help with it.
Would @s77rt and @hannojg be interested in working on this a bit together? Not sure how you like to work but there could be help with debugging nad testing for sure :) That do you think?

@hannojg I wouldn't be too surprised if the upstream fix is a lot of work but this was actually working on RN, so maybe we can just drop the "features" that broke it?
Also please note that we are looking to fix the buggy selection on iOS too (which would also require upstream changes)
I'm interested on working on this as @mountiny suggested, let me know what do you think

Can we move this discussion to Slack? I think that would be easier (the problem solution statement can be nailed later, I will close this Pr for now)


Animation for 'Best Trip Package' section image done

Title and Issue number

Title : Feature_Request Add animation to 'Best Trip Package' Section
Issue No. : #46
Code Stack : HTML , CSS
Close #46

Video/Screenshots (mandatory)

https://github.com/apu52/Travel_Website/assets/149994109/a8ce3e83-f21a-46d3-a907-88068c694971
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
I have added animation to images in 'Best Trip Package' section and also made sizes of all images uniform for same.
Are you contributing under any Open-source programme?

Yes, I have completed this issue under JWOC.

@Shravani-Korde Thank you for your valuable Contribution


Everything needed to get it deployed

setup_local_mssql_docker.sh and `setup_local_mssql_docker.sh' both have db-migrate scripts in them that won't work with the new package.json. This won't be captured by the tests.

setup_local_mssql_docker.sh and `setup_local_mssql_docker.sh' both have db-migrate scripts in them that won't work with the new package.json. This won't be captured by the tests.

Thanks, fixed in those. We can probably later as part of removing postgres, make these scripts simpler to understand/not refer to mssql.


Fix godoc deprecated comments.

Changes

Fix godoc deprecated comments.
From https://github.com/golang/go/wiki/Deprecated:

To signal that an identifier should not be used, add a paragraph to its doc
comment that begins with "Deprecated:" followed by some information about the
deprecation, and a recommendation on what to use instead, if applicable.

This should help IDEs and pkgsite to pick up on deprecated fields.
Most changes fall into a few categories:

"Deprecated." -> "Deprecated:"
Added paragraph new line.
Added deprecation description.

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

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

Release Notes
NONE

/kind documentation

/test pull-tekton-pipeline-alpha-integration-tests

/retest


Training with intermediate checkpoints

Hi,
My training went into divergence. I am trying to train using intermediate checkpoints.
Command used is :
lstmtraining --stop_training --continue_from /home/administrator/pooja/training_output/training_changed3.118_1971.checkpoint --traineddata /home/administrator/pooja/training_changed_lstmf/eng/eng.traineddata --model_output /home/administrator/pooja/training_output/test.traineddata
But it is giving error:
mgr_.Init(traineddata_path.c_str()):Error:Assert failed:in file ../../src/lstm/lstmtrainer.h, line 110
Illegal instruction (core dumped)
Please help

Issue resolved.

how did you solve it?

Issue resolved.


Update customer-key-manage.md

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@sathana87 Gentle reminder

@sathana87 Please update

@sathana87 Please implement the suggested changes to initiate the merger of this PR. Thanks

@sathana87 Please implement the suggested changes, in the event of no response we may may have to close this PR. Thanks

@sathana87 We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Check team id

https://developer.apple.com/help/account/manage-your-team/locate-your-team-id?ssp=1&darkschemeovr=1&setlang=en&cc=FI&safesearch=moderate

Fixed by https://github.com/NigelBreslaw/guardian-ghost/commit/20c2b5e7ad27cce4bf86b48bd16f1fd56f27afa1


Reorder, reduce simplify structure of Features dependencies

We want to know wich Features depends from others Feature.
To accomplish this, we need to create graphs that show this clearly.

Technically this can be done with yaml files, Rakefile Tasks. Wach yaml File contains feauees seta. Rakefile Will orchestrate the execution order.
Additionaly we could draw a three od sets dependencies.

Yes, we need to split the test suite, and basing it on Rakefiles just makes sense. +1.
How would you determine the sets? "Based on my experience" is a valid answer ;-) or do you envision using a tool that could give us a graph we could use as a base?

this will be adressed too soon by idempotent, closing


[bug] useEnsName Returns null When Attempting To Fetch ENS From Gnosis Safe

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.4.12
Current Behavior
Calling useEnsName on an Ethereum address that is a gnosis safe returns null
Sample Code.
  const { data: ensName } = useEnsName({
    address: "0xd2aa6323f32dae7aa8e20734260204a6ee7c1e94",
  });

In this example ensName will be null.
Expected Behavior
Calling useEnsName on an Ethereum address that is a gnosis safe returns atomize.eth
  const { data: ensName } = useEnsName({
    address: "0xd2aa6323f32dae7aa8e20734260204a6ee7c1e94",
  });

In this example ensName should be atomize.eth
Steps To Reproduce
git clone https://github.com/joeblau/wagmi-gnosis
cd wagmi-gnosis
yarn install
yarn dev
open https://localhost:3000

In pages/index.tsx toggle between the two sets of code to view the results.
  // doodles.eth/poopie.eth wallet works
  const { data: ensName } = useEnsName({
    address: "0xC35f3F92A9F27A157B309a9656CfEA30E5C9cCe3",
  });

  // atomize.eth gnosis safe does not work
  const { data: ensName } = useEnsName({
    address: "0xd2aa6323f32dae7aa8e20734260204a6ee7c1e94",
  });

  return (
    <div>
      <h1>{`ensName: ${ensName}`} </h1>
    </div>
  );

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
https://github.com/joeblau/wagmi-gnosis
Anything else?
The result for 0xd2aa6323f32dae7aa8e20734260204a6ee7c1e94 or atomize.eth

The result for 0xC35f3F92A9F27A157B309a9656CfEA30E5C9cCe3 or poopie.eth

0xd2aa6323f32dae7aa8e20734260204a6ee7c1e94 is the registrant/controller of atomize.eth, but it's reverse record is set to 0xb6542DE25941D3ca4Eb839F6C9096823e09Ab5B4.


unable to hide tabbarcontroller when pushed!

unable to hide tabbarcontroller when pushed!

Did you find a solution to this?  I'm trying to hide the tabbarycontroller when EAIntroView https://github.com/ealeksandrov/EAIntroView is displayed.

I tried a clumsy way but worked.
Here is what I did:

Make a globel variable tabBarController in AppDelegate;

Use this in every child controller of tabBarController:

-(void)viewWillAppear:(BOOL)animated{
    AppDelegate *appDelegate = [[UIApplication sharedApplication] delegate];
    if ([appDelegate tabBarController].tabBarView.hidden == NO) {
    } else {
        [appDelegate tabBarController].tabBarView.hidden = NO;
    }
}

3.Add this in the UIViewController you pushed:
-(void)viewWillAppear:(BOOL)animated{
    AppDelegate *appDelegate = [[UIApplication sharedApplication] delegate];
    if ([appDelegate tabBarController].tabBarView.hidden == YES) {
    } else {
        [appDelegate tabBarController].tabBarView.hidden = YES;
    }
}

I know that is not a smart way, is there any other ideas ?

Any solution ? I'm stuck on this issue :(

I'll follow the discussion me too!

Similar but less code than TindleWei's answer.

Just create a YalFoldingTabBarController property in AppDelegate.h
(i.e YalFoldingTabBarController *myTabBarController)

Import AppDelegate.h in the view controller that you want to hide the tabbarcontroller
(i.e #import "AppDelegate.h")

in ViewDidAppear method of that view controller, reference your tabBarController property in app delegate and set the tabBarView to hidden

example:
desiredViewController ()
- (void) viewWillAppear
{
AppDelegate *appDelegate = [[UIApplication sharedApplication] delegate];
[appDelegate myTabBarController].tabBarView.hidden = YES;
}

Guys, it can be done much simpler:

(void)viewWillAppear:(BOOL)animated {
[super viewWillAppear:animated];
YALFoldingTabBarController *tabBarController = (YALFoldingTabBarController *)self.tabBarController;
tabBarController.tabBarView.hidden = YES;
}


Suggestions for a more beginners friendly CONTRIBUTING.md page

As a beginner looking to contribute to the QIIME 2 project, I found that having the project forum URL as the only piece of information on the CONTRIBUTING.md page is a bit vague. Thus, the QIIME 2 project forum itself is quite massive and it is not easy to find instructions/guidelines on basic questions for beginners like me.
Therefore, I would suggest adding more information to the CONTRIBUTING.md page that can help out beginners such as:

Whom should I contact in case I want to contribute to this project?
What is the general contribution process for this project?
Are there any coding guidelines and/or code of conducts that beginners like me should know about before contributing to this project?
Basic instructions on how to set up development environment (specific tools, software packages, files, dependencies...etc.), how to report issues/bugs, and the pull request process.

I agree, and think it could look something like this. This is a problem that would turn away potential contributors who are new to the project and/or open-source in general.

That's a really good prototype. I will start working on it when I have more free time next week.

I made all suggested changes to the CONTRIBUTING.md page and created a pull request here:
https://github.com/qiime2/qiime2/pull/459

Thanks for discussing this! We actually had two contribution documents in this repo, I finally got around to removing the deprecated one (CONTRIBUTING.md), in favor of the .github/CONTRIBUTING.md doc. The advantage to using the .github/ one is that it integrates more deeply with GitHub. Anyhow, thanks for thinking of us! Hope to see you around here more in the future.
Just curious, how did y'all get hooked up with QIIME 2? What prompted you to take a closer look? Thanks!

Thank you for your clarification. I'm a college student taking a course in open-source software development currently. My professor encourages all of us to look up interesting projects on GitHub and try to contribute something to them. That's how I found out about QIIME 2. Hopefully, with more time, I would be able to offer more significant contributions to your project.


msal don't work if B2C susi policy initiated manually

Steps to reproduce:

Open b2c susi policy manually by going in browser to url like the following: https://login.microsoftonline.com/te/mytenant.onmicrosoft.com/b2c_1a_signup_signin/oauth2/v2.0/authorize?response_type=id_token&scope=https%3A%2F%2Fmytenant.onmicrosoft.com%2Fapi%2Fread%20openid%20profile&client_id=client-guid&redirect_uri=http%3A%2F%2Flocalhost%3A8080%2F&state=8bf581a4-a581-4347-8e20-f8dcb532a6e6&nonce=f0b68acb-35f8-4cc6-9bf5-1554c91ed623&client_info=1&x-client-SKU=MSAL.JS&x-client-Ver=0.1.3&brand=mybrand&client-request-id=12e14a79-5261-43e0-a7c5-3642ef0a3ec1&prompt=select_account&response_mode=fragment
sign in
browser is redirected to SPA: http://localhost:8080/#state=8bf581a4-a581-4347-8e20-f8dcb532a6e6&client_info=eyJ1aWQiOiI1MDk2MWQzMi0wY2FkLTQyOTMtYmYwNS1mNTI5MjY3M2NjZGMtYjJjXzFhX3NpZ251cF9zaWduaW4iLCJ1dGlkIjoiNjJhMGM5MzItMTgyMC00YzM4LTg3YTQtOWE5YWIyYzRjMmZhIn0&id_token=eyJ0eXAic...
msal redirects browser to http://localhost:8080/
msalApp.getUser() unexpectedly returns null :(

I expect to msalApp.getUser() return signed in user object.
If I use msalApp.loginRedirect, then msalApp.getUser() return correct user object

@ed-ilyin This is by design. Msal saves parameters like state and nonce when you call loginRedirect in the browser storage and it then matches the parameters in the response with the ones saved in the storage. If you make the authorization request manually, no such parameters will be saved and Msal must not process the response for security reasons. In this case , user object will be returned as null.


Update EditPlacement.cshtml

If the javascript snippet is not located in the footer then jQuery will not be recognized.

Dear @sfmskywalker could you please review this pull request. may be I did not know how to make the request correctly.

Hi ramiassi, it looks like you created a pull request for your own fork. Please create a PR on the Orchard repository. Please refer to the documentation on how to create a PR for Orchard: http://docs.orchardproject.net/en/latest/Documentation/Contributing-patches/
Thanks


Cannot use a secure OTLP endpoint

Describe the bug
I receive a context deadline exceeded error when the agent is configured to send to a TLS OTLP endpoint.
Environment

OS: Linux (Ubuntu 22.04.2 LTS)

uname: Linux 5.19.0-1023-aws #24~22.04.1-Ubuntu SMP Wed Mar 29 15:23:31 UTC 2023 x86_64 GNU/Linux

Go Version: go version go1.18.1 linux/amd64
Version: 83cae247d736997efa1efee8d74cd2e999899089

To Reproduce
Steps to reproduce the behavior:

make build
run:

# sudo isn't needed to repro, but to rule out a permissions issue
sudo \
  OTEL_TARGET_EXE=/usr/bin/ls \
  OTEL_SERVICE_NAME=context-deadline-sadness \
  OTEL_EXPORTER_OTLP_ENDPOINT=api.honeycomb.io:443 \
  ./otel-go-instrumentation

Behold! The Error! A deadline has not been met!

}
{
	"level": "info",
	"ts":<PHONE_NUMBER>.3007176,
	"caller": "opentelemetry/controller.go:107",
	"msg": "Establishing connection to OpenTelemetry collector ..."
}
{
	"level": "error",
	"ts":<PHONE_NUMBER>.3095894,
	"caller": "opentelemetry/controller.go:113",
	"msg": "unable to connect to OpenTelemetry collector",
	"addr": "api.honeycomb.io:443",
	"error": "context deadline exceeded",
	"stacktrace": "github.com/open-telemetry/opentelemetry-go-instrumentation/pkg/opentelemetry.NewController\n\t/home/ubuntu/opentelemetry-go-instrumentation/pkg/opentelemetry/controller.go:113\nmain.main\n\t/home/ubuntu/opentelemetry-go-instrumentation/cli/main.go:45\nruntime.main\n\t/usr/lib/go-1.18/src/runtime/proc.go:250"
}
{
	"level": "error",
	"ts":<PHONE_NUMBER>.3097224,
	"caller": "cli/main.go:47",
	"msg": "unable to create OpenTelemetry controller",
	"error": "context deadline exceeded",
	"stacktrace": "main.main\n\t/home/ubuntu/opentelemetry-go-instrumentation/cli/main.go:47\nruntime.main\n\t/usr/lib/go-1.18/src/runtime/proc.go:250"
}

Expected behavior
I expect a secure connection to be established. I can make it happen by changing the current hard-coded insecure gRPC config to a hard-coded TLS config:
- conn, err := grpc.DialContext(timeoutContext, endpoint, grpc.WithInsecure(), grpc.WithBlock())
+ conn, err := grpc.DialContext(timeoutContext, endpoint, grpc.WithTransportCredentials(credentials.NewTLS(&tls.Config{})), grpc.WithBlock())

I imagine the proper fix will be some shenanigans for supporting more of the OTEL_EXPORTER_OTLP_* environment variable config surface area.

I think I might have accidentally-on-purpose have a fix for this while trying to add an "OTel-Go-Auto-Instrumentation" user-agent to the agent's OTLP exporter connection. … which is an issue I meant to open, but could just open a PR for. 😁