gamnit: add an example to showcase fonts support

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: implement new `TextSprites` settings in `BMFont`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: add customization settings to `TextSprites`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: fix actually using the attribute partial_line_skip

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: accept float values in `BMFont`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: intro `CustomTexture::fill`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: move `pixelated=` up to `Texture`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: intro custom textures, built programmatically pixel per pixel

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: if a texture fails to load, use the checker texture

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: CheckerTexture with custom size and move up implementation

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

Merge: gamnit: fix README and docker latest-full references to the use of SDL2 mixer

Update gamnit README and latest-full docker for #2466.

Pull-Request: #2481

Merge: lib/meta: add a *goret* approach to get the name of random types.

may be used to improve deserialization?

Pull-Request: #2482
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>

lib/meta: add a *goret* approach to get the name of random types.

may be used to improve deserialization?

Signed-off-by: Jean Privat <jean@pryen.org>

docker full: install missing package for SDL2 mixer

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: don't recommend to install a package that is not used anymore

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

Merge: nitin: use GPL-3.0 instead of GPL-3

the tag is used to fill the catalog, http://nitlanguage.org/catalog/p/nitin.html but the link is broken whereas https://opensource.org/licenses/GPL-3.0 works

Signed-off-by: Jean Privat <jean@pryen.org>

Pull-Request: #2476

Merge: listfull.sh: Quote paths

Make the list unambiguous and easy to consume by POSIX shell scripts.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2473
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: modelbuilder_base: Refactor the signature of `resolve_mtype*`

Only `modelize_mclass` require support for partial context. Furthermore, in
a future PR, it may need to resolve types just before modeling the class
definition (in a place where the `MClass` is available and must be used).

Also include various related cleanups.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2472

Merge: gamnit: Blinn-Phong specular lighting, 3 axes of rotation and more

Implement a Blinn-Phong specular lighting shader to replace the previous Lamber diffuse lighting. It creates a reflection effect either from `specular_color` (as on the shields) or `specular_texture` (as on the gold tower).


The required material attributes for the specular effect were already present but not implemented. This PR adds similar attributes for normal maps, but leave them as a TODO. The new shader is fine but not optimal, I'll probably need to rewrite it to gain some performance, support normal maps and improve normals per vertex.

Add two angles of rotation to actors for full 3D rotation with Euler angles. All rotation angles follow the right-hand rule and are consistent between the camera and actors.

Other changes:
* Intro `Cuboid` that can be created by `Boxed3d::to_mesh`.
* Better error management when loading models.
* Fix missing `lock_cursor` implementation on desktop. It allows creating FPS like cameras.
* Fix and update `angle_lerp` to follow the style of `lerp` (where `self` is the weight).
* Improve the template project so 3D objects don't blend in the background and explain the printed performance statistics.

In a future PR, I'll standardize the assets classes and move some service up to `Asset`. This would allow for easier loading of all assets at once, no matter their type, and offer a standard behavior regarding errors and lazy loading.

Pull-Request: #2471
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>

Merge: Wrapper gmp

gmp: Initial implementation of the GMP wrapper and simple test for it.

Pull-Request: #2460
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: lib/core: introduce replace_first in text/string_search

Replace the first occurrence of `pattern` with `string`

~~~nit
assert "hlelo".replace_first("le", "el") == "hello"
assert "hello".replace_first('l', "")    == "helo"
~~~

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Pull-Request: #2475

Merge: gamnit::network: add server discovery via UDP

Move server discovery logic from Tinks! up to `gamnit::network` and extend existing services with more options. These new services allow for an easy discovery of local servers over UDP, as in the following example which also creates a server if none is found:

~~~nit
import gamnit::network

# Discover local servers
var servers = discover_local_servers
if servers.not_empty then
# Try to connect to the first local server
var server_info = servers.first
var server = new RemoteServer(server_info)

if not server.connect then
print_error "Failed to connect to {server_info.address}:{server_info.port}"
else if not server.handshake then
print_error "Failed handshake with {server_info.address}:{server_info.port}"
else
# Connected!
print "Connected to {server_info.address}:{server_info.port}"

