Uncaught TypeError: undefined is not a function

[Enter steps to reproduce below:]

... Ctrl-n -> error!

Atom Version: 0.208.0
System: Mac OS X 10.10.3
Thrown From: atomic-emacs package, v0.7.0
Stack Trace
Uncaught TypeError: undefined is not a function
At /Users/jasper/git/noblejasper/dotfiles/macosx/atom/packages/atomic-emacs/lib/atomic-emacs.coffee:29

TypeError: undefined is not a function
  at AtomicEmacs.editor (/Users/jasper/git/noblejasper/dotfiles/macosx/atom/packages/atomic-emacs/lib/atomic-emacs.coffee:29:20)
  at AtomicEmacs.nextLine (/Users/jasper/git/noblejasper/dotfiles/macosx/atom/packages/atomic-emacs/lib/atomic-emacs.coffee:164:6)
  at atom-text-editor.disposables.add.atom.commands.add.atomic-emacs:next-line (/Users/jasper/git/noblejasper/dotfiles/macosx/atom/packages/atomic-emacs/lib/atomic-emacs.coffee:311:56)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:238:29)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:520:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:355:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:177:20)

Commands
     -0:01.1.0 atomic-emacs:next-line (input.hidden-input)

Config
{
  "core": {
    "followSymlinks": true,
    "themes": [
      "seti-ui",
      "monokai"
    ],
    "ignoredNames": [
      ".git",
      ".bundle",
      ".sass-cache",
      "vendor"
    ],
    "disabledPackages": [
      "atom-alignment",
      "language-csharp",
      "sort-lines",
      "atom-color-highlight"
    ]
  }
}

Installed Packages
# User
Sublime-Style-Column-Selection, v1.3.0
atom-jinja2, v0.5.0
atomic-emacs, v0.7.0
autocomplete-elixir, v1.3.5
compass, v0.9.1
emmet, v2.3.10
japanese-wrap, v0.2.10
javascript-snippets, v1.0.0
language-elixir, v0.5.1
linter, v0.12.7
linter-elixirc, v0.2.2
linter-jshint, v0.1.6
linter-python-pyflakes, v0.1.0
monokai, v0.14.0
seti-ui, v0.7.1
symbols-tree-view, v0.9.3
visual-bell, v1.0.0

# Dev
No dev packages

Same for Ctrl-X Ctrl-S

I'm guessing these are fixed now, with the 1.0 compatibility work. Please reopen if see this with the latest versions. Thanks for reporting!

Just saw this today in Atom and it linked me here.

[Enter steps to reproduce below:]

... Ctrl-Space -> Error!

Atom Version: 1.0.11
System: Mac OS X 10.10.5
Thrown From: atomic-emacs package, v0.7.2
Stack Trace
Uncaught TypeError: undefined is not a function
At /Users/thaddeus/.atom/packages/atomic-emacs/lib/mark.coffee:63

TypeError: undefined is not a function
  at Mark.deactivate (/Users/thaddeus/.atom/packages/atomic-emacs/lib/mark.coffee:49:23)
  at Mark.set (/Users/thaddeus/.atom/packages/atomic-emacs/lib/mark.coffee:26:6)
  at AtomicEmacs.setMark (/Users/thaddeus/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:144:24)
  at atom-text-editor.disposables.add.atom.commands.add.atomic-emacs:set-mark (/Users/thaddeus/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:345:55)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:245:29)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:524:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:347:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:184:20)

Commands
     -1:46.6.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused)
  5x -1:45.2.0 core:undo (atom-text-editor.editor.is-focused)
     -1:41 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused)
     -1:40.8.0 core:move-right (atom-text-editor.editor.is-focused)
     -1:27.6.0 autocomplete-plus:activate (atom-text-editor.editor.is-focused)
     -1:22.5.0 core:backspace (atom-text-editor.editor.is-focused)
     -1:21.2.0 core:save (atom-text-editor.editor.is-focused)
  2x -1:18.6.0 core:move-up (atom-text-editor.editor.is-focused)
  4x -1:18.1.0 editor:move-to-end-of-word (atom-text-editor.editor.is-focused)
  2x -1:16.2.0 pane:show-previous-item (atom-text-editor.editor.is-focused)
     -1:12 settings-view:open (atom-text-editor.editor.is-focused)
     -0:30.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:02.8.0 core:move-to-top (atom-text-editor.editor.is-focused)
     -0:02.1.0 core:move-to-bottom (atom-text-editor.editor.is-focused)
     -0:01.1.0 atomic-emacs:set-mark (atom-text-editor.editor.is-focused)
  3x -0:00.6.0 atomic-emacs:previous-line (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "ignoredNames": [
      ".git",
      ".svn",
      ".DS_Store",
      "node_modules"
    ],
    "disabledPackages": [
      "archive-view"
    ]
  }
}

Installed Packages
# User
atomic-emacs, v0.7.2
editorconfig, v1.1.1
project-manager, v2.3.0

# Dev
No dev packages

Atom 1.0.11
C-e command not work properly. this show me the "Find in current buffer" option, so every time  i press the C-g to close it an Uncaught TypeError: undefined is not a function . it appears,  i must close the editor and reopen
TypeError: undefined is not a function
    at Mark.deactivate (/home/foo/.atom/packages/atomic-emacs/lib/mark.coffee:63:36)
    at deactivateCursors (/home/foo/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:37:41)
    at AtomicEmacs.keyboardQuit (/home/foo/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:231:14)
    at atom-text-editor.disposables.add.atom.commands.add.core:cancel (/home/foo/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:618:30)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:245:29)
    at /usr/share/atom/resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/usr/share/atom/resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:524:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/usr/share/atom/resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:347:22)
    at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/usr/share/atom/resources/app.asar/src/window-event-handler.js:184:20)

I just updated to the most recent atomic-emacs 0.7.2 in atom 1.0.11 and I had an error that redirected here. On OSX, I'm running into this issue after I cmd+F to open the search and replace dialog, I see this error when I hit esc, which normally closes the search and replace dialog.
TypeError: undefined is not a function
    at Mark.deactivate (/Users/foo/.atom/packages/atomic-emacs/lib/mark.coffee:63:36)
    at deactivateCursors (/Users/foo/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:22:22)
    at AtomicEmacs.keyboardQuit (/Users/foo/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:147:5)
    at atom-text-editor.disposables.add.atom.commands.add.core:cancel (/Users/foo/.atom/packages/atomic-emacs/lib/atomic-emacs.coffee:350:45)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:245:29)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:524:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:347:22)
    at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:184:20)

Thanks guys. I just pushed the fix out (v0.7.3) -- upgrading atomic-emacs should fix it.

thanks @oggy. that fixed it!

:metal:

:+1:


Dash board calendar picker

-added calendar picker to filter from & to date for data
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1645 (623e7f2) into main (8cd2152) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 623e7f2 differs from pull request most recent head 76ac9bd. Consider uploading reports for the commit 76ac9bd to get more accurate results

@@           Coverage Diff           @@
##             main    #1645   +/-   ##
=======================================
  Coverage   71.82%   71.82%
=======================================
  Files         341      341
  Lines       10235    10235
  Branches      775      775
=======================================
  Hits         7351     7351
  Misses       2761     2761
  Partials      123      123

Flag
Coverage Δ

metweb
65.04% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

met-web/src/components/common/dateHelper.tsx
88.23% <100.00%> (ø)

The other pr is suppose to be a 1 file change. I was going to rebase once the other one got merged since they are separate tasks. I could just do put them all in one though


LD_LIBRARY_PATH is not respected

Environment data

VS Code version: 1.51.1

Jupyter Extension version (available under the Extensions sidebar): !ldd /ccs/home/hzfmer/file_back/programs/andes/anaconda3/lib/python3.8/site-packages/osgeo/_gdal.cpython-38-x86_64-linux-gnu.so | grep libnsl

Python Extension version (available under the Extensions sidebar): !ldd /ccs/home/hzfmer/file_back/programs/andes/anaconda3/lib/python3.8/site-packages/osgeo/_gdal.cpython-38-x86_64-linux-gnu.so | grep libnsl

OS (Windows | Mac | Linux distro) and version: Linux-Red Hat 7.6

Python and/or Anaconda version: 3.8.3-Anaconda

Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda

Jupyter server running: Remote

Expected behaviour
I was trying to import a package "gdal" from "osgeo", which is successful if I ran it in the system or VSCode terminal, while in vscode-jupyter it was unable to find the dynamic library "libnsl.so.1".  It's strange that ldd in the terminal finds it, but Jupyter doesn't.
I have added the location of this library to LD_LIBRARY_PATH, and Jupyter seems to read the variable, but somehow doesn't respect it.
Actual behaviour

