not building in windows 7

I could not build the package
I'm using:

Windows 7 32bits
Python 3.4.4 32bits
SDK Windows 7.1 and Visual C++ 2010 ( from https://www.microsoft.com/download/details.aspx?id=8279)
fix to compiler applied(from http://support.microsoft.com/kb/2519277)

links helped me:

https://blogs.msdn.microsoft.com/pythonengineering/2016/04/11/unable-to-find-vcvarsall-bat/
https://stackoverflow.com/questions/26473854/python-pip-has-issues-with-path-for-ms-visual-studio-2010-express-for-64-bit-ins

log of building:
C:\Users\IEUser\Downloads\splitstream>c:\python34\python.exe setup.py build
running build
running build_ext
building 'splitstream' extension
creating build
creating build\temp.win32-3.4
creating build\temp.win32-3.4\Release
creating build\temp.win32-3.4\Release\src
creating build\temp.win32-3.4\Release\src\python
c:\Program Files\Microsoft Visual Studio 10.0\VC\Bin\cl.exe /c /nologo /Ox /MD /
W3 /GS- /DNDEBUG -Ic:\python34\include -Ic:\python34\include /Tcsrc/pyth
on/splitstream_py.c /Fobuild\temp.win32-3.4\Release\src/python/splitstream_py.ob
j
splitstream_py.c
src/python/splitstream_py.c(98) : error C2099: initializer is not a constant
src/python/splitstream_py.c(115) : error C2143: syntax error : missing ';' before 'type'
src/python/splitstream_py.c(126) : error C2065: 'kwarg_list' : undeclared identifier
src/python/splitstream_py.c(126) : warning C4047: 'function' : 'char **' differs in levels of indirection from 'int'
src/python/splitstream_py.c(126) : warning C4024: 'PyArg_ParseTupleAndKeywords': different types for formal and actual parameter 4
src/python/splitstream_py.c(228) : error C2143: syntax error : missing '{' before '*'
src/python/splitstream_py.c(229) : error C2371: 'splitstream_generator_new' : redefinition; different basic types
        src/python/splitstream_py.c(42) : see declaration of 'splitstream_generator_new'
src/python/splitstream_py.c(239) : warning C4133: 'return' : incompatible types- from 'Generator *' to 'int *'
src/python/splitstream_py.c(274) : error C2275: 'PyObject' : illegal use of this type as an expression
        c:\python34\include\object.h(109) : see declaration of 'PyObject'
src/python/splitstream_py.c(274) : error C2065: 'readargs' : undeclared identifier
src/python/splitstream_py.c(275) : error C2275: 'PyObject' : illegal use of this type as an expression
        c:\python34\include\object.h(109) : see declaration of 'PyObject'
src/python/splitstream_py.c(275) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(284) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(284) : warning C4047: '=' : 'int' differs in levelsof indirection from 'PyObject *'
src/python/splitstream_py.c(294) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(294) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
src/python/splitstream_py.c(304) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(304) : warning C4047: '=' : 'int' differs in levels of indirection from 'void *'
src/python/splitstream_py.c(309) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(309) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
src/python/splitstream_py.c(316) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(316) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
src/python/splitstream_py.c(323) : error C2065: 'readargs' : undeclared identifier
src/python/splitstream_py.c(323) : warning C4047: 'function' : 'PyObject *' differs in levels of indirection from 'int'
src/python/splitstream_py.c(323) : warning C4024: 'splitfile_pure_once' : different types for formal and actual parameter 3
src/python/splitstream_py.c(324) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(324) : warning C4047: '=' : 'int' differs in levelsof indirection from 'void *'
src/python/splitstream_py.c(327) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(327) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
src/python/splitstream_py.c(334) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(334) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
src/python/splitstream_py.c(339) : error C2065: 'readargs' : undeclared identifier
src/python/splitstream_py.c(341) : error C2065: 'ret' : undeclared identifier
src/python/splitstream_py.c(341) : warning C4047: 'return' : 'PyObject *' differs in levels of indirection from 'int'
error: command 'c:\\Program Files\\Microsoft Visual Studio 10.0\\VC\\Bin\\cl.exe
' failed with exit status 2

other log building:
C:\python34\Scripts>..\python.exe -m pip install splitstream
Collecting splitstream
  Using cached splitstream-1.2.1.tar.gz
Installing collected packages: splitstream
  Running setup.py install for splitstream ... error
    Complete output from command C:\python34\python.exe -u -c "import setupt
ools, tokenize;__file__='C:\\Users\\IEUser\\AppData\\Local\\Temp\\pip-build-5_lp
h0n_\\splitstream\\setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.
read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" in
stall --record C:\Users\IEUser\AppData\Local\Temp\pip-m3ik0w3g-record\install-re
cord.txt --single-version-externally-managed --compile:
    running install
    running build
    running build_ext
    building 'splitstream' extension
    creating build
    creating build\temp.win32-3.4
    creating build\temp.win32-3.4\Release
    creating build\temp.win32-3.4\Release\src
    creating build\temp.win32-3.4\Release\src\python
    c:\Program Files\Microsoft Visual Studio 10.0\VC\Bin\cl.exe /c /nologo /Ox /
MD /W3 /GS- /DNDEBUG -IC:\python34\include -IC:\python34\include /Tcsrc/
python/splitstream_py.c /Fobuild\temp.win32-3.4\Release\src/python/splitstream_p
y.obj
    splitstream_py.c
    src/python/splitstream_py.c(106) : error C2099: initializer is not a constant
    src/python/splitstream_py.c(123) : error C2143: syntax error : missing ';' before 'type'
    src/python/splitstream_py.c(134) : error C2065: 'kwarg_list' : undeclared identifier
    src/python/splitstream_py.c(134) : warning C4047: 'function' : 'char **' differs in levels of indirection from 'int'
    src/python/splitstream_py.c(134) : warning C4024: 'PyArg_ParseTupleAndKeywords' : different types for formal and actual parameter 4
    src/python/splitstream_py.c(236) : error C2143: syntax error : missing '{' before '*'
    src/python/splitstream_py.c(237) : error C2371: 'splitstream_generator_new': redefinition; different basic types
            src/python/splitstream_py.c(42) : see declaration of 'splitstream_generator_new'
    src/python/splitstream_py.c(247) : warning C4133: 'return' : incompatible types - from 'Generator *' to 'int *'
    src/python/splitstream_py.c(282) : error C2275: 'PyObject' : illegal use of this type as an expression
            c:\python34\include\object.h(109) : see declaration of 'PyObject'
    src/python/splitstream_py.c(282) : error C2065: 'readargs' : undeclared identifier
    src/python/splitstream_py.c(283) : error C2275: 'PyObject' : illegal use ofthis type as an expression
            c:\python34\include\object.h(109) : see declaration of 'PyObject'
    src/python/splitstream_py.c(283) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(292) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(292) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
    src/python/splitstream_py.c(302) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(302) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
    src/python/splitstream_py.c(312) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(312) : warning C4047: '=' : 'int' differs in levels of indirection from 'void *'
    src/python/splitstream_py.c(317) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(317) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
    src/python/splitstream_py.c(324) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(324) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
    src/python/splitstream_py.c(331) : error C2065: 'readargs' : undeclared identifier
    src/python/splitstream_py.c(331) : warning C4047: 'function' : 'PyObject *' differs in levels of indirection from 'int'
    src/python/splitstream_py.c(331) : warning C4024: 'splitfile_pure_once' : different types for formal and actual parameter 3
    src/python/splitstream_py.c(332) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(332) : warning C4047: '=' : 'int' differs in levels of indirection from 'void *'
    src/python/splitstream_py.c(335) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(335) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
    src/python/splitstream_py.c(342) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(342) : warning C4047: '=' : 'int' differs in levels of indirection from 'PyObject *'
    src/python/splitstream_py.c(347) : error C2065: 'readargs' : undeclared identifier
    src/python/splitstream_py.c(349) : error C2065: 'ret' : undeclared identifier
    src/python/splitstream_py.c(349) : warning C4047: 'return' : 'PyObject *' differs in levels of indirection from 'int'
    error: command 'c:\\Program Files\\Microsoft Visual Studio 10.0\\VC\\Bin\\cl.exe' failed with exit status 2

    ----------------------------------------
Command "C:\python34\python.exe -u -c "import setuptools, tokenize;__file__='C:\\Users\\IEUser\\AppData\\Local\\Temp\\pip-build-5_lph0n_\\splitstream\\setup
.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record C:\Users\IEUser\AppData\Local\Temp\pip-m3ik0w3g-record\install-record.txt --single-version-externally-managed --compile" failed with error code 1 in C:\Users\IEUser\AppData\Local\Temp\pip-build-5_lph0n_\splitstream\

Than you for reporting! It looks like Visual Studio is not allowing C99 syntax. This is indeed something that should work "out of the box" if pip install does this. I will try to reproduce on a clean VM, as I don't have a working Windows installation right now.

Hi!
I'm using MS isos: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/
Thanks!

Hi!
So, this could be the real problem? VS 2010 compiler do not support C99?
https://stackoverflow.com/a/7614436
thanks!

Oh that is seriously bad, I didn't know that (they were only lagging ten years behind the C standard..), I would have guessed there was just a compiler flag missing.
Is there a reason you have to use VS2010 when compiling, or could you use a more recent VS version?
Currently my idea is that the library should be buildable on a still-supported version of Windows running the dev tools available for download from Microsoft (currently VS2017).

my python app must running on windows 7 32 bits.
I try to compiling using visual studio build tools 2015, but give me an error, telling me that visual studio 2010 not found and are required.
I think it's because the version of Python I'm using: 3.4.4. I need to use this version because in windows 7 I don't need to install VC++ Redistributable, so I can use cx_freeze to packing and distribute my app without any third party installation.
Did you think in remake your code using cython?
Could this way be more quickly and more "portable" (for missing a better word) your code?
thanks for your return, hope I'm be help in some way..
Samuel

The sad story is that Python 2.7 on Windows does not support using C99 as it is hard coded to use a specific compiler (http://aka.ms/vcpython27).
I don't plan on porting this library from C99 to something that the obsolete VC++ 9.0 supports. The unfortunate fact of this is that Python 2.7 is not, and will not, be supported on Windows.
Contributions are of course always welcome, but I am documented this as a known limitation for now.


Use TunIn with french radio

Thank you for reporting an issue with AlexaPi.
Make sure you are running the latest version of AlexaPi, and look through issues before submitting.
Check out our Wiki, and make sure to look at Debugging, and Audio debugging in particular before filing an issue.
In order to help troubleshooting, be sure to include the following information:
Output of the audio debugging script.
Your OS (including version) where you are running AlexaPi:

Your hardware platform and model you are running on:

Python release (python2 --version):

Description of problem:

Expected:

Problem-relevant config.yaml entries:

Steps to reproduce:

Traceback (if applicable):

Additional info:

Sorry.
Is it possible to use alexa TuneIn with french radio? it' great with United Kingdom but my favorites are in french. I can't see language to change in configuration files. Thanks


Npm Package Missing Dependency: react-timer-mixin

https://github.com/leecade/react-native-swiper/blob/890c0ce64e2192c2ca7830e6699f67b88171e74b/src/index.js#L20
It appears react-timer-mixin isn't being included as a dependency and required above.

hi @cuongvo
react-timer-mixin is provided by react-native.
see: https://github.com/facebook/react-native/blob/master/package.json#L57
I guess you forgot run react-native init project before?
Anyway I will improve the problem in README.md later.

@leecade Thanks for replying! Maybe I'm misunderstanding how npm dependencies work, but what you linked shouldn't resolve the require in your package -- only react-native. I think it's working due to the default packager.
If you use something else to bundle your dependencies and handle transpilation (such as react-native-webpack-server) it looks to break.

:+1: on this, you should specify the dependency now

I'm having this exact issue at the moment, is there a temporary fix?

Looks like react-native doesn't include react-timer-mixin any longer.
@tomtang539 I was able to continue using Swiper by installing react-timer-mixin myself, using the command npm install react-timer-mixin --save (--save to add it to the package.)

Issue should be re-opened. As @paulmederos mentioned, it needs to be installed manually. This is on react-native 0.11.x


forbidden: User "system:anonymous" cannot list resource "workflows"

What version are you running?
v3.3.6
After loging into argo workflows UI,  i see this  error in workflows tab :
Forbidden: [workflows.argoproj.io](http://workflows.argoproj.io/) is forbidden: User "system:anonymous" cannot list resource "workflows" in API group "[argoproj.io](http://argoproj.io/)" in the namespace=...

level=warning msg="finished unary call with code PermissionDenied" error="rpc error: code = PermissionDenied desc = [workflowtemplates.argoproj.io](http://workflowtemplates.argoproj.io/) is forbidden: User \"system:anonymous\" cannot list resource \"workflowtemplates\" in API group \"[argoproj.io](http://argoproj.io/)\" in the namespace \"\"" grpc.code=PermissionDenied grpc.method=ListWorkflowTemplates grpc.service=workflowtemplate.WorkflowTemplateService grpc.start_time="2022-09-06T11:49:11Z" grpc.time_ms=22.017 span.kind=server system=grpc
level=warning msg="finished unary call with code PermissionDenied" error="rpc error: code = PermissionDenied desc = [workflows.argoproj.io](http://workflows.argoproj.io/) is forbidden: User \"system:anonymous\" cannot list resource \"workflows\" in API group \"[argoproj.io](http://argoproj.io/)\" in the namespace \"p\"" grpc.code=PermissionDenied grpc.method=ListWorkflows grpc.service=workflow.WorkflowService grpc.start_time="2022-09-06T11:49:11Z" grpc.time_ms=23.442 span.kind=server system=grpc

I am sure that the rolebinding for SSO SA has been defined properly , because i also tried to login with SA's token , and it works and it has proper access to workflows.
Logs from the workflow controller:
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}
age=5m0s err="<nil>" instanceID= labelSelector="![workflows.argoproj.io/phase,!workflows.argoproj.io/controller-instanceid](http://workflows.argoproj.io/phase,!workflows.argoproj.io/controller-instanceid)"

Logs from the workflow server:
level=info msg="selected SSO RBAC service account for user" email="" loginServiceAccount=argo-admin-user serviceAccount=argo-admin-user ssoDelegated=false ssoDelegationAllowed=false subject=e2jyg5bf3-7ec6-0005-acf7-69332gg98626
level=info msg="finished unary call with code OK" grpc.code=OK grpc.method=GetUserInfo grpc.service=info.InfoService grpc.start_time="2022-09-06T13:29:57Z" grpc.time_ms=2.585 span.kind=server system=grpc

Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Check your anonymous SA's permissions on workflows.

But i got the correct serviceaccount after loging into argo with cloudidp , so why it checks the SA's permissions and not argo-admin-user (my SA)?

hello @alexec
About my SSO serviceaccount issue , i mentioned that in UI user info i see the correct SA (which in my case is argo-admin-user) so i got the correct SA , so i don't login anonymously. If i login with argo-admin-user bearer token , i have correct access without mentioned error.
My SSO and SAs configuration is like this but i get this error in UI when i login with SSO:
forbidden: [workflows.argoproj.io](http://workflows.argoproj.io/) is forbidden: User "system:anonymous" cannot list resource "workflows" in API group "[argoproj.io](http://argoproj.io/)" in the namespace ""

This is part of the role which binded to my SA:
- verbs:
      - get
      - list
      - watch
      - create
      - update
      - patch
      - deletecollection
      - delete
    apiGroups:
      - [argoproj.io](http://argoproj.io/)
    resources:
      - workflows
      - workflows/finalizers
      - cronworkflows
      - cronworkflows/finalizers
      - workfloweventbindings
      - workfloweventbindings/finalizers
      - workflowtaskresults
      - workflowtaskresults/finalizers
      - workflowtasksets
      - workflowtasksets/finalizers
      - workflowtasksets/status
      - workflowtemplates
      - workflowtemplates/finalizers
      - eventsources
      - sensors
  - verbs:
      - get
      - list
      - watch
    apiGroups:
      - [argoproj.io](http://argoproj.io/)
    resources:
      - clusterworkflowtemplates
      - clusterworkflowtemplates/finalizers

I also defined read-only default SA like bellow (i think @terrytangyuan meant this by anonymous SA ) and related RB, but was not helpful:
  name: user-default-login
  annotations:
    workflows.argoproj.io/rbac-rule: "true"
    workflows.argoproj.io/rbac-rule-precedence: "0"

Do you have any idea to fix it?

@terrytangyuan might be a similar issue as with my question here: https://github.com/argoproj/argo-workflows/discussions/9496
I think it might be an issue in combination with OpenShift. But I am not sure.


DOC: clarify cluster-based permutation testing

closes #10595
this is a massive PR to update the docs with all I have learned in:

#10595
#10602
#10604

I will also want to update references to the new plot_ch_adjacency from #10633 once that one is merged.
I think I also identified and fixed several typos and/or mistakes, but I would be very happy for some thorough reviews to ensure that I don't introduce false facts.

I'll get back to reviewing later today or tomorrow but I'm happy to see this happen. One PR at a time we'll make cluster-based permutation testing in mne more straightforward!

@sappelhoff can you please ping me once you think you're pretty much done here? I can then take a look at the result and give feedback :)

@sappelhoff can you please ping me once you think you're pretty much done here? I can then take a look at the result and give feedback :)

@sappelhoff can you please ping me once you think you're pretty much done here? I can then take a look at the result and give feedback :)

@sappelhoff can you please ping me once you think you're pretty much done here? I can then take a look at the result and give feedback :)

@hoechenberger I made a pass over comments by @mmagnuski and @drammock now, so I'd be ready for a new round of reviews :+1:

Do you think these changes can be easily backported?

I don't know -- I don't think I have ever backported something :)
Would it be as easy as "cherry picking" the squashed commit of this PR (when it lands in main) and adding it to maint/1.0, and then making a release from the maint/1.0 branch?

Thanks @sappelhoff!


DBZ-3626 Renaming master branches to gh-pages/main

https://issues.redhat.com/browse/DBZ-3626

@uidoyen, this one is a bit odd: the preview env has not been destroyed, still the comment above was added. Something wrong with checking return values of the Surge execution perhaps? I am going to tear down that env manually now.


Fix issue #54

Description
Adds an alarm to indicate that the tank is about to overflow
Fixes # 54
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

Reviewer: Vinit Shahdeo

@jainpawan21 @ArpitKotecha please review and merge if all is well.

@IuliaElizaS Please resolve the conflicts.


Shadow service typename filter

Code look good to me but I missed some test showing side effect of when isShadowingEnabled.


MM-11848 Allowing clicking on username to open profile popover for webhook posts.

@crspeller Can you help cherry-pick this to v5.9?


[client] Fix #19351 - vFillBuffer fix uint16 summation overflow.

Problem
Fix #19351
NCC-E003350-HRX
uint16 overflow issue in vFillBuffer() flagged by security audit.
Change overview
suggested fix
Testing
CI

client-api.cpp was removed alltogether


[Bug] Custom Distillation MMSeg CWD loss nan problem

Describe the bug
I am training the segnext_l model as a standard teacher on my data and keeping the checkpoint obtained for distillation(mmseg/cwd) from segnext_l--->segnext_tiny. When doing this after starting few iterations i am getting all losses as nan for all upcoming iterations.
I am using all the latest versions .

The results of student model also remains 0.

 After I doing several times experiment, I think this is because the lr rate is too large which makes gradient explosion. In mmrazor schedules modules, they do learning rate warm up, which will first make lr larger and the reduce lr by weight decay. So I change my learning rate number(which make it smaller) it works. I hope this can help you.


Zoom and scrolling in drawing mode

Same as in Legacy version: https://github.com/ARK-Builders/ARK-Retouch-Legacy/issues/14

Implemented in #83


Exclude RandomTestChiSquared Tests

Exclude RandomTestChiSquared Tests
Related issue https://github.com/eclipse-openj9/openj9/issues/13202
Signed-off-by: Tobi Ajila<EMAIL_ADDRESS>

@llxia Please review these changes


Add "Apache-2.0" license?

Should this project add the "Apache-2.0" license as a dual-license (for protection against "patent aggression")? "MIT/Apache-2.0" (equivalently, "MIT or Apache-2.0") is the license used by rust itself.
ref: https://opensource.stackexchange.com/questions/217/what-are-the-essential-differences-between-the-bsd-and-mit-licences
ref: https://dwheeler.com/essays/floss-license-slide.html @@ http://archive.today/4QHQ0
ref: https://github.com/dtolnay/rust-faq#why-a-dual-mitasl2-license

I would argue against this and rather argue in favor of adding a GPT license for the sake of retaining the work committed to this project instead of having it concealed by the works of proprietary software.

I think a project like this will never gain traction without a GPL license. What is the incentive for people to donate their time when there is no assurance that big corporations must also do the same?

With more than 150 contributors, I think the ship has sailed for the license.
Sorry!


[test] Add OSU Allreduce test

we might do it more general and summarise alltoall and allreduce to a single check with parameters in osu_tests.py

@ajocksch The test is failing on both Daint/Dom. Please have a look and run manually.


LDAP for local auth

I am slightly put off by a tool that purports to be a self-hosted mesh network solution requiring a hosted 3rd party to auth against.
Why can't we use simple LDAP against a local AD?

Have you looked into self-hosting an IdP that supports auth via LDAP, e.g. Keycloak, Zitadel?


Chore/Vertically center text labels in MuiDrawer side menu

Summary
This pull request addresses vertical alignment issues in the side menu (MuiDrawer component) by making the following improvements:

Vertical Alignment of Side Menu Button Labels

Fixed the vertical alignment of labels, such as 'Agentflows', which were not centered within their containers.
Applied automatic vertical margins to <ListItemText> and added a 4px vertical margin to the primary text element <Typography> to maintain existing margins.

Vertical Alignment of Side Menu BETA Chips

Corrected the vertical alignment of 'BETA' chips in the side menu.
Ensured that the chips are consistently centered regardless of the presence of captions in the corresponding text labels.

Adjust Margin for Secondary Labels (a.k.a., captions)

Added mt: -0.6 to the secondary (caption) label to replicate the original gap between the primary and secondary labels.

Before and After Screenshots

Before
After

Before and After Screenshots with Added Caption Labels

Before with Caption
After with Caption

Detailed Changes

Updated <ListItemText> components with sx={{ my: 'auto' }} to automatically center the nav button labels vertically.
Modified <Typography> elements within <ListItemText> to include sx={{ my: 0.5 }} for consistent vertical margins, replicating the 4px vertical margins that existed before this change.
Updated 'BETA' chip elements with sx={{ my: 'auto' }} to automatically center them vertically.
Added mt: -0.6 to the secondary (caption) label to replicate the original gap between the primary and secondary labels.

Additional Information
These changes enhance the visual alignment and consistency of the side menu items, improving the overall user experience.

Not sure what three-word information would fit in there as a useful caption for the user, but if this change goes ahead, I would prefer captions to appear on hover for a minimalist approach.

@knakamura13 I see, that makes more sense.
A change I would pay for is to add *{ border: 0; } as a feature for the entire app. It would look sleek.
There is still some room for visual improvement here.
Thank you for your work on this.


Windows R Setup should also include JDK installation?

I just ran into an issue on my Windows 10 laptop where I was trying to set up a Strategus module and renv was unable to install rJava.  Installing rJava outside of renv::restore() worked fine.
The error message seemed to point to the absence of the Java JDK as the reason rJava could not be installed (which renv does from source by default).  I was able to solve this issue by installing the Amazon Corretto 8 JDK.  I had previously only installed the JRE per HADES setup instructions: https://ohdsi.github.io/Hades/rSetup.html#Installing_Java
The reason this is an Issue and not a PR is that I'm not sure that we want all users to install the JDK, or if this is a better recommendation just for Strategus users, or if there's another better solution :)

Would you happen to have saved the error message?

Unfortunately not. But it was related to being unable to find javac. When I looked into this, I gathered that javac was needed in order to install rJava from source (which is what renv was doing), and that in order to get javac I needed the JDK.


The CustomValidationPopup should have a property to prevent opening on Keyboard focus

Describe the feature
Sometimes the UI gets disturbed by opening the Popups frequently. An AttachedProperty like ShowValidationErrorOnMouseOver for the HasFocus property could be useful here. The Popup can still be shown on MouseOver
Additional context
Idea:
ShowValidationErrorOnKeyboardFocus= true | false ;  // The default value would be true like the current behavior

I can implement this when the other things are implemented. I don't want to spam too many PR at a time.
Happy coding
Tim

@timunie Any chance to implement this now?


PR: Use get_python_executable to run in external terminals

Fixes #5165.
Attempt to fix #5165 based on the suggestion to use get_python_executable() from @jitseniesen
I tried to run the tests but got errors which seem unrelated to my changes (they also occur in clean 3.x branch.)

Thanks for your contribution @cknoll! I'll test it locally, and if it works well, this will be part of 3.2.3, to be released tomorrow.


Bug found in newest preview: Object Microsoft.SqlServer.Management.Common.ServerConnection not working?

Hello,
I found a potential Bug, in the Preview Version 22.0.49 of the SQLServer.
We updated our SQLServer-Module Version, so we can automatically deploy SQL2022 Instances, after that our Script to add newly created instances to our CMS suddenly  didnt work anymore.
It seems like there is a Problem with Object:
Microsoft.SqlServer.Management.Common.ServerConnection
OR
Microsoft.SqlServer.Management.RegisteredServers.RegisteredServersStore.
When I try to create the RegisteredServersStore-Object I get an Error Message, that the connection failed to server "System.Data.SqlClient.SqlConnection"
The reason for this, is the Object Microsoft.SqlServer.Management.Common.ServerConnection which i gave as parameter to the constructor.
When using the old SQLServer Module 21.1.18256 the script runs without a Problem!
I attached 2 Txt-Files that shows the code we ran and the output of Microsoft.SqlServer.Management.Common.ServerConnection Object (called in $cnn in the Script). In the preview Version its pretty empty, while in the current stable version the Parameters are filled with information.
Bellow the output you see the results of trying to create the registeredServers-Object. In the preview it fails, in the stable version it works without a problem.
Using_SQLServer_Preview.txt
Using SQLServer21.1.18256.txt

@acul99 -- thank you for your well-researched feedback.  Please add this here:

SQL Server feedback

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Add onions!

Add onions and corresponding instructions to the following sections:

Ingredients
Instructions

Yes!  It does need onions


Loading a quantized TFlite model fails when allocating tensors

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.2.0-dev20200407
Python version: 3.6 / 3.8
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the current behavior
Loading a tf.keras model to tflite causes an error when trying to allocate the tensors for inference. The error is:
RuntimeError: tensorflow/lite/kernels/conv.cc:337 bias->type != kTfLiteInt32 (9 != 2)Node number 1 (CONV_2D) failed to prepare.
Describe the expected behavior
The exported model should load without error. Inference with the same model in tensorflow is working.
Standalone code to reproduce the issue
Colab gist reproducing the error can be found here Slight modifications to the network structure (ie removing random layers) make the error go, but without any understandable scheme.
Other info / logs
RuntimeError                              Traceback (most recent call last)

<ipython-input-2-6b478363f7a3> in <module>()
     68     interpreter = tf.lite.Interpreter(model_path="model.tflite")
     69     input_details = interpreter.get_input_details()
---> 70     interpreter.allocate_tensors()
     71     print("DONE")

/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/interpreter.py in allocate_tensors(self)
    241   def allocate_tensors(self):
    242     self._ensure_safe()
--> 243     return self._interpreter.AllocateTensors()
    244
    245   def _safe_to_run(self):

RuntimeError: tensorflow/lite/kernels/conv.cc:337 bias->type != kTfLiteInt32 (9 != 2)Node number 1 (CONV_2D) failed to prepare.

@moberweger,
I am unable to open the Colab link you have provided as I do not have access to it.
Please go to File -> Save a copy as a GitHub Gist and share the link of the new gist created with us. Thanks!

@amahendrakar Thanks for the notice. I updated the original issue with the new link

Was able to reproduce the issue with TF-nightly i.e. v2.2.0.dev20200410.
On running the above code with TF v2.2.0rc2, facing an error stating AttributeError: 'TFLiteConverterV2' object has no attribute 'experimental_new_quantizer'
Please find the attached gist. Thanks!

Hello, and thank you so much for your time! Unfortunately, we are running into the exact same issue. Has there been a workaround/fix yet?

I have tried in colab with TF nightly version(2.4.0-dev20200807) and was able to reproduce the issue.Please, find the gist here.Thanks!

Facing the same issue in both TF 2.3.0 and TF nightly version(tf-nightly-2.4.0.dev20200922) for my model involving Conv1D layers. Are Conv layers not yet supported for tflite or is it something else?
Is there a way to resolve this or do we need to wait till the next tf stable release (TF 2.4.0).
Please help.

@moberweger Looks like there was some in compatible dtype issue. The following is the error .
RuntimeError: tensorflow/lite/kernels/conv.cc:350 bias->type != kTfLiteInt32 (INT8 != INT32)Node number 1 (CONV_2D) failed to prepare.
When I remove BatchNorm layer, it worked without any error.  I remember there was some issue related to BatchNorm but it was long time back. Need to check that. Thanks!

@tromedlov22 Please share a simple standalone code to reproduce the issue. Thanks!


[KYUUBI #5925] Kyuubi TPC-DS support running benchmark with skipping some queries

:mag: Description
Issue References 🔗

When running Kyuubi's TPCDS, some SQL runs slowly, but there are no parameters to skip it.
Describe Your Solution 🔧
Add the skip parameter, specifying a comma-separated list of SQL
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
no parameters to skip it.
Behavior With This Pull Request :tada:
$SPARK_HOME/bin/spark-submit \
  --class org.apache.kyuubi.tpcds.benchmark.RunBenchmark \
  kyuubi-tpcds_*.jar --db tpcds_sf10 --skip q2,q4

== QUERY LIST ==
q1-v2.4
q3-v2.4
q5-v2.4
q6-v2.4
q7-v2.4
q8-v2.4
q9-v2.4
.....

Related Unit Tests

Checklists
📝 Author Self Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

[ ] Pull request title is okay.
[ ] No license issues.
[ ] Milestone correctly set?
[ ] Test coverage is ok
[ ] Assignees are selected.
[ ] Minimum number of approvals
[ ] No changes are requested

Be nice. Be informative.

how about changing filter to include and skip to exclude? And we should define the priority of these two options

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

Comparison is base (c6177ab) 61.57% compared to head (13744e5) 61.57%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #5925   +/-   ##
=========================================
  Coverage     61.57%   61.57%
  Complexity       23       23
=========================================
  Files           616      616
  Lines         36388    36388
  Branches       4979     4979
=========================================
+ Hits          22405    22407    +2
+ Misses        11568    11561    -7
- Partials       2415     2420    +5

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

how about changing filter to include and skip to exclude? And we should define the priority of these two options

It sounds great, but now there is also the queries parameter. Its function is similar to filter. Filter can only include one SQL, and queries can include multiple SQLs.

Regarding priority, I think exclude should always take precedence over include.

would it be clear if we replace the queries and filter with include(list) and exclude(list)? it's kind of a developer tool, such breaking changes are acceptable.
for priority, especially when include and exclude are present at the same time， there are some candidates:

the final result is include minus exclude
as you proposed, simply ignore the include

I lean to option 1, but both are fine, as long as we document the behavior clearly

It's cleaner to just keep include and exclude in the end. I can make changes.
My previous description of priority may have some misunderstandings. The meaning of exclude taking precedence over include is the same as option 1.

that's fine, please go ahead

LGTM, only minor comments

UT failure is unrelated, thanks, merged to master

@rhh777 BTW, you may want to link your email<EMAIL_ADDRESS>to your GitHub account, otherwise, your contribution will not be counted by GitHub :)

@rhh777 BTW, you may want to link your email<EMAIL_ADDRESS>to your GitHub account, otherwise, your contribution will not be counted by GitHub :)
Thanks, I'll adjust it later.

BTW, it seems that 📝 Committer Pre-Merge Checklist is not welcomed at all. @pan3793

BTW, it seems that 📝 Committer Pre-Merge Checklist is not welcomed at all. @pan3793

Is it my job to fill in this list? I wasn't sure before.

thank you for your contribution and there is extra effort for you. @rhh777
this is a reviewer's field

@rhh777 it's my responsibility to fill in the committer checklist, you just need to fill in the "Author Self Checklist"

BTW, it seems that 📝 Committer Pre-Merge Checklist is not welcomed at all. @pan3793

TBH, the checklist is too long, especially when there are images in the description and the network is slow, selecting the long checkbox makes the screen flicker many times. I prefer to the previous brief template, only listing a few most important items and leave others to scripts, e.g. checking assignees, milestone are selected.

BTW, I recommend avoid using emoji but prefer plain text in PR title and description, it doesn't render well on some platforms


Unable to use Hash#to_a in certain case

in /opt/crystal/src/hash.cr:190: no overload matches 'Array({String+, Int32})#<<' with types {String, Int32}
Overloads are:
 - Array({String+, Int32})#<<(value : {String+, Int32})

      ary << {key, value}
          ^~

I don't understand, why {String, Int32} doesn't count as {String+, Int32}..

@waterlink It's because of this bug. You can use the workaround of using record in the meantime until we implement it :-)

I see. But in that case tuple is formed by Hash#to_a, so I have no control over it..

I'm using workaround with Hash#each and accumulating in local variable (yuck).

I'm closing this because it is a duplicate


Error when adding source

New to grafana, so this may be a rookie question. I see 4 entries for the MQTT data source in the configuration view (default, 1, 2 and 3).
I built mqtt-datasource but got the following error in grafana when I tried to add it. No obvious grafana logs to share.
Running on Ubuntu 20.04, not using a container. Grafana v7.0.
An unexpected error happened
Details
Error: Minified React error #130
in form
    in tm
    in Unknown
    in div
    in t
    in form
    in div
    in div
    in t
    in div
    in div
    in div
    in t
    in t
    in div
    in t
    in t
    in x
    in ConnectWithCleanUp(Connect(t))
    in r
    in Unknown
    in t
    in Unknown
    in m
    in Unknown
    in Unknown
    in t
    in t
    in s

@whbruce do you want to maybe try with Grafana v8.0?
for a streaming backend datasource plugin like this one a pre-requisite is Grafana v8.0+. see: https://grafana.com/tutorials/build-a-streaming-data-source-plugin/
will add this to the docs 😬

Thanks updating to v8.0 fixed it.


Potential bugs in apps/speed.c, which caused by missing error code checkings

HMAC_Init_ex, HMAC_Update, HMAC_Final are hash functions used for message authentication, which is based on a hash function. The return value of them contains the error code, where 0 is for failure. It seems that the return value should be checked carefully.
In method HMAC_loop() of apps/speed.c, the error checks of return values are missed.
apps/speed.c
static int HMAC_loop(void *args)
{
    loopargs_t *tempargs = *(loopargs_t **) args;
    unsigned char *buf = tempargs->buf;
    HMAC_CTX *hctx = tempargs->hctx;
    unsigned char hmac[MD5_DIGEST_LENGTH];
    int count;

    for (count = 0; COND(c[D_HMAC][testnum]); count++) {
        HMAC_Init_ex(hctx, NULL, 0, NULL, NULL);
        HMAC_Update(hctx, buf, lengths[testnum]);
        HMAC_Final(hctx, hmac, NULL);
    }
    return count;
}

Besides, we find all the other call sites of these methods have been check.
This problem exists in many versions:
OpenSSL_1_1_0-stable
OpenSSL_1_1_1-pre6-8
Zuxing Gu, Jiecheng Wu

The speed.c is reworked to use EVP_MAC for HMAC and there is error checking in master.


Make prep Phony

When I was setting up the Makefile for wren-io, I realized prep wasn't PHONY.

:+1:


FALCON-2283 Issue with jersey multi part with mime type 1.6 version

@sandeepSamudrala Can you elaborate on your analysis of what was the underlying cause and how this change fixes the issue?

Nevermind. I think I found the probable issue. I believe it is something on these lines. For the benefit of others it seems the issues is: https://java.net/jira/browse/JAX_WS-965 and fix is on the lines of http://www.javaroots.com/2013/09/jersey-rest-service-stream-already.html
Please, try to add as much details as possible for bug fixes. It not only helps in reviewing, but will also help users who might be affected by the same issue.
Can you please add a test for the same? A test which will fail without the fix and pass with the fix.

@ajayyadava : There are test cases for the same in ExtensionManagerIT, but unfortunately they are disabled because of which this was not caught in tests.
There is a jira already parked for the same
https://issues.apache.org/jira/browse/FALCON-2262

@ajayyadava : Do you still have any concerns? If not would like to get this merged, as this is blocker for the release.

+1 Thanks @sandeepSamudrala

Thanks @sandeepSamudrala , @ajayyadava


Added a hidden Save button, to change the default Enter action

Since pressing Enter while in a text field of an HTML form submits via the first submit button, adding a hidden one will cause it to be used instead of the first visible one. Result: Enter saves the form without hitting the Save as New button.

Thanks for the patch. What about using tabindex instead?
https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/tabindex

I tried tabindex. While it does prevent the user from tabbing to the other submit buttons, or change the order in which tabbing occurs, it does not in any way change what happens when the user hits Enter while a text field has the focused.. i.e. the "default" submit button in use.

That's  a pity, choosing the first submit button as default without respecting tabindex is a bit lame though.


[JOSS Review] ipywidgets dependency

The documentation is generally excellent.
I only tested the basic_tutorial.ipynb file, but I can confirm it ran successfully and the outputs are virtually identical to what's shown in the documentation.
One note that may just be an artifact of me running the notebook in VS Code:  I had to install ipykernel to get the code to execute, and some of the outputs wouldn't display without installing ipywidgets.

Fixed by https://github.com/tvwenger/bayes_spec/pull/49


[0.5.1] Fix & Updates for windows and crates

This pull request "fixes" issue #9 and allow new version to be pushed on crates.
This also fix a critical issue making unable to build or run on windows.
(I took the opportunity to update the dependencies)
I've tested build on windows and it was working.
I also hesitate to delete show_debug since it is deprecated...
Notes:
chore: update dependencies
fix: main function wasn't returning anything on windows config.
fix: notification wasn't sent on windows.
bump version to 0.5.1

While waiting for the pull request to be merged, you can install toastify this way:
cargo install --git https://github.com/GGLinnk/toastify.git

I fixed this on my own branch, since I was not able to roll back the version change that was included here
thank you for the help :+1:


Upon job failure job data is not removed from disk

When a job fails, the job logs from the disk is not deleted. It would nice to have a clean job button as well.

I'm not sure what you mean. There are "Delete job" buttons on the home page and on the job page. What else are you looking for?

Thanks, I wanted the default behavior to be clean the job logs after job finishes. There may be others who might want to tweak the network. If there is an auto clean method, I can setup multiple jobs without worrying about disk being used up.
Hope that helps.

Hmm, that's the opposite of what I usually want. I usually want all information possible to tell me what went wrong. Thanks for explaining, I'll mark this as a feature request.

I do want the debug logs to be available. However, the folder within which the job data is stored grows quite large. Does Digits create the images again? Is there a possibility of compression?

On second thought, we're not going to do this. Closing.

However, the folder within which the job data is stored grows quite large. Does Digits create the images again? Is there a possibility of compression?

Try storing JPEGs in your LMDBs instead of PNGs. Your datasets should take up much less space.

Also, you may be interested in #405.


Reduce the amount of layers docker uses

What does this PR do?
This PR changes the docker image to use 7 layers instead of the current 9 and reduces the size of the image.

👍


maximum number of search results displayed

currently, maximum number of search result to be displayed is not specified.
The storyboard doesn't mention pagination.
The view shall be specified more in detail.

As we're using js already anyway, dynamic loading (while scrolling down) is the nicer way to go, as it doesn't interrupt the interaction to load the next page. The best would be to start with 1.5 screen-heights worth of results.

hmmm i agree with the dynamic loading on mobile devices, but when we view the page on a pc, i am not sure how it should look, because the post details shall also be displayed on the right of the screen?
should the post detail view slide down together while scrolling?

Isn't the simplest solution to dynamically create a new div next to the search result when a user clicks on it?

@fkleedorfer hmmm yeah but that's not what we were talking about..
we were talking about loading the thumbnails on the left side.
how many thumbnails are to be displayed
and whether to do it with dynamic loading as the user scrolls down or via pagination

I meant this to be used together with dynamic loading while scrolling.

old ui -> search results not applicable -> close


SAK-41886 Gradebook Classic is not listed in tools to import in Site …

…Info -> Import from site

added a commit to only import one of the gradebook tool's content
Sorry, I'm not sure if this was the correct way to do it, but I also created a seprate PR to merge this additional fix into 'master'
https://github.com/sakaiproject/sakai/pull/7035

Thanks for addressing the issue


SPARQL AST editing in spargebra

We have internally implemented a few extensions to spargebra, to allow us in building and composing SPARQL queries, that we could possibly upstream.
@Tpt I'm wondering if that is something that you would like to be a part of spargebra? If so, where would be a good place to put it? (A new modulespargebra::algebra::edit?). I could also understand if you feel that it's best kept separate as that is possibly an endless topic.
One example would be the RenameVariable trait:
/// Trait that allows to recursively rename all occurrences of a variable.
pub trait RenameVariable {
    /// Replaces all nested occurrences of the variable `old_name` with the variable `new_name`.
    fn rename_variable(&mut self, old_name: &str, new_name: &str);
}

Nice! Thank you!
Yes to include well defined operations in spargebra. If it's general use like variable renaming it might even be nice to include them as inherent method instead of traits. I am planning to implement a query optimiser inside of spargebra or an other related crate and having manipulation methods would be very convenient.


Feature: Generic pre-post hook commands

Supporting generic pre/post hook commands would allow users to employ their existing infrastructure/scripts - rather then have to re-write everything in Rhai.
Enforced behavior would be to treat prompt_pre and prompt_post as returning false.  Template authors don't get to override the type nor the default setting for prompt_pre and prompt_post.
Default  prompt_pre and prompt_post strings could be:
....
prompt_pre = "Do you agree to execute pre-setup commands for `cargo generate /what/was/given here`? (y/n)"
prompt_post = "Do you agree to execute post-setup commands for `cargo generate /what/was/given here`? (y/n)"
....

Example:
[hooks]
prompt_pre = "Do you agree to execute xyz's pre-setup commands? (y/n)"
prompt_post = "Do you agree to execute xyz's post-setup commands? (y/n)"
pre_commands = [["rake", "init", "&>generate-pre.log"], ["make", "config", "&>>generate-pre.log"]]
post_commands = [["redo", "all", "&>generate-post.log"], ["make", "install", "&>>generate-post.log"]]

YEAH!.... templates becomes platform dependent.... TBH I am not a big fan of this one - if I really wanted to do it, I'd just utilize #598 from the existing functionality.
... Sorry if it came out a little harsh.

I think something that cannot be controlled by the template author are the prompts.
[hooks]
prompt_pre = "Do you agree to execute xyz's pre-setup commands? (y/n)"
prompt_post = "Do you agree to execute xyz's post-setup commands? (y/n)"

for security and safety reasons those must come from cargo-generate, whenever pre/post commands are given in the configuration of a template.
Also, I would think the safest way would be to present the user before the given command is executed with the specific command that is going to be executed. So that he actually can see what is going to happen and confirms that then.
The rust std lib has the command API which we could feed the args into. So that we could loop over the array of commands, and take the first entry as the command, and all following values as their arguments and prepare it accordingly.

Per @sassman 's observation in PR #611  - just because we're generating rust projects doesn't mean we're tied or get any particular benefit from using cargo-generate, and our use case probably reguires one of the 3-rd or 4th generation generators.... we'll keep an eye on cargo-generate.  Thanks again for your efforts.

and our use case probably reguires one of the 3-rd or 4th generation generators

I would love to understand your particular use case a bit better. What exactly do you mean with the 3rd or 4th generation generators? And what feature in particular is it they bring to the table?


Multiple Entities Extract

Hi!!
I have loaded intents in GUI of rasa ..Suppose I have a text saying "I want to play Badminton on Monday and Friday" and I highlight Monday and Friday to Days Entity. How should I extract days in a list format in tracker in actions.py? to process further?
Also, last entity is coming I think its getting overwritten.
Kindly help

You can get them using get_latest_entity_values(entity_type)

I'll close this for now, please let us know if you have any further questions.


[FEATURE-REQUEST] Record just audio

How do I record just audio and not the video too? The docs (https://docs.daily.co/reference/daily-js/instance-methods/start-recording) show the following command that records video as well. I do not want to turn off the displayed video, just not record it.
call.startRecording({
  width: 1280,
  height: 720,
  fps: 25,
  backgroundColor: '#FF1F2D3D',
  layout: {
    preset: 'default',
    max_cam_streams: 5,
  },
});

Hey @cinjon,
this sounds like you'll want to record using the raw-tracks configuration.
While this might not be exactly what you're asking for, it's the closest Daily is currently offering.
With raw-tracks recordings you'll receive individual media tracks in your custom S3 bucket. From there you'll be able to post-process the audio tracks into a single audio recording.
I'll close this issue for now, as this is a more general question about Daily as a platform and not necessarily a feature request for the Daily React library.
I'd like to invite you to ask similar questions in our community or checkout other questions that were already posted there.
Best,
Christian


Updating the configuration in runtime

It would be nice to have a logic to update configuration parameter at runtime without a re-deploying application.

This is now implemented, as Application Parameter updates (Application Update, no process restart) in develop branch:
https://github.com/microsoft/service-fabric-observer/tree/develop


Call python scripts as argument to virtualenv python command

This fixes #994, where the 127 character limit on shebang line lengths was
triggering failures of commands inside virtualenvs that had long names or
long branch names. This PR adds some path environment variable handling to
allow for shorter commands, and makes all python script call wrapped by a
call to the virtual env symlinked python binary.
This depends on the pending pull request #1524, for adding docker
containerization and should be rebased after that PR is merged.

:+1: other than the above nit.

:ship:

Very concise PR which makes it easy to review. :+1: Good work.


The table did not show if not refresh the page.

My table only displays when refreshing the page. The table did not show if not refresh the page.  You can see the issue in the video below. I try to open this page by using the route name or typing in the search bar but it does not work.
How to fix it?
https://user-images.githubusercontent.com/59549741/162574429-b886b1a2-0cd0-4f04-999a-77f9472f9934.mp4

I don't know. Is my issue related to https://stackoverflow.com/questions/65807514/flutter-widget-is-not-showing-its-showing-in-hot-reload ?
But I try to use state management and It's working!
I can use setState to reset the UI.
However, if you have some ways to fix that, please give me.

A minimal sample code is required to verify your problem.
I would appreciate it if you could register a new issue by attaching a sample code to check the issue.


Add comments to the code to help newcomers

Please add descriptions to the structs, traits and macros in gobject-subclass code.
Thanks :)

That's definitely a good idea! Until that point, what are you trying to do? Maybe I can link you to some example code then for the time being :)

I need to implement GListModel and gtk_list_box_bind_model method for gtk-rs.
I'm not very experienced in GObject, but would love to learn more. :)

Once I understand how things work, I would like to write comments for gobject-subclass too.

Ah that one is going to be tricky in any case because of all the gpointer / void * usage in the C API.
In gtk-rs, this probably has to become a generic type ListModel<T: StaticType + FromGlibPtrNone + ToGlibPtr>, as completely manual bindings. Same thing goes for GListStore. It's very bindings unfriendly API, I don't think any of this can be autogenerated.
As a first step you'll have to start with the bindings part in the gio and gtk bindings. And can then make the GListStore work first. After that implementing everything for subclassing should be relatively straightforward, but the other part has to happen first.

You would have to write manual bindings for that too, around a FnMut. There are some like that in glib/src/source.rs IIRC, take those as an example.
But the important part here is that you somehow need to make the gpointer everywhere into a generic type parameter. You can take a look at gstreamer/src/iterator.rs for some code doing something similar.

What is the use of struct ClassInitToken(()); and struct TypeInitToken(()); ?

To make it impossible to call the functions that need them as parameter from arbitrary places

Could you give some examples from where it is impossible to call functions with ClassInitToken as a parameter?

ClassInitToken can only be created from inside this crate, and you only get a reference to one passed to the class init function. So you can only (with safe code) call functions taken a ClassInitToken from inside the class init function.

But it is declared public here: https://github.com/gtk-rs/gobject-subclass/blob/f42869195b1022223885889c99f375f9f8e4e4f4/gobject-subclass/src/object.rs#L67
And object.rs is imported as pub mod object here: https://github.com/gtk-rs/gobject-subclass/blob/f42869195b1022223885889c99f375f9f8e4e4f4/gobject-subclass/src/lib.rs#L25

Yes but the constructor is private, it would otherwise have been pub struct ClassInitToken(pub ());

What does Ext stand for in the trait names like ObjectClassExt or ListBoxExt?

They're extension traits for the type in their name, and its subclasses

https://github.com/gtk-rs/examples/raw/master/src/bin/listbox_model.rs is a very well documented example implementation.
Please add a link to this in Readme.md

@happycoder97 Do you want to send a PR to add this to the README.md?

Done :)

The code is now merged into https://github.com/gtk-rs/glib/tree/master/src/subclass in a quit different and cleaned up form, and that has documentation/comments on everything now.
Please open a new issue if there's still something missing there. This repo here is obsolete :)


Infinity loop fix

Fix infinity loop in ExpandableArrayBuffer.calculateExpansion, add reproducer. Fix a bit performance case when second "put" requires more than two iterations for "while", than greatest value passed than more iterations accrue

@mjpt777 Hello ! Could you please review this pull request ? LGTM analysis failed, but i see not for my changes :)

I took a different approach and put the tests in the correct place. https://github.com/real-logic/agrona/commit/d4558507ef03312f8eecba5a992087a66f561bc7

I took a different approach and put the tests in the correct place. d455850

Yep, looks good thx. @mjpt777 But, could you please explain why you still use cycle instead of calculation for new size?
INITIAL_CAPACITY still low enough if you don't know size for next 'put'. As example second 'put' after initialisation may have size x100*INITIAL_CAPACITY  and you still need few iterations for 'while' cycle and do value = value + (value >> 1);  for each.

It keeps the code simple and a few iterations does not take very long. Must buffers are small in size and grow incrementally.

I get your point thx :) But, "simple" it's subjectively and depends on.. For me, cycle always looks more complex that 1+1 :D
Whatever, even if some of JMH benchmarks could show huuuge different between cycle and math approach it's nothing within reallocation new memory. Thx again for attention, have a good day !

