Showing Python not installed when executing Notebook

Type: Bug
Steps:-Step 1:-Open vscode
Step 2:-press CTRL+SHIFT+P
Step 3:-Then look for python interpreter
Actual:-When executing notebook it is showing no python install and also when checking in Python Interpreter it still showing no python install. Even in my machine python is already installed.

Python is already installed in my machine:-

Expected:-Jupyter notebook will run fine when python installed.
VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:29:14.590Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i5-1235U (12 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.71GB (3.01GB free)

Process Argv
--crash-reporter-id 9cf35ac9-2b4c-493d-a774-1d8180f862e5

Screen Reader
no

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

markdown-preview-github-styles
bie
1.0.1

python-environment-manager
don
1.0.4

vscode-pull-request-github
Git
0.56.0

python
ms-
2022.20.1

vscode-pylance
ms-
2022.12.20

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.12

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

LiveServer
rit
5.7.9

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848:30577860
nodejswelcome1cf:30587006
3biah626:30602489
3d0df643:30613357
89544117:30613380
fim-prod:30623723

Please could you enable logging as follows:

Open settings & go into Jupyter->Logging
Change the value to verbose
Reload VS Code,
Attempt to repro the issue & then copy the output from the Jupyter output panel.

Hi,
I tried the steps.

Visual Studio Code (1.74.2, undefined, desktop)
Jupyter Extension Version: 2022.11.1003412109.
Python Extension Version: 2022.20.1.
Workspace folder d:\Data Analyst Project
info 10:42:00.363: ZMQ install verified.
User belongs to experiment group 'jupyterTest'
User belongs to experiment group 'jupyterEnhancedDataViewer'
debug 10:42:00.630: Updating cache with Local kernels
, Added =
, Updated =
, Removed =
debug 10:42:00.630: Updating cache with Local kernels
, Added =
, Updated =
, Removed =
debug 10:42:00.630: Updating cache with Python kernels
, Added =
, Updated =
, Removed =
debug 10:42:00.630: Get Custom Env Variables, Class name = S (started execution), Arg 1: undefined, Arg 2: "RunPythonCode"
debug 10:42:00.630: Start refreshing Kernel Picker<PHONE_NUMBER>630)
debug 10:42:00.631: Get Active Interpreter, Class name = x (started execution), Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb"
info 10:42:00.631: LSP Notebooks experiment is enabled
debug 10:42:00.631: Get Active Interpreter, Class name = x (started execution), Arg 1: "/d:/Data Analyst Project"
debug 10:42:00.634: getCondaFile, Class name = k (started execution)
debug 10:42:00.634: getCondaVersion, Class name = k (started execution)
debug 10:42:00.634: getCondaVersionFromPython, Class name = k (started execution)
debug 10:42:00.634: Get Active Interpreter, Class name = x (started execution), Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb"
debug 10:42:00.634: Get Active Interpreter, Class name = x (started execution), Arg 1: undefined
debug 10:42:00.883: Get Active Interpreter, Class name = x (started execution), Arg 1: undefined
debug 10:42:00.885: Get Custom Env Variables, Class name = S, completed in 255ms, has a truthy return value, Arg 1: undefined, Arg 2: "RunPythonCode"
debug 10:42:00.885: Get Jupyter Sub Paths, Class name = j, completed in 255ms, has a truthy return value, Arg 1: "", Arg 2: "kernels", Return Value:
debug 10:42:00.885: Get Jupyter Kernel Paths, Class name = j, completed in 255ms, has a truthy return value, Arg 1: "", Return Value:
debug 10:42:00.886: Getting Jupyter KernelSpec Root Path, Class name = j, completed in 1ms, has a truthy return value, , Return Value: Uri:/c:/Users/utsav/AppData/Roaming/jupyter/kernels
debug 10:42:00.886: Get KernelSpec root path, Class name = j, completed in 256ms, has a truthy return value, Arg 1: {"a":false,"b":null}, Return Value: Uri:/c:/Users/utsav/AppData/Roaming/jupyter/kernels, Uri:/C:/ProgramData/jupyter/kernels
debug 10:42:00.914: Not Searching for kernels as path does not exist, c:\Users\utsav\AppData\Roaming\jupyter\kernels
debug 10:42:00.915: Not Searching for kernels as path does not exist, c:\ProgramData\jupyter\kernels
debug 10:42:00.915: Updating cache with Local kernels
, Added =
, Updated =
, Removed =
debug 10:42:01.736: Detected change in Python environments via Python API
debug 10:42:01.743: Detected change in Python environments via Python API
debug 10:42:02.009: getCondaVersionFromPython, Class name = k, completed in 1375ms, has a falsy return value
debug 10:42:02.009: getCondaFile, Class name = k, completed in 1375ms, has a truthy return value
debug 10:42:02.009: getCondaVersion, Class name = k, completed in 1375ms, has a falsy return value
debug 10:42:02.010: Detected change in Python environments via Python API
debug 10:42:03.759: Detected change in Python environments via Python API
debug 10:42:03.763: Get Active Interpreter, Class name = x, completed in 3132ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project", Return Value: undefined
debug 10:42:03.763: Get Active Interpreter, Class name = x, completed in 3129ms, has a falsy return value, Arg 1: undefined, Return Value: undefined
debug 10:42:03.763: Get Active Interpreter, Class name = x, completed in 2880ms, has a falsy return value, Arg 1: undefined, Return Value: undefined
debug 10:42:03.764: Get Active Interpreter, Class name = x, completed in 3133ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Return Value: undefined
debug 10:42:03.764: Rank Kernels, Class name = p (started execution), Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Arg 2: "", Arg 3: {"kernelspec":{"display_name":"Python 3 (ipykernel)","language":"python","name":"python3"},"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.10.7"}}, Arg 4: undefined, Arg 5: "", Arg 6: undefined
debug 10:42:03.764: Get Active Interpreter, Class name = x, completed in 3130ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Return Value: undefined
debug 10:42:03.764: Found 0 cached controllers
debug 10:42:03.764: Creating 0 controllers
info 10:42:03.764: Find preferred kernel for d:\Data Analyst Project\test\Netflix Dataset.ipynb with metadata {"kernelspec":{"display_name":"Python 3 (ipykernel)","language":"python","name":"python3"},"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.10.7"}} & preferred interpreter
debug 10:42:03.765: Rank Kernels, Class name = p, completed in 1ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Arg 2: "", Arg 3: {"kernelspec":{"display_name":"Python 3 (ipykernel)","language":"python","name":"python3"},"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.10.7"}}, Arg 4: undefined, Arg 5: "", Arg 6: undefined
debug 10:42:03.765: Rank Kernels, Class name = p (started execution), Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Arg 2: "", Arg 3: {"kernelspec":{"display_name":"Python 3 (ipykernel)","language":"python","name":"python3"},"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.10.7"}}, Arg 4: undefined, Arg 5: "", Arg 6: undefined
info 10:42:03.765: Find preferred kernel for d:\Data Analyst Project\test\Netflix Dataset.ipynb with metadata {"kernelspec":{"display_name":"Python 3 (ipykernel)","language":"python","name":"python3"},"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.10.7"}} & preferred interpreter
debug 10:42:03.766: Rank Kernels, Class name = p, completed in 1ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Arg 2: "", Arg 3: {"kernelspec":{"display_name":"Python 3 (ipykernel)","language":"python","name":"python3"},"language_info":{"codemirror_mode":{"name":"ipython","version":3},"file_extension":".py","mimetype":"text/x-python","name":"python","nbconvert_exporter":"python","pygments_lexer":"ipython3","version":"3.10.7"}}, Arg 4: undefined, Arg 5: "", Arg 6: undefined
debug 10:42:03.766: Compute Preferred Controller, Class name = P, completed in 3135ms, has a truthy return value, Arg 1: <Uri:/d:/Data Analyst Project/test/Netflix Dataset.ipynb>, Arg 2: undefined, Arg 3: {"a":false,"b":{"j":false,"q":{"a":{},"b":{},"c":0}}}, Return Value: {}
debug 10:42:03.768: Detected change in Python environments via Python API
debug 10:42:03.770: Get Active Interpreter, Class name = x (started execution), Arg 1: "/d:/Data Analyst Project"
debug 10:42:03.770: Get Active Interpreter, Class name = x (started execution), Arg 1: undefined
debug 10:42:03.770: Updating cache with Local kernels
, Added =
, Updated =
, Removed =
debug 10:42:03.770: loadData after detecting changes to interpreters
debug 10:42:03.770: Updating cache with Python kernels
, Added =
, Updated =
, Removed =
debug 10:42:03.770: refreshData after detecting changes to interpreters
debug 10:42:03.771: Interpreter removed C:\USERS\UTSAV\APPDATA\LOCAL\PROGRAMS\PYTHON\PYTHON39\PYTHON.EXE
debug 10:42:03.771: Get Active Interpreter, Class name = x (started execution), Arg 1: "/d:/Data Analyst Project"
debug 10:42:03.771: Get Active Interpreter, Class name = x (started execution), Arg 1: undefined
debug 10:42:03.771: Updating cache with Local kernels
, Added =
, Updated =
, Removed =
debug 10:42:03.771: loadData after detecting changes to interpreters
debug 10:42:03.771: Updating cache with Python kernels
, Added =
, Updated =
, Removed =
debug 10:42:03.771: refreshData after detecting changes to interpreters
debug 10:42:03.771: Interpreter removed C:\USERS\UTSAV\APPDATA\LOCAL\MICROSOFT\WINDOWSAPPS\PYTHON3.10.EXE
debug 10:42:03.774: Full interpreter list after refreshing (total 5ms, resolve 0ms, refresh 5ms) is length: 0,
debug 10:42:03.776: Full interpreter list is length: 0,
debug 10:42:03.776: Refreshed Environments
debug 10:42:03.777: Refreshed Environments
debug 10:42:03.777: Refreshed Environments
debug 10:42:03.777: Refreshed Environments
debug 10:42:03.777: Refreshed Environments
debug 10:42:03.777: Refreshed Environments
debug 10:42:03.778: Full interpreter list after refreshing (total 0ms, resolve 0ms, refresh 0ms) is length: 0,
debug 10:42:03.778: Full interpreter list is length: 0,
debug 10:42:03.778: Refreshed Environments
debug 10:42:03.809: Get Active Interpreter, Class name = x, completed in 39ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project", Return Value: undefined
debug 10:42:03.809: Get Active Interpreter, Class name = x, completed in 38ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project", Return Value: undefined
debug 10:42:03.810: Get Active Interpreter, Class name = x, completed in 39ms, has a falsy return value, Arg 1: undefined, Return Value: undefined
debug 10:42:03.810: Get Active Interpreter, Class name = x, completed in 40ms, has a falsy return value, Arg 1: undefined, Return Value: undefined
debug 10:42:04.331: _jupyterPythonPathFunction: "file:///d%3A/Data%20Analyst%20Project" is not a notebook
debug 10:42:04.332: Get Active Interpreter, Class name = x (started execution), Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb"
debug 10:42:04.354: Get Active Interpreter, Class name = x, completed in 22ms, has a falsy return value, Arg 1: "/d:/Data Analyst Project/test/Netflix Dataset.ipynb", Return Value: undefined
debug 10:42:04.354: _jupyterPythonPathFunction: Couldn't find interpreter for "file:///d%3A/Data%20Analyst%20Project/test/Netflix%20Dataset.ipynb"
debug 10:42:04.791: End refreshing Kernel Picker<PHONE_NUMBER>630)
info 10:42:36.890: Loading webview. View is notset
info 10:42:36.890: Loading web view...
info 10:42:36.891: Webview panel created.
info 10:42:37.027: Web view react rendered
debug 10:46:59.076: No controller, hence notebook communications cannot be initialized for editor d:\Data Analyst Project\test\Netflix Dataset.ipynb
debug 10:46:59.081: Full interpreter list after refreshing (total 0ms, resolve 0ms, refresh 0ms) is length: 0,
debug 10:46:59.081: Full interpreter list is length: 0,
debug 10:46:59.081: Refreshed Environments
debug 10:47:00.083: End refreshing Kernel Picker<PHONE_NUMBER>630)