Steps to reproduce:
This might be not very likely to reproduce, since I am working on an HPC cluster.
I would expect some setting/configuration as a workaround.
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

@hzfmer Sorry about the issue here.
Are you able to load this notebook in normal jupyter (just starting jupyter notebook from the command line) and have it work outside of VS Code? I wasn't sure from your example how you were setting that LD_LIBRARY_PATH for jupyter. But in general I would think that you'd have to specify it in the kernel.json file to actually get it picked up in the Jupyter (not terminal scenario). Is that how you are specifying it for the notebook?
Example would be the second reply on this thread here:
https://stackoverflow.com/questions/37890898/how-to-set-env-variable-in-jupyter-notebook

@IanMatthewHuff
Thanks.  I can run this notebook directly, and this module can be imported if I use Chrome to remotely loading the notebook. So Web and VSCode perform differently.
My LD_LIBRARY_PATH was set in my ~/.bashrc, and as you can see os.environ['LD_LIBRARY_PATH'] does include this path.
I tried to add

"env": {"LD_LIBRARY_PATH":"~/.local/lib"}

to ~/file_back/programs/andes/anaconda3/share/jupyter/kernels/python3/kernel.json, which doesn't work either.
I don't understand why the same command gives different results if running in Jupyter?
!ldd ~/file_back/programs/andes/anaconda3/lib/python3.8/site-packages/osgeo/_gdal.cpython-38-x86_64-linux-gnu.so | grep libnsl

# libnsl.so.1 => not found

While in Vscode integrated terminal:
ldd ~/file_back/programs/andes/anaconda3/lib/python3.8/site-packages/osgeo/_gdal.cpython-38-x86_64-linux-gnu.so | grep libnsl

# libnsl.so.1 => ~/.local/lib/libnsl.so.1 (0x00007f8ea120b000)

Ok, so a few more questions here. Sorry about this, but without a local repro to look at it's a bit trickier to drill down on this.

You mentioned that you are running against a remote jupyter server in your initial issue post. Does this mean that you are connecting to a remote machine using the "Jupyter: Specify local or remote server for connection" setting in vs code? Or is VS Code actually running on the machine that is executing the notebook?
Can you tell me more about the conda environment that you are using to run this?
If you print your current working directory, is it the same in the vscode terminal versus in the jupyter notebook?

The thought here is that your context for executing in Jupyter is not the same as executing the terminal. This is expected to some degree, you might have a different working directory, environment, interpreter and other things that would cause one to fail and the other to succeed. Just trying to narrow down what the difference is given my questions above.
Given what I've seen online we do need to make sure that LD_LIBRARY_PATH is set before Jupyter / IPython is started up.
https://github.com/jupyter/notebook/issues/1290

@IanMatthewHuff
Please check below:

using the "Jupyter: Specify local or remote server for connection" setting in vs code? Or is VS Code actually running on the machine that is executing the notebook?

I used "remote-ssh"  to connect to the server first, and then open the jupyter file. So VS Code is running on the remote server.

about the conda environment that you are using to run this
I recently reinstalled the Anaconda on the server, which is migrating to a new system.

>> lsb_release -a
LSB Version:	:core-4.1-amd64:core-4.1-noarch
Distributor ID:	RedHatEnterprise
Description:	Red Hat Enterprise Linux release 8.1 (Ootpa)
Release:	8.1
Codename:	Ootpa

>> conda --version
conda 4.9.2
>> python -c "import sys; print(sys.version)"
3.8.3 | packaged by conda-forge | (default, Jun  1 2020, 17:43:00)
[GCC 7.5.0]

##  gdal is installed using conda:
>> conda install -c conda-forge gdal
>> conda list gdal
packages in environment at ~/file_back/programs/andes/anaconda3:

Name                    Version                   Build  Channel
gdal                      3.0.4            py38h172510d_4    conda-forge
libgdal                   3.0.4                h9e6407a_4    conda-forge

My environment is a bit unusual that my default SHELL is /bin/zsh, but for some reasons I have to switch to using /bin/bash on the server. Yet Jupyter is started with zsh, which probably messed things up. Web remote Jupyter connection executes my RemoteCommand bash -l, which doesn't have this problem; while Remote-SSH doesn't respect RemoteCommand in the ssh configuration
It might be worth trying to force Jupyter to start with bash, which I don't know how.
I am sorry I cannot set up a local environment for you to reproduce it, but I would like to explain more information to you, if it will help.
Thank you so much!

For any one reads this in similar situation, Jupyter via Remote-SSH uses the default SHELL, and even if you specify a different SHELL through RemoteCommand in ssh config, it's not respected. Please read the comment above to change your environment variable is needed, or change the default shell instead.

Thanks for looping back with the solution in case any one else hits this.


Fix trailing commas and missing json marker

I am parsing the documentation in order to generate data types for the Go programming language and encountered some simple consistency issues. This pull request adds a json marker to the markdown and removes several trailing commas in JSON payloads.

Cool idea! We have a tool that we use to generate test cases and other collateral directly out of the documentation. You might find the information in JSON format in an HTML comment before most code blocks to be interesting/useful as well.
You might also want to look at https://api.onedrive.com/v1.0/$metadata which provides CSDL for the entities and actions defined in the service.

Thanks for the pointer I'll check that out and see if is a better source for generating these sorts of data types.


Implement "publish on release"

Publish to NPM when we tag and create a release: https://medium.com/faun/automate-your-npm-publish-with-github-actions-dfe8059645dd

It seems very interesting for me, May I help you?
For testing purposes, are you thinking about CI/CD using for example  travis ?

Hi @hugolesta ! If I got it right, this change will require me linking my personal npm account with this repo's settings. That's why I'd like to do it myself.
The CI/CD would definitely be awesome. To get more value though, perhaps we want to add some tests. Is that something you have experience with? Feel free to create an issue and add a harness, if you want, that we can later integrate in Travis or a similar service.

Hello xnt, it sounds awesome, yep I have experience in automation using travis and other tools, perhaps I could open new issue but first let me think what we can perform to, probably I can build a dummy project using docker and upload to someplace like heroku or something like that. Let me know if you've some suggestion.

Nope, sounds awesome. Try creating a separate issue if possible, though.


Integrate a new API: Get the list of projects in a repository

https://developer.github.com/v3/projects/#list-repository-projects

done in #365


hashring replica point concatenation can create slightly imbalanced rings in particular circumstances

Replica points are added by concatenation onto the server name, which is currently of the form a.b.c.d:X. If you have a=<IP_ADDRESS>:55 and b=<IP_ADDRESS>:555, you will have some overlap, e.g. with replica points 5 and 55, respectively, where a + '55' === b + '5', and therefore hash identically.
This is probably an atypical configuration, and probably not a major concern, but as we consider a future with logical IDs, this could become a bigger of a potential question mark.
Probably the simplest thing is simply to insert '/' or '#' in between. In any consistent naming scheme, this would make all points unique.
Some trivial code to demonstrate this is below -- no matter how much you adjust the offset, 's1:11' is always underrepresented.
test('random hosts with good points show good distribution', function t(assert) {
    var ring = new HashRing({ replicaPoints: 100 });
    var servers = ['s1:1', 's1:11', 's2:2', 's2:5'];
    var counts = {};
    for (var i = 0; i < servers.length; i++) {
        var server = servers[i];
        ring.addServer(server);
        counts[server] = 0;
    }

    var offset = 20 * 1000000;
    for (var i = 0; i < 1000000; i++) {
        s = ring.lookup((i + offset) + '');
        counts[s]++
    }
    console.log(counts);
    assert.end();
});

@CorgiMan Recently found the same. Although the fix is simple it is hard to deploy to running systems without a lot of forwarding inconsistencies. The moment you start upgrading your system after such a fix there will be disagreement on key ownership till the upgrade has completely completed.
Since the chances of such port configurations are slim in our current situation it might break more than it solves.


Linking issue

