@event_trigger : event_type (first parameter) cannot be longer than 8 ??)

registered_triggers = list()

def create_event_trigger():
    @event_trigger("my_event1")
    def got_my_event(**kwargs):
        log.info(f"got my_event: kwargs={kwargs}")
    registered_triggers.append(got_my_event)

@time_trigger('startup')
def event_default_startup():
    create_event_trigger()
    event.fire("my_event1", my_param1=100, fruit="apple")

this doesn't work. It's almost the same as one of the examples.
when the event_type is "my_event", then it works. Apparenty, the event_type cannot be longer than 8 characters?
when I look in "developers utilties", and listen for "my_event1", it let me see that de event is fired. So, firing the event is no problem.
But, the      @event_trigger doesn't pick up an event type longer than 8 characters

What is your workflow?  Are you running this code in a single jupyter block?  Or is this code in a .py file?

I had a problem with the event trigger. So.i created a new py file with the above statements.
When I name the event_type 'my_event', it just works. The entire trigger is triggered.
When I do the same, with event_type 'my_event1'
Or any name longer than 8 characters, the event trigger is not triggered.
I can see in home assistant that the event is fired.
( I don't use Jupiter)

I'm sure this isn't an issue with the length of the event name.
pyscript doesn't enable triggers until after the script is completely loaded (eg, run).  So the event.fire happens before the event trigger is armed, which means it is missed.
Here's a modified version that fires the event 0.2 seconds after the script runs:
@event_trigger("my_event_long_name_1")
def got_my_event(**kwargs):
    log.info(f"got my_event: kwargs={kwargs}")

def delay_event_fire():
    task.sleep(0.2)
    event.fire("my_event_long_name_1", my_param1=100, fruit="apple")

task.create(delay_event_fire)

I will test this one of this days. I did something similar.... Don't understand why it didn't work. I'll let you know the result

Yes, seems to work.
added a task.sleep after my create_event_trigger, and then it seems to work.
Did that during testing too.... Anyway, I'll add the delay in my code...
Thanks you very much for your time.
and... happy new year ! :)

Thanks, a delay solved it.


Dino Game Clone

Hi, I am TRIDIB BAG. I'm  GSSoC22 participant.
I want to add Dino Game  clone with proper documentation as there is nothing regarding this . I'm really want to contribute in this topic .

/assign


cannot build crab-llvm due to an ocaml-llvm binding issue

Hi,
When building crab-llvm, I encountered an error at the step:
cmake --build . --target llvm && cmake ..
The reason is due to the compilation of ocaml-llvm binding. Here is the detailed log produced in the file crab-llvm/build/llvm-prefix/src/llvm-stamp/llvm-build-err.log.
File "llvm_executionengine.ml", line 54, characters 28-77:
Error: This expression has type nativeint
       but an expression was expected of type int64
make[6]: *** [bindings/ocaml/executionengine/llvm_executionengine.cma] Error 2
make[5]: *** [bindings/ocaml/executionengine/CMakeFiles/ocaml_llvm_executionengine.dir/all] Error 2
make[4]: *** [all] Error 2

This issue occurs with both OCaml 4.02.3 and OCaml 4.04.0 I googled for the solution of this issue but there is not much information, except the bug report: http://lists.llvm.org/pipermail/llvm-bugs/2015-June/040629.html
Do you know how to overcome this issue?
Thank you!

Crab-llvm does not use ocaml bindings so the solution is to disable it.
I'm afraid that I've not been able to find the cmake option to disable it but I'll keep trying.
In any case, my understanding was that llvm 3.6 disables ocaml bindings by default
https://www.wzdftpd.net/blog/ocaml-llvm-01.html
but I could be wrong.
Meanwhile, you can modify by-hand the file: build/llvm-prefix/src/llvm-build/CMakeLists.txt
and make sure you have this:
 //Path to a program.
 OCAMLFIND:FILEPATH=OCAMLFIND-NOTFOUND

With this change you might be able to continue with the compilation process.

Commit f25e469e8feb868645528e0e35905fda2d96fe42 should fix your problem.
Let me know if it does.

Thank you for the information! I build the latest version of crab-llvm https://github.com/seahorn/crab-llvm/commit/c5a3e7f9db5fba993b56197a0f9e2dcc00d003e3, but the OCaml-binding error still occurs:
File "llvm_executionengine.ml", line 54, characters 28-77:
Error: This expression has type nativeint
       but an expression was expected of type int64
make[6]: *** [bindings/ocaml/executionengine/llvm_executionengine.cma] Error 2
make[5]: *** [bindings/ocaml/executionengine/CMakeFiles/ocaml_llvm_executionengine.dir/all] Error 2
make[4]: *** [all] Error 2

I also have other versions of llvm installed on my machine: versions 2.9, 3.2, 3.9, 4.0. Do you think if this could a reason for the compilation error with llvm-3.6?

more likely the reason is that you have ocaml and llvm decides to build
ocaml bindings for you. We don't generally use ocaml, so we don't have it
on our system and this problem has gone unnoticed. Disabling ocaml, either
temporary on your machine or configuring  llvm to not build ocaml bindings
should solve it.
On Sun, Aug 20, 2017 at 21:52 tddsg<EMAIL_ADDRESS>wrote:

Thank you for the information! I build the latest version of crab-llvm
c5a3e7f
https://github.com/seahorn/crab-llvm/commit/c5a3e7f9db5fba993b56197a0f9e2dcc00d003e3,
but the OCaml-binding error still occurs:
File "llvm_executionengine.ml", line 54, characters 28-77:
Error: This expression has type nativeint
but an expression was expected of type int64
make[6]: *** [bindings/ocaml/executionengine/llvm_executionengine.cma] Error 2
make[5]: *** [bindings/ocaml/executionengine/CMakeFiles/ocaml_llvm_executionengine.dir/all] Error 2
make[4]: *** [all] Error 2
I also have other versions of llvm installed on my machine: versions 2.9,
3.2, 3.9, 4.0. Do you think if this could a reason for the compilation
error with llvm-3.6?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/seahorn/crab-llvm/issues/13#issuecomment-323628567,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFzN56WhuhClIq0nPK23iqLxNXg0TQPaks5saOLAgaJpZM4O7Psi
.

--
On the move ...

The commit f25e469e8feb868645528e0e35905fda2d96fe42 was trying to tell LLVM not to build ocaml bindings but you say it didn't work. I'm afraid that you will need to disable ocaml temporary while building llvm.

Thanks for the update! I try your manual solution to modify the file build/llvm-prefix/src/llvm-build/CMakeLists.txt but it seems that file doesn't exist in the llvm-build folder.
I see only the file build/llvm-prefix/src/llvm-build/CMakeCache.txt, but it is override every time I compile, hence modify the file CMakeCache.txt doesn't work either.

Why not make sure that ocaml cannot be found? i.e., make sure it's not in PATH

the file should be in the source, not the build directory. It would be
under ext/llvm/CMakeLists.txt
On Sun, Aug 20, 2017 at 10:05 PM, tddsg<EMAIL_ADDRESS>wrote:

Thanks for the update! I try your manual solution to modify the file
build/llvm-prefix/src/llvm-build/CMakeLists.txt but it seems that file
doesn't exist in the llvm-build folder.
I see only the file build/llvm-prefix/src/llvm-build/CMakeCache.txt, but
it is override every time I compile, hence modify the file CMakeCache.txt
doesn't work either.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/seahorn/crab-llvm/issues/13#issuecomment-323629939,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFzN55Ok1Ske_c1sE_r0TnOV86s_9pjMks5saOXrgaJpZM4O7Psi
.

Thank you all for your kind helps! I finally can compile crab-llvm by remove OCaml from PATH.
@agurfinkel : I also commented out the bindings setting in ext/llvm/CMakeLists.txt but it didn't work.

I'll close this issue even if it has not been solved properly.


New package: Thrift2 v0.1.0

Registering package: Thrift2
Repository: https://gitlab.com/ExpandingMan/Thrift2.jl
Created by: https://gitlab.com/ExpandingMan
Version: v0.1.0
Commit: acc603343c8087d99cfd85981a4ed4a8a6309884
Git reference: HEAD

Name similarity is expected, I want to merge this.


Add Friedrich-Ebert-Schule Pfungstadt

Add fes365-pfungstadt.de

@Silas2701 I am sorry for a delay with processing your request.
Pull request merged. Thank you!

@Silas2701 I am sorry for a delay with processing your request.
Pull request merged. Thank you!

Hi Philip, thanks a lot :)--Diese Nachricht wurde von meinem Android Mobiltelefon mit GMX Mail gesendet.Am 10.01.21, 00:06 schrieb Philip Torchinsky<EMAIL_ADDRESS>@Silas2701 I am sorry for a delay with processing your request. Pull request merged. Thank you!
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

Hi Philip, thanks a lot :)--Diese Nachricht wurde von meinem Android Mobiltelefon mit GMX Mail gesendet.Am 10.01.21, 00:06 schrieb Philip Torchinsky<EMAIL_ADDRESS>@Silas2701 I am sorry for a delay with processing your request. Pull request merged. Thank you!
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.


Will there be support for version 10?

These days I wanted to connect the plugin to Grafana but I have not been able to because it has been updated to version 10. I would like to know if there will be support for this version in the near future. Thank you very much!

yes, the plugin will support Grafana 10 version. I'll spend sometime on this next week.


Gem requires 2.3.0dev ?

In trying to upgrade to the latest, I'm given the warning that it requires 2.3.0dev (currently running ruby 2.1.5).  Is this correct?
 $ gem update did_you_mean                                                                                22:57:42
Updating installed gems
Updating did_you_mean
ERROR:  Error installing did_you_mean:
	did_you_mean requires Ruby version >= 2.3.0dev.

