Build errors on FreeBSD

I'm trying to build juCi++ on FreeBSD 10.3-Release which is my main development box. Unfortunately I'm getting a few build errors:
First it was not able to find libclang.so. That's because FreeBSD uses a different directory structure in order to allow the user install different compiler versions. So, I made the following changes:
cmake_modules/FindLibClang.cmake
find_path(LIBCLANG_INCLUDE_DIR clang-c/Index.h
  PATHS ${libclang_llvm_header_search_paths}
  PATH_SUFFIXES LLVM/include llvm33/include llvm34/include llvm35/include llvm36/include llvm37/include llvm38/include #Windows package from http://llvm.org/releases/
  DOC "The path to the directory that contains clang-c/Index.h")

# On Windows with MSVC, the import library uses the ".imp" file extension
# instead of the comon ".lib"
if (MSVC)
  find_file(LIBCLANG_LIBRARY libclang.imp
    PATH_SUFFIXES LLVM/lib llvm33/lib llvm34/lib llvm35/lib llvm36/lib llvm37/lib llvm38/lib
    DOC "The file that corresponds to the libclang library.")
endif()

find_library(LIBCLANG_LIBRARY NAMES libclang.imp libclang clang
  PATHS ${libclang_llvm_lib_search_paths}
  PATH_SUFFIXES LLVM/lib llvm33/lib llvm34/lib llvm35/lib llvm36/lib llvm37/lib llvm38/lib #Windows package from http://llvm.org/releases/
  DOC "The file that corresponds to the libclang library.")

Then cmake was successful:
-- The C compiler identification is Clang 3.4.1
-- The CXX compiler identification is Clang 3.4.1
-- Check for working C compiler using: Ninja
-- Check for working C compiler using: Ninja -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler using: Ninja
-- Check for working CXX compiler using: Ninja -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found LibClang: /usr/local/llvm38/lib/libclang.so
-- Boost version: 1.55.0
-- Found the following Boost libraries:
--   regex
--   system
--   filesystem
-- Found ASPELL: /usr/local/lib/libaspell.so
-- Found PkgConfig: /usr/local/bin/pkg-config (found version "0.28")
-- Checking for module 'gtkmm-3.0'
--   Found gtkmm-3.0, version 3.16.0
-- Checking for module 'gtksourceviewmm-3.0'
--   Found gtksourceviewmm-3.0, version 3.2.0
-- Checking for module 'libgit2'
--   Found libgit2, version 0.24.0
-- Could NOT find plantuml (missing:  PLANTUML_JARFILE)
-- Could NOT find Doxygen (missing:  DOXYGEN_EXECUTABLE)
-- Configuring done
-- Generating done
-- Build files have been written to: /home/babaei/Desktop/jucipp/build

Now, I'm getting the following error at the build stage:
[  9%] Building CXX object src/CMakeFiles/project_shared.dir/git.cc.o
/home/babaei/Desktop/jucipp/src/git.cc:106:44: error: no member named
      'FILE_MONITOR_WATCH_MOVES' in 'Gio::FileMonitorFlags'; did you mean
      'G_FILE_MONITOR_WATCH_MOVES'?
  ...Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     G_FILE_MONITOR_WATCH_MOVES
/usr/local/include/glib-2.0/gio/gioenums.h:347:3: note:
      'G_FILE_MONITOR_WATCH_MOVES' declared here
  G_FILE_MONITOR_WATCH_MOVES      = (1 << 3)
  ^