# Write something and close connection
server.writer.serialize "hello server"
server.socket.as(not null).close
end
else
# Create a local server
var connect_port = 38271
print "Launching server: connect on {connect_port}, discovery on {discovery_port}"
var server = new Server(connect_port)

# Don't loop if testing
if "NIT_TESTING".environ == "true" then exit 0

loop
# Respond to discovery requests
server.answer_discovery_requests

# Accept new clients
var new_clients = server.accept_clients
for client in new_clients do
# Read something and close connection
assert client.reader.deserialize == "hello server"
client.socket.close
end
end
end
~~~

Pull-Request: #2470
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>

Merge: parser_nodes: Fix spelling mistakes

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2468
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: modelize_class: Remove the `useless-bound` warning

With class subsets, there is no bound that is useless.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2467

Merge: popcorn: some useful services for JSON related REST API

This PR introduces a way to validate and deserialize requests bodies with some shortcuts.

I'm not a big fan of the `new_body_object` but I don't see how to do this elegantly...

~~~nit
class MyJsonHandler
  super Handler

  # Validator used do validate the body
  redef var validator = new MyFormValidator

  # Define the kind of objects expected by the deserialization process
  redef type BODY: MyForm

  # Instanciate the object from a deserializer
  redef fun new_body_object(d) do return new MyForm.from_deserializer(d)

  redef fun post(req, res) do
      var post = validate_body(req, res)
      if post == null then return # Validation error: let popcorn return a HTTP 400
      var form = deserialize_body(req, res)
      if form == null then return # Deserialization error: let popcorn return a HTTP 400

      # TODO do something with the input
      print form.name
  end
end

class MyForm
  serialize

  var name: String
end

class MyFormValidator
  super ObjectValidator

  init do
      add new StringField("name", min_size=1, max_size=255)
  end
end
~~~

Also, I moved json related services to this module and updated the clients.

Pull-Request: #2465
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: model: Remove truthful conditions

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2442
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>

Merge: Actors upgrade and fixes

`BlockingQueue` and `ReverseBlockingQueue` classes have been removed from  `lib/actors`  and are now in `lib/pthreads/concurrent_collections` because it's a better fit for them.
Also, since the implementation of `BlockingQueue` was too coupled with actors, the actor-specific implementation was kept in `lib/actors` and renamed as `Mailbox`.

`ReverseBlockingQueue` is a fun name and could be useful for some specific cases (as it was used in `lib/actors` or for synchronization on a list of dynamic tasks for example), but was too heavy for it's purpose in the `lib/actors` module. It's been replaced with a `SynchronizedCounter`. The `wait` method of this counter is blocking until the counter value equals 0.

There was multiple synchronization problems with the actor library :
* The creation of `async`, aka the proxy wasn't synchronized, which means we could create multiple instances of an actor if multiple threads called `async` on an object simultaneously
* Due to the asynchronous start of an actor (a new on a Nit Thread can return before the actual underlying C pthread started running), there was some cases where the mecanism used to wait for every actors to be idle before exiting a program was broken.

This PR adresses theses 2 bugs. The first one is just adding a mutex used for the creation of the `async` object.

The second one was adressed by moving the logic dealing with the idle state of an actor directly into his mailbox.

This PR also makes it possible to redefine an actor class. `lib/actors/agent_simulation` provides an example of this feature:
* `agent_simulation.nit` defines a simple api to which defines a Multi-agent simulation. The `ClockWorker` is in charge of the simulation and executes every steps, waiting for one to be finished before launching the other. the list of `Agent` it manages are all the agents of the simulation, which are actors too.
* `simple_simulation.nit` redefine the `Agent` class to add specific services for a specific simulation. Here, each agent knows every other one in the simluation and greets them, wait to be greeted back by everyone, then signals the clock that it finished the step.

This feature can be useful, but hasn't been fully tested yet. Furthermore, the Nit actor model still doesn't support subclassing an actor, which could also be really useful.

Well, that's pretty much it, enjoy reading :p

Pull-Request: #2425
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>
Reviewed-by: Jean Privat <jean@pryen.org>

nitin: use GPL-3.0 instead of GPL-3

because https://opensource.org/licenses/GPL-3.0 works
whereas it is broken without the .0