You also need to consider the allocation pattern and why we have chosen to grow by golden ratio. Memory fragmentation is a real issue for native, i.e. direct byte buffer, allocation.

Yep, i understand why you chosen this pattern, that why i leaved yours pattern with cycle and added new one. Just my solution for "fast" calculation, "fast" calculation cover only case when you need reallocate more than twice new array. In real world it shouldn't happens often, even i guess it's very rare case.
Ye, i tried to get a formula for 'golden ratio' without cycle but... your "simplification" better :D


Upgrade Unity & TMPro versions, allow fixing numbers and using rich tags at the same time

Changes made in this pull request:

Unity upgraded to 2019.4 and TextMeshPro to 2.1.3. I tested with TextMeshPro 2.1.3, 2.1.1, 2.1.0, 2.0.1 and added the version define so that the package compiles regardless of the TextMeshPro version.
Replaced RTL_OVERRIDE flag with the one from version defines (feature available in Unity 2019.4 or newer)
Updated RTLSupport code to allow fixing numbers and using rich tags in the same text. The number fixing will ignore the numbers inside tags.
Made code more concise (~200 lines less) without a performance hit.
Fixed minor bugs in tag detection, tests, etc.

Thanks for the PR.
Please change you braces style to match the rest of the project. Opening and closing braces should be on a new line:
if (...)
{

}
else
{

}

