why() misses exception involving complex numbers

In [1]: a = 3+1j
In [2]: b = 1+3j
In [3]: divmod(a, b)
Traceback (most recent call last):
  Code block [3], line 1, in <module>
    divmod(a, b)
  TypeError: unsupported operand type(s) for divmod(): 'complex' and 'complex'
In [4]: why()
No information is known about this exception. Please report […]

Python 3.10.1
IPython 7.31.0
friendly_traceback 0.5.10
friendly 0.5.5

Thanks for the report.
I know that I have many TypeError cases not covered yet, but I will try to take care of this one next.

Take your time, mate. It's not like I run into this exception every day, but I thoght I might as well report it when friendly asked me to.

Fixed in version 0.5.11 for friendly-traceback.
There will be more cases to consider eventually, but this one (with at least one complex number) has been taken care of.


Add lightning schemas

https://learn.microsoft.com/en-us/minecraft/creator/documents/deferredlighting?view=minecraft-bedrock-stable

Duplicate of #272


Support for angular 2 above 2.1.1

Any plans to support 2.1.2 and upcoming versions? The peer dependency is currently fixed at 2.1.1.

1 month and no answer on this, last commit 22 days ago is this project still in progress?

@NicolasDepoilly yes it's in progress, I believe it should work with newer versions. There's just other important stuff at the moment that we need to take care of.
In any case, updating dependency is kinda of a big deal, because we need to carefully test everything.

@lugovsky good to hear!
to be honest I am starting a new project from scratch with ng-admin and i used some ng2-smart-table. I was completely lost and thought I had to change of idea until I saw it was just a dependency issue.
Do your best guys and thanks for the answer!

@NicolasDepoilly No worries :)

I had the same warning of peer dependency while running "npm install". I ignored it, but when I tried to start the app with "npm start", it rejected saying "Ng2SmartTableModule is not a NgModule".
Any idea why its causing this error? did I install it wrong ?

@siddharthaReddy Probably you are using angular-cli. If so, see the issue linked below. There is also described workaround for that temporary problem.
https://github.com/angular/angular-cli/issues/3426


imylu/imylu/utils/load_data.py/

def gen_data(low, high, n_rows, n_cols=None):
Returns:
        list -- 1d or 2d list with int

but in imylu/imylu/utils/preprocessing.py/def min_max_scale(X): X is ndarray .
and you can see imylu/examples/kd_tree_example.py/ 's line 51~line 53
X = gen_data(low, high, n_rows, n_cols)
y = gen_data(low, high, n_rows)
Xi = gen_data(low, high, n_cols)

I think just add a line in def gen_data():
def gen_data(low, high, n_rows, n_cols=None):
    """Generate dataset randomly.
    Arguments:
        low {int} -- The minimum value of element generated.
        high {int} -- The maximum value of element generated.
        n_rows {int} -- Number of rows.
        n_cols {int} -- Number of columns.
    Returns:
        list -- 1d or 2d list with int
    """
    if n_cols is None:
        ret = [randint(low, high) for _ in range(n_rows)]
    else:
        ret = [[randint(low, high) for _ in range(n_cols)]
               for _ in range(n_rows)]
    ret = array(ret)  # This is my add.
    return ret

Maybe I ignore something in somewhere, thanks!
打扰了，非常感谢~

Thanks @Jacob-xyb, this repo was originally implemented by native python objects like set, list, dict, and then re-implemented by numpy. So the doc-strings in this repo may need to be updated. I feel like I should find some time to update all the doc-strings with mistakes, and not make people confused.

After reading my codes again, I found that some of the algorithms are still implemented by native Python objects like array but not ndarray. So that's might be the reason why gen_data function is returning with List.

Thanks for your reply!


Can't view commits from Graph view

Got this error:
Traceback (most recent call last):
  File "C:\Program Files\Sublime Text 3\sublime_plugin.py", line 818, in run_
    return self.run(edit)
TypeError: run() missing 1 required positional argument: 'action'

When pressing Enter on a commit in the Graph view.

I cannot reproduce, what is you GitSavvy version?

2.13.0

Could you restart Sublime Text and try again, I suspect it was caused by the automatic upgrade and Sublime Text was not restarted.

You are correct, works fine after a restart, I see new functionality on the commit selection and I guess ST didn't unbind the previous shortcut assignment when the new one was added.


ARM Test

Hi @victorgrenke ,
Thanks for your work on this art repo. And now I want to test this repo on ARM v8 platform. However, it seems like that ws-rtsp-proxy does not support ARM v8. My question is: could I get support for testing this repo on ARM v8 ?
Any help will be appreciated.
Thank you.

Hello @lorlor  the proxy didn't compiled for the ARM v8 platform. but it's possible to do. what board do you use? or do you use your own design?

Hi @victorgrenke
I'd like to deploy the proxy on a Jetson Xavier (aarch64/ubuntu18.04).
Is it possible to have a build for this platform?
BR
Alex

Hi @victorgrenke I'd like to deploy the proxy on a Jetson Xavier (aarch64/ubuntu18.04). Is it possible to have a build for this platform? BR Alex

Hi, Alex. Do you have got the ARM deb package? I have the same request, and send e-mails also. But not response currently @StreamSupp

Hi @WanHui666
Unfortunately no response so far...
If you'd get a response on your mail please keep us updated!

Hi @WanHui666 Unfortunately no response so far... If you'd get a response on your mail please keep us updated!

Ah.. Thanks for your response. If I got the response, I will sync to you.
I need to find the other way to show video on web page, do you have other ideas?

Hello @lorlor the proxy didn't compiled for the ARM v8 platform. but it's possible to do. what board do you use? or do you use your own design?

Hi, VictorG
Do you have the plan to support ARM platform for proxy server?Maybe it's hard to find an ARM platform device.
Is it OK to build in virtual machine? Such as Azure or AWS? You can use it for trial and choose the ARM arch machine, then push code and build DEB package...
Looking forward to hearing from you.

Hi @WanHui666
Previous versions of proxy were built for ARM (raspberry pi + ubuntu), but they were of little demand and DEB packets were removed from the website. You are right to find a ARM hardware is specific task :) Thanks for the advice on how to get around this with virtual machines. I will try to do this and get back to you.


Release/7.0.0

:white_check_mark: Build TheIdServer 7.2.2.build.3027 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/a4c9fb43de by @semantic-release-bot)


Create Gitlab Project YML to call unity-app-generator

Acceptance Criteria:

Gitlab runner can be triggered through API call
Gitlab runner executes YML without error
Gitlab runner successfully deploy and example application unity-sds/unity-ads-deployment#47

Now, the unity-app-generator can be successfully installed in Amazon Linux.  The Python version issue, “sudo” issue, docker server connection problem … are all resolved.  A basic YAML file that can execute the first two unity app generator commands is created, but the file will need to be improved over time.

Where is this YAML file version controlled?

This task is complete.  All unity-app-gen commands (init, docker-build, docker-push, cwl-build, cal-push) are called in this commit https://github.com/unity-sds/unity-app-build-trigger/commit/54963ada4c3586439a78e1874bb6ee09858aeff7


Quantize Pointrend

I compressed a model of pointrend based on mmdet-2.2.1，the training looks normal, but a error occurs when i convert it to onnx using functions of pytorch2onnx.py in mmdet-2.3.1 (commit id:6495391) . It seems like _bbox_forward() and _mask_forward() have some problems, do you know how to fix it? the error info is as follows:
File "/home/mechmind/projects/mech_learning/mmdet/models/detectors/base.py", line 180, in forward
return self.forward_test(img, img_metas, **kwargs)
File "/home/mechmind/projects/mech_learning/mmdet/models/detectors/base.py", line 138, in forward_test
return self.forward_dummy(imgs[0])
File "/home/mechmind/projects/mech_learning/mmdet/models/detectors/two_stage.py", line 101, in forward_dummy
roi_outs = self.roi_head.forward_dummy(x, proposals)
File "/home/mechmind/projects/mech_learning/mmdet/models/roi_heads/standard_roi_head.py", line 60, in forward_dummy
bbox_results = self._bbox_forward(x, rois)
File "/home/mechmind/projects/mech_learning/mmdet/models/roi_heads/standard_roi_head.py", line 139, in _bbox_forward
x[:self.bbox_roi_extractor.num_inputs], rois)
File "/home/mechmind/projects/nncf_pytorch/nncf/dynamic_graph/wrappers.py", line 83, in wrapped
retval = module_call(self, *args, **kwargs)
File "/home/mechmind/miniconda3/envs/nncf/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in call_impl
result = self.forward(*input, **kwargs)
File "/home/mechmind/projects/mech_learning/mmdet/core/fp16/decorators.py", line 131, in new_func
return old_func(*args, **kwargs)
File "/home/mechmind/projects/mech_learning/mmdet/models/roi_heads/roi_extractors/single_level_roi_extractor.py", line 73, in forward
roi_feats_t = self.roi_layers[i](feats[i], rois)
File "/home/mechmind/projects/nncf_pytorch/nncf/dynamic_graph/wrappers.py", line 83, in wrapped
retval = module_call(self, *args, **kwargs)
File "/home/mechmind/miniconda3/envs/nncf/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/mechmind/projects/mech_learning/mmdet/ops/roi_align/roi_align.py", line 144, in forward
self.sample_num, self.aligned)
File "/home/mechmind/projects/mech_learning/mmdet/ops/roi_align/roi_align.py", line 30, in forward
aligned)
RuntimeError: roi_width >= 0 && roi_height >= 0 INTERNAL ASSERT FAILED at "/home/mechmind/projects/mech_learning/mmdet/ops/roi_align/src/cpu/roi_align_v2.cpp":134, please report a bug to PyTorch. ROIs in ROIAlign cannot have non-negative size!

@aleksei-kashapov, @AlexKoff88

@Ivan Lazarevich，could you please give me some suggestions?

@vanyalzr Could you please give me some suggestions? Thanks.

I compress a pointrend model based on mmdet-2.3.0(commit id: 7ab8f44 ) with nncf patch, the same error occurs, could you please give me a direction? @aleksei-kashapov

Hi @adamantboy,
Unfortunately the ONNX export in the original mmdetection repo is still limited in terms of the scope of exportable models. You can try to export the model to ONNX using our mmdetection fork.
The export script is located in tools/export.py and the corresponding config for INT8-quantized PointRend model is in configs/nncf_compression/point_rend/point_rend_r50_caffe_fpn_mstrain_1x_coco_int8.py. You can specify your compressed .pth checkpoint to loaded in the config script and export it to ONNX using the export.py script:
python tools/export.py configs/nncf_compression/point_rend/point_rend_r50_caffe_fpn_mstrain_1x_coco_int8.py /path/to/your/checkpoint/ ./ onnx.
cc @aleksei-kashapov


