error LNK2019: unresolved external symbol WinMain

Hi, wanted to take the project for a test spin :) Not really an expert with cmake, but I got this peculiar error when building ZeldaPython - not mentioning any particular src file
PS C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine> cmake .
-- Building for: Visual Studio 17 2022
-- Selecting Windows SDK version 10.0.22621.0 to target Windows 10.0.22631.
-- The C compiler identification is MSVC 19.38.33139.0
-- The CXX compiler identification is MSVC 19.38.33139.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.38.33130/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.38.33130/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Using module to find Vulkan
-- Found Vulkan: C:/VulkanSDK/1.3.275.0/Lib/vulkan-1.lib (found version "1.3.275") found components: glslc glslangValidator
-- Vulkan include dir: C:/VulkanSDK/1.3.275.0/Include
-- Vulkan library dir: C:/VulkanSDK/1.3.275.0/Lib/vulkan-1.lib
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - not found
-- Found Threads: TRUE
-- Including Win32 support
-- GLM: Version 1.0.0
-- GLM: Build with C++ features auto detection
-- No build type selected, default to Debug
-- Generating project file for example in C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaEngine/Engine/ZeldaEngine
-- Generating tool file in C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaEngine/Engine/ZeldaMeshlet
-- Generating tool file in C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaEngine/Engine/ZeldaPython
-- Generating tool file in C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaEngine/Engine/ZeldaPython
-- Configuring done (4.2s)
-- Generating done (0.1s)
-- Build files have been written to: C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaEngine

PS C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine> cmake --build .
MSBuild version 17.10.4+10fbfbf2e for .NET Framework

  1>Checking Build System
  Update Zelda Engine Resource Success!
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Engine/CMakeLists.txt
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5): w
arning MSB8065: Custom build for item "C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprin
cefounder\ZeldaEngine\CMakeFiles\880f9780ff667a04bb60a835a0e0a527\always_rebuild.rule" succeeded, but specified output
"c:\users\olliv\desktop\art and development\render engines\zeldaengine by iceprincefounder\zeldaengine\engine\always_re
build" has not been created. This may cause incremental build to work incorrectly. [C:\Users\olliv\Desktop\Art And Deve
lopment\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Engine\ZeldaEngineShaders.vcxproj]
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glslang/glslang/CMakeLists.txt
  CodeGen.cpp
  Link.cpp
  Generating Code...
  GenericCodeGen.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\Z
  eldaEngine\Third_Party\glslang\glslang\Debug\GenericCodeGend.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glslang/glslang/OSDependent/Windows/CMakeLists.txt
  ossource.cpp
  OSDependent.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\Zeld
  aEngine\Third_Party\glslang\glslang\OSDependent\Windows\Debug\OSDependentd.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glslang/glslang/CMakeLists.txt
  cmake_pch.cxx
  glslang_tab.cpp
  attribute.cpp
  Constant.cpp
  iomapper.cpp
  InfoSink.cpp
  Initialize.cpp
  IntermTraverse.cpp
  Intermediate.cpp
  ParseContextBase.cpp
  ParseHelper.cpp
C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Third_Party\glsla
ng\glslang\MachineIndependent\ParseHelper.cpp(7539,78): warning C4267: 'argument': conversion from 'size_t' to 'int', p
ossible loss of data [C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEn
gine\Third_Party\glslang\glslang\MachineIndependent.vcxproj]
  PoolAlloc.cpp
  RemoveTree.cpp
  Scan.cpp
  ShaderLang.cpp
  SpirvIntrinsics.cpp
  SymbolTable.cpp
  Versions.cpp
  intermOut.cpp
  limits.cpp
  linkValidate.cpp
  Generating Code...
  Compiling...
  parseConst.cpp
  reflection.cpp
  Pp.cpp
  PpAtom.cpp
  PpContext.cpp
  PpScanner.cpp
  PpTokens.cpp
  propagateNoContraction.cpp
  Generating Code...
  MachineIndependent.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefound
  er\ZeldaEngine\Third_Party\glslang\glslang\Debug\MachineIndependentd.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glslang/SPIRV/CMakeLists.txt
  GlslangToSpv.cpp
  InReadableOrder.cpp
  Logger.cpp
  SpvBuilder.cpp
C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Third_Party\glsla
ng\SPIRV\SpvBuilder.cpp(1119,45): warning C4267: 'argument': conversion from 'size_t' to 'unsigned int', possible loss
of data [C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Third_Pa
rty\glslang\SPIRV\SPIRV.vcxproj]
  SpvPostProcess.cpp
  doc.cpp
  SpvTools.cpp
  disassemble.cpp
  spirv_c_interface.cpp
  Generating Code...
  SPIRV.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngin
  e\Third_Party\glslang\SPIRV\Debug\SPIRVd.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glfw/src/CMakeLists.txt
  context.c
  init.c
  input.c
  monitor.c
  platform.c
  vulkan.c
  window.c
  egl_context.c
  osmesa_context.c
  null_init.c
  null_monitor.c
  null_window.c
  null_joystick.c
  win32_module.c
  win32_time.c
  win32_thread.c
  win32_init.c
  win32_joystick.c
  win32_monitor.c
  win32_window.c
  Generating Code...
  Compiling...
  wgl_context.c
  Generating Code...
  glfw.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine
  \Third_Party\glfw\src\Debug\glfw3.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glm/glm/CMakeLists.txt
  glm.cpp
  glm.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\
  Third_Party\glm\glm\Debug\glm.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glslang/glslang/CMakeLists.txt
  glslang_c_interface.cpp
  glslang.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEng
  ine\Third_Party\glslang\glslang\Debug\glslangd.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/glslang/glslang/CMakeLists.txt
  ResourceLimits.cpp
  resource_limits_c.cpp
  Generating Code...
  glslang-default-resource-limits.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by i
  ceprincefounder\ZeldaEngine\Third_Party\glslang\glslang\Debug\glslang-default-resource-limitsd.lib
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/meshoptimizer/CMakeLists.txt
  allocator.cpp
  clusterizer.cpp
  indexcodec.cpp
  indexgenerator.cpp
  overdrawanalyzer.cpp
  overdrawoptimizer.cpp
  quantization.cpp
  simplifier.cpp
  spatialorder.cpp
  stripifier.cpp
  vcacheanalyzer.cpp
  vcacheoptimizer.cpp
  vertexcodec.cpp
  vertexfilter.cpp
  vfetchanalyzer.cpp
  vfetchoptimizer.cpp
  Generating Code...
  meshoptimizer.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\Ze
  ldaEngine\Third_Party\meshoptimizer\Debug\meshoptimizer.lib
  Generating vulkan_renderer.rc
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Engine/CMakeLists.txt
  ZeldaEngine.cpp
  GLM_GTX_hash requires C++11 standard library support
C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Engine\ZeldaEngin
e\ZeldaEngine.cpp(1622,18): warning C4101: 'iSendResult': unreferenced local variable [C:\Users\olliv\Desktop\Art And D
evelopment\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Engine\ZeldaEngine.vcxproj]
  imgui.cpp
  imgui_draw.cpp
  imgui_widgets.cpp
  imgui_tables.cpp
  imgui_impl_glfw.cpp
  imgui_impl_vulkan.cpp
  ofbx.cpp
  Generating Code...
  libdeflate.c
  ZeldaEngine.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\Zeld
  aEngine\Debug\ZeldaEngine.exe
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Engine/CMakeLists.txt
  ZeldaMeshlet.cpp
MSVCRTD.lib(exe_winmain.obj) : error LNK2019: unresolved external symbol WinMain referenced in function "int __cdecl in
voke_main(void)" (?invoke_main@@YAHXZ) [C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by icepri
ncefounder\ZeldaEngine\Engine\ZeldaMeshlet.vcxproj]
C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Debug\ZeldaMeshle
t.exe : fatal error LNK1120: 1 unresolved externals [C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEng
ine by iceprincefounder\ZeldaEngine\Engine\ZeldaMeshlet.vcxproj]
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Engine/CMakeLists.txt
  ZeldaPython.cpp
MSVCRTD.lib(exe_winmain.obj) : error LNK2019: unresolved external symbol WinMain referenced in function "int __cdecl in
voke_main(void)" (?invoke_main@@YAHXZ) [C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by icepri
ncefounder\ZeldaEngine\Engine\ZeldaPython.vcxproj]
C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\ZeldaEngine\Debug\ZeldaPython
.exe : fatal error LNK1120: 1 unresolved externals [C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngi
ne by iceprincefounder\ZeldaEngine\Engine\ZeldaPython.vcxproj]
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Engine/CMakeLists.txt
  ZeldaPythonLib.cpp
  ZeldaPythonLib.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\Z
  eldaEngine\Debug\ZeldaPythonLib.dll
  Building Custom Rule C:/Users/olliv/Desktop/Art And Development/Render Engines/ZeldaEngine by iceprincefounder/ZeldaE
  ngine/Third_Party/tinyobjloader/CMakeLists.txt
  tiny_obj_loader.cc
  tinyobjloader.vcxproj -> C:\Users\olliv\Desktop\Art And Development\Render Engines\ZeldaEngine by iceprincefounder\Ze
  ldaEngine\Third_Party\tinyobjloader\Debug\tinyobjloader.lib

This seemed to work (open up the sln file and change the below mentioned property for ZeldaPython & ZeldaMeshlet):
https://stackoverflow.com/questions/6626397/error-lnk2019-unresolved-external-symbol-winmain16-referenced-in-function#:~:text=Try to change,Console (/SUBSYSTEM%3ACONSOLE)

Try to change Properties -> Linker -> System -> SubSystem (in Visual Studio).
from Windows (/SUBSYSTEM:WINDOWS) to Console (/SUBSYSTEM:CONSOLE)


Future ESLint compatibility

In the roadmap presented by the TypeScript team for the first half of 2019 there was an announcement to migrate their own linting from TSLint to ESLint as well as starting to do major contributions to bring ESLint up to parity with TSLint. Where I'm unsure how much of the work would be required in the main ESLint repository compared to the monorepo for all the tooling which enables ESLint to support TypeScript. Read more here
Given these news, I wonder if the maintainers of this library are interested in migrating the TSLint rules to work with some current or future version of ESLint? Would be interesting to hear the different aspects under consideration:

Does the current version of ESLint have enough feature parity or are many more API's or API changes required to allow for implementing Codelyzer with ESLint?
Are there very many similarities between the ESLint and TSLint APIs? Or will it essentially be a full re-write?
Would it be interesting to begin supporting both TSLint and ESLint? And would it be in this repository then or would a separate one be more appropriate?

On a personal note I am intrigued on the idea of migrating to the generally more popular and performant ESLint. But if the price is losing the rules of this project for any Angular projects I'm using I don't really think it'll be worth the hassle.
Usage of ESLint has been brought up before in #16 but figured this announcement coupled with the long time allowed for a new issue.

This is currently not on the roadmap and I don't see clear benefits of such migration.
Closing the issue for now, but I'll keep it in the back of my head for Q3-Q4 this year.

Palantir has just announced TSLint will be deprecated in favor of ESLint.

thks @mgol for your link; We are going to think about this problem.

Really looking forward to this one. We are in a rough spot currently where we are using ESLint for most stuff but still need TSLint running just for these rules.

FYI https://twitter.com/MrJamesHenry/status/1130143387292786688

I have migrated over 50% of the rules now, and will coordinate with Minko on making an alpha available soon.
I have also created the builder necessary to run ESLint via “ng lint” and will add schematics soon.
Stay tuned!

Angular CLI it's waiting for the migration to ESLint to complete in order to switch to it from TSLint. Maybe this should be coordinated as well.
angular/angular-cli#13732

@JamesHenry Is your work public somewhere? I would be interested in following the progress. 🙂

@glen-84 I had a meeting with Minko yesterday about it :) It will be publicly accessibly (although not documented initially) very soon!

@JamesHenry Any update on this?

@dumptruckman you can follow the progress here. If you have a spare moment, feel free to push a port of any of the not implemented rules to the repo.
Moving the CLI to ESLint is not part of the version 9 timeline, but @JamesHenry already has a CLI builder which will let you opt in.

Unfortunately there is not much progress or any update in this repository.
Could you please give it some attention and at least update the README with an ETA, a TODO list and a way for users to test existing functionality :)

I will add some more information this week, on my vacation.
It’s always important to remember that these are not products - they are communities. I do not get paid to work on it any more than you do.
As Minko noted on the 23rd August, it has been open for a while and we’ve been happy to accept contributions from other members of the community we share together.

As @mgechev notes, you can try ESLint out at any time with your Angular projects by using (and I would suggest watching) the https://github.com/angular-eslint/angular-eslint project on github.
You can check out the latest v9 Angular CLI integration test for how to manually set up ESLint with your Angular CLI projects today:

ESLint config: https://github.com/angular-eslint/angular-eslint/blob/master/packages/integration-tests/fixtures/angular-cli-workspace/.eslintrc.js

angular.json: https://github.com/angular-eslint/angular-eslint/blob/master/packages/integration-tests/fixtures/angular-cli-workspace/angular.json

Eventually we will introduce an ng add schematic for it. PRs for that and further conversions of Codelyzer rules are more than welcome!
Now that we have quite a few utilities set up in angular-eslint they are pretty straightforward to convert to ESLint! All but one of the rules that the Angular CLI ships with today for TSLint have already been done.

Important Notes

@naeramarth7's comment is not accurate with regards to template rules, they are supported and two have already been converted (the two that the Angular CLI ships with and therefore the highest priority).

In response to @fr0 and @naeramarth7 I would not particularly recommend setting up the eslint-plugin-tslint (which I created) unless you absolutely have to. It runs a full instance of TSLint within ESLint and was only intended to provide an ultimate fallback in the early days before many rules had been converted (not just in an Angular context, in a more general TypeScript context). Give angular-eslint another look and please consider contributing! :)

Closing this issue since we've been working on a replacement on top of ESLint.


Tooltip for touch devices

Do you have any plans to make tooltips usable on touch devices?
If you are not planning on doing this, would you accept such PR?

How would you expect the Tooltip behavior to be on mobile?
On Apr 9, 2015 7:58 PM, "Kestutis"<EMAIL_ADDRESS>wrote:

Do you have any plans to make tooltips usable on touch devices?
If you are not planning on doing this, would you accept such PR?
—
Reply to this email directly or view it on GitHub
https://github.com/Dogfalo/materialize/issues/1141.

It would be nice to have an option (e.x. data-trigger) to define how the tooltip will be triggered: click | hover | focus | manual.
In this case click option would trigger tooltip on touch devices when user taps on the element.
What are your thoughts?

This is how it looks on mobile device https://vimeo.com/120788406 when the user taps on the element.

Found the bug in my own code, it does work on mobile just the way I expected.
Sorry for the fuss.

I also have the issue, that the tooptip dos not work on mobile. Looking at the code I wonder how a mouseenter event should be triggered by touching on a mobile device?

Wow apparently mouse events get called on mobile. However if you use Meteor's fastclick package they get skipped somehow. So I added the needsclick to my tooltip trigger and it is working fine now.

@PhilippSpo I think I'm having fastclick issues too. When I run the app normally in a browser, it works fine, but fullscreen modes on Android/Chrome and iOS/Safari both fail - probably because its falling back to fastclick there? .needsclick didn't seem to fix it for me...

It worked just adding .needsclick, however i am not able to close the tooltip :S  how did you achieve this @PhilippSpo


point-system page

Point-system page added ( In leaderboard page button point added)
removing scroll-bar in whole project (changes in index.css )

some changes remain


Missing woff2 file

Font awesome uses a woff2 file that was not included in the 1.0.5 release

And the non minified FA css should be removed (too lazy to open a new bug lol).

Can I blame the personal message controller? :tongue:

XD
Yes, without any doubt!

New Version of Font Awesome 4.5
https://fortawesome.github.io/Font-Awesome/whats-new/


Can't Manage extension settings

I've installed Code Cracker as an extension for Visual Studio 2015, but there appears to be no way to manage which diagnostics are on and what levels they are on at. I've found the analyzers node of References in solution explorer. While I can make changes there they don't seem to save as the new set the next time I open the solution or project. Even if they did, this is installed for my entire VS2015, so I'd like to manage it at that level rather than have to make changes to every project I open.

Duplicate of https://github.com/code-cracker/code-cracker/issues/524. Already discussing there.


Coherence between 2 MultipleChoiceFilter

Hello,
I notice a problem when using your excellent library.
When two MultipleChoiceFilter are proposed in a datagrid, the selection of some elements in the first one will modify the results in the datagrid but it has no effect on the values proposed in the second MultipleChoiceFilter.
Can that be done ?

You can bind to DataGridFilterColumnControl.Values instead of DataGridFilterColumnControl.SourceValues, that should give you the distinct list of all currently visible items in that column.

Added an additional DataGridFilterColumnControl.SelectableValues property that holds all values that are currently selectable in the column.


Implement a JS gate to the vsimem interface

As an alternative to #4 , I propose that a gate to the vsimem interface is implemented, allowing to use gdal.open on a JS object of Buffer type
Instead of relying on the rudimentary support in GDAL to make the network requests, a Node.js program which could build upon the rich JS ecosystem, will be much better suited to do the networking and leave the processing to GDAL
This is a very interesting JS-specific project and will be a useful addition to node-gdal-next
Alas, GDAL requires random-access I/O, so a Read/WriteStream can't be compatible - when using curl it will repeatedly ask the HTTP server for various parts of the file that is being read, often in a non-sequential manner
In particular, GeoJSON is infamous for being impossible to incrementally parse (which is the reason behind GeoJSONSeq)
Still, when coupled with the native http module, this approach can have excellent performance - only the setup will be done by JS/V8 with the C++ native code taking care of the actual data transfers

@andrewharvey @Ysgorg
I have an initial version at https://github.com/mmomtchev/node-gdal-async/tree/vsimem
You can call gdal.open on a Buffer object - everything works as for a normal file
Persistence is handled, but not enforced - if the GC deletes your buffer, the application won't crash, but subsequent operations will fail with an I/O error
I wonder if I shall keep a strong reference on the Buffer
If it works, it will probably be merged very soon

This is ready for merging
I added a reference to the Buffer object in the dataset in gdal.Dataset.buffer - this way you can be sure that the lifetime of the dataset won't exceed the lifetime of the buffer

Shipped in 2.6.0


Send the time-range dates in format conforming to the standard

Dates in time-range have to be sent in the caldav's DATE-TIME format,
so it should be 20210817T120000 and not 2021-08-17T12:00.00Z.
I've tested v1.0.5 with Nextcloud and filtering by time-range doesn't work. Changes in this PR make it work.
Ref calendso/calendso#479

Looking good, I'll merge this after some testing

Published

@llldar could you publish a new patch version to npm? :)

@llldar could you publish a new patch version to npm? :)

@kuba-orlik isn't version 1.0.6 the published version you need?

@llldar, oh, that's right. Must've been looking at a cached page. Thanks!


watch task is always run on initialization, regardless of ignoreinitial option value and file system contents

What were you expecting to happen?
Nothing
What actually happened?
Watch task ran during initialization
Please give us a sample of your gulpfile
const { watch } = require("gulp");

function watchAction(foo) {
  console.log("foo=", foo);
}

function testingTask(cb) {
  watch("../.mydir/.myfile", { events: "add", ignoreinitial: true }, watchAction(true));
  cb();
}

exports.testing = testingTask;

Terminal output / screenshots

$ gulp testing

[10:20:53] Using gulpfile D:\temp\gulpfile.js
[10:20:53] Starting 'testing'...
foo= true
[10:20:53] Finished 'testing' after 19 ms

Please provide the following information:

OS: Windows 10 x64 v20H2 (OS Build 19042.985)
node: 14.16.0
npm: 6.14.1
gulp: CLI version: 2.3.0, Local version: 4.0.2

Additional information
N/A

watchAction(true) actually calls the function, thus executing it immediately.
You need to pass a function to the watch function:
e.g. replace watchAction(true) with () => watchAction(true) (untested, but it should work 😀)

Brilliant, thank you @simonhammes


A non well formed numeric value encountered" in "partials/base.html.twig"

Accessing my website (index file), I now suddenly receive the following message:
("A non well formed numeric value encountered") in "partials/base.html.twig" at line 27
Snippet of the template where this takes place:
24 {% block stylesheets %}
25     {% do assets.addCss('theme://css/style.css', 100) %}
26 {% endblock %}
27 {{ assets.css() }}

The stylesheet is properly formatted. I have tried many things:

Cleared Cache
Used empty stylesheet
Checked Grav pages headers for proper use of date format

If I remove line 27, the page gets rendered but of course without styling. Anybody else experienced this also?
I am using PHP 5.6.23 and Grav 1.0.10

There's anything that looks strange in that snippet. Maybe caused by some plugin?
You can PM me the user/ folder via gitter.im/flaviocopes and I can give a look.

Are you sure you are not running PHP 7.1 alpha??? That is a known problem with this early alpha version of PHP.

Let me check the plugins if they cause anything strange... I tried different versions of PHP (7.0, 5.6 and 5.5); however all gave the same result.

After verifying many things (removed plugins, installed default theme, switched PHP versions, etc.), it caused by my HTTP server. I am using Laravel's Valet which is based on Dnsmasq and CaddyServer.
Restarting Valet solved the issue (ie. error no longer showing), however assets are not found. I will further look into how to solve this. So it's not a Grav issue :)

I know it works perfect on Caddy as I use it too, but I never tried Valet

Yeah, the strange thing is that it was working before. Perhaps because I did some composer updates, causing this error. Once I restarted Valet it kind of worked. Still don't know the real reason, but hey... :)


Toggle switch widget animations

I played around a bit and added animations to the toggle switch widget:

This looks amazing.

Yes, I'm really impressed.


Feature Request - Analytics and Reporting

Feature Request.
Could any reporting info be provided as to how many people have read or opened the message in Teams chat? Then if they have clicked the link/button? Maybe the data could somehow be captured via an event logged with the Graph API?
If it could be provided in a excel report or written to an azure table (which could then be pulled into PowerBi for example). That would be great.

Hello @bungleweed ,
Thanks for reaching out to us.
We are currently only making changes to fix critical bugs or deployment failures. Please consider forking the repository to implement your improvement and post back here if you're successful.

You can export Excel file of  the message notification status using Export detailed results*. There you will be able to see if message is sent successfully and what is team/user name.


Use getter methods of proto message on FromProto functions

Pull Request Test Coverage Report for Build 369167673

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 88.859%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gqlgenplugin/protomodelgen/plugin.go
4
6
66.67%

Totals

Change from base Build 363856416:
0.007%

Covered Lines:
1332

Relevant Lines:
1499

💛  - Coveralls


JsonPost example code minimizing

JsonPost example code minimizing

well, i have just change an example code, and then the check have failed?

 This is not even named react-something...

 This is not even named react-something...

it looks like this project is better then react!

 This is not even named react-something...

btw i hate react


nixosTests.cage: fix OCR properly

Description of changes
The underlying problem with OCR in this test has been that the only font installed was DejaVu Sans, a proportional font, which xterm would try to render as monospace.  This produced very broken looking text, which the OCR understandably had trouble with.  With an actual monospace font installed, there are no more problems and we don't need the hacks.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build nixosTests.cage


Updated license term dialog should not happen on dev or oss builds

https://github.com/Microsoft/vscode/blob/5b9ec15ce526decc5dd0488339e798f6bcb4ec46/src/vs/workbench/parts/update/electron-browser/update.contribution.ts#L63
Maybe check that env.licenseUrl does not point to https://github.com/Microsoft/vscode/blob/master/LICENSE.txt also?

Since this is a one off, from prior to 1.0 to post 1.0, I won't do anything now.


n+1 queries issue

Right now has_many association still have N+1 issue:
Entity Load (1.0ms)  SELECT "entities".*  FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "b0d49d56-16d2-4d9f-a946-db7504271799"]]
  ↳ app/controllers/graphql_controller.rb:10
  Location Load (0.3ms)  SELECT "locations".* FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "1a9bfbec-b2e6-4eb8-a64b-17a3ae77cf2a"]]
  ↳ app/controllers/graphql_controller.rb:10
  Location Load (0.3ms)  SELECT "locations".* FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "c88f263b-eea1-43cb-9a8d-acf55014a875"]]
  ↳ app/controllers/graphql_controller.rb:10
  Location Load (0.3ms)  SELECT "locations".* FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "8adb8c4a-bc72-4d9d-932d-5e2a34a65fdb"]]
  ↳ app/controllers/graphql_controller.rb:10
  Location Load (0.4ms)  SELECT "locations".* FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "c3892ae1-f9e3-487c-9224-4a28b097b575"]]
  ↳ app/controllers/graphql_controller.rb:10
  Location Load (0.3ms)  SELECT "locations".* FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "0d350ec8-9b81-4194-8623-3f5079173c89"]]
  ↳ app/controllers/graphql_controller.rb:10
  Location Load (0.2ms)  SELECT "locations".* FROM "locations" WHERE "locations"."deleted_at" IS NULL AND "locations"."entity_uuid" = $1  [["entity_uuid", "78a38941-a0c1-4066-b740-d9382bf6b78f"]]
  ↳ app/controllers/graphql_controller.rb:10

You may want to take a look at this link: https://graphql-ruby.org/schema/lazy_execution

@samsonw hey this feature was not developed for AR yet.
I just sent a patch that adds it.
Just FTR this feature works when using mongoid too, for 1 nested level. For more than 2 nested levels, to avoid n+1 queries of inclusions you are forced to use mongoid < 7 and install the gem mongoid_includes


Vercel: Unable to load translations using getServerSideProps/loadTranslations

While the setup works fine running locally, the deployed to vercel app returns empty translations by loadTranslations in getServerSideProps
GET | https://app.vercel.app/_next/data/HZ7wajlS0AuNQGjuW_9so/en/app/dashboard.json
{"pageProps":{"__ni18n_server__":{"resources":{"en":{"dashboard":{},"common":{}}},"ns":["dashboard","common"]}},"__N_SSP":true}

ni18n.config.ts
import ChainedBackend from 'i18next-chained-backend'
import HttpBackend from 'i18next-http-backend'
import LocalStorageBackend from 'i18next-localstorage-backend'

const isBrowser = typeof window !== 'undefined'

const isRunningOnLocalhost = !!process.env['NEXT_PUBLIC_IS_RUNNING_ON_LOCALHOST']

const localePath = '{{lng}}/{{ns}}.json'

export const ni18nConfig = {
  supportedLngs: ['en'],
  ns: ['common', 'auth', 'dashboard'],
  use: isBrowser ? [ChainedBackend] : undefined,
  defaultNS: 'common',
  fallbackLng: 'en',

  backend: isBrowser
    ? {
        backends: [LocalStorageBackend, HttpBackend],
        backendOptions: [{ expirationTime: 24 * 60 * 60 * 1000 }, { loadPath: `locales/${localePath}` }]
      }
    : isRunningOnLocalhost
    ? { loadPath: `apps/ui-web-app/public/locales/${localePath}` }
    : { loadPath: `locales/${localePath}` },

  interpolation: {
    format: (value, format, lng, options) => {
    ...
    }
  }
}

next.config.js
// eslint-disable-next-line @typescript-eslint/no-var-requires
const withNx = require('@nrwl/next/plugins/with-nx')

"i18next": "^21.6.5",
"i18next-chained-backend": "^3.0.2",
"i18next-http-backend": "^1.3.1",
"i18next-localstorage-backend": "^3.1.3",
"next": "12.0.7",
"ni18n": "^1.0.3-rc.0",

/**
 * @type {import('@nrwl/next/plugins/with-nx').WithNxOptions}
 **/

const withBundleAnalyzer = require('@next/bundle-analyzer')({
  enabled: process.env.ANALYZE === 'true'
})

const nextConfig = {
  nx: {
    // Set this to true if you would like to to use SVGR
    // See: https://github.com/gregberge/svgr
    svgr: false
  },
  images: {
    domains: ['cdn.nordigen.com']
  },
  i18n: {
    defaultLocale: 'en',
    locales: ['en', 'es']
  }
}

module.exports = withBundleAnalyzer(withNx(nextConfig))

const Dashboard: NextPage = () => (
  <AuthGuard>
    <RoutePermissionsGuard>
      <SubscriptionGuard>
        <DashboardLayout>
          <DashboardPage />
        </DashboardLayout>
      </SubscriptionGuard>
    </RoutePermissionsGuard>
  </AuthGuard>
)

export const getServerSideProps = async ({ locale }) => {
  const something = {
    ...(await loadTranslations(ni18nConfig, locale, ['dashboard']))
  }
  console.log('file: index.tsx ~ line 25 ~ getServerSideProps ~ something', JSON.stringify(something))

  return {
    props: something
  }
}

export default Dashboard

 {"i18next": "^21.6.5",
    "i18next-chained-backend": "^3.0.2",
    "i18next-http-backend": "^1.3.1",
    "i18next-localstorage-backend": "^3.1.3",
    "next": "12.0.7",
    "ni18n": "^1.0.3-rc.0"}

Hi @atanaskanchev, if the prebuilt JSON is empty it probably means the issue is on the build step.
I would start looking at the path's configuration after the isRunningOnLocalhost conditional.
It is possible you don't need the conditional at all, can you try one of the following solutions?

use undefined
use { loadPath: `apps/ui-web-app/public/locales/${localePath}` }
use { loadPath: `locales/${localePath}` }

If none of these work, would you mind creating a repository where with the reproducible behaviour where I can try to find a solution?

Hi @JCQuintas thanks for the quick response.
I've recreated the issue here https://github.com/atanaskanchev/ni18n-nx-issue and it's deployed to https://ni18n-nx-issue-9y6puo3xe-atanaskanchev.vercel.app/dashboard
localhost

vercel

vercel workspace

In the config, if !isRunningOnLocalhost I am setting the locales path { loadPath: locales/${localePath} } and it seems it matches the vercel workspace
 backend: isBrowser
    ? {
        backends: [LocalStorageBackend, HttpBackend],
        backendOptions: [
          { expirationTime: 24 * 60 * 60 * 1000 },
          { loadPath: `locales/${localePath}` },
        ],
      }
    : isRunningOnLocalhost
    ? { loadPath: `apps/ni18n-nx-issue/public/locales/${localePath}` }
    : { loadPath: `locales/${localePath}` },

So it seems when deploying to vercel, we don't have access to the public folder from the getServerSideProps. :(
The current behaviour work if you are deploying using a docker container though... I've opened a bug issue on nextjs repo and will keep you posted.
A quick solution for your case right now would be to use getStaticProps instead for the time being, or use clientNamespaces to have the translations loaded on the client only.

Hi @JCQuintas thanks for your quick resolution for this issue. Awaiting the release 🚀

Hi @JCQuintas  I've had a try with v1.0.4-rc.3 but it seems the issue with Vercel still persist.

Indeed, it didn't work. Looking at it deeper, it seems that the issue is on the https://github.com/vercel/nft packager that doesn't allow a "third party module" to flag a file as "necessary" out of its scope. So at the moment we can't really fix that automatically without creating a patch to the nft package and then waiting for vercel to update it.
In the meantime, one workaround would be to create a function on your repository to do that for us
// force-locales-packing.js
import path from 'path'

export const forceLocalesPacking = () => path.join('./public/locales')

Then you should be able to run that inside your getServerSideProps and it will "tag" the files to be bundled. I intend on creating a documentation for this as well as soon as I have time. Then I try to work on a fix on the nft package itself.
Let me know if it works for you, else I can update the repo you linked above with the steps necessary to make it work.

This worked for me.
import type { NamespacesNeeded, Ni18nOptions } from 'ni18n';
import { loadTranslations as ni18nLoadTranslations } from 'ni18n';
import path from 'path';

import { i18n } from '../../next.config';

export const namespaces = [
  'common',
  'register',
  'dashboard',
  'sidebar',
] as const;

export const ni18nConfig: Ni18nOptions = {
  supportedLngs: i18n?.locales,
  ns: namespaces,
};

export const loadTranslations = async (
  initialLocale?: string | undefined,
  namespacesNeeded?: NamespacesNeeded | undefined
) => {
  const locales = path.resolve('./', './public/locales');

  return await ni18nLoadTranslations(
    ni18nConfig,
    initialLocale,
    namespacesNeeded
  );
};

Has anyone found a solution to this?

This solved my problem https://github.com/JCQuintas/ni18n/issues/49#issuecomment-1024515204
const locales = path.resolve('./', './public/locales');

Hi @ZaniLuca, the main issue is on how the nft package decides which file to bundle or not. The main issue can be found here: https://github.com/vercel/nft/issues/77
I haven't had the time to look at that more in depth and propose a solution, but in the meantime, the solution provided by @RodriguesCosta should work.

Any solution on that?


Please invite me to the Recode-Hive GitHub Community Organization

Name
ARYAN CHAURASIA
Discord Username (if applicable)
No response
Additional Context
I want to be a part of this community and contribute to it with the best of my abilities.

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


Possible to design a confirmation view

Description
For some application owners, it is better to show a designed view of the data and not pdf. See Altinn/apps-backlog#1
Screenshots

Considerations
If we introduce a "confirmation view" design capability this might be used as input for a "receipt generated". This should be input to a more general "Do we need PDF" discussion.

Is there any need for components updates to be presented in this view?
Can this be solved as a backend functionality? Generate HTML based on the layout and data and save it as data just like PDF?

Analysis
Currently we support the following steps. Here data can be customized with formlayout as you wish, while the confirm step is static.
stateDiagram
    direction LR
    [*] --> Data
    Data --> Confirm
    Confirm --> End

With the following process definition:
<bpmn2:process id="Altinn_Data_Confirmation_Process" isExecutable="false">
    <bpmn2:startEvent id="StartEvent_1">
      <bpmn2:outgoing>SequenceFlow_1</bpmn2:outgoing>
    </bpmn2:startEvent>
    <bpmn2:task id="Task_1" name="Fyll ut skjema" altinn:tasktype="data">
      <bpmn2:incoming>SequenceFlow_1</bpmn2:incoming>
      <bpmn2:outgoing>SequenceFlow_2</bpmn2:outgoing>
    </bpmn2:task>
    <bpmn2:task id="Task_2" name="Bekreft skjemadata" altinn:tasktype="confirmation">
      <bpmn2:incoming>SequenceFlow_2</bpmn2:incoming>
      <bpmn2:outgoing>SequenceFlow_3</bpmn2:outgoing>
    </bpmn2:task>
    <bpmn2:endEvent id="EndEvent_1">
      <bpmn2:incoming>SequenceFlow_3</bpmn2:incoming>
    </bpmn2:endEvent>
    <bpmn2:sequenceFlow id="SequenceFlow_1" sourceRef="StartEvent_1" targetRef="Task_1" />
    <bpmn2:sequenceFlow id="SequenceFlow_2" sourceRef="Task_1" targetRef="Task_2" />
    <bpmn2:sequenceFlow id="SequenceFlow_3" sourceRef="Task_2" targetRef="EndEvent_1" />
  </bpmn2:process>

The confirm step looks like this in the applicaiton:

BRG has a need to customize this view. How can we support this?
Alt 1.
Keep the process as is:
stateDiagram
    direction LR
    [*] --> Data
    Data --> Confirm
    Confirm --> End

But extend the layout-sets.json (see docs) and make it possible to setup a layout set for the confirm step, where we would reference the datamodel in the previous step. Example configuration:
{
    "sets": [
      {
        "id": "data-step",
        "dataType": "someDataModel",
        "tasks": [
          "Task_1"
        ]
      },
      {
        "id": "custom-receipt",
        "dataType": "someDataModel",
        "tasks": [
          "Task_2"
        ]
      }
    ]
  }

With the following app structure:
App:
| ....
├───ui
│   ├───custom-receipt
│   │   └───layouts
│   └───data-step
│       └───layouts

Changes needed:

Frontend

If confirm step has setup a connected layout set => render.

Alt 2.
As discussed in slack, another alternative would be to define a seperate data step that acts as a "Confirm" step.
stateDiagram
    direction LR
    [*] --> Data
    Data --> Data_2
    Data_2--> End

What configuration would we need?
What must be implemented to support re-using the same data model?
How do we stay backward compat with existing applications using confirm (are there any?)

Conclusion
After deliberation with the team + lead architect and our product owner we have landed on going forward with Alt 1.
Altinn Studio
How can we add designer capabilities for different tasks?
How can we map design to a given data model?  Do we need to? Can we make the assumption that it is a data model for the previous data task?
What if the previous task had several data elements?
Ops requirements

Are there any requirements for monitoring? What is being built and what could go wrong?
Are there any requirements related to backup?

Acceptance criteria

Possible to design the confirmation view with the elements?
Still possible to save something that can be used as proof?

Specification tasks

[ ] Development tasks are defined

Development tasks

[ ] Add design capability in UI designer from another task
[ ] Update Confirmation view to support uilayout
[ ]

Test

Add test cases here as checkboxes that are being tested as part of the changes.

Definition of done
Verify that this issue meets DoD (Only for project members) before closing.

[ ] Documentation is updated (if relevant)

[ ] Technical documentation (docs.altinn.studio)
[ ] User documentation (altinn.github.io/docs)

[ ] QA
[ ] Manual test is complete (if relevant)
[ ] Automated test is implemented (if relevant)
[ ] All tasks in this userstory are closed (i.e. remaining tasks are moved to other user stories or marked obsolete)

Skatteetaten har et behov for å hindre revisorer fra å trykke "Send inn"-knappen fra selve Altinn3-appen til skattemeldingen. Det skal ikke begrenses på rolle - da det skal være mulig å bruke next-API fra et sluttbrukersystem.
Vi har kartlagt noen mulige løsninger for å øke brukeropplevelsen i innsendingsprosessen.
Mulighet 1: Deaktivere innsendingsknapp i Altinn3-app for revisors bekreftelse
"Send inn"-knappen (se skjermbilde nedenfor) ikke bør vises i Altinn3-appen i steget for revisors bekreftelse, for å forhindre revisor i å gjøre feil.
Revisors bekreftelse må gjøres i et sluttbrukersystem eller i Skatteetatens Visningsklient (Skatteetatens egen brukerflate). Den åpnes ved å følge lenka "Se opplysninger som er mottatt".
Det er greit om "Send inn"-knappen skjules i begge bekreftelsesstegene. Skatteetatens Visningsklient får støtte for å bekrefte begge steg.

Mulighet 2: Lenke rett fra Altinn Portal innboks-element til Skatteetatens Visningsklient
Fra innboks i Altinn Portal (se skjermbilde nedenfor), så kan lenka "Gå til skjema" føre brukeren direkte til Skatteetatens Visningsklient. Dette kan gjøres for begge bekreftelsesstegene.
SIRIUS bruker lite av mulighetene som ligger i brukergrensesnitt i Altinn3-apper. Hvis denne løsningen velges, så ønsker SIRIUS at dette også gjøres for feedback-steg.
Denne løsningen skal ikke endre den arkiverte kvitteringssiden.

Hei @knut-olav-hoven-ske ! Denne endringen er nå merget og tilgjengelig til å tas i bruk. Så vidt vi forstår vil den støtte opp under mulighet 1 som du har beskrevet over, ved at man kan velge å ikke inkludere noen send-inn knapp på siden når man er i confirm-steget. Dere må da bygge opp visningen på siden selv, med de komponentene/tekstene dere ønsker å bruke, akkurat som i skjema.
Dokumentasjon ligger her: https://docs.altinn.studio/app/development/configuration/process/customize/#custom-form-layout
Ta kontakt på studio-skd kanalen på Altinn-Slack'en om dere har noen spørsmål 🙂

Adding support for this in studio should be handled through https://github.com/Altinn/altinn-studio/issues/2058. Following up SKD and BRG for their use of this. Any other bugs/needs will be reported as seperate issues. Closing issue as done.

Hei, er det mulig å bruke dette til å lage tilpassede visninger i feedback-steget? Vi vurderer å bruke flere feedback-steg.