I'am trying to compile my own software that uses libpointmatcher. However, when linking, I get the following error I couldn't solve in any way:
Linking CXX executable /home/simone/catkin_ws/devel/lib/dem_simulator/dem_simulator_test
/home/simone/libpointmatcher/build/libpointmatcher.so: undefined reference to `boost::filesystem::detail::initial_path(boost::system::error_code*)'
/home/simone/libpointmatcher/build/libpointmatcher.so: undefined reference to `boost::filesystem::detail::status(boost::filesystem::path const&, boost::system::error_code*)'
/home/simone/libpointmatcher/build/libpointmatcher.so: undefined reference to `boost::filesystem::path::extension() const'
/home/simone/libpointmatcher/build/libpointmatcher.so: undefined reference to `boost::filesystem::absolute(boost::filesystem::path const&, boost::filesystem::path const&)'
/home/simone/libpointmatcher/build/libpointmatcher.so: undefined reference to `boost::filesystem::path::parent_path() const'
/home/simone/libpointmatcher/build/libpointmatcher.so: undefined reference to `boost::filesystem::path::operator/=(boost::filesystem::path const&)'
collect2: error: ld returned 1 exit status

I am using the following CMakeLists.txt:
cmake_minimum_required (VERSION 2.8)
project(dem_simulator)

find_package(catkin_simple REQUIRED)
find_package(Boost COMPONENTS system filesystem REQUIRED)
find_package(libpointmatcher REQUIRED)

catkin_simple(ALL_DEPS_REQUIRED)

add_definitions(--std=c++11)

include_directories(${CMAKE_CURRENT_BINARY_DIR} ${Boost_INCLUDE_DIRS} ${libpointmatcher_INCLUDE_DIRS})

cs_add_library(${PROJECT_NAME} src/dem_simulator.cc)
cs_add_executable(dem_simulator_test src/main.cc)

target_link_libraries(dem_simulator_test ${PROJECT_NAME} ${catkin_LIBRARIES} ${libpointmatcher_LIBRARIES} ${Boost_LIBRARIES})

cs_install()
cs_export()

which version of boost are you using?

I am using libboost 1.54

Just FYI. I also am having this problem. Also with Boost 1.54. If I can help at all from ETH, let me know :)

ok, I'm investigating this issue now. It seems that boost is not link to the library but only to the example applications, which make the all thing work within our build hierarchy but most probably not for external projects. It seems that drhalftone was reporting the same problem.
What is the output of  ${libpointmatcher_LIBRARIES} on your side?
In your CMakeLists.txt, you can add:
message("libpointmatcher_LIBRARIES= ${libpointmatcher_LIBRARIES}")

Hi Francois,
I've already solved the problem on my machine. Sorry I forgot to send a
message with the fix. If I remember correctly, all I had to do was add a
link to the Boost libraries for 'pointmatcher'.
Line 215 of CMakeLists: target_link_libraries(pointmatcher
${yaml-cpp_LIBRARIES} ${libnabo_LIBRARIES} ${Boost_LIBRARIES})
Hopefully this works for everyone.
Best,
Tim
On Thu, Feb 19, 2015 at 6:13 PM, François Pomerleau <
<EMAIL_ADDRESS>wrote:

ok, I'm investigating this issue now. It seems that boost is not link to
the library but only to the example applications, which make the all thing
work within our build hierarchy but most probably not for external
projects. It seems that drhalftone was reporting the same problem.
What is the output of ${libpointmatcher_LIBRARIES} on your side?
In your CMakeLists.txt, you can add:
message("libpointmatcher_LIBRARIES= ${libpointmatcher_LIBRARIES}")
—
Reply to this email directly or view it on GitHub
https://github.com/ethz-asl/libpointmatcher/issues/61#issuecomment-75093740
.

@simone-fontana the last commit 48b1a61ddd1d1c19ea6440b6d401c373fc3e9a39 should fix your problem. I also add a small CMakeFile.txt inspired from your example along with an executable to make sure that all information is properly included using find_package(). If you could try on your side, it would be great to confirm everything before I close this issue.
The example project is in /examples/demo_external_link/.
@tsandy thanks for the tip. I cleaned the CMakeList.txt to track the external libraries more easily. There was also some difference whether the library was static or dynamic. If you could update and confirm that it cover your case too, it would be great!
Also, if everything work on your side, could you give me your system spec. It seems that you're on a more recent system than me and would like to add that in the table on the top of tutorial page. All the commands to quickly write the versions are at the end of that page.
Cheers!

I'm still having problems.  Here is the error message:
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang++ -headerpad_max_install_names -Wl,-syslibroot,/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.10.sdk -mmacosx-version-min=10.7 -Wl,-rpath,/Qt/5.4/clang_64/lib -o LAUPointMatcher.app/Contents/MacOS/LAUPointMatcher main.o laupointmatcherwidget.o moc_laupointmatcherwidget.o   -F/Qt/5.4/clang_64/lib -L/usr/local/lib/ -lboost_thread-mt -lboost_filesystem-mt -lboost_system-mt -lboost_program_options-mt -lboost_date_time-mt -lboost_chrono-mt -framework QtWidgets -framework QtGui -framework QtCore -framework DiskArbitration -framework IOKit -framework OpenGL -framework AGL
Undefined symbols for architecture x86_64:
"PointMatcher::DataPoints::Label::Label(std::string const&, unsigned long)", referenced from:
LAUPointMatcherWidget::LAUPointMatcherWidget(QWidget*) in laupointmatcherwidget.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [LAUPointMatcher.app/Contents/MacOS/LAUPointMatcher] Error 1
22:54:26: The process "/usr/bin/make" exited with code 2.
Error while building/deploying project LAUPointMatcher (kit: Desktop Qt 5.4.0 clang 64bit)
When executing step "Make"
Here is the Qt Project file that I'm using:
QT       += core gui widgets
TARGET    = LAUPointMatcher
TEMPLATE  = app
SOURCES  += main.cpp
laupointmatcherwidget.cpp
HEADERS  += laupointmatcherwidget.h
INCLUDEPATH += /Users/dllau/Libraries/libpointmatcher/pointmatcher
/usr/local/include/eigen3/
/usr/local/include/
LIBS     += -L/usr/local/lib/ -lboost_thread-mt -lboost_filesystem-mt -lboost_system-mt -lboost_program_options-mt -lboost_date_time-mt -lboost_chrono-mt
/Users/dllau/Libraries/libpointmatcher/build/libpointmatcher.a
Here is the header for the object that is triggering the undefined symbols:
#include
#include "PointMatcher.h"
class LAUPointMatcherWidget : public QWidget
{
Q_OBJECT
public:
LAUPointMatcherWidget(QWidget *parent = 0);
~LAUPointMatcherWidget();
private:
PointMatcher::DataPoints::Label label;
};


Eliminate homepage redirect?

@ritch has noted that the redirect from http://expressjs.com/ to http://expressjs.com/en/index.html is somewhat annoying.  It would be better if we could eliminate the redirect and just load the English home page directly (by default).
It might also be slick to detect the user's (or browser's) locale setting and load the appropriate localized home page.  Not sure how easy that would be.

huh. I felt like that was something new. It is quite annoying, as typically I will be looking at that page for several seconds.

Yes, it's relatively new, and allows us to "fallback" to English pages when there is no localized page.
A very simplistic solution would be to simply duplicate en/index.md at the top level, and remove the line
redirect_from: "/"
from that page.
We'd need to be sure to copy any changes to the English home page into the top-level home page.
This wouldn't present the appropriate home page for non-en locales, though.  I'm not sure how we could do that.   I think that could be considered an enhancement.
@hacksparrow Is there any drawback or "gotcha" to my suggestion above?

The home page will no longer redirect to /en/.
@crandmck It's not a good idea to have duplicates, for search engines as well as dev/maintenance. I moved index.md from /en/ with a directive to handle the request to /en.


Broken connections are reused in connection pools

Describe the bug
If an application has a connection break with the network, it accumulates broken connections with the Redis server and tries to reuse them to execute commands. This is due to the fact that the connect() methods do not check whether an exception is set in the StreamReaders:
class Connection:
    async def connect(self):
        """Connects to the Redis server if not already connected"""
        if self.is_connected:   # <-- self._reader may contain an exception (e.g. OSError No route to host).
            return

class SentinelManagedConnection:
    async def connect(self):
        if self._reader:  # <-- self._reader may contain an exception (e.g. OSError No route to host).
            return  # already connected

This cases should be handled.
To Reproduce

Initialize base Redis client or Sentinel client, connect to Redis server
Send pings periodically
Turn off wi-fi
Look at ping errors
Turn on wi-fi
Look at ping errors

Expected behavior
There should be no errors in step 6
Logs/tracebacks
https://gist.github.com/evgenymarkov/1af3a5cb12887068cb9c6a72c09168ea

Python Version
3.9.10

aioredis Version
2.0.1

Additional context
No response
Code of Conduct