@karrtikr looks like python didn't return any interpreters to jupyter extension on this case


Unable to resolve module os react native firebase-admin !

Error: Unable to resolve module os from /Users/ghassen_gh/React/Okito/node_modules/firebase-admin/lib/credential/credential-internal.js: os could not be found within the project or in these directories:
node_modules/firebase-admin/node_modules
node_modules

The Admin SDK is a server library and requires a Node.js environment. The SDK will not work in a react native environment.


Make spec.namespace fields immutable

Moving a control plane around has consequences beyond just the templates installed by helm, as the control plane namespace has its own semantics within Istio: it is often the place where you can put Istio configs that serve as defaults. By making the field immutable, we avoid strange side- effects that might occur when moving a control plane to another namespace.

So, what's the recommended way to move the control plane to another namespace? I think this deserve a small paragraph in the docs.

I'm not sure. Istio docs don't state anything about moving control planes. Personally, I'd suggest copying the CA root cert over and creating a second control plane, then slowly migrating workloads over. But that's just a guess on my part.
I think this is something the Istio community needs to document. We can then follow their best practices.

Also, a test for this scenario would be great too.

A test that verifies the namespace change is denied? Sure, I can add one.

So, what's the recommended way to move the control plane to another namespace? I think this deserve a small paragraph in the docs.
Also, a test for this scenario would be great too.

IIUC this change would mean it's not possible to move. Only remove/create. Is there any scenario where remove/create is more complicated than the move where the user would have to make sure root cert is moved correctly anyway plus some other unknowns?

So, what's the recommended way to move the control plane to another namespace? I think this deserve a small paragraph in the docs.

I'm not sure. Istio docs don't state anything about moving control planes. Personally, I'd suggest copying the CA root cert over and creating a second control plane, then slowly migrating workloads over. But that's just a guess on my part.

This could be another benefit of using our operator. The operator could handle all of this, no?

I added a test!

LGTM.
Should we add a note on the docs saying the namespace cannot be changed?
/hold

Done!

@dgn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_sail-operator_main
a1fa671d1ba4c54480fa0624ccfe4d91eb28e8d5
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@dgn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test unit-tests-arm64

integ-tests-arm64_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test integ-tests-arm64

scorecard_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test scorecard

e2e-kind_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test e2e-kind

e2e-kind-dualstack_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test e2e-kind-dualstack

lint_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test lint

integ-tests_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test integ-tests

e2e-kind-olm_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test e2e-kind-olm

e2e-kind-multicluster_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test e2e-kind-multicluster

gencheck_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test gencheck

unit-tests_sail-operator_main
e43a287617d6f781f08ab5c403def5dcda16f310
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


[ISSUE][CLOSED] [Bug] kill the existing pdebuild process and run when retrigger happens

 Issue by geunsik-lim
Thursday Jul 05, 2018 at 07:17 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/issues/101

Recently, We added ubuild build procedure to test Ubuntu platform as well as Tizen platform. So, we have to append an opeation to kill the existing pdebuild process and run when retrigger happens

 Comment by git-bot-sec
Thursday Jul 05, 2018 at 07:17 GMT

:octocat: cibot: Thank you for posting issue #101. The person in charge will reply soon.

:octocat: cibot: Thank you for posting issue #125. The person in charge will reply soon.

 Comment by geunsik-lim
Thursday Jul 05, 2018 at 07:36 GMT

This issue is disappeared by killing process that runs checker-pr-audit-async.sh and checker-pr-audit.sh


Wrong answer confirmation with Anki 2.1.45 (355e66e8)

Hi,
Thanks for NDFS! It makes my Anki reviews much more pleasant 😊
I’ve just installed the newest version of Anki, and NDFS’s answer confirmation doesn’t work well with it: every time a question is shown (including the first one in the review session, before answering any question), NDFS displays a confirmation for the current card, including its next review time.
I’m not sure if it is a bug in the new version of Anki or a compatibility issue.
For now I disabled the confirmation option as a workaround.
Júda

Seems to be an issue with the new scheduler. Can you try the v2 scheduler
and see if it works?
Unfortunately I don't have time rn to fix the issue. I'll get to it in a
few weeks
On Tue, Aug 3, 2021, 9:13 AM Júda Ronén @.***> wrote:

Hi,
Thanks for NDFS! It makes my Anki reviews much more pleasant 😊
I’ve just installed the newest version of Anki, and NDFS’s answer
confirmation doesn’t work well with it: every time a question is shown
(including the first one in the review session, before answering any
question), NDFS displays a confirmation for the current card, including
its next review time.
I’m not sure if it is a bug in the new version of Anki or a compatibility
issue.
For now I disabled the confirmation option as a workaround.
Júda
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ccz-2/No-Distractions-Full-Screen/issues/34, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACY5RNSWW47LYFWUJXB2B73T27TN5ANCNFSM5BOZ7IGQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

This occurs with v2. I haven’t switch to v3 yet, because I use desktop Anki in tandem with AnkiDroid, which doesn’t have v3 support yet.

Thanks for the info. I'll have to do a deeper dive into this when I get some free time

Thanks! 🙂

Hey @rwmpelstilzchen, did you find any other workaround for this particular issue by any chance?


Fix Sharpe ratio calculation

