Add Ori to the metallb owners

Adding Ori, he is already an upstream maintainer and this would help in sharing the reviews.

Also, removed mark and casey as they are not with us anymore.

/override ci/prow/metallb-e2e-metal-ipi

/lgtm


Issue #69: Mainnet gas settings

Closes #69
Description
There were two issues with why OP wasn't working w/the bot

Optimism wasn't added to the available networks

I added it to the code

The payer wallet was not paying the bot wallet because the bot was over the minimum drop threshold (it had a 0.000203447032002334 balance and it needs to be lower than 0.0002)

I added a variable called MINIMUM_BOT_BALANCE to gasConfig.js that is the value of MINIMUM_GAS_DROP_THRESHOLD but another decimal larger. This ensures that we won't get into an infinite loop of the bot not having enough gas to do a transaction but it'll have enough
Note that we will have to make sure our bot initially has enough gas for each of these MINIMUM_BOT_BALANCE values or else the calls will keep failing until the bot has been given enough gas by the payer to meet the balance
Screenshot
Log when gas is below MINIMUM_BOT_BALANCE, airdrop goes to bot

Transactions from payer -> bot

going to wait to merge this until https://github.com/open-dollar/od-bot/issues/76#event-12296925677 is finished


Add support for jsrun and bsub/jsrun for launching parallel jobs.

Description
Resolves #19099
I added proper support for using jsrun and bsub/jsrun for launching parallel jobs on IBM systems. I added options in the GUI. I added support in the internallauncher and updated the LLNL customlauncher to leverage the new support. I updated the host profiles for lassen, rzansel and sierra.
Type of change

[X] New feature~~

How Has This Been Tested?
I built and installed a private version on rzansel. I then updated the host profiles to point to my private installation and proceeded to test launching using bsub/jsrun and just jsrun.
I tested bsub/jsrun by creating a pseudocolor plot of procid(mesh1) from multi_ucd3d.silo to verify that it was running in parallel.

I tested jsrun by first launching an xterm in a bsub job using:
rzansel61{brugger}236: /usr/tcetmp/bin/bsub -core_isolation 2 -nnodes 2 -q pdebug -W 30 -G guests xterm
Job <2504251> is submitted to queue <pdebug>.

Then I ran VisIt and created a pseudcolor plot of procid(mesh1) from multi_ucd3d.silo, which looked correct. Here is the terminal output in the xterm window.
[brugger@rzansel26 visit_jsrun_dev]$ visit/bin/visit
Running: gui3.4.2
Running: viewer3.4.2 -geometry 1558x1080+362+0 -borders 29,5,5,5 -shift 17,27 -preshift -12,2 -defer -host <IP_ADDRESS> -port 5600
Running: mdserver3.4.2 -host <IP_ADDRESS> -port 5601
Running: env LD_LIBRARY_PATH=/usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2/linux-ppc64le/lib/osmesa:/usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2/linux-ppc64le/lib/mesagl:/usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2/linux-ppc64le/lib:/usr/local/tools/ifort-11.1.046/lib /usr/tce/packages/jsrun/jsrun-2019.05.02/bin/jsrun --np 80 --nrs 2 -c ALL_CPUS -g ALL_GPUS -d plane:40 -b none -X 1 /usr/tce/packages/lrun/lrun-2019.05.07/bin/mpibind10 /usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2/linux-ppc64le/bin/engine_par -plugindir /g/g17/brugger/.visit/3.4.2/linux-ppc64le/plugins:/usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2/linux-ppc64le/plugins -visithome /usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2 -visitarchhome /usr/WS1/brugger/visit_jsrun_dev/visit/3.4.2/linux-ppc64le -dir /usr/workspace/brugger/visit_jsrun_dev/visit -forcestatic -idle-timeout 480 -noloopback -host rzansel26 -port 5600

To verify that I was using 40 cores on each of 2 nodes, I used bjobs -l to determine that it was running on nodes rzansel26 and rzansel5. I then used ssh to login to each of the nodes and checked that there were 40 engine_par processes running on each.
On rzansel26:
rzansel26{brugger}22: ps -u brugger | grep engine_par | wc -l
40

On rzansel5:
rzansel5{brugger}21: ps -u brugger | grep engine_par | wc -l
40

Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[X] I have commented my code where applicable.~~
[X] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have confirmed new and existing unit tests pass locally with my changes.
[ ] I have added new baselines for any new tests to the repo.
[X] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

Awesome!


Add deposit/withdraw fee text to the frontend

Added in https://github.com/threefoldfoundation/tft/commit/2dda3eb04c157b3249f5db9ef09529160617a6d4
Waiting on https://github.com/threefoldtech/tf_operations/issues/430


Use pytest env for pytest CI check

our pytest env is insufficient / out of date it seems

sorry this is in progress to fix #517 among other issues.
This first commit gives the expected failure; I'll push a fix -- and we'll be left with the other failures which then I'll think about how to fix :p

ok. This is now ready. I changed the pytest job to run the runtime tests with runtime dependencies and added a new, separate job to test the dev dependencies. Previously: the dev environment was used for testing so badly configured factorized dependencies could sneak by causing #517

ahh I had that commit locally but didn't push it because I didn't want to have to start the CI over :D

a [skip ci] might slide.

is this good now @fdmalone ?


Azure Document Intelligence Error: InvalidContentDimensions

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

I included 1000s of files with MD file format and images to the data/ folder. azd up kept failing for every single file which has a smaller dimension. It is ok to skip those file but currently I've to delete those files to proceed with azd up.

Any log messages given by the failure

For files with very small dimensions (un-supported), the azd up kept failing and had to fix each of the files to proceed.