[X] I agree to follow the aio-libs Code of Conduct

https://github.com/aio-libs/aioredis-py/pull/1313

My issue can be related with https://github.com/aio-libs/aioredis-py/issues/1174


Create the functions to be able to score whether an evaluation was useful or not

Tasks ✅

[x] Create path operations to evaluate useful and not useful

Done ✅


update(build): updated libs version for Falco 0.32.0 release.

Signed-off-by: Federico Di Pierro<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

update(build): set libs version to bd94f3189774e5298dd383bf913da7b3b620a94a.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 244e393d4ec2df9cd8ee62ceda004a652fe3b7b8

/milestone 0.32.0


Fix typo in nerdtree.txt

Description of Changes
Fix map setting typo in doc/nerdtree.txt.

Thanks!👌


Move Golang Extension from contrib to main extensions

Title: Moving Golang Extension from contrib to main extensions
Description:

Golang is a widely used language and very easy to implement envoy filter extensions using the Envoy's Golang extension.

Do we foresee moving Golang extension from contrib to main extension. I'm not aware of the requirements to mark a extension as contrib or main extension, any details on this would also help.
Also from a stability point of view, where do we see the golang extension as of today?

We are currently looking into adopting Golang extensions for our use cases and use it in production. So any detail would really help and much appreciated.
[Relevant Links:]

https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/golang_filter

@itsLucario Thanks for your insteresting in Golang extension.
We do plan to move it to main, there is a issue, but sorry for busying to not make it done yet.
Moving to main is still high priority in our TODO list.
I think it's stable enough to use it in production, see: https://github.com/envoyproxy/envoy/issues/31685#issuecomment-2354469227

Awesome 🚀 !
Thanks for the update @doujiang24 and great to hear it's in priority TODO 😄
Looking forward to seeing it in standard extensions.


fix: ensure feature segments are cloned correctly

Changes
Currently feature segments are generated for the new environment... but not used at all. This PR ensures that feature segments are cloned correctly and used on the resultant feature states.
How did you test this code?
Updated / added unit tests.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (5b46cc0) 95.47% compared to head (dd16278) 95.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2706   +/-   ##
=======================================
  Coverage   95.47%   95.47%
=======================================
  Files         986      986
  Lines       27709    27728   +19
=======================================
+ Hits        26454    26473   +19
  Misses       1255     1255

Files Changed
Coverage Δ

api/environments/models.py
95.36% <ø> (-0.05%)
:arrow_down:

api/features/models.py
93.60% <100.00%> (+0.01%)
:arrow_up:

api/tests/integration/conftest.py
100.00% <100.00%> (ø)

...integration/environments/test_clone_environment.py
100.00% <100.00%> (ø)

...i/tests/unit/features/test_unit_features_models.py
100.00% <100.00%> (ø)

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


[SPARK-22450][Core][MLLib][FollowUp] Safely register MultivariateGaussian

What changes were proposed in this pull request?
register following classes in Kryo:
"org.apache.spark.ml.stat.distribution.MultivariateGaussian",
"org.apache.spark.mllib.stat.distribution.MultivariateGaussian"
How was this patch tested?
added tests
Due to existing module dependency, I can not import spark-core in mllib-local's testsuits, so I do not add testsuite in org.apache.spark.ml.stat.distribution.MultivariateGaussianSuite.
And I notice that class ClusterStats in ClusteringEvaluator is registered in a different way, should it be modified to keep in line with others in ML? @srowen

Test build #98590 has started for PR 22974 at commit 0b9ed17.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4842/
Test PASSed.

Test build #98590 has finished for PR 22974 at commit 0b9ed17.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98590/
Test FAILed.

Can you explain the issue here? Spark's tests work fine right now.
mllib-local is not supposed to depend on core.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4874/
Test FAILed.

Test build #98635 has started for PR 22974 at commit 90a4d54.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4875/
Test PASSed.

@srowen Existing kryo-register testsuite need to import spark-core:
import org.apache.spark.SparkConf
import org.apache.spark.serializer.KryoSerializer

val conf = new SparkConf(false)
conf.set("spark.kryo.registrationRequired", "true")
val ser = new KryoSerializer(conf).newInstance()

Since mllib-local is not dependent on spark-core, current classes in mllib-local do not test kryo-serialization at all. E.g. mllib.linalg.VectorsSuite contains test test("kryo class register"), while ml.linalg.VectorsSuite  do not have it.

You're requiring registration, which is what makes this fail, right? why do that? I think I'm missing something.

Do you mean fail in this pr? It was caused by a non-registered filed BDM[Double].
MultivariateGaussian is used in GMM, kryo-registration should help performance.
As to mllib-local's dependency, it is another thing: current kryo-regiestered classes, like 'ml.linalg.Vector', 'ml.linalg.Matrix', do not have kryo test in their testsuites.

OK, that's the issue, yeah. Registration is an optimization. I wonder, what other classes should we add if we're going to add this one? I don't know if it needs a test. But if there are 10 other somewhat commonly-used classes that are serialized during Spark ML operations, they should be registered.

I am not sure, but maybe all serializable classes need to be registered.
Since MultivariateGaussian is a public class, so I think we need to add it.
I also wonder whether a test is needed. If no longer needed, I can list all other public ones in ML in this PR.

not all public serializable classes are needed to registered. Only those one which needed ser-deser should be registered, one important groups should be transformers and prediction models.

Test build #98635 has finished for PR 22974 at commit 90a4d54.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98635/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4881/
Test PASSed.

Test build #98643 has started for PR 22974 at commit 2fc7247.

Test build #98643 has finished for PR 22974 at commit 2fc7247.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98643/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4885/
Test PASSed.

Test build #98648 has started for PR 22974 at commit 2fc7247.

Test build #98648 has finished for PR 22974 at commit 2fc7247.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98648/
Test FAILed.

OK, I wonder if you have time to take a quick pass over the "Serializable" classes in MLlib to identify a few more that are likely important to register with Kryo? there are probably a few more. If so then that would help improve this all at once rather than class by class.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4938/
Test PASSed.

Test build #98712 has started for PR 22974 at commit 7e97e45.

Test build #98712 has finished for PR 22974 at commit 7e97e45.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98712/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4945/
Test PASSed.

Test build #98719 has started for PR 22974 at commit 0c529fb.

Test build #98719 has finished for PR 22974 at commit 0c529fb.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98719/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4946/
Test PASSed.

Test build #98720 has started for PR 22974 at commit 0c529fb.

Test build #98720 has finished for PR 22974 at commit 0c529fb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98720/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4960/
Test PASSed.

Test build #98747 has finished for PR 22974 at commit b0eb584.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98747/
Test FAILed.

Test build #98757 has started for PR 22974 at commit d965752.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4969/
Test PASSed.

Test build #98757 has finished for PR 22974 at commit d965752.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98757/
Test FAILed.

retest this please

Test build #98762 has started for PR 22974 at commit d965752.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4974/
Test PASSed.

Test build #98762 has finished for PR 22974 at commit d965752.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98762/
Test FAILed.

Hm. On the one hand, I don't actually think we need to test that certain classes are registered except for some performance critical ones, maybe. However if a class can't be registered except by registering every class it touches (like internal anonymous ones), it may not have any effect to register many of these; Kryo won't work. Further, it may then break usages that have kryo registration required otherwise. I guess back out any registration of classes that seems to require a bunch of anonymous classes be registered too?

@srowen Yes, this is the problem. I have to register Param* before any prediction model, but there are too many anonymous classes  in ParamValidators and other places, and I have not found an efficient way to list those anonymous ones.
So I wonder whether we could continue to register all of them.  Or only list simple classes like MultivariateGaussian ?

@zhengruifeng you are right, let's stay conservative. Thanks for investigating. Are there any simple, commonly-used classes that should be serializable by themselves without registering other classes, anything else obvious to add? it may just be MultivariateGaussian

Test build #98855 has started for PR 22974 at commit ec9b83f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5040/
Test PASSed.

Test build #98855 has finished for PR 22974 at commit ec9b83f.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98855/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5047/
Test PASSed.

Test build #98864 has started for PR 22974 at commit ec9b83f.

Test build #98864 has finished for PR 22974 at commit ec9b83f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98864/
Test PASSed.

Merged build finished. Test PASSed.

Merged to master


WebControlExtension.Exists() fails when control does not exist

Fails on getting control.Instance
Throws ControlNotFound