[ ] Fix the zipline implementation to use risk-adjusted returns in the denominator (https://github.com/quantopian/zipline/blob/master/zipline/finance/risk/risk.py#L106) Source: http://www.pipsafe.com/Forex-E-Books/The_Sharpe_Ratio.pdf
[ ] Default zipline to use 1M T-Bills instead of 10Y
[ ] Pass in mean returns instead of cumulative returns: https://github.com/quantopian/zipline/blob/master/zipline/finance/risk/risk.py#L91 is called by https://github.com/quantopian/zipline/blob/master/zipline/finance/risk/period.py#L222 with algorithm_period_returns calculated by: https://github.com/quantopian/zipline/blob/master/zipline/finance/risk/period.py#L210

See https://www.quantopian.com/posts/sharpe-ratio-in-backtest-result-differs-from-than-one-in-pyfolio?utm_campaign=sharpe-ratio-in-backtest-result-differs-from-than-one-in-pyfolio&utm_medium=email&utm_source=forums for context.

Does this mean the "answer key" is also wrong?

I'm not sure, I find it hard to navigate that. @justinlent could you help taking a look at what's going on there?

Sure. Maybe I can take the answer key's daily returns and import them into pyfolio to try to figure it out

@dunster @twiecki I've done a deep dive into the answer key document.  It looks like there are some issues as I look closely into the formulas and their interconnectedness.  There is not consistency between using arithmetic and compounded returns which is causing imprecise calculation of Sharpe.  I've added new columns to the answer key which I will share with Jess and Thomas, and whomever else would like to provide a fresh set of eyes to review my modifications. Once all are in agreement we can propagate the changes to zipline and pyfolio (only 1 minor change to pyfolio, more to zipline it seems)

I had a conversation a couple of days ago with Jess about the risk metrics in zipline vs pyfolio. This was brought up because of the issue with information ratio that we recently found. What I would really like to do is move all of the risk metric calculations to a new project. Both pyfolio and zipline would consume this project for the risk metrics. This would mean making the risk free rate a parameter to some of these functions where I belive pyfolio currently uses 0 implicitly.
The main reason for moving this to a new repo would be to make all the risk metrics that we report consistent. We can share implementation and tests for zipline and pyfolio which is great for avoiding duplication and introducing subtle differences. I would want this to be a standalone project because I don't think it makes sense for zipline to be a dependency of pyfolio or for pyfolio to be a dependency of zipline.

@llllllllll This sounds great to me.  Do you propose us waiting to migrate to the soon-to-be-developed risk module without fixing the existing bugs in zipline?  I think we should at least fix the Sharpe Ratio (and the underlying annual returns intermediate calculation), unless you think migrating to the to-be-developed module won't take that long. How long do you think it would take?

I am not sure. I don't know who will own making this change but I imagine in will need to be a joint effort from the pyfolio and zipline teams. @dunster and @richafrank, any thoughts on this?

Lets move the discussion to #855.

We should definitely fix it in zipline first. Sharpe is a critical statistic we use throughout. The fix also shouldn't be too difficult.

I think it would be a good idea to show @ehebert because he did a lot of work on the answer key in the past.  Definitely @Jstauth too.

@twiecki your changes seem reasonable from an implementation stand point (and I defer to your and @justinlent's expertise on the why).
If the spreadsheet is too difficult work with, it may have outlived its usefulness, especially know that we have more experts looking at the risk calculations.
While making this fix, we could deprecate its usage (at least for Sharpe) and have the Sharpe tests in tests.risk.test_risk_cumalitive be checked with input defined in Python in the test suite instead of checking the answer sheet.

https://github.com/quantopian/zipline/pull/1322


Trying to install using documented command:
go get -u github.com/radovskyb/watcher/...
Got error:

go: downloading github.com/radovskyb/watcher v1.0.7
go get: installing executables with 'go get' in module mode is deprecated.
        Use 'go install pkg@version' instead.
        For more information, see https://golang.org/doc/go-get-install-deprecation
        or run 'go help get' or 'go help install'.

I got the same problem, did you fix it ?

This worked for me (1.19)
go install github.com/radovskyb/watcher/...@latest


Pruning

Haskell code for pruning jets.
Adds C example test cases for pruned and unpruned Simplicity programs.

Code looks good to me (at least, the C code). But I cannot build it -- I get the error
Preprocessing library 'Simplicity-Indef' for Simplicity-0.0.0..
Building library 'Simplicity-Indef' instantiated with
  Simplicity.Primitive = <Simplicity.Primitive>
for Simplicity-0.0.0..
[1 of 8] Compiling Simplicity.Primitive[sig] ( Haskell/Indef/Simplicity/Primitive.hsig, nothing )
[2 of 8] Compiling Simplicity.Term  ( Haskell/Indef/Simplicity/Term.hs, nothing )
[3 of 8] Compiling Simplicity.JetType ( Haskell/Indef/Simplicity/JetType.hs, nothing )
[4 of 8] Compiling Simplicity.Semantics ( Haskell/Indef/Simplicity/Semantics.hs, nothing )

Haskell/Indef/Simplicity/Semantics.hs:71:41: error:
    Variable not in scope:
      lift :: ReaderT PrimEnv Maybe b -> ReaderT PrimEnv Maybe b
    Suggested fix:
      Perhaps you want to add ‘lift’ to the import list in the import of
      ‘Control.Monad.Reader’
      (Haskell/Indef/Simplicity/Semantics.hs:14:1-41).
   |
71 |   optfs a | Just jt <- matcher =<< jm = lift . ReaderT $ flip (proxyImplementation fe jt) a
   |                                         ^^^^

which isn't even in our library, but I'm not sure what to do about it.

It's commit 5e4c132dbdf2010438aef34ce39202293d7c675e.

I think this might be my fault for not building intermediate commits.

Should be fixed now.


Doesn't support the f-sharp included with .NET Core CLI (intended?)

Is this intended or is this functionality set to be exclusively for the https://github.com/OmniSharp/generator-aspnet generator?

Functionality intended for the yo-aspnet generator.


Different paths per loader

I have a base html tag in my document and reference files via src="assets/bla.css" instead of src="/assets/bla.css" (note the leading /).
The issue that causes is that the url-loader references fonts via assets/fonts/bla.otf instead of /assets/fonts/bla.otf inside a CSS file which then causes the browser to try to load the font from /assets/css/assets/fonts/bla.otf.
Is there some way to tell the url loader how to reference the file inside the CSS file so that it would use a leading /?
output: {
  publicPath: ''
},

{
  test: /\.(woff2?|eot|ttf|otf)(\?.*)?$/,
  loader: 'url-loader',
  options: {
    limit: 10000,
    name: utils.assetsPath('fonts/[name].[hash:7].[ext]')
  }
}

In case the asset exceeds the limit the file-loader is used which provides options for this (see the file-loader README). publicPath should default to '/', since the file-loader will generate a URL looking like this
module.exports = __webpack_public_path__ + url

You can pass all  file-loader options to url-loader like you already did with name

The issue is that the paths inside the CSS are relative to the CSS file itself so unless I manually go into the CSS file and prefix the font paths with a / it will try to load them from a wrong path.
That is why I am asking if there is a way to specify 1 path that is used for output and 1 format that is used inside the CSS. I basically want to tell Webpack to prefix all font paths inside a CSS file /.

Thats what the file-loader && css-loader should do already. webpack picks up the relative path and replaces it with the output path of the asset in the CSS

Solved it by setting the domain as publicPath.


emcee documentation - Incrementally saving progress error

Hi,
In the current documentation the suggestion for saving the progress of a chain as it goes does not work (at least on my system- Python 3.5.1, numpy 1.10.4). It raises an error:

TypeError: sequence item 0: expected str instance, numpy.float64 found

I suggest a change of:
f.write("{0:4d} {1:s}\n".format(k, " ".join(position[k])))
to
f.write("{0:4d} {1:s}\n".format(k, " ".join(map(str, position[k]))))
which then works as intended for me.

This has been fixed finally for the next release. Took long enough :-\


2016 year inreview

Coverage remained the same at 64.74% when pulling c420c99354437a4138b61748160fd90b0b7d0668 on 2016-year-inreview into 42d5780bedba7d9dc82d6d8dd68b447b03cf17b6 on master.


RSR-153 Create new SSD file with its Substation, VoltageLevel and Bay section - Part 1/2 Creation mode

As a COMPAS service user, this US purpose is to create a new SSD file with its Header but also its Substation, VoltageLevel, Voltage and Bay sections.
As service user, I would like to create a new SSD file with its Header but also its Substation, VoltageLevel, Voltage and Bay sections.
The US has to fulfill the 61850-6 XSD schema. This US is contributing to the following functional services as described in the 61850-6:

S45 Edit / create substation topology (connections between primary equipment (9.2.4)

To define context, COMPAS service checks:

If the SSD file Header.id or header.version or header.revision is not provided as input, COMPAS service is in creation mode to create new SSD file.
If the SSD file Header.id and header.revision and header.version are provided as input, COMPAS service is in update mode to edit an already existing SSD file and add new inputs objets.

SSD file creation UseCase:
The COMPAS service checks if the SSD file defined by the header.id, the header.version and header.revision are existing or not. If not, the COMPAS service create the new SSD file.
The Header.toolID is fixed string "COMPAS" and is not provided as input.
The inputs of this service are:

SSD filename (string) (required). It shall be unique within the database.
Header.id, Header.version (string) and Header.revision (string) (required if header.id, header.revision and header.version is not referring to existing create the SSD file)
Hitem.who, what, why attributes (optional) ; the Header/History/Hitem.when is automatically defined as current system date/time
Substation.name (string) (required). It shall be unique within SSD file.
VoltageLevel.nomFreq (string) (optional), VoltageLevel.numPhases (number) (optional) and VoltageLevel.name (string) (required). VoltageLevel.name shall be unique within a Substation section.
Voltage section is optional :

Case 1) All Voltage attributes are kept empty or omitted then Voltage section will not be created
Case 2) If Voltage section exists, provide Voltage.unit (string) (required), Voltage.multiplier (string) (optional) and Voltage/Value (number) (required)

Bay.name (string) (required). It shall be unique within a VoltageLevel section.

US process
SSD file creation UseCase:
The following steps have to be considered to create a new SSD file with its Header and its Substation, VoltageLevel and Bay section.:
If Header.id and Header.revision and Header.version is not referring to an existing SSD file the COMPAS service creates the new SSD file.
If a mandatory attribute is missing then an error message is returned and the processing is aborted

Create the SSD file with its Name, Header and Hitem
Insert into Substation.name, the VoltageLevel node with its attributes:
-- VoltageLevel.nomFreq (optional)
-- VoltageLevel.numPhases (optional)
-- VoltageLevel.name (required)

If at least Voltage value is specified as input, insert into VoltageLevel the Voltage:

Voltage.unit (required if Voltage section exists)
Voltage.multiplier (optional if Voltage section exists)
Voltage/Value (required if Voltage section exists)

Insert into VoltageLevel, the Bay section with its name
For new SSD file creation, a new Hitem node is appended:

Hitem.revision = current Header.revision and Hitem.version=current Header.version (required)
Hitem.who, what, why attributes (optional).

Output
The header.id, header.revision and header.version are output in case of new SSD file creation
A Substation, VoltageLevel, Voltage (optional) and Bay sections are inserted into SSD file
The SSD file is stored in the database with the Header.id, Header.version and Header.revision as attributes
A message is returned to user in case of success or not of the processing
Error messages
If a mandatory attribute is missing then an error message is returned: "This attribut is mandatory attribut_name. End of processing."

CoMPAS has already:
Database to store SSD files
The exsisting CoMPAS service already makes sure that a SSD file has unique by adding a UUID to the header.
Technical refinement is needed to re-use the existing functionality.


Add tool: Streamlab

Here's a suggestion to include a stream test tool I have coded, and am also frequently using in my day-to-day work with streams. I believe it should be useful also for readers of this list.
More info about the project: https://github.com/vimond/streamlab

@torerikal thanks for the contribution, I have merged. I'm gonna have to play around with the project, seems really useful!

@krzemienski Thanks for merging my addition, however I can't find it in the commit log. Did your other PR (#32) merge at the same time block or overwrite this PR?

@torerikal I believe your commit had a conflict with the other commit so i resolved the conflict manually and it probably got rolled up into that commit.

Actually it seems it not there so something must have happened with my merge....git....do you mind opening again from the current master? @torerikal

Sure, please see #33.


Update dependency to avoid security flawness

It is an upgrade of a dependency to resolve the CVE-2019-16769 security issue.

What's the current status of it? I think this security issue should be addressed.

The vulnerability was resolved with the patch update 2.1.1, hence this change to bump to that version. Looking at the release log of the serialize-javascript package (https://github.com/yahoo/serialize-javascript/releases) it seems the bump from v1 to v2 has breaking changes. Have these been tested?

Not by me. But serialize-javascript is used by terser-webpack-plugin which is used by webpack which is then used by laravel-mix.
There are newer version of all packages in between but I'm not sure what breaking changes this comes with.

@JeffreyWay @taylorotwell - could we have an update on this one? Its to patch a CVE.

I don't know the status of this pull request. My trouble is that I don't know that much of Laravel Mix to fully understand the consquences of updating serialize-javascript to the newer version. I even don't know where the package is being used in. However I ran my laravel code with the updated package and nothing broke. But as I stated, I don't know the full usage of the package....
I can see that Webpack already has a pull request that would resolve this issue, but it's blocked: https://github.com/webpack/webpack/pull/10108
It's better if @JeffreyWay can take a look at this pull request, and said if we need to wait for Webpack to resolve it self?

A little disappointing to see this has not progressed. For me, it looks like a reason to move from mix if security vulnerabilities don't get patched in a timely fashion.

I think running npm audit fix will sort this without changes being needed. Let me know if not and we can get our pitchforks and head for @JeffreyWay 😂


Restructure README installation & configuration instructions.

Readme should get restructured:

[ ] The DNS configuration and Testing it out - sections should follow the Install section.
[ ] Troubleshooting section should get added after Testing it out - section.
[ ] Different sections should be linked to from the top of the README for easier access.

i spent a few hours on this but didnt get it to work. i tried the docker container but it didnt create the requested txt records. i dont want to go into details but just vote for "better" documentation/instructions.


Fix iso collision a bit and implement fireball sound effect

Fixes a few random things:

Ralph now runs correctly out of the Trulu cave, the DIFF of 1/128 was making his bounding box just slightly too high and clipping the top of the door. Slightly hacky fix but seems to work ok.
Implement using the correct sound effect for the fire magic ball.
Slightly tighten up the spike collision detection to match original implementation.

Preview here: https://pr-495.lba2remake.net

Nah ok this has broken a bunch of other things with iso collision, will try and resolve.

Nah ok this has broken a bunch of other things with iso collision, will try and resolve.

ok found a much nicer way of doing this, rather than moving the entire actor box up by 1/128 I just move the bottom of the actor box. This means the top of Ralph's bounding box no longer clips the top of the door, but all the other corner cases (like running up steps) still work as expected.

ok found a much nicer way of doing this, rather than moving the entire actor box up by 1/128 I just move the bottom of the actor box. This means the top of Ralph's bounding box no longer clips the top of the door, but all the other corner cases (like running up steps) still work as expected.


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #57.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or Other) nor an SPDX license expression.

The license should be CECILL-C if you want to fix that too.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


empty garbage buffer when opening an entry in a new window from lsp provider

Info

Operating System: Ubuntu 22.04

Shell: bash

Terminal: gnome-terminal

nvim --version: 0.10, 6d93bdd45

fzf --version: 0.42.0

[x] The issue is reproducible with mini.sh

Some modifications to `mini.sh`
diff --git a/scripts/init.lua b/scripts/init.lua
index 5a685c6..f58f4c3 100644
--- a/scripts/init.lua
+++ b/scripts/init.lua
@@ -7,6 +7,12 @@ vim.o.termguicolors = true

 require "fzf-lua".setup { global_git_icons = false }

+require'mason'.setup()
+vim.cmd[[MasonInstall lua-language-server]]
+
+require'mason-lspconfig'.setup()
+require'lspconfig'.lua_ls.setup{}
+
 vim.api.nvim_set_keymap("n", "<C-\\>", [[<Cmd>lua require"fzf-lua".buffers()<CR>]], {})
 vim.api.nvim_set_keymap("n", "<C-k>", [[<Cmd>lua require"fzf-lua".builtin()<CR>]], {})
 vim.api.nvim_set_keymap("n", "<C-p>", [[<Cmd>lua require"fzf-lua".files()<CR>]], {})
diff --git a/scripts/mini.sh b/scripts/mini.sh
index 69d86fb..1b61eed 100755
--- a/scripts/mini.sh
+++ b/scripts/mini.sh
@@ -36,6 +36,9 @@ if [ "${1:-}" = "reset" ]; then
 fi

 download_plugin "nvim-tree" "nvim-web-devicons"
+download_plugin "neovim" "nvim-lspconfig"
+download_plugin 'williamboman' 'mason.nvim'
+download_plugin 'williamboman' 'mason-lspconfig.nvim'

 # if exists, link to local folder so we can test local changes
 if [ -d "${plug_dir}" ]; then

Description
With lsp providers, opening the entry in a new window leaves an empty garbage buffer.

Run nvim with the above modified mini.sh. This sets up lua_ls.
Edit any lua file in this repository, e.g. lua/fzf-lua/init.lua
Use :FzfLua lsp_references e.g. on require
Select an item with <C-s> or <C-t>.
Run :ls. There is an empty [No Name] buffer.

https://github.com/ibhagwan/fzf-lua/blob/257aafb3c3a3475821db74d1a48ea5b097ac53a9/lua/fzf-lua/actions.lua#L127-L139
Here, :new (for <C-s>) is unconditionally run if the entry is a URI entry, even if it has file:// scheme. I think it's more appropriate to use :split, :vsplit, and :tab split for file_split, file_vsplit, and file_tabedit. In fact, the git actions use them.

Ty @tomtomjhj, I’ve been busy traveling in recent weeks, I’ll look into this and solve it as soon I can.

think it's more appropriate to use :split, :vsplit, and :tab split

Correct, these were one of the first actions ever written and this slipping through as URI jump to location was added at a later stage.
Ty @tomtomjhj!


Only show the first line of multi-line tasks in the task list

Feature Request
What problem does it solve?
Sleek supports creating tasks that consist of multiple lines and encodes the newlines as DLE in the todo.txt file. However, the newlines only seem to have some effect in the task editor. In the task list, they are just shown as spaces and the entire content of the multi-line task is just put in one line. If there are long task descriptions, the task list becomes very cluttered.
I think it would be a lot better if only the first line is shown in the task list (with some kind of indicator that there is more content).
This issue goes in the direction of #510 but is a lot simpler with a much narrower scope. I hope that makes it relatively easy to implement. I would be really really glad to see the multi-line feature becoming useful for my use case in the near future. 😃
How does the user interface change to accommodate this request?
There are a few ways of implementing this, so I'll try to give some suggestions. As long as by default only the first line of text is visible in the task list, I'm happy.

The probably fastest implementation would be to show a badge (like for due text, possibly with just "..." as the content) that indicates that some content is missing. To see the full text, you can just click on the task and open it in the editor.
Clicking on the "..." badge could also open a dialog with the full text.
A bit nicer would be a way to expand the text within the task list, e.g., with some < / v arrow button. When expanding the item, the newlines should probably be rendered as newlines instead of spaces.

@ransome1 Is this something that aligns with your vision of sleek and that you might be willing to implement someday? Or is this a feature you don't agree with?
I understand that your time is limited and this message is absolutely not intended to push for getting this feature implemented sooner. I am really just curious what you think about my idea and if I maybe also totally misunderstood how the multi line feature was intended to be used.

@dlaidig I am afraid that I can not contribute any time towards sleek at the moment and also in the near future, since my private situation has changed. Unless somebody else contributes something like this, please don't  expect it any time soon. I wish I could give you a bit more optimistic news.

Like I said, I didn't mean to push, I just wanted to ask if the feature I described sounds reasonable to you or if it conflicts with your idea of how sleek should work.


vendor: update virtcontainers to pull in latest kata gRPC updates

We have added support for passing network details to the agent in
virtcontainers' kata-agent.  Revendor to pull this in.
Shortlog:
43c9ed3 kata-agent: add initial test for kata-agent networking
082ec3f kata-agent: add network gRPC support
4b30262 setup: #!/bin/bash in the wrong place
8b7fddf vendor: re-vendor kata-containers agent
07c2c88 test: Modify block device test to test for SCSI as well
a1a569a scsi: Add SCSI support for volumes as well
3c038f1 scsi: Pass SCSI address to hyperstart agent
80bdc50 tests: Fix tests to take into account default driver for block devices
7df3f45 scsi: Add support for scsi driver for hotplugging drives
10e2349 1.0.7 release
10d462c block: Fix disabling block device use
fd6f6ed pod: Factorize network creation and removal into pod functions
2fec63f pod: StartVM() does not need any parameter
0dd84be container: Remove the container from pod structure on deletion
b852d93 api: Stop the VM when stopping the pod
17065de pod: Simplify pod stop
bcb8643 pod: Delete containers when deleting the pod
Signed-off-by: Eric Ernst<EMAIL_ADDRESS>

cc/ @jcvenegas @sboeuf @amshinde

@amshinde Could you please take a look at those failures ? I am asking you cause they're all related to the same test failing run hot plug ....
I have the same failures on https://github.com/containers/virtcontainers/pull/637 which has nothing to do with device hotplug.

@sboeuf Yup, will take a look.

Replaced with #1044
Can be closed now


Gets 301 redirect when request path contains "images"

First, create a server that contains some resource:

Then, try /group and it goes as expected:

But when try /images, it responses with a 301 redirect (from /images to /images/), which result in failure in ajax request。

I guess it might be the problem that images is also the directory of the serve's public assets (remaining to be tested.)

@hyiso remove the last /, you should request http://localhost:9999/images not http://localhost:9999/images/

Nice catch, I do think this is a bug in json-server as it serve /images, stylesheets statically and causes the static middleware before reaching the actual routes.
for now,TEMPORARY, you could do one of the following:

Use something other than images as a key for your resource
Find your json-server installation folder

If you installed it locally: you will find it in your project root directory node_modules/json-server, navigate to node_modules/json-server/lib/server/public and remove the images folder, then restart your server, it should work (don't worry, the only flaw of removing this folder is you are going to loose json-server logo-like image

if you installed it globally: it depends on your OS, node/npm installation, you can google "global npm package root"

@typicode: I do suggest one of the following:

hash images and stylesheets folder something like  images_static_756476574 and stylesheets_static_8288488343
further enclosing the static files into unlikely-to-chose-name folder like public/statically_served_for_json-server_349938

I could do PR with this or whatever else you suggest


Cleanup pkg

attempt to clean up
files and folder structure more reasonable (i.e. no filtering functions in the WT_class file, etc.).
also added tests to raise coverage....
...in some modules. I can document the differences (will update soon)

Enhancing test coverage


Enhancement: Paste from Clipboard

It would be helpful if DasherG could at least paste-in keyboard input from the system clipboard.
This will need to work on both Windows and Linux...

Fixed via commit https://github.com/SMerrony/DasherG/commit/b896f37c15b4d8ccbd2dee5a04ac890b5383e206


Set image as background

Hello! Can I set image as background of the bar?
I think it would be very useful. For example, we can have blurred pseude-transparent background with this.

No, you can't.

For example, we can have blurred pseude-transparent background with this.

I'm not sure how an image as background can achieve this. If you want a blurred transparent background, you should use a compositor that does this for you.

I'm not sure how an image as background can achieve this.

Just prepare blurred backround image and set it as background of the bar

If you want a blurred transparent background, you should use a compositor that does this for you.

But why then polybar has pseudo-transparent option? What if I just don't want to use compositor?

But why then polybar has pseudo-transparent option?

For people that want transparency without using a compositor. But you can't have everything without a compositor. For fancy things like blurring, shadows and fading effects you generally need one.

What if I just don't want to use compositor?

You could blur the background image in the region where polybar is and have a pseudo-transparent polybar.

For fancy things like blurring, shadows and fading effects you generally need one.

I know :D

You could blur the background image in the region where polybar is and have a pseudo-transparent polybar.

Ok, thanks. But actually I was using video wallpaper set with xwinwrap. So I will try to blur this part video region

Does polybar's pseudo-transparency work with the video wallpaper?

Does polybar's pseudo-transparency work with the video wallpaper?

Nope

Yeah, I thought so.


Add maintenance notice for migration

We believe this is necessary for the migration, to have a maintenance notice on the root page, because DNS records that we repoint to the github.io page from e.g. girder.dandiarchive.org will not get redirected to the /maintenance.html page but to the root page.

looks good - perhaps wait till noon tomorrow to merge?


Error: Invariant failed: As of Slate 0.42.0, value object are no longer schema-aware, and the value.change() method is no longer available. Use the editor.change() method on the new Editor controller instead.

I use editor like demo but occur this error:
Error: Invariant failed: As of Slate 0.42.0, value object are no longer schema-aware, and the value.change() method is no longer available. Use the editor.change() method on the new Editor controller instead.
The code is :
import React from "react";
import initialValue from "./initialValue";
import CannerEditor from "canner-slate-editor";

export default class DemoEditor extends React.Component {
  // Set the initial state when the app is first constructed.
  state = {
    value: initialValue
  };

  render() {
    const { value } = this.state;
    const onChange = ({ value }) => this.setState({ value });

    return (
      <div style={{ margin: "20px" }}>
        <CannerEditor
          value={value}
          onChange={onChange}
          serviceConfig={{
            name: "image",
            accept: "image/*",
            action: "https://api.imgur.com/3/image",
            headers: {
              Authorization: "Client-ID a214c4836559c77",
              "X-Requested-With": null
            }
          }}
          galleryConfig={null}
        />
      </div>
    );
  }
}

@rui-teixeira canner-slate-editor's slate core version is too low


Can you have nested lists?

Cna I have a sortable with another sortable list inside it, whereby all items in both lists can be dropped into the other list?

Sortable - does not support nesting.

exactly what i"m struggling with, i managed to get it working easily but no the data propagation back up
(everything gets re-rendered and items get copied instead of being transferred)

shame... It would be great to be able to use this library in a tree like component.

As I said earlier, if someone has a laconic solution (plugin), I with pleasure will review PR.

Would love to see this as well.

actually with some work you can have nested lists with Sortable, but you have to handle building 'tree' yourself.
Here is demo. It's not the best, but it is good to start from. I've achieved this using groups of Sortable with same name.

  <title>Sortable</title>

Sortable

Main

divitem 1

divitem 2

divitem 3

Floor 1

item 1

item 2

item 3

Zone 1

item 12

item 22

item 32

I'm using the following work-around for nested lists: in _onTapStart :
if(nestedStarted){
  //do nothing, drag already initiated
} else {
  nestedStarted = true; // drag and drop is starting flag
//... original code
}

In  _onDrop: nestedStarted = false; // end of the drag & drop
The goal is to avoid a start tap call for each list in the hierarchy of nested lists, where, in my case, the deepest lists are created last. It seems to do the job for my web app (touch device not tested). Any comments welcome.
Great plugin anyway.

I need similar functionality (nested lists and drag&drop between them) in Angular.js. I added an example of it here. I might work on this as it is a critical functionality for the project I am working on. Should submit a PR once I am done (if I am done).

+1

This is my take on, in building sortable tree:
http://localhost:8888/omaido/builder/
you can check code for clarification.

@goranefbl You'll have to give us access to your computer so we can see it... :)

hahaha sorry, wrong link pasted
http://itsgoran.com/builder/

That looks awesome, good stuff!

I have a nested list hierarchy, and have set each list in that hierarchy to be sortable. This allows me to sort elements within each list, including the elements that contain sub-lists. It all works nicely out of the box with nothing special configured.
I am using jQuery to select each list in the hierarchy and then apply Sortable to it. That does mean a separate Sortable instance for each list, which is fine for my application.
To help sort the hierachy, I have buttons using bootstrap to show and hide different levels, i.e. the number of nested lists that are shown, so you can focus on sorting each level at a time.
Is this what the OP is asking?

There is actually a way to have nested Sortables but the trick is that the "draggable" class has to be the same.

Currently does the Sortablejs support nested lists out of the box??
Can someone point to some nice demo or a plunker link ?


match git_user_email option not working

New Issue Checklist

[ X] Updated fastlane to the latest version
[X ] I have read the Contribution Guidelines

Issue Description
I don't have a global git email configured, and I've configured git_user_email in my Matchfile, but it still doesn't seem to work because pushing my cert repo still fails. See output below.
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

$ bundle exec fastlane match nuke development
[11:42:29]: Successfully loaded '/Users/jpaas/Documents/karma/worker-react-app/fastlane/Matchfile' 📄

+----------------+------------------------------------------------+
|           Detected Values from './fastlane/Matchfile'           |
+----------------+------------------------------------------------+
| app_identifier | ["com.karmacasting.karmacasting"]              |
| git_full_name  | Karma Apple Dev aka Cody McCodeface            |
| git_url        | git@bitbucket.org:karmacasting/karma-certs.git |
| git_user_email |<EMAIL_ADDRESS>                    |
| type           | development                                    |
| username       |<EMAIL_ADDRESS>                    |
+----------------+------------------------------------------------+

[11:42:29]: Cloning remote git repo...
[11:42:31]: Migrating to new match...
[11:42:31]: Cloning remote git repo...
[11:42:34]: 🔓  Successfully decrypted certificates repo
keychain: "/Users/jpaas/Library/Keychains/login.keychain-db"
version: 512
class: "inet"
attributes:
    0x00000007 <blob>="match_git@bitbucket.org:karmacasting/karma-certs.git"
    0x00000008 <blob>=<NULL>
    "acct"<blob>=<NULL>
    "atyp"<blob>="dflt"
    "cdat"<timedate>=0x32303137303332383135323733345A00  "20170328152734Z\000"
    "crtr"<uint32>=<NULL>
    "cusi"<sint32>=<NULL>
    "desc"<blob>=<NULL>
    "icmt"<blob>=<NULL>
    "invi"<sint32>=<NULL>
    "mdat"<timedate>=0x32303137303332383135323733345A00  "20170328152734Z\000"
    "nega"<sint32>=<NULL>
    "path"<blob>=<NULL>
    "port"<uint32>=0x00000000
    "prot"<blob>=<NULL>
    "ptcl"<uint32>=0x00000000
    "scrp"<sint32>=<NULL>
    "sdmn"<blob>=<NULL>
    "srvr"<blob>="match_git@bitbucket.org:karmacasting/karma-certs.git"
    "type"<uint32>=<NULL>
password has been deleted.
[11:42:34]: 🔒  Successfully encrypted certificates repo
[11:42:34]: Pushing changes to remote git repo...

*** Please tell me who you are.

Run

git config --global user.email<EMAIL_ADDRESS>git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'jpaas@slimshady.(none)')
[11:42:34]: Exit status: 128
[11:42:34]: Couldn't commit or push changes back to git...
[11:42:34]: Exit status: 128

+-----------------------+------------------------------------------------+
|                     Summary for match nuke 2.23.0                      |
+-----------------------+------------------------------------------------+
| git_full_name         | Karma Apple Dev aka Cody McCodeface            |
| git_url               | git@bitbucket.org:karmacasting/karma-certs.git |
| git_user_email        |<EMAIL_ADDRESS>                    |
| type                  | development                                    |
| username              |<EMAIL_ADDRESS>                    |
| git_branch            | master                                         |
| keychain_name         | login.keychain                                 |
| readonly              | false                                          |
| team_id               | 258N8Y58G5                                     |
| verbose               | false                                          |
| force                 | false                                          |
| skip_confirmation     | false                                          |
| shallow_clone         | false                                          |
| force_for_new_devices | false                                          |
| skip_docs             | false                                          |
| platform              | ios                                            |
+-----------------------+------------------------------------------------+

[11:42:34]: Fetching certificates and profiles...

[11:42:39]: No relevant certificates or provisioning profiles found, nothing to nuke here :)

Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
<details><summary>✅ fastlane environment ✅</summary>

### Stack

| Key                         | Value                                                |
| --------------------------- | ---------------------------------------------------- |
| OS                          | 10.12.3                                              |
| Ruby                        | 2.4.0                                                |
| Bundler?                    | true                                                 |
| Git                         | git version 2.11.0 (Apple Git-81)                    |
| Installation Source         | ~/.rvm/gems/ruby-2.4.0@worker-react-app/bin/fastlane |
| Host                        | Mac OS X 10.12.3 (16D32)                             |
| Ruby Lib Dir                | ~/.rvm/rubies/ruby-2.4.0/lib                         |
| OpenSSL Version             | OpenSSL 1.1.0e  16 Feb 2017                          |
| Is contained                | false                                                |
| Is homebrew                 | false                                                |
| Is installed via Fabric.app | false                                                |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/          |
| Xcode Version               | 8.3                                                  |

### System Locale

| Variable | Value       |   |
| -------- | ----------- | - |
| LANG     | en_CA.UTF-8 | ✅ |
| LC_ALL   |             |   |
| LANGUAGE |             |   |

### fastlane files:

<details><summary>`./fastlane/Fastfile`</summary>

```ruby
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://docs.fastlane.tools/actions
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "2.23.0"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."

  end

  desc "Runs all the tests"
  lane :test do
    scan
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    gym(scheme: "KarmaWorkerApp development") # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # match(type: "appstore")
    # snapshot
    gym(scheme: "KarmaWorkerApp development") # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://docs.fastlane.tools/actions

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

`./fastlane/Appfile`
app_identifier "com.karmacasting.karmacasting" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "258N8Y58G5"  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

fastlane
2.23.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.1.0

io-console
0.4.6

CFPropertyList
2.3.5

i18n
0.8.1

minitest
5.10.1

thread_safe
0.3.6

tzinfo
1.2.3

activesupport
4.2.8

public_suffix
2.0.5

addressable
2.5.0

babosa
1.0.2

bundler
1.14.6

claide
1.0.1

colored
1.2

highline
1.7.8

commander-fastlane
4.4.4

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170223

dotenv
2.2.0

excon
0.55.0

multipart-post
2.0.0

faraday
0.11.0

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

fastimage
2.1.0

gh_inspector
1.0.3

jwt
1.5.6

little-plugger
1.1.4

multi_json
1.12.1

logging
2.2.0

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

hurley
0.2

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.0.15

representable
2.3.0

retriable
2.1.0

google-api-client
0.9.28

json
2.0.3

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.2.0

rubyzip
1.2.1

security
0.1.3

slack-notifier
1.5.1

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

tty-screen
0.5.0

word_wrap
1.0.0

nanaimo
0.2.3

xcodeproj
1.4.2

rouge
1.11.1

xcpretty
0.2.4

xcpretty-travis-formatter
0.0.4

generated on: 2017-03-28

```

When I did configure a global git email, my certs repo was committed using the global email instead of the one configured using git_user_email.

It seems like it never gets to the point where it prints Add git user config to local git repo..., I'll take a look into this. Thanks for submitting this issue.

That's so weird, I just tried it and things work fine on my machine. So in your log, git user config to local git is nowhere to be found?

@KrauseFx I don't think so since I posted the complete console output. Another weird thing I noticed, is that this seemed to only happen with nuke. Later when I generated certs using fastlane match appstore, it used the email that was configured using git_user_email.


Augment README with dependency management guidance

Some simple guidance which assists in streamlining dependency management would go a long way. It looks like others have already noticed this as well in #180.
PR coming up...

Hi @lewismc,
PR#217 has been merged.
Therefore, closing this issue.


Merge with distutils@58fe058e4

Apply ruff/tryceratops rule TRY300
Apply ruff/tryceratops rule TRY301
Enforce ruff/tryceratops rules (TRY)
Enforce ruff/flake8-2020 rule (YTT301)
handle failures to find a user home directory
Remove outdated import error check for warnings
Satisfy ruff import format
Remove unused wininst-*.exe files
Apply ruff/Perflint rule PERF102
Apply ruff/Perflint rule PERF402
Apply ruff/Perflint rule PERF403
Enforce ruff/Perflint rules (PERF)
Add # local to ignore conflicts with upstrea
Use arm64 MSVC on arm64 Windows
Update distutils/dist.py
format
conditionally construct PLAT_TO_VCVARS
function-ify
nit
python supports comparison chains
apply feedback
Simply suppress the exception.
Sort imports
restore the lost comment
Define the variable in one place.
Extract logic around the mappings into a function to compute the spec.
Tweak docstring
Remove legacy msvc compiler modules.
Remove associated tests
Apply downstream VS 2017 support.
Apply error message from downstream.

Summary of changes

Closes
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in newsfragments/.
(See documentation for details)

Here's the more useful summary:
 🐚 git log HEAD^1..HEAD^2 --first-parent --oneline
58fe058e4 Merge pull request pypa/distutils#289 from pypa/debt/vs2017
b8c70bbba Merge pull request pypa/distutils#285 from saschanaz/arm64
23e3bfa72 Merge pull request pypa/distutils#287 from pypa/debt/remove-legacy-msvc
0107b075f Merge pull request pypa/distutils#264 from DimitriPapadopoulos/YTT
d42a6aa54 Merge pull request pypa/distutils#266 from DimitriPapadopoulos/TRY
6c224d3c6 Merge pull request pypa/distutils#278 from cj81499/handle-failure-to-find-user-home
2e8c2a25c Merge https://github.com/jaraco/skeleton
7f0e0c92c Merge pull request pypa/distutils#279 from abravalheri/warnings-exist
dcb0a648c Merge pull request pypa/distutils#282 from lazka/remove-wininst-exe
d7766971e Merge pull request pypa/distutils#265 from DimitriPapadopoulos/PERF
164f27685 Merge https://github.com/jaraco/skeleton


Seofon Action Ids (english)

All of seofon's action have been recorded for english, only thing that I can note that seemed strange was his Prison de Amars seems to use the action ID of 5000 (Ether Round). Everything else was functional have been labeled though.

I'll leave Skill ID 5000 alone for now, there's most likely a flag that gets set maybe for cannons. Need to dig into what the event data looks like.
Thanks for the work here! 👍


Divination option blocks auto-Play.

thanks for update
There still seems to be a bug with the Divination option.
I am checking with <IP_ADDRESS>.
--

turn on all options.
start the fight and use draw.
use Malefic, but the play is not executed automatically.
(If you do not target anything manually, the play is auto-targeted.)
(Auto-targeting of Play, "If Divination is executable , stop auto play/redraw" option is in effect. )
use Divination.
5, When I do a Malefic, neither auto Play nor Redraw is executed as I want it to be.
6, After this, there is no automatic Play.
I may be using only my environment, but with ver <IP_ADDRESS>, ffxiv sometimes crashes.
--

1.turn on all options.
2. use Divination and wait for Divination's recast.
3. start combat and use Draw.
4. automatic Play should be executed, but it is not.

I'll try it. I'm sorry for the inconvenience.

Just out of curiosity, are the following four items correct for the problems that are currently occurring?
Or are the 5 hyphenated items also included?

They are both different things and it happens with each.
I am trying different things.
Sometimes it seems that even if I think it is not playing automatically, it will Play automatically after 1 GCD.

I have released ver 1.3 and you can check it out.
I think we were able to fix these problems by using the method of checking the recast timer seconds, as it was sometimes executed and sometimes not executed in our environment as well.
Regarding the option to stop autoplay/redraw when Divination is executable, am I correct in understanding that you want autoplay/redraw even when Divination is executable?

Regarding the option to stop autoplay/redraw when Divination is executable, am I correct in understanding that you want autoplay/redraw even when Divination is executable?

I would like to
When Divination is not available, autoplay/redraw is performed
When Divination is available autoplay/redraw stops.
Maybe this should be a separate Issue.
I think I may have figured out when the application crashes.
It seems that the application crashes when the Play button is pressed while no cards are being drawn.
The same thing happens in v1.3.

There are cases where AutoPlay still works as expected in v1.3, and there are cases where it does not.
Sometimes it would not AutoPlay with or without the Divination option.
Perhaps I was mistaken from the beginning.
When Astrodyne is buffed, it does not AutoPlay even if I keep running Malefic after Draw.
(I thought it might be a problem with Astrodyne's chanting speed, so I checked the character's Spell Speed 400 and 1089, but that did not seem to be the problem.)
Translated with www.DeepL.com/Translator (free version)

Thanks for the reply.
I will add "when not holding a card" to the conditional statement that is currently implemented "when not executing a play, it will execute some action that was performed as is".
It is more than possible that maybe Sloth or other combo plugin implementations are interfering with Astro's implementation, so I will look at the implementation and try to remove the cause of the crash.
I have created a new Issue on Divination.
Sorry, but please send any complaints or requests about the behavior here.
https://github.com/aiczk/Astro/issues/4

We have confirmed a bug that prevents autoplay when Astrodyne is buffed, we will fix it immediately.

Fixed in ver 1.4


Starting bash from start menu doesn't process .profile

Bash launched from the start menu will not process the .profile file. This seems to be because bash is not being launched as a login shell, so it doesn't read the .profile.
I would expect bash launched from the start menu to start up a login shell (bash -l)

To start bash as a login shell use "bash.exe -l". You can also start in your home directory by using "bash.exe ~ -l"

Just curious, is it seen as intentional behavior that the default shortcut doesn't start a login shell?  I would expect that it would start one.

The shortcut is "bash.exe ~" so it does not start a login shell by default. I agree it probably makes sense as for at least the shortcut to launch a login shell. I'll check with the rest of the team today and see what they think.

Just heard back from the team and non-login shell is going to be the default for the anniversary update.  If you'd like to launch bash as a login shell I'd suggest modifying the "Target" of the C:\Users\<username>\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Bash on Ubuntu on Windows.lnk shortcut to read C:\Windows\System32\bash.exe ~ -l

Would y'all consider changing this in a future release?  Worth creating a UserVoice entry?  I feel like it's something that has come up periodically as a point of confusion.

non-login shell is going to be the default for the anniversary update
:tada:

I've changed the .lnk as suggested, and happy with the change :smile:

@pplu I'm glad that works for you.
@aseering It's absolutely something we will consider changing in the future.  A UsreVoice entry would not be a bad idea. I will also file a work item internally to revisit this.

Ok, I've created a UserVoice post here:
https://wpdev.uservoice.com/forums/266908-command-prompt-console-bash-on-ubuntu-on-windo/suggestions/14825565-open-bash-as-a-login-shell-by-default

The user voice entry is marked as "CLOSED - VS MERGE LEGACY (DO NOT USE)". Is this still an open item? .profile is the proper place to put PATH settings that you want to apply across multiple shells. Yes, I can use .bashrc but bash is not the only shell that I or others may want to use. That WSL never sources this when launching it in the default manner is a subtle yet serious problem. For example, the entry in the default .profile that adds ~/bin to your path does not take effect when launching bash.exe from the start menu entry (which is all I ever do: WINKEY-b-a-s-h-enter).
Even with the available workaround, this is confusing to any new users who are just getting started. In my case, I just wasted a couple hours trying to chase down why the PATH settings in .profile were not working. This is where the official installation instructions for golang tell you to place the command to add /usr/local/go/bin to your PATH. This type of native-Linux/WSL inconsistency is exactly the kind of problem I would think you wouldn't want developers dealing with.
IMHO this is a significant pain point that will keep cropping up until fixed.

the workaround doesnt work for me as the lnk file doesnt appear in my C:\Users<username>\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Bash on Ubuntu on Windows.lnk
Also, creating a new lnk file doesnt solve the issue for me :(

Please use wsl.exe or the distro-specific executables - here are the docs


TD3 not deterministic

Hi all,
Can't get the same results twice with TD3 but it works with PPO2.
import gym
from stable_baselines import TD3
from stable_baselines.td3.policies import MlpPolicy

env = gym.make('Pendulum-v0')
# env.seed(12345) not working
# env.reset()
model = TD3(MlpPolicy, env, verbose = 1, seed = 12345, n_cpu_tf_sess = 1)
model.learn(total_timesteps = 50000, log_interval = 10)
model.save("td3_pendulum")

System Info
Conda env, Python 3.6.9, TF 1.14 CPU from conda-forge, SB 2.10.0 from pip
Additional context
I need to study parameters effect on my custom env, but I'm not able to remove the randomness so far. Any idea ?
Thibaut

Hello,
As mentioned in the documentation (and in the PR https://github.com/hill-a/stable-baselines/pull/492), TD3 sometimes fail to be deterministic for obscure reasons.
Apparently, the tensorflow version affects the results. I could get deterministic results with TF 1.8.0 on cpu (but not with higher versions...).
PS: I could not find the other duplicated issue yet, but I'm pretty sure there was one...

Couldn't find that in the TD3 doc but ok I'll try with TF 1.8 ! Thanks.

Couldn't find that in the TD3 doc but ok I'll try with TF 1.8 ! Thanks.

See the note here

Ok thanks again! Indeed it works with TF 1.8.0 I'm closing the issue then!


Formula#keg_only should be a boolean

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run brew tests with your changes locally?

Since the left half of this conditional is false if keg_only_reason is nil, this was returning nil or true instead of false or true. Noticed this via the JSON output, where this value is exposed publicly.
Before:
    "linked_keg": "8.27",
    "pinned": false,
    "outdated": false,
    "keg_only": null,

After:
    "linked_keg": "8.27",
    "pinned": false,
    "outdated": false,
    "keg_only": false,

Rubocop doesn't like !!, but it feels like the most natural solution in this case. It's not like #2699 where there was a more natural method to use.
Library/Homebrew/formula.rb:1011:5: C: Avoid the use of double negation (!!).
    !!keg_only_reason && keg_only_reason.valid?
    ^

This was originally opened at #2862, which was accidentally pushed to origin rather than my fork.

it feels like the most natural solution in this case.

Maybe a guard clause? return false unless keg_only_reason
Or duck-typing? keg_only_reason.respond_to?(:valid?) && keg_only_reason.valid?

Pushed a commit to disable the Style/DoubleNegation cop. There's more discussion of this in bbatsov/rubocop#3344 and the Ruby style guide, but there's a reasonable agreement that returning values via double negative is just fine.

I believe I remember you saying that Ruby 2.3 couldn't be vendored because of some bug, is that still the case?
Would be nice once High Sierra ships, so we can use keg_only_reason&.valid?. 😉

Yeah, that's right - https://bugs.ruby-lang.org/issues/13413 hasn't been fixed yet. I might take a stab at it sometime.

Thanks again @mistydemeo!


Silence build warnings in Xcode 7 GM Seed

This silences the warnings that we see in Xcode 7 GM Seed (7A218 ) when building both Expecta itself as a standalone as well as the warnings we see when we build and use Expecta as a Cocoapod.
We like to have warning-free builds even in our tests.  :-)
All tests in the Expecta project pass.
Thanks for the awesome work on Expecta, I love it!

OOPS, and maybe bump Cocoapods to a new version 1.0.3 and include this change if possible so we can pull down the change as a pod?  :-)
Thanks!

Sure, this sounds good to me.


GraphQL?

Any chance this app / tutorial will one day demonstrate importing a GraphQL schema and using GraphQL in the app? 👯
(I have loved following this to learn more FQL but would also love to see a sample GraphQL schema and how FQL might handle more complex indexing / functions in the client)

Not this specific app but we plan to create a GraphQL sample soon which will demonstrate how to use GraphQL and FQL together. Given that I'm no longer the only DA @ Fauna, things will accelarate :)


Remove resource ids

Motivation for this change: Fetching with an id other than 0 is only possible outside a system or with a custom struct (which can only fetch a constant id). The plan was to use this for scripting, but resources added by scripts can easily be maintained by another resource, since access of those "script resources" is special anyways. This PR removes the unnecessary complexity and improves Resources access.
Resource ids were also a source of confusion in Specs where people thought that resource ids / component ids have something to do with entities.

bors r=Xaeroxe


Static Framework Generator

There's a bug for framework generator that is it will always generate chi http server. It because of hardcoded argument passed on here. It should take from the Main() function so it can generate dynamically based on user input.

implemented at #32, merged at #33


Set up docs container

What this PR does

Set up configuration to build a docs container that starts up mkdocs
Copy latest API specification documentation from eligibility-server/docs@8223615 into a docs folder

Thanks for the feedback @thekaveman . I pushed new changes - let me know if anything looks off.


Camorph Meshroom to InstantNGP doesn't provide desired outcome

Dear
I tried to use your Camorph script but did not arrive at the expected result. I generated camera postions from 17 pictures in Meshroom via the StructureFromMotion node. I used the cameras.sfm file to then run your script to generate the transforms.json file to use in InstantNGP and arrived at following results.
As example 1 of 17 source pictures:

Results:

The dense point cloud I generated from Meshroom some nodes down the line however seems perfectly fine so I do not really doubt the correctness of the generated camerapositions in Meshroom.

Then I tested to generate the transforms.json file anew via the instantngp-batch procedure described over here:
https://github.com/jonstephens85/instantngp-batch
The procedure first runs colmap to detect camera postions anew and then uses the script colmap2nerf.py to generate the transfroms.json file. This procedure yields significantly better results. See below.

However, I have needed to calculate the camerapositions twice now, once in Meshroom and once via Colmap. I would prefer to only do this once and see promise for your script if it would yield the same results as the Colmap procedure.
I am under the impression that there must be a matrix tranformation bug in the script as I can see from the camera symbols and axis colors on the first Result image that they appear to be flipped along their view direction (blue Z axis). Also I see this because I took the pictures in 2 rows of distance to object and I a know from the pictures that on the nearest row I took 2 pictures each time while on the farthest row I only took 1 picture on each position. From your generated camera symbols I can see that the row order is flipped, which means that the cameras have flipped their viewing direction.
Looking forward to any answer you can give. Please let me know if you need any of my testfiles. I am more than happy to provide those.
For your convenience I have already included both transforms.json files generated by Colmap and by Meshroom and your script:
transforms_FromColmap.json.txt
transforms_FromCamorph.json.txt
Kind regards
Paulus

Hey Paulus,
Could you provide me the meshroom files (e.g. the .sfm file) ? Then I can look into the issue!

Can you please try the new version 0.2.5? Should be fixed now.

Hi Paulus,
First of all, thanks for the detailed feedback -- very much appreciated! ^^
I tried to reproduce the issue with the SFM file parameters on our end and figured out that so far, we experimented with an older version of Meshroom (2021.1.0) which in fact yields pxFocalLength in the SFM file. After downloading the latest Meshroom (2023.2.0), I observed that this parameter was replaced with focalLength (in mm), so we adapted our code accordingly to also support the latest Meshroom version.
Please try again with Camorph v0.3.0 to see if your problem got solved. ;)
Important note though: I noticed that in particular in your provided SFM file, the parameter focalLength was capitalized for some reason (FocalLength) -- which wasn't the case for me when using Meshroom 2023.2.0! After comparing with the major bulk of the parameters given in the SFM file, we decided to support the camel case with initial lower case -> focalLength.
I hope with the latest update of Camorph, your problems have been fixed. If not, please do let us know.
Best,
Michel