private void Method(...)
{

}

Other than that, everything looks good to me.

Thanks for the PR.
Please change you braces style to match the rest of the project. Opening and closing braces should be on a new line:
if (...)
{

}
else
{

}

private void Method(...)
{

}

Other than that, everything looks good to me.

Fixed styling.
Probably make a new release from master before this pull request as the final release for Unity versions before 2019.4.

Fixed styling.
Probably make a new release from master before this pull request as the final release for Unity versions before 2019.4.

What makes this version incompatible with Unity versions below 2019.4?

What makes this version incompatible with Unity versions below 2019.4?


fix(expand volume) Display an error message when expansion fails

https://github.com/longhorn/longhorn/issues/1674
Signed-off-by: Siye Wang<EMAIL_ADDRESS>

@Mergifyio backport v1.4.x


Use common transformer

What? and Why?

Make use of sdx-common code to remove code from sdx-transform-cs

Checklist

[ ] CHANGELOG.md updated? (if required)

Codecov Report

Merging #111 into master will increase coverage by 1.29%.
The diff coverage is 80.95%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
+ Coverage   87.48%   88.77%   +1.29%
==========================================
  Files          17       12       -5
  Lines        1390      918     -472
==========================================
- Hits         1216      815     -401
+ Misses        174      103      -71

