Porting ChipmunkDemo

Please try running this.
chipmunk_demo.zip

Great update!
I like one file wrappers.
FYI, Araq explained that the {.inheritable.} pragma adds runtime checks for subtype
memory safety and a hidden typeinfo field to the object. To get rid of the overhead and
the typeinfo field, {.inheritable, pure.} should be used, but then there is no memory check.

Hey BlaXpirit,
In the next demo I'm porting i make a rounded player box using this code:
    body = space.addBody(newBody(1.0, INFINITY))
    body.position = chipmunk.Vect(x: 0.0, y: -200.0)
    body.velocityUpdateFunc = player_update_velocity
    player_body = body
    shape = space.addShape(
        newBoxShape(
            body=body,
            box=newBB(-30.0, -30.0, 50.0, 50.0),
            radius=10.0
        )
    )
    shape.elasticity = 0.0
    shape.friction = 0.0
    shape.collisionType = cast[CollisionType](1)
    player_shape = shape

But when drawing the shape using the 'debugDraw' function, in the 'SpaceDebugDrawOptions:drawPolygon' callback, I always get radius 0.0!
Any ideas?

Please, a minimal complete code example.

Here is the code, chipmunk_demo.nim is the main compilation module:
chipmunk_demo.zip
In 'data.nim' procedure 'default_draw_implementation' in line 305 the drawing is setup, then check the 'debug_draw_polygon' callback procedure in line 219. In 'debug_draw_polygon' the radius is always 0.0 when the callback fires.
The default_draw_implementation is used in the demos/player.nim example, which is the demo
that should draw the rounded box.
Run the demo, press '1' and use the cursor keys to move the player box.
The player box should be rounded, but it is not and I don't know why?

Please, a minimal complete code example.
I can't run this, at least because

An error occured while opening ttf font file!

Sorry, forgot the font file:
EffectsEighty.zip
Put the extracted file next to the executable.
About the minimal example, just look at 'demos/player.nim' module's 'init_space' procedure and
the 'default_draw_implementation' and 'debug_draw_polygon' in the 'data.nim' module and
you'll get the idea.

Code for initializing the player rounded box in C:
	// Set up the player
	body = cpSpaceAddBody(space, cpBodyNew(1.0f, INFINITY));
	body->p = cpv(0, -200);
	body->velocity_func = playerUpdateVelocity;
	playerBody = body;

	shape = cpSpaceAddShape(space, cpBoxShapeNew2(body, cpBBNew(-15.0, -27.5, 15.0, 27.5), 10.0));
	shape->e = 0.0f; shape->u = 0.0f;
	shape->type = 1;
	playerShape = shape;

Am I missing anything?

I don't know what's going on here. I give up.

Thanks anyway. Will keep hacking at it.

Something isn't right here:
    body = space.addBody(newBody(1.0, INFINITY))
    body.position = chipmunk.Vect(x: 0.0, y: -200.0)
    body.velocityUpdateFunc = player_update_velocity
    player_body = body
    shape = space.addShape(
        newBoxShape(
            body=body,
            box=newBB(-30.0, -30.0, 50.0, 50.0),
            radius=10.0
        )
    )
    shape.elasticity = 0.0
    shape.friction = 0.0
    shape.collisionType = cast[CollisionType](1)
    player_shape = shape
    echo cast[PolyShape](player_shape).radius # <-- THIS GIVES 0.0

The echo in the last line should give 10.0, the underlying C code initializes the 'shape->r'
and the wrappers 'radius' procedure should return it.
The 'radius' wrapped C 'cpPolyShapeGetRadius' function:
cpFloat
cpPolyShapeGetRadius(const cpShape *shape)
{
	cpAssertHard(shape->klass == &polyClass, "Shape is not a poly shape.");
	return ((cpPolyShape *)shape)->r;
}

Is this any help?

You were so close to making a minimal complete example, but no.
Anyway,
import chipmunk
var space = newSpace()
var body = newBody(1.0, 1e1000)
discard space.addBody(body)
var shape = newBoxShape(
    body=body,
    box=newBB(-30.0, -30.0, 50.0, 50.0),
    radius=10.0
)
discard space.addShape(shape)
echo cast[PolyShape](shape).radius

10

This is very weird!
Just recompiled the chipmunk.dll with MSVC and GCC, both cases give the same result: 0.0

Just inserted a 'printf' into the 'cpBoxShapeNew2' in the chipmunk source:
cpShape *
cpBoxShapeNew2(cpBody *body, cpBB box, cpFloat radius)
{
	printf("%f", radius);
	return (cpShape *)cpBoxShapeInit2(cpPolyShapeAlloc(), body, box, radius);
}

compiled it and ran it with the example you gave and the 'printf' shows '0.0000'

Good diagnostics, thanks.
Are you on a 32-bit system? Might need to compile Chipmunk with #define CP_USE_DOUBLES 1 or change nim-chipmunk: type Float* = cfloat.