Confirm, code fails in case when control is absent. It appears that the extension is obsolete (forgot to mark it).
There is a new UI agnostic extension appeared in UI.Core lib starting from version 3.1.0. @nortimirino93 please use it instead.
Unicorn.UI.Core.PageObject.ControlExtension.ExistsInPageObject


prevent upscaling

Is there a way to disable upscaling? If I use
->focusCrop(1200,600)

and if the image is just 800px width it blows it up to 1200px.

Thanks for your feedback. This should be possible and I'll check it out.
But what do you think should happen, if only one dimension (width or height) is larger than the original?
// original 800px x 800px

->focusCrop(1200,600)

Possible results

Crop only the height: 800px x 600px
Crop to desired reatio: 800px x 400px
Don't crop at all: 800px x 800px

I would prefer to crop to the desired ratio (2).

Crop means crop, not CropAndScale. For this reason I would select 1 :)

I also prefer 1, but I will check how the Kirby crop() method handles this.

Alright, Kirby crop() makes an upscale to match the larger value and then crops the image to the final dimensions.


Generating plugin.js without digest

Currently assets:precompile generates plugin.js with digest, but tinymce tries to load plugin.js file. After my modification, assets:precompile task creates public/assets/tinymce/plugins/uploadimage/plugin.js file.

Closing due to the lack of activity but everything seems fine on latest version, reopen if you still see this issue on master.


⚠ Jenkins Plugins Artifactory has degraded performance

In c92d313, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1301 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 6521079 after 49 minutes.


[Feature] Multi-level categories

Problem
Organizing related notes within sub-categories is essential for keeping semantically related notes together but separate
Suggested Approach
Ability to add a category under another category, or add a note to multiple categories.
Note

We need to consider the UI behavior of the sidebar when there are too many levels to display
If a category of the same name is dragged in, what happens? Do we append something like (1)?

I think about notes and categories like this:

Categories: One (category) to many (notes)
Tags: Manu (tags) to one (category)

While writing this, I just decided on categories for simplicity, because if I could only have one I'd choose categories. I would propose, if anything, a tag system in addition to categories as opposed to multi-level category hierarchies.

Ah I was thinking of categories like folders. One category to many (notes | categories) just like a standard file system.
I'm trying to think how I would categorize something like:
I have a set of notes for work, school, and hobbies (three categories) and then for work I have a sub-category for each of the three features each with three stories.
So with how you described it I might:

Make nine categories for work
Tag them

work, feature 1, story 1
work, feature 1, story 2
work, feature 1, story 3
...
work, feature 3, story 3

Make nine categories for school
Make nine categories for hobbies

Resulting in a huge flat list with a large set of tags.

I'm just thinking if you can nest categories, how much nesting can you do? Infinite nesting? Would it be easy for the UI to handle that? I think adding tags solves the problem without having to worry too much about that, while satisfying the second part of your suggested approach ("or add a note to multiple categories" - in this case, add a note to multiple tags). Perhaps adding tags and allowing one level of sub-categories would be sufficient.

@taniarascia @codytowstik
A tag system makes sense to me. We can add tags within each file and then we can add a 'Tags' option beneath 'Categories'.
Question(s) about the implementation:
How are we tagging each note? A note taking app called 'Bear' does this with this syntax #tag—any text written with a # next to it will be deemed a tag. If we do something similar, I was thinking of using regex to search for this (and then update the tags option) whenever you focus on a different note (search in the note you were just in) or when you sync (search in the current note).

I think a tag system would be a good approach for clean and simple organization :)

@taniarascia I just thought about this again. I must have misread your description. I think a simple implementation would be to add an 'add tags' field to the category context menu. Tags can then be viewed within the category by clicking on the category icon. I think it would be useful to display the tags a note is associated with a note in the editor.
Perhaps creating a component that can be hidden/expanded that holds frontmatter data (or just the tags) that sits above the editor is the way to go? When you expand it, the editor shrinks in height and when you hide it, the editor looks like it does currently.

I'm going to close this ticket in favor of potentially adding a tag system.


EKSA CLI Pre-flight validation for AWS authentication issues

What would you like to be added:

Post creation of EKSA Cluster, we were not able to deploy Emissary Curated Packages to Workload clusster, due to authentication issue with AWS API's.
Root cause was that AWS CLI Secret Access key was incorrect.

Why is this needed:

Curated Packages is feature supported for License Paying customers. EKSA command line should add details when CLI fails with authentication issues for AWS API endpoint.

Thanks for reporting that. Did you set up environment variables EKSA_AWS_SECRET_ACCESS_KEY, EKSA_AWS_ACCESS_KEY_ID and EKSA_AWS_REGION when you were creating the cluster?  You can refer to our doc for details.


Issue with relative path and Get-SCPFolder

I am trying to use Get-SCPFolder with a relative path but it seems to be using the wrong full path when executing the command.

I am able to make an SSH connection using Posh-SSH.
I have 3 drivers mapped on my machine C, E, and H
When I used a relative path with Set-SCPFolder it works as expected  Set-SCPFolder -LocalFolder ".." -RemoteFolder "/root/SB-36030S1" -ComputerName $sship -Credential $sshcred -Port $sshport -AcceptKey
Notice my working directory is "E:\AlsProjects\SB-36030S1\factory" and for the parameter -LocalFolder I am using the path "../libraries/sb36030/bin" so this should not change driver letters. Looking at the error when the path is expanded into a full path it's trying to use another driver. "H:\ddccm\bin\ddccm"

CODE:
    $pathtest=pwd
    Write-Host "PWD="$pathtest
    Get-SCPFolder -LocalFolder "../libraries/sb36030/bin" -RemoteFolder "/root/SB-36030S1/libraries/sb36030/bin" -ComputerName $sship -Credential $sshcred -Port $sshport
    $pathtest=pwd
    Write-Host "PWD="$pathtest
    Get-SCPFolder -LocalFolder "../apps/stopmotion/bin" -RemoteFolder "/root/SB-36030S1/apps/stopmotion/bin" -ComputerName $sship -Credential $sshcred -Port $sshport
    $pathtest=pwd
    Write-Host "PWD="$pathtest
    Get-SCPFolder -LocalFolder "../ddccm/bin" -RemoteFolder "/root/SB-36030S1/ddccm/bin" -ComputerName $sship -Credential $sshcred -Port $sshport
    $pathtest=pwd
    Write-Host "PWD="$pathtest

OUTPUT
PWD= E:\AlsProjects\SB-36030S1\factory
PWD= E:\AlsProjects\SB-36030S1\factory
Get-SCPFolder : Could not find a part of the path 'H:\apps\stopmotion\bin\stopmotion'.
At E:\AlsProjects\SB-36030S1\factory\vm_compile.ps1:197 char:5
+     Get-SCPFolder -LocalFolder "../apps/stopmotion/bin" -RemoteFolder ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Renci.SshNet.ScpClient:ScpClient) [Get-SCPFolder], DirectoryNotFoundException
    + FullyQualifiedErrorId : SSH.GetScpFolder

PWD= E:\AlsProjects\SB-36030S1\factory
Get-SCPFolder : Could not find a part of the path 'H:\ddccm\bin\ddccm'.
At E:\AlsProjects\SB-36030S1\factory\vm_compile.ps1:200 char:5
+     Get-SCPFolder -LocalFolder "../ddccm/bin" -RemoteFolder "/root/SB ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Renci.SshNet.ScpClient:ScpClient) [Get-SCPFolder], DirectoryNotFoundException
    + FullyQualifiedErrorId : SSH.GetScpFolder

PWD= E:\AlsProjects\SB-36030S1\factory

As a workaround to the issue I determined the absolute path myself and passed it to the command. The path I pass into -LocalFolder is "E:\AlsProjects\SB-36030S1\apps\stopmotion\bin" but the command fails looking for the path "E:\AlsProjects\SB-36030S1\apps\stopmotion\bin\stopmotion"
CODE2
    $testvar=Resolve-Path -LiteralPath ".."
    $pathroot=$testvar.Path

    $pathtest=pwd
    $localpath = Join-Path $pathroot "/libraries/sb36030/bin"
    Write-Host "PWD ="$pathtest
    Write-Host "Local Pwth ="$localpath
    Get-SCPFolder -LocalFolder $localpath -RemoteFolder "/root/SB-36030S1/libraries/sb36030/bin" -ComputerName $sship -Credential $sshcred -Port $sshport
    $pathtest=pwd
    $localpath = Join-Path $pathroot "/apps/stopmotion/bin"
    Write-Host "PWD ="$pathtest
    Write-Host "Local Pwth ="$localpath
    Get-SCPFolder -LocalFolder $localpath -RemoteFolder "/root/SB-36030S1/apps/stopmotion/bin" -ComputerName $sship -Credential $sshcred -Port $sshport
    $pathtest=pwd
    $localpath = Join-Path $pathroot "/ddccm/bin"
    Write-Host "PWD ="$pathtest
    Write-Host "Local Pwth ="$localpath
    Get-SCPFolder -LocalFolder $localpath -RemoteFolder "/root/SB-36030S1/ddccm/bin" -ComputerName $sship -Credential $sshcred -Port $sshport
    $pathtest=pwd
    Write-Host "PWD="$pathtest