Ingesting 'efficiency.png'
Extracting text from './data/efficiency.png' using Azure Document Intelligence
Traceback (most recent call last):
  File "/workspaces/azure-search-openai-demo/./app/backend/prepdocs.py", line 470, in <module>
    loop.run_until_complete(main(ingestion_strategy, setup_index=not args.remove and not args.removeall))
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 654, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/./app/backend/prepdocs.py", line 213, in main
    await strategy.run()
  File "/workspaces/azure-search-openai-demo/app/backend/prepdocslib/filestrategy.py", line 84, in run
    sections = await parse_file(file, self.file_processors, self.category, self.image_embeddings)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/app/backend/prepdocslib/filestrategy.py", line 26, in parse_file
    pages = [page async for page in processor.parser.parse(content=file.content)]
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/app/backend/prepdocslib/filestrategy.py", line 26, in <listcomp>
    pages = [page async for page in processor.parser.parse(content=file.content)]
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/app/backend/prepdocslib/pdfparser.py", line 54, in parse
    poller = await document_intelligence_client.begin_analyze_document(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/.venv/lib/python3.11/site-packages/azure/core/tracing/decorator_async.py", line 94, in wrapper_use_tracer
    return await func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/.venv/lib/python3.11/site-packages/azure/ai/documentintelligence/aio/_operations/_operations.py", line 3241, in begin_analyze_document
    raw_result = await self._analyze_document_initial(  # type: ignore
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/.venv/lib/python3.11/site-packages/azure/ai/documentintelligence/aio/_operations/_operations.py", line 132, in _analyze_document_initial
    raise HttpResponseError(response=response, model=error)
azure.core.exceptions.HttpResponseError: (InvalidRequest) Invalid request.
Code: InvalidRequest
Message: Invalid request.
Inner error: {
    "code": "InvalidContentDimensions",
    "message": "The input image dimensions are out of range. Refer to documentation for supported image dimensions."
}

ERROR: error executing step command 'provision': failed running post hooks: 'postprovision' hook failed with exit code: '1', Path: '/tmp/azd-postprovision-1671022109.sh'. : exit code: 1

Expected/desired behavior

Skip the files which has InvalidContentDimensions and provide a summary of the files skipped later.

OS and Version?

GitHub Codespaces

uname -ra
Linux codespaces-50920e 6.5.0-1021-azure #22~22.04.1-Ubuntu SMP Tue Apr 30 16:08:18 UTC 2024 x86_64 GNU/Linux

azd version?

run azd version and copy paste here.

azd version 1.9.3 (commit e1624330dcc7dde440ecc1eda06aac40e68aa0a3)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Ah, thanks for filing, I haven't done much data ingestion with images. I'm a little worried that if we skipped over images, people wouldn't notice the logs. I think that'd have to be an optional flag. You should be able to modify the code to try/catch for that particular exception for your use case.
If any other developers run into this as well, please comment so we can see how often developers are using this repo with images/small images.


CORS Middleware should set Content-Length = 0 when returning a 204 response

https://github.com/aspnet/AspNetCore/blob/master/src/Middleware/CORS/src/Infrastructure/CorsMiddleware.cs#L189
Can we please allow this to be set either in AddCors or UserCors?
I need to send a 200 because IIS is seeing the 204, it doesn't see a Content-Length header so it tries to chunk the response and that explodes our F5 load balancer because 204 with Transfer-Encoding: chunked does not conform to HTTP spec.

@VictorioBerra would it suffice if the middleware set Content-Length=0 as part of the response?
@jkotalik would the changes to use Pipelines now send a Content-Length header when it wasn't explicitly specified?

Yes that would be a good idea.

👍 that seems do-able. For 2.x, you could consider setting the Content-Length in a middleware when you see a 204 response code and the header is missing.

How can I do this before cors middleware exits and returns the response? I believe it does not call _next when it sees a preflight and sets the 204.

@VictorioBerra Hi, you can subscribe to Response.OnStarting event with action which wiil check response.
public async Task Invoke(HttpContext context)
{
    context.Response.OnStarting(() =>
    {
        context.Response.Headers["X-Trace-Id"] = context.TraceIdentifier;
        return Task.CompletedTask;
    });

    await _next(context);
}

I think this should work.

I didn't know about those response events. Thanks!!!
On Sat, Mar 16, 2019, 10:33 AM Andrey Mityagov<EMAIL_ADDRESS>wrote:

@VictorioBerra https://github.com/VictorioBerra Hi, you can subscribe
to Response.OnStarting event with action which wiil check response.
public async Task Invoke(HttpContext context)
{
context.Response.OnStarting(() =>
{
context.Response.Headers["X-Trace-Id"] = context.TraceIdentifier;
return Task.CompletedTask;
});
await _next(context);

}
I think this should work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/AspNetCore/issues/8555#issuecomment-473547476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACzG6z8_iqQe-cHpInccTp_fpR608YUpks5vXQ7bgaJpZM4b2uae
.

In case other people come across this, another option:
using ...

namespace YourApp.API.Services
{
    public class IISCorsService : CorsService
    {
        public IISCorsService(IOptions<CorsOptions> options) : base(options)
        {
        }

        public IISCorsService(IOptions<CorsOptions> options, ILoggerFactory loggerFactory) : base(options, loggerFactory)
        {
        }

        public override void EvaluatePreflightRequest(HttpContext context, CorsPolicy policy, CorsResult result)
        {
            base.EvaluatePreflightRequest(context, policy, result);

            if (result.IsPreflightRequest)
            {
                context.Response.Headers.Add("Content-Length", "0");
            }

        }

    }
}

@VictorioBerra for my own knowledge, are you seeing this behavior with 204s using ANCM/IIS In-process or Kestrel behind ANCM? We fixed a similar bug related to in-process, so I want to see if this was related.

@jkotalik We are hosting our API on IIS using <AspNetCoreHostingModel>InProcess</AspNetCoreHostingModel> and targetting netcoreapp2.2.
When did you fix this? Is there a way for me to test the fix today? We might be a few minor versions behind on our ANCM/Hosting Bundle CLR.

@VictorioBerra can you try removing that line for AspNetCoreHostingModel and try seeing if this issue reproduces without setting the content length? This issue should have been fixed in 2.2.1 in ASP.NET Core, check what version you are currently on. If you are on 2.2.1 or later and the issue doesn't reproduce when removing AspNetCoreHostingModel, we may have an issue.

Looks like we have 2.2.0 on our dev server. Updating and will test momentarily.

Looks like updating maybe have solved the problem.

Awesome, I’m going to close this issue then.


Env-dependand configuration, docker-compose.yaml

Description
This PR does:

uses EXTERNAL_URL and WEBVOWL_URL env variables to set respective properties

Checklist

[X] I followed the CONTRIBUTING guidelines.
[ ] I updated the documentation.

LGTM


fix: pass pug attribute to wrapped html in _attribute_backed.pug

Pass PUG attributes to the underlying HTML generation in _attribute_backed.pug. This use the same system as e.g. https://github.com/Kurohyou-Studios/k-scaffold/blob/61897632a8ee11a56d641a3792b1ee719fc0db37/lib/attribute_holders/_inputs.pug#L21

Merging to Dev/2.3.0


Correct syntax for parameters not described or linked to

For az cdn endpoint rule add, the following parameters exist:
--match-variable
Name of the match condition.

--operator
Operator of the match condition.

--action-name
Name of the action.

The examples given show the following format for these parameters:
--action-name CacheExpiration

--match-variable RequestScheme --operator Equal --match-values HTTPS --action-name "UrlRedirect"

The syntax of these examples does not match the actions listed here: https://docs.microsoft.com/en-us/azure/cdn/cdn-standard-rules-engine-actions
or the match conditions listed here: https://docs.microsoft.com/en-us/azure/cdn/cdn-standard-rules-engine-match-conditions
and there is no link to the correct format for whatever values these parameters expect.
I consider the documentation for the az CLI to be inadequate and the tool unusable as a result.  This is a consistent problem with many subcommands.

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

ID: fef54fc4-b4a8-d366-40de-a0ae2586961d
Version Independent ID: 1231852d-d3b4-9bcf-e9d4-ba12cffb7603
Content: az cdn endpoint rule
Content Source: latest/docs-ref-autogen/cdn/endpoint/rule.yml
Service: cdn
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

cdn

@cpdohert , could you try upgrade to version 2.35.0 or after. We've added more context in the help output.


split parse and validate into 2 phases!

I have written an api framework based on falcon, however, within it, the framework already parse the params. which makes using webargs a little bit ugly.
heres the demo for my frame work falsy
because I already parse the req into function param name, so, is there any way to use the webargs field validation directly?

webargs uses marshmallow for validation/deserialization, so I suppose you could use marshmallow directly for validation. Would that meet your use case?

yeah, because our project is based on flask, and now want to change to our own framework.
we used webargs in our project, which defines schema in json. we do not want to recode the validation part. just check the source of this project and find the correct part may help.
thx a lot.
def argmap2schema(argmap, instance=False, **kwargs):
    """Generate a `marshmallow.Schema` class given a dictionary of argument
    names to `Fields <marshmallow.fields.Field>`.
    """
    class Meta(object):
        strict = True
    attrs = dict(argmap, Meta=Meta)
    cls = type(str(''), (ma.Schema,), attrs)
    return cls if not instance else cls(**kwargs)

I'm not sure I understand the problem completely, but validation is done almost exclusively by marshmallow, so could you use marshmallow.Schema#validate to validate the request data?

yeah, thx, the problem is solved by using the argmap2schema func.
because our old project is using webargs, we define the schema with json style
{
   'name': fields.Str(validate=lambda p: len(p) >= 6, required=True),
}

however, we are moving to our own framework, if we change to marshmallow directly, we need to change to the schema style(we need to define the schema class), which force us to rewrite hundreds of dicts.
the thing behind this is that our framework parses the params directly, the controller is something like this,
def get_it(name):
    return {
        'get': name
    }

the name can be something in query, path or header etc.
however, the decorator doesn't work for this situitation, for it doesn't expose the req object.
so, if there's a decorator can be used with the function params(not the req object) directly, it will solve the problem.
currently, I just use the argmap2schema function and marshmallow directly to make things work.

yeah, thx, the problem is solved by using the argmap2schema func.
because our old project is using webargs, we define the schema with json style
{
'name': fields.Str(validate=lambda p: len(p) >= 6, required=True),
}
however, we are moving to our own framework, if we change to marshmallow directly, we need to change to the schema style(we need to define the schema class), which force us to rewrite hundreds of dicts.
the thing behind this is that our framework parses the params directly, the controller is something like this,
def get_it(name):
return {
'get': name
}
the name can be something in query, path or header etc.
however, the decorator doesn't work for this situitation, for it doesn't expose the req object.
so, if there's a decorator can be used with the function params(not the req object) directly, it will solve the problem.
currently, I just use the argmap2schema function and marshmallow directly to make things work.

You should be able to write a decorator to validate the input arguments for your controllers.
How about something along the lines of:
def validate(argmap):
    schema = argmap2schema(argmap, instance=True)
    def decorator(func):
        @functools.wraps(func)
        def decorated(*args, **kwargs):
            schema.validate(kwargs)
            return func(*args, **kwargs)
        return decorated
    return decorator

You would use it like:
from marshmallow import fields

@validate({
    'x': fields.Int(),
    'y': fields.Int(),
})
def add(x, y):
    return x + y

add(x='invalid', y=2)  # ValidationError: {'x': ['Not a valid integer.']}

wow, this is real black magic, thx a lot~

sorry for bother you again, the argmap2schema may have some problems, see the code below,
the required check is neglected, however, it has some relations with the strict attr in the meta class.
import functools

from marshmallow import fields
from webargs import argmap2schema

from marshmallow import Schema
def argmap2schema(argmap, instance=False, **kwargs):
    class Meta(object):
        strict = True
    attrs = dict(argmap, Meta=Meta)
    cls = type(str(''), (Schema,), attrs)
    return cls if not instance else cls(**kwargs)

def validate(argmap):
    schema = argmap2schema(argmap, instance=True)
    def decorator(func):
        @functools.wraps(func)
        def decorated(*args, **kwargs):
            print(kwargs)
            schema.validate(kwargs)
            return func(*args, **kwargs)
        return decorated
    return decorator

@validate({
    's': fields.Str(required=False),
})
def add(s):
    print(s)
    return s

add(s=None)


new ML-PACE plugin for LAMMPS MD simulation software

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-No-Executables


Default workspace's variable has a too high reference count

And loaded variables too...

Fixed by 9d269b2


Add template for Governance reviews

Creates a Markdown template for governance reviews.
Includes input by @aliok @TheFoxAtWork @geekygirldawn
Please review, folks!  Also @dzolotusky
After this merge, we will also need:

Documentation on the review process
Self-assessment Guide

@jberkus All that I'm seeing here is an empty file being added as governance/reviews/template.md. I assume that that's not what I should see.
Am I doing it wrong on my end, or is this PR missing the content?

Same here, looks like the committed file is empty.

sigh  Text present now.

This looks great, and super thorough.


installs Razor v1 now instead of v2, fixes for all of Xpand.DLL directory being added to the GAC

merged in <IP_ADDRESS>


Error running node helloworld.js

I am trying to run the Hello World bot. I am getting an error at the last step, node helloworld.js. I am running ngrok as, ngrok http 3000 or ngrok http 8080.
D:\[REDACTED]\cisco-spark\github\botkit-ciscospark-samples\node_mo
dules\botkit\lib\CiscoSparkbot.js:42
            throw new Error('Could not determine hostname of public address: ' +
 controller.config.public_address);
            ^

Error: Could not determine hostname of public address: "https://[REDACTED].ngrok.i
o"
    at Object.Sparkbot (D:\[REDACTED]\cisco-spark\github\botkit-ci
scospark-samples\node_modules\botkit\lib\CiscoSparkbot.js:42:19)
    at Object.<anonymous> (D:\[REDACTED]\cisco-spark\github\botkit
-ciscospark-samples\helloworld.js:32:25)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:389:7)
    at startup (bootstrap_node.js:149:9)

The Emoji bot is working. The Hello World bot and the DevNet bot are not working.

Hi @MGTHEBOSS
The issue seems to be related to the formatting of the bot public URL
Here is the code responsible for the issue
       var endpoint = url.parse(controller.config.public_address);
        if (!endpoint.hostname) {
            throw new Error('Could not determine hostname of public address: ' + controller.config.public_address);
        }

Can you paste the command line you're using to launch the bot, and your platform OS?
Note that the emojibot is implemented via WebSockets. As such it does not need ngrok nor PUBLIC_URL env variable. Confirms the issue comes from the PUBLIC_URL .

I'm on windows 7 professional cmd.

Reproduced on Win7, with DOS shell.
Must remove the quotes on the command line
=> doc updated.
Thank you @MGTHEBOSS


fix: missing properties in python when no additional properties

fixes #5037

some integration tests seem to be failing now, I'll take a look at it tomorrow


updated readme with detailed instructions

@dirtmaxim Can you please add details about the player.py and cvat2slowfast.py files?
@zhong-al Can you please provide details on Step 3? Please add any scripts you used to label the mini-scenes. thanks!

@egrace479, I'm pushing this change so Alison and Maksim can contribute to this track. Once that is done, I have a few details I still need to add, especially the time budget visualizations from #20. Thanks!

@jennamk14, thanks for getting them up on HF; I updated the links here (you can go back to the "Files changed" tab to add both suggestions to batch to commit together).
Let me know if you want me to address the merge conflicts (or help you with them) to add this into the repo.

hi @egrace479 yes i do need some help to merge these. I can see where the conflicts are, but im having trouble figuring out how to make github accept the new changes. thanks!

I believe these suggestions cover all the overwrites.

@egrace479 thanks! i committed all the suggestions. I realized i need to go back and update the file paths now that the .py scripts are in the src directory, ill track these changes here #33


Dokyoomintashun? I'm not playin' no mo'. Get Down To O'Shack Hennessy's Office...

The project was restored using Microsoft.NETCore.App version blah, but with current settings, version blah would be used instead
Really?? I am saddened that this is the best guidance you can provide.  Why do you struggle so to provide useful troubleshooting information?
The cream RISES to the top.
https://youtu.be/rWrlAmT4jyA
Oh yeah, Nothing Means NOTHING. Bad documentation means NOTHING.  ...Zero.
There is nobody who does troubleshooting better than the following. Here is the kind of info we in the development community need when we get such bizarre compilation errors: https://stackoverflow.com/questions/51642172/the-project-was-restored-using-microsoft-netcore-app-version-2-1-0-but-with-cur
We really don't need Self-contained deployment roll forward. Nope.  What's the point of this information?  Come on man.

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

ID: a901ae9f-cc78-b5de-a031-b4671c7a85ac
Version Independent ID: 89276cf0-167d-c108-f9d6-f5bfe1317851
Content: Runtime roll forward for .NET Core self-contained app deployments. - .NET Core
Content Source: docs/core/deploying/runtime-patch-selection.md
Product: dotnet-core
GitHub Login: @KathleenDollard
Microsoft Alias: mairaw

We aren't finding it in search - the error message tells us to go to http://aka.ms/dotnet-runtime-patch-selection for more information, which sends us here


copy_blob calls have started to fail in system tests (400 no file found in request)

@thobrla Any idea why these requests would start to become flaky?
Some recent examples:

https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158783443
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158777834
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158764459
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158717892
https://travis-ci.org/GoogleCloudPlatform/google-cloud-python/builds/158645485

Do you have a example of the full request / response for this error case?  I'm not aware of anything that would lead to a 400 like this.  Possibly the request had a connection drop and was only partially sent, but I would expect that to be quite rare.

@thobrla The whole response was just No file found in request. (not a JSON or HTML response, just text) with a 400 status code (I don't have any of the other headers, sorry).
The request is a POST with no body to
https://www.googleapis.com/storage/v1/b/{BUKCET}/o/{FILE1}/copyTo/b/{BUCKET}/o/{FILE2}

@thobrla It happened consistently for several days (see the Travis failures linked above). I checked some recent test runs and it doesn't seem to be triggering anymore, so that's good.

Thanks.  I wonder if there was a bug in the service during this time.  Please let me know if it resurfaces.

Will do. We may not detect it, right now we are retrying on the failure up to 4 times.
CONTEXT: We have been sprinkling lots of retries in our system tests to get them to go green. Talking to 14 Google services at once (in 2 different Python environments) essentially guarantees a failure if no retries are used.

That's understandable but 400 is not intended to be a retryable error, so you really shouldn't retry a 400 as in theory it will never succeed.

@thobrla Indeed. We added a special error predicate check to make sure it wasn't any old 400, but it was the specific 400 we're discussing here.


Browser Build

Has anyone tried or succeeded in building eslint-plugin-react to run in the browser?
Use case: linting inside a browser-based IDE https://vizhub.com/curran/c3b14112dae34ef395999cef5783324f?edit=files&file=index.js
Thanks in advance!

Linting requires filesystem access at times - in particular, eslint-plugin-import, but also this plugin detects the react and flow versions, and also needs to load the TypeScript config.
I don't think it's realistic to lint in the browser, I'm afraid.

Thanks @ljharb for your response!
FWIW I have got linting working in the browser using this package for now https://www.npmjs.com/package/eslint4b
Assuming no Flow and no Typescript, and an explicit React version passed in rather than detected from the file system, in theory it should be possible to run eslint-plugin-react in the browser. There may be some rules that have a fundamental incompatibility and strictly depend on file system access (indeed as you mention eslint-plugin-import), but I suspect many React-specific rules could work in a browser environment. I understand there are technical hurdles, but I think it is feasible, at least conceptually.
I suppose the question is, is this feature something that the team behind eslint-plugin-react would be interested in supporting? I could take a crack at a PR for a browser build.

What changes do you anticipate being needed, in that constrained scenario?

I'm imagining a different flow for the browser build that stubs out or otherwise disables the fs and dynamic require calls.
Rollup does this quite elegantly, as a reference:

https://github.com/rollup/rollup/blob/master/browser/fs.ts
https://github.com/rollup/rollup/blob/0fa9758cb7b1976537ae0875d085669e3a21e918/rollup.config.js#L142

Sure, but you can do that already in every JS bundler, including browserify, webpack, etc.


BUG - rollback snowflake fix for dbt_utils.star

Are you a current Fivetran customer?

Fivetran created PR
Describe the bug

With the patch release of dbt-utils v0.7.3 the snowflake uppercase fix no longer is needed / may cause ambiguous column errors in the following models

iterable__campaigns
iterable__events
iterable__users

Additional context

Can adapt the fix from jira https://github.com/fivetran/dbt_jira/pull/40

PR has been merged and release has been integrated; deleting this issue


Refresh Team Banner

This is going to be my focus for awhile, I want to mess around with the design a bit in an issue before I move it to a pull request.
My first draft is:

My goals were:

Supporting teams on Gratipay is AMAZING! Let's show that off
Highlight you (the user) setting up an easy gift to the team

I'd love some initial thoughts! Please be aware my "Gratipay Terminology" is way old :)
To Do:

[ ] Ugly black border!
[ ] What are tip amounts? Buttons? Anchors?
[ ] I added a semi-black transparency to the banner for the white text, good solution?
[ ] Update terminology

Side note, in order to build this out, I had to tear up the rest of the site haha

My $0.02:

Is the amount for the latest week or average?
are the numbers ($5.00, $10.00..) clickable? If so it's not quite obvious. And maybe $1 (without .00) is more concise
is "setup gift" to setup weekly giving? If it's just to give different number, maybe relabel to something like "other"?

I'm thinking when you click on any of the numbers OR the button, it slides to this menu:

@clone1018 Maybe keep that up all the time, no slide?

@clone1018 Can we close this in favor of #4215?

Yeah I meant to! Thanks


chore: replace resolveZoneFileToProfile from blockstack SDK

Description
While browsing the repo I found this function that can be extracted from the blockstack package. I will submit other pull requests if I find other parts that can be migrated easily.
Type of Change
Code change.
Does this introduce a breaking change?
No
Checklist

[ ] Code is commented where needed
[ ] Unit test coverage for new or modified code paths
[ ] npm run test passes
[ ] Changelog is updated
[ ] Tag 1 of @wileyj or @CharlieC3

Closing then re-opening PR to try and re-trigger CI.

I see that CI / build-publish (pull_request) is failing but it doesn't seems related to that pr

@pradel I'll submit a fix for that in a separate PR.

Closing then re-opening PR to try and re-trigger CI.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: fix if writer already exists, logx.Setup not working

SetUp should called first.

SetUp is no guarantee that it must be called first.
In previous versions, such as version 1.3.3, this situation did not exist.

Once the version is upgraded, it should be backward compatible as long as the api is not modified


Enable https

https://github.com/parvardegr/sharing/issues/19
By default it'll launch as http, if anyone wants to enable https instead, just add -S and path to cert files with -C and -K.
Example:
node index.js ../../demo/demo/pom.xml -S -C /d/cert/cert.pem -K /d/cert/key.pem

Will this support self-signed cert? It would be nice not to have the option not to set anything up

As I have created new PR based on express to enable the ssl, close this one.
https://github.com/parvardegr/sharing/pull/26


Replace LinkedBlockingQueue with ConcurrentLinkedQueue

Hi Team
We are noticing performance issue when using this client on a java app that emits high volume of metric updates/sec ie; in the order of > 600k (600 thousand)  metric updates/sec. We did see improvement in throughput of the app when we replaced LinkedBlockingQueue with ConcurrentLinkedQueue.
Our app consumes messages from kafka and writes to Elasticsearch , in peak it has a throughput of 70k-90k messages per host per second and it emits/updates metrics for every consumed message.
The performance issue when using the LinkedBlockingQueue is due to the fact multiple threads of the app are contending to acquire lock to the queue and are in waiting state(as per thread dump and flamegraphs). Are there any reasons to not use ConcurrentLinkedQueue. Would you accept the upstream PR to replace the queue implementation or making the queue implementation configurable.

cc @ baojr

Hi @prudhvi, thanks for bringing this issue to our attention. We are reviewing your questions and will get back to you asap.

Just a small update incase if you think ConcurrentLinkedQueue in unbounded , what do you think of using http://javadox.com/org.jctools/jctools-core/1.0/org/jctools/queues/MpscArrayQueue.html from https://github.com/JCTools/JCTools. We tried this queue and it seems to do better than Linked BlockingQueue but not as good as ConcurrentLinkedQueue which is unbounded and requires more heap. The bottleneck we ran with MpscArrayQueue is that the single consumer thread that's writing to statsd can't consume as fast as producers are writing metrics to the queue( with queuesize of 10000) , so might involve more work there to remove the overhead in consumer thread or using multiple consumer threads using MPMC queue.

Hi @prudhvi, thanks for your suggestions on the queue implementation! Just so you know, we have work planned to improve the performance of this client: the queue implementation is something we're looking into, but we're also looking into the way the client currently groups metrics in UDP datagrams, and into reducing the memory allocations done by the client.

before we dig into details of the queue implementation, could you share the options with which you initialize the client? If you could share some code snippets that reflect how the dogstatsd client is used within your app, that could help as well. (If you'd prefer not to share this publicly here, please open a support case by emailing the info to<EMAIL_ADDRESS>and mention this github issue in the email).
do you have more details on the metric types (gauge, count, distribution, etc) of the metrics that you send with the client? Depending on the metric type that your app uses, some simple aggregation in the app could significantly reduce the metric throughput.
do you use one instance of the client for the whole app, or one instance per worker thread? Having one instance of the client per worker thread could improve performance as well.

If none of these questions allows identifying low-hanging fruit to improve the client's performance, let's continue the conversation on the queue implementation and other code change we could make to improve the client's performance.

Thanks for the follow up @olivielpeau
1)Sure will send the snippets showing usage to your support email.
2)For the app that I mentioned 80% of the metrics are counters and remaining 20% are histograms.  Prior to statsd integration we were using Yammer MetricsRegistry from which we are periodically scraping metrics and sending them to kafka. But with statsd integration using this client we are intercepting every call to update a metric in MetricRegistry(eg: counter inc , dec and histogram update) and sending that update to statsd agent( veneur) via this client.  We are not periodically scraping from MetricsRegistry for statsd integration as we want to support first class tags in future(where counter inc and dec api's should support adding arbitrary tags) and don't want to be tightly coupled with MetricRegistry (which doesn't have firstclass tags support and even in upcoming version 5 it supports tags by storing every uniquely tagged metric in MetricRegistry) . Although I understand MetricRegistry provides local aggregation of counters which would reduce the number of updates. For now we are resorting to down sampling heavily to support this volume.

Currently we are using one instance  per whole app, as discussed above in 2) to support easier migration to statsd via existing code we are intercepting every call to MetricRegistry metric update and sending that update to Statsd agent via Singleton statsd client(Intercepting setup is done via Guice modules where we bind MetricRegistry to a custom implementation of our MetricRegistryWrapper which sends update to in memory MetricRegistry(Yammer singleton) as well as statsd.

@olivielpeau FYI included snippets for 1) at https://help.datadoghq.com/hc/en-us/requests/278328

Thanks Prudhvi for all these details!
With a volume of 600K points/sec on a single client, allowing for fully arbitrary tags is a challenge.
Have you identified the hot loop in the app that submits the highest volume of points? And would it be conceivable in the short-term to modify that part of the app so that it uses a slightly different interface with non-arbitrary tags (which would more easily allow for pre-aggregation, in particular for counters)? I don't know if you have access to the app's code itself, and if it's conceivable as part of the migration to modify the app's code.
Also, how many threads (roughly) does this app have?

Hey, folks! We've noticed a similar pattern in our applications where we observed these two issues:

Extensive CPU usage on EC2 spent sending UDP packets.

Depending on the application, the CPU consumption can vary from 5% to 25%. Here's an exempt from a related CPU flame-graph:

but we're also looking into the way the client currently groups metrics in UDP datagrams, and into reducing the memory allocations done by the client.

@olivielpeau, I assume that this is exactly that problem, and you're already on top of it?

CPU usage, and locking on the shared metrics queue.

I believe this is the exact issue that was discussed here initially. IMO, apart from ConcurrentLinkedQueue, a viable solution here would be to use an MPSC (Multiple-Producer Single-Consumer) queue from JCTools. This can give quick and significant benefits with minimum code changes. Or, maybe even thread-local-based constructs if we can assume that the data is completely independent.

Thanks for the followup @SerCeMan . I have tried MPSC queue with queuesize of 10k and noticed that the single consumer wasn't able to consume fast enough if we have multiple producers sending statsd messages to the queue at the rate I mentioned( > 600k metric updates per second) so we are seeing incorrect values as the queue drops the messages if it's full. So suggested if MPMC could be looked into or optimizing the consumer thread so that it can keep up with producer rate.

Hey, @prudhvi! What do you mean by "wasn't able to consume fast enough"? As far as I can see, there is only a single consume in the NonBlockingStatsDClient - c.t.s.NonBlockingStatsDClient#executor thread, that's why I suggested MPSC queue.
Are you seeing thread starvation where the consumer thread doesn't get enough quota to send the data? If yes, can it be a sign that the system is now working much faster overall? Or are you seeing something different?

Yeah it only has single consumer for now, but suggesting if it could be made multiple consumers so that the queue doesn't fill up and drop messages. We noticed that with MPSC and 10k queuesize and 5000 as maxPacketSizeBytes metrics updates from producers are getting dropped.
Not sure if it's starvation or just slowness in writing to Unix Domain Socket in the consumer thread that prevents it from keeping up to speed.(we are using Unix Domain socket not UDP)

Thanks for all the additional details @prudhvi, and thanks @SerCeMan for chiming in (and the suggestions)
@prudhvi When using the MPSC queue, were the producers at least fast enough to not noticeably slow down your application? (I'm assuming they were not anymore, but double-checking)
When using dogstatsd with Unix Domain Socket, the thread that writes to the socket is blocked if the socket is full, and on most systems the socket has a relatively small max buffer size by default. So a small burst of data (or a small timeframe during which the statsd agent doesn't read from the socket) can easily end up with the writing thread filling up the socket and then having to wait on the statsd agent to read from the socket.
Therefore it may be worth looking into the kernel parameters defining the buffer sizes of the sockets. Increasing the buffer size can help when the writing thread is blocked waiting on the statsd agent to read from the socket. Could you run the following commands to find out the values of these parameters on your hosts:

sysctl net.unix.max_dgram_qlen: defines the max number of datagrams in the buffer of unix domain sockets
sysctl net.core.wmem_default and sysctl net.core.wmem_max: defines the max buffer size (in bytes) a unix domain socket is allowed to have.

You can increase net.unix.max_dgram_qlen (with sysctl -w net.unix.max_dgram_qlen=X) roughly as long as net.unix.max_dgram_qlen * maxPacketSizeBytes remains below net.core.wmem_default.
This may help increase the throughput, let us know, in any case the info will be helpful. I'm not sure at the moment if increasing the number of consumer threads would actually improve the problem here (as they may all end up blocking on the write to the socket).

@olivielpeau when using MPSC queue it's still spending 30% CPU instead of 40% CPU (when I searched for yammer in svg) and still having overhead for the application. I will attach the flamegraphs for MPSC queue in the ticket.
Thanks aware of socket size limitations so we bumped the net.core.wmem_max to 8MB on our hosts from a low value. We didn't want to bump the default as it would effect all sockets consuming more memory irrespective of usecase and we run multiple LXC app containers on single host
our default are  net.core.wmem_default = 212992.  net.unix.max_dgram_qlen = 512
Veneur(statsd agent) starts listening on socket by setting the socket size to 2MB (less than the max 8MB configured via net.core.wmem_max) . However I tested by configuring socket size to 8MB as well and didn't see any difference in CPU usage by the consumer thread as per the flamegraphs. But yeah you are right multiple consumer threads might not help if writing to socket is blocker. However I am not sure how to check(what tools to use) if the socket buffer is full for Unix Datagram sockets and is blocking the consumer thread in writing to socket. Let me know if there is a way to verify it.

Hi All!
Once again thank you @prudhvi and @SerCeMan for your feedback. We have started a bit of a revamp to improve client performance - first focusing on throughput (though we will later focus on resource efficiency, ie. cpu/memory usage).
We have 3 PRs:

https://github.com/DataDog/java-dogstatsd-client/pull/94
https://github.com/DataDog/java-dogstatsd-client/pull/95
https://github.com/DataDog/java-dogstatsd-client/pull/96

Trying to address some of the issues described here, or at least trying to provide different options to our users who may be in need of greater performance. As we can see from @SerCeMan flamegraph we spend quite a bit of time writing to the socket, so the first quick gain was to move the socket writes to a different thread. Decoupling the datagram assembly logic from the actual submission should allow the dogstatsd application to continue to write statsd messages to the queue. The new approach with sender workers (writing to the socket) will also improve our client behavior while waiting on IO as the IO-blocked thread - now separate - should give up it's time-slice at the kernel level. The number of worker threads (ie. consumers) is configurable and we have currently not experienced any gains by increasing the number of socket-writing consumers.
To deal with the architectural change we have introduced a direct buffer pool, configurable in size, that should allow us to curb memory allocations, helping performance, in the new scenario. Currently the only scenario that would block the client packet-assembling thread would be running out of byte buffers (ie. having them all queued up for submission, with none remaining in the pool). This is an unlikely scenario which can be dealt with by dropping incoming messages.
To further improve performance, an option to use a non-blocking queue has been introduced as well. This should definitely reduce contention. The actual queue implementation we might finally use is still undecided - we are currently testing a ConcurrentLinkedQueue with a soft upper bound to curb memory usage, but we have to try MPSC, and other array-based queues to make the final decision.
I will post some numbers soon to get a sense of the gains introduced by the proposed changes. Thank you all for your feedback and ideas with respect to the client and way we can improve it.
Feel free to test it out if you wish. The PRs are stacked meaning they build up on top of each other, so https://github.com/DataDog/java-dogstatsd-client/pull/96 includes the changes in the other PRs as well. Please note that there are some breaking changes (in particular in the way the client is instantiated - better documentation should be added to the PR; sorry about that), so please keep that in mind if you decide to test these with your applications.

Thanks a lot for these improvements @truthbk , we will try these changes. It would be nice to merge in  the benchmark tests too.

cc @AndrooHan

@prudhvi @AndrooHan we have release v2.10.0 with a large set of improvements, and a bunch of new knobs we can use to improve performance. We have also been able to reduce the latency considerably for metric submission. I wanted to let you guys know in case you wanted to give it a shot. It should help. Also, if you encounter any issues please let us know and we will make sure we address them ASAP.
I will leave the issue open a few more days, couple of weeks, and if there's no feedback I will likely go ahead and close it.
Regardless, please feel free to reach out if you need anything.

Sorry to join late the discussion, but if you need a fast unbounded queue on JCTools that work better under contention, if it turns to be the issue with the other MPSC (including JCTools ones), we've added  https://github.com/JCTools/JCTools/blob/master/jctools-core/src/main/java/org/jctools/queues/MpscUnboundedXaddArrayQueue.java (that has a MPMC variant as well). Feel free to try and report any question :)


add pyupgrade pre-commit hook

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@11842f3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage          ?   98.19%
=========================================
  Files             ?        2
  Lines             ?      610
  Branches          ?        0
=========================================
  Hits              ?      599
  Misses            ?       11
  Partials          ?        0

Continue to review full report at Codecov.

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


ARROW-12944: [C++] String capitalize kernel

This PR adds scalar compute functions for string capitalization, namely "ascii_capitalize" and "utf8_capitalize".

Rebased, will merge if green.

@pitrou I pushed a change after you rebased, so I will fix conflicts and re-push.

Hmm, another commit was lost then...

Recommend starting from https://github.com/apache/arrow/commit/9d7f298bf1515c0014b2cd138c475e7bc1deda87 and then applying the buffer overflow fix.

Some tests invoke the incorrect kernel (ASCII test uses utf8_capitalize and viceversa). These "typos" are fixed in https://github.com/apache/arrow/pull/10869


ARROW-16570: [R] Make pkg-config commands find all of the libs

See discussion at https://lists.apache.org/thread/7brfhv6mzc392jqo04o1jp5vos7g7lnj
We don't currently have any CI that triggers the case @rvernica reported there.

It seems that nobody can't confirm this before we merge.
I'll merge this.


factor importance and information scores

Thanks for the amazing package!
I successfully managed to run spectra on a few datasets and I was eager to calculate the importance and information scores for the factors that spectra found. However, while reading the utils functions it was not immediately clear how to use these functions to calculate the scores for each factor:

There is no mention of an importance score function, am I right to assume that this is calculated with the holdout_loss() function? For this function I am unsure what cell_type and labels arguments should contain. The lines below suggest that there should be a loop over each unique cell_type, where labels is an array of the the cell_type annotations for each cell. Am I correct that this loop missing from the current code?
https://github.com/dpeerlab/spectra/blob/474a3c708a1fe7336f9368839c8e7058ace12f9d/src/Spectra/Spectra_util.py#L371 https://github.com/dpeerlab/spectra/blob/474a3c708a1fe7336f9368839c8e7058ace12f9d/src/Spectra/Spectra_util.py#L382
The get_information_score returns and empty list because of the commented out code. In here the labels parameter is missing as mentioned in the #todo https://github.com/dpeerlab/spectra/blob/474a3c708a1fe7336f9368839c8e7058ace12f9d/src/Spectra/Spectra_util.py#L428 Islabels the supposed to contain the same array of cell_type annotations as in holdout_loss()?

In https://github.com/dpeerlab/spectra/issues/24#issuecomment-1600431648 it is mentioned that an example would be added to the tutorial, but I have not found it there. Ideally I would like to make a figure similar to "Extended Data Fig. 6a".
Any help would be greatly appreciated.

Hello,
I'm also trying to apply Spectra to my data at the moment and was having the same question and wondering how to derive the importance scores.
@bpkwee did you already find a solution for this.

@russellkune for using Spectra which seems to be a very cool method it would be very helpful for me if you and/or one of the other developers could maybe provide some insights on how to calculate the factor importance and information scores based on the model output.
Thanks a lot!


Cannot make tuple flow to array

I'd like to pass a tuple to a function that takes an array and uses Array.find(), but I'm not sure how to do that without having a tuple vs. array type mismatch. To my knowledge there's no way to do Tuple<*>, so I'm at a loss here; I'd rather not have to use any.
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVAXAngBwKZgByArgLYBGeATmALxgCMYAPmAEwtgDMA3JrgRIVqAZwAqxHDAL0A2kTKUqAGgXCVapWAC6fVAGM4AOxEYwUAJZGAJlzpgAFKjBgji0QC4wAQSpUAhlgAPEJKAHzKqACUXqHUnABucBbWdGGu7lQiAHSWNg5u6mkZRbRl3FF6hiZmhUoisZniktL2sgyqbKpcuuh5tgVNlUA

You can't pass tuple instead of Array, because arrays are mutable, which means that you can do e.g. array.pop(), which might invalidate the tuple. If you switch Array to $ReadOnlyArray, it'd work.

that's great, thanks!


Please close all Serial sessions on Abnormal Program Termination? Spyder 4.x

When my program terminates abnormally  (frequently... MY problem not Python's nor Spyder's) all the serial connections the program opened at run start... stay open.  On re-run... I can not re-establish serial sessions as the port was not previously released.
Is there a switch / control / feature in Spyder where all serial sessions are CLOSED on any program termination?
CTRL + . in the Console restarts the Kernel... but is slow.
(LiClipse shut down all Serial connections on any termination and I didn't realize how needed this feature was... until it was gone.)
Cheers! Doug

Hey @dougshade, what do you mean exactly by serial connections?

Early in the program....
import serial
ser = serial.Serial(
port='COM20',
baudrate=115200,
timeout=1,
parity=serial.PARITY_NONE,
stopbits=serial.STOPBITS_ONE,
bytesize=serial.EIGHTBITS
)
in the middle of the program.... some of my (errant) code causes abnormal termination of program which prevents the "clean up" portion of my code from running: specifically:
ser.close()
So... the Kernel still has an open serial COM port.
Re-execution of the program fails now... because the COM port has not been released.
Restarting the kernel is a slow work around.
typing ser.close() in the iPython console works only sometimes (weird)
LiClipse IDE apparently closed all open serial connections at program termination... regardless of how it terminated.

(just 6 hours or so of stick time with Spyder... so probably overlooking some feature)

I have a similar problem but I don't think we can do anything.
What happens with me:

I open a kernel, open a serial connection
The program crashes, so can't close the serial connection properly.

At that point it is up to the OS to notice that the program that opened the serial connection is dead. Unfortunately Windows doesn't appear to do that. What I do is:

Close the kernel, open a new one, connect, it fails
Close the kernel, open a new one, connect, it succeeds

I guess the OS is resetting something during the first try.

What do you mean exactly by "Abnormal Program Termination"? Is it just an exception but the kernel is still running? (The "In[]" prompt didn't reset)
In that case I am not sure the closing should happen at the spyder level (not everyone would want to have their serial connection randomly closed.)
I would advise that you add to your code:
try:
    # Here your code that might crash
except Exception:
    ser.close()

I agree with @impact27, this is not something we can solve here, sorry.


Feature/dispose controls

Updates in short:

Add key release event
Add ability to dispose OrbitControls
Add first person and fly controls

Care to also include a simple example using the new controls?

I've dropped the keyaction enum and property.

Merging this. Example for FirstPersonControls can come later.


feat: add fink localization editor

Added Fink Localization Editor
Fink is a lightweight localization editor that works with svelte-i18next projects.

-> https://fink.inlang.com/github.com/NilsJacobsen/obsidian-projects
What I added

settings.json with config for editor
repo id file was generated by the editor SDK

-> Everything is opt-in

There is also this translation status badge. That helps indicating translation problems.

Yeah that looks great! Please don't my following changes.
Some questions:

How can I maintain the id?
Is there a way to add a new translation via the Fink editor, or should this be created directly in the repo?

Hey @Acylation,
The badge needs some seconds for the initial load, after that it is cached. The timeout of the markdown editor is too short for generating the reports which is the reason why it is not displayed.

How can I maintain the id?

The id doesn't need to be maintained. You do not need to think about the project.inlang dir after setup. Everything can be done in the editor.

Is there a way to add a new translation via the Fink editor, or should this be created directly in the repo?

I recommend using inlang's ide extension for message extraction. You could also add the extension in the .vscode dir. This way all contributor can use it.
{
  "recommendations": [
    "inlang.vs-code-extension"
  ]
}


'new_rebus_ONEPLUS

Associated Issue: #
Summary of Changes
New Redbus is added

Pull Request Test Coverage Report for Build 716

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

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


FIPS conditional test for entropy sources.

Add the FIPS related continuous random number generator (CRNG) testing.
Refer to FIPS 140-2 section 4.9.2 Conditional Tests.
The check is fairly simplistic, being for the entropy sources to not feed
the DRBG the same block of seed material twice in a row.  Only the first
DRBG in a chain is subject to this check, latter DRBGs are assumed to be
safely seeded via the earlier DRBGs.
This is an alternative for ~#8185~

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

I've no idea what is failing in the Travis builds.  It's an independent test program.

Travis failures are relevant (leaked memory).

Fix is in for the oops and the leaking memory.

Merged, thanks all.


OnRemoveTab event didn't sent the tab object

Hi,
When implementing this code:
<XTabs OnTabRemoved="ClosedTabCard" CloseTabs="true" CloseAllTabs="true" >
    @foreach (var item in _cardItems ?? new List<CatalogCardItem>())
    {
        <XTab Title="@item.TabShowTitle()" InactiveRender="true" >
            <CatalogCard CardItem="@item"></CatalogCard>
        </XTab>
    }
</XTabs>

An then the code behind:
 void ClosedTabCard(XTab xTab)
  {
      var cardItem = _cardItems.FirstOrDefault(x => x.TabShowTitle() == xTab.Title);
      _cardItems.Remove(cardItem);
  }

I was hoping that the current closed tab was sent by the OnTabRemove but came null.
Checking your code, you forget to pass the object "tab" 😅
_tabContent.Remove(tab);
    if (OnTabRemoved.HasDelegate)
        await OnTabRemoved.InvokeAsync();

Oops, going to hotfix. :)


Add support for version parameter to winget task

This PR adds the "version" parameter for use in the winget task, allowing users to specify a version of the package to install.
This was tested by locally running the main.ps1 script for the winget task with the following scenarios:

Passing -package Microsoft.DotNet.Framework.DeveloperPack_4 -version 4.6.2 -- successfully installed .NET Framework 4.6.2
Passing -package Microsoft.DotNet.Framework.DeveloperPack_4 -- successfully installed .NET Framework 4.8.1 (the default, since no version was provided)

Also validated this by: utilize test DevCenter => add my branch (with this change) to the catalog => spun up a DevBox w/ customization file (with version specified) => checked .NET Framework version on the box


golang: support patch versions in go.mod

# go.mod
go 1.21.4

then
cd .
o version
bash: command not found: go

works fine if .go-version is added

this was intentionally left out to avoid issues such as https://github.com/asdf-community/asdf-golang/issues/99

the issue is about 1.20 not saying what the patch version is, but the new go 1.21.4 supplies the full version
mise supplies 1.21 / 1 when installing 1.21.4 so afaik it would make sense to support even go 1.21 syntax
if you disagree then what about making this opt-in ?

mise doesn't read go.mod at all so I'm not really clear on what you're talking about patch versions here

the linked issue https://github.com/asdf-community/asdf-golang/issues/99 was about not supporting go.mod because it does not have a fully defined version (i.e. go 1.21 and not go 1.21.4)
I'm proposing to read that file as fallback to .go-version and use it's version if it's a full version go 1.21.4 for example

the issue is that the field isn't "deterministic" which it arguably could be in mise because of fuzzy matching but we would have to answer the question of what the determinism should be. My reading of the go docs is that this field is for defining the minimum supported version, not the minor version of the toolchain. If all the field defines is "latest" there isn't any value in supporting it.
This seems similar to the engines field in node where you should only bump it when you want to remove support for particular versions not for defining which version of node to actually use. We don't use this field for node for the same reason.
I suspect this is the reason the go designers added toolchain as a separate field. One is for the expected version and the other is for defining compatibility.
I don't think we should be using go.mod, it sounds like the asdf-golang maintainers don't either and I suspect the go designers also feel this way. I don't even think this should be the option since to me it's promoting an actively harmful pattern. If it were me I would just add a .go-version file but if you really want this you could just use asdf-golang which has that capability.

I mostly want it so I don't have to add .go-version to every go repo
since I usually match that to the go.mod / dockerfile / ci setup ... since that "minimum version" is not really something reliable :)
an alternative could be to pick a version that is >= than what the go.mod says, not really what I need but could be a nice usability feature
atm it's kinda rough because you cd into a go project and there is no go available

Why not just set go@latest globally?

hehe true that would work too :)

newer go versions tend to break older go code, so latest is not a great option
I saw goenv has GOENV_GOMOD_VERSION_ENABLE for this, would that be ok ?
(as in: MIISE_GOMOD_VERSION_ENABLE=true -> read the version from go.mod)


Null is not an object (evaluating RNNetworkinfo.getIPAddress)

Hi,
I have a problem with use that package.
Linked without problems.
Dependencies:
"react-native": "0.58.3",
"react": "16.6.3",
App created with react-native init.

what was the solution?  I have the same issue

I too am interested in the solution? Why do you keep closing issues without providing a solution? I get that you've helped us immensely by making this plugin in the first place, but that's pretty frustrating.

@inf3rnus - it's not me that's closing the issue.  You can see in the thread above that the user that created the issue closed it, then reopened it, then closed it again.
If you are having the same issue, please feel free to open a new issue and provide a little more info than this user did.  What OS, Android/iOS, how are you linking the library, example of your code, etc.

I found the solution last day... We need to use the callback version instead await... If you look at *.java file, there are the old version methods with callback as required param.
The readme file is actually wrong, I don't know why but Android version is out of date.
Hope it works!

ah, yes, thanks @shuffledex -- the readme is what is currently in master and that version uses async/await.  I need to release a 5.0 version that will switch from using callbacks.  I'll get the readme updated so it shows how to use both the < 5 version and >=5 version.
Thank you!


[bug]: Confusing error message

Describe the bug
I did this

Which gave this error message:

But I didn't use DialogTitle, so I spent a long time figuring out what's going on. It should say:

Error: SheetTitle must be used within Dialog or Sheet

Affected component/components
Sheet
How to reproduce

Use <SheetTitle> without a <Sheet> parent

Codesandbox/StackBlitz link
it's pretty easy to trigger this bug
Logs
No response
System Info
none, it's straightforward.

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

Sheet extends the Dialog component to display content that complements the main content of the screen. (source)

The error message should technically say Sheet and SheetTitle, but either way you can't arbitrarily use SheetTitle without a Sheet.


Traffic Dump: Allow unlimited disk utilization

The traffic_dump.so plugin has a --limit argument that specifies how much disk space the plugin will use before stopping writing out new sessions to replay files. This patch allows the user to configure unrestricted disk utilization by passing a value of 0 to --limit.

A couple of suggestions:

We could make it such that --limit is optional, and not specifying --limit means unlimited.
I think the implementation would be simpler if the "unlimited" value was INT64_MAX.

A couple of suggestions:

We could make it such that --limit is optional, and not specifying --limit means unlimited.
I think the implementation would be simpler if the "unlimited" value was INT64_MAX.

Yes, good points. I'll just make the --limit optional. Thanks @moonchen .


Load from module searches that require user interaction

For example, Birmingham uses session variables to store search parameters https://program-and-modules-handbook.bham.ac.uk/webhandbooks/WebHandbooks-control-servlet?Action=getModuleSearchList&pgDspAllOpts=Y
Could either carry out synthetic browser actions after loading the page, or save an HTML version of the generated index page

Information on calling browser actions at https://www.scaler.com/topics/selenium-tutorial/form-input-in-selenium/


[SQOOP-3002] sqoop merge tool composite merge-key

JIRA Issue:https://issues.apache.org/jira/browse/SQOOP-3002
Sqoop Merge Tool can only specify one column by using --merge-key argument.
When i need to specify two or more column, Sqoop-1.4.6 will throw an Exception.

Hi @kevin00chen ,
Sqoop project currently does not accept pull requests. To contribute, you will need to generate patch and upload it to the JIRA (of course if there's no corresponding JIRA issue, you will also need to create one first).
See the instructions here: https://cwiki.apache.org/confluence/display/SQOOP/How+to+Contribute
If you need more directions, I'd be happy to help!


Create face model from 5 obj's

Hi Patrikhuber,
i want to create my own base face model with 5 sample obj 3D model, i am not familiar with MatLap software, do you have any guideline or tutorial to create base face model?
( or )
please describe with simple steps

Hi,
I'm not really sure what you want to do... I suggest you read through #26 and #23.

Thanks, i need some MatLab tutorial for generate PCA, EV from OBJ model

Sounds like a good idea. You can also find more information in the issues I linked and in our VISAPP paper. The most important things is the dense correspondence (registration).
I'll close this, feel free to reopen later.

@samsgates What is EV?

@tiancaipipi110 It's a commonly/sometimes used abbreviation for eigenvectors or eigenvalues (which are two separate things, so the abbreviation is a bit ambiguous). I suggest you do an introductory tutorial to PCA to get familiar with these terms.


change directory structure

1.change the directory structure to support StratifyLabs.UI Module in file test.qml
2.fixed some bug

treat StratifyLabs.UI as an Identified Module


Fix: Handle duplicate containers across accounts

The C# version of this function throws an exception if it finds duplicate containers; in the Python code the last one wins. Update the C# code to match the Python code.
Closes #2580.

Codecov Report

Merging #2596 (a8b1500) into main (e6dd1ac) will decrease coverage by 4.87%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2596      +/-   ##
==========================================
- Coverage   29.66%   24.78%   -4.88%
==========================================
  Files         282      120     -162
  Lines       34202    12481   -21721
==========================================
- Hits        10145     3094    -7051
+ Misses      24057     9387   -14670

Impacted Files
Coverage Δ

src/ApiService/ApiService/onefuzzlib/Containers.cs
67.17% <100.00%> (-0.50%)
:arrow_down:

src/agent/onefuzz/src/expand.rs

src/agent/srcview/src/bin/srcview.rs

src/agent/onefuzz-agent/src/setup/double.rs

...gent/onefuzz-task/src/tasks/report/crash_report.rs

src/agent/debugger/src/debugger.rs

src/agent/onefuzz-agent/src/setup.rs

src/agent/win-util/src/com.rs

src/agent/win-util/src/jobs.rs

src/agent/debugger/src/dbghelp.rs

... and 153 more

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


Dockerize

Tested Dockerfile and touched up a couple things so it would compile. Although I still have not been able to run the model provided, it seems as though Docker + dependencies are working correctly.
I also added a setup.py and version.py file (modeled after pybids)

Oh,  I removed the entrypoint command because that directory was not found. I assume that you haven't made into an executable yet and we can add that back later.

Yeah, sorry, failed to add setup.py to git. I've updated master and added a CircleCI build to make sure that the Docker container builds from a clean environment.
Could you merge master and reduce this to the necessary changes?


[Improvement][dashboard] Shuffle server page list sorting function.

What changes were proposed in this pull request?
On the Shuffle Server list page, you can sort the Shuffle Server by column to facilitate you to view information about the Shuffle Server.

Why are the changes needed?
Fix: #1867
Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
Page view.

@xianjingfeng @jerqi @rickyma PTAL

Great feature! I hope some of the features in https://github.com/apache/incubator-uniffle/issues/1401 are already on the roadmap.
cc @maobaolong

This should be a feature instead of an improvement.

I think most of codes in the node list pages are the same. I think we should create a component for node list.  WDYT? @yl09099

I think most of codes in the node list pages are the same. I think we should create a component for node list. WDYT? @yl09099

That's the right thing to do, but my research.

I think most of codes in the node list pages are the same. I think we should create a component for node list. WDYT? @yl09099

It's the right thing to do, but I have to study it.

I think most of codes in the node list pages are the same. I think we should create a component for node list. WDYT? @yl09099

This one's ready. I hope I'm taking a look at it.

Merged to master.
Thanks for your contribution @yl09099
Thanks for your review @xianjingfeng


Standardize on wait time for test expectations

@ehmadzubair please review. Hopefully this will prevent screenshot generation failures from slow machines.

:+1:


MRG: [MINOR, BUG] Extent bug in evoked_image

A tiny, subtle bug in evoked.plot_image I introduced: if the extent is given as floats, the ticklabels can be wrong.

Fixing another weird bug that mean the order of the evoked image would depend on if exclude was an empty list or anything else.
Also arguably makes the code more pythonic.

This isn't easy to test because it concerns a function which only returns a figure. In principle, we could test for it by accessing the image or something like that, if you guys think it's necessary.

No we don't usually do that, and instead rely on visual confirmation

I can't force a full Circle build from the web interface, right?

"Rebuild without Cache"?

If you want to build all examples in a PR you need to add [circle full] to a commit message (and it will only do it for that commit, so in subsequent commits add the same thing if you want it to keep happening)

@jona-sassenhagen done? if so merge when green

I'm waiting for the circle build, will check the examples .. then I'll merge.

Merged, thanks guys

BTW did you mean to push this feature branch to upstream? Usually we try not to do this.

By that I mean this PR was:
	jona-sassenhagen merged 5 commits into master from fix_image_plot

When it should have been
	jona-sassenhagen merged 5 commits into master from jona-sassenhagen:fix_image_plot

Accident. I started it via the web interface and it sent it to upstream. Was the first time I used the web interface since I got upstream rights, and I didn't know it'd do this.

Ahh okay. I just looked and it doesn't look like there is a way for people with push rights to  to do it directly from the mne-tools repo. You could, though, make the changes in your fork, open a PR to your master, close it, and then manually open one to mne-tools/master and it would work. But that's a bit of a pain.

I think I'll just stick to creating branches locally :)


[feat] Support merge command in databend-sqlalchemy

We plan to support the merge into command in sqlalchemy like:

Ha, no way. I didn't see this yesterday, but we needed the statement. It's not quite the same as the snowflake implementation, but was based off it. #32

Fixed by https://github.com/datafuselabs/databend-sqlalchemy/pull/32


Update README.md

Prerequisites for KnowledgeBases

That's nice!. Thanks. Knowledge bases don't necessarily need nomic-embed-text, as there should be a bunch of other options. But it's still a good start point for most of the users. Getting it documented is helpful.


Python: Having Issues setting up a planner when working with a PromptFlow + Semantic Kernel Setup

my main file has something like this
async def main():
    kernel = sk.Kernel(log=sk.NullLogger())

    kernel.add_chat_service(
            "chat_completion",
            OpenAIChatCompletion(
                api_key="xxxxx",
                ai_model_id="gpt-4"

            ),
        )

    planner = SequentialPlanner(kernel=kernel)

    # Import the native functions
    llama_rag_plugin = kernel.import_skill(LlamaIndexRAGPlugin(), "LlamaIndexRAGPlugin")
    question = "what do i do if i want a refund?"
    ask = "Here is a question: " + question

    print(f"ask: {ask}")

    planner = SequentialPlanner(kernel=kernel)

    print(f"ask: {ask}")
    print(kernel.__dict__)

    plan = await planner.create_plan_async(ask)

    print("created plan")
    print(plan)
    print(plan.__dict__)

and then my plugin code looks like so
class LlamaIndexRAGPlugin:

    @sk_function(
        description="Asks the LlamaIndex RAG to give back a generative answer in a chat like interface. Use for answering questions",
        name="llama_rag_func",
        input_description="User question query string",
    )
    def llama_rag_func(self, question: str) -> str:

        # initialize client
        db = chromadb.PersistentClient(path="/Users/pedrogarzon/Documents/UpworkCode/DeepEvalCustomerSupport/chroma_db")

        # get collection
        chroma_collection = db.get_or_create_collection("rag_colection")

        # assign chroma as the vector_store to the context
        vector_store = ChromaVectorStore(chroma_collection=chroma_collection)
        storage_context = StorageContext.from_defaults(vector_store=vector_store)

        # load your index from stored vectors
        index = VectorStoreIndex.from_vector_store(
            vector_store, storage_context=storage_context
        )

        # create a query engine
        query_engine = index.as_query_engine()
        response = query_engine.query(question)

        print("found response in LlamaIndexRAG")
        print(response.__dict__)

        return response.response

My problem is that when I create the Planer it always come out empty like {} and my actual Plugin code never gets executed. Am I missing a step?
I have been able to successfuly call my plugin manually, but not with using a Plan as desired
result = await kernel.run_async(
llama_rag_plugin["llama_rag_func"],
input_str="What is the best way to get a refund?", )

As an update, I tried going into the Semantic Kernel source code and it looks like in the sequential_planner_parser.py file around line 80 the plan is not able to extract the function.
The code in that file looks like this
if plugin_function is not None:
                            plan_step = Plan.from_function(plugin_function)

                            print(f"plugin func: {plugin_function.__dict__}")
                            print(f"plan steps: {plan_step}")
                            print("plan step details")
                            print(plan_step.__dict__)

And that code prints
plugin func: {'_description': 'Asks the LlamaIndex RAG to give back a generative answer in a chat like interface. Use for answering questions', '_plugin_name': 'LlamaIndexRAGPlugin', '_name': 'llama_rag_func', '_is_semantic': False, '_stream_function': <bound method LlamaIndexRAGPlugin.llama_rag_func of <Plugins.RAGPlugins.LlamaIndexRAG.LlamaIndexRAGPlugin object at 0x28609bf10>>}
plan steps:
plan step details
{}

@juliomenendez, @moonbox3 said that you may have already fixed this; is that true?

@matthewbolanos I worked on a SK + PromptFlow issue 2 weeks ago, let me take a look at this one later today.


Test and increase SPI bitrate for the flash

Description
According to the datasheets, the maximum bitrates for the SPI interfaces are:

STM32F411RE: flash SPI = SPI1: 50 Mbit/s
W25Q01JVZEIQ (flash chip): 133 MHz

The Initialize() function takes any baud rate, but the HW is limited by the clock and prescaler. To check the baud rate that is actually configured, use spi.status(RODOS::SPI_STATUS_BAUDRATE).

The chosen and tested SPI bitrate for the flash is 48 MHz.


Ignore the origin [0,0] inView.

Fix issue #3895, by ignore the origin [0,0] inView.
[0,0] not inView could avoid keyboard trigger slide change in hidden swiper.
When hidden swiper , the swiperCoord was:
0: (2) [0, 0]
1: (2) [NaN, 0]
2: (2) [0, NaN]
3: (2) [NaN, NaN]

Thanks! Merged

I’m reluctant to submit a PR without doing more testing, but for the project I’m working on currently this change isn’t preventing slides inside an inactive tab (display: none) from changing when arrow keys are pressed.
If I swap continue for break it all works as anticipated.
Before:
if (point[0] === 0 && point[1] === 0) continue; // eslint-disable-line

After:
if (point[0] === 0 && point[1] === 0) break; // eslint-disable-line

The swiperCoord I’m seeing for the hidden element looks like:
0: (2) [0, 0]
1: (2) [1051, 0]
2: (2) [0, 24]
3: (2) [1051, 24]


Add Mutator API to experimental sub export

Adds experimental sub export for exposing unstable APIs.

This PR exposes an unstable Mutators API to iterate on.
Also added a warn when importing from experimental sub-path to let consumers know APIs there are unstable.

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler

Show changes

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


Showing a lot of underlines in Alacritty terminal

In my C++ code opened in Alacritty (Mac OS X) starting from some point in code, whole text is underline. Can you please help? There are no errors at those lines.

Oops, it turned out Treesitter was givinig error at those lines, adding highlight link TSError Normal to configs solved the issue.


test_nistbeaconvalue.py needs some re-working

test_nistbeaconvalue.py is a massive unit test file with a TON of static data. It should be re-worked (without impacting testing or coverage) for easier management in the future.
This was previously considered in #15 to be pushed back until after version 1.0, but instead this will be completed before the first full public release.

I've started to break tests and test data out into smaller files. This began with commit 0bd857321bca631501a5765d51ecc7212d12e883

Some great headway has actually been made on this item.

Tests from the top level test_nistbeaconvalue.py have been split up into sub-files. Specifically:

pseudo_random
json
xml

Sample data has been migrated to the nist_records.py file if appropriate. Otherwise, one-off test data was pushed into the test itself.

Work still needs to be done on this item, but this is a great start.


Add function component names

First of all, thanks for GitHub Blocks, really cool! I still need to internalize the idea behind the concept to think about all of the possibilities that they enable (and also try to find out when they will be available for everyone as a part of github.com), but I sense that they are powerful and will enable new experiences in viewing and interacting with files which have been (mostly) static up until now.
Just a small PR: For better debuggability and to align with idiomatic React, add names to the function components.
cc @Wattenberger @mattrothenberg since you have been most active in these files

great call, thanks for the PR! Would love to see you around the Discord, if you aren't already https://discord.gg/githubnext


[Feature] webpack config being able to process either sass or scss

Submit a feature request or bug report

[x] I've read the guidelines for Contributing to Roots Projects
[x] This is a feature request
[ ] This is a bug report
[x] This request isn't a duplicate of an existing issue
[x] I've read the docs and NPM Debugging Guidelines post and followed them (if applicable)
[x] This is not a personal support request that should be posted on the Roots Discourse forums

Feature Request
Feature request: Use whatever extension is setup in config.json
Nowadays you need to change line 77 on webpack.config.js in order to change from scss to sass.
No matter if you specify in config.json sass as the extension of the main style file.
It would be great if you can use it whatever you want and only set it up only on config.json
It's also necessary to edit line 177  for sass lint messages
      syntax: 'sass',

+1 on this.
Using several Laravel Framework modules already. Maybe use Jeffrey Way's Laravel Mix which provides an API for defining basic webpack build steps. You can even mix two or three style types together if you wish (LESS, SCSS, Stylus).
This repository works well with stand-alone projects too, not just Laravel projects. I am using it for a custom WordPress theme that I am working on, and it is fantastic.

#2172


vcpkg example

Added a duplicate of the example app using vcpkg/cmake for #191
Also fixed a few markdown things in the readme my IDE was complaining about

Is it possible to update the cmake and change file hierarchy to not have duplicate files?

I could, but I believe that would impact the clarity of the example. As it stands the vcpkg-example folder can be taken independently of everything else in the repo and build a working executable which is why I chose to duplicate the file from the other example directory. Ultimately your choice though, so let me know and I'll update either way

Sorry for the extra work but I prefer to not duplicate.

I tried but there is a conflict with the copy of imgui files in the example folder

Merged! Thank you Jonathan!


SSL handshake issue

We are facing a issue with netty 4.0.33 in https ssl handshake issue where server receives a CLIENT HELLO responds with ACK but never sends SERVER hello back . Once this happens then the only option is to kill the service and start again and it continue to work fine for few hours .
When the problem comes the wireshark file of that time is attached which shows the problem . Pls let me know as to how to go about debugging this issue and resolve
dump_desk.zip

Can you share a reproducer?

Am 13.01.2016 um 08:04 schrieb abhayavis<EMAIL_ADDRESS>We are facing a issue with netty 4033 in https ssl handshake issue where server receives a CLIENT HELLO responds with ACK but never sends SERVER hello back Once this happens then the only option is to kill the service and start again and it continue to work fine for few hours
When the problem comes the wireshark file of that time is attached which shows the problem Pls let me know as to how to go about debugging this issue and resolve
dump_deskzip
—
Reply to this email directly or view it on GitHub.

We have not been able to simulate it , it comes on the production server and how it is triggered is unknown but the problem comes almost everyday .

When it happens can you also get a thread-dump ?

On 13 Jan 2016, at 12:50, abhayavis<EMAIL_ADDRESS>wrote:
We have not been able to simulate it , it comes on the production server and how it is triggered is unknown but the problem comes almost everyday .
—
Reply to this email directly or view it on GitHub https://github.com/netty/netty/issues/4698#issuecomment-171264419.

Yes i have that from visual VM and it uses three different engine for normal socket , http and https . We had switched over to native epoll and the tump is with native epoll.
ThreadDumpHang.txt

I tried to change from JDK to openssl with netty-tcnative and still have the problem . So took a tcpdump of the server and found that the server (<IP_ADDRESS>) is busy sending sever hello and FIN,ACK to a client <IP_ADDRESS> and on receiving RST from client goes on to send the same server hello and FIN,ACK again to the same client on different ports which is weird .
dump.pcap.zip

There is some analysis that is done . I am attaching a pcap file and starting from packet number 3983 looks like the client ip <IP_ADDRESS> is trying to tell the server <IP_ADDRESS> about the window full to which netty server probably not understanding the packet sends "Ignore unknown record" . Once the client recovers and is ready to receive it sends a "TCP Wondow Update" and netty still do not understand that and sends "Ignore Unknown Record" and is de-stabilized after that and as can be seen does not responds to client hello message with server hello message .
file627.zip

@abhayavis sorry but without a way to reproduce I'm not able to help much atm :/

Closing this for now... if there is still a problem please reopen with more details


protocols: vpnc: Add vpnc as a network protocol

Add VPNC as a network protocol in LuCI (UCI already exists).  Note that this pull request assumes the validator pull request has already been applied.

Merged, thanks!


Add Julia tests to GitHub actions

Closes #48
Closes #46
I don't think a third-party can add new GitHub action scripts and have them run, so @cvanaret probably needs to cherry-pick my commit into a branch on the cvanaret/Uno remote. I guess I have permission because I previously committed.
But we first need a way of compiling uno_ampl in CI.

Now we can test if things like #45 are actually fixed

I've marked this as closing #46 as well, since it isn't failing anymore. I don't know which commit was responsible.

I've marked this as closing #46 as well, since it isn't failing anymore. I don't know which commit was responsible.

Interesting. I've temporarily disabled function scaling, which seems to have solved the issue!

@odow thanks for the PR!


Inline redefs crash during eval

(ns scratch.repro)

(def a 1)

(def a (inc a))

crashes with class clojure.lang.Var$Unbound cannot be cast to class java.lang.Number. The first def is not evaluated at all since the second definition overrides the first in the analysis-info map at key scratch.repro/a.

Similar failure with the code:
(def a 1)

(inc a)

(def a :boom)


Add ADRs: service merge and category hierarchy

Todays ADRs!

@lesser1 I'll give you until the next daily to comment, if there's no further comment, I'll merge the PR.


Async load page specific styles and scripts?

I play with intercooler and async page load. Works fine with simple commands / element attributes! :)
Ajax calls work fine. I haven't tested dependencies, but that sounds great to!
But how could I async load page specific stylesheets and scripts? Is there a intercoolerjs / simple way to do it?
For example first page need a accordion with additional styles, but second page is a gallery with a slider.
During search a solution a found jQuery getScript and loadCss, but maybe there is a better way. So maybe you have suggestions how I could implement this feature a better way?

You can include

include should help me with another problem, thanks! :)
To reduce the ajax calls and server side logic I would transport css / js information "inline" inside the html response. Any hint how to handle it?
I transport css / js files and css / js code json encoded and need to

get the information from the response.
remove it from html response after extracted at client side with javascript.

At the moment I added a hidden div with the json css / js output / information.

I have found a solution :)
Intercooler.ready(function (elt) {...}) to handle the injected response. remove() the transfer DIV from DOM.


Extract prometheus.yaml into assets

It's a lot easier to show and explain the Prometheus config if available
in the standard yaml format. The Kubernetes ConfigMap specs can always
be auto-generated.

First outside contribution 👍 If you could just rebase then this lgtm

@brancz rebased, made the path explicit as requested, unified scrape + evaluation interval.

Thanks @grobie ! 👍


JSON parse error

EDIT: Happens when running the wwdcdownloader script
/var/lib/gems/1.9.1/gems/json-1.8.6/lib/json/common.rb:155:in parse': 784: unexpected token at '' (JSON::ParserError) 	from /var/lib/gems/1.9.1/gems/json-1.8.6/lib/json/common.rb:155:in parse'
from /var/lib/gems/1.9.1/gems/wwdcdownloader-20.17.0/lib/wwdcdownloader.rb:124:in block in load' 	from /var/lib/gems/1.9.1/gems/wwdcdownloader-20.17.0/lib/wwdcdownloader.rb:72:in block in read_url'
from /usr/lib/ruby/1.9.1/net/http.rb:746:in start' 	from /var/lib/gems/1.9.1/gems/wwdcdownloader-20.17.0/lib/wwdcdownloader.rb:67:in read_url'
from /var/lib/gems/1.9.1/gems/wwdcdownloader-20.17.0/lib/wwdcdownloader.rb:123:in load' 	from /var/lib/gems/1.9.1/gems/wwdcdownloader-20.17.0/lib/wwdcdownloader.rb:243:in run!'
from /var/lib/gems/1.9.1/gems/wwdcdownloader-20.17.0/bin/wwdcdownloader:5:in <top (required)>' 	from /usr/local/bin/wwdcdownloader:23:in load'
from /usr/local/bin/wwdcdownloader:23:in `'

@klische Thank you for the report. Is this still happening for you? I can't reproduce it.

@jfahrenkrug Just tried it, and still happening :( Perhaps something wrong with my Ruby installation?

Totally my fault. Updated to latest ruby by following directions here https://www.brightbox.com/blog/2017/04/25/updated-ruby-ubuntu-packages/
PS. For others with the same issue, make sure to apt-get install ruby2.4 and ruby2.4-dev and then reinstall the wwdcdownloader gem


color variable for split pane borders only

Is there such a variable or how to do it otherwise with css?
    termCSS: `
       .spllitpane_dividers {
         background-color: red;
       }
    `,

doesn't seem to work

.splitpane_divider should be the class you're looking for there. Looks like you have a few spelling mistakes in the selector you provided.
I also think this should be within the css propert.
Could you try this instead:
css: `.splitpane_divider { background-color: red; }`,

@CodeTheory
css: `.splitpane_divider { background-color: red !important; }`,

did the trick! thanks!


Entered unreachable code on 128x128 matrix multiplication

Aim
I am trying to implement 128x128 matrix multiplication, but I've had to modify it to be flattened due to Noir not supporting multi-dimensional arrays. My code works on smaller inputs, but once I move to 128x128 it breaks.
Expected behavior
I expect it to work as at does on smaller inputs, such as 10x10
Bug
Message: internal error: entered unreachable code: failed on standard_example__get_exact_circuit_size call
Location: /home/runner/.cargo/git/checkouts/aztec_backend-a697fb631cbad807/2cb523d/barretenberg_wasm/src/composer.rs:114
To reproduce

on local system, do nargo new matmult
Put the following in main.nr

fn main(a : [Field; 16384], b : [Field; 16384], c : pub [Field; 16384]) {
    let mut temp_field = [0; 16384];
    for i in 0..128 {
        for j in 0..128 {
            for k in 0..128 {
                temp_field[i*128 + j] = temp_field[i*128 + j] + a[i * 128 + k] * b[k * 128 + j];
            }
        }
    }

    for i in 0..16384 {
        constrain c[i] == temp_field[i];
    }
}

Set Prover.toml to the following: PasteBin
Use the following dockerfile:

FROM ubuntu:22.04 as base

RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get update && apt-get install \
    -y build-essential cmake git libgmp3-dev libprocps-dev libboost-all-dev libssl-dev libsodium-dev nano wget clang lld libomp-dev curl bash

RUN curl https://sh.rustup.rs -sSf | bash -s -- -y
ENV PATH="/root/.cargo/bin:${PATH}"

RUN git clone https://github.com/noir-lang/noir.git

WORKDIR noir

RUN curl -L https://raw.githubusercontent.com/noir-lang/noirup/main/install | bash

RUN cd /root/.nargo/bin/ && rm noirup && wget https://raw.githubusercontent.com/noir-lang/noirup/main/noirup && chmod +x noirup

ENV PATH="${PATH}:/root/.nargo/bin"

RUN noirup

# RUN nargo new hello_world

COPY matmult/ matmult/

WORKDIR matmult

RUN nargo check

RUN nargo prove p

RUN nargo verify p

Installation method
Binary
Nargo version
nargo 0.3.2 (git version hash: 29b1f7df4d563849a62e64c533cb62932188135b, is dirty: false)
@noir-lang/noir_wasm version
No response
@noir-lang/barretenberg version
No response
@noir-lang/aztec_backend version
No response
Additional context
No response
Submission Checklist

[ ] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

@f01dab1e can you check if this is an issue error occurs in our newer code, ie with --experimental-ssa enabled?

Assigning myself as I cannot assign f01dab1e

Operating system kills the process :]
❯ nargo prove --experimental-ssa
Killed

The triple loop here would mean loop unrolling would have to unroll 2,097,152 times. Plus an extra 16384 times for the second loop. We can look at memory usage but I'm not sure if this would even prove and verify in a reasonable time if that is fixed.


Gradle dependencyCheckAggregate task failed

Started getting this exception during gradle build in last 2-3 days only:

Invocation of method getHighestSeverityText in class org.owasp.dependencycheck.dependency.Vulnerability threw exception java.lang.NullPointerException

@jeremylong my educated guess for this happening would be a vulnerability that has only CVSSv4 scoring attached to it (or one for which NVD has associated a CPE, but no CVSS score has been associated yet).
https://github.com/jeremylong/DependencyCheck/blob/main/core/src/main/java/org/owasp/dependencycheck/dependency/Vulnerability.java#L537-L545

@aikebah good catch - I suppose it is time I put together the PR to actually use the CVSS4v data in the reports... I hate velocity... ;)

@jeremylong Thank you! In which version can we expect the fix?

12.0.0. I'll release 11.2.0 this week and do more testing before we release 12.0.0.