Cumulative CLI fixes

https://issues.jboss.org/browse/ISPN-6516
https://issues.jboss.org/browse/ISPN-6562
https://issues.jboss.org/browse/ISPN-6565
https://issues.jboss.org/browse/ISPN-6567
https://issues.jboss.org/browse/ISPN-6569
https://issues.jboss.org/browse/ISPN-6570

Some of those test failures seems related.

One small comment and those CI failures from my side. Otherwise looks nice.

Fixed the serviceloader failures

@tristantarrant isn't missing the help description for disconnect?

Pedro, I've addressed all your comments. I'm now using your proposed solution for adding completions

@tristantarrant waiting for the CI to finish and I'll integrate it.

hey @tristantarrant. JmxManagementIT.testMemCachedConnectionCount failed. is it related?

Totally unrelated.

integrated! thanks @tristantarrant !


nimble-toggle-tip Component

😯 Problem to Solve
Complex forms and editor views built with nimble input components require field specific context help. While we've previously considered creating a nimble-tooltip to show contextual information, a nimble-toggle-tip has several benefits over the tooltip.
Persistence: A toggle-tip stays visible until the user decides to close it. This allows the user to interact with the information, such as copying text or clicking links. Tooltips, on the other hand, typically disappear when the user moves the cursor.
Keyboard accessibility: The user can use the keyboard to focus on the toggle-tip icon and open or close the toggle-tip with only the keyboard.
Complex Information: Toggle-tips can be better for displaying more complex information, including images, links, or formatted text. Tooltips are usually limited to simple, short text.
Mobile Friendliness: Toggle-tips can be more mobile-friendly as they can be triggered by taps and remain visible. Tooltips often rely on hover interactions, which don't translate well to touch interfaces.
💁 Proposed Solution

The toggle-tip will be predominately added to control labels.
Consider:

Should we add a 'no outline' appearance mode for the icon button and use that here?
Add a slot to the label or all controls?

References:

https://inclusive-components.design/tooltips-toggletips/
https://accessibility.pearson.com/resources/developers-corner/reference-library/tooltips-and-toggletips/index.php

Other reference components:

https://grafana.com/developers/saga/Components/toggletip
https://carbondesignsystem.com/components/toggletip/usage/

📋 Tasks

[ ] nimble-toggle-tip interaction design issue
[ ] nimble-toggle-tip interaction design spec
[x] nimble-toggle-tip visual design issue
[x] nimble-toggle-tip visual design spec
[ ] nimble-toggle-tip spec
[ ] nimble-toggle-tip custom element
[ ] nimble-toggle-tip unit tests
[ ] nimble-toggle-tip Storybook tests
[ ] nimble-toggle-tip Storybook docs
[ ] nimble-toggle-tip Angular wrapper
[ ] nimble-toggle-tip Blazor wrapper
[ ] Add to Angular example app
[ ] Add to Blazor example app
[ ] Update Component Status table in README and incubating status

The RDSS team working on the SLE Config Utility app mentioned they'll be interested in this component. They're currently building it themselves with a button and tooltip.

We should consider re-enabling the tooltip matrix tests when we work on this feature. #1106

The SLE schedule view introduced a toggle-tip-like component in this PR. It uses the info icon but only shows more info on hover, not click. When this component is available we should look at replacing that (and see if there are other SLE apps that have adopted similar patterns, possibly by searching source for that icon).


Debugging guide and/or internal tracing for debugging purposes

While debugging libuv changes I often resort to printf's.

Is there a way to run a libuv test/ within gdb and/or under valgrind? If so, can we put notes on how to do this in a DEBUGGING.md-type document?
Would there be interest in introducing some kind of internal tracing mechanism?

For valgrind, does something like valgrind -v --tool=memcheck --leak-check=full --track-origins=yes --leak-resolution=high --show-reachable=yes --log-file=memcheck.log ./test/run-tests tmpdir tmpdir work for you? I wouldn't be opposed to a wiki page or something for keeping that kind of information in one place.
Do you mean something like https://github.com/libuv/libuv/pull/1815?

For valgrind, does something like valgrind -v --tool=memcheck --leak-check=full --track-origins=yes --leak-resolution=high --show-reachable=yes --log-file=memcheck.log ./test/run-tests tmpdir tmpdir work for you?

Haven't tried with valgrind but I'm not sure it will work as ./test/run-tests is a script.
What works for me, at least wit gdb is:
$ ./gyp_uv.py -f make
$ make -C out
$ gdb --args out/Debug/run-tests test-name test-name

Would there be interest in introducing some kind of internal tracing mechanism?

Do you mean something like #1815?

OK, wrong word. I meant logging -- e.g. UV__LOG(UV_LOG_FS, 4, "This is a log statement at verbosity level 4"), with UV__LOG under a DEBUG ifdef.
I think (the splendid) #1815 is about monitoring the health of libuv from the embedder's perspective, not really what I had in mind.

I ask about valgrind specifically because the tests have MAKE_VALGRIND_HAPPY() in them. So presumably someone knows how to do this!

I ask about valgrind specifically

I would use the command https://github.com/libuv/libuv/issues/1966#issuecomment-416939989 but using out/Debug/run-tests instead of ./test/run-tests. For example, this works for me:
valgrind -v --tool=memcheck --leak-check=full --track-origins=yes --leak-resolution=high --show-reachable=yes --log-file=memcheck.log out/Debug/run-tests  udp_open udp_open

@santigimeno is right. FWIW, test/.libs/run-tests may also work.

Thanks folks. See #1971.
FWIW, the command from @santigimeno works but only traces the execution of run-tests. For tests that involve a fork, we need to add --trace-children=yes.
For example, I deliberately inserted a memory leak into one of the libuv paths traversed by the getaddrinfo_basic test, and this invocation catches it:
valgrind -v --tool=memcheck --trace-children=yes --leak-check=full --track-origins=yes --leak-resolution=high --show-reachable=yes --log-file=memcheck.log out/Debug/run-tests getaddrinfo_basic

Without --trace-children=yes, no leak is reported.

#1217 has used it in abundance (though used in conjunction with Node, practically it was calibrating libuv)

Thanks @gireeshpunathil

Fixed by c0c672e1a0117b88a2b43a4472cfc50dcb33db45


Error thrown when decoding multiple attributes

Does this library support multiple attributes?
Decoding something like this throws an error.
<MyObject>
    <Item id="1" value="ok">Hello</Item>
    <Item id="2" name=”not-ok”>Goodbye</Item>
</MyObject>

The error thrown is "Error Domain=NSXMLParserErrorDomain Code=111 "(null)""
Any ideas?

did you manage to resolve this issue ? I am also facing the same issue.

Running into the same decoding this little guy
struct CategoryXML: Decodable {
    @Attribute var citationFormat: String?
    @Attribute var field: String?

    enum CodingKeys: String, CodingKey {
        case citationFormat = "citation-format", field
    }
}

error: -[CitationStyleLanguageTests.InfoXMLTests testCategoryDecode] : failed: caught error: "Error Domain=NSXMLParserErrorDomain Code=111 "(null)”

FWIW, tried this and got same result
struct CategoryXML: Decodable, DynamicNodeEncoding {
    let citationFormat: String?
    let field: String?

    enum CodingKeys: String, CodingKey { case citationFormat = "citation-format", field }

    static func nodeEncoding(for _: CodingKey) -> XMLEncoder.NodeEncoding { .attribute }
}

FWIW, I thought I had this issue, to the point that I forked the project and began adding a test to find the problem. But the test passed—lots of attributes, all good. So I pasted my XML into the test and it failed.
It turned out I had an extraneous comma (,) between a couple elements.
With that sorted, my current project is parsing elements with dozens of attributes and child elements, no problem.

I'll leave this open for a bit for @SwapnilNighot . Let me know what the current status is. If it becomes stale for another month or so I'll close it.


get_ase() error

from aiida import orm, load_profile
load_profile()

from aiida_atomistic.data.structure import StructureData

properties_dict = {
    "cell":{"value":[[3.5, 0.0, 0.0], [0.0, 3.5, 0.0], [0.0, 0.0, 3.5]]},
    "pbc":{"value":[True,True,True]},
    "positions":{"value":[[0.0, 0.0, 0.0],[1.5, 1.5, 1.5]]},
    "charge": {"value": [1.0, 1.0]},
    "symbols":{"value":["Li","Li"]},
    }
structure = StructureData(properties = properties_dict)
structure.get_ase()

Raise error:
   1992         _kinds = self.kinds
   1993