OUTPUT2
PWD = E:\AlsProjects\SB-36030S1\factory
Local Pwth = E:\AlsProjects\SB-36030S1\libraries\sb36030\bin
PWD = E:\AlsProjects\SB-36030S1\factory
Local Pwth = E:\AlsProjects\SB-36030S1\apps\stopmotion\bin
Get-SCPFolder : Could not find a part of the path 'E:\AlsProjects\SB-36030S1\apps\stopmotion\bin\stopmotion'.
At E:\AlsProjects\SB-36030S1\factory\vm_compile.ps1:204 char:5
+     Get-SCPFolder -LocalFolder $localpath -RemoteFolder "/root/SB-360 ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Renci.SshNet.ScpClient:ScpClient) [Get-SCPFolder], DirectoryNotFoundException
    + FullyQualifiedErrorId : SSH.GetScpFolder

PWD = E:\AlsProjects\SB-36030S1\factory
Local Pwth = E:\AlsProjects\SB-36030S1\ddccm\bin
Get-SCPFolder : Could not find a part of the path 'E:\AlsProjects\SB-36030S1\ddccm\bin\ddccm'.
At E:\AlsProjects\SB-36030S1\factory\vm_compile.ps1:209 char:5
+     Get-SCPFolder -LocalFolder $localpath -RemoteFolder "/root/SB-360 ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Renci.SshNet.ScpClient:ScpClient) [Get-SCPFolder], DirectoryNotFoundException
    + FullyQualifiedErrorId : SSH.GetScpFolder

It looks like you already had to fix this before in several other places and it was just missed in Get-SCPFolder
File GetScpFolder.cs line 422:
replace:
                        var localfullPath = Path.GetFullPath(_localfolder);
with:
                        // Resolve the path even if a relative one is given.
                        ProviderInfo provider;
                        var pathinfo = GetResolvedProviderPathFromPSPath(_localfolder, out provider);
                        var localfullPath = pathinfo[0];

Just pushed version 2.0.2 in to the gallery. Reopen if the issue persist.


Re-queue labelled tasks.

Describe the solution you'd like
I have an image set with cars. I've labelled all of my images and I am getting message "No More Tasks Left in Queue". After some time, I've decided to provide new labels to my project. Now I want to label also trucks. I would like to bring back all of my currently labelled images to the queue. However, I don't want to delete current annotations. I just want to have images  back in the queue and create a new annotation based on the one that I have already created. That annotation should show in annotation history like usually.
Describe alternatives you've considered
You can copy current annotation for each individual image and submit that annotation. The problem with that solution is that we don't have a clear divide between tasks considered as labelled and the ones  that needs to be labelled.

So there really is no function for that?

All I want to do is add annotations to an already existing, partially annotated dataset and I am disgusted by how hard it is to achieve ANYTHING with this label-studio bs
loading a locally stored, pre annotated dataset -> here's the most complicated way a human could possibly come up with
labeling images with annotations? -> can't do

I am facing the same issue. If there is any kind of workaround. it would be a huge help.


Remove deprecated Pod Security Policies

Describe the solution you'd like
https://docs.microsoft.com/en-us/azure/aks/use-pod-security-policies
As you can see, pod security policy is getting deprecated on the 15th of October, superseded by Azure policies for AKS
https://docs.microsoft.com/en-us/azure/aks/use-pod-security-on-azure-policy
Since I use Azure policies for pods, I'd like if we update the PSP definitions with the newer definitions
Anything else you would like to add:
Environment:

Secrets Store CSI Driver version: (use the image tag): 0.0.13
Azure Key Vault provider version: (use the image tag): 0.0.8
Kubernetes version: (use kubectl version): 1.18.6
Cluster type: (e.g. AKS, aks-engine, etc): AKS

How do you even enable pod security policies in deployment/pod-security-policy.yaml. setting rbac.enable: true still gives an error of Host network is not allowed to be used and Host port 9808 is not allowed to be used. There is nothing else in values.yaml that covers enabling pod security policies

How do you even enable pod security policies in deployment/pod-security-policy.yaml. setting rbac.enable: true still gives an error of Host network is not allowed to be used and Host port 9808 is not allowed to be used. There is nothing else in values.yaml that covers enabling pod security policies


Fix the header in home and help page

Working on it sir

Sir I fixed the Header thing.
Need to know there to place the toggle button for modes and that select language thing

Like this

Sir I fixed the Header thing.
Need to know there to place the toggle button for modes and that select language thing

Also, zoom and select language buttons are in only home page not on the help page.

Done sir.
Can i now go for PR

Sure.
Also, share screenshots of both pages and a live demo link so I can review your changes.

Done sir
Can i now go for a PR

Fine

Headers of Help page as well as home page
Rectified.

sir guide me through if My PR was successful or not

@stellashank u=you have not made PR yet.

After making changes you have to click on the make pull request button.

Done sir. Do u confirm?!


Warn about async rendering of Collections in README

We spent nearly an hour to find out that automatic re-rendering of Collections does indeed work. We took so long, since the same Jasmine code to assert the changed DOM did not work on collections. (We mocked the AJAX call with a sync callback, so we assumed nothing is async).
However, after looking in the code, rendering of a single model is done in sync and rendering of a collection is done async (using debounce of UnderscoreJS).
I think it would be good to at least warn in the README about this behaviour to spare others the wasted time. Better yet, both update mechanisms should work the same way.

I agree, I think we shouldn'the make promises on updating synchronously (react doesn'the really do this either).  A warning would help. I'm not sure on making model asynchronous as well. It would make this problem more clear, but it would make debugging harder (I think). But a PR for an improved readme is very welcome of course


uv init normalize directory name

Summary
Resolves #5255

We may need to revisit the normalization for dots (e.g. foo.bar) later, but always using underscores doesn't seems correcter than the current solution

Do we reject packages with invalid names, like uv init "foo bar"? What happens there?

error: Not a valid package or extra name: "foo bar". Names must start and end with a letter or digit and may only contain -, _, ., and alphanumeric characters.


reset develop on latest master, and several others

These commits are based on master, not develop, since develop was modified after 1.0.0 and before 1.0.1.  As such, this will require a force-push to develop.
It includes these outstanding PRs (as well as other commits):

https://github.com/marklogic/marklogic-samplestack/pull/584
https://github.com/marklogic/marklogic-samplestack/pull/575
https://github.com/marklogic/marklogic-samplestack/pull/573

Please seen individual commits.
Note: there is a major reorganization of the Javascript automation code in here, which means that:

