Log packing failures

Description

[x] Write error blob to file if Packmol fails (just throw it in working path - simpler than adding a new argument for where to write this to)

Checklist

[x] Add tests
[x] Lint
[ ] Update docstrings

Here's the UX for a straightforward case, stripping down #1062 a bit
In [1]: from openff.toolkit import Molecule, Topology, Quantity
   ...: from openff.interchange.components._packmol import pack_box, UNIT_CUBE
   ...:
   ...:
   ...: solvent = Molecule.from_smiles('CCCCO')
   ...: solvent.generate_conformers()
   ...:
   ...: ligand = Molecule.from_smiles('CCCCCCCC')
   ...: ligand.generate_conformers()
   ...:
   ...: pack_box(
   ...:     molecules=[solvent],
   ...:     number_of_copies=[100],
   ...:     solute=ligand.to_topology(),
   ...:     tolerance=Quantity("1.0 angstrom"),
   ...:     mass_density=Quantity("100 * gram/milliliter"),
   ...:     box_shape=UNIT_CUBE,
   ...:     center_solute=True,
   ...:     working_directory='.',
   ...:     retain_working_files=False,
   ...: )
   ...:
---------------------------------------------------------------------------
CalledProcessError                        Traceback (most recent call last)
File ~/software/openff-interchange/openff/interchange/components/_packmol.py:705, in pack_box(molecules, number_of_copies, solute, tolerance, box_vectors, mass_density, box_shape, center_solute, working_directory, retain_working_files)
    704 try:
--> 705     result = subprocess.check_output(
    706         packmol_path,
    707         stdin=file_handle,
    708         stderr=subprocess.STDOUT,
    709     )
    710 except subprocess.CalledProcessError as error:
    711     # Custom error codes seem to start at 170
    712     # https://github.com/m3g/packmol/blob/v20.15.1/src/exit_codes.f90#L13-L16

File ~/micromamba/envs/new-models/lib/python3.11/subprocess.py:465, in check_output(timeout, *popenargs, **kwargs)
    463     kwargs['input'] = empty
--> 465 return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
    466            **kwargs).stdout

File ~/micromamba/envs/new-models/lib/python3.11/subprocess.py:569, in run(input, capture_output, timeout, check, *popenargs, **kwargs)
    568     if check and retcode:
--> 569         raise CalledProcessError(retcode, process.args,
    570                                  output=stdout, stderr=stderr)
    571 return CompletedProcess(process.args, retcode, stdout, stderr)

CalledProcessError: Command '/Users/mattthompson/micromamba/envs/new-models/bin/packmol' returned non-zero exit status 173.

The above exception was the direct cause of the following exception:

PACKMOLRuntimeError                       Traceback (most recent call last)
Cell In[1], line 11
      8 ligand = Molecule.from_smiles('CCCCCCCC')
      9 ligand.generate_conformers()
---> 11 pack_box(
     12     molecules=[solvent],
     13     number_of_copies=[100],
     14     solute=ligand.to_topology(),
     15     tolerance=Quantity("1.0 angstrom"),
     16     mass_density=Quantity("100 * gram/milliliter"),
     17     box_shape=UNIT_CUBE,
     18     center_solute=True,
     19     working_directory='.',
     20     retain_working_files=False,
     21 )

File ~/micromamba/envs/new-models/lib/python3.11/site-packages/openff/utilities/utilities.py:80, in requires_package.<locals>.inner_decorator.<locals>.wrapper(*args, **kwargs)
     77 except Exception as e:
     78     raise e
---> 80 return function(*args, **kwargs)

File ~/software/openff-interchange/openff/interchange/components/_packmol.py:715, in pack_box(molecules, number_of_copies, solute, tolerance, box_vectors, mass_density, box_shape, center_solute, working_directory, retain_working_files)
    710     except subprocess.CalledProcessError as error:
    711         # Custom error codes seem to start at 170
    712         # https://github.com/m3g/packmol/blob/v20.15.1/src/exit_codes.f90#L13-L16
    713         open("packmol_error.log", "w").write(error.stdout.decode("utf-8"))
--> 715         raise PACKMOLRuntimeError(
    716             f"PACKMOL failed with error code {error.returncode}. Wrote file packmol_error.log in working "
    717             "directory, which might be a temporary directory. Set the argument `working_directory` to "
    718             "point this to a persistent path.",
    719         ) from error
    721     packmol_succeeded = result.decode("utf-8").find("Success!") > 0
    723 if not packmol_succeeded:

PACKMOLRuntimeError: PACKMOL failed with error code 173. Wrote file packmol_error.log in working directory, which might be a temporary directory. Set the argument `working_directory` to point this to a persistent path.

In [2]: !tail -n 100 packmol_error.log
  Moving worst molecules ...
  Function value before moving molecules:   39.466416204891054
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.466416204786036
  Packing:|0                                                             100%|
          |******************************************************************|
          |******************************************************************|

  Fixing bad orientations ...     35 of     40
  Moving worst molecules ...
  Function value before moving molecules:   39.466416204786036
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.466416204724517
  Packing:|0                                                             100%|
          |******************************************************************|
          |******************************************************************|

  Fixing bad orientations ...     36 of     40
  Moving worst molecules ...
  Function value before moving molecules:   39.466416204724517
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.466416204627130
  Packing:|0                                                             100%|
          |******************************************************************|
          |******************************************************************|

  Fixing bad orientations ...     37 of     40
  Moving worst molecules ...
  Function value before moving molecules:   39.466416204627130
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.466416204593855
  Packing:|0                                                             100%|
          |******************************************************************|
          |******************************************************************|

  Fixing bad orientations ...     38 of     40
  Moving worst molecules ...
  Function value before moving molecules:   39.466416204593855
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.466416204587837
  Packing:|0                                                             100%|
          |******************************************************************|
          |******************************************************************|

  Fixing bad orientations ...     39 of     40
  Moving worst molecules ...
  Function value before moving molecules:   39.466416204587837
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.467380674970428
  Packing:|0                                                             100%|
          |******************************************************************|
          |******************************************************************|
  Fixing bad orientations ...     40 of     40
  Moving worst molecules ...
  Function value before moving molecules:   39.466416265153697
  Type         1 molecules with non-zero contributions:  100.00%
  Moving         5 molecules of type         1
  New positions will be based on good molecules (movebadrandom is not set)
   Moving:|0                                                             100%|
          |******************************************************************|
  Function value after moving molecules:   39.466416258718127

  Restraint-only function value:    39.466416258718127
  Maximum violation of the restraints:   0.18227051192478103
  ERROR: Packmol was unable to put the molecules
         in the desired regions even without
         considering distance tolerances.
         Probably there is something wrong with
         the constraints, since it seems that
         the molecules cannot satisfy them at
         at all.
         Please check the spatial constraints and
         try again.
  >The maximum number of cycles (          40 ) was achieved.
   You may try increasing it with the nloop0 keyword, as in: nloop0 1000
STOP 173

Thanks for the review @ntBre!


Enhancement assessment: VK_PRIMITIVE_TOPOLOGY_TRIANGLE_FAN

Metal does not support triangle fans (VK_PRIMITIVE_TOPOLOGY_TRIANGLE_FAN). This is reported accurately by MoltenVK as part of VkPhysicalDevicePortabilitySubsetFeaturesKHR in the VK_KHR_portability_subset extension.
However, some apps, including many CTS tests, continue to need triangle fans, which opens the question of whether MoltenVK should attempt to support triangle fans, regardless of Metal support.
Triangle fans can be treated as triangle strips, with a different vertex order. Specifically, a triangle fan can be converted to a triangle strip by reorganizing the vertex indexing using a standard algorithm:
    if( vtxIdx % 2 == 0 )
        vtxIdx = vtxIdx / 2;
    else
        vtxIdx = vtxCnt - 1 - vtxIdx / 2;

There are a couple of options about where and how this remapping should be done. I've done some initial research into options within MoltenVK, and present them here for discussion. There are no doubt other options, and I may be over-or-under-estimating the complexity of each option in my initial review.
Remap Vertices or Vertex Indices Prior to Draw
In this first option, since the vertex data can be updated in the vertex buffer anytime before a draw call is encoded, including via other GPU activity, the remapping of the vertices would need to be done via copying the vertices or vertex indices into a temporary MTLBuffer using a Compute Shader encoded just before the draw command.
We could add a kMVKGraphicsStageVertexRemap to MVKGraphicsStage enumeration, and include it in the results of MVKGraphicsPipeline::getStages() when the pipeline requires triangle fan topology. The various MVKCmdDraw*::encode() functions could then incorporate this into their stage management operations, similarly to how they incorporate Compute Shaders as part of tessellation operations.
The scheduling of the kMVKGraphicsStageVertexRemap would need to take into consideration both tessellated and non-tessellated pipeline handling, and the passing of the temporary vertex buffer in place of the original buffer. This could be quite a complex modification, given the complexity of the existing MVKCmdDraw*::encode() functions.
Remap Each Vertex in the vertex Shader
In this second option, the MSL of the vertex shader could be modified to perform the above remapping on each vertex, by modifying the effective value of glVertexID before grabbing the vertex data from the array of vertices.
Currently, both MoltenVK and SPIRV-Cross support directly accessing the vertex as an [[ attribute(n) ]] in the [[ stage_in ]] vertex attribute structure, and do not support random indexing into the vertex array. For this remapping option to be available, both MoltenVK and SPIRV-Cross would require significant modifications to their handling of vertex attributes. This is made more complex by certain vertex attribute modifications within MoltenVK to work around differences in vertex attribute offset and stride handling between Vulkan and MoltenVK.
@cdavis5e @HansKristian-Work I would appreciate any thoughts from you.

Hey guys.  any update on this one?  Is triangle support coming for moltenvk?

Hi Blender developer responsible for the Vulkan port.
Although our use case is compatibility/maintainability. Blender already has support for OpenGL and Metal. Our user base can have create scripts using Python that will draw overlays onto any editor. There are massive amounts of these scripts and out of our hand. And it isn't realistic to drop this feature.
Our current Metal Backend already has a work-around for triangle fans. Although we could port this one over to Vulkan and only activate it when MoltenVK reports that it isn't available; this could become a maintenance issue in the longer run as this code-path might not be tested. Regular developers and all users would use Metal, gpu developers would want to use Apple hardware as it is the only platform that can run and perform compile time validation on all supported 3 backends (OpenGL/Vulkan/Metal).
We don't think it is a show stopper, but it could be that we would not support MoltenVK on Apple devices as more work-arounds are needed to get the same support level. Adding this to MoltenVK would ease cross platform support.
On the other hand, what would be the effect when 'all applications' consider to remove using triangle fans. Vulkan drivers/sillicon would still need to support it as it is part of the Vulkan spec and we might have a soft-deprecated part of the API.