/home/babaei/Desktop/jucipp/src/git.cc:106:26: error: no matching member
      function for call to 'monitor_directory'
  monitor=git_directory->monitor_directory(Gio::FileMonitorFlags::FILE_M...
	  ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
/usr/local/include/giomm-2.4/giomm/file.h:2735:29: note: candidate function not
      viable: no known conversion from 'GFileMonitorFlags' to 'const
      Glib::RefPtr<Cancellable>' for 1st argument
  Glib::RefPtr<FileMonitor> monitor_directory(const Glib::RefPtr<Cancell...
	                    ^
/usr/local/include/giomm-2.4/giomm/file.h:2743:29: note: candidate function not
      viable: no known conversion from 'GFileMonitorFlags' to
      'Gio::FileMonitorFlags' for 1st argument
  Glib::RefPtr<FileMonitor> monitor_directory(FileMonitorFlags flags = F...
	                    ^
2 errors generated.
*** Error code 1

Stop.
make[2]: stopped in /usr/home/babaei/Desktop/jucipp/build
*** Error code 1

Stop.
make[1]: stopped in /usr/home/babaei/Desktop/jucipp/build
*** Error code 1

Stop.
make: stopped in /usr/home/babaei/Desktop/jucipp/build

It seems cmake picks up the wrong version of gtkmm library since I do have two different version of it installed at the same time (despite the fact that it says it picked up version 3 of the library):
    $ pkg info | grep gtkmm
    gtkmm24-2.24.4_2               C++ wrapper for Gtk+
    gtkmm30-3.16.0                 C++ wrapper for Gtk+3

I'm really excited about this brand new IDE and like to check it out, or even create a port for it. So, any help would be greatly appreciated.

Thank you for testing on FreeBSD. There seems to be a version mismatch between your glib and glibmm. We should really test glibmm version instead of glib version here: https://github.com/cppit/jucipp/blob/master/src/git.cc#L107. I'll try fix it now.

Would you mind test the above commit?

Sorry, there was a similar check in src/directories.cc as well. I did a rebase too, please use the latest commit above.

Thank you so much for the quick reply. I'm not at home right now. But, I'll try to test and report back as soon as I can.

I just checked out the master branch and I'm getting the following error:
[ 12%] Building CXX object src/CMakeFiles/project_shared.dir/git.cc.o
/home/babaei/Desktop/jucipp-master/src/git.cc:108:44: error: no member named
      'FILE_MONITOR_WATCH_MOVES' in 'Gio::FileMonitorFlags'; did you mean
      'G_FILE_MONITOR_WATCH_MOVES'?
  monitor=git_directory->monitor_directory(Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES);
                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                           G_FILE_MONITOR_WATCH_MOVES
/usr/local/include/glib-2.0/gio/gioenums.h:347:3: note: 'G_FILE_MONITOR_WATCH_MOVES' declared
      here
  G_FILE_MONITOR_WATCH_MOVES      = (1 << 3)
  ^
/home/babaei/Desktop/jucipp-master/src/git.cc:108:26: error: no matching member function for
      call to 'monitor_directory'
  monitor=git_directory->monitor_directory(Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES);
          ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
/usr/local/include/giomm-2.4/giomm/file.h:2735:29: note: candidate function not viable: no
      known conversion from 'GFileMonitorFlags' to 'const Glib::RefPtr<Cancellable>' for 1st
      argument
  Glib::RefPtr<FileMonitor> monitor_directory(const Glib::RefPtr<Cancellable>& cancell...
                            ^
/usr/local/include/giomm-2.4/giomm/file.h:2743:29: note: candidate function not viable: no
      known conversion from 'GFileMonitorFlags' to 'Gio::FileMonitorFlags' for 1st argument
  Glib::RefPtr<FileMonitor> monitor_directory(FileMonitorFlags flags = FILE_MONITOR_NONE);
                            ^
2 errors generated.
gmake[2]: *** [src/CMakeFiles/project_shared.dir/build.make:159: src/CMakeFiles/project_shared.dir/git.cc.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:125: src/CMakeFiles/project_shared.dir/all] Error 2
gmake: *** [Makefile:150: all] Error 2

What version of glib and glibmm do you have?

$ pkg info | grep glib
dbus-glib-0.104                GLib bindings for the D-BUS messaging system
glib-2.46.2_2                  Some useful routines of C programming (current stable version)
glib-networking-2.46.1_1       Network-related giomodules for glib
glibmm-2.44.0,1                C++ interfaces for glib2
json-glib-1.1.2                JSON (RFC 4627) interface for Glib
libgit2-glib-0.24.0            Glib wrapper library around the libgit2 git access library
poppler-glib-0.40.0            GLib bindings to poppler
taglib-1.10                    Library for manipulating ID3 tags and Ogg comments
telepathy-glib-0.24.1          GLib utility library for the Telepathy framework

That is strange, you have the required glibmm version (2.44) for Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES. You can however replace Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES with Gio::FileMonitorFlags::FILE_MONITOR_SEND_MOVED in src/git.cc and src/directories.cc.

Or, is there a separate giomm package on FreeBSD? In that case, what version is your gio and giomm?

Or, is there a separate giomm package on FreeBSD? In that case, what version is your gio and giomm?
No, there is not:
https://www.freebsd.org/cgi/ports.cgi?query=giomm&stype=name&sektion=all

That is strange, you have the required glibmm version (2.44) for Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES. You can however replace Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES with Gio::FileMonitorFlags::FILE_MONITOR_SEND_MOVED in src/git.cc and src/directories.cc.

Now I get an linker error, despite the fact that I have /usr/local/lib/libgtkmm-3.0.so on my system:
[41/41] Linking CXX executable src/juci
FAILED: src/juci
: && /usr/bin/CC   -DJUCI_VERSION=\"1.2.1\" -std=c++1y -pthread -Wall -Wextra -Wno-unused-parameter -Wno-reorder -O3   src/CMakeFiles/project_shared.dir/cmake.cc.o src/CMakeFiles/project_shared.dir/ctags.cc.o src/CMakeFiles/project_shared.dir/dispatcher.cc.o src/CMakeFiles/project_shared.dir/filesystem.cc.o src/CMakeFiles/project_shared.dir/git.cc.o src/CMakeFiles/project_shared.dir/project_build.cc.o src/CMakeFiles/project_shared.dir/source.cc.o src/CMakeFiles/project_shared.dir/source_clang.cc.o src/CMakeFiles/project_shared.dir/source_diff.cc.o src/CMakeFiles/project_shared.dir/source_spellcheck.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/CodeCompleteResults.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/CompilationDatabase.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/CompileCommand.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/CompileCommands.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/CompletionString.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/Cursor.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/Diagnostic.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/Index.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/SourceLocation.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/SourceRange.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/Token.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/Tokens.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/TranslationUnit.cc.o src/CMakeFiles/project_shared.dir/__/libclangmm/src/Utility.cc.o src/CMakeFiles/project_shared.dir/__/tiny-process-library/process.cpp.o src/CMakeFiles/project_shared.dir/__/tiny-process-library/process_unix.cpp.o src/CMakeFiles/juci.dir/config.cc.o src/CMakeFiles/juci.dir/dialogs.cc.o src/CMakeFiles/juci.dir/dialogs_unix.cc.o src/CMakeFiles/juci.dir/directories.cc.o src/CMakeFiles/juci.dir/entrybox.cc.o src/CMakeFiles/juci.dir/info.cc.o src/CMakeFiles/juci.dir/juci.cc.o src/CMakeFiles/juci.dir/menu.cc.o src/CMakeFiles/juci.dir/notebook.cc.o src/CMakeFiles/juci.dir/project.cc.o src/CMakeFiles/juci.dir/selectiondialog.cc.o src/CMakeFiles/juci.dir/terminal.cc.o src/CMakeFiles/juci.dir/tooltips.cc.o src/CMakeFiles/juci.dir/window.cc.o  -o src/juci  -lgtkmm-3.0 -latkmm-1.6 -lgdkmm-3.0 -lgiomm-2.4 -lpangomm-1.4 -lglibmm-2.4 -lgtk-3 -lgdk-3 -lpangocairo-1.0 -lpango-1.0 -latk-1.0 -lcairo-gobject -lgio-2.0 -lcairomm-1.0 -lcairo -lpthread -lsigc-2.0 -lgdk_pixbuf-2.0 -lgobject-2.0 -lglib-2.0 -lintl -lgtksourceviewmm-3.0 -lgtkmm-3.0 -latkmm-1.6 -lgdkmm-3.0 -lgiomm-2.4 -lpangomm-1.4 -lglibmm-2.4 -lcairomm-1.0 -lsigc-2.0 -lgtksourceview-3.0 -lgtk-3 -lgdk-3 -lpangocairo-1.0 -lpango-1.0 -latk-1.0 -lcairo-gobject -lcairo -lpthread -lgdk_pixbuf-2.0 -lgio-2.0 -lgobject-2.0 -lglib-2.0 -lintl /usr/local/lib/libboost_system.so /usr/local/lib/libboost_filesystem.so /usr/local/llvm36/lib/libclang.so /usr/local/lib/libaspell.so -lgit2 -lgtkmm-3.0 -latkmm-1.6 -lgdkmm-3.0 -lgiomm-2.4 -lpangomm-1.4 -lglibmm-2.4 -lcairomm-1.0 -lsigc-2.0 -lgtksourceviewmm-3.0 -lgtksourceview-3.0 /usr/local/lib/libboost_system.so /usr/local/lib/libboost_filesystem.so /usr/local/llvm36/lib/libclang.so /usr/local/lib/libaspell.so -lgit2 -Wl,-rpath,/usr/local/lib:/usr/local/llvm36/lib: && :
/usr/bin/ld: cannot find -lgtkmm-3.0
c++: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

I should not that traditionally FreeBSD systems install all packages inside /usr/local and leaves the base system untouched by ports and packages. e.g:
/usr/local/lib
/usr/local/include
/usr/local/share
/usr/local/etc

Or, is there a separate giomm package on FreeBSD? In that case, what version is your gio and giomm?

No, there is not:
https://www.freebsd.org/cgi/ports.cgi?query=giomm&stype=name&sektion=all

$ pkg which /usr/local/include/giomm-2.4/giomm/file.h
/usr/local/include/giomm-2.4/giomm/file.h was installed by package glibmm-2.44.0,1

I fixed the gio check in https://github.com/cppit/jucipp/tree/freebsd I hope.
Regarding the linking error, try add -L/usr/local/lib to the compile flags, or modify the path to the libgtkmm-3.0.so(or something similar) file.

OK, thanks. I checked out the freebsd branch and the gio check issue still persists:
$ git status
On branch freebsd
Your branch is up-to-date with 'origin/freebsd'.
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

    modified:   ../tiny-process-library (new commits)

no changes added to commit (use "git add" and/or "git commit -a")

The actual build error:
Scanning dependencies of target project_shared
[  2%] Building CXX object src/CMakeFiles/project_shared.dir/cmake.cc.o
[  4%] Building CXX object src/CMakeFiles/project_shared.dir/ctags.cc.o
[  7%] Building CXX object src/CMakeFiles/project_shared.dir/dispatcher.cc.o
[  9%] Building CXX object src/CMakeFiles/project_shared.dir/filesystem.cc.o
[ 12%] Building CXX object src/CMakeFiles/project_shared.dir/git.cc.o
/usr/home/babaei/Desktop/jucipp-freebsd/src/git.cc:108:44: error: no member named
      'FILE_MONITOR_WATCH_MOVES' in 'Gio::FileMonitorFlags'; did you mean
      'G_FILE_MONITOR_WATCH_MOVES'?
  monitor=git_directory->monitor_directory(Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES);
                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                           G_FILE_MONITOR_WATCH_MOVES
/usr/local/include/glib-2.0/gio/gioenums.h:347:3: note: 'G_FILE_MONITOR_WATCH_MOVES' declared
      here
  G_FILE_MONITOR_WATCH_MOVES      = (1 << 3)
  ^
/usr/home/babaei/Desktop/jucipp-freebsd/src/git.cc:108:26: error: no matching member function
      for call to 'monitor_directory'
  monitor=git_directory->monitor_directory(Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES);
          ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
/usr/local/include/giomm-2.4/giomm/file.h:2735:29: note: candidate function not viable: no
      known conversion from 'GFileMonitorFlags' to 'const Glib::RefPtr<Cancellable>' for 1st
      argument
  Glib::RefPtr<FileMonitor> monitor_directory(const Glib::RefPtr<Cancellable>& cancell...
                            ^
/usr/local/include/giomm-2.4/giomm/file.h:2743:29: note: candidate function not viable: no
      known conversion from 'GFileMonitorFlags' to 'Gio::FileMonitorFlags' for 1st argument
  Glib::RefPtr<FileMonitor> monitor_directory(FileMonitorFlags flags = FILE_MONITOR_NONE);
                            ^
2 errors generated.
gmake[2]: *** [src/CMakeFiles/project_shared.dir/build.make:159: src/CMakeFiles/project_shared.dir/git.cc.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:125: src/CMakeFiles/project_shared.dir/all] Error 2
gmake: *** [Makefile:150: all] Error 2

In addition to that now I noticed that CMake is not able to find liblldb in both master and freebsd branches. In the cmake_modules/FindLibClang.cmake file I manipulated from v1.2.0 tag there is no warning about liblldb is missing:
-- The C compiler identification is Clang 3.4.1
-- The CXX compiler identification is Clang 3.4.1
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/CC
-- Check for working CXX compiler: /usr/bin/CC -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found LibClang: /usr/local/llvm36/lib/libclang.so
liblldb not found. Building juCi++ without debugging support
-- Boost version: 1.55.0
-- Found the following Boost libraries:
--   system
--   filesystem
-- Found ASPELL: /usr/local/lib/libaspell.so
-- Found PkgConfig: /usr/local/bin/pkg-config (found version "0.28")
-- Checking for module 'gtkmm-3.0'
--   Found gtkmm-3.0, version 3.16.0
-- Checking for module 'gtksourceviewmm-3.0'
--   Found gtksourceviewmm-3.0, version 3.2.0
-- Checking for module 'libgit2'
--   Found libgit2, version 0.24.0
-- Could NOT find plantuml (missing:  PLANTUML_JARFILE)
-- Could NOT find Doxygen (missing:  DOXYGEN_EXECUTABLE)
-- Configuring done
-- Generating done
-- Build files have been written to: /usr/home/babaei/Desktop/jucipp-freebsd/build

Thank you very much for testing this! According to https://developer.gnome.org/glibmm/stable/group__giommEnums.html#ga469f795e9a4dddb2421c0e5568be6649, the check is correct in master branch. Not sure how to fix the compilation error actually, other than doing the manual replace.
Where is your liblldb located? I might need the full path and file name.

OK, I found the issue with: liblldb not found. Building juCi++ without debugging support
I have 3 versions of LLVM installed at the same time:
/usr/local/llvm36
/usr/local/llvm37
/usr/local/llvm38

Now if change the following lines inside ./cmake_modules/FindLibClang.cmake from
find_path(LIBCLANG_INCLUDE_DIR clang-c/Index.h
  PATHS ${libclang_llvm_header_search_paths}
  PATH_SUFFIXES LLVM/include #Windows package from http://llvm.org/releases/
    llvm33/include llvm34/include llvm35/include llvm36/include llvm37/include llvm38/include llvm39/include # FreeBSD
  DOC "The path to the directory that contains clang-c/Index.h")

find_library(LIBCLANG_LIBRARY NAMES libclang.imp libclang clang
  PATHS ${libclang_llvm_lib_search_paths}
  PATH_SUFFIXES LLVM/lib #Windows package from http://llvm.org/releases/
    llvm33/lib llvm34/lib llvm35/lib llvm36/lib llvm37/lib llvm38/lib llvm39/lib # FreeBSD
  DOC "The file that corresponds to the libclang library.")

To, for example
find_path(LIBCLANG_INCLUDE_DIR clang-c/Index.h
  PATHS ${libclang_llvm_header_search_paths}
  PATH_SUFFIXES LLVM/include #Windows package from http://llvm.org/releases/
    llvm38/include # FreeBSD
  DOC "The path to the directory that contains clang-c/Index.h")

find_library(LIBCLANG_LIBRARY NAMES libclang.imp libclang clang
  PATHS ${libclang_llvm_lib_search_paths}
  PATH_SUFFIXES LLVM/lib #Windows package from http://llvm.org/releases/
    llvm38/lib # FreeBSD
  DOC "The file that corresponds to the libclang library.")

CMake won't get confused and won't spit out that warning.

Thank you very much for testing this! According to https://developer.gnome.org/glibmm/stable/group__giommEnums.html#ga469f795e9a4dddb2421c0e5568be6649, the check is correct in master branch. Not sure how to fix the compilation error actually, other than doing the manual replace.

Maybe an #if defined ( __FreeBSD__ ) is necessary on that check.
I'll try to make it work and make the necessary patches with your approval.

Thank you, I'll change the search order so that the newest version is checked first. I also removed older llvm versions. llvm 3.6 should probably be minimum.

That would be great @NuLL3rr0r

When I think about it, the gio build error might be a glibmm issue on freebsd, so the #if check might be needed as a workaround.

Well, I investigated that http://svnweb.freebsd.org/ports/head/devel/glibmm/
In the Makefile the MASTER_SITES variable points to gnome official website and if you check the distinfo file the sha hash is the same. Moreover, in FreeBSD Ports system if they patch a port/package there should be a files folder too which is not there (all patch files go into that folder). So, the package is a vanilla upstream package.

The file operations are very platform dependent. What I'm thinking is that, Gio::FileMonitorFlags::FILE_MONITOR_WATCH_MOVES is missing or not implemented in your version of glibmm. Since it might be implemented in glib (the C-library), this might be an issue with glibmm where there might be an incorrect platform check. I've seen this before for gtksourceview (and posted a fix) for MSYS2. This is only a hunch though, the problem might lie elsewhere.

Thanks for the info, I'm not familiar with these gtk stuff. So, perhaps your guess might be correct. I checked this package maintainer's email and it's the FreeBSD Gnome Team at<EMAIL_ADDRESS>Probably they are the best to ask.

I'll have a look at the glibmm bug reports and the source code later in a day or two. We probably have to add the #if defined ( __FreeBSD__ ) expression anyway though, since a potential fix would not arrive until a later glibmm version.

Thank you so much. Now I'm able to build and install it on my FreeBSD system. I'll try to create a FreeBSD port and submit it for review on FreeBSD ports tree.
Looks promising BTW! Thanks for the great work.

Thank you as well! Let me know if there are any more issues on FreeBSD. I guess it's about time I give FreeBSD a try too, maybe next week:)

My pleasure! That's awesome :) I'm sure you'll like it.


[Feature Request] Display full request path when selecting a request for a Response tag

It's really nice that Insomnia can grab parameters from other responses in order to build requests, it saves me a lot of copy-pasting. I also like to keep my stuff neatly organized so I use a lot of folder nesting with Insomnia (which kind of backfired).
Currently, in the window where you get to configure a Response tag, you can select the request you want to grab the response from, in the list you can see the request name and the name of the folder it is in (if any).
[Auth] GET Authorize URL

However, when you have a lot of folder nesting, and the structure of the upper level folders is similar, it becomes really hard to identify the requests you're looking for. For example:
cool_apis
- API_1
- - Auth
- - - GET Authorize URL
- API_2
- - Auth
- - - GET Authorize URL

For the structure shown above you will see the following in the select list:
[Auth] GET Authorize URL
[Auth] GET Authorize URL

without being able to tell which request comes from API_1 and which comes from API_2.
It would be nice to have them look like:
[API_1][Auth] GET Authorize URL
[API_2][Auth] GET Authorize URL

Perhaps make the the displayed nesting level configurable somehow?
As in for 2 it will be [API_1][Auth], for 3 [cool_apis][API_1][Auth] etc.

That's a good idea 💡
Sent with GitHawk

Hi, I would like to work on this :smile:.

@rickychandra Hey thanks for taking this on. I took a look at your PR and the issue seems to be a lot simpler to solve than I thought, haha. Guess I was just scared of diving yet into another codebase.

Hi @TeeSeal, I think this one is quite simple too lol. I am new to contributing to Insomnia, but I find the codebase is pretty easy to follow though.


question: CPU time needed for hog feature in SDM code

Hi Patrikhuber,
when we run the SDM test(predict) code, the CPU time needed for each image is so much. For different size of images, the time needed is much different.
My qestions are:

is there any way to optimize the time needed to calculate hog feature?
for different faces with different size, the CPU time needed is much different. can we resize the image to small size for hog calculation? will this method lead to the bad accuracy of SDM?

Many thanks for your answer!

Hi,
Sure you can resize the image. You'd have to test the accuracy if you want to know. If you go through the issues list here, there's a few more tickets with some ideas to improve the speed. Alternatively you can reduce the feature size and retrain, or use different features if you're not happy with vlhog (though vlhog is pretty fast in my experience). Also you can check the current literature.

Thanks patrikhuber!


Integration with Chrome Dev Tools Protocol

It should be possible to integrate/wrap something like https://github.com/cavalle/chrome_remote/ into various commands on the browser.
The gain here is that you have direct access to the Chrome Dev Tools Protocol and can setup things like network intercepts or listening for particular messages in the console (among other things, more found here - https://chromedevtools.github.io/devtools-protocol/).
For reference, in the Javascript space I've been using webdriver.io and there is a package (https://github.com/webdriverio/wdio-devtools-service) that extends the CDP commands onto the browser.
General Thoughts/Concerns:
This would only work in Chrome.
Can this live in Watir or become a "plugin" Gem?
Should people that want to use this just use chrome_remote directly?

Selenium has executeCdpCommand which should be enough.
https://github.com/SeleniumHQ/selenium/blob/07a18746ff756e90fd79ef253a328bd7dfa9e6dc/py/selenium/webdriver/chrome/webdriver.py#L118

Selenium 4 is working to implement a cross-browser bidirectional API
One option is for Watir to wrap this once all browsers are supported. Another option is to just let the user access the functionality they need with browser.driver.register / browser.driver.devtools, etc.
We're going to go with the latter unless/until someone has a reason for a change.


Sample does not work

I believe someone else already raised this issue, but it looks like this sample project is not being updated.  The CreateSubscription fails with some unknown error.

Hi @ewahner , sorry for the late reply. We're working on new samples and suggest you try those out instead https://github.com/microsoftgraph/csharp-webhook-with-resource-data


permissions_aftergroupexpansion.csv data contains commas not in quotes

I'm looking at the Permissions_aftergroupexpansion.csv file and I'm seeing Identity and User values that contain commas and they aren't contained in quotes.  I am concerned that the Calculate executable is not then able to parse that data correctly when generating subsequent output files since a split on the comma character would result in more than three fields.  I'm starting to suspect that is why I have so many users getting lumped into one giant group.
It looks like there was previously an issue fixed in RC1 regarding data with commas, but it wasn't clear to me which step in the process that referred to.  The CSV files generated by ExportScript.ps1 have all relevant fields contained in quotes, so I suspect that's what was referred to.

Quotas do not influence calculation. Try new version.


Add ability to write math expression in amount field

When creating new expenses, currently there is no support to write mathematical expressions.
My solution includes a package  math-expression-evaluator.
On change in the amount field, it tries to evaluate the string and shows the evaluated value if correct and 'Invalid expression' otherwise.
The evaluated amount is shown below the input box.
'This is a reimbursement' is renamed to 'Reimbursement' and shifted to the right side of the total value field.
Fixes #183

I have updated the formatting issues that was failing the Prettier check.

@scastiel hello, I have fixed the merge problems as well. Can you check if you are free.


[REF-2273] Implement .setvar special EventHandler

All State subclasses will now have a special setvar EventHandler which appears in the autocomplete drop down, passes static analysis, and can be used to set State Vars in response to event triggers.
Before:
rx.input(value=State.a, on_change=State.set_a)

After:
rx.input(value=State.a, on_change=State.setvar("a"))

This reduces the "magic" because setvar is statically defined on all State subclasses.

To make this actually work nicely, this PR also enables partial application of EventHandler.
When an EventHandler is called with an incomplete set of args it creates a partial EventSpec. This change allows Component._create_event_chain to apply remaining args from an args_spec to an existing EventSpec to make it functional.
Instead of requiring the use of lambda functions to pass arguments to an EventHandler, they can now be passed directly and any remaining args defined in the event trigger will be applied after those.

Deprecations: this removes the old-style args_spec which hasn't been used since 0.2.x and was slated for removal in 0.5.0.

When using setvar with a non-existing state var, this is actually a runtime error instead of a compile time error. That may be non-ideal behavior.
Maybe instead of setvar, we should expose a .set EventSpec on each BaseVar instead.

@masenf how about this?
rx.input(value=State.a, on_change=State.setvar(State.a))

@benedikt-bartscher we considered that, but it's actually kind of ambiguous... is it setting State.a itself, or is it an indirection that sets the var name referenced by State.a.
class State(rx.State):
    a: str = "b"
    b: str = "foo"

...

rx.input(value=State.a, on_change=State.setvar(State.a))

Does this set State.a to the value of the input... or does it set State.b to the value of the input (because State.a is "b").

You may be right, it is kind of ambiguous... I wanted to find a typed alternative to setvar("a") and this was the first notation that came into my mind.

Maybe instead of setvar, we should expose a .set EventSpec on each BaseVar instead.
I like this one. It is not that ambiguous, and "typed".

So i tried the `State.a.set` thing

diff --git a/reflex/state.py b/reflex/state.py
index a84ea7f8..2845d280 100644
--- a/reflex/state.py
+++ b/reflex/state.py
@@ -909,6 +909,7 @@ class BaseState(Base, ABC, extra=pydantic.Extra.allow):
         if setter_name not in cls.__dict__:
             event_handler = cls._create_event_handler(prop.get_setter())
             cls.event_handlers[setter_name] = event_handler
+            prop._set = event_handler
             setattr(cls, setter_name, event_handler)

     @classmethod
diff --git a/reflex/vars.py b/reflex/vars.py
index 4a8e6b30..8268d003 100644
--- a/reflex/vars.py
+++ b/reflex/vars.py
@@ -40,6 +40,7 @@ from reflex.utils import console, format, imports, serializers, types
 from reflex.utils.imports import ImportDict, ImportVar

 if TYPE_CHECKING:
+    from reflex.event import EventHandler
     from reflex.state import BaseState

 # Set of unique variable names.
@@ -1739,6 +1740,9 @@ class BaseVar(Var):
     # Extra metadata associated with the Var
     _var_data: Optional[VarData] = dataclasses.field(default=None)

+    # An EventHandler for setting the Var from an event trigger
+    _set: "EventHandler" | None = dataclasses.field(default=None)
+
     def __hash__(self) -> int:
         """Define a hash function for a var.

@@ -1835,6 +1839,19 @@ class BaseVar(Var):

         return setter

+    @property
+    def set(self) -> "EventHandler":
+        """Get the EventHandler for setting the var.
+
+        Returns:
+            An EventHandler for setting the var.
+        """
+        if self._set is None:
+            raise ValueError(
+                f"Var {self._var_full_name} does not have a setter defined."
+            )
+        return self._set
+

 @dataclasses.dataclass(init=False, eq=False)
 class ComputedVar(Var, property):
diff --git a/reflex/vars.pyi b/reflex/vars.pyi
index fb2ed465..51d68b98 100644
--- a/reflex/vars.pyi
+++ b/reflex/vars.pyi
@@ -6,6 +6,7 @@ from dataclasses import dataclass
 from _typeshed import Incomplete
 from reflex import constants as constants
 from reflex.base import Base as Base
+from reflex.event import EventHandler as EventHandler
 from reflex.state import State as State
 from reflex.state import BaseState as BaseState
 from reflex.utils import console as console, format as format, types as types
@@ -126,10 +127,13 @@ class BaseVar(Var):
     _var_is_string: bool = False
     _var_full_name_needs_state_prefix: bool = False
     _var_data: VarData | None = None
+    _set: EventHandler | None = None
     def __hash__(self) -> int: ...
     def get_default_value(self) -> Any: ...
     def get_setter_name(self, include_state: bool = ...) -> str: ...
     def get_setter(self) -> Callable[[BaseState, Any], None]: ...
+    @property
+    def set(self) -> EventHandler: ...

 @dataclass(init=False)
 class ComputedVar(Var):
diff --git a/tests/test_state.py b/tests/test_state.py
index ce62e9c6..c13f3a64 100644
--- a/tests/test_state.py
+++ b/tests/test_state.py
@@ -2883,3 +2883,29 @@ async def test_setvar(mock_app: rx.App, token: str):
     # Cannot setvar with non-string
     with pytest.raises(ValueError):
         TestState.setvar(42, 42)
+
+
<EMAIL_ADDRESS>+async def test_base_var_set(mock_app: rx.App, token: str):
+    """Test that Var.set works correctly.
+
+    Args:
+        mock_app: An app that will be returned by `get_app()`
+        token: A token.
+    """
+    state = await mock_app.state_manager.get_state(_substate_key(token, TestState))
+
+    # Set Var in same state (with Var type casting)
+    for event in rx.event.fix_events(
+        [TestState.num1.set(42), TestState.num2.set("4.2")], token
+    ):
+        async for update in state._process(event):
+            print(update)
+    assert state.num1 == 42
+    assert state.num2 == 4.2
+
+    # Set Var in parent state
+    for event in rx.event.fix_events([GrandchildState.array.set([43])], token):
+        async for update in state._process(event):
+            print(update)
+    assert state.array == [43]

And while it does work, it still does not pass typing because the Vars in a state are treated like the actual type they are (`str`, `bool`, `int`, etc). Statically, they are not known as `Var`, so the `set` property still throws a typing error.

/Users/masenf/code/reflex-dev/reflex/tests/test_state.py
  /Users/masenf/code/reflex-dev/reflex/tests/test_state.py:2900:25 - error: Cannot access member "set" for type "int"
    Member "set" is unknown (reportGeneralTypeIssues)
  /Users/masenf/code/reflex-dev/reflex/tests/test_state.py:2900:49 - error: Cannot access member "set" for type "float"
    Member "set" is unknown (reportGeneralTypeIssues)
  /Users/masenf/code/reflex-dev/reflex/tests/test_state.py:2908:61 - error: Cannot access member "set" for type "List[float]"
    Member "set" is unknown (reportGeneralTypeIssues)

Ultimately, we need to make Var and EventHandler into descriptors so that we can statically type the State class (and get rid of a bunch of dodgy __init_subclass__, __getattribute__ and __setattr__ logic along the way as well). But that's a much bigger project.
Probably will take this change as-is for now, and circle back around to the typing improvements once we have real descriptors for these fields.
This week, the focus is on getting a 0.5.0 release out with radix 3.0 and some other Component API changes, so any descriptor work would have to wait a few weeks at minimum.

Yes, I know that it won't pass pyright for base_vars, but (as you explained) this is another issue not related to your State.a.set implementation. It should pass pyright with ComputedVars though. I would prefer implementing a typed variant directly and annotate the related lines with ignore comments (like we already need to do for other problems introduced by the __init_subclass__ magic).
My main intention is to reduce reflex's api changes.


Excessive rate limit violations

Does the plugin need the same changes as mentioned in https://github.com/zbirenbaum/copilot-cmp/issues/117 to avoid any issues?

That was one of my concerns, since I knew this was a problem in copilot-cmp. Good to know this was quickly fixed :)


Umbrella issue for Netlify access amendments

Discussed in k/website discussion and RT 1.26 retro

One member from CNCF's Dev Advocate/Tech writer team to have access to K8s Netlify
Netlify access cleanup for emeritus co-chairs/tech leads
Netlify access cleanup for other non-Docs accounts
RT Docs lead Netlify access

Proposal:
--> Since no sub-teams within Netlify give elevated privileges, the CNCF Dev Advocate/Tech Writer will be added individually as a collaborator.
--> Current list of emeritus docs co-chairs & tech leads to be cleaned up manually by docs co-chairs.
--> Separate onboarding and offboarding documentation for co-chairs & leads to be created toward including this access.
--> For non-docs accounts, for example, RT Docs, Emeritus Steering, etc., access is to be removed.
--> Per Netlify documentation, a collaborator role allows the actions required for the Release team Docs lead to perform the necessary actions on a release without causing any damage. If in agreement, document explicitly in the role handbook to seek Netlify collaborator access from the Docs leads.

+1 on the overall cleanup.

(With the hat of a GitHub Admin)
Would our access be removed? Asking since we get requests to enable subproject sites (*.sigs.k8s.io) on kubernetes/org and we handle those requests.

On that note, can a list be maintained in k/community on who has Owner access to Netlify? This can keep the membership transparent to everyone.

/triage accepted

+1 on the overall cleanup.
(With the hat of a GitHub Admin)
Would our access be removed? Asking since we get requests to enable subproject sites (*.sigs.k8s.io) on kubernetes/org and we handle those requests.

Nope. For the current cleanup, we'll be manually cross-checking and figuring out the retention. But the aim is to have an updated Netlify members list by end of Q1 (hopefully) & a way to, potentially, automate it.

On that note, can a list be maintained in k/community on who has Owner access to Netlify? This can keep the membership transparent to everyone.

Agreed!

+1 to moving forward with this. @chalin and @nate-double-u have both raised Netlify access as issues/blockers for ongoing CNCF projects, like out Google analytics 4 migration (issue: https://github.com/kubernetes/website/issues/35299).

Amending the description format to a checklist of items that will be completed after the consensus date i.e. February 8th.

One member from CNCF's Dev Advocate/Tech writer team to have access to K8s Netlify

Is it too late to ask for 2 members from CNCF's Dev Advocate/Tech writer team to have access? Would be good for vacation coverage and whatnot.

Closing this issue off as the audit work + associated documentation for SIG Docs has been completed. For non-docs accounts on Netlify, we have raised it with SIG Contribex and will be following it up there.
/close


HSEARCH-3529 MassIndexer fails for entities with primitive ID type

https://hibernate.atlassian.net//browse/HSEARCH-3529
Waiting for CI...

Pull Request Test Coverage Report for Build 1

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 88.649%

Totals

Change from base Build 173:
0.005%

Covered Lines:
16197

Relevant Lines:
18271

💛  - Coveralls


OCPBUGS-22112: Add workload partitioning annotations to guard pods

I tested this PR locally with cluster-bot and the guard pods are properly annotated:
$ oc get pods -n openshift-kube-apiserver -l app=guard -o<EMAIL_ADDRESS>\"PreferredDuringScheduling\"}")].metadata.name}'
kube-apiserver-guard-ip-10-0-110-120.ec2.internal kube-apiserver-guard-ip-10-0-25-80.ec2.internal kube-apiserver-guard-ip-10-0-98-231.ec2.internal

/jira refresh

Closing since the library-go bump was done in a separate PR.


Add initial prettier integration

What kind of change does this PR introduce? (check at least one)

[x] Code style update

Does this PR introduce a breaking change? (check one)

[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup

Other information:
Here's an initial implementation of Prettier integration, with changes made to each file so that we can see how the source would change. Once we've confirmed the configs in .eslintrc and .prettierrc look good, I think it would be a good idea to do the following prior to merging:

Build the Prettier ESLint overrides into eslint-plugin-vue-libs.

Ship the Prettier config with eslint-plugin-vue-libs, so that we can maintain consistency between projects with the following in a .prettierrc.js file:
module.exports = require('eslint-plugin-vue-libs/.prettierrc.js')

Then once this is merged, I can create PRs for our other repos.

Closing as this is quite outdated - we will likely move to prettier eventually, but after we cleanup existing PRs to avoid back and forth formatting churn.


Update mobilenet_v3.py

Hi, I found that normalization operation is included in the definition of mobilenetv3 model and Mobile netv2 doesn't do that.
I don't think include normalization in the model a good idea, it should be outside the model. First of all, it limits the ability to customize. Secondly, I find that the normalization of 255 directly results in the accuracy of the model almost reduced to random after the model is converted to tflite int-8 version.
So I think it's better to remove it in the model.

@googlebot I signed it!


Databricks Bad Request Error

What happened?
trying to run AutoGen Databricks Hello World example
Running on  - Databricks LTS 14.3 ML
%pip install autogen-agentchat==0.2.40 openai==1.21.2 typing_extensions==4.11.0 --upgrade

the rest of the configurations are as specified in the example
os.environ["DATABRICKS_TOKEN"] = "dapi...."

llm_config = {
    "config_list": [
        {
            "model": "databricks-dbrx-instruct",
            "api_key": str(os.environ["DATABRICKS_TOKEN"]),
            "base_url": str(os.getenv("DATABRICKS_HOST")) + "/serving-endpoints",
        }
    ],
}

import autogen

# Create Assistant and User
assistant = autogen.AssistantAgent(name="assistant", llm_config=llm_config)

user_proxy = autogen.UserProxyAgent(name="user", code_execution_config=False)

# Initiate chat from user_proxy side
chat_result = user_proxy.initiate_chat(assistant, message="What is MLflow?")

------
user (to assistant):

What is MLflow?
-------
BadRequestError: Error code: 400 - {'error_code': 'BAD_REQUEST', 'message': 'Bad request: json: unknown field "name"\n'}

...

.../openai/_base_client.py 922
return ...

What did you expect to happen?
I expected a response like in the referenced article. Something like
Sure, I'd be happy to explain MLflow to you. MLflow is an open-source platform for managing machine learning workflows. It was developed by Databricks and was open-sourced in 2018. MLflow provides a number of features to help data scientists and machine learning engineers manage the end-to-end machine learning lifecycle, including:

1. **MLflow Tracking**: This is a logging API that allows you to record and query experiments, including code, data, config, and results.
2. **MLflow Projects**: This is a format for packaging reusable and reproducible data science code, which can be run on different platforms.
3. **MLflow Models**: This is a convention for packaging machine learning models in multiple formats, making it easy to deploy in different environments.
4. **MLflow Model Registry**: This is a central repository to store, manage, and serve machine learning models.
...

How can we reproduce it (as minimally and precisely as possible)?
Try to run the code (as in the blog post) on a Databricks cluster. Does it still work. Note, in addition to the code above, I had to add to get around the error" "OpenAI Client.init() got an unexpected keyword argument proxies"
%pip install httpx==0.27.2

AutoGen version
0
Which package was this bug in
Core
Model used
No response
Python version
0.2.40
Operating system
Databricks LTS 14.3 ML
Any additional info you think would be helpful for fixing this bug
No response

@tj-cycyota Could you take a look at this?


Update test harness and handling .reference files

This PR fixes (hopefully) #139:

by introducing control over colors when formatting errors
making the test harness not use colors when formatting diagnostics: the very readable result is visible in the "bless colorless test expectations" commit.
if I understood #139 correctly: all .ref files were to be "ignored" during tests (except when blessing, where they are still generated), removed from the repo, and gitignored ? I hope this understanding is correct, as the last few commits do that.

Thanks @lqd!
bors r+


Pylon targeting

Is there a way to set the targeting of Pylons? I can't find anything related to that on the README nor in the code.

Ah, didn't think about that. Nope, I only set the targeting for gems, I'll look into it, thanks for reporting!

Added in 1.3


php5-mcrypt is installed, but not enabled

For reasons unknown, despite installing php5-mcrypt here, it doesn't actually get enabled.
(See output of $ php -m|grep mcrypt, it's blank.)
Modules we install should actually work.

Terrible Workaround:
vagrant ssh -c "sudo php5enmod mcrypt && sudo apache2ctl graceful && php -m|grep mcrypt"

This appears to have been resolved by PR #12:
~/www/precip (master)$ vagrant ssh -c "php -m|grep mcrypt"
mcrypt
Connection to <IP_ADDRESS> closed.


Allow clusters to be assigned IAM roles during launch

When you launch a cluster you want to access a dataset somewhere, and that somewhere is often S3. You need credentials to access non-public locations on S3.
Instead of copying around AWS keys or doing some other janky setup, the Amazon-recommended way is to assign the instances in your cluster to an appropriate IAM role that lets them access what they need.
We should support this.

Following what seems to be the project naming standards and parameter names of AWS, --ec2-instance-profile-name seems like an appropriate name. It's long, but it if someone were to Google for "instance-profile-name", it would lead them to the AWS docs.
Feedback?

Sounds good to me. :+1:
Long option names are not a big obstacle in Flintrock since you can persist your preferences to a file and avoid having to type them up. It's better to have descriptive names, in general.


Unable to use environment variables in development mode

Issue summary
I created a new shopify app (node-react) using the shopify-cli and then I created a .env file in the root folder and ran the npm run shopify app env pull command. The relevant environment variables were added to my .env file but I am not able to access them in the code right now.

Node version: 18
Operating system: MacOS

Expected behavior
Should return the relevant environment variables when I log them.
Actual behavior
Only returns the default created variables which were generated when I ran the npm run shopify app env pull command.
Steps to reproduce the problem

Create a new app using shopify-cli
Install the app in store
Run the app using npm run dev

npm run shopify app env pull will generate a .env file but I don't think it uses that file when passing environment variables into the application (it builds the environment independently of .env).
To pull the variables in .env into your app, you'll need to add something like the dotenv package (to web/package.json) and add
import * as dotenv from 'dotenv';
dotenv.config();

to the index.js file.

Yes, that is correct ... I should have mentioned that it needs to be inside the web folder (where the app server is), not the root folder (which is mainly for the CLI).

Can you help me out with something else as well though?
When I run the app using npm run dev, it automatically generates an ngrok URL and asks me to update the App URLs inside the partners' dashboard. Is there a way I can give it a static ngrok URL and not the one it generates by default?

Yes, that is correct ... I should have mentioned that it needs to be inside the web folder (where the app server is), not the root folder (which is mainly for the CLI).

I tried this and it didn't work.

The .env file is in web folder

dotenv is installed & showing in web/package.json

index.js file contains:
import * as dotenv from 'dotenv';
dotenv.config();

restarted the server countless times in between every change

this is how I am accessing my key: process.env.KEY_NAME
Any advice to troubleshoot this?


Use a more compact format for the fallback EventBus-listeners in PDFViewerApplication_initializeJavaScript

Please refer to the individual commit messages for additional details.

/botio integrationtest

Looks good!


dragonboard-410c: create 32-bit machine configuration

This adds a 32-bit db410c machine.
On of the usage scenarios is to build a 32-bit rootfs using this machine, and to combine it with the kernel and the modules from a 64-bit build. The 32-bit linux kernel is not needed, so we use "linux-dummy" for virtual/kernel provider, and don't include the kernel into the rootfs image.

Thanks for the review!
I've updated the branch [1].
[1] https://github.com/andrey-konovalov/meta-qcom/tree/master-add-db410c-32

On 09/22/2016 04:07 PM, Nicolas Dechesne wrote:

@ndechesne commented on this pull request.

In conf/machine/dragonboard-410c-32.conf https://github.com/ndechesne/meta-qcom/pull/24#pullrequestreview-1136370:

@@ -0,0 +1,23 @@
+#@TYPE: Machine
+#@NAME: dragonboard-410c

fix NAME value

Oops.. My bad.

In conf/machine/dragonboard-410c-32.conf https://github.com/ndechesne/meta-qcom/pull/24#pullrequestreview-1136370:

+MACHINE_FEATURES = "usbhost usbgadget alsa screen wifi bluetooth ext2"

+SERIAL_CONSOLE = "115200 ttyMSM0"
+
+# Building 32-bit kernel is not supported.
+PREFERRED_PROVIDER_virtual/kernel = "linux-dummy"
+RDEPENDS_kernel-base = ""
+
+MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += " \

${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'mesa-driver-msm', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'wifi', 'wcnss-config wcnss-start', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'bluez5', 'bluez5-noinst-tools', '', d)} \
${@'firmware-qcom-dragonboard410c' if d.getVar('ACCEPT_EULA_dragonboard-410c-32', True) == '1' else ''}
+"

+CMDLINE = "console=ttyMSM0,115200n8 root=/dev/mmcblk0p10 rootwait"
that line was removed in the other machines. please remove it from here too.

Thanks! I've missed commit e6e83d7 "... remove unused variable"

master-add-db410c-32 branch has been updated.

@ndechesne ping

Thanks Nicolas!


[BUG] The loaders module in the document has a lot of examples that have expired and cannot be used properly.

Required prerequisites

[X] I have read the documentation https://camel-ai.github.io/camel/camel.html.
[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[X] Consider asking first in a Discussion.

What version of camel are you using?
0.2.9
System information
I recently discovered that a lot of the documentation in this part of the Loader module was unusable while writing code about chunkr and firecrawl. Cookbook also has some code that cannot be used.
Problem description
I recently discovered that a lot of the documentation in this part of the Loader module was unusable while writing code about chunkr and firecrawl. Cookbook also has some code that cannot be used.in camel 0.2.9
Reproducible example code
The Python snippets:

Command lines:

Extra dependencies:

Steps to reproduce:

from io import BytesIO
from camel.loaders import read_file
Read a pdf file from disk
with open("test.pdf", "rb") as file:
file_content = BytesIO(file.read())
file_content.name = "test.pdf"
Use the read_file function to create an object based on the file extension
file_obj = read_file(file_content)
Once you have the File object, you can access its content
print(file_obj.docs[0]["page_content"])
Traceback
No response
Expected behavior
If you have time, I'll help rewrite the example code in the loaders section.
Additional context
If you have time, I'll help rewrite the example code in the loaders section.

Thanks @xzjjj ! The documentation for loader is indeed outdated, it would be great if you can contribute to fix this, here is our contribution guideline for your reference: https://github.com/camel-ai/camel/blob/master/CONTRIBUTING.md
Could @WHALEEYE  also help with this?


feat: copy repo page PR chart as image

Description
Repository page's PR chart can now be copied as an image by clicking a button available on hover. Implemented with html2canvas-pro via CopyContainer; implements PostHog captures for analytics.

Related Tickets & Documents
Closes #4022

Mobile & Desktop Screenshots/Recordings

Steps to QA

Go to a repo page and hover over the 'PR' chart
Click the button that appeared on the top right corner to copy the chart
Paste image to GitHub, Twitter, etc. textbox to see image and confirm information
Go to PostHog events and check if the click sent the event Repo Pages: clicked PR chart

Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[x] Tier 3
[ ] Tier 4

[optional] What gif best describes this PR or how it makes you feel?

Same problem as: #4026 (review)

I have an issue (https://github.com/open-sauced/app/issues/4028) for this, would love for this PR to go through and I'll address the fix alongside the other charts in another PR.


Skin converter cannot be saved.

Skin converter cannot be saved.
When I selected several skins, I restarted the game. After loading, the selected skin disappeared.

But the configuration of Aimbot, Triggerbot, Backtrack, Glow, Chams and so on is not a problem.
Is that the only way my computer works?

The gun has too many skins, so it's difficult to choose. Can you add a sort or search?

That's a problem listed to be fixed in the near future.


Added tsvd tests and fixed one small bug

I added a few test cases for tsvd module, and while working on them, I found another bug in the tsvd implementation within the svd_flip function, which I have fixed.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Two last days only in JSON (by location)

Am I missed something? Could it be possible to restore access to full dataset?  Re-deploying now!

Re-deployed, this should be fixed. Thanks!

@lazd btw, i can help in automatization of deployment process: ansible, raw docker, kubernetes - on your choice!


params get replaced by [params.id] when they shouldn't

Hello, I tried to pass some params to route function, but they were ignored. So i was trying to understand why and found this piece of code:
if (
    params.hasOwnProperty('id') &&
    this.template.indexOf('{id}') == -1
) {
    params = [params.id];
}

In my route template i have {id?} instead of just {id}, so the condition this.template.indexOf('{id}') == -1 evaluates as true and that means all my params get replaced by just [params.id].
I don't know, if that's something important for you and for this package. For now I'm gonna make some workaround, that will work for me. I just wanted to let you know... and to say thanks for great package 👍

@briska this was fixed a while ago but the fix has not been released yet, technically it's a breaking change so it's going to be in 1.0. Hopefully that release will be out some time in the next few weeks. Thanks for your patience!


unetbootin: 681 -> 702

Motivation for this change
Update to latest upstream release 702
Change log:

https://github.com/unetbootin/unetbootin/releases/tag/702
https://github.com/unetbootin/unetbootin/releases/tag/700

unetbootin now requires QT5. GUI is coming up but I get some errors.
$ sudo ./result/bin/unetbootin
QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-root'
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile
libpng warning: iCCP: known incorrect sRGB profile

I would appreciate if somebody with proper knowledge about QT/qmake could take a look. Thanks.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129306 run on x86_64-linux 1

1 package built:

unetbootin

Result of nixpkgs-review pr 129306 run on x86_64-linux 1

1 package built:

unetbootin


Fix upload is not working

Closes: https://github.com/metarhia/Example/issues/212
Currently the issue in client side code:
https://github.com/metarhia/Example/blob/2435b8e643af3fc4a62fea1f3f20bdedc1e1d9c1/application/static/console.js#L187-L193
streamId isn't exists in return contract of application.metacom.createBlobUploader from
https://github.com/metarhia/Example/blob/2435b8e643af3fc4a62fea1f3f20bdedc1e1d9c1/application/static/metacom.js#L74-L75
Current state at the metacom repository is the same https://github.com/metarhia/metacom/blob/acb90a6d8cb358fe1db9e28cec8f9b4a8107cffc/dist/metacom.js#L76-L77

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

Thank you @tshemsedinov for the approval of workflow testing. I really missed that static.test relied on console.js file size that has been changed due to code fix.  All test expected to pass.


JS code for retrieving weather

Create the JS code for retrieving the weather forecast from the National Weather Service API. Should be separated from the geocoding code for use if the latitude/longitude are already stored.

This is complete pending the sign-off on the PR

PR was approved


Fix for prvConnectHelperWithRetry function in iot_test_tcp.c

Description

The function prvTcpSocketHelper called by prvConnectHelperWithRetry returns error code, if Socket cannot be allocated and the error code is stored in variable xResult.
Previously this variable was never addressed after an error is received. Thus, the code runs in an infinite loop(which is not expected). This is the reason for failure of AFQP_SOCKETS_Threadsafe_DifferentSocketsDifferentTasks test-case.
The expected behaviour is; the function should retry obtaining a socket repetitively after a specific delay.
In prvConnectHelperWithRetry, check is added to retry until valid socket is received.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

Hi shubhamkulkarni97,
Thank you for raising this PR. I agree with you that if unable to create socket in prvTcpSocketHelper(), prvConnectHelperWithRetry() will stuck in do-while loop.
Though, I think there may be a better fix than exponential backoff and retry -- break out of the loop when failing to create socket without retry. Reasoning as below.
The original code looks like
        /* Create the socket. */
        *pxSocket = prvTcpSocketHelper( pxSocketIsAllocated );

        if( SOCKETS_INVALID_SOCKET == *pxSocket )
        {
            xResult = SOCKETS_SOCKET_ERROR;
        }

And in prvTcpSocketHelper() we create a socket by calling SOCKETS_Socket(). The reason the system may fail to create socket can be:

fail to allocate memory to hold internal structure.
board does not support more than a number of sockets.

In the former case, though memory can be freed in between retries (assuming dynamic allocation is supported), system is already struggling with insufficient memory. In the latter case, retry would not make too much difference if system ran out of available socket, given when testing we don't have other things running.
Given above, I think adding a break; statement is sufficient. Test will fail as expected. Let me know your thoughts.

What is the root cause of the socket allocation failure? Would be helpful to know if it’s RAM, max connections, or etc.

What is the root cause of the socket allocation failure? Would be helpful to know if it’s RAM, max connections, or etc.

If a closed socket is in time-wait state, PCB for that socket is not cleared. Hence, if the total count of active PCBs exceed the maximum count, socket allocation fails.

Hi shubhamkulkarni97,
Thank you for raising this PR. I agree with you that if unable to create socket in prvTcpSocketHelper(), prvConnectHelperWithRetry() will stuck in do-while loop.
Though, I think there may be a better fix than exponential backoff and retry -- break out of the loop when failing to create socket without retry. Reasoning as below.
The original code looks like
        /* Create the socket. */
        *pxSocket = prvTcpSocketHelper( pxSocketIsAllocated );

        if( SOCKETS_INVALID_SOCKET == *pxSocket )
        {
            xResult = SOCKETS_SOCKET_ERROR;
        }

And in prvTcpSocketHelper() we create a socket by calling SOCKETS_Socket(). The reason the system may fail to create socket can be:

fail to allocate memory to hold internal structure.
board does not support more than a number of sockets.

In the former case, though memory can be freed in between retries (assuming dynamic allocation is supported), system is already struggling with insufficient memory. In the latter case, retry would not make too much difference if system ran out of available socket, given when testing we don't have other things running.
Given above, I think adding a break; statement is sufficient. Test will fail as expected. Let me know your thoughts.

If a closed socket is in time-wait state, it will be cleared in finite amount of time. Hence, I think adding a retry with some delay may help in allocating a socket.
Let me know your views.
If you still think adding a break; is a better alternative, I can update the PR.

I understand the motivation for the retry. If this part of the code is in application, I would've agreed with the fix. However, prvConnectHelperWithRetry() is part of the test. And with every function call to prvConnectHelperWithRetry() in the same file, there is either an assert followed or check followed. Thus, breaking out the do-while loop early in the test, will make test more efficient. Another fact for our tests is that, while test is running, there is nothing else running. Thus, if we have trouble creating more than one socket already, then the test is solidly failed.
So, yes, I'd vote for adding a break; statement after socket creation failure in the original code.
-------- more details --------

Without any fix, this particular test case will stay in do-while loop upon failure to create more than one socket. The test group will timeout with other tests behind this test case unfinished.
With shubhamkulkarni97's original proposal for adding retries-timeout after socket creation failure, the test group will break out of do-while loop after retries. The test group will fail faster.
With break; after socket creation failure, the test fails immediately. This is given during test, nothing else is running.

@yuhui-zheng I have pushed required changes. Please take a look.

/bot run checks

/bot run checks

/bot run checks

/bot run checks

/bot run checks


SafeRegex: need script to determine impact

We should create a script to determine if a user may be impacted by https://github.com/istio/istio/pull/17490. This would entail something similar to `kubectl get virtualservice --all-namespaces -ojsonpath='{..regex}' and smoke checking each regex used

@fpesce what is the status of this? This is probably not useful unless done before 1.4 ships (nov 12)

@fpesce is this feasible for 1.4?

Not feasible or as useful now that we have shipped. No one complained...


feat: updates cache

What
feat: updates cache

:tada: This PR is included in version 1.76.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix error with hugo container

Apparently there was a bug with running hugo-extended in alpine containers. I thought it was working, but it wasn't.
This PR switches from alpine to the base versions to resolve it.

Also added the "Add your own run statement" comment for #409.

LGTM! Thanks!


Third tranche of NdArray refinement.

Continuation of work on #330. This PR primarily focused on the read/write interface of NdArray
Changes:

Added types for various paramater constants (eg, read batch-format)
Ported SOMAMeasurement to the new SOMACollection MutableMapping ABC interface
alignment of NdArray read/write interface to latest spec and added unit tests.

Dense: now centered around dense subarray read/write using Arrow Tensor (and numpy ndarray)
Sparse: support both Arrow sparse tensors, Arrow Table, and pandas (read only)
NdArray can now read/write more than 2D (the previous implementation was hard-wired to 2 dimensions)
Changed the read/write API names to be explicit about what types they consume/produce (eg, read_tensor)

Awesome, thanks @bkmartinjr ! :)


[Android] Incorrectly getting Bluetooth Powered off error

Sometimes when trying to reconnect to a device I get the following error:
{message: "Bluetooth is powered off", code: 102}

Even though the Bluetooth is definitely on. It also results in me not being able to reconnect.

Hello,
I will try to update RxAndroidBle backend to new 1.1.0 version and then we will see if it's still reproducible.

Great!

Could you check latest 0.5.0 version?

Sorry, moved to another library so not using this atm

Closing as this is an overdue issue. Please feel free to reopen if you tested it with the latest version of a library.


fix: remove-display-table for mobile device

While I reading typescript handbook with tablet PC, I found that part of document is hidden in tablet PC screen.

To fix this I remove 'display: table' property from 'table.full-width-table'

table.full-width-table selector is used in 2 pages. (one of them is deprecated)
and after removing property, it works well.

@microsoft-github-policy-service agree

@jakebailey
I'm sorry. I did not notice that my code changes original layout of document. Since I checked the page with my laptop. Therefore It seemed like layout was same as original one. I fixed my code. Please check it.


python311Packages.hahomematic: 2023.9.2 -> 2023.9.3

Diff: https://github.com/danielperna84/hahomematic/compare/refs/tags/2023.9.2...2023.9.3
Changelog: https://github.com/danielperna84/hahomematic/releases/tag/2023.9.3
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] 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.

Result of nixpkgs-review pr 256040 run on x86_64-linux 1

2 packages built:

python311Packages.hahomematic
python311Packages.hahomematic.dist


[2203] DiskRP migration progress change

Upgrade API version from 20210401 to 20210901

/azp run

/azp run


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20231214163910130 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20231214163910130 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.604%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30818

Relevant Lines:
69389

💛  - Coveralls


GAE (Flex_Java/Flex_Custom/Flex_Compat/Std_Java8), GCE, GKE testing app for gcj

https://github.com/GoogleCloudPlatform/google-cloud-java/issues/1752

Changes Unknown when pulling 4ca6737c1afcf8d80f5afc2816911c136b08d067 on vam-google:master into ** on GoogleCloudPlatform:master**.

Changes Unknown when pulling 4ca6737c1afcf8d80f5afc2816911c136b08d067 on vam-google:master into ** on GoogleCloudPlatform:master**.

Addressed review comments. PTAL

Uncommented. PTAL

Changes Unknown when pulling f4fde1113ae976e1e92e3a5d1ef2174340e357d1 on vam-google:master into ** on GoogleCloudPlatform:master**.

Changes Unknown when pulling 5ab123b2cc8eea35b66bf679f66d0b782831a766 on vam-google:master into ** on GoogleCloudPlatform:master**.

More seds, apparently mvn clean install -DskipITs can be used to skip tests. PTAL.

Changes Unknown when pulling e97f961e029b704ecb2501f346e2ae84cc0d3457 on vam-google:master into ** on GoogleCloudPlatform:master**.

LGTM after tests pass

Changes Unknown when pulling 798ff0de2eab6e8272c52231848330e85c1610a0 on vam-google:master into ** on GoogleCloudPlatform:master**.


Create a user authentication demo in nodejs restify framework

Need to create a small user authentication demo using LoginRadius solution, Check our Demo Guidelines for how to make the authentication demo and Contributing Guidelines to know about the contribution flow.
Platform - NodeJS (latest stable  version)
Framework - Restify (latest stable version)
Code Folder - https://github.com/LoginRadius/developer-authentication-demos/tree/master/demos/nodejs/restify

I can take this one.

Hello, @karmanya007  Thanks for your interest Please feel free to create a demo and let us know here once you raise a PR.
Also please go through https://www.loginradius.com/engineering/page/hacktoberfest2020 to get free swags from LoginRadius as well.
Happy Coding :)

Done!!
PR opened.


SatSale doesn't work with Python 3.10

When starting with Python 3.10, there is error (which seems to be issue with gunicorn / flask_restplus, not SatSale itself):
$ python3.10 ~/.local/bin/gunicorn -w 1 -b <IP_ADDRESS>:8000 satsale:app
[2022-08-29 18:07:29 +0300] [24238] [INFO] Starting gunicorn 20.0.4
[2022-08-29 18:07:29 +0300] [24238] [INFO] Listening at: http://<IP_ADDRESS>:8000 (24238)
[2022-08-29 18:07:29 +0300] [24238] [INFO] Using worker: sync
[2022-08-29 18:07:29 +0300] [24239] [INFO] Booting worker with pid: 24239
[2022-08-29 18:07:29 +0300] [24239] [ERROR] Exception in worker process
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/arbiter.py", line 583, in spawn_worker
    worker.init_process()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/workers/base.py", line 119, in init_process
    self.load_wsgi()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/workers/base.py", line 144, in load_wsgi
    self.wsgi = self.app.wsgi()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/app/base.py", line 67, in wsgi
    self.callable = self.load()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/app/wsgiapp.py", line 49, in load
    return self.load_wsgiapp()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/app/wsgiapp.py", line 39, in load_wsgiapp
    return util.import_app(self.app_uri)
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/util.py", line 358, in import_app
    mod = importlib.import_module(module)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/user/git/SatSale/satsale.py", line 7, in <module>
    from flask_restplus import Resource, Api, fields
  File "/home/user/.local/lib/python3.10/site-packages/flask_restplus/__init__.py", line 4, in <module>
    from . import fields, reqparse, apidoc, inputs, cors
  File "/home/user/.local/lib/python3.10/site-packages/flask_restplus/reqparse.py", line 19, in <module>
    from .model import Model
  File "/home/user/.local/lib/python3.10/site-packages/flask_restplus/model.py", line 8, in <module>
    from collections import OrderedDict, MutableMapping
ImportError: cannot import name 'MutableMapping' from 'collections' (/usr/lib/python3.10/collections/__init__.py)
[2022-08-29 18:07:29 +0300] [24239] [INFO] Worker exiting (pid: 24239)
[2022-08-29 18:07:29 +0300] [24238] [INFO] Shutting down: Master
[2022-08-29 18:07:29 +0300] [24238] [INFO] Reason: Worker failed to boot.

Seems to be this issue - https://stackoverflow.com/questions/70870041/cannot-import-name-mutablemapping-from-collections.

After #99 solving above error, now have next one:
$ python3.10 ~/.local/bin/gunicorn -w 1 -b <IP_ADDRESS>:8000 satsale:app
[2022-09-17 12:25:42 +0300] [28957] [INFO] Starting gunicorn 20.0.4
[2022-09-17 12:25:42 +0300] [28957] [INFO] Listening at: http://<IP_ADDRESS>:8000 (28957)
[2022-09-17 12:25:42 +0300] [28957] [INFO] Using worker: sync
[2022-09-17 12:25:42 +0300] [28959] [INFO] Booting worker with pid: 28959
[2022-09-17 12:25:43 +0300] [28959] [ERROR] Exception in worker process
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/arbiter.py", line 583, in spawn_worker
    worker.init_process()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/workers/base.py", line 119, in init_process
    self.load_wsgi()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/workers/base.py", line 144, in load_wsgi
    self.wsgi = self.app.wsgi()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/app/base.py", line 67, in wsgi
    self.callable = self.load()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/app/wsgiapp.py", line 49, in load
    return self.load_wsgiapp()
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/app/wsgiapp.py", line 39, in load_wsgiapp
    return util.import_app(self.app_uri)
  File "/home/user/.local/lib/python3.10/site-packages/gunicorn/util.py", line 358, in import_app
    mod = importlib.import_module(module)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/user/git/SatSale/satsale.py", line 28, in <module>
    from node import xpub
  File "/home/user/git/SatSale/node/xpub.py", line 5, in <module>
    from bip_utils import Bip84, Bip44Changes, Bip84Coins, Bip44, Bip44Coins
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/__init__.py", line 5, in <module>
    from bip_utils.addr import (
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/addr/__init__.py", line 1, in <module>
    from bip_utils.addr.ada_byron_addr import (
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/addr/ada_byron_addr.py", line 41, in <module>
    from bip_utils.base58 import Base58Decoder, Base58Encoder
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/base58/__init__.py", line 1, in <module>
    from bip_utils.base58.base58 import Base58Alphabets, Base58Decoder, Base58Encoder
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/base58/base58.py", line 28, in <module>
    from bip_utils.utils.crypto import DoubleSha256
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/utils/crypto/__init__.py", line 1, in <module>
    from bip_utils.utils.crypto.aes_ecb import AesEcbDecrypter, AesEcbEncrypter
  File "/home/user/.local/lib/python3.10/site-packages/bip_utils/utils/crypto/aes_ecb.py", line 29, in <module>
    from Crypto.Cipher import AES
  File "/home/user/.local/lib/python3.10/site-packages/Crypto/Cipher/__init__.py", line 31, in <module>
    from Crypto.Cipher._mode_ctr import _create_ctr_cipher
  File "/home/user/.local/lib/python3.10/site-packages/Crypto/Cipher/_mode_ctr.py", line 37, in <module>
    from Crypto.Util.number import long_to_bytes
  File "/home/user/.local/lib/python3.10/site-packages/Crypto/Util/number.py", line 399
    s = pack('>I', n & 0xffffffffL) + s
                                ^
SyntaxError: invalid hexadecimal literal
[2022-09-17 12:25:43 +0300] [28959] [INFO] Worker exiting (pid: 28959)
[2022-09-17 12:25:43 +0300] [28957] [INFO] Shutting down: Master
[2022-09-17 12:25:43 +0300] [28957] [INFO] Reason: Worker failed to boot.

Found the problem - even current master of bip_utils requires pycryptodome 3.6, which has 0xffffffffL instead of 0xFFFFFFFF in code, incompatible with latest Python versions. So, we need fix in bip_utils, not here (bip_utils itself is compatible with Python 3.10, problem is dependency on old version of pycryptodome).

Hmm, or it is just problem on my machine. Did pip uninstall pycryptodome and then pip install -r requirements.txt --user again and now it has recent enough pycyptodome and no more error.

Did upgrade to Python 3.10 on a different machine, everything seems to working after merge of #99, so closing.


feat: pass command line arguments for start command

Following on from #530
We now have:
$ karma start &
$ karma run -- --grep someTest

But that sucks for batch runs (like CI scripts), because you need to:

run karma start
parse its output (!) to see when it's ready
run karma run -- <args>
kill the first process
deal with other messy stuff related to managing multiple processes

There is real need for this feature, e.g segmenting travis builds by test type:

karma start --single-run -- --tag unit
karma start --single-run -- --tag integration

Options:

treat everything after -- as clientArgs, just like we do for run.
@vojtajina doesn't like this, because it removes the possibility of using this for something else in the future.
an explicit --client-args would trigger that ALL following arguments should be taken as clientArgs:
karma start --single-run --client-args --grep unit
This should probably grab only up to the next "--" argument though.
more tediously, specifying them individually:
karma start --single-run --client-arg=--grep --client-arg=unit

Other suggestions?

@gaelazzo A bit late to the party, but since this there's a much simpler solution now:
karma start --grep integration

karma.conf.js:
module.exports = function (config) {
  config.set({
    ...,
    client: {
      args: ['--grep', config.grep]
    }
  })
}

Many thanks.

@danielsiwiec, still getting Executed 1 of 12.
./node_modules/.bin/karma start --grep test/user.spec.ts

./node_modules/.bin/karma start --grep UserService

./node_modules/.bin/karma start -- --grep="UserService"

None of those codes above seems to make any diference.
Is it really possible run a single test?


[REFACTOR]: [MOLLIE] Add amount conversion framework to Mollie

:memo: Feature Description
Currently, amounts are represented as i64 values throughout the application. We want to introduce a Unit struct that explicitly states the denomination. A new type, MinorUnit, has been added to standardize the flow of amounts across the application. This type will now be used by all the connector flows.
Rather than handling conversions in each connector, we will centralize the conversion logic in one place within the core of the application.
:hammer: Possible Implementation

For each connector, we need to create an amount conversion function. Connectors will specify the format they require, and the core framework will handle the conversion accordingly.
Connectors should invoke the convert function to receive the amount in their required format.
Refer to the connector documentation to determine the required amount format for each connector.
You can refer this PR for more context.

🔖 Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/ , crates/router/src/types/api.rs , crates/router/tests/connectors/
:package: Have you spent some time checking if this feature request has been raised before?

[ ] I checked and didn't find a similar issue

:package: Have you read the Contributing Guidelines?

[ ] I have read the Contributing Guidelines

:package: Submission Process:

Ask the maintainers for assignment of the issue, you can request for assignment by commenting on the issue itself.
Once assigned, submit a pull request (PR).
Maintainers will review and provide feedback, if any.
Maintainers can unassign issues due to inactivity, read more here.

Refer here for Terms and conditions for the contest.
:sparkles: Are you willing to submit a PR?

Hey @swangi-kumari , please assign me this issue

Hey @sahil9001 Thanks for your interest, As you have requested another issue to you, i would suggest to keep working on it.
Please reach out here, once that is done.
This will help us maintain fairness in the assignment process.

Hey @swangi-kumari I would like to work on this issue.

Sure @aayushdhiman01 ,
I have assigned this issue to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server

Hey @aayushdhiman01 ,
Quick check-in on the status of this issue. If you have any questions or need assistance, feel free to reach out in the community.

Hey @gorakhnathy7 , I would like to work on this issue.

Sure @Sidharth-Singh10 assigning the issue to you!


Production. CKEditor TypeError: c[a] is undefined in admin page

django-ckeditor-5.2.2
I've some problem with deployment my application to production server. If I've set in settings.py
DEBUG = False
TEMPLATE_DEBUG = DEBUG

STATIC_URL = '/static/'
STATICFILES_FINDERS = (
    'django.contrib.staticfiles.finders.FileSystemFinder',
    'django.contrib.staticfiles.finders.AppDirectoriesFinder',
)

error
TypeError: c[a] is undefined

and content not showing
html:
<script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/2.2.4/jquery.min.js"></script>
<script type="text/javascript" src="/static/ckeditor/ckeditor-init.4374360ba35c.js"></script>
<script type="text/javascript" src="/static/ckeditor/ckeditor/ckeditor.8bd276b5ef4c.js"></script>

Does it work without the ManifestStaticFilesStorage? Are you using other libraries wrapping django-ckeditor? Can you show the full traceback for the JS error?

not working without the ManifestStaticFilesStorage .
when i false DEBUG not wokring, but other all scripts worked.
JS errors:

"NetworkError: 404 Not Found - http://domain_name/admin/news/article/add/config.js/change/"
/manage...change/
"NetworkError: 404 Not Found - http://domain_name/admin/news/article/add/skins/moono-lisa/editor_gecko.css/change/"
/manage...change/
"NetworkError: 404 Not Found - http://domain_name/admin/news/article/add/lang/en.js/change/"
/manage...change/
TypeError: c[a] is undefined

...s.detect(d,a));var c=this;d=function(){c[a].dir=c.rtl[a]?"rtl":"ltr";b(a,c[a])};...

Thanks.
@matthiask worked !
:+1:

Great!


Fixes #1207

Description
Fixes #1207
login_credentials textview added to landscape and x-large activity_login.xml
Screenshots showing what changed

Codecov Report

Merging #1208 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1208   +/-   ##
======================================
  Coverage    3.82%   3.82%
======================================
  Files         125     125
  Lines        5810    5810
  Branches      568     568
======================================
  Hits          222     222
  Misses       5573    5573
  Partials       15      15

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 215c97e...541976c. Read the comment docs.

The PR fixes the crash. Tested on a real device. Going ahead and merging it.
Thanks for your contribution @knight-shade. Keep up the good work! :)

Thank you very much @maskaravivek


add support for installation behind SSL proxy

If beeing behind a SSL proxy (nginx in my case), wordpress tried to load assets using http instead of https. These lines change the protocol to https if behind a proxy.

Thank you for your pull request!
I'll check the changes and give you feedback!

Since this image is meant for testing and development, I would prefer to use a simple base WordPress configuration with minimal side effects.
If you need some custom configurations you could either use this image as the base image and override the wp-config.php file with your own configuration:
FROM kaihofstetter/wordpress-cli:4.4.1
MAINTAINER ...

# WordPress configuration
ADD wp-config.php /var/www/html/wp-config.php
...

Or you could simply copy your own wp-config.php file in the container:
docker cp wp-config.php my-wordpress:/var/www/html/wp-config.php

Thank you for your pull request!


Change default experiments instructions to accommodate multiple assignments to expert workspaces

Currently, experiments are by default created with instructions that tell users to contact an admin if they're assigned to an expert workspace more than once. There are, however, legitimate reasons why this might occur, so the default instructions should be changed to reflect this.

Resolved in #612


How to make the feature rotate angle

when draw the group of feature,I want to make the feature rotate angle.
Example：

I has draw it,and now I want make it

rotate 45°,
Can you show me a demo?
And My English is not good,so sorry, hope you can got it. Thank you very much!

Please ask such kind of usage question on StackOverflow. There are some plugins that might be interesting for you: http://viglino.github.io/ol3-ext/examples/map.interaction.transform.html https://ghettovoice.github.io/ol3-rotate-feature/

thank you


updated README

reflects community now ;-)

Hello,
I am not sure how to separate the acceleration magnitude to x,y,z with Python. In the Experiment 2, it says we can do that with small modifications but I am not sure how to do it. Any help or suggestions will be helpful. Thanks!


Allow multiple wallets

Is it not the case that MetaMask already has a provider which allows multiple wallets? I mean, I have multiple wallets in my browser's MetaMask presently. I'm not actually familiar with this code base, but in regards to https://github.com/trufflesuite/truffle-hdwallet-provider/pull/3 it would be cleaner to require a new provider from here that already supports this.
But like I said I'm not familiar with this codebase so I could be smoking reefer.

MetaMask is able to have multiple wallets by using Hooked-Waller-Subprovider, a different, more open ended subprovider for providing accounts and signing functions.
That means this change isn't entirely necessary, but it also probably doesn't hurt. Needs a good review, but if it's solid I don't see anything wrong with it.

@flyswatter Is there an example of how to instantiate a Hooked-Wallet-Subprovider for truffle-hdwallet-provider? I'd like the same functionality proposed in the above linked PR of the latter repo, but I'd rather not have to change both repos.

Please see https://github.com/trufflesuite/truffle-hdwallet-provider/pull/5

@cemozerr im closing this because its a bit old and theres not a lot of requests for this feature
the hooked subprovider provides lots of hooks to customize how you get available addresses, so i think this can be solved without changes to provider-engine. let me know if you think this deserves more attention


Replace pcre with pcre2

SiriDB is using the older pcre3 library and should switch to the newer pcre2 lib.

Replaced pcre with pcre2


Custom Style are not applied in Copilot mode

Describe the bug
Custom Style are not applied in Copilot mode
To Reproduce
Define custom css in a file.
Run the application.
Create another html file with necessary scripts included for copilot.
THe custom styles dont have an effect.
Expected behavior
Custom css styles are applied
Screenshots

You can see that the watermark is still visible. I have styles to hide it. Works in desktop mode. but not in copilot mode.
Desktop :
Windows, chrome

OS:Windows
Browser chrome

I found a work around, see here: https://github.com/Chainlit/chainlit/issues/1022


Pihole V5

Hi there,
Does this work with Pi Hole v5? There have been quit a few changes I think.
Many thanks
Jason

It works with v5 now.


routePrefix option in host.json

I added routePrefix option to host.json to use custom route and domain.
For example, if custom domain is webapp.com, I want to call API with /azure/api route.
It worked in the local environment, but on GitHub action process, got below error.
---End of Oryx build logs---
Function Runtime Information. OS: Linux, Functions Runtime: v3, Node version: 12.X
Finished building function app with Oryx
Found and validated 'routes.json' within build artifacts folder.
Error in processing api build artifacts: the host.json file cannot specify a http.routePrefix value other than 'api'.

My host.json settings:
{
  "version": "2.0",
  "watchDirectories": [ "lib" ],
  "extensions": {
    "http": {
      "routePrefix": "azure/api",
    }
  },
  "extensionBundle": {
    "id": "Microsoft.Azure.Functions.ExtensionBundle",
    "version": "[1.*, 2.0.0)"
  }
}

Is the routePrefix option is not supported in the azure static web app?

This is currently not supported. Are you already using the /api route? Wondering what the reason to change the api path is

Would also love to customize this. api.example.com/api/... is confusing and undesirable.

If you set the api_location value to an empty string ("") in the workflow configuration, you might be able to work around the limitation.
Obviously, this means additional config in your WebApp and creation of Bring Your Own Functions, etc.

This would not work. Currently, Static Web Apps will only proxy requests for /api/* to the function app (managed or bring your own).

any chance to enhance this in the near future?


Initial support for 'file' SCM

Handle file:// protocol in source with fs.Copy()
GIT is no longer required when using file://
Initial work on supporting more SCM

@soltysh @bparees PTAL

[test]

LGTM [merge]


(bugfix) Fixes bug in folder tree, related to special chars

Resolving #469 by simplifying collapse IDs in folder tree.
Make use of integers for collapse IDs rather than folder names to avoid
problem with any kind of non-allowed characters as ID value.
Simplifies corresponding code as no character replacement is required
for collapse IDs anymore.

This looks a lot easier to me, but I don't know, if there are any other reasons for the old implementation.

This looks a lot easier to me, but I don't know, if there are any other reasons for the old implementation.


Extract context definition into a different section

This PR extracts the context definitions into a different section.
The list of context names are:

"compact": Any compact, tabular presentation of data from multiple entities.
"detailed": Any detailed read-only, entity-level presentation context.
"entry": Any data-entry presentation context, i.e. when prompting the user for input column values.

"entry/edit": A sub-context of entry that only applies to editing existing resources.
"entry/create": A sub-context of entry that only applies to creating new resources.

"filter": Any data-filtering control context, i.e. when prompting the user for column constraints or facets.
"name": Any abbreviated title-like presentation context.
"*": A default to apply for any context not matched by a more specific context name.

Questions

I removed the record context in favor of detailed. Their definitions are the same and I think detailed is a better name for this context. Is this fine?

The name context is only defined under 2016 Table Display annotation. Should I remove this context?

In each annotation only a portion of supported contexts was mentioned. I removed those list and instead added a link to list of all contexts. Should I mention the contexts that make sense for a particular annotation too?

Replacing record with detailed seems fine, as they are accidental
synoyms.
I just added name yesterday so no, don't remove it. :-)  This allows us to drop the separate row_name option and just use the row markdown pattern with this name context instead.
If we collapse to one section itemizing the context definitions (a good idea), we should probably also include a matrix of which context is meaningful in which annotation, or just keep a short list of contexts in each annotation section (with links back to their definitions rather than repeating definitions).

I generated this matrix based on the documentation.

 Shouldn't visible-foreign-keys have the * context too?

I think all of them should have *...

This is very helpful. Matrix is great too.
Regarding those "supported context name" sub-section that appear under several annotations.

Now that you've pulled out the definitions for the context's, let's not redefine them in each annotation. At most, we just want a set of context names that are supported (w/out definitions).

But I kind of wonder maybe we should not even define what is supported in the individual annotations. Just leave that up to the clients to decide what makes sense to support in a given context (?)

@RFSH can you add the matrix to the PR before I merge it?

@karlcz I added the matrix.

The matrix preview here https://github.com/RFSH/ermrest/blob/af1911157f1f1e2cf0f01a8a69cc88820c928cf4/user-doc/annotation.md looks garbled

Fixed it.
https://github.com/RFSH/ermrest/blob/context-documentation/user-doc/annotation.md#context-names