Is this correct?  gemspec and rubygems.org both indicate:
REQUIRED RUBY VERSION:
>= 1.9.3

Note: I just upgraded my OSX, so this could be an un-related issue.

@tleish It's totally correct. The required ruby version (which is 2.3.0dev) looks incorrect but is actually also correct. The version 1.0.0 has been prepared exclusively for Ruby 2.3 and doesn't work on any other Ruby implementations. I'm not going to talk about the circumstances here, but please use 0.10.0 if you use Ruby 2.2 or older.


Toggle Menu Bar

My Setup

Operating System: Windows
App Version: 2.10.5

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description

It is not possible to toggle the menu bar if it is hidden
The shortcut [Alt] + [V] isn't working
I think it isn't an electron-bug, because in Atom this shortcut works
Current Behavior

The menu-bar is hidden for ever :(
Expected Behavior

I'd like to get my menu-bar back.. sometimes..

Just hit [alt] once (not [alt gr]) and the menu bar will show up temporary. Now you able to show it permanent via Toggle Menu Bar.

Thank you so much!


Doesn't actually move to the selected pane when it's in the same session

It appears to be there is a bug and the behavior that I discovered is like this: it can choose specific tab and specific pane as long as choosing a different session but nothing happens when choosing things within the same session.
tested versions:

zellij 0.41.1
zellij-choose-tree 0.4.1

Thanks for bringing this up, I think the switch function provided by the zellij plugin api fails if the session is already active and the other params are just ignored. I'll look into this edge case and use other api calls when I have some time.


Handle boolean properties

Same as for attributes, boolean properties must be handled correctly. For example, in the Chrome console:
> elem
<input type="checkbox" name="collab_privs" class="js-collab-option" checked>
> elem.checked
true
> elem.checked = "false"
"false"
> elem.checked
true
> elem.checked = false
false
> elem.checked
false

Isn't this already handled by #62 ? @cornerman ?

Good catch! This is also not handled in the scala-dom-types branch. Maybe fix it there?

Yes, fixing it in #62.

Actually, I think we do not need to handle this. You are just experiencing type conversion in javascript, the truthy value "false" is interpreted as true. You can set it to "" or 0 and it is false. Now checked is just a property, which is the current value of the radio/checkbox. And the attribute checked which you set in your html element corresponds to the property defaultChecked. Initially checked attribute and property are the same but they are not reflected.
For snabbdom, boolean handling only exists for attributes and not for properties (see https://github.com/snabbdom/snabbdom#the-props-module - this readme is a bit outdated with regards to boolean attributes, check the release text for that part: https://github.com/snabbdom/snabbdom/releases/tag/v0.7.0).

I was actually using checked as a prop on my branch at it wasn't working properly until I fixed this. I'll test if the issue persists with #62 and if it does, I'll add a test for it.

Thanks, please add a test there and check whether you need the commit for boolean properties (best with a reflected attribute like e.g. disabled. You can hopefully remove the commit :)

I verified that we need to handler boolean props on #62. I pushed a fix and a test case.


Building omnisharp on new windows box: missing 4.0 reference assemblines

At the moment I have installed:

windows 10 home edition 10.0.15063 Build 15063
visual studio community 2017 15.2
sync'd the omnisharp server repo
updated submodules

msbuild doesn't seem to be in my path (forgive my windows dev n00bness), so I'm using c:/Program Files (x86)/Microsoft Visual Studio/2017/Community/MSBuild/15.0/Bin/MSBuild.
When I run it, I get repeated errors about missing .net 4.0 reference assemblies:
c:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(
1111,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v4.0" were not found. To resolve
this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the fra
mework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global As
sembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targ
eted for the framework you intend. [c:\Users\Andrew\Documents\dev\personal\omnisharp-server\OmniSharp\OmniSharp.csproj]

When I try to install 4.0 from the web installer, I'm told I have it already, in c:\window\microsoft.net.
Any ideas what I'm doing wrong? What more information can I give you?

fwiw I managed to get it installed vial the install-omnisharp command in emacs, but would still like to know why my direct compilation didn't work.

I suspect this was a bad install of vs2017: I was having other issues until I repaired via the installer, and now those other issues are resolved. I bet this is too.


Hello 👋(Code-Review)

Hey people 👋
Great work so far 🎉 You've been able to keep a good balance of front-end / back-end / documentation / tests. I know you've decided to switch DBs to Airtable which has set you back a little, but you're doing an excellent job so keep it up 👍
I'm aware you've only got a week left, so feel free to ignore my suggestions if you've got more urgent things to focus on. Hopefully they will also be useful for future projects.
Also please do let me know if I've said something which doesn't make sense, and apologies if you were already aware of an issue, or you had a good reason for doing something a certain way.
Best of luck for the final MVP 🥇

Cheers man.
I am a little confused about the 500 error still, are you saying it should be on the front end? Rather than the Backend? Or should I have one in both? Your suggestions have definitely helped! Really useful for fixing certain glaring errors.
Also, have you looked at our routes tests on the front end? Do you think I should test every route? Also do you think it might be a good idea to somehow test the URL or status code on the front end rather than the text in the first element?

Hi @harryyy27
You want both for 500 error.
A 500 error means an Internal Server Error, so you definitely want to catch it first from the backend. You've almost setup the 500 error route (don't forget about the bug mentioned in #74):
app.use((err, req, res, next) => {
    res.status(500)
    res.render('500') // res.render() will throw an error because you're not using a rendering engine
    // you want res.send();
})

Once you send the 500 response back to the client - one option would be to route the user to an error page. For example, in contactUs.js lets say when you do the api call, it returns a 500 error. It will go to the catch(), which we can use to redirect the user:
fetch('/api/contactUs', {
            method: 'POST',
            headers: {
                "Content-Type": "application/json"
            },
            body: data
        })
            .then(res => console.log(res))
            .catch(err => {
                console.log('ERROR IS', err)
                this.props.history.push('/server-error');
            })

Then you could have a /server-error route set up which renders a ServerError component (ie: a message telling the user there has been a server error):
<Route path ='/server-error' component = {ServerError} />
You don't have to do this exact approach, but the general idea is:

Pick up error on express server - the express docs say synchronous errors are automatically sent to your error handling route, but for asynchronous errors - you need to call next(err)
Send a response to front end with a status of 500, and optionally a message with more detail
Pick this up in the front-end, and do something useful with it (eg: reroute user, show them a message)

Does that make sense?

 I haven't used react-testing-library, but I've heard good things! Might be worth talking to someone with more experience of it.
What I would say is try to not to test things about the implementation that need to be flexible. For example, if the text-content is going to be changed and updated, it's probably not going to helpful to test (your tests will keep breaking).
Testing things that should remain consistent will be useful. For example:

URLs probably would be helpful as they should be consistent.
The behaviour of a menu bar opening and closing.
An error page rendering if your api responds with an error


Sub menu 'Import / Export CSV' not shown in 'System' menu in other languages

I have Symphony 2.3.1 and also the french translation. My Symphony is setup to use french.
Once your extension installed, I don't see the sub menu "Import / Export CSV" in the "System" menu.
But if, in this file, I change line 26 to:
'location'   => _('System'),
it works.
It also creates the same problem with the Italian language, but I didn't test other translations.
How to reproduce:

Install french language extension
Enable french in Preferences
Install 'Import / Export CSV'
Problem: the menu is not added to 'System'

Repo moved to: https://github.com/symphonists/importcsv


[Release] sdk/resourcemanager/networkfunction/armnetworkfunction/2.0.0

https://github.com/Azure/sdk-release-request/issues/3426

Hi @ArcturusZhang @lirenhe @tadelesh can we please get an approval if everything looks ok?

I regenerated, but found no new changes

Do you mean the breaking changes are gone?

I regenerated, but found no new changes

Do you mean the breaking changes are gone?

swagger no new changes https://github.com/Azure/azure-rest-api-specs/tree/main/specification/networkfunction/resource-manager/Microsoft.NetworkFunction/stable/2022-11-01

The breaking changes have been approved in this PR: https://github.com/Azure/azure-rest-api-specs/pull/20942


Failing in runing "00-classification.ipynb" example GPU mode with ARM Mali GPU

Hey @psyhtest and @gfursin:
I really appreciate your help thus far. Sorry that I run into more problems again.
I installed my caffe as suggested by you with the command
ck install package:lib-caffe-bvlc-opencl-clblast-universal --env.DISABLE_DEVICE_HOST_UNIFIED_MEMORY=ON --env.CK_HOST_CPU_NUMBER_OF_PROCESSORS=2 --env.CAFFE_BUILD_PYTHON=ON --env.CK_MAKE_CMD2="make pycaffe"

I have some problems run the example "00-classification.ipynb," provided by Caffe in the directory
~/CK-TOOLS/lib-caffe-bvlc-opencl-clblast-master-gcc-5.4.0-linux-32/install/examples

I run by the command
ck xset env tags=lib,caffe && . ./tmp-ck-env.bat && jupyter notebook

I can perfectly import caffe, and make the cpu load the weights, deploy, and run.
But I cannot make the gpu run. the program died in the line
caffe.set_device(0)  # if we have multiple GPUs, pick the first one
caffe.set_mode_gpu()
net.forward()  # run once before timing to set up memory
%timeit net.forward()

it produced error message
Error Message:
I0724 17:09:56.982797 25836 device.cpp:56] CL_DEVICE_HOST_UNIFIED_MEMORY: disabled
std::exception

when I opened the debugger, the error message changed into

Can you guys reproduce the error? Have you guy successfully run the Caffe Example on the Mali GPU ?
///////////////////////////////////////////////////////////////////////////////////////////////////////////////
P.S.
Suddenly, it reminded me the command for the installation of ck-caffe in #114 sugggested by @psyhtest :
ck install package:lib-caffe-bvlc-opencl-clblast-universal \
  --env.DISABLE_DEVICE_HOST_UNIFIED_MEMORY=ON \
  --env.CK_HOST_CPU_NUMBER_OF_PROCESSORS=2

Espeically
--env.DISABLE_DEVICE_HOST_UNIFIED_MEMORY=ON

what does flag do? why does it cause the kernel to die?
Furthermore, I have tried to install by command
ck install package:lib-caffe-bvlc-opencl-clblast-universal \
  --env.CK_HOST_CPU_NUMBER_OF_PROCESSORS=2

without the flag
--env.DISABLE_DEVICE_HOST_UNIFIED_MEMORY=ON

With this option, I can import caffe, but when I am trying to load the weights, the program's kernel died with after line
net = caffe.Net(model_def,      # defines the structure of the model
                model_weights,  # contains the trained weights
                caffe.TEST)     # use test mode (e.g., don't perform dropout)

with error

the error in command line is

It died while attemping to load data for weights.

@XiaoMaol First, let me explain what the DISABLE_DEVICE_HOST_UNIFIED_MEMORY flag does. It was introduced as a workaround for a rather annoying ambiguity of the OpenCL specification that says that CL_DEVICE_HOST_UNIFIED_MEMORY property of clGetDeviceInfo():

Is CL_TRUE if the device and the host have a unified memory subsystem and is CL_FALSE otherwise.

Now, what exactly is "unified memory subsystem" is open to interpretation. When people come from desktop and server background, they assumed that memory is unified in the OpenCL 2.0 sense: the host and the device can share the same pointers; therefore, no copy between the host address space and the device address space is required. As a consequence, you can see code like this:
#define ZEROCOPY_SUPPORTED(device, ptr, size) \
             (device->is_host_unified())
<...>
              CHECK_EQ(mapped_ptr, cpu_ptr_)
                << "Device claims it support zero copy"
                << " but failed to create correct user ptr buffer";

(see syncedmem.cpp)
Unfortunately, mobile GPU vendors started returning CL_TRUE even for OpenCL 1.x implementations. In their interpretation, the CPU and the GPU in a system-on-chip typically share the same physical memory; therefore, they argue, the memory subsystem is unified. In fact, the CPU and the GPU cannot share the same pointers. You will hopefully see how this is at odds with the expectation from applications like OpenCL Caffe, and leads to checks similar to the above failing at runtime.
To workaround this issue, you can explicitly specify that you wish Caffe to ignore whatever the driver is saying, and assume that the memory subsystem is not unified.
Now, the output you posted seems to suggest that disabling this property leads to an exception:
Error Message:
I0724 17:09:56.982797 25836 device.cpp:56] CL_DEVICE_HOST_UNIFIED_MEMORY: disabled
std::exception

I believe, however, this is just an unfortunate intermixing of log info (which gets printed even when everything goes well) with the exception message.

You may now ask, why do you have this Jupyter error in the first place? The honest answer is that I have no idea :). But if I were to guess, the device query seemed to be somewhat different for OpenCL. For example, for [program:caffe](https://github.com/dividiti/ck-caffe/blob/master/program/caffe/.cm/meta.json), the query_gpu_cuda command looks like:
"run_cmd_main": "$<<CK_CAFFE_BIN>>$ device_query --gpu=$<<CAFFE_COMPUTE_DEVICE_ID>>$"

while the query_gpu_opencl command looks like:
"run_cmd_main": "$<<CK_CAFFE_BIN>>$ device_query"

So there may well be some difference in how a device gets selected too.
Maybe @naibaf7 has a better idea?

Without having read the whole context, it is just important to use set_mode_gpu() before set_device(), otherwise it will fail. But this is the only "more strict" rule compared to CUDA Caffe.

From @XiaoMaol's initial comment, the order of these calls is reversed:
caffe.set_device(0)  # if we have multiple GPUs, pick the first one
caffe.set_mode_gpu()

Hey, @naibaf7 and @psyhtest:
I have tried to reverse the order of the call, but it still does not work. I think the problem is somewhere else, but I do not know where.
what other more information can I provide to give a better description of the problem? Thanks!

Today, I have tried Caffe without the Python Layer, and this is what I get.
When I tried to run with the GPU device 0, the program is killed:

when I run the program with device1, I have gotten that (Notice that the first iteration takes around 6 mins, then the program is killed):

However, when I have tried to run device 1 again, the program seems to freeze.
When I tried Caffe with gdb,  the program just seems to freeze.

Sounds a lot like a faulty driver. Can you try an absolute minimal network with just one fully connected layer to differentiate if Caffe works on that driver at all or not?

Hey @naibaf7, @psyhtest :
The Caffe can run a smaller network. In addition, when I reduce the input image size of the Alexnet into 1 from 10, it also works. Thus, it seems to be a problem of memory rather than Caffe itself.
However, I am suspecting the crash is due to the memory of my device rather than GPU memory.
I am currently also using ARM computing library as well. Its AlexNet also crash when the input batch number is more than 5 or 6. (Here is the issue: https://github.com/ARM-software/ComputeLibrary/issues/190)
and the memory of the device is

The platform is
Linux odroid 3.10.103-124 #1 SMP PREEMPT Tue Oct 11 11:51:06 UTC 2016 armv7l armv7l armv7l GNU/Linux

I am just wondering, Is 2GB of the memory too small to properly Caffe?
The GPU info is:

@XiaoMaol On Odroid and similar system-on-a-chip platform, the total system memory (e.g. 2 GB) is shared between all the devices. The GPU doesn't have any dedicated memory like on desktop or server cards. When you run out of this memory, you cannot use the GPU either.


Selecting enum dropdown drags node

When selecting an enum in node it drags the node and has to be repositioned.
https://streamable.com/nnf101

Hello,
I already experienced this issue with ShaderGraph blackboard, I think this is a GraphView bug as it was reported here: https://issuetracker.unity3d.com/issues/hdrp-graph-inspector-window-sticks-to-the-mouse-cursor-on-changing-any-dropdown-value-in-node-settings

Hello,
I already experienced this issue with ShaderGraph blackboard, I think this is a GraphView bug as it was reported here: https://issuetracker.unity3d.com/issues/hdrp-graph-inspector-window-sticks-to-the-mouse-cursor-on-changing-any-dropdown-value-in-node-settings

Issue fixed by Graphview team for 2021.2.X


Pipfile.lock install faild

Installation of the firmware scanning environment EMBArk
Reading package lists...
Building dependency tree...
Reading state information...
build-essential is already the newest version (12.9).
default-libmysqlclient-dev is already the newest version (1.1.0).
The following packages were automatically installed and are no longer required:
g++-9 libstdc++-9-dev
Use 'apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 461 not upgraded.
Requirement already satisfied: pipenv in /usr/lib/python3/dist-packages (2022.12.19)
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
www-embark
Install embark daemon
Creating SSL Cert
Certs already generated, skipping
Install hostnames for local dns-resolve
hostname already in use!
Installing dependencies from Pipfile.lock (bcd04c)...
An error occurred while installing apscheduler==3.10.0 ; python_version >= '3.6' --hash=sha256:575299f20073c60a2cc9d4fa5906024cdde33c5c0ce6087c4e3c14be3b50fdd4 --hash=sha256:a49fc23269218416f0e41890eea7a75ed6b284f10630dcfe866ab659621a3696! Will try again.
An error occurred while installing asgiref==3.6.0 ; python_version >= '3.7' --hash=sha256:9567dfe7bd8d3c8c892227827c41cce860b368104c3431da67a0c5a65a949506 --hash=sha256:71e68008da809b957b7ee4b43dbccff33d1b23519fb8344e33f049897077afac! Will try again.
An error occurred while installing async-timeout==4.0.2 ; python_version >= '3.6' --hash=sha256:8ca1e4fcf50d07413d66d1a5e416e42cfdf5851c981d679a09851a6853383b3c --hash=sha256:2163e1640ddb52b7a8c80d0a67a08587e5d245cc9c553a74a847056bc2976b15! Will try again.
An error occurred while installing attrs==22.2.0 ; python_version >= '3.6' --hash=sha256:29e95c7f6778868dbd49170f98f8818f78f3dc5e0e37c0b1f474e3561b240836 --hash=sha256:c9227bfc2f01993c03f68db37d1d15c9690188323c067c641f1a35ca58185f99! Will try again.
An error occurred while installing autobahn==23.1.2 ; python_version >= '3.7' --hash=sha256:c5ef8ca7422015a1af774a883b8aef73d4954c9fcd182c9b5244e08e973f7c3a! Will try again.
An error occurred while installing automat==22.10.0 --hash=sha256:e56beb84edad19dcc11d30e8d9b895f75deeb5ef5e96b84a467066b3b84bb04e --hash=sha256:c3164f8742b9dc440f3682482d32aaff7bb53f71740dd018533f9de286b64180! Will try again.
An error occurred while installing certifi==2022.12.7 ; python_version >= '3.6' --hash=sha256:35824b4c3a97115964b408844d64aa14db1cc518f6562e8d7261699d1350a9e3 --hash=sha256:4ad3232f5e926d6718ec31cfc1fcadfde020920e278684144551c91769c7bc18! Will try again.
An error occurred while installing cffi==1.15.1 --hash=sha256:04ed324bda3cda42b9b695d51bb7d54b680b9719cfab04227cdd1e04e5de3104 --hash=sha256:198caafb44239b60e252492445da556afafc7d1e3ab7a1fb3f0584ef6d742375 --hash=sha256:2012c72d854c2d03e45d06ae57f40d78e5770d252f195b93f581acf3ba44496e --hash=sha256:3799aecf2e17cf585d977b780ce79ff0dc9b78d799fc694221ce814c2c19db83 --hash=sha256:1e74c6b51a9ed6589199c787bf5f9875612ca4a8a0785fb2d4a84429badaf22a --hash=sha256:3bcde07039e586f91b45c88f8583ea7cf7a0770df3a1649627bf598332cb6984 --hash=sha256:40f4774f5a9d4f5e344f31a32b5096977b5d48560c5592e2f3d2c4374bd543ee --hash=sha256:3d08afd128ddaa624a48cf2b859afef385b720bb4b43df214f85616922e6a5ac --hash=sha256:a66d3508133af6e8548451b25058d5812812ec3798c886bf38ed24a98216fab2 --hash=sha256:87c450779d0914f2861b8526e035c5e6da0a3199d8f1add1a665e1cbc6fc6d02 --hash=sha256:a0f100c8912c114ff53e1202d0078b425bee3649ae34d7b070e9697f93c5d52d --hash=sha256:e263d77ee3dd201c3a142934a086a4450861778baaeeb45db4591ef65550b0a6 --hash=sha256:3548db281cd7d2561c9ad9984681c95f7b0e38881201e157833a2342c30d5e8c --hash=sha256:8102eaf27e1e448db915d08afa8b41d6c7ca7a04b7d73af6514df10a3e74bd82 --hash=sha256:a591fe9e525846e4d154205572a029f653ada1a78b93697f3b5a8f1f2bc055b9 --hash=sha256:e00b098126fd45523dd056d2efba6c

Sorry, I can't really help you like that.
The pipenv install  command is throwing errors, I can tell only this much
Try executing that command by hand.
Since this is the python-package-manager it can have a number of reasons.

no internet connection
pip needs a proxy setup
etc

@Roksolana969 can we have an update on this issue?

Please re-open it with further details.


Request for Contribution

Hey Alexander,
do you accept pull requests for this project?.
I've made some changes to the UI:

replaced your Check Button with an Icon. The small icon is a Folder with a check mark and a X on it. Color primary.dark
change the Icons on the RoundCard.  I'd like to have them red and green, but the original icons didn't look nice then. So i change the check mark and the x and also the empty box is now an icon instead of an div with a border. I just looks better then. Colors: primary.dark, secondary.dark
removed some white space on the RoundCard to have it more compact

Add some features:

added a very simple, not yet complete, language switch. Needs to be changed to a dropdown, persistence  need's to be changed.
add the possibility to download/upload the state Clipboard with corresponding icons , not yet started. Just something I've in mind, due to the fact that I used to different  devices to log my plays.

Kind Regards Frank

Hey Frank,
the changes look great. Feel free to create the pull request.
Importing/Exporting the state from the Clipboard should be quite easy. The whole app state is already persisted in LocalStorage.

I'm done with the UI changes, more or less. There is a change in the layout of the criteria cards that is completely up to my preference.
I don't like the layout of the criteria cards in an extreme game.
A  B      but         A  A
A  B      ------>     B  B
C  D      i like      C  C
C  D                  D  D

So I've changed Comments.tsx and `Comment.tsx. Where the layout is much simpler and less cluttered now.
I currently stopped working on import/export. Due to the fact that the persistence layer has some flaws.
The game hash in saves the is the primary key, therefore it's not possible to play a game twice, without overwriting the state of the previous play. And it's also not multi user capable, so an other 'user' will overwrite the state too.

@frudolph77 I mostly play on standard or nightmare mode. If this feels better for you, and cleans up the code, this seems like a good change.

I will create a PR later this weekend, and you can take a look a the code.


Update config.yaml

@stefanstrydom I believe this PR was not supposed to be created here, right? :)

@stefanstrydom I believe this PR was not supposed to be created here, right? :)