Data science and ml assets

data-science-and-ml content update:

created subsection and markdown for oracle-machine-learning
created subsection and markdown for oracle-graph-for-data-science

I have read the CLA Document and I hereby sign the CLA


securing-your-github-pages-site-with-https page should mention cases with Cloudflare

What article on docs.github.com is affected?
https://docs.github.com/en/free-pro-team@latest/github/working-with-github-pages/securing-your-github-pages-site-with-https
What part(s) of the article would you like to see updated?
The page doesn't mention, that the issue with the HTTPS Enforcement not being available could also be caused by using Cloudflare to resolve your DNS settings, as the proxying of the DNS records/requests makes it unable for GitHub to actually check the DNS settings properly (See this answer for the actual info)
Additional information
Many people use Cloudflare and may also use GitHub Pages, so mentioning this as a common cause would go a long way in helping people.

Thanks for opening an issue @Andre601! I've triaged your pull request for review 🎉

Thanks for opening an issue @Andre601! I've triaged your pull request for review 🎉

Thanks so much for opening this issue! I've checked back in with the team again and we've decided that we're not going to make this change.
We appreciate your interest and passion in improving the GitHub docs ✨

That issue took quite some time to be closed considering the PR related to it was denied much, much earlier.... Guess it went under the radar.


Can't add a script cdn to my html

Please provide us with the following information:

1. OS
Mac OSX El Capitan
2. Versions.
angular-cli: 1.0.0-beta.9
node: 6.3.0
3. Repro steps.
When I add the script:    into the Head of my index.html file, and then later click on on a routerLink, then I wont get redirected until I press it again. this happens on all of my routerlinks.

Thanks! We'll be in touch soon.

Can you show some code here what is included in head, what is in routerLink and when it fails

Sorry, added some more details of the code now.

I'm not sure this is a CLI problem to be honest. It seems that either the CDN scripts your are adding or the router itself has a problem, but in either way I don't see it as a CLI issue since whatever you put there is being served.


[BEAM-6767] SamzaRunner: Add more parameters to SamzaPipelineOptions

Add the following parameter support:

Allow enable/disable beam metrics in SamzaPipelineOptions
Enable durable state with Samza pipeline options
Enable host-affinity for durable state

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

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

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

R: @lhaiesp

Run Java PreCommit

Run Java PreCommit

Run Spotless PreCommit

Run Python PreCommit

Run Portable_Python PreCommit


Avoid -Wshadow warnings

Building with a Kokkos installation that has compiler warnings enabled, forces, i.a., -Wshadow.
This is what I currently need to compile warning-free.

What variables are they shadowing?

The respective local variables, e.g. https://github.com/arborx/ArborX/pull/344/files#diff-99291d7886a818fee666071fb9fbe18eL24.

I would rather rename local variables than add test_ to the name.

I also dislike the resolution proposed here.
I would rather

rename variables empty_tree -> tree in tstLinearBVH.cpp.
change BOOST_AUTO_TEST_CASE_TEMPLATE({test_morton_codes -> assign_morton_codes}, ...
possibly BOOST_AUTO_TEST_CASE({test_stack -> stack_})

I followed the suggestions.


Custom Video Upload - fast forward/rewind issues

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
Hi guys,
Not sure if this is an expected one or not, as I can only find a few records in Open/Closed issues relating to custom videos, however, my custom uploaded videos to pages don't seem to allow me to rewind/fast forward using either keyboard or mouse.
My embedded videos from alternative sources like YouTube/AWS are fine.
Is this expected? I also tested using a custom video with the Bookstack demo site and had the same issue.
Exact BookStack Version
v23.02.1
Log Content
No response
PHP Version
8.2
Hosting Environment
Ubuntu 22.04

Thanks @patsyk1, but yeah, this is known. We don't explicitly support any video-specific hosting features, which includes not supporting range-requests to properly allow video scrubbing.
I feel this is already covered by existing issues #3274 and #882 so I'm going to close this off.


Fixing examples.

Fixes #4331. You can't link with grpc++_unsecure and grpc. You have to pick one of the two operations.

Let's choose the other option?
Reasoning: we want folks to default to secure, and so the closer our examples are to that, the less questions we get :)

Yeah, I don't care which we're choosing. But we can't have both, that's all.

Done.

LGTM