I'm on Windows Vista 64-bit, these are the results of the different compilers:
(#define CP_USE_DOUBLES 1 is always defined)
Nim (GCC 5.2.0 32-bit) and Chipmunk (GCC 5.2.0 32-bit):
echo's '0.0'  (type Float* = cdouble)
echo's '0.0'  (type Float* = cfloat)
Nim (MSVC 2010 32-bit) and Chipmunk (MSVC 2010 32-bit):
echo's '-30.0'  (type Float* = cdouble)
echo's '206158462976.0'  (type Float* = cfloat)

Just tried it on a Windows XP 32-bit machine and it's the exact same as above.

The BB type has to have the 'bycopy' pragma:
http://forum.nim-lang.org/t/2291

OK, thanks for info. Had no idea / forgot about this


ouibounce modal containing a form

I am using ouibounce and it's working as it should, except...
I am trying to add a small form to the modal window that I will submit with ajax.  For some reason I cannot handle the form with jquery, but if I set the form up traditionally (posting to another page) it works.  and I don't know if it's just a bug on my end or there's something in ouibounce that's preventing it.
I've commented out e.stopPropagation(); but it doesn't appear to help.
To test, I added this code the the same page that triggers OB:
$(document).ready(function() {
$("#myform").submit(function(e){
alert("clicked");
});
});
form id="myform" is the form on the ouibounce modal.  The alert doesn't show when I submit the form.
Is there some jquery conflict?  I'm using the latest OB and jquery 1.9.1

Nevermind - this issue was on my end :-/


add FAQ to Referral page

Based on @tzarebczan ideas

Thanks @ykris45 ! Not sure if Referrals should be capitalized or not. We should also just say "Read our FAQ to learn more".


Questions for evaluation

I want to replicate results of ATEPC model from the paper A Multi-task Learning Model for Chinese-oriented Aspect Polarity Classification and Aspect Term Extraction. Specifically for the APC task. But ATEPC model gives the extracted aspects and then classify the sentiment on the extracted aspects. If the model extracts the aspects wrongly then the further classification evaluation will be hampered.

Can you tell how did you evaluate for only APC task

Could you share your evaluation script or any pointers how to replicate the numbers reported in the paper

In the paper, APC and ATE are evaluated separately, rather than extracting terms first. Secondly, the APC results in the paper are based on domain adapted BERT. The original LCF-ATEPC code is based on old repos (e.g., pytorch_pretrianed_bert, etc.) no longer available, but we provide a version compatible with the Transformers (https://github.com/yangheng95/LCF-ATEPC), you can try to reproduce the results (log_step=5 is recommended).


Decal Rotation

Currently, if I try to rotate the decal that I've positioned, it will totally move and lose the position that I've already set.
lilToon
https://github.com/lilxyzw/lilToon/assets/32463720/e5ef559a-8f7d-47df-adea-4b9ac3518f4c
Poiyomi (for example)
https://github.com/lilxyzw/lilToon/assets/32463720/71953808-d3c3-4538-aac8-208de0e80e62
Seems that the decal isn't rotating in its own axis

I've tried to fix it by myself, but I've never touched shader coding and my brain was on fire trying to understand math. But I was able to almost fix it by changing the end of lilCalcDecalUV function
float2 lilCalcDecalUV(float2 uv, float4 uv_ST, float angle, bool isLeftOnly, bool isRightOnly, bool shouldCopy, bool shouldFlipMirror, bool shouldFlipCopy, bool isRightHand)
{
    float2 outUV = uv;

    // Copy
    if(shouldCopy) outUV.x = abs(outUV.x - 0.5) + 0.5;

    // Scale & Offset
    outUV = outUV * uv_ST.xy + uv_ST.zw;

    // Flip
    if(shouldFlipCopy && uv.x<0.5) outUV.x = 1.0 - outUV.x;
    if(shouldFlipMirror && isRightHand) outUV.x = 1.0 - outUV.x;

    // Hide
    if(isLeftOnly && isRightHand) outUV.x = -1.0;
    if(isRightOnly && !isRightHand) outUV.x = -1.0;

    // Rotate

    // FROM THIS
    //outUV = (outUV - uv_ST.zw) / uv_ST.xy;
    //outUV = lilRotateUV(outUV, angle);
    //outUV = outUV * uv_ST.xy + uv_ST.zw;

    // TO THIS
    float2 center = ((uv_ST.zw / uv_ST.xy) / 2) + 0.5;
    outUV = lilRotateUV(outUV - center, angle) + center;

    return outUV;
}

It's kinda a hacky way, but I think with some polishment it will be totally fixed.

Thank you! I will also think about the calculation method.

@lilxyzw
Updating the center variable to this seems to completely fix my issue, feel free to give some tests before merging. (full code in PR)
float2 center = (uv_ST.zw / 2) / (uv_ST.xy * uv_ST.zw);

@lilxyzw Any news about this? I'm waiting for the implementation of this fix to release a thing I made


Updating to container1.0-alpha

Updating to containerd 1.0 alpha6 with minor changes.
Signed-off-by: Abhinandan Prativadi<EMAIL_ADDRESS>

LGTM


Only allow end date after start date

I used as input 2015-06-22 as begin date and 2015-07-07 as end date I got 2015-09-01-ttt-online into the filter. The filters is fine but 2015-09-01-ttt-online is inconsistent since the end date is before the start date when it should be after.
Can we have something to avoid this inconsistency?

This is not an issue with debrief query, it's an issue with 2015-09-01-ttt-online event. It has end date in the past (2015-04-20), and start date in future (2015-09-01).


Custom color for party page

Each party page should have custom color taken from color field.
Please refer to party color at https://github.com/codeforthailand/election-live/blob/master/data/party.json

I'd like to work on this

Hey, is this issue still open? Can I work on this?

Yes, please send PRs.


Refactor into pkg using an options input and add Loki App

Signed-off-by: Alistair Hey<EMAIL_ADDRESS>Refactor into pkg for creating apps using a "options" block passed into the specific apps.
See  #21
Add Loki app, first part of #115

Description

Refactor some of the common stuff out into pkg, creating a "InstallerOptions" struct that would be passed to the pkg function that does the install. This would allow other programs to use these functions to install the apps (once migrated)
The Loki app has also been added
Motivation and Context

[ ] I have raised an issue to propose this change (required)
#115 #21

How Has This Been Tested?

Installed Loki, Loki with Grafana, OpenFaaS, cert-manager, ingress-nginx, minio
and verified they installed without errors
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

[ ] I have tested this on arm, or have added code to prevent deployment
I dont have any ARM booted, but there's a closed issue pointing at ARM support in the repo https://github.com/grafana/loki/issues/653#issuecomment-517358019

CI is failing?

CI is failing?

missed committing after fmt. Apologies

@Waterdrips could you work on the Loki app for the openfaas component from @LucasRoesler? I would probably suggest calling it openfaas-loki


WDDG & WDCG hardware readiness tool: typo & locale issues

As discussed in ticket #5972 (even if that ticket is closed), the script on this page stops working in localized versions of Windows (Dutch mentioned in ticket #5972 specifically). This is caused by the difference in how the strings 64-bit and 32-bit occur in non-English localized versions of Windows.
The suggestion in ticket #5972 is to add some regex functionality inside function CheckOSArchitecture, like this:
function CheckOSArchitecture
{
    $OSArch = $(gwmi win32_operatingsystem).OSArchitecture.ToLower()
    Log $OSArch
    if($OSArch -match ("64\-?\s?bits?"))
    {
        LogAndConsoleSuccess "64 bit archictecture"
    }
    elseif($OSArch -match ("32\-?\s?bits?"))
    {
        LogAndConsoleError "32 bit  archictecture"
        $DGVerifyCrit.AppendLine("32 Bit OS, OS Architecture failure.") | Out-Null
    }
    else
    {
        LogAndConsoleError "Unknown architecture"
        $DGVerifyCrit.AppendLine("Unknown OS, OS Architecture failure.") | Out-Null
    }
}

The word "archictecture" is also misspelled, twice in that function, luckily only in text strings used for logging.

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

ID: c12e9c4a-af05-9ecd-a791-ac89aa43787c
Version Independent ID: 76517523-a960-c5fb-ce76-2c3efe642f37
Content: Windows Defender Device Guard and Windows Defender Credential Guard hardware readiness tool - Microsoft 365 Security
Content Source: windows/security/identity-protection/credential-guard/dg-readiness-tool.md
Product: w10
Technology: windows
GitHub Login: @SteveSyfuhs
Microsoft Alias: stsyfuhs

May as well just mention related issues from the other page, since they are all related to the Windows Defender Device Guard and Windows Defender Credential Guard hardware readiness tool :
#5690 #5723 #6054 #6055 (open issues)
The main issue reported here (and described in the closed ticket #5972 ) is possibly related to an unresolved question in the closed ticket #4679 .

Great sir. Author plesse accept his PR

Please note that the author of the script modification is @RvdHout (Ruud van den Hout).


Install createSlice redux

Is your feature request related to a problem? Please describe.
The reducers are a mess
Describe the solution you'd like
You need to install createSlice https://redux-toolkit.js.org/api/createSlice

https://stackoverflow.com/a/66722569/10294022


hashed url for _memory

thanks so much for getting this plugin set up, @simonw. everything is now working well now (or better since https://github.com/simonw/datasette/issues/1581 is no longer an issue).
it would be a nice feature addition if the _memory endpoint could also have a hashed-url for crossdb queries. (relates to https://github.com/simonw/datasette/issues/1561).
a way that this could work is that a hash of the db hashes could be calculated and that used as the _memory suffix.

This seems like it could be quite straight forward. That logic would go here:
https://github.com/simonw/datasette-hashed-urls/blob/87e4da21db3b970423daa95ecca24d777498d256/datasette_hashed_urls/__init__.py#L6-L14
It could have a special case for _memory but only if datasette.crossdb is True. It could derive a hash by hashing together the hashes of the other immutable databases.

This will be available in 0.4 once it ships - currently waiting for the GitHub Actions workflow to run: https://github.com/simonw/datasette-hashed-urls/actions/workflows/publish.yml

doesn't look like it was triggered?


fixes #39 require either title or bibId

... and don't attempt to retrieve external metadata if bibId isn't present

fixes #39
fixes #33


dapr telemetry

Please provide us with the following information:

This issue is a: (mark with an x)

[ ] bug report -> please search issues before submitting
[x] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
there is no documentation that shows how to setup telemetry correctly while using dapr in container apps, only kuburnetes. We can specify app insights in the container apps environment but no metrics regarding the dapr sidecar are logged, including errors. I would expect to see sidecar telemetry and errors, we are only seeing service invocation and http requests with no errors.
does container apps utilise the "open telemetry collector" as referenced in the dapr documentation for setting up kuburnetes?
Steps to reproduce

setup container apps environment with app insights
create 2 services, one should invoke the other through the dapr sidecar
build the 2 images
push the 2 images to container apps specifying a dapr sidecar setup
call the service , which should call the other service through the sidecar

Expected behavior [What you expected to happen.]
telemetry similar to zipkin when running dapr locally including errors being caught by sidecar . telemetry should show the sidecars talking to each other and then the service, this helps with resolving where an error occurred especially through a long chain of requests. because no errors are logged we cannot see what the request payloads were to resolve.
Actual behavior [What actually happened.]
service makes a http call to the other service. the sidecar forwarded request is hidden.  no errors logged
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
calling a dapr workflow through a sidecar that errors doesn't log anything in errors or app insights to show the request failed. this is an issue because the dapr workflow service contains no error logging and therefore there is no flag of an error occurring. running locally these errors are captured through zipkin.

Please refer to the following documentation: https://docs.microsoft.com/en-us/azure/container-apps/dapr-overview
Application insights is currently the supported export location for distributed tracing, and you can provide the app insights connection info during environment create to wire this up! We will likely expose the configuration for this in the future.

please refer to my "steps to reproduce" section , step 1 . I have setup the environment with app insights but have no sidecar telemetry/errors.


Add retry mechanism

Thanks for the token handle store implementation but any reason why you needed to add retry? Were you running into some specific issue?


Add "shortCircuit" task attribute to Parallel task

This will allow the Parallel task to fail when one child fails. If a child fails, the Parallel task will call end() on all running children and prevent starting any remaining children.
I mimicked the original Dog test, but instead of Rest, I created Play. The main difference is that Play is not checking for any conditions itself, this is instead handled by the Parallel task.

TBH I'm hesitant to merge this as it is right now.
I think that just calling end() to terminate children is not enough. Task termination should go down the hierarchy and terminate nested children I guess.
For example, with your implementation, SemaphoreGuard is properly terminated (i.e. the semaphore is released) only if it is a direct child of Parallel.

@davebaol
Thanks for taking the time to look at this and provide feedback.
I added end() to BranchTask that will loop through all the children to terminate them. Parallel is the only one that overrides this call since it needs to set runningTasks to false.
Does this sound like the right approach to you? I will work on writing up some tests to confirm this functionality before I do another pull request but would like to hear your input.

I'll look into it asap. Rather busy right now :/

Perhaps a notion of cancel() or something would be useful to distinguish it from regular end()?

Yeah I think so.
I'm going to release gdx-ai 1.6.0 in the week end then l'll look into this.

Closing this. Sorry, I got inspired. :)
I'm currently reviewing the entire API and parallel is one of the things I'm working on.
I'm going to push changes soon.

Sounds awesome!

Done, see https://github.com/libgdx/gdx-ai/issues/12#issuecomment-145060773


Implement method in EventHubConsumerClient to receive from all partitions

Implement method in EventHubConsumerClient to receive from all partitions with two overloads - one with default "earliest" event position and second with a user-provided event position (same position will be used for all partitions).

@srnagar Regarding this one, I thought the overloads would be int and Duration similar to our existing receives.  I didn't know that our receivers could start from new positions... if that's the case, should this also be added into our async consumer since it does not have the capability to start from another partition?

Related PR #6258


Switched from AsyncDuplicateLock to AsyncKeyedLock

Switched from AsyncDuplicateLock to AsyncKeyedLock, which provides better performance and lower memory usage.

@f2calv

Closing as you will anyway get the obsolete message once you update the CasCap.Common package.


Adjusting Charts' frame manually

Hi!
As far as i understood, the frame is set via Env.Ipad? (or any other interface), however I am trying to customise this view so that the charts are not so large.
Is this possible?

You can remove that and pass as frame anything you want.

Duh thanks, that worked :) Also on that note, in your hello world example, when i tried to input a larger value to see how the graph will wrap them up it just didnt fit in the screen and on the grid. How can i fix this? Sorry not sure if this was asked before. Thanks for your help!

A larger value where? A chart point? You have to ensure that everything is in range. Not sure if this is what you mean.

Ok, so as I said: you have to ensure that the chart points are in range. That is select min max in axis according to your chart points. There is also ChartAxisValuesGenerator which you can use to generate the axis values from you, there's also a flag you can set to add an extra value when the chart point is at the edge like in (16, 16) in your screenshot.


consolidating the secure-handlebars-helpers into the npm package of secure-handlebars

in order to have a better management of the secure-handlebars package, this package makes sense to be consolidated into the secure-handlebars.

agree. low priority though


Add iterator support to url_search_params

Does two things:

Adds simple iterators to ada::url_search_params. These are implemented as distinct types with next() and is_done() methods as opposed to traditional c++ iterators.

Adds url search params apis to ada_c

@anonrig ... not entirely sure why the tests are failing here. They are passing locally for me :-/

@jasnell Can you add documentation to each function and a small example to README?

@jasnell Looks fine to me.
However, I am unsure why you don't want to use conventional C++ iterators, which allow you to automagically do this:
 for (auto key : search_params.get_keys()) {
   // do something with key
 }

The approach you document...
auto keys = search_params.get_keys();
 while (!keys.is_done()) {
   auto key = keys.next();  // "a", "c", "e", "g"
 }

will appear odd (I don't mean to be insulting) to many modern C++ users. It is also objectively 'busier' and easier to get wrong.
Furthermore, why 'is_done' instead of the conventional 'has_next... That is, the following is easier to understand...
auto keys = search_params.get_keys();
 while (keys.has_next()) {
   auto key = keys.next();  // "a", "c", "e", "g"
 }

Isn't it?
Can you elaborate on your design decisions ?

Effectively, I think you need very strong arguments if you are going to bypass the standard C++ iterators. They are easy to implement, and they lead to all sorts of goodies. They are expected in C++.

Yep, I'd considered implementing normal c++ iterators but opted for this model to keep it conceptually similar to JavaScripts iterator pattern (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Iteration_protocols#the_iterator_protocol), which as next(), done, etc. The key expectation being that these are most likely to be wrapped in an implementation of the URLSearchParams class where next() is called to advance.
There the pattern for wrapping these is more likely something like (forgive the mild pseudocode)
class URLStringParamsKeysIterator {
 public:
  struct Result {
    bool done;
    JsValue value;
  };
  Result next() {
    if (iter.done) return { true, undefined };
    return { false, iter.next() };
  }

private:
  ada::url_search_params_keys_iter iter;
}

I wanted also to leave room for someone to implement the conventional c++ iterator pattern if they wanted -- these should be able to co-exist. I just really have no immediate need for the c++ style iterators. I need this only to support the JS pattern.

@lemire @anonrig ... ok, I've updated to add the conventional c++ iterator in addition to the JS-style. For the c++ iterator I'm really just deferring to the underlying std::vector for entries, whereas the js-style has distinct instances for keys, values, and entries.
I also renamed is_done to has_next and updated accordingly. Hopefully this will strike a nice balance for both JS and C++ use.

Approved.

I see that many of the operations in ada_c are intended to be safe when !r but the current approach means that these will crash after the various ada_free* calls. In a separate PR I'll go through and make these all safer (not just the search_params ones) so that double frees and the other ops are more resilient after free.


Web Server GPU Usage Anomaly

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[Y] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[Y] I carefully followed the README.md.
[Y] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[Y] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
That the webserver would utilise 100% of the GPU (or close to it).
Current Behavior
Currently it is only using 25% of each GPU it never really goes beyond this:

Environment and Context
This is Linux Ubuntu running latest version of Llama-cpp-python with 2x A100 GPU (all the cuda is up to date, cblas is on etc.  This works at 100% using vLLM - so I don't believe there is any issue with the hardware.
Steps to Reproduce
Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.
I am running the web server using the command: python -m llama_cpp.server --config_file serverconfig.json
My config file is below, I show this because I know I had a problem with GPU usage when not using offload_kqv = true previously and wanted to demonstrate that I was aware of this and my config has this set as true.
If any paths, or anything looks odd in the syntax, that is because I have anonymised it - the server runs, it accepts requests, it responds all of that is fine, it is just the utilisation that is a problem.
{
    "host": "<IP_ADDRESS>",
    "port": 8000,
    "models": [
        {
            "model": "/models/mistral-7b-instruct-v0.2.Q5_K_M.gguf",
            "n_gpu_layers": -1,
            "offload_kqv": true,
            "n_threads": 12,
            "n_batch": 512,
            "n_ctx": 8192
        }
    ]
}

This loads up the two A100 GPU, it recognises them, offloads 33 layers to them and provides this response:

I have also run this same machine using vLLM's server.  It utilises 100% of the GPUs both of them at times.  So I know it is possible to get them both to 100% utilisation, but I can't seem to get there with the llama-cpp-python web server.
I was hoping that this is a me problem, that I have not set a flag I need to set or that the 25% being recorded is actually the full utilisation but this is an Nvidia thing that interacts with Llama-cpp this way, but doesn't using whatever vLLM is using...
Just thought I would add, this is the GPU usage when running vLLM:

Did you ever resolve this?  I've come back to llama-cpp-python after being force to move to llama.cpp directly due to post 2.27 performance issues and am experiencing like behavior.  GPU peaks only during initial prompt assessment until first token is produced and then caps at ~25% utilization.
My config file amounts to the same:
{             "model": "/models/SFR-Iterative-DPO-LLaMA-3-8B-R-Q6_K.gguf",             "model_alias": "gpt-4",             "chat_format": "chatml",             "n_gpu_layers": -1,             "offload_kqv": true,             "cache": false,             "cache_size": 16,             "verbose": true,             "n_threads": 40,             "n_batch": 512,             "use_mlock": true,             "n_ctx": 8192,             "last_n_tokens_size": 128         }


TP unique value error trying to store NULL IPv6 address for server

While updating MSO origin server properties in the new TP UI, I am getting 500 errors when trying to modify the second of the two servers. It appears that it is attempting to store a "null" IPv6 address for the server (server does not have IPv6 address).
[2017-11-13 20:41:18,833] [ERROR] DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::Pg::st execute failed: ERROR:  duplicate key value violates unique constraint "idx_140441_ip6_profile"
DETAIL:  Key (ip6_address, profile)=(, 192) already exists. [for Statement "UPDATE server SET ilo_username = ?, ip6_address = ? WHERE ( ( cachegroup = ? AND id = ? AND profile = ? AND status = ? AND type = ? ) )" with ParamValues: 1='None', 2='', 3='509', 4='2077', 5='192', 6='2', 7='3'] at /opt/traffic_ops/app/lib/API/Server.pm line 235
[2017-11-13 20:41:18,836] [ERROR]

We should not be attempting to store a new IPv6 address here, as it shouldn't have a value at all (and is not changing).
Problem does not seem to exist in the legacy Traffic Ops UI.

There is a unique constraint on the server table which is ip6_address + profile, therefore you can't have 2 servers with null ip6_address + profile A (for example) however that seems silly so i think this unique constraint simply needs to be modified to only be enforced if ip6_address != null.
looks like you can do a conditional constraint https://www.postgresql.org/docs/9.6/static/ddl-constraints.html using the CHECK function.

actually, this is occurring because a empty string is being passed in and multiple rows in the server table where ip6=empty string and profile=X is a violation of the unique constraint.

this is related to #1535


How do I select the quality of the embeddable thumbnail (on audio files that support it)?

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.07.18 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I want to test the cover art capabilities of a music player app for android, so I downloaded something from youtube with the --embed-thumbnail parameter and it works as it should. However, and as seen in the verbose log below on line 19, it downloads the maximum resolution image (id 41) for the thumbnail, which is at 1080p.
$ yt-dlp --list-thumbnails https://www.youtube.com/watch?v=dQw4w9WgXcQ
[youtube] dQw4w9WgXcQ: Downloading webpage
[youtube] dQw4w9WgXcQ: Downloading android player API JSON
[info] Available thumbnails for dQw4w9WgXcQ:
ID Width   Height  URL
0  unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/3.jpg
1  unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/3.webp
2  unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/2.jpg
3  unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/2.webp
4  unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/1.jpg
5  unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/1.webp
6  unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/mq3.jpg
7  unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/mq3.webp
8  unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/mq2.jpg
9  unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/mq2.webp
10 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/mq1.jpg
11 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/mq1.webp
12 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/hq3.jpg
13 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/hq3.webp
14 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/hq2.jpg
15 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/hq2.webp
16 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/hq1.jpg
17 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/hq1.webp
18 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/sd3.jpg
19 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/sd3.webp
20 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/sd2.jpg
21 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/sd2.webp
22 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/sd1.jpg
23 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/sd1.webp
24 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/default.jpg
25 120     90      https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/default.webp
26 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/mqdefault.jpg
27 320     180     https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/mqdefault.webp
28 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/0.jpg
29 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/0.webp
30 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/hqdefault.jpg
31 168     94      https://i.ytimg.com/vi/dQw4w9WgXcQ/hqdefault.jpg?sqp=-oaymwEbCKgBEF5IVfKriqkDDggBFQAAiEIYAXABwAEG&rs=AOn4CLDd2KtelLHaNSXrI9_5K-NvTscKNw
32 196     110     https://i.ytimg.com/vi/dQw4w9WgXcQ/hqdefault.jpg?sqp=-oaymwEbCMQBEG5IVfKriqkDDggBFQAAiEIYAXABwAEG&rs=AOn4CLBUpEOOWUXWkNyijQuZ4UPzp2BE-w
33 246     138     https://i.ytimg.com/vi/dQw4w9WgXcQ/hqdefault.jpg?sqp=-oaymwEcCPYBEIoBSFXyq4qpAw4IARUAAIhCGAFwAcABBg==&rs=AOn4CLBCyhr8AqpJ1SxKVU6SyK5ODJ_IpA
34 336     188     https://i.ytimg.com/vi/dQw4w9WgXcQ/hqdefault.jpg?sqp=-oaymwEcCNACELwBSFXyq4qpAw4IARUAAIhCGAFwAcABBg==&rs=AOn4CLB_p0PncTtkrhaNDZtntrE3gKkoYw
35 480     360     https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/hqdefault.webp
36 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/sddefault.jpg
37 640     480     https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/sddefault.webp
38 unknown unknown https://i.ytimg.com/vi/dQw4w9WgXcQ/hq720.jpg
39 unknown unknown https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/hq720.webp
40 1280    720     https://i.ytimg.com/vi/dQw4w9WgXcQ/maxresdefault.jpg
41 1920    1080    https://i.ytimg.com/vi_webp/dQw4w9WgXcQ/maxresdefault.webp

Is there a way to limit that to a smaller resolution, e.g. 480p, or select it by its id, e.g. 37?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ yt-dlp -vU -f 140 --embed-thumbnail https://www.youtube.com/watch?v=dQw4w9WgXcQ
[debug] Command-line config: ['-vU', '-f', '140', '--embed-thumbnail', 'https://www.youtube.com/watch?v=dQw4w9WgXcQ']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05ef6]
[debug] Python 3.10.5 (CPython 64bit) - Linux-5.18.0-2-amd64-x86_64-with-glibc2.33 (glibc 2.33)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.1 (fdk,setts), ffprobe 5.1
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.2
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.07.18
yt-dlp is up to date (2022.07.18)
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=dQw4w9WgXcQ
[youtube] dQw4w9WgXcQ: Downloading webpage
[youtube] dQw4w9WgXcQ: Downloading android player API JSON
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] dQw4w9WgXcQ: Downloading 1 format(s): 140
[info] Downloading video thumbnail 41 ...
[info] Writing video thumbnail 41 to: Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].webp
[debug] Invoking http downloader on "https://rr1---sn-vuxbavcx-n3bs.googlevideo.com/videoplayback?expire=1659354810&ei=WmrnYr7AHZCN6dsPhPqiqA4&ip=<IP_ADDRESS>&id=o-APesyoCZUNYanjl-1_XYn1FClzyojkeglx2VMtpxxSwt&itag=140&source=youtube&requiressl=yes&mh=7c&mm=31%2C29&mn=sn-vuxbavcx-n3bs%2Csn-4g5e6nzl&ms=au%2Crdu&mv=m&mvi=1&pcm2cms=yes&pl=19&initcwndbps=712500&vprv=1&mime=audio%2Fmp4&gir=yes&clen=3433514&dur=212.091&lmt=1651669827062637&mt=1659332933&fvip=4&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=4532434&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIgbFYHZM-UBjkLNQSsI1zAMzLVwf9L8tHIGXPde-0qt48CIQDl4W9Wt4yhBIHl2w7HOyolOrxObolN1ricUVqjxU27UA%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgf2YJBFMI5XrEj6QK4326G1Ku7wB2umzt9bXd8ChJXk0CID5yAdiMKjQJO1mULQkgXdiDDyfufh3PgEs8COLQJ1s0"
[download] Destination: Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].m4a
[download] 100% of 3.27MiB in 00:02
[FixupM4a] Correcting container of "Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].m4a' -map 0 -dn -ignore_unknown -c copy -f mp4 -movflags +faststart 'file:Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].temp.m4a'
[ThumbnailsConvertor] Converting thumbnail "Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].webp" to png
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -f image2 -pattern_type none -i 'file:Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].webp' -movflags +faststart 'file:Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].png'
[EmbedThumbnail] mutagen: Adding thumbnail to "Rick Astley - Never Gonna Give You Up (Official Music Video) [dQw4w9WgXcQ].m4a"

Not currently possible https://github.com/yt-dlp/yt-dlp/issues/237


Error in schema validation for ItemBatcher

Describe the bug
Error when validating DISTRIBUTED mode Map step with ItemBatcher option specified.
Error:
✕ State machine "batchedMachine" definition is invalid:
SCHEMA_VALIDATION_FAILED: /States/Batched/ItemBatcher is invalid. must match exactly one schema in oneOf
To Reproduce
Please provide a complete state machine definition:
    batchedMachine:
      name: 'batchedMachine'
      definition:
        Comment: 'batched distributed map step function'
        StartAt: Batched
        States:
          Batched:
            Type: Map
            MaxConcurrency: 1
            ItemsPath: $.items
            ResultPath: $.batched
            End: true
            ItemBatcher:
              MaxItemsPerBatch: 2
              BatchInput:
                batchInput: inputValue
            ItemProcessor:
              ProcessorConfig:
                Mode: DISTRIBUTED
                ExecutionType: EXPRESS
              StartAt: Test-Batched
              States:
                Test-Batched:
                  Type: Pass
                  End: true

Expected behavior
State machine definition should pass validation
Version:
node 16.11.1
serverless: 3.28.1
SDK: 4.3.2
serverless-step-functions: ^3.13.0

I believe the current validation schema is not specific enough, causing valid ItemBatcher configurations to match both entries in the oneOf case.
There are several combinations allowed for this feature by AWS. There are two paired properties:
MaxItemsPerBatch + MaxItemsPerBatchPath
MaxInputBytesPerBatch + MaxInputBytesPerBatchPath
You must specify at least one of the four properties. You may specify one property from each pair. You may not specify both properties from a single pair.

Just for clarity - I closed the automatically linked PR and created a cleaner one with the suggested solution:
https://github.com/ChristopheBougere/asl-validator/pull/134

Released in 3.6.1
Thanks for your contribution!


CXFSA-51: Currency Formating Improvements

https://jira.tools.sap/browse/CXFSA-51
---Code refactoring is needed to be done additionally---

Code refactoring is needed to be done on this branch to improve efficiency and to follow best practices regarding frontend coding.


Could a GitHub release replace a build manifest, should we try it?

Build manifests as are used in OpenSearch / OpenSearch Dashboards contain a lot of information, is all that information necessary or is a GitHub release and associated artifacts enough?
Acceptance Criteria:

Determine if common use cases can fallback to a release driven datastore rather than a manifest file.

After doing a little digging, the release supports a direct publish mode, and a draft that can be published later.  The draft release can take the intended tag name and when the release it published it would automatically be created.  By using a template, this can be pre-populated.
Releases can be editing afterwards, but the tag pointer possible, it would have to be switched.  While releases could be driven by this tool, the pre-release repository is not accessible publicly so additional access would be needed for each repo that supports this.
To use the release feature as part of github, I would advise a system that:

Repo is setup to release on demand
Maintainer wants to make a release
They create a draft release, template includes fields that might / might not be used for releasing
Periodic scan for unpublished releases is found
Run the release workflow in the repo
If the repo successes, publish the release

This automatically creates a release tag at the commit that was pushed
Allows opportunity to associate artifacts to the github release page
Allows opportunity to include links/refs/notes that are release specific

If there are details that need to be managed in a format in-line with the existing input manifest model, a shared github action could create manifest file after a release draft is created.
Potential areas of concern

How will maintainers know not to hit 'publish' and instead rely on the deployment workflow?
If there is a release failure, how does the team find out?
Each repository will need to allow the workflow system permissions to scan for draft releases.

Github action on release
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#release
Release object reference for github
https://docs.github.com/en/graphql/reference/objects#release
Creating templates for release notes:
https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes
Api reference release notes:
https://docs.github.com/en/rest/reference/releases

The GitHub release seems like a better option that makes the client release a push based model. Removing the manifest file completely. The GitHub release needs to be created/drafted for the jenkins release workflow to run. The trigger payload gives the necessary details that can be used by the jenkins lib functions https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#release
As stated above release assets can be used for releasing. For example, the release is based on tags, the high level end to end workflow would look like:

The GitHub release seems like a better option that makes the client release a push based model. Removing the manifest file completely. The GitHub release needs to be created/drafted for the jenkins release workflow to run. The trigger payload gives the necessary details that can be used by the jenkins lib functions https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#release As stated above release assets can be used for releasing.
For example, the release workflow is triggered on tags, the high level end to end workflow would look like:
In the long run, this is low/no maintenance compared to manifest files. The jenkins File used for releasing can differ the source of artifacts used for publishing (download from s3 / GH release / any other source)

Can you explain a bit on what draft a release means here?

Can you explain a bit on what draft a release means here?

@peterzhuamazon See https://github.com/marketplace/actions/gh-release
You can decide if you need to create/draft/publish/delete a release on Github https://github.com/marketplace/actions/gh-release#inputs

@gaiksaya I like this solution but does this tie our release process with GitHub repo and tools? I am good with this model but want to make sure that the process can be easily reproduced outside of GitHub as well.

It does not directly tie it to GH but since our repos, their CIs are on GH it makes sense if part of it is tied. The jenkins libraries that will take care of the actual release will be independent of GH rather GH is a bridge that will trigger, bring the artifact to jenkins.

+1 on the staging area being GitHub. I think this will work at least for Java (Maven) and Ruby (pkg) artifacts.

Hey, few thoughts:

One option is, the build and compile should completely happen on Jenkins, so the artifacts can be shipped from Jenkins workspace to the external repos.

The build and compile code can run on gitHub workflows but again should be shipped to a central place (s3, minio or amy other option), where Jenkins can pick the artifact and just run the code to publish the artifacts, with this I see management of jenkins, agents and shared libraries will be minimal, as the heavy lifting is being done with gitHub workflows, jenkins is just used to ship the artifact.

Github has an option to store artifacts on the repo itself https://git-lfs.github.com/ (should explore more on this) where jenkins can pull the built artifacts directly.

Please note that git LFS is not free, and you can easily DDoS someone else's account.

We will be going ahead with GitHub release. For artifact staging, if release assets are not sufficient we can look into bringing up the infrastructure to support additional storage. Updated the design doc with the above approach.
Thanks!


Retrieve Current Air Temperature

Hello Everyone,
I'm not a developper and I can barely read the Pyton code of this extension.
Whatever, I use this plugin to Set On or Off my Ravelli pellet stove, from Home Assistant.
I would really like to stop totally using the Ravelli application, but for that, I would like to know what is the actual temperature from the remote device. (remote controller from ravelli)
If somoene could explain easily, in Home Assistant : How to retrieve the current temperature and Print it in an entity or in the existing Climate entity.
Many Thanks,;
Best Regards.
Gauthier

Hello Gauthier,
The climate entity exposed by the plugin is aware of both target and current air temp. On climate card, current is on first line, bold, and target on second line.
Using entity card, you can have access to the properties of climate entity of your stove. But it's a bit limitating.
The integration is laking of direct access to sensors.
I'm thinking to add additionnals entities for each device, mainly using sensor / bynaries sensors, to be able to uses easily data given by stove. Similar thoughts for handling alarms and popup (low wood, etc...) in HA.
As the plugin have been splitted from backend, this issue apply most to the plugin : https://github.com/vincentwolsink/home_assistant_micronova_agua_iot

hi @GauthierCrp and @jipem01
In Home Assistant most of the sensors can be found in the status attributes of the climate entity.

For displaying your climate entity I recommend using the Simple Thermostat Card.
You can also make it show those sensors clearly.


[EC2] Preserve spaces in EC2 path

Simple demonstration of the problem and the fix:
$ spacey_path="/path/with some/spaces"
$ dirname $spacey_path
usage: dirname path
$ echo $?
1
$ dirname "$spacey_path"
/path/with some
$ echo $?
0

cc @JoshRosen

Test build #26169 has started for   PR 4224 at commit 960711a.

This patch merges cleanly.

Test build #26169 has finished for   PR 4224 at commit 960711a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26169/
Test PASSed.

LGTM. Would be good to create a JIRA especially if we want to backport. @JoshRosen might have more ideas on backporting

@shivaram JIRA created.

Ok I'm merging this into master and marking it as backport needed (since we're in the middle of a 1.2.1 release)