[~/miniconda3/envs/aiida/lib/python3.11/site-packages/aiida/orm/nodes/node.py](https://file+.vscode-resource.vscode-cdn.net/home/xing/tests/aiida/atomistic/~/miniconda3/envs/aiida/lib/python3.11/site-packages/aiida/orm/nodes/node.py) in __getattr__(self, name)
    720             return getattr(self.base.links, new_name)
    721
--> 722         raise AttributeError(name)

AttributeError: cell

Should be related to #12 .

Hi @superstar54 , yes indeed it is related to https://github.com/aiidateam/aiida-atomistic/issues/12. I have not yet adapted the method, it is still the old one which works with orm.StructureData

The method is no more there, in favour of to_ase


Test rich diff editor for notebook

Refs: https://github.com/microsoft/vscode/issues/112486

[x] anyOS @jrieken
[ ] anyOS @Tyriar

Complexity: 5
Create Issue

We added support for rendering outputs for notebook in the diff editor this iteration. To test this feature, you would need a bit of time setting things up but once that's done, you should be able to use it just like the normal text diff editor.
Setup

Install Jupyter extension from master

go to https://github.com/microsoft/vscode-jupyter/actions/runs/510651036#artifacts and download ms-toolsai-jupyter-insiders.vsix

Install the extension from vsix
If you don't want to mess up with python or jupyter

you can create a .ipynb file with following content  plotly.txt
open it in VS Code, which should be opened in notebook editor

Or, if you already have Python installed, Jupyter can use it as a kernel, with which you don't need future customization

run command "create new blank jupyter notebook"
create a code cell with language Python, type in !pip install plotly=4.5.2, run the cell
create another code cell with following content, run the cell

  import sys
  import plotly.graph_objects as go
  fig = go.Figure(
    data=[go.Bar(y=[2, 1, 3])], layout_title_text="A Figure Displayed with fig.show()"
  )
  fig.show()

you should be able to see a graph

Testing
Once you can open a jupyter notebook with rich outputs in VS Code, you can then use git command to change its state in order to see it in SCM viewlet, or you can duplicate the file and then compare them directly through file explorer

A diff editor with document generated by above python code will look like below

Please test that:

Rich outputs can be rendered in the diff editor
If an output has multiple mimetypes, you can switch between mimetypes (for this you would need  plotly.txt)
You toggle the visibility of outputs
You can choose to see the outputs' raw data
Scrolling in the notebook diff editor with rich output rendered is reasonablly performant (no siganificant performance drop)

This is equally cool as buggy

This is equally cool as buggy


ImportError when trying to import PydanticValueError from pydantic

Description
When trying to execute dbt2looker from a virtual environment, I encountered an ImportError related to the pydantic package. The error occurs when attempting to import PydanticValueError from pydantic.
Steps to Reproduce

python3.11 -m venv dbt2looker-venv
source dbt2looker-venv/bin/activate
pip install dbt2looker
dbt2looker
Observe the error during the import process.

Expected Behavior
The dbt2looker tool should start without any import errors.
Actual Behavior
The following ImportError is thrown:
Traceback (most recent call last):
File "/Users/username/Projects/dbt/dbt2looker-venv/bin/dbt2looker", line 5, in <module>
from dbt2looker.cli import run
File "/Users/username/Projects/dbt/dbt2looker-venv/lib/python3.11/site-packages/dbt2looker/cli.py", line 18, in <module>
from . import parser
File "/Users/username/Projects/dbt/dbt2looker-venv/lib/python3.11/site-packages/dbt2looker/parser.py", line 5, in <module>
from . import models
File "/Users/username/Projects/dbt/dbt2looker-venv/lib/python3.11/site-packages/dbt2looker/models.py", line 7, in <module>
from pydantic import BaseModel, Field, PydanticValueError, validator
ImportError: cannot import name 'PydanticValueError' from 'pydantic' (/Users/username/Projects/dbt/dbt2looker-venv/lib/python3.11/site-packages/pydantic/init.py)

Additional Information

Python Version: 3.11
dbt2looker Version: 0.11.0
Operating System: macOS Sonoma 14.4.1
Installation Method: pip

Tested with python 3.7.16 with the following packages installed in a conda venv:
Package            Version
------------------ ---------
annotated-types    0.5.0
certifi            2022.12.7
dbt2looker         0.11.0
importlib-metadata 6.7.0
lkml               1.3.4
pip                22.3.1
pydantic           2.5.3
pydantic_core      2.14.6
PyYAML             6.0.1
setuptools         65.6.3
typing_extensions  4.7.1
wheel              0.38.4
zipp               3.15.0

Got the same error:
Traceback (most recent call last):
  File "/Users/username/anaconda3/envs/dbt2looker/bin/dbt2looker", line 5, in <module>
    from dbt2looker.cli import run
  File "/Users/username/anaconda3/envs/dbt2looker/lib/python3.7/site-packages/dbt2looker/cli.py", line 18, in <module>
    from . import parser
  File "/Users/username/anaconda3/envs/dbt2looker/lib/python3.7/site-packages/dbt2looker/parser.py", line 5, in <module>
    from . import models
  File "/Users/username/anaconda3/envs/dbt2looker/lib/python3.7/site-packages/dbt2looker/models.py", line 7, in <module>
    from pydantic import BaseModel, Field, PydanticValueError, validator
ImportError: cannot import name 'PydanticValueError' from 'pydantic' (/Users/username/anaconda3/envs/dbt2looker/lib/python3.7/site-packages/pydantic/__init__.py)

pip install dbt2looker will install pydantc 2.7.0 as dependency which no longer supports 'PydanticValueError' .
pip uninstall pydantic
pip install pydantic==1.10.2

will solve the above ImportError


remove old issue reporter flow

[x] enable aux window issue reporter by default everywhere (https://github.com/microsoft/vscode/pull/226370)
[ ] continue to monitor issues in stable and insiders. when we are at a good spot, we can continue with the step below.
[ ] remove all components of the old issue reporter (https://github.com/microsoft/vscode/issues/213332)

Would be great to remove the old BrowserWindow based flow 🙏


Enhancement: Remove Headbob and Lean when moving

Is it feasible to add a toggle to disable headbob and the camera leaning when sidestepping?
Thank you!

Hey, thanks for the suggestion!
It's a good suggestion, but I feel that it might be a bit out of scope for an anti-aliasing mod.
This might be a better suggestion for the likes of https://github.com/OpenCAGE/CinematicTools instead.
There's not really a "nice" way to hook that deep into the game in a way that would work across all the variants (Epic, GOG, Steam) on PC without a good bit of work, as each one was recompiled differently as far as I know.


top-level/impure.nix: fix overlay directory check

Description of changes
nix 2.16 and newer return true for pathExists (path + "/.") regardless of whether path is a file or directory, resulting in a new error after updating:
error: /home/arc/.config/nixpkgs/overlays.nix should be a file

This issue likely also affects stable nixpkgs (when evaluated with a newer nix) and may be worth backporting.
This implementation of isDir was stolen from filesystem.nix
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

We can just use lib.filesystem.isDirectory instead. It won't be a problem to import lib in this file

Ah, I assumed that would be circular (probably biased by thinking the original author had a good reason to redefine isDir inline), but I guess it will work just like this one in default.nix? https://github.com/NixOS/nixpkgs/blob/1c1df7f74e38cd208cec3b69d3e2ee9683f4a70a/pkgs/top-level/default.nix#L50

Actually no we can't do that, because lib.filesystem.pathIsDirectory always treats symlinks as non-directories, even if they link to a directory.
That's the main reason this pathExists (toString ./path + "/") hack is necessary.

We can just remove the . for now, it was never necessary in the first place I'm pretty sure.

That seems like a reasonable partial mitigation for now... Though I think we should prefer using lib.filesystem.pathType path anyway and only fall back to using the hack when it's `== "symlink".
Then I'd argue that we should remove most of the checking here. Given that the check is an unreliable hack, there's no real benefit of displaying a message like

error: ~/.config/nixpkgs/overlays.nix should be a file

vs

error: opening directory '~/.config/nixpkgs/overlays.nix': Not a directory

(we may still need a form of isDir to handle <nixpkgs-overlays> properly but if we can avoid using it for homeOverlaysFile and homeOverlaysDir then that eliminates most of our problems)

It would be great to have a lib.filesystem.isDirectory as described in https://github.com/NixOS/nixpkgs/pull/224834#issuecomment-1497793328

Wondering if this has since been fixed? I just installed nix 2.18.5, and have a symlink at ~/.config/nixpkgs/overlays.nix, and everything appears to work.


Does not restore secret after full restart

After docker flow proxy and swarm listener (as a docker swarm mode services, 1 replica each) are restarted fully (destroyed/rm'ed, then started again), the proxy configuration does not pick up the previously granted secret.
This may be an intractable problem, depending on how docker deals with the granting part.

There are a few ways to tell DFP about the existence of a secret. Can you post the commands that would help me reproduce the problem?

I used "docker service update --secret-add" to grant the proxy access to the secret

Start proxy with its own overlay proxy_network
docker service update --secret-add to grant the proxy the secret
Start service A
Access service A. Everything works
docker stack/service rm proxy
docker stack ps and docker service ls to ensure proxy is gone
proxy_network remains
Start proxy again
Check configuration with docker container exec -it $containerId cat /cfg/haproxy.cfg
There are no references to the secret on the "bind *:443" line

frontend services
bind *:80
bind *:443
mode http

@patran How are you doing the point 8? The service needs to be created with the secrets. If they're not included, the config will not have them. If a secret is not inside a service, proxy cannot invent it (it would not be much of a secret, wouldn't it?).
I would recommend creating a stack file that would, among other things, have the secret defined.
I'm not sure if I explained it well. Please let me know if I didn't and I'll do my best to be more descriptive. Can you please join http://slack.devops20toolkit.com/? We can have a chat there.

@patran Is this issue resolved through my comments? Can we close this issue? If not, can you please clarify point 8 of your steps?

Closing the issue due to inactivity. Please post a comment if you'd like me to reopen the issue.


Houdini 16.5 boost linking

Hi all
I was building on OSX for houdini 16.5 and ran into some stumbling blocks when compiling the houdini SOPs because of the changes to houdini's boost. Some sops seem to use the houdini error logging which uses hboost. Also the vector merge SOP uses boost regex which didn't link straight away.
Adding these to the openvdb_houdini CMakeLists target link libraries sorted everything out. Does someone want to do some fancier CMakelist file editing?
example line 133...
TARGET_LINK_LIBRARIES ( openvdb_houdini
${OPENVDB_SHARED_LIB}
HoudiniPRM
HoudiniGEO
HoudiniUT
HoudiniOP2
HoudiniOP3
boost_regex
hboost_iostreams
hboost_system
)

Hey @dokipen3d this should now be resolved. Closing, but let me know if you still have any issues with the OpenVDB Houdini CMake builder


Remove dependency on jvm 1.7?

Hi,
This is more of a question. I used your scalaPB on Spark, which used jvm 1.6
I am trying to fork and remove this line.
scalacOptions += "-target:jvm-1.7"
Will it work and how do I package to get a jar? Thanks

Just remove this line, and run package. The path of the packages should appear in the console.


Update docs for AzureADHybridAuthenticationManagement with MgGraph (Graph API)

Please update the docs on how to use "AzureADHybridAuthenticationManagement" with the Module "Microsoft.Graph" instead of "AzureAD"

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

ID: 50b27a01-622e-d980-83f2-1ed27060613b
Version Independent ID: e6472e15-10d7-bd4a-da02-558f2774dbaf
Content: Passwordless security key sign-in to on-premises resources - Microsoft Entra ID
Content Source: docs/identity/authentication/howto-authentication-passwordless-security-key-on-premises.md
Service: entra-id
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

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

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

@weyCC81 thanks for raising this. The engineering team has a work item to track the need to replace this option. There's dependencies to be worked out, and we don't have an ETA at this time. #please-close


Kubernetes discovery

Bit of a brain dump while I think about it.... Kubernetes components expose Prometheus metrics for the master, nodes, etcd, etc. Autodiscovery of nodes would mean all nodes & running containers are automatically discovered. Nodes expose Prometheus metrics for all Docker containers that Kubernetes manages. As every resource can be labelled in Kubernetes these labels could be reused as metric labels in Prometheus.
Following on from this, Kubernetes annotations could be used on Kubernetes services to monitor service endpoints.
If you're interested in this I'll try to put in a PR for review as soon as I have something basic working to get feedback?

This sounds very interesting to me! I'd be interested to hear how you envision the integration and necessary changes/additions in Prometheus. Happy to take a look at a PRl, will read up on interfaces Kubernetes provides has so far.

We are absolutely interested in this!

To begin with I'm thinking of just adding kubernetes node discovery. This will be enough to collect container (cgroup) metrics for all containers in the cluster as well as cluster level components. Each resource on kubernetes can be labeled (key/value pairs). Each node can be labeled independently & reusing these node labels as Prometheus target labels makes sense so I'm thinking of adding each node as a separate target group. This will mean each target group will only ever contain 1 target. Is there any performance impact of that compared to using a target group with multiple targets? I don't think targets in the same target group can have different labels can they? Well without relabelling which isn't part of discovery.

This sounds great! For pods, it's already kind of possible using DNS and SRV records but from my limited playing with it this only works if the pod is covered by a service.
Once all the bits for monitoring cluster, nodes and pods I think it would be great to have a blessed example configuration/replication controller/prometheus service definition that new kubernetes/Prometheus users can drop in to get started quickly.

@matthiasr while using DNS is possible for pods that are service endpoints, you won't be able to use the kubernetes labels as metric labels. Once I've added in node metrics, I hope to expand this initial node only functionality to cover kubernetes service endpoints via the kubernetes api. This will enable reusing kubernetes labels in metrics labels. Haven't thought about that too much yet but any ideas most welcome.
Your idea for a blessed config sounds great. Once I have something basically working I'll let you know & perhaps you could help me work out a generic config that will work out of the box for most people. I know kubernetes better than Prometheus so I have a lot to learn here.

@matthiasr BTW if you wanted to try out a drop in config for kubernetes nodes that uses file config targets & a separate watcher utility to update the targets as changes happen we provide one in fabric8. The kubernetes manifest is available at http://repo1.maven.org/maven2/io/fabric8/jube/images/fabric8/prometheus/<IP_ADDRESS>/prometheus-<IP_ADDRESS>-kubernetes.json It's this work that I'm trying to convert over to built in Prometheus discovery.


ffmpeg:x64-windows build failure

Host Environment

OS: Windows 10
Compiler: Visual Studio 2019 CMAKE

To Reproduce
Steps to reproduce the behavior:
./vcpkg install ffmpeg:x64-windows
Failure logs
-- Using cached C:/Program Files/VCPkg/downloads/ffmpeg-ffmpeg-n4.2.tar.gz
-- Extracting source C:/Program Files/VCPkg/downloads/ffmpeg-ffmpeg-n4.2.tar.gz
-- Applying patch 0001-create-lib-libraries.patch
-- Applying patch 0003-fix-windowsinclude.patch
-- Applying patch 0004-fix-debug-build.patch
-- Applying patch 0005-fix-libvpx-linking.patch
-- Applying patch 0006-fix-StaticFeatures.patch
-- Using source at C:/Program Files/VCPkg/buildtrees/ffmpeg/src/n4.2-a4d149a5c6
CMake Error at ports/ffmpeg/portfile.cmake:16 (message):
Error: ffmpeg will not build with spaces in the path.  Please use a
directory with no spaces
Call Stack (most recent call first):
scripts/ports.cmake:90 (include)
*** INFO
zlib 9733f1ea63ddd8ae97810e5598daeaa5e7da26e0
CONTROL 16fe63bb95b1e9f309b54c7bea9521cb72d20be3
vcpkg-cmake-wrapper.cmake 185f3ce346a5ced573f6c7fd41ab9f5146b77bb5
0006-fix-StaticFeatures.patch 2019b72a6e7af153bdd54cb574ae6fc2d2f43836
0004-fix-debug-build.patch 24fb49856631788d651d27b1ecdc669cc99b450a
build.sh 7a3fcf869b5017c3763afe90e5894486e74029bb
0001-create-lib-libraries.patch 82fd250e880296562bee8b4c9342e5a735cb39aa
build_linux.sh 88c253cadd25d2e59dd99bc4ec57abc1b4684680
FindFFMPEG.cmake.in 9be54e6c701714f18bc8aac485db5eb8996816f3
0005-fix-libvpx-linking.patch ca631d186b134bd84fe324210eff8ff5832552c2
portfile.cmake d8c993a6f7e8619d4ff141702dd59463db7aa005
0003-fix-windowsinclude.patch ebeecc1f6d119d180ffbb7e6686d6bfdbd19c2a7
cmake 3.14.7
powershell ab1effc926b000a6adc12198a1886514ec203621a53b0cd7ec1cd9a8225dccda7e857feaabcfba4004bea73129b986abaad777c4573f44e0af70411226ce08b0
vcpkg_fixup_cmake_targets dac334f3bf50b8262be92623168023aa565aa729
post_build_checks 2
triplet 3eab5c64d9cf628fc0aec277fde66dbdc69b8516
features avresample;core
stdout-x64-windows.log
x64-windows.vcpkg_abi_info.txt

Error: ffmpeg will not build with spaces in the path. Please use a directory with no spaces
C:/Program Files/VCPkg/buildtrees/ffmpeg/src/n4.2-a4d149a5c6 contains a space.
Put vcpkg in a location without any spaces.

@EnzoDAndrea, please try the solution provided by timautry, and reopen this issue if it still be a problem for you.


First and last occurrence of x in array.

🚀 Feature
(I write a code in cpp, can you assign me this issue.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Motivation
(Help to people to crack coding interview.)
Pitch
(As this question ask in many technical Interview.)

Need to add a description and can you please bit more elaborate how you're going to implement and also tell us whether you are working under Gssoc as a participant? Thanks, Closing as of now.


Investigate using CSS grids for article page

Type
🚑 Health report
Investigate whether using CSS grids could simplify the article template to get rid of all the complicated maths in the CSS

Made a start on this, but it's on hold - see 'grids-experiment' branch

@gestchild What would we need to get this done?
In terms of value - it would give us a better toolkit to develop the UI for new pages, especially when we get a new designer in, we can say "This is the framework" which I remember @Norvard saying he would have loved to have done more of.

Are we able to close this? Or shall I leave it open until we know more about whether we'll make adjustments to the article body?

I think it's worth keeping it open


Patches for Lux integration

I am pulling out the patches from https://github.com/EnzymeAD/Reactant.jl/pull/5, so that we can get them in faster

Could array interface be done in an extension package if it's not in base

Done

Ah looking at that error I think we should trace the primal and shadow
arrays into tracedrarrays so then the duplicates invariant holds. We'll
need the data as an rarray anyways to run it
On Mon, May 27, 2024, 11:33 PM Avik Pal @.***> wrote:

@.**** commented on this pull request.
In src/utils.jl
https://github.com/EnzymeAD/Reactant.jl/pull/7#discussion_r1616400841:

@@ -7,7 +7,7 @@ function transpose_val(val)
return MLIR.IR.result(MLIR.Dialects.stablehlo.transpose(val, permutation=attr), 1)
end

-function apply(f, args...; kwargs)
+function apply(f, args...; kwargs...)
This came up in computing the gradient of the model with Enzyme. But that
code is not completely functional yet, see LuxDL/Lux.jl#665 (comment)
https://github.com/LuxDL/Lux.jl/pull/665#issuecomment-2133946153
—
Reply to this email directly, view it on GitHub
https://github.com/EnzymeAD/Reactant.jl/pull/7#discussion_r1616400841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJTUXFWCLE53NPEVDMVQGLZEORCZAVCNFSM6AAAAABIKKBQG6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOBRGQ4DANZQGY
.
You are receiving this because you commented.Message ID:
@.***>

However separate from this fix (which ought be done regardless) it is
surprising that the original model is not an rarray as I would expect. That
should probably be fixed in lux
On Mon, May 27, 2024, 11:36 PM Billy Moses @.***> wrote:

Ah looking at that error I think we should trace the primal and shadow
arrays into tracedrarrays so then the duplicates invariant holds. We'll
need the data as an rarray anyways to run it
On Mon, May 27, 2024, 11:33 PM Avik Pal @.***> wrote:

@.**** commented on this pull request.
In src/utils.jl
https://github.com/EnzymeAD/Reactant.jl/pull/7#discussion_r1616400841:

@@ -7,7 +7,7 @@ function transpose_val(val)
return MLIR.IR.result(MLIR.Dialects.stablehlo.transpose(val, permutation=attr), 1)
end

-function apply(f, args...; kwargs)
+function apply(f, args...; kwargs...)
This came up in computing the gradient of the model with Enzyme. But that
code is not completely functional yet, see LuxDL/Lux.jl#665 (comment)
https://github.com/LuxDL/Lux.jl/pull/665#issuecomment-2133946153
—
Reply to this email directly, view it on GitHub
https://github.com/EnzymeAD/Reactant.jl/pull/7#discussion_r1616400841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJTUXFWCLE53NPEVDMVQGLZEORCZAVCNFSM6AAAAABIKKBQG6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOBRGQ4DANZQGY
.
You are receiving this because you commented.Message ID:
@.***>


trigger to update manifestwork when cluster is updated

Refers: https://issues.redhat.com/browse/ACM-5109

/cc @clyang82 @morvencao

/lgtm


Update common_smtp_server_bounce_responses.md

Removing potentially offensive language.

Description of the change:
Reason for the change:
Link to original source:
@ksigler7

I can't even believe this was in our doc


[BEAM-12697] Add basics for SBE schema

This isn't really usable yet. It's just to get things started and establish one of the major restrictions on constructing it.
Schema generation and the PayloadSerializer will be handled in future PRs. This is just to get the foundation set.

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

R: @TheNeuralBit

I'm not clear on why we need this, can't we use the Beam schema builder, and have it's field order be the same as the SBE schema's field order?

I'm not clear on why we need this, can't we use the Beam schema builder, and have it's field order be the same as the SBE schema's field order?

One thing I know is going to be added is semantic type mapping. SBE XML schemas allow for a "semanticType" attribute on a field, and outside of a direct field mapping, as done in this PR, that's the best way to detect certain logical types (e.g. UTCTimestamp). Otherwise, we have little way to distinguish types with unique semantic meanings. More on that is here. That could just be another set of overloaded methods, but at a point, it seems like the builder pattern done here will be cleaner, especially if anything new comes up.
I'm also planning to generate the PayloadSerializer at build time, and this method will clearly keep that in line with how the schema is generated (stubs, IR, or XML). Since there are some benefits and limitations between stubs and IR/XML, that seems like it will minimize unexpected behavior.

I'm still waiting for more feedback. I think this PR just got lost amid vacations.

Hey @zhoufek sorry for the delay. I'm still not entirely clear on why this is necessary, and right now I'm just inclined to hold off until more of the implementation is ready which could clarify that. Alternatively, maybe we could just have a chat offline to discuss this -  if that makes sense could you schedule something?

I'm basically replacing this with PR #16762


enhancement: contributing rules

Description
Create a development branch for users to develop template.
Main branch will be locked for all users, except GitHub actions
External users will fork wrappid-module and they will create Pull Request, after verification, it'll be merged to development branch
References
https://github.com/wrappid/.github-private/issues/1 (not followed)
https://github.com/wrappid/.github/issues/8

below commit adds contributing file
f39316155ccb798368fc94bfd6466eac8e1e9260
closing


Implement perf module(script) to optimize link latency for different vendors.

Scope:

Determine vendors/upstream driver(iwlwifi/ath9k/10k/11k/rtlwifi/mt).
Determine available knobs to optimize link delay and segregate common and   vendor specific knobs.
Apply the configuration dynamically (if up/down cb)
Example:
Common knobs:
Disable power save.
Disable/reduce aggregation size(amsdu/ampdu): All vendor does not have this knob.
WMM QOS: classify critical latency data to VO ( mostly application layer handling)
CPU governor setting (bus bandwidth detection?).
TCP delack/window size configuration.

Vendor specific knobs:

Any latency mode if available(disable off-channel operation ex:background roam scan)
Rx interrupt/bottom half(NAPI) affinity.

High level idea is drafted in below commit for review/inputs.
https://github.com/govindsi/mesh_com/commit/f679b298f2a3376e97a865a2d474c013e4d890ca

Seems like a good idea. Would be fun to play around with at the very least. Thoughts @joenpera?

Good idea, but takes time to find all knobs for all listed wifi drivers.  Of course work can be started with wifi cards which we have in our hands.  Let's keep this open and develope further.

No needed any more!


Adding different properties\theorems for laplace transformations

I would like to add theorems used in laplace transformations
Eg: first shifting theorem, second shifting theorem, multiplication theorem and many more .
Please assign this issue to me.

Go ahead
Deadline 31 March,2021 (11:59 PM IST)


Bugs under Organization creation

[x] not able to create new organization-500 internal server error

[x] dropdown values are not visible while hovering mouse over it

[x] make currency,time zone and country as mandatory fields

[x] no currency values under dropdown list of currency and same for timezone dropdown(backend issue)

issue fixed!


Masking cookies as key/pair instead of a single string

This closes #900 and #904

Codecov Report

Merging #983 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #983   +/-   ##
=======================================
  Coverage   97.35%   97.35%
=======================================
  Files          46       46
  Lines        2080     2080
=======================================
  Hits         2025     2025
  Misses         55       55

Impacted Files
Coverage Δ

spec/raven/processors/cookies_spec.rb
100.00% <100.00%> (ø)

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 b9d150e...1bb8d45. Read the comment docs.


Fix #1306: Project rename input doesn't lose focus anymore

Here is the fix for https://github.com/mozilla/thimble.mozilla.org/issues/1306.  Project rename input doesn't lose focus anymore when an auto-save is triggered

Hopefully we won't find that there is a focus bug waiting to emerge due to this change.  I agree that it seems safe/harmless.  Let's try to remember that we did this down the road if we hit odd behaviour with focus and (background) saves.


Fixes #7 - Upgrade JUnit - current version causes hard crash for sure…

…fire if also using latest JUnit

Thanks!

This'll be fixed in v1.2.0


Change order of $factories

Allows users to use the service container across other factories, for
example in the session storage factory

:+1: I'll test.

@b3njamin could you rebase your branch against the master to avoid these extra commits? Thanks

Sure - done :)

@miniche could you test that?


Add RealSense dynamic_reconfigure configuration.

The only change from the default configuration (inherited from image_transport) is to use png compression for the aligned depth data. The default is jpg, which isn't able to handle the 16-bit depths provided by the RealSense. There may be a speed penalty (I haven't profiled yet), but OpenCV/libjpeg will just drop the upper 8 bits if it uses jpg, which renders the data unusable for distances farther than 0.25m.
(I don't fully understand why, but the cv::imwrite documentation specifically says that 16-bit unsigned images can only be encoded as png/jpeg2000 and not jpeg.)

LGTM. Is there more doc that you used to write this dynamic_reconfigure? maybe helpful to link it here?

Thanks! I generated this by manually running dynamic_reconfigure to set the format and dumping the resulting configuration, something like
$ rosrun dynamic_reconfigure dynparam set /camera/aligned_depth_to_color/image_raw/compressed/format png
$ rosrun dynamic_reconfigure dynparam dump


Looks like there was a server error! (500)

A few things to check first:
Is the issue you are filing related to a specific kata?
If so then please file an issue on the discourse page of the kata. You can do so by creating a comment and marking it as an issue. Only issues related to site functionality should be filed here.
Is the issue you are filing related to how a specific language behaves when being executed?
If so then it will likely be better to file the issue on our github.com/Codewars/codewars-runner-cli/issues repo, which is where we have our open source code execution functionality. You also may want to look into forking that repo and fixing the issue yourself if you have the means.
Otherwise delete all of this text and file your issue. Make sure to include steps to reproduce the issue, including which pages/kata you found the issue on, and your Codewars username if its related to your account.

What is the issue about ? You have not posted anything just the preloaded text.

I get the above error on signup or trying to view my profile...
Url: https://www.codewars.com/users/username-redacted

Looks like there was a server error! (500)
We have taken note of the situation and will be looking into it.
If you would like to supply us with some details as to how this happened please visit our Github issues list. This will help us get things fixed faster.
ApplicationController::ResourceNotFoundError

I get this also when trying to sign up. It doesn't create my user and crashes with a server error.

I just tried signing up again and it worked after that.

Sorry for the late response. If this issue is about server error after signing up, it will be tracked at #1516


[Request] How to access local dataset locally?

Hi,
Thanks for amazing work. You have many predefined datasets in your library. My usecase is that I have a dataset which is not listed in your repo. Also, I dont want to upload the dataset to your servers as it is of huge size and also privacy concerns.
Can you show me a way how can I analyze the dataset locally?

Hello @meet-minimalist thanks for reaching out. The visuallayer SDK is meant to work with our cloud product VL Profiler so there isn't a way to run it locally currently. We are working on a solution to let users run the product on your own premise at the moment. Get it touch with us if this solution would work for you.
In the meantime, please try out fastdup the open source version of the tool. You can run it locally on your own dataset.
Hope it helps :)

Hello @ShSi1001 you can try out fastdup - https://github.com/visual-layer/fastdup

Hi @dnth. Thanks for the reply. I have been working with fastdup.
On windows 10 and python 3.10 version, i am getting the error
ERROR: Could not find a version that satisfies the requirement fastdup (from versions: none)
ERROR: No matching distribution found for fastdup
On windows 10 python 3.9 version, I am not getting the above error however, the command fd.vis.stats_gallery(metric='dark') is not being implemented properly


Downloading smaller file with NoResume doesn't truncate

This is related to #23 and should have been fixed by 799551cb35da12b490c245c195e7293d7e11e591, but it's not. I'm not sure whether it's expected behaviour, but I don't think it is.
We have a file on disk, and we're downloading a newer version of it with grab. If the newer version is smaller, the file is not truncated, even with NoResume enabled.
I followed the code path (using good ol' fmt.Printf :) ) and discovered that this is because none of the blocks that set writeFlags are accessed in our case. In validateLocal, we exit before the NoResume block is reached, via this block:
        // determine expected file size
	size := resp.Request.Size
	if size == 0 && resp.HTTPResponse != nil {
		size = resp.HTTPResponse.ContentLength
	}
	if size == 0 {
		return c.headRequest
	}

as size is zero.
Then, in headRequest, there's this block:
        if resp.Request.NoResume {
		return c.getRequest
	}

that doesn't add the proper writeFlags. This means that when we get to openWriter, the writeFlags are untouched, as initialized by Do:
        writeFlags: os.O_CREATE | os.O_WRONLY,

And thus the file is not truncated.
What I did as a temporary fix is to set the flags in headRequest, just like you do in validateLocal:
        if resp.Request.NoResume {
                resp.writeFlags = os.O_TRUNC | os.O_WRONLY
		return c.getRequest
	}

However, I think that perhaps the NoResume check and the setting of the flags should be in one place, rather than two, but I'm not familiar enough with the library to know where that would be, so I thought I'd report this and leave it to you. Sorry this isn't a pull request, we're not using github plus my time is unfortunately limited at the moment.

Oh, and one more thing: we also ran into a problem with only O_TRUNC: it deleted our files. To fix it, we needed to change it to:
resp.writeFlags = os.O_CREATE | os.O_TRUNC | os.O_WRONLY

Might be related or not, just mentioning it.

Thanks for raising this. It was a bug as you described.
The desired behavior if the remote file is smaller than the local file is:

return ErrBadSize
or truncate the local file if Request.NoResume is true

I've cleaned up the code so all write flags are set in one place (openWriter) and added a test to catch regressions for this issue.

Thanks a lot! Glad I could help!


tf2 tutorials

I am currently in the process of migrating tf2 tutorials to ROS2. I have created a list of tutorials and corresponding PRs as shown below to keep the progress:
tf2 Concepts page:

[x] tf2 Concept: About tf2 page. https://github.com/ros2/ros2_documentation/pull/1951

Top level tutorial pages:

[x] Main tf2 tutorials page. https://github.com/ros2/ros2_documentation/pull/1628
Link to source.
[x]  Introduction to tf2. https://github.com/ros2/ros2_documentation/pull/1628
Link to source.

Python tutorials:

[x]  Writing a tf2 static broadcaster (Python). https://github.com/ros2/ros2_documentation/pull/1628
Link to source.
[x]  Writing a tf2 broadcaster (Python). https://github.com/ros2/ros2_documentation/pull/1715
Link to source.
[x]  Writing a tf2 listener (Python). https://github.com/ros2/ros2_documentation/pull/1715
Link to source.
[x]  Adding a frame (Python). https://github.com/ros2/ros2_documentation/pull/1737
Link to source.
[x]  Learning about tf2 and time (Python). https://github.com/ros2/ros2_documentation/pull/1770
Link to source.
[x]  Time travel with tf2 (Python). https://github.com/ros2/ros2_documentation/pull/1770
Link to source.

C++ tutorials:

[x]  Writing a tf2 static broadcaster (C++). https://github.com/ros2/ros2_documentation/pull/1788
Link to source.
[x]  Writing a tf2 broadcaster (C++). https://github.com/ros2/ros2_documentation/pull/1835
Link to source.
[x]  Writing a tf2 listener (C++). https://github.com/ros2/ros2_documentation/pull/1836
Link to source.
[x]  Adding a frame (C++). https://github.com/ros2/ros2_documentation/pull/1895
Link to source
[x]  Learning about tf2 and time (C++). https://github.com/ros2/ros2_documentation/pull/1898
Link to source
[x]  Time travel with tf2 (C++). https://github.com/ros2/ros2_documentation/pull/1899
Link to source.

General:

[x] Quaterion Fundamentals. https://github.com/ros2/ros2_documentation/pull/1968
[x] tf2 Tutorial: Debugging tf2 problems. https://github.com/ros2/ros2_documentation/pull/1923
[ ] Add tutorial "Using Stamped datatypes with tf2_ros::MessageFilter". https://github.com/ros2/ros2_documentation/pull/1929https://github.com/ros2/ros2_documentation/pull/1929

Additional PRs in ros2_documentation:

[x] https://github.com/ros2/ros2_documentation/pull/1807
[x] https://github.com/ros2/ros2_documentation/pull/1818
[x] https://github.com/ros2/ros2_documentation/pull/1882
[x] https://github.com/ros2/ros2_documentation/pull/1886
[x] https://github.com/ros2/ros2_documentation/pull/1936

Associated PRs in the geometry_tutorials repository:

https://github.com/ros/geometry_tutorials/pull/34
https://github.com/ros/geometry_tutorials/pull/38
https://github.com/ros/geometry_tutorials/pull/39
https://github.com/ros/geometry_tutorials/pull/40
https://github.com/ros/geometry_tutorials/pull/41
https://github.com/ros/geometry_tutorials/pull/44
https://github.com/ros/geometry_tutorials/pull/48
https://github.com/ros/geometry_tutorials/pull/49
https://github.com/ros/geometry_tutorials/pull/50
https://github.com/ros/geometry_tutorials/pull/51
https://github.com/ros/geometry_tutorials/pull/52
https://github.com/ros/geometry_tutorials/pull/53
https://github.com/ros/geometry_tutorials/pull/54
https://github.com/ros/geometry_tutorials/pull/56
https://github.com/ros/geometry_tutorials/pull/57
https://github.com/ros/geometry_tutorials/pull/58

The first Introduction to tf2 tutroial is completed and is shown in: https://github.com/ros2/ros2_documentation/pull/1628
For now I added it to Beginner: CLI Tools section, but not sure if it is the correct way.

I think that it would be good to have an intermediate page for the tf2 tutorials. Specifically, on the tutorials page, link to a page for tf2 tutorials, and from there, you can add the specific tutorials.

@audrow where should I place the link to the intermediate page in the tutorials page. Should it be under the Examples section?

I think probably in the "Intermediate" section. If any of the tutorials should probably be advanced, perhaps we can have a larger discussion on how to organize them.

I think the whole thing should be in the intermediate section (even the introduction).  While tf2 is an important concept in ROS, it's not really required to understand it to get started.
One other thing I'll mention here is that we can put these tutorials into ros2_documentation to start with, but long-term we may want to consider moving these into https://github.com/ros2/geometry2 .  We can't do that right now because we don't have a good mechanism to deploy tutorials from individual packages, but eventually we'll get there.

Hey, this looks interesting, anything I can do here?
Thanks!

Hey, this looks interesting, anything I can do here?
Thanks!

Hi! I am working on this myself currently, but you can take a look at other open issues and try to resolve them. Your help will be appreciated 😉

@kurshakuz Finally I finished tutorial "Using Stamped datatypes with tf2_ros::MessageFilter" last night, so can I create a PR for it, or do I need to wait for #1923 PR merged to rolling branch? You can look it in my repo. https://github.com/kennywangaigraphx/ros2_documentation

@kennywangaigraphx no, feel free to create it. We will make a review as soon as possible

General:

[ ] Quaterion Fundamentals

@kurshakuz Do you plan to write a page for this?

General:

[ ]  Quaterion Fundamentals

@kurshakuz Do you plan to write a page for this?

I am currently working on other PRs, and was planning to do it next week, but if you want to do it, your help will be very appreciated! Just let me know if you are going to do it

Hello @clalancette! The biggest part of tutorials mentioned in your initial issue https://github.com/ros2/geometry2/issues/387 is covered so far. However, I would like to get your advice on the last ones. My current observation is that "Setting up your robot using tf", "Using the robot state publisher on your own robot", "Using urdf with robot_state_publisher" are already covered in one or other sense. However, they are currently spread out among different pages as shown below:

Setting up your robot using tf
https://navigation.ros.org/setup_guides/transformation/setup_transforms.html

Using the robot state publisher on your own robot
https://navigation.ros.org/setup_guides/urdf/setup_urdf.html
https://github.com/ros/robot_state_publisher/tree/ros2

Using urdf with robot_state_publisher
https://docs.ros.org/en/rolling/Tutorials/URDF/Using-URDF-with-Robot-State-Publisher.html

I am currently thinking adding those links with corresponding descriptions to the About tf2 page. What do you think about it?

* Setting up your robot using tf
  https://navigation.ros.org/setup_guides/transformation/setup_transforms.html

For this one, I'd actually suggest we contact the Navigation team and see if they would be OK migrating that guide from the Navigation documentation into the core ROS 2 documentation.  Then they can link to it from their own documentation.  @SteveMacenski , thoughts about doing that?

* Using the robot state publisher on your own robot
  https://navigation.ros.org/setup_guides/urdf/setup_urdf.html

This is another one that we could consider moving into the core ROS 2 documentation, as it is not navigation specific.  Additionally, we could definitely use some more advanced URDF and robot_state_publisher tutorials, as those are frequently asked about.  It may be outside of the scope of this issue, though.

* Using urdf with robot_state_publisher
  https://docs.ros.org/en/rolling/Tutorials/URDF/Using-URDF-with-Robot-State-Publisher.html

That one is fine as-is.  If we are going to link to it from "About-Tf2", I think we should make it clear how that tutorial relates to TF2 (basically that the combination of the URDF, robot_state_publisher, and joint_state_publisher publishes transforms to TF2).

These are pretty specific pages to Nav2. I can see they would have general utility, but keep in mind our goal in these pages are to teach people to set up the transforms / sensors for Nav2, not in general (e.g. not in cars, manipulators, or other robot types). We don't bring up the conventions in naming or styling for those other forms so it would probably need some additions to be more generic for the ROS 2 documentation.
The code this references is in the Nav2 tutorials github repo, how would you suggest dealing with that?


[release-1.2] mem-hotplug: enable only for VMs with memory >= 1Gi

Manual backport of https://github.com/kubevirt/kubevirt/pull/12212.
The 1GiB mark is chosen as a tradeoff, it is enough memory for the guest kernel to allocate its internal data structures and to allocate the SWIOTLB.
It also means we can memory map all PCI devices
as they're memory mapped in the first 1Gi (PCI hole).
If there are 32bit devices that need to do DMA,
the kernel will allocate memory for the SWIOTLB, which is a bounce buffer to address memory after the 4Gi Mark, the SWIOTLB in Linux at the moment of this commit is 64MB and this causes an issue for the kernel with VMs with very small amounts of memory, if not enough memory is available this can lead to the guest kernel panicking while trying to allocate memory.
This SWIOTLB issue is present when setting maxMemory (which is set in all VMs when LiveUpdate is used as the vmRolloutStrategy) in the XML domain of the VM because the kernel will have to allow DMA to happen also for virtio-mem hotplugged memory, which is always mapped after the 4Gi mark and thus always needs SWIOTLB for devices that use 32bit addresses.

What this PR does
Before this PR:
VMs with less that 1GiB of guest memory might see the guest kernel panicking at boot.
After this PR:
VMs with less that 1GiB of guest memory work as expected but cannot hotplug memory.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

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

Release note

enable only for VMs with memory >= 1Gi

/lgtm

/approve

/test pull-kubevirt-unit-test-1.2

/kind bug

/test pull-kubevirt-e2e-k8s-1.28-sig-operator-1.2


chore(master): release butler-sos 11.0.3

:robot: I have created a release beep boop
11.0.3 (2024-10-23)
Bug Fixes

config: Add missing property in sample config file (51b64b3), closes #930
config: Allow empty New Relic destination array in config file (9601a9c), closes #929
influxdb2: Allow pos/neg engine cache values (f9d8ecb), closes #933
startup: Make sure log level can be set via command line (45733c9), closes #932
startup: Make sure no attempts to access InfluxDB if it is disabled in config file (4010a1c), closes #931

This PR was generated with Release Please. See documentation.

:robot: Created releases:

butler-sos-v11.0.3
:sunflower:


Login Timeout

I have deployed and configured the ldap usermanager plugin for azkaban.
On the same server Azkaban runs on I've verified I can communicate to the LDAP server via ldapsearch.  Azkaban logs indicate the server starts up fine, I did see one or two errors when I had a typo, and once corrected, no more errors in logs indicating config is good.
When I try and login, the web page just sits there and after about 60 seconds I see...
Incorrect Login. LDAP error: TimeOut occurred
I've tried this with multiple accounts, and logged in to a windows server in same domain with same accounts, so I know its not a mistyped password.
Any ideas, I don't see anything in any of the Azkaban logs or system logs.

Ok, so posting this just in case someone else comes searching.  I was specifying the wrong field for userIdProperty, instead of cn, it was needed sAMAAccountName in my case.


unexpected auto-lagging results & other questions

Originally posted this on the answers board, but received no response. You may check this page more frequently.
Working a bunch with the Trackme app and it's showing a lot of promise. I finally got the right MLTK and Python applications installed, hopefully that will help some of the issues I'm having.
When I modify a data source, choosing "auto lagging" comes back with unexpected results. In particular, I have a data source that only ingests M-F, between 9am and 5pm. The auto percentile lag of this data source for 7-30 days at 1-3 seconds. How would I go about getting a longer average lag time for this source?
In addition, how can I tell trackme to not show an alert state on the same sourcetype on a monday morning, since it hasn't gotten any events since the friday before? I don't want to set the lag time as too high as that will interfere w/ monitoring during weekdays.
Thanks for your help.

Hi @manderso7
My apologises I didn't see a notification from Splunk Answers, I need to check why ;-)
Happy you like the trackMe promise, let's convert that in results!
I will review your questions very shortly, if you have some screenshots of the use case we are speaking about this would help understand it better.
Thanks
Guilhem

No need to apologize, thanks for looking.

@manderso7
Thanks, I understand better the use case and the question.

When I modify a data source, choosing "auto lagging" comes back with unexpected results. In particular, I have a data source that only ingests M-F, between 9am and 5pm. The auto percentile lag of this data source for 7-30 days at 1-3 seconds. How would I go about getting a longer average lag time for this source?
So effectively, TrackMe records 2 essential metrics can it come to lagging:

lag_ingestion_sec: delta between index time and latest event timestamp
lag_event_sec: delta between now when the measure is taken, and the latest event timestamp

When you define a limit fo the max lagging, say 3600 seconds (1 hour), trackMe will define a green / red (essentially) status based on the condition if the max lagging is  superior to 300 seconds either for the event lag, or for the event lag, or for both.
If you open of the single forms on the left of the screen you shared, you will see:
eval delta=(indextime-_time)
This measure refers to the delta in seconds between the indexing time of the event (when it was indexed versus the event timestamp as parsed by Splunk)
So basically what it says is that your events were indexed with a very low latency.
The max lagging value you can interact is effectively applied against both measures, and any of these 2 measures would influence the state of it.
Normally on this specific data source, you should have one of the 2 measures showing the difference between and the latest time of you had events for it, can you confirm ? (it's not visible in the screenshot)
Technically you would define a max lagging value which matches the level where you want to get alerted, this would indeed affect your capability to detect ingestion latency on this data source in favor of detecting if you basically have events as expected or not.
I need to think about more about this use case, basically a batch use use. (and find a way to allow dissociating both if that is required)

In addition, how can I tell trackme to not show an alert state on the same sourcetype on a monday morning, since it hasn't gotten any events since the friday before? I don't want to set the lag time as too high as that will interfere w/ monitoring during weekdays.
The week days definition basically influence the results in the following way:

If a data source would be red due to monitoring conditions, but if the current days is not a monitored day, the state will be orange instead of red, the status message would reflect the information, and the OOTB alert would not trigger.

Let me know your thinking / understanding of my answer ;-)

Is this what you're looking for?

To verify we're on the same page, you're talking about the auto lagging option measuring the longest time between events?
That makes sense. The status can show orange, until it turns green when data comes in, or red when it reaches the max data lag, ignoring the unmonitored days.

@manderso7
Thanks, yes this is correct.
The above shows that the time of your writing, there was 2393 seconds between now (when the last execution of the tracker exactly), and that the ingestion was near 0 seconds. (certainly between 0 and 0.5 but round at one decimal)
Yes I think we are on the same page, you are talking about:

What this feature does is basically calculating different versions of what you could choose for the max lagging value, instead of defining it yourself statically in the previous window.
Note that once defined, this will not change anymore and will not be re-calculated or updated to reflect the current behaviour of the ingestion latency)
So to be clear, this is the ingestion latency max accepted definition with a feature that provides different options you might want to choose.
Does it make sense ?
Guilhem

Yes, thanks. Though I'm just wondering why the suggested lag times are so different from the lag times witnessed in the data source above. If the suggested lag times could be more similar to the visible gaps in the data they would be more useful.

@manderso7
Because technically you didn't have any latency at the ingestion time (the difference between the Splunk _indextime and the _time which is extracted from the events according to your sourcetype definition or what Splunk used for the time stamp recognition)
The latency at ingestion time is what shows the windows exposing the 4 choices for an auto determinate potential choice for the max lagging.
The delta between now, and the latest time you had data (which in true is the latest time recognized by Splunk, the _time) is what you call the visible gaps in the data, and is called lagging event in the app.
Note: I understand there might some confusion in these single forms as they don't explicitly say it and I will fix that)
So basically, the gaps between 2 bunch of data is the value for lag event (and will grow by time until you get new data) and the lag ingestion is the latency at which your events were indexed in Splunk.
One option I could provide in TrackMe would be to allow you to choose if you want to consider the lag event or the lag ingestion for the calculation of the values, would you think this addresses the problem ?

I think the calculation should be based on the amount of time between events, and not the latency of events being ingested.
Also, I think _indextime is more valuable than _time to measure lag ingestion.
If possible, maybe have the option you suggested, and it's worth it to see if that shows the data more clearly.
Thanks for listening to my suggestions.

More than happy to take your suggestions in consideration ;-)

the amount of time between events is a third metric that can indeed be interested specially for batch like use cases, I will consider it
_indextime is already taken in charge and consideration, when you calculate the latency you need to understand the time taken by an event to be indexed on the disk (the _indextime) and the time the event was produced (the _time), this is already at the root of everything and is the lag ingestion as explained previously

@manderso7
Work is being achieved to better handle a similar batch like use case, and shall be on-boarded within Version 1.2.18.
Guilhem

Hi @manderso7
TrackMe version 1.2.19 has been published with some new features that should address your use case.
Especially, you can now choose from which KPI the alert status will be derived, as well as a better rendering of the differences between the lag at ingestion time and the lag event.
The documentation: https://trackme.readthedocs.io
Has been seriously reviewed and updated too.
Let me know your thinking, if this addresses properly your requirements, in such a case we will close this issue.
Guilhem


feat: Add basics.pronouns

Based on @veeara282's work on https://github.com/jsonresume/resume-schema/pull/325. I lifted the schema verbatim, and added unit tests.
Motivation
Why should a JSON résumé schema facilitate a pronouns field:

People aren't always accurately gendered by their photo. Whether a person "looks like a man" varies by culture and circumstance. If you didn't know this, you need to get out more.
Recruiters and others reading résumés should not be expected to guess an applicant's forms of address on sight. (That's just rude.)
It's cool to normalize providing a simple detail that answers questions like "how should people refer to this person in a respectful manner?"
Adding the field is super easy, barely an inconvenience.
The field may be optional. Users don't have to declare their forms of address if they do not want to.
Declaring one's own forms of address hurts no one.
It's becoming more common to add pronouns to résumé documents.
Standardizing this field in the JSON schema helps theme developers to adopt the field more readily, rather than forcing them each to guess where users might put the field in custom configurations. (Somewhere in meta, perhaps? Are we talking meta.pronouns, or something more structured like meta.basics.pronouns? I get the impression that meta is more about the document than the author, so basics would be a better spot for the author's forms of address, I think.)
It's just plain polite to refer to people the way they want to be referred to, such as by their own name, or by the appropriate title. Pronouns are even more common than formal titles, since everyone gets referred to in the third person at some point.
Pronouns are very common. Declaring one's own forms of address (and having a simple, neutral, easy, and unobtrusive way to do so without social stigma or ridicule) eliminates a lot of extra brain steps.
Why not add the field?

Why omit the pronouns field:

The basics property already permits additional properties. Custom properties there already pass validation, so users ought to add their own. An ecosystem decision on the matter will surface naturally in several years or so.
We don't know the best place for it, nor the best shape.
@thomasdavis doesn't want to. Fair enough, it's his project I guess.

These lists are not comprehensive. I may have rendered my arguments inefficiently or veered into straw-man territory in some cases. I'm certainly biased, but I think that having a standard pronouns field does more good than harm.
I figure if @thomasdavis is so adamant that resume-schema omit the pronouns field as a standard, then a fork will do well enough. Even if it's only me using this fork, this is a fun project for me, and it makes the tooling around my own résumé more complete.
These proposed changes are flexible enough for most users to declare their forms of third-person address. As the language evolves, so too can the schema.
Not a political stunt, only trying to be helpful.
See also discussion at https://github.com/jsonresume/resume-schema/issues/324, https://github.com/jsonresume/resume-schema/issues/447, and https://github.com/jsonresume/resume-schema/issues/475.

Forgive the brain splatter above. It's late, and I just wanna get this out there asap so I can get back to work on other things.
I welcome discussion here, of course.

(Also also, sorry about the pings, Thomas! I figured I should mention you, but idk if I can do that without sending a notif. It's late. So sorry 😅)

I for one am glad you did it. Perhaps the next step would be to have a published site and whatnot for it, to get visibility on it and not just redirect back to the old one as though it also has this feature?

Of course, the ideal scenario would be for these changes to work their way upstream, but as we've seen, that's not likely to happen except maybe the type declarations and the Promise-ified interface.
I might open a PR later on anyway, just to see what happens


In Chrome 38 beta on Android, taps don't move the cursor

In Chrome 37 tapping somewhere in CodeMirror brought the cursor there. In Chrome 38 this stopped working. I reported it in http://crbug.org/418177, but it may not get fixed before broader release =/

This is fixed in the current code (which handles mobile browsers in a completely different way)


one-file-system does not correctly handle sub-paths

/mnt/storage is a filesystem. / is a filesystem. / has many file systems mounted underneath it, including /mnt/storage.
What happens if I run:
$ du -shcx / /mnt/storage/Archives/

It will give me the sum total of files used on /, on the / filesystem, and of /mnt/storage/Archives/ on the /mnt/storage filesystem. This is the correct behavior.
However, if I run:
$ restic -r ... backup -x / /mnt/storage/Archives/

Then it will backup everything in / and everything in /mnt/storage/, since /mnt/storage/ is under /, and the addition of /mnt/storage/Archives/ to the command line added in the /mnt/storage filesystem to the allowed lists. This is the incorrect behavior.
Rather, -x should anchor each traversal point to that root point's filesystem, like what du does.

Hey, thanks for taking the time to report this. It is actually a known issue, the code responsible for --one-file-system is unable to handle nested mount points. I'm going to close this as a duplicate of #702, please subscribe to that issue if you want to be notified once it's resolved.
In the meantime, you could make separate backups (but running at the same time) of / and /mnt/storage/Archives.

Hey @fd0. Thanks. Just saw that bug a few seconds before you posted. Sorry for the duplicate.
Question: if I do two separate backups, and then later, after you fix this issue, move to doing one single backup, will --parent still auto-populate with the identical information?

No big deal, I'll just close duplicates ;)
When you move to a separate backup, the first run will re-read all files, but restic will recognize that all data has already been saved in the repo, so no new data is uploaded. The next backup after that will again use a parent snapshot.

Could you please check if the code in #908 if that works for you?


Do golang profiling to find out any memory leak in cephcsi

Golang supports profiling , we can also make use of profiling and check are we leaking any memory and if required we can also do some optimization to improve the performance.

This should probably be a configurable option, either by setting a commandline parameter or environment variable.
heketi/heketi#1396 does something of the sort too.

Assigning this to you @nixpanic as you have more idea and you already implemented the same for heketi

@Madhu-1  you can assign this to me. I will discuss it with @nixpanic .

@Madhu-1  you can assign this to me. I will discuss it with @nixpanic .

Thanks @Madhu-1  for assigning. Will start working on this once I am done with the snapshot issue.

Thanks @Madhu-1  for assigning. Will start working on this once I am done with the snapshot issue.


Fabric in Fabric in Fabric ... what?

In the last few days I noticed that when I update/uninstall mods using Mod Updater, upon reopening the game (manually or via the "Relaunch Minecraft" button), at the bottom left side of the screen, the word "Fabric" repeats multiple times (the amount of duplicates seems to somewhat correlate with the amount of mods that were modified, installed or uninstalled).

This bug does not seem to affect gameplay from the short testing that I did.
Note: The duplicates of the word "Fabric" disappear after reopening the game again.
Additionally, I want to point out that the launching/relaunching process of the game usually takes a minute or more (even for a small amount of mods that are being modified) to finally load, which is quite a long time to wait.
Here are some logs that may help you:
2022-09-23-7.log.gz
latest.log

Hello, thank you for the report. The repeating Fabric text is a known issue, but I don't think it affects gameplay. As for long update times, I will look into improving that. Thanks!


Can't connect to server with mixed SMB shares authorizations

As AndreyLavrov unfortunately closed issue https://github.com/yoichiro/chromeos-filesystem-cifs/issues/32, I open another one.
Please have a look at my 3 last posts in that issue 32 issue.
Edit: Never mind. I copy them here below.
============================================
Hi, I just checked with your 1.1.8 version.
Unfortunately it still doesn't work. See below. I removed the password.

I quickly setup a samba share and connected.
That works!
Unfortunately, as described above, I still can't connect to my NAS.
=======================================
hvdwolf commented 2 hours ago
Hi,
Some observations.
My NAS has a public share, 2 multi-user shares and 1 specific configuration share.
If I specifiy the ip-address without username and password, I get the popup and option to connect to all 4 shares. Of course I can only connect to the Public share in this case (But this is already a great step forward :) ).
As soon as I specify a username and password I get the errors.
Is it possible to provide a share name as well?
I understand that you want to keep it as simple as possible, but if you would be able to specify a share could that solve this issue? Possibly via a checkbox setting like "Specify share name"?

@hvdwolf I can't know the reason of "NT_STATUS_ACCESS_DENIED". As the error name, you rejected to connect from your server. You should check the server log of your server.

Below my smbd.log. All the preallocate errors are from the startup or restart of nmbd/smbd
There are several authentication methods for samba which are set in the smb.conf
I had:
auth methods = guest, sam_ignoredomain, winbind:ntdomain
I changed that to:
auth methods = guest, sam_ignoredomain, winbind:ntdomain, ntdomain
Look at line "2015/06/07 16:02:25,  0".  If there is no domain, it can't be used.
So I removed that again and restarted it. I get the NT_STATUS_LOGON_FAILURE again, but nothing is written to the log..
cat log.smbd
[2015/06/07 16:02:06,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:02:06,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:02:06,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:02:06,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:02:06,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:02:06,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:02:06,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:02:06,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:02:25,  0] source/auth/auth_domain.c:check_ntdomain_security(319)
check_ntdomain_security: Configuration error!  Cannot use ntdomain auth method when not a member of a domain.
[2015/06/07 16:03:18,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:03:18,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:03:18,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:03:18,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:03:18,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:03:18,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
[2015/06/07 16:03:18,  1] source/param/loadparm.c:map_parameter(6090)
Unknown parameter encountered: "preallocate"
[2015/06/07 16:03:18,  0] source/param/loadparm.c:lp_do_parameter(7176)
Ignoring unknown parameter "preallocate"
My security is set to "security = user", which is the default for samba. Setting it to share or domain gives other errors but does not solve it.

@hvdwolf I have already released the version including supporting "share" mode and etc. Could you confirm whether you can reproduce this issue with the latest version?

Hi, I'm sorry. I completely missed the mail notification.
It works fine now. No matter whether I configure my server for "security = share" (deprecated) or "security = user" (default), I can connect.

However, looking back at the the title of my issue: The mixed shares still don't work.
If I connect only on ip address (and port), the SMB app shows all available shares but I can only connect to the public ones.
When I add my username/password (for the protected shares) I still get the NT_STATUS_LOGON_FAILURE

I can now mount both my user/password protected shares as well as my public shares. Very nice.


Packaging feature, unwanted dependencies in package.json and more

Describe the bug
Hi
I didn't know that this packaging feature is experimental, I just jumped into it😂. Let me start from the beginning to explain what is needed and etc.
I have a very simple and small package. It's a 1 .svelte file, a Svelte component. It doesn't need any dependencies. This is the package.
My first issue was that I needed to create an index.js file. I thought that I can just link to the component file from alias config in svelte.config.js or from jsconfig.json. In the community, I find out that someone had the same issue and someone else said that she need an index.js file to export the component as default... I was able to fix it.
After doing some changes to my package, I realized that all of my package.json dependencies are added to the package's package.json file too!. My package doesn't even need a single dependency.
What if I want to have a separate LICENSE or README or... for the package and the demo? There is nothing to do that.

Wasn't it better to just add an absolute path to the dependency value in package.json without creating something that does something and doesn't do so much stuff?
"devDependencies" : {
  "my-package": "C/some-folder/package-folder"
}

What if someone has a CSS file that can't be placed in the lib folder? Like this.
This description must be placed at the start, so everyone before starting to use the Kit packaging feature know about all the negative things about it.
That's all❤
Reproduction
_
Logs
No response
System Info
_

Severity
blocking an upgrade
Additional Information
No response

From what I understand, you want to

place the caveat description at the top -> I'm against this
get better docs around the package.json contents. I'll create a PR where the dependencies/devDependencies thing is explained
you want more flexibility ("what if..") but there's no compelling reasons given why. Many of SvelteKit's features are opinionated, which is great to prevent decision fatigue and provide a consistent developer experience, but it also means it may not suit all needs.

From the the three things only one is actionable, so I'll close this issue through that docs PR.
Please open one issue per feature request or bug report in the future. It's hard for us to track issues when there are multiple in one.

Can I create a post in Discussions to talk about this feature? And I'll also check out the current issues related to this feature and talk about them too...


Provides APIs to limit Atom autoupdate version in a specific range.

Hey guys. I am working on Nuclide and one thing I am doing right now is improve the quality of Nuclide.
As it happened to us previously that Atom auto-updated a new version and made a non-backwards-compatible change and caused bugs and regressions, it would nice to have APIs to set the autoupdate version range for Atom programatically so that we can control the version of Atom user is using.
What's your guys thought about this? I am willing to write up PR's if it does make sense to you.

Which change was that? Was it the one that I fixed via https://github.com/atom/atom/commit/5031284ce58aab3c86a04c1ffe59a76822d0d62b?

@bolinfest  yeah, although https://github.com/atom/atom/commit/5031284ce58aab3c86a04c1ffe59a76822d0d62b didn't break Nuclide, but it's really close.

Addressed in https://github.com/atom/atom/pull/8721


[#1587] Added Creation Policy support for DisruptorCommandBus

Implemented Repository#loadOrCreate on the DisruptorCommandBus.DisruptorRepository, so that the Creation Policy feature can be used in conjunction with the DisruptorCommandBus.
This issue resolves #1587.

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

Coverage is calculated incorrectly, is it covers both the axon-disruptor and axon-modelling modules.
The current set up does not take into account multi-module testing.


Can't view DataFrame through Variable explorer

Description
What steps will reproduce the problem?

Create dataframe of size (387652 , 73)
Try to view in variable explorer.
The error message pops up saying:

"Spyder was unable to retrieve the value of this variable from the console.
The error message was:
Kernel did not send the value."

That means that your dataframe is too big and can't be visualized through Spyder (most probably you don't have enough memory to perform this operation).
We plan to address this in a future Spyder version, but there's nothing we can do about this for the moment, sorry.

Maybe put that as an error message? ‘Your data frame is too big to visualise’ :)
Get Outlook for iOShttps://aka.ms/o0ukef

From: Carlos Cordoba<EMAIL_ADDRESS>Sent: Tuesday, October 15, 2019 2:43:05 PM
To: spyder-ide/spyder<EMAIL_ADDRESS>Cc: Peter<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [spyder-ide/spyder] Can't view DataFrame through Variable explorer (#10409)
Closed #10409https://github.com/spyder-ide/spyder/issues/10409.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/spyder-ide/spyder/issues/10409?email_source=notifications&email_token=AI27BYVFYDWO2WPP7AHNZDTQOU34TA5CNFSM4JAWQL2KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUGWKLZA#event-2712446436, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AI27BYUDY2R7NL6ICGNHKE3QOU34TANCNFSM4JAWQL2A.

Spyder was unable to retrieve the value of this variable from the console

I think this is enough.


Warn user when using "_" instead of "-" in iconNames

Summary
Proposed changes
This PR brings the following changes:

Warn user when using _ instead of - in iconNames

Related issue
#229 #298

Testing

[ ] I have added or updated tests that prove my fix is effective or that my feature works (if applicable)

[ ] Unit test
[ ] Snapshot test
[ ] Manual test

Checklist

[ ] I have added corresponding labels to this PR (like bug, enhancement...);
[ ] My commits follow the Conventional Commits 1.0 Guidelines;
[ ] My code follows the style guidelines of this project;
[ ] I have performed a self-review of my own code;
[ ] I have mapped technical debts found on my changes;
[ ] I have made changes to the documentation (if applicable);
[ ] My changes generate no new warnings or errors;

Codecov Report
Patch coverage: 80.00% and project coverage change: -0.11 :warning:

Comparison is base (6df7aa1) 94.50% compared to head (240f5a8) 94.40%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #299      +/-   ##
==========================================
- Coverage   94.50%   94.40%   -0.11%
==========================================
  Files          16       16
  Lines         764      768       +4
  Branches      133      135       +2
==========================================
+ Hits          722      725       +3
  Misses         18       18
- Partials       24       25       +1

Impacted Files
Coverage Δ

src/providers/base.ts
86.57% <80.00%> (-0.32%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

closes #298


lighten('black') is still black

Hi Frederik, thanks for creating this library - I'm currently using it in a TypeScript project and the API is really nice.
I'm trying to lighten a black color, but it remains black:
const {lighten} = require("@wessberg/color")
console.log(lighten('black', 10)) // => '#000000'
console.log(lighten('#000000', 10)) // => '#000000'

(check it out on Runkit)
It works just fine for other colors.
I had a look at the source for lightenHsl, and the lightness is calculated as follows:
Math.max(0, Math.min(100, ((100 + percentage) * parseFloat(L)) / 100))

(where percentage = 10)
For black, L = 0, so the resulting lightness value is 0.
For L = 50, the resulting lightness is 55.
In general, it seems strange that the percentage is proportional to the existing lightness value, because colors that are already light will become much lighter than darker colors. I expected the lightness to simply be added to the existing lightness, ie. that
for L = 0, the resulting lightness would be 10,
and for L = 50 it would be 60.
in JS:
Math.max(0, Math.min(100, parseFloat(L) + percentage))

This change would obviously affect all of the colors generated by your library so I don't know if this is something you'd want to do if you're relying on the way specific colors look already - just wanted to flag this as I'll unfortunately have to use a different library for the time being.
Anyway let me know if this makes sense & if you'd consider changing the behavior. Thanks a lot :)

Hey there. Thanks for submitting this issue. Yeah, I've been wanting to address this for some time. The same goes for saturate() which can also lead to the exact same kind of issues when the S value is 0.
I've released a new version (v1.0.5) that fixes the behavior of lighten and saturate.

Awesome, thanks so much for the speedy fix :)

Can confirm this works perfectly - thanks a lot!


add empty files to please the windows installer

as part of our installation process, wix creates a database of things to delete on uninstall. the template that helps to generate this list assumes that every folder has a file in it, but every now and again, there comes along a folder that only contains folders. then it doesn't get marked for deletion and there's an error like this:
error LGHT0204: ICE64: The directory is in the user profile but is not listed in the RemoveFile table

so, rather than futz with the template (which is much more prone to me screwing something up), i added a bit to the build process. After generating all the files but before passing them to wix, recursively step through every directory and add an empty file if there arent any other files present. currently this only happens in one place, but hopefully this fix is future-proof enough that no one needs to come back to look at this message in the future.

Perhaps we should fix the template? This seems somewhat hacky. But of course if that is not possible, then this is the way to go.

Perhaps we should fix the template? This seems somewhat hacky. But of course if that is not possible, then this is the way to go.

yeah - i did. my previous comment is a little disingenuous. i tried to tweak the template to add these RemoveFolder nodes for each directory instead of each file (and with various arguments to heat), but i couldn't get it to work. i'll give it another serious look though because you're right. this solution, while i learned a lot about windows batch scripting making it, is a good bit hackier than doing it directly in heat.

ok, lets do it.

is there something you'd like us to test on our machines before getting this in?

is there something you'd like us to test on our machines before getting this in?

@zapu nah thanks. if it works to build the app on my machine and the build bot, i suspect we don't need more data points.


Task: Renew certificate before March 9th

Metabase cert expires by March 9th and needs to be renewed.
Certbot is no longer an option, so the process is now manual.
Resources:
NRIDS SSL Cert: https://apps.nrs.gov.bc.ca/int/confluence/display/DEVGUILD/SSL+Certificates
NRIDS Generating a CSR: https://apps.nrs.gov.bc.ca/int/confluence/display/DEVGUILD/Generating+a+CSR
@dleard's writeup on getting a brand new certificate: https://bcgov.sharepoint.com/:w:/r/teams/00608-ScrumTeam/Shared Documents/Scrum Team/OBPS/Domain Setup/Domain_Request_and_SSL_Certificate_Documentation.docx?d=w3140fee140ef4eb2abf7615a53fb391e&csf=1&web=1&e=vxstGj

Renewal request submitted to NRIDS on Feb 28 2024. Waiting for confirmation.

@patriciarussellCAS to verify this, visit https://cas-metabase.nrs.gov.bc.ca/ , click the little padlock in the URL bar of your browser and view the certificate information. The validity should be March 4, 2025


How do I get started with this thing?

I added {:user {:plugins [[nightlight/lein-nightlight "1.6.3"]]}} to my ~/.lein/profiles.clj, and tried to run lein nightlight --port 4000, but the terminal said 'nightlight' is not a task.
I am a complete noob to 1) Leiningen, and 2) Clojure. Can you help me out with some kind of absolute beginner's guide? I would like to learn Clojure, and Nightlight sounds like a really interesting project concept.

Hmm I can't reproduce this...that is exactly what I have in my profiles.clj. Can you paste the entire output?

It just says 'nightlight' is not a task. run lein help
I just ran:
lein new app clojure-test
cd clojure-test
lein nightlight --port 4000

I literally haven't done anything else though. Is that all I'm supposed to do?

Oh, I figured it out. I also had this line in my profiles.clj:
{:user {:plugins [[cider/cider-nrepl "0.8.1"]]}}

For when I was using CIDER with Emacs, working through Brave Clojure. Interesting.

Ah gotcha, yeah you'd have to merge them together.


Add laravel-s library

Hello guys, I introduce an interesting project here https://github.com/hhxsv5/laravel-s.
LaravelS makes Laravel resident in memory, creating an HTTP server no longer depends on Nginx + FPM, and it can also quickly create WebSocket/TCP Server.

Thanks! 👍


Log user actions

Here is implementation of logging user actions:

Object create/edit/remove
Objects link/unlink
User login/logout
Reset device total distance

Made it non-switchable, only with global log. I do not think log size will grow dramatically in comparing with device communications debug.