Closed due to inactivity


Measurement breakdown of IPFS TTFB performance based on content resolution path

Specifically trying to understand the differences in TTFB performance between ipfs.io gateways and go-ipfs nodes across the following resolution paths:

DHT (that hits a Hydra node) - may be related to https://github.com/libp2p/hydra-booster/issues/93
DHT (that does not hit a Hydra node)
Peered cluster (i.e. Pinata)

These performance metrics can help inform where bottlenecks are happening, and how to think through setting a reasonable SLA for services that build on top of IPFS.
cc @yiannisbot @guseggert

Thanks @yiannisbot, these graphs are great. If I'm understanding them correctly, it looks like the PL gateway has a p80 TTFB of < 75ms, where the fastest go-ipfs TTFB (yellow) has a p80 TTFB of ~1.2s (500ms DHT walk + 700ms content fetch). Is there a way to see the breakdown of response latency for the PL gateway based on whether it hits the cache, node store, or looks up a peer / DHT?
Your suggestion to retrospectively filter out the results and compare the analysis for requests that do and do not hit a Hydra peer, makes sense to answer (1) and (2).
For peering (3), an experiment with known CIDs could help show the difference in TTFB for go-ipfs nodes and the ipfs.io gateway. The trick may be having to control for ipfs nodes' proximity to the content. It may also be interesting to see if there's a significant difference between different IPFS clusters, as in compare response times between a Pinata CID to a web3.storage CID.

