[qt5] update to 5.12.3

closes #5929

dont forget to change line 5 in  ports/qt5-base/vcpkg-cmake-wrapper.cmake to link the debug libraries

hit a problem on qt5-tools not using the correct path for a debug library, will look into this tomorrow

hit a problem on qt5-tools not using the correct path for a debug library, will look into this tomorrow

@TheScarfix: let me have a wild guess without looking into it: libpq.

@Neumann-A no, some qtqml library which is searched in the lib path not the debug/lib

still hitting an error on qt5-tools
Error: dependent 'C:\dev\vcpkg\installed\x64-windows\lib\Qt5QmlDevToolsd.lib' does not exist.
jom: C:\dev\vcpkg\buildtrees\qt5-tools\x64-windows-dbg\src\linguist\lupdate\Makefile [debug] Error 2
C:\dev\vcpkg\buildtrees\qt5-tools\src\5.12.3-2e47618cfc\src\linguist\linguist\main.cpp(0): Note: No relevant classes found. No output generated.
jom: C:\dev\vcpkg\buildtrees\qt5-tools\x64-windows-dbg\src\linguist\Makefile [sub-lupdate-make_first] Error 2
jom: C:\dev\vcpkg\buildtrees\qt5-tools\x64-windows-dbg\src\Makefile [sub-linguist-make_first] Error 2
jom: C:\dev\vcpkg\buildtrees\qt5-tools\x64-windows-dbg\Makefile [sub-src-make_first] Error 2

does C:\dev\vcpkg\installed\x64-windows\lib\Qt5QmlDevToolsd.lib exist in C:\dev\vcpkg\installed\x64-windows\lib\debug\Qt5QmlDevToolsd.lib ?

it's debug/lib not lib/debug but yes, it exists

don't think qt building uses cmake in any way. Qts build config says it looks for libraries in debug/lib 🤔

what I would do in these circumstances is a search all for Qt5QmlDevToolsd within the buildtree with Notepad++

okay, maybe it's an error in the Makefile.Debug of qt5-tools

no idea how to fix that though 😅

So the next questions would be:

Where does the makefile come from? Is it generated? (see log files)
If it is generated who does generate it? qmake/cmake?
Where does the information for the generator come from.

