Nicolas Lelong rotoglup

`IMPORTED` library

add_library(NvGPUDirect::DVP SHARED IMPORTED)
target_include_directories(NvGPUDirect::DVP INTERFACE "${NV_GPU_DIRECT_INCLUDE_DIR}")
set_target_properties(NvGPUDirect::DVP 
  PROPERTIES
    IMPORTED_IMPLIB ${NV_GPU_DIRECT_IMPLIB}
)

Some thoughts about code structure, as I read code, and find some things harder to follow than others.

Pass the real argument, not the owner

❌ void someMethod(const DataOwner& owner) { ...; owner.data.doSomething(); owner.data.somethingElse(); ... }

✅ void someMethod(const Data& owner) { ...; data.doSomething(); data.somethingElse(); ... }

Mark methods `static` when possible

My GIT habits :

On Windows, through GIT bash
No UI, except occasional gitk --all& to visually check branches and commits
Use git fetch then git merge --ff-only origin/master
- I don't like git pull auto-merging feature that messes up the history
Use git add -p to :
make a quick review of changes before commit

Agile

Et si on redémarrait l'agile ?, Arnaud Lemaire

Agile in "management" vs. Agile in "production"

You Should Stop Doing Daily Scrum Meetings, Vitaly Chicago, archive

A good list a common problems in daily standups

Conan (on Windows)

conan search <name> -r conan-center

What I liked :

easy install through python's pip

What I don't like :

In an administrator PowerShell console

Get-NetConnectionProfile
Set-NetConnectionProfile -InterfaceAlias "Ethernet" -NetworkCategory Private

or

Set-NetConnectionProfile -Name "Ethernet" -NetworkCategory Private

ucrtbase.dll

Can be broken for C Runtime, depending on versions of Windows, see fmod broken here:

https://developercommunity.visualstudio.com/comments/1243113/view.html

This is complicated as the UCRT is shipped by windows, but the c++ team first has to fix it then convince the windows team to ship a hotfix.

Microsoft Visual C++ files

applyMatrix on Geometry does not flip faces when needed

applyMatrix, on Geometry or BufferGeometry, does not flip faces indices when applying a matrix that flips normals (determinant < 0).

see mrdoob/three.js#17361

	#include <Cocoa/Cocoa.h>

	#import <Metal/Metal.h>
	#import <MetalKit/MetalKit.h>
	#import <simd/simd.h>

	// BUG: on OSX 10.15.6, the App menu is not clickable at the first activation, but becomes so after switching to another app

	//----------------------------------------------------------------------------

	//******************************************************************************
	//
	// Copyright (c) 2016 Microsoft Corporation. All rights reserved.
	//
	// This code is licensed under the MIT License (MIT).
	//
	// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
	// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
	// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
	// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER