Error Running the Request-ocp-roks-play plabook to deploy a ROKS cluster

Describe the bug
I am using the playbook request ocp roks to deploys a ROKS cluster. I have followed the instructions on the readme of this playbook and populated the roks-vars.yaml with the details.
I am finding the error :
TASK [request-ocp-roks : get-resource-group] *********************************************************************************************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "", "rc": 1, "resource": {"_name": "cloudpaksvt-user-ci-cd", "_type": "ibm_resource_group", "target": "ibm_resource_group.cloudpaksvt-user-ci-cd"}, "stderr": "\nError: Error occured while fetching account user details: \"token contains an invalid number of segments\"\n\n  on ibm_resource_group_cloudpaksvt-user-ci-cd.tf line 1, in data \"ibm_resource_group\" \"cloudpaksvt-user-ci-cd\":\n   1: data ibm_resource_group \"cloudpaksvt-user-ci-cd\" {\n\n\n", "stderr_lines": ["", "Error: Error occured while fetching account user details: \"token contains an invalid number of segments\"", "", "  on ibm_resource_group_cloudpaksvt-user-ci-cd.tf line 1, in data \"ibm_resource_group\" \"cloudpaksvt-user-ci-cd\":", "   1: data ibm_resource_group \"cloudpaksvt-user-ci-cd\" {", "", ""], "stdout": "data.ibm_resource_group.cloudpaksvt-user-ci-cd: Refreshing state...\n", "stdout_lines": ["data.ibm_resource_group.cloudpaksvt-user-ci-cd: Refreshing state..."]}

PLAY RECAP *******************************************************************************************************************************************************************************************************
localhost                  : ok=2    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

This would suggest that the IBMCloud APIKey I am using to login is wrong? But when running the command ibmcloud login --apikey <apikey> the login suceeds and it shows me the required resource group.
Expected behavior
Provided the same login details, the playbook should be able to run the task to get the given resource group.
Desktop (please complete the following information):

OS: Linux

Am I perhaps missing out on some dependancies?
I have following the instructions here https://github.com/IBM/community-automation/tree/master/ansible/request-ocp-roks-play
But have seen that the page here specifies additional requirements https://github.com/IBM/community-automation/tree/master/ansible/roles/request-ocp-roks which I have not installed.
Which page should I be following? As a side note upon finding that second set of instructions in the roles directory, I tried to not pass in the resourcegroup as the readme said not required but then the playbook returned saying that you are missing that variable in the yaml/

File passed in
---
ansible_python_interpreter: /usr/bin/python3
cloudregion: "us-south" # Provide the resource i.e. us-south to use
clusterName: "<clusterName>" # Provide a unique cluster name
dataCenter: "dal12" # Provide the data center (zone) to deploy to
defaultPoolSize: 1 # Set the size of the default worker pool
hardware: shared # shared for virtual workers
kubeVersion: 4.5.24_openshift # kube version to use
machineType: "b3c.16x64" # machine flavor to use
privateVLAN: <privLan> # Private VLAN to use, must be available
publicVLAN: <pubLan> # Public VLAN to use, optional
resourceGroup: "<resGroup>" # Provide the Resouce Group name where the cluster is deployed -- default is "default" resource group
workerCount: "6" # Set the size of the default worker pool
icaccount: "<7 digitaccount number >" # Target the account to use
entitlement: "cloud_pak" # Set entitlement to "cloud_pak" when deploying a cloud pak to the cluster otherwise you will be charged for OCP licenses
apikey: "<apikey>" # Set the api key here to use for IBM Cloud authentication

is the error due to passing in some variables with ""?
When I passed in the resourceGroup as a command line argument ansible-playbook -i localhost request-roks.yml -e resourceGroup=cloudpaksvt-user-ci-cd The playbook completed the task correctly.

@jasonthink any thoughts on this one?

I think it is going to be a good idea to add the role readme link to all of our playbook readme, in case the role has additional setup or prereq's.

The last time I ran this, I filled in all of the variables, and made sure I installed the terraform pieces.

Hey Ray, so It seems to be no longer a problem but I think the ROKS-Vars yaml instructions are a bit incorrect.
Eg: https://github.com/IBM/community-automation/blob/master/ansible/request-ocp-roks-play/examples/roks-vars.yml#L6 and https://github.com/IBM/community-automation/blob/master/ansible/request-ocp-roks-play/examples/roks-vars.yml#L13 seem to reference the same topic but only the line 6 input is taken.
The main problem this issue was tackling I was able to fix by removing the quotes on the resourceGroup variable.
Please let me know how you would like to proceed with this? Perhaps a change to the yaml instructions, perhaps more info on the readme. Or if you are happy to close this, then plpease let me know that too.

@prattyushmangal woud you like to make the doc changes in your own fork and create a PR for review?

I'd be happy to make the changes in the Readme and the Roks-vars yaml about not needing the quotation marks in the resourceGroup variable.
But I am not sure about the changes regarding the workerCount vs defaultPoolSize. Not sure if there is code reading in these vars as a requirement but then not using them?
Does that sound okay?

I'm working on this now. I did find some undesirable behavior in how variables are handled and I'm working on that. As far as the original error posted, it's hard to say what it was pointing to as the issue. Most likely was an inability to load the resource group from IBM Cloud (for some reason). Errors returned are not often clear or in direct relation to the problem (in my experience). I also understand some documentation is confusing and I'm working through that now. I'll update soon.

@prattyushmangal please close this issue if you are satisfied with the updates.

Hi @rayashworth just testing the playbook now and will close if the changes added by @jasonthink are good to go. Thank you Jason