theoretically it looks like vcpkg_build_qmake.cmake should have fixed the debug build paths
  file(GLOB_RECURSE DEBUG_MAKEFILES ${DEBUG_DIR}/*Makefile*)

       foreach(DEBUG_MAKEFILE ${DEBUG_MAKEFILES})
           file(READ "${DEBUG_MAKEFILE}" _contents)
           string(REPLACE "zlib.lib" "zlibd.lib" _contents "${_contents}")
           string(REPLACE "installed\\${TARGET_TRIPLET}\\lib" "installed\\${TARGET_TRIPLET}\\debug\\lib" _contents "${_contents}")
           string(REPLACE "/LIBPATH:${NATIVE_INSTALLED_DIR}\\debug\\lib" "/LIBPATH:${NATIVE_INSTALLED_DIR}\\debug\\lib\\manual-link /LIBPATH:${NATIVE_INSTALLED_DIR}\\debug\\lib shell32.lib" _contents "${_contents}")
           string(REPLACE "tools\\qt5\\qmlcachegen.exe" "tools\\qt5-declarative\\qmlcachegen.exe" _contents "${_contents}")
           string(REPLACE "tools/qt5/qmlcachegen" "tools/qt5-declarative/qmlcachegen" _contents "${_contents}")
           string(REPLACE "debug\\lib\\Qt5Bootstrap.lib" "tools\\qt5\\Qt5Bootstrap.lib" _contents "${_contents}")
           string(REPLACE "lib\\Qt5Bootstrap.lib" "tools\\qt5\\Qt5Bootstrap.lib" _contents "${_contents}")
           string(REPLACE " Qt5Bootstrap.lib " " ${NATIVE_INSTALLED_DIR}\\tools\\qt5\\Qt5Bootstrap.lib Ole32.lib Netapi32.lib Advapi32.lib ${NATIVE_INSTALLED_DIR}\\lib\\zlib.lib Shell32.lib " _contents "${_contents}")
           file(WRITE "${DEBUG_MAKEFILE}" "${_contents}")
       endforeach()

hmm
\vcpkg\installed\x86-windows\share\qt5\debug\mkspecs\features\qt_module_pris.prf
lines 175 & 176
to
            "QT_MODULE_LIB_BASE = $$val_escape(MODULE_BASE_OUTDIR)/debug/lib" \
            "QT_MODULE_HOST_LIB_BASE = $$val_escape(MODULE_BASE_OUTDIR)/debug/lib" \

they do not include the debug normally
the next error will be:
LINK : fatal error LNK1104: cannot open file 'Qt5Helpd.lib'
(which does not exist in my installed folder)

After thinking about it. The problem may be in all files in C:\Sources\test\vcpkg\installed\x86-windows\share\qt5\debug\mkspecs\modules having the wrong library path. Which seems to make qmake and vcpkg for external projects in debug mode a no go without manually fixing the makefiles.
It would be interesting to stop cmake right after/before https://github.com/Microsoft/vcpkg/pull/6127#issuecomment-484692227 and see if the offending line is already written to the makefile or not.

Something is strange....
..\..\..\bin\lupdate.exe: C:\Sources\test\vcpkg\installed\x86-windows\debug\lib\Qt5QmlDevToolsd.lib $(OBJECTS) .obj\debug\lupdate_resource.res
stopped right after makefile generation. So the file is changed by someone else again?

it is really weird because it's the only library with problems 😕

looked at it with procmon and it looks like qmake rewrites the file wrong. Can't see why though

Its probably because of what is standing in the mkspecs for the different modules in \vcpkg\installed\x86-windows\share\qt5\debug\mkspecs\ which also makes the string replacements in the build files necessary. Seems like one of the makefiles is rerunning the qmake configuration for that module and reverting the string replacement..

does d299ad61079bc77c331751623308c7739ff977e3 solve the problem?

no, still failing

I changed line 4 in qt_lib_qmldevtools_private.pri to:
QT.qmldevtools_private.libs = $$QT_MODULE_HOST_LIB_BASE/../../debug/lib
with the following result:
G:\qt\vcpkg>vcpkg install qt5-tools
The following packages will be built and installed:
    qt5-tools[core]:x86-windows
Starting package 1/1: qt5-tools:x86-windows
Building package qt5-tools[core]:x86-windows...
-- Using cached G:/qt/vcpkg/downloads/qttools-everywhere-src-5.12.3.tar.xz
-- Using source at G:/qt/vcpkg/buildtrees/qt5-tools/src/5.12.3-2e47618cfc
-- Configuring x86-windows-rel
-- Configuring x86-windows-rel done
-- Configuring x86-windows-dbg
-- Configuring x86-windows-dbg done
-- Package makefiles-x86-windows-dbg
-- Package build-x86-windows-dbg
-- Package makefiles-x86-windows-rel
-- Package build-x86-windows-rel
-- Package install-x86-windows-dbg
-- Package install-x86-windows-rel
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/bin/Qt5Designer.dll
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/bin/Qt5DesignerComponents.dll
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/bin/Qt5Help.dll
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/debug/bin/Qt5DesignerComponentsd.dll
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/debug/bin/Qt5Designerd.dll
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/debug/bin/Qt5Helpd.dll
-- Installing: G:/qt/vcpkg/packages/qt5-tools_x86-windows/share/qt5-tools/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package qt5-tools[core]:x86-windows... done
Installing package qt5-tools[core]:x86-windows...
Installing package qt5-tools[core]:x86-windows... done
Elapsed time for package qt5-tools:x86-windows: 7.724 min

Total elapsed time: 7.724 min

The package qt5-tools:x86-windows provides CMake targets:

    find_package(Qt5Designer CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Qt5::Designer Qt5::DesignerPrivate Qt5::QQuickWidgetPlugin)

    find_package(Qt5Help CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Qt5::Help Qt5::HelpPrivate)

    find_package(Qt5UiPlugin CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Qt5::UiPlugin Qt5::UiPluginPrivate)

    find_package(Qt5UiTools CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Qt5::UiTools Qt5::UiToolsPrivate)

I also had merged in latest vpckg master and pr #6259

could be that it should be QT_MODULE_LIB_BASE instead of QT_MODULE_HOST_LIB_BASE

yeah works also with QT_MODULE_LIB_BASE. I don't know what the difference is since I cannot tell what values are set. The only thing I found was:
        QT_MODULE_LIB_BASE = $$[QT_INSTALL_LIBS]
        QT_MODULE_HOST_LIB_BASE = $$[QT_HOST_LIBS]

so there are two questions:
Why does QT.qmldevtools_private.libs = $$QT_MODULE_HOST_LIB_BASE/../../debug/lib even solve the problem? (QT_MODULE_HOST_LIB_BASE will point to the debug directory in the makefile meaning: somepath/debug/lib/../../debug/lib/ but will not do so without the path additions) This is most likely a bug in qmake and wrong chaching of variables or something alike.
should we simply set QT_MODULE_HOST_LIB_BASE to QT_MODULE_LIB_BASE in qt_lib_qmldevtools_private.pri? Probably yes after the installation of qt5-base or declarative? (Who creates the file?)

will you make a pr to my branch like last time?

@TheScarfix made a pr. You must test the rest. Compiling qt with nmake always takes forever... (jom stalls on my pc for some unidentified reason)

If you resolve the merge conflicts with latest master then the CI system should pick it up.

@Rastaban: Cannot reproduce x86-windows regressions. Will try x64-windows after full build of qt on x86.

With the latest merge the failures have gone away.  To me this PR looks good, is there anything else that needs to be done before we merge?

I think it's good now :)


[stable-4.4] cypress.yml - backport changes from master (#2480)

Backports cypress.yml changes from #2480, #2478, non-signing parts of #2240, #1979, #2233, #1827, #1558, #1562, #1323
The remaining difference between 4.4 and master cypress.yml are..

galaxykit 0.7.0 vs main
no signing in 4.4 => most of #2240
build from ghcr.io/pulp/pulp-ci-centos version 3.15 (not latest)
galaxykit queries namespaces not collections (also #2265)

Cc @ironfroggy - this ensures 4.4 tests on github run against current 4.4 with pulpcore 3.15, instead of June 4.4 with pulpcore 3.18


Adding content to certificate rotation sections in v2.5 and v2.6

Fixes #3960
Content in the v2.0-2.4 docs has been copied as-is to v2.5 and 2.6.
Since this is not novel content, pushing it to master branch.

@catherineluse / @watarumohawk : Please can you sign off so that I can merge these changes?

Merging the pull request, since requested changes have been addressed.


Truck Volume Percentages

Below is a description of how truck volumes are estimated in PeMS. The methodology has it's limitations and drawbacks as described below. @kengodleskidot is currently evaluating Weigh-In-Motion (WIM) data that the Caltrans Census group collects and that is reported to FHWA. The preferred option is to incorporate the WIM data in a way where truck volume percentages can be determined and applied to 5-minute aggregate data. We will have further discussions regarding this topic in the future and this issue will be expanded on in subsequent posts.
PeMS Existing Estimation of Truck Volumes:
For detectors that don't report truck volumes (currently none of the VDS stations or associated detectors report truck volume), the PeMS system has the ability to estimate them at each detector based on the measured 5-minute, lane-by-lane values of flow and occupancy. If there are sensors that do report truck volumes then we can use those directly. This is a configuration option that is configured during the initial installation.
The initial work on this was in the paper, "Estimation of truck traffic volume from single loop detectors using lane-to-lane speed correlation," by J. Kwon, P. Varaiya, A. Skabardonis (2002), which is available for download in our Resources section.
The algorithm attempts to break down the total flow into passenger cars and large trucks. It does not attempt to identify the proportion of vehicles in each of the 13 classes defined by the FHWA in the United States - that requires detailed measurements that are typically not collected with standard detectors (we assume that since the detectors can't report truck volumes that they are relatively simple sensors like single-loop detectors). The algorithm starts with the following assumptions:
There are no heavy trucks on the inner lanes.
For multi-lane freeways, the vehicle speeds over different lanes are synchronized.
The traffic volume consists mostly of short passenger cars and long trucks.
The average length of passenger cars is 16 feet and the average length of trucks is 60 feet.
With these assumptions, the algorithm estimates the proportion of trucks in each lane, starting with the first lane (where we know by assumption that the proportion of trucks is zero), and working to the outer lanes. We store the proportion of trucks in each lane, but we usually plot it only as an aggregate across all lanes since individual per-lane estimates tend to be noisy.
Warning: It is very important to understand that the truck volume estimates from the PeMS algorithm are not measured but estimated from 5-minute aggregated volume and occupancy data. The algorithm works best when the assumptions listed above are satisfied by the traffic at the given location. There are a number of cases when these assumptions can be violated and thus the estimates are unreliable. The two main cases are:
When there is a configuration error and the lane numbers are switched, especially when lane 1 is switched with one of outer lanes, the first assumption is clearly violated. In this case, the algorithm will incorrectly think that an outer lane is actually lane 1 and assign zero trucks to that lane. Further, since the algorithm hinges on the ratios of occupancy in lane 1 and outer lanes, results for other lanes also will be off.
When there is a huge speed difference among lanes, the second assumption is violated. Such speed difference occurs especially when there is a lot of merging/diverging, i.e. near on- and off-ramps, or when traffic speed is very low. In these cases, the algorithm will perform poorly since it requires that the speeds in the adjacent lanes be correlated.
It is not recommended to look at the truck volume estimate from a single location over a single day. A better way to use the PeMS truck volume estimates is to look at the outputs over many locations, preferably along the corridor in question over multiple days - say over 5 weekdays. The estimates may be unreliable for some of those locations on some of those days, but the empirical studies suggested that the overall trend is captured by the average over locations and times. Specifically, we recommend that the user computes the ratio (daily truck volume to the daily total volume) over each day and location and take their median as the final estimate of the ratio of truck traffic in the corridor. The PeMS displays allow users to view the ratio of truck traffic down the freeway as an easy way to get the estimate.

Ken to look into WIM data, daily variance on traffic that is trucks. Mintu will keep the item backlogged until the issue is prioritized.

Meeting is set for June 5th.

Data gathering phase has begun - access to historical time period data to be provided to Ken and Mintu. Provided data needs to be reviewed and analyzed to create scope of effort. WIM team will be updated once this has been completed.

May 5, 2024 Meeting resolutions with Census group:
Action Item:
Stanley will help to get FHWA account to download WIM class historical data
Ken will contact with the corresponding person of TSN to check if they have already processed data
Mintu will schedule follow up meeting with Stanley and Vicky to discuss imputation progress and outcomes.
Leanings:
Consider Hour of day (HOD), Day of week (DOW) and Season of the Year (SOY) variation to develop the factors (24 (hours) X 7 (days) X 4 season)= total 672 factors)
Define Truck before calculating the factors (may be >class4)
Consider lane closure impacts in imputation process
WIM data quality is pretty good as it is prepared good data only for FHWA submission
Truck percentage does not really change over time, we may apply the factors couple of longitudinal direction easily.

Per Ken this will likely not happen during the project timeline, will likely come later.

Per Mintu there is no update on this as of 7/18 so keeping in the backlog.


Disable dead stripping

With dead stripping, some optimized targets would fail to start fuzzing with the following error:
ERROR: The size of coverage PC tables does not match the
number of instrumented PCs. This might be a compiler bug,
please contact the libFuzzer developers.
Also check https://bugs.llvm.org/show_bug.cgi?id=34636
for possible workarounds (tl;dr: don't use the old GNU ld)

If you'd like to reproduce the issue, clone https://github.com/cloudflare/quiche and run cargo fuzz run packet_recv_client --release from the fuzz directory. You should see the above error.
This PR disables dead stripping, which removes the above error. Alternatively, one could use ld.gold by adding -Clink-arg=-fuse-ld=gold to RUSTFLAGS. This would also fix the error. However, I did not want to assume that ld.gold was present on users' systems. Others have also successfully switched to disabling dead stripping, like OSS-Fuzz, so it seemed like a better solution.
Environment
cargo-fuzz 0.5.4
cargo 1.41.0-nightly (8280633db 2019-11-11)
rustc 1.41.0-nightly (3e525e3f6 2019-11-18)
Debian bullseye container running on Mac OS 10.15.1 and Docker Desktop <IP_ADDRESS>

The build is currently broken due to https://github.com/rust-lang/rust/issues/66140. The same failure is on master as well.
I assume that the pipeline will be fixed once https://github.com/rust-lang/rust/pull/65241 makes it to the nightly build. Once that's the case, we should try re-running the pipeline on this PR to make sure it's passing.

We can manually merge this, this seems harmless enough and fixes an important issue.


GPUImage 如何实现加上滤镜的HDR效果

by reference to Apple's demo, achieved using the GPUImage HDR effect, due to the project using a saturation of the filter, in the regulation of the saturation parameter and save the HDR images without filter effects, I should how to obtain the video frame after the combination of filters, and then do processing

I don't understand what you're asking. You obtain frames as you do after any other kind of filtering in the framework. There's nothing special about saturation adjustment.


🐞 - Bug in InputDateTime component when trying to overwrite it

Which @taiga-ui/* package(s) are the source of the bug?
cdk
Please provide a link to a minimal reproduction of the bug
https://taiga-ui.dev/components/input-date-time/API
Is this issue blocking you?
Blocking
Description
There is a bug in the InputDateTime component.
Reproducing
Type "01.11.2023, 11:11" and then press "Pos1" button it is not possible to type a new number. Same problem with "
10.11.2023, 23:15" and "05.02.1982, 02:54".
Following date works "12.12.1212, 12:12"
Therefore, it is not possible to use the component as an user correctly
Angular version
14.0
Taiga UI version
3.15.0
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

Looks like a mask issue, completely new mask is under development right now, it would most likely solve this.


Quorum logs in JSON

Juraj took a look at https://geth.ethereum.org/docs/interface/command-line-options and found an option to set log format to JSON. Would you please add the --log.json flag to the Helm Chart  (from MSD/Cheryl)


Drop PIL (and use e.g. Pillow instead)

To cite someone on stackoverflow:

Use Pillow instead, as PIL is basically dead. Pillow is a maintained fork of PIL.

Pillow could indeed be an alternative, maybe with none to minial effort:

https://pypi.python.org/pypi/Pillow/2.2.1 => https://pypi.python.org/pypi/Pillow/3.0.0
https://github.com/python-pillow/Pillow

I know, we can do all(?) the image stuff with ImageMagick, but to install it on windows is was a pain, and since PIL is supported at the moment...
Background: I'm collecting the dependencies for all the beets plugins I use in an requirements file and PIL throws errors since its dead...

Yes, sounds good. IIRC, Pillow is a drop-in replacement—i.e., it gets imported under the name PIL—so we just need to update the docs and metadata. Any chance you could volunteer to update that?

Cool! Yes, i think i can do this.
Should I mention the original PIL in the docs or just replace PIL by Pillow?
Also, since you're mentioning metadata, i would have thought of https://github.com/sampsyo/beets/search?l=rst&q=PIL&utf8=✓, which files more?
Mention Pillow in python comments?

Yeah, let's just say Pillow; no need to also call out the original.
I was thinking of metadata in setup.py, but it looks like it's not listed there—so no problem.


Allow creating entities containing StateMachineStateFields via the REST API

Description
It is currently not possible to create an instance of any entity that contains a field of type StateMachineStateField (e.g. Order::stateId) using the admin REST API. Attempts to do so result in an error:
{
  "errors": [
    {
      "code": "FRAMEWORK__WRITE_CONSTRAINT_VIOLATION",
      "detail": "This field is write-protected. (Got: \"user\" scope and \"system\" is required)",
      "meta": {
        "parameters": [
          "user",
          "system"
        ]
      },
      "source": {
        "pointer": "/0/stateIdstateId"
      },
      "status": "400",
      "template": "This field is write-protected. (Got: \"%s\" scope and \"%s\" is required)"
    }
  ]
}

It should be possible though to create any entity via the REST API, no matter what fields they have.
Possible Solution
There are a few solutions for this:

Define a default state for each entity which is applied by the REST API while not allowing to set a set.
Disable the write validation for REST API writes.
Preferred solution based on 1.: Use the StateMachineRegistry to validate and apply the state passed to the REST API after the entity itself was created to ensure the correct transition events are fired etc.

Additional context
There is already an issue for this regarding the order creation: https://issues.shopware.com/issues/NEXT-7800

Hello,
thank you for creating this issue.
I have opened a ticket on our Issue Tracker for you. See the ticket link: https://issues.shopware.com/issues/NEXT-10842
Please use this ticket to track the state of your issue.


Add ZiMengSheng into approvers alias

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


Add Subscribe button for subscriber only items in item store.

Fix for #4421

Coverage remained the same at 57.433% when pulling 6ea6b4fb0914d4be156c56a157bb6dd2053cad48 on item-store-subscribe into 6b99a216f855f69fcfaf846ff399473f72fe2d1f on master.

Reviewed and tested, added a tweak and a fix from testing. Looks good to go!

Coverage remained the same at 57.433% when pulling dcd44fedbf11a51f5fc71ffdd8786259f46dc3b4 on item-store-subscribe into 6b99a216f855f69fcfaf846ff399473f72fe2d1f on master.


doc: consistent comment style

nitpicked comment style a few places

Sorry, as mentioned on the other issue we don't do standalone style fixup commits.


C-x 3 should probably be mentioned

To get a better display in emacs, you can do the following
Load your .agda file and do C-c C-l.
Type C-x 1 to get only your agda file showing, then type C-x 3 to split the window horizontally.
Now move to the right-hand half of your frame and switch to the buffer called "Agda information" by typing C-x b Agda TAB.
Now your errors, contexts, etc. will appear next to your file, rather than squashed beneath it.
I had seen somebody with this style of display, but had to ask somebody how to do it.   I couldn't find a way of doing this in Customize agda2-mode.
This is probably worth mentioning somewhere.
Sorry for the flurry of issues; Phil said this was the best way of giving you folks feedback.

Thanks! I've added this text to the "About" page.

Thanks! I've added this text to the "About" page. Cheers, -- P
.   \ Philip Wadler, Professor of Theoretical Computer Science,
.   /\ School of Informatics, University of Edinburgh
.  /  \ and Senior Research Fellow, IOHK
. http://homepages.inf.ed.ac.uk/wadler/
Too brief? Here's why: http://www.emailcharter.org/
On Sun, 30 Sep 2018 at 20:41, Mitchell Wand<EMAIL_ADDRESS>wrote:

To get a better display in emacs, you can do the following
Load your .agda file and do C-c C-l.
Type C-x 1 to get only your agda file showing, then type C-x 3 to split
the window horizontally.
Now move to the right-hand half of your frame and switch to the buffer
called "Agda information" by typing C-x b Agda TAB.
Now your errors, contexts, etc. will appear next to your file, rather
than squashed beneath it.
I had seen somebody with this style of display, but had to ask somebody
how to do it. I couldn't find a way of doing this in Customize agda2-mode.
This is probably worth mentioning somewhere.
Sorry for the flurry of issues; Phil said this was the best way of giving
you folks feedback.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/plfa/plfa.github.io/issues/72, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEqfsianTeARCID9d-mERHL2a-wDJUheks5ugR5ZgaJpZM4XBHwG
.

The University of Edinburgh is a charitable body, registered in
Scotland, with registration number SC005336.


[BACKLOG-35399] - Update projects to include locale properties for integration test execution

@smmribeiro
@andreramos89
@renato-s

:white_check_mark: Build finished in 36m 45s
Build command:
mvn clean verify -B -e -Daudit -amd

:exclamation: No tests found!
:information_source:  This is an automatic message


Multiple processes for NetMHC and NetMHCpan

From @iskandr, we should check this:

I think that NetMHC and NetMHCpan don't allow you to specify a distinct temp directory for each process, which is why I didn't use multiple processes for them. I need to check if that's actually true.

And based on that, decide if we can also split NetMHC and NetMHCpan runs into multiple processes.

From @iskandr:

@tavinathanson I checked the commandline flags and both NetMHCpan and NetMHCcons allow you to specify the tempdir (NetMHCcons commandline flags). I'm suspicious that internally it only passes that flag to -tdir flag to NetMHCpan and not NetMHC (which would cause weird bugs when running multiple predictors concurrently). Let's remember this thread if NetMHCcons causes us trouble later?

@iskandr I guess I misunderstood your concern. So you're talking about NetMHCcons's usage of NetMHC and NetMHCpan? But you had cons split into multiple processes already; does my PR #12 (which just adds flexibility to that) introduce a new concern?

Sorry, I misunderstood the PR. I think it's fine.

Hey @tavinathanson, ideally I want to run topiary with 8 alleles * 5 peptide lengths and mhc-cons which can spin out 100+ processes (and that's too many for my system).  Does this addition to mhctools allow me to set an upper limit on mhc-cons children that will run at once?  If so, how do I implement setting such a limit?

Hey @kippakers, yup! Does setting process_limit in the NetMHCcons constructor work for you? Looks like I should add this to the README and add some better comments!

@tavinathanson Yeah, OK that did it, thanks for your help!

@tavinathanson I think that with my refactoring, all of the predictors that can run in parallel do-- should I close this issue?

Agreed, closing it out


Work on README clarity based on feedback

As I said in a different commented, though not so explicitly, if this is for people who basically know what they're doing as far as git/github/node goes, we can simplify these instructions a ton.

I don't know how to un-hide supposedly 'outdated' review comments that are not actually outdated.

@lianilychee: Shall I merge this to get most of the setup instructions cleaned up and we can work on the paragraphs below later?


floppy_files never set in Veewee::Definition.initialize from */definition.rb

Looks like @floppyfiles is set to nil at https://github.com/hh/veewee/blob/feature/windows/lib/veewee/definition.rb#L58 and then never set to what's defined at https://github.com/hh/veewee/blob/feature/windows/templates/windows-2008R1-serverweb-amd64/definition.rb#L47
module Veewee
  class Definition
    ...
    attr_accessor :floppy_files
    ...
    def initialize(name,path,env)
      ...
      # Default is no floppy mounted, and we never set it again 8(
      @floppy_files = nil
      ...
   end
end

You can test by using the following in your Gemfile:
gem 'vagrant', :git => "git://github.com/hh/vagrant.git", :branch => "feature/winrm"
gem 'veewee', :git => "git://github.com/hh/veewee.git", :branch => "feature/winrm"

And running the following commands:
bundle install && bundle exec

bundle exec veewee vbox build windows-2008R2-serverweb-amd64
Downloading vbox guest additions iso v 4.1.8 - http://download.virtualbox.org/virtualbox/4.1.8/VBoxGuestAdditions_4.1.8.iso
Checking if isofile VBoxGuestAdditions_4.1.8.iso already exists.
Full path: /home/chris/chef/winrmtest/iso/VBoxGuestAdditions_4.1.8.iso

The isofile VBoxGuestAdditions_4.1.8.iso already exists.
/home/chris/chef/veewee/lib/veewee/provider/virtualbox/box/build.rb:9:in `build': undefined method `floppy_files' for nil:NilClass (NoMethodError)
	from /home/chris/chef/veewee/lib/veewee/command/virtualbox.rb:17:in `build'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor/task.rb:22:in `run'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor/invocation.rb:118:in `invoke_task'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor.rb:263:in `dispatch'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor/invocation.rb:109:in `invoke'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor.rb:205:in `block in subcommand'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor/task.rb:22:in `run'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor/invocation.rb:118:in `invoke_task'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor.rb:263:in `dispatch'
	from /home/chris/chef/winrmtest/.bundle/vendor/gems/thor-0.14.6/lib/thor/base.rb:389:in `start'
	from /home/chris/chef/veewee/bin/veewee:18:in `<top (required)>'
	from /home/chris/chef/winrmtest/.bundle/vendor/bin/veewee:19:in `load'
	from /home/chris/chef/winrmtest/.bundle/vendor/bin/veewee:19:in `<main>'

I thought you could use templates directly without copying them to an immediate definitions/ dir within your project. My bad.


[New Feature]: Blue / Green Deployment for PCM

Checked for duplicates
Yes - I've already checked
Alternatives considered
Yes - and alternatives don't suffice
Related problems
Currently, the PCM deployment procedure requires our system to be offline during a major PCM upgrade. This downtime amount will only increase in duration as more complex and longer-running PGEs are active in our system - since we'd want to wait for processing to complete before an upgrade commences.
Describe the feature request
We should design, ascertain the cost, and eventually implement a "blue / green" deployment system or procedure that ensure our PCM has zero downtime during upgrades.

Operational Needs
Assuming "blue" is the legacy system, and "green" is the new system:

(Required) Zero operational downtime between "blue" wind-down and "green" wind-up
(Required) Ability to revert to using "blue" should "green" be problematic
(Required) Maintaining interface agreements with external systems such that the data flow in or out with respect to "blue" or "green" is maintained
(Desired) Operational state is maintained between "blue" and "green"

OPERA SDS doesn't need to keep the current states of the processes. So we don't need to perform blue/green deployment when upgrading to newer version.


Emit a warning when data sources have duplicate functionality

Includes an additional check during subgraph validation to identify data sources which are functionally equivalent.
If a subgraph is deployed with functionally equivalent data sources it will likely lead to unexpected entity data such as double counting.

This is currently setup to emit a warning, but I'm now thinking we should fail the subgraph on this condition.  Thoughts?


Cached Users not updated in feed

If User A retrieves the feed and user B changes the name. User A will still see User Bs name in the feed.

@onurakpolat
might be fixed with this commit > needs testing  https://github.com/tapglue/ios_sdk/commit/b087f776357d683b52938712e56ca8d5655813ee


returning raw response for non-JSON Content-Type

A number of Gerrit's APIs return (or may return) response encoded as text or
HTTP 204 No Content
With this change, gerrit2 looks at response's content-type header and skips
JSON parsing for responses other than application/json

Thanks!


How to Uninstall?

I'm trying to remove the 3 extensions this script installs: Privacy Badger, uMatrix, and the Cookie Remover extensions. They are redundant for me and I don't need them. However even after manually deleting the extension files from the extensions folder, they reinstall. How can I prevent this from happening?

I'm trying to remove the 3 extensions this script installs: Privacy Badger, uMatrix, and the Cookie Remover extensions. They are redundant for me and I don't need them. However even after manually deleting the extension files from the extensions folder, they reinstall. How can I prevent this from happening?

Go to your firefox configuration folder
and remove the following file https://github.com/simeononsecurity/FireFox-Privacy-Script/blob/master/Files/distribution/policies.json to remove the extensions that auto install.

Thanks for the help. Does this extension edit any settings with WebAssembly or WebGL? I've noticed my settings being undid as soon as I restart Firefox.

Thanks for the help. Does this script edit any settings with WebAssembly or WebGL? I've noticed my settings being undid as soon as I restart Firefox.

The vast majority of configurations are in https://github.com/simeononsecurity/FireFox-Privacy-Script/blob/master/Files/mozilla.cfg
Please take care to review scripts before running random scripts online.
If you skimmed or searched, you would've been able to determine this yourself ;)
https://github.com/simeononsecurity/FireFox-Privacy-Script/blob/3b9e545e6353542fd313cd2de9af21ca1bb01fe9/Files/mozilla.cfg#L204

Fair points, I was just recommended this and I had no clue what I was doing. With enough digging I fixed everything though. Sorry for being an annoyance

I'm on osx 14.6.1 and I've trying to do everything to remove these settlings as it was was causing more issue with some extension. I tried everything to remove the settings/folders/etc but the extension from this still install on a clean install.

Go into your mac and find the files the script installed in
    preferences_dir="/Applications/Firefox.app/Contents/Resources/defaults/pref/"
    extensions_dir="/Applications/Firefox.app//Contents/Resources/browser/features/"
    config_dir="/Applications/Firefox.app/Contents/Resources/"

    checkInstall

    rm /Library/Preferences/org.mozilla.firefox.plist
    defaults import /Library/Preferences/org.mozilla.firefox ./Files/distribution/org.mozilla.firefox.plist


Adapt the data analysis code to pick up manual segmentations if they are present in the dataset

Currently the jupyter notebook containing the analysis code downloads the datasets from OSF and proceeds to segment the spinal cord using MP2RAGE data. For some subjects this segmentation is not ideal and a manual segmentation is needed (see #3 ). Once #3 has been addressed, the OSF repo should contain manually segmentation files. The code should be adapted so that manual segmentation files are searched for, and if found, no further segmentation using MP2RAGE data is attempted.

See resolution to https://github.com/spinal-cord-7t/coil-qc-code/issues/3


How do I remove sentinel from workspace - This workspace is locked.

I get below message while trying to remove the workspace.
This workspace is locked. You might not be able to make certain changes.

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

ID: d272cb77-30a3-15c5-eb77-c15e5b9937ed
Version Independent ID: a1a18221-fe5a-aed8-3908-cbce1aa9502d
Content: Remove Azure Sentinel
Content Source: articles/sentinel/offboard.md
Service: azure-sentinel
Sub-service: azure-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@codemaker Any resource which has been locked by the owner or user access administrator does not allow either Edit or delete option when set accordingly.
If the workspace is locked, you must delete the lock first, if you are the owner, you can follow step 4 on this article to delete it :
https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/lock-resources?tabs=json#configure-locks
If not, please contact the owner for that workspace, you can help you remove the lock and then you can delete it.


Not working on Oreo and above

+1

Fixed in 1.2.3.
Note: I've changed the dependency name from com.akexorcist:localizationactivity to com.akexorcist.localization. Please see more info the readme.


MessageCollection items with empty key

Describe the bug
Hi, I have a little problem when I try to get messages wihout Message-ID header.
When I make a query and I apply the get function on mails without Message-ID header, the get method returns a collection with an unique result but the total has an another value.
In the items array, I have an empty key.
To Reproduce
Steps to reproduce the behavior:

Send emails on a box without Message-ID header

$client = new Client([
            'host' => "",
            'port' =>  "",
            'encryption' => "SSL",
            'validate_cert' => true,
            'username' => "",
            'password' => "",
            'protocol' => "imap",
        ]);

$messages = $client->getFolder('INBOX')->from('email@example.com')->get();

dd($messages);

Expected behavior
If no Message-ID is presents in headers add an incremetal key or a random key to preserve this messages.
Desktop / Server (please complete the following information):

OS: linux
PHP: v7.0.33
Version: 1.6.2

Hi @Gmacko ,
you can change the used key in your config/imap.php config file.
Just change imap.options.message_key from id to number or list :)
Best regards,

Thanks for your reply.


Port ember warning about local cluster members

The warning about adding users to local clusters does not exist in vue. We should update it to say:

Caution: This is the cluster that Rancher is using as a data store. Only administrators should be given write access to this cluster. Users with write access to this cluster can use it to grant themselves access to any part of this installation.

Ember page

Vue page

Test Environment:
Rancher version: v2.6-head 6e7308f
Rancher cluster type: HA
Docker version: 20.10

Testing:
Tested this issue with the following steps:

Go to local cluster
Go to Cluster Members under Cluster
Verify the Caution message is there

Results
The message is now present, but the message has not been updated as mentioned in the description of the ticket @codyrancher could you take a look?

Test Environment:
Rancher version: v2.6-head 2c21373
Rancher cluster type: HA
Docker version: 20.10

Testing:
Retested this issue with the following steps:

Go to local cluster
Go to Cluster Members under Cluster
Verify the Caution message is there

Results
The message has been updated correctly.

How do I get rid of this message? Is this some bug on the UI? Is there a workaround?

@tbernacchi the message is intended and will always show when adding members to the local cluster.


ERR_MODULE__NOT_FOUND importing unplugin as related dependency

Environment
Node version 20.12.2
npm 10.8.3
Reproduction
I can't, we run in the problem since some hours and we tried to understand what was before finding out that a new release of the module has been performed.
Maybe because the libraries that use it still bind the version 1.13.0, but actually we can't work anymore.
Describe the bug
After installing the libraries, once npm run dev is executed, after some seconds the following error is shown:
failed to load config from /home/luca/Projects/vue-template/frontend-base-lib/vite.config.mts
error when starting dev server:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'webpack-sources' imported from /[project_path]/node_modules/unplugin/dist/index.mjs
    at packageResolve (node:internal/modules/esm/resolve:854:9)
    at moduleResolve (node:internal/modules/esm/resolve:927:18)
    at defaultResolve (node:internal/modules/esm/resolve:1157:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:390:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:359:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:234:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:87:39)
    at link (node:internal/modules/esm/module_job:86:36)

Additional context
We use the following libraries:
...
  "peerDependencies": {
    "@intlify/unplugin-vue-i18n": "4.x.x",
    "@vueuse/core": "10.x.x",
    "axios": "1.x.x",
    "pinia": "2.x.x",
    "primevue": "4.x.x",
    "vue": "3.x.x",
    "vue-i18n": "9.x.x",
    "vue-router": "^4.x.x"
  },
  "devDependencies": {
    "@types/node": "20.x.x",
    "@vitejs/plugin-basic-ssl": "1.x.x",
    "@vitejs/plugin-vue": "5.x.x",
    "autoprefixer": "10.x.x",
    "postcss": "8.x.x",
    "tailwindcss": "3.x.x",
    "typescript": "5.x.x",
    "vite": "5.x.x",
    "vite-plugin-css-injected-by-js": "3.x.x",
    "vite-plugin-dts": "3.x.x",
    "vue-tsc": "2.x.x"
  },
  "dependencies": {
    "quill": "^2.0.2",
    "unplugin": "1.x.x" <-- tried to check if the new version is used, but it gets always the 1.13.0
  }

Logs
No response

Fairly, I tried to create a really minimal project with anything execpt some code to run a vue+vite app, but the problem doesn't appear.  But, the version linked is the 1.13.1.
That means, maybe the problem is the version 1.13.0.
I wait tomorrow, if nothing is still working then I try to create a better project for testing it out.


Focus in card when clicked

The textbox should be focusable when the user clicks inside a card

i'm doing this with JS....


Deep learning

Need to try deep learning approaches to classify classes with few observations more accurately.
Models to explore:

CNN,
LSTM,
bidirectional LSTM.

Done


Failed test on windows

Test fail on windows because of the slashing are "" instead of "/"
141 passing (2s)
5 failing

External Formatter formats failures:
actual expected
[1, 1]formatters/\externalFormatter.test.ts
[2, 11]formatters/\externalFormatter.test.ts
[9, 2]formatters/\externalFormatter.test.ts

Fixed by 10234a59125d12e91b86f828cf660e7a4ce98cd4?

This should be fixed by #291


release: v0.15.0

See CHANGELOG

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

Rebased on top of the build fix.


browserWindowOverrides option has no effect on the created preference window

Describe the bug
Trying to change the preference window by setting the browserWindowOverrides option in constructor, but no effect
To Reproduce
Steps to reproduce the behavior:

create preference window like this:
const preferences = new ElectronPreferences({ ... browserWindowOverrides: {  title: 'My title', },  ... });

Expected behavior
the preferenec window should have the specified title.
Screenshots

Machine (please complete the following information):

OS: Windows 10
Electron version: 21.1.0
electron-preferences version: 2.8.1

Additional context
Add any other context we should know

Hi @shiyeling. Thank you for reporting this bug.
Seems to be an issue due to changes in electron which update the browserWindow title to the HTML header title when loading a page.
I'll create a fix and try to release a patch in the near future.

Has been released in 2.8.2. Sorry for the delay


Engine Nacelle Implementation

Issues that still need to be addressed:

blending surfaces are only G0
core cowl is not implemented yet, only fan and center cowl
The current definition for the nacelle guide curves (specifically the x-component) should be revised

Great...! Let's hope we can soon leave this behind us. Let's create some issues from the remaining todos.

Yes, that makes sense. I will open those issues as soon as this is reviewed and merged.

@joergbrech I think you used an old version of the generator! Can you please update you submodule to the linked one and rebuild the generated files?

@joergbrech Your cpacs_gen code is still at  r34f55be ?

yes, I only committed the updated generated code, not the submodule.


Feature Request: Export Templates/Profiles

Debug log ID
none
What happened?
I'm pretty new to this utility, but already I've encountered something that I think many others might appreciate.
I have a bib export for my Obsidian use. And I have a new one for my manuscript.
For my manuscript, I don't need that as many bib fields (don't need the urldate date accessed, etc.), but for Obsidian I like more details.
It would be handy to be able to create different exporting templates/profiles to fit different contexts.