I'm getting the following error message after this change:
Package grpc++ was not found in the pkg-config search path.
Perhaps you should add the directory containing `grpc++.pc'
to the PKG_CONFIG_PATH environment variable
No package 'grpc++' found
After adding
export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig
I'm getting the following error:
/usr/bin/ld: greeter_async_client.o: undefined reference to symbol 'gpr_log'
/usr/bin/ld: note: 'gpr_log' is defined in DSO /usr/local/lib/libgrpc.so.0 so try adding it to the linker command line
/usr/local/lib/libgrpc.so.0: could not read symbols: Invalid operation
collect2: error: ld returned 1 exit status
I tried adding the libraries mutually exclusive and it seems to only work when both are included.
If I manually run the command to before this change it works.  I'm seeing this error on mac,centos7

If that's the case, it would seem that we're not generating the .pc file properly, with the dependency between grpc++ and grpc. @murgatroid99 ? Do you recall something like this ?

I don't know. The Makefile appears to be explicitly listing grpc as a requirement of grpc++ (https://github.com/grpc/grpc/blob/master/templates/Makefile.template#L637). Since the error refers to gpr_log, maybe there's something wrong with how gpr is getting included.

Can we enter an issue to track and investigate this ? I feel we missed something in the pkg-config files.


Can I use this add-on on Australia jiotv

Hello can I use your playlist in Australia ?
If so can I buy the VIP pack ?
Also jiotv add-on can I use in Australia if I have jiotv login from India ?
Does it work in Australia as I need indian channels and I have Jiotv login
I tried your add-on and added jio login but channels does not work
Please help
Also I cannot send message on your discord channels as it says I cannot add server and I am not your friend
Please help email me on<EMAIL_ADDRESS>

Streams are Geo-restricted. So you must be on Indian network while playing channels.

Can I use vpn India??
On Sun, 1 Aug 2021, 7:26 pm abhi, @.***> wrote:

Streams are Geo-restricted. So you must be on Indian network while playing
channels.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/botallen/repository.botallen/issues/162#issuecomment-890501929,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVBOBTL7JFBO6MA3JIHGRSDT2UVQFANCNFSM5BK5IWNQ
.


Broken link to parameter plist file

The link to the plist file is broken:
https://github.com/cloudflare/cloudflare-docs/blob/478cb6260585652434d6c8732fb89d30b8de9c39/content/cloudflare-one/connections/connect-devices/warp/deployment/mdm-deployment/_index.md?plain=1#L64

Howdy @securitygeneration, should be fixed now. Apologies, we're rolling out a new transform rule and didn't account for that file ending.


Update plugin-collabnet.yml

Repository:
https://github.com/jenkinsci/collabnet-plugin
My commits:
https://github.com/jenkinsci/collabnet-plugin/commits?author=BoraAksoy
Existing maintainer: @jmcnally

[x] Add link to plugin/component Git repository in description above
[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

I think we'll need approval from dharmsheta to being removed.

Please approve this change to add Bora as a maintainer of the plugin.


Spelling error in first paragraph

There is spelling issue in the first paragraph of this page. It says "The code to configure your web API so that it calls downstream web APIs builds on top of the code used to project a web API. For more info, see Protected web API - app configuration." I believe that it should instead be "The code to configure your web API so that it calls downstream web APIs builds on top of the code used to protect a web API. For more info, see Protected web API - app configuration."

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

ID: 2fc5b4a3-0d85-102b-8b4c-8dfdd664b2d8
Version Independent ID: 13086ddd-1cf0-a8ca-46f2-588e59734122
Content: Web API that calls downstream web APIs (app's code configuration) - Microsoft identity platform
Content Source: articles/active-directory/develop/scenario-web-api-call-api-app-configuration.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

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

Good catch, @TyHorn !
I created a pull request which should go live in 24 hours. https://github.com/MicrosoftDocs/azure-docs-pr/pull/76765


add new internal worker + worker/temporal validations

workflows worker on based on version, internal worker off. ✅
$ helm template -f ~/retool-helm/charts/retool/values.yaml foo ~/retool-helm/charts/retool --set config.encryptionKey="foo" --set image.tag="5.6.10" --set internalWorker.enabled=false | grep "worker"
# Source: retool/templates/deployment_workflows_worker.yaml
  name: foo-retool-workflow-worker
    retoolService: foo-retool-workflow-worker
# Source: retool/templates/deployment_workflows_worker.yaml
  name: foo-retool-workflow-worker
    retoolService: foo-retool-workflow-worker
      retoolService: foo-retool-workflow-worker
        prometheus.io/job: foo-retool-workflow-worker
        retoolService: foo-retool-workflow-worker

workflows worker on version, internalWorker on ✅
$ helm template -f ~/retool-helm/charts/retool/values.yaml foo ~/retool-helm/charts/retool --set config.encryptionKey="foo" --set image.tag="5.6.10" --set internalWorker.enabled=true | grep "worker"
# Source: retool/templates/deployment_internal_worker.yaml
  name: foo-retool-internal-worker
    retoolService: foo-retool-internal-worker
# Source: retool/templates/deployment_workflows_worker.yaml
  name: foo-retool-workflow-worker
    retoolService: foo-retool-workflow-worker
# Source: retool/templates/deployment_internal_worker.yaml
  name: foo-retool-internal-worker
    retoolService: foo-retool-internal-worker
      retoolService: foo-retool-internal-worker
        prometheus.io/job: foo-retool-internal-worker
        retoolService: foo-retool-internal-worker
# Source: retool/templates/deployment_workflows_worker.yaml
  name: foo-retool-workflow-worker
    retoolService: foo-retool-workflow-worker
      retoolService: foo-retool-workflow-worker
        prometheus.io/job: foo-retool-workflow-worker
        retoolService: foo-retool-workflow-worker

workflows worker off based on version, internalWorker on -- validation error! ❌
➜  $ helm template -f ~/retool-helm/charts/retool/values.yaml foo ~/retool-helm/charts/retool --set config.encryptionKey="foo" --set image.tag="2.6.10" --set internalWorker.enabled=true | grep "worker"
Error: execution error at (retool/templates/NOTES.txt:34:4):
VALUES VALIDATION:
internalWorker:
  Internal worker is enabled (via internalWorker.enabled), but Temporal is not enabled via retool-temporal-services-helm.enabled or workflows.temporal.enabled or via Retool's Managed Temporal for Retool Workflows (via workflows.enabled explicitly, or workflows.enabled implicitly based on image.tag > 3.6.11)

Use --debug flag to render out invalid YAML

workflows worker off based on version, internalWorker on + temporal configured ✅
➜  $ helm template -f ~/retool-helm/charts/retool/values.yaml foo ~/retool-helm/charts/retool --set config.encryptionKey="foo" --set image.tag="2.6.10" --set internalWorker.enabled=true --set workflows.temporal.enabled=true | grep "worker"
# Source: retool/templates/deployment_internal_worker.yaml
  name: foo-retool-internal-worker
    retoolService: foo-retool-internal-worker
# Source: retool/templates/deployment_internal_worker.yaml
  name: foo-retool-internal-worker
    retoolService: foo-retool-internal-worker
      retoolService: foo-retool-internal-worker
        prometheus.io/job: foo-retool-internal-worker
        retoolService: foo-retool-internal-worker

will take this on


optimizations for GoogleEventSet, speeding up merging 20+%

All objects from takeout pass through this merging HPI so worth speeding it up

add add_if_not_present method to avoid computing key twice (which is quite expensive!)
This is intended to be used as a replacement for (e.g. in HPI)
if event in emitted:
    continue
emitted.add(event)
yield event

With this method, we could rewrite as:
if emitted.add_if_not_present(event):
    yield event

This could be introduced to hpi with backwards compatibility.

use type directly as key, types are hashable (very tiny speedup, but it also feels more natural anyway)

Thanks ❤
Will merge when I get home

@karlicoss  bumped the version, since changing how to dedupe events might lead to some weird duplicates/errors if people were merging a cachew database with some other standalone export.
feel free to ping me to bump pypi versions if you ever need em


[Issue]: Temporary files aren't properly moved between drives

Confirmations

[X] I have checked the issues tab, and couldn't find an existing open issue for the issue I want to report.

OS Type
Linux
Python Version
3.8.9
Package Version
2.4.0
Description
Files aren't properly moved from the temporary directory, if that happens to be in another drive. That's because Path.replace is used which only attempts to rename the file, changing that to shutil.move fixes the issue.
Config
No response
Output Log
isubrip https://itunes.apple.com/ar/movie/id1641189067
Scraping https://itunes.apple.com/ar/movie/id1641189067?l=en
Found movie: Farewell Mr. Haffmann (2021)
Español (Latinoamérica) (es-419) subtitles were successfully downloaded.
Português (Brasil) (pt-BR) subtitles were successfully downloaded.
Error: [Errno 18] Invalid cross-device link: '/dev/shm/isubrip/Farewell.Mr.Haffmann.2021.iT.WEB/Farewell.Mr.Haffmann.2021.iT.WEB.es-419.vtt' -> 'Farewell.Mr.Haffmann.2021.iT.WEB.es-419.vtt'

Fixed in v2.4.1


Using own SSL-Certificates / Certificate-Exchange

Hi,
since the default certificates created by bootkube are only valid 1y, i was trying to replace them with my own, but seem to fail somehow when authenticating during bootstrap.
Here is what i did:

<IP_ADDRESS>/16 is the service-cidr

removed all *crt *pub files in the tls-folder

create new apiserver.crt:

~> cat > openssl_apiserver.cnf <<EOF
[req]
req_extensions = v3_req
distinguished_name = req_distinguished_name
[req_distinguished_name]
[ v3_req ]
basicConstraints = CA:FALSE
keyUsage = nonRepudiation, digitalSignature, keyEncipherment
subjectAltName = @alt_names
[alt_names]
DNS.1 = kubernetes
DNS.2 = kubernetes.default
DNS.3 = kubernetes.default.svc
DNS.4 = kubernetes.default.svc.cluster.local
DNS.5 = api.example.com

IP.1 = <IP_ADDRESS>
EOF

~> openssl req -new -key apiserver.key -out apiserver.csr -subj "/O=kube-master, CN=kube-apiserver" -config openssl_apiserver.cnf
~> openssl x509 -req -in apiserver.csr -CA ca.crt -CAkey ca.key -CAcreateserial -out apiserver.crt -days 3650 -extensions v3_req -extfile openssl_apiserver.cnf

same thing for the kubelet.crt:
~> cat > openssl_kubelet.cnf <<EOF
[req]
req_extensions = v3_req
distinguished_name = req_distinguished_name
[req_distinguished_name]
[ v3_req ]
extendedKeyUsage       = serverAuth, clientAuth
keyUsage               = critical,digitalSignature, keyEncipherment
EOF

~> openssl req -new -key kubelet.key -out kubelet.csr -subj "/O=system:masters, CN=kubelet" -config openssl_kubelet.cnf
~> openssl x509 -req -in kubelet.csr -CA ca.crt -CAkey ca.key -CAcreateserial -out kubelet.crt -days 3650 -extensions v3_req -extfile openssl_kubelet.cnf

And then i did replace the bas64-encoded parts in

auth/kubeconfig
manifests/kube-apiserver-secret.yaml
kube-controller-manager-secret.yaml

But on bootkube-start i get the following authentication error:
Starting temporary bootstrap control plane...
Waiting for api-server...
W0802 07:14:31.346322       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:14:36.343600       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:14:41.343842       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:14:46.343672       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:14:51.343575       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:14:56.343890       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:15:01.343832       1 create.go:31] Unable to determine api-server readiness: API Server http status: 0
W0802 07:15:06.367023       1 create.go:31] Unable to determine api-server readiness: User "system:anonymous" cannot get namespaces in the namespace "kube-system". (get namespaces kube-system)
W0802 07:15:11.344025       1 create.go:31] Unable to determine api-server readiness: User "system:anonymous" cannot get namespaces in the namespace "kube-system". (get namespaces kube-system)

Which i believe is a problem with the kubelet-cert?
Am i missing some cert-option or another important step?
Is there an alternative easy way to replace the certificates?
thx in advance,
Roman

bootkube render has a --ca-certificate-path switch, so I think (but I'm not sure at all) that you can just run:
bootkube render --asset-dir=foo --ca-certificate-path=<path to ca.crt> --ca-private-key-path=<path to ca.key> --api-servers=<something>
for f in $(foo/manifests/{*-secret,etcd*-tls}.yaml; do
    kubectl apply -f $f
done

I'm also looking into this, as the certificates is gonna expire at some point and I need to create some doc.

thx, this is a good start. in addition the ca.crt needs to be copied to /etc/kubernetes/ca.crt.
But that should be  all i guess.
Not sure though, if/how the secrets are automatically being re-read

I don't inherently see anything wrong with those TLS assets but it's hard to tell without logs from your API server.
Closing for now. Consider taking a look at:
https://github.com/coreos/tectonic-installer/tree/master/modules/tls/kube/self-signed
Or:
https://github.com/poseidon/terraform-render-bootkube/blob/master/tls-etcd.tf
https://github.com/poseidon/terraform-render-bootkube/blob/master/tls-k8s.tf


[Proposal] Final placeholder must work as a catch-all mask

Good day here:)
I have meet multiple issues with route templates that seems to be widespread (#1264, #1067, #649 etc).
As a workaround I've implemented custom logic for route template handling. As far as I can see from reading the code and documentation there is almost no changes in logic. The only difference is, last placeholder in the template path part is treated as .*, not .+.If you're interested in it I'will prepare a PR with code to discuss.
How it's implemented
To be honest, code is simpler than the description:)
UPD I've changed tokenization logic a little to match current ocelot behavior. As it is for now all tests are green except the
should_return_response_200_favouring_forward_slash and should_return_not_found routing tests. These two are failing as the match behavior for /{url} and /products/{productId} has been changed.
1  Tokenizer
There is a TemplateTokenizer, a class that splits route template into a sequence of template tokens (actually, a ReadOnlyMemory<char> span with token kind attached).
I've introduced minimal set of token kinds but it can be changed to better match desired behavior. Current set is

normal text
inner uri placeholder
catch all placeholder of a path part
catch all placeholder of a path part (prev char is separator)
placeholder in a query part.

I've preferred to make token kinds a bit verbose to simplify rest of the code. Outside of the tokenizer, there is no need to check whether there is a / before last path placeholder, or whether the placeholder is located at query or path part.
As example,
/a/{sample}/route/{rest}?someArg=1&{queryRest} will be splitted to:
/a/                # normal
{sample}           # inner uri placeholder
/route/            # normal
{rest}             # catch all path part placeholder, prev char is '/'
?someArg=1&        # normal
{queryRest}        # query part placeholder

2 Regex generation helper
The helper produces a regex from a sequence of the tokens, placeholders are represented as a named groups. For the sample above output will be
^(?in)/a/(?'sample'[^/?]+)/route(|/?(?'rest'.*))\?someArg=1&(?'queryRest'[^&]+).*$
or, expanded
^                     # begin of string
(?in)                 # case-insensitive, named groups only
/a/                   # normal text
(?'sample'[^/?]+)     # named group 'Sample', matches everything up to '/' or '?' char
/route                # normal text, note that trailing slash is trimmed
(|/?(?'rest'.*))      # named group 'rest', matches empty string -or- slash followed by any number of characters
\?someArg=1&          # normal text, the '?' is escaped
(?'queryRest'[^&]+)   # named group 'queryRest', matches everything up to '&' char
.*                    # catch-all stub to match rest of the query part
$                     # endof string

3 Integrate regex generator/tokenizer into ocelot stack.
The rest of the code is pretty straightforward. I've replaced
IUpstreamTemplatePatternCreator, IPlaceholderNameAndValueFinder, IDownstreamPathPlaceholderReplacer and it just works:)
4 Set of tests
Set of test with cases that proofs that every service is working as desired.
Final thoughts
Actually, all above was not a big deal due to the good architecture. Almost every part of the Ocelot is a DI service and implementation is replaceable, nice thing to see:)
Also, I prefer proposed code over existing even without the fix. It is much more compact and readable and have no code duplication. If you need to change template parsing logic, all you have to do is to change list of token kinds, modify the tokenizer (hard part, but it is covered with tests) and modify rest of the pattern-related code (easy part and also is covered with tests);
Performance-related: we saw no difference in our loadtests. There are some optimizations to do in the future (as example, value finder may reuse match results from the template matcher), but they are definitely out of the scope of current proposal.

Up?:)

Up again :)

Up

Up

Ok, feel free to tag me if you're still interested in the PR

Is there a PR to go with this?
I'm certainly interested in fixing the /api/blah/{everything} vs /api/blah{everything} duplication issue

I have working implementation but I do not want to spend time on a PR it is of no interest.

Here's a raw code. I did not backport tests yet so there may be silly typos added at backporting time:)

@ig-sinicyn - I'm also interested in seeing this as a PR. Currently hit this exact issue.

@ig-sinicyn Would it make sense to mark the optional placeholder as {?name} so we can differentiate between optional and hard requirement?

@winromulus such decisions have to be made by ocelot team. There are no new commits or merged PRs since end of May so I do not expect any progress here:(

Has this been completed? I just ran into the issue as well

@TomPallister is there any willingness to accept a potential PR and get this issue resolved?

And, down!

#649 double checked.
URLs ending (not ending) with slash are supported
#748 by PR #1911
So, empty placeholders for Catch-All are supported also starting version 23.0.0

Just give me feedback please, what else do you need?


Nothing worked :)

I have prosody-0.10, all required modules, after I enter my credentials and WS Url I see "Connecting..." and then silently redirected to login page. node server output is empty, JS console in browser is empty, prosody log:
Dec 08 15:40:38 c2s85bd450      info    Client connected
Dec 08 15:40:38 mod_websocket   debug   Websocket received frame: opcode=1, 132 bytes
Dec 08 15:40:38 c2s85bd450      debug   Client sent opening <stream:stream> to xxx.yyy.zzz.aaa
Dec 08 15:40:38 c2s85bd450      debug   Sent reply <stream:stream> to client
Dec 08 15:40:38 mod_websocket   debug   Websocket received frame: opcode=1, 16 bytes
Dec 08 15:40:38 c2s85bd450      debug   Received </stream:stream>
Dec 08 15:40:38 c2s85bd450      debug   c2s stream for <xxx.yyy.zzz.aaa> closed: session closed

What is the problem?

I was running into this problem and for me it looked like the authentication was failing.  Are you seeing a similar issue perhaps?

Same happens to me.

@andrey-utkin that error was mean otalk-client uses "old-websocket" (as specified in xmpp-websocket draft specs) transport while server want real "websocket" (RFC7395 spec). It was fixed in later versions of stanza.io library, but otalk still uses old one. Check other forks, there are updated ones.

@vitalyster so you don't care to update the instance on the original site?

@andrey-utkin it looks abandoned, I think original authors does not care

Oh sorry I thought you are maintainer )


Update to use new Creek Kafka docs site

Reviewer checklist

[ ] Read the contributing guide
[ ] PR should be motivated, i.e. what does it fix, why, and if relevant, how
[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage: 98.343%. Remained the same when pulling dddd079ee0f3600470619eb3a18f28b44658a13e on docs into 5ffb2fcdc12c9230507a56560d787196bae92e0f on main.


Fix failed line missing from TAP output

Fixes #569.

Thank you, @sotojuan, keep the good PRs coming!


UDS connect error with node.js

A GRPC node client which is using UDS for communication with the GRPC server, will use a HTTP proxy if the environment variable http_proxy is set. This is not correct, UDS must not use a HTTP proxy.
Below you can see the program output   (GRPC_TRACE=all GRPC_VERBOSITY=DEBUG) - the first line is is the important one.
To reproduce the problem you can use the greeter client & server from the grpc/examples/node/dynamic_codegen directory and change the TCP endpoint to UDS e.g. /tmp/test.sock. Before starting the client make sure the http_proxy env variable is set.
I1225 09:06:05.005114486    6037 http_connect_handshaker.c:209] Connecting to server /tmp/test.sock via HTTP proxy localhost:8080 D1225 09:06:05.005125689    6037 tcp_posix.c:440]            WRITE 0x2658a8552d40 (peer=unix:/tmp/test.sock): 43 4f 4e 4e 45 43 54 20 2f 74 6d 70 2f 74 65 73 74 2e 73 6f 63 6b 20 48 54 54 50 2f 31 2e 30 0d 0a 48 6f 73 74 3a 20 6c 6f 63 61 6c 68 6f 73 74 3a 38 30 38 30 0d 0a 55 73 65 72 2d 41 67 65 6e 74 3a 20 67 72 70 63 2d 68 74 74 70 63 6c 69 2f 30 2e 30 0d 0a 0d 0a 'CONNECT /tmp/test.sock HTTP/1.0..Host: localhost:8080..User-Agent: grpc-httpcli/0.0....'
D1225 09:06:05.005142817    6037 connectivity_state.c:156]   SET: 0x2658a86b1bd8 client_channel: IDLE --> CONNECTING [lb_changed] error=(nil) "No Error" D1225 09:06:05.005146384    6037 connectivity_state.c:109]   CONWATCH: 0x2658a8651998 (null): from CONNECTING [cur=CONNECTING] notify=0x2658a864ef08
D1225 09:06:06.005774154    6045 tcp_posix.c:176]            read: error={"created":"@1482653166.005708271","description":"FD shutdown","file":"../src/core/lib/iomgr/ev_epoll_linux.c","file_line":1045}
D1225 09:06:06.005820710    6045 connectivity_state.c:156]   SET: 0x2658a86b1d00 subchannel: CONNECTING --> TRANSIENT_FAILURE [connect_failed] error=0x2658a8ac5460 {"created":"@1482653166.005788374","description":"Connect Failed","file":"../src/core/ext/client_channel/subchannel.c","file_line":635,"grpc_status":14,"referenced_errors":[{"created":"@1482653166.005708271","description":"FD shutdown","file":"../src/core/lib/iomgr/ev_epoll_linux.c","file_line":1045}]}

We can probably change the code that looks at the http_proxy environment variable skip what it's doing if the target is a UDS.


Add function signature to the sharpen function

We call functions like resize and modulate with their parameters in an object, which results in calls like:
sharp(input)
  .modulate({
    brightness: 0.5,
    saturation: 0.5,
    hue: 90
  });

sharp(input)
  .resize({ width: 100, height: 300 });

But this does not work:
sharp(input).sharpen({ sigma: 2, flat: 2, jagged: 3 }

What are you trying to achieve?
Symmetry with how I call other functions
Have you searched for similar feature requests?
Yes. My search did not turn up any issues.

Happy to accept a PR, if you're able, that adds the ability to use this approach whilst maintaining the existing API.

The Object-based approach for setting sharpen parameters is now available as of v0.30.3. The previous approach still works and is now deprecated.
https://sharp.pixelplumbing.com/api-operation#sharpen


[Batch Mode] Add frontend check to ensure that all primaries are present in the supplementary output filemap.

…pplementary output filemap.

If the supplementary output file map does not get transmitted correctly from the driver into the frontend, one symptom can be that not all primary inputs are represented when the frontend parses the map. This PR adds a check for that condition to the compiler.

@swift-ci please smoke test

@swift-ci please smoke test

Ah, you forgot to push.

@swift-ci please smoke test

Oops! Sorry. Now, I've pushed.

@swift-ci please smoke test


Autocorrect browser extension

A lightweight browser extension to autocorrect what you type without touching anything. There could also be additional features like using a Find feature to replace certain letters or words with something different, creating automatic text replacements (like expanding "omw" to "on my way"), or saving phrases for quick reuse later.

It's worth pointing out that Grammarly can autocorrect certain words automatically, although it doesn't do that for everything.


Not Working in Blazor 0.6.0

Try running it on Blazor 0.6.0 but get error

Yes, I need to publish an update to get this working on 0.6. Not sure when I will have time but hopefully in the next week or so

I believe the latest release will work on 0.6. I only tested against 0.7. It was fixed by #5


Implement proper multi-window support

Storage Explorer has the "New Window" feature but it has lots of problems today.
Related issues:

Quick Access data is not visible to the secondary instances.
https://github.com/microsoft/AzureStorageExplorer/issues/638

Local storage data is not visible to the secondary instances.
https://github.com/microsoft/AzureStorageExplorer/issues/723

Secondary instances may overwrite local storage data of primary instance.
https://github.com/microsoft/AzureStorageExplorer/issues/8139

Subscription filter data is not visible to the secondary instances.
https://github.com/microsoft/AzureStorageExplorer/issues/213

"New Window" doesn't work for snap.
https://github.com/microsoft/AzureStorageExplorer/issues/4529

Due to lack of coordination, primary and secondary instances may race and corrupt data when writing to the shared resources such as files on disk or objects in local credential store. This is currently mitigated by not allowing the secondary instance from writing to shared resources at all.

We need proper multi-window support where Storage Explorer application manages all the windows of itself and coordinate them using the shared main process. This would allow us to implement our own concurrency control to prevent race conditions and maybe allow it to work for snap since the application no longer need to have the permission to spawn new processes.

@richardMSFT please investigate this over the holiday lock down period. Please come up with a plan/list of items needed to fix the broken aspects of multi window support. At a high level, the current hypothesis is that the main process needs to be able to manage and coordinate the existance of multiple renderer windows. What achieving that goal actually entails is up for you to figure out and communicate to the team. Good luck! It'll be a lot.


naming typo

https://github.com/Sammyjo20/Saloon/blob/44fe2d29aba0148f96e5d874e9c6534b1e0c5ca6/src/Managers/RequestManager.php#L327

Thanks @shoxton
I get this updated in the next release ;)


ci: fix timeouts

@ishank011 merged to unblock ci and created this new one: https://github.com/cs3org/reva/pull/858


dot product dimension mismatch

I'm trying to use the dot product operator, but always receiving broadcasting errors.  For example:
        from mxfusion import Model, Variable
        from mxfusion.components.distributions import Normal
        from mxfusion.components.variables import PositiveTransformation
        from mxfusion.inference import StochasticVariationalInference
        from mxfusion.inference.grad_based_inference import GradBasedInference
        from mxfusion.inference import BatchInferenceLoop
        from mxfusion.inference.meanfield import create_Gaussian_meanfield
        from mxfusion.components.functions.operators import broadcast_to
        from mxfusion.components.functions.operators import dot

        k = 5
        [m, n] = 20, 10

        X = mx.nd.array(np.random.normal(size=(m, n)))

        #  MXFusion model
        model = Model()
        zero = mx.nd.array([0.])
        one = mx.nd.array([1.])
        model.Z = Normal.define_variable(mean=broadcast_to(zero, (m, k)), variance=broadcast_to(one, (m, k)), shape=(m, k))
        model.D = Normal.define_variable(mean=broadcast_to(zero, (k, n)), variance=broadcast_to(one, (k, n)), shape=(k, n))
        sigma_2 = Variable(shape=(1,), transformation=PositiveTransformation())
        model.X = Normal.define_variable(mean=dot(model.Z, model.D), variance=broadcast_to(sigma_2, (m, n)), shape=(m, n))

        observed = [model.X]
        q = create_Gaussian_meanfield(model=model, observed=observed)

        print(model)

        svi = StochasticVariationalInference(model=model, num_samples=100, observed=observed, posterior=q)
        infr = GradBasedInference(inference_algorithm=svi, grad_loop=BatchInferenceLoop())
        infr.run(X=X, max_iter=10)


Style login screen - fix #70

This is a fix for #70.
This styles the login page to match the style when you log in a little better:

Remove the top navbar. It's already obvious it's a login page. I would also remove all the links under the form. If users need that they can just use the web version of Twitter.

Fixed. The vertical centering feels hacky but whatever.

It would be nice with some kind of logo at the top.

I could bring back the header but only keep the logo then center it. What about something like that @SamVerschueren?

Happy now @SamVerschueren? 😄

The Twitter logo looks a bit ugly though. It has dropshadow and the color is incorrect. I would rather use the one in the navbar on twitter.com. It should also not be clickable.
On the login form, can you use border: 1px solid #e8e8e8; instead of the box-shadow?

The Twitter logo looks a bit ugly though. It has dropshadow and the color is incorrect. I would rather use the one in the navbar on twitter.com.

@sindresorhus This is an img tag in the DOM out of the box, but the one in the navbar is a :before element using a font that's not available on this page. I tried sourcing it from the CSS using @font-face but it doesn't seem to load the font.
/* Twitters icon font */
@font-face {
	font-family: 'rosettaicons';
	src: url('https://abs.twimg.com/a/1464326506/font/rosetta-icons-Regular.woff') format('woff');
}

If you know how to import external fonts in electron a nice svg link or something I can make that the background of the entire header or something.

It should also not be clickable.

Again, this is out of the box. I can push a small JS preventDefault(). If we find a nice image like I mentioned above I'll wind up hiding the a tag entirely, so I didn't do this yet hoping we can improve the icon.

Here you go. The logo as SVG:
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 273.4 222.2"><path fill="#55ACEE" d="M273.4 26.3c-10.1 4.5-20.9 7.5-32.2 8.8 11.6-6.9 20.5-17.9 24.7-31-10.9 6.4-22.9 11.1-35.7 13.6A55.92 55.92 0 0 0 189.3 0c-31 0-56.1 25.1-56.1 56.1 0 4.4.5 8.7 1.5 12.8C88 66.5 46.7 44.2 19 10.3c-4.8 8.3-7.6 17.9-7.6 28.2 0 19.5 9.9 36.6 25 46.7-9.2-.3-17.8-2.8-25.4-7v.7c0 27.2 19.3 49.8 45 55-4.7 1.3-9.7 2-14.8 2-3.6 0-7.1-.4-10.6-1 7.1 22.3 27.9 38.5 52.4 39-19.2 15-43.4 24-69.7 24-4.5 0-9-.3-13.4-.8 24.8 15.9 54.3 25.2 86 25.2 103.2 0 159.6-85.5 159.6-159.6 0-2.4-.1-4.9-.2-7.3 11.1-8 20.6-17.9 28.1-29.1z"/></svg>

OK, I've updated with the svg version provided by @sindresorhus. Thanks for that! The color is the fill in the svg that was provided.
This is a BG image on a :before pseudo-element, so there's no link to click.
Screenshots:

Looks amazing! Thanks for all the effort on this @paulmolluzzo :D

Super duper nitpick, but maybe also some right/left margin on the login form so it looks nice when narrow. Current:

Awesome :)


Rewrite ellipse discretization from scratch

There was many problem with ellipse discretization, which was losing a
lot of informations about the ellipse orientation in 3D. (Aka, fas often completely wrong if not axis aligned).
This almost completely rewrite it to be more general, as well as a bit
more efficient in the generation of the triangulation indices.
I'm not sure it is more readable (but it looks way more generic).
Main difficulty in reading this code is that the shapes of objects
(transforms), depends on Wether you are in 2 or 3D, but the math is the
same.
I've tried to heavily comment it for the next modification, as I'm
pretty sure we will hit some degenerate cases.
I've also modified one of the existing example that display ellipses
in addition to points where the 4 corners would be in order to properly
visually check the rendering it correct.
See #4326 for some visualisation of incorrect rendering.
The triangle index generation at the end is a bit hard to read:
-    triangles = np.array([[0, i + 1, i + 2] for i in range(num_segments)])
+    triangles = (
+        np.arange(num_segments) + np.array([[0], [1], [2]])
+    ).T * np.array([0, 1, 1])

So I'm' happy to revert it, but it is about an order of magnitude faster
than the Python list comprehension.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

How has this been tested?
Currently only visually, let's see how the test suite reacts.
I likely need to add a few test to make sure some discretizations are ok.

I think @sofroniewn and @jni you did most of that in #100 originally, so your input would be appreciated.

Hi @Carreau - thanks for picking this up. I think the previous ellipse code was limited to axis aligned planar projections and we didn't support any non-axis aligned ellipses - that probably should have been more clearly documented. Can you check whether this is true for our rectangles and polygon code too.
It's fine if we expand generality with ellipses first, but we should have a plan to support the other shapes in the same way if we need to as well.
I'll give this a more through review and demo when I can

Can you check whether this is true for our rectangles and polygon code too.

AFAICT, rectangles and polygons are fine, as some of the examples actually show off-axis/off-plane rectangles and polygon.
Really here it's only the triangulation and display of ellipses that is/was wrong as the for the the rectangles/polygon it is already an polygon so you don't need to change base to make it a polygon. At least the above is true for polygon that live in a 3D "slice".
I'm wondering if we should verify that for all coordinate of vertices except 3 the values are the same.


Setting next array index nils previous indexes

irb(main):001:0> require 'rodash'
=> true
irb(main):002:0> object = {}
=> {}
irb(main):003:0> Rodash.set(object, 'a[0].b.c', 4)
=> {"a"=>[{"b"=>{"c"=>4}}]}
irb(main):004:0> Rodash.set(object, 'a[1].b.c', 4)
=> {"a"=>[nil, {"b"=>{"c"=>4}}]}
irb(main):005:0> Rodash.set(object, 'a[2].b.c', 4)
=> {"a"=>[nil, nil, {"b"=>{"c"=>4}}]}
irb(main):006:0>

@yellowiscool thanks! If you could release a new version of the updated Gem it would be awesome, as my current project depends on your library. :-)


Add "preserve" quote-style to mimic Black's skip-string-normalization

Summary
Adds a new option to the quote-style formatter configuration called "preserve":

normalizes the prefixes of all strings
enforces double quotes for triple-quoted strings (docstrings and multiline strings)
normalizes unnecessary escapes

In other words, using preserve leaves the quotes of inline-strings as is, but performs all other normalisation, the same as for single or double.
Fixes #7525
Test Plan
Added tests.  Tested that quote-style="preserve without --preview fails with an error.
Edited by @MichaReiser: I expanded the summary to be explicit about what preserve does and updated the test plan.

Thanks for implementing this new option. It is a good signal on how important this functionality is for the community.
I haven't looked at the code yet and have some more catch-up work to do before I'll find the time for the review. I'm overall okay with adding this new option but I want to make sure that it removes the need for --skip-string-normalization because I want to avoid eventually having to support both.

Awesome, thanks @sciyoshi, great to see you back on the repo :) I have basically the same reaction: I support adding this, though it'd be nice if doing so could definitively preclude us from adding --skip-string-normalization in the future.

Thanks :) I think there's two cases to think about for the preserve option and how it compares to skip-string-normalization:

Whether it should keep existing string prefixes or normalize them
Whether it should format docstrings to use double quotes

Black's behavior with skip-string-normalization is to not touch any string prefixes or docstrings. Personally, I think that having ruff normalize string prefixes in preserve mode is OK, although docstrings probably should be kept as-is. As it stands, this PR does both (but for #2, only if the docstring is triple-quoted).

Black's behavior with skip-string-normalization is to not touch any string prefixes or docstrings, and also to skip changing any character escapes. Personally, I think that having ruff normalize string prefixes and escapes (1. and 3.) in preserve mode is OK, although docstrings probably should be kept as-is. As it stands, this PR does all three (but for 2., only if the docstring is triple-quoted).

To make sure I understand you correctly. This PR

normalizes the prefixes of all strings
enforces double quotes for triple-quoted strings (docstrings and multiline strings)
normalizes unnecessary escapes

In other words, using preserve leaves the quotes of inline-strings as is, but performs all other normalisation as with single or double.

Correct, that's the current behavior of this PR.

@MichaReiser
The "preserve" quote style looks like a step in the right direction.
What would this PR do about this?
s = '''
"""
Example docstring
"""
'''

Or this?
return('''
"""
Example docstring
"""
'''
)

Are you sure you know what an "unnecessary" escape is? What about?
pattern = re.compile('''
whatever
''')

Or this?
pattern = re.compile(fr"""
whatever
""")

Finally, what about backslash-newlines?
s = textwrap.dedent("""\
unit test data
""")

Removing the trailing backslash would break some of Leo's unit tests.

@edreamleo the result for your examples would be
import re
import textwrap

s = '''"""
Example docstring
"""
'''

def foo():
    return('''
    """
    Example docstring
    """
    '''
    )

pattern = re.compile('''
whatever
''')

pattern = re.compile(fr"""
whatever
""")

s = textwrap.dedent("""\
unit test data
""")

--- example.py
+++ example.py
@@ -6,22 +6,29 @@
 """
 '''

+
 def foo():
-    return('''
+    return '''
     """
     Example docstring
     """
     '''
-    )

-pattern = re.compile('''
+
+pattern = re.compile(
+    """
 whatever
-''')
+"""
+)

-pattern = re.compile(fr"""
+pattern = re.compile(
+    rf"""
 whatever
-""")
+"""
+)

-s = textwrap.dedent("""\
+s = textwrap.dedent(
+    """\
 unit test data
-""")
+"""
+)

This happens to be identical to the formatting without the preserve option.

@zanieb Thanks for your comment!
My only remaining concern is the backslash-newline question. By "whatever" I meant various cases containing escapes that might be considered "unnecessary". Perhaps this PR has no effect on that question.

I've added an argument to ruff format which will allow setting --quote-style via the CLI.
Backslash-newlines will be preserved, as will other escape sequences. From my testing I think it is only unnecessarily-escaped quotes that are changed.
If @MichaReiser and @charliermarsh are OK with it, I think this could probably be merged as-is. Nothing precludes us from changing the behavior of preserve to be weaker if there are still concerns with the reformatting of triple quotes & string prefixes - making it stronger on the other hand would be a more of a breaking change. If on the other hand we want preserve to match skip-string-normalization exactly, I can take a look at changing that behavior as well. We probably do also want a test or two for this option.

I rebased the changes, updated the documentation, and gated the new option behind the --preview flag to get more feedback.
Something I noticed when updating the documentation is that preserve is special in the sense that it won't pick the quote that requires the few escapes, at least not the way it's implemented now. This is a slight deviation from single and double but I think it's subtle enough that adding preserve to quote-style is fine.
The reason for gating this behind preview is that we had some internal discussions and @charliermarsh brought up this valid concern:

I think more users are likely to be confused that preserve changes docstrings than they are happy that preserve changes docstrings.

We gate this behind preview because we want to ship the functionality that allows users to preserve their quote style, but we need to gather more feedback if configuring this using quote-style is the right way to go.
@charliermarsh or @BurntSushi would appreciate if you could review the changes that I made to this PR.

I agree with this, although like I mentioned this can be relaxed in a followup without breaking compatibility.

I continue to feel conflicted about this. The documentation for quote-style already become more complicated because some parts no longer apply for preserve. Potentially preserving all quotes styles and having this configured under the same setting will make this worse.
The other challenge is that we wouldn't be able to support enforcing double quotes for triple quoted strings and docstrings but preserve the other quotes.
Anyway, I think this is something we can re-visit when we get more feedback. This looks good to me. Thanks for pushing us to revisit adding this option by creating the PR.

I was trying out the "preserve" quote-style on a code base formatted with black using "skip-string-normalization" and was surprised to see non-docstring multiline strings using ''' have their quotes replaced. I came looking for a bug on that, but discovered it was intentional.

enforces double quotes for triple-quoted strings (docstrings and multiline strings)

Just a little feedback.

Thanks for the feedback. Does this behavior prevent you from adopting ruff or are okay with the changes that it introduces?

I wouldn't say the behavior prevents adoption, but it does add a little friction. If there were only a small number of changes between black and ruff on our code base, I'd just open a pull request to switch to ruff and expect my team to be happy that formatting is faster. Without "preserve" there are hundreds of files changed. Since "preserve" is only currently available with the preview setting and that changes other things, using it also results in many files reformatted.
I'm personally fine normalizing all quotes, and I imagine my teammates probably are too, but it becomes a point of discussion when lots of files change and cause merge conflicts with their outstanding pull requests.

Just like @genericmoniker I also expected preserve to completely skip quotes normalization (just like black's --skip-string-normalization which is what we've been using on our code base). I understand this is by-default but I guess it would at least delay adopting ruff until we have fewer ongoing pull requests to avoid conflicts (I mean, to adopt it as a formatter, obviously; we are already using ruff as a linter and its awesome).


isort forgetting 1st party when extending in sub-configs

In today's version of monorepo hurdles, here is a minimal repro repo with ruff=0.0.267 and numpy==1.24.3 installed:
.
├── folder_1
│   └── module.py
├── folder_2
│   ├── main.py
│   ├── module.py
│   └── ruff.toml
└── pyproject.toml

# pyroject.toml

[tool.ruff]

select = ["ALL"]
ignore = [
    "D2",
    "INP001",
]

# ruff.toml

"""folder_1/module.py."""

SOME_OTHER_MODULE_IMPORT = 0

"""folder_2/module.py."""

SOME_SAME_MODULE_IMPORT = 0

"""folder_2/main.py."""

import time  # noqa: F401

import numpy as np  # noqa: F401

from folder_1.module import SOME_OTHER_MODULE_IMPORT  # noqa: F401
from folder_2.module import SOME_SAME_MODULE_IMPORT  # noqa: F401

All is well, ruff reports no issues from the repo root:
repo > ruff .

Issue
Now, changing ruff.toml to be:
# ruff.toml

extend = "../pyproject.toml"

Suddenly we get:
repo > ruff .
folder_2/main.py:3:1: I001 [*] Import block is un-sorted or un-formatted
Found 1 error.
[*] 1 potentially fixable with the --fix option.

So, what's the issue about?

With sub-config ruff.toml withoutextend: ruff from root correctly infers folder_1 as 1st party
With subconfig ruff.toml using extend: ruff from root now infers folder_1 as 3rd party

I think sub-config extensions should inherit 1st party understandings from parent config.  What do you think?

For the sake of completeness in my explanation and evaluation, should your modules here contain init.py files? Or do they intentionally not?

Yeah one can add them, just for a minimal repro it wasn't necessary.
Having an __init__.py in folder_1 will not change how ruff's isort sees folder_1 as 1st --> 3rd party.

The issue here comes from ruff's automatic inference of 1st vs 3rd party changing with sub-configs.
If you hardcode known-first-party in the top-level config and use extend in the sub-config (thus defeating ruff's inference), this issue goes away.

If you add select = ["I"] to your empty ruff.toml, I believe you see the same behavior regardless of the extend. In the first example provided, the I rules aren't actually enabled for folder_2.

(There may actually be an issue here, I'm just trying to pinpoint a faithful reproduction so we can discuss it properly, sorry, not intending to nitpick.)

Yeah no offense taken!  And you're right about having select in the sub-config.

Better Repro
# pyroject.toml

[tool.ruff]

select = ["I"]

Without the sub-config, ruff has no complaints.
Then, adding in the sub-config:
# ruff.toml

select = ["I"]

Now, ruff changes it's opinions on 1st vs 3rd party on folder_2/main.py's import from folder_1.
Turns out it wasn't the extend, but actually I being selected in the sub-config, that caused ruff to change its opinions.

So, if you omit the src field (which is what we use to do first-party eligibility checks), then we use the "project root", which is generally determined by finding the pyproject.toml or ruff.toml file. This applies regardless of whether you use extend.
You can fix it in one of two ways. First, by explicitly setting the src in the root directory:
# pyproject.toml

[tool.ruff]
# Explicitly set the source to the current directory
src = ["."]

# ruff.toml

extend = "../pyproject.toml"

Second, by explicitly setting the src in the subdirectory:
# ruff.toml

extend = "../pyproject.toml"
src = ["../"]

Either of these works as expected. (If you run with --verbose, Ruff will also tell you why it's resolving modules as first- or third-party, which can help with debugging.)
I can see why the current behavior might be confusing as seen above, but alternative designs are confusing in other ways. E.g., you could be using extend with some shared configuration that doesn't even live in the project hierarchy. In that case, it would be weird to consider that folder the "project root".

Okay I follow what you're saying, thank you!

E.g., you could be using extend with some shared configuration that doesn't even live in the project hierarchy. In that case, it would be weird to consider that folder the "project root".

This makes sense.  And I guess isort doesn't hit this case because it doesn't support sub-configs (as far as I know).

Here are a few possible resolutions:

Documenting src field in isort's settings page for known-first-party and known-third-party

Bonus points: documenting discerning logic for 1st party vs 3rd party vs local

Creating a new boolean isort settings option inherit-src (defaulted false) to pull src from parent config when extend is present
Just close this out 🤷♂

Regardless, at least I have a working solution, so thanks!

Per --verbose, I didn't know that, it's helpful.  One comment is, it only shows up if one includes --no-cache:
repo > ruff --no-cache --verbose .
...
[2023-05-26][20:51:46][ruff::rules::isort::categorize][DEBUG] Categorized 'time' as Known(StandardLibrary) (KnownStandardLibrary)
[2023-05-26][20:51:46][ruff::rules::isort::categorize][DEBUG] Categorized 'numpy' as Known(ThirdParty) (NoMatch)
[2023-05-26][20:51:46][ruff::rules::isort::categorize][DEBUG] Categorized 'folder_2.module' as Known(FirstParty) (SamePackage)
[2023-05-26][20:51:46][ruff::rules::isort::categorize][DEBUG] Categorized 'folder_1.module' as Known(ThirdParty) (NoMatch)
...

Totally, I'll try to clarify this a bit in the docs.

Feel free to tag me as a reviewer!


CMS: Correct CMS -sign and -verify with binary input on Windows

apps/cms.c: Correct -sign output and -verify input with -binary.
Also add related warnings on irrelevant use of -nodetach and -content options.
Fixes #15347.
This PR contains a preliminary commit just to demonstrate effectiveness of the solution on Windows,
with this message:
TODO REMOVE THIS TMP COMMIT: Add basic test_cms build to the Windows CI workflow
Checklist

[x] documentation is added or updated
[x] tests are added or updated

This pull request is ready to merge

Approved. Assuming the CI addition is dropped from the PR as there is a separate PR #15349 for that.

Done and merged - thanks @t8m


4.0 Ideas

The Cmd API released in version 3 was the first attempt at simplifying the API and making effects more reusable. But there are still more improvements that can be made. Let's use this thread to post feedback on the current version and ways to improve it.

New Cmd types like batch and sequence, except they dispatch resulting actions as they become ready instead of all at once at the end.
Break out integrations with popular libraries into separate projects (such as immutablejs-optimized combineReducers)

I'd personally propose replacing the Cmd.x functions with individual exports with names like queuePromise and queueDispatch. Similarly, Cmd.getState and Cmd.dispatch would probably be better as something like getStatePlaceholder and dispatchPlaceholder.
There's also the issue that loop as a name for the function to return is kind of weird, since in the common parlance it's not actually looping anything; that's what got me thinking of names using terminology like queue in the first place.

I guess at some point we should decide if we want to try and mimic elm's function names or not. Task and Effect? thunk ? schedule ?

"Thunk" seems like a bad idea, given that in this space it tends to mean redux-thunk pretty specifically.

i'm not married to any particular name, but i'd want to not change them without evidence that Cmd was confusing people or that a new name would be much more clear

I like having things be the same as elm. I kind of think of redux-loop as an elm polyfill for redux and should strive to match the implementation as closely as possible. With additional features as necessary, of course.
That said, configurable batching behavior would probably be at the top of the list in terms of features I would like to see. A parallel batch would be nice, which @bdwain already mentioned in the first comment.

Version 4.0 was just released. It was a small update to remove deprecated functionality and clean up the build and tests to make future changes easier.
Now that that's done, I'd love to start tackling some of these ideas and coming up with more.
The first one I'd like to do is configurable batching behavior. My thought is that we have batch and sequence now, but it'd probably be simplest for there to be one Cmd type for an array of Cmds, with 2 options available. Option 1 is a boolean to choose batch vs sequence (parallel vs series) behavior. Option 2 is to determine whether all actions dispatch at once at the end (the current behavior) or each action gets dispatched as it becomes ready.
I'm not married to any particular name, but I was thinking of calling it Cmd.array or Cmd.set and deprecating Cmd.batch and sequence. I'd rather not deprecate things again if possible, but I think it'd be confusing because I think option 2 should be called batch (and be off by default, which is not the current behavior). Option 1 can be called sequence and be off by default.
If we did that, we could release the new Cmd with a minor version bump and add deprecation warnings for batch and sequence in 4.0 and remove them in 5.0.
Thoughts?

#151  was made to address the batching issue.

Release 4.1.0 with the new batching changes.

I opened #153 to address the remaining two points. I'm going to close this issue as there's not much discussion anymore and all of the original ideas have been implemented.


Need to add "bind interfaces only = no" to config for Podman

Behaviour
I'm not sure if this is specific to Podman, but I'm getting an issue when running this container with a config file that resembles the example in this repo.
When starting the container under Podman, and ensuring that the port is appropriately forwarded with --publish, I find that I can't connect to the SMB server, even though I can see the conmon service listening on the port when using ss on the host.
Running a cat /etc/samba/smb.conf inside the container shows that Samba is configuring itself to listen only on certain interfaces, but the list of interfaces is empty for whatever reason:
hosts allow = <IP_ADDRESS>/8 <IP_ADDRESS>/8 <IP_ADDRESS>/12 <IP_ADDRESS>/16
hosts deny = <IP_ADDRESS>/0
interfaces =
bind interfaces only = yes

I found that I could fix this and make the container work by adding this to the config.yml:
global:
  - "bind interfaces only = no"

Expected behaviour
I'm not sure if this is specific to Podman as I don't really have Docker available to test with, but it seems like the "bind interfaces only" setting isn't going to add any security inside a container so it should probably be set to "no" by default.
Configuration

podman version 3.3.1
RHEL 8.5

Logs
Container starts fine on host when checking with podman logs samba, but this is what I see from a remote host:
# smbclient -L <IP_ADDRESS>
protocol negotiation failed: NT_STATUS_IO_TIMEOUT

And a remote nmap run shows the ports as listening even when the service isn't working, but I think this is a bit of a red herring caused by the way conmon works with Podman. The server is "listening" on 445 because I told it to with the --publish flag, but this doesn't indicate that the app inside the container is actually listening.
# sudo nmap <IP_ADDRESS>
Starting Nmap 7.91 ( https://nmap.org ) at 2021-12-26 21:03 GMT
Nmap scan report for nas.home.arpa (<IP_ADDRESS>)
Host is up (0.0018s latency).
Not shown: 996 closed ports
PORT    STATE SERVICE
22/tcp  open  ssh
111/tcp open  rpcbind
445/tcp open  microsoft-ds

Ah, I guess the problem comes from commit e604d61b5bb299ccc91ab86c6d63dd45f55fb02b, which commented out the line that parses the SAMBA_INTERFACES env var (which had a default value of eth0 and would have worked):
https://github.com/crazy-max/docker-samba/blob/cb8edcdb1b27e326ca5220da16257203f220b173/entrypoint.sh#L11
But entrypoint.sh is still trying to use that var even though it's unset/empty:
https://github.com/crazy-max/docker-samba/blob/cb8edcdb1b27e326ca5220da16257203f220b173/entrypoint.sh#L42


python312Packages.citeproc-py: Remove nose dependency and re-enable tests

Description of changes
Part of #326513

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

(Sorry about all the nitpicking! I really appreciate everyone taking the initiative to deal with this massive list of packages; I just want to make sure we do what we can to get packages in a good state or else get rid of unnecessary burdens in the process.)

No worries! I appreciate the nitpicks, they help improve future packages I submit and/or update!


EM evaluation failed

Hi there,
Thanks for releasing the code. I ran the model with a filtered spider dataset excluding extra hard samples. However I encountered errors below, although the training process can still go on, the evaluation accuracy is 0. Any idea on this situation?

Hey,
In this line, there is a try except block which is failing, can you removing and let me know what the actual exception is?


Ultrasonic sensor problem

Mr.hamuchiwa
Even if the Ultrasonic sensor is detected the car doesn't stop.pls help

Mr. Hamuchiwa I also have another problem, I included Cascade classifiers for left arrow and detected it,but the problem is I can't steer the vehicle in accordance as def(self,prediction) does not have a special condition for taking left . This is in rc_driver_helper.py file. Please tell me how to add our own steer() condition
Thank you

Mr.hamuchiwa,
After i run collect_training_data.py and collect data and i convert .npz file to .xml file and run rc_driver.py the car doesn't follow the lane and goes in a wrong direction. i want to know,

what will be the height of the camera from the car?
2.what should be the color of the lane and the road?
Thank You

Steering function call is in rc_driver.py, feel free to add your functions to handle special steering case.
Color doesn't matter, image is converted to grayscale before saving. Check out #139 if you have doubts on training.

@sundar19 my car keep running randomly after training..... did you get the car going well?

If you want to train your car for the lane

you should speed control your car
you should good streaming speed
3.try adding more hidden layers to your neural network
if all these don't work you need to go for convolutional neural network using tensorflow

@sundar19 how do i add more hidden layers to my neural network? i am new to this.....no really know how to do in the coding part

https://docs.opencv.org/2.4/modules/ml/doc/neural_networks.html#cvann-mlp-trainparams
This will help you.


[Hub Generated] Review request for Microsoft.DigitalTwins to add version stable/2020-10-31

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

Validation tools were run, but an exception occurred when running the linter in OpenApi hub: x-ms-secret was not recognized as a valid extension when checking against the schema.

@sjiherzig, do you plan to release SDKs for this version after merging this PR? Please update readme.md if your answer is yes.

/azp run automation - sdk

@sjiherzig, do you plan to release SDKs for this version after merging this PR? Please update readme.md if your answer is yes.

Thanks! I've updated the readmes accordingly.

@sjiherzig, if you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Changes looks good. Marked it as ARMSignedoff.

@erich-wang What would be the next step? We're all set for merging in our changes, I think.

@sjiherzig, I'm going to merge this PR, but I noticed that the API version is 2020-10-31, please let me know if this PR is ready to merge from service team perspective.

@erich-wang I discussed it internally, and we are ready to merge!


add "use strict" for node.js v4 support

Without the 'use strict'; Node.js v4.x will choke on let/const

Thanks! At first I was confused – how can Travis succeed on Node.js 4? Then I realized this was in cli.js. I'll add a sanity check test for it.

Hah, yea, I only noticed it because I decided to run the config-check in my test travis flow, and I was a bit confused at first because I'm also using eslint-plugin-node which catches that sort of thing in my codebase.


[BUG] Wrong number of vibrational frequencies

Describe the bug
The number of vibrational frequencies do not correspond to natoms * ndim.
To Reproduce
Steps to reproduce the behavior:
moldesc = 'C          -1.28652        0.25524        0.08013;C           0.07762        0.16535       -0.63495;O           0.22083        0.47033       -1.80613;C           1.24790       -0.33747        0.23567;H          -2.04623        0.61618       -0.60539;H          -1.22080        0.93293        0.92681;H          -1.57890       -0.72292        0.45183;H           1.03268       -1.33368        0.61210;H           2.16265       -0.36812       -0.34714;H           1.39077        0.32217        1.08708'
mol = dqc.Mol(moldesc=moldesc, basis="sto-3g")
print(len(mol.atomzs))
hf = dqc.HF(mol).run()
mol.atompos.requires_grad_()
vib = dqc.vibration(qc, freq_unit='cm^-1') # calculate vibrational frequencies
print(vib[0].shape, vib[1].shape)

Here I am trying to calculate the vibrational spectrum of acetone, which as 10 atoms in it (I'm using positions I obtained from a previous geometry optimization in PySCF). The output of the above script is
10
torch.Size([6]) torch.Size([6, 6])

So I have 10 atoms as expected, but I only have six vibrational frequencies.
Expected behavior
I expect to get 10 * 3 or 30 frequencies (of which 24 should correspond to vibrations).
Desktop (please complete the following information):

DQC version: 0.1.0
Python version: 3.9
PyTorch version: 1.10.2
OS: macOS

Additional context
I will also note that some of the frequencies as negative - that surprised me - and doesn't match what I expect or get from other comp chem programs like PySCF and ASE

This was a bug in my code from a copy paste error, my apologies!! The above won't actually run as I was calling vibration on qc (something I had defined in another cell of my notebook) but should I should have been calling it on hf.


Cannot copy files with names that contain supplementary characters / surrogate pairs

When copying files that have file names containing supplementary characters / surrogate pairs, using a CopySpec or from the resources folder when using the java plugin, Gradle is not capable of copying them, reporting that the files do not exist.
Expected Behavior
Copy should work.
Current Behavior
Copy fails:
Caused by: org.gradle.api.GradleException: Could not copy file '/Users/cgal0002/workspace/github.com/cjbooms/japanese-characters-maven/src/test/resources/supplementary/アンドリューは本当に凄いですawesomeだと思います𩸽.jpg' to '/Users/cgal0002/workspace/github.ccharacters-maven/build/resources/test/supplementary/アンドリューは本当に凄いですawesomeだと思います𩸽.jpg'.
        at org.gradle.api.internal.file.AbstractFileTreeElement.copyTo(AbstractFileTreeElement.java:79)
        ... 68 more
Caused by: org.gradle.internal.nativeintegration.filesystem.FileException: Could not get file mode for '/Users/cgal0002/workspace/github.com/cjbooms/japanese-characters-maven/src/test/resources/supplementary/アンドリューは本当に凄いですawesomeだと思います𩸽.jpg'.
        at org.gradle.internal.nativeintegration.filesystem.services.GenericFileSystem.getUnixMode(GenericFileSystem.java:72)
        ... 105 more
Caused by: net.rubygrapefruit.platform.NativeException: Could not get UNIX mode on /Users/cgal0002/workspace/github.com/cjbooms/japanese-characters-maven/src/test/resources/supplementary/アンドリューは本当に凄いですawesomeだと思います𩸽.jpg: file does not exist.
        at net.rubygrapefruit.platform.internal.DefaultPosixFiles.getMode(DefaultPosixFiles.java:48)
        at org.gradle.internal.nativeintegration.filesystem.services.NativePlatformBackedStat.getUnixMode(NativePlatformBackedStat.java:34)
        at org.gradle.internal.nativeintegration.filesystem.services.GenericFileSystem.getUnixMode(GenericFileSystem.java:70)
        ... 107 more

Context
This issue has been reported on the forum: https://discuss.gradle.org/t/build-fails-to-copy-resources-with-special-characters-in-file-name/20294
Steps to Reproduce (for bugs)
A reproducer project is available: https://github.com/cjbooms/japanese-filenames
Your Environment
Gradle Version: 3.1
Operating System: MAC OS X El Capitan
JVM Version:
java version "1.8.0_51"
Java(TM) SE Runtime Environment (build 1.8.0_51-b16)
Java HotSpot(TM) 64-Bit Server VM (build 25.51-b03, mixed mode)

Fails with Gradle 2.x, 3.x.

Reproduced on macOS, Linux and Windows.
Root cause and stack trace are the same.

This might be related to  GRADLE-2181 . These links might contain useful background information: http://happygiraffe.net/blog/2009/09/24/java-platform-encoding/ , http://stackoverflow.com/questions/11343828/file-encoding-has-no-effect-lc-all-environment-variable-does-it . TL;DR The file.encoding doesn't control file name encoding. There is a hidden system property sun.jnu.encoding, but that shouldn't be manipulated directly.
Search the Gradle source code with GRADLE-2181 to find some tests related to that issue.

Anyone working on it? Can I grab?

Thanks @hotchemi for your interest in this issue. No one is assigned to this issue. I see no problem for you to take the lead on this. Don't hesitate to ask questions if you feel stuck.

@lacasseio Thx, I'll try!

@lacasseio Sorry for the late. I digged into the problem and being little bit stuck...so let me share current status.
Cause
I followed the stack trace and it turned out the original exception is caused inside of nativeplatform's DefaultPosixFiles.java and posix.cpp.

DefaultPosixFiles.java
posix.cpp

It seems we gotta give unicode escaped string to this library to execute stat(here's a reproducer test).
Workaround?
Therefore, I tried to add a simple test case that adding convert/reconvert file name procedures before and after copy task. And it works well...I know it's kinda hucky though.
The reason I add this procedures is as  @lhotari said global encoding option shouldn't be manipulated directly.
Things to discuss?
So I'm just thinking two ways to fix the problem...

Adding a convert/reconvert file name procedures if file name contains surrogate pairs or so as I added above inside of gradle.
Fix adammurdoch/native-platform to allow surrogate pairs(sorry I'm not sure C++ and this library yet🙇)

Say, can you give me your thoughts or idea to fix the problem cleverly? This is the first time I dig into Gradle source code so I know you know a lot of things that I don't know. Thx😃

Thanks for looking into this @hotchemi. I think the preferred solution should be done in adammurdoch/native-platform. I suspect the java_to_char call should be changed to java_to_wchar. Unfortunately, stat and lstat doesn't support wchar_t path in Posix. You will need to find a way to achieve this.
@eskatos Would you be able to comment on how handling wchar in native-platform affect the performance?

@lacasseio Thx for your quick response. I tried to address with wcstombs on my local but java_to_wchar is only defined in win.cpp...I'd like to hear @eskatos's thought, sorry to bother you🙇

On the native side of JNI you get Java Strings as jstrings. These are Modified UTF-8 strings.
char and wchar arrays can be used to convey differently encoded strings. How they should be encoded depends on the targeted native method and what it expects.
java_to_char in native-platform already does a great job converting jstrings but maybe it does it in a way that is not expected by stat and lstat.
A good first step would be to add a test case to native-platform that exhibits the issue.

Still an active issue :(

In 2021, the problem is still relevant for cyrillic filenames
> Task :core:processResources FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':core:processResources'.
> Could not copy file '/builds/[project]/external-autotests/core/src/main/resources/clientfiles/?????????????????????? ??????????????.txt' to '/builds/[project]/external-autotests/core/build/resources/main/clientfiles/?????????????????????? ??????????????.txt'.
   > Problems opening file input stream for file: /builds/[project]/external-autotests/core/src/main/resources/clientfiles/?????????????????????? ??????????????.txt


Module requirements

One thing we do often at the company I work for is have modules require other modules. We used to go by "order" but that only goes so far: explicit requirements are extremely useful. The ability to say "I need module X, and X requires functionality from Y" has been a reoccurring theme. Therefore, I submit this PR.

Hello,
This looks good.
I'm assuming you have something on your end that uses that method to require the required modules ?

At the moment, yes. I was debating about adding it into Module::register() here, but some people may want to have different requiring procedures (on a SAAS platform, for instance, you may want to check that someone's plan allows a certain module before enabling it). But with that being said, listening for the module.enabling would allow a great range of functionality with this.

I see, thanks. 👍


GraphRNN with node feature

Firstly, thanks a lot for your perfect work, especially the comprehensive baselines. However, as you mentioned in appendix A.6, GraphRNN can be used as a node and edge feature generation. I wonder if you have finished this part in this repository?

Hello, did you figure out if they have finished this part?


Include a short intro

Just a couple sentences above the TOC that would at least include the git version on which these were tested (see also #47). Many operating systems are pinned on older versions with varying behavior. Ideally each command would have an "added in x.x" clause, but I understand if that's going a bit too far.

Thanks for the suggestion @mahmoud, I shall work on it ASAP.

Yeah +1 on this!

Looks fine now?

Sure. You could probably go on, but the key part is there :)

@mahmoud PR welcomed 😆


Add customizable main loop iterator

This is an implementation for the proposal godotengine/godot-proposals#1592 bases on the existing PR GH-42875 by @AndreaCatania
I have rewritten the PR for:

Bring it up to date with master
Allow multiple custom iterators from different modules and extensions

The changes are:
Add class that allows specifying custom iterators
Add class that allows registering of custom iterators
Add possibility to set the enabled iterators in the project settings (default, if unset, is all are enabled)
Call the enabled iterators at the specific place in main where the default iterators are called
I added the classes CustomIterator (for modules) and CustomIteratorExtension (for gdextension)
Both work about the same, but the methods in CustomIteratorExtension have "_" prepended.
Mandatory methods to override are

get_name: Must return a unique name. This is used as an identifier.
get_type: Must return a IterationServer::IteratorType flag value, which can have one or a (bitwise) combination of the follow values

ITERATOR_TYPE_SEPARATE: Iterator uses one or multiple of the separate methods for process, physics or audio iteration

ITERATOR_TYPE_MIXED: Iterator uses the mixed iteration method to for an iterator that combines several types.

There are also several methods that are optional or used conditionally:

process_iteration, physics_iteration, audio_iteration: Called when get_type returns a flag with ITERATOR_TYPE_SEPARATE. These either add to, or replace the _process, _physics_process or AudioServer functionality.
mixed_iteration: Called when get_type returns a flag with ITERATOR_TYPE_MIXED. This is when the iterator does not fit into a single type.

The "*_iteration" methods are called in the "Main::iteration()" methods at specific locations:

mixed_iteration: The first iterator to be called. It is called before the default "_physics_process" is called.
physics_iteration: Called right before the default "_physics_process" is called, but after mixed_iteration
process_iteration: Called right before the default "_process" is called.
audio_iteration: Called before the update of the AudioServer is be called

These methods should return a boolean value that indicates if the engine should be exited (by setting the "exit" variable in the Main::iteration() method to the returned value)
Registering works similar to ScriptServer, using the class "IterationServer":
Create instance of the custom iterator (using memnew) and call register_iterator passing it.
The enabled iterators can be configured in the project settings. There base path is "application/config" withthe following fields: process_iterators, physics_iterators, audio_iterators and mixed_iterators.
They are each a list of strings containing the names of the iterators that are enabled for the specific usage.
For the setting there is one special name: "main". By leaving that out one can disable the default functionality of either "_process", "_pysics_process" or updating of the AudioServer. When main is disabled for either process or physics_process a warning will be printed when opening the project in the editor.
There are some things that can/should be added (like tests and (more) documentation), but I wanted to open this PR at this point to see if this is an acceptable change in general and/or if there are things that should be changed - especially as I generally build on what @AndreaCatania  started - which was, as I understand, written with godex in mind - and am not absolutely sure of what is actually needed.

The reason the original enhancement was limboed was because anyone who needed godex could patch the engine themselves, so there was push back. Looking for a new approach for the use of custom iterators.

Ah, nevermind then, in that case


Invitation Confirmation Screen Tweaks

Summary
Fixing the following tweaks:

Min width of the table is off from design spec ... needs to be fixed
Remove up arrow icon in header for now
In mobile, the header bar overlaps the fullscreen modal when it shouldn't

Ticket Link
https://mattermost.atlassian.net/browse/MM-17333

Other than the failing tests, the changes looks good. This will be tested as part of the Guest Account Feature. Removing the QA Review Label

Hey @srkgupta sorry I think this requires a follow up round of QA review as the tests were fixed based on your previous comments, and i think i forgot to follow up with a new QA review request

Changes looks good. This PR will be tested as part of the 5.16 Release Guest Account feature. Removing the QA Review label.


[MINOR] fixed link

Fixed broken link to spec
Signed-off-by: Sally MacFarlane<EMAIL_ADDRESS>

This is probably the best link, but worth noting that we currently follow the 0.9.4 version of the spec which is not the current dev branch.


Add some missing cfi frame info in aesni-x86_64.pl

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Merge to master and 1.1.1
Thanks!


Fix org.hbbtv_HTML5-DASH017

Copy improvements from RDK equivalent.

The functions that are exactly the same could get moved into the javascript code using them.
Not sure what would happen if a third platform was added.

I've just rebased the pull request. No change was needed.


Port new page-streaming code to REST APIs.

(Also includes an extra bit of documentation in Google.Api.Gax that I missed before.)

LGTM


README: typo and style fixes

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: schu sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @seh for the extra fixes, all addressed I believe.

Thank you @schu and @seh


bindPopup

Is it still possible to use L.marker leaflet calls such as bindPopup ?

I didn't realise it yet. I use this plugin to display static data only, without any interaction.

I encounter  the same problem,I have too many markers to show on map and it's so slow , when I use this plugin it can be rendered by canvas but the click event expired ,is there any way both rendered by canvas and retain the event for the maker? thanks

support for popups would be awesome

@lmiao2017 that's exactly my situation.
Maybe @eJuke has an idea?

I ended up writing this :
https://github.com/sgentile/pixi-overlay-marker-examples
It needs updated but uses pixi.js
On Tue, May 29, 2018, 4:09 PM Felipe Menezes<EMAIL_ADDRESS>wrote:

@lmiao2017 https://github.com/lmiao2017 that's exactly my situation.
Maybe @eJuke https://github.com/eJuke has an idea?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eJuke/Leaflet.Canvas-Markers/issues/1#issuecomment-392926537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIFNDy81FAgUWRAINNcBZHiro1txn_Fks5t3arngaJpZM4QKDn7
.

Have a Pull request that I just submitted that should handles this as well as binding toolTips.

Thank you @Spaction
I merged it into master


Setup the serve command

We are implementing a simple HTTP server that will provide a few endpoints which can be used to fetch the data extracted
by the scanner.

/scan returns a tarball with the the files generated by the scanner
/scan/extracted returns a json payload with the data present in the extracted.json file
/scan/convertedreturns a json payload with the data present in the converted.json file

This requests receive data from the body json payload:
{
  "repo_url": "https://github.com/elhmn/ckp",
  "ref":  "master" // default to the main branch
}

How to test the command:
Run the server using:
$ cargo run --features bevy/dynamic serve -p 3000

The run the requests using curl on any of the request:
$  curl -vsS localhost:3000/scan

TODO: edit the curl request with the json payload sent
should solve #32

This should be rebased off of https://github.com/elhmn/waking-git/pull/49

The server code is in an acceptable state, still need to write tests for each of this endpoints.

@Git-Math, I updated the description with a few instructions on how to test the pull request. It is now ready for review, 🎉