test-branch/execute-job/quoted-authorization/41400f7d-1ba2-4c92-a324-a17f27f39f64

PR for testing bceb338662c3c6b59120238decc5170999c05970

/canonical/self-hosted-runners/run-workflows bceb338662c3c6b59120238decc5170999c05970


Comparison to Capn Proto

rkyv looks like a very interesting project. We are using Capn Proto, but since we are developing in Rust this looks like it may be a good replacement. How does rkyv compare in terms of speed/size/security, etc. to Capn Proto?

Thanks for your interest! I'll try to address those three areas as best I can:
Speed
I'll preface this by saying that I have not properly benchmarked rkyv yet.
rkyv's speed should be at least comparable to Cap'n Proto, and may be better in some cases. Mostly this would be due to lower overhead traversing the data. For data of simple and intermediate complexity, I would expect both frameworks to perform the same.
Size
It's really tough for me to say on this one. I'm not familiar with the details of Cap'n Proto's binary format, but I do know that they claim to compress padding and unset fields. My guess is that Cap'n Proto would have the edge in size.
Security
It looks like Cap'n Proto has some validation on their message format, but they do also caution against handling messages from untrusted sources as they have not undergone a security review. rkyv is in the same boat, although it does not currently perform any message validation. I have some ideas floating around on how basic message validation could be introduced into rkyv.
In conclusion, I would not expect rkyv to perform much differently from Cap'n Proto. The main differences between the two frameworks will definitely be their feature sets. I would definitely encourage you to try rkyv out, find some gaps in the feature set, and file feature requests so I can evaluate what it would take to add them.

Thanks!

Just wondering, is it possible to make rkyv support capn' proto? Like serde supporting multiple protocols.

Unfortunately it is likely impossible. rkyv is tied directly to a portable representation of types and adapting it to Cap'n Proto's format is not something that can be done. If you wanted to, you could always deserialize objects from Cap'n Proto's format into a native structure and then archive it with rkyv.


System.Net.WebException: Unable to connect to the remote server

Please assist with this problem I encounter:
On the page http://localhost:8081/Home/Stateful, when I enter a key value paid and click the Add button, a "404 Not Found" is displayed (Figure 1).
The debug output is as follows:
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker: Information: Executing action method WebService.Controllers.StatefulBackendServiceController.PutAsync (WebService) with arguments ([B, Bing]) - ModelState is Valid
...
Exception thrown: 'System.Net.Http.HttpRequestException' in mscorlib.dll
Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware: Error: An unhandled exception has occurred: An error occurred while sending the request.

System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.WebException: Unable to connect to the remote server ---> System.Net.Sockets.SocketException: No connection could be made because the target machine actively refused it <IP_ADDRESS>:19081
   at System.Net.Sockets.Socket.EndConnect(IAsyncResult asyncResult)
   at System.Net.ServicePoint.ConnectSocketInternal(Boolean connectFailure, Socket s4, Socket s6, Socket& socket, IPAddress& address, ConnectSocketState state, IAsyncResult asyncResult, Exception& exception)
   --- End of inner exception stack trace ---
   at System.Net.HttpWebRequest.EndGetRequestStream(IAsyncResult asyncResult, TransportContext& context)
   at System.Net.Http.HttpClientHandler.GetRequestStreamCallback(IAsyncResult ar)
   --- End of inner exception stack trace ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at WebService.Controllers.StatefulBackendServiceController.<PutAsync>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeActionFilterAsync>d__28.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeAsync>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()

Figure 1

The same error on GetAll clicks as well. It tries to perform GET Operation at http://localhost:19081/GettingStartedApplication/StatefulBackendService/api/values?PartitionKind=Int64Range&PartitionKey=-9223372036854775808
which returns "Not Response!"
I'm not sure where that port "19081" comes into pictures, where the stateful service is listening on some random port. Needs some clarification here!

It looks like the partition count is 6 for cloud deployment and the range is all UInt64
https://github.com/Azure-Samples/service-fabric-dotnet-getting-started/blob/e5641cba3a284a84c00937ddcbfb4ac4b01463a9/src/GettingStartedApplication/GettingStartedApplication/ApplicationPackageRoot/ApplicationManifest.xml#L66
Shouldn't the partition number be 26 and the range from 0 to 25, if the partition scheme is the first capital letter?
https://github.com/Azure-Samples/service-fabric-dotnet-getting-started/blob/cf8070ea3a1c5931badf7dc55da064be47e5b38d/src/GettingStartedApplication/WebService/Controllers/StatefulBackendServiceController.cs#L141

Hi snpdev,
Did you ever solve this problem? I'm trying to get an application up and running and the reverse proxy is not working. I'm following the following article...
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-quickstart-dotnet
Thanks.

Port 19081 is the default reverse proxy port. You need to deploy the sample to a Azure Service Fabric cluster with the reverse proxy feature enabled. More information on the reverse proxy feature of Azure Service Fabric can be found here.

Thanks. I was trying to get it working in my local development environment using the SDK.
I found online that it doesn't work on Windows 7 unfortunately.


Question about Remote Site Setting and Gov Cloud Orgs

Hi Andrew and Friends, a quick question about the Remote Site Setting and whether I likely need to explain the usage of it by DLRS when installing in a Gov Cloud org. I'm in the process of confirming the IT approval process for managed packages. Any description of how DLRS uses the Remote Site Setting (e.g., metadata API access only?, no org data sent?, what about the org id / domain?) would be helpful and most appreciated. Thank you!

The Remote Site setting that DLRS installs is to enable DLRS to make remote calls into the same org to execute metadata deployments. It is through this use of the Metadata API that DLRS can create triggers on objects that have realtime and scheduled rollups configured.


Fix for clipping and camera navigation issues below ellipsoid

After re-looking at https://github.com/CesiumGS/cesium/issues/9198 it seems to be related to DepthPlane usage in Scene.  It seems the DepthPlane is used for the following purpose:

https://github.com/CesiumGS/cesium/blob/master/Source/Scene/Scene.js#L3243-L3245
I tried a quick workaround to offset down by an arbitrary amount.  It seems to address the reported issue.

Assuming this is a valid fix, how should it be implemented?

Simple magic number (as shown in https://github.com/CesiumGS/cesium/commit/b96d673b8030cff6f8becc7dc6f48a15ac295568) Perhaps it should be -11000m to cater for bathymetry?
Viewer constructor options argument that is passed Viewer=>Scene=>DepthPlane
Other

You should find the issues are improved or resolved running a local Cesium from this branch and the same sandcastle code from a local server:
local sandcastle link

Thanks for the pull request @chris-cooper!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Thanks for opening a PR Chris! There was some discussion here about pushing the depth plane down or other techniques that may be relevant here: https://github.com/CesiumGS/cesium/issues/7879

Thanks @OmarShehata .  After looking at https://github.com/CesiumGS/cesium/pull/8398/files it looks like the default for frameState.minimumTerrainHeight could be set differently (e.g. an option argument for Scene) and then used by DepthBuffer.  I've tweaked this PR as an example.

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

What would you like to do with this PR @OmarShehata ?

What would you like to do with this PR @OmarShehata ?

What would you like to do with this PR @OmarShehata ?

Is this PR waiting on anything, this PR would possibly fix an issue we are having.

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

@OmarShehata @chris-cooper Hey folks, is this PR going to be in the next release? This appears to relate to a issue our customers face in locations such as Florida Keys.
I've looked at the travis-ci and it looks like the tests are failing is that what is currently blocking the merge?

We are looking into this PR and other possible ways to resolve the issue. The current estimate date is the May release. @grantis

Hi @dzungpng , I have tested this locally on our dataset and with the latest version of cesium + these PR changes and it appears this change is causes further issues, I lose the model completely and it doesn't help with the clipping issue.

We got another bump for this issue (and request for an updated ETA) from the forum: https://community.cesium.com/t/new-cesiumjs-release/11110/15

Any news on this? Thanks

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 90 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 90 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @chris-cooper!
No one has commented on this pull request in 90 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 90 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Hi @chris-cooper and all, we just talked through this PR with @lilleyse. This is definitely a problem we've run into ourselves. While this does technically address the issue, it's a bit of a band-aid fix. We agree that 1000 seems a bit arbitrary. Can we get a real world constant, keeping in mind that some users are not using earth as their ellipsoid?
Also it may be better for this constant to live on the depth plane.
Also generally for bathymetry we disable the depth plan entirely.

Also generally for bathymetry we disable the depth plan entirely.

Slight correction - we disable the depth plan when the camera is underground or the globe is translucent, but not necessarily for bathymetry.

Thanks @ggetz and @lilleyse .  I've updated this PR to the latest master + tried to address your comment @ggetz .  Is this what you meant?
local sandcastle example

Thanks for the update @chris-cooper! This approach looks better. Would you be able to add documentation and unit tests, and update CHANGES.md?

I've added some of those things @ggetz .  For now it is just testing the option gets passed through.  I might need some help from someone with knowledge of that area of the rendering code if we want to test it more rigorously.

Thanks @chris-cooper! I suppose to test this more rigorously, we could add a new test that confirms the depth plane draw command has boundingVolume set to the expected value. But no similar tests currently exist for DepthPlane. @lilleyse Any thoughts?

A test like that could be added SceneSpec.js

I've tried adding a spec for that thanks @ggetz @lilleyse .

Awesome! Thanks for your patience with this PR @chris-cooper !

Without the logarithmic depth buffer this issue still reproduces. Here is a sandcastle where you can see the error by tilting the camera up.
https://sandcastle.cesium.com/#c=ZZJfb9MwFMW/ipWXZlLqJGvLRJtVQFuJB1gnNsFLJOTaN4mFY0e2k6mgfXdu/hTYyJPv9fndnHOTjlnSSXgCS26JhieyAyfbmn4deuGMD+XOaM+kBjuLyK9cE3w8WIute2s6KcCuLyC3wDx8M1aJx1ESXkUjIqDx1b1iGg5KycYZKY5F4cCvyTxNkiTXz1ebXOd6NEQdBw20VOYEdGAfwfn3JY50fpqNpgumHGxeQcqUzEpf1ZLve/JDWxRDxIsa9ZgcsDNxoL30EhxlQoRDxsY47BhN/mTbMevxxPSCFtbUeygtgAvnaXpD0+VqcZNEZHFN3ywX12+XJCLzVTJmr40AhXOm3bVWYjGjNH5gdaNgzzyLB42Lx1d9Zno6fccj7uA0m5ZYgSwr/wUwDmgOf819fHlB7453h4lxnClYk1XSl8/Tmqfc3jL+A8Shj3/GdcAmiILM+bOC7Ui/k3VjrEfXKkTLHtAyfmIXn1okPeXO9fOy+AJlQnZEits8ePX35AHhijmHN0Wr1IP8CXmwzWLUv8CUYULq8tiBVezcS6p0+2lsUkqzGMv/KW+MOjH7z8Tf

Thanks for the report @v8gx. I opened https://github.com/CesiumGS/cesium/issues/10166 to track. Would you possibly be able to contribute a fix?


chore(perf): rename switchFirst to exhaust on micro perf

Oh, nice catch.

This makes me remind that we also have renamed scheduler..

@kwonoj Yeah, you're right. I'll create an issue for this and start working on it.

Meanwhile, would you able to update commit message to be aligned with current history? I could see commits like https://github.com/ReactiveX/RxJS/commit/1127f13c4e40201d24ca7b501987d33601c9b30f#diff-4d51848d1226d013c854a78d7868ad24 or https://github.com/ReactiveX/RxJS/commit/8e6d560c862da0d034a0b7b9dda6d6e002c18b08#diff-4d51848d1226d013c854a78d7868ad24 mention it's micro perf changes. (I know I also made inconsistency of strict foramt.. :( )

@kwonoj done. ;)

