text:
Pooling backward solvers (except GetWorkspaceSize)

This PR contains both backward pooling solvers.
Testing in process.

@DrizztDoUrden

[2021-11-19T21:50:02.144Z] 50/92 Test #39: test_pooling3d ........................................***Failed  Error regular expression found in output. Regex=[FAILED]271.25 sec
[2021-11-19T21:50:02.144Z] /var/jenkins/workspace/s_MIOpen_ddu_pooling-bwd-solvers/build/bin/test_pooling3d --float --index_type miopenIndexUint32 --mode miopenPoolingMax --verify_indices 0 --input 16 64 3 4 4 --lens 2 2 2 --strides 2 2 2 --pads 0 0 0 --wsidx 1
[2021-11-19T21:50:02.144Z] FAILED: /var/jenkins/workspace/s_MIOpen_ddu_pooling-bwd-solvers/src/ocl_kernel.cpp:87: Running kernel failed:  Invalid Kernel Args

@DrizztDoUrden Is testing done? Description says "in process"

@zjing14 It would be nice to get your blessing before merging this.

@averinevg If you have time, could you please review? I would like to double-check if this refactoring PR is really clean and safe (No Functional Changes).

LGTM

@DrizztDoUrden Can you please fix merge conflicts, thanks!

There are some CI issues when latest develop is merged:
http://micimaster.amd.com/blue/organizations/jenkins/MLLibs%2FMIOpen/detail/ddu%2Fpooling-bwd-solvers/9/pipeline
[2022-01-04T23:29:23.889Z] In file included from /var/jenkins/workspace/s_MIOpen_ddu_pooling-bwd-solvers/src/solver/pooling/forward2d.cpp:27:

[2022-01-04T23:29:23.889Z] /var/jenkins/workspace/s_MIOpen_ddu_pooling-bwd-solvers/src/include/miopen/pooling/solvers.hpp:90:35: error: unknown template name 'SolverBase'

[2022-01-04T23:29:23.889Z] struct PoolingBackward2d : public SolverBase<OldStyleProblemDescription>

[2022-01-04T23:29:23.889Z]                                   ^

@DrizztDoUrden @junliume

... error: unknown template name 'SolverBase'

I would fix it

@DrizztDoUrden could you merge develop into this branch and resolve the conflicts? Thanks!

@DrizztDoUrden

Also it may worth updating from develop.

@junliume I have a problem with the API here. Backward pooling call has a constant workspace pointer. That doesn't allow us to use it for the transpose. But making it non-const shouldn't be a breaking change. Should I update the API?

@junliume I have a problem with the API here. Backward pooling call has a constant workspace pointer. That doesn't allow us to use it for the transpose. But making it non-const shouldn't be a breaking change. Should I update the API?

@DrizztDoUrden sorry for the late reply. I would vote for updating the API as long as it is not breaking.
@averinevg @atamazov @JehandadKhan @ce1adon @zjing14  @asroy  what do you think?

@junliume I have a problem with the API here. Backward pooling call has a constant workspace pointer. That doesn't allow us to use it for the transpose. But making it non-const shouldn't be a breaking change. Should I update the API?

@DrizztDoUrden sorry for the late reply. I would vote for updating the API as long as it is not breaking. @averinevg @atamazov @JehandadKhan @ce1adon @zjing14 @asroy what do you think?

@DrizztDoUrden Could you please show how the new API will look like compared to the old one?

new:
MIOPEN_EXPORT miopenStatus_t miopenPoolingBackward(miopenHandle_t handle,
                                                   const miopenPoolingDescriptor_t poolDesc,
                                                   const void* alpha,
                                                   const miopenTensorDescriptor_t yDesc,
                                                   const void* y,
                                                   const miopenTensorDescriptor_t dyDesc,
                                                   const void* dy,
                                                   const miopenTensorDescriptor_t xDesc,
                                                   const void* x,
                                                   const void* beta,
                                                   const miopenTensorDescriptor_t dxDesc,
                                                   void* dx,
                                                   void* workSpace);

old:
MIOPEN_EXPORT miopenStatus_t miopenPoolingBackward(miopenHandle_t handle,
                                                   const miopenPoolingDescriptor_t poolDesc,
                                                   const void* alpha,
                                                   const miopenTensorDescriptor_t yDesc,
                                                   const void* y,
                                                   const miopenTensorDescriptor_t dyDesc,
                                                   const void* dy,
                                                   const miopenTensorDescriptor_t xDesc,
                                                   const void* x,
                                                   const void* beta,
                                                   const miopenTensorDescriptor_t dxDesc,
                                                   void* dx,
                                                   const void* workSpace);

@junliume I have a problem with the API here. Backward pooling call has a constant workspace pointer. That doesn't allow us to use it for the transpose. But making it non-const shouldn't be a breaking change. Should I update the API?

@averinevg @atamazov @JehandadKhan @ce1adon @zjing14 @asroy what do you think?

No objections. Actually we do not have anything better than removal of constness, if we want to engage transpose to implement NHWC pooling quickly.
:warning: This may lead to build errors of the user apps, if they actually pass const workspace ptr to miopenPoolingBackward(). But the fix of such user apps should be trivial.

They would have to create a new const buffer and to copy data from the forward workspace (which may not be const). But yes, I have somehow missed that it is actually kind of breaking in theory.

@DrizztDoUrden we have some HIP tidy issues: http://micimaster.amd.com/blue/rest/organizations/jenkins/pipelines/MLLibs/pipelines/MIOpen/branches/ddu%252Fpooling-bwd-solvers/runs/21/nodes/13/log/?start=0

Updated the API

I recommend marking this PR with "[API change]" in order to make this commit immediately visible in the history.

authors: DrizztDoUrden atamazov averinevg junliume zjing14

repo: ROCmSoftwarePlatform/MIOpen


text:
Improve workaround for SWDEV-330460

Improvement of W/A introduced in #1616: this allows to enable ConvAsmBwdWrw3x3 explicitly by means of MIOPEN_DEBUG_CONV_DIRECT_ASM_WRW3X3. Might be useful for experimentation when developing the full-blown fix.

@JehandadKhan One detail I've forgot during review of #1616.

/cc @junliume @Kirpich30000 FYI this is for the one who will look into SWDEV.

@junliume  CI passes

@junliume @JehandadKhan BTW what is the current status of SWDEV-330460? Have we decided to keep this solver disabled for MI200 (and thus transform the W/A into the persistent part of the design of this solver)?

SWDEV-330460

Hi @atamazov  so far we are keeping this WA, since it is for FP32 we are not fixing it at kernel level at high urgencies.

source: gharchive/pull-request

authors: JehandadKhan atamazov junliume

license: mit

license_type: permissive

license_source: bigquery


text:
import fails after installation.

Ubuntu 1804:
root@sriov-guest:~# python3 --version
Python 3.6.9
Installed as usual:
pip3 install --user tensorflow-rocm --upgrade
There seems online discussion that it happens when CWD is tensorflow source but that should not be the cause of import.
tensorboard-data-server 0.6.1
tensorboard-plugin-wit  1.8.0
tensorflow-estimator    2.4.0
tensorflow-rocm         2.4.3
termcolor               1.1.0
testresources           2.0.0
traceback2              1.4.0
typing-extensions       3.7.4.3
ubuntu-advantage-tools  27.0
ubuntu-drivers-common   0.0.0
ufw                     0.36
unattended-upgrades     0.1
unittest2               1.1.0
urllib3                 1.22
usb-creator             0.3.3
verboselogs             1.7
wadllib                 1.3.2
Werkzeug                2.0.1
wheel                   0.36.2
wrapt                   1.12.1
xkit                    0.0.0
zipp                    3.4.1
zope.interface          4.3.2
root@sriov-guest:/git.co/tensorflow# python3
Python 3.6.9 (default, Jan 26 2021, 15:33:00)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/git.co/tensorflow/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
  File "/git.co/tensorflow/tensorflow/python/__init__.py", line 40, in <module>
    from tensorflow.python.eager import context
  File "/git.co/tensorflow/tensorflow/python/eager/context.py", line 32, in <module>
    from tensorflow.core.framework import function_pb2
ImportError: cannot import name 'function_pb2'
>>> quit()

after apt-get update && apt dist-upgrade + rocm full installation, it appears to work.

Fresh installation still failing:
-bazel/linux-x86_64/crypto/fipsmodule/x86_64-mont5.o python_build/temp.linux-x86_64-3.6/third_party/boringssl-with-bazel/linux-x86_64/crypto/fipsmodule/x86_64-mont.o python_build/temp.linux-x86_64-3.6/third_party/boringssl-with-bazel/src/crypto/hrss/asm/poly_rq_mul.o -lrt -lm -o python_build/lib.linux-x86_64-3.6/grpc/_cython/cygrpc.cpython-36m-x86_64-linux-gnu.so -lpthread -Wl,-wrap,memcpy -static-libgcc
Traceback (most recent call last):
File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/commands.py", line 264, in build_extensions
build_ext.build_ext.build_extensions(self)
File "/usr/lib/python3.6/distutils/command/build_ext.py", line 448, in build_extensions
self._build_extensions_serial()
File "/usr/lib/python3.6/distutils/command/build_ext.py", line 473, in _build_extensions_serial
self.build_extension(ext)
File "/root/.local/lib/python3.6/site-packages/setuptools/command/build_ext.py", line 203, in build_extension
if ext._needs_stub:
AttributeError: 'Extension' object has no attribute '_needs_stub'
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/tmp/pip-build-3myzovp3/grpcio/setup.py", line 465, in <module>
    cmdclass=COMMAND_CLASS,
  File "/root/.local/lib/python3.6/site-packages/setuptools/__init__.py", line 153, in setup
    return distutils.core.setup(**attrs)
  File "/usr/lib/python3.6/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/usr/lib/python3.6/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/root/.local/lib/python3.6/site-packages/setuptools/command/install.py", line 61, in run
    return orig.install.run(self)
  File "/usr/lib/python3.6/distutils/command/install.py", line 589, in run
    self.run_command('build')
  File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/usr/lib/python3.6/distutils/command/build.py", line 135, in run
    self.run_command(cmd_name)
  File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/root/.local/lib/python3.6/site-packages/setuptools/command/build_ext.py", line 79, in run
    _build_ext.run(self)
  File "/usr/lib/python3.6/distutils/command/build_ext.py", line 339, in run
    self.build_extensions()
  File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/commands.py", line 267, in build_extensions
    support.diagnose_build_ext_error(self, error, formatted_exception)
  File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/support.py", line 118, in diagnose_build_ext_error
    diagnostic(build_ext, error)
  File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/support.py", line 98, in diagnose_attribute_error
    "We expect a missing `_needs_stub` attribute from older versions of "