no longer necessary to the developer to install bower globally (sudo npm install -g bower) at all
bower installation is automatically performed during npm install (developer doesn't need to run bower install at all)
npm install and all gulp tasks are from from the root of the repository instead of in the browser sub-directory

Sounds big, exciting!

These commits are based on master, not develop, since develop was modified after 1.0.0 and before 1.0.1. As such, this will require a force-push to develop.

Does this mean you are pushing commits that made it to master (via rc) back to develop to get it up to date? If I want to switch back to develop do I need to do something special (this force-push)?

Sounds good to me @laurelnaiad

@popzip asked

Does this mean you are pushing commits that made it to master (via rc) back to develop to get it up to date? If I want to switch back to develop do I need to do something special (this force-push)?

It means that when this merges, develop will be unseated from where it sits today so as to rebase it onto the commit that happened post 1.0.0.
Yes, it means that when this lands, you'll need to reset develop to HEAD.
git checkout develop
git fetch origin
git reset --hard HEAD

ought to do it.  Unless you have changes in your local copy of develop that you are intending to preserve and push up to a remote branch some day, the above is always a safe operation.


Variable Blocks no longer include a dropdown

Expected Behavior
When making a new variable, which creates a variable block, that variable block should have a dropdown in order to be able to select different existing variables.
Actual Behavior
The variable block that is created has no dropdown.
Steps to Reproduce
Create 2 variables. Notice when making the 2nd one there is no way to switch it to the 1st one.
Operating System and Browser
All
Notes
Ok, I know this is not a bug but instead a design choice and how Scratch works... but this behavior is different from how Blockly does it (which is how this repo has had it for a long time). I'm posting this because I'd like to ask some questions around this, and am not sure of a better place (please let me know if there is indeed a better place for this sort of discussion).
I assume Scratch 3.0 will follow how Scratch 2.0 has this and NOT include the drop downs by default. Why is this the case? It seems inconsistent to have things this way but I have a feeling you guys have good reasons... I'm curious what they are.
Also, if we choose to follow the Blockly design... I think we can maintain that behavior by simply reverting some of the recent changes. Is there any reason you can see where this would be a real issue to maintain?

I assume you're talking about the fact that there's no dropdown on the "foo" and "bar" reporter blocks above, even though there's a dropdown on the "bar" in the variable setter block.
Scratch lets you click on a block to see its value at runtime:

Why does that mean there's no dropdown?
If we wanted to be able to use a dropdown and tap on a block, we would have to have two touch targets in a small area.  One way to do that is to use the dropdown triangle as the touch target for opening the dropdown, and the text for evaluating the variable.  That's annoying anywhere else you use a dropdown (tiny touch target) and it isn't actually clear why sometimes you get the dropdown and sometimes you evaluate the variable--especially on a touch screen or if the user isn't great at using a mouse yet.
How do I select another variable?
You should be able to right-click on the variable getter and select a variable from the context menu, as you can in Scratch 2.0, but that looks broken right now (#964).
Why do it this way in Scratch-Blocks/why not in Blockly?
For Scratch/scratch-blocks: tinkerability.  Blockly doesn't do that, in part because Blockly doesn't have a runtime and therefore has no idea what the value of a block would be.  Obviously someone could add this behaviour into Blockly as part of integrating it into an app, if they felt like it.
Can I just revert recent changes to get the Blockly behaviour?
You can probably use a field_variable instead of a field_variable_getter to get the dropdown menu back.  Offhand I'm not sure how well or poorly that would work.  You will run into similar usability issues with evaluating the variable on tap, though, if that's a thing that you want.

That makes sense now! Thanks for the explanation :)
I know we certainly want to have features that expose the runtime value of variables but I don't think we've gotten to designing that part yet. Good to know this is closely related to this question of variable drop downs.

One quick question: why not make the set (and changeBy) blocks follow that design? Meaning when the variable shows in these blocks... have no dropdown by default? Is that in these cases it's more common to change the variable name and less common to inspect the value?

Clicking on a set variable block runs the block (in scratch) if you clicked anywhere other than an editable field, and edits the field if you clicked on an editable field.  Clicking on a block doesn't display its value if it's a statement block.
Since there's so much more space to be touch targets, and since the block and its field are clearly delineated, it's much easier to discern intent.  As a result clicking on the set block runs it and clicking on the variable field edits it.

Got it, thanks for the explanation!

Note that when you have a dropdown, it's technically possible to drop a block inside that dropdown (though that may be disabled via UI) and get the value of a variable by its name "dynamically" (example). Similar capability is discussed here and here; it might be interesting to read!

@MichaelZawadzki - Thanks for bringing this up.
@rachel-fenichel @liam4 - Thanks for explaining and pointing at previous discussions.


Create mobile renderer

This PR adds a mobile version of the Vulkan renderer to Godot as the full clustered renderer uses a few techniques not supported on mobile hardware. Changes (will) include:

no depth pre-pass
normal light culling as we can't used the clustered approach
no GI and likely no volumetric fog (maybe)
move most things related to the mobile renderer into a subfolder (just like we did for clustered)
instance information is now sent through the push constant instead of a bigger UBO

Got to a point today where it is starting to render stuff unshaded. Rendering of objects is now done through push constants. I do want to pretty up how the push constants are populated
Next step is to get lights to work again

Slow progress today but directional lights seem to work and transparency now works

There are a few bugs that snuck in that I will look at over the weekend, for instance this is totally screwed up now:

Then I'm going to make the shadow render pass work.
I think I'll also make the background work first because it'll make things look much nicer :)

Also currently still crashes when directional_shadow_atlas_set_size is called on exit. Not sure why:

I put back most of the shadow code but it's failing to run it. I'm getting a bunch of errors that the shader variant is not setup:
ERROR: Attempted to use an unused shader variant (shader is null),
   at: (D:\Development\godot4-git\servers/rendering/renderer_rd/pipeline_cache_rd.h:71)
ERROR: Condition "!pipeline" is true.
   at: RenderingDeviceVulkan::draw_list_bind_render_pipeline (drivers\vulkan\rendering_device_vulkan.cpp:6379)
ERROR: The vertex format used to create the pipeline does not match the vertex format bound.
   at: (drivers\vulkan\rendering_device_vulkan.cpp:6586)
ERROR: Attempted to use an unused shader variant (shader is null),
   at: (D:\Development\godot4-git\servers/rendering/renderer_rd/pipeline_cache_rd.h:71)
ERROR: This render pipeline requires (0) bytes of push constant data, supplied: (128)
   at: (drivers\vulkan\rendering_device_vulkan.cpp:6562)
ERROR: No render pipeline was set before attempting to draw.
   at: (drivers\vulkan\rendering_device_vulkan.cpp:6579)
ERROR: Attempted to use an unused shader variant (shader is null),
   at: (D:\Development\godot4-git\servers/rendering/renderer_rd/pipeline_cache_rd.h:71)
ERROR: This render pipeline requires (0) bytes of push constant data, supplied: (128)
   at: (drivers\vulkan\rendering_device_vulkan.cpp:6562)
ERROR: No render pipeline was set before attempting to draw.
   at: (drivers\vulkan\rendering_device_vulkan.cpp:6579)
ERROR: Attempted to use an unused shader variant (shader is null),
   at: (D:\Development\godot4-git\servers/rendering/renderer_rd/pipeline_cache_rd.h:71)
ERROR: This render pipeline requires (0) bytes of push constant data, supplied: (128)
   at: (drivers\vulkan\rendering_device_vulkan.cpp:6562)

I think this is due to the way the shadow variants are parsed in SceneShaderForwardClustered::ShaderData::set_code, looks like it won't include them because we're not using the depth prepass but I could be misreading this code and thus not have changed the code correctly. Old code from the forward clustered shader is commented out in the same block.

Have to commit the changes but the sky works:

Full kudos to @clayjohn though, the code was well separated out of the main engine so all that work I did moving it into its own class paid off. It was really easy to hang it back in there and make it work.
I also have shadows!
I'm still getting the above mentioned errors, I think this is related to the material shader

Just wondering, will we be able to use "mobile" renderer on desktop (power saving on a laptop and/or bad Vulkan support)?

Yup, it'll just be a setting in project settings, similar to GLES2/GLES3 switch

Today I managed to get all the lights, reflection probes and decals working. Reflection probes do have issues but those also are apparent in the clustered renderer.

Tomorrow I'm going to add msaa back in and try to finally figure out why one of the shader variants isn't building causing weird error spam.

I've added MSAA support but there is an issue around the 16x version giving a black screen on my laptop that I still need to look into.
Multimesh seems to work
Bones seem to work.
Both of these basically use the same approach as before loading the needed data into the transform UBO
Also less errors around the shaders so that's nearly working.
Agreed with Reduz that we're going to do further improvements on lightmaps in another PR. I do still want to test if it works (or @JFonS maybe you want to give my branch a quick twirl? In theory they should work)
Basically all thats left is cleanup I think.

Agreed with Reduz that we're going to do further improvements on lightmaps in another PR. I do still want to test if it works (or @JFonS maybe you want to give my branch a quick twirl? In theory they should work)

Lightmaps are not easy to test because unwrapping is not working right now. We can wait until I re-add it and then revisit lightmaps on the mobile renderer.

I've added MSAA support but there is an issue around the 16x version giving a black screen on my laptop that I still need to look into.

See https://github.com/godotengine/godot/issues/37917.

Agreed with Reduz that we're going to do further improvements on lightmaps in another PR. I do still want to test if it works (or @JFonS maybe you want to give my branch a quick twirl? In theory they should work)

Lightmaps are not easy to test because unwrapping is not working right now. We can wait until I re-add it and then revisit lightmaps on the mobile renderer.