Correct! lol Sorry clicked the wrong thing! If you able to remove/Delete this PR please! :)

Correct! lol Sorry clicked the wrong thing! If you able to remove/Delete this PR please! :)

No worries at all! :)

No worries at all! :)


Create basic tests for validating parity with Ergast

It was suggested on today's status call to create some basic and automated testing scripts to help validate that we've reached feature parity with Ergast and to ensure that while we are fixing items during beta testing that we don't inadvertently introduce regressions.

I think this can be closed as we already have tests comparing against ergast responses for parity and preventing regression.
See here: tests/fixtures/ergast_responses


Fix widgets ui

@javierarce could you take a look?

I think this could have been extracted and merged to master (since there's not directly related to analysis), but anyways, it's :+1:


Move IDL files *.matter to reside together with input zap files

Problem
zzz_generated files are generally not expanded for code review, but we do want .matter files to be so (they are designed explicitly for reviewing changes for spec/sanity etc).
Change overview
Change zap generate to move Cluster.matter files to sit and be named as their corresponding .zap file.
Testing
Manually ran generation. Also CI will validate that zap passes.

fast track: a rename of files that do not affect how matter sdk actually runs. A review of the python bits should be sufficient.


Fix mobile first responsiveness #220

In dev screenshot of resolved issue,

@jain-anshika @jayesh2474 The checks failed. Can you review the PR?

@ashishk1331  There are some issues and we are fixing it! We will review and merge your PR soon!!

It seams that you add new packages in the package.json file which has installed node-gyp as an optional package which always causes problem with the deployment.
To address issue I ask of you to remove all the node-gyp installation that is present in the yarn.lock file
May I also know what packages did you add into the package.json file

Okay. I'll remove the lock files and try again.

Since the deployment works and the checks have all passed, I will merge it
Thank you for your contribution

Yes sir.


chore(deps): upgrade "Avalonia" dependencies to 11.0.0

Breaking Changes (0.10->11.0)
Avalonia 11.0.0 Release Candidate 1.1 was released on May 31st.

I've tried 11.0.5, there are big breaking changes there.
For example some interfaces are restricted form implementation:

Interface member 'void Avalonia.Platform.IGeometryImpl.(This interface or abstract class is -not- implementable by user code !)()' is not implemented Interface member 'void Avalonia.Platform.ITransformedGeometryImpl.(This interface or abstract class is -not- implementable by user code !)()' is not implemented

Since Consolonia implements a custom backend, it makes sense to enable private apis in the csproj:
<AvaloniaAccessUnstablePrivateApis>true</AvaloniaAccessUnstablePrivateApis>

I would love to help with this port. Is there a branch?

@tomlm I have tried to upgrade to 11.0.9, just tried to introduce required members etc. Pushed the branch here https://github.com/jinek/Consolonia/tree/105-choredeps-upgrade-avalonia-dependencies-to-1100
Let me know if you started to work on this, please. Otherwise I will work within this branch.
My branch has many compilation errors, probably need to address this at first. The abstraction changed more than I thought.

I have brought it to the point where it draws something. From now what's left is we need new control templates (old does not work well). Primitives drawing seems broken a bit, text formatting does not work well etc.
At the screenshot there is an example old Consolonia vs new Consolonia.

DataGrid is broken itself thus manual navigation in the gallery is not possible. It should be run with an argument - name of the name of the case in gallery:

For keyboard navigation you might consider using XYFocus class APIs that is available in 11.1.
It was primarily implemented for TVs with only arrow keys as a navigation, but might be applied here too.
https://github.com/AvaloniaUI/Avalonia/pull/13947

For keyboard navigation you might consider using XYFocus class APIs that is available in 11.1. It was primarily implemented for TVs with only arrow keys as a navigation, but might be applied here too. https://docs.avaloniaui.net/docs/concepts/input/focus#directional-focus-navigation- AvaloniaUI/Avalonia#13947

Then either in this task we need to replace Consolonia XY navigation https://github.com/jinek/Consolonia/blob/773b1e6669e77324cbc9bb89b3c1fb7bd47f9526/src/Consolonia.Core/Infrastructure/ArrowsAndKeyboardNavigationHandler.cs or we can create a separate one.
But still, currently even TAB navigation does not work, needs investigation.

I'm currently stuck with this version: https://github.com/jinek/Consolonia/tree/105-choredeps-upgrade-avalonia-dependencies-to-1100
Initially, everything renders correctly:

However, after clicking a button, only that button and its parent elements are rendered, while the rest of the interface disappears:

Resizing the window triggers a complete redraw of the entire screen and everything rendered fine.
First, I thought implementing layers correctly would resolve the problem, but it turned out to be irrelevant.

@tomlm I have tried to upgrade to 11.0.9, just tried to introduce required members etc. Pushed the branch here https://github.com/jinek/Consolonia/tree/105-choredeps-upgrade-avalonia-dependencies-to-1100 Let me know if you started to work on this, please. Otherwise I will work within this branch. My branch has many compilation errors, probably need to address this at first. The abstraction changed more than I thought.
I have gotten caught up with summer fun and completely not started this (not in any meaningful way). Sorry!

I've finished fixing major issues, now some controls either don't work properly or does not render properly. It needs template/styling update and may be some extension of them:

DataGrid

No header bottom border (line)
No alternate rows
Items can not be selected by clicking on the row outside of text

ComboBox

DropDown - overflow of the bottom border
Array is not rendered as several elements anymore (3rd ComboBox in gallery)

Calendar

Does not work at all

Calendar Picker

Shift+Tab does not bring focus back to previous element

ListBox

Same as DataGrid: Items can not be selected by clicking on the row outside of text
Check what "Toggle" is supposed to do
Check what is "Always selected"

TextBox

CTRL+RIGHT/LEFT does not do anything
Center/right aligned text: cursor is shifted one character left
Wrapped text - wrong behavior when text line hits the right end of textbox

Dialog

Show dialog does not work

Menu

Separator is probably not rendered

ProgressBar

Undetermined progress bar does not move the position

Working branch is here: https://github.com/jinek/Consolonia/tree/105-choredeps-upgrade-avalonia-dependencies-to-1100

I have created the pull request, now it's need a cleanup, quality checks etc
https://github.com/jinek/Consolonia/actions/runs/11420348010/job/31775946159?pr=119
and
https://github.com/jinek/Consolonia/actions/runs/11420348011/job/31775950380?pr=119


Entry not found in cache

Does this occur consistently?
Repro steps:

Action: azureFunctions.deploy
Error type: Error
Error Message: Entry not found in cache.
Version: 1.0.1
OS: win32
OS Release: 10.0.19042
Product: Visual Studio Code
Product Version: 1.51.1
Language: en

Call Stack
s.createError extension.js:16:58035
b.<anonymous> extension.js:393:42004
extension.js:393:36599extension.js:393:36599
extension.js:243:75486extension.js:243:75486
extension.js:243:73467extension.js:243:73467
extension.js:243:73087extension.js:243:73087
i.find extension.js:243:107619
extension.js:243:53535extension.js:243:53535
processTicksAndRejections task_queues.js:94:5

As a workaround you could try reloading your window or signing out and signing back in to your Azure Account using the command palette.
Unfortunately there's not much we can do in this extension, but this is being tracked upstream here: microsoft/vscode-azure-account#53


Lock a series when it's being edited

We should lock a series when it's being edited

Reassigning to @Coenego for review

Coverage remained the same at 100.0% when pulling a6dec8a88f485084725e12fd6f855d0b31ac0a63 on bp323:locking into 045b1097d37c759fa3b9452f3605e0b74a7cafbd on CUL-DigitalServices:master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>55bfe7a93c73eba3e4deef6d160ae2 on bp323:locking into 045b1097d37c759fa3b9452f3605e0b74a7cafbd on CUL-DigitalServices:master.


Attack Path: Unauthorized access to vehicular cloud interfaces (APIs)

Exploit weaknesses in authentication mechanisms to gain unauthorized access to vehicle data or control systems through cloud-based interfaces.


spelling: geometry

I'm not entirely certain what this test is trying to accomplish...

Thanks, @jsoref.  The test is only asserting that the source has a feature, but not that the feature has the expected geometry property, so the typo didn’t cause a failure.


Change "Skip Story & Attempt" on buttons to just "Attempt"

Simple change to remove "Skip Story" on the buttons displayed when trying to access an assessment for the first time. I took the opportunity to modify the icon used as well, from "step-forward" (similar to a 'skip' icon on videos) to "play".

This change is meant to resolve issue #725.

Pull Request Test Coverage Report for Build 2233

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 34.296%

Totals

Change from base Build 2207:
-0.005%

Covered Lines:
1777

Relevant Lines:
4562

💛  - Coveralls

I propose we merge this. @geshuming , what do you think?
There is no need to mention any "story" in the mission overview. We let students do the "obvious" thing and go straight to the mission.
The "stories" are what you see in the game. They should not appear elsewhere, in my view.


516 replace include throught source in test-py action

Merge Request - GuideLine Checklist
Guideline to check code before resolve WIP and approval, respectively.
As many checkboxes as possible should be ticked.
Checks by code author:
Always to be checked:

[x] There is at least one issue associated with the pull request.
[x] The branch follows the naming conventions as defined in the git workflow.
[x] New code adheres with the coding guidelines
[ ] Make sure that the pre-commit linting/style checks pass.

If functions were changed or functionality was added:

[ ] Tests for new functionality has been added
[x] A local test was succesful

If new functionality was added:

[ ] There is appropriate documentation of your work. (use doxygen style comments)

If new third party software is used:

[ ] Did you pay attention to its license? Please remember to add it to the wiki after successful merging.

If new mathematical methods or epidemiological terms are used:

[ ] Are new methods referenced? Did you provide further documentation? Has the glossary been updated?

The following questions are addressed in the documentation if need be:

[ ] Developers (what did you do?, how can it be maintained?)

[ ] For users (how to use your work?)

[ ] For admins (how to install and configure your work?)

For documentation: Please write or update the Readme in the current working directory!

Checks by code reviewer(s):

[ ] Is the code clean of development artifacts e.g., unnecessary comments, prints, ...
[ ] The ticket goals for each associated issue are reached or problems are clearly addressed (i.e., a new issue was introduced).
[ ] There are appropriate unit tests and they pass.
[ ] The git history is clean and linearized for the merge request. All reviewers should squash commits and write a simple and meaningful commit message.
[ ] Coverage report for new code is acceptable.

closes #516

Codecov Report
Base: 92.06% // Head: 92.13% // Increases project coverage by +0.07% :tada:

Coverage data is based on head (2fbffd3) compared to base (740fb37).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #517      +/-   ##
==========================================
+ Coverage   92.06%   92.13%   +0.07%
==========================================
  Files          91       93       +2
  Lines        7084     7149      +65
==========================================
+ Hits         6522     6587      +65
  Misses        562      562

Impacted Files
Coverage Δ

cpp/models/abm/location.cpp
99.12% <100.00%> (+0.01%)
:arrow_up:

cpp/models/abm/location.h
95.12% <100.00%> (+1.57%)
:arrow_up:

cpp/models/abm/mask.cpp
100.00% <100.00%> (ø)

cpp/models/abm/mask.h
100.00% <100.00%> (ø)

cpp/models/abm/parameters.h
100.00% <100.00%> (ø)

cpp/models/abm/person.cpp
97.45% <100.00%> (+0.58%)
:arrow_up:

cpp/models/abm/person.h
100.00% <100.00%> (ø)

cpp/models/abm/world.cpp
81.81% <100.00%> (+0.46%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


support for other models

Are there any parallel efforts ongoing to add support for other models in FLAP?
models such as phi, Gemma, mistral, mistral, etc.
If so, what is the timeline for the same?

I apologize for any inconvenience. Due to our current focus on other projects, we may not be able to extend support to additional models like phi, Gemma, and Mistral in the existing version of FLAP. We appreciate your interest and patience, and we recommend staying tuned for future updates regarding FLAP's development.


ClientContext::GetServerTrailingMetadata and GetServerInitialMetadata should be const.

ClientContext::GetServerTrailingMetadata and GetServerInitialMetadata should be const
so that ClientContext can be passed by const ref.

And ServerContext::client_metadata as well.


interactive container and connecting to ARM

the below command runs container and let us access it at the same time, so if this is used each time, multiple containers will be run.
docker run -it azure-stack-powershell powershell
It might worth mentioning that this command can be used to access the container, instead of creating one each time:
docker exec -it "Container name" powershell

Tried the below and it works, but figured that I connected to public Azure
Connect-AzureRmAccount -ServicePrincipal -Credential $pscredential -TenantId

Have used the below and it works fine with Azure Stack
Add-AzureRMEnvironment -Name "AzureStackUser" -ArmEndpoint "Arm Endpoint"
Add-AzureRmAccount -EnvironmentName "AzureStackUser" -TenantId "tenant ID" -ServicePrincipal -Credential $pscredential

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

ID: 55e2558f-e434-e5cc-acd4-0d3bd2dd979f
Version Independent ID: be6bfe5f-bfdd-6550-588d-6d627c16c3f8
Content: Use Docker to run PowerShell in Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/azure-stack-powershell-user-docker.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@HassanAlmanasrah Thanks for this detailed feedback. I will update the article.
#in-progress

updated. #please-close


rtnl: add IFLA_IF_NETNSID for operating in other namespaces

IFLA_IF_NETNSID allows RTM_GETLINK et. al. to operate in other network namespaces. For example I use it to check if a given interface exists in a given namespace.
  let reply = rtnl.request(rtnl.const.RTM_GETLINK, rtnl.const.NLM_F_REQUEST, {
    "ifname": ifname,
    "if_netnsid": nsid,
  });

The nsid is set with ip netns set <nsname> <nsid> and obtained with ip -j netns list-id.

Btw I noticed that the IFLA_NETNS_PID spec attribute is commented out, I assume because it doesn't work yet? I think it has a typo and should be called IFLA_NET_NS_PID, there's also an equivalent for /var/run/netns/ file descriptors called IFLA_NET_NS_FD.

I don't exactly remember why I left this attribute commented out, I think because I thought it entails more work than just passing through the integer value as-is, but maybe I was mistaken.
If you have a use-case for it and are able to test it out then I'd appreciate a patch implementing it.

Btw, upstream renamed IFLA_IF_NETNSID to IFLA_TARGET_NETNSID - maybe it makes sense to go with the new attribute name right away?

Ah yes I noticed that, but thought it was the other way around. It makes sense, I'll test and push.
And I do actually have a use-case for IFLA_NET_NS_PID - moving an interface from a custom namespace into the default namespace, which is usually identified by PID 1.
I'll get back to you with that.

Okay let's leave it at only IFLA_TARGET_NETNSID for now :-)
I noticed that the combination of IFLA_TARGET_NETNSID and IFLA_NET_NS_PID (or _FD) actually isn't supported yet (which is what I wanted to try and test). There was a kernel patch proposed in 2019 to allow it, but it stalled.
But I will probably eventually get to testing IFLA_NET_NS_PID and _FD on their own.


Can you add Cull Leaves

Mod name
Cull Leaves
Curseforge link
No response
Other links
https://github.com/TeamMidnightDust/CullLeaves
What it does
Adds culling to leaf blocks, providing a huge performance boost over vanilla
Why should it be in the modpack
It just boost
Why shouldn't it be in the modpack
the leaves looks a bit weird but it just culling
Categories
Graphics optimization
Additional details
No response

Already exists since 1.5.0 🙂


cli command to create templated subfolder in books folder

with current jninja templates and regexps to customize

goodreads-export init <folder, config by default>
write config with the templates set name for the folder and write embeded timplates renamed as "default-copy",..
so we have embeded templates and templates loaded from the config folder
if the template set name in the folder is the same as one of embeded - config error, halt
--template


Always create waitable D3D swapchains

Definitely don't want this by itself as it causes extra overhead when tearing is enabled.


Memoisation with Time To Live

For functions that perform I/O, or produce a result that may depend on some global state, having a TTL may enable memoisiation / caching to be useful.
Should be implemented as a new class TTLMemoise or similar.

A custom cache factory can be passed to Memoize now in cyclops v6.0.1


https://docs.rs/releases/search?query= gives a division by zero error

Oct 30 16:05:44 docsrs cratesfyi[8377]: 2020/10/30 16:05:44 [ERROR] docs_rs::web::releases: called ctry!() on an `Err` value: db error: ERROR: division by zero
Oct 30 16:05:44 docsrs cratesfyi[8377]: note: while attempting to fetch the route Url { generic_url: "http://docs.rs/releases/search?query=" }
Oct 30 16:05:44 docsrs cratesfyi[8377]:    0: docs_rs::web::releases::search_handler

cc @Kixiron

I'm completely baffled by this, the best guess is that it's because of the eternal fixme??

(why are page and limit not u64?)

Because postgres takes in i64s, I was told not to bother


Invalid ID EEPROM content

The Base HAT I got seems to have invalid EEPROM contents. This makes it more difficult to auto-detect its presence due to missing entries in /proc/device-tree/hat. Do I have a bad HAT or do you have an error in your eeprom data?
Output of eepdump from https://github.com/raspberrypi/hats :
# ---------- Dump generated by eepdump handling format version 0x01 ----------
#
# --Header--
# signature=0x69502d52
# version=0x01
# reserved=0
# numatoms=2
# eeplen=96
# ----------

Reading atom 0 (type = 0x0001 (vendor type), length = 36 bytes)...
# Start of atom #0 of type 0x0001 and length 36
# Vendor info
product_uuid 30084008-0001-4181-9120-80202ac24220
product_id 0x0002
product_ver 0x0000
vendor "Sixfab"   # length=6
product "Sixfab "   # length=8
# End of atom. CRC16=0x0040
Error: atom CRC16 mismatch
# Error: atom CRC16 mismatch. Calculated CRC16=0x40

Reading atom 1 (type = 0x4020 (unknown), length = 1040 bytes)...
# Start of atom #17472 of type 0x4020 and length 1040
Error: atom count mismatch
# Error: atom count mismatch
Error: unrecognised atom type
# Error: unrecognised atom type
Unexpected EOF or error occurred

Never mind, I think the problem is due to the fact that I stacked a Base HAT and a UPS hat...


RepeatBehavior infinite not working for certain GIFs

I have a bunch of images using the same code to set the URI source on the gif. I have set their RepeatBehavior to infinite, however I noticed it only works for some images. On others, they only run once and then stop. I can't compile the code myself, but I wonder if the only differences lay in the GIFs innate frame count settings.
Thanks for all your hard work!

This is the not repeating GIF in question!

It seems to work fine for me... Just tried it in the WPF demo app. The repeat count encoded in this image is "forever", so even if you don't set RepeatBehavior="Forever", it will run forever.
Could you post the code you used?

Thanks for the response, it actually seems that something about uploading the image to imgur fixed the issue for me as well. I found where I originally got the image, and taking it directly from there again still has issues:

Just tried with the new image, and it still works... Could you send a link to the raw file? I'm not sure what Github does with images included in comments, but it might also transform them.

Sure: here's where I got it from
http://everlastingeden.tumblr.com/post/137041867540/2-8-1-4

Still no luck, I can't reproduce... the animation is repeating forever by default. You still haven't shown your code; the problem might be there.

You're right, it must be something in my code. I did a unit test and got this image working - I'll report back if I find anything interesting out!

Dang I realized my old project had an outdated NuGet version somehow! That did the trick sorry for the trouble! Coffee on me - Cheers!


Add knex withRaw and withWrapped operations

These methods are useful for building complex CTE queries, especially necessary for row-level locking (FOR UPDATE ... SKIP LOCKED).
I didn't see tests for the existing with method or other straightforward knex operations, so I didn't include them here.

Coverage decreased (-0.04%) to 94.4% when pulling 64b73645a7c59c8633cd5b500a1c52f4d901b633 on jordansexton:patch-1 into d0921c2970e6b34dfdc01afa9916f296aa070a80 on Vincit:master.

Actually, argument argument against this: as withRaw and withWrapped and not documented by Knex, they may be regarded as private APIs, and you may as well get to them by just passing a raw query or a function.

It's worth noting that the TypeScript definitions of Objection include withRaw and withWrapped, so maybe this should be supported.


[BUG] .gitignore does not ignore python venv folders

Describe the issue:
Upon cloning the repository and setting up a virtual environment in Python, the new folders are tracked in the repository, which they should not be.
I propose adding the following lines to .gitignore:
.Python
[Bb]in
[Ii]nclude
[Ll]ib
[Ll]ib64
[Ll]ocal
[Ss]cripts
pyvenv.cfg
.venv
pip-selfcheck.json

Reproduce the code example:
In the directory above Fast-F1:
python3 -m venv Fast-F1

Error message:

The problem here is that you use the project root directory as the folder where you create the venv. You should use an empty (!) folder. This folder should preferably be outside the project directory. As an alternative, you can use a folder that is named venv (which is excluded by .gitignore) inside the project root. But you should never create a venv inside a folder that is used for anything else.

Actually, if you place a venv inside a project directory you more commonly would name the subfolder .venv.
Still, you should really prefer to place them outside as placing them inside the project directory really starts cluttering up stuff, when you start to have multiple environments.

I appreciate that you don't want to pollute your .gitignore for different local development styles, given that these can vary significantly. Not least because the virtualenv folder could be named various things (although apparently venv is the recommendation - see below). It totally makes sense and this decision seems entirely justified.
I'd just like to point out that keeping the virtualenv folder inside the project folder is probably a lot more common than you realise. As an open source project, I imagine many more members of your community are doing this than you might think. We can see evidence of this in various places:
https://docs.python-guide.org/dev/virtualenvs/#basic-usage

Create a virtual environment for a project:
$ cd project_folder
$ virtualenv venv

virtualenv venv will create a folder in the current directory which will contain the Python executable files, and a copy of the pip library which you can use to install other packages. The name of the virtual environment (in this case, it was venv) can be anything; omitting the name will place the files in the current directory instead.
Note
‘venv’ is the general convention used globally. As it is readily available in ignore files (eg: .gitignore’)

https://code.visualstudio.com/docs/python/environments

The extension automatically looks for interpreters in the following locations:

Standard install paths such as /usr/local/bin, /usr/sbin, /sbin, c:\python27, c:\python36, etc.
Virtual environments located directly under the workspace (project) folder.

https://stackoverflow.com/questions/40332383/where-do-i-store-my-python-program-files-when-using-a-virtual-environment

However, most projects put the environment right next to their source code within their project folder

But as I say it's totally fair to want to  keep your .gitignore file clean.

@nottrobin I agree that this is fairly common and I do this myself for some projects. But this was not the issue here. The fairly common  venv folder is already added in .gitignore to allow this.


Consider storing password hash parameters in the key file

Evidenced in gen_key() function, though it's a more general problem with the file format.
The problem here is that password hashing is a rather fast moving field, where the relatively weak entropy of passwords encourage us to routinely update the strength of our password hashes, or even the algorithm itself. And depending on context (are you using your beefy desktop or your crappy palmtop?) you may want to adjust the password strength. Version numbers are insufficient to deal with such a fast evolution, or the various use cases. Which is why I believe it is worth putting at least Argon2 parameters in the file format itself:

At least 1 bit to chose between Argon2i and Argon2id, though I would recommend a full byte for (i) easy of coding, and (ii) having some room to use other password hashing algorithms. Note : while I used to favour Argon2i, and still believe the case for Argon2id is quite flimsy, I'm now mostly uncertain about which is best, most notably because this is so highly dependent on the threat model: how much do you fear timing attacks, vs just having your key file stolen and attacked offline?
At least 7 bits for the number of passes.
32 bits for the number of blocks.
At least 4 bits for the number of lanes. Or make it 8, we never know when we'll get those 256 core CPUs. Or give up and just use 1 lane, I don't think it's the most important parameter.

Speaking of lanes, I recall some papers noting that parallel versions of Argon2 are a bit weaker than just running several serial instances in parallel (with different additional data to distinguish them), then hashing the results together. So if you use a parallelism parameter, you should probably do that instead of using Argon2 lanes (whose support is not universal anyway, see Monocypher not supporting threads, and libsodium not even supporting lanes).

Of course, nothing here stops you from setting good defaults, or even hard code the parameters you encode into the key file. You can always add options or change the defaults later, as long as the file format itself (and the reader side) is prepared for the change.

I agree 100%. We just need to revise the gen_key function using a fifth parameter of type crypto_argon2_config as follows:
static enum error gen_key(uint8_t *passphrase, size_t passphrase_size,
                          uint8_t *salt, const crypto_argon2_config config,
                          uint8_t key[KEY_SIZE])

and directly use config  in the call to crypto_argon2 as follows:
crypto_argon2(key, KEY_SIZE, work_area, config, inputs, extras);

That would call for a new key format, and a new version of the program. Thus, my plan would be to freeze the current version of the app in a v-2.0 tag in its own "legacy" branch, and work on a new version with longer keys. Speaking of which, we can start with some hard-coded values for the Argon2i algorithm and eventually offer them as advanced parameters in the key generation operation.
In other words, one could invoke kr as follows with the default parameters:
$ kr -g ~/key

Or with advanced parameters as follows:
$kr -g -i5 -b200 -l4 ~/key

Where -i if for passes (iterations), -bfor blocks, and l for lanes.

A new version of kr has been pushed into the new-keyformat branch to address this issue. Some more reviewing (and eventually refactoring) might be needed before merging the changes back to the master branch, but for now it seems to work fine, and the tests passed just fine. Thanks again @LoupVaillant. Again your comments and suggestions on this issue (or the project as a whole) are more than welcome :smile:


Resulting file size vs gifski-cli

Hello,
I'm getting 30%-40% smaller file sizes while using this app versus the gifski cli tool on the same input material.
Could anyone give some pointers into where/how this difference happening?
Thank you very much,
pedro

The app and CLI use exactly the same compression, so the difference can be caused by:

different output from video decoder (e.g. I've noticed ffmpeg gives more blocky color channels), but it's unlikely to cause that much difference,

or different settings. That's most likely. The app chooses defaults for size and quality, and the cli doesn't.

Thank you.
I did try to mimick the app settings but these in some cases do not display a precise value (quality, just a slider)
Will investigate further.
Thanks again!


Fix NPS version format in nzgo

Problem:
nzgo_client_version is missing a mandatory word before the version number. This causes the server to fail to parse nzgo's client version.
Solution:
Add the word "Release" to the beginning of the  client version string nzgo_client_version

This PR is not sufficient for JSON support. PR #17 is. Closing this PR.


Font:getWidth and lovr.graphics.print now support a first-line indent

Motivation
I want to be able to do some really basic text layout. In particular, I want to highlight exactly one word as red in a text printout. The best way I can think of to do this is to print the text before the word in white, the word in red, and then the text after the word in white. In order to do this, I must print each of the three substrings, then after each print use getWidth to "advance" my text pointer. The new "last line" feature in getWidth makes this possible.
Problem
The problem is once I have the "last line" width, I cannot do anything with it. If I add that width to the print location, then the entire paragraph will be pushed right, not just the line I'm trying to advance.
Solution
The patch allows the "last line" number to be passed back in to print or getWidth as an "indent". The indent is in in-font sizing (IE it is before the fontScale transformation).
Test
Copy this main.lua.txt into a folder as main.lua and run it with the patch.

Other approaches?
lovr.graphics.print is already calculating the values getWidth computes (it calls Measure()), so it could return those values when it's called and save a call and a little bit of time.

This solves the problem but I'd prefer to add something to Font that lets you measure out the individual lines.
Font just has a lot of arguments already, and it's nice to separate out the layout functionality so you can do other stuff like switch fonts, insert icons between ranges of text, or render a range of lines.
The algo to do this would be something like this
local x, y = 0, 0
for i, entry in ipairs({ { font, text }, { font2, text2 } }) do
  local font, text = unpack(entry)
  local width, lines, final = font:getWidth(text, wrap)
  lovr.graphics.setFont(font)

  -- if this line is indented, just draw rest of line and reset x to zero
  if x ~= 0 then
    local tail = text:sub(font:getLines(text, wrap - x)[1]) -- get text of remainder of line, hypothetically
    lovr.graphics.print(tail, x, y, ...)
    x = 0
    y = y + font:getHeight()
  end

  -- draw remaining lines in text chunk, if any
  lovr.graphics.print(text, x, y, ..., wrap)
  x = final
  y = y + lines * font:getHeight()
end

I'd also like to add a LÖVE-style variant of print that lets you print text with different colors by passing a table of colors/strings instead of a single string.


about  Mag combined with model

Thanks for your great work mag. The chronos model performs very well on my dataset. I wonder if you could provide me with an example of how to train adapters for specific tasks using mag as discussed in your paper. Thanks a lot.

Also, if I want to combine mag, do I need to retrain the model or fine-tune it? Or train the mag model separately?

What is MAG?


Initial Loading overlay

Just wondering if there is an interest for a loading overlay? Can clean up and give proper commit messages? Just thought I'd get the conversation going.

Thanks for your proposal! However, this isn't something I think belongs in Material itself (versus Material providing the individual pieces that compose it).


cpu？gpu ？memory size？

cpu？gpu ？memory size？

CPU: i7 7700K
GPU: gtx 1080 Ti . This GPU has 11Go, but you only need around 3Go for Openpose on 640x480 frames.

CPU: i7 6700k
GPU: GEX 1080 (not Ti) should be working fine, isn't it?


Update to allow access to call objects

Supports https://github.com/googleapis/toolkit/pull/1310

Codecov Report

Merging #72 into master will decrease coverage by 0.15%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
- Coverage   89.73%   89.57%   -0.16%
==========================================
  Files          28       28
  Lines        1724     1736      +12
==========================================
+ Hits         1547     1555       +8
- Misses        177      181       +4

Impacted Files
Coverage Δ

src/Testing/MockStubTrait.php
84.74% <75%> (-4.62%)
:arrow_down:

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 eecc8e4...3b0bf07. Read the comment docs.

LGTM

Toolkit PR: https://github.com/googleapis/toolkit/pull/1310


Linting refactoring

BpmnLinter is a dummy
CloudBpmnLinter configures linter based on execution platform and adjusts error messages

Closed in favor of https://github.com/camunda/linting/pull/1.


How can I put the vscode pythonnode configuration

Hello
How can I use vscode editor instead of sublime text to edit a pythonnode in the configuration?
thanks you.

You can edit in preferences and instead of sublime.exe put the path to your vscode installation


Print LoadFailedException cause on load failure

LoadFailedException wraps a list of error reports. In the load path,
these reports are primarily added by the parser, but we also construct
LoadFailedException objects to wrap FileNotFoundException instances.
The compiler previously only printed the formatted reports. Other
wrapped exception causes were effectively swallowed which lead to
ambiguous compilation failures when an included .thrift file couldn't be
loaded, for example.
This change prints the underlying cause when the LoadFailedException
doesn't have any error reports. This feels like a good compromise that
addresses the root problem of certain exception cases being swallowed.
Alternatively, we could avoid wrapping FileNotFoundException and let is
bubble up on its own, but that would be a change from the currently
documented "contract" where we except all load errors to be represented
by LoadFailedException.

Codecov Report

Merging #328 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #328   +/-   ##
=========================================
  Coverage     76.74%   76.74%
  Complexity      381      381
=========================================
  Files            29       29
  Lines          1664     1664
  Branches        148      148
=========================================
  Hits           1277     1277
  Misses          291      291
  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 4b8cc86...717d12b. Read the comment docs.

Sometimes I regret implementing the contract this way - feels like any new code in the parser will do the wrong thing by default.  Thanks for plugging one more hole in the boat :)


Wrong navigation bar padding when presented as second modal view controller

This seems to be an iOS 13 bug but easy fix would be appreciated.
https://forums.developer.apple.com/thread/121861

Adding these lines in TatsiPickerViewController fixed current issue

    public override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        if #available(iOS 13.0, *) {
            self.view.layoutIfNeeded()
            self.view.updateConstraintsIfNeeded()
        }
    }


Add backwards compat support for existing recipe.yml and package.json

There exists two methods for defining which packages need to be installed after a first boot from main and base. Will need to workshop a method for locating this file in a well known and doing what's expected when that file is loaded

I'm not 100% sure about this, but I believe packages.json was never used to declare flatpaks to install.
And on startingpoint, which is the only non-deprecated repo still using recipe.yml, I merged #13 (feat: use yafti), which adds a "package group" called Custom which is enabled by default and contains all of the flatpaks declared in recipe.yml.
As such, it's probably not necessary (and probably even beneficial) for yafti to handle such legacy support.


[NoncopyableGenerics] handle ~Copyable in where, some, and compositions.

This is a pile of fixes and improvement for NoncopyableGenerics:

Fixes ownership checking for some parameters that are noncopyable.
Fixes an issue with conditional Copyable conformances.
Fixes generic signatures for some types.
Correctly handle ~Copyable in protocol compositions.
Handle ~Copyable constraints in where clauses wherever they may appear.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test

@swift-ci please test


Add fallback path capability

This pull request adds the ability to define a "fallback path" that can be used, when enabled, as a 200 OK response instead of a 404 when a file is not found. This can be used with front-end single-page applications to allow the application to do partial view routing based on the URL.
Created as a possible resolution to #26
Notes

Because it uses an environment variable, I had to add --test-threads 1 so that the test I added to verify the functionality does not alter the behavior of another test.
I'm pretty new to Rust so any and all critical feedback here will help me learn

Thanks for looking!
Signed-off-by: Justin Pflueger<EMAIL_ADDRESS>

Don't use an environment variable, because then it won't work in Cloud.
Instead, use a component config setting:
[[component.config]]
fallback_path = "/foo"

and access it as:
spin_sdk::config::get("foo")

(falling back to existing 404 ciode if the config entry is not present)

Actually I guess it works in cloud if you set the environment variable directly on the component... I'd still recommend config to reduce the risk of "it worked on my machine (because I had an ambient environment variable)."

No, you've even pre-empted me by writing docs!  :shipit:

Should we release a v0.0.2 based on this?


teleop_twist_joy: 2.2.2-1 in 'dashing/distribution.yaml' [bloom]

Increasing version of package(s) in repository teleop_twist_joy to 2.2.2-1:

upstream repository: https://github.com/ros2/teleop_twist_joy.git
release repository: https://github.com/ros2-gbp/teleop_twist_joy-release.git
distro file: dashing/distribution.yaml
bloom version: 0.10.1
previous version for package: 2.2.1-1

teleop_twist_joy
* Export interfaces for Shared Lib on Windows.
* Make teleop_twist_joy composable.
* Reenable cppcheck.
* Rename teleop_twist_joy.h to teleop_twist_joy.hpp
* Get some basic tests running (#10 <https://github.com/ros2/teleop_twist_joy/issues/10>)
* Port config and launch to ROS 2 (#11 <https://github.com/ros2/teleop_twist_joy/issues/11>)
* Contributors: Chris Lalancette, Scott K Logan, seanyen

Never mind; this update will break teleop_twist_joy in dashing.  I'll close this and have to revert some stuff.


Improve documentation on NativeCall

To me, the doc of NativeCall helps you to quickly create an hello world application, but from beyond there you're on your own specially when dealing with more complex datatypes. Maybe we need an "advanced" page as well.

We do have a few of requests of this kind, latest one #1376. Also #1513. Thanks anyways, we do need to move on this.

My excuses I missed those.
C.

2018-03-29 10:32 GMT+02:00 nxadm<EMAIL_ADDRESS>
My excuses I missed those.

No problem at all. It's a reminder we should do something about that :-).
Thanks.

See also #1902

:loudspeaker: Can you please check #1513 and see if most of your concerns are addressed? If they aren't, please create a checklist and add it here.


Fix bug with school_scope in EducatorSectionAssignmentsImporter

Follow-on to https://github.com/studentinsights/studentinsights/pull/2036.
Who is this PR for?
developers
What problem does this PR fix?
There's a bug in filtering by school that meant filtering didn't work.
What does this PR do?
Fixes the bug.

selfie


Cosmetic refactoring

@jarthod
i had another set of purely cosmetic code changes that i hadn't tested yet when you released 4.1.0.
This has all been tested in production and should have no functional impact at all, so take your time on merging this.  Also not sure it's worth releasing as 4.1.1 given the lack of functional change.
This should be the last of this round of improvements.
edit i guess there is the one change about allowing single filters, so maybe it's worth a 4.1.1, maybe not.  totally up to you.

Ok I'll let this in master then, it'll release with the next one ;)


⚠ Bofrost.fr has degraded performance

In 23a56c6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2149 ms

Resolved: Bofrost.fr performance has improved in e4d9aee after 9 minutes.


Write a post for Rackspace DevOps Blog

Gentlemen. We've got a little developer blog here at Rackspace that @jnoller maintains. If you're down, we'd love to have you write a post about this Hubot plugin. Idea for article outline:

Describe the issue you had to deal with,
the idea to use Hubot,
the development of the plugin,
and how it's helped.

The entire dev site is Jekyll-based, which is awesome, and the source for which is publicly available on GitHub via rackerlabs/devsite.
What do you think?

Closing this since it's forever old. Hope all is well with you guys.


ignore node_modules

the case is requireDir with recurse: true, and there are node_modules exist.

ping @aseemk

@aseemk any advice?

Sorry for missing this! Will TAL.

Thanks for doing this!
My only major piece of feedback is that instead of handling this at the top of the requireDir function, we should instead handle it where we recurse. i.e. Don't recurse if base === 'node_modules'.
(Note that base has already been derived there too.)
The advantage of that is it'll support an explicit requireDir('./node_modules'), and it won't return an empty node_modules entry if we're skipping it.
It'd also be nice if there were a test for this, and also if the readme were updated to mention this. I don't mind doing these things, but might not be able to get to it right away.
Finally, ideally, this would be an option, just set to the default. But perhaps it makes sense to do a more general ignore option instead, so I don't mind not worrying about the option for now.

@aseemk done
ci failed on<EMAIL_ADDRESS>how about rm 0.8 and add 0.12, iojs?

Great work, and thanks! Happy to merge after these final changes.

everything should be OK :smile:

Published to npm as v0.2.0.


Fix Python 3.x

Coverage remained the same at 99.142% when pulling 4d0580f5124934d53ac656b91b80c86257fdc360 on mwojcikowski:python3 into a3c99c7d93463243bdb3d0411b01f34ab9cf92d3 on ajtulloch:master.


Add new filters to Search Divisions

[ ] Add new filters to Search Divisions
region_name
district_name
settlement_name
edrpou - legal_entity edrpou
legal_entity_id

[ ] Change response for Search Divisions
add legal entity information

Apiary
specs

@svetavedmed deployed


Update docs to use Helm chart from OCI registry

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm


How do I use Trix in rails 7 with cssbundling

How do I use Trix in rails 7 with cssbundling

judging from the time since the repo has been updated, and the age of the pull requests and issues, I'm guessing that this has been abandoned and it would be far more work to get it working with Rails 7 than is worth it.
I think it should be easy enough to get Trix working in Rails 7 without a gem like this.  Here are directions for not only getting Trix working, but also adding a table editor to Trix: https://blog.appsignal.com/2022/10/26/build-a-table-editor-with-trix-and-turbo-frames-in-rails.html. You can of course stop once you get Trix installed.


Fix the broken U2F test by partially re-decoding the public key

This PR implements a fix for the broken test introduced in the target branch.

You got an extra commit in there. And the last two commits should be squashed together.

You still have an extra commit in there, there should only be one


Rust backend - safe_prime sometimes generates 1025 instead of 1024 bits

While running the tests for the rust backend, I received the following error:
➜  unknown_order git:(main) cargo test --release
    Finished release [optimized] target(s) in 0.03s
     Running unittests (target/release/deps/unknown_order-ed6009b21a5a84b7)

running 1 test
test rust_backend::safe_prime ... FAILED

failures:

---- rust_backend::safe_prime stdout ----
thread 'rust_backend::safe_prime' panicked at 'assertion failed: `(left == right)`
  left: `1025`,
 right: `1024`', src/rust_backend.rs:322:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    rust_backend::safe_prime

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 34.91s

error: test failed, to rerun pass '--lib'

in most executions, the test passes but occasionally the prime size is off by one. I assume this is a bug in glass_pumpkin which is used to generate the safe primes for this backend.

Fixed with 47f036c96b39881c9120f5a0ae24cb637a0ddee1


EnvironmentVariables does not work

When adding two environment variables to a config, only the first one is added to the psrc file
The config is:
Configuration Test1
{
    Import-DscResource -Module JeaDsc

    JeaRoleCapabilities TechnicianRoleCapability
    {
        Path = 'C:\Program Files\WindowsPowerShell\Modules\JeaDsc\RoleCapabilities\Technician.psrc'
        VisibleCmdlets = 'Get-WinEvent','Get-Service'
        EnvironmentVariables = "@{ V1 = 'test1' }; @{ V2 = 'test2' }"
        VisibleProviders = 'FileSystem', 'Environment'
    }

    JeaSessionConfiguration JeaEndpoint
    {
        EndpointName = "JeaEndpoint"
        RoleDefinitions = "@{ 'Everyone' = @{ RoleCapabilities = 'Technician' }; }"
        TranscriptDirectory = 'C:\ProgramData\JeaEndpoint\Transcripts'
        DependsOn = '[JeaRoleCapabilities]TechnicianRoleCapability'
    }
}

Test1 -OutputPath d:\dsc
Start-DscConfiguration -Path d:\dsc -Wait -Verbose -Force

The psrc looks like this:
# Environment variables to define when applied to a session
EnvironmentVariables = @{
    'V1' = 'test1' }

# Type files (.ps1xml) to load when applied to a session
# TypesToProcess = 'C:\ConfigData\MyTypes.ps1xml', 'C:\ConfigData\OtherTypes.ps1xml'

Test-DscConfiguration always returns false, I guess due to the same issue in #19.

I'll have a dig into this and see what's going on with it. Hopefully have time to get a fix out later this weekend.

After a bit of digging and looking at the code, the problem is you've got  2 hashtables there when you should really have 1.  It should really be @{V1 = 'Test1'; V2 = 'Test2'}  and then it works correctly.


chore: add benchmark action

Added benchmark workflow
Description
Added github-action-benchmark to run for changes to main. I've tried to include the changes from the previous discussion here #419.
Resovles #477
This needs a branch called gh-pages to visualize the benchmarks, See here for more information.

From comment

IMO the benchmarks should run on each PR shouldn't they?
But the results should only be updated when merged onto main. WDYT?

Enabling the benchmark for all pull requests will result in the file being written to and is not recommended, I could either add a separate workflow for pull requests or try to use save-data-file with github.event_name != 'pullrequest' (not sure if this is possible)

Why did it take 6 hours for the CI to fail?

@TimothyMakkison no Idea, which job took so long?

Looks like the test. Wonder if github was having issues yesterday.


GET device by name returning wrong Content-Type

The Content-Type should be application/json, but got text/plain.
13:01:47 ↳ http://localhost:48082/api/v1/device/name/:name
13:01:47   GET http://edgex-core-command:48082/api/v1/device/name/Livingroom automation [200 OK, 773B, 37ms]
13:01:47   ┌
13:01:47   │ 'Validation failed: ', null
13:01:47   └
13:01:47   ✓  Status code is 200
13:01:47   ✓  Response time is less than 1200 ms
13:01:47   ✓  Has Content-Type
13:01:47   4. Content-Type is application/json
13:01:47   ✓  Is Valid Response Schema

@jinlinGuan Making a note of your observation (above), and will investigate 👍

@jinlinGuan Making a note of your observation (above), and will investigate very shortly 👍

@jinlinGuan Good catch. I have created a fix for this issue and requested @AnthonyMBonafide to review that PR: https://github.com/edgexfoundry/edgex-go/pull/2506 ✅

@jinlinGuan The PR that I refer to above has been approved and merged into master. I'll leave this issue open for you to please confirm and close out, thanks.

Verified on Jenkins test.
Closing this issue.

Verified on Jenkins test.
Closing this issue.

@jinlinGuan Thank you for your prompt follow-up 👍


Many lines printed in terminal during "npm install" step if package names wrap to the next line

I ran an interactive update and npm-check started installing newer versions of some of my dependencies like this:
⠼ npm install<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
My terminal wasn't wide enough to fit all of the package names, though, so they wrapped to the next line. npm-check then started printing out a new line each time it advanced the spinner animation instead of overwriting the current line.
Platform: OS X El Cap with the built-in Terminal.app.

This sounds like an issue for ora, I'll create an issue there.