Following the PR, spotted that there was an error returned when we tried to run the playbook.
fatal: [localhost]: FAILED! => {"changed": false, "msg": "", "rc": 1, "resource": {"_name": "<cluster name>", "_type": "ibm_container_cluster", "target": "ibm_container_cluster.<name>"}, "stderr": "\nError: \"worker_num\"  must be greater than 0...

Have corrected this in my fork, will link PR to this issue, if someone could please review. Thank You


Error while installing PyTornado

 ERROR: Command errored out with exit status 1:
     command: 'C:\Users\Guset User\AppData\Local\Microsoft\WindowsApps\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\GUSETU~1\\AppData\\Local\\Temp\\pip-install-kscujy3m\\pytornado\\setup.py'"'"'; __file__='"'"'C:\\Users\\GUSETU~1\\AppData\\Local\\Temp\\pip-install-kscujy3m\\pytornado\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\GUSETU~1\AppData\Local\Temp\pip-install-kscujy3m\pytornado\pip-egg-info'
         cwd: C:\Users\GUSETU~1\AppData\Local\Temp\pip-install-kscujy3m\pytornado\
    Complete output (9 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\GUSETU~1\AppData\Local\Temp\pip-install-kscujy3m\pytornado\setup.py", line 90, in <module>
        packages=find_packages(where=PACKAGE_DIR),
      File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.7_3.7.1776.0_x64__qbz5n2kfra8p0\lib\site-packages\setuptools\__init__.py", line 71, in find
        convert_path(where),
      File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.7_3.7.1776.0_x64__qbz5n2kfra8p0\lib\distutils\util.py", line 112, in convert_path
        raise ValueError("path '%s' cannot end with '/'" % pathname)
    ValueError: path 'src/lib/' cannot end with '/'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Thanks for reporting this. I don't have a Windows system available at the moment to test the installation myself. From your error log, I suspect that this error occurs because the file paths in setup.py are not of Windows style. I have added a fix for that.
Please try to install PyTornado with this latest fix (https://github.com/airinnova/pytornado/archive/master.zip) and let me know if the error still remains or if something else doesn't work.

The command with the error pasted here:
c:\Bharath\IIT PhD\Phase 3\Tornado\pytornado-master>pip install pytornado
Collecting pytornado
Using cached https://files.pythonhosted.org/packages/ee/39/8e2cc637772fc6e8fb4e518e3d926139aa3d32d62255981473a6fe93e270/pytornado-0.5.0.tar.gz
ERROR: Command errored out with exit status 1:
command: 'c:\users\guset user\appdata\local\programs\python\python38\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\Users\Guset User\AppData\Local\Temp\pip-install-5p903tdx\pytornado\setup.py'"'"'; file='"'"'C:\Users\Guset User\AppData\Local\Temp\pip-install-5p903tdx\pytornado\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\Guset User\AppData\Local\Temp\pip-install-5p903tdx\pytornado\pip-egg-info'
cwd: C:\Users\Guset User\AppData\Local\Temp\pip-install-5p903tdx\pytornado
Complete output (9 lines):
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Guset User\AppData\Local\Temp\pip-install-5p903tdx\pytornado\setup.py", line 90, in
packages=find_packages(where=PACKAGE_DIR),
File "c:\users\guset user\appdata\local\programs\python\python38\lib\site-packages\setuptools_init_.py", line 71, in find
convert_path(where),
File "c:\users\guset user\appdata\local\programs\python\python38\lib\distutils\util.py", line 124, in convert_path
raise ValueError("path '%s' cannot end with '/'" % pathname)
ValueError: path 'src/lib/' cannot end with '/'
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Could you please try again following these installation instructions:

https://pytornado.readthedocs.io/en/latest/user_guide/installation.html

You will need to download the latest version here (https://github.com/airinnova/pytornado/archive/master.zip) and run pip install . in the PyTornado root directory. pip install pytornado does not work at the moment as pointed out here.

I apparently need Visual C++ 14.0 which I am downloading now.
In the meantime, can you please get me the results for a simple aircraft geometry from PyTornado? I want to see if the results are different from Matlab Tornado.  I would really appreciate the help.

Aaron
I was able to install PyTornado but I am getting the following error:
'pytornado' is not recognized as an internal or external command,
operable program or batch file.
Why is the command not being recognised?
On Thu, Jan 2, 2020 at 10:22 AM Aaron Dettmann<EMAIL_ADDRESS>wrote:

Could you please try again following these installation instructions:

https://pytornado.readthedocs.io/en/latest/user_guide/installation.html
You will need to download the latest version here (
https://github.com/airinnova/pytornado/archive/master.zip) and run pip
install . in the PyTornado root directory. pip install pytornado does
not work at the moment as pointed out here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5AMOPC3MLMT7YW2STPDQ3YWH5A5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH7AIRI#issuecomment-570295365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5AJEPCDSG2VOWYGO4VLQ3YWH5ANCNFSM4KCDWK3Q
.

If you still get any error messages, it would help me if you can send them here, since I don't have a Windows machine to test the installation myself. Then I will be able to fix the installer and to update the documentation :)
How to generate a simple rectangular wing is described here: https://pytornado.readthedocs.io/en/latest/user_guide/getting_started.html The geometry and analysis can easily be set up in Matlab Tornado too. Numerical results can be found in the _results folder which will be generated when you run pytornado -v --run settings/template.json.

When I call pytornado, this is the error message:
'pytornado' is not recognized as an internal or external command,
operable program or batch file.
On Thu, Jan 2, 2020 at 12:49 PM Aaron Dettmann<EMAIL_ADDRESS>wrote:

If you still get any error messages, it would help me if you can send them
here, since I don't have a Windows machine to test the installation myself.
Then I will be able to fix the installer and to update the documentation :)
How to generate a simple rectangular wing is described here:
https://pytornado.readthedocs.io/en/latest/user_guide/getting_started.html
The geometry and analysis can easily be set up in Matlab Tornado too.
Numerical results can be found in the _results folder which will be
generated when you run pytornado -v --run settings/template.json.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5AJ37FKPBHNAX46HYUTQ3ZHPFA5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH7L2AQ#issuecomment-570342658,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5AONHG4AS7XL3I6HCC3Q3ZHPFANCNFSM4KCDWK3Q
.

This is a Windows issue. It has to do with where Windows looks for executable files. There is little I can do here I am afraid. Try to google the error, it seems to be a common with Python. It should also work to run python.exe C:\path\to\tornado (but I am not sure where pip installs its packages on Windows). See also:

https://docs.python.org/3.7/using/windows.html

Hi Aaron
Thanks. I am able to run pytornado by:

Changing the name of the file "pytornado" to something different -->
"pytornado_".
Calling the command "py pytornado_ --vr
Now I need another help. I have an aircraft configuration and I am having
trouble making the template file. Can you please make one for me so that I
can use it for other configurations in future?

On Thu, Jan 2, 2020 at 1:14 PM Aaron Dettmann<EMAIL_ADDRESS>wrote:

This is a Windows issue. It has to do with where Windows looks for
executable files. There is little I can do here I am afraid. Try to google
the error, it seems to be a common with Python. It should also work to run python.exe
C:\path\to\tornado (but I am not sure where pip installs its packages
on Windows). See also:

https://docs.python.org/3.7/using/windows.html

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5APOE4662C2CH2QTVPLQ3ZKNBA5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH7OFUA#issuecomment-570352336,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5AIQ4R2J4FRUQBS4XYDQ3ZKNBANCNFSM4KCDWK3Q
.

The reason I need your help is because I am getting wrong force values. I
think it's because of the setup file. Hope you can help me out.
On Thu, Jan 2, 2020 at 1:53 PM Bharath ae15d019<EMAIL_ADDRESS>wrote:

Hi Aaron
Thanks. I am able to run pytornado by:

Changing the name of the file "pytornado" to something different -->
"pytornado_".
Calling the command "py pytornado_ --vr
Now I need another help. I have an aircraft configuration and I am having
trouble making the template file. Can you please make one for me so that I
can use it for other configurations in future?

On Thu, Jan 2, 2020 at 1:14 PM Aaron Dettmann<EMAIL_ADDRESS>wrote:

This is a Windows issue. It has to do with where Windows looks for
executable files. There is little I can do here I am afraid. Try to google
the error, it seems to be a common with Python. It should also work to run python.exe
C:\path\to\tornado (but I am not sure where pip installs its packages
on Windows). See also:

https://docs.python.org/3.7/using/windows.html

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5APOE4662C2CH2QTVPLQ3ZKNBA5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH7OFUA#issuecomment-570352336,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5AIQ4R2J4FRUQBS4XYDQ3ZKNBANCNFSM4KCDWK3Q
.

I was able to fix this.
On Thu, Jan 2, 2020 at 2:10 PM Bharath ae15d019<EMAIL_ADDRESS>wrote:

The reason I need your help is because I am getting wrong force values. I
think it's because of the setup file. Hope you can help me out.
On Thu, Jan 2, 2020 at 1:53 PM Bharath ae15d019<EMAIL_ADDRESS>wrote:

Hi Aaron
Thanks. I am able to run pytornado by:

Changing the name of the file "pytornado" to something different -->
"pytornado_".
Calling the command "py pytornado_ --vr
Now I need another help. I have an aircraft configuration and I am having
trouble making the template file. Can you please make one for me so that I
can use it for other configurations in future?

On Thu, Jan 2, 2020 at 1:14 PM Aaron Dettmann<EMAIL_ADDRESS>wrote:

This is a Windows issue. It has to do with where Windows looks for
executable files. There is little I can do here I am afraid. Try to google
the error, it seems to be a common with Python. It should also work to run python.exe
C:\path\to\tornado (but I am not sure where pip installs its packages
on Windows). See also:

https://docs.python.org/3.7/using/windows.html

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5APOE4662C2CH2QTVPLQ3ZKNBA5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH7OFUA#issuecomment-570352336,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5AIQ4R2J4FRUQBS4XYDQ3ZKNBANCNFSM4KCDWK3Q
.

Glad to hear that the installation seemed to work now. Do you have any other issues with the installation procedure itself? Otherwise, I can try to help with your other questions in one of the other issues, and we close this one. We will try to address the issues with the installation on Windows that you reported here and update the installation scripts and documentation.

Sounds good.
On Fri, Jan 3, 2020 at 3:57 AM Aaron Dettmann<EMAIL_ADDRESS>wrote:

Glad to hear that the installation seemed to work now. Do you have any
other issues with the installation procedure itself? Otherwise, I can try
to help with your other questions in one of the other issues, and we close
this one. We will try to address the issues with the installation on
Windows that you reported here and update the installation scripts and
documentation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5AKEVQ2ACCFMTFILGMDQ34R4DA5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIA7GIA#issuecomment-570553120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5ALI45WMTVQOO6PJGOLQ34R4DANCNFSM4KCDWK3Q
.

Thanks for your help.
On Fri, Jan 3, 2020 at 3:57 AM Aaron Dettmann<EMAIL_ADDRESS>wrote:

Glad to hear that the installation seemed to work now. Do you have any
other issues with the installation procedure itself? Otherwise, I can try
to help with your other questions in one of the other issues, and we close
this one. We will try to address the issues with the installation on
Windows that you reported here and update the installation scripts and
documentation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airinnova/pytornado/issues/12?email_source=notifications&email_token=AHHO5AKEVQ2ACCFMTFILGMDQ34R4DA5CNFSM4KCDWK32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIA7GIA#issuecomment-570553120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHO5ALI45WMTVQOO6PJGOLQ34R4DANCNFSM4KCDWK3Q
.


Checking for balanced paranthesis using stack

This is useful and implicit code part used in IDEs like VScode to detect errors.

Excluded repo for Hacktoberfest 2022.


Better version outpout

Hello !
I follow the development of Magic of Jenkins and use it like a changelog, but It's hard to know what build I use cause "/ver magic" didn't tell what build I'm using
Can you add the build number in /ver magic output ?
/ver magic

/ver Citizens show the build

Sorry, I prefer the git revision. It tells me what you're running, even if you didn't get it from my build server :)


Prediction of Recession

Prediction of recession using ml if this is good please assign the necessary labels and taggings under gssoc

this repo aint considering this issue, we are only focusing on stack over flow


About Image transform

Hello! I tried to use your released p3d pytorch model. But i got a wrong result, every class got a low score under 0.01.
Is it my way to transform image into tensor result in this?
model = P3D199(pretrained=True)
model = model.eval().to(device)

transforms = Compose([Resize(160), CenterCrop(160), ToTensor(),
             Normalize(mean=(0.485,0.456,0.406),std=(0.229, 0.224, 0.255),inplace=True)])

clip = [transforms(img) for img in clip] # 16 frames
clip = torch.stack(clip, 0).permute(1, 0, 2, 3).unsqueeze(0).to(device)

out=model(clip)

Thank you for your advice

Hi! Did you solve your problem? If yes, how?

Hi! Did you solve your problem? If yes, how?

No, I use another repository

Can I have the link? Il gio 27 feb 2020, 11:39 万昕<EMAIL_ADDRESS>ha scritto:
…
Hi! Did you solve your problem? If yes, how? No, I use another repository — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#31?email_source=notifications&email_token=ALCKVNRPVCG5QT4TAJB7RCTRE6J5HA5CNFSM4IZC6IF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEND3DWI#issuecomment-591901145>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALCKVNVGOKYFOJGOHDE5CALRE6J5HANCNFSM4IZC6IFQ .

I use 3d-resnet or 3d-resnext

I have also some problem, who can help me?


reimplement rgbas

fixes a bug when the 'scheme' argument is passed
reimplement using matplotlib's vectorized method; much faster
give back numpy instead of lists

updated and renamed with an example if @martinfleis or @sjsrey want to have a look


Charmaps and other issues

This tool is awesome and I managed to gain some success with the running of the converter however, I had to add a bunch of error handling, particularly with handling charmaps. Also seems to be some issues with deleting files. Once I work out the issues, I will submit a PR but I thought I would flag.
Also the repo of converted sigma is now 404.
Many thanks!

Hey @XGREENi3 -- there's a lot of room for improvement.
If you can post some sample inputs that produce errors, that would be super useful

Utils.py
def escape_special_characters(content):
# Replace problematic special characters with an escaped version
return re.sub(r'[\x00-\x08\x0B\x0C\x0E-\x1F\x7F-\x9F\xAD\u0378\u0379\u0380-\u0383\u038B\u038D\u03A2\u0530\u0560\u0588\u058B\u058C\u058E\u058F\u0590\u05C8\u05C9\u05EC\u05ED\u05F5\u05F6\u05F8\u05F9\u0600\u0604\u0605\u061C\u061D\u070E\u070F\u074B\u074C\u07B2-\u07BF\u07FB\u07FC\u082E\u082F\u083F\u085C\u085D\u085F-\u089F\u08A1\u08AD\u08E4\u08E5\u08E8-\u08EF\u08F2-\u08FF\u0978\u0980\u0984\u098D\u098E\u0991\u0992\u09A9\u09B1\u09B3\u09B5\u09BA\u09BB\u09C5\u09C6\u09C9\u09CA\u09CF\u09D0\u09D8\u09DB\u09DE\u09E4\u09E5\u09FC-\u0A00\u0A04\u0A0B-\u0A0E\u0A11\u0A12\u0A29\u0A31\u0A34\u0A37\u0A3A\u0A3B\u0A3D\u0A43-\u0A46\u0A49\u0A4A\u0A4E\u0A50\u0A52-\u0A58\u0A5D\u0A5F-\u0A65\u0A76-\u0A80\u0A84\u0A8E\u0A92\u0AA9\u0AB1\u0AB4\u0ABA\u0ABB\u0AC6\u0ACA\u0ACE\u0ACF\u0AD1-\u0ADF\u0AE4\u0AE5\u0AF2\u0AF4-\u0B00\u0B04\u0B0D\u0B0E\u0B11\u0B12\u0B29\u0B31\u0B34\u0B3A\u0B3B\u0B45\u0B46\u0B49\u0B4A\u0B4E\u0B55\u0B58\u0B5B\u0B5E\u0B64\u0B65\u0B78-\u0B81\u0B84\u0B8B\u0B8D\u0B91\u0B96-\u0B98\u0B9B\u0B9D\u0BA0\u0BA2\u0BA5\u0BA7\u0BAB\u0BAC\u0BB6\u0BBA-\u0BBD\u0BC3\u0BC5\u0BC9\u0BCE\u0BCF\u0BD1-\u0BD6\u0BD8-\u0BE5\u0BFB-\u0C00\u0C04\u0C0D\u0C11\u0C29\u0C34\u0C3A\u0C3B\u0C45\u0C49\u0C4E\u0C54\u0C57\u0C5B\u0C5E\u0C64\u0C65\u0C70\u0C71\u0C75\u0C80\u0C81\u0C84\u0C8D\u0C91\u0CA9\u0CB4\u0CBA\u0CBB\u0CC5\u0CC9\u0CCE\u0CD4\u0CD7-\u0CDD\u0CDF\u0CE4\u0CE5\u0CF0\u0CF3\u0D04\u0D0D\u0D11\u0D3B\u0D3C\u0D45\u0D49\u0D4F\u0D56\u0D58-\u0D5F\u0D64\u0D65\u0D76-\u0D78\u0D80\u0D81\u0D84\u0D97-\u0D99\u0DB2\u0DBC\u0DBE\u0DBF\u0DC7-\u0DC9\u0DCB-\u0DCE\u0DD5\u0DD7\u0DE0-\u0DF1\u0DF5-\u0E00\u0E3B\u0E3C\u0E45\u0E4F\u0E5A\u0E5B\u0E83\u0E85\u0E86\u0E89\u0E8B\u0E8C\u0E8E-\u0E93\u0E98\u0EA0\u0EA4\u0EA6\u0EA8\u0EA9\u0EAC\u0EBA\u0EBE\u0EBF\u0EC5\u0EC7\u0ECE\u0ECF\u0EDA\u0EDB\u0EE0-\u0EFF\u0F48\u0F6D-\u0F70\u0F98\u0FBD\u0FCD\u0FDB-\u0FFF\u10C6\u10C8\u10C9\u10CC\u10CE\u10CF\u1249\u124E\u124F\u1257\u1259\u125E\u125F\u1289\u128E\u128F\u12B1\u12B6\u12B7\u12BF\u12C1\u12C6\u12C7\u12D7\u1311\u1316\u1317\u135B\u135C\u137D\u139A-\u139F\u13F5-\u13FF\u169D-\u169F\u16F1-\u16FF\u170D\u1715-\u171F\u1737-\u173F\u1754-\u175F\u176D\u1771\u1774-\u177F\u17DE\u17DF\u17EA-\u17EF\u17FA-\u17FF\u180F\u181A-\u181F\u1878-\u187F\u18AB-\u18AF\u18F6-\u18FF\u191D-\u191F\u192C-\u192F\u193C-\u193F\u1941-\u1943\u196E\u196F\u1975-\u197F\u19AC-\u19AF\u19CA-\u19CF\u19DB-\u19DD\u1A1C\u1A1D\u1A5F\u1A7D\u1A7E\u1A8A-\u1A8F\u1A9A-\u1A9F\u1AAE-\u1AFF\u1B4C-\u1B4F\u1B7D-\u1B7F\u1BF4-\u1BFB\u1C38-\u1C3A\u1C4A-\u1C4C\u1C80-\u1CBF\u1CC8-\u1CCF\u1CF7-\u1CFF\u1DE7-\u1DFB\u1E9C-\u1E9F\u1EFA-\u1EFF\u1F16\u1F17\u1F1E\u1F1F\u1F46\u1F47\u1F4E\u1F4F\u1F58\u1F5A\u1F5C\u1F5E\u1F7E\u1F7F\u1FB5\u1FC5\u1FD4\u1FD5\u1FDC\u1FF0\u1FF1\u1FF5\u1FFF\u200B-\u200F\u202A-\u202E\u2060-\u206F\u2072\u2073\u208F\u209D-\u209F\u20BB-\u20CF\u20F1-\u20FF\u218A\u218B\u23F4-\u23FF\u2427-\u243F\u244B-\u245F\u2B74-\u2B75\u2B96-\u2B97\u2BBA-\u2BBC\u2BC9-\u2BCF\u2BD2\u2BD3\u2BFE-\u2BFF\u2C2F\u2C5F\u2CF4-\u2CF8\u2D26\u2D28-\u2D2C\u2D2E\u2D2F\u2D68-\u2D6E\u2D71-\u2D7E\u2D97\u2D9F\u2DA7\u2DAF\u2DB7\u2DBF\u2DC7\u2DCF\u2DD7\u2DDF\u2E3C-\u2E7F\u2E9A\u2EF4-\u2EFF\u2FD6-\u2FEF\u2FFC-\u2FFF\u3040\u3097\u3098\u3100-\u3104\u312E\u3130\u318F\u31BB-\u31BF\u31E4-\u31EF\u321F\u32FF\u4DB6-\u4DBF\u9FCD-\u9FFF\uA48D-\uA48F\uA4C7-\uA4CF\uA62C-\uA63F\uA698-\uA69E\uA6F8-\uA6FF\uA78F\uA794-\uA79F\uA7AB-\uA7F7\uA82C-\uA82F\uA83A-\uA83F\uA878-\uA87F\uA8C5-\uA8CD\uA8DA-\uA8DF\uA954-\uA95E\uA97D-\uA97F\uA9CE\uA9DA-\uA9DD\uA9E0-\uA9FF\uAA37-\uAA3F\uAA4E-\uAA4F\uAA5A\uAA5B\uAA7C-\uAA7F\uAAC3-\uAADA\uAAF7\uAB07\uAB08\uAB0F\uAB10\uAB17\uAB1F\uAB27\uAB2F\uAB6C-\uAB6F\uABEE\uABEF\uABFA-\uABFF\uD7A4-\uD7AF\uD7C7-\uD7CA\uD7FC-\uD7FF\uFA6E\uFA6F\uFADA-\uFAFF\uFB07\uFB12\uFB18-\uFB1C\uFB37\uFB3D\uFB3F\uFB42\uFB45\uFBC3\uFBC4\uFBCB-\uFBCC\uFBEA-\uFBFB\uFE10\uFE16\uFE17\uFE1A-\uFE1F\uFE27-\uFE2F\uFE53\uFE67\uFE6C-\uFE6F\uFE75\uFEFD\uFF00\uFFBF-\uFFC1\uFFC8\uFFC9\uFFD0\uFFD1\uFFD8\uFFD9\uFFDD\uFFDE\uFFE7\uFFEF-\uFFFB]', '', content)
def read_yaml_file(file_path):
try:
with open(file_path, 'r', encoding='utf-8') as file:
content = file.read()
content = escape_special_characters(content)
yaml_data = yaml.safe_load(content)
return yaml_data
except yaml.YAMLError as e:
print(f"YAML error in file {file_path}: {e}")
except Exception as e:
print(f"Error reading YAML file {file_path}: {e}")
return None
This seems to handle the non-UTF chars

def append_data():
results = []
for root, _, files in os.walk(config.file_system_path):
if ".git" in root:
continue
for filename in files:
if filename.endswith(".json"):
file_path = os.path.join(root, filename)
try:
with open(file_path, "r", encoding="utf-8") as file:
stix_object = json.load(file)
results.append(stix_object)
except json.JSONDecodeError as e:
print(f"Error decoding JSON in file {file_path}: {e}")
except UnicodeDecodeError as e:
print(f"Encoding error in file {file_path}: {e}")
except Exception as e:
print(f"Error reading file {file_path}: {e}")
return results
This handles the non-utf at the end of the process

Made a PR

Thanks @XGREENi3 ! Do you have an example rule I can use to test with?

I did this on the full repo pull, as that is where I was getting the errors. I could not identify specific rules with that. I should have tried to identify those, apologies!

all good! that's useful info and will help me find them, thanks!


[Docs] update guidance for test

fix: https://github.com/Azure/azure-sdk-for-python/issues/23865

fix: https://github.com/Azure/azure-sdk-for-python/issues/23865

/check-enforcer override


Gradle Compiler Plugin - Exclude does not work at the ruleset level

As already discussed here: https://github.com/detekt/detekt/discussions/6604
Expected Behavior
naming:
  active: true
  excludes: [ '**/src/**' ]
  VariableNaming:
    active: true
    variablePattern: '[a-z][A-Za-z0-9]*'
    privateVariablePattern: '(_)?[a-z][A-Za-z0-9]*'
    excludeClassPattern: '$^'

Should exclude all files in "src".
Observed Behavior
It is not excluded.
Steps to Reproduce
I created a minimal example here: https://github.com/mrclrchtr/gradle-detekt-compiler-plugin-bug with the exclusion here https://github.com/mrclrchtr/gradle-detekt-compiler-plugin-bug/blob/7da17a8ed8e6e368a7049f9d8640ff2874d4b74e/config.yml#L16 and the failing build here: https://github.com/mrclrchtr/gradle-detekt-compiler-plugin-bug/actions/runs/7832556886/job/21371525166
Context
I tried to exclude generated sources.
Your Environment

Version of detekt used: 1.23.5
Version of Gradle used (if applicable): 8.5
Gradle scan link (add --scan option when running the gradle task):
Link to your project (if it's a public repository): https://github.com/mrclrchtr/gradle-detekt-compiler-plugin-bug

I didn't dig on this issue but I think that the problem is that the compiler plugin doesn't provide the full path of the files, just the relative path. For that reason when you exclude **/src/** it does nothing because on the path that detekt core receives there is not /src/.
I'm not sure how difficult this would be to fix but I assume that we don't have anything like this in the compiler plugin:
https://github.com/detekt/detekt/blob/3c6eee5df6d7cea00055d3bf258393a0b2108812/detekt-parser/src/main/kotlin/io/github/detekt/parser/KtCompiler.kt#L29-L46

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

ping

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

ping


Specify default configuration extension on save

Fixes issue #610 by specifying default extension in QFileDialog for config saves

It seems like this PR also has several changes related to textured markers in it; did you mean for those to be included?


Fatal error throwing empty initializers should be removed

Describe the bug
Many Node types like WhiteNoise can no longer be initialized with an empty initializer like WhiteNoise() yet this is allowed at compile time and only detectable when run and a fatal error is thrown at run time.
To Reproduce
Use an initializer like WhiteNoise()
Expected behavior
Unsupported initializer should be removed or have better guardrails or guidance added to them.
Additional context
This applies to the latest AK4 release as well.

Latest AK4 release "as well" What do you mean by "as well"? Is this happening elsewhere?

On my AK5 build this doesn't raise any compile time issues but seemed to crash at runtime. The same was true with my AK4 build.

This does not seem to be the case in the Cookbook's NoiseGenerators Recipe: https://github.com/AudioKit/Cookbook/blob/main/Cookbook/Cookbook/Recipes/NoiseGenerators.swift#L14
Runs just fine, so what's different on your system.

@megastep seems to be a big headache for v4. As for V5, I'll close this issue until more information comes in.

Here is on 4.11.2

Switching from WhiteNoise() to WhiteNoise(amplitude: 0.5) stopped the crash.
I thought I had tested with with AK5 and observed the same but may be been confused. I'll double-check as soon as I can but I recommend reopening this issue since many will continue to stay on AK4 until AK5 is stable and this does appear to be a regression.

This never happened on 5. I seem to remember something like this with objective-C.

This somehow started to happen when switching from regular universal frameworks to XCFrameworks. Really not too clear on what would trigger this.


How can I evaluate neuralangelo in RTK4090 without memory out error

Describe the bug
Thank you for your great work in sdfstudio, especially the latest work neuralangelo. However, I found that neuralangelo consume too much GPU memory when evaluation(no memory out error in trainning) which resulted in can not run in the RTK4090 with 24GB memory.
To Reproduce
The running command is: ns-train neuralangelo --experiment-name neuralangelo-truck-lego --vis wandb --trainer.steps-per-eval-image 200 --trainer.max-num-iterations 300001 --pipeline.datamanager.train-num-rays-per-batch 512 --pipeline.datamanager.eval-num-rays-per-batch 512 --pipeline.model.sdf-field.use-appearance-embedding False --pipeline.model.sdf-field.inside-outside False --pipeline.model.background-model none --pipeline.model.background-color white --pipeline.model.sdf-field.bias 0.1 --pipeline.model.level-init 4  blender-data --data ./nerf_synthetic/nerf_synthetic/lego
Expected behavior
No memory out error in both training and evaluation.
Screenshots

Hello, I found that the --pipeline.model.eval-num-rays-per-chunk parameter may have a greater impact on GPU memory usage.
Also，I use the following commands which can run normally on the 32G V100:
ns-train neuralangelo  --trainer.steps-per-eval-batch 10000   --trainer.steps-per-eval-image 50000  --pipeline.model.sdf-field.inside-outside False    --pipeline.model.eval-num-rays-per-chunk 1024    --vis tensorboard

Yes, neuralangelo evaluation can run normally without memory out error with your adviced setting  --pipeline.model.eval-num-rays-per-chunk 1024.
Thank you very much.


fix: show L format for a previous day LTS for current day

The current code keeps showing the LTS format if the dialog is less than 24 hours old, but this means you can have time stamps in the list that appear to be in the future which should not be possible.
Example: I create a dialog yesterday at 4:15 PM and I look in the list today and still see 4:15 PM even though it's only 12:50 PM. This is because it still less than 24
This changes is so it only shows LTS for dialogs created on the current day. Examples below:
Occurred today: "11:59:24 AM"
Yesterday or before: "02/14/2019"

Did you only request changes to indicate @SDESkowronski should see this or what changes did you want?
What was his opinion? This doesn't change how it looks only the conditions it's shown in so if he has code looking specifically for the 'LTS' or 'L' format I would think they would still work.

Hmm, I would be interested to see the sites that were presenting times like this. I don't understand how you would understand that an item was yesterday.  If you have no other time related information about the item and you didn't create it or don't remember it would look like: A is 4:00 PM and B is 12:30 PM even though A is yesterday and B is actually after / later than A.
As for tests that seems to restrictive.  I would imagine there is a test for times like "Train Dialogs created less than or equal to 24 hours ago show L format" and "Train dialogs created more than 24 hours ago show LTS format".  All the other tests shouldn't care about the format only that the time is correct since the above test already cover that.  I would say just test for both for both formats. I assume it's getting the text content and trying parse it back into a time.  Can try L and if it doesn't match try LTS and if it still doesn't match fail.

:tada: This PR is included in version 0.341.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Mitigate array-of-statuses' impact on JSON bodies

fixes a regression in #715 / v6.1.4 which introduces a breaking change where any array passed as the first argument to .expect() is treated as an array of status codes, rather than as a body which was the previous behaviour. This tries to minimise the impact of status code arrays.
Given, this will still break if the expected body is an array of numbers, but it will at least break less, and perhaps array bodies can be deprecated/removed in the next major release providing proper notice of the change?

v6.1.5 released, can you further improve this by ensuring that besides being a number, they are one of the valid status codes as well in the 200-500 range?

[].every(val => typeof val === 'number'))  still return true, so incases where we are expecting the body to be empty array, this wrongly identifies the intention to be for testing status codes.
Changing the condition to if (Array.isArray(a) && a.length > 0 && a.every(val => typeof val === 'number')) { should fix that. (Basically add a check to be falsy when array length is zero as .every is truthy for empty array https://en.wikipedia.org/wiki/Vacuous_truth)

I have raised a PR for that https://github.com/visionmedia/supertest/pull/735 @niftylettuce

v6.1.6 released

thanks, I'm not sure limiting to 200-599 is the correct route to me, sure it'd probably mitigate more cases, but http status codes can technically be any number from 0 to 999, for example that's how the fetch spec treat then, that way it feels like it could just lead to more confusion why [200, 204] works, but suddenly when you add your own vendored status code [200, 204, 609] it doesn't


EFS - VolumeDriver.Mount: docker-legacy: Mount: vol: failed: ThrottlingException: Rate exceeded status code: 400

Summary
Hi !
I'm using the docker plugin rexray EFS and I have some errors.
Bug Reports
When my docker service try to mount a volume, I got this error :
VolumeDriver.Mount: docker-legacy: Mount: vol: failed: ThrottlingException: Rate exceeded status code: 400
I read some others issues and it's due to the AWS API calls.
Version
I use the docker plugin version 0.11.4 with the defaults parameters.
Question
I would like to know, which API call must be increased by the AWS support to fix this issue ?
Thanks

check this link.
https://github.com/rexray/rexray/issues/1282#issuecomment-1602546601

Same problem here 😭

Hi @JCenatus , I don't have the solution yet, but I use EBS for temporary fixing.
It supposes that you have only one replicas of your service.
For example :
volumes:
 elasticsearch:
    driver: rexray/ebs:0.11.4
    name: "elastic-vol"
    driver_opts:
      size: 32


cartesian position control

Here, https://frankaemika.github.io/docs/libfranka.html#realtime-commands it says that motion generator can be for Cartesian coordinates also, but from here, https://frankaemika.github.io/libfranka/classMotionGenerator.html it seems that this is only for joint space angles, can you please specify is it possible to do motion generation in Cartesian coordinates.

As the docu says, the MotionGenerator class is only an example. See https://frankaemika.github.io/libfranka/examples.html (or directly in source code) for more examples (generate_*) using all four motion generator interfaces.


Localization of date in header/footer

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.3.0
Platform / OS version: Windows 10
URLs (if applicable): ..
Node.js version: 9.11.1

What steps will reproduce the problem?
Using date in header/footer.
Please include code that reproduces the issue.
await page.pdf({
      //..
      displayHeaderFooter: true,
      headerTemplate: '

',
      footerTemplate: '',
      margin: { bottom: '60px' }
});
What is the expected result?
18-04-2018
What happens instead?
4/18/2018
Or simply put.. I'm located in The Netherlands where date's are normally formatted like dd-MM-yyyy.
Is it possible to get some kind of localization in the header/footer?

@yrshaikh Hmm.. Strangely for a moment I was looking at the templates as being some static kind of thing in stead of being programmable. Sometimes you get blind for something and hold on to it.
Thanks for the input.
I'll close the issue.

{ format: "A4", printBackground: true, displayHeaderFooter: true, headerTemplate:
${new Date().toDateString()}
Generated PDF
, margin: {   top: '38px',   right: '38px',   bottom: '38px',   left: '38px' } }

@Pankajtanwarbanna The one which you added did not work for me. It prints ${new Date().toDateString() this as it is in pdf. Am I missing anything?

Later, I stored the value in a variable first, and then used ${print_this_date} instead of ${new Date().toDateString()}


Roll Flutter Engine from 965501a25d92 to c3f57b77f611 (3 revisions)

https://github.com/flutter/engine/compare/965501a25d92...c3f57b77f611
2023-08-24<EMAIL_ADDRESS>Roll Skia from<PHONE_NUMBER>89 to 177e8477faf9 (1 revision) (flutter/engine#45076)
2023-08-24<EMAIL_ADDRESS>Turn on the skia_enable_optimize_size flag to save a bit of binary size (flutter/engine#45029)
2023-08-24<EMAIL_ADDRESS>Roll Skia from b17ee34f3378 to<PHONE_NUMBER>89 (1 revision) (flutter/engine#45075)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Alphabet rendering issue

VSCode Version: 1.39.0-insider
OS Version: macOS Mojave 10.14.6

Steps to Reproduce:

If you type "f" and "l" or "f" and "I", you will see these two characters stick together.

Does this issue occur when all extensions are disabled?: Yes

Your font has ligatures enabled. Set "editor.fontLigatures": false


fix(amplify-category-api): check to see resolverConfig exists

if resolver config exists and it either has syncconfig for a project for model(s) then write it to
the transform conf
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #3045 into master will decrease coverage by 2.09%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #3045     +/-   ##
=========================================
- Coverage   63.28%   61.18%   -2.1%
=========================================
  Files         218      223      +5
  Lines       11078    11458    +380
  Branches     2157     2370    +213
=========================================
  Hits         7011     7011
- Misses       3759     4134    +375
- Partials      308      313      +5

Impacted Files
Coverage Δ

packages/amplify-cli/src/context-extensions.ts
25.69% <0%> (ø)
:arrow_up:

...lify-graphql-types-generator/src/compiler/index.ts
90.32% <0%> (ø)
:arrow_up:

...y-graphql-types-generator/src/compiler/legacyIR.ts
96.72% <0%> (ø)
:arrow_up:

...ages/amplify-appsync-simulator/src/server/index.ts
0% <0%> (ø)
:arrow_up:

...lify-appsync-simulator/src/velocity/util/errors.ts

...mplify-appsync-simulator/src/mqtt-server/server.ts
0% <0%> (ø)

...y-appsync-simulator/src/velocity/util/map-utils.ts
0% <0%> (ø)

...mplify-appsync-simulator/src/mqtt-server/client.ts
0% <0%> (ø)

...simulator/src/data-loader/dynamo-db/utils/index.ts
0% <0%> (ø)

.../amplify-appsync-simulator/src/mqtt-server/util.ts
0% <0%> (ø)

... and 1 more

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 7f5d320...6783ba7. Read the comment docs.


[10.x] Fix Redis store set int value, but get returns string

Fix the issues in #31345
Cache::put('test', 1);
Cache::get('test'); // expected: 1, actual: "1"
Cache::put('test', 1.0);
Cache::get('test'); // expected: 1.0, actual: "1"

//  throws an error on the second call
function foo(): int
{
    return Cache::remember('foo', 60, function() {
        return 1;
    });
}

Implement

Store raw value of int and float to Redis
Use "." to distinguish int and float
Serialize other type(including string) before store to Redis

BREAKING CHANGES

Reading existing numeric values in Redis will be converted from string to int or float
Increment new stored string value wont work, use int instead

Cache::set('test', '1'); // store "s:1:"1";" in Redis
Cache::increment('test');  // (error) ERR value is not an integer or out of range

To avoid breaking changes, we can add option like keepNumeric/keepType ? to enable this feature.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


The months and weekdays are shown in English regardless of current configured language

Expected behaviour: months and weekdays are shown in language according to configured via configureUiKit.

@dfilatov Hi!
It is a little bit tricky to change locale for DatePicker because of date-utils usage. Here you can find an example of locale changing.
You can reopen this issue if you still have questions.

@korvin89 It works, thank you!


Change signature of AugmentedMap constructor

Fixes #23

@rultor merge

@rultor merge

@suseika OK, I'll try to merge now. You can check the progress of the merge here

@rultor release, tag=0.8.0

@rultor merge

@suseika Done! FYI, the full log is here (took me 2min)

@rultor release, tag=0.8.0

@suseika OK, I will release it now. Please check the progress here

@rultor release, tag=0.8.0

@suseika Done! FYI, the full log is here (took me 4min)


iter_content slow with large chunk size on HTTPS connection

https = requests.get(url, stream=True)
for content in https.iter_content(100 * 2 ** 20): # 100MB
    pass

This causes 100% CPU and slows down throughput to less than 1MB/s.

Sorry, but right now you haven't given us anything like enough to go on. Please answer all of the following questions:

How does this behave on smaller chunk sizes?
What about larger ones?
What version of Requests are you using?
What version of Python are you using?
Can you share the URL in question?
What is the throughput when using .content directly?
What do the response headers include?

I had wrongly assumed it's an universal bug affecting every HTTPS connection, which is why I didn't provide further information. I've done further investigation and it seems to affect only some CDNs, like Akamai and Azure.
Example (affected). Counter-example (not affected).
At 10MB, there is no noticeable increase in CPU load, and no impact on throughput. At 1GB, CPU load is at 100% as well, but throughout is reduced to below 100KB/s, which makes me believe that CPU is not the primary reason for this problem.
I'm using the latest version on Python 2.7.12, installed through pip.

Is this true even with empty loop bodies as in your sample code? I ask because my normal instinct here would be that the repeated context switches back into Python are causing your performance problem, but larger chunks shouldn't exhibit that problem at all: they should be proportionally more efficient than the smaller chunk sizes.
This suggests that the problem is the manipulation of increasingly large chunks of memory. If this happens to you even with empty loop bodies then we have something really interesting to have to investigate.
Do you have access to Python 3 on your machine?

Is this true even with empty loop bodies as in your sample code?

Yes.

Do you have access to Python 3 on your machine?

I've tried Python 3.4.3 with requests 2.2.1 as provided by Ubuntu repositories, and the problem does not occur, not even at 1GB chunk size.
I have figured out why. When I Ctrl-C the connection, the stack traces hold the clue.
Python 2.7.12
File "/home/user/.local/lib/python2.7/site-packages/OpenSSL/SSL.py", line 1299, in recv
  buf = _ffi.new("char[]", bufsiz)

Python 3.4.3
File "/usr/lib/python3.4/ssl.py", line 641, in read
  v = self._sslobj.read(len, buffer)

Maybe this works.
$ pip uninstall PyOpenSSL

Python 2.7.12 again.
File "/usr/local/lib/python2.7/ssl.py", line 643, in read
  v = self._sslobj.read(len)

And the problem does not occur. No decrease in throughout or increase in CPU load.
It still doesn't answer the question why it manifests itself only with large chunk size in the first place.

I've also figured out why the counter-example provided a few posts earlier does not exhibit the problem: it redirects to a HTTP server.

I've only installed PyOpenSSL to silence the InsecurePlatformWarning on Python 2.7.6, which I've since replaced with self-compiled Python 2.7.12, so PyOpenSSL is now obsolete and the problem fixed for me.
I think there is still a bug here somewhere related to chunk-size.

Unfortunately I just noticed that some pip-installed packages do require PyOpenSSL.

Ok, so then the next question is: is this merely a case of CFFI buffer allocation being slow, or is there something else going on? I'll need to investigate this further, as PyOpenSSL may not be at fault directly: it may be something to do with how we use it.

@pdknsk It would be interesting to see if you continue to have this problem on Python 3 with pip install requests[security]. That will also bring in PyOpenSSL, which should cause the problem again.

So, a quick test reveals that our use of CFFI is almost certainly the bottleneck here. I ran the following script (note the import from PyOpenSSL's FFI object, this was intended only for consistency and ease):
from OpenSSL._util import ffi

MB = 1024 * 1024

def test(alloc_size):
    buf = ffi.new("char[]", alloc_size)

import timeit

for size in [1, 10, 100, 1000]:
    print "Doing allocations of {}MB".format(size)
    size = size * MB
    print timeit.repeat(
        "test({})".format(size), setup="from __main__ import test", number=100
    )

That gave me the following results:
% python test_cffi.py
Doing allocations of 1MB
[0.004097938537597656, 0.003545999526977539, 0.003551006317138672]
Doing allocations of 10MB
[0.07877898216247559, 0.07785892486572266, 0.06536316871643066]
Doing allocations of 100MB
[3.7023580074310303, 3.6858859062194824, 3.602185010910034]
Doing allocations of 1000MB
[65.49478697776794, 67.42595791816711, 67.89322805404663]

Note, essentially, that increasing the allocation size increases the cost of allocation by a factor of between 20 to 40. That's really quite unexpected. The next step is going to be trying to work out if this is a problem with CFFI or with malloc more generally.

So I can't reproduce this behaviour in plain C code. That suggests that the problem is elsewhere. We might need to look at CFFI here.

Ok, this is kinda CFFI's fault, but not really.
When we do the initial buffer allocation CFFI kindly gives us a zeroed buffer. This zeroed buffer requires something that is basically the equivalent of malloc followed by memset (it's actually calloc, but that's neither here nor there). That means that allocations of buffers like this in CFFI have an approximately linear cost in the size of the allocation.
So, this isn't really CFFI's fault. Its strategy is reasonable. This is a problem higher up the stack. In the case of reading from a socket we don't need a zeroed buffer: we know how many bytes the code will write into it, so we can avoid doing anything silly. That's why CPython doesn't encounter the issue: I think they just directly malloc the underlying byte buffer.
I guess the first line of attack would be to see if CFFI is interested in adding a non-zeroed buffer acquisition API. That, however, only solves the problem on newer CFFIs.
Another option is to adjust the logic in either PyOpenSSL or Requests to quietly ignore really large arguments to recv and to clamp them to the size of the socket buffer in the kernel. The socket buffer represents a hard upper limit on how much data the kernel can actually return in one call to recv on a socket. So we could pursue that fix as well.

I read the CFFI documentation briefly yesterday and noticed that ffi.new returns zero-initialised memory, but didn't mention it because I'm unfamiliar with all the machinery involved here, so wasn't sure that's the performance limitation.
What I also noticed is function ffi.new_allocator which provides a way to use malloc directly.

Yup, this has already been proposed as an alternative solution in the relevant pyopenssl issue.

I just tried the benchmark you posted earlier, with the line from pyca/pyopenssl#577, and it returns instantly regardless of allocation size, as you'd except. Problem solved.

Ok, the underlying issue at PyOpenSSL has had a fix merged. This should drastically improve our performance in this edge case. Thanks for the report @pdknsk and thanks for working with us to narrow down the problem! :sparkles:

It's interesting how a rare problem produced a fix with much wider application. I figure this will save quite a bit of CPU and thus electricity, and perhaps even yield noticeable performance improvements for every HTTPS connection on slow devices. A problem well worth reporting.

...What's particularly mysterious here is that for large buffers (these definitely qualify), calloc should be about as cheap as malloc, and much cheaper than malloc+memset! Once you pass the size threshold where the heap allocator decides to go to the kernel to satisfy the request (128 KiB for glibc by default), then the kernel returns lazily-zeroed memory regardless. So malloc and calloc should be indistinguishable, and both should be O(1) because the kernel will zero the memory lazily as its faulted in.

...darn, I got curious and investigated further. I have poor impulse control. Anyway, this is totally a bug in cffi: https://bitbucket.org/cffi/cffi/issues/295/cffinew-is-way-slower-than-it-should-be-it

...annnnnd it's also a bug in MacOS: https://github.com/pyca/pyopenssl/pull/578#issuecomment-264800709
@pdknsk what have you done

So @pdknsk, after further investigation prodded by @njsmith, you've actually stumbled onto two bugs that are wasting a lot of CPU and electricity.
The first is that CFFI should be using more efficient allocation. The second, and much more important, is that there is a region of allocations between about 128kB and 125MB where macOS (and, presumably, iOS/watchOS/tvOS as well) forcibly page-in and zero memory when they don't need to, wasting a ton of CPU to do it.
Nicely done!

This bug now also filed as rdar://29508271


Make Revert button work

@ritikpal1122 we dont do assignments
/bounty $8

/attempt #8
I’ll implement the functionality for the Revert button to restore the previous state of changes. The plan includes adding the necessary event handler, managing the state to enable reverting, and testing for various use cases. I’ll submit a PR with /claim #8 upon completion. Let me know if you have any suggestions!

/attempt #8


Control-Meta-Enter is not working

I have tried a number of combinations and none of the pushes the code from the editor panel to run in the console (which works just great in the python plugin but with different shortcut).
I am on macos high sierra on 2017.1.5 with the latest R plugin.

This functionality was refactored out into a separate plugin called "Send to Terminal"

As mentioned in the Documentation bug : this error is about not pushing code from the editor panel to the R console (not bash). I have tried numerous times and it does not work.

When using cmd+shift+a to browse all actions, search for evaluate which should list it along with the current key binding:

This is interesting: running it that way  ie from the Search Actions
DOES work.  But invoking ctl-shitf-enter does .. nothing .
2018-02-18 11:34 GMT-08:00 Holger Brandl<EMAIL_ADDRESS>
When using cmd+shift+a to browse all actions, search for evaluate which
should list it along with the current key binding:
[image: image]
https://user-images.githubusercontent.com/200952/36355929-1db2ca26-14eb-11e8-9fae-99556172c23f.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/holgerbrandl/r4intellij/issues/144#issuecomment-366541082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZEkX0ncxg2LWjoTWGc8UQsm5cdkTO7ks5tWHtNgaJpZM4R4tKR
.

Did you make sure that the focus is in the R editor in a line that contains code (or that some code is selected)? For debugging purpose you may want to try it also in a document without syntax errors.

This is actually not the problem: it is that the console is not scrolling - so I did not realize anything were happening.  (I opened a separate bug).


rev-parse --symbolic-full-name master@{u}: fatal: no upstream configured for branch 'master'

VSCode Version: 1.42.1
OS Version: Windows 8.1

Steps to Reproduce:

Create local repo with no upstream.
Work on the repo.

Git output:
> git status -z -u
> git symbolic-ref --short HEAD
> git rev-parse master
> git rev-parse --symbolic-full-name master@{u}
fatal: no upstream configured for branch 'master'
> git for-each-ref --format %(refname) %(objectname) --sort -committerdate
> git remote --verbose
> git config --get commit.template

Does this issue occur when all extensions are disabled?: Yes
Related: https://github.com/arialdomartini/oh-my-git/issues/14

Is this causing a problem? I think an error message in the git log by itself isn't an issue.

Actually there is no problem. Just a suggestion to consider, maybe vscode can add some kind of logic to check whether it has upstream or not before running that command? Imho, simple edit and save should not trigger this message, unless the user explicitly performs an action that requires the existence of upstream.
Come to think of it, it looks like the title has to be changed to match what I explained above.

This is fine.


Allow toggling Buddy Project matching

Requires https://github.com/Yes-Theory-Fam/yestheory.family/issues/146

Completed in https://github.com/Yes-Theory-Fam/yesbot-ts/pull/580/commits/715854fb8396a9a6a1bbbf10b6d2aa712a62c9d4


Plots not showing up

Evaluating in atom the lines
using Gadfly
plot(sin, 0, 1)

results in the last line returning a Plot object, but nothing is displayed in the browser. The same happens with PyPlot.
Is this a bug or a feature not implemented yet? In the latter case sorry for the noise.
Bye,
Carlo

Just not implemented yet

This will work for Gadfly now, I'm working on getting a plotting pane + other libraries working.

it  does work in atom. any solution?
using Gadfly
xvalues = rand(1000);
yvalues = rand(1000);
Gadfly.plot(x=xvalues,y=xvalues,Geom.point,Geom.line)
Gadfly.plot(x=1:10, y=2.^rand(10),Scale.y_sqrt, Geom.point, Geom.smooth,
Guide.xlabel("Stimulus"), Guide.ylabel("Response"), Guide.title("Dog Training"))

see https://github.com/JunoLab/Juno.jl/issues/583


[Feature Request] HIGH SCORES

Is your feature request related to a problem? Please describe.
I can't find a way to show players' HighScores between sessions. When I exit the program and run it again, the previous statistics are not displayed. I would like to be able to know and see my record at all times, and not just by session.
Describe the solution you'd like
Show ./save/LocalProfiles/00000xxxx/Stats.xml parsed, similar to how this web shows http://dev.kserver.dy.fi/smpv/#
And also show the three best results of each song somewhere on the game screen to know what score I should beat at my next dance.
Describe alternatives you've considered

Additional context

This is something we will be doing in the upcoming new default theme.


[beta] Fix publication of packages with metadata and resolver

This is as backport of #9300

r? @Eh2406
(rust-highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.52.0. Please double check that you specified the right target!

@bors r+

:pushpin: Commit 129f4da371f998d1ec67f8774f4be32c243d0b8f has been approved by Eh2406

:hourglass: Testing commit 129f4da371f998d1ec67f8774f4be32c243d0b8f with merge 69b5bbca217677124882a91146eece80ee85bd84...

:sunny: Test successful - checks-actions
Approved by: Eh2406
Pushing 69b5bbca217677124882a91146eece80ee85bd84 to rust-1.52.0...


Your MB not on the list, then failed to load drivers.

Hi. I got msi b450-pro-m2-max. I am trying to load the program and following all the instructions exactly but, when I start msirgb.exe it says your motherboard not on the list, and then I click to continue, it fails again and warns me that failed to load drivers. What can I do ? Help pls.

Do you have any anticheat installed? Vanguard, FACEIT, etc.

Nope. I don't have any of them.

Can anyone please help me? :(

https://github.com/ixjf/MSIRGB/issues/132#issuecomment-674405037
Can you try this debug build and tell me the output of the debug log? (Explained there how to do that)

The same thing happend.
Debug_log :
start_drv: StartService failed with error 647
load_drv: start_drv() != ERROR_SUCCESS
load_drv failed

This sort of error is usually caused by some program interfering with MSIRGB's driver. Can you try running MSIRGB in a sort of 'safe mode' boot? Just disable all non-Windows services and startup items (from 'msconfig' and Task Manager) and reboot, then try.

Nope :( Sorry but the same thing happened again.  Thanks for your struggle. Happy new year.

I guarantee that's not a problem with your motherboard, though it is possible MSIRGB won't support your motherboard either way. If you have any games installed, I suggest you check if they don't include any anti cheat.
Happy New Year.

Did you fix this?

Nope I didn't

Can you open a command prompt and run these commands?
tasklist > userprc.txt```

It should create the files 'drivers.txt' and 'userprc.txt' in the directory where you ran the command. Please upload these.

 :( I really appreciate your effort but as I said before, this motherboard sucks.

These commands are not meant to fix your issue, but to help me figure out what is causing it.

Will you try this, or should I close the issue?

I'm closing the issue then.


"How To Write Unmaintainable Code"

funny list of specific techniques for writing unmaintainable code

I guess this was meant: https://www.doc.ic.ac.uk/~susan/475/unmain.html


Operator crashes when numShards is omitted from the AtlasDeployment resource

What did you do to encounter the bug?
We are trying to use the operator with an Atlas project and cluster that are managed by Terraform with the sole purpose of managing database users. I've created minimalist AtlasProject and AtlasDeployment custom resources like this:
---
apiVersion: atlas.mongodb.com/v1
kind: AtlasProject
metadata:
  name: test-uk
spec:
  name: test-uk
---
apiVersion: atlas.mongodb.com/v1
kind: AtlasDeployment
metadata:
  name: test-uk-test-uk
spec:
  advancedDeploymentSpec:
    name: test-uk
    # replicationSpecs:
    # - numShards: 1
  projectRef:
    name: test-uk

The goal was to add as little as possible given that this is all managed elsewhere. I've found that if I omit the numShards parameter (the lines commented out above) then the custom resource will pass validation but the operator will crashloop with a panic.
What did you expect?
I expect the operator to not manage the aspects of the database cluster I omit from the spec.
What happened instead?
The operator crashed with the following panic message and continued to crash after leader election each time after Kubernetes restarted it:
mongodb-atlas-operator-6db86785d8-4mtsq manager {"level":"INFO","time":"2023-10-24T09:57:58.391Z","msg":"Status update","atlasdeployment":"default/test-uk-test-uk","lastCondition":{"type":"ValidationSucceeded","status":"True","lastTransitionTime":null}}
mongodb-atlas-operator-6db86785d8-4mtsq manager {"level":"INFO","time":"2023-10-24T09:57:58.405Z","msg":"Observed a panic in reconciler: runtime error: index out of range [0] with length 0","controller":"AtlasDeployment","object":{"name":"test-uk-test-uk","namespace":"default"},"namespace":"default","name":"test-uk-test-uk","reconcileID":"d96a5d1d-e829-422a-b45e-ea6c58fbd65f"}
mongodb-atlas-operator-6db86785d8-4mtsq manager panic: runtime error: index out of range [0] with length 0 [recovered]
mongodb-atlas-operator-6db86785d8-4mtsq manager 	panic: runtime error: index out of range [0] with length 0
mongodb-atlas-operator-6db86785d8-4mtsq manager
mongodb-atlas-operator-6db86785d8-4mtsq manager goroutine 342 [running]:
mongodb-atlas-operator-6db86785d8-4mtsq manager sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile.func1()
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:119 +0x1a4
mongodb-atlas-operator-6db86785d8-4mtsq manager panic({0x12cdc00?, 0x40008a1b60?})
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/opt/hostedtoolcache/go/1.21.3/x64/src/runtime/panic.go:914 +0x218
mongodb-atlas-operator-6db86785d8-4mtsq manager github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment.MergedAdvancedDeployment({0x40006cdf98, 0x4000ead638, {0x40006cdfc0, 0xa}, 0x40002f2850, 0x40006cdfd0, {0x40006cdfe0, 0x4}, {0x40008a1a70, 0x18}, ...}, ...)
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/work/mongodb-atlas-kubernetes/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment/advanced_deployment.go:129 +0x300
mongodb-atlas-operator-6db86785d8-4mtsq manager github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment.advancedDeploymentIdle(0x400031c500, 0x400002e000, 0x40005a2480, 0x4000de1680)
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/work/mongodb-atlas-kubernetes/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment/advanced_deployment.go:81 +0x10c
mongodb-atlas-operator-6db86785d8-4mtsq manager github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment.(*AtlasDeploymentReconciler).ensureAdvancedDeploymentState(0x0?, 0x400031c500, 0x400002e000, 0x40005a2480)
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/work/mongodb-atlas-kubernetes/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment/advanced_deployment.go:65 +0x644
mongodb-atlas-operator-6db86785d8-4mtsq manager github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment.(*AtlasDeploymentReconciler).handleAdvancedDeployment(0x164d7c8?, {0x164a810, 0x4000bd7ce0}, 0x400031c500, 0x400002e000, 0x40005a2480, {{{0x163b2b8?, 0x4000b411c0?}, {0x16335e0?, 0x4000b411d0?}}})
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/work/mongodb-atlas-kubernetes/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment/atlasdeployment_controller.go:379 +0x58
mongodb-atlas-operator-6db86785d8-4mtsq manager github.com/mongodb/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment.(*AtlasDeploymentReconciler).Reconcile(0x400040ecf0, {0x164a810, 0x4000bd7ce0}, {{{0x4000f80f50, 0x7}, {0x4000f80f40, 0xf}}})
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/work/mongodb-atlas-kubernetes/mongodb-atlas-kubernetes/pkg/controller/atlasdeployment/atlasdeployment_controller.go:191 +0x1b3c
mongodb-atlas-operator-6db86785d8-4mtsq manager sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile(0x164a810?, {0x164a810?, 0x4000bd7ce0?}, {{{0x4000f80f50?, 0x110e280?}, {0x4000f80f40?, 0x0?}}})
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:122 +0x8c
mongodb-atlas-operator-6db86785d8-4mtsq manager sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler(0x40003a4500, {0x164a848, 0x4000315680}, {0x12258c0?, 0x4000991460?})
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:323 +0x29c
mongodb-atlas-operator-6db86785d8-4mtsq manager sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem(0x40003a4500, {0x164a848, 0x4000315680})
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:274 +0x198
mongodb-atlas-operator-6db86785d8-4mtsq manager sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2()
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:235 +0x74
mongodb-atlas-operator-6db86785d8-4mtsq manager created by sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2 in goroutine 87
mongodb-atlas-operator-6db86785d8-4mtsq manager 	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:231 +0x43c

Operator Information
image: docker.io/mongodb/mongodb-atlas-kubernetes-operator:1.9.0
Kubernetes Cluster Information
EKS version: v1.27.4-eks-2d98532

Hi, @viktorradnai
Sorry to hear that you are having issues using the Atlas Operator.
We will look into avoiding those panics, but the fix will probably be done by rejecting the Custom Resource definition at validation time, which will not solve your issue.
The Atlas Operator is designed to fully own an Atlas project & deployment combo, including its dependencies, like user definitions.  This is by design as Operators are usually thought to fully automate the deployment and live management of infrastructure as code setups.
In this case we would suggest you can either manage everything from Terraform or allow the Operator to fully manage the deployment.

Hi @josvazg , thanks for the quick reply. The validation fix makes sense based on the use cases you are intending the operator for.
Also thanks for clarifying the design intention. It seems like the operator isn't a great match for our use case and we will not be adopting it at this point but we'll follow its development.

I am closing this one now @viktorradnai, but please, do no hesitate to re-open if you have related concerns.


show documentation when writing function like in brackets editor,

Please provide more information about this feature request, like screenshots, samples, etc.

in brackets editor when writing code, let say
a file scripts.js
we write console, the code editor (brackets ) show the documentation
The console object provides access to the browser's debugging console, the specifics of how it works varies from browser,.. mdn web docs

We have that with IntelliSense and SignatureHelp


namespace for numeric constants

Initial attempt to enclose constants in namespace.
Used constants:: when only few constants where needed, but using in the function when more needed.
In the cpp files, using added to the top of the file.

Does that fix the warnings?

No all of them.
Here is another:
In rectangle.h function inline bool operator< the input parameter name is b which is same as the field with the same name in that class:

warning C4458: declaration of 'b' hides class member
note: see declaration of 'dlib::rectangle::b'

I think in this case for example, the b in the argument list should be just renamed to rect

I can send more PR while I encounter them, or find a way to reveal them all and one PR?

You might as well wait until the PR covers all the warnings that are bothering you :)


I have successfully run this code, but it did not respond to "[[self rac_signalForSelector:@selector(collectionView:didSelectItemAtIndexPath:) fromProtocol:@protocol(UICollectionViewDelegate)] subscribeNext:^(RACTuple *arguments)"method when I clicked item in collectionView,how to solve it?

In the previous code, this method can be responded, and I do not know why in the new code it doesn't respond to this method,my environment is xcode7,iphone6plus.

Hey there! Could you paste the full view controller code? It's probably setting the delegate at the wrong time – common problem to have!

i just download it,then run the project , and i found the problem,i don't know why

Could be something that broke with Xcode 7. I'll check it out and get back to you.

Thank you .

The project is OK?

i added one line code to reset the delegate to nil, then reset it to self again. It's OK now.
but i don't know why.
// Need to "reset" the cached values of respondsToSelector: of UIKit
    self.collectionView.delegate = nil;
    self.collectionView.delegate = self;

@calvingit Interesting. UIKit queries the delegate/datasource when you set it to determine if it responds to different selectors (optional protocol methods). Since the RAC delegate changes the view controller's answers to respondsToSelector:, it must need to be reset. This is likely a change in iOS 9 or so that wasn't needed with the project before that.
I see that you made a commit that fixes this issue on your fork – would you mind sending a pull request? I'd love to merge it into this repo!


fix: retriever module follow LSP, utils function use GPT_3_5_TURBO

Description

The original design of vector retriever didn't follow Liskov Substitution Principle
recreate_collection method will be removed in the future by Qdrant.
test of bm25_retriever is making real http request, which is slow
utils function used GPT_4_TURBO before but didn't switch to ChatGPTVisionConfig with the new featurehttps://github.com/camel-ai/camel/pull/473

Motivation and Context

refactor the retriever module, move storage into attribute, make the class follow Liskov Substitution Principle
use create_collection instead of recreate_collection in Qdrant module
use mock to do the pytest for bm25_retriever
switch to GPT_3_5_TURBO, also can reduce the cost

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of example)

Implemented Tasks

[ ] Subtask 1
[ ] Subtask 2
[ ] Subtask 3

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[ ] I have read the CONTRIBUTION guide. (required)
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly. (required for a bug fix or a new feature)
[ ] I have updated the documentation accordingly.

Hi @Wendong-Fan . After some searches I suggest discarding the parent class and using Union to group the two retrievers. Below are my considerations:

We don't seem to need to allow the user to extend the type set, that makes an abc kind of useless but troubles ourselves since it either constrains the signature or allow useless key words (imagining user can use a non-exist argument threshold when querying bm25 without error, however it does not affect the final result).
In the current two retrievers, as we discussed, bm25 is strictly without optional kwargs but that's not the case for vectorretriever. In fact, they are less common when querying and should not share signatures.
As I understand we don't have too many different types of retrievers to implement. That makes a base more useless.

The drawback is when we query a retriever which only being annotated as base, we need to add a if statement to judge which retriever it exactly is if kwargs are passed.

Hi @Wendong-Fan . After some searches I suggest discarding the parent class and using Union to group the two retrievers. Below are my considerations:

We don't seem to need to allow the user to extend the type set, that makes an abc kind of useless but troubles ourselves since it either constrains the signature or allow useless key words (imagining user can use a non-exist argument threshold when querying bm25 without error, however it does not affect the final result).
In the current two retrievers, as we discussed, bm25 is strictly without optional kwargs but that's not the case for vectorretriever. In fact, they are less common when querying and should not share signatures.
As I understand we don't have too many different types of retrievers to implement. That makes a base more useless.

The drawback is when we query a retriever which only being annotated as base, we need to add a if statement to judge which retriever it exactly is if kwargs are passed.

Hey @ocss884 , thanks for your input,I agree with you that current it's not so meaningful by using abc, but I think having this base class with abc helps enforce a uniform interface and is useful for maintaining consistent if any contributor want to add new retriever types, even for now it's little bit over engineering. How about keep it for now for code consistency and extensibility?

Hi @Wendong-Fan . After some searches I suggest discarding the parent class and using Union to group the two retrievers. Below are my considerations:

We don't seem to need to allow the user to extend the type set, that makes an abc kind of useless but troubles ourselves since it either constrain the signature or allow useless key words (imagining user can use a non-exist argument threshold when querying bm25 without error, however it does not affect the final result).
In the current two retrievers, as we discussed, bm25 is strictly without optional kwargs but that's not the case for vectorretriever. In fact, they are less common when querying and should not share signatures.
As I understand we don't have too many different types of retrievers to implement. That makes a base more useless.

The drawback is when we query a retriever which only being annotated as base, we need to add a if statement to judge which retriever it exactly is if kwargs are passed.

Hey @ocss884 , thanks for your input,I agree with you that current it's not so meaningful by using abc, but I think having this base class with abc helps enforce a uniform interface and is useful for maintaining consistent if any contributor want to add new retriever types, even for now it's little bit over engineering. How about keep it for now for code consistency and extensibility? We also have one additional retriever under review #471

I check #471 and find in the cohere query, the signature is again different. In base we have top_k but it is missing in #471. That's to say, in fact, these retrievers has less in common in nature since the class itself is a wrapper of third-party functionalities. They must be less common in implementation and the only thing in common is that we wish them to do similar works but that doesn't mean they must have a parent.
Adding the cohere one now we have 3 retrievers, and two of them have a signature mismatching with base class and need to be addressed by type-ignore. It is a problem out of base class design.
Imaging we are implementing a function which exploits these retrievers, the retriever variable is only claimed as a base. Can we switch from one to another by only changing the instance type without changing query arguments? If not, it is in nature not somewhere for such an inheritance relationship for polymorphism. I even haven't mentioned there is zero code reuse.
In fact, I prefer to remove the base now, not only the ABC but my suggestion of using a union. I don't see any point of using that.

If you just want children retrivers are always implemented with process and query. I suggest to take a look at torch.nn.Module.
In short, just declare them as Callable and assign a default function which will raise a NotImplementedError. This way is tricky to get around the parent method signature and remind the user to implement their own method.
https://github.com/pytorch/pytorch/blob/e3b9b71684ae4f81d6128854b12771f83064c5ce/torch/nn/modules/module.py#L468.
But remember, this way does not guarantee children's methods have a uniform interface (in fact, they should not share). The purpose is code reuse.

LGTM! @Wendong-Fan pls address the rest comments


koreader: 2022.07 -> 2022.08

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/koreader/koreader/releases.
meta.description for koreader is: "An ebook reader application supporting PDF, DjVu, EPUB, FB2 and many more formats, running on Cervantes, Kindle, Kobo, PocketBook and Android devices"
meta.homepage for koreader is: "https://github.com/koreader/koreader"
meta.changelog for koreader is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2022.08 with grep in /nix/store/2jrkj8szrl34zkaqdyc7xv81c9slz1w2-koreader-2022.08
found 2022.08 in filename of file in /nix/store/2jrkj8szrl34zkaqdyc7xv81c9slz1w2-koreader-2022.08

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
koreader

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2jrkj8szrl34zkaqdyc7xv81c9slz1w2-koreader-2022.08 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A koreader https://github.com/r-ryantm/nixpkgs/archive/3305fd81772270a25d5b5706690c21f845470886.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2jrkj8szrl34zkaqdyc7xv81c9slz1w2-koreader-2022.08
ls -la /nix/store/2jrkj8szrl34zkaqdyc7xv81c9slz1w2-koreader-2022.08/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

koreader

Maintainer pings
cc @contrun @neonfuz for testing.

Result of nixpkgs-review pr 189147 run on x86_64-linux 1

1 package built:

koreader


rust Cardano

https://github.com/free-honey/naumachia

The Plutus-bench repo is a good example of how it is used https://t.co/LI9JOx5Cni

The Plutus-bench repo is a good example of how it is used
https://arxiv.org/abs/2303.05491


1.0.0 release changed the results of operations

And this causes the Assimilation project to fail about half of its tests.
It's possible this is the result of a bug fix. Will fill in details as I discover them. But it's definitely a change.
ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__AddrFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__IpPortFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__IpPortFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__IntFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__UnknownFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__SeqnoFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__SignFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame
E       ctypes.ArgumentError: argument 3: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__SignFrame
E       ctypes.ArgumentError: argument 3: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__SignFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame
E       ctypes.ArgumentError: argument 1: <class 'TypeError'>: expected LP_struct__Frame instance instead of LP_struct__CstringFrame

In this case, these are effectively subclasses of Frame (CstringFrame, SignFrame, etc). So, they are equivalent. That is, the first element in a CstringFrame is a the base class Frame. But it's a noticeable change.
@olsonse Was this intended?

It turns out there are several other changes where return results simply appear to be be wrong.
In one case, it appears that a return result of calling a function is quite different:
E       assert<PHONE_NUMBER>3709551607 == -9
E       AssertionError: assert None == 'b'
In any case, there are quite a few changes...

I started looking at the TypeError problems above. The message as given is likely correct.
However, when I started fixing it by adding a cast() call, I ran into other problems.
A function which is declared to return a gboolean (which is a gint, which in turn is a c_int) is treated as though it returns a pointer. This is a change from the former correct behavior, to the current incorrect behavior. This is almost certainly a real bug. I could add another cast to get around it, but it's pretty clearly broken as is.

I have put all the fixes into my code, but I'm still having trouble with function pointers that return plain basic values - in particular ints or integer variants. If I replace UNCHECKED(type) with just (type) in the field definitions, then all seems well (at least as far as my tests are concerned).
So, there's something going on with UNCHECKED that is different, maybe broken?

Is this fixed by your commit that fixes the UNCHECKED function (that indeed should not have been comparing ctypes.*.type with instances of bytes)?  I am not sure that I understand this issue otherwise.  Is there a definite example in the tests?

Yes. That's the fix. I didn't know what the fix was at the time I created this issue.
On Thu, Sep 26, 2019, at 9:31 AM, Spencer E. Olson wrote:

Is this fixed by your commit that fixes the UNCHECKED function (that indeed should not have been comparing ctypes.*.type with instances of bytes)? I am not sure that I understand this issue otherwise. Is there a definite example in the tests?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/davidjamesca/ctypesgen/issues/70?email_source=notifications&email_token=AAJTJIZSXF7J23RURSMSGJTQLTITPA5CNFSM4IVTKKHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7V7ZUQ#issuecomment-535559378, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJTJIYV6ZNU6FPQV36NGELQLTITPANCNFSM4IVTKKHA.

--
Alan Robertson
<EMAIL_ADDRESS>

There is no example in the tests. I should have made one. Mea culpa!
Here's the situation:

A function pointer for a function which returns an int.
Call the function through the pointer.
Return result is not an int. It should be. I seem to recall it treats it as a void *.


⚠ Bofrost.at has degraded performance

In d775768, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 830 ms

Resolved: Bofrost.at performance has improved in 3510a16 after 34 minutes.


mux MJPEG into .mp4 if possible

@zanshi
It'd be nice for the mp4 example app to mux MJPEG into .mp4 files, and for a new VideoParameters::sample_entry to create a suitable VisualSampleEntry for this for other applications (e.g. Moonfire NVR) to use for writing .mp4s without codec-specific logic.
mediamtx appears to have logic for doing this here: https://github.com/bluenviron/mediamtx/blob/3d69edb533a94b55bf3a98fb5391268ee324825c/internal/playback/mp4/track.go#L645-L694
ffmpeg also seems to do this. Just ffmpeg -i in.mjpeg -c copy out.mp4 produces something that plays with apps such as ffplay, VLC, and Apple QuickTime Player and still is reported by ffprobe as MJPEG.
I'm confused though. The VisualSampleEntry type is mp4v which I thought was MPEG-4 video (ISO/IEC 14496-2)? Is that standard backward compatible with MJPEG? I'm not actually sure what it's writing is correct even though it does play.

Ahh, this bit of mediamtx source was helpful, a ways above what I quoted:
// Specification: ISO 14496-1, Table 5
const (
	objectTypeIndicationVisualISO14496part2    = 0x20
	objectTypeIndicationAudioISO14496part3     = 0x40
	objectTypeIndicationVisualISO1318part2Main = 0x61
	objectTypeIndicationAudioISO11172part3     = 0x6B
	objectTypeIndicationVisualISO10918part1    = 0x6C
)

ISO 10918-1 is JPEG; wikipedia says:

In 1987, ISO TC 97 became ISO/IEC JTC 1 and, in 1992, CCITT became ITU-T. Currently on the JTC1 side, JPEG is one of two sub-groups of ISO/IEC Joint Technical Committee 1, Subcommittee 29, Working Group 1 (ISO/IEC JTC 1/SC 29/WG 1) – titled as Coding of still pictures.[14][15][16] On the ITU-T side, ITU-T SG16 is the respective body. The original JPEG Group was organized in 1986,[17] issuing the first JPEG standard in 1992, which was approved in September 1992 as ITU-T Recommendation T.81[18] and, in 1994, as ISO/IEC 10918-1.

so I guess there is a standard way to embed JPEG into elemental streams and thus the mp4v thing is valid. And that means we also can say it has an RFC 6381 codec of mp4v.6C. I'm not sure any web browser claims support for that and will play it, though.


Initial tab/space is trimmed at the starting of each new line

I have recently used this library to highlight the syntax. But it trimmed initial spaces and white space characters at the beginning of each new line.  This has made difficult to understand the code. Please add this feature.

Hi, thanks for reporting the issue! Fixed and soon will be published in forthcoming release.

@kbiakov  can I work on this issue and open PR?

Yesterday I found new issue which breaks highlighting: it's fixed, but not merged to develop branch. Also I need to fix Travis CI build fail related to not applied licenses. If you want to - you can, but as I think it's more convenient to do it by my own.

Hi, check latest release! Note that I moved repository under my ownership: see dependency in README.


Fix omitting of chunk on Seek()

We just get rid of the whole binary search complexity. The general use case (and only really allowed usage) is to stream forward through a series. In general, we will touch every chunk in the course of a query. So just iterating through them instead of binary searching is not only a lot simpler but also cheaper.
@Gouthamve

LGTM


Update Regression Tests

Please include in our regression test a single evaluation of the criterion function at the starting values in addition to the overall statistic on the simulated dataset.

Does Criterion function mean the likelihood function, an estimation with 1 evaluation or anything else?
If one of the mentioned options is appropriate, there would be the problem that our battery generates dictionaries that are deterministic with probability 0.1. That means that for both options the function would fail to provide neither the log likelihood value nor an estimation (zero division error ...). An easy solution would be to constrain the generating process in the whole loop.
But maybe my i am wrong regarding my starting point, so please let me know what you mean by 'single evaluation of the criterion function'.

Please only create non-deterministic regression tests so we can avoid the case distinction
     if 0.00 in dict_['DIST']['coeff']:             stat = np.sum(df.sum())             tests += [(stat, dict_)]         else:             stat = np.sum(df.sum())
Also, isn't the if statement too restrictive. How about zero covariances?


Key or string?

Under the section "How should users be prepared for default number matching?"
#3 says to "Set the following Key Value Pair: Key: OVERRIDE_NUMBER_MATCHING_WITH_OTP Value = FALSE"
Is the intention to create a new subkey under the AzureMfa key? If so, then it cannot be assigned the value FALSE as keys cannot be assigned values. If the intention is to create a string within the AzureMfa key, the say "create a new string value"!

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

ID: 5d46abb2-6ad9-e86e-8808-b69ca11922d8
Version Independent ID: c4ccd9d6-13c7-6150-6a44-241bdf9b0982
Content: Use number matching in multifactor authentication (MFA) notifications - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/authentication/how-to-mfa-number-match.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@ianc911
Thanks for your feedback! We will investigate and update as appropriate.

@ianc911
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly

@Justinha
Can you please check and add your comments on this doc update request as applicable.

This is fixed. #please-close


Error matching of meta/nbt for jei + click

Issue description:
When a recipe requires an item with a damage value or NBT data, a matching item in the storage (or player) inventory won't be found on clicking the JEI +, or by a refined storage crafter attempting to craft the item.
However, after placing the items into the crafting grid manually, and then taking out the finished item, the correct items in the storage network do refill the crafting grid.
What you expected to happen:
The grid and crafter should be able to find the matching item that exists in the storage network under all circumstances.
Steps to reproduce:

Open crafting grid UI
Click into JEI recipe view of an item whose ingredients have meta/nbt values  (e.g. Mekanism Basic Energy cube)
Click + button in JEI
Grid is filled with all items except the ones with meta/nbt
Find correct item in the storage network with the grid
Place item into appropriate spaces in recipe
Click to remove crafted item
Grid is refilled with all items in recipe, including the ones with meta/nbt

Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>7
Refined Storage: 1.0.2

Not much I can do about that. The JEI recipe says it takes / needs a full durability item. So I do what JEI says. There is no way for me to detect if said mod allows to be autocompleted with a mod with a different damage value.

It doesn't look like the JEI recipe is specifying a full-damage item.  The damage bar and the nbt tooltip match exactly between what is in JEI and what is in the network.


Fizz sound still plays

Fizz sound is now controllable through config, but if set to false the default sound when lava touches water still plays

Ok so i dig through source code and found nothing. Still this can be fixed by using a resource pack with empty audio as fizz sound, by doing so the plugin will work as intended

I think this can be stopped using SoundPackets?


Added copying of pan info to AVPro audio sources

Allowed AVPro audio sources to copy stereo pan for simulated global stereo mix on non-standard channels.
This is so we can have channel 3 + 4 as a fake global stereo, to allow for channel mixing shenanigans for isolating sources between twitch streaming vs VRCDN streaming with minimal components in the middle.
The setup here is to set the stereo pan left/right of channel 3 and 4 to -1 and 1 respectively, with otherwise identical setup to the "Global" prefab. This way we get global audio on non-standard channels.
This PR enables this behavior by copying over pan information to the audio source.

Closing - this doesn't actually work in AVPro, my bad


ListItem spans two columns with two-column layouted List

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
I have a  component with an odd number of . These ListItems do have a  and a  included. I want to have the List in two columns. Therefore I am utilizing the CSS property columns.  The last item on the left column spans over the first and the second col. You can see this in the screenshot

Expected Behavior 🤔
It should behave like it behaves if there is an even number of ListItems in the List.

Steps to Reproduce 🕹
Steps:

open the sandbox

Context 🔦

Your Environment 🌎
Win 10, Chrome 93

Take a look on https://next.material-ui.com/system/flexbox/ for inspiration of how you may style the list