I think that's fine because even if the current logic doesn't work we'll have to redo it anyway. The problem as I understand it with the mobile renderer doesn't allow as large buffers to be loaded as on desktop which is also why we're now sending some of the code data right before the draw call.
The plan is to do the same with selecting the lightmap. By sorting our drawbuffer by lightmap we can change the current selected lightmap as few times as possible and access it during rendering, instead of looking up a lightmap without our lightmap buffer.
But reduz already mentioned we should do this in a follow up PR.

I've added MSAA support but there is an issue around the 16x version giving a black screen on my laptop that I still need to look into.

See #37917. Given the high cost of 16× MSAA, I wouldn't bother supporting it in 2021, especially in a renderer targeting mobile.

Buffer, well that explains it. Agreed it probably has no purpose in a mobile renderer but seeing its a global settings it's possible to set it. That may need to be resolved in a follow up PR or in the PR that fixes the MSAA :)

Early results look great! I am getting about half the frame time that I do with the default renderer with a very simple scene.

Thanks!


Enqueue one task per root_promise_id

Enqueue a single task per root_promise_id. This is necessary to prevent extra work happening for a single promise id.

@dfarr I am still unsure how to add dst to this change, any pointers will be appreciated.

Sorry for being anal here:
enqueable ->
enqueueable


No transactions show in Spring based java tomcat application hosted in AWS.

We have Spring based java tomcat application which is using Oracle and MySQL as backend databases. We have deployed this as Kubernetes micro service in our on-prem Kubernetes cluster and We are able to get all transactions properly and populated correlation diagram as we expected.
Now we need to deploy this micro service in AWS and when we doing the testing, we saw that, we are not able to see any transactions and only see Spring icon in correlation diagram.
We are using same Spring based java tomcat application in both environments and Pinpoint agent also same.  We have manually profiled by providing the Main class but no luck.
Our Pinpoint version is 2.6 and agent also same version.
Please help to resolve this.

$AGENT_DIR/logs

Check the agent log in the above directory.

hello,
I have checked both log files and all are empty.  I'm able to see pinpoint agent bound with application properly through application log.
thanks.


Missing CHANGELOG

A changelog / version history is missing.
As far as we're aware there's no way to understand protocol changes between v1.x / v2.x apart from manually comparing / diffing the PDFs, which is a quite tedious work. Please let us know how / where to retrieve a change history of the protocol.

the problem it the not existing history of the mark down document.
see explanation #152

Hi @AntonDueck, thanks for your response. However, this issue is not related to the file format or git history. When maintainers release new major versions of the protocol, the community wants to know what changed and how the versions are (backward)-compatible.
Currently all engineers have to manually "diff" the PDFs and then identify the changes between the payloads/topics and the potential compatibility-issues. That's not only tedious, but will also make it very hard for the protocol to find adoption across engineers.

@FPurchess We have added the deprecated version 1.1 as a new branch. You can compare the changes from version 1.1 to version 2.0 here: https://github.com/VDA5050/VDA5050/compare/1.1.0...2.0.0
Please Note: As already described by @AntonDueck, the project was only created as an open source project after the release of version 1.1. So the branch 1.1 was only a working version at any time. Officially the versions published by the VDA are always valid. Nevertheless, the above diff makes it easier to understand the main changes (adding the factsheet and adjusting variable names).
We hope it helps you anyway!


Ignore deep/* tests when torch is not installed

Closes #132

@tgnassou Seems to be working now. Would you please double check?


Linking of rule '//modules/perception:perception' failed

Hi，
I builded apollo3.0 ARM version based on docker 14.4 enviroment, but builded error. I tried many methods but it was not resolved，how to solve this problem
System information
OS Platform and Distribution (e.g., Linux Ubuntu 18.04):docker 14.4 enviroment
Apollo installed from (source or binary):source
Apollo version (3.5, 5.0, 5.5, 6.0):3.0
Steps to reproduce the issue:
nvidia@in_dev_docker:/apollo$ ./apollo.sh build_opt_gpu
System check passed. Build continue ...
[WARNING] ESD CAN library supplied by ESD Electronics does not exist. If you need ESD CAN, please refer to third_party/can_card_library/esd_can/README.md.
Running build under GPU mode. GPU is required to run the build.
[INFO] Start building, please wait ...
INFO: Reading 'startup' options from /apollo/tools/bazel.rc: --batch_cpu_scheduling --host_jvm_args=-XX:-UseParallelGC
.

[INFO] Building with --jobs=8 --local_resources 16384,8,1.0 for aarch64
INFO: Reading 'startup' options from /apollo/tools/bazel.rc: --batch_cpu_scheduling --host_jvm_args=-XX:-UseParallelGC
INFO: (09-10 00:51:48.851) Found 2029 targets...
ERROR: (09-10 00:52:42.207) /apollo/modules/perception/BUILD:40:1: Linking of rule '//modules/perception:perception' failed (Exit 1).
/usr/lib/aarch64-linux-gnu/libnvinfer.so.6: undefined reference to `NvMediaDlaLoadFromMemory'
collect2: error: ld returned 1 exit status
INFO: (09-10 00:52:45.423) Elapsed time: 97.220s, Critical Path: 39.41s
[ERROR] Build failed!
[INFO] Took 153 seconds

Seems to be a problem with aarch64 libnvinfer.so.6, although the new version does not fully support aarch64, it is not recommended that you use 3.0


feat: added collection_type to schema

Codecov Report

Merging #71 (4ba9841) into main (560fdd1) will increase coverage by 0.03%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
+ Coverage   91.27%   91.31%   +0.03%
==========================================
  Files           8        8
  Lines        1868     1876       +8
  Branches      277      278       +1
==========================================
+ Hits         1705     1713       +8
  Misses        161      161
  Partials        2        2

Impacted Files
Coverage Δ

src/db.ts
91.74% <80.00%> (+0.03%)
:arrow_up:

src/types.ts
93.30% <100.00%> (+0.07%)
:arrow_up:

src/utility.ts
97.63% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add troubleshooting page for SignalR

https://github.com/dotnet/AspNetCore.Docs/issues/20063
Needs some cleanup.
This is definitely missing some common scenarios. Don't need to get them all in this first pass.
@Rick-Anderson  edit
Internal review URL

@FredHan001   FYI

@BrennanConroy  let me know what's need to get this merged. Great info, thanks.

I guess it needs author to be filled in to get rid of the warnings

@Rick-Anderson I see your screenshot with the suggested pending changes to add @bradygaster as an author to this doc. Did you mean to submit a new review with those suggestion?

No, I made a bunch of suggestions, most were accepted. We can't merge this without the author meta-data. These are my suggestions, if you agree, I'll add them.

The following needs to be added:
Q: Customer would like to know on the server the reason of the disconnection. However the exception parameter of the OnDisconnectedAsync seems to be always null:
public override async Task OnDisconnectedAsync(Exception exception)
A: I suspect you're right that there was some sort of network issue. If the connection closes between SignalR messages, the server may interpret that as an unexceptional client disconnect. I filed https://github.com/dotnet/aspnetcore/issues/26701 to track the issue of not passing an Exception to Hub.OnDisconnectedAsync in this case.

The following needs to be added:
Q: Customer would like to know on the server the reason of the disconnection. However the exception parameter of the OnDisconnectedAsync seems to be always null:
public override async Task OnDisconnectedAsync(Exception exception)
A: I suspect you're right that there was some sort of network issue. If the connection closes between SignalR messages, the server may interpret that as an unexceptional client disconnect. I filed dotnet/aspnetcore#26701 to track the issue of not passing an Exception to Hub.OnDisconnectedAsync in this case.

I don't know what kind of wording we want in the doc for something like this.

Whilst not very friendly, I think the statement

If the connection closes between SignalR messages, the server may interpret that as an unexceptional client disconnect.

Sums it up concisely. That said, i'd love some wordsmithery by @Rick-Anderson

If the connection closes between SignalR messages, the server may interpret that as an unexceptional client disconnect.

That's clear to me.

If the connection closes between SignalR messages, the server may interpret that as an unexceptional client disconnect.

That's clear to me.

Sure it's clear, but it doesn't mean anything. If we rephrase it a little it's equivalent to: "If the client closes the connection, it could be a graceful or ungraceful disconnect."

@BrennanConroy  I see your point. How about:
Transient network failures may close the connection between SignalR messages. The server may interpret the closed connection as an unexceptional client disconnect.

@BrennanConroy  added
For more information, see this GitHub issue.
What I have is pretty incomplete, but the issue fleshes it out and we might get feedback on the issue.

@BrennanConroy  let me know when I can merge this.

Lets wait till middle of the day tomorrow to give folks some time to take a look

LGTM

Ok, think it's good to go