I backported this to 1.2


chatcommands: add shorthand names for all agility courses

Adds shorthand names for all agility courses, including common misspellings.
I have confirmed each longBossName in game, as they don't all match what is shown when talking to Grace.
Closes #12529

Maybe Pollnivneach should also include poly and/or polly, and maybe Gnome Stronghold should also include gnome.

Imo we should not normalize the usage of misspelled versions of city names such as canafis, relekka or polnivneach.

I've removed the misspellings.


x axis offset

Hi,
Sometimes it seems my bar charts have a strange offset on the x-axis and the labels for each column do not show up, only for some of them. Note, the tool tips all render fine.
Any ideas?:

@blubberbo this is already fixed in the master branch and will be in the next release. PR was #3508


Suggestion / request: use Dependabot to keep dependencies up-to-date

First of all, thanks for speakerinnen.org!
I've got a suggestion / request: would you be up for using Dependabot to automatically create dependency update PRs for this repo? I ran it against my fork and it generated these PRs. I'll port the nokogiri, sprockets and rubyzip ones across to this repo now because they're security related.
I built Dependabot, but I'm honestly only suggesting it because I hope it can save you some time. I'd love any feedback, and obviously having open source repos using Dependabot helps boost its profile, but if it's not helpful to you then it's not really worth anything.
You can install it from here or here if you decide to give it a try. It's been through GitHub's security testing (to be allowed in the GitHub Marketplace) and is used by a few thousand organisations, and the source code is here.
:octocat:

Thank you for the suggestion and the PR's. We will discuss it in the team but it seems to be reasonable to add this gem.

I installed it. Thank you for your suggestion!

Awesome! Looks like you've got it running on https://github.com/rubymonsters/fiftypercent but not enabled on this repo (although you've given Dependabot permission for this repo). Want me to set it up to send PRs to this repo too?


Update circle executor image to a supported ubuntu version

#167  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


Lots of warnings logged

Having just JavaDungeons 1.2.1 and FAPI 0.16.2 build 385 on loader 0.9.0 build 204 causes a lot of warnings logged. This is not a major problem but a bit inconvenient when trying to find things in the log.
Specifically:
[15:46:58] [Worker-Main-10/ERROR]: Using missing texture, unable to load dungeons:textures/particle/particle/green_flame.png : java.io.FileNotFoundException: dungeons:textures/particle/particle/green_flame.png
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: dungeons:blockstates/dungeons_water.json: java.io.FileNotFoundException: dungeons:blockstates/dungeons_water.json
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=0'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=5'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=6'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=7'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=8'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=1'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=2'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=3'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=4'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=15'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=14'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=9'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=11'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=10'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=13'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/dungeons_water.json' missing model for variant: 'dungeons:dungeons_water#level=12'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: dungeons:blockstates/soggy_swamp_water.json: java.io.FileNotFoundException: dungeons:blockstates/soggy_swamp_water.json
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=12'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=11'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=14'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=13'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=6'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=7'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=8'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=10'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=9'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=2'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=3'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=4'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=5'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=15'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=0'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/soggy_swamp_water.json' missing model for variant: 'dungeons:soggy_swamp_water#level=1'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/tent.json' missing model for variant: 'dungeons:tent#facing=down'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/tent.json' missing model for variant: 'dungeons:tent#facing=up'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/teapot.json' missing model for variant: 'dungeons:teapot#facing=down'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/teapot.json' missing model for variant: 'dungeons:teapot#facing=up'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/chains.json' missing model for variant: 'dungeons:chains#above=true,below=true'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/cw_gravestone.json' missing model for variant: 'dungeons:cw_gravestone#facing=down'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/cw_gravestone.json' missing model for variant: 'dungeons:cw_gravestone#facing=up'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/cw_mossy_gravestone.json' missing model for variant: 'dungeons:cw_mossy_gravestone#facing=down'
[15:47:01] [Worker-Main-9/WARN]: Exception loading blockstate definition: 'dungeons:blockstates/cw_mossy_gravestone.json' missing model for variant: 'dungeons:cw_mossy_gravestone#facing=up'
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/mossy_gravestone
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/teapot
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/unlit_brazier
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/cw_gravestone
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/brazier/green_lit
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/glow_mushroom/0
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/gravestone
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/ripped_banner
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/cw_glow_mushroom
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/glow_mushroom/2
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/food_box/fish
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/green_lit_brazier
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/glow_mushroom/1
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/pop_flower/0
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/pop_flower/1
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/pop_flower/2
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/pop_flower/3
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/food_box/orange
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/flower_patch
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/fish_food_box
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/teapot
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/open_book
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/creeper_woods/glow_mushroom/3
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/ripped_banner_rotated
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/ripped_banner
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/corn_food_box
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/cw_mossy_gravestone
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/water_plant
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/closed_book
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/brazier/lit
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/book/open_0
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/book/open_1
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/book/open_2
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/food_box/corn
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/orange_food_box
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/book/closed_0
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/book/closed_1
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/book/closed_2
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/tray/yellow_tulip
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/apple_food_box
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/brazier/unlit
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:block/food_box/apple
[15:47:02] [Worker-Main-9/WARN]: Unable to resolve texture reference: #missing in dungeons:item/lit_brazier
[15:47:02] [Worker-Main-9/WARN]: Texture dungeons:block/coals with size 8x8 limits mip level from 4 to 3