commands.CommandError: We expect a missing `_needs_stub` attribute from older versions of setuptools. Consider upgrading setuptools.

----------------------------------------

Command "/usr/bin/python3 -u -c "import setuptools, tokenize;file='/tmp/pip-build-3myzovp3/grpcio/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-lrbza8vl-record/install-record.txt --single-version-externally-managed --compile --user --prefix=" failed with error code 1 in /tmp/pip-build-3myzovp3/grpcio/

setuptools upgrade warning is misleading as i am using latest one:
-bazel/linux-x86_64/crypto/fipsmodule/x86_64-mont5.o python_build/temp.linux-x86_64-3.6/third_party/boringssl-with-bazel/linux-x86_64/crypto/fipsmodule/x86_64-mont.o python_build/temp.linux-x86_64-3.6/third_party/boringssl-with-bazel/src/crypto/hrss/asm/poly_rq_mul.o -lrt -lm -o python_build/lib.linux-x86_64-3.6/grpc/_cython/cygrpc.cpython-36m-x86_64-linux-gnu.so -lpthread -Wl,-wrap,memcpy -static-libgcc
Traceback (most recent call last):
File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/commands.py", line 264, in build_extensions
build_ext.build_ext.build_extensions(self)
File "/usr/lib/python3.6/distutils/command/build_ext.py", line 448, in build_extensions
self._build_extensions_serial()
File "/usr/lib/python3.6/distutils/command/build_ext.py", line 473, in _build_extensions_serial
self.build_extension(ext)
File "/root/.local/lib/python3.6/site-packages/setuptools/command/build_ext.py", line 203, in build_extension
if ext._needs_stub:
AttributeError: 'Extension' object has no attribute '_needs_stub'
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/tmp/pip-build-3myzovp3/grpcio/setup.py", line 465, in <module>
    cmdclass=COMMAND_CLASS,
  File "/root/.local/lib/python3.6/site-packages/setuptools/__init__.py", line 153, in setup
    return distutils.core.setup(**attrs)
  File "/usr/lib/python3.6/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/usr/lib/python3.6/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/root/.local/lib/python3.6/site-packages/setuptools/command/install.py", line 61, in run
    return orig.install.run(self)
  File "/usr/lib/python3.6/distutils/command/install.py", line 589, in run
    self.run_command('build')
  File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/usr/lib/python3.6/distutils/command/build.py", line 135, in run
    self.run_command(cmd_name)
  File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
    self.distribution.run_command(command)
  File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/root/.local/lib/python3.6/site-packages/setuptools/command/build_ext.py", line 79, in run
    _build_ext.run(self)
  File "/usr/lib/python3.6/distutils/command/build_ext.py", line 339, in run
    self.build_extensions()
  File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/commands.py", line 267, in build_extensions
    support.diagnose_build_ext_error(self, error, formatted_exception)
  File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/support.py", line 118, in diagnose_build_ext_error
    diagnostic(build_ext, error)
  File "/tmp/pip-build-3myzovp3/grpcio/src/python/grpcio/support.py", line 98, in diagnose_attribute_error
    "We expect a missing `_needs_stub` attribute from older versions of "
commands.CommandError: We expect a missing `_needs_stub` attribute from older versions of setuptools. Consider upgrading setuptools.

----------------------------------------

Command "/usr/bin/python3 -u -c "import setuptools, tokenize;file='/tmp/pip-build-3myzovp3/grpcio/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-lrbza8vl-record/install-record.txt --single-version-externally-managed --compile --user --prefix=" failed with error code 1 in /tmp/pip-build-3myzovp3/grpcio/

Hi @gggh000 , can you provide a Dockerfile to reproduce the failures you have observed, in case you're expecting inputs from the ROCm team?

source: gharchive/issue

license_source: github-api


text:
fix(ecau): copy MusicBrainz complex artwork types

"Matrix/Runout" and "Raw/Unedited" could not be copied when a MB release's images were copied because the enum used the "Matrix" and "Raw" keys only. Instead, we'll use the full artwork type name as keys. This also requires changes in VGMdb's caption to artwork type mappings and related tests. Also add test case with a real release.
Fixes #708

/deploy-preview

repo: ROpdebee/mb-userscripts

license: MIT


text:
Use NextAuth.js for auth instead

https://nextjs.org/docs/authentication
https://next-auth.js.org/

Use <Provider> to increase performance.
https://github.com/nextauthjs/next-auth-example/blob/d74009aac8e70a9c6f49168051df440904786860/pages/_app.js#L4

repo: RaenonX-DL/dragalia-site-front


text:
The bot is spamming messages

It looks like that when loop=all is selected, at some point, it starts skipping all songs and sending each song in the queue without stopping, like if the songs were 0 seconds long.
I unforunately haven't any screenshot because I cleared the channel.

I'll try to reproduce it locally, any informations about when or how it happens will help a lot

Ok, so here are the setps to reproduce it:

In the config file, set SHOULD_AUTO_DISCONNECT_WHEN_ALONE = False
Play 2 songs ( the ones i played where https://www.youtube.com/watch?v=hgr4b-ve6XU&ab_channel=ArthurSena & https://www.youtube.com/watch?v=h8YZMT94rR4&t=7665s&ab_channel=smugkat )
Set Loop = all
Wait
At some point, the bot will stop playing music and start spamming 🎧 Song playing now messages, skipping the songs like they were 0 minutes long.

How long do you usually wait until the bot gets this behavior?
I've fixed some edge issues I've run into over the past few days, but nothing that seems to be related to your issue, and looking through the code I couldn't find a situation that causes this issue.
Maybe the problem is the music source expires, songs downloaded by the Bot don't actually get downloaded, the Bot just takes a URL to a server and passes that URL to discord to play, as you can see in this code below.

Maybe after some time that URL can become invalid and the music actually takes 0 seconds to play.
Can you download the commits I made over the past few days and try to find your error again?
Just for you to better understand how the execution works, look for the PlayerProcess.py file in the Parallelism folders, in this class there are two functions that call each other, __playNext() and __playSong(song)
The second tells Discord to play the song and after finishing the song it calls the __playNext function, this checks if there is another song to be played and calls the __playSong function

https://rr1---sn-oxunxg8pjvn-4aol.googlevideo.com/videoplayback?expire=1674758495&ei=_3TSY7C0HYPD8wKyw56YCw&ip=2804%3A14d%3Abaa1%3A8c12%3A2636%3A107b%3Af404%3A2618&id=o-APoUozIQlZIgSbA4mDOMDQAKMzi-bF7v-jNbpCuKRsQf&itag=251&source=youtube&requiressl=yes&mh=Pz&mm=31%2C29&mn=sn-oxunxg8pjvn-4aol%2Csn-bg0eznsy&ms=au%2Crdu&mv=m&mvi=1&pl=48&gcr=br&initcwndbps=1078750&spc=H3gIhlsjDS8x1seE447mrUnFkJvDUpU&vprv=1&svpuc=1&mime=audio%2Fwebm&gir=yes&clen=4395259&dur=254.801&lmt=1565941960611701&mt=1674736382&fvip=2&keepalive=yes&fexp=24007246&c=ANDROID&txp=5431432&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cgcr%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhAIXfnAKt9ghFlPPt4iq5geWr9Wh6E9HjH_roioF2ibzeAiAZriZzkgf_PmZ5JU52RMm-I7VHHVN89Cq8JoW3jvV_dg%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIhAPNisP3pvaELL0ogZQf4_0h5NwnO2naFCn9lN4UL0eIKAiBRJrTn_-Nacz7W3ALQ526ahc1opsWn127bgvmxss4fUg%3D%3D
This is an example of a song.source, if you look closely, there's this parameter in the URL expire=1674758495, and if you convert that epoch value to a time, it's closer to 6 hours from now, so I think in 6 hours the Bot won't be able to play the songs, I'm going to try this theory and if it's true, I'm going to fix it.
Also, if you put this URL in your browser, you will hear a song

The problem is exactly with this url, i already uploaded a patch to fix this error, in this commit
If you got any other problems, don't mind asking

Thanks a lot!
Great jib!

authors: Paillat-dev RafaelSolVargas

repo: RafaelSolVargas/Vulkan


text:
No notification on Exited container.

Hi,
I was wondering if there is a way to notify on Exited containers in same way as is already possible for Stopped?
I have large-ish docker-compose vpn media server stack (https://paste.o5.ddns.net/?01c203401ee4a8ff#47Sw1AmHptxYVkFnc5Cs41B59QRRD5R5aDdzWN6jtwg3) with lots of container dependencies. After pc reboot 70% of the time it works every time, the remaining 30% a random number of the containers doesn't start and remains in Exited state.
Would love a way to get notified if this happens. Thanks for any insights.

Hello!
Do you have CONTAINERMON_CHECK_STOPPED flag set to true?
As far as I can tell exited counts as being stopped, so it should be picked up!

Yep I do, and I get notified when Stopped, but not when Exited. I think Exited results when the docker stack starts shutting down and reaches some timeout before all the containers gracefully stop and some containers get killed.
I can't reboot the pc now as folks are streaming, but next time it happens I'll try to see if there's anything in container-mon log.

That's really weird, I'm not sure what the difference in exited is supposed to be
I just tried bringing up a random container, then did docker stop <container_name> and checked it's status with docker inspect <container_name> which shows me:
"State": {
            "Status": "exited",
            "Running": false,

and docker ps -a:
CONTAINER ID   IMAGE           COMMAND       CREATED          STATUS
10989fd32808   ubuntu:latest   "/bin/bash"   14 seconds ago   Exited (0) 4 seconds ago

and container_mon notified me for that
Next time it happens could you send me the output for docker ps -a and the logs for docker inspect <container_name> for one of the exited ones you didn't get a notification for?

Will do, it might take some time, but the issue is not pressing. Only thing I can remember is It wasn't (0) after the Exited, I think it was (137) and I found this: https://betterprogramming.pub/understanding-docker-container-exit-codes-5ee79a1d58f6 which claims 137 "Indicates failure as container received SIGKILL (Manual intervention or ‘oom-killer’ [OUT-OF-MEMORY])" which sounds to me like it could be the kill when rebooting?

Hmm I tried it with docker kill too, same thing with status  Exited (137) 🤷♂

Oh boy do I feel stupid now. TL;DR Error was in my config
I duplicated the stack on a different vm, kept rebooting it until I got into Exited (the number was 128 though) ... and got notification within 3 minutes. It turns out I was under impression I use same settings for container-mon in all vm's, but for reasons I forgot just the one with the "issue" had checking stopped disabled.
Sorry for wasting your time!

Haha no worries, glad you got it working 😄

authors: RafhaanShah hugalafutro

repo: RafhaanShah/Container-Mon


text:
Completition and ChatCompletition stream_options

Feature Request
Add support of stream_options
Additional context
[Usage stats now available when using streaming](https://community.openai.com/t/usage-stats-now-available-when-using-streaming-with-the-chat-completions-api-or-completions-api/738156)
Thanks!

Duplicate of #285

authors: AissistmeMT StephenHodgson

repo: RageAgainstThePixel/OpenAI-DotNet


text:
Add new OpenAI embedding models + dimensions parameter

This PR adds the new OpenAI embedding models, as announced here:
https://openai.com/blog/new-embedding-models-and-api-updates
Also added the new dimensions parameter described in the announcement.

See comments

updated!


text:
🛑 Buildatscale is down

In d8f5b6a, Buildatscale (https://buildatscale.tech) was down:

HTTP code: 429
Response time: 140 ms

Resolved: Buildatscale is back up in f629281 after 6 minutes.


text:
Low performance in search with more 10.000 e-mails

RainLoop version, browser, OS: 1.9.4.402
Expected behavior and actual behavior:
Steps to reproduce the problem: Using e-mail survey of more than 10,000 e-mails in the box.
Logs or screenshots:

Hi
When doing research in e-mail lists and when it contains about 10,000 e-mails or more in the box, the system takes too long to perform the search.
Is there any way to improve is to research performance?
Thank's

This is not a rainloop problem, this is related to your dovecot (imap)
server.
Some of my clients have 100 000+ emails in their inbox and a full text
search takes less than 0.1seconds.
My personal inbox is 43 000 mails with a search taking 0.07sec
.    https://eXtremeSHOK.com     .
On 07-Oct-16 1:27 PM, marlon-micrologi wrote:

  RainLoop version, browser, OS: 1.9.4.402

  Expected behavior and actual behavior:

  Steps to reproduce the problem: Using e-mail survey of more than
  10,000 e-mails in the box.

  Logs or screenshots:

capturar
https://cloud.githubusercontent.com/assets/7561179/19188324/75b8b54a-8c67-11e6-8be0-45f4761968e8.JPG
Hi
When doing research in e-mail lists and when it contains about 10,000
e-mails or more in the box, the system takes too long to perform the
search.
Is there any way to improve is to research performance?
Thank's
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RainLoop/rainloop-webmail/issues/1211, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFrm0N1H0m8HXFuK-o8WMAzuMZTGa3V1ks5qxiy0gaJpZM4KQ7jb.

Take a look at Solr FTS: http://wiki2.dovecot.org/Plugins/FTS/Solr
We have mailboxes with a million messages and search takes a couple of seconds.

authors: azidhaka extremeshok marlon-micrologi

repo: RainLoop/rainloop-webmail


text:
🛑 RSOS服务控制面板 is down

In 3802e2a, RSOS服务控制面板 (https://$US_SITE) was down:

HTTP code: 502
Response time: 500 ms

Resolved: RSOS服务控制面板 is back up in ac53499 after 16 minutes.

repo: RainThings/uptime


text:
🛑 HK区服 is down

In 6eb3d2c, HK区服 (https://$HK_SERVER) was down:

HTTP code: 0
Response time: 0 ms

Resolved: HK区服 is back up in cbd12d5.


text:
Question about PropertyMethodExtensions.kt

DBFlow Version: 4.0.5
Issue Kind (Bug, Question, Feature): Question
Description: There are few infix property methods extensions implemented in PropertyMethodExtensions.kt such as eq, isNot, notEq  and etc. have non-null parameter. But the same functions in Property.java have @Nullable annotation for parameter.
For example eq function. Infix representation:
infix fun <T : Any> Property<T>.eq(value: T) = this.eq(value)

and source function:
@NonNull
@Override
public Operator<T> eq(@Nullable T value) {
    return getCondition().eq(value);
}

Question is why not to set nullability for infix function parameter too? Because currently compiler won't let us to do something like this:
var employee = (select from Employee::class where (Employee_Table.id eq employeeId)).result

cause employeeId is defined like var employeeId: Int? = null and can be null. Compiler shows the following error -
'infix' modifier is required on 'eq' in 'com.raizlabs.android.dbflow.sql.language.property.Property'.
But if we call eq without infix, all is working just fine:
var employee = (select from Employee::class where (Employee_Table.id.eq(employeeId))).result

good catch.

use d624e8f4 for version to see it in action.

Thanks!

authors: agrosner epetrenko

repo: Raizlabs/DBFlow


text:
[Style] i want to improve footer theme and link to there necessory pages

What's the style idea?
it improve user connectivity
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Pls provide the supporting screenshot of your issue @kunalmaurya6

background of the page and footer is looking the same which does not looking good

Go ahead @kunalmaurya6


text:
[Style] Guiding Buttons are not dynamic in hover effect.

What's the style idea?
We need to improve
The guiding buttons for directing to the next and prev buttons are not dynamic in hover effect when there is a change in the dark and llight theme.
And also I expect to style the buttons more modern and suitable for CalcDiverse.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a KWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100  Kindly assign this issue to me

Go ahead @isid555

PR update pls @isid555

Kindly review the PR
@Rakesh9100


text:
python3 not recognized as "python"

With my new computer I had been getting an error in the build process that "python" was not a recognized command
thread 'main' panicked at '

 couldn't find required command: "python"

  '

I was able to fix it by installing the package python-is-python3, but it would be incredibly helpful for other users if they didn't need to do so, or if the troubleshooting message was more informative.

This is probably an issue for upstream https://github.com/google/shaderc-rs.

authors: DimpyRed Ralith

repo: Ralith/hypermine


text:
Define local API version for PreferredDecision

This helps remove the dependency on multicloud-operators-placementrule dependency in go.mod. Enabling a more cleaner module for dependent components to import and use.
Fixes #1343

The change looks mostly safe to me.
If there is any consumer of DRPC status which is using an older version of the Ramen API, would they get an error or will the unmarshal happen seamlessly, I am not sure.
Another thing to check for is what happens in the upgrade path. Will the API server transition that status properly once the CRD is updated.
The json tags are the same so JSON to GO and vice-versa should be fine.
@ShyamsundarR If you can test a upgrade scenario that would be good. LGTM!

The change looks mostly safe to me.
If there is any consumer of DRPC status which is using an older version of the Ramen API, would they get an error or will the unmarshal happen seamlessly, I am not sure.
Another thing to check for is what happens in the upgrade path. Will the API server transition that status properly once the CRD is updated.
The json tags are the same so JSON to GO and vice-versa should be fine.
@ShyamsundarR If you can test a upgrade scenario that would be good. LGTM!

Upgrade using a DRPC from an older image to the newer image and CRD spec tested, and passed.

The change looks mostly safe to me.
If there is any consumer of DRPC status which is using an older version of the Ramen API, would they get an error or will the unmarshal happen seamlessly, I am not sure.
Another thing to check for is what happens in the upgrade path. Will the API server transition that status properly once the CRD is updated.
The json tags are the same so JSON to GO and vice-versa should be fine.
@ShyamsundarR If you can test a upgrade scenario that would be good. LGTM!

Upgrade using a DRPC from an older image to the newer image and CRD spec tested, and passed.

Thanks!

authors: ShyamsundarR raghavendra-talur

repo: RamenDR/ramen


text:
Feature request: Option to show or hide Smart Feeds group

Another idea: Personally, I never use Smart Feeds on any newsreader app I have used before, even though I like the idea. To save screen real estate, maybe consider having the option to not display the group at all (rather than just collapse it). Thanks again everyone!

I disagree. Yes, the latter would solve my specific problem, but its not a duplicate.
The state of disclosure should be preserved regardless of whether the folders can be hidden.

authors: githubnavigator peternlewis

repo: Ranchero-Software/NetNewsWire


text:
Scala Almond

is there a way to make this extension to work with Almond Scala ?

What is Almond Scala?
This extension is designed to work with any Jupyter notebook cell outputs in VSCode that produce json, csv, or geojson data.

Thanks. Almond is an scala kernel.
I was able to integrate up to a certain point. But i have some large csv do
you have a recomendation of a proper way to stream a big file ?
On Sun, Jan 23, 2022 at 10:23 Taras Novak @.***> wrote:

What is Almond Scala?
This extension is designed to work with any Jupyter notebook cell outputs
in VSCode that produce json, csv, or geojson data.
—
Reply to this email directly, view it on GitHub
https://github.com/RandomFractals/vscode-data-table/issues/112#issuecomment-1019518659,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAM3XTUW7TDAV6P6UNYHSTUXQTOTANCNFSM5MSVASNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

@orellabac I've done some preliminary work on large data streaming in this new Tabular Data Viewer extension recently:
https://github.com/RandomFractals/tabular-data-viewer
It's not for Jupyter notebooks, but you can try it with CSV files over 1GB in size.
New release is coming this week with Perspective view, data grouping, filtering, and basic charts.
That said I will close this request as I don't plan to provide tight integration for different Jupyter notebook kernels in this renderer extension at the moment.

authors: RandomFractals orellabac

repo: RandomFractals/vscode-data-table


text:
Tag Fixes

archive.createTag and archive.getAllTags require callbacks

archive.version uses ID, not name

Added some tests

Apparently this was already sorted by https://github.com/geut/hyperdrive-promise; it didn't seem to be.
Will investigate and see if something is wrong on my end.

I did a silly and changed two things with one.
Closing this one.

repo: RangerMauve/dat-fetch


text:
unable to init server: Could not connect: Connection refused

我在训练过程中，总是出现“unable to init server: Could not connect: Connection refused”这个报错。详细报错见下面。
/usr/local/lib/python3.6/dist-packages/pytorch_lightning/metrics/init.py:44: LightningDeprecationWarning: pytorch_lightning.metrics.* module has been renamed to torchmetrics.* and split off to its own package (https://github.com/PyTorchLightning/metrics) since v1.3 and will be removed in v1.5
"pytorch_lightning.metrics.* module has been renamed to torchmetrics.* and split off to its own package"
Unable to init server: Could not connect: Connection refused
Unable to init server: Could not connect: Connection refused
(train.py:34): Gdk-CRITICAL **: 10:35:14.994: gdk_cursor_new_for_display: assertion 'GDK_IS_DISPLAY (display)' failed
[root][06-29 10:35:15]INFO:Using Tensorboard, logs will be saved in workspace/nanodet_m_persion/logs
[root][06-29 10:35:15]INFO:Setting up data...
[root][06-29 10:35:15]INFO:loading annotations into memory...
我是在服务器上，通过创建docker容器来运行的。我发现创建docker容器时，如果不增加-v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=unix$DISPLAY -e GDK_SCALE -e GDK_DPI_SCALE这几个参数，那么训练nanodet时，就会停在“unable to init server: Could not connect: Connection refused”这个报错几分钟，之后才会继续输出训练打印信息。如果我增加了上述几个参数，虽然训练时，依然会有unable to init server: Could not connect: Connection refused报错，但是很快就会继续输出下面的打印，开始进行训练。
我知道这个几个参数都是和显示图片有关的命令，但是具体不知道为什么无论是否增加参数，都会输出unable to init server: Could not connect: Connection refused，而且，这个报错到底是和谁链接失败导致的。

nanodet 在训练时应该没有和GUI相关的调用

nanodet 在训练时应该没有和GUI相关的调用

这个警告每次都出来，(train.py:34): Gdk-CRITICAL **: 10:35:14.994: gdk_cursor_new_for_display: assertion 'GDK_IS_DISPLAY (display)' failed

Would you like to share your Dockerfile? @liuqinglong110

authors: RangiLyu imyhxy liuqinglong110

repo: RangiLyu/nanodet


text:
fix(readme): Typo 'ants' -> 'wants'

Thanks for all the cool work Danny 🙏
:)

Thanks.

authors: Rapptz TheOnlyWayUp

repo: Rapptz/eos


text:
dependency seems to cause segmentation fault

I'm collecting issues where users have started recently seeing segmentation faults here. For multiple it seems that installing gym 0.15.3 or 0.15.4 fixed the issue, but some of them seemed to have needed to change the sanic-plugins-framework as well. It's worth noting that this thread seems to indicate that the release of 0.15.6 could be causing the issue, but when I install all the dependencies on my Mac, I end up with gym 0.14.0. The issues seem to occur only on Ubuntu (maybe because it gets this newer version?), but on different versions and different platforms.
https://github.com/RasaHQ/rasa/issues/5196
https://github.com/RasaHQ/rasa/issues/5228
https://forum.rasa.com/t/segmentation-fault-core-dumped/4214
https://forum.rasa.com/t/rasa-shell-segmentation-fault/24485/3
https://forum.rasa.com/t/odd-problem-of-rasa-train-segmentation-fault-core-dumped/24695/4
https://forum.rasa.com/t/issue-running-rasa-on-google-cloud-compute-engine-instance/24683

Hello! Your post falls well because not able to install Rasa on my raspberry 4 4gb with Ubuntu (I created an exit for this problem), I tried to install it on a mini small laptop PC and the installation ended without error!
Only at launch I had an error:  (core dumped) . I can't launch.

@Suniron will you please create a forum post on the topic? It seems like it could be a different issue. Feel free to tag me in it there.

I'm going to create a subject right now. Thanks for the advice.

https://forum.rasa.com/t/ubuntu-18-04-64-bits-core-dumped-error-on-launch/24798 :1st_place_medal:

Original issue closed by #5248

authors: Suniron erohmensing

repo: RasaHQ/rasa


text:
Mark only part of words as entities (versus whole words)?

Rasa NLU version: 0.10.1
Used backend / pipeline: spacy_sklearn
Operating system: linux
If I mark only a part of a word as entity the python trainer returns a warning. Is this something that should work?
WARNING:rasa_nlu.extractors.crf_entity_extractor:Misaligned entity annotation in sentence 'Zeig mir Bilder unter einer CC0-Lizenz.'. Make sure the start and end values of the annotated training examples end at token boundaries (e.g. don't include trailing whitespaces).
This is the relevant part of the training file:
      {
        "text": "Zeig mir Bilder unter einer CC0-Lizenz.",
        "intent": "oer_search",
        "entities": [
          {
            "start": 9,
            "end": 13,
            "value": "Bild",
            "entity": "resource"
          },
          {
            "start": 28,
            "end": 31,
            "value": "CC0",
            "entity": "license"
          }
        ]
      },

OK, now I understand the entity value can differ from the selected text, so I modified the training file:
      {
        "text": "Zeig mir Bilder unter einer CC0-Lizenz.",
        "intent": "oer_search",
        "entities": [
          {
            "start": 9,
            "end": 15,
            "value": "Bild",
            "entity": "resource"
          },
          {
            "start": 28,
            "end": 31,
            "value": "CC0",
            "entity": "license"
          }
        ]
      },

And the python trainer no longer complains. :-)

here X is also an entity but i'm unable to extract it
pprint(interpreter.parse("Our yearly revenue has grown at a 50X rate to $22,500 over the last two years"))
{
"intent": {
"name": "revenue",
"confidence": 0.9284462221808045
},
"entities": [
{
"start": 23,
"end": 28,
"value": "increased",
"entity": "verb",
"confidence": 0.960265495627011,
"extractor": "ner_crf",
"processors": [
"ner_synonyms"
]
},
{
"start": 46,
"end": 47,
"value": "$",
"entity": "unit",
"confidence": 0.9452229431259471,
"extractor": "ner_crf"
},
{
"start": 47,
"end": 53,
"value": "22,500",
"entity": "value",
"confidence": 0.9166615710795952,
"extractor": "ner_crf"
},
{
"start": 63,
"end": 77,
"value": "last two years",
"entity": "time",
"confidence": 0.978303264391344,
"extractor": "ner_crf"
}
],
"intent_ranking": [
{
"name": "revenue",
"confidence": 0.9284462221808045
},
{
"name": "customers",
"confidence": 0.07155377781919549
}
],
"text": "Our yearly revenue has grown at a 50X rate to $22,500 over the last two years"
}

authors: pheraph saishaga


text:
🛑 Showcase Image Server DZI Array is down

In 7d55461, Showcase Image Server DZI Array (https://rationai-vis.ics.muni.cz/iipsrv-demo/iipsrv.fcgi?DeepZoomExt=CancerDetection/annotation.tif,CancerDetection/probability.tif,CancerDetection/explainability.tif.dzi) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Showcase Image Server DZI Array is back up in f2ff15a after 7 hours, 56 minutes.

repo: RationAI/demo-uptime


text:
🛑 GiteeSite is down

In 5f2cf94, GiteeSite (https://ravelloh.gitee.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: GiteeSite is back up in 019389f.

repo: RavelloH/sitemonitor


text:
Why static render methods?

Just curious why mercury component render methods are static since naively it would be convenient for a parent component to call foo.render() as an instance method. Is that mainly to enforce separation from the state object and discourage mutating local state?
Thanks very much for the excellent docs and examples. Mercury is an enlightening project.

You can definitely do that. I advocate separating render functions completely into their own files because

Hot reloading is easier to do
It makes it more difficult for the view to update model state in unexpected ways (the only sanctioned way is by binding channels to the DOM) There is always the possibility for local state in renderers, what you want is for the model to control access to itself via channels
Ideologically, model and view should be separate. There are many ways one might want to render a boolean state; so instead of duplicating the state in several components (tickbox component, switch component), separate state and renderer (boolean state + tickbox renderer or switch renderer)

2 and 3 are just my opinion. Raynos does seem to lean more componentwards in some of the examples, for example how most render methods are attached to an object, not completely free.

@shoover  is not a requirement of the library just a way in which Raynos do the examples,  because he likes more functional programming,  in your I see that you want to use and OO paradigm, a option then is create local state in each component, at attach them to his parent component state,  and the parent state to its parent-parent state, and soon on till the root State (making you State tree in the process) when the class are loaded, using observ-struct, observ-array, or observ-varhash. if you are careful and understand the code that  is really small and easy to comprehend, you will archive the same using the OO paradigm.
this approximation is good  for me because In my case when I think about  the model of the app, I relate it more with the db data (users, tasks, messages, etc. very similar and what you should have in a db).  the ui for me can represent  that data in different ways, so the state of db-data like should not be tied to a UI component. and the Ui component should has its own state. (is just my approximation)
I archive this in this way: ( Warning: this is pseudo code can contains  many errors)
    // Create global variable that you will use  ref the db models
    GLOBAL_VARIABLE = {};

    // load your model here, you can define all your model structure like  you do when use db- schemes
    // and use observ-struct (prefer this), or be more be scheme less and use observ-varhash
    // and a reference to the db model in the global variable

    GLOBAL_VARIABLE.model = mercury.varhash({});

    // load the ui classes tree
    var ui = new UI()

    // Create the app state
    var appState = mercury.struct("model": GLOBAL_VARIABLE.model, "ui": ui.state);

    mercury.app(document.body, appState, ui.render);

    // is important that you UI.render has the state as argument,  you just can ignore it after, because you
    // are using the component state for render

    class UI{
         constructor():{
                state = {}
                //load your component 1
                this.c1 = Component1();
                state["c1"] = c1.state
                // load your component n
                this.cn = Component_n();
                state["cn"] = cn.state

               //the magic occurs here
              this.sate = mercury.struct(state);
         }
        render(state):{
           //just ignore the state
            return mercury.h("div", {}, this.c1, ...., this.cn);
         }
    }

For this has effect the child state should be Observ-values, Obsev-Arrays, Observ-Struct, or Observ-Varhash, or some similar object.

you can access to you db model from the component using the global variable.

also I will recommend you use EventEmmiter or EventEmmiter2 or any similar thing for communication between components ( to  communicate events-things-stuff like Modal.onClose), just avoid modify  state outside of its  component.

Sorry for my broken English I hope that this example can help you.

Thank you, @eightyeight and @criloz, for the helpful explanations. I can see there is room for changing designs to suit the application but the examples seem to do it one particular way that ensures a functional perspective. I see this separation enforced in many Elm designs, as well.

authors: criloz eightyeight shoover

repo: Raynos/mercury


text:
C-API - crop the mastered image

These are the data structures that control the size of the raster image.
Lets say, I wish to fit the image to 1000x1000 but only wish to render a window of 100x100.
Would that be possible ?
typedef enum resvg_fit_to_type {
    RESVG_FIT_TO_ORIGINAL, /**< Use an original image size. */
    RESVG_FIT_TO_WIDTH, /**< Fit an image to a specified width. */
    RESVG_FIT_TO_HEIGHT, /**< Fit an image to a specified height. */
    RESVG_FIT_TO_ZOOM, /**< Zoom an image using scaling factor */
} resvg_fit_to_type;

typedef struct resvg_fit_to {
    resvg_fit_to_type type; /**< Fit type. */

    /**
     * @brief Fit to value
     *
     * Not used by RESVG_FIT_TO_ORIGINAL.
     * Must be >= 1 for RESVG_FIT_TO_WIDTH and RESVG_FIT_TO_HEIGHT.
     * Must be > 0 for RESVG_FIT_TO_ZOOM.
     */
    float value;
} resvg_fit_to;

Do you want to crop the final image or do you want to skip rendering of the other tiles altogether?

If the final image has to be cropped, its more efficient to skip rendering of the other tiles. At the current moment, I can crop the final image, I'm curious if a more efficient way can be used.

I guess you are looking for #354

OK. So, no support ATM. I am not versed with rust so I cannot work on render.rs - thanks.

authors: RazrFalcon zcream

repo: RazrFalcon/resvg


text:
Followers  shows wrong content.

fixed

authors: Razviar apoapostolov

repo: Razviar/mtga-pro-tracker


text:
[FrontEnd] Implement User Profile page after the login

Create the user profile and information pages that display after a user logs in.

As a user I Want a profile page after logging in. So That I can view and manage my information and see relevant data visually.

#42

repo: RbAvci/Read-me-Hire-me

license: ISC


text:
[BUG] Tooltip doesn't show when generating anchorSelect dynamically

Describe the bug
Tooltip doesn't show when generating anchorSelect dynamically
Version of Package
v5.10.0
To Reproduce
Create a list of items where each one has a different tooltip and h-generate the id and anchorId dynamically
    <Body weight="semiBold" size="small" id={ '#teammate-${props.teammate.id}-emails-tooltip' }>         { props.teammate.name }     </Body>     <ReactTooltip render={ emailTooltipContent } anchorSelect={ '#teammate-${props.teammate.id}-emails-tooltip' } />
Expected behavior
The tooltip should be shown
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information if possible or delete this section):

OS: MacOS
Browser: Chrome
Version: 110.0.5481.177
Frameworks: React 18

Additional context
It's working when adding a static id, but every line of the table show the same tooltip content (the last one)

I created a code sandbox to reproduce the issue:
https://codesandbox.io/s/hopeful-bhabha-w7nqho?file=/src/App.js

Actually, it's working on the code sandbox, it seems to be related to my application.
Any idea of what could produce this issue?
When I load the page, sometimes, some are showing, but the majority of tooltip are not working :(

It seems I found the issue, I was generating an uuid on the fly as id.
Adding some text in front of it fixed the issue. Weird :)

@vddgil just a suggestion, if you're using React 18, you can test the useId from React

authors: danielbarion vddgil

repo: ReactTooltip/react-tooltip


text:
redirect from redux action in react router v4[Question]

I have used react-router v4 for routing in my application. In homepage, there is a form. When user fills up form and hits the submit button, then the action is dispatched(showResultofCar) and it should be redirected to result page which is not a child in the homepage instead it is a different page with different UI from top to bottom.
I tried to do this way but the action is not dispatched only the routing has been transitioned but shows the same homepage instead of new page(result)
**index.js**

    ReactDOM.render(
      <Provider store={createStoreWithMiddleware(reducers)}>
        <ConnectedIntlProvider>
          <Router>
            <App />
          </Router>
        </ConnectedIntlProvider>
      </Provider>
      , document.querySelector('.app'));

**app.js**

    render() {
      return (

             this.showModal(e)}
              hideModal={() => this.hideModal()}
              show={this.state.show}
              onHide={() => this.hideModal()}
            />
              <Banner />
              <Media />

            );
    }

**form.js(it is a child component of banner which is a child component of app)**

    onSubmit = (e) => {
      e.preventDefault();
      const originwithCountry = e.target.Origen.value;
      const originwithCity = originwithCountry.split(', ')[0];
      const cityFrom = base64.encode(originwithCity);
      const startDate = (new Date(e.target.startDate.value).getTime() / 1000);
      this.props.showResultofCar(cityFrom, cityTo, startDate);
      this.context.router.transitionTo('/result');
      }

    render() {
      const { focusedInput } = this.state;
      const { intl } = this.props;
      return (

            <form className="form-inline" onSubmit={this.onSubmit}>

                <Field
                  name='Origen'
                  component={renderGeoSuggestField}
                />

                <Field
                  name="daterange"
                  onFocusChange={this.onFocusChange}
                />

              <Link to="/result">
              <button type="submit" className="btn btn-default buscar">
                { intl.formatMessage({ id: 'buscar.text' })}
              </button>
            </Link>
            </form>

      );
    }

**result-parent.js**

    class ResultParent extends Component {
      render() {
        return (

            <Match pattern='/result' component={Result} />

        );
      }
    }

**result.js**

    class Result extends Component {
    render() {
      return (

          <ResultNav />
          <Filtering />
          <Results />

      );
    }
    }

**actions/index.js**

    export function showResultofCar(cityFrom, cityTo, date) {
      return (dispatch) => {
        dispatch({ type: 'CAR_FETCH_START' });
        const token = localStorage.getItem('token');
        console.log('date', date);
        return axios.get(`${API_URL}/car/{"cityFrom":"${cityFrom}","cityTo":"${cityTo}","date":${date}}.json/null/${token}`)
          .then((response) => {
            console.log('response is', response);
            dispatch({ type: 'CAR_FETCH_SUCCESS', payload: response.data });
          })
          .catch((err) => {
            dispatch({ type: 'CAR_FETCH_FAILURE', payload: err });
          });
      };
    }

My way is not working. How can i now redirect using react router v4 inside action?
Also i don't want the result to be shown inside App component(parent) because result page will be completely different with its own navbar,filtering and results option.

Have your App component grab the router off of context and make it available in your store somehow. Best bet is using a middleware to intercept those kinds of actions.

Are there any examples of how to do this? I haven't had much luck. In my situation, I rely on forms for navigating a website, not just links.
I don't think using context is an appropriate option. Facebook certainly doesn't recommend it. "If you want your application to be stable, don't use context."

I'm in the same boat. I need to push a new route to history inside my redux actions. What do we do about this? It was so much easier with Router v3. :/

I found the answer:
https://github.com/ReactTraining/react-router/tree/master/packages/react-router-redux
Worked perfect for me. With a Redux action, just use dispatch(push('path'));

@chbrules I also get trouble, I use react-router-redux, but push can not work in action, url can change ,but component not.
When I use store.dispatch(push('path')) in store.js,  it works. but in action.js, it can not work.

@yujiayinshi  Did you include 'push' in your action.js?
import { push } from 'react-router-redux'
Also, you have to setup a reducer for it in your root reducer:
`import { routerReducer } from 'react-router-redux'
const rootReducer = combineReducers({
router: routerReducer,
other reducers...
});
export default rootReducer;`

@MilanRgm Did you ever find a suitable solution for this matter?  I have now encountered the problem and tried @chbrules proposed solution to no avail.
"react-router-dom": "^4.1.2"
"react-router-redux": "4.0.8"

@chbrules @MilanRgm @timdorr Any other advice? dispatch(push('/path')) changes the URL in the browser but does not render the page with the new container associated with that path. routerReducer is included. Seems like this should be a pretty basic and straightforward feature as it is a universal design pattern to redirect after a completed action.

Little old thread, but I found it easy to pass this.props.history to the action. From there you can do a simple history.push('/').  For this the component needs to be wrapped in a connect

authors: MilanRgm amboy00 chbrules mindtonic pvanwoensel-lv skiingandhiking timdorr yujiayinshi

repo: ReactTraining/react-router


text:
Switch depth

Is is possible to rethink Switch to work like this?
<Switch>
  <PublicLayout >
    <Route
      path='/'
      component={Login}
    />
    <Route
      path='/recover'
      component={PasswordRecovery}
    />
  </ PublicLayout>
  <AdminLayout >
    <Route
      path='/admin'
      component={Dashboard}
    />
  </ AdminLayout>
  <Route
    component={NoMatch}
  />
</Switch>

Currently needs that its children be routes.

The way I would have that is like this:
<Switch>
  <Route path='/admin' component="AdminLayout/>
  <Route path='/' component={PublicLayout} />
  <Route component={NoMatch} />
</Switch>

AdminLayout
  <AdminLayout >
<Switch>
    <Route
      path={path}
      exact
      component={Dashboard}
    />
  <Route
    component={NoMatch}
  />
</Switch>
</AdminLayout>

Public Layout:
<PublicLayout >
    <Route
      path={path}
exact
      component={Login}
    />
    <Route
      path={`${path}/recover}
      component={PasswordRecovery}
    />
  <Route
    component={NoMatch}
  />
  </Switch>
</ PublicLayout>

No, that would not work. The <Switch> works by iterating over its children prop and attempting to match each one against the current location.pathname until a match is found. Think of a switch more along the lines of an array of route config objects:
<Switch routes={[
  { path: '/', exact: true, component: Login },
  { path: '/recover', component: PasswordRecovery },
  { path: '/admin', component: Dashboard },
  { component: NoMatch }
]} />

authors: Lube kwelch pshrmn


text:
Add nullability annotations to RACSubscriber

In general, it seems helpful to have nullability annotations specified within RACSubscriber.h, though perhaps there's a strong reason for which this file hasn't already been annotated that I'm not aware of.
However, according to SE-0140:

when an Optional is intentionally passed into Objective-C as a nonnull object, we should bridge some value by bridging the wrapped value, and bridge nones to a singleton such as NSNull

With SE-0140, which has already been implemented in Swift 3.0.1 (available in the Xcode 8.1 beta 3), it is important that nullability annotations be specified on RACSubscriber. Otherwise, when RACSubscriber.sendNext() is passed an Optional.none from within Swift, Swift will (post Swift 3.0.1) be bridge this to NSNull, when previously (pre-Swift 3.0.1) it was visible within the Objective-C sendNext: implementation simply as nil. This results in a change in behavior when ReactiveObjC is compiled with the Swift 3.0.1 compiler
@raylillywhite and I observed this change in behavior in Xcode 8.1 beta 3, and have verified that adding nullability annotations restores the previous behavior that users of ReactiveObjC are relying on.
Here's the difference in the Swift generated interface before and after this change:
Previous Swift generated interface (from Xcode 8.1 beta 3 with Swift 3.0.1)
public protocol RACSubscriber : NSObjectProtocol {

    public func sendNext(_ value: Any!)

    public func sendError(_ error: Error!)

    public func sendCompleted()

    public func didSubscribe(with disposable: RACCompoundDisposable!)
}

New Swift generated interface (from Xcode 8.1 beta 3 with Swift 3.0.1):
public protocol RACSubscriber : NSObjectProtocol {

    public func sendNext(_ value: Any?)

    public func sendError(_ error: Error)

    public func sendCompleted()

    public func didSubscribe(with disposable: RACCompoundDisposable)
}

Relatedly, there seems to be a bug or limitation in the Swift compiler that causes ReactiveObjCBridging's RACSignal.toRACSignal() function to bridge Optional.none to NSNull (in next events' values) even after added the nullability annotation to -[RACSubscriber sendNext:]. @raylillywhite and I are in the process of investigating this and have created ReactiveCocoa/ReactiveObjCBridge/pull/5 to address this.

👍 @mdiep I just amended my commit to use NS_ASSUME_NONNULL_* as suggested.

@erichoracek is correct. NS_ASSUME_NONNULL_* should only have replaced the _Nonnull uses.

Yep, good catch @erichoracek. Too much rushing on my part. Updated.

Thanks!

authors: mdiep sibljon

repo: ReactiveCocoa/ReactiveObjC


text:
fix Error on Xcode 15

Replace deprecated methods with UIAlertController.
comment Deprecations+Removals.swift
reformat code

Sorry guys, I wanted to merge it to my repo because the reformat change too many code, and this change do not have enough test,  but I was not familiar with fork operations and opened it by mistake. If you think this request is needed please inform me


text:
Comprehensive marble tests for switch

Should duplicate the current RxJS 3/4 tests
Should adhere to our contribution guidelines and operator authoring docs.

This issue can be closed.
PR #467 added the tests that were missing.

authors: blesh staltz

repo: ReactiveX/RxJS


text:
combineLatest without project returns mutating array.

We have been using combineLatest in combination with distinctUntilChanged and ran into an issue where the array returned from combineLatst was mutation and reused, resulting in distinctUntilChanged comparing the array against itself.
Unsafe:
console.clear();

let first$ =  Rx.Observable.interval(500).take(3).map(i => `A${i}`);
let second$ =  Rx.Observable.interval(500).take(3).map(i => `B${i}`);
let third$ =  Rx.Observable.interval(500).take(3).map(i => `C${i}`);

let combined$ = Rx.Observable
          .combineLatest(first$, second$, third$)
          .debounceTime(1)
          .flatMap(state => Rx.Observable.interval(10).take(2).map(i => state))
          .distinctUntilChanged(x=>x, (a, b) => {
            console.log("A:" + a);
            console.log("B:" + b);
            return a !== b;
          });

combined$.subscribe(console.log);

Safe:
console.clear();

let first$ =  Rx.Observable.interval(500).take(3).map(i => `A${i}`);
let second$ =  Rx.Observable.interval(500).take(3).map(i => `B${i}`);
let third$ =  Rx.Observable.interval(500).take(3).map(i => `C${i}`);

let combined$ = Rx.Observable
          .combineLatest(first$, second$, third$, (a, b, c) => [a, b, c]))
          .debounceTime(1)
          .flatMap(state => Rx.Observable.interval(10).take(2).map(i => state))
          .distinctUntilChanged(x=>x, (a, b) => {
            console.log("A:" + a);
            console.log("B:" + b);
            return a !== b;
          });

combined$.subscribe(console.log);

I would expect both snippets to give the same result, as the project function from combineLatest is optional.
Ps. @babeal brought up this snipped on gitter on 28 juli 2016 19:07.

I assume this is dupe of issue https://github.com/ReactiveX/rxjs/issues/1686 with PR https://github.com/ReactiveX/rxjs/pull/1841?

Looks very similar.

Closed by #1841

authors: Dorus blesh kwonoj

repo: ReactiveX/rxjs


text:
对12.2.2节allocator类例子的疑惑

#include <iostream>
#include <string>
#include <memory>
#include <typeinfo>
using namespace std;
int main()
{
    char *cp = "helloworld";
    allocator<string> alloc;  //可以分配string的allocator对象
    auto const p = alloc.allocate(10);  //分配10个未初始化的string,返回分配的内存的首地址
    auto q = p;
    alloc.construct(q++,10,'c');
    cout<<typeid(p).name()<<endl;
    cout<<p<<" "<<q<<endl;
    cout<<*p<<endl;
    cout<<*q<<endl;
    cout<<cp<<endl;
    return 0;
}

1.我查资料发现allocate返回内存的首地址，传递给q的时候为什么成了“q指向最后构造的元素之后的位置”
2.为什么要把p传递给q，而且p和q表示的地址不一致
我的环境是win7 32位 + codeblocks带有-std=C++11。

。。。哦，原来答案就在下面，我还一直在上面想，没想明白，看到下面的话我明白了。

repo: ReadingLab/Discussion-for-Cpp


text:
Display the Authentication Response

Display the username and profile from GitHub authentication response on user profile screen .

Done with console log.

its done

repo: Real-Dev-Squad/mobile-app


text:
Dev to Main Sync

Date: 15-10-2024

Developer Name: vinit

Issue Ticket Number

#822
#880
#876
#890

PR's going in this sync

#850
#881
#878
#889

Here is screenshot it is working properly over staging.

authors: Achintya-Chatterjee PunditWhoCodes

repo: Real-Dev-Squad/website-dashboard


text:
Task Card Title Display Enhancement

Closes : #565
Description:
This pull request addresses the issue of task card titles not being displayed correctly when the title is long and wrapping to multiple lines. The proposed changes aim to improve the task card title display by ensuring that long titles are shown in a single line with an ellipsis (...) to indicate that the title has been truncated.
Changes Made:

Updated the CSS styles for task card titles to enforce a single line display.
Added the text-overflow: ellipsis property to truncate long titles with an ellipsis when they exceed the available width.

After Changes
https://github.com/Real-Dev-Squad/website-status/assets/70854507/e51cac31-de91-489a-95e9-28db10d09209

As discussed please add tooltip for the ellipsis.

As discussed please add tooltip for the ellipsis.

Currently we don't have a design for tooltip so i can't add it.

In that case can you create a ticket for adding tooltip and mention there that design is awaited. and add that ticket link here.

As discussed please add tooltip for the ellipsis.

Currently we don't have a design for tooltip so i can't add it.

In that case can you create a ticket for adding tooltip and mention there that design is awaited. and add that ticket link here.

Issue ticket for add tooltip - https://github.com/Real-Dev-Squad/website-status/issues/570

authors: heyrandhir sahsisunny

repo: Real-Dev-Squad/website-status


text:
Set up Prettier CLI for consistent formatting of code

We need the whole codebase to have consistent formatting of the code, otherwise every developer will have their formatter messing up with the formatting of the code
This documentation would help: https://prettier.io/docs/en/install.html
Also, we're having it set up on welcome-site. You can refer: https://github.com/Real-Dev-Squad/website-welcome/pull/120

Remember, DO NOT make any other change in the codebase than setting up formatter and changing formatting

repo: Real-Dev-Squad/website-www


text:
Define priority semantics

The priority field is currently an opaque positive integer. As currently implemented:

"Higher priority" (ie more urgent) is encoded as a greater priority value
Tasks which don't define a priority are set as 0.

This presents a few issues:

The maximum value is unbounded, which could cause issues with different database backends
It's not possible to define a task as being lower priority than the default (without defining your own default and using it across your application (which doesn't work for external tasks))

Therefore, the scale needs to be better defined:

How many increments are needed? 10, 50, 100, 1000 etc
How are lower-than-default priority tasks encoded? Use 0 as the default and support negative? Set the default as half way through the range?

I would suggest a default of zero, a range of -128 to +127, and also define some constants, e.g.:
    class Priority(IntEnum):
        LOWEST = -128
        LOWER = -96
        LOW = -48
        DEFAULT = 0
        HIGH = 48
        HIGHER = 96
        HIGHEST = 127

and then people can use the constants to increase the likelihood of priorities set by different packages being suitable when used together, and also they can say things like Priority.HIGH + 1 to mean "a little bit higher priority than HIGH but not as high as HIGHEST".
(One might reasonably say "why would anyone care if a value fits in 8 bits in the modern world" but one might also reasonably reply "why on earth would you need more than 256 different priorities?". There are lessons to be learned I think from things like CSS z-index where people end up using ridiculous values like 999999999 because there are no guidelines whatsoever as to what values to use - there's no way of saying what you actually mean rather than just picking a random number and hoping.)
Also I don't think the DEP defines what the priority actually means. Presumably it means "while tasks of priority X are ready to run, no tasks with priority less than X will be run, no matter how old/overdue they are"? This should probably be made explicit.

Priority is indeed the biggest measure - not how overdue it is.
I'd not thought of constants - I really like this idea, even if just as a basis. Having 0 in the middle is definitely the way I'm leaning.
256 priorities ought to be enough for anyone (famous last words). Unbounded is going to almost certainly cause issues with other backends. At least with a fixed range it's possible to re-divide that to any smaller range (albeit losing some resolution). Even with the above, you could argue 7 steps is probably enough - fine-grained priorities could end in bike-shedding. I'd be tempted to use 100 either side instead, simply because as humans it's easier to reason about, and the extra 56 priorities probably aren't useful to most.

authors: RealOrangeOne jribbens

repo: RealOrangeOne/django-tasks


text:
crash

Issue description
i cant even enter
oritech.txt
my world but the issue seems to be with the coal generator and oritech machines ticking
oritech.txt
Steps to reproduce
i just placed these oritech machines assembler,foundry,lava generator,centrifuge, and a pulvirizer
and 4 coal generator for power form the mod wormholes i dont know a good power option right now that would help.
Also when ever i exit fullscreen alt-tab the game crashes
Minecraft version
1.21.1 (Latest)
Fabric loader version
0.16.5
Fabric API version
fabric-api-0.102.0+1.21
Oritech version
oritech-0.9.0
Other relevant versions
the crash error said oritech,wormholes,fabric api lookup,and the conif and library needed for the wormhole mod wich are SuperMartijin642
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
https://pastebin.com/RCDpiZgy

How exactly did you achieve this? One of the machine cores seems to think it's connected to the coal generator, which can't ever happen without some weird worldfile manipulation or something. Oritech also has a basic generator which can burn coal btw.

Yeah i dont know how it happaned buy i just delted and reisntaled the mod that adds coal generator and it dwlted the generators so now i can llay

How exactly did you achieve this? One of the machine cores seems to think it's connected to the coal generator, which can't ever happen without some weird worldfile manipulation or something. Oritech also has a basic generator which can burn coal btw.

i think i know the issue the carryon mod lets u move the machines with multiblocks and it doesnt pick up the multiblocks since it a multiblock but the machine cores remain in their old place when you pickup the machine i can send a video to you with mediafire since the size is big even tho its 43 seconds long
https://www.mediafire.com/file/488wayuc2gu9nlw/Java-runtime-delta+2024.10.31+-+14.40.35.10.mp4/file

repo: Rearth/Oritech


text:
our server does not load spec file

Hello!
I am trying to view our redoc spec with the index.html hosted on our site but it is not rendering.  seems like it can't pull in the swagger.yml file.  any ideas?

Draft Orders API Specification

<!--
ReDoc uses font options from the parent element
So override default browser styles
-->
<style>
  body {
    margin: 0;
    padding: 0;
  }
</style>

<!DOCTYPE html>
<html>
  <head>
    <title>Draft Orders API Specification</title>
    <!-- needed for adaptive design -->
    <meta name="viewport" content="width=device-width, initial-scale=1">

    <!--
    ReDoc uses font options from the parent element
    So override default browser styles
    -->
    <style>
      body {
        margin: 0;
        padding: 0;
      }
    </style>
  </head>
  <body>
    <redoc spec-url="https://andrewjtech123.github.io/imagehost/swagger.yml">
    </redoc>
    <script src="//rebilly.github.io/ReDoc/releases/latest/redoc.min.js"> </script>
  </body>
</html>

@andrewjtech123  What is printed to browser console?

Hi.  I think we fixed the issue.  Our local jekyll server runs without https.  When I run it locally I have to append https to the url, in order to view the Redoc output correctly.
One other related question - is it possible to tun off the swagger warnings from outputting to redoc?

Yes - use suppress-warnings: docs
Example:
<redoc spec-url="..." suppress-warnings></redoc>

@andrewjtech123 can this issue be closed?

@andrewjtech123 closing this issue for now but feel free to reopen in case it is not solved

repo: Rebilly/ReDoc


text:
Fix bug updating images version > 1

Description
Fix bug updating images version > 1
Fixes #THEEDGE-1481
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I run go fmt ./... to check that my code is properly formatted
[x] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #509 (0dff812) into main (dd4dd2c) will increase coverage by 0.06%.
The diff coverage is 22.85%.

@@            Coverage Diff             @@
##             main     #509      +/-   ##
==========================================
+ Coverage   31.21%   31.28%   +0.06%
==========================================
  Files          33       33
  Lines        3232     3235       +3
==========================================
+ Hits         1009     1012       +3
  Misses       2126     2126
  Partials       97       97

Impacted Files
Coverage Δ

pkg/services/images.go
12.05% <15.62%> (+0.11%)
:arrow_up:

pkg/routes/images.go
28.32% <100.00%> (ø)

pkg/services/services.go
100.00% <100.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd4dd2c...0dff812. Read the comment docs.

authors: biancarosa codecov-commenter

repo: RedHatInsights/edge-api


text:
Add link to package total and additional list in image detail page

Signed-off-by: Adelia Ferreira tay.figueira@gmail.com
Description
Add links to show total and additional packages in image detail page
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I run npm run lint:js:fix to check that my code is properly formatted

Codecov Report

Merging #282 (a0eb8b4) into master (9cd5bf8) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #282      +/-   ##
==========================================
- Coverage   31.41%   31.38%   -0.04%
==========================================
  Files         102      102
  Lines        2085     2087       +2
  Branches      599      599
==========================================
  Hits          655      655
- Misses       1312     1314       +2
  Partials      118      118

Impacted Files
Coverage Δ

src/Routes/ImageManagerDetail/ImageDetailTab.js
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9cd5bf8...a0eb8b4. Read the comment docs.

authors: adeliaferreira codecov-commenter

repo: RedHatInsights/edge-frontend


text:
Wizard: change the image name

fixes #1196
this commit update the wording on image name step
when user create new image.

Perfect! Just two small nitpicks and a then a rebase

Looks good! It only needs a small lint fix :)

@jrusz Changes in this PR will affect most of the texts on the "Name image" step. Could that cause any problems with the integration tests?

@jrusz Changes in this PR will affect most of the texts on the "Name image" step. Could that cause any problems with the integration tests?

So the tests don't seem to care as they don't really check if they're on the "Image name" step :smile: ... But I am going to prepare an update regardless. Thanks for bringing this to my attention @regexowl :slightly_smiling_face:

Btw, what is that description good for? I don't see it anywhere after I create the image...

Thanks for checking this @jrusz!
The description should be an optional field for adding more information about the image. For now it should show on the Review step when recreating it as far as I can tell from the mocks.

Thank you @mgold1234, this looks great! Can you please rebase and update the image name helperText with "The image name can be 3-63 characters long. It can contain lowercase letters, digits and hyphens, has to start with a letter and cannot end with a hyphen."? We've recently updated the name validation :)

authors: jrusz mgold1234 regexowl

repo: RedHatInsights/image-builder-frontend


text:
test/fixtures: Update timestamps of mock composes

The format of the timestamps of mock composes was causing an error in msw browser after merging https://github.com/RedHatInsights/image-builder-frontend/pull/1137
This updates the mock timestamps to the format of the actual ones fetched from the API.

Would it make sense to update timeUtilities to Typescript? Would that have helped catch this bug? Could be in a follow-up PR, of course.

Actually thinking about it more - what would be the scenario when we would need to check for the right format of the date? I mean the dates we are fetching from API will be always what we need and the only problem are our fixtures. Maybe we could put together validator just for those? But at the same time, how often do we change them?
Sorry, not sure if that makes sense. Just thinking about real usage of the validator.
Also weird that the tests weren't screaming even though the mocked data crashed the page in browser.

authors: lucasgarfield regexowl


text:
Mention VirtualBox port forwarding

The run.sh script spawns a Docker container with the app, that is exposed on port 1337. If your Docker machine runs in VirtualBox, like if using Mac or Windows, it is necessary to set up NAT port forwarding in VirtualBox too. It can be done by a command like this [1]
VBoxManage modifyvm "default" --natpf1 "insights,tcp,,1337,,1337"

or in the VirtualBox machine settings UI.
I’d put this information to the README as it might not be obvious if you’re not familiar with Docker.

Hey, can you make a note/section in the README and submit a PR?

Sure! #4

authors: Glutexo iphands

repo: RedHatInsights/insights-proxy


text:
Documentation for components: 3rd part

Description
Documentation for components: 3rd part
Type of change

Documentation update

Testing steps
N/A
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Base: 60.56% // Head: 60.56% // No change to project coverage :thumbsup:

Coverage data is based on head (cb8277f) compared to base (19e2c75).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #974   +/-   ##
=======================================
  Coverage   60.56%   60.56%
=======================================
  Files          23       23
  Lines        3231     3231
=======================================
  Hits         1957     1957
  Misses       1053     1053
  Partials      221      221

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter tisnik


text:
Revert "Revert "Refs #RHINENG-2640 - set reporter to "discovery" or "…

…satellite" (#71)" (#75)"
This reverts commit ab64ae18fdfeefebe675c2eecbc8f67206b11a05.
Redo this change to main branch.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.79%. Comparing base (77f3398) to head (12d11c7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #78   +/-   ##
=======================================
  Coverage   78.79%   78.79%
=======================================
  Files          23       23
  Lines         613      613
=======================================
  Hits          483      483
  Misses        130      130

Flag
Coverage Δ

unittests
78.79% <100.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

/retest

/retest

fixed pr checks -  https://gitlab.cee.redhat.com/insights-qe/iqe-foreman-rh-cloud-plugin/-/merge_requests/174/diffs

Many thanks @Aurobinda55 for the fix.

@JoySnow - this change looks good to me

authors: Aurobinda55 JoySnow codecov-commenter xiangce

repo: RedHatInsights/yuptoo


text:
Move reuse resource and skip teardown to deploy/clean_up (#637)

Move reuse resource and skip teardown to deploy/clean_up

Minor fix

Add return self.deploy()

!verified

repo: RedHatQE/openshift-python-wrapper


text:
create(): Fix initial_resource_version call

initial_resource_version use get and this fail when it's done with user without permissions

Fix for https://github.com/RedHatQE/openshift-python-wrapper/issues/997

!verified

!cherry-pick v4.12


text:
Add documentation for cluster scale-up

Super easy, but good to have it noted:
## You should permanently set count_bootstrap=0 by updating your vars.tf, but for the sake of time:
terraform apply -var="count_compute=4" -var="count_bootstrap=0" --auto-approve

Once the node has been provisioned, approve the pending CSR as cluster admin:
oc get csr -oname | xargs oc adm certificate approve

Once CSR is approved, you can quickly see the node appear, first NotReady then Ready:
[jrmorgan@localhost terraform]$ oc get nodes
NAME                        STATUS     ROLES    AGE   VERSION
master-0.og.pkt.shifti.us   Ready      master   35h   v1.17.1+912792b
master-1.og.pkt.shifti.us   Ready      master   35h   v1.17.1+912792b
master-2.og.pkt.shifti.us   Ready      master   35h   v1.17.1+912792b
worker-0.og.pkt.shifti.us   Ready      worker   34h   v1.17.1+912792b
worker-1.og.pkt.shifti.us   Ready      worker   34h   v1.17.1+912792b
worker-2.og.pkt.shifti.us   Ready      worker   34h   v1.17.1+912792b
worker-3.og.pkt.shifti.us   NotReady   worker   3s    v1.17.1+912792b
[jrmorgan@localhost terraform]$ oc get nodes
NAME                        STATUS   ROLES    AGE     VERSION
master-0.og.pkt.shifti.us   Ready    master   35h     v1.17.1+912792b
master-1.og.pkt.shifti.us   Ready    master   35h     v1.17.1+912792b
master-2.og.pkt.shifti.us   Ready    master   35h     v1.17.1+912792b
worker-0.og.pkt.shifti.us   Ready    worker   34h     v1.17.1+912792b
worker-1.og.pkt.shifti.us   Ready    worker   34h     v1.17.1+912792b
worker-2.og.pkt.shifti.us   Ready    worker   34h     v1.17.1+912792b
worker-3.og.pkt.shifti.us   Ready    worker   2m45s   v1.17.1+912792b

Need to link to docs regarding scale-up after the initial cert (for bootstrapping) has expired within 24hrs. Not hard to execute/explain, but we should document it.

repo: RedHatSI/openshift-packet-deploy


text:
use six library for isinstance

fix #64

@quintonparker can you confirm this fixed the issue

@ashtul works on my machine ™
😄

authors: ashtul quintonparker

repo: RediSearch/redisearch-py


text:
next-gen CI for bloom, migrating to poetry

This takes a next-gen approach to CI builds for python clients, using Circle. I don't have permissions to modify the repository - so let's discuss the approach, before more events are triggered.
The stock requirements.txt is used to install the dependencies for the 'build', tox and poetry. For the purpose of the branch - those builds have been configured to remove all logic from CircleCI, meaning anything can call these in the future. Circle is only responsible for installing them, and calling tox to do its work.
As below - we now build (in parallel) across all of the supported python versions. To add python versions to CI, all we have to do is edit the .circleci config and add a new python docker.
If we like this approach my proposal is to complete it by:

[x] Updating the README
[x] Add support for auto build, and publish to PyPI on tag, removing that from GitHub
[x] Changing the nightly to use the same code
[ ] (Optionally) Replace tox with pytest integrate since it can run codecov
[x] Integrating the codecov upload (currently broken because I don't have permissions)
[ ] Changing GitHub requirements to pass on the tests, rather than a specific build
[x] Lifting the early return bits I've introduced to other modules here.
[x] Applying to our other python projects.

@filipecosta90 or @guykorlandredis Do you have the ability to change the GitHub repo to depend on ci/circleci:build-latest instead of ci/circleci:build?

repo: RedisBloom/redisbloom-py


text:
Use MinVer NuGet package for Versioning NuGet packages

MinVer does versioning using git tags, git height, build number and git commit SHA all combined.
Added Directory.Build.props to configure common .csproj properties.
Added Directory.Build.targets to configure MinVer.

I'm considering going all in with Directory.Build.props and adding another one under the Source directory to configure assembly signing, turning on xml doc generation and configuring source link. Perhaps even another one under Tests to add xunit etc.

Looks good to me! Definitely cleans up a lot of double-ups having a global Directory.Build.props.
Regarding having additional ones under tests and source, I guess it depends whether there will be more projects under those folders or whether the idea is to bring all non-specific configuration out of the single .csproj files.
That said, I'd probably still do it because there is still a chance we might have additional projects later on - whether that be we have separate projects for the extended schema.org types and then require splitting the tests into separate projects for each. 🙂

Regarding having additional ones under tests and source, I guess it depends whether there will be more projects under those folders or whether the idea is to bring all non-specific configuration out of the single .csproj files.

I guess it doesn't matter in this one. I have other projects where it will. I try to keep them all consistent though.

Turns out you have to add a nasty <Import> if you want to use multiple nested Directory.Build.props files. I'll do it in another PR.

authors: RehanSaeed Turnerj


text:
FileNotFoundException for System.Diagnostics.DiagnosticSource with version 3.0.0 #237

Describe the bug
If I include this project reference <PackageReference Include="Serilog.Enrichers.Span" Version="3.0.0" /> and start an Acitivity in an Azure Function project, I get the following exception:
System.IO.FileNotFoundException: Could not load file or assembly 'System.Diagnostics.DiagnosticSource, Version=7.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. The system cannot find the file specified.
File name: 'System.Diagnostics.DiagnosticSource, Version=7.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
   at serilog_span_bug.Test.Run(HttpRequest req, ILogger log)
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at serilog_span_bug.Test.Run(HttpRequest req, ILogger log)
   at lambda_method110(Closure , Test , Object[] )
   at Microsoft.Azure.WebJobs.Host.Executors.TaskMethodInvoker`2.InvokeAsync(TReflected instance, Object[] arguments) in C:\projects\azure-webjobs-sdk-rqm4t\src\Microsoft.Azure.WebJobs.Host\Executors\TaskMethodInvoker.cs:line 21
   at Microsoft.Azure.WebJobs.Host.Executors.FunctionInvoker`2.InvokeAsync(Object instance, Object[] arguments) in C:\projects\azure-webjobs-sdk-rqm4t\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionInvoker.cs:line 52

I've created a minimal repro by just creating a new Function project, included the package reference, and created a new Activity.
I've tried to understand what happens, but I'm afraid it's a bit beyond me. I guess it's some kind of dependency conflict. Even if I explicitly add that reference, I still get the same exception. Downgrading to version 2.3.0 fixes the issue as well.
Steps to reproduce

Clone/Fork https://github.com/jooooel/serilog-span-bug
Run the function called Test (just F5 in VS/Rider)
Invoke the function by going to http://localhost:7071/api/Test in a browser

Expected behaviour
That there isn't an exception...

I realised this is due to the fact that there is a reference to System.Diagnostics.DiagnosticSource 7.0.0 in the csproj file and that I'm using it from an Azure Function. So I've open a separate issue here. I guess you can close this one if you'd like 😊

Thanks for reporting.

authors: RehanSaeed jooooel


text:
Getting compatibility error with VS Code 1.20.0

Getting the following error message:

Unable to install 'mischah.relaxed-theme'; there is no available version compatible with VS Code '1.20.0'.

I just have released a new version which should be compatible with VS Code '1.20.0'.

authors: Fralleee mischah

repo: Relaxed-Theme/vscode-theme-relaxed


text:
Problem on implementation of HAPPO

After testing HAPPO, I found that in happo_surrogate_loss, no other agents are considered for each self-agent. I wonder if there is any problem?

hello, the Ray RLlib is designed with singleton updating. Therefore, in order to implement the heterogeneous updating, we cannot use the loss function directly.
Here is the implementation:
First, in https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/core/CC/happo.py#L74
    for i, iter_train_info in enumerate(get_each_agent_train(model, policy, dist_class, train_batch)):
        iter_model, iter_dist_class, iter_train_batch, iter_mask, \
            iter_reduce_mean, iter_actions, iter_policy, iter_prev_action_logp = iter_train_info

There is a loop. In this loop, we get each agent's model, batch, etc. These data were collected in postprocessing:
.
Secondly, we will train each model in https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/core/CC/happo.py#L114
Thirdly, we will use trained each agent's model to get the updated sampling importance to get the $M-advantage$
Fourthly, in order to get each agent's model result, there is an important class need to be mentioned:
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/heterogeneous_updateing.py#L54
Because of this class, we could get each agent's data by re-implement the builtin functions: getitem
By the combination of the above ways,  we could implement the updating for related agents sequentially and dependently in these ways.
Am I clear on this?

hello, the Ray RLlib is designed with singleton updating. Therefore, in order to implement the heterogeneous updating, we cannot use the loss function directly.
Here is the implementation:
First, in
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/core/CC/happo.py#L74
    for i, iter_train_info in enumerate(get_each_agent_train(model, policy, dist_class, train_batch)):
        iter_model, iter_dist_class, iter_train_batch, iter_mask, \
            iter_reduce_mean, iter_actions, iter_policy, iter_prev_action_logp = iter_train_info

There is a loop. In this loop, we get each agent's model, batch, etc. These data were collected in postprocessing:
.
Secondly, we will train each model in
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/core/CC/happo.py#L114
Thirdly, we will use trained each agent's model to get the updated sampling importance to get the M−advantage
Fourthly, in order to get each agent's model result, there is an important class need to be mentioned:
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/heterogeneous_updateing.py#L54
Because of this class, we could get each agent's data by re-implement the builtin functions: getitem
By the combination of the above ways, we could implement the updating for related agents sequentially and dependently in these ways.
Am I clear on this?

Thanks for the response. Basically, I understand the logic. What I was confused about is that I ran the command:
python main.py --algo_config=happo --env_config=lbf.
Then I output the all_polices_with_names in the https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/heterogeneous_updateing.py#L54:~:text=get_each_agent_train, it turns out that only the self policy is in the all_polices_with_names. Then what is the meaning of iteration in all_polices_with_names ?

hello, the Ray RLlib is designed with singleton updating. Therefore, in order to implement the heterogeneous updating, we cannot use the loss function directly.
Here is the implementation:
First, in
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/core/CC/happo.py#L74
    for i, iter_train_info in enumerate(get_each_agent_train(model, policy, dist_class, train_batch)):
        iter_model, iter_dist_class, iter_train_batch, iter_mask, \
            iter_reduce_mean, iter_actions, iter_policy, iter_prev_action_logp = iter_train_info

There is a loop. In this loop, we get each agent's model, batch, etc. These data were collected in postprocessing:
.
Secondly, we will train each model in
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/core/CC/happo.py#L114
Thirdly, we will use trained each agent's model to get the updated sampling importance to get the M−advantage
Fourthly, in order to get each agent's model result, there is an important class need to be mentioned:
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/heterogeneous_updateing.py#L54
Because of this class, we could get each agent's data by re-implement the builtin functions: getitem
By the combination of the above ways, we could implement the updating for related agents sequentially and dependently in these ways.
Am I clear on this?

Thanks for the response. Basically, I understand the logic. What I was confused about is that I ran the command: python main.py --algo_config=happo --env_config=lbf. Then I output the all_polices_with_names in the https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/heterogeneous_updateing.py#L54:~:text=get_each_agent_train, it turns out that only the self policy is in the all_polices_with_names. Then what is the meaning of iteration in all_polices_with_names ?

hello, This is a feature that existed in Ray actually. It is pretty tricky.
Here is the thing:
If there are N agents existing in a task.
At the first steps sampling step, the RLlib will only run ONLY ONE agent when sampling.
But after some steps, maybe like some kind of "warming up", all agents will occur.
For example, if we check the postprocessing function in this place:
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/centralized_critic_hetero.py#L301
We will find, in the beginning, steps, the crucial parameters: other_agent_batches is None, which means RLlib only run a single agent.
In order to solve this problem,  I implement the connect and share function here:
https://github.com/Replicable-MARL/MARLlib/blob/bb3b9a296a7792486d4cceb6e3175a8140ba4db1/marl/algos/utils/centralized_critic_hetero.py#L201
This function will check if there are 'other_agent_batches' in the sampling batch. And there is it, the function will firstly share the critic with each agents and connect the other agent's information to the main agent, which is sent to the loss running.
In short,  in order to solve this,
you need:
Firstly, make sure that there are more than one agent in this task;
Secondly, waiting for a short minute, more agent information will occur.
Am I clear about it?

Awesome! Thanks for the clarification.

Awesome! Thanks for the clarification.

You are welcome. It's my pleasure. :)

repo: Replicable-MARL/MARLlib


text:
关于数据的问题

非常感谢您分享代码。我想请问您数据中power和power_value代表的是什么？power和intensity有什么联系吗？

具体我也不清楚，可能是回波强度和某种归一化之后的值。我只关注算法。