Awesome, merged with 7dc98c3. Thanks @luisgabriel !


build: migrate the build tool from vue-cli to vite4

Closes https://github.com/apache/skywalking/issues/10106
Signed-off-by: Qiuxia Fan<EMAIL_ADDRESS>

Please review @wu-sheng


Using Latest Versions of Passenger - how to do?

Right now Beanstalk comes with 4.x version of Passenger as an environment.
There are ton of advanced features with 5.x version of Passenger.
Is there any ways we can use latest version of Passenger with Nginx? Or Enterprise Passenger version??

Change solution stack name (in config file eb_deployer.yml) to what you like to use:
Solution stack for elastic beanstalk, default is 64bit tomcat 7 for JAVA app
solution_stack_name: 64bit Amazon Linux 2014.03 v1.0.3 running Tomcat 7 Java 7


Fix mobile title changes

fix: update mobile title when convo changes.
doc: add magic of generate auth code

I merged this, but not sure when this issue started: currently, I'm trying to debug the latest changes to the master branch as chatgpt is not retaining conversations. I'm not very familiar with how the conversation handling has evolved since I first designed it, so I may limit backend changes until there's a proper test env.
Rolling back the node-chatgpt-api version didn't fix the issue. Luckily, my search branch is fully functional, and I may end up using it as the main branch and manually walking back the latest commits made to master.


Allow adding custom urls as service to main menu

It would be great if we could add any custom urls to the main menu, or implement bookmarks to open other streaming websites that are not included in the default services list.
If not, would be great if we could have the following spanish tv services added:
orangetv.orange.es (Orange TV in Spain)
atresplayer.com
www.rtve.es/play/

Absolutely!


Reducing number of concurrent segments tested 72s -> 51s locally

Codecov Report

Merging #1646 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1646      +/-   ##
==========================================
- Coverage   67.72%   67.67%   -0.05%
==========================================
  Files         825      825
  Lines       40128    40128
  Branches     5279     5279
==========================================
- Hits        27175    27157      -18
- Misses      11144    11165      +21
+ Partials     1809     1806       -3

Impacted Files
Coverage Δ

...not/common/segment/fetcher/HttpSegmentFetcher.java
28.57% <0%> (-19.05%)
:arrow_down:

...m/linkedin/pinot/common/utils/FileUploadUtils.java
45.58% <0%> (-7.36%)
:arrow_down:

...nkedin/pinot/common/metrics/ValidationMetrics.java
77.04% <0%> (-4.92%)
:arrow_down:

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-3.58%)
:arrow_down:

...m/linkedin/pinot/client/DynamicBrokerSelector.java
69.69% <0%> (-3.04%)
:arrow_down:

...filter/SortedInvertedIndexBasedFilterOperator.java
77.02% <0%> (-2.71%)
:arrow_down:

...ain/java/com/linkedin/pinot/client/Connection.java
51.35% <0%> (-2.71%)
:arrow_down:

...pinot/controller/validation/ValidationManager.java
81.01% <0%> (-2.32%)
:arrow_down:

...lix/core/realtime/PinotRealtimeSegmentManager.java
74.28% <0%> (-1.15%)
:arrow_down:

.../broker/routing/HelixExternalViewBasedRouting.java
89.54% <0%> (-1.05%)
:arrow_down:

... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 17b8dc8...94805de. Read the comment docs.


Error adding CONTRACT_ABI: Custom code is over 10000 character limit

I am having issues adding the contract ABI to the code. When I add it, the custom code is over the 10000 character limit for webflow. Is there any way to call the abi from an external source like a CDN? something like this?

I tried to just add this code with the correct user/rep@version/file and the mint button is not connecting to the contract and just sends me to the metamask website to download metamask.

The possible solutions are:

remove unused parts in ABI (ABI is just an array of { ... }, you can safely remove events and onlyOwner methods, basically you only need mint, getPrice and totalSupply)
use DEFAULTS.abiCacheURL to return JSON in a format { [address]: [...abi...] }:

Example:
DEFAULTS = { abiCacheURL: "https://ipfs.io/ipfs/Qm...." }

which returns:
{
    "0xYourContractAddress": [  /* .... your ABI array goes here */ ]
}


Implement pagination

This now properly fixes the pagination approach.
In some cases, n=max can return less results (not sure when, but for me it makes a difference if Iḿ accessing my wiki from remote, or through localhost), and this helps with this issue.
It's not the cleanest solution, since it duplicates some code, but without backporting my other refactorings done in my master branch, this was the easiest solution I could come up with.

Thank you @peschmae


Review usage of {u,i}size

I only see it being used for numbers of things that could conceivably be limited by memory.


Edit - unpredictable chaotic behavior when setting the control style Transparent

I am using "mIRC v7.66" + "DCX dll v3.1-git394" in system "Windows 8.1 x64".
When testing a control "Edit" with a control style set transparent a significant flaw in the form of unpredictable behavior was revealed of the element. This is one of the large and notable BUG that needs fixing. Thanks.
Brief description of what is happening:

When moving the mouse up and down, the text that is inside the field begins to superimpose many times, increasing gradually the font thickness and all this is accompanied by periodic flickering.
When typing, it is superimposed on top of the ghost message and with each subsequent letter, the font thickness also increases.
If, after the typed text, you go to the second field, the typed text is transferred there with the same text overlap.
When you try to erase the typed message, nothing happens, except for moving the cursor to the left, but if you move to the second field, the text is partially deleted.
If set focus to the first field, and then go to the adjacent open script editor window, then a ghostly "OK" button will appear in this field.
If set style transparent together with multi, then the effect will be slightly different, but similar to the described above.

Here is a test piece of script to reproduce this the problem. To run the script click on "F5" or enter the command "/edit_test":
alias F5 { edit_test }
alias edit_test {
  if ($dialog(edit_test)) .dialog -x edit_test
  else { .dialog -m edit_test edit_test }
}
dialog edit_test {
  title "Edit test"
  icon $mircexe,0
  option pixels
  size -1 -1 400 300
}
on *:DIALOG:edit_test:init:*:{
  .dcx Mark $dname edit_test_work
  .xdialog -b $dname +tyz
  .xdialog -g $dname +b $rgb(20,20,27)
  if ($exists($mircdirimage\edit-test.jpg)) .xdialog -g $dname +is $mircdirimage\edit-test.jpg
  .xdialog -c $dname 1 edit 90 75 210 23 showsel transparent | edit_decor $dname 1
  .xdialog -c $dname 2 edit 90 105 210 23 showsel transparent | edit_decor $dname 2
  .xdialog -c $dname 3 edit 90 135 210 78 multi showlinenumbers return showsel autovs | edit_decor $dname 3
}
alias edit_decor {
  .xdid -x $1 $2 +b
  .xdid -C $1 $2 +bk $rgb(25,25,32)
  .xdid -C $1 $2 +t $rgb(235,227,203)
  .xdid -f $1 $2 + ansi 10 Comic Sans MS
  if ($2 isnum 1-2) {
    .xdid -R $1 $2 +b 1 160
    .xdid -E $1 $2 Enter your text here ...
  }
  if ($2 == 3) {
    .xdid -o $1 $2 1 test line 1
    .xdid -o $1 $2 2 test line 2
    .xdid -o $1 $2 3 test line 3
    .xdid -o $1 $2 4 test line 4
    .xdid -S $1 $2 24 | .xdid -F $1 $2
  }
}
alias edit_test_work {}

Provisionally fixed.
The text displayed when transparent can have some odd artefacts depending on the background.
Working on something better.


Blade template not resolved

Hello,
When an exception is thrown inside a view, the compiled cache view is shown instead of the initial blade template.
Is this the intended behaviour?

Thanks,
Karel

I have the same question, the previous setup in Laravel used the regular blade view and I would like to get that functionality back. It's a real pain to try to match up the compiled view with the blade view

#132

@shaffe-fr @VincentVanWijk  Fixed https://github.com/spatie/laravel-ignition/releases/tag/1.6.4

@SocolaDaiCa Perhaps out of scope for this issue, but the collapse vendor frames button does not work yet. I was hoping it had something to do with the paths that were fixed with your pull request

@Vincent-avant what is collapse vendor. Could you explain It.

@SocolaDaiCa
As you can see on the screenshot at the top of this issue, there is a button on the top-left named "collapse vendor frames".
It collapses the stack trace on the left, to filter out the method calls of the framework itself, and only shows you 'your' files, if I understand it correctly.
You can see it working in this video on Freek van der Herten's channel :
https://www.youtube.com/watch?v=LEY0N0Bteew&t=739s
You can see a dropdown in the video that says "7 vendor frames" and "44 vendor frames".
This is what is currently not working on windows, the button does not collapse the stack trace.

