Can I run pcfdev-pas.v.2.0.22 on Windows 7?

I have downloaded pcfdev-pas.v.<IP_ADDRESS>-build.0.101.0.iso and the cf installer and followed the instructions to install the cfdev plugin. Following the instructions for Windows 7 I have also installed VirtualBox.
When I try the command "cf dev start -f ...iso" it fails at "checking whether hyperv is enabled". Of course HyperV is not available on Windows 7.
Returning to the instructions for Windows 7 I see they are really for the older releases of PCF-Dev such as pcfdev-v0.30.0+PCF1.11.0. Is it possible that pcfdev-pas.v.<IP_ADDRESS> rquires Windows 10 and cannot be run on Windows 7?

@Kairys - Correct the latest version of PCF Dev only supports Windows 10.  We are interested in hearing from users who are using earlier version of 10.
Would you mind sharing your PCF Dev use case and what type of system you are running?


improvements to log collector kubeadm collection

This PR addresses issue #222

Removed collection of cluster-info dump to avoid gathering of sensible data

reduced collection of logs in /var/log/pods/ to only system clusters namespaces

fixed user variable when ran with sudo

a custom kubeconfig file location can now be specified with the -k switch, however in case of failure the script fallback to ~/.kube/config. A new function verify-kubeconfig has been added

Example:
sudo ./rancher2_logs_collector.sh -r kubeadm -k /home/tux/kubeconfig

Hi Any update on this PR? I have accepted the modification and clarified the queries raised.Thanks

@gnuninu To specify a kubeconfig.yml file with the new changes, the proposed approach requires passing -k. However, instead of introducing this new flag, we can leverage the KUBECONFIG environment variable directly from the shell where the script is invoked. This approach is more intuitive and eliminates the need for an additional flag.

IMHO I actually think -k it's an useful way to override KUBECONFIG in flexible way.
I see environments where users retrieve a kubeconfig file either from rancher or from other unmanaged clusters and they want to simply be able to quickly run log collector without necessarily configure nor defining a local KUBECONFIG variable at all. they simply want to run it from their computer with no other interaction with the clusters whatsoever. a -k option would work nicely in such cases.

It's important to note that setting a default value for KUBECONFIG can have unintended side effects, especially since this script runs in various cluster types like RKE, RKE2, and k3s.

I don't see how this could break other parts of the code though.
The function verify-kubeconifg is executed only when a kubeadm cluster collection is selected, hosted in turn inside the related kubeadm-k8s function, leaving the other parts of the code untouched.

Discussed offline with @leodotcloud, will close this, however please open a fresh PR with only the proposed changes if needed.


Update ch5.md

“request all the data changes that occurred during the time when the follower was disconnected.”
请求的是在从库断开其实发生的所有数据变更

Thanks for your contribution!


[Test suite]: log snapshot resources from all namespaces

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

NONE

/retest

/retest
/cc @mhenriks @awels

/lgtm
/approve

/retest

/test pull-kubevirt-e2e-k8s-1.27-sig-compute

/cherrypick release-1.1


[chooseTerminal] issue

Describe the bug
My VSCode had been buggy after I installed Radian, every time I open VSCode and use run resource, it will prompt "Error identifying terminal".  This is what it says:
[chooseTerminal] A. There are 2 terminals: Terminal 0: powershell Terminal 1: radian B. There are 2 terminals: Terminal 0: powershell Terminal 1: radian C. There are 2 terminals: Terminal 0: powershell Terminal 1: radian

Environment (please complete the following information):

OS: Windows
VSCode Version: 1.62.3
R Version: 4.1.0
vscode-R version: 2.3.4

Additional context
Add any other context about the problem here.

Duplicate of #899


Regression: package was not installed in a way that PackageLoader understands.

Expected Behavior
PackageLoader should continue to load from a namespace package even under pytest.
Actual Behavior
Starting with jinja2 2.11, the tests for pmxbot have started failing. They fail when pytest attempts to rewrite the modules for running under test and in doing so, cause the jinja2.loaders.PackageLoader not to recognize the 'pmxbot.web' package. Note that pmxbot is a setuptools namespace package.
Full Traceback
pmxbot/web/__main__.py:1: in <module>
    import pmxbot.web.viewer
<frozen importlib._bootstrap>:991: in _find_and_load
    ???
<frozen importlib._bootstrap>:975: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:671: in _load_unlocked
    ???
.tox/python/lib/python3.8/site-packages/_pytest/assertion/rewrite.py:143: in exec_module
    exec(co, module.__dict__)
pmxbot/web/viewer.py:24: in <module>
    jenv = jinja2.Environment(loader=jinja2.loaders.PackageLoader('pmxbot.web'))
.tox/python/lib/python3.8/site-packages/jinja2/loaders.py:288: in __init__
    raise ValueError(
E   ValueError: The 'pmxbot.web' package was not installed in a way that PackageLoader understands.

The error is not triggered when simply importing the package naturally; it only seems to happen during the pytest rewrite.
Your Environment

Python version: 3.8.1
Jinja version: 2.11.x

Downgrading to Jinja 2.10.3 works around the failure. I suspect #970 is implicated.

We've switched to using packaging.resources. I think you're a maintainer for that, any insight into how to support this style of namespace?

I don't see that you're using importlib.resources, at least not in the #1082. Indeed, even in master, importlib.resources (or its backport) isn't imported.
I do agree that using importlib.resources should make the implementation cleaner and it's a laudable goal to drop pkg_resources (it's also one of my goals). However, I believe lack of support for subdirectory traversal is a blocker to adopt importlib.resources.
I'm not sure what it is about the "pytest assert rewrite" functionality that's triggering the error, and I don't know how difficult it will be to debug either (given that it happens during test runner setup and not during test execution).
I don't know when I'll get the time to troubleshoot, but I'll try to help when I have the time.

You're right, I was remembering what we started with, but we ended up only using pkgutils and doing a few things manually since importlib didn't handle it.
I recall a pytest issue we opened that had to do with monkeypatching modules, I'll see if I can find it.

See pytest-dev/pytest#5098. Their monkeypatch.syspath_prepend() has to call pkg_resources.fixup_namespace_packages() and importlib.invalidate_caches() in order to get dynamic changes to be recognized during tests. Perhaps there's something similar going on in your test setup?

I think this is a specific case of #1168, do you think we can close this in favor of the discussion there?

Agreed, that ticket gets closer to the root of the issue, so I'll close this as a manifestation of that issue.


Bump version to 1.0

I think version 0.1.0 is unfair. This project has been used in production by a few users, and it's also classified as "Production/stable" quality (in setup.py), which is is not consistent with the 0.1.0 version.
Shall we bump it to 1.0 so? /cc @Curita @kmike

I agree, I copied that convention from other scrapy projects (and scrapy itself) but this project should be already at 1.0.

Cool, can you make the change @Curita and push the new version for pypi?

Done it, and the new version is uploaded to pypi. I had to force a commit because of a mistake I made before with .travis.yml, sorry for that.


Deprecations and removals in Chrome 66

Target Live Date: 2018-03-26

[x] This has been reviewed and approved by feature owners
[x] I have run gulp test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:+1:

:+1:

I have it on my MDN list now, but that's not a deprecation or removal.

:+1:

:+1:

:tada: This has been pushed live to https://developers.google.com/web/


Add more common serializers

Oak requests its users to provide serializers for their objects representing keys and values.
Oak has internal common package including sub-packages for common serializers for common objects: IntBuffer, Integer and String. It would be good to have more of such. What kind of objects? It is up to implementer decision.
@liran-funaro can you please add more details to this issue?
For more explanations feel free to add questions in this issue.

how would a FloatSerializer be different from an IntSerializer?
wouldn't this suffice:
public class OakFloatSerializer implements OakSerializer<Float> {

    private final int size;

    public OakFloatSerializer() {
        this(Float.BYTES);
    }

    public OakFloatSerializer(int size) {
        this.size = size;
    }

    @Override
    public void serialize(Floatvalue, OakScopedWriteBuffer targetBuffer) {
        targetBuffer.putFloat(0, value);
    }

Yes. That is exactly what we expect.
It would also help if a simple test would be added that validates that this serializer works correctly with OakMap.


Make an AMD-Friendly build

This plugin is currently difficult to load with require.js. I could wrap it manually but that would break package management with volo. It would be nice to see it wrapped in a UMD https://github.com/umdjs/umd/blob/master/returnExports.js

Problem still occurs on master bunch (version 1.4.4
This is relay simple fix just add a UMD factory to your namespace
(function (factory) {
    if (typeof define === 'function' && define.amd) {
        define(['jquery', window], factory);
    } else if (typeof exports === 'object') {
        module.exports = factory(require('jquery'), window);
    } else {
        factory(jQuery || Zepto, window);
    }
}(function($, window){
 ..... your code goes here ....
}));

any news?


Write font files as binary instead of text

This caused the font files to be corrupt and rejected by the browser's sanitising engine.
Fonts have been updated as well.

I'm not sure why github's diff is showing "Empty file" for each font, but I can assure you they are not empty: https://github.com/matrix-org/matrix-doc/tree/anoa/write_fonts_as_binary/static/fonts


temp file should be removed when sync interrupted.

What happened:
root@bench-01:~/juicefs# ./juicefs sync /tmp/jfs/bigfile /tmp/jfs/sync
2024/04/29 18:45:43.743250 juicefs[3859900] <INFO>: Prometheus metrics listening on <IP_ADDRESS>:36403 [mount.go:135]
2024/04/29 18:45:43.745302 juicefs[3859900] <INFO>: Syncing from file:///tmp/jfs/bigfile to file:///tmp/jfs/sync [sync.go:1300]
Scanned objects: 0/1 [--------------------------------------------------------------]  0.0/s ETA: 0s
Skipped objects: 0                            0.0/s                                       ⠦
  Skipped bytes: 0.0 b   (0 Bytes)            0.0 b/s                                     ⠦
Pending objects: 0                            0.0/s                                       ⠦
 Copied objects: 0                            0.0/s                                       ⠦
   Copied bytes: 1.9 GiB<PHONE_NUMBER> Bytes)   337.2 MiB/s                                 ⠦
 Failed objects: 0                            0.0/s                                       ⠦
^C
root@bench-01:~/juicefs# ls /tmp/jfs/ -a
.  ..  abc  .accesslog  big_dir  bigdir2  bigfile  .config  fsrand  no_dismount.txt  .stats  .sync.tmp5862692998825721621  .trash  vdb.1_1.dir  vdb.1_2.dir
root@bench-01:~/juicefs# ls -lh /tmp/jfs/.sync.tmp5862692998825721621
-rw-r--r-- 1 root root 1.9G Apr 29 18:45 /tmp/jfs/.sync.tmp5862692998825721621

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?
Environment:

JuiceFS version (use juicefs --version) or Hadoop Java SDK version:
Cloud provider or hardware configuration running JuiceFS:
OS (e.g cat /etc/os-release):
Kernel (e.g. uname -a):
Object storage (cloud provider and region, or self maintained):
Metadata engine info (version, cloud provider managed or self maintained):
Network connectivity (JuiceFS to metadata engine, JuiceFS to object storage):
Others:

should done https://github.com/juicedata/juicefs/issues/4642 first


Rebrand to NEO - port from @CJSS

This is a PR porting @CJSS and fixing the conflicts, should go first before others.

Tks, if you can find anything more please add the comments to the code itself, is easier to track..

@canesin @CJSS This PR is too big. When I go to the "Files changed" section, my browser is just stuck. And I can't add the comments to the code itself.
So, I will port this PR to a branch in this repo so that I can modify it directly.
See #72


Create tag from main window and context menu

What feature or improvement do you think would benefit Files?
Hi,
Do you plane to add creating tag from main window and context menu ?
It would be more useful
Thanks
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2428
Comments
No response

Thank you for your feedback, I think this can be done by adding an option to the "edit tags" menu to "Create tag". This option should deep link into the Tags setting page.

@Beuzbeuz78 It seems like we're already tracking this in #12146. You can get notifications if you click the subscribe button on that issue.


Added pyright checking

https://github.com/semiotic-ai/autoagora/issues/80

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 84.11%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
307

Relevant Lines:
365

💛  - Coveralls


chore: upgrade webapp only

Restrict upgrade workflow to only update construct hub webapp.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@eladb This needs to into dev as well since the workflows run from the default branch


update account balance only after line creation is successful

Had instances where during Line creation, AccountWouldBeSentNegative is thrown. This rolled back the Line record but the balance update for AccountBalance wasn't getting rolled back. This made positive-only accounts go into negative balances without creating corresponding line item records.
Ideally we want to make sure all Line items are created successfully before we update the account balance. That way, when any error gets thrown while creating a Line record, we wouldn't worry about any updates to the Account Balance.

Isn't this all happening in a transaction anyway? I don't understand how this would possibly be happening ... could you perhaps write a failing test that passes after your changes?


When indenting code, Godot crashes because of expected indentation

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 2.1.3 stable official, Antergos Linux x86_64, 4.12.8-2-ARCH
Issue description:
I added an inbuilt script to a viewport node and copied over the size and stretch override code example from Godot's docs inside the _ready()
It would be copied over unindented as so:
extends Viewport

func _ready():
viewport.set_size_override(w,h) #custom size for 2D
viewport.set_size_override_stretch(true/false) #enable stretch for custom size

And then, if I indented both lines of code as such:
extends Viewport

func _ready():
    viewport.set_size_override(w,h) #custom size for 2D
    viewport.set_size_override_stretch(true/false) #enable stretch for custom size

Godot will crash. This does not just apply to the viewport node, but to any node, it seems, as I tried it out on a panel node as well and it led to a crash.
P.S: For some reason, Godot's editor didn't allow code to be copied from it :/
Steps to reproduce:
Create a node. Attach a script. Copy/paste the aforementioned code from the Godot docs into the _ready() function and, sometimes without saving, Godot will crash.
THIS DOES NOT HAPPEN IF THE CODE IS WRITTEN BY HAND.

Well, when I look more closely... it happens because after I fix the indentation of the first line and the error parser gets to the second line where the supplied argument is (true/false), which isn't possible and that's what probably crashes the engine.

Please fill in:

Godot version

THIS DOES NOT HAPPEN IF THE CODE IS WRITTEN BY HAND.

WHY NOT.

The crash happens here
https://github.com/godotengine/godot/blob/bd282ff43f23fe845f29a3e25c8efc01bd65ffb0/core/variant_op.cpp#L673
because true/false is a division by zero.

False resolves to 0, doesn't it? true/false is then 1/0. Doesn't this mean that this isn't actually a bug?

It's a big. The editor should have a check for this instead of crashing. Same for games in debug mode, for proper error reporting.
The crash is ok on release builds.

Closing this issue, as #11291 is more explicit about the problem :)


Release 1.1.3

Codecov Report

Merging #125 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   88.65%   88.65%
=======================================
  Files          49       49
  Lines        2212     2212
=======================================
  Hits         1961     1961
  Misses        251      251

Impacted Files
Coverage Δ

atcodertools/release_management/version.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Does not show Subscription info

Does this occur consistently? Yes
Repro steps:

Open Extension
Under Resources -> Select Subscription -> No Subscription Available to select

Version: 0.8.4
OS: win32
OS Release: 10.0.22635
Product: Visual Studio Code
Product Version: 1.87.0
Language: en

Could you share a screenshot? Also, try using the "Azure: Sign in to Directory..." command.

@cpich3g   @alexweininger I have the exact same issue.
clicking on sign into directory does nothing. Yes, I tried signing out and signing back in again.
I have the exact same issue.
Version 0.8.4  has a big problem signing into Azure.
I revered back to version 0.7.3 and the resources appeared immediately.
This should be a P0 bug and correct ASAP.  Thank you!

Hi @longtimedeveloper could you include more detailed repro steps? Please include screenshots, your account type, which extensions you are using, and any other special info about your account if possible. Thank you!

@motm32 no steps are required.  Install version 0.8.4 and try to sign into Azure. It does not work. You have two people telling you the exact same thing.
I am using Windows 10. Can't upgrade due to processor requirements.  Windows 11 does not like my Core i7 processor.
What is an account type? Never heard of this before.
Chrome is my default browser and opens when I sign into Azure using the 0.7.3 extension.
I am a volunteer in Ukraine. I apologize, I don't have time to reconfigure my system to get you a screenshot of a broken extension.
These are my vscode extensions:
alefragnani.Bookmarks
alexisvt.flutter-snippets
aljazsim.tsco
austenc.tailwind-docs
blakeh.jump-to-test
bradlc.vscode-tailwindcss
Dart-Code.dart-code
Dart-Code.flutter
dbaeumer.vscode-eslint
dsznajder.es7-react-js-snippets
EditorConfig.EditorConfig
esbenp.prettier-vscode
forbeslindesay.forbeslindesay-taskrunner
formulahendry.auto-rename-tag
formulahendry.code-runner
george-alisson.html-preview-vscode
gmlewis-vscode.flutter-stylizer
Gruntfuggly.todo-tree
kreativ-software.csharpextensions
markis.code-coverage
mechatroner.rainbow-csv
mikeburgh.xml-format
miquelddg.dart-barrel-file-generator
ms-azuretools.vscode-azurefunctions
ms-azuretools.vscode-azureresourcegroups
ms-dotnettools.csdevkit
ms-dotnettools.csharp
ms-dotnettools.vscode-dotnet-runtime
ms-vscode.azure-account
plibither8.remove-comments
ritwickdey.LiveServer
shd101wyy.markdown-preview-enhanced
tamasfe.even-better-toml
Tyriar.sort-lines
usernamehw.errorlens
valentjn.vscode-ltex
vincaslt.highlight-matching-tag
vscode-icons-team.vscode-icons
yatki.vscode-surround
I do hope this helps to fix this.  Many people have been reporting problems signing into Azure from vscode for years. Some reports are different from this one, but never the less, problems signing in. I really hope you can finally get this sorted out.
Have a nice and productive day!

Same problem in here.
Had a lot of trouble getting the subscriptions of the first two tenants to be visible.
Now I'm adding another tenant and I don't see any subscriptions and all the other subscriptions are gone.
But if I click on the user on the footer of the screen from Azure sign in Azure account login, it shows that I have a subscription.
Unfortunately selecting the subscription doesn't change the visible subscriptions on Azure resources.

@Dangerous-Mind did you try installing version 0.7.3 ?  I hope you can get your situation sorted out.  Thank you for adding your feedback and information.  I pray Microsoft can sort this out.

@longtimedeveloper
I gave permission on this new tenant Resource Group to a user that was already working on another tenant and the resource group appeared.
Removed the permission and the resource group disappeared.
It looks like he's using the wrong user on this new tenant.

I removed the user permission of the working tenant on VS Code and the Resource Group appeared with the second user.
Then added back the first user and it seams to be working fine for both users.
But sometimes it still messes things up not showing the correct RG.
It's definitely using the wrong user to get the Resources on that second tenant.


Forbid using the classes function inside style blocks

Currently, both of the following lines are valid and work the same:
Div({ classes("abc", "def") })
Div({ style { classes("abc", "def") } })

However, it's better to avoid having multiple ways of doing something to make code more consistent, so I suggest forbidding specifying classes inside style. Probably adding @DslMarker will help.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


apache airflow comparison

how is it different compared to apache airflow still in incubator process. I wonder what are key differences I see a lot of similarities.

I have not had much time to look into cadence, but after spending some time browsing through the code I can see what you guys trying to achieve it not exactly a workflow engine in a traditional sense.

Can you clarify what you mean by "workflow engine in a traditional sense"?  Most of the "traditional" workflow engines expose some kind of DSL to represent long running applications.  In that sense both Cadence and AirFlow are non-traditional engines as they allow users to express their application as regular code.
Having said that, it should be pretty straightforward to expose any DSL on top of Cadence.  Here is an example which how to write a workflow definition which parses a DSL and executes steps in the provided definition:
https://github.com/samarabbas/cadence-samples/tree/master/cmd/samples/dsl

Resolving this task as I have answered the original question.
Feel free to reactivate if you have more questions.

Hi @samarabbas , Does this comparison changes or has any more points as of March 2020?

Just tacking on some HN discussions for people who might find this in future:

https://news.ycombinator.com/item?id=19732447
https://news.ycombinator.com/item?id=23349507
https://news.ycombinator.com/item?id=23844177

(i work at temporal and this github issue came up)


Add Extended WADL support

From @LexeY4eg on October 1, 2013 15:22
Please, add an extended WADL support, or advise another way how to generate documentation for REST services (javadoc support would be nice).
see: https://wikis.oracle.com/display/Jersey/HowToConfigureExtendedWADL
Thanks
Copied from original issue: krasserm/grails-jaxrs#31

From @michaelrice on October 1, 2013 17:34
I would love to see that added as well.

From @noamt on October 2, 2013 7:35
Thanks for the suggestion; Not quite sure when we'll be able to get to it, but feel free to open a pull request for this.


No Express+Vue (ssr) boilerplate

I want to use ream instead of nuxt because of its flexibility but the problem is I just don't know how to do that... SSR is a thing of server i.e. Expressjs & all docs are related to Vue. So how I am supposed to use that? I can make a simple express server and render some template based on handlebars engine but I don't know how to integrate REAM with experss and without that or easy DX, not many devs are going to use it no matter how flexible/powerful REAM is.

from the master
https://github.com/ream/ream/blob/master/server.js

@cannap thanks!


TinkerbotsCubieBoxWithRoundSlots slots

TinkerbotsCubieBoxWithRoundSlots.leftSlot and TinkerbotsCubieBoxWithRoundSlots.axisSlot are located at the same position, and TinkerbotsCubieBoxWithRoundSlots.front/backSlot are missing.
We should certainly drop TinkerbotsCubieBoxWithRoundSlots.leftSlot and TinkerbotsCubieBoxWithRoundSlots.rightSlot.

Note that the slots name/position in TinkerbotsCubieBoxWithRoundSlots have at least to be consistent with the slots in TinkerbotsCubieBoxWithCrossSlots (exact same shape).
This means that the slots in TinkerbotsCubieBoxWithCrossSlots have to be updated as well.

I also remarked a very minor issue which could be fixed here: the TinkerbotsCube slots are not sorted consistently (backSlot and frontSlot are inverted).

Left/right slot positions seems to not be consistent in TinkerbotsCubieMaleCube and TinkerbotsCubieFemaleCube.

Left/right slot positions seems to not be consistent in TinkerbotsCubieMaleCube and TinkerbotsCubieFemaleCube.

I checked again and they are correct in Webots.

I also remarked a very minor issue which could be fixed here: the TinkerbotsCube slots are not sorted consistently (backSlot and frontSlot are inverted).

If I understand correctly the issue, i.e. that based on the up, down, left and right faces the position of the back and front faces are inverted, then this is affecting also TinkerbotsDistanceSensor, TinkerbotsLightSensor, and TinkerbotsMotor.
Given that for the TinkerbotsDistanceSensor and TinkerbotsLightSensor it makes sense to have the device itself on the front face, then maybe it is better to switch the right and left faces.

Oops, I was speaking on a much simplest issue: the definition of the back/frontSlot are strangely sorted here (e.g. backSlot appears on top of the frontXSlots while frontSlot appears on top of the backXSlots):
https://github.com/omichel/webots/blob/master/projects/robots/tinkerbots/protos/TinkerbotsCube.proto#L198
What you raise here is indeed another issue :-(


how can I show graph if don't use jupyter notebook

model = torchvision.models.vgg16()
hl_graph = hl.build_graph(model, torch.zeros([1, 3, 224, 224]))
hl_graph.save("xxx", format="png")

Hi @aikudexiaohai,
If the information provided above doesn't answer your question or you're still facing issues, feel free to add more details to this thread and re-open this issue.
Thanks!


wrong kernel is selected for IW in web

repro steps

open insiders.code.dev, make sure you're not connected to a remote jupyter server
create an interactive window
conncect to a remote jupyter server

actual result:
The top item of the kernel picker is selected
expected
Python 3 is selected, as is the case if you created another IW at this point

Woudln't this also apply to desktop with remote jupyter?

we don't open the IW without the python extension installed for desktop, but maybe if python wasn't on your machine

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/11967


Client Crash

Hello,
looks like there's a crash when hooking up Simple Tubes between chests:
https://pastebin.com/PzCnLFGF
MC: 1.11.2
Simple Tubes: 0.0.2
Forge: 2414
Thank you :)
Darkosto

That's not caused by SimpleTubes, an error occured at

Caused by: net.minecraftforge.fml.common.asm.ASMTransformerWrapper$TransformerException: Exception in class transformer toughasnails.asm.transformer.CropDecayTransformer@51a8313b from coremod TANLoadingPlugin

One of the mods you have uses a coremod that's conflicting with SimpleTubes, I don't use coremod so it's not down to me to fix
Please contact the author of said coremod.
Thanks, oMilkyy

Thanks.


[tor-browser] Proxy server refusing connections

Describe the bug
When you try to connect to the Tor network using the Tor browser you get the error message

When you try to visit a website after being shown the big red error message your get the error message

This problem appeared between one and two weeks ago. Elserjo on the Discourse forum reported this issue eight days ago; can't find an existing Nixpkgs issue.
Steps To Reproduce
Steps to reproduce the behavior:

Install the Tor browser from Nixpkgs
Start the Tor browser
Click on Connect
Read error message
Try to go to a website

Expected behavior
When clicking on Connect the browser should connect to the Tor network and be able to browse.
Screenshots
Additional context
Troubleshooting
The Tor project has a troubleshooting guide on their website.

The browsers says that it can detect my internet connection.

The Tor logs doesn't seem to indicate any problems (from what I can interpret)

2023-03-05 20:53:12.260 [NOTICE] New control connection opened.
2023-03-05 20:53:12.260 [NOTICE] New control connection opened.
2023-03-05 20:53:12.266 [NOTICE] DisableNetwork is set. Tor will not make or accept non-control network connections.

I live in Sweden and this issue has appeared both when I've connected to the internet using my home connection (Sunet) and when I've connected over a VPN. I.e. no fuckery by an oppressive government.
The timeline match up-ish with when #217407 would have become available in the binary caches. Unsure if the error appeared before or after updating around that time.
After checking the original PR I see that this problem was noticed, but a workaround was applied. Doesn't seem to work for me :-/
The network.proxy.socks value is <IP_ADDRESS> for me. I'm using root  on tmpfs, and nothing under /run is persistently saved between boots.

Notify maintainers

@offline @matejc @thoughtpolice @joachifm @hax404 @KarlJoad
@panicgh and @SuperSandro2000 because you seem to have actually researched the problem.
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.95, NixOS, 22.11 (Raccoon)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

The system is fully up-to-date as of 2023-03-05 22:15 CEST.

When you try to visit a website after being shown the big red error message your get the error message

The red error message indicates that Tor cannot find the tor socket. Everything else is bound to not work unless it goes over the normal internet.
Please see https://github.com/NixOS/nixpkgs/pull/205463#issuecomment-1345749870 and https://github.com/NixOS/nixpkgs/pull/205463#issuecomment-1436064480 for the underlying problem  and please check if the firefox setting points to the right socket file.

When you try to visit a website after being shown the big red error message your get the error message

The red error message indicates that Tor cannot find the tor socket. Everything else is bound to not work unless it goes over the normal internet.
Please see #205463 (comment) and #205463 (comment) for the underlying problem and please check if the firefox setting points to the right socket file.

$ ls  /run/user/1000/
bus  dbus-1  dconf  env-vars  podman  pulse  ssh-agent  systemd  Tor

$ ls  /run/user/1000/Tor/
[empty]

I tested to set network.proxy.socks to  /run/user/1000/Tor/, but that didn't help.

Setting network.proxy.socks to file:///run/user/1000/Tor also doesn't work.

Just to avoid ambiguities,  did you set network.proxy.socks to file:///run/user/1000/Tor/socks.socket (with socks.socket file name part)?
TBB is still working fine for me with the workaround.

nothing under /run is persistently saved between boots.

Likewise for me, /run/user/1000 is a tmpfs.
Have you modified (and forgotten about it) your torrc, e.g. to use a different control port/mechanism?

Just to avoid ambiguities, did you set network.proxy.socks to file:///run/user/1000/Tor/socks.socket (with socks.socket file name part)?

That did it! I missed that you should specify the socks.socket file in the path.
$ echo $XDG_SESSION_TYPE
x11

I'm using AwesomeWM.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/tor-browser-crash/25799/3

The person on [Discourse[(https://discourse.nixos.org/t/tor-browser-crash/25799/4) had the same problem and the same solution :-)

Why did you close this issue @panicgh? You can't seriously think that this temporary workaround is an acceptable permanent solution to this problem? While we still don't have a permanent solution, it should be as easy as possible for users to find information about this workaround. By closing this issue we signal to people that this problem has been solved. And people currently experiencing this problem on a fully up-to-date system will probably not think about looking into closed issues, because the issue is still there.

I am also experiencing this issue. The fix worked for me.

Confirmed. The issue is still present. Default installed value of network.proxy.socks is <IP_ADDRESS>. Changing it to file:///run/user/1000/Tor/socks.socket resolves the problem.

This issue only started happening on my machine after deleting ~/.local/share/tor-browser today.

Still an issue. I'm using TBB with firejail and I have to apply the workaround at every restart. TBB creates the ~/.tor-browser/ folder but not a profile inside (this wouldn't be a problem for me if connection works).
about:config and then make network.proxy.socks = file:///run/user/1000/Tor/socks.socket
tor-browser-bundle-bin 12.0.5
firejail-0.9.72
nixos-unstable

By default, the service runs on port 9050 and the control port on 9051. This is configured in the torrc file:
https://github.com/torproject/tor/blob/main/src/config/torrc.sample.in
Your firewall may be blocking some of these ports bound by default to localhost which normally point to <IP_ADDRESS>

The PR for Tor Browser 13 removes the current workaround again, because it seems the issue was fixed upstream:
https://github.com/NixOS/nixpkgs/pull/256934
I would be grateful if some of you could give this a try and report if it's still working (or working again).


Fix for report_broken_client not working when passphrase is set

Small fix for report_broken_client not reporting when a passphrase is set for MunkiReport on the client machine.

:1:


Code review

Codecov Report

Merging #9 into master will not change coverage.

@@           Coverage Diff           @@
##           master       #9   +/-   ##
=======================================
  Coverage   61.46%   61.46%
=======================================
  Files          28       28
  Lines        1217     1217
=======================================
  Hits          748      748
  Misses        469      469

Impacted Files
Coverage Δ

...fications/management/commands/parse_pending_sms.py
0% <ø> (ø)
:white_check_mark:

universal_notifications/admin.py
100% <ø> (ø)
:white_check_mark:

universal_notifications/backends/websockets.py
74.07% <ø> (ø)
:white_check_mark:

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 b45f08e...e40e906. Read the comment docs.


Neuroglancer fragments cannot be merged

This causes maps such as cortical layers on Bigbrain to fail when fetched without a fragment being provided.

solved as of 2d7a0e49ca793e1d41ea3833f78fc2d94935eb78


Proposed: Allow users to encrypt EC2 Access and Secret keys in their configuration

Some people (notably, me) aren't comfortable with sensitive information lurking in configuration files.  I'd like to make it possible for users to encrypt certain fields in their machine configurations such that they either have to specify a passphrase as a cli option or be prompted for it ever time they run a machine command.

It's definitely something we need to do!

Yes.  This is an important issue for me and I ultimately want to move away from storing the secrets on a per-driver basis entirely, instead having one unified Config store where various configuration options including API keys are kept which has a few "backends" including one for plaintext as well as one for encryption.

+1 for encryption

+1


CFP: read-only filesystem support

Check sysctl values before writes to avoid errors on potentially read-only filesystem
Fixes: #30059
Check sysctl values before writes to avoid errors on potentially read-only filesystem

/test

/ci-ipsec-upgrade)

/ci-ipsec-upgrade

/ci-clustermesh

/ci-eks

/kind release-note/bug


RAD - Generate the boilerplate classes from Entity

Hello, this is simply awesome, with its simplicity, love the videos as well, I highly encourage others to view them.
Can you help explain, how we can create the various hooks, classes if we have entities, for e.g. EventManagement, with all its supporting needs. Will there be a T4 template or some code gen.
Also whats your road map, and can we use this in a commercial closed source for free
thanks

Hi @fasteddys thanks for your questions. Glad to hear you like the videos, we have more coming.

Also whats your road map,

We target two milestones at all times, a minor milestone (currently v0.9.5) and a major milestone (v1.0.0). Minor milestones contain bug fixes and small feature enhancements. Major milestones will be significant enhancements that may require a database migration. Minor releases are released more frequently than major releases.
For v1.0.0 major release: https://github.com/RaythaHQ/raytha/milestone/2 features that are coming include:

Automatic Headless REST API
Support for running in a clustered environment (like Azure Kubernetes Service)
Support for running at a PathBase route (mydomain.com/mysite instead of just mydomain.com)

After that, there's plenty up for discussion as to what the next major release should contain. It will probably be a mix of tackling tech tebt and some of which is listed in Discussions here: https://github.com/RaythaHQ/raytha/discussions. But some low hanging fruit that I am strongly considering in the 3-6 month time frame are:

Tech debt: https://github.com/RaythaHQ/raytha/issues?q=is%3Aissue+is%3Aopen+label%3A"tech+debt"
Code editor for templates: https://github.com/RaythaHQ/raytha/discussions/17
New field type for advanced editing: https://github.com/RaythaHQ/raytha/discussions/16
301 redirects: https://github.com/RaythaHQ/raytha/discussions/15
Add text to dashboard: https://github.com/RaythaHQ/raytha/discussions/47

6+ months road map is hard to say, but open to ideas, so feel free to post in the Discussions section. We will likely at some point shift gears to building a "Try it Out" feature on raytha.com to reduce the barrier to playing around with the platform. This will be a precursor to eventually having a Cloud hosted version.

can we use this in a commercial closed source for free

Yes, Raytha is MIT open source, you can use freely in commercial, closed source products. We offer optional paid support packages, consulting, and hosting services.

Can you help explain, how we can create the various hooks, classes if we have entities, for e.g. EventManagement, with all its supporting needs. Will there be a T4 template or some code gen.

Can you expand further on this question. Do you mean if you wanted to add more custom database entities (not a Raytha Content Type), what the steps would be?
Thanks
**ps going to transfer this to Discussions board.


ci: add aarch64-unknown-linux-musl release artifact

Fixes #5690.
I tested the workflow on my fork:

CI run
test release

The resulting artifact works as expected:
$ file coreutils
coreutils: ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), statically linked, with debug_info, not stripped

$ ./coreutils
coreutils 0.0.23 (multi-call binary)

Usage: coreutils [function [arguments...]]

Currently defined functions:

    [, arch, b2sum, b3sum, base32, base64, basename, basenc, cat, chgrp, chmod, chown, chroot,
    cksum, comm, cp, csplit, cut, date, dd, df, dir, dircolors, dirname, du, echo, env, expand,
    expr, factor, false, fmt, fold, groups, hashsum, head, hostid, hostname, id, install,
    join, kill, link, ln, logname, ls, md5sum, mkdir, mkfifo, mknod, mktemp, more, mv, nice,
    nl, nohup, nproc, numfmt, od, paste, pathchk, pr, printenv, printf, ptx, pwd, readlink,
    realpath, rm, rmdir, seq, sha1sum, sha224sum, sha256sum, sha3-224sum, sha3-256sum, sha3-
    384sum, sha3-512sum, sha384sum, sha3sum, sha512sum, shake128sum, shake256sum, shred, shuf,
    sleep, sort, split, stat, stty, sum, sync, tac, tail, tee, test, timeout, touch, tr, true,
    truncate, tsort, tty, uname, unexpand, uniq, unlink, vdir, wc, whoami, yes

Thanks


User Creation Unknown Error Occured | Bookstack

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
Dear Bookstack Community
Since 1 week we have the problem that we cannot create any new users in our Bookstack website (wiki).
When I try to create the User with the Admin account and all credentials and the correct persmissions and press on create there
a new error message opens with the the message: An Error Occured (An unknown error occured).
I've checked the logs before and after user creation.
The New error message from the error Log that  appears is the following when i try to create an account:
IP Hidden - - [11/Aug/2022:08:27:18 +0200] "POST /settings/users/create HTTP/1.1" 500 15562 "http://wiki.hiddenurl.local/settings/users/create" "Mozilla/5.0 (Windows NT 10.0; W>

Has somebody already had the same error before and if so, how did you find a solution for this problem?
Thanks in Advance for the Help
yours Sincerly
Santis
Exact BookStack Version
v22.04.2
Log Content
Accesslog after user creation.txt
accesslogbeforeusercreation.txt
errorlog.txt
errorlog1.txt
PHP Version
8.1.2
Hosting Environment
Ubuntu 22.04.1 LTS
Release 22.04

Hi @SantisTraining,
None of the log files shared are relevant error log files, but are access and general service log files.
Please view the error log file, as described in our docs to gain details on this particular error.

@ssddanbrown Hey Thx for the quick response.
Iwasn't sure if i sent you the right logs.
I got the Laravel log now you also have to know that im not the one who installed this application so its all kind of new form me and my workpartners. As i see in this log it tells me the mail cannont be sent because smtp.mailserver isnt configured correctly is this true ?
and thx for the help in advance
laravel.log

Hi @SantisTraining,
That is correct, It looks like the system is having trouble sending email through the configured outlook-based SMTP provider.
In your /var/www/bookstack/.env file, what is the current value (If existing) of the MAIL_ENCRYPTION option?
If it helps, while testing & configuring things, you can send test emails via the "Settings > Maintenance" area within BookStack.

Hello @ssddanbrown
The Lines are listed as followed:
MAIL_HOST=smtp-mail.outlook.com
MAIL_PORT=587
<EMAIL_ADDRESS>MAIL_PASSWORD=null
MAIL_ENCRYPTION=STARTTLS
without quotation and closing marks
Greetings
Santis

Hi @SantisTraining,
You could try changing changing the MAIL_ENCRYPTION=STARTTLS to instead be MAIL_ENCRYPTION=tls.
Additionally, you might need to change your MAIL_HOST option to be MAIL_HOST=smtp.office365.com.
You may also need to provide a password, sometimes (Such as if MFA is enabled) you may need to generate a seperate "app password" for this kind of usage.

@ssddanbrown
i get nnow 2 errors when i played arround with the config a bit.
I switched back to STARTLS and used port 465 gave me this error.
Error thrown when sending a test email: Connection could not be established with host smtp.office365.com :stream_socket_client(): Unable to connect to starttls://smtp.office365.com:465 (Unable to find the socket transport "starttls" - did you forget to enable it when you configured PHP?)
then i switched back to tls and got this one. its the same.
Error thrown when sending a test email: Connection could not be established with host smtp.office365.com :stream_socket_client(): Unable to connect to starttls://smtp.office365.com:465 (Unable to find the socket transport "starttls" - did you forget to enable it when you configured PHP?)
now i got these settings x is just the mail we use.
Mail sender details
MAIL_FROM_NAME="Proxmox-Team Santis-Wiki"
MAIL_FROM=x
SMTP mail options
These settings can be checked using the "Send a Test Email"
feature found in the "Settings > Maintenance" area of the system.
MAIL_HOST=smtp.office365.com
MAIL_PORT=587
MAIL_USERNAME=x
MAIL_PASSWORD=null (not set cause its an email without password probably the problem ?)
MAIL_ENCRYPTION=tls
Error thrown when sending a test email: Failed to authenticate on SMTP server with username "X" using 2 possible authenticators. Authenticator LOGIN returned Expected response code 235 but got code "535", with message "535 5.7.139 Authentication unsuccessful, the user credentials were incorrect. [ZR0P278CA0163.CHEP278.PROD.OUTLOOK.COM] ". Authenticator XOAUTH2 returned Expected response code 235 but got code "535", with message "535 5.7.3 Authentication unsuccessful [ZR0P278CA0163.CHEP278.PROD.OUTLOOK.COM] ".
So i guess it wont work with a Mail account that has no password ? is this correct so we need a Mail we can login to it ?
thx for the help.

So i guess it wont work with a Mail account that has no password ? is this correct so we need a Mail we can login to it ?

I am not completely sure on the rules Microsoft imposes, but I'd be surprised if you could send mail through their SMTP server without a proper authenticated account. Since that error is explicitly failing authentication due to credentials, I would assume it's therefore required.

@ssddanbrown
thx that is what i thought. im gonna check it out i guess we can close the ticket


Install fails on arm64 MacOS

System: MacOS Ventura 13.0
Python: 3.11.0 (inside Conda env)
  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for tweedledum
Failed to build tweedledum
ERROR: Could not build wheels for tweedledum, which is required to install pyproject.toml-based projects

Problem is in pip install tweedledum
This seems like a well known issue in tweedledum and in qiskit-terra

Hi plutoniumm,
Thanks for the feedback! Did you try pip install tweedledum==1.1.0 qiskit according to the suggestions in the issue you pointed here?

Hey yeah I tried all variations i could think of

Installing a couple of older versions
Installing it from source
No cached
In a sequestered conda env

And maybe something more here at there. It fails everytime and I'm really not able to get why

Hello!
I am having the same issue here.
Do you think it might be because of the python3 version? I am using 3.11.3

hi
I think tweedledum is a package required by qiskit, I can work on separating tq and qiskit, which means you can use tq alone. But when it comes to converting circuit to qiskit, the package will still be needed.

System: MacOS Ventura 13.0
Python: 3.11.0 (inside Conda env)
While installing Quantum Matcha Tea, which requires qiskit, I got the same issue:
pip install tweedledum
Collecting tweedledum
Using cached tweedledum-1.1.1.tar.gz (4.8 MB)
Installing build dependencies ... done
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
Any suggestions for fixing it?


Allow page breaks within rows

I am using gridx with rows containing many lines of text. It would be nice if there was an option to allow page breaks within these rows, similar to the default in grid. Additionally, this would allow the use of rows that are higher than the page.

I intentionally left this out as I believed it could cause problems with header repetition, but I think I could add experimental support for this which just has the header appear below the long row (which probably isn't very desirable, but I don't think there's much else I could do without modifying the Typst compiler itself).

I assume that this is an example of this? https://typst.app/project/rTJdSEwyv8rGJZKYSvlaFJ
In this case the table is effectively one very long row with some nested tables in the right-hand column.


What is a Jupyter Server for Testing?

Testing #13585
When I choose "Select kernel," I see 2 options:

I'm unsure what a Jupyter Server for Testing is, so I click the (i) to go to docs. I'm brought to https://code.visualstudio.com/docs/datascience/jupyter-kernel-management#_existing-jupyter-server. This section doesn't really answer my question. When I ctrl+f the whole doc, I also don't get any results.
cc @minsa110 for the docs aspect too

@bamurtaugh thats a bug I will remove that, it should only show up during testing.


Format of input data from multiple individuals

Great work! I'm trying to see if I could apply this to some of our own data, and have a question in terms of how to feed input data from multiple individuals. It appears that the example dataset is from one individual, with 27 samples at different time points. How would I structure input data from multiple individuals, would I just merge the data to the same matrix?

I think I figured it out- I hadn't seen the UBCSamples variable that encodes which columns belong to which individuals. I'm assuming these are ordered in chronological order from left to right (ex. if samples 22-25 are from patient 5, then sample 22 would be the first time point and 25 the last).

Hey Sam,
I'm happy that you liked the idea and the method.
You are right, the UBCSamples is the id vector used to signal the algorithm which sample came from which patient. TimeAx assumes that the order of samples in the data (within each patient) is based on the chronological time so the first sample of patient 5 will be the first time point, the second will be the second time point and so on.
To be clear, the example data is just a scrambled data so there should be no actual biological effect there but this is the way to run the method.
If you have any more questions, please let me know and good luck with your analysis.
Amit


QueryRequestAnalyzer throwing a Null exception

This issue is obscuring another issue that I'm having that is preventing me from calling a Mutation of mine.  The error that I'm receiving is:
4>CSC : warning AD0001: Analyzer 'ZeroQL.SourceGenerators.Analyzers.QueryRequestAnalyzer' threw an exception of type 'System.NullReferenceException' with message 'Object reference not set to an instance of an object.'.
I started debugging the analyer and see the root cause here.  At https://github.com/byme8/ZeroQL/blob/main/src/ZeroQL.SourceGenerators/Analyzers/QueryRequestAnalyzer.cs#L73 the ZeroQLRequestLikeContextResolver.Resolve method can return an Error in its Result<GraphQLSourceGenerationContext> instance, but the if condition following the Resolve call only checks for an ErrorWithData<Diagnostic>.  Its else then gets called which assumes that there is no error and that requestContext is not null (but it is null).  Therefore, an exception occurs at https://github.com/byme8/ZeroQL/blob/main/src/ZeroQL.SourceGenerators/Analyzers/QueryRequestAnalyzer.cs#L83 when trying to get the OperationQuery property.
As for the other error, I'm still trying to pin it down and will create an issue for that, if it isn't something that I'm doing wrong.  Love this library by the way.  Thanks.

Checkout v2.1.1. Now it handles the case with the Error properly.


LoggingEvent functionality in NLog

Question
Hi, is there a similiar functionality in NLog as LoggingEvent in Log4Net?
https://logging.apache.org/log4net/log4net-1.2.13/release/sdk/log4net.Core.LoggingEvent.html

The answer is sort of ... by using MethodCall target and triggering the event from called method ...
https://github.com/nlog/nlog/wiki/MethodCall-target


Folder sync (opt-in)

Call POST /$/sync/path/to/sync "folder/on/fs/to/sync"

I think this feature would be useful for hyper.el.

I think it could pose a security risk to implement, so I'd prefer it if we leave folder sync to client libraries. 👀


save wallet inside lock

Coverage decreased (-0.01%) to 94.7% when pulling 8195e067c3a3835e5a8aa6e88c176608d723ed32 on isocolsky:scanning into a73b9bec1d31dc579aafe38e46830fe92438f9b7 on bitpay:master.


coqPackages.Ordinal: init at 0.5.3

Description of changes
I added myself to the maintainer list, and added coqPackages.Ordinal.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@ofborg build coqPackages_8_18.Ordinal

Please squash the last two commits

Test: https://github.com/coq-community/coq-nix-toolbox/pull/291


Add the sensitive_post_parameters decorator to views.

Fixes #1869

@jerivas Please review.

Thanks! I can't try out the PR right now, but can you confirm the server error messages are being properly protected by the decorators?

@jerivas I am having some trouble setting up the project. It would be better if someone can confirm this.


Question: Fastest way how to insert data into DB

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[X] 3.1

On this operating system:

[x] Linux

[x] Debian .deb

Hello,
I want implement snapshot of DB. We have stored JSON documents which represent data what is necessary for recovering DB in particular time, so I'm looking for the fastest solution how to insert data into the database.
I thought that FOXX should be faster than ArangoJS driver, because Foxx lives inside DB, so there will be eliminated requests to HTTP API.
I wrote simple test, 2 000 devices who have 20 routes each. See implementation using ArangoJS. This test took 19s
Promise.map(listOfDev, dev => {
    return storeDev(arDb, dev.devName)
    .then(sDev => {
      return Promise.map(dev.route, r => storeRoute(arDb, sDev._id, r));
    });
});

function storeRoute(db, devId, name) {
  return db.query(`for r in vRoute filter r.vDevId == @devId && r.name == @name return r`, {name, devId})
  .then(cursor => cursor.all())
  .then(results => {
    switch (results.length) {
      case 0:
        return db.collection('vRoute').save({vDevId: devId, name});
      default:
        return results[0];
    }
  });
}

function storeDev(db, name) {
  return db.query(`for d in vDev filter d.name == @name return d`, {name})
  .then(cursor => cursor.all())
  .then(results => {
    switch (results.length) {
      case 0:
        return db.collection('vDev').save({name}).then(r => r);
      default:
        return results[0];
    }
  });
}

Then I wrote same test using FOXX, this test took 394s - it's because FOXX is sync
function storeDev(dev){
  const d = vDevItems.byExample({name: dev.devName}).toArray();
  switch(d.length) {
    case 0:
     return vDevItems.insert({name: dev.devName});
    default:
     return d[0];
  }
}

function storeRoute(vDevId, name){
  const r = vRouteItems.byExample({vDevId, name}).toArray();
  switch(r.length) {
    case 0:
     return vRouteItems.insert({vDevId, name});
    default:
     return r[0];
  }
}

router.get(function (req, res) {
  vRouteItems.truncate();
  vDevItems.truncate();
  const vLogItems = db.vLog.all().toArray();
  const start = Date.now();
  vLogItems.forEach(dev => {
    const d = storeDev(dev);
    dev.route.forEach(r => storeRoute(d._id, r));
  })
  res.send([{time: Date.now() - start}]);
}, 'list')

Then I tried to use FOXX Queues (200 workes), only insertion jobs to queues took 6.8s and all tasks was completed after 321s! (_jobs.modified - _jobs.created)
router.get(function (req, res) {
  vRouteItems.truncate();
  vDevItems.truncate();
  const vLogItems = db.vLog.all().toArray();
  const start = Date.now();

  vLogItems.forEach(dev => q.push(
    {mount: '/test', name: 'saveDev'},
    dev
  ))
  res.send([{time: Date.now() - start}]);
}, 'list')

/***** saveDev.js script *****/
'use strict'
const argv = module.context.argv;

const vDevItems = module.context.collection('vDev');
const vRouteItems = module.context.collection('vRoute');

function storeDev(dev){
  const d = vDevItems.byExample({name: dev.devName}).toArray();
  switch(d.length) {
    case 0:
     return vDevItems.insert({name: dev.devName});
    default:
     return d[0];
  }
}

function storeRoute(vDevId, name){
  const r = vRouteItems.byExample({vDevId, name}).toArray();
  switch(r.length) {
    case 0:
     return vRouteItems.insert({vDevId, name});
    default:
     return r[0];
  }
}

const d = argv[0];
const dev = storeDev(d);
d.route.forEach(r => storeRoute(d._id, r));

Is it normal that the foxx is so slow? Or I missed something? What is the fastest way insert data to db (instead of dump)?

I saw that when using the JS driver, the collections were named vDev and vRoute, and for the Foxx version they are named vDevItems or vRouteItems. This is not a problem, but I was wondering whether the collections have identical index definitions.
vDev/vDevItems should have an index on name, vRoute/vRouteItems should have an index on [vDevId, name]. If indexes are missing, the lookup operations will be unnecessarily slow.
For example, when I run the inserts in Foxx without indexes, it takes 265 seconds. With indexes, it takes only 10 seconds. So probably the different insertion times are due to missing indexes in one case.
Additionally, is any of these collections created with waitForSync === true? If yes, this will make insert operations slower than when setting it to false.
Finally, the approach you've taken to ensure uniqueness is more expensive than necessary. For each item you're about to insert the code is performing an all() or toArray() query. If the normal case is that the item is not present yet (so insert will normally succeed), then it will be better to create a unique index and try the insert. Only if it fails a document lookup needs to be done.
I have created the unique indexes and changed the relevant code section as follows:
function storeDev(dev){
  try {
    // first try insert...
    return vDevItems.insert({name: dev.devName});
  } catch (err) {
    // insert failed. now return the first document matching the search pattern
    return vDevItems.firstExample({name: dev.devName});
  }
}

function storeRoute(vDevId, name){
  try {
    // first try insert...
    return vRouteItems.insert({vDevId, name});
  } catch (err) {
    // insert failed. now return the first document matching the search pattern
    return vRouteItems.firstExample({vDevId, name});
  }
}

This change cut down the runtime to less than a second in Foxx.
The commands to create the unique indexes in ArangoShell were:
db.vDevItems.ensureIndex({ type: "hash", fields: ["name"], unique: true });
db.vRouteItems.ensureIndex({ type: "hash", fields: ["vDevId", "name"], unique: true });

Firstly let me clarify setup. Yes, JS driver vs FOXX use different collections, it's only for testing purposes (comparing results). Each of collections had set waitForSync === false and also there wasn't any index. I thought that index optimization isn't needed for testing purposes.
But I just tested your suggested solution (unique indexes) and now both have similiar times.

JS driver (async) 9529.924ms
JS driver (sync Promise.mapSeries)  20933.033ms
Foxx (no queue) 9121ms
Foxx (using queue 200 workers)

Fistr insert into queue 8121ms all jobs done 1020ms
Second insert into queue 10239ms all jobs done 1020ms
Third insert into queue 13179ms all jobs done 1020ms (how _jobs collection grow then insertion is slower)

Now I can see that FOXX can be little bit faster than JS driver, but it isn't still too big difference and also it depends on good DB design. Do you plan async in FOXX?

By using Foxx you may also save a lot of HTTP traffic. This can be beneficial if the network is slow or client application and database are on different hosts.
For example, if a Foxx route can insert all documents at once, it will only require one HTTP request from the client application. Compare this to issuing 2,000 times 20 HTTP requests for the individual inserts (probably even more, because at least in the old code each insert was actually a series of check-and-insert-only-if-not-already-exists operations).
And getting the indexes right will definitely help. Not only will the lookups be faster, but the index uniqueness will also save you from data races that the non-unique-index code will be suffering from. Without the unique indexes, two parallel client threads could at the same time detect that an entry was not yet present and both try to insert it (and succeed). By using a unique index only one thread will succeed. And by catching the error and turning the insert into a lookup as in my example the results will always be correct provided that documents aren't deleted from the collections in between.
Finally, regarding async operations in Foxx: at the moment they are not supported, and I myself don't if they are on the roadmap and for which version. I'll leave this up for others to answer.


[Windows] 'keytool' is not recognized as an internal or external command

Describe the bug
A clear and concise description of what the bug is.
When running bubblewrap init  it fails with the error: 'keytool' is not recognized as an internal or external command
I tried in CMD and in Powershell
To Reproduce
Steps to reproduce the behavior:

run bubblewrap init --manifest {MANIFEST}
have no keystore yet, create one when asked
See error

Expected behavior
A clear and concise description of what you expected to happen.
I expected it to generate the application
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10

Additional context
Add any other context about the problem here.

Partial reason was I didn't have the JDK in my path -> but the bubblewrap cli asked for the path to the JDK but it seems not to be used

It does use the JDK in the path. Any change the jdk path is pointing to the incorrect folder? There's more info on how to check it here: https://github.com/GoogleChromeLabs/bubblewrap/tree/master/packages/cli#updating-the-location-of-the-jdk-and--or-the-android-command-line-tools
Overall, the doctor command tracked on #181 and implemented on #220  should make it easier to debug this in the near future.

Closing this as the doctor command now exists and dependencies can be installed automatically.

I hit this problem while trying to run bubblewrap on windows using gitbash. Even bubblewrap installed JDK fails to run keytool.
Executing bubblewrap using windows cmd works fine.


Fixed deprecated warning for simple_form.

See plataformatec/simple_form#997 for an
explanation. Fixes issue #284 .
But at the same time it breaks the testing suite since this only works for newer simple form versions. The testing suite is using simple_form 1.5.2 at the time of writing.
See also pull request for old gem: https://github.com/crowdint/rails3-jquery-autocomplete/pull/287

:+1:

@panmari Can you update the tests for this?  At this point I'm comfortable dropping support for the old version of Simple Form.  Most users have or will be upgrading; if not then they should be able to use a prior release of the gem.

@panmari @14113 Could you try running against the https://github.com/bigtunacan/rails-jquery-autocomplete/tree/simple_form_fix branch?
It is still compatible with older versions of simple form and I believe it should work with 3.0.2 and above as well, but am not setup to test that very easily.

I've fixed this via pull request #10 which also keeps backwards compatibility with older versions of simple form gem.


Deprecation warning for default value of boundary kwarg in map_overlap

This PR is the first step in updating the default boundary kwarg value in Dask's map_overlap function. The current default value is "reflect", in future this will be changed to "none".
We want to do this primarily for performance reasons, as discussed here: https://github.com/dask/dask/issues/8391
Questions:

Should we choose a specific version to change this in? Something 6 months or a year away?
Do we need to do this as a two step process? Potentially I could use the disutils LooseVersion to check whether the dask version is above or below a certain number, so this change would turn on automatically.

...although then I guess you'd still need a second step to prune out that stuff at a later date, it's just less important to remember to do it at a specific time

Is FutureWarning correct to use here? This seems the most commonly used in the Dask codebase, although there is also a PendingDeprecationWarning used a handful of times, so I thought I should check.

[x] Related to https://github.com/dask/dask/issues/8391
[ ] Tests added / passed
[x] Passes pre-commit run --all-files

cc @jakirkham

There are a number of tests that are failing because of the FutureWarning added here. I've updated the tests to explicitly specify what boundary value they need here.

Ok, latest commits should fix up those doctest errors (doctests were not explicitly specifying boundary= kwarg and then failing when the warning was raised)

...I don't really understand why the macOS tests are failing here. Something's a bit weird. I'm going to investigate on my personal laptop & see if I can reproduce it.

I should add - the kinds of test errors I'm seeing on my laptop are not the same event loop errors that are happening in the CI.
ERROR dask/array/tests/test_xarray.py - RuntimeError: There is no current eve...
ERROR dask/bytes/tests/test_hdfs.py - RuntimeError: There is no current event...
ERROR dask/tests/test_distributed.py - RuntimeError: There is no current even...
ERROR dask/tests/test_layers.py - RuntimeError: There is no current event loo...
ERROR gw0

Here's hoping those are just a bit of a fluke.

I've just scheduled an email that should hopefully remind me to make a follow up PR to this in the last week of February (when we should actually change the kwarg default value). Fingers crossed the reminder works!

Whoa. I've never thought of doing something like that.


Build cryptography

Should we build only wheels for certain platforms/archs?

I guess linux is fine ? Only need it because it's compiled against GLIBC_2.28


test(e2e): test multiple volumes in a single test pod MQ-361

Verify that within an E2E context and single pod can mount and
use (run fio) on multiple volumes provisioned with
- a single replica
-  multiple replicas

Create a single test pod
mount 6 volumes on the test pod
run fio on all volumes

Run these steps for volumes with 1 and 2 replicas

bors try

bors merge


Duplicate entry prevention

In some cases backend could generate multiple requests to the ledger with the same transaction data resulting in a duplicate entry. One way to prevent this could be setting transaction id to HASH(order.id) - posting this more than once should result in failure.
Is this possible? Or there is a better approach to this?

Hey @minvs1, we will solve this with #8
Closing this one for now.


${machinename} breaks NLog in .NET Core

Hi! Thanks for reporting this feature/bug/question!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
Type (choose one): Bug
NLog version: 4.4.0-beta12
Platform: .Net Core 1.0
Current NLog config (xml or C#, if relevant)
This breaks:

This works:

This is all despite not even using the target.   The only logger I have is:

Desired Result is for ${machinename} to work on .NET Core.

This one is indeed not supported yet: https://github.com/NLog/NLog/wiki/platform-support
Any idea how we get the machine name in c# with netcore?

Looks like it has been added in rc2
https://github.com/dotnet/corefx/pull/5428

Yep! Was just going to reply with that.
On Thu, Jun 30, 2016 at 1:53 PM -0700, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:
Looks like it has been added in rc2
dotnet/corefx#5428
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Although this will bump you up to netstandard 1.5. You'll probably want to conditionally target 1.5.
On Thu, Jun 30, 2016 at 2:06 PM -0700, "Andrew Katz"<EMAIL_ADDRESS>wrote:
Yep! Was just going to reply with that.
On Thu, Jun 30, 2016 at 1:53 PM -0700, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:
Looks like it has been added in rc2
dotnet/corefx#5428
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ah good to know. Any idea where you  could see that netstandard1.5 is needed?

http://packagesearch.azurewebsites.net/?q=Machinename
On Thu, Jun 30, 2016 at 2:19 PM -0700, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:
Ah good to know. Any idea where you  could see that netstandard1.5 is needed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

PR would be nice (coreCLR branch)
My VS is broken after update3. Can't open any project :(

@304NotModified would you point me to the area of code.  Will get PR submitted immediately.

@304NotModified https://github.com/NLog/NLog/pull/1526

When do you think you'll push to Nuget?

This weekend. Build pc not fixed yet.

https://www.nuget.org/packages/NLog/4.4.0-beta13
Published from AppVeyor :)
Thanks for the submit!


Video FIle Size limitations

Hi there,
I am having some issues with how your example implements the loading of of video assets into android media player.
I am trying to load a rather large vertical stereo video from the assets directory the same way in which is demonstrated in the minimal 360 video example, but keep hitting and IOException error. I am guessing it is due to the (not sure if this is till relevant after android 2.3???) file size limits for the assets folder or not?
Is there a workaround for this if the app needs to be distributed with the video embeded? It cannot be loaded into an external directory by the user

Okay so I managed to get the video loading buy re-compressing it down to a smaller size -> which is not ideal as I would like to be able to playback the bst quality stereoscopic content I can.
I then came across another issue where playback speeds were incredibly slow! Bare in mind, I am also loading in another  (much smaller) video file from assets for my UI menu. I am not sure if this is what was generating drop in frame rate.
I managed to get back to an acceptable frame rate by modifying the way in which the video assets were loaded into the mediaPlayer object, and am loading them in now from res/raw:
`mMediaPlayer = MediaPlayer.create(mGVRContext.getContext(), R.raw.showreel_1);
    mMediaPlayer.setLooping( false );

    video = new GVRVideoSceneObject(mGVRContext, mesh, mMediaPlayer, GVRVideoType.VERTICAL_STEREO);`

This now loads the compressed video into mediaPlayer fine, however when I tried this with the uncompressed version (larger file size), the application would crash on open.
The large file in question plays absolutely perfectly when being loaded from external storage.
Any wise words from anybody would be much appreciated

Hi, can you share the file size that doesn't work and the one that does? What is the resolution of the video?

Sure thing, sorry for leaving that out...
The uncompressed file:
2000x2000 pixels
60fps
progressive field order
audio: 48000Hz, Stereo
filesize: 1.32 GB
The compressed file:
2000x2000 pixels
60fps
progressive field order
VBR, 1 pass 10 -12Mbps
AAC, 320 Kbps, 48khtz Stereo
filesize: 196 MB

Is rescaling the video an option? I would say more than 1000x1000 is an overkill considering the pixel area the video is shown onto.
It is very likely something is running out of memory with the 1GB+ file.

Hmmm I dont really want to rescale the content as I know what 2k content looks like, and it looks great! cutting it in half would be a dramatic loss in quality...
Are there any recommended compression settings for 2k content?
Cheers

Just asking, you could try it at least and see how it looks in stereo. We will look into the issue but since this requires embedding a 1GB video in the apk it could take some time.  Can you share the logcat output? Thanks.

This problem occur if you try to do this without gvrf?  In other words, as a regular android app using the media player?  Iirc, the hw should be able to playback 2kx2k video just fine.  Maybe it is just a memory issue.  I'm not as familiar with the media player, but is there a way to stream rather than load the whole file in?

It seems to work fine when building for gear out of unity - haven't had any issue with the uncompressed file regarding playback or memory there... I just prefer developing with gearvrf as it is a lot less restrictive than unity.
Unfortunately my hands are also tied when it comes to down-scaling - my boss eventually wants to be at 4000x4000 pixels and dropping resolution would be a step backward.
GearVRf has definitely been able to play back 2kx2k at 1+ GB from external storage perfectly fine, but now in the process of wanting to put a free download up on the oculus store, so the content needs to be packaged with the application.
I just tried loading the 1GB+ file from the res/raw folder and the application crashes on start.
LogCat file attached from initial error when attempting to load 1+GB file from assets folder using the method demonstrated in the examples.
logcat1.txt
I also notice after recreating that issue, that tap events were no longer being listened to
Looking into alternative compression techniques to see if I can drop the file size without losing too much quality - the 196MB file is ghastly!
Alternatively, a way of not having the file embedded in the assets or raw directory would be to have to embedded in an android extension file (ogg) -> have you guys had any experience with extension files and GearVRf?
On another note, with the embedded file,
even at the smaller size, I am seeing a lot of dropped frames and video stutter.

I'm not an expert at the media player stuff, but I'm taking a look anyway.    Googling reveals:  http://stackoverflow.com/questions/9008770/media-player-called-in-state-0-error-38-0
That may not be the core of your problem, but it might not be helping either.  I'll keep looking...

what file extension is the video?

ok, so, i created my own 1.2GB video in mp4 by recording a video on my phone.  added it to the assets directory, pointed the media player to it and also got a crash.  here are the relevant logs:
05-11 12:23:51.039 16607 16658 V MediaPlayer: setListener
05-11 12:23:51.039 16607 16658 E filemap : mmap(0,1298322698) failed: Out of memory
05-11 12:23:51.039 16607 16658 W asset   : create map from entry failed
05-11 12:23:51.049 16607 16658 W System.err: java.io.FileNotFoundException: 20160511_104221.mp4
05-11 12:23:51.049  1314  2709 V BroadcastQueue: [background] Process cur broadcast BroadcastRecord{e279268 u0 android.intent.action.PACKAGE_REMOVED qIdx=4}, state= (APP_RECEIVE) DELIVERED for app ProcessRecord{7cf05ee 16641:com.android.settings/1000}
05-11 12:23:51.059 16607 16658 W System.err: 	at android.content.res.AssetManager.openAssetFd(Native Method)
05-11 12:23:51.059 16607 16658 W System.err: 	at android.content.res.AssetManager.openFd(AssetManager.java:379)
05-11 12:23:51.059 16607 16658 W System.err: 	at org.gearvrf.gvr360video.Minimal360Video.onInit(Minimal360Video.java:56)
key part is that mmap failed because it was out of memory.  there's gotta be a way to stream from the assets directory rather than load the whole file into memory.

so, i've been trying this all day.  (with a 1+GB file, the adb install time is around 10 minutes...slowing down iteration time).  There's no way you can make that video just a little shorter?  I seem to not get the out of memory problem if i make the file 500MB.  I do get it at 999MB.  Haven't finished binary searching what size will be accepted.
I tried passing the file as a Uri as illustrated in these stackoverflow links:
http://stackoverflow.com/questions/8269353/how-to-load-videos-from-assets-folder-to-play-them-with-videoview
http://stackoverflow.com/questions/3028717/how-to-play-videos-in-android-from-assets-folder-or-raw-folder
of course another option is to have your application copy the file to sdcard (since you said that works) on startup and then play it.

Hey @thomasflynn  sorry for my slow reply - Im guessing you are on the other side of the world from me (Im in the UK)
I had another look into that log file where you found the error(-38, 0), and that was a schoolboy error of myself not naming a file reference correctly and has been fixed.
I did however manage to get better playback by removing one of the the mediaplayer objects - one I have in my 'home' menu system, and instead using a stereoscopic static image - which again, I dont like as I would like to be abel to have animated content in the menu. The video that was used for that object was small, but perhaps having multiple mediaPlayers in one scene is draining a heap memory - I am going to look at dynamic content swapping and have a single media player for the entire application.
Back to the file and size - it is an .mp4, and currently gettign sufficient playback out of the assets folder with a 600MB file.
I did look into streaming via a URI, but couldnt quite get it working - are you able to share your code on your approach there?
I also have looked into using the videoView but im sure I read somewhere that videeo view is simply a wrapper for mediaplayer with embedded controls, so shouldnt have any impact on performance - if that is what you are suggesting?
i will look further into streaming from assets, and/or copying to ssd. The one hang up i have with copying the file to ssd / external storage is the potential of exposing the content to the user. we work in the medical/pharmaceutical animation realm, where content is very sensitive and we cannot allow for it to be accessed by users (well without at least a bit of hard work and know how). Ill have some more updates for you today.
O.

After spending most of today trying to find a solution where I can stream directly from the assets or raw folder, I am at a loss..
I have decided to go the route of saving the file to storage in onCreate() when the application is first opened.
The next time it is opened it checks to see if the file exists, if so, it doesn't need to overwrite.
At this stage I am currently saving this to external storage, just to get the ball rolling.
This is problematic for a store app where content is sensitive as
A. Even if the file location the video is being copied to is is "hidden" - by placing a '.' at the beginning of the file you create - this is easily discoverable by enabling display of hidden files.
B. Updates to the application and content will not remove any data stored in external storage, and will not overwrite files with the same name. Therefore content naming will have to change for each update, only bulking out external storage - some users may not be aware of what it is clogging up all of their storage.
Another problem I have come across by saving the content to storage in onCreate is again hitting an out of memory error with the 1.24GB file during copy, crashing the application.
With a smaller file being copies - ~400MB - there is also a noticeable hang the first time the application is opened while copying the file over. In the oculus store submission requirements, there must be no hangs, freezing, crashing etc for it to be accepted.
Potential solutions / next areas of exploration
Save the content to internal internal storage where the content will be removed when the application is removed/uninstalled, and the user does not have access to.
Implement a system (god knows how at this stage) that will overwrite any video content if an update is installed. Any ideas are welcome here.
Move the file copying into an asynchronous task and display some form of loading graphic before the content can be played. This is to avoid any hanging/freezing on create.
But the problem with memory still persists!
Regardless of all of this additional carry-on, we are still met with the exact same problem we had in the beginning! We are still unable to embed and and read/play large stereoscopic video content into a GearVRf application for Oculus Store submission!!!

Yeah, we're in California.  I'll keep looking for solutions today.  FWIW, this doesn't appear to be specific to gvrf, but to using the Android MediaPlayer.  In any case, off-hand, i don't know what the Oculus store policy is about large apks, but I know Google Play prefers to have any large files in a separate binary blob:  http://developer.android.com/google/play/expansion-files.html
again, i'll keep looking.  there's gotta be a way to trick the media player to stream for resource/assets files like it does for /sdcard and urls.  instead, it seems to want to mmap the whole file :(

as far as protecting the content, you can maybe look at the DRM support in Android: http://developer.android.com/reference/android/drm/package-summary.html

i took an attempt at using MediaDataSource: http://developer.android.com/reference/android/media/MediaDataSource.html
found an example here:
<EMAIL_ADDRESS>i figured with that, i could manually buffer it.  I was trying to get it to work with the small video first and i couldn't get it to work.  I'll show you what i got, but i got my git tree a little messed up in the process, but you can just focus on the changes in gvr-360video.  here's a link to the tree:  https://github.com/thomasflynn/GearVRf-Demos/tree/bigvideo
it seems like it starts reading from it, but then gives a totally bogus position to start at<PHONE_NUMBER>).

Ok, figured out why that didn't work.  MediaDataSource needs to be seekable.  I can't just read in the stream sequentially. :(

@thomasflynn The seek at<PHONE_NUMBER> is quite strange since MediaDataSource needs to specify the file size by implementing getSize(). Mediaplayer should not seek beyond that. If the error persists, maybe we can try another approach.
I think in our remote scripting example, we could play a video from a URL. Since it's an acceptable approach for @oliverellmers, why won't we point to that approach:
To create a media player that plays a URI:
        MediaPlayer mediaPlayer = new MediaPlayer();
        try {
            mediaPlayer.setDataSource(filenameOrURL);
            mediaPlayer.prepare();
        } catch (IOException e) {
            e.printStackTrace();
            return null;
        }

After it is successfully prepared, we can call mediaPlayer.start() to play the video. To play it automatically, we can add a MediaPlayer.OnPreparedListener to the media player, and call mediaPlayer.start() from the onPrepared method.

Yeah, look back a couple days ago, that's exactly what i tried.  It still tried to mmap the file.

Hey guys,
Apologies for not getting back on this sooner. Have been tasked with a different problem at work.
Anyway, I have still not managed to get this going, and have set up a work around for the mean time by copying the video to internal storage asynchronously when the application is first loaded, displaying a "preparing content" message while we wait. I have compressed the file down to 700MB which give me about a 5-6 second preparation time.
As the copying is being done asynchronously, there is no halt/lag in my 'home' menu, allowing free movement with the sensors, which will hopefully allow a pass on the oculus store.
I unfortunately still run into an out of memory error if I try to copy a file to storage that is any larger than the 700BM file, suggesting a limitation on the device (I am working with S6).
At this stage, it looks like our troubles are limitations of the devices and android itself, and not directly related to GearVRf.

Thanks for the update. Glad to hear that you are able to somehow reduce the file size and do the decompression (from APK to app data directory) before playing the video. Please let us know if you see other issues.

@oliverellmers Can we close this issue or you need further help?

Planning to close tomorrow

And update in this topic(trying to display big video file from assets)?

I think we pretty much ran into Android limitations with this :(

It's a little bit off-topic, but can be very helpful for people seeking answer about video size limitations.
Why, in the whole gearvr documentation there is no information about maximum APK size ? Why you can upload much bigger APK to oculus store, and without any problems test it on alpha/beta channels?
Like a lot of people in this topic, I tried to load big video file from assets. I faced a lot of problems, but in the end I figured the way it works. Application passed content review(after a lot of other problems). And now, I received information from support, that APK size limit is 500MB (even if oculus store accepts bigger files, and it passes content review...).
tl;dr : Don't try to solve problems with playing big files from assets, APK size is limited to 500MB.
hint : Gear VR apps have OBB support! (but only uploading by CLI, not a web panel)


[$250] User is able to disable 2FA when connected to Xero accounting

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.64-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @allgandalf
Slack conversation (hyperlinked to channel name): ts_external_expensify_bugs
Action Performed:

Connect to Xero
Disable two-factor authentication in settings

Expected Result:
We should block this with a RHP stating that we cannot disable two-factor auth while connected to xero
Actual Result:
User is able to disable two-factor authentication and on each login it just asks the user to re-enable it
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/a45269eb-8c61-4e30-83f5-5e589b882baf
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021859734183969700022
Upwork Job ID:<PHONE_NUMBER>969700022
Last Price Increase: 2024-11-28
Automatic offers:

allgandalf | Reviewer | 105196111
FitseTLT | Contributor | 105196113

Issue OwnerCurrent Issue Owner: @
Issue OwnerCurrent Issue Owner: @neil-marcellini

Proposal
Please re-state the problem that we are trying to solve in this issue.
User is able to disable 2FA when connected to Xero accounting
What is the root cause of that problem?
We are not disabling the disabling of 2FA while there is xero accounting connection
https://github.com/FitseTLT/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/pages/settings/Security/TwoFactorAuth/Steps/EnabledStep.tsx#L31-L33
What changes do you think we should make in order to solve the problem?

We can check for xero connection by checking for the key CONST.POLICY.CONNECTIONS.NAME.XERO in  POLICY_CONNECTION_SYNC_PROGRESS or checking policy.connection.xero just similar to what we do
https://github.com/Expensify/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/libs/PolicyUtils.ts#L1027-L1028
And if there is a xero connection we can hide/disable the disable 2FA button and inform users with some message on the RHP
What alternative solutions did you explore? (Optional)

I think when the user taps Disable two-factor authentication we should open one of our confirmation modals letting them know they can't do that because of their Xero connection. Something like this but with help from @jamesdeanexpensify on the copy.

Proposal
Please re-state the problem that we are trying to solve in this issue.
User is able to disable 2FA if connected to xero
What is the root cause of that problem?
we are adding the disableTwoFactorAuth to menu items, without checking for Xero or other connections:
https://github.com/Expensify/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/pages/settings/Security/TwoFactorAuth/Steps/EnabledStep.tsx#L29
What changes do you think we should make in order to solve the problem?

I think we can simply don't add this item if user has a connection at all Or Xero connection:
 menuItems={
+    !hasAccountingConnection ? [
        {
            title: translate('twoFactorAuth.disableTwoFactorAuth'),
            onPress: () => {
                setStep(CONST.TWO_FACTOR_AUTH_STEPS.GETCODE);
            },
            icon: Expensicons.Close,
            iconFill: theme.danger,
            wrapperStyle: [styles.cardMenuItem],
        },
    ] : []}

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
User is able to disable 2FA when connected to Xero accounting
What is the root cause of that problem?
We are not disabling the disabling of 2FA while there is xero accounting connection
https://github.com/Expensify/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/pages/settings/Security/TwoFactorAuth/Steps/EnabledStep.tsx#L27-L39
What changes do you think we should make in order to solve the problem?

We should first check for xero connection by checking if there is a single policy that the current user owns/admins where policy.connections.xero exists
function hasXeroConnections() {
    return !!allPolicies?.some((policy) => isPolicyOwner(policy, getCurrentUserAccountID()) && !!policy?.connections?.[CONST.POLICY.CONNECTIONS.NAME.XERO]);
}

we can also use isPolicyAdmin instead of isPolicyOwner according to the requirements
then we can show cannot disable 2FA ConfirmModal (some what similar to our FocusModeNotification with the respective copys and buttons with callbacks)
In here
https://github.com/Expensify/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/pages/settings/Security/TwoFactorAuth/Steps/EnabledStep.tsx#L33-L34
we will set the modal visibility if PolicyUtils.hasXeroConnections() is true
when they press the disable button in EnabledStep when pressing on the Got it button on the modal we can either dismiss only the confirm modal or we can additionally also dismissModal the 2FA RHP too.
or optionally we can hide/disable the disable 2FA button and inform users with some message on the bottom of the RHP
by avoiding the menu item or passing disable prop respectively when hasXeroConnections
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
User is able to disable 2FA when connected to Xero accounting
What is the root cause of that problem?
We are not disabling the disabling of 2FA while there is xero accounting connection
https://github.com/Expensify/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/pages/settings/Security/TwoFactorAuth/Steps/EnabledStep.tsx#L27-L39
What changes do you think we should make in order to solve the problem?

We should first check for xero connection by checking if there is a single policy that the current user owns/admins where policy.connections.xero exists
function hasXeroConnections() {
    return !!allPolicies?.some((policy) => isPolicyOwner(policy, getCurrentUserAccountID()) && !!policy?.connections?.[CONST.POLICY.CONNECTIONS.NAME.XERO]);
}

we can also use isPolicyAdmin instead of isPolicyOwner according to the requirements
then we can show cannot disable 2FA ConfirmModal (some what similar to our FocusModeNotification with the respective copys and buttons with callbacks)
In here when they press the disable button in EnabledStep
https://github.com/Expensify/App/blob/2d2555b173b47c0bce0b1c16b275f1f98639664c/src/pages/settings/Security/TwoFactorAuth/Steps/EnabledStep.tsx#L33-L34
we will set the modal visibility and return if PolicyUtils.hasXeroConnections() is true
when pressing on the Got it button on the modal we can either dismiss only the confirm modal or we can additionally also dismissModal the 2FA RHP too.
or optionally we can hide/disable the disable 2FA button and inform users with some message on the bottom of the RHP
by avoiding the menu item or passing disable prop respectively when hasXeroConnections
What alternative solutions did you explore? (Optional)

Danny's solution makes total sense to me. We might consider linking to Settings > Workspaces from the alert modal copy, but I don't feel strongly about that.

A little late here, but what about for the copy, something like:

Cannot disable 2FA
Two-factor authentication (2FA) is required for your Xero connection and cannot be disabled.

Normally I would give them next steps if they really want to disable 2FA, but generally it's bad practice and I also wouldn't want them to disconnect their accounting integration. So I kinda feel....ok?...not saying anything else here.

Works for me!

Definitely able to repro this.

Updated modal mock based on James' copy:

How are you going here @allgandalf ?

Thanks for proposals everyone 🙏 , Let's go with @FitseTLT 's proposal here, They were the first to suggest a correct solution.
@Krishna2323 I don't see any major difference in your proposal and it is an elaborated version of @FitseTLT 's proposal. So I would recommend @FitseTLT 's proposal here
🎀👀🎀 C+ reviewed

Sounds good to me.

Will be ready tomorrow 👍

@dannymcclain Can I get a confirmation on the look below?

@Expensify/design have we always had that drop shadow on these modals? I don't remember it and we don't have it in Figma (but it does appear in the product on staging). Is that new or am I crazy?
Other than that it looks good to me. @jamesdeanexpensify for a final copy check.

Looks good!

@Expensify/design have we always had that drop shadow on these modals? I don't remember it and we don't have it in Figma (but it does appear in the product on staging). Is that new or am I crazy?

I think this has always been there, but we don't see it very well in dark mode, but in light mode it's pretty meh. Let's tweak that as a separate issue, @dannymcclain ? I'd love to see it a bit more subtle in light mode

Let's tweak that as a separate issue, @dannymcclain ? I'd love to see it a bit more subtle in light mode

Totally down for that. In Figma we don't have one at all and IMO it's totally fine and good.

Yeah, chiming in late but I do think that's how it's always been. Big fan of tweaking it to be exactly how we want it (if we even want it at all) and having specific values for both light and dark modes.


Tabs: if first tab is disabled, it should not be set active

Just like title suggests. It only makes sense to set tab active if it's not disabled.

Addressed by 91b5fb62eedbb600d6a6abe32376846f327a903d

Is this ever getting fixed?