Signed-off-by: Jean Privat <jean@pryen.org>

lib/core: introduce replace_first in text/string_search

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

tinks, model_viewer & action_nitro: print model loading errors and fix one

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: collect errors when loading models and its textures

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: don't load the same .mat material more than once

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: replace the Lambert shader by a Blinn-Phong shader

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

geometry: rework `Float::angle_lerp` in the style of `Float::lerp`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: 3 Euler rotation angles for actors

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

search_tests.sh: Unquote results

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

search_tests.sh: Update to the new format of `listfull.sh`

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

listfull.sh: Quote paths

Make the list unambiguous and easy to consume by POSIX shell scripts.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

testfull.sh: Use `xargs`

That way, `listfull.sh` can quote the paths

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

modelbuilder_base: Refactor the signature of `resolve_mtype*`

Only `modelize_mclass` require support for partial context. Furthermore, in
a future PR, it may need to resolve types just before modeling the class
definition (in a place where the `MClass` is available and must be used).

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

modelbuilder_base: Rename some local variables

Avoid a name conflict in future commits.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

typing: Make `TypeVisitor::mmodule` implicit

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

typing: Make `TypeVisitor::anchor` non-nullable

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

lib/popcorn: import pop_json when needed

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/popcorn: move json related services to `pop_json`

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/popcorn: add useful services for JSON REST handlers

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

typing: Make `TypeVisitor::mclassdef` non-nullable

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

typing: Make `TypeVisitor::mpropdef` non-nullable

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

typing: Remove duplicate assignment to `TypeVisitor::mpropdef`

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

modelize_class: Fix spacing

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

gmp: Introduce NativeMPZ and NativeMPQ with simple test

Signed-off-by: PatrickBlanchette <PatrickBlanchette@users.noreply.github.com>

gamnit: extract Cuboid as super class to cube and intro Boxed3d::to_mesh

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: differentiate background from default material and explain perf stats

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: implement `lock_cursor` using SDL2 `relative_mouse_mode`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

Introducing agent simulation: an actor redefinition example

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

Generate appropriate code for actors redefinitions

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

use `SynchronizedCounter` in chameneos-redux

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

Fix synchronizations problem to better track active actors

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

Move `BlockingQueue` and `ReverseBlockingQueue` to `concurrent_collection`

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

gamnit: add server discovery or create example

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: add more options to `accept_clients` & `broadcast`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: tweak network doc

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: move up UDP discovery logic from Tinks! to the lib

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

parser_nodes: Fix spelling mistakes

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

modelize_class: Remove the `useless-bound` warning

With class subsets, there is no bound that is useless.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Merge: geometry: implement 3D `dist` and improve API

The main change in this PR is the reimplementation of `Point::dist` and `dist2` to support 3D points, including comparing 2D and 3D points. The double dispatch logic is a bit complex but it works with any receiver.

This PR also tweaks existing services to improve the API and doc:
* Make all attributes of `Point` optional for gradual creation/space allocation. In practice, points are often created with a default value, as a kind of buffer, or with the Z axis at 0.
* Rework `BoxedArray` as a subclass of `Array` to inherit its services (incl. `remove`) while using less code.
* Move the custom `to_s` from the concrete boxes to the interfaces as it is very useful when debugging.

Pull-Request: #2462

Merge: lib/popcorn: introduce `AppConfig::app_hostname`

Introduce `app_hostname` in popcorn AppConfig.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Pull-Request: #2464

Merge: gamnit & sdl2: implement app::audio using SDL2 mixer

Intro a decent implementation of `app::audio` for desktop computers using SDL2 mixer. It supports WAV, MP3, OGG and more. The underlying SDL2 mixer bindings are minimal, they offer only the required features, more can be added as needed.

Clients of the gamnit framework will need to `apt-get install libsdl2-mixer-dev`.

Pull-Request: #2466

geometry: implement `Point::dist` for 3D points

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

geometry: make `BoxedArray` a subclass of `Array`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

geometry: make the attributes of points optional

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

geometry: move up custom `to_s` to the interfaces

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: implement app::audio for desktop using SDL2 mixer

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

sdl2: intro minimal bindings to SDL2 mixer

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