@jeroenbakker-atmind
Thanks for your recommendation, and for the background on Blender's interest in this.
Don't bother adding a special path in your Vulkan-MoltenVK backend. Triangle-fans are now fairly high on our priority list, and we will be adding support fairly soon.
Since you stress consistency, if you care to share how Blender handles it in your Metal backend, we can strive to be consistent with that functionality.
BTW...we were unaware that Blender supports MoltenVK as part of the backend options. Do you bundle MoltenVK with Blender on macOS, or do you rely on the Vulkan SDK to manage installing and finding MoltenVK, as it finds any Vulkan driver on any other platform?

That is good news.
Quick overview of Blender GPU backend architecture: Due our long existence and different versions of GPU backends we have developed our own GPU abstraction layer called gpu module (https://projects.blender.org/blender/blender/src/branch/main/source/blender/gpu). All GPU commands pass this module one way or another. This module is also accessible to add-on developers. The GPU has different backends for OpenGL/Metal/Vulkan. At the start of Blender one backend is loaded. The active backend can be chosen in the preferences. As our origin is OpenGL some OpenGL features had to be implemented on both Metal/Vulkan for example we track state a globally for bindings and they are bound to the shader just before the drawing/dispatch command is sent to the active backend.
In our backend the topology primitive is part of the index buffer data. When the metal backend detects unsupported configurations it converts the index buffer to a new GPU buffer https://projects.blender.org/blender/blender/src/branch/main/source/blender/gpu/metal/mtl_index_buffer.mm#L307. As input data is CPU bound it can happen on the fly when uploading it to GPU bound memory.
The triangle fan is just the first issue we detected when starting Blender using MoltenVK. Fun fact: The initial development of the Vulkan Backend has been conducted using MoltenVK using test cases.
Our current implementation is to distribute MoltenVK as an internal library. It hasn't been decided if this is the best option and would be available to end users. Changes can happen to policy and implementation before we do an initial release (ETA is to be available for early adaptors in Blender 4.1).

PR # #1962 implements triangle fans.


What is the right way to get the full path of the request.

I am using websockets to run a server which caters to clients. The way it is setup, I run multiple instances of the code behind a load-balancer to allow for more HA. When a message is received from the client, I would like to get the entire path of the HTTP request that was used to create the connection. For example, if the client was connecting to the websocket using the URL : "wss://stream-vaibhav.com/ws", then I would like to get the full URL "stream-vaibhav.com/ws" in some way.
How do I get this information? Is there a field in WebSocketServerProtocol that can help me get the entire path?
Would be good to add this to the documentation as well.

Hello,
Have you tried the host header of handshake? You can access to GET handshake request's headers in your handler :
async def handler(proto, path):
    print(proto.request_headers.get("host", None)
    ...

Either it works like @AmirHmZz said, or you can configure your load balancer to add the original Host in the non-standard-but-widely-used X-Forwarded-Host header.


[URGENT] New update broke a part of the transportation function

Bug explanation
Self-explanatory, the new game update changed the transport elements so it is not functional.
Operating System
N/A
Ikabot installation
I downloaded the pre-built Windows binary
Ikabot version
Latest
Python version
No response

After further testing, I noticed that the issue is only affecting the sellResources and buyResources functions which use the transport function.

The issue should be fixed now that the two PR that are referenced have been merged.


Update sysctl.sh

restart procps service
defects
on some situations, after enable ip forward and sysctl -p commands,  the network is broken, so the next steps, such as npm install will exit abnormally.
solution
at Ubuntu 14.04, restart procps service can fix this problem

thank you :)


Search doesn't allow to search for anything other than Tracks

The combobox doesn't allow choosing anything else. Trying to show available options shows nothing and "Tracks" is selected by default.

You need to:

tap "Tracks"  so icon on the right will be a tick ( instead of arrow down )
swipe from "Tracks" to top until you'll find option you need

I don't know whether they changed this control or what. It was very non-obvious to me too.

This is true for the version in Ubuntu Store (1.4.3) but it does not work for the version in Open Store (1.6.1). At least in a BQ Aquaris M10 FHD Ubuntu Edition. Anyway, it would benice if the combobox expanded and showed all options.


fix(markdown): preserve underscores in image URLs during markdown export

Solved the problem in this issue(#97)
Previously, all underscores in markdown output were escaped to prevent emphasis styling, which broke image URLs containing underscores. This fix modifies the underscore escaping logic to:

Identify image URL patterns in markdown text
Skip underscore escaping within image URL sections
Continue escaping underscores in regular text content

LGTM


DataObject->write() without changes does not update LastEdited in the Database, but does on the instance

write() updates LastEdited on the object itself, but only writes it to database if the DataObject has changes.
$foo = Foo::get()->first();
echo $foo->LastEdited; // 2014-11-14 18:46:14
$foo->write();
echo $foo->LastEdited; // 2014-11-14 22:00:32
$foo = Foo::get()->first();
echo $foo->LastEdited; // 2014-11-14 18:46:14

@tractorcow Could not confirm this as of ff562e1.
$foo->write(); updates the db record despite nothing had changed.

It probably SHOULDN'T update the database record if nothing has changed. That means there's a new bug, or perhaps this isn't solved. :)

@tractorcow I see your point, but I can't manage myself to agree with it.
write() is self-explanatory and purposes an explicit action; write, is write, and not write if.
Only writing when something as changed should be a best practice and not an imposition.
One could use:
if ($foo->isChanged())
    $foo->write();

Or explicitly
$foo->write();

Please, try and help me change my mind if I'm really wrong about this.

Well the phpdoc for $forceWrite is as below
/**
* @param boolean $forceWrite Write to database even if there are no changes
*/

It could be documented a bit more explicitly that this "writes any changes, if necessary, to the database"

Just having a look at the code, in 3.1 at least the bug where LastEdited is updated without a write (possible) is still there. It is weird code though...
In 3.2 the code is there, but since the function was cleaned up (and thus no longer as weird) it should be easy to fix.

I see.

@tractorcow 3.2 is on the 3 branch, I'm I right?

@pedro2555 that's right

Closing as this isn't a bug.

I am a bit confused, it is in fact a bug as discussed previously.
But I think closing it is still valid though, because in 3.6 it no longer exists, so I presume it has been fixed.
It would appear now that a ->write() will now always write, regardless of the changed fields.
   $foo = Page::get()->first();
echo $foo->LastEdited; // 2017-05-11 23:48:04
   $foo->write();
echo $foo->LastEdited; // 2017-05-11 23:51:58
   $foo = Page::get()->first();
echo $foo->LastEdited; // 2017-05-11 23:51:58
so I am happy now. but felt like I had to point it out anyway.

That feels bad to me because the record is reporting a last edit time when the record wasn't actually edited... if the record remains the same, it's not edited. :(

I re-read the discussion, sorry, and have re-opened the issue.
I've renamed the issue slightly ;)

Should have stayed quiet and enjoy my victory :D

@tractorcow does this mean 2 write calls after one another wil result in a DB update?

Ah, it could possibly. I'd need to test.

Fixed with https://github.com/silverstripe/silverstripe-framework/pull/7288


Fix bug when validating custom types

I was interested in looking at https://github.com/dry-rb/dry-validation/issues/103 - I've managed to get a passing spec (without breaking the build) but I can't help feeling that that this isn't the actual solution. I'd love to seeing another failing spec.

I pushed the actual solution :)

@solnic I had that as my original solution but was able to get the tests to pass using DEFAULT_TYPE_NODE.first - I'm guessing the changes you made to the test exposed the need for type.rule.to_ast.

@coop yes they did because otherwise everything would be treated as a string so no coercions would kick in


CMSIS-DSP fill flash with tables not in actual use

I'm working with a chip that has very limited space: 32KB Flash, 4KB SRAM.  When doing FFT on q15_t types, I can't finish linking because of excess constant data.  Initially I used 256 samples, but tried 128 and even 64 for identical outcomes.
Inspecting the .map file there's a very clear reason why this is the case:
twiddleCoef_1024_q15         0x8e6c   0xc00  Data  Gb  CommonTables.o [3]
twiddleCoef_128_q15          0xc6fc   0x180  Data  Gb  CommonTables.o [3]
twiddleCoef_16_q15           0xcba8    0x30  Data  Gb  CommonTables.o [3]
twiddleCoef_2048_q15         0x4f80  0x1800  Data  Gb  CommonTables.o [3]
twiddleCoef_256_q15          0xb2f4   0x300  Data  Gb  CommonTables.o [3]
twiddleCoef_32_q15           0xcb0c    0x60  Data  Gb  CommonTables.o [3]
twiddleCoef_4096_q15            0x0  0x3000  Data  Gb  CommonTables.o [3]
twiddleCoef_512_q15          0xa22c   0x600  Data  Gb  CommonTables.o [3]
twiddleCoef_64_q15           0xc95c    0xc0  Data  Gb  CommonTables.o [3]

A single one of these twiddle coefficient tables alone is eating 12KB of Flash space by itself.  CommonTables.o in aggregate is taking up 40KB.  Of 32.
    CommonTables.o                    40'664
    ComplexMathFunctions.o       64
    FastMathFunctions.o         384
    TransformFunctions.o      5'994
    -------------------------------------------------
    Total:                    6'442   40'664

Looking into CommonTables.c (and its included files) I don't see any way to pare back the contents.  Despite me never using anything over 256 samples in FFT operations, my map file is full of references to as high as 4096.  For example:
armBitRevIndexTable_fixed_1024
                             0x9a6c   0x7c0  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_128
                             0xc87c    0xe0  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_16
                             0xce20    0x18  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_2048
                             0x7eec   0xf80  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_256
                             0xc51c   0x1e0  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_32
                             0xcbd8    0x30  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_4096
                             0x3000  0x1f80  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_512
                             0xaf34   0x3c0  Data  Gb  CommonTables.o [3]
armBitRevIndexTable_fixed_64
                             0xca9c    0x70  Data  Gb  CommonTables.o [3]

And there doesn't appear to be any way to turn this all off.
What am I missing?  (Or, rather, perhaps more accurately, what is the documentation missing?)

@ACleverDisguise You need to avoid using initialization functions like arm_cfft_init_f32 and instead use dedicated versions for the FFT lengths you need. For instance arm_cfft_init_512_f32
There are similar functions for other data types, and other transforms (rfft ...)
Like that, the linker will be able to deduce the table that are useless and they won't be included in the final build.
Those functions are documented in the doxygen generated documentation.
There is also a new section in the README : https://github.com/ARM-software/CMSIS-DSP?tab=readme-ov-file#code-size
(The required build and link options that are compiler dependent must also be used : like -ffunction-sections, -fdata-sections, -Wl,--gc-sections ...)

IAR and I have no idea: the IDE does its level best to conceal that from me.  I'll go digging to see what command line it comes up with.

Ah, never mind.  A third-party (as in another coworker's) file had a call for the general init and I wasn't expecting it to have any calls to the math lib at all, not to mention FFT.
Stripping that out and replacing it with the appropriately-sized init (64 in this case) solved our problem.  Thanks, @christophe0606!

Problem solved, but this really needs to be documented in the actual library docs, not buried in a README.  Like perhaps arm_cfft_init_<type>()'s documentation should mention the sized variants?
Or optionally initialization could be a macro call that parametrizes by size and type like this: ARM_CFFT_INIT(&my_init_struct, q15, 256) and have that assemble the call to arm_cfft_init_256_q15(&my_init_struct)?
A macro I've (loosely) tested that does this is below:
#define ARM_CFFT_INIT_IMPL(S, T, N) arm_cfft_init ##N ##T(S)
#define ARM_CFFT_INIT_PREP(S, T, N) ARM_CFFT_INIT_IMPL(S, _ ##T, _ ##N)
#define ARM_CFFT_INIT(S, T, N) ARM_CFFT_INIT_PREP(S, T, N)

This permits me to write code like this:
#define DATA_TYPE q15
#define SAMPLE_SIZE 256
ARM_CFFT_INIT(&my_struct, DATA_TYPE, SAMPLE_SIZE)

And if I decide I want to expand or contract my sample size, I change it in one place and all uses are updated.  Similarly if I decide I want to move to f32, I change it in one place and it gets updated everywhere.

@ACleverDisguise We will upgrade the documentation so that it is more visible.
The macro is a good suggestion. Thank you.

I've written now about a dozen such macros for the features I use.  They've really helped in rapid changes while testing and prototyping.  I'm sure if I spent actual time on it, I'd make macros that sucked slightly less.  ;)


Gatling scenario returning before completion - All users of scenario custom are stopped

I am doing a prototype using the Custom RPC scenario on karate-gatling (0.9.6) ,Gatling Gradle Plugin (gradle.plugin.io.gatling.gradle:gatling-gradle-plugin:<IP_ADDRESS>) by setting the gatling version to (gatlingVersion=3.0.1 in the plugin configuration , due to the scala version restriction in my project .My project is on scala 2.12.12 , so chose these gatling and the karate-gatling library version. The program flow is simulation calling karate feature that calls my java utility that calls my test software (this makes a rest service get call to a server. When running my karate feature independently it is able to call my project and provide valid results. However when i invoke this feature from the gatling simulation(following the custom-rpc sample code) , my simulations complete but without executing all my java code. It seems to get stuck at the place where the REST call is being made and returns without waiting for a response from the server.
With logs enabled
These are the logs i get
023-02-10T14:29:03.427+0530 [QUIET] [system.out] 14:29:03.427 [GatlingSystem-akka.actor.default-dispatcher-46] INFO i.g.core.controller.inject.Injector - All users of scenario custom are stopped
2023-02-10T14:29:03.427+0530 [QUIET] [system.out] 14:29:03.427 [GatlingSystem-akka.actor.default-dispatcher-46] INFO i.g.core.controller.inject.Injector - Stopping
2023-02-10T14:29:03.429+0530 [QUIET] [system.out] 14:29:03.428 [GatlingSystem-akka.actor.default-dispatcher-49] INFO i.gatling.core.controller.Controller - Injector has stopped, initiating graceful stop
2023-02-10T14:29:03.450+0530 [QUIET] [system.out]
2023-02-10T14:29:03.450+0530 [QUIET] [system.out] ================================================================================
2023-02-10T14:29:03.450+0530 [QUIET] [system.out] 2023-02-10 14:29:03 4s elapsed
2023-02-10T14:29:03.450+0530 [QUIET] [system.out] ---- Requests ------------------------------------------------------------------
2023-02-10T14:29:03.450+0530 [QUIET] [system.out] > Global (OK=0 KO=0 )
2023-02-10T14:29:03.450+0530 [QUIET] [system.out]
2023-02-10T14:29:03.450+0530 [QUIET] [system.out]
2023-02-10T14:29:03.450+0530 [QUIET] [system.out] ---- custom --------------------------------------------------------------------
2023-02-10T14:29:03.451+0530 [QUIET] [system.out] [##########################################################################]100%
2023-02-10T14:29:03.451+0530 [QUIET] [system.out] waiting: 0 / active: 0 / done: 1
2023-02-10T14:29:03.451+0530 [QUIET] [system.out] ================================================================================
2023-02-10T14:29:03.451+0530 [QUIET] [system.out]
2023-02-10T14:29:03.452+0530 [QUIET] [system.out] 14:29:03.451 [GatlingSystem-akka.actor.default-dispatcher-54] INFO i.gatling.core.controller.Controller - StatsEngineStopped
2023-02-10T14:29:03.452+0530 [QUIET] [system.out] Simulation mock.CatsSimulation completed in 3 seconds

@rmelethodiyil1 first, we don't support old versions. please use 1.4.0.RC3, because that is where we are focused.
second, it is absolutely a must that you follow this process: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue - there are many teams who have collaborated to solve issues in the gatling integration, see this example: https://github.com/karatelabs/karate/issues/1544


TabView add header HorizontalOptions

Can you add properties to center for exemple the header , actually the tab view header is force to start position.

TabHeaderItemColumnWidth was added for this reason:
https://github.com/enisn/UraniumUI/blob/50bd77b0f4c004c4f113bae45214576d85b19c60/src/UraniumUI.Material/Controls/TabView.BindableProperties.cs#L23-L24
So, we can close this


Added firmware update docs

firmware_update_workflow.md - provide steps on how to perform firmware
update on system running Pelion Edge Yocto image.
port_firmware_update.md - Guide to port the Device Management Update client to
systems running Edge Core.

@AnotherButler  FYI, I reused the "Initiate a firmware update campaign" section from https://github.com/ARMmbed/pelion-dm-edge-docs/pull/43

Why are we duplicating content, and why are we creating content in this repo instead of in the public docs repo?

@AnotherButler Sure, we can move the content there. I wasn't aware of the process. So raise a PR against master? Also I am not sure of the folder structure and in which folder these docs should reside in. Should I consult you before raising the PR?

@ygoyal18 Thanks. Yes, please raise a PR against master. Feel free to put them loose on the main level or in a folder on the main level. (We'll create a new book for updating.)

@AnotherButler Created PR in the other docs repo. Closing this PR.


Update AES GCM IV max length to be 1024 bits (was 512)

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

See query in PR #11843

This pull request is ready to merge

Thanks @t8m.
Merged to master.


Handling empty date

Sometimes the date in DICOM file is 0000-00-00. Is it a common case in the ecosystem when the date is empty?

It is fairly common for date elements to either be all zeros, empty, or not present altogether. Including SeriesDate / StudyDate.

This kind of relates to #9. We shouldn't attempt to parse the VR's while loading the DICOM files.
IMO, all parsing should be done after the fact, for example, using Result<[u8]> for VR::IS, or Result<DateTime> for VR::DT.
@Enet4, what do you think about making this the "default" approach?

As mentioned in #9, it is reasonable to preserve textual values as strings when loading DICOM objects and only convert when requested.
Regarding this issue (and others of similar nature), it helps if you can make a reproducible example, so that it can be independently reasoned with and tested. In part, this also means that there should be an easy way to build and write DICOM objects, yet to be worked on.

Closing this as partially resolved by #24, and for discussion to continue in #9.


Fix crash of generator when property type is not explicitly set.

Is this doable?

The crash is now somewhere in the code. It should at least throw some error with description.

I see, you're right.


Error: pngquant failed to build, make sure that libpng-dev is installed (Win10)

Getting this error when doing npm install in Windows 10
‼ Command failed: C:\projects\react-core-boilerplate-master\ReactCoreBoilerplate\node_modules\pngquant-bin\vendor\pngquant.exe --version

  ‼ pngquant pre-build test failed
  i compiling from source
  × Error: pngquant failed to build, make sure that libpng-dev is installed
    at Promise.all.then.arr (C:\projects\react-core-boilerplate-master\ReactCoreBoilerplate\node_modules\execa\index.js:231:11)
    at process._tickCallback (internal/process/next_tick.js:68:7)
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^5.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `node lib/install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\<username>\AppData\Roaming\npm-cache\_logs\2019-03-02T16_54_05_901Z-debug.log

I think that the problem was in floating dependencies of the package image-webpack-loader. Is it ok now?

I have the same problem when downloading and building the project in Visual Studio:
Built at: 2019-03-19 12:27:05
Asset     Size       Chunks                    Chunk Names
main-server.js  648 KiB  main-server  [emitted]  [big]  main-server
Entrypoint main-server [big] = main-server.js
ERROR in ./ClientApp/images/logo.png
Module build failed (from ./node_modules/image-webpack-loader/index.js):
Error: 'C:\Users\gunna\source\repos\ReactCoreBoilerplate\ReactCoreBoilerplate\node_modules\pngquant-bin\vendor\pngquant.exe' is not recognized as an internal or external command,
operable program or batch file.

    at WriteWrap.afterWrite [as oncomplete] (net.js:779:14)
 @ ./ClientApp/pages/HomePage.tsx 1:486-521 1:904-908
 @ ./ClientApp/routes.tsx
 @ ./ClientApp/boot-server.tsx

npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build:dev: node ./node_modules/webpack/bin/webpack.js --config webpack.config.vendor.js && node ./node_modules/webpack/bin/webpack.js --config webpack.config.js
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\gunna\AppData\Roaming\npm-cache_logs\2019-03-19T11_27_06_171Z-debug.log

Package image-webpack-loader has been removed due to instability.


make submit button functional

navigate user to home page on submit the form

I think there is no need to navigate user to home page after submitting.


Simplify package checksum verification

There's no need for get_url/shell/stat/etc. tasks when you can simply use a url lookup for fetching the package checksum.

Jinja2 Ninja! Thanks


SkyFactory 2.5.6 crash [1.7.10]

Issue Description:
I have a crash on SF 2.5.6.  The money quote appears to be here:
Chisel: Errors like "[FML]: Unable to lookup ..." are NOT the cause of this crash. You can safely ignore these errors. And update forge while you're at it.
EnderIO: Found the following problem(s) with your installation:

An unknown AE2 API is installed (rv3 from appliedenergistics2-rv3-beta-6.jar).
Ender IO was build against API version rv2 and may or may not work with a newer version.
This may have caused the error. Try reproducing the crash WITHOUT this/these mod(s) before reporting it.

A fresh build?
I don't know if it is reasonable to try to get a build for 1.7.10 against rv3 or not.  I've not a java guy, but I know my way around a Linux prompt.  If you would like to give me pointers for a build, I could play with that.
What happens:
"Random" crash
Steps to reproduce:

Install FTB 2.5.6, with Thermal expansion moved to an available release
???
crash
...

Affected Versions (Do not use "latest"):

EnderIO:  1.7.10-<IP_ADDRESS>9_beta
EnderCore: 1.7.10-<IP_ADDRESS>_beta
Minecraft: 1.7.10
Forge: 1.7.10-<IP_ADDRESS>8-1.7.10

Your most recent log file where the issue was present:
https://gist.github.com/NathanZook/a38635db823d54a6c148942c0b3eac28

This output has been updated in later versions to read:

Found the following problem(s) with your installation (That does NOT mean that Ender IO caused the crash or was involved in it in any way. We add this information to help finding common problems, not as an invitation to post any crash you encounter to Ender IO's issue tracker. Always check the stack trace above to see which mod is most likely failing.):

This output has been updated in later versions to read:

and still people are ignoring it :D

Thanks for the help.  Sorry for the noise.


Using VRTK with Leap Motion

Hi, I'm trying to get VRTK working with the leap motion. Specifically i am working on UI inteaction. I have figured out that VRTK_UIGraphicRaycaster does not work with the Leap motion interaction system. Enabling the graphicsraycaster make it work again. I wonder if it's normal that a input module will not work with the  VRTK_UIGraphicRaycaster, or if it should work and more likely is a problem in the leap sdk.
Another idea is to make leap working using the vrtk ui interaction system and use it's touch and interaction methods to control the ui. But i am unsure how much work this would be.

The custom graphics raycaster that VRTK uses is required to allow the 3d worldspace objects to cast rays and connect with UI elements. As the normal raycaster uses screen space this is no good for 3d world objects.

The leap motion sdk does this as well, but i think (not having the code with me right now) it's doing it in the input module. That makes the setup a bit easier (no need to attach a script to change graphicsraycasters) but i think it suffers from a problem that the input module cant handle multiple inputs simultanously. But i'm not sure about the multiple input part, i just had lots of problems activating leap, gaze and vive at the same time.
I think i will try to get leap working with VRTK to solve my problem. I hope the touch input is exact enough for this use (leap activates buttons on dragging the finger out of the ui element).
thanks & keep up your work!

Maybe #927 is fixing this?

Once you get this working, I’d love to see a sample project, article with a walkthrough how to set this up :)

On Jan 26, 2017, at 4:29 AM, ludos1978<EMAIL_ADDRESS>wrote:
The leap motion sdk does this as well, but i think (not having the code with me right now) it's doing it in the input module. That makes the setup a bit easier (no need to attach a script to change graphicsraycasters) but i think it suffers from a problem that the input module cant handle multiple inputs simultanously. But i'm not sure about the multiple input part, i just had lots of problems activating leap, gaze and vive at the same time.
I think i will try to get leap working with VRTK to solve my problem. I hope the touch input is exact enough for this use (leap activates buttons on dragging the finger out of the ui element).
thanks & keep up your work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/thestonefox/VRTK/issues/871#issuecomment-275343517, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEMEGE_n1IUNl8DeIxaQDzYnBXLgOxfks5rWGd0gaJpZM4LrQ3x.


Adds param-immediately-invoked-callable in Option

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5013

Relevant Lines:
5156

💛  - Coveralls


Error while building the app after twitter-kit-android integration: com.android.builder.dexing.DexArchiveBuilderException

After adding the dependency on app module Gradle file, unable to run the application. Below is the Dex exception received in build console.
-- | --
Invoke-customs are only supported starting with Android O (--min-api 26) |
Stripped invalid locals information from 2 methods. |
null |
com.android.builder.dexing.DexArchiveBuilderException: Failed to process /home/user/.gradle/caches/transforms-1/files-1.1/twitter-core-3.2.0.aar/0e1c71d7ba7eca159f5f854fdb021b6a/jars/classes.jar |
com.android.builder.dexing.DexArchiveBuilderException: Error while dexing. |
com.android.tools.r8.CompilationFailedException: Compilation failed to complete |
com.android.tools.r8.utils.AbortException |
Using latest Android studio 3.1 version to compile and execute the project with compileSdkVersion and targetSdkVersion 27.

After setting java version to 8 working properly.


F-Droid can't build - Google Play Services dep

ref: https://gitlab.com/fdroid/fdroiddata/-/jobs/3800573610#L3284
#114 vaults ( https://github.com/deckerst/aves/commit/bc6d75e92820b7108c794f5016717c7cfab8ded0 )
added dependency on pinput ( https://github.com/deckerst/aves/blob/v1.8.0/pubspec.yaml#L86 )
which depends on smart_auth ( https://github.com/Tkko/Flutter_Pinput/blob/master/pubspec.yaml#L23 )
which depends on com.google.android.gms:play-services-auth ( https://github.com/Tkko/flutter_smart_auth/blob/main/android/build.gradle#L49-L50 )
/LE: disabled https://gitlab.com/fdroid/fdroiddata/-/commit/c9fa1682736b5c543c19dce372bf2a3dd0d5707b

done https://gitlab.com/fdroid/fdroiddata/-/commit/1263d5493af1f7c3e333cde47f5284d81d9623b9


Update ammonite, ammonite-terminal to 3.0.0-M0-5-0af4d9e7

Updates

com.lihaoyi:ammonite
com.lihaoyi:ammonite-terminal

from 3.0.0-M0-3-c2d1e706 to 3.0.0-M0-5-0af4d9e7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.lihaoyi" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #2354 2353


fix: make directories gruvboxblue

I think has a lot more sense making Directory gruvboxblue since is by default the colors of the directories in the terminal. Also I think we are using gruvboxblue for NvimTree directories too.

hey @saccarosium makes sense, what do you think about keeping it bold as well?

In my setup is not very apperent when you use bold font but I see that some people might prefer bold.

@saccarosium let's keep it bold for now, and you can always override it if you want. Thanks for the contribution!


Incorrect path determined for some dependencies resulting in invalid cached licenses

When caching dependencies for our app,  several dependencies are resulting in warnings related to incorrect dependency paths. Those incorrect paths cause failures in license caching if the license needs to be updated and dependencies are cached with a license of "none".
I haven't looked into this too deeply so far but I did notice that Licensed::Dependency is initialized multiple times for activerecord (the one I was picking out). The first time, the path given was correct. The second time it was not.

Chatted with @cpruitt and looked into this - the problem is that the bundler source is surfacing multiple versions of the same dependency from Gemfile.lock, while only a single version of the dependency is present.  When multiple versions of the same dependency are present, and the path of the cached file isn't keyed on the dependency version, results are dependent on the order of evaluation.
As an example, both activerecord 5.x and 6.x were listed in Gemfile.lock but only 6.x was installed.  Since the 5.x dependency version was evaluated last, it is overwriting what's cached for the valid 6.x dependency.
I'm going to look into a fix for the bundler source that handles multiple versions of the same dependency, similar to how the npm source works.

this workaround to find the right gem_dir is only needed when if ruby_packer? is true.
add version requirement info to this call
allow multiple versions of a dependency to be cached separately, e.g. in this case activerecord-6.x... would be cached properly while activerecord-5.x... would continue to raise an error.  This at least allows you to ignore the missing activerecord-5.x... dependency while still have a valid 6.x file.

What's increasingly confusing to me is why bundle install is not reporting any errors.  It seems like some dependencies are requiring versions of a dependency that aren't installed.


Package "unittest" uses deprecated library syntax

FAILED: dartc-none release_ia32 html/document_test
Expected: pass
Actual: fail
stderr:
ERROR|PARSER|DEPRECATED_LIBRARY_DIRECTIVE|file:/Users/scheglov/Source/Dart/dart/pkg/unittest/lib/html_individual_config.dart|15|1|8|The '#library(url)' directive has been deprecated, use 'library name' instead
    14:  */
    15: #library('unittest');
        ~~~~~~~~
ERROR|PARSER|DEPRECATED_IMPORT_DIRECTIVE|file:/Users/scheglov/Source/Dart/dart/pkg/unittest/lib/html_individual_config.dart|17|1|7|The '#import(url)' directive has been deprecated, use 'import url' instead
    16:
    17: #import('unittest.dart', prefix: 'unittest');
        ~~~~~~~

This issue has been moved to dart-lang/test#172.


Update requirements and confirm notebook runs

Update featuretools to >0.16.0
Restrict baytune verions
Fix read csv statements
Fix cutoff time naming issue

Updated code in PR #9 instead - this is no longer needed.


Universal Slice Selection Deviates from the Preference

When using the setting files.universal.architecturePreference, the preference does not work sometimes.
For example, open /bin/ls while having the setting set to ["arm64e", "x86_64"]

Fixed in 3.3.4040-dev.


Attract Active data-class

Hi,
I have a problem to attract the active class with "onSlideChangeEnd":
Show my script:
var MySwiper = new Swiper('.swiper-container-v', {
	direction: 'vertical',
	paginationClickable: true,
	onSlideChangeEnd: function(swiper){
		var nfo = $(MySwiper.activeIndex).attr("data-class");
		alert(nfo);
	}
});

and body

...

Can you help me please ?

It should be:
var nfo = $(MySwiper.slides[MySwiper.activeIndex]).attr("data-class")


Added back calling Natives with IHandleable types

Added UI Changes
todo fix returning from native with IHandleable types

Sounds reasonable. But something went wrong with intendation, there's a few spaces instead of tabs in some places.

thought this one was long over due but casting enums(only ones based off int/uint/int64/uint64) is no longer necessary for calling natives.This means code like this
Function.Call<bool>(Hash.IS_THIS_MODEL_A_CAR, VehicleHash.Adder);
will compile and run as expected

What's up with the scripting stuff being C# files?

@Fireboyd78 See #489

Ohhh, that's what the "experimental" branch was about. My bad, lol.


Add formatting to the example block for the Endpoints doc

Wrap the example section with in the code  block tags so that it appears with proper formatting on the website. This is for the en web site. This resolves issue #30878

@benignbala : Please sign the CLA for the PR to be reviewed.

closing per https://github.com/kubernetes/website/pull/30879#pullrequestreview-829673985
@benignbala Feel free to reopen or continue the conversation if you like to continue working on it
/close


Maybe allow html in markdown without fences?

Other markdown I've been using allows to place html right into markdown.
Having the possibility to easily switch back to markdown within some html div would be awesome as well.
One big advantage of going with such standard is that syntax highlighting can be borrowed from other markdown linters.
Having custom latex-commands is nice and handy, however they fall short in some cases with syntax highlighting and autocompletion.
Was there a particular reason why html is only allowed within ~~~  blocks?

Probably because the Markdown stdlib doesn't support it: https://github.com/JuliaLang/julia/issues/17837

Even CommonMark.jl have special syntax for it: https://github.com/MichaelHatherly/CommonMark.jl#raw-content

So this is actually an easy one: there's no way I will do it; you could criticise the initial decision but it was a choice from very early on to have the MD and the HTML fully separated which simplifies a lot of stuff. And @fredrikekre is right that Base.Markdown doesn't support it but even if it did I'd have done the same.
Another one that I kind of had supported but will definitely not be supported in the future is indented code blocks.
Blocking these two allows to (much more easily) handle custom parsing rules for things like latex-like commands or hfuns which is by design.

One side note
I'd be fully ready to acknowledge that it's not always a good idea to reinvent the wheel that some people thought about markdown for a while and that lacing HTML into the markdown and/or having indented code blocks is cool. So in the next version there will be the possibility to basically switch off Franklin's parser in favour of another one (typically CommonMark's) and then you can do whatever you want in there that is compatible with that parser but you won't be able to use any of Franklin's goodies. That should make it nice for people who want to migrate sites and don't want to have to bother with adjusting old pages so they're Franklin-compliant.

Ah and yes I agree for the syntax hl... we'd have at some point to define custom rules for "Franklin markdown" that could be a good project 🤪

thank you all for your input.
very valuable and appreciated


Add default options object if one doesn't exist

If options are not defined in the Dropkick object
some of the default logic will fail. This ensures
an options object, even if empty, is included.

While I like being able to test the bug we had I don't think adding opts to the object solely to test is the best way.


PPA repository for install via APT

Hello, looks great!
If you've already set up your .deb building from deb-src, could you please put it up in a PPA?  It would make installation much simpler, particularly for those of us who have our CI server configuration in puppet or chef scripts.
Cheers!
Michael

We now have Bintray integration which supports apt repositories. We should be able to publish the Drone .deb file to Bintray and take advantage of this feature. I'm on it.

Bintray looks good. how far is the work on the repo?
you should upload the pgp key to public keyservers that one can simply get it:
gpg --keyserver keyserver.ubuntu.com --send-keys 12345678
apt-key adv --keyserver keyserver.ubuntu.com --recv-keys 12345678

@bradrydzewski did you finish setting this up? Looks like Bintray isn't aware of any files. Can you make it pull from S3 automatically or something?

drone is now being distributed as a Docker image


Add translations for profile card screen

We have "Share" and "Edit" buttons, but we don't have translations for Japanese.
Please use Compose Resources to display the translation
https://github.com/DroidKaigi/conference-app-2024/blob/b69506419c6c5555aae87d9e6f485ecbec15d7f9/feature/profilecard/src/commonMain/composeResources/values/strings.xml
https://github.com/DroidKaigi/conference-app-2024/blob/b69506419c6c5555aae87d9e6f485ecbec15d7f9/feature/profilecard/src/commonMain/kotlin/io/github/droidkaigi/confsched/profilecard/ProfileCardScreen.kt#L750

✋


Classifying and documenting queries for JMX QueryStats monitoring.

Use case:
Adding CrateDB JMX Monitoring metrics to Prometheus and visualizing them through Grafana, queries are also classified as Management and DDL. Although it is obvious for most of the cases it is not for all the case.
E.g. creating and restoring a snapshot is classified as DDL.
There might be reasons for this and it might make sense, but there's no way atm, besides digging into the source code in clarifying this.
Feature description:

Document which query is classified how. https://crate.io/docs/crate/reference/en/latest/admin/monitoring.html#querystats-mbean
I'd like to have a conversation if creating and restoring a snapshot isn't more of a Management (or it's own type) of QueryStat.

any solution how can i collect cratedb metrics using jxm ?

any solution how can i collect cratedb metrics using jxm ?

Maybe this article is helpful?
https://crate.io/blog/monitoring-cratedb-with-prometheus-and-grafana
Getting the JMX metrics via http is done via the JMX exporter:
https://github.com/crate/jmx_exporter


Update Twitter Logo to New "X" Branding

Issue: Update Twitter Logo to New "X" Logo
Description
While scrolling down the page, the Twitter logo displayed is still the older version. We need to update it to the new logo, which is now referred to as "X".
Steps to Reproduce

Navigate to the https://manifest.build
Scroll down the page.
Observe the Twitter logo.

Expected Behavior
The page should display the new "X" logo instead of the older Twitter logo.
Current Behavior
The old Twitter logo is still visible when scrolling down.
Screenshot

Suggested Solution
Replace the existing Twitter logo with the new "X" logo in the relevant files.
Thank you!

@SebConejo I would like to work on this issue.

@Piyush-Goenka go ahead, the concerned repo is https://github.com/mnfst/website
You can follow the same contributing rules as in this repo !

@brunobuddy
I am unable to solve this issue please assign this issue to someone else .

@brunobuddy assign me.

@nitesh2920 we use lineicon. This font doesn't contain the x icon for now. Maybe we could just use an svg icon to display this one instead of calling another whole font-icon.

@SebConejo yeah I just wanted to ask that question.

@SebConejo I have one question I have to raise a PR in https://github.com/mnfst/website this repo instead of this as it is for backend only.

@nitesh2920 yes the repo is https://github.com/mnfst/website

@brunobuddy so I have to firstcreate  a issue there then raise a PR for that issue.

@brunobuddy @SebConejo raised the PR please check https://github.com/mnfst/website/pull/29#issue-2576495831

Hello @nitesh2920 sorry this is not applicable anymore as we are not displaying personal accounts on the website


Have integers be deserialized as number not bigint

hi, passing bigints to the aws-sdk will result in an error. maybe use numbers instead?

The way I solved this is by converting to number during serialization of a bigint. So you should always pass down the plain object to the AWD SDK with the serialize method.
I wanted to keep a distinction between integer and float numbers, but we can revisit that in case it becomes too complicated or not intuitive.

The reason for creating our own BigInt type is lack of serialization support (more details here)


Release version before next driver development steps (deletes, etc.)

Release complete version of implementations before adding new features.

Released at https://github.com/ldbc/ldbc_snb_interactive_impls/releases/tag/1.0.0


Convert Active DLRS to Custom metadata via Custom metadata loader

I want to convert the active existing DLRS records to Custom metadata via Custom metadata loader.
Since the field Calculate Job id is absent on Custom metadata DLRS so the migration tool throws the error
"Webservices returned with Soap Fault: Field dlrs_CalculateJobId__c does not exist on entity dlrs__lookupRollupSummary2"

@Shraddha-Ingale indeed, per @aheber that field is not needed in the Custom Metadata version of DLRS, so you are safe to drop it from your import file. 👍


Error in Chrome 75.0.3770.100: Uncaught Instance of 'minified:p'

steps to reproduce after fresh install of windows 10
choco install dart-sdk
pub global activate webdev
pub global activate stagehand
mkdir angular_dart_web
cd angular_dart_web
stagehand web-angular
pub get
webdev serve

open http://<IP_ADDRESS>:8080/ in Chrome:
Uncaught Instance of 'minified:p'
    at Object.b (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:367:3)
    at nq.$0 (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:5445:20)
    at Object.uK (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:1268:5)
    at kK.uP (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:1270:7)
    at lL.$1 (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:4865:3)
    at vk (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:441:17)
    at MutationObserver.<anonymous> (http://<IP_ADDRESS>:8080/webdev/src/serve/injected/client.js:449:51)

@johnpryan or @domesticmouse or @vsmenon, does this sound familiar?

@jakemac53 @grouma

Okay it's weird the error doesn't appear after reinstalling dart-sdk
But now after executing webdev serve the page is loading but hangs here:

Found the problem.
webdev indicates that it is serving on http://<IP_ADDRESS>:8080
after using http://localhost:8080 everything works

That's weird. I can't reproduce this issue but we will now serve the client unminified which will hopefully give a better error.


Install examples with an nipkg

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Create an nipkg to install examples.
Update Scripting API build specs to install to vi.lib as a post-build action.
Modify the examples to refer to the scripting API in the installed location.
Why should this Pull Request be merged?
We want to install examples!
What testing has been done?
Branch build built installers successfully: \\nirvana\Measurements\VeriStandAddons\routing_and_faulting_custom_device\ni\export\dev\examples_use_installed_vis\Build 13\2018\installer
New examples package installs files to C:\Program Files (x86)\National Instruments\LabVIEW 20xx\examples\VeriStand Custom Devices\Routing and Faulting

[ ] Exclude project aliases file from installer.

[ ] Change build destination directory to "Scripting Examples" to match source folder layout.

[ ] Update build.toml with folder rename

[[package]]
type = 'nipkg'
payload_dir = 'Source\Built\Examples'
install_destination = 'ni-paths-LV{veristand_version}DIR\examples\VeriStand Custom Devices\Routing and Faulting'
control_file = 'control_examples'
package_output_dir = 'Source\Built'


task status isn't showing not error if one of several problem matchers reports a problem

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.81.1
OS Version: Windows 10 Pro

I have a background task with multiple problem matchers:
{
        "label": "Storybook A11Y",
        "type": "shell", //"process" does not work as value
        "command": "npm run test-storybook-a11y:watch",
        "args": [],
        "isBackground": true, //this makes it keep running in the background without interrupting what you're doing
        "runOptions": {
          "runOn": "folderOpen" //start this task when folder is opened in VS Code
        },
        "presentation": {
          "panel": "dedicated",

        },
        "problemMatcher": [
          {

            "fileLocation": ["relative", "${workspaceFolder}"],
            "severity": "error", //probably redundant as it is the default
            "pattern": [

              {
                "kind": "file",
                "regexp": "^\\s*(name):\\s*'(Error)'\\s*$",
                "message": 2,
                "file": 1
              }

            ],
            "background": {
              "activeOnStart": true,
              "beginsPattern": "^\\s*Starting Storybook A11Y tests\\.\\.\\.\\s*$",
              "endsPattern": "^\\s*Completed Storybook A11Y tests\\.\\.\\.\\s*$"
            }
          },
          {

            "fileLocation": ["relative", "${workspaceFolder}"],
            "severity": "error", //probably redundant as it is the default
            "pattern": {
              "kind": "file",
              "regexp": "^\\s*Detected the following (accessibility)\\s(violations).\\s*$",
              "message": 2,
              "file": 1
            },
            "background": {
              "activeOnStart": true,
              "beginsPattern": "^\\s*Starting Storybook A11Y tests\\.\\.\\.\\s*$",
              "endsPattern": "^\\s*Completed Storybook A11Y tests\\.\\.\\.\\s*$"
            }
          },
          {

            "fileLocation": ["relative", "${workspaceFolder}"],
            "severity": "error", //probably redundant as it is the default
            "pattern": {
                "kind": "file",
                "regexp": "^\\s*(Error)(.*)$",
                "message": 2,
                "file": 1
              },
            "background": {
              "activeOnStart": true,
              "beginsPattern": "^\\s*Starting Storybook A11Y tests\\.\\.\\.\\s*$",
              "endsPattern": "^\\s*Completed Storybook A11Y tests\\.\\.\\.\\s*$"
                          }
          }
        ]
      },

If only one of the three problem matchers reports a problem, I see it in Problems pane:

But the terminal decoration does not indicate error:

I only get an error in terminal decoration when there is a single problem matcher in the task. So if I comment out the first and last problem matchers in the above task so the only remaining one matches the line Detected the following accessibility violations!, then a terminal decoration of error is shown correctly:

I believe this is already fixed, can you see if you can reproduce in the Insiders build?
/label info-needed

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


manifest mapper doesn't map router-path property

Currently you can't set the route-path in the manifest.yml. I created a PR which implemented this missing feature (#747).

Hi @floriangosse,
We have a story on supporting this new feature in app manifest files and the reason I haven't prioritised it yet is because we're still reviewing the UX of it. Can you explain your PR's UX, how it compares to the one suggested in our story?
https://www.pivotaltracker.com/story/show/104193622
Cheers,
Dies Koper
CF CLI PM

Hi @dkoper,
I like like the way, to move all the route specific properties into a list and make is possible to define multiple of it.
For me it isn't needed define multiple routes for my applications, but for someone this will be the thing that is needed.
My solution was simple. I simply added the route-path value of the yamlMap to the appParams. This was the logical and missing step for me after the last release which makes it possible to define a route path with the command line options.

Note that we have a proposal (two actually) on supporting routes with paths in App Manifests to the CF Dev mailing list:
<EMAIL_ADDRESS>Please take a look and join the discussion!

Hi Dies,
What's the status of this? The docs suggest it has been implemented, if it has then there looks to be a bug that I can't map a path onto another host name via the manifest (can do via CLI).
So for example:
---
  name: app-v1
  routes:
  - route: app.domain.com/v1

Doesn't work?

@sradnidge I failed to update & close this issue. It was implemented in cf CLI 6.21.0, two weeks ago.
https://github.com/cloudfoundry/cli#downloads
Please confirm what cf CLI version you're using, and if 6.21.0 or later, please submit a new issue with all the details (cf CLI version, cf trace).
Cheers,
Dies Koper
CF CLI PM

Awesome - cheers! I was trying with 6.20, will give 6.21 a try and report back :)

Can confirm - all working as expected. FWIW I'm glad you went with that format for the 'UI', its nice.


io_ssl.c:786:10: fatal error: keys/target_key.c: No such file or directory

I'm trying to build OpenSSL 1.1.1 from source (branch OpenSSL_1_1_1-stable) in Arch Linux, but during the building I get this error:
/usr/bin/cc -Wall -Wextra -std=c99 -pedantic -Os -DOPENSSL -I../openssl/include -c -o string_to_vector.o string_to_vector.c
/usr/bin/cc -Wall -Wextra -std=c99 -pedantic -Os -DOPENSSL -I../openssl/include -c -o io.o io.c
Generating DH parameters, 2048 bit long safe prime, generator 2
This is going to take a long time
............................In file included from io.c:5:
io_ssl.c: In function ‘init_io_control’:
io_ssl.c:436:30: warning: implicit declaration of function ‘get_dh’; did you mean ‘getwd’? [-Wimplicit-function-declaration]
  436 |                 if((io->dh = get_dh()) == NULL){
      |                              ^~~~~~
      |                              getwd
io_ssl.c:436:28: warning: assignment to ‘DH *’ {aka ‘struct dh_st *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
  436 |                 if((io->dh = get_dh()) == NULL){
      |                            ^
io_ssl.c: In function ‘init_io_target’:
io_ssl.c:786:10: fatal error: keys/target_key.c: No such file or directory
  786 | #include "keys/target_key.c"
      |          ^~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:117: io.o] Error 1

In my system, I installed openssl-1.1 but still get that error. Tested also by installing openssl as dependency but same error.

I am not sure where these .c files come from but they are not part of the OpenSSL codebase.

Any idea could be the library?


declaration names link to anchor; github links at bottom

The declaration names now link to their own position on the page.
Maybe the spacing needs to be adjusted a little, I'm not sure. (Or we could use a normal bullet point.)

I also found it unintuitive that the declaration names linked to the source, unlike all other names.  The explicit View declaration on GitHub link on every declaration is IMHO quite distracting.  Every theorem now has a second line that conveys no information.
My first reaction to reduce the visual clutter would be to move the github link to the right:

https://github.com/gebner/doc-gen/commit/6c50c3b77ae107fe5e15ddecd52fa4c1ab4ddbbf

I agree putting it at the bottom isn't ideal. On a wide screen, floating it to the right kind of disassociates it from the declaration, especially for declarations that don't have many arguments.

Maybe repeating the colored line on the right of the declaration div would improve it? I'm not sure -- I'll experiment a bit.

How about this?

I think the horizontal line looks good.  However putting borders on all sides looks a bit too heavy to me.  How about just putting borders on the left & top, like this?

I like it!


SW-4028 Use 'contain' object fit for thumbnail images

this will use correct aspect ratio to display thumbnail, with the downside that there will be horizontal empty space for portrait images

@chudo76 take a look at this and lmk.. my proposal it to track a separate effort that standardizes how we display thumbnails in the app. There are a few places where we show thumbnails.

Lgtm!

Thanks @constanzauanini

@chudo76 take a look at this and lmk.. my proposal it to track a separate effort that standardizes how we display thumbnails in the app. There are a few places where we show thumbnails.

@karthikbtf Sounds good; I created: https://terraformation.atlassian.net/browse/SW-4080


Videos with non latin titles fail to sync

И wилл конверт дем то латин

not sure why i left this open. it was fixed.


No module named grumpy.compiler when compile python code to go

Traceback (most recent call last):
  File "./tools/grumpc", line 25, in <module>
    from grumpy.compiler import block
ImportError: No module named grumpy.compiler

I just add the information in __init__.py,however it doesn't work

export PYTHONPATH=$PYTHONPATH:$PWD/build/lib/python2.7/site-packages/

Even standard library support is still on the way, I am not sure which libraries tensorflow depends on, so no idea for me.

All right, thanks~


Why the name Alice?

Why is it called Alice?

There is a rock band called Alice in Chains. alice chains HTTP handlers. That's about it :smile:

CLASSIC! I like this library even more now...


Update fails if kubectl is running [kubernetes]

There is a bug in the Windows installer (installed version <IP_ADDRESS>61, trying to install 18.4.17070)
When kubernetes was activated and kubectl is still running, because you run the command kubectl proxy (or maybe kubectl proxy & in the background), the installer cant replace the kubectl.exe and installation fails. Finally the old Docker version is uninstalled (but not the kubectl.exe) and you have to kill the running process, and install manually.

BTW why has the installer grown from 382 MB to 467 MB?

This issue was moved to docker/for-win#1944


Right-apply for column-wise operations

Ginkgo currently implements column-permutations (and column-scaling from what I discussed with @fritzgoebel) by slightly abusing the M->apply(b,x) to mean x <- bM. Aside from this undocumented deviation from the expected behavior, it also doesn't work with rectangular matrices, because the wrong dimensions are checked against each other.
One possible fix would be to use b->apply(M,x), but that would mean that the right-application of M would become the responsibility of b, which means that the control flow for column-permutation would be moved away from row-permutation to IMO the wrong place.
I would instead propose to add another RightApplyable interface that provides M->rapply(b,x) computing x <- bM. We would need to derive from this and implement rapply_impl only for LinOps for which this right-apply operation is actually needed (like Permutation and diagonal matrices).

I no longer consider this a good idea, as the apply interface has been overloaded with functionality already. member functions and LinOpFactory should be able to deal with explicit permutation/scaling and reordering in the solver hierarchy explicitly.


SSH port mapping

Could you consider SSH port-mapping based on specific requirements to manage cEOS instances?

yeah, shouldn't be a problem. what's your use case? why not "docker exec -it ... Cli" ?

It means you can provide access to a container-based network model directly
without having to have multiple sessions open on the Docker host
On Mon, 18 Jun 2018 at 22:36, networkop<EMAIL_ADDRESS>wrote:

yeah, shouldn't be a problem. what's your use case? why not "docker exec
-it ... Cli" ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/networkop/arista-ceos-topo/issues/7#issuecomment-398203710,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APSYHBo05oYL59k_NlBhBHSurql6EN-9ks5t-B1egaJpZM4UrwVt
.

--
Sent from Mobile, excuse typos

Done James,
Have a look at the https://github.com/networkop/arista-ceos-topo/blob/master/topo-extra-files/examples/v2/3-node_ssh.yml
In order to be able to login, you'd need to configure root password inside EOS:
aaa root secret secret!23

and then you can ssh to cEOS, e.g. the first container:
ssh -p 2000 root@localhost

issue resolved in commit https://github.com/networkop/arista-ceos-topo/commit/43d1a679ebbd9588d2f0c2b89fe27f2cf142a46e
Please see README and https://github.com/networkop/arista-ceos-topo/blob/master/topo-extra-files/examples/v1-legacy/mport.yml


[Bug] Spline scene will sometimes not render in a production build of React app

Environment is CRA and deployed using GitHub Pages. When you visit the site, one of two things will happen:

Scene does not render (sometimes you see a flash before it goes)
Scene renders after a brief delay

Here is a post from Stack Overflow describing a similar problem:
https://stackoverflow.com/questions/72244874/react-spline-bug-after-built-react-project-the-spline-content-is-gone-in-spli
I've tried a variety of workarounds and potential solutions to no avail. Note that the issue described above is not encountered during development.

After digging some more, it seems that the issue lies with the responsive size setting in the Spline app. Setting a custom size and exporting it that way fixed the issue for me.

Seams to be the same with my issue #40. But when you set a custom size, is there a problem with fullscreen application or dose it behave normally like before.

I just tried it and am running into the issue again unfortunately.

We will take a look into this soon. Sorry for the delay 🙌

Fixed with version 0.9.62 of @splinetool/runtime


Allow collapsible form in box

See issue #523

What happens if we just remove > instead of adding the > form to the query?

Hello,
I just tested your suggestion and it works, but I have no idea whether this would cause any other sideeffects. Was there a special reason why you chose the direct children selector?

What happens is that we would reproduce an issue that we fixed long ago #321.
I'll merge this PR then since it sounds like the best solution. Thank!

OK, for the moment it seems like the best solution, experienced some lag in the slideUp-effect but this was not reproducable anymore ...
BTW: I also added the data-collapse to the header so the whole header is acting as trigger for the collapse (sort of accordeon) which works nicely but is a very special case.
Thanks for merging


Rewrite .js files into .ts files

As described in the title, I rewrote backend files in Javascript into those in Typescript. Since the strict option is currently disabled, the migration hasn't finished yet.

Thank you for your review!
I removed unused variables named _ as you suggested. But I decided migrating to ts-node and a proper setup for eslint to be in another PR.


Request to add values of encrypted fields to the list in Document the values of encrypted fields

A partner in Japan reports us the two values of encrypted fields below and requests to add  them to the list in Document the values of encrypted fields.
Field name Where to set the value
PersonnelIntegrationConfiguration.STORAGEACCOUNTCONNECTIONSTRING	Select System administration > Setup > Integration configuration > Azure storage account connection string.
BIMEASUREMENTPARAMETERS.AZUREAPPSECRET	Select System administration > Setup > System parameters. On the Document connections Tab, in the AZURE KEY VALUE group, select Application Secret.

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

ID: 277452eb-70c4-3d29-5219-1355a40bb714
Version Independent ID: a9d425c4-c627-d20f-4098-307eceb2dd07
Content: Golden configuration promotion - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/database/dbmovement-scenario-goldenconfig.md
Service: dynamics-ax-platform
GitHub Login: @laneswenka
Microsoft Alias: laneswenka

Updated in latest update


[Discussion] Database Security

How do you protect users from writing to other users data, once they are authenticated?

There are some solution such as [firebase database rules] (https://firebase.google.com/docs/database/security/), security token and do some checks with cloud functions.

What rules must to be deployed in production? If I use own rules, there are more errors than I can see. Please, create usable production rules before release 1.0


Nodemon seems to not fire my shutdown function

According to this nodemon issue, nodemon does not appear to give child processes a chance to handle signals. Nothing either of us can do about it but go fix nodemon.

hmm k

hmm maybe you could fix this by doing what they suggest in here? https://github.com/remy/nodemon/blob/5e88b04eaa94ba72a18fca7d28517a0316fe4cb3/README.md#controlling-shutdown-of-your-script

Good find, but SIGUSR2 isn't a shutdown signal. An application receiving it could do anything it wants and need not shut down. So node-cleanup can't shutdown upon detecting it.
You might implement that yourself in your app. It's kinda weird that an app would need to implement a special shutdown process just to run properly under nodemon.

true.. maybe there's a way to figure out if you are running under node mon and use that event only in such a case?


How to handle unproper connection of CP [question]

Hello...
Just wonder how to proper maintain connection.
When CP connects to server it should 1st send BootNotification. But I have station that can send StatusNotification (or others) before BootNotification. So how to proper handle that? I assume that all traffic before BootNotification should be suspended or connection colsed/refused.

Hey @eslavko
So the BootNotification is sent after an start-up, booting or rebooting process. Check section 4.2 on OCPP 1.6, so I believe not necessarily needs to be the first message after the charger connects to Central System.

What happens if you proceed? Does any error appear from your implementation?

Hello...
Actually I want to be little more paranoic and want to allow just known stations to connect. So the URL must match with the known data from boot notification data. As data is not available when connection is made I just want to reject all traffic until got BootNottification. After that if data here is expected one will allow all other functionality or reject station and droop connection.

@eslavko can you the wait for certain amount of time for the BootNotification then if you do not receive after this amount of time, you can close/refuse the connection? Would this work in your case? Just an idea! :)

Could work. For now I changed source charge_point.py and modify start routilne with this:
`#----------------------added by CAA
self.bootNotify=False
self.Exit=False
async def start(self):
while self.Exit==False:
message = await self._connection.recv()
if self.bootNotify==True or "BootNotification" in message:
LOGGER.info('%s: receive message %s', self.id, message)
await self.route_message(message)
#LOGGER.info('%s: Closing connection.', self.id)
await self._connection.close()
#LOGGER.info('%s: Connection closed. %s' % (self.id,self.Exit))
#----------------------
`
So when station is connected all message different of BootNotification is not routed. When BootNotification is received and data are valid I set bootNotify=True and all mesage from this time will pass.


Permalinks to features do not work if the feature is after the first page

How are you running Flagsmith

[ ] Self Hosted with Docker
[ ] Self Hosted with Kubernetes
[X] SaaS at flagsmith.com
[ ] Some other way (add details in description below)

Describe the bug
This link points to a feature in a project with more than 50 features, which causes pagination: https://app.flagsmith.com/project/18444/environment/UgLNVEFpLooRAwiwLdY8bR/features?feature=97961&tab=value
Because the target feature is located on the second page, the feature modal does not open and the link just lands you on the first page of features.
Steps To Reproduce

Create over 50 features in a project
Try to open a link to any feature after the first page

Expected behavior
Permalinks to features should work no matter how many features a project has.
Screenshots
No response

I suspect this will be resolved when #3301 is resolved (by #4188)


Import matmul/conv/attention tests from iree/tests/e2e/

Could start with what already exists, including the C++ binaries files and CMake build system:

https://github.com/iree-org/iree/tree/main/tests/e2e/matmul
https://github.com/iree-org/iree/tree/main/tests/e2e/convolution
https://github.com/iree-org/iree/tree/main/tools/testing/e2e
https://github.com/iree-org/iree/blob/main/build_tools/cmake/iree_e2e_generated_runner_test.cmake

Or... run the generator scripts and jump straight to a prototype replacement test runner (pytest? other cmake functions?)
The test generators seem to be using mostly upstream dialects (e.g. linalg.conv_2d_nchw_fchw). The build system usage does some fancy things with flags though... should watch out for layering in there. For example:
  COMPILER_FLAGS
    "--iree-opt-data-tiling=false"
    "--iree-llvmcpu-enable-scalable-vectorization"
    "--iree-llvmcpu-target-triple=aarch64-unknown-unknown"
    "--iree-preprocessing-pass-pipeline=builtin.module\(util.func\(iree-preprocessing-transpose-matmul-pass{input=lhs}\)\)"
  LABELS
    "requires-arm-sme"
  TARGET_CPU_FEATURES_VARIANTS
    "arm_64:sme:+sve,+sme"

See this related thread on Discord: https://discord.com/channels/689900678990135345/1270451599231156266

Can also draw on code from https://github.com/nod-ai/rocm-gemm-benchmark/. That has test case generators and checked in .mlir files.

Current plan (cc @erman-gurses ):

Land https://github.com/iree-org/iree/pull/17751 for interim coverage of attention on CPU (and possibly ROCm)
Fork C++ and CMake matmul test code to this repo (including all transitive deps, a new CMake project that depends on the runtime, etc.)
Fork C++ and CMake conv and attention test code to this repo
Prototype a new test structure (check in generated files, rework CMake/ctest project or use pytest, etc.)

Finding lots to clean up while refactoring the matmul tests. Debating starting fresh or cleaning up in-place.
Notes/ideas:

[ ] https://github.com/iree-org/iree-test-suites/pull/14 (run the generator offline and check in the test case files)
[ ] https://github.com/iree-org/iree-test-suites/pull/15
[ ] Remove the concept of "sizes" from the generator. Generate one file per test case with only a single function / matmul shape.
[ ] Maintain a list of interesting problem sizes sourced from general sweeps of common sizes and specific sizes used in popular workloads, like in https://github.com/nod-ai/rocm-gemm-benchmark/blob/main/gemmbench/problems.py
[ ] Come up with some directory structure that gets the number of files / test cases per folder down to under 100-1000 as needed. Maybe group by LHS/RHS data type, with deeper subfolders for transpose variants?
[ ] Drop the infer_acc_type helper function and make all calls into the generator fully explicit about what data types they want to use for the LHS, RHS, and accumulator


invalid Cookie.Domain "localhost:3000"; dropping domain attribute   - Which is valid!

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.10.3 darwin/amd64
NEXT STEP :: sudo port install go
go version go1.11.1 darwin/amd64
Does this issue reproduce with the latest release?
YES
What operating system and processor architecture are you using (go env)?
NAME  -  Is my username
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/NAME/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/NAME/go"
GOPROXY=""
GORACE=""
GOROOT="/opt/local/lib/go"
GOTMPDIR=""
GOTOOLDIR="/opt/local/lib/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="/usr/bin/clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/lc/2r1wlzgn6jb12zmbjk0syyh40000gn/T/go-build259772903=/tmp/go-build -gno-record-gcc-switches -fno-common"
What did you do?
If possible, provide a recipe for reproducing the error.
http - is import "net/http"
wr - is http.ResponeWriter
http.SetCookie(wr, &http.Cookie{
Name: "sid",
Value: cookie.Value, <------- Some Md5 value
Expires: expires, <----- NOW+1h
Domain: self.ConfGet("server.domain"),       // <------  self.ConfGet returns string  "localhost:3000"
Path: "/",
HttpOnly: true,
})
What did you expect to see?
Hmmm...  localhost:3000 is valid domain for any browser!!!!   "localhost" domain and "localhost:3000" are different domains!!!
What did you see instead?
2018/10/21 02:14:17 net/http: invalid Cookie.Domain "localhost:3000"; dropping domain attribute
2018/10/21 02:14:17 [::1]:53538 GET /static/ 200
2018/10/21 02:14:17 [::1]:53548 GET /echo 101
2018/10/21 02:14:17 net/http: invalid Cookie.Domain "localhost:3000"; dropping domain attribute

"localhost" domain and "localhost:3000" are different domains!!!

This is not accurate. A browser today shares cookies on localhost, regardless of the port.


Add artefact caching on github for Go builds & tests

Background
Currently, when running tests, go needs to download all dependencies. This takes about 1min and 40seconds on our CI.
Proposed solution
Add action/cache in between multiple runs to speed up the tests.

@mszuyev maybe good optimization for speed.

Moved to Volume


Error parsing reference: "ubuntu:14.04 As ubuntu" is not a valid repository/tag

When I try to install digits with Dockerfile 6.0 with command:
"nvidia-docker build ."
I got the error message.
"Error parsing reference: "ubuntu:14.04 as protobuf" is not a valid repository/tag"
If I remove the "as protobuf", the instruction is correct. what's the problem?
and I can run the digits  directly when I use this command :
"nvidia-docker run --name digits -d -p 5000:5000 nvidia/digits"
my dockre version 1.10.3

+1

https://docs.docker.com/engine/userguide/eng-image/multistage-build/

Multi-stage builds are a new feature requiring Docker 17.05 or higher on the daemon and client.

Docker minimum version requirement should be documented on the README.

I'm getting a similar error when I run on git lab CI:
$ docker --version
Docker version 17.10.0-ce, build f4ffd25

$ docker build -t $DOCKER_IMAGE_NAME:$GitVersion_SemVer --build-arg PROJECT=$WEB_PROJECT --build-arg VERSION=$GitVersion_SemVer   .
Sending build context to Docker daemon  933.9kB

Step 1/13 : FROM microsoft/aspnetcore-build:2.0 as builder
Error parsing reference: "microsoft/aspnetcore-build:2.0 as builder" is not a valid repository/tag: invalid reference format

But it works when I run locally, using the same dockerfile and all, but on docker on windows an "older" version:
$ docker --version
Docker version 17.09.0-ce, build afdb6d4


prkrishnPrkrishn/3135

Fixes #3135

cc @rynowak

prkrishnPrkrishn/3135 is probably the most important bug we've encountered. Glad to see a fix in flight!

Better?

:shipit:


add support for publishing percentile time series for the histogram m…

…etrics cql-requests, cql-messages and throttling delay.
Motivation:
Histogram metrics is generating too many metrics overloading the promethous servers. if application has 500 Vms and 1000 cassandra nodes, The histogram metrics generates<PHONE_NUMBER> = 50,000,000 time series every 30 seconds. This is just too much metrics.  Let us say we can generate percentile 95 timeseries for for every cassandra nodes, then we only have 1*500 = 500 metrics and in applciation side, we can ignore the _bucket time series. This way there will be very less metrics.
Modifications:
add configurable pre-defined percentiles to Micrometer Timer.Builder.publishPercentiles. This change is being added to cql-requests, cql-messages and throttling delay.
Result:
Based on the configuration, we will see additonal quantile time series for cql-requests, cql-messages and throttling delay histogram metrics.

Thanks for the PR @nparaddi-walmart !
Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

i already signed the CLA
Username
nparaddi-walmart
Name
Nagappa Paraddi
Email
<EMAIL_ADDRESS>Company
Walmart Associates Inc
Status
Completed

@absurdfarce you may be already aware. Just FYI. Snyk is failed with
"snyk requires an authenticated account. Please run snyk auth and try again."

One unrelated test-failure (ChannelPoolResizeTest#should_grow_during_reconnection / JAVA-3113)


parallel nodeSetup-execution breaks build

Hi there,
we use the gradle-node-plugin for our builds to have project specific node versions.
However, in our jenkins CI-Environment builds may break under the condition, that two or more projects with the same node version are built at the same time.
The nodeSetup-Task fails, because the node runtime is being downloaded twice.
:cms-client:nodeSetup FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':cms-client:nodeSetup'.
> Unable to expand TAR '/data/var/lib/jenkins/.gradle/caches/modules-2/files-2.1/org.nodejs/node/0.12.7/a861a6ded50c8cda32ac39c3a54699c767742e6/node-0.12.7-linux-x64.tar.gz'
    The tar might be corrupted or it is compressed in an unexpected way.

Any idea how to fix the problem so that we can run our builds parallel?

Same than #91

Fixed in 0.12-SNAPSHOT.


Updates DG

Codecov Report

Merging #304 (7b24501) into master (379c956) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #304   +/-   ##
=========================================
  Coverage     57.50%   57.50%
  Complexity      677      677
=========================================
  Files           131      131
  Lines          2605     2605
  Branches        274      274
=========================================
  Hits           1498     1498
  Misses         1011     1011
  Partials         96       96

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 379c956...7b24501. Read the comment docs.


[Request] char.escape scope support for common languages

Most languages hljs parser are not implementing char.escape.
Here are those who do it:
https://github.com/highlightjs/highlight.js/blob/2bd6ff65635640c99b04cd4217cfe45ca682f884/src/languages/nix.js#L49-L52
https://github.com/highlightjs/highlight.js/blob/2bd6ff65635640c99b04cd4217cfe45ca682f884/src/languages/elixir.js#L75-L79
https://github.com/highlightjs/highlight.js/blob/2bd6ff65635640c99b04cd4217cfe45ca682f884/src/languages/wren.js#L212-L220
https://github.com/highlightjs/highlight.js/blob/2bd6ff65635640c99b04cd4217cfe45ca682f884/src/languages/haskell.js#L198-L201
https://github.com/highlightjs/highlight.js/blob/2bd6ff65635640c99b04cd4217cfe45ca682f884/src/languages/llvm.js#L118-L122
https://github.com/highlightjs/highlight.js/blob/2bd6ff65635640c99b04cd4217cfe45ca682f884/src/languages/nsis.js#L190-L194
but all those are less common or very uncommon languages.
It would be nice to have escape sequence highlight in common languages such as ruby, json, javascript, python, etc.
Example for JSON:
Without:

With:

Would love to see anyone work on this - higher fidelity highlighting is always welcome - esp for simple things like this.  This is technically already covered by #2500 though, so I'm closing this as a duplicate.  If you'd like to contribute any of this work, just pick a grammar and jump in with a PR.


Thoughts/Notes on AlienFx

Thank you for the new release, sir.
Version <IP_ADDRESS> uses 2 megabytes more ram than previous version (3.x vs 1.x megabytes).
Would be good to use SHIFT, CNTL A, or click AND drag to select ALL the keys at once in SELECT LIGHTS FROM LIST (ZONES).
COLORS Window RESIZE Sliders are inoperative/greyed out. On GRIDS Page, sliders work fine. Is this normal procedure?
COLORS Window, beneath Action Settings, The MODE is set COLOR and greyed out. Are there other OPTIONS here to select or only COLORS?
My TRON items still do not respond to EFFECTS. The proper colors are displayed on the lights but items remain only one of the colors selected, BACKGROUND. Note, when I select RANDOM on the LIGHTS Page, My TRON items DO change color.
When I DETECT DEVICES, TOUCHPAD is included but my touchpad does not have lights. I cannot delete this entry. Do I need to create custom light grid?
DIM LIGHTS option works fine. No problem here. To this end, have you considered separating your versions? Is it reasonable to discontinue supporting older models in each new release and to have two separate versions?
I refer to what AWCC does. One version supports up to a certain level and another that picks up from where that version leaves off?
Pop up TOOL TIPS might be helpful in the SETTINGS Window for some of the items.
Thank you for your effort and your time.

Version <IP_ADDRESS> uses 2 megabytes more ram than previous version (3.x vs 1.x megabytes).

It's OK. Price for stability.

Would be good to use SHIFT, CNTL A, or click AND drag to select ALL the keys at once in SELECT LIGHTS FROM LIST (ZONES).

I check what i can do. There are 2 modes of list, now i use first one.

COLORS Window RESIZE Sliders are inoperative/greyed out. On GRIDS Page, sliders work fine. Is this normal procedure?

It's OK. Grid size can only be changed at "Devices&Grids" tab to avoid mess with zones.

COLORS Window, beneath Action Settings, The MODE is set COLOR and greyed out. Are there other OPTIONS here to select or only COLORS?

This is OK in case no action present/selected.

My TRON items still do not respond to EFFECTS. The proper colors are displayed on the lights but items remain only one of the colors selected, BACKGROUND. Note, when I select RANDOM on the LIGHTS Page, My TRON items DO change color.

Increase WIDTH - i mention it into release notes/readme before. WIDTH define number of phases for morph.

When I DETECT DEVICES, TOUCHPAD is included but my touchpad does not have lights. I cannot delete this entry. Do I need to create custom light grid?

You are got mappings from incorrect device. Just select it from light list there and press "Reset" button.

To this end, have you considered separating your versions? Is it reasonable to discontinue supporting older models in each new release and to have two separate versions?

No. One of the ideas behind these tools was i support ANY hardware in one app.
V1/V2 difference is fan access method only (some systems feels better with old V1 - it uses some hack tricks), but both support the same hardware.

Pop up TOOL TIPS might be helpful in the SETTINGS Window for some of the items.

A lot of effort and app size grow for small profit. Press "Help" in top menu - it opens manual with all options explained.

RAM Usage is back to 1.x ram.
FINALLY got the lights to work properly, duh... I added  new group in LIGHTS and moved the TRON keys to it. Works great now. TRON Lights are shown in Devices and Grids but did not transfer over to LIGHTS like the LIGHTS did. Thank you.
"Increase WIDTH - i mention it into release notes/readme before. WIDTH define number of phases for morph.
BTW, for Tron lights, effects defined at the "Color" tab using actions."

I did this and the keyboard had no other entries besides the keyboard. No Tron, etc. Also re-detected DEVICES and saw no change in keyboard options.
Unchecking POWER/INDICATOR Lights in SETTINGS does not extinguish the Power Light.
Just comments, my friend. I am very happy with current results.

Question: Can I sync the TRON items and the KEYBOARD to same effects or by using a single control?

You can create one zone for all of them and set it.

Unchecking POWER/INDICATOR Lights in SETTINGS does not extinguish the Power Light.

Yes - checking it at this tab, just report about type. It handled other way - then off/dim lights. Power/indicator can not set as off or dim with others. Power button is also tricky - it should have at least 2 colors (AC/BATT), so Color tab can not allow you to set only one.

AlienFX Configuration pop out window hangs when saving configurations.

It's a simple .cmd calling regedit. So it's about your system settings.