A quick update on the results of DHT lookup times with and without considering Hydra peers.
In the below graphs, we conducted two parallel DHT experiments with six nodes each. Those nodes ran in all different AWS regions - similar to the experimental setup in our SigComm paper. The first set of six nodes was operated without any special modifications. The second set of six nodes was fed a list of Hydra Head PeerIDs that were excluded from any DHT operation. E.g., if another DHT server node serves us a Hydra head as a closer peer to the desired CID we don't take that response into account. Different to the experiment in the SigComm paper we pumped the kubo version to 0.16.0.

References

DHT Lookup Performance Experiment Code
Hydra Booster Study
DHT Lookup Graphs
Comparison charts


Message Edition randomly bug

Hello,
I have a phpBB 3.0.13, with the theme COMBOOT. When I create a topic, sometimes it works very well. But sometimes it bug at the message editing. the forum is installed http://dogecoin.fr/forums this address. The bug seems to be random. In the "Forum Commerce", the first subject bug, the second subject "test" do not bug him .. In the forum "A propos du site" the subject bug too.

Would be nice, if you can send me the details here:
http://zoker.me/go/kontakt
Thank you :)

When I try to access the url it gives me this message: "Enter your password to unlock this URL".
I can't access the url.

Sorry, fixed it :)

Ty :)