audio: move some services between the abstraction and the Android layers

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/popcorn: introduce `AppConfig::app_hostname`

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: Popcorn templates

# Template rendering for popcorn

## Basic templates

Use TemplateString to render really basic templates that just need macro
replacements.

Example:
~~~nit
class TemplateStringHandler
        super Handler
redef fun get(req, res) do
# Values to add in the template
var values = new HashMap[String, String]
values["USER"] = "Morriar"
values["MYSITE"] = "My super website"

# Render it with a shortcut
res.template_string("""
<h1>Hello %USER%!</h1>
<p>Welcome to %MYSITE%.</p>
""", values)
end
 end
 ~~~

 For larger templates, you can also use external files (makes your Nit code cleaner):
 ~~~nit
 class TemplateFileHandler
super Handler

redef fun get(req, res) do
# Values to add in the template
var values = new HashMap[String, String]
values["USER"] = "Morriar"
values["MYSITE"] = "My super website"

# Render it from an external file
res.template_file("example_template.tpl", values)
end
 end
 ~~~

 ## Using pug templates

 Pug is a templating format provided by the external command `pug`.
 For complex templates that need conditional or loop statements, pug can be a solution.

 See the pug syntax here: https://pugjs.org/api/getting-started.html

 ~~~nit
 class PugFileHandler
super Handler

redef fun get(req, res) do
# Values to add in the template
var json = new JsonObject
json["user"] = "Morriar"
json["mysite"] = "My super website"

# Render it from an external file
res.pug_file("example_template.pug", json)
end
 end
 ~~~

Pull-Request: #2449
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: coloring: Only generate strictly positive IDs

This will allow to reserve negative IDs (including 0) for special purposes.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2457
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: model: Do not assume enums to be final

An upcoming feature will destroy this assumption.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2458

Merge: gamnit: intro support for Angel Code bitmap font

The BMFont format supports packed textures, varying advance per character and even kernings. It can be generated by a number of tools, including:
* BMFont, free software Windows app, http://www.angelcode.com/products/bmfont/
* Littera, a web app, http://kvazars.com/littera/

This PR also generalizes the `Font` and `TextSprites` concepts, previously limited to tile set bitmap fonts. And add a new shortcut service to the excellent DOM parser used to parse the XML description.

The result is much nicer than what was previously available, you can see it in a preview of my latest project:

The parsing method crash easily on malformatted font description file. However, since the description file should be generated, loaded at game start and not often modified, all expected crash errors should occur in dev only. Errors that can happen in production, like a truncated file or a misplaced file, should be detected by the basic sanity checks.

If an error occurs while loading a font, there is currently no automatic fallback and it will probably lead to an unusable game. It could be remedied by adding a default font, similar to the default material and mesh. It would be very useful, even at the beginning of any projects, but there is some technical limitations to implementing it as of now.

In the future, I plan on moving up more services from `TileSetFont` and add text alignment features. The `flat` module needs to be split in a group to include more services, including fonts.

Pull-Request: #2459
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

gamnit: intro BMFont parser and drawing services

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

gamnit: extract general Font from TileSetFont

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

dom: intro `XMLTag::attributes_to_map` mapping `XMLStringAttr` to their value

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

tests: Update error messages

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

model: Do not assume enums to be final

An upcoming feature will destroy this assumption.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

tests: Do not rely on the implicit fixing of type bounds

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Merge: model: Fix minor documentation mistakes

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2451

Merge: nitvm: Factorize the loading of superclasses

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2452

Merge: parser_nodes: Fix documentation

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2453

Merge: rosettacode: Fix spelling mistakes

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2454

Merge: rapid_type_analysis: Fix a spelling mistake

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2455

dom: return a more precise `Array[XMLTag]` as the doc already said

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

metrics_base: Do not rely on the implicit fixing of type bounds

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

opts: Do not rely on the implicit fixing of type bounds

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

coloring: Only generate strictly positive IDs

This will allow to reserve negative IDs (including 0) for special purposes.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

coloring: Fix documentation formatting

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

rapid_type_analysis: Fix a spelling mistake

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

rosettacode: Fix spelling mistakes

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

parser_nodes: Fix documentation

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

model: Fix minor documentation mistakes

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>