I have the same issue on Windows. You can maybe create a dedicated issue @Vincent-avant ?


don't special case intensity target for pq

Benchmarking with
build/lib/jxl_gbench --benchmark_filter=".*PQ.*"

showed that TF_PQ() was not made significantly slower by this change.
Drive-by: add static_assert when evaluating polynomials

How sure are we that this is not triggering because we do something not OK
with a TF_PQ array/pointer in the tonemapping file?
On Wed, 30 Aug 2023 at 22:37, Sami Boukortt @.***>
wrote:

@.**** commented on this pull request.
In lib/jxl/transfer_functions-inl.h
https://github.com/libjxl/libjxl/pull/2762#discussion_r1310793251:

@@ -165,7 +165,7 @@ class TF_709 {
// Perceptual Quantization
class TF_PQ {
public:

explicit TF_PQ(float display_intensity_target)

explicit TF_PQ(float display_intensity_target = 255.0)

Is it because of the explicit? Because then we could have the explicit
one without a default value for the argument, and then add a no-parameter
constructor like this:
TF_PQ() : TF_PQ(kDefaultIntensityTarget) {}
—
Reply to this email directly, view it on GitHub
https://github.com/libjxl/libjxl/pull/2762#discussion_r1310793251, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOPAIYR7NLUQU4ATL67CO3XX6P67ANCNFSM6AAAAAA4EHOCBU
.
You are receiving this because you commented.Message ID:
@.***>


A mongo codec to serialize longs as strings + extra timing output

Longs were formatted in a similar way to Date's so I needed to do something like this. Also added some extra output so I could see exactly how long the loads were taking.

Thank you @winder ! Merged.


Make it possible to select what happens with errors

The current implementation simply raises NotImplementedError on linux if none of the existing methods (jeepney, dbus-python, systemd) can be used. It would be nicer for the user to be able to control what happens if using keepawake is not possible.
Currently, if someone would like to continue when setting keepawake is not possible, something like this must be used:
Option A
try:
    from wakepy import keepawake
except NotImplementedError:
	# Make a mock for replacing keepawake..

    from contextlib import contextmanager
	@contextmanager
	def keepawake(*args, **kwargs):
		yield

... # some code

with keepawake():
    # do something

Option B
try:
    from wakepy import set_keepawake, unset_keepawake
except NotImplementedError:
    # Make mocks for replacing set_keepawake and unset_keepawake..
    set_keepawake = lambda:None
    unset_keepawake = lambda:None

... # some code

set_keepawake()
# do something
unset_keepawake()

Idea
It would be nicer if the user could decide what occurs when:
(a)  A keepawake method fails (dbus address not found, etc..)  --> on_method_failure
(b) Setting keepawake fails completely --> on_failure
Then, code could look something like this:
from wakepy import keepawake

with keepawake(on_failure='warn'):
    # do something

This is much cleaner code and nicer to the user. Suggested options for on_failure and on_method_failure could be

error (raise wakepy.KeepAwakeError) -- default for on_failure?
warn (warnings.warn)
logwarn (logger.warning)
loginfo (logger.info) -- default for on_method_failure?
logdebug (logger.debug)
pass (do nothing)
callable (custom callable called with single string as argument)

So, on linux, with multiple methods defined: keepawake(method_linux=('jeepney', 'systemd')): would, in absense of dbus address and sudo rights, by default:

Fail jeepney (dbus) because no DBUS_SESSION_BUS_ADDRESS found in environment variables. Tell that with logger.info
Fail systemd because no sudo rights. Tell that with logger.info
Fail to set keepawake completely as no more methods to try. Raise KeepAwakeError and tell that jeepney cannot be used because DBUS_SESSION_BUS_ADDRESS is not set and systemd cannot be used because sudo rights are missing.

Working on this and planning to include in the next release.

This should be done now, and available in the v.1.0.0 branch. Will merge to master and publish a version in PyPI once tests and docs are also ready.

Nice work. Will include that in Minarca Data Backup whenever you make the release.
Did you also check if the exception is properly raised by keepawake instead of import wakepy ?
Thanks

Thanks for commenting! Yeah I had some nice Easter holidays writing an update :)
It is definitely a goal to remove all exceptions for any errors occuring at import time ( e.g. when doing from wakepy import keepawake). That should be already true in the v.1.0.0 branch, but I think it is still possible that an error occurs when importing from specific implementation module (e.g. wakepy._implementations._linux._dbus). That's of course not something what a normal package user should do, but I felt that it's making writing tests more difficult than it should, so I'll refactor also those and make all imports to be exception-free. Much nicer if the exception is raised at possible with keepawake(on_failure='error')..

Ok great !

That should be already true in the v.1.0.0 branch, but I think it is still possible that an error occurs when importing from specific implementation module (e.g. wakepy._implementations._linux._dbus). That's of course not something what a normal package user should do

Agree with you. You should focus only on import wakepy not raising an error.

@ikus060
This is now fixed in a released version 0.7.1. The new syntax is
from wakepy import keep

with keep.running() as m:
  if not m.success:
    # optional: signal to user?
  do_something()

The idea is that you may now decide to signal to the user somehow if the inhibitor lock could not be taken; this can be checked with m.success. There is also keep.presenting() context manager which also prevents screensaver & screen lock.
The keep.running and keep.presenting can be used as direct replacements for keepawake()

Reopening this. I think the 0.7.1 implementation had in mind that entering the context manager might be a non-blocking call. Since it will be a blocking call in the future, I think that an input argument on_fail would make sense. I will add this to the 0.8.0 release. Have to think about the possible values and the default value for the on_fail later.

Planning to make the following changes. The modes will have on_fail argument:
with keep.running(on_fail="warn"):
    ...

The on fail can either a string or a callable.

"error" (raise wakepy.KeepAwakeError) -- default for on_failure?
"warn" (warnings.warn)
"pass" (do nothing)
callable (custom callable, which takes single argument: ActivationResult)

The ActivationResult has more information about the activation process, and can later be extended if needed.


Add compressed dataset

Add a compressed dataset for the ML model so that contributors can try their hand at training it and tweaking the code.

Cannot attach datasets larger than 2GB. Will not fix.


feat: add crossplane provider-aws crds

Hi, I've added crossplane aws-provider's crds based on:

https://github.com/upbound/provider-aws/tree/main/package/crds

There are more than 1700 files in your commit.
is this really necessary to validate all those schemas?


Corrected typo in docfx from azps 610 publish process

Docs Build status updates of commit 55058d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Suggested changes for SC member selection

Per conversations in the October SC call and the SC Slack channel.
This PR is to get conversation going and to iterate as necessary.

Specify appointed member terms start Feb 1 & last 2 years
Give more wiggle room in the nomination/election timing (Sept-Nov
instead of Sept only)
• Specify appointments happen at start of year after newly elected
member terms begin

Looks good to me, thanks for doing this!

I'm +1 for merging these changes to the charter.

+1 from me.

+1 to merge, my feedback was addressed

Since we've all already voted to make these changes the charter, the resolution embedded in this PR should be 'approved' and not 'proposed'. @vmbrasseur do you want me to update the branch to relfect that?

@kpfleming I've become confused by the order of operations on this one. The resolution will need updating to reflect approval date, votes, etc. along with being moved to approved. Is the idea to do all that—updating the PR—and then merging it?

In that case we have a minor and quickly fixable problem. I'm working on filling in the vote results and the most approves I can puzzle out is four:

Me
KPF
Duane
Ashley

I can't find any record of +1/-1 from @decause, @jeffmcaffer, or @caniszczyk. Changes to the charter require ⅔ majority, so we need one more person to sign off on this.
In the meantime, I'll send over a mostly-complete update to this PR. Once we get one more +1, I can update just that line in the resolution and we can merge this.

@vmbrasseur @kpfleming This has been sitting around a while - is it ready to be merged?

@vmbrasseur @kpfleming This has been sitting around a while - is it ready to be merged?

Merge conflict with https://github.com/todogroup/governance/pull/104.
https://github.com/todogroup/governance/pull/158 should help with that.

This will have to taken up by the 2022 SC to get it completed.

Summary: This was only "approved", not APPROVED. One more vote was needed and that didn't happen before the end of my 2020 term.
The entire SC has changed since then, so I suspect this will need to start from scratch. We can talk about it on tomorrow's call if there's time. (I've added it to the agenda)

I think this is related to this new issue: https://github.com/todogroup/governance/issues/150

I think this is related to this new issue: #150

We will be using this issue now


Unforunately,  * has stopped. After invalid StyleSheet. From Getting Started

Description
Using Getting started Tutorial to reproduct an error I have with my current project.
When an error occurs during live reload, like having an invalid StyleSheet attribute, application crashes on my android phone as with emulator with "Unfortunately, myapp has stopped."
Reproduction Steps and Sample Code
Install blank react-native application. Setup Android SDK, start Emulator. Start packager and then react-native run-android.
When its running simply rename an StyleSheet value and save to reload application.
Solution
Error page should be shown and not leave the application.
Additional Information

React Native version: 0.42.3
Platform: Android
Development Operating System: Windows 10
Dev tools: VSC, Android SDK Tools 25.2.5 / Platform 23.4 (Emulator running @)

I can confirm the issue. Passing an invalid value to a stylesheet prop will cause a native crash, it would be nice to handle the case properly and show a helpful error message.
Repro:
const styles = StyleSheet.create({ marginTop: 'hello' });

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


wrap component inserts "Empty div" span above element being wrapped

small one, but a bit annoying.

Select an element in Hierarchy View
Select "wrap", and wrap in a div
Component is wrapped, but with an extra span above it

    Empty div
    <button>Button</button>

Got it. Will think how to avoid such an inconvenience.

Think I have solved in https://github.com/psimyn/structor/tree/wrap
in client/src/api/utils.js, I changed:
  case 'wrap':
    if (buffer && buffer.length > 0) {
+        options.children = buffer;
-        options.children.push(buffer[0]);
    }

This possibly will have terrible results, I haven't gone through fully but it seems to work..

Think I have solved in https://github.com/psimyn/structor/tree/wrap
in client/src/api/utils.js, I changed:
  case 'wrap':
    if (buffer && buffer.length > 0) {
+        options.children = buffer;
-        options.children.push(buffer[0]);
    }

This possibly will have terrible results, I haven't gone through fully but it seems to work..


destroying wheelchair and people disappear

if wheelchair destroyed from explosion(for example), player will disappear

just a guess but it might be using delete behavior instead of doing destruction act

fixed


Not implemented methods to get column data

We have some not implemented methods in DuckDBDataReader. We need to check whether DuckDb supports these data types and implement the methods for the supported type. Maybe we should throw something like NotSupportedException for the types DuckDB does not support.
These are the not implemented methods:
byte GetByte(int ordinal)
long GetBytes(int ordinal, long dataOffset, byte[] buffer, int bufferOffset, int length)
char GetChar(int ordinal)
long GetChars(int ordinal, long dataOffset, char[] buffer, int bufferOffset, int length)

What if we treat byte as char too?


adds :edited_message to @keys_of_message in Nadia.Parser

solves #131

LGTM
Thanks for your contribution!


add ETL support for optimus bundles

Revived dcp_zarr_store.py to workaround ! file separator convention and access DCP zarr structures.
Testing:

Transformed d65ef9e6-3fd1-4026-9843-ff573c9e66c8 integration bundle (included bundle in repo for unit testing)

Review by commit to reduce Files changed

Yup that makes sense


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 16:12:29 on 2023.04.10
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 6830
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=6830

Problem has been resolved in 3m 0s at 16:15:29 on 2023.04.10
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 6830


Make everything work perfectly

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

fixed everything, all is working with commit 9b8f57d


feat(ocm): add sorting to the ocm cluster table

Description
Adds sorting to the OCM cluster table
What issue(s) does this PR fix
Fixes #769

/cc @jerolimov

/retest

@Zaperex test job seems to be failing

Hmm it seems to be failing due to tests from the @janus-idp/backstage-plugin-web-terminal package at https://github.com/janus-idp/backstage-plugins/actions/runs/7494511967/job/20482135625#step:3:1257

@divyanshiGupta Not sure how to do that without destroying the commit history at the moment since I've merged from main a few times during the lifetime of this PR. Shouldn't there be a merge option in github that allows for a PR's commits to be squashed before it gets merged?


Option to omit spaces from yeartoyear output

Currently the yeartoyear function outputs in the format YYYY - YYYY. I would prefer it if it were YYYY-YYYY (no spaces), but the format is hardcoded. Would it be possible to add some sort of option to control the output format?

Hi Simon,
I am in the middle of moving house (to a different "country") but I should be up and running again early next week. I'll take a look and let you know what I can do.
Regards
David

Hello, is this feature available? If yes, how can I remove the spaces?

this has been released in v1.18.0


feat: Add contact page

Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[x] I made my own code-review before requesting one

Description of the changes
Add the contact page to the website.
Technical highlight/advice
Make the contact form bot-proof

I think that this PR is clearly over-scoped, why remove k8s files ?

I've readded them and removed them in another PR: https://github.com/lenra-io/website/pull/83

:tada: This PR is included in version 1.1.0-beta.6 :tada:
The release is available on:

GitHub release
v1.1.0-beta.6

Your semantic-release bot :package::rocket:


Correct filetype detection for bsdl

In PR #5945 it's added as "*.bsd", but in commit 0fc1288ae it got added
as "*bsd", without a dot.
I guess that's probably an error? So correct that, as it highlights
files named "freebsd", "openbsd", etc. wrong.

The test for this also uses the files "bsd" and "some-bsd".  It looks like the missing dot was intentional.
But it catches too many names.  Perhaps "bsd" would be OK and a file ending in "-bsd"?

Oh right; looks like the test was added in 10a1a534d1 later by someone else who added a bunch of filetype tests; I think they just "copied" the typo.
I tried looking up if these files are being used, but I can't really find all that much info about this BSDL thing beyond an ungodly amount of SEO linkspam, but as near as I can determine from the git history you "applied" the small patch by just editing the file, with this small typo.
I just changed the test now.
CC: @danielkho

CI failure is unrelated to this patch:

curl: (35) schannel: next InitializeSecurityContext failed: SEC_E_ILLEGAL_MESSAGE (0x80090326) - This error usually occurs when a fatal SSL/TLS alert is received (e.g. handshake failed). More detail may be available in the Windows System event log.

OK, if someone really wants "bsd" and "some-bsd" to be recognized as bsdl we'll await comments.


Append to FZF_DEFAULT_OPTS variable

base16-fzf should append to the FZF_DEFAULT_OPTS variable rather than overwriting it.

I believe you rebased on your own master, since your diff doesn't include some of the new themes. I'll merge this one and then add a commit to fix the diff.

Thanks for finishing up the missing files for me

Related fix for fish https://github.com/fnune/base16-fzf/commit/954e5cbe8a75beb32aa1c3750d76bcbc9d32cabb

@ayyess why was this change done? Why append rather than overwrite?


Upgrade Android gradle plugin to version 7.2.1

The blocking issue was resolved in https://github.com/GodotVR/godot_openxr/pull/239.
It was caused by a change in the output directory in AGP v7.1.

Cherry-picked for 3.6.

Cherry-picked for 3.5.2.


Directly run &shell when opening a new terminal

I have a prompt that show me the level of nested shells.
This is accomplished by taking the value of the environment variable $SHLVL
which is incremented at every new shell opened.
I noticed that when I used :terminal to open a terminal, the $SHLVL was 2
but when I used :ToggleTerminal, the $SHLVL was 3.
So I search and found the termopen() call with &shell instead of [&shell] which was
responsible of spawning bash in bash.

Good catch! Thanks :)


[stable/mongodb] Add chart label to template

Signed-off-by: Javier J. Salmeron Garcia<EMAIL_ADDRESS>
What this PR does / why we need it:
This fixes an issue in the Bitnami automation pipeline of the charts. Without this tag, the charts are not being tested
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
- fixes #
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

I tested the upgrade and found no issues

/ok-to-test

/lgtm


Cat get the config.ovpn into the container

Hey,
I can't get the .ovpn file from the host mounted. The mount should target /config right?
I alway get the error "erro: no vpn configuration file found".
The mounting works on other containers so it should work the same way in here as well.
Thankfull for any ideas.
Regards

For everyone with the same issue. Mounting on /data/vpn works

For future readers, this is the exact command that worked for me.
My config file was /home/azureuser/vpn/client.conf.
Mounting it to /data/vpn would be --volume='/home/azureuser/vpn:/data/vpn', and so the final command was:
docker run -it  --name=openvpn-client  --cap-add=NET_ADMIN  --device=/dev/net/tun  --volume='/home/azureuser/vpn:/data/vpn' ghcr.io/wfg/openvpn-client

I had the same error. And same result, mounting on /data/vpn works.
It would be great to have either Readme or the entry.sh updated.
My mount option FYI:
--mount type=bind,source=/usr/vpns/openvpn.ovpn,target=/data/vpn/openvpn.ovpn


Correct estimation when selling

Also add slippage to the estimation
And a tooltip explaining the slippage
Make sure output 0 is render when no amount is added
Use three decimals

Screenshots would be nice

Added just after your comment... hehe also on more commit

the set max button doens't work anymore

hmm any reproduce steps?
works for me:
https://user-images.githubusercontent.com/5688912/186847800-59bf37a4-2339-4f7b-914b-2f2e83dd3471.mov


Mismatching @jit signature falls back on object mode

This is a bit surprising and confusing:
>>> def f(x, y): return x+y
...
>>> jit(int32(int32,))(f)
CPUOverloaded(<function f at 0x7fafa3c3fbf8>)

Here, the signature is obviously wrong (one argument type is given for a two-arguments function), but Numba just goes ahead and falls back on object mode. It would be more helpful to raise an error.

Apparently this error is allowed in object mode... test_blackscholes relies on it. Do we want to support it?

I'm assuming that blacksholes relies on this behavior on accident due to a bug in the test as well.  We should fix the bug and raise a proper error when the signature doesn't match.


Not working in react-native-web

how can we use this react-router-native in react-native-web
i have observed uncaught TypeError: Cannot read property 'Transitioner' of undefined
but project is complied successfully

@ashanjayasundara I haven't tried yet, but could try including it in webpack, the the babel-loader config. Or create an alias, "react-router-native: "react-router". Can you be more specific with that you tried?


pyperclip must be installed globally?

Hi guys, I am doing automate the boring stuff v2.0, run into this problem with pyperclip.
I am running on python 3.8, as I always do, I have set up a venv, and pip install it but no module found, and then I tried by install globally after deactivate, and it works, anyone know why is that?
thanks

I am reading the same book, got over this problem when I checked appendix A. Tells you there to use the --target feature on pip (or admin window?). Works ok there. Pyperclip isn't in the automate-win-requirements.txt so has to be done seperately. Hope this helps, trying pyperclip on the webscraping chapter (webbrowser module) but doesn't seem to be pasting on windows, can you help me there?


Change name of column with link to file metadata on Publications Requiring Permissions Review on a File page

On QA right now, the Publications Requiring Permissions Review on a File page is a list with two columns -- one column has the Publication title with a link to edit Publication metadata. The other has the filename(s) and version(s) with link(s) to edit file metadata.
Let's change the name of the file metadata column to "File metadata: Filename (Version)" to make it clearer what's in that column.

@anaelizabethenriquez Preferred Version Review has the same thing. I assume that one should be updated as well?

@Smullz622 Yep! Thanks for catching that. Let's make the change there too.


Allow config for Electron Release Server to be read from envars

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[ ] The changes are appropriately documented (if applicable).
[x] The changes have sufficient test coverage (if applicable).
[x] The testsuite passes successfully on my local machine (if applicable).

Adds electronReleaseServer to the Forge Config template so configuration can be read from environment variables.

The commit message should be Angular-style if you want it to show up in CHANGELOG.md.

Oh right. Just rewrote it using the npm run commit command.

Thanks!