Should be fixed now! Please download the latest version :)

Thank you so much :)
2015-06-27 20:44 GMT+02:00 Florian Gareis<EMAIL_ADDRESS>
Should be fixed now! Please download the latest version :)
—
Reply to this email directly or view it on GitHub
https://github.com/TheZoker/ComBoot/issues/25#issuecomment-116115503.


Add failsafe cache in local storage or similar

I just started to build a bunch of levels. I wrote the most beatiful story for it until I wanted to Backspace-Delete one falsly plut letter in one dialog. I missed to focus the textarea. The Backspace made the browser go history back. All work lost.
To solve this issue: I'd suggest to add a dialog about unsaved work whenever leaving the page for whatever reason and there is unfinished work. I can add it unless that idea has not been discussed yet.

Aw dang, huge bummer that you lost some of you work/progress @ohcibi. I think this would be a great edition! Want to add it as long as some portion of a new level has been created? I think that'll be a nice touc

@pcottle sounds about right. I just noticed that my issue title is another task than my issue text. I'll put - [ ] in it, but to clarify the confusion please tell me which one you were thinking about 😆.
While trying to add things to avoid further confusion I confused myself again by introducing a third idea - which is autosave - into this mess in my mind 🤣. Lets design the feature properly before implementing 😄.

Ah I was suggesting warning before leaving the page. that'll be easier than educating people about an emergency backup cache (and then how to access that cache)


azcopy copy command to allow passing --account-name and --account-key options for source storage account.

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
AzCopy 10.16.0
Which platform are you using? (ex: Windows, Mac, Linux)
Debian GNU/Linux 11 (bullseye)
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy copy 'https://s2mscompanybucket.blob.core.windows.net/s2mscompanybucket' 'https://s2mspersonalbucket.blob.core.windows.net/s2mspersonalbucket?[SAS TOKEN]' --recursive --account-name <source storage account name> --account-key <source storage account key>

What problem was encountered?
Passing the account-name and the account-key for source destination is not supported. However, with az storage blob copy the equivalent parameters are available.
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
No, I don't want to produce a SAS token for the source container.

Hi @mgiannakopoulos, thanks for reaching out and providing this feedback.
Unfortunately, this was a conscious decision by our security team, as account keys are way too powerful and they give super user permission to the entire account. SAS are safer since they are scoped in terms of permission and short-lived.
If you don't like SAS, please consider using AAD. More info here: https://docs.microsoft.com/en-us/azure/storage/blobs/authorize-access-azure-active-directory


Installing instructions

Ok, so I’ve followed the keiser2zwift install instructions altering anything that needed changing from k2z to your m3iPiZeroW - so far can’t get it working - any chance of some written instructions please?
is there something I didn’t need to do ? Or additional missing steps??
thanks in advance