Impacted Files
Coverage Δ

transform/__init__.py
100% <ø> (ø)
:arrow_up:

transform/transformers/__init__.py
100% <ø> (ø)
:arrow_up:

tests/test_transform.py
86.07% <ø> (-13.08%)
:arrow_down:

tests/test_mwss_transform.py
100% <100%> (ø)
:arrow_up:

transform/views/main.py
66.66% <60%> (-1.37%)
:arrow_down:

tests/test_views.py
56.09% <85.71%> (ø)

transform/views/image_filters.py
25.64% <0%> (-66.67%)
:arrow_down:

Continue to review full report at Codecov.

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

 (5) Update transformers to use common formatters

The thing I've noticed is that a lot of tests are being deleted and it doesn't look like they're being replaced by anything.  Is that because we're using the common transformer and that's already got unit tests?

@insacuri Yes and no, there are unit tests in common that are being used instead, but I think we could do with more tests, but we do have card for that.


LoadError: Could not open library 'glib-2.0'

Hello, this is continuation of an issue i opened on image_processing gem issue tracker (link to original issue). Long story short: i am not able to use ruby-vips on alpine in any possible way. I've tried to set it up using this and this Dockerfiles and also i tried to use apk packages and still no luck. It always ends up in:
Traceback (most recent call last):
	8: from app.rb:11:in `<main>'
	7: from app.rb:11:in `require'
	6: from /usr/local/bundle/gems/image_processing-1.7.1/lib/image_processing/vips.rb:1:in `<top (required)>'
	5: from /usr/local/bundle/gems/image_processing-1.7.1/lib/image_processing/vips.rb:1:in `require'
	4: from /usr/local/bundle/gems/ruby-vips-2.0.13/lib/vips.rb:13:in `<top (required)>'
	3: from /usr/local/bundle/gems/ruby-vips-2.0.13/lib/vips.rb:28:in `<module:GLib>'
	2: from /usr/local/bundle/gems/ffi-1.9.25/lib/ffi/library.rb:99:in `ffi_lib'
	1: from /usr/local/bundle/gems/ffi-1.9.25/lib/ffi/library.rb:99:in `map'
/usr/local/bundle/gems/ffi-1.9.25/lib/ffi/library.rb:145:in `block in ffi_lib': Could not open library 'glib-2.0': Error loading shared library glib-2.0: No such file or directory. (LoadError)
Could not open library 'libglib-2.0.so': Error loading shared library libglib-2.0.so: No such file or directory

I've also prepared a simple docker-based repo to make it able to reproduce it easily. To do so, you can clone it: https://github.com/mbajur/ruby-vips-example
and run:
docker build -t ruby-vips-example .
docker run --rm -ti ruby-vips-example bundle exec ruby app.rb

Please take note i can't be considered as an super experienced unix power-user. I know some stuff but that's probably not enough and it's highly possible i've did some really silly mistake in here - and i'm really sorry if so.
Thanks in advance!

Your dockerfile is using a testing (I think) alpine, not alpine latest. libvips is still not available via apk on alpine (I think), so I built from source.

Yes, I think you are mixing up alpine packages from edge and testing with regular alpine. Until vips (finally!) lands in alpine:latest, it's safest to build your own libvips (I think).

Thank you! Following your example did the trick :)


Player MMR not shown correctly

I recently had a game with average 2.2k MMR.
When I review the game one player is shown as ~1.6k MMR. I know, that it shows the current MMR not the MMR during the game, but he couldn't have lost 600 MMR in 2 hours...
https://www.opendota.com/matches/3455767578
I had another game where a player is shown as 3k MMR.
Is this an issue with international ranked? If so can both MMR be shown?

This player hasn't had their MMR visible for a year:
https://www.opendota.com/players/164482777/mmr
The value is outdated

Ah, ok, that explains it. Can we have that information in the overview on mouseover over the person icon (last updated:)?

Yeah, I guess that'd be possible but would require adding some data to the database and pass it down to the UI

Great! :+1:
And also on player overview?

Yeah it would be the same data.  The main thing is to add a column to the MMR table tracking when the data was collected.

I understand. It was just a question if you think that it's a good proposal ;)

I dont know why they refuse to do it so badly. Please add it.

No one's refusing to do it, but someone has to volunteer to actually write the code :)

I am volunteering to write it. Like I said I just need a bit of guidance about what would be required to do it properly.

Few steps needed:

Check out https://github.com/odota/core
Add updated_at columns to solo_competitive_rank and competitive_rank tables (https://github.com/odota/core/blob/master/sql/create_tables.sql#L394)
Update the insertion function to put the current time in the database (https://github.com/odota/core/blob/master/store/queries.js#L449)

It's already available in the row object as row.time, the mmr update service code is here which called insertPlayerRating: https://github.com/odota/core/blob/master/svc/mmr.js

I think this is moot with the new ranked system


Serilog dont logging

After this line in Program.cs, nothing is logged
https://github.com/iammukeshm/CleanArchitecture.WebApi/blob/4d89c2c5fc0cf79b71995ac11d9b6d828ead7d7e/WebApi/Program.cs#L54

Had the same issue, you need to move this line inside try catch block in Program.cs
await host.RunAsync();


moved docusaurus config redirects to server side

Moved docusaurus config redirects to server side using _redirects file for server-side processing

Tested preview site with success. Example:
Original URL: https://deploy-preview-483--celo-docs.netlify.app/developer-resources/integrations/cloud-hsm
Redirected URL: https://deploy-preview-483--celo-docs.netlify.app/integration/cloud-hsm
Ready for review & approval


Standardising what the affix styles look like

Changes:

navbar links with affix are now always #fff
under navbar affix the brand image changes to match the navy background (#fff coloured brand image)

Hi @coolsebz!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

The CNAME file should never be modified.
Changes must be made to the original Sass or Less source code file(s), not just the compiled CSS file(s).
Changes must be made to the original JS source code file(s), not just the generated concatenated JS file(s).

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


API renaming: of_file

How about renaming of_file to something like file_lines (which conveys its meaning more explicitly), and letting of_file return just chunks read by input to enable other segmentations/primary parsing?

Thanks for the suggestion!
I have thought about this. The reason Stream.of_file is currently reading lines is because that's the most immediately useful and common use case, in my opinion. The current version of streaming does not attempt to expose multiple and flexible IO streams. I'd like to address this in the next release though.
For a more complete solution, you're right, we need to expose streams with chunked inputs. I'm hesitant to just rename of_file to file_lines without considering how other IO operations would be integrated. In particular, we would certainly also want to write lines, chars, chunks, etc.
Maybe something like:

Stream.File.read_lines
Stream.File.write_lines
Stream.File.read_chars
Stream.File.write_chars
...

would result in a better API?

I'll start making some research in this direction and will post my thoughts here.

[...] would result in a better API?

yes, definitely.
One sure I don't understand yet is why read_lines is a stream rather than a source. Is it to avoid a systematic call to Stream.from?

Stream is a convenience module, so, yes – instead of calling Stream.from and Stream.into directly, it provides of_ and to_ functions.
In some situations it's also possible to implement a slightly more efficient stream "source" given that actual Source.t have to allocate an option value per element.

Now that I'm thinking about this, for the IO/File functions mentioned above, providing decoupled sources and sinks is probably a better decision. Makes them much easier to reuse in pipelines and provides nice support for leftover sources with Stream.run.
For example, reading a header from a CSV file with:
let (header, csv_data) =
  Stream.run
    ~from:(Source.File.lines "foo.csv")
    ~via:parse_header
    ~into:Sink.list

That looks nice to me! This is a bit off-topic but I tried to define a source for gziped files:
let gziped_file ?(buffer_size = 1_024) path =
  let buf = Bytes.create buffer_size in
  let init () = Gzip.open_in path in
  let pull ic =
    match Gzip.input ic buf 0 buffer_size with
    | 0 -> None
    | n -> Some (String.init n (Bytes.get buf), ic)
  in
  let stop ic = Gzip.close_in ic in
  Source.make ~init ~pull ~stop ()

Does it seem correct?

Hi @pveber! Sorry for the delay. I've been thinking recently about buffered streams. To answer your question: yes, I think currently that's the recommended way to create a gzip source. Your implementation is a well-behaved source: (1) it's lazy - the path is only open on init; (2) it's safe - terminates the channel on stop; and (3) it buffers the input exposing a configurable buffer size.
I like these three properties, but think this could be further improved in the future versions of streaming.
In particular, it's common to have the buffer allocation local to the source like in your example, but the reality is that it's very likely that the string produced by this source (from the buffer) will be put into another buffer in a sink. This means that there will be unnecessary copying from the source buffer into the sink buffer.
Can we avoid this? I think we can if we allow the source to write data directly into a buffer provided by a sink. I'm not certain yet how exactly this interface would look like but I have been experimenting with some encodings.
My current source of inspiration is C#'s System.IO.Pipeline (https://blog.marcgravell.com/2018/07/pipe-dreams-part-1.html) and ZIO's ZTransducer type (it's similar to Streaming.Flow but works on "chunks").

I wrote some notes on lazy resource initialization for streaming: https://github.com/odis-labs/streaming/blob/7a42d248cf0ec687c34890e18d6495189c0fb196/NOTES.md#lazier-resource-initialization
Would love to know if you have any thoughts.
With my proposal, your gziped_file source example would look like:
let gziped_file ?(buffer_size = 1_024) path =
  let buf = Bytes.create buffer_size in
  let init = lazy (Gzip.open_in path) in
  let pull ic =
    match Gzip.input (Lazy.force ic) buf 0 buffer_size with
    | 0 -> None
    | n -> Some (String.init n (Bytes.get buf), ic)
  in
  let stop ic = if Lazy.is_val ic then Gzip.close_in Lazy.force ic in
  Source.make ~init ~pull ~stop ()

Are there any drawbacks with this approach?

Sorry, forgot to answer on that. Wrapping the resource in a lazy container adds a very tiny overhead when reading, but I think the benefit is worth it.

Sorry, forgot to answer on that. Wrapping the resource in a lazy container adds a very tiny overhead when reading, but I think the benefit is worth it.

No worries.
By the way, I'd make one last modification to the gziped_file source above:
let gziped_file ?(alloc = int -> bytes) path =
  let buf = alloc 1_024 in
  let init = lazy (Gzip.open_in path) in
  let pull ic =
    match Gzip.input (Lazy.force ic) buf 0 buffer_size with
    | 0 -> None
    | n -> Some (String.init n (Bytes.get buf), ic)
  in
  let stop ic = if Lazy.is_val ic then Gzip.close_in Lazy.force ic in
  Source.make ~init ~pull ~stop ()

Having alloc provided by externally allows for "zero-copy"

No worries.
By the way, I'd make one last modification to the gziped_file source above:
let gziped_file ?(alloc = int -> bytes) path =
  let buf = alloc 1_024 in
  let init = lazy (Gzip.open_in path) in
  let pull ic =
    match Gzip.input (Lazy.force ic) buf 0 buffer_size with
    | 0 -> None
    | n -> Some (String.init n (Bytes.get buf), ic)
  in
  let stop ic = if Lazy.is_val ic then Gzip.close_in Lazy.force ic in
  Source.make ~init ~pull ~stop ()

Having alloc provided by externally allows for "zero-copy"

Closing this for now. I'm working on a buffered API for IO operation that will replace the existing file functions.

Closing this for now. I'm working on a buffered API for IO operation that will replace the existing file functions.


Fix cursor  overlapping label on iOS and macOS

Description
This PR updates the cursor height computation on macOS and iOS in order to get a better result with M3 default text style.
Previously, on macOS and iOS, the cursor height was slightly extended for fidelity. Because M3 default text styles relies on line height (for instance 1.5 for bodyLarge), and because the cursor height reflects this line height, extending the cursor height leads to the cursor overlapping the label or being very close to it.
This PR applies the cursor height extension only when line height is small enough (< 1.3). When line height is equal or superior to 1.3, the cursor height is slightly decreased to get a better result.
Default M3 TextField on macOS:

Before
After

macOS

iOS

Related Issue
Fixes https://github.com/flutter/flutter/issues/141354.
Tests
Adds 2 tests for M3.
Updates 1 existing test (replace golden with a paint expect clause. (Will do the same in another PR to make existing tests shorter and more accurate).

We should follow the spec if possible. I think we have a lot of inconsistencies with M3 right now so I wouldn't be surprised if the height is wrong, though I couldn't find an issue about it.

After writing this comment, I pursued my investigation and filed https://github.com/flutter/flutter/issues/142972 and fixed it in https://github.com/flutter/flutter/pull/142981. So now text field height should be compliant (unless the PR is reverted, but as it was merged several days ago there is less chance of a revert).

Ah right. I was also worried about a revert of https://github.com/flutter/flutter/pull/142981 but it looks like the roll succeeded.

It looks like https://github.com/flutter/flutter/pull/142981 actually did break a bunch of Google tests during the roll, due to shrinking the size of text fields. They all look like expected changes, so I don't think it will be reverted.
However, we should be careful with this and other PRs in the same way, though. They're likely to break visual diff tests.

However, we should be careful with this and other PRs in the same way, though. They're likely to break visual diff tests.

Totaly agree, that's why this fix takes care of not modifying the rendering on M2 and on non-Apple platforms. A deeper refactoring of the cursor rendering might be difficult to get merged without breaking visual diff tests for M2 and non-Apple platforms).


Revert "Sync eng/common directory with azure-sdk-tools for PR 6823"

Reverts Azure/azure-sdk-for-ios#1596

/checkenforcer override


feat: throw error on unknown section name

Stentor now throws an error when it parses a section name
that does not exist in its list of sections

@wfscheper better?


TypeError: undefined is not an object (evaluating 'currentView.caption')

I'm trying to reproduce your simplest example with <Carousel views={images} />. Getting an error TypeError: undefined is not an object (evaluating 'currentView.caption').
FooterCaption
node_modules/react-images/dist/react-images.es.js:324
  321 |     getStyles = props.getStyles,
  322 |     isFullscreen = props.isFullscreen,
  323 |     isModal = props.isModal;
> 324 | var caption = currentView.caption;
  325 | ^
  326 | var state = { isFullscreen: isFullscreen, isModal: isModal };
  327 |

This page http://jossmac.github.io/react-images/#/patterns also doesn't work. The error is also related to FooterCaption:
index.js:22 ReferenceError: string is not defined
    at t.FooterCaption (index.js:6)

All good. It just doesn't like empty arrays...


Add Atomic Tanks 6.5

Requires Visual C++ 2010 Redist (x86) to run. Keeping as a draft until we can actually define dependencies, unless that's not a problem. Other than that, installs fine and runs in my dev environment.
Part of https://github.com/microsoft/winget-pkgs/issues/897
Microsoft Reviewers: Open in CodeFlow

The app is failing to run due to missing MSVCR100.dll.


Fix up tune control docs

Addresses #10610
This provides fix for tune_control docs statement that there are 15 default tunes.
To fix this the docs now refer to system tunes (rather than default tunes) and don't mention the number of tunes.

@bkueng I was wondering if we should perhaps take the time to add tune_control to module documentation. Something like this:
	PRINT_MODULE_DESCRIPTION(
		R"DESCR_STR(
### Description
External tune control for testing.

Information about the tune format and predfined system tunes can be found here:
https://github.com/PX4/Firmware/blob/master/src/lib/tunes/tune_definition.desc

### Examples

tune_control play -t 2

)DESCR_STR");

	PRINT_MODULE_USAGE_NAME("tune_control", "system");
	PRINT_MODULE_USAGE_COMMAND_DESCR("play","Play system tune, tone, or melody");
        PRINT_MODULE_USAGE_PARAM_INT('t', 1, 1, 21, "Play predefined system tune", true);
        PRINT_MODULE_USAGE_PARAM_INT('f', 0, 0, 22, "Frequency of tone in Hz (0-22kHz)", true);
	PRINT_MODULE_USAGE_PARAM_INT('d', 1, 1, 21, "Duration of tone in us", true);
	PRINT_MODULE_USAGE_PARAM_INT('s', 40, 0, 100, "Strength of tone (0-100)", true);
	PRINT_MODULE_USAGE_PARAM_STRING('m', nullptr, "<string> - e.g. \"MFT200e8a8a\"",
					 "Melody in string form", true);
	PRINT_MODULE_USAGE_COMMAND("libtest");
	PRINT_MODULE_USAGE_COMMAND_DESCR("stop","Stop playback (use for repeated tunes)");

Questions

what is libtest argument?
Is there a way to indicate "there is no default" (at least in docs)? For example, the frequency above must be defined if a tone is defined. There is no default value though, so would be best this was not shown in docs.
Any reason not to include the px4_module.h which defines these doc features?

Thanks @hamishwillee, your suggestion sounds good.

what is libtest argument?

Not quite sure what it does but it's there to test the tunes library.

Is there a way to indicate "there is no default" (at least in docs)? For example, the frequency above must be defined if a tone is defined. There is no default value though, so would be best this was not shown in docs.

no, you can write this in the description. There's only the optional boolean flag (last argument).

Any reason not to include the px4_module.h which defines these doc features?

Yes you're going to need that when using the PRINT_MODULE_USAGE_NAME methods.

OK, I have added this as module docs. Couldn't get libtest to do anything so just put in placeholder text. @bkueng Feel free to take this and modify to whatever state you think it needs :-)
The current version renders out .md as shown below (and the console output in simulator below that)

tune_control
Source: systemcmds/tune_control
Description
Command-line tool to control & test the (external) tunes.
Tunes are used to provide audible notification and warnings (e.g. when the system arms, gets position lock, etc.).
The tool requires that a driver is running that can handle the tune_control uorb topic.
Information about the tune format and predefined system tunes can be found here:
https://github.com/PX4/Firmware/blob/master/src/lib/tunes/tune_definition.desc
Examples
Play system tune #2:
tune_control play -t 2

Usage {#tune_control_usage}
tune_control <command> [arguments...]
 Commands:
   play          Play system tune, tone, or melody
     [-t <val>]  Play predefined system tune
                 default: 1
     [-f <val>]  Frequency of tone in Hz (0-22kHz)
                 default: 0
     [-d <val>]  Duration of tone in us
                 default: 1
     [-s <val>]  Strength of tone (0-100)
                 default: 40
     [-m <val>]  Melody in string form
                 values: <string> - e.g. "MFT200e8a8a"

   libtest       Test library

   stop          Stop playback (use for repeated tunes)

There's a CI failure in this PR, which is still now present in master.

Thanks @dagar . Fix in https://github.com/PX4/Firmware/pull/10675


Ubuntu 14 or16 installing

Hello there. when I installed the gekko bot to my vps server(The same error in both ubuntu 14 and 16 LTS)   I compared it with an error like this:
root@ubuntu:~# certbot certonly --standalone -d example.com -d gekko.example.com
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Obtaining a new certificate
Performing the following challenges:
tls-sni-01 challenge for example.com
tls-sni-01 challenge for gekko.example.com
Waiting for verification...
Cleaning up challenges
Failed authorization procedure. example.com (tls-sni-01): urn:acme:error:unauthorized :: The client lacks sufficient authorization :: Incorrect validation certificate for tls-sni-01 challenge. Requested 0bc5b2042f3a165890141f639398042c.9742e28ceba767d599a712dbc3698f7d.acme.invalid from [2606:2800:220:1:248:1893:25c8:1946]:443. Received 2 certificate(s), first certificate had names "example.com, example.edu, example.net, example.org, www.example.com, www.example.edu, www.example.net, www.example.org", gekko.example.com (tls-sni-01): urn:acme:error:connection :: The server could not connect to the client to verify the domain :: DNS problem: NXDOMAIN looking up A for gekko.example.com
IMPORTANT NOTES:

The following errors were reported by the server:
Domain: example.com
Type:   unauthorized
Detail: Incorrect validation certificate for tls-sni-01 challenge.
Requested
0bc5b2042f3a165890141f639398042c.9742e28ceba767d599a712dbc3698f7d.acme.invalid
from [2606:2800:220:1:248:1893:25c8:1946]:443. Received 2
certificate(s), first certificate had names "example.com,
example.edu, example.net, example.org, www.example.com,
www.example.edu, www.example.net, www.example.org"
To fix these errors, please make sure that your domain name was
entered correctly and the DNS A/AAAA record(s) for that domain
contain(s) the right IP address.

The following errors were reported by the server:
Domain: gekko.example.com
Type:   connection
Detail: DNS problem: NXDOMAIN looking up A for gekko.example.com
To fix these errors, please make sure that your domain name was
entered correctly and the DNS A/AAAA record(s) for that domain
contain(s) the right IP address. Additionally, please check that
your computer has a publicly routable IP address and that no
firewalls are preventing the server from communicating with the
client. If you're using the webroot plugin, you should also verify
that you are serving files from the webroot path you provided.

root@ubuntu:~#
please help me :(

You can only use SSL and the certbot if you own a domain, you unfortunately do not own the domain example.com. I emailed you what instructions to follow (not these ones).


Missing functionality - how to run command in background

I needed to run command (binary) in the background which will run during the k6 execution and found that (probably because of exec.Command) it will wait forever if I run it like:
console.log(exec.command("mycmd", ["&"]));

I found one way which is working for me as a workaround:
console.log(exec.command("bash",  ["-c", "setsid mycmd >/dev/null 2>&1 < /dev/null &"]));

but this way I'm loosing all logs or info about the activity.
Does it make sense to include handling of background task functionality into this library?

Does it make sense to include handling of background task functionality into this library?

@robooo Do you have a proposal for the API? I'm not sure how we could expose the output from this background process, or detect its finalization (or force the finalization when the test ends)

@pablochacin sorry I wrote like 0 lines in golang but found this:
https://github.com/takama/daemon
maybe this can be integrated?

@robooo this daemon package is interesting, thanks for sharing. However I'm still unsure about how to integrate running a background process in the exec API:

I'm not sure how we could expose the output from this background process, or detect its finalization (or force the finalization when the test ends)


[FEATURE] Add _kind property to entities/types

Description
Currently, there's no way to distinguish between entities and types in the generated files.
Suggested Solution
Add a _kind property to entities and types as done for actions/functions in #200
Alternatives
No response
Additional Context
No response

Hi Andreas,
thanks for this suggestion. While I am not opposed, would you mind showing a use case where you need this kind property to be present to distinguish between types and entities?
Best,
Daniel

Hi @aschmidt93,
I just looked into it and I'm afraid the best I can currently offer you is something that looks a bit like this:
class MyType {
  static readonly kind: 'type' | 'entity' | 'aspect' = 'type'
}

class MyAspect {
  static readonly kind: 'type' | 'entity' | 'aspect' = 'aspect'
}

class MyEntity {
  static readonly kind: 'type' | 'entity' | 'aspect' = 'entity'
}

so the type remains a bit vague. The reason for this is that types, entities, and aspects can extend one another:
class MyType {
  static readonly kind = 'type'  // implicitly also strictly typed as 'type'
}

class MyAspect extends MyType {
  static readonly kind = 'aspect'  // wants to strictly type as 'aspect', which clashes with the inherited declaration
}

will this still be useful to you?
Best,
Daniel

@aschmidt93 ping to see if this is still relevant to you 🙂

Thank you, I'll try that.

The solution works for me :)


✨ Update Schema Implentation

Implementation of #27
closes #27

Thanks for the idea and contribution @andreipopovici. Your feature is now released. 🥇


lidar2imu diverge

Hi, Thanks for your sharing!
I have used the lidar2imu tool, however, the result seems to diverge.
Ceres Solver Report: Iterations: 15, Initial cost: 2.748471e+03, Final cost: 2.717503e+03, Termination: CONVERGENCE
I have given it a proper initialization. What could be the possible other reason?
Thanks!

Hi, Thanks for your sharing! I have used the lidar2imu tool, however, the result seems to diverge.
Ceres Solver Report: Iterations: 15, Initial cost: 2.748471e+03, Final cost: 2.717503e+03, Termination: CONVERGENCE
I have given it a proper initialization. What could be the possible other reason?
Thanks!

You can pull the latest code and try again. It is better to have more surface features in the calibration environment.

Hi, Thanks for your quick reply, I will try running the latest code.
Besides, I wonder is there other built-in debug way? For example, can I visualize the diverging process or the surface feature's distribution? now I can only received feedback from terminal.

@linwk20 您好，请问你用自己的数据auto calib标定lidar2imu标定成功了吗？我用自己的数据集标定效果不理想，可以交流一下吗？期待您的回复

no, i use other tools

@linwk20 请问您用的是什么工具，lidar align吗？还是什么其他工具？可以推荐一下吗？能否加你微信或者QQ交流一下，QQ2985782675,v:19516191665

Actually, other people take care of the calibration for me, I am not sure what they use too. The result is not good as well.

实际上，其他人帮我做了校准，我不确定他们用什么。结果也不好。我建议你使用其他开源校准工具。或者在更多数据集上尝试这个 repo，也许有些会起作用。

谢谢


⚠ GLDDNS Server (ns2.glddns.com) has degraded performance

In d20195d, GLDDNS Server (ns2.glddns.com) (ns2.glddns.com) experienced degraded performance:

HTTP code: 200
Response time: 428 ms

Resolved: GLDDNS Server (ns2.glddns.com) performance has improved in a0d09ca after 6 minutes.


flutter run error : Execution failed for task ':app:checkDebugDuplicateClasses'.

flutter run :
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugDuplicateClasses'.
> 1 exception was raised by workers:
  java.lang.RuntimeException: Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules guava-20.0.jar (com.google.guava:guava:20.0) and listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)

  Go to the documentation to learn how to Fix dependency resolution errors.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 10s

flutter --version
Flutter 1.24.0-1.0.pre • channel dev • https://github.com/flutter/flutter.git
Framework • revision 12bea61c29 (5 days ago) • 2020-10-14 15:57:04 -0700
Engine • revision 069b3cf8f0
Tools • Dart 2.11.0 (build 2.11.0-218.0.dev)

flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel dev, 1.24.0-1.0.pre, on Microsoft Windows [Version 10.0.18363.1139], locale fa-IR)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Android Studio (version 4.0)
[√] IntelliJ IDEA Ultimate Edition (version 2018.1)
[√] VS Code (version 1.50.1)
[√] Connected device (1 available)

what is the problem?

solved:
add
dependencies {

    implementation 'com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava'

}

in
build.gradle


Color-blind/visual impaired friendly (or configurable)

I'd love to be able to modify the width and color of markers. They are small and hard for me to differentiate.

Since UI themes can modify this and we haven't heard a ton of interest in it here, I'm going to close this one out.


Fix og-image logo

Used x512.png from resources repo. Should fix broken embed image in discord.

Closing because PR is just redundant fixes from #152! Moxi was faster on the draw today...


opentextfile-method.md: the ForWriting constant isn't documented consistently

In the opentextfile-method.md file, the code sample contains a reference to a ForWriting constant, but the "iomode" documentation in the tables above the code sample doesn't mention it at all. I'm confused by this: ForWriting (constant value 2) works perfectly fine for me with VBA in Excel 2013 and with Windows Script Host version 5.8. This constant is also documented elsewhere on pages scattered across the web, including other Microsoft pages. Was this purposefully (but incompletely) left out of the documentation for some reason? (If not, I can always submit a patch myself.)

@dhgutteridge Hi, can you provide me the URL to the documentation topic you're looking at? I see this when Iook at OpenTextFile Method.

Hi @lindalu-MSFT,
Yes, that's exactly the URL I'm talking about. If you look at the code example at the bottom of the page, note that it references a ForWriting constant ("iomode" value 2) that's not documented elsewhere in the page. The code example is out of sync with the tables above.
Constant 2 for "iomode" works fine for me. I'm not sure why it's not referenced in the tables.
Compare also with older Microsoft documentation on the same topic, e.g. https://technet.microsoft.com/en-us/library/ee198715.aspx, where you'll see this constant value is explicitly discussed.

Hi @lindalu-MSFT,
I've opened this as pull #556. (Like I said, I'm assuming the ForWriting reference wasn't deliberately omitted, since it works for me, and has been previously documented by Microsoft.)

Thanks @dhgutteridge I appreciate you bringing this to our attention! I merged this PR, but then I forgot that the content is in "freeze" right now. It may be a couple weeks before this update appears.

Thanks for your contribution! This content has migrated to https://github.com/MicrosoftDocs/VBA-Docs. Please go to the new repo, and if you still see the issue occuring there, please open a brand new issue. Closing this issue now. Regards, lindalu