Why is the Feature Request combined with bug reports??

Because years of doing this has shown it to be effective for my workflow.

Do you have any idea how bizarre it is being asked to justify how I provide my free support for my free software?
https://retorque.re/zotero-better-bibtex/installation/preferences/hidden-preferences/index.html#preferencesoverride


Fixed type error in 'brew audit'

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Resolves:
$ brew audit --eval-all                                                                                [master|⚑6]
vde:
  * fully scope test system calls, e.g. system "#{bin}/vde_switch"
Error: Return value: Expected type String, got type NilClass
Caller: /opt/homebrew/Library/Homebrew/cask/auditor.rb:68
Definition: /opt/homebrew/Library/Homebrew/cask/audit.rb:123
Please report this issue:
  https://docs.brew.sh/Troubleshooting
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/sorbet-runtime-0.5.10461/lib/types/configuration.rb:296:in `call_validation_error_handler_default'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/sorbet-runtime-0.5.10461/lib/types/configuration.rb:303:in `call_validation_error_handler'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/sorbet-runtime-0.5.10461/lib/types/private/methods/call_validation.rb:187:in `report_error'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/sorbet-runtime-0.5.10461/lib/types/private/methods/call_validation.rb:166:in `validate_call'
/opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/sorbet-runtime-0.5.10461/lib/types/private/methods/_methods.rb:270:in `block in _on_method_added'
/opt/homebrew/Library/Homebrew/cask/auditor.rb:68:in `audit'
/opt/homebrew/Library/Homebrew/cask/auditor.rb:12:in `audit'
/opt/homebrew/Library/Homebrew/cask/cmd/audit.rb:115:in `block in audit_casks'
/opt/homebrew/Library/Homebrew/cask/cmd/audit.rb:113:in `to_h'
/opt/homebrew/Library/Homebrew/cask/cmd/audit.rb:113:in `audit_casks'
/opt/homebrew/Library/Homebrew/dev-cmd/audit.rb:239:in `audit'
/opt/homebrew/Library/Homebrew/brew.rb:93:in `<main>'

(I made the other necessary changes in oder to enable typed: true, which I consider to be a form of writing new tests, so I'm checking that box 😄)

Review period will end on 2023-02-28 at 00:00:00 UTC.

Review period skipped due to critical label.

Thanks again @dduugg!


Complaint Management & Notifications

*1. Complaint Submission
a. Create models for complaints (fields: VIN, description, photos, etc.).
b. Implement submission forms with validation for mandatory fields.
*2. Tracking and Status Management
a. Add functionality to update complaint statuses.
*3. Attachments Management
a. Implement file upload and association with complaints.
*4. Notification System
a. Automate emails for complaint submission confirmations (to customers)
and new assignments (to agents).
When completed: Complete complaint submission and notification system.


don't ignore relative phase of sub_gate in QubitSubspaceGate

currently equal_up_to_global_phase checks if the sub_gate s of two QubitSubspaceGates are equal *up to global phase*, but this isn't correct because the global phase of the sub gates becomes physical when expanded to higher dimensions
this pr changes QubitSubspaceGate._equal_up_to_global_phase_ to instead use the absolute comparison of the two sub gates

I don't quite understand the physics

basically QubitSubspaceGate just embeds a qubit gate in a given subspace of higher-dimensional qudit operation, while performing an identity on the new energy levels. Which means for example the gate
css.QubitSubspaceGate(cirq.X, (3,))

has the unitary
[[0.+0.j 1.+0.j 0.+0.j]
 [1.+0.j 0.+0.j 0.+0.j]
 [0.+0.j 0.+0.j 1.+0.j]],

while
css.QubitSubspaceGate(cirq.rx(np.pi), (3,))

has the unitary
[[0.+0.j 0.-1.j 0.+0.j]
 [0.-1.j 0.+0.j 0.+0.j]
 [0.+0.j 0.+0.j 1.+0.j]].

So the different global phases of cirq.X and cirq.rx(pi) become different physical phases of the (0, 1) and (1, 0) terms relative to the (2, 2) term

Ah I see. Makes sense. Thanks!

(will merge after server-side changes have been merged)


Pioneer vsx-831

Hi @TG908
Newby here.
Could you help please. I can’t seem to get the plugin working on my vsx-831
I keep on getting connection refused for you have any idea what I’m missing ?

Hey @mark125431,
I no longer have access to any Pioneer device so I cannot help you with that.
I'm sorry. If you want to give it a shot yourself a good starting point would be to reverse engineer the iControlAV5 App. Try capturing the network traffic between the device and the app or decompile the Android app.

Hi Tim,
Not sure where to start really,  i'm quite new to all this.


Dependency Check 5.0.0-M1 - Autoupdate is disabled and the database does not exist

Describe the bug
Using Dependency Check 5.0.0-M1 make the build fail.
Issue with: org.owasp:dependency-check-maven:5.0.0-M1
Works with: org.owasp:dependency-check-maven:4.0.2
Log file
See the attachment please.
To Reproduce
I've launched the job from Jenkins with dependency-check:check.
Autoupdate is false and we have a local DB of the last NVID.
Trying with the older version makes the build have success.
Expected behavior
It should works like the v4.0.2 without given the error "database does not exist".
Log.txt

I believe this has been fixed in 5.0.0-M2. Please retest this with 5.0.0-M2.

Hi Jeremy,
now, using 5.0.0-M2, I've this issue:
Error resolving version for plugin 'dependency-check-maven:5.0.0-M2' from the repositories [local (/data/jenkins/.m2/repository), Edenred.it-mirror (xxxxxxxx)]: Plugin not found in any plugin repository -> [Help 1]
Is a normal behavior because we don't have in local this plugin?

If you are using the gradle plugin - 5.0.0-M2 has not been publshed yet.  We had a few additional updates.  It should be released in the next day or two.

Thanks Jeremy, I'll check with the next release.
I confirm again after have tested that works with Issue with org.owasp:dependency-check-maven:4.0.2.
I've updated the Description for make more clear where the command is inserted.

Issue is not resolved with 5.0.0-M2
[ERROR] Failed to execute goal org.owasp:dependency-check-maven:5.0.0-M2:check (default-cli) on project xxx-service: Fatal exception(s) analyzing xxx-service: One or more exceptions occurred during analysis:
[ERROR] Autoupdate is disabled and the database does not exist

Using 4.0.2 version with same configuration and setup works.

The database schema (and event the file name changed).  I'm assuming you had created the database while connected to the Internet and then used the cached version for future scanning.  This works as long as you are doing something to keep the locally cached database updated.
Most organizations mirror the NVD in their private network; see: https://github.com/stevespringett/nist-data-mirror

Thanks for getting back to me Jeremy,
We used the DependencyChecker Jenkins Plugin's "Invoke depenency checker update only" to update the database daily and run mvn dependency-check-maven:check using that database for our builds.
The problem appears to be that the Jenkins plugin's latest version is only 4.0.2, but the Maven plugin is 5.0.0-M2 however as you stated the database schema has changed which caused our issue.
Sorry for the confusion.

Note that 5.0.0-M2 is a milestone release - not a final release.  We are working to finalize the 5.0.0 release over the next few weeks.


Speed up relative re-imports into absolute imports

Cython has to know the qualified module name at compile time, so the absolute package path of a relative import is actually known at compile time. We cannot change the way relative first imports work in general, because they are actually looked up relative to the current package, but we can still speed up re-imports (which are actually not that uncommon). Cython can construct the qualified name of a relative module import at compile time and look that up in sys.modules (as done in the ImportDottedModule helper function for absolute imports).
Requires C / C-API knowledge.

Mind if I take this as a "good first issue"?

Mind if I take/attempt this as a "good first issue"?

Sure, if you know C? The import code is mostly written using C-API code. Look at the FromImportStatNode in Nodes.py and the ImportNode in ExprNodes.py that generate the relevant C code. I suggest writing a test case (see the HackerGuide) with repeated relative imports and looking at the generated C code to see what we currently do and what can be done.

Sure, if you know C?

Yup.
@scoder Does this have any relation to #2854 and #2859 in terms of steps taken or good ways to approach it? I'm also having a look at tests/run/absolute_import.srctree.

Yes, related. The __Pyx__ImportDottedModule_Lookup helper function does the right module lookup, once you have the fully qualified module name, so that can be reused (move it to a new /////… section in ImportExport.c and use //@requires to depend on it). I'm also ok with doing this only for Py3, if Py2 proves any problematic.
The qualified module name needs to be constructed for relative imports in FromImportStatNode (I think), then passed as a Python string constant (e.g. created with code.intern_identifier()) into the helper, if that returns NULL, it means that the module needs to be looked up normally.


intensity-normalization: init at 2.0.1

python3Packages.scikit-fuzzy:  unstable-2020-10-03 -> unstable-2021-03-31
Motivation for this change
Add a package.
Since this package was primarily intended as a CLI package but now has a Python API, I've added to both python-packages and all-packages.
Things done

Built on platform(s)

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

[NA] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[NA] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages 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/): some binaries fail with an ImportError due to missing optional antspy dependency
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

nixpkgs-review happy, macos 10.15 & nixos x86_64

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

4 packages built:

intensity-normalization (python39Packages.intensity-normalization)
python38Packages.intensity-normalization
python38Packages.scikit-fuzzy
python39Packages.scikit-fuzzy


Mac command uses deprecated -K flag when setting up SSH keys

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/authentication/connecting-to-github-with-ssh/generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent
What part(s) of the article would you like to see updated?
Running ssh-add -K ~/.ssh/id_ed25519 on a Mac results in the following console warning, so both the command and the callout below it need to be updated.
WARNING: The -K and -A flags are deprecated and have been replaced
         by the --apple-use-keychain and --apple-load-keychain
         flags, respectively.  To suppress this warning, set the
         environment variable APPLE_SSH_ADD_BEHAVIOR as described in
         the ssh-add(1) manual page.

Additional information
I'm on a new 16" MacBook Pro.

@tsongas
Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

Hi @tsongas, thanks for opening this! ✨
When we first spotted the deprecation notice, I wanted to update all the commands in the article with the replacement flags but learnt that support for them only appeared recently (as far as I can see, they were first available in in Monterey).
Without providing separate commands for different versions of MacOS, I think we're best keeping the deprecated flags for the time being so we can be confident they will function for all users. The deprecation notice is better than the flags simply not being recognised.
We added a callout about the deprecation notice in https://github.com/github/docs/pull/11443 and we'll continue to keep an eye on this. Once we can be confident that they will work for the vast majority of users, or Apple remove support for -K and -A entirely, we'll update the article.
I'll close this issue for now but do @mention me if you have any extra information about the change; it was a bit of a struggle to determine when support was added! 😄


Add one CVPR GAN paper.

Thanks for sharing the collection. I added my CVPR19 Oral paper. Thanks a lot.

Thanks!


Move PathData/RttEstimator to a paths module

The refactoring parts of #804 so we can get this out of the way.

(Also, #822 was actually necessary to make the abstractions line up here, so no stepping on toes at all...)


Keep the ignored files during the upgrade process

Motivation
This PR fixes the issue #12237 relative to react-native-git-upgrade.
When the user adds new files to the .gitignore file, those files are deleted during the upgrade process because it starts by creating a fresh .gitignore file without user's modification, so the user's ignored files are no longer ignored and they are deleted during the upgrading process.
The best solution I've found to keep the user's ignored files... ignored, consists in appending the content of the .gitignore file in the <TEMP DIR>/.git-rn/info/exclude file.
The user's ignored files are now ignored at the repository level, they no longer interfere with the upgrade process.
Reminder: The tool uses a temporary local Git repository generated on the fly, so we can do whatever we want in it.
Test Plan

Publish react-native-git-upgrade to sinopia
npm install -g react-native-git-upgrade
Init a new project with an old version: react-native init MyApp --version=0.50.0
Create a new file named dummy-ignored-file.json in the project's root
Append dummy-ignored-file.json to the .gitignore file
Run react-native-git-upgrade

👉 The dummy-ignored-file.json is not deleted.
Release Notes
[CLI][BUGFIX][react-native-git-upgrade] - Keep the user's ignored files while upgrading

Actually I can just fix it myself :)
@facebook-github-bot shipit

cc @hramos


Add documentation and references to the Calculations code

[ ] find and add the materials (in general + code)
[ ] let us know

@white07S is this commit the explanations?

@bujowskis Indded this commit is the explanations for those calculations.
I am not sure why Cython automatically created that file but I will try to fix it asap.

@white07S I mean, I'm not sure if there's anything wrong they are being created, the question is - should cython be committed?

@white07S also, there seem to have risen some errors with the imports/structure and the tests are failing?


Custom model API service interface

How to customize the model service API?
For example, how to modify openai's base_url?

Hi @tom54007, have you checked the provider "custom_host"? Does this meet your requirements
Related Docs: https://docs.tasking.ai/docs/integration/models/language_models/custom-host

I am also looking on changing the openai's base_url. It is possible with the custom_host, BUT my proxy needs custom headers so custom_host does not fit my use case.

I am also looking on changing the openai's base_url. It is possible with the custom_host, BUT my proxy needs custom headers so custom_host does not fit my use case.
Gotcha, then we'll try to design a mechanism to allow custom headers. please stay tuned :-)

Thank you very much. After fit in this project(one-api), it completely solved my needs.


M3-4937 Add: Interfaces section to Linode Configs modal

Description

Do you know if there's logic on the back-end for sorting the interfaces to place the public interface in index 0 in the array after they get created? If there isn't, I think we need logic in LinodeConfigDialog.tsx doing that before the data gets submitted
Edit: This also raises the question that since a VLAN interface can be specified without a public one, should index 0 of the interfaces array be null (since only a public interface should occupy that index) in that case?
i.e.
[ null, { label: 'example_interface', purpose: 'VLAN', ipam_address: '<IP_ADDRESS>/24' } ]

There is logic for this on the backend, but we shouldn't allow putting public anywhere other than first. I was going to do that and forgot 🤷 . I'll push that up soon. The first item can be a VLAN, though, so having it be a normal select seems best.


Failed to create  release

Failed to create release in local.
The commands are
git clone https://github.com/cloudfoundry-community/docker-boshrelease.git
cd docker-boshrelease
./update
bosh create release --force --name docker

And the errors are:
  >   * cf-uaa-lib-3.2.5.gem
  >   * concurrent-ruby-1.0.1.gem
  >   * daemons-1.2.3.gem
  >   * excon-0.48.0.gem
  >   * docker-api-1.26.2.gem
  >   * eventmachine-1.0.7.gem
  >   * hashie-3.4.3.gem
  >   * json_pure-1.8.3.gem
  >   * kgio-2.10.0.gem
  >   * thor-0.19.1.gem
  >   * railties-4.2.6.gem
  >   * lograge-0.3.6.gem
  >   * thin-1.6.4.gem
  >   * nats-0.5.1.gem
  >   * omniauth-1.3.1.gem
  > /tmp/d20161201-27761-d4nfvr/d20161201-27761-ez3qbb/cf-containers-broker/vendor/cache/omniauth-uaa-oauth2-f892fd3415f9/lib/omniauth/uaa_oauth2/version.rb:16: warning: already initialized constant OmniAuth::Cloudfoundry::VERSION
  > /home/ubuntu/.rvm/gems/ruby-2.2.1@test/bundler/gems/omniauth-uaa-oauth2-f892fd3415f9/lib/omniauth/uaa_oauth2/version.rb:16: warning: previous definition of VERSION was here
  >   * sprockets-3.5.2.gem
  >   * sprockets-rails-3.0.4.gem
  >   * rails-4.2.6.gem
  >   * rails-api-0.4.0.gem
  >   * raindrops-0.16.0.gem
  >   * sass-3.4.21.gem
  >   * tilt-2.0.2.gem
  >   * sass-rails-5.0.4.gem
  >   * settingslogic-2.0.9.gem
  >   * unicorn-5.0.1.gem
  > + RAILS_ENV=assets
  > + bundle exec rake assets:precompile
  > /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/spec_set.rb:92:in `block in materialize': Could not find addressable-2.4.0 in any of the sources (Bundler::GemNotFound)
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/spec_set.rb:85:in `map!'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/spec_set.rb:85:in `materialize'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/definition.rb:132:in `specs'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/definition.rb:177:in `specs_for'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/definition.rb:166:in `requested_specs'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/environment.rb:18:in `requested_specs'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/runtime.rb:13:in `setup'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler.rb:122:in `setup'
  > 	from /home/ubuntu/.rvm/gems/ruby-2.2.1@global/gems/bundler-1.8.4/lib/bundler/setup.rb:18:in `<top (required)>'
  > 	from /home/ubuntu/.rvm/rubies/ruby-2.2.1/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
  > 	from /home/ubuntu/.rvm/rubies/ruby-2.2.1/lib/ruby/site_ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
'cf-containers-broker' pre-packaging failed

Hi,
Try to install addressable-2.4.0
gem install addressable-2.4.0
https://rubygems.org/gems/addressable/versions/2.4.0
Best regards,

Yes, I can make it work to add "bundle install" in "cf-containers-broker" source code.
But I think we should fix it in this release or "cf-containers-broker" project, but not each user to do this manually if she/he want to generate one release?

We may ask for gem.lock file to be provided.

Is this still an issue with the v30+ releases?


Return URL instead of string in swagger

@alzimmermsft, you were suspicious this may not be fixable.  Please take a look.

Swagger doesn't support URL in an intuitive way and would require changes to the core generator. Given that we have already added convenience methods/functionality to handle the conversion of URLs in the clients I think it is safe to close this issue as can't/won't be done.


uWSGI integration

Flask recommend use uWSGI for production projects.
Additional references:

https://www.digitalocean.com/community/tutorials/how-to-build-and-deploy-a-flask-application-using-docker-on-ubuntu-18-04-pt
https://medium.com/@cirolini/docker-flask-e-uwsgi-d10e58c56489

It is not needed. Just implement a docker with Flask and uWSGI.
https://github.com/tiangolo/uwsgi-nginx-flask-docker


Configure options for autowiring

I added autowired metadata to a few component options. These were mainly driven by the needs of camel-quarkus and what its test suite expects to be auto configured. But it should be useful for users in any case.
I did the changes as separate commits in case any have to be rolled back for some reason.

Thanks a lot, I'm looking at the AWS stuff, I do believe it would be nice to have the autowire well done for these components also on the quarkus side.


select multiple tags for better search

What feature or improvement do you think would benefit Files?
for example i have tags

cats
dogs
flowers
...

I would like to choose a cat in flowers by choise tags in left menu
(now it can be done in the search bar by phrase "tag: cats, flowers" , but each time you have to type the phrase manually to search)
Requirements

This proposal will improve searching whis using tags

Files Version
<IP_ADDRESS>
Windows Version
22h2
Comments
No response

You can click a tag in the details layout to search for it, is this what you're looking for?

You can click a tag in the details layout to search for it, is this what you're looking for?

I think this will make it clearer what I mean:

it would be nice if it worked with keypresses ("+" to add a new tag to the search list and "-" to remove it)

We're looking into adding search filters, this isn't going to be ready soon but it's on the roadmap and I think it'll help for this.

We're looking into adding search filters, this isn't going to be ready soon but it's on the roadmap and I think it'll help for this.

That is the related issue btw:
https://github.com/files-community/Files/issues/4813


Deprecate package 'org.opensearch.action.support.master'

Is your feature request related to a problem? Please describe.
A part of issue https://github.com/opensearch-project/OpenSearch/issues/1684.
To support inclusive language, the master terminology is going to be replaced by cluster manager in the code base.
The goal for this issue is:

Deprecate the package org.opensearch.action.support.master, which is the only package contains "master" in the name, and it will be removed in the next major version (3.0)
Have alternative package org.opensearch.action.support.clustermanager as the replacement.
Rename all public/protected classes, methods and variable contain "master" in the old package, and have alternative new class, method or variable name in the new package. Each class in the new package preserves the git history of the deprecated class as the successor.

Describe the solution you'd like
For overall solution to replace "master" in public Java APIs: https://github.com/opensearch-project/OpenSearch/issues/1684#issuecomment-1145541374
Key points:

At the time of deprecating the package, all public/protected classes, methods or variables that contain "master" in the package will be deprecated as well, to avoid spending time modifying the same file again.
Any public/protected Class with "master" in the name will extend the renamed class to maintain only one class while supporting the old class.
Any public/protected method or variable with "master" in the name will directly call the renamed method or variable, for the same reason as above.
Renaming and deprecating class/method/variable must be in 2 separate Pull Requests, in order to be in separate git commits after PR merged.
Because there will be 2 classes as the result, one with new the name and one with the old name, and the class with new name should be the successor, only in this way can preserve the git file history in the renamed class.

In detail:

Copy the folder of the package and keep it in another place.
Replace master word with clustermanager in the package name. This is done by the Rename refactoring feature of IntelliJ IDEA, so that both the definition and reference can be renamed.
Replace Master with ClusterManager in the class name, using the same way as above.
replace master with clusterManager in the method name, using the same way as above.
Create a Pull Request for the above changes, so that the file renaming can be tracked in the git history.
Wait until the PR merged.
Paste the original master folder back.
Add @Deprecated annotation and @deprecated Javadoc tag in package-info.java to mark the whole package as deprecated.
Remove all the existing implementation in the old classes, except any public/protected method or variable that contains "master" name. Make the old class extends the new renamed class.
For class, adding all the constructors of the super class.
For the method or variable contains inclusive name, change its implementation to call the corresponding ones in super class.
Add @Deprecated annotation and @deprecated Javadoc tag for the class and method that contains "master" in the name.
If the renamed class has unit test, copy the unit test to test the deprecated class.
Backport all the changes to 2.x branch.

Example of the 2 commits:
In branch: https://github.com/tlfeng/OpenSearch/commits/rename-master-package-preview
First step: rename - https://github.com/tlfeng/OpenSearch/commit/cdc3de435b6f32bff692733c340d0d2463cc559a
Second step: add old usage back and deprecate: https://github.com/tlfeng/OpenSearch/commit/e484e0f5225ae337cc0521f3427d5ace413ee1ec
Describe alternatives you've considered
None.
Additional context
The package org.opensearch.action.support.master has got 15 Java classes inside.
6 classes of those having "master" in the names.
8 methods and 2 variables in these classes having "master" in the names.
1 Class to be renamed:
MasterNodeOperationRequestBuilder
MasterNodeReadOperationRequestBuilder
MasterNodeReadRequest
MasterNodeRequest
TransportMasterNodeAction
TransportMasterNodeReadAction
2 Method and variable to be renamed:
In class MasterNodeRequest -
TimeValue DEFAULT_MASTER_NODE_TIMEOUT
TimeValue masterNodeTimeout
Request masterNodeTimeout(TimeValue timeout)
Request masterNodeTimeout(String timeout)
TimeValue masterNodeTimeout()
In class TransportMasterNodeAction -
void masterOperation(Request request, ClusterState state, ActionListener listener)
void masterOperation(Task task, Request request, ClusterState state, ActionListener listener)
String getMasterActionName(DiscoveryNode node)
In class TransportClusterInfoAction -
void masterOperation(final Request request, final ClusterState state, final ActionListener listener)
void doMasterOperation(Request request, String[] concreteIndices, ClusterState state, ActionListener listener)

Progress update on 07/19/2022:
To achieve the goal to deprecate the package org.opensearch.action.support.master, and use org.opensearch.action.support.clustermanager instead,
the following classes that have been deprecated and alternative classes have been created:
MasterNodeOperationRequestBuilder
MasterNodeReadOperationRequestBuilder
MasterNodeReadRequest
MasterNodeRequest
TransportMasterNodeAction
TransportMasterNodeReadAction
The following classes are pending to be deprecated:
AcknowledgedRequest
AcknowledgedRequestBuilder
AcknowledgedResponse
ShardsAcknowledgedResponse
ClusterInfoRequest
ClusterInfoRequestBuilder
TrasnportClusterInfoAction

Progress update on 07/29/2022:
Completed:
The following methods are deprecated and renamed by PR https://github.com/opensearch-project/OpenSearch/pull/4032 (backport to 2.x branch by https://github.com/opensearch-project/OpenSearch/pull/4048)
In class TransportMasterNodeAction -
abstract void masterOperation(Request request, ClusterState state, ActionListener listener)
void masterOperation(Task task, Request request, ClusterState state, ActionListener<Response> listener)

In class TransportClusterInfoAction -
abstract void doMasterOperation(Request request, String[] concreteIndices, ClusterState state, ActionListener listener)
void masterOperation(final Request request, final ClusterState state, final ActionListener<Response> listener)

Remaining:
The following classes still exist in package org.opensearch.action.support.master
AcknowledgedRequest
AcknowledgedRequestBuilder
AcknowledgedResponse
ShardsAcknowledgedResponse

But they will not be deprecated and provided an alternative usage, but be renamed in a future major version directly as a breaking change. There is the reason https://github.com/opensearch-project/OpenSearch/issues/3663#issuecomment-1195873346

As mentioned above, the 4 classes (AcknowledgedRequest, AcknowledgedRequestBuilder, AcknowledgedResponse, ShardsAcknowledgedResponse) in package org.opensearch.action.support.master will not be deprecated in version 2.x, and will be moved to package org.opensearch.action.support.clustermanager directly in a future major version as a breaking change.
Other than that, there is no remaining for this issue.


Appveyor and Travis badge

Now that we have ci, you might want to add the travis status image and the appveyor status badge. This way users can immediately see that you project has automated tests and feel a little more comfortable in using it. And potential contributors can be confident that their contributions are automatically tested.

I'm still thinking about this. I am considering disabling CI for master instead, since it will be redundant – everything will already have been tested before it gets merged. That would leave no good branch to use for a status image, unless I commit to having some kind of "staging" branch. If I use the latest build over the whole project, it might scare people away instead, since those can fail on any pull request.
There is also the matter of not making the README uglier :)

Currently master seems to be the latest stable branch. If the badge is limited to that branch, the users can see that this project has
a) tests
b) passing tests
just by looking at your README. These status images are not for you. You already tested the code and know that it works. But your users don't.

I already understand that it's not for me, and that there is value in using something people immediately recognize.
Nonetheless, the badges communicate not exactly the right information about the project. The information I want to efficiently communicate is that master is always stable. The information the badges do communicate is that master is sometimes stable, but might not be, and that there are intervals when this information is pending because the build has not finished. In fact, there are never any such intervals. So, as you can see, the badges don't do what I really want.
I am even considering things such as using static images (with some ALT text to justify the deception for those who discover it). But I am also quite likely to just use the badges normally because it's simple, even though sloppy. I simply want to look for better options first.
I am aware of the ability to limit the badges to master. As I explained before, having builds for master is redundant because it wastes time building. It's also imprecise, because the build can be pending, when in fact a build of the exact same commit has already completed. I am trying to avoid these drawbacks, if it is possible in a reasonable way. Yes, they are minor, but, sorry for repeating, it's not such an emergency that I can't spend a few days thinking about it in my downtime. It's worth something to me if I can communicate the reliability of master as easily as a badge communicates the status of the latest build. If I can't, then I fall back to the regular badges.
I would appreciate any useful information you have. Please don't repeatedly tell me what I and everyone else who uses GitHub already knows.

If you are looking for static images, shields.io might do what you want.
AFAIK if a build is pending/cancelled/in progress, travis/appveyor will display the last build status instead.

Thanks, that looks cool :)
I don't know for sure if Travis/AppVeyor would display a pending status, but AppVeyor docs do mention configurable pendingText. Maybe they only show that if there is no previous completed build for a branch/project, but it still spooks me, at least at first glance.

I just checked and travis does indeed wait for the build to complete before updating the badge. Appveyor displays a "pending" status image instead.

Right, thanks. So, this behavior is acceptable, but not optimal.
It's a valid issue, just not immediately clear how to best resolve it. There is, after all, some value in even "redundant" master builds, since I might make a mistake when fast-forwarding, or doing something else.

FYI

Looks good!


Migrate math backend kernels to a separate file and interface defining inputs / outputs.

This is the first PR preparing us for eager mode.
This PR separates NDArrayMath from backends. Backends now need to implement NDArrayMathBackend, which contains a set of core kernels. The NDArrayMath layer is backend-agnostic - it does shape validation and can compute combinations of kernels.
This PR also:

Wraps NDArrayMath - NDArrayMathCPU and NDArrayMathGPU are now thin wrappers that initialize NDArrayMath with different backends. The API remains the same.
Separates the optimizer tests into their own files, away from session_test.ts.

This change is

 This is a great step towards eager! Nice work!!

Reviewed 2 of 9 files at r1, 119 of 120 files at r2.
Review status: 80 of 81 files reviewed at latest revision, 4 unresolved discussions.

src/math/math.ts, line 23 at r2 (raw file):

import * as axis_util from './axis_util';
import {MatrixOrientation, NDArrayMathBackend} from './backends/math_backend';
import {NDArrayMathBackendCPU} from './backends/math_backend_cpu';

I think we should move the legacy NDArrayMathGPU and NDArrayMathCPU c-tors to their backends: NDArrayMathGPU goes to math/backends/math_backend_webgl.ts, and analogously for NDArrayMathCPU
Then math.ts won't import any backends directly other than the interface NDArrayMathBackend. This won't cause any cyclic deps since the CPU/GPU backend will depend on math, but math depends only on backend interface.
This allows us to ship only webgl backend, or only cpu backend later on, with treeshaking.

src/math/math.ts, line 44 at r2 (raw file):

abstract

remove abstract. We should be able to create a global NDArrayMath without going through the legacy NDArrayMathGPU/NDArrayMathGPU

src/math/math.ts, line 2082 at r2 (raw file):

}

export class NDArrayMathCPU extends NDArrayMath {

add jsdoc on top
/** @deprecated Please use NDArrayMath */
here and in NDArrayMathGPU below

src/math/backends/math_backend_cpu.ts, line 33 at r2 (raw file):

NDArrayMathBackendCPU
optional: Consider calling it just NDArrayBackend/NDArrayBackendWebGL/NDArrayBackendCPU (math is implicit) And rename the files to backend_cpu.ts and backend_webgl.ts.

Comments from Reviewable


Question: does caddy load plugins in specific order?

Hi,
I'm developing small caddy plugin (http midleware) and I wonder in which order my plugin will load and wrap other middlewares. Is there any specific order on which I can rely? Thx @mholt

Hey Mateusz! There's two ways to answer your question depending on what you mean.

Plugins are imported in an arbitrary order. Meaning, their init() functions (and thus registering them with Caddy) happen in some arbitrary order. This is by design, and makes it very convenient.

If a plugin has a directive for the Caddyfile, the setup functions are executed in a specific order, given absolutely in the server type's code. For the HTTP server, the list of directives is in plugin.go. This is also by design, and ensures that code is executed in the proper order when setting up the server. Because, for example, the tls directive needs to know what addresses bind has configured for each site before it sets up HTTP->HTTPS redirects.

Hope this helps! (Whatcha writing? :wink:)

Got it, thx! I will share when I'll be ready 😄 Hope by the end of the week 🙏


Adding InMemoryLogger to the Getting Started Notebook

Stores training data in InMemoryLogger instances and plots the results to compare speed updates

I think the reason for only doing 3 epochs on the baseline run is to demonstrate that 3 epochs running without Composer takes approximately the same amount of time as 4 epochs running with Composer. I can still modify the baseline if demonstrating that speedup effect is not as important, though.


Course adding bug

API server expects tucan id in add course endpoint. This is not documented and also throws an error in the add.coursestats.de app

Resolved in recent API changes!


An error occurred while executing the "rosservice call /finish_trajectory 0"

I run demo_backpack_2d.launch successfully.(roslaunch cartographer_ros demo_backpack_2d.launch bag_filename:=${HOME}/Downloads/cartographer_paper_deutsches_museum.bag
)
when i run "rosservice call /finish_trajectory 0" to save the map ,an error occured.
ERROR: Unable to load type [cartographer_ros_msgs/FinishTrajectory].
Have you typed 'make' in [cartographer_ros_msgs]?
I build and install cartographer_ros according to The official reference：https://google-cartographer-ros.readthedocs.io/en/latest/compilation.html#building-installation
I want to know how to solve this problem or how to save map.

You probably didn't build cartographer_ros_msgs which is a sub package of cartographer_ros.

Thanks for your response.Actually i know this ERROR means cartographer_ros_msgs didn't build,but i don't know why and i just follow official document.I would appreciate if you give me some advice about how to rebuild cartographer_ros_msgs.

I can't say more because I don't use the catkin_make_isolatedcommands written in the documentation (I use catkin build).
But maybe someone else or one of the maintainers can help.

#1078 [(https://github.com/googlecartographer/cartographer_ros/issues/1078)]
this issue provides the solution.if you follow the official document to compiling cartographer_ros successfully,you need to do is source before calling service.


Pyspark tutorial?

Hi, I have gone through the tutorial and would like to try pyspark on hdfs. I notice pyspark is pre-installed (2.0.x). But it doesn't support the pre-installed python (version 2.6.6). To make it work, need to upgrade to newer version of python. I tried pip, yum to update python. But these commands are not recognized in the VM.

@skahler-pivotal do you know if thats doable in the image?

Not sure if that is doable in the image. pyspark must be coming down as part of Zepplin, but I imagine the fact that the system is still running RHEL6 is causing an issue.
I'll take this as another flag that the process needs to be upgraded to use RHEL7
There are a couple tutorials on upgrading or adding python2.7 that aren't that tough, but yum and some of the system stuff relies on 2.6 at the base. So you've got potential to mess that up. It is a VM though so probably not much lost if it does go sideways.

Thanks! I figure I can use yum when login as root. Previously I login as gpadmin and couldn't use yum

gpadmin should be able to sudo to root in the VM


Add the chat in the panel window

It could be great to have the chat box in the panel box, not as a "file" tab.

@fubaduba a feature request for chat

Hey @kud we have to work within the user interface provided by VS Code, which means chat needs to be placed where it is. But we are working to make the chat more visible. Thank you for your feedback.

Alright, thank you @fubaduba


[2.2] Bump Mesos to nightly 1.11.x cb6cfe9

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/d4678d33b223fec5d48007f8246f1ed1cda5e90d...cb6cfe9b122d1b60a8264b28b6abb38a3c8417b4

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review


Fortune Recording Oracle - Integration with Reputation Oracle

Description
Develop an api client to communicate with Reputation Oracle.
Motivation
When the job is fully completed, Recording Oracle should be able to send a request to Reputation Oracle to start calculating reputation points for tellers.
Implementation details

[ ] Develop an api client to communicate with Reputation Oracle

Since the recording oracle already has to write the data to steady storage. It can just provide a notification to the reputation oracle that there's data to fetch from a URL


Specifying multiple name filters in components GET requests

Is there a way to specify multiple names of metrics/calculatedmetrics/segments in the respective GET endpoints' parameters so that the response contains data only about the specified components?
E.g. calling the \segments endpoint with parameter names="segment1,segment2,segment3" returns a JSON response with data on the 3 segments in it's content.
From what I understand from the documentation we can specify name keywords in the \segments endpoint, but the response only contains data on segments that have all of the keywords in their names, so if we want multiple segments with completely different names, we have to either do 1 GET request per segment, or do 1 (more if there are more than 1000 total segments) GET request that returns all segments and filter them on client side.
The same thing happens for \calculatedmetrics and the \metrics endpoint does not even have a name filter parameter.

Hi @goncalocapelalopes, the behavior and workaround you described are correct. Providing an "OR"-type list of names to filter on is not currently supported.
Feel free to submit a feature feedback request for this via experience.adobe.com by clicking the question mark icon in the top right of the screen and selecting the "Feedback" tab:


Feature/geode 1952 2

This merges feature/GEODE-1952, which was branched from the
docs donation staging branch, into feature/GEODE-1952-2, which
was branched from develop.
In addition to merging the docs donation, it adds the
geode-book directory for building the end-user documentation
from the content in the geode-docs directory. It also updates the
geode-docs README for building the docs locally.

Hi @joeymcallister The CI failed because the new files are missing the Apache License header. Can you please fix this and re-submit the pull request?
71 Unknown Licenses
*******************************
Unapproved licenses:
  /home/travis/build/apache/incubator-geode/geode-book/Gemfile
  /home/travis/build/apache/incubator-geode/geode-book/Gemfile.lock
  /home/travis/build/apache/incubator-geode/geode-book/config.yml
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/index.html.erb
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/javascripts/book.js
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/javascripts/waypoints/context.js
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/javascripts/waypoints/group.js
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/javascripts/waypoints/noframeworkAdapter.js
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/javascripts/waypoints/sticky.js
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/javascripts/waypoints/waypoint.js
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/layouts/_book-footer.erb
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/layouts/_title.erb
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/stylesheets/book-styles.css.scss
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/stylesheets/partials/_book-base-values.scss
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/stylesheets/partials/_book-vars.scss
  /home/travis/build/apache/incubator-geode/geode-book/master_middleman/source/subnavs/geode-subnav.erb
  /home/travis/build/apache/incubator-geode/geode-book/redirects.rb
  /home/travis/build/apache/incubator-geode/geode-docs/images_svg/JMX_Architecture.svg
  /home/travis/build/apache/incubator-geode/geode-docs/images_svg/MBeans.svg
...

Is geode-book a generated dir or a source directory?

geode-book is a source directory containing the pieces (e.g., the
config.yml that points Bookbinder to content, and the geode-subnav.erb
that provides the book's table of contents) that Bookbinder uses to
generate directories geode-book/output/ and geode-book/final_app from
the markdown files in geode-docs.
On Thu, Oct 13, 2016 at 1:44 PM Anthony Baker<EMAIL_ADDRESS>wrote:
Is geode-book a generated dir or a source directory?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-geode/pull/259#issuecomment-253633135,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXEUTUuhqkV-oHKwTUCuOLUm8zUxP1bks5qzpg7gaJpZM4KVYQ9
.

+1
Bookbinder publishing procedure worked for me.

@sbawaska Commit 2b6fec8 corrects the license issues and I was able to build successfully.

+1
I built with bookbinder and viewed with rackup, per the README.md instructions.
Running ./gradlew rat passed the license check:  no licensing issues.

I think the README could use a few clarifications:

You also need to run bundle install in the final_app dir.
How do we take the generated html site and merge it into the asf-site branch so it can be hosted on geode.apache.org.

I also ran into some strange ruby errors installing libv8 / therubyracer that required google's help.

I think geode-docs should be a subdirectory of geode-book so we don't have two unrelated directories for these docks.
It should probably output to a directory called build to be consistent with the rest of the build.

@upthewaterspout Bookbinder requires the "book" directory to be at the same dir level as the content, so a subdirectory wouldn't work here. If the goal is to clean up the top level, we could move both of these directories into one directory—perhaps as follows:

geode-docs

geode-docs-book
geode-docs-content

@metatype

Running bundle install in the final_app dir isn't something I'm familiar with. We don't usually have to do it, and I'm currently able to build and rackup the Geode book without doing that but, well ... who knows what Ruby's doing behind the scenes? (@upthewaterspout Did you happen to encounter this when running through the process the first time?)
I postponed the docs-to-website step in order to focus on getting the docs buildable and licensed. The short version is that you can move the contents of the directory geode-book/final_app/public/docs to the directory geode-site/website/content/docs and then follow the instructions for updating the website located at geode-site/website/README.md. (I need to tweak some stylesheet stuff in geode-site to make the docs display correctly, but that should be a one-time thing.)

@upthewaterspout:

Add the final_app and output directories to the rat excludes, so people don't have rat failures after building the docs.

Excellent suggestion. Thank you. 👍

Is there a clean target to remove the docs?

My experience with this has always been with a dedicated docs-book repo, so git clean was sufficient. If there's a way to specifically clean only the geode-book dir, I'd love to implement it.

@joeymcallister - Ok, the parallel directories makes more sense now. We can always move stuff around after 1.0 if we decide to.
I also had to do bundle install in the final_app directory in order to run rackup. I don't know if we really need to use rackup anyway, this seems like an easier way to quickly view the book:
cd final_app/public
python -m SimpleHTTPServer 8080

Tweaked the build instructions to include build install before rackup. Also moved the README, which focuses on building the docs, to the geode-book dir (and pointed to it from geode-docs/CONTRIBUTE.md, which focuses on working with the raw content).
If it's okay with everyone, I'd like to call GEODE-1952 done and revisit any outstanding issues from the thread (e.g., dir structure) in future work/tickets.

LGTM

+1


hystrix-metrics-event-stream uses org.codehaus.jackson

org.codehaus.jackson:jackson-core-asl is the old version of Jackson, currently (rather, forever) at 1.9.2.
The real current version is 2.5.0 and has a new group id com.fasterxml.jackson.core; also, some classes moved around.

Thanks for the report @piefel .  Fixed in #763


Pin pandas in conda-forge environment

Addresses #1896
Description
This PR adds a short-term fix to pin pandas in the conda-forge environment, so that the GitHub Actions checks on PRs will pass.
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

Superseded by #1899