@Lloydster2000
I have been dragging my feet writing instructions, but will try to do so in the next couple of days (I want use a fresh SD card and start from scratch so I capture everything). So hopefully a tested install procedure available soon!
In the mean time, when you say that you can't get it working -- were you able to trouble-shoot at all?  For instance, were you able to try to start the software and get any output at the linux console?  Or was the Bluetooth service running on the Pi but just not seen by Zwift?  It's hard for me to tell what might not be working without more info...

Thanks for the quick reply,
Well, wasn’t sure how closely I was supposed to follow the k2z instructions seeing as there’s no second bl dongle required for starters (just skipped ‘sudo hciconfig hci1 up’ otherwise followed everything else on keiser2zwift
it just came with an error when I ran - sudo npm run,
Copied m3iPixxx.service to /etc/systemd/system and rebooted and it came up with another error - (think I missed a step ‘running without root or sudo’ actually looking back through the instructions)- will try again this evening and add what error if any, is mentioned on the screen as I just gave up on it. (It was getting warm by lunch time so I gave up)

Heres what Ive tried -
sudo systemctl stop bluetooth
sudo systemctl disable bluetooth
sudo mv /usr/lib/bluetoothd/bluetoothd bluetoothd.bak
Install the development requirements
sudo apt-get install bluetooth bluez libbluetooth-dev libudev-dev
Install NVVM to manager Node.js versions (Make sure to close and reopen your shell after this step)
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.35.3/install.sh | bash
Install the latest binary build of Node.js to support Pis (anything newer and your need to spend a lot time with building it)
nvm install 10.22.0
Download this repo or clone it locally. I put it in a directory off the home folder for pi called /home/pi/code/M3iPiZeroW
Go to that directory and run
npm install
Make sure both bluetooth adapters are connected and issues the following two commands
sudo hciconfig hci0 up
Verify that you see two devices when you run:
hcitool dev
Start this project manually as root. You should see log messages that indicate its looking for an M3i.
sudo npm start <------THIS DID NOT WORK
Wake up your bike and do a few revolutions of the pedals. You should see log messages indicating that its connected to that bike
Launch Zwift and connect to the Power meter whose name starts with KeiserM3-. Then connect to the Cadence sensor. Enjoy
Running without root/sudo
Run the following command:
sudo setcap cap_net_raw+eip $(eval readlink -f which node)
This grants the node binary cap_net_raw privileges, so it can start/stop BLE advertising.
Note: The above command requires setcap to be installed, it can be installed using the following:
apt: sudo apt-get install libcap2-bin
Setting up to always run
You can setup the program to run at Pi boot time by doing the following steps:
Edit the M3iPiZeroW.service file in the root of this project to change the path if you didnt take my suggestion.
Copy the service definition into the correct location
sudo cp M3iPiZeroW.service /etc/systemd/system
Enable the service
sudo systemctl enable M3iPiZeroW
Start the service (Make sure you aren't still manually running the project using npm)
sudo systemctl start M3iPiZeroW
Verify it started
systemctl status M3iPiZeroW
Reboot the system and verify that the service started automatically using step 5

@Lloydster2000
This issure looks familiar.... please take a look at this response to the closed issue "The console is stuck at npm start" in Keiser2Zwift
(also note that there must be "back ticks" surrounding which node in the sudo setcap command which was pointed out in a later response.)
If you can get the "npm start" command (without the sudo prefix) to run from the linux command line, then you should be ok to move on to the next steps to get the service to start automatically.  But even without setting up things to start automatically, once you have the app running with npm start you should be able to pedal the Keiser and see power/cadence in Zwift.
Hope this helps!

found npm and update file, backticks are there it’s just the way it’s formatted in here, still no luck

I’ll just wait till you get round to writing out some instructions.

@Lloydster2000
ok -- I created detailed instructions at the end of the readme file.  I started from scratch (i.e. blank miniSD card) and everything worked.  Based on what I am seeing in your screen shot, I am guessing that your system in not finding "node".   In step 9 of my instructions I copy the node files to a directory that I know is in path:
sudo cp -r /home/pi/.nvm/versions/node/v10.22.0/* /usr/local/

You might give that a try to see if it fixes your issue,  Otherwise the best advice I have is to start with a blank miniSD card and follow the installation instructions.
Hope this helps!

Fab- thanks , will have a clean go tomorrow morning.

@djwasser   Working! thanks.
Apart from removing the ~4x // from hr in ble/keiser bike- is there anything else I need to do for it to work with my polar h10??

@Lloydster2000
Great!
Actually there are 5 "//" to remove to enable HR service.  I went ahead and removed them so if you update your copy of the repo you should get the updated file.   For reference, here are the five lines (with line #s) to uncomment/comment to enable/disable HR service:
5    //const HeartRateService = require('./heart-rate-service');
20   //this.hrs = new HeartRateService();
49   //self.hrs,
93   //this.hrs.notify(event);
118  //this.hrs.uuid,

Since this all worked for you, I will go ahead and close this issue!

All working (incl HR)👍🏻👍🏻
Only app that hasn’t worked so far is RGT ftms shows up but won’t connect

@Lloydster2000
ok - great that HR works.  I have not tried RGT cycling.  I know that ftms typically also has feature where software can adjust the resistance of the trainer (which obviously is not available for Keiser!) so maybe that is why it won't connect?  When I get a chance I will investigate what is involved in creating a "power profile"...

@Lloydster2000
ok - great that HR works. I have not tried RGT cycling. I know that ftms typically also has feature where software can adjust the resistance of the trainer (which obviously is not available for Keiser!) so maybe that is why it won't connect? When I get a chance I will investigate what is involved in creating a "power profile"...

Rouvy connected as ftms and worked - but it did show an error at the top stating no gear/resistance.
Fabulous work - thank you v much.

@Lloydster2000
Looking through my notes, it does not look like I have connected any software to just the cyclng power service.... so it may be that I did not implement it correctly :-(
I will play around with it when I have some time to see if I can figure out what is going on... but it may not be anytime soon!
But glad you are happy with what is working so far  :-)


Fix import_gk_values

Why:
The old functionality of import_gk_values was linked to the internal
setup of calibration systems. It did not handle the ffxml residues,
such as ligands. I've now added bookkeeping of residue types, so that
gk values for non-standard ligands can also be provided.
This change addresses the need by:

adding a residue type specifier to add_titratable_group
updating import_gk_values

Codecov Report

Merging #85 into master will decrease coverage by 0.17%.
The diff coverage is 29.16%.

@@            Coverage Diff            @@
##           master     #85      +/-   ##
=========================================
- Coverage   63.88%   63.7%   -0.18%
=========================================
  Files          19      19
  Lines        2733    2747      +14
=========================================
+ Hits         1746    1750       +4
- Misses        987     997      +10

Impacted Files
Coverage Δ

protons/driver.py
85.57% <29.16%> (-1.3%)
:arrow_down:

Continue to review full report at Codecov.

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

Tests pass. Merging.


Adding some trees problems

Adding some trees problems in cpp like flatten tree to linked list, kth ancestor etc.
Please assign this to me under HacktoberFest 2023 so I can add it to the repo.

I wish to contribute. Kindly assign this to me.

Hi @Saikat2407,
Please assign the issue to me so I can add some more significant problems on trees.

Hi @Saikat2407, Please assign the issue to me so I can add some more significant problems on trees.

assigned to you let me know when you are done with the changes!

@Saikat2407 kindly asssign this to me also under Hacktoberfest 2023.