(This log is from a fresh MultiMC instance btw)

Duplicate of #16
this has been fixed, but the release with the fix is not out yet (should be out sometime in the near future)


Just for info: bootstrap like checkboxes and radios

I suppose you already know this .. https://github.com/flatlogic/awesome-bootstrap-checkbox

There is a little problem with this checkboxes you may fix I think.
You currently layout the label/input as below::

    <input type="hidden" name="preload" class="form-control "  value="0" />
    <input type="checkbox" name="preload" value="1" id="preload" checked="checked">Préchargement du point d'intérêt

And according to the doc here https://github.com/flatlogic/awesome-bootstrap-checkbox#use, that cause the checkbox to not working.
So the correct layout should be as follow, notice the start  tag put after . That won't change anything visualy or functionnaly I think::

    <input type="hidden" name="preload" class="form-control "  value="0" />
    <input type="checkbox" name="preload" value="1" id="preload" checked="checked">
    Préchargement du point d'intérêt

Hi Alain, the helpers are made to work with default bootstrap, the plugin you linked is not using the default bootstrap markup (see http://getbootstrap.com/css/#forms).

Ok Mickael but as shows the first code block which, if I didn't mistaken is the layout you provide, you don't respect the rule http://getbootstrap.com/css/#don't-mix-form-groups-with-input-groups
input should be outside the label, as also specify the link I gave you about the new checkbox shape no?

Check the link I provided (http://getbootstrap.com/css/#forms), and you'll see that checkbox and radio inputs are inside the label:

    <input type="checkbox"> Check me out

The only thing that change between the bootstrap example and my code is the hidden input which has no effect on the global output.

Ah! for checkboxes ok, which is strange as all others are not made in the same way !?!
I didn't found an explanation about that choice.


Satistics -> Wave Population

See snapshot below for directional wave population

Thank you,
I have corrected this one


MRG, MAINT: Cleaner split, fewer skips in _brain tests

This PR refactors renderer_interactive into two forms: one that includes mayavi (renderer_interactive) and makes it clear in the code that PySurfer is required, too; and another that just uses PyVista (renderer_interactive_pyvista). This makes it so that we don't get a bunch of unnecessary skip messages for tests that were never meant to be run on Mayavi/PySurfer.
This is a first step toward making it so that we can iterate over 'notebook' as a backend type in the renderer_interactive tests. @GuillaumeFavelier feel free to review and merge if you and CIs are happy

Thanks for pushing fixes @GuillaumeFavelier


Heading right after bullet

Fixes #337
When H1 heading is right after the bullet (one new line between) compiler can't parse it as heading.
Exaple:
- **bullet**
# [Heading](http://domain.com)

## Test

Every markdown editor, I can find, interpret this as heading.
Bellow GitHup interpretation with the same markup:

bullet

Heading
Test

@czterystaczwarty thanks, can you mark the test with it.skip for merging purposes and I'll unskip it with the fix later?


fix: Keep falsy JS values in success handler

This is just a quick (untested) workaround and might lead to other problems.
As suggested in #12 I'd return T instead of Option<T> anyway.

Closing this as the issue was resolved in #14.


"Internal Server Error" when accessing pyDash

I followed the instructions of using pyDash with Nginx + Gunicorn, with the exception of cloning the repo into a directory that's not owned by a normal user (actually owned by nginx, where I put all my server apps), and installing it into a virtualenv.
But when I run the server with:

sudo gunicorn -b <IP_ADDRESS>:25304 wsgi

Page on my browser shows Internal Server Error, and the following crap got spit out in my console:
https://gist.github.com/SammDu/b3b0867210540bf52e883cb87c648ab3
(Yup please click on the link to the gist cuz it's a nightmare to paste it directly in here lol)
Yeah so that where I'm stuck at and please take a look, thanks very much!

BTW if you want a record of what I did from the very beginning,
please take a look at this gist:
https://gist.github.com/SammDu/41fbe3e16b37ec62a838a2093b186b06

@k3oni please take a look thanks bro X-D

@SammDu - Try running the pip install with the -H flag when using sudo.  Without the -H flag the perms on the install could be incorrect resulting in your traceback (as shown in your gist)

Try
`sudo gunicorn -b <IP_ADDRESS>:25304 pydash.wsgi

Sorry for the delay...I've been kindda busy these days, but I'll be sure to try both suggestions out as soon as I can get my head around it. Thanks very much for responding, I appreciate the effort!


Hololens Pointer disappear while project deployed into the Hololens

I use Unity 2019 and MRTK 2.5.0

After I deploy this project into Hololens, the pointer is disappeared and can't operate anything

Hololens Preview:

XR Setting:

Seems like some error

Fixed

this error appears again

Can you take a look at https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/GettingStartedWithMRTKAndXRSDK.html and make sure that you have followed all the steps? It seems that this issue is related to the version of Windows XR Plugin (i.e. you might need to upgrade).

Can you take a look at https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/GettingStartedWithMRTKAndXRSDK.html and make sure that you have followed all the steps? It seems that this issue is related to the version of Windows XR Plugin (i.e. you might need to upgrade) and the profile of the MRTK GameObject.

I use 2.3.0 version for Windows XR Plugin, don't know whether it is the newest version

Do you still see the compilation error in the console? If not please describe what is the problem you are experiencing now.

Do you still see the compilation error in the console? If not please describe what is the problem you are experiencing now.
I always meet this problem since I have already added the MixedRealityInputModule into Canvas, even Camera and Event System

@Microsoft

You should not add that input module to anything other than the main camera under the mixed reality place space GameObject (and even for that GameObject the module should be added automatically already). You can try run the example scene first and remember to change the MRTK profile to a XRSDK one.

Solved


fix(minecraft-proxy): use generic naming

As per https://github.com/itzg/minecraft-server-charts/pull/50#issuecomment-781757398, added to the readme.
Also bumped the version for major.minor.patch & replaced explicit references to bungee with proxy.

Should this be a major version bump as it's a breaking change?


Upgrade bitcoinconsenus

Upgrade to the most recent bitcoinconsensus version that excludes Taproot verification i.e., one version before latest.

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 84.043%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19540

Relevant Lines:
23250

💛  - Coveralls

Flagging that I hit an ASAN CI fail, re-ran the job and it went green.


set GSP threshold to > 0 for

Pull Request
Description
Simple change to filter on GSP threshold mw > 0
Related to issues #303 and #205
How Has This Been Tested?
normal unittests

[ ] No
[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

LGTM! Good work! You're really powering through lots of issues! Very exciting to see!
One tiny comment: Maybe change the comment on line 96 of gsp_data_source.py so it says something like "drop any GSP below threshold_mw." (i.e. remove the "20 MW").
I took the liberty of very slightly editing the PR description: I changed "Fixes #303" to "Related to issues #303 and #205" because, in 2022, it might still be nice to implement a config option so users can set the GSP threshold_mw. And I moved issue #303 from WP1 to WP2

Thank you


Feature/stop codon readthrough edit

Description
ENSCORESW-3110- Feature to handle stop codon readthrough in amino acid sequence.
Use case
At the moment, for a set of 12 transcripts, the stop codon readthrough is identified by an asterisk. This feature will modify the amino acid sequence to display the stop codon readthrough with an 'X'.
Benefits
This will bring stop codon readthrough representation in line with the stop codon readthrough representation in RefSeq.
Possible Drawbacks
None.
Testing
One dummy test added in translation.t which executes successfully. An actual test case cannot be added at the moment as sequences containing asterisk representation for stop codon readthrough is unavailable in core database. This is available in loutre database.

@s-mm Let us wait until #374 has been merged so that Travis (hopefully) stops failing on unrelated problems. Or you could merge bugfix/e97_testdb_patches_master into your branch once (if?) the Travis build for #374 has succeeded.

I have just changed the PR base to bugfix/e97_testdb_patches_master to make it easier to see what it actually changes. Note to whoever ends up merging this: please change the base back to master beforehand.

Coverage increased (+0.005%) to 81.459% when pulling 331fec91d975d2a8e4a34d14e3f60550485d92a2 on feature/stop_codon_readthrough_edit into 29abf57415cd2707afdc1747ebdbfe7b778c2f18 on bugfix/e97_testdb_patches_master.

...and I have just realised the attribute code introduced in this PR is not valid because the type of attrib_type.code is VARCHAR(20). Have let @s-mm know this will have to be fixed. I wonder why the test harness doesn't fail on that new line in attrib_type.txt; another fun property of MyISAM?

I think MySQL silently truncates unless it is configured to barf.


imaplib.error: command SEARCH illegal in state AUTH, only allowed in states SELECTED

I'm trying to setup a cron job for django-helpdesk to create tickets from my inbox. I've configured a cron job and then I tried to do this:
python manage.py get_email
It gives me this error:
imaplib.error: command SEARCH illegal in state AUTH, only allowed in states SELECTED
Here's the full error log:
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/django/core/management/__init__.py", line 351, in execute_from_command_line
    utility.execute()
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/django/core/management/__init__.py", line 343, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/helpdesk/management/commands/get_email.py", line 73, in handle
    process_email(quiet=quiet)
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/helpdesk/management/commands/get_email.py", line 106, in process_email
    process_queue(q, logger=logger)
  File "/home/commercial/.virtualenvs/projects/lib/python2.7/site-packages/helpdesk/management/commands/get_email.py", line 198, in process_queue
    status, data = server.search(None, 'NOT', 'DELETED')
  File "/usr/lib64/python2.7/imaplib.py", line 639, in search
    typ, dat = self._simple_command(name, *criteria)
  File "/usr/lib64/python2.7/imaplib.py", line 1082, in _simple_command
    return self._command_complete(name, self._command(name, *args))
  File "/usr/lib64/python2.7/imaplib.py", line 837, in _command
    ', '.join(Commands[name])))
imaplib.error: command SEARCH illegal in state AUTH, only allowed in states SELECTED

Are there steps I'm missing somewhere? I'm following what is mentioned here:
https://django-helpdesk.readthedocs.io/en/latest/configuration.html

You may have stumbled on a bug here. I think the IMAP code is a little less well tested than other areas.
It looks like it's failing to select your IMAP folder before performing a search. Since the line immediately before the one that crashes is a select() statement, I'm thinking the IMAP folder you're trying to reach might not exist, and we're not properly checking that and notifying you before it crashes hard, so at a minimum we should provide a better error message here.
Please double-check that the IMAP folder you requested exists on the server and that you have added the folder name to your Queue configuration (if it is blank, it tries the default folder of 'INBOX' which maybe your email server doesn't have?).

Yes, you're right, the folder name had a typo. That's why it wasn't seeing it. I fixed the name and that fixed the issue. Thank you so much!

Awesome! Glad it was an easy fix. I still think we need a better error message here so I'll keep the ticket open for now to remind myself. Let us know if you have any other problems.

Added a logger error message suggesting user check if IMAP folder is spelled correctly and exists. Committed to master for the 0.2.0 release. Hopefully this will help anyone else encountering the error!


[USMON-1010] pktbuf prep

What does this PR do?
Introduces infrastructure changes in the pktbuf API to ease the migration of HTTP2 codebase

Motivation
Preliminary change for http2 migration

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Astropy equivalencies not fully supported

We have this doctest example:
x = Quantity([1, 2, 3], "Kelvin")
with u.add_enabled_equivalencies(u.temperature()):
    y = x.to(u.deg_C)

which works, but this one doesn't work:
x = Quantity([1, 2, 3], "radian")
with u.add_enabled_equivalencies(u.dimensionless_angles()):
    y = x.to(u.one)

I think this is because the physical type (dimension) changes in the latter case and not the first. See https://github.com/GalacticDynamics/unxt/blob/main/src/unxt/_src/quantity/base_parametric.py#L43

Actually I think the main issue is with this line - I don't think we want to do a replace here (that line just replaces the unit and value of a quantity), because conversions with equivalencies can change the physical type (dimensions) of a quantity (such as with dimensionless_angles()).

Right. Though do we expect other arguments? If we don't mind type coercion, one option would be to swap the replace for:
return Quantity(_apy7_unit_to(x.unit, unit, x.value), unit)

We can support all fields by using the dataclassish.field_items()
Preventing unnecessary type coercion is a bit more fiddly. I think we can do this by checking the dimensions of the output unit against the input units and only doing type coercion if they are different.


Initialize SDKClient with instance of ExtensionSettings

Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #579 (332aaa8) into main (4dc426a) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #579      +/-   ##
============================================
+ Coverage     69.85%   69.92%   +0.06%
  Complexity      216      216
============================================
  Files            40       40
  Lines           889      891       +2
  Branches         26       26
============================================
+ Hits            621      623       +2
  Misses          257      257
  Partials         11       11

Impacted Files
Coverage Δ

...main/java/org/opensearch/sdk/ExtensionsRunner.java
77.34% <100.00%> (ø)

src/main/java/org/opensearch/sdk/SDKClient.java
91.66% <100.00%> (+0.17%)
:arrow_up:

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


Schedule & Timeout annotated method cannot be decorated

Integrating the library I came across having some methods not publishing metrics as expected.
Those methods are some EJB methods which are timer based.
Thus the followings example do not work
1. Example with @Timeout
@Timed(name = "schedule")
@Timeout
public void scheduledMethod() {
}

2. Example with @Schedule
@Schedule(hour = "*", minute = "*", second = "*", persistent = false)
@Timed(name = "schedule")
public void scheduledMethod() {
}

I'll provide test cases in a fork branch.
Not sure it is normal or if it comes from a miss usage. Feel free to comment if I am doing soemthing wrong.

Hi Matthieu,
Thanks a lot for reporting. Indeed, instrumentation of @Schedule and @Timeout methods isn't currently supported. It should be straightforward though to add that support by adding a @AroundTimeout interceptor method along with the existing @AroundConstruct and @AroundInvoke ones in https://github.com/astefanutti/metrics-cdi/blob/964e26a321b777613d07f33812bc32f57a5eb13e/impl/src/main/java/io/astefanutti/metrics/cdi/TimedInterceptor.java#L46-L54 and the other interceptors.
I'll work on it ASAP though feel free to contribute if you will in the meantime :-)

I will provide it. I didn't thought it was as easy as that. I am not totally new to CDI but I was not aware that timeouts methods had their own interceptor scheme.

Thanks. I'll do a release ASAP.

Ok thank you, that will save me to have to maintain my own release in my company.

I've just release version 1.1.0 to Maven Central: http://repo1.maven.org/maven2/io/astefanutti/metrics/cdi/metrics-cdi/1.1.0/
Thanks for your contribution!

You're welcome and thanks for the fast release.


Out-of-box integration with istio

As mentioned in our roadmap, let's address this after we bump the default Kubernetes version to v1.8, which supports the Initializers admission control required by automatic-injection of istio sidercars.

I began to think that we should wait at least until istio v0.6+ for:

Autoscaling in Istio using horizontal pod autoscaler
Support for Istio upgrade in Kubernetes
Istio Scale & Performance · Issue #369 · istio/istio
Lots of broken pipe messages · Issue #2728 · istio/istio
Scale issues in istio-pilot discovery service, affecting alpha perf targets and installation · Issue #1485 · istio/istio

Wish I could have more time to actively contribute to Istio but at this point I don't have enough confidence to recommend it for kube-aws users (as, literally, we produce kube-aws for production usage).

Hello All, I was looking into getting this working with the most recent kube-aws release:
kube-aws version v0.10.2
Sadly, the Istio docs only mention requisite configurations for Kops in AWS
I asked the Istio Users and Developers groups for any info and also posted in
istio rocket chat forum
Specifically if anyone in the groups or chat channel had information on how to integrate istio with default vanilla kubernetes (if not kube-aws), etc.
Tl;DR: Only got one response (which just stated "you need to get the order correct") from all of those forums. Not much to go on for me.
I also pinged the  kube-aws slack channel re istio/kube-aws integration thread:
I got some great feedback and thanks again @c-knowles
Must add the below to cluster.yaml and then kube-aws update
    experimental:
    mutatingAdmissionWebhook:
      enabled: true
    validatingAdmissionWebhook:
      enabled: true

Okay, so moving along, getting back to istio docs...
ref istio docs side-car-injecttion

Sidecars can be automatically added to applicable Kubernetes pods using a mutating webhook admission controller. This feature requires Kubernetes 1.9 or later. Verify that the kube-apiserver process has the admission-control flag set with the MutatingAdmissionWebhook and ValidatingAdmissionWebhook admission controllers added and listed in the correct order and the admissionregistration API is enabled.

# istio docs
$ kubectl api-versions | grep admissionregistration
admissionregistration.k8s.io/v1alpha1
admissionregistration.k8s.io/v1beta1

So, now that I've added the initial requisite cluster.yaml file tweaks from Chris in the above kube-aws thread, and comparing against the above Istio doc specified output, I only have
# my kube-aws cluster
kubectl api-versions | grep admissionregistration
admissionregistration.k8s.io/v1beta1

'Not knowing how to hack the next step, I looked at the Istio AWS Kops doc (link above) and thought I might be able to proceed by taking the Istio Kops-specific instructions and trying to do the equivalent on my kube-aws provisioined cluster, but could use some advice, confirmation,  etc.  I.e.:
Below istio aws platform setup only shows kops
It does have a different order for the kubeAPIServer than my kube-aws provisioned cluster,
What is the process to safely hack/modify this?

(For Kops) Add the following in the configuration file:
kubeAPIServer:
    admissionControl:
    - NamespaceLifecycle
    - LimitRanger
    - ServiceAccount
    - PersistentVolumeLabel
    - DefaultStorageClass
    - DefaultTolerationSeconds
    - MutatingAdmissionWebhook
    - ValidatingAdmissionWebhook
    - ResourceQuota
    - NodeRestriction
    - Priority

Looking at my kube-aws provisioned cluster, I do see a different order with the kube-apiserver
for i in $(kk get pods | grep 'kube-apiserver' | awk '{print $1}'); do kk describe pods $i | grep -e "(Node|admission-control)"; done
Node:         ip-10-1-8-102.us-west-1.compute.internal/<IP_ADDRESS>
      --authorization-mode=Node,RBAC
      --admission-control=NamespaceLifecycle,LimitRanger,ServiceAccount,PersistentVolumeLabel,DefaultStorageClass,NodeRestriction,ResourceQuota,DefaultTolerationSeconds,MutatingAdmissionWebhook,ValidatingAdmissionWebhook
Node-Selectors:    <none>
Node:         ip-10-1-8-68.us-west-1.compute.internal/<IP_ADDRESS>
      --authorization-mode=Node,RBAC
      --admission-control=NamespaceLifecycle,LimitRanger,ServiceAccount,PersistentVolumeLabel,DefaultStorageClass,NodeRestriction,ResourceQuota,DefaultTolerationSeconds,MutatingAdmissionWebhook,ValidatingAdmissionWebhook
Node-Selectors:    <none>

If I could get help getting Istio && kube-aws integration working and get your confirmation(s) that the proposed/recommended process is the "kube-aws approved method", I could then do a P/R for the doc enhancement update to support that functionality.
Thank you

@cmcconnell1 any particular problem you are facing? I'm not sure if the alpha admission registration is needed for Istio still?
If you do need that, I dug a little bit and found this:
https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/#enable-initializers-alpha-feature
That is reflected in this line:
https://github.com/kubernetes-incubator/kube-aws/blob/e8d6bea5760fb07a0b5cf83399fc6f0ba23fbbd7/core/controlplane/config/templates/cloud-config-controller#L3073
So can be enabled using this in cluster.yaml:
experimental:
  admission:
    initializers:
      enabled: true

I believe Istio uses to use Initializers but swapped to webhook admission which is why I'm not sure if it's needed any more.
I agree better out of the box integration using the kube-aws plugin support would be good.

Thanks for your follow-up.  I made those changes to the config and applied them but haven't got back to istio testing yet.  I've been working on a few POC's for service mesh integration (of course using kube-aws clusters ;-)  ) and wanted to spend some time with the most popular options.  Being new to service mesh in general, I wanted to make sure that things worked as specified before I invested a lot of time and then introduced to our team.  As often the case, the docs might be a bit out of date on the Istio side.
However, what I can say for me is that I really like Linkerd having a three word cluster validation command
Step 2: Validate your Kubernetes cluster

Kubernetes clusters can be configured in many different ways. To ensure that the control plane will install correctly, the Linkerd CLI can check and validate that everything is configured correctly.
To check that your cluster is configured correctly and ready to install the control plane, you can run:
linkerd check --pre

I would appreciate recommendations and opinions on both Istio and Linkerd service mesh options.  I.e. was there a reason that Istio was targeted for kube-aws integration instead of Linkerd, etc?  This would be very helpful for us trying to herd the cats of kubernetes ecosystem components, integrations, etc.
Thank you

@cmcconnell1 @c-knowles Thanks a lot for chiming in, and your supports!

was there a reason that Istio was targeted for kube-aws integration instead of Linkerd, etc?

This is my personal opinion. I thought it makes sense to provide a smooth path for integrating kube-aws and istio, rather than linkerd, because istio seems to cover much many use-cases and linkerd is already super easy to get started.
Regarding the covered use-cases, I think istio's benefits over linkered(2) are:

cross-cluster mesh
automatic mutual tls
cilium integration
serving as a platform for other high-level platforms like knative

The cross-cluster mesh sounds especially useful FOR ME because then I can use istio as the global load balancer that can be relatively safely rolling-updated.
The global LB can then be used as a solid foundation to achieve blue-green deployments of kube-aws clusters with smooth traffic shifting. It will drastically improve how we could continuously and automatically upgrade kube-aws clusters #1050.

Good morning!  We are trialing Istio in a few of our kube-aws clusters and to the best of my knowledge all we had to do is enable the mutating and validating webhooks and installed istio via helm charts to get it up and running.
I do think that both out-of-the-box Istio and ingress controllers would be great additions to kube-aws and allow users to bootstrap fuller, more useful clusters more easily.  I appreciate that these are areas where there are admittedly quite a few options and lots of movement but I think it would be good for us to offer a couple of opinionated choices - much like we do with networking.


Artisan make:model only creates form requests when selecting resource controller as well

Laravel Version
11.7.0
PHP Version
8.2.18
Database Driver & Version
MySQL 10.11.7
Description
When running php artisan make:model and declare that I would like to have "Form Requests" generated, it does so ONLY when I also check "Resource Controller"

Use case:
As I already had the controller created I would have liked to only create the model and form requests.
Steps To Reproduce

php artisan make:model
check "Form Requests" without checking "Resource Controller"
-> No form requests are created

This is on purpose as these are only tied in with resource controllers. Just selecting form requests won't do anything because you didn't need a resource controller in the first place. Might indeed be a little weird gotcha.

@driesvints Thanks for the explanation 😊
The options would be more clear like:

...
Resource Controller
Resource Controller (with From Requests)


Add support for Amazon linux

NAME="Amazon Linux"
OS Type          : Linux
VERSION_ID       : amzn
VERSION_ID_LIKE  : fedora
VERSION_PRETTY   : Amazon Linux 2023.4.20240429
VERSION_STR      : 2023

Thank you for the fix.
I have committed these changes with minor edits, and added the code to handle the required packages.


Unable to import a 2MB file

Hi,
I'm trying out 'tbx' on a 2MB file.
Command used is"tbx services google sheets sheet import -id 1M3n9tA378VY3TXrjU2LENMD3NxcJCG3UvxEDTZnytsI -range Sheet1! -data OSSBooking_test.csv"
Error is
Unable to read file OSSBooking_test.csv' due to an error record on line 23: wrong number of fields Unable to read file OSSBooking_test.csv` due to an error record on line 23: wrong number of fields
The command finished: 4.343s
Operation failed with an error: record on line 23: wrong number of fields"
Can you help me where to look and what to do as I'm hoping tbx is the solution to my dilemma when importing CSV files greater than 2MB and the file contains all columns.
Yours,
Bing

Thank you for reporting. This program, tbx, uses https://pkg.go.dev/encoding/csv for parsing CSV. That means the tbx expect the file formatted with RFC 4180 compliant CSV file.
I believe it's not a matter of file size. I frequently import CSV > 2MB without an error.
Please verify your CSV file line number 23.


a discussion than an issue

Hi,
I tried out the sample, it works perfectly fine. Thing doesn't work as expected when I added a new file.
I added a file UnitTestSample in main/java/src.
`public class UnitTestSample {
public static int add(int i, int j){
    return i+j;
}

}`
and a UnitTest file in test/java/src
public class UnitTest {     @Test     public void testAdd(){         assertEquals(2,UnitTestSample.add(1,1));     } }
What I expected from the coverage report is that it should shows 2 classes, MainActivity & UnitTestSample under package net.rafaeltoledo.coverage but it shows only MainActivity. The coverage report is at app/build/reports/jacoco/jacocoTestReport/html/index.html.
Is there anything I missed out?

After playing around with the gradle task, I found that the filter has been filtered out the UnitTestSample class with '**/Test.*'.


Windows Filetime mod

Small change to Windows Filetime input and output to allow hex. Hex is often seen in registry settings and Tool output. Decimal is often seen in LDAP and PowerShell usage.

Thanks, this is a useful addition. I've updated the tests to specify the new argument values.


Perkeep on Raspberry Pi 3

Hello everyone,
I'm interested in this project and wanted to test it on a Raspberry Pi 3. Since there aren't binary packages for ARM, I resorted to compiling it from source, but I run in the following issues:

The master branch compiles and starts fine, but crashes after a while when trying to upload two video files filling the terminal with errors about goroutines.
The staging branch refuses to compile with an error about version length that I don't remember well
The releases/0.10 branch, which is the one I'm mostly interested in since at the moment is the latest stable release (although I saw that in issue #1307 that you're thinking about a new stable release), compiles and starts fine, but when I log in I only see a blank page and in the terminal it trows errors about being unable to load resource files (mostly CSS files).

I can't post here the exact errors right now because I decided to try again with master before writing and compiling different versions involves changing the version of go as well, but if you need it I can post them here in the next days.
What should I do to be able to compile and run Perkeep on my Raspberry Pi?
I saw that somebody else had issues running it on a Raspberry Pi 3, but that issue involved Docker and I would prefer running it natively. Thanks for your help

Quick update: I tried uploading two small files, one at a time, using the master version. It seems to have worked, so maybe the crash was due to the small RAM of the Raspberry?
But now I'm unable to see the two files using the web UI. One of them is called "arte_moderna.txt" but the search query  "filename:arte_moderna.txt" returns no file, the other is a png image, but "is:image" returns no result.

@GTP95 I was thinking of trying to build Perkeep on Pi 4. Did you solve the out of RAM on Pi 3 issue?

How much ram do you have, and how much data are you storing? I wonder if you're running afoul of #1271

8GB. Any chance you have a stable Perkeep build for Pi? (Amazon van just delivered shiny new Pi 4)

Sadly, no -- my boxes are all x86.

Ok, I will try to see what I can build. Thanks for #1271

@GTP95 I was thinking of trying to build Perkeep for Pi 4. Did you solve the out of RAM on Pi 3 issue?

No, for now I'm just storing smaller files like photos, I may try again with the two videos, maybe uploading one at the time and with increased swap storage could help.

Ok, I will try to see what I can build. Thanks for #1271

If you have any problems with the build I can send you my binaries compiled on my PI 3

I've tried one video at a time and with more swap space. This time perkeep didn't crash, but I'm unsure if it really uploaded the videos, since I'm unable to find them.


[WIP] Add GraphQL mutation to REST endpoints

It's a PR for https://github.com/elitan/hasura-backend-plus/issues/15
Now it's just support register as mutation.
After some test we can add other endpoint as GraphQL mutation and use Hasura remote schema feature with it.

We must add GRAPHQL_PORT environment variable too.
Also Maybe it's better to add another environment variable as flag called ENABLE_GRAPHQL to able to enable or disable graphQL endpoint.
@elitan What is your idea?

The main reason would be that you can not send files via the graphql protocol so the storage part of hasura-backend-plus need to be run as a express server (or similar).


Delete spaces at ends of lines

Especially double spaces at ends of lines often create strange formatting
artifacts in various renderings.

@mlinksva thanks for approving. You might consider going ahead and merging this one, so you don't end up with merge conflicts all over the place as folks send new stuff off master.