how to instrument a bottle app

Hi, this a newbie question. How can I instrument a python WSGI  bottle or flask app that has its own server ?
two threads ?
import sys

from bottle import route, run
from prometheus_client import start_http_server, Summary
import random
import time

REQUEST_TIME = Summary('request_processing_seconds','Time Spent processing request')

@route('/')
@REQUEST_TIME.time()
def handler():
    print "Hello world"
    time.sleep(random.random)

run(host='<IP_ADDRESS>', port=8080)

I managed to do it with tornado, but is there a best practice without tornado:
from tornado.wsgi import WSGIContainer
from tornado.ioloop import IOLoop
from tornado.web import FallbackHandler, RequestHandler, Application
from prometheus_client import start_http_server, Summary

from flask import Flask
from prometheus_client import Counter

app = Flask(__name__)

REQUEST_TIME = Summary('request_processing_seconds','Time Spent processing request')

@app.route('/')
@REQUEST_TIME.time()
def handler():
    return "hello world"

class MainHandler(RequestHandler):
    def get(self):
        self.write("This message comes from Tornado")

tr = WSGIContainer(app)

application = Application([
                          (r"/tornado", MainHandler),
                          (r".*", FallbackHandler, dict(fallback=tr)),
                          ])

if __name__ == "__main__":
    application.listen(5000)
    start_http_server(9999)
    IOLoop.instance().start()

Here is a working example of instrumenting a few routes in a basic bottle application and making the metrics route available an attached to the application. This also uses the MultiProcessCollector since it would be fairly atypical (at all but the smallest scale) to run a wsgi application as a single process. This requires that the prometheus_multiproc_dir variable be set. This can typically be accomplished in your wsgi runner, for example in uwsgi you could add the following to your config
env = prometheus_multiproc_dir=/tmp/helloworld.stats
from bottle import route, run, template, Bottle
from prometheus_client import multiprocess
from prometheus_client import generate_latest, CollectorRegistry, Gauge, Counter

application = Bottle()
registry = CollectorRegistry()
multiprocess.MultiProcessCollector(registry)

IN_PROGRESS = Gauge("inprogress_requests", "help", multiprocess_mode='livesum')
REQUESTS = Counter('http_requests_total', 'Description of counter', ['method', 'endpoint'])

@IN_PROGRESS.track_inprogress()
@application.route('/hello/<name>')
def index(name):
    REQUESTS.labels(method='GET', endpoint="hello").inc()
    return template('Hello {{name}}!', name=name)

@IN_PROGRESS.track_inprogress()
@application.route('/metrics')
def metrics():
    data = generate_latest(registry)
    return data

For sack of completeness, here is the equivalent example with none of the multiprocessing stuff:
from bottle import route, run, template, Bottle
from prometheus_client import multiprocess
from prometheus_client import generate_latest, REGISTRY, Gauge, Counter

application = Bottle()

IN_PROGRESS = Gauge("inprogress_requests", "help")
REQUESTS = Counter('http_requests_total', 'Description of counter', ['method', 'endpoint'])

@IN_PROGRESS.track_inprogress()
@application.route('/hello/<name>')
def index(name):
    REQUESTS.labels(method='GET', endpoint="hello").inc()
    return template('Hello {{name}}!', name=name)

@IN_PROGRESS.track_inprogress()
@application.route('/metrics')
def metrics():
    return generate_latest(REGISTRY)

application.run()

thanks a lot for your answer, I will try it asap.

Ok this is working fine. I just need to figure out how to export the metrics into my prometheus server which is running on a different node...


HTTP Path is not passed in to handler in http mode

Expected Behaviour

Paths should be forwarded to the upstream function.
Current Behaviour

I tried to run a SimpleHTTPServer in Python and noticed the file-browser wasn't passing through the path.
Possible Solution

Update the proxying code to pass the requestURI
Steps to Reproduce (for bugs)

port=8081 mode=http fprocess="python -m SimpleHTTPServer" upstream_url="http://<IP_ADDRESS>:8000" ./of-watchdog

Browse into a directory and see it doesn't work
Context

Running as a Go binary outside of Docker.
We should be careful to test this change since the templates only bind to a single path anyway of /
https://github.com/openfaas-incubator/node8-express-template/blob/master/template/node8-express/index.js#L83

I experienced the same issue with querystring variables not being passed to the nodejs handler. Do you plan to merge the patch anytime soon?

I am having the same problem: not being able to access GET parameters of the query. It would be nice if you could merge the patch.
Many thanks

I tried with the patch, where should I access the Query ? I tried to print Header & Body of handler.Request instance but I cannot find the Query.

QueryString is passed, use the latest release of the project:
https://github.com/openfaas-incubator/of-watchdog/blob/master/executor/http_runner.go#L102
This issue is about the URL Path, not the QueryString.
Alex


Automated cherry pick of #9657: fix(monitor): monitor notify international and 3.6 bugfix

Cherry pick of #9657 on release/3.6.
#9657: fix(monitor): monitor notify international and 3.6 bugfix

Codecov Report

Merging #9658 (ab14706) into release/3.6 (b9d30c0) will increase coverage by 2.20%.
The diff coverage is 2.31%.

@@               Coverage Diff               @@
##           release/3.6   #9658       +/-   ##
===============================================
+ Coverage         5.86%   8.07%    +2.20%
===============================================
  Files              828     107      -721
  Lines           129133   12967   -116166
===============================================
- Hits              7577    1047     -6530
+ Misses          120876   11800   -109076
+ Partials           680     120      -560

Flag
Coverage Δ

aFlag
8.07% <2.31%> (+2.20%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/hostman/diskutils/nbd/driver.go
0.75% <0.75%> (ø)

pkg/hostman/diskutils/nbd/lvmutils.go
4.93% <4.93%> (ø)

pkg/monitor/alerting/eval_context.go
37.00% <0.00%> (-1.15%)
:arrow_down:

pkg/monitor/alerting/rule.go
1.86% <0.00%> (-1.03%)
:arrow_down:

pkg/multicloud/esxi/manager.go
0.62% <0.00%> (-0.09%)
:arrow_down:

pkg/multicloud/esxi/vnic.go
0.00% <0.00%> (ø)

pkg/multicloud/esxi/network.go
0.00% <0.00%> (ø)

pkg/monitor/alerting/notifier.go
0.00% <0.00%> (ø)

pkg/cloudprovider/snapshot_policy.go

... and 621 more

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 22d31a3...ab14706. Read the comment docs.

/lgtm
/approve


[AsParameter] attribute generates a string query parameter when it should generate a decimal

Issue
The [AsParameters] attribute is not generating the correct type of decimal when using Swashbuckle to generate a WebApi. It generates a string.
When a decimal type is used on the object that is annotated with the [AsParameters] attribute the type generated is always a string rather than a decimal.
Reproduction
I have attached a zip file with a net7.0 solution with a basic replication of the problem. It contains a net7 minimal web API and you need to run it to see the type generated on the /swagger URL page is a string, not a decimal as expected.
TestWebApp.zip
Expected Behaviour
Generate a decimal when the client runs the build step.

I tested this using DotSwashbuckle and .NET 8 and it seems to work.
https://github.com/Havunen/DotSwashbuckle
https://www.nuget.org/packages/DotSwashbuckle.AspNetCore

I've run into simmilar issue with enum type
endpoint registered like that

As parameters object

swagger endpoint

swagger json

I can provide some more context if it will be needed

More information would be good if you can.

To make sure it's not any external or domain dependency I've created  sample project that shows the issue
https://github.com/Liandrel/Swashbuckle-AsParameters-sample
.NET 8 minimal api with no additional nugget packages except of pattern ones
one endpoint:

RequestModel:

Swagger Page:

Swagger.json

It seems the ApiExplorer provides the value for enum as string... using metadata driven approach works tho...
// Works
                endpoints.MapGet("/asparameter/openapi/enum", ([FromQuery] ConditionState? ConditionStateFromQuery,
                    [AsParameters] RequestModel model) =>
                {
                    return $"{ConditionStateFromQuery}, {model.ConditionStateAsParameters}";
                }).WithOpenApi();

// Does not work
                endpoints.MapGet("/asparameter/openapi/enum2", ([FromQuery] ConditionState? ConditionStateFromQuery,
                    [AsParameters] RequestModel model) =>
                {
                    return $"{ConditionStateFromQuery}, {model.ConditionStateAsParameters}";
                });

I tried it and for me it still renders as string field in swagger, did u changed anything else then adding .WithOpenApi() extension ?


Derive a conversion trait from Rust struct to R list

So the existing way that extendr handles Rust structs seems to be by creating an env with a pointer to the struct, and then providing wrapper functions for the methods in the annotated impl block. This is a great baseline behaviour, but sometimes I want a different behaviour from my structs; to convert them recursively into an R data structure. For instance, if my function has a complicated result struct, rather than forcing my users to call getter functions, I would rather just convert the whole result struct into some native R types.
For this reason I'm wondering if we should create a derive attribute, such that we can:
#[derive(extendr_list)]
struct Foo {
    a: str
    b: usize
}

Which would expand to:
impl TryFrom<Foo> for Robj {
    fn try_from(value: Foo) -> Result<Robj, Error> {
            let mut ret = List::new(2);
            ret.set_elt(0, value.a.into())?;
            ret.set_elt(1, value.b.into())?;
            ret.as_robj_mut().set_names(["a", "b"])?.as_list()?
    }
}

(there might be a more idiomatic way to create a named list but I don't know it)

I am busy implementing serde::Serializer for Robj.
This should allow you to use any serde derived type to generate R objects as well as
to convert them to JSON etc.
Serialize is not ideal as it doesn't deal with vectors very efficiently. So a more specific
object converter would be a good idea. There was some discussion of derive(TryFrom)
in the Rust community.
The code above can be done very simply with the new list! macro.
list!(a=value.a, b=value.b)

Yes, I found out about that great macro recently. Is there any merit in me adding a derive macro that implements TryFrom<T> for Robj for any struct using that macro?

Absolutely!
I can give some help if you like.
It needs to go in extendr_macros where we have entrypoints for macros already.

It may be a little more stable than the components that it references.
On Sun, 26 Dec 2021, 00:41 Michael Milton, @.***> wrote:

Yes, I found out about that great macro recently. Is there any merit in me
adding a derive macro that implements TryFrom for Robj for any struct
using that macro?
—
Reply to this email directly, view it on GitHub
https://github.com/extendr/extendr/issues/333#issuecomment-1001088464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL36XCTZVGSJ42LZVGWVYLUSZQEVANCNFSM5JNLX6VQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

What does that mean?

Macros are dependent on the API matching them. If the API changes, we
need to update the macros.
Don't worry, though just give it a try.


[Helper] Protractor feature parity and bug fixes

Highlights
New Helper Functions
Protractor
Protractor helper now has same API as WebDriverIO and Puppeteer (excluding WebDriverIO and Puppeteer specific functions)
List of newly added functions

_locateCheckable (only available from other helpers)
_locateClickable (only available from other helpers)
_locateFields (only available from other helpers)
acceptPopup
cancelPopup
dragAndDrop
grabBrowserLogs
grabCssPropertyFrom
grabHTMLFrom
grabNumberOfVisibleElements
grabPageScrollPosition (new)
rightClick
scrollPageToBottom
scrollPageToTop
scrollTo
seeAttributesOnElements
seeCssPropertiesOnElements
seeInPopup
seeNumberOfVisibleElements
switchTo
waitForEnabled
waitForValue
waitInUrl
waitNumberOfVisibleElements
waitToHide
waitUntil
waitUrlEquals

Nightmare

grabPageScrollPosition (new)
seeNumberOfVisibleElements
waitToHide

Puppeteer

grabPageScrollPosition (new)

WebDriverIO

grabPageScrollPosition (new)

Bug Fixes

[Helpers] Fixed I.switchTo(0) bug
[Helpers] Fixed I.scrollTo(100, 100) api inconsistencies between all web helpers

Wow, awesome stuff!
Looks like we need to publish a new version )


Connection passed with Message Event

This is a minor tweak that would make life much easier. The connect and close events both pass the connection, but message does not. This makes things a bit awkward if you want to respond to the message. You're forced to use variables outside of the local scope.
This is easily changed by modifying WebSocketConnection.js on line 340, 356, 399, 405 from:
this.emit('message', {

to
this.emit('message', this, {

Thank you for this great library!

I don't conceptually agree with this suggestion.  It would indeed make one particular coding style simpler, but the culture with JavaScript and Node is to use closures to maintain a reference to the connection at hand.  Also, the close event on the connection object does not (any longer) pass the connection as a parameter.  It used to, errantly, but the official documentation never showed the connection being passed.  The examples did, unfortunately, show it used that way, unnecessarily, and has been revised to remove the connection parameter.
Example of correct idiomatic usage:
ws.on('connect', function(connection) {
    // the connection parameter is available within the closure, so any further functions defined inside this function will have access to it.
    connection.on('message', function(message) {
        // connection is still available here:
        if (message.type !== 'utf8') { return; }
        console.log("Message from " + connection.remoteAddress + ": " + message.utf8Data;
    });
    connection.on('close', function() {
        console.log("Connection from " + connection.remoteAddress + " has been closed");
    });
});

Where this breaks down is if you use a single event handler function, passed by reference to the on() method.  Don't do that.


Celke channel added in PHP list

This sir has some good content about PHP even though his channel is not very known.

@mateussoares1997 Valeu pela sugestão !


iscroll scroll hardly with chrome dev mode in version ^5.1.3

Iscroll has some problem in chrome mobile dev mode, It scroll hardly.
It is good in chrome of pc, but bad in chrome of mobile phone, also bad in chrome with device dev mode. is something wrong with touchmove?
maybe something wrong.
in version before 5.1.3 is good, 5.1.3 bad, 5.2.0 bad, thank you very much:D

I've notice a hard scrolling issue too, did you solve it?

look at the demo. https://github.com/cubiq/iscroll/blob/master/demos/simple/index.html
use this script https://github.com/cubiq/iscroll/blob/master/demos/demoUtils.js then chrome mobile dev moda is OK.

@chang0022  Thank you very much,Demo can be used！


Can ModSecurity create a GUI panel?

Can ModSecurity create a GUI panel?

I am afraid I did not get exactly what you meant. Do you mind to explain a little bit what you want to accomplish? Is that a UI for logging? ui for rules configuration? ui using ModSecurity secrule language?

Can ModSecurity create a GUI panel?

No.
May be you're looking something like this:
https://fossil.include-once.org/modseccfg/index
It's available through PyPI:
https://pypi.org/project/modseccfg/
HTH.

@Hello-Linux: Please re-open the issues if you did not found your answer already.


65 fix posneg function issue

Fixed logic to correct mishandling of negative numbers and changed some logic for equals button click to eliminate console error when clicked sequentially. Added to test cases and ensured all passed.

Awesome, looks like a great fix!


Development

Description
GitHub Issues

Coverage increased (+0.2%) to 91.434% when pulling 06a7d3002797ad35341c9ee9e70b5ae049a55a29 on development into 37126d5d691e789b929af4df5a3e7e99df480263 on master.


fix-u1f454-svg

Made the shirt opaque. Used a white background as reference.

Thanks but unfortunately, the SVG are not sources for us, so we generally don't make these kinds of fixes in the repo.  Instead we fix our the Adobe Illustrator images in our internal repo, then regenerate the .png and .svg from them.
Sometimes the SVG generated by AI and then massaged by tooling is damaged, and we'll take fixes to the damaged SVG that don't change the appearance.

Can't you put the AI files in the repo then?

@arjenzorgdoc Thanks, and like @dougfelt said we applied to fix to the artwork in the internal repo. I don't think Google is planning on making that repo public, but if they do I'll ping you.


Login Page: Initialization

Initializing the Login Page

routing the url to the appropriate function

Update here when it's ready for review

Closing this PR, and opening a different one instead #80


NIFI-6302: Ensuring Process Group contents are pruned when appropriate

NIFI-6302:

Ensuring Process Group contents are pruned when appropriate.

All looks good on this, @mcgilman. Thanks! Have merged to master.


Use release_urls() XMLRPC method for the time being

This replaces the package_urls() XMLRPC method by the release_urls() method,
since the former was deprecated and no longer works. Invoking the endpoint
results in an error:
This API has been deprecated. Use https://pypi.org/xxx/yyy/json instead.
The XMLRPC method release_urls can be used in the interim, but will be
deprecated in the future.

However, this is only a quick-fix, since the package_urls() method will also be
deprecated in the future.

The test failure is due to copyright yeear change

The test failure is due to copyright year change

Addressed in https://github.com/openSUSE/py2pack/pull/97


Add lodash_require wrapper module.

So after tonnes of frustration, I managed to get lodash working on the Deno runtime, I thought I'd make it into a small wrapper module since the lodash module linked here doesn't provide documentation or support.
See #4316 and #4240 on the lodash/lodash repository.

@tokenchingy can you elaborate on the problems you had using the raw lodash source code? I haven’t tried it myself...

You’re unable to import the module as is, or use the ES modules, if you try, it throws import errors, but using createRequire you can load the lodash CJS modules. This wrapper just makes does that, it includes the CJS sub module from lodash and uses createRequire to import it and then exports it for use as you would. Really a convenience thing.

@TokenChingy you should be able to use Lodash directly in Deno if you use lodash-es (ES Modules format). Try using this one<EMAIL_ADDRESS>

@bartlomeiju I have done so and it’ll throw an error when it gets to `_nodeUtils.

Any update on this? @ry

@TokenChingy Can you use a different name for the module? Module names shouldn't include a dash and having "deno" in the name seems redundant... I can't think of a good suggestion "require_lodash" ?

@ry will sort it out tonight.

Renamed the packaged. I ended up choosing lodash_require as I'd like it to appear alphabetically after lodash, this package should only exist until lodash ES modules works out of the box without a createRequire().

@TokenChingy can you comment on the proper usage of lodash_require? I'm trying to import Lodash and have had an assortment of errors.
Additionally it was mentioned using the unpkg lodash-es but that results in:
...
Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>error: Uncaught ImportPrefixMissing: relative import path "util" not prefixed with / or ./ or ../ Imported from<EMAIL_ADDRESS>► $deno$/dispatch_json.ts:40:11
    at DenoError ($deno$/errors.ts:20:5)
    at unwrapResponse ($deno$/dispatch_json.ts:40:11)
    at sendSync ($deno$/dispatch_json.ts:67:10)
    at resolveModules ($deno$/compiler_imports.ts:71:10)
    at processImports ($deno$/compiler_imports.ts:166:27)
    at processImports ($deno$/compiler_imports.ts:175:13)

@frankhale , see the instructions -> https://deno.land/x/lodash_require/

Yes I’ve reviewed the instructions before I made the comment. I had to
modify them in order to get it to work properly. I’ve made comments on this
issue about how I got it to work:
https://github.com/denoland/deno/issues/3957
On Mon, Feb 10, 2020 at 23:03 Jason Lei<EMAIL_ADDRESS>wrote:

@frankhale https://github.com/frankhale , see the instructions ->
https://deno.land/x/lodash_require/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/denoland/deno_website2/pull/128?email_source=notifications&email_token=AABONSREWGKQYWYZTLJLGRLRCIPRLA5CNFSM4JTXWYLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELLE3GI#issuecomment-584469913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABONSQ45ZMSCU2T3LQLSILRCIPRLANCNFSM4JTXWYLA
.

I will look into implementing your fix, maybe something changed. This should be done soon.


Roll Flutter Engine from b494143fb0bc to c679fb0f950e (2 revisions)

https://github.com/flutter/engine/compare/b494143fb0bc...c679fb0f950e
2023-07-20<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 7kGBjmX-zBXcg1svE... to SmAtKPfGzPllC9gfO... (flutter/engine#43845)
2023-07-20<EMAIL_ADDRESS>Roll Skia from b238c09fe959 to d09e9869f84c (1 revision) (flutter/engine#43842)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from 7kGBjmX-zBXc to SmAtKPfGzPll
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add hooks.generate_with_anthropic_prompt_caching

Closes #763.
Anthropic doesn't support n>1. Also, Middleman may return fewer results than expected for a request to an Anthropic model with n>1.
This PR adds a new hook that does generations while taking these things into account. First, it makes a single generation to put the prompt in the prompt cache (assuming the agent added cache_control objects to the prompt). Then, it makes repeated generation requests until it gets back as many completions as the agent originally requested.
This PR adds a new hook because we don't want agents to behave like this for model providers that do support n>1.

@sjawhar Could you please give this another look today?


Allow hint in iframe to bypass proxy

Allow a page to have iframes that remain unproxied when the data-h-no-proxy attribute is set.
Resolves hypothesis/viahtml#457

I suggest changing the attribute name to data-via-no-proxy to make it more obvious which service this attribute is for. h is the Hypothesis data storage server, but isn't involved in processing this attribute.

@robertknight
I've renamed the attribute to directly reference viahtml. Stylistically, does it matter that it just looks for the naked attribute? (Ie, would it preferable if it handled cases like: data-viahtml-no-proxy="true" or data-viahtml-no-proxy="false"?

Stylistically, does it matter that it just looks for the naked attribute?

That's a good question. We don't have an established rule within Hypothesis. The behavior of built-in boolean HTML attributes is to treat a present attribute (disabled) as true and an absent one as false, so it seems reasonable to align with that.


Two Spaces Indention

Since the last code @AngusJohnson added had two space indention, it looks like that is the preferred method of indention.  I have modified the .editorconfig and formatted the codebase to the same indention style that you are using.  This should prevent future unnecessary space indention changes on my and other's ends that you were referring to previously.
This should be the last massive PR for a while.  After I address the warnings, that should be the end of the growing pains.

I'm happy with 2 space indentation but was somewhat mislead by VisualStudio's editor's view of your .editorconfig as it indicated Tab Size(2) and Indentation Size(2). If these weren't settings from .editorconfig I had assumed that these would have been grayed. Anyhow I evidently wrongly assumed that your suggestion (and the default settings) was 2 space indentation.

This should be the last massive PR for a while. After I address the warnings, that should be the end of the growing pains.

Daniel, I have several minor changes to Clipper.Engine.cs pending and am wondering if it'd be easier for you to finish your cleanup of warnings before I upload anything, or a should I go ahead and upload my changes?

Go ahead and make the modifications on your end and I'll incorporate the warning fixes afterword.


Allow simultaneous use of multiple IPAM modes

Proposal / RFE
Is your feature request related to a problem?
When using the Overlay network on EKS (possibly other managed kubernetes services) webhook services do not work and give an error of "Address is not allowed".  this is because the IP addresses that the pods are assigned are not within the VPC CIDR range (which is expected) and the K8S Control Plane does not allow the Cilium DaemonSet pods to be scheduled on them, so the Control Plane is not within the overlay network.
Describe the solution you'd like
Cilium can currently support the Overlay network OR the ENI network to issue out VPC routable IP addresses.  It would be nice if Cilium could support both of these at the same time, with one being the default and the other being able to be applied to a pod perhaps with an annotation (maybe something like io.cilium.ipam-mode).  This would allow pods by default to exist within the Overlay network, but then pods which require it (pods that handle webhooks/CRDs/etc) could be opted into having an ENI IP Address assigned enabling connectivity with the K8S API layer.

I am still very interested in this feature.  I see the issue went stale.  Is there any chance that this is something that would be included into Cilium?

Hi @cryptk , I think this suggestion makes sense and could be integrated into Cilium in future. I'm not aware of anyone actively working on it today however.

Are there any known workarounds to allow us to use the overlay network while still allowing pods which need to be accessed by the control plane for things such as validating/mutating webhooks to still allow those connections?

+1 on this issue, we encounter a similar usecase when using Cilium on EKS where we have the requirement that only a specific set of pods need VPC IPs (ipam mode set to ENI) while we would like the rest of the pods to be able to run with the overlay IPs.

I have the same problems 😢


Error after install on fresh server

Hi, if I try to execute the webhook, I will receive an empty reply
curl http://localhost:9000/energy-web
curl: (52) Empty reply from server

pm2 logs pm2-hooks
3|pm2-hook | ReferenceError: regeneratorRuntime is not defined
3|pm2-hook |     at WebhookServer._handleCall (/root/.pm2/modules/pm2-hooks/node_modules/pm2-hooks/dist/WebhookServer.js:96:28)
3|pm2-hook |     at emitTwo (events.js:126:13)
3|pm2-hook |     at Server.emit (events.js:214:7)
3|pm2-hook |     at parserOnIncoming (_http_server.js:619:12)
3|pm2-hook |     at HTTPParser.parserOnHeadersComplete (_http_common.js:112:17)
3|pm2-hook | >>>>>
3|pm2-hook | name energy-web
3|pm2-hook | cwd /root/applications/energy-web/source
3|pm2-hook | webhook undefined
3|pm2-hook | name pm2-hooks
3|pm2-hook | cwd /root/.pm2/modules/pm2-hooks/node_modules/pm2-hooks
3|pm2-hook | webhook undefined
3|pm2-hook | fin
3|pm2-hook | [2018-05-29T16:46:09.076Z] Started. Routes:

Distributor ID: Ubuntu
Description:    Ubuntu 18.04 LTS
Release:        18.04
Codename:       bionic

Anything I can do?

Hi @Fchen48, I am sorry this is not working for you! The only thing I think can cause your problem is the Node version, which one are you using? I think you need 7.10 in order to make this project to work

I'm using the latest LTS version, v8.11.2

Oh! Yeah I remember now I used babel for this project. I am sure it's related to how I configured babel. I will take a look to be sure I am using the polyfills. If you want you can make a PR or I will take a look this weekend. Thanks for the issue! I will keep you informed

Hi @Fchen48! I was able to replicate your error. The issue was I didn't published to npmjs.com since a long time ago. The current version is 1.1.5 but in npmjs had the 1.1.1 where the issue with babel was not resolved yet. I just created a new release, 1.1.8 with updated dependencies etc and published it to npm. Now you can uninstall it (pm2 uninstall pm2-hooks) and install again and everything must be working.
I will wait to you to confirm the issue is gone before closing this issue. Thanks!
PS: I realized it was an error on the readme. It said the property to write your config on ecosystem was 'env_hooks' but it's actually 'env_hook'. Please double check you have it correctly.

Problem solved


Missing stim config causes error

If I already have a vault token and use stim (running the stim docker container) to get kube config, it errors and complains that the ~/.stim/config.yaml file doesn't exist. It would be nice if stim would check if it needs the config file to run the command or at least have the docker container create an empty config file

do you mean the warning:
2019-06-25 19:54:06.8682203	[ WARN  ]	No config file exits at :"/home/<user>/.stim/config.yaml"

?
If so I dont think its appropriate to completely removing this, even with a valid token and vault addr set.  There are plenty of other things that could be needed from the config if its missing.  Could probably improve the IsAutomated() check and maybe even add a cli/env flag for it so it skips certain user errors/warns in that case.  Does that sound like an appropriate solution?

The warning is fine. It actually errors. In this case, it is writing to the file so maybe it could create it if it doesn't exist. Or maybe a cli flag to force IsAutomated() to be true.
Below is the output from running VAULT_TOKEN='...' stim kube config ... without a config file.
2019-06-26 11:40:25.566293	[ WARN  ]	No config file exits at :"/home/<user>/.stim/config.yaml"
2019-06-26 11:40:25.566446	[ WARN  ]	Issue loading config file use -verbose for more info
2019-06-26 11:40:25.568909	[ WARN  ]	Stim-vault: time: invalid duration
2019-06-26 11:40:25.870283	[ FATAL ]	Stim-Vault: Error Updating username in configuration file: open /home/<user>/.stim/config.yaml: no such file or directory

Ahh yeah thats a problem.


asp-prerender-module isue with service fabric

i am using angular 2 app and hosting it in service fabric cluster.
locally(local cluster) every thing works.
but when i host in azure cluster it is not working.
i have used asp-prerender-module tag.
i have installed node.js manually on cluster VM.  but it is still not working.
when i remove asp-prerender-module tag it start rendering.
Note: every script in copied on cluster i  have verified vendor.js and client.js

Thanks for letting us know. Do you have specific error messages, or something else that would give an indication of what might need to be fixed?

I'll close this due to inactivity. If you do manage to find out what the error messages are, please let us know.


AppMetrics Maintenance Plan

There seem to have been no releases in this repo since 2021, are there any plans to maintain this repo in the future .
We want to implement this nuget for many of our project and wondering on the roadmap for this project in the future.

@alhardy may be you can guide us on this?

I've stopped maintaining the project given projects like open telemetry, open telemetry I think is the way forward for observability for now.
If AppMetrics is still useful, happy to add those that wish to contribute as owners to the repo.

Thanks for the quick response. Closing the issue as its answered.


Refactor(plugins): Move WAN internet-exit zscaler integration to eos_designs

Change Summary

Fixes for cv_zscaler_endpoints lookup plugin
Related Issue(s)
Fixes #3994
Component(s) name
arista.avd.cv_zscaler_endpoints
Proposed changes

Various fixes and updates.

How to test

Manual testing with Vinay.

Checklist
Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

The functionality of this PR has been tested and confirmed by @vinaygcArista


Render all our JSON-LD as server-side props

This continues a trend in a few other places – rendering our JSON-LD on the server-side and passing it down through the props. The theory is that it should mean we have to send less data to the front-end, because:

We can delete large objects from the *Basic types (contributors are always large)
We can shake the JSON-LD transformer code out of the bundle, and just run it server-side

This actually came out of staring at the PageLayout component and realising it does a bunch of stuff we should probably be doing server-side. We shouldn't really be sending any Prismic models to the client.

😮
I was expecting a bundle size improvement, but not that substantial!

There's also some discussion here: https://wellcome.slack.com/archives/C3TQSF63C/p1648730771823929


Add ability to remove unrestricted items when there are linearly dependent recipes

Suggestion
I am not sure if this is a bug but I feel like it's just a behavior that was not considered.
I got into a situation in which I needed to remove a lot of recipes to continue because I couldn't get past the error. I got into this situation by choosing an unrestricted item. So it should be possible to undo this state by going back into the unrestricted item selection. Or by removing the previous recipe.

Hello, thanks for the suggestion. I changed the UI for the matrix solver configuration in a recent update which should help with this. Please have a look at that. If you still think it should be improved, please feel free to open a new issue, thanks!


Method is too large in init method

I intermittently get following when compiling nballerina project
Compiling source
        wso2/nballerina:0.1.0
ERROR [nballerina.front:(1:1,1:1)] method is too large: '.<init>'
error: compilation contains errors

We have optimised the generated code specially related to init function by properly splitting the large functions and moving some utility methods to separate class.  We have removed some duplicate code generation for union types. Please refer the following PRs for more information.
https://github.com/ballerina-platform/ballerina-lang/pull/32059
https://github.com/ballerina-platform/ballerina-lang/pull/32325
I am closing this issue since above fixes will solve the method too large issue with init. Please reopen the issue if you have further queries on this.


Guide for D365 F&O consultants and clients to set up inventory posting profiles

Contact details
<EMAIL_ADDRESS>Organization type
Partner / ISV / Independant consultant
End-to-end business process
Inventory to deliver
Which business process area is this article related to?
guidance/business-processes/inventory-to-deliver-overview.md
Which business process is this article related to?
Record financial transactions
Which pattern or practice is this request related to?
Manage ledger postings
Enter any additional comments or information you want us to know.
Hello Sir/Ma'am,
My name is Rajendra Sherikar and I am a veteran in D365 F&O consulting having spent more than 20 years in this domain of expertise. Over the years I have realized that it's very difficult for new consultants and clients to understand and set up financial posting profiles. Based on my experience over the years I have created an excel spread that I use as a guide to set up inventory financial posting profiles for Sales, Purchase, and Inventory, and Production. Like this I have also set up posting profiles guides for many other transactions. But for this specific submittal, I would like publish the inventory posting profiles guide to help new consultants and clients understand and set them up. I seek your approval to publish the article in git bub and hence am writing up this issue. Please get back to me on the subject. Thanks.
(Business process workstreams: Source to pay, Order to cash, Plan to produce)
Specify the date you expect the article to be completed and ready for review.
11/15/2024
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Hello Rachel,
I am aspiring to apply for fast track recognition for which, publishing a business process guide here is one of the requirements. Deadline for this application is soon in January. In view of this I would be grateful if you could kindly approve my request to apply. Looking forward to your reply.
Thanks
Rajendra Sherikar


404 Not Found when new asset is virus scanned but not uploaded to S3

I haven't actually checked this, but I'm pretty sure an asset's state is updated to clean if the virus scan job completes successfully and the upload to S3 is only triggered at this point. If the upload to S3 takes some time or fails for some reason, I think the app will attempt to proxy a request for the asset to S3 which will presumably respond with a 404 Not Found.
Pragmatically, the S3 upload usually runs immediately after the virus scan and doesn't usually take very long to complete, so the time window in which this could happen is very short. However, it feels as if we're modelling the asset state incorrectly and this feels more significant now we're serving all assets from S3 in production.

I've just realised that Asset Manager responds with a 404 Not Found when a Mainstream asset that hasn't been virus scanned is requested and so this isn't so much of an issue for Mainstream assets. However, for Whitehall assets this bug will result in a placeholder image/page redirect while the asset is being virus scanned, then a 404 Not Found (from S3) until the asset is uploaded to S3, and then finally the asset will be served correctly. The behaviour should be to continue to do the placeholder redirect until the asset is uploaded to S3.

I've opened #369 to address this.


Use latest scala rules to fix tests

This is for https://github.com/bazelbuild/rules_scala/issues/532 which is breaking the builds at the moment.

It seems like the scala rules have changed a bit since the last pinned commit, maybe to include toolchains. Hopefully the SSL cert issue is fixed very soon so this doesn't have to be looked at again very soon, as I don't have time to get into this more.

Hi @Code0x58,
Firstly, thanks to having a look this.
It looks like no matching toolchains found from latest travis log:
(18:43:50) ERROR: While resolving toolchains for target //heron/api/src/scala:api-scala: no matching toolchains found for types @io_bazel_rules_scala//scala:toolchain_type
(18:43:51) ERROR: Analysis of target '//heron/api/src/scala:api-scala' failed; build aborted: no matching toolchains found for types @io_bazel_rules_scala//scala:toolchain_type

From bazel doc, we can try setting default toolchain if no concern.
# WORKSPACE
# register default scala toolchain
load("@io_bazel_rules_scala//scala:toolchains.bzl", "scala_register_toolchains")
scala_register_toolchains()

Ref: https://github.com/bazelbuild/rules_scala/blob/master/README.md

@Code0x58 @erenavsarogullari - is this good to go?

Yep, I think this was merged as a part of #2928 though

Yep, this is in. The build failure on master is down to #2925 now


MacOS Crash 1.19.2

Minecraft client instance fails to load when using a MacOS system. Tested & confirmed with 1.19.2 running on Ventura 13.1. Best assumption is that the issue is caused by a file access issue.

Determined as user error. Server member had an incompatible owo-lib installed.


fix [#77] : Ask confirmation on reset

Fixes #77
Added an "Are you sure" dialog to Suitcase App, that briefly explains what is about to happen.

runtests

fixes odk-x/tool-suite-X#77

Thank you for reviewing my work!
You can let me know if you need something added or modified.

Closes: https://github.com/odk-x/tool-suite-X/issues/77


missing button to return from tag overview

go to the menu for motions
click there on the button "Tags"

result: one sees the list of existing tags. What is missing is a button to go back to motions. For all other things there is such a back button.

Right, but we have the problem that this tag list view is used from agenda, motions and elections list view. So we can't remember from which list view the user is come to the tag list.
I think this is not so bad because you can navigate via main menu items. I would close this issue if nobody has a better idea. Ok?

Is there something like a referer in AngularJS State?

There is a open discussion about a "previous state" in ui-router:
https://github.com/angular-ui/ui-router/issues/92

So we can't remember from which list view the user is come to the tag list.

Hmm, why not simply adding a button that does the same as if I would use the back button of the webbrowser?

Because you cannot be sure that the user comes directly to tag list view without any turn about e.g. tag edit view. One example:
motion list -> tag list -> tag form -> tag list
(a back action would be go to tag form view)

But from my view as bug reporter this would do the job. I just want a way to go back without the need to use the browser. So it is fine if the button is named "go back" and does the same as the browser back button.

Ok, this is easy: A new "Back" button on the top of the tag list view which use the browser back action.

@matakuka Can you try this issue?

Works fine. many thanks!


Create Experiment Modal: Metrics Step Design issues

Describe the bug
User Interface for Metrics Step has design and user experience issues
To Reproduce
Steps to reproduce the behavior:

Create new experiment, or edit existing
Fill out first 3 steps

Expected Behavior
Table rows and headers differ from designs, and/or have unexpected behaviors:

Small Font sizes
Wide vertical spacing
Form fields have missing underlines
First item "Metric*" is a dropdown, but no triangle indicator given
Selected Items in "Metric" show as grey and disabled, which seems like a weird state
Description is required?

Screenshots
Expected:

Actual:

Additional context
Add any other context about the problem here.

@danoswaltCL Few notes on the above expected behaviour:
Small Font sizes (Will do that)
Wide vertical spacing (Will do that)
Form fields have missing underlines (Will do that, but it's not there in figma design. If that's required, will have to do that in other stepper pages as well)
First item "Metric*" is a dropdown, but no triangle indicator given (Will do that, make it select option)
Selected Items in "Metric" show as grey and disabled, which seems like a weird state (This is as it was in earlier monitored metrics page and it is to avoid various issues in case of repeated metrics)
Description is required? (Yes, this will be the name of the query in experiment details page)

@zackcl okay, so let's confirm this in the meeting. Do we want metrics to be type ans select or dropdown.

Per the discussion on 7/7, a combo box is acceptable, but it should validate against the metadata


Added fliptopbox to the XXIIVV index

Hi I hope this conforms with the guidelines :crossed_fingers:  been wanting to join for almost a year. my work is mostly front-end JavaScript, the homepage is a gallery of my favourite experiments, (and respective links to the code/articles).
The about and creative writing and webring links are on global naviagion, plus a link to a random neighbour :smile:

Nice website but all the content is hosted offsite, the website is mostly a single-pager to jump to 3rd party services. It does not qualify for the webring, sorry.


Add run export

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #27.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

@conda-forge-admin please rerender

@conda-forge-admin please rerender


Country is not shown in report BC365

Hey guys, in my report I would like that not only the address but also the country is displayed under the sender. I do all this in Business Central. Unfortunately, this does not seem to work for me ... If I look at the layout of the report, under the sender there are several fields, which are intended for the address data. Is it normal that the country is never printed or do I have to change the company data?
Thank you for your help in advance

This repository no longer handles general questions and is only dedicated to bug reports and feature suggestions for AL compiler and developer tools
To get help please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


Reconstruct xml from new LaCo (layout content) block in vxml files

LaCo is an AST (or so Valve calls it), which should pretty trivially map back to xml.
As DATA is now empty in these files, you can just check if that block exists after this:
https://github.com/SteamDatabase/ValveResourceFormat/blob/75c8ec96f31494947c279d7455a3ca2bc864b0b3/ValveResourceFormat/Resource/ResourceTypes/Panorama.cs#L53
And just do LaCo.ToString and put it into DATA so it will just work as-is in the GUI/CLI.
This should be all the possible types:

Fixed in 7641998af978989bc86b168d4a9ec486b3184713


Pilz Sequence Motion MoveGroupSequence example

MoveGroupSequence example for generating a sequence of trajectories with a blend radius.

pilz_moveit.launch.py replaces mtc_demo.launch.py since it needs to include the action server and service server capabilities.

pilz_sequence.cpp plans and shows a trajectory using the MoveGroupSequence service, then plans and executes the trajectory using the MoveGroupSequence action. Finally, it executes the same sequence, but stops the motion after 2 seconds.

@sea-bass

Very cool to see! A few initial suggestions
1. Make sure you don't commit the .vscode folder that your IDE may create

2. Can you instead rebase this and put up the PR to the `main` branch instead of humble? That's where development happens.

3. Also, make sure you modify the actual docs page to add instructions for your great new example!

Try making those changes and I'll do a more thorough review/test after.

Sure, I will do those changes. Thanks for the reply!

Gave a more thorough look and tested it -- looks great!
Two major comments:
1. I think calling and waiting for the service and action results may be done incorrectly, as `future.get()` does not block. You should do something like what is shown in [this ROS 2 tutorial](https://docs.ros.org/en/humble/Tutorials/Beginner-Client-Libraries/Writing-A-Simple-Cpp-Service-And-Client.html#write-the-client-node) and [this example for actions](https://github.com/ros2/examples/blob/rolling/rclcpp/actions/minimal_action_client/not_composable.cpp#L44) -- I suspect this is why the execution fails, I even got a segfault!

2. The docs show the action interface and then the service interface first, but the example does service and then action. I recommend shuffling the docs to also do first service and then action.

BTW were you getting these errors?
[ros2_control_node-5] [ERROR] [1719959728.962806900] [panda_arm_controller]:
Time between points 27 and 28 is not strictly increasing, it is 2.700000 and 2.700000 respectively

Nop, I didn´t get any errors. Did you changed or added points in the trajectory? I could try to replicate it.
For (2) I think you are right, I already change the order in the tutorial.
One last thing: after doing the main rebase, I'm getting error in others examples that I didn´t modify, so I can´t compile the package. Do you know how to solve this?

It's possible that my trajectory errors show up because I am testing on rolling, not Humble.
Similarly, versions of packages could be causing compilation errors on your end. What are you seeing?

I have changed the server's future. Now it waits with a while loop and future.wait_for() unitil it is ready. I still have to update the tutorial, let me know if the changes are ok to do so.

One last thing: after doing the main rebase, I'm getting error in others examples that I didn´t modify, so I can´t compile the package. Do you know how to solve this?

These are some of the issues I have when compiling the package
user@Lenovo:~/ROS2/moveit_fork_ws$ colcon build --packages-select moveit2_tutorials
Starting >>> moveit2_tutorials
[Processing: moveit2_tutorials]
--- stderr: moveit2_tutorials
moveit2_tutorials: You did not request a specific build type: Choosing 'Release' for maximum performance
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/realtime_servo/src/pose_tracking_tutorial.cpp:49:10: fatal error: moveit_servo/utils/common.hpp: No such file or directory
   49 | #include <moveit_servo/utils/common.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
gmake[2]: *** [doc/examples/realtime_servo/CMakeFiles/pose_tracking_tutorial.dir/build.make:76: doc/examples/realtime_servo/CMakeFiles/pose_tracking_tutorial.dir/src/pose_tracking_tutorial.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:736: doc/examples/realtime_servo/CMakeFiles/pose_tracking_tutorial.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp: In function ‘int main(int, char**)’:
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp:156:46: error: ‘struct moveit::planning_interface::MoveGroupInterface::Plan’ has no member named ‘trajectory’; did you mean ‘trajectory_’?
  156 |   visual_tools.publishTrajectoryLine(my_plan.trajectory, joint_model_group);
      |                                              ^~~~~~~~~~
      |                                              trajectory_
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp:209:46: error: ‘struct moveit::planning_interface::MoveGroupInterface::Plan’ has no member named ‘trajectory’; did you mean ‘trajectory_’?
  209 |   visual_tools.publishTrajectoryLine(my_plan.trajectory, joint_model_group);
      |                                              ^~~~~~~~~~
      |                                              trajectory_
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp:278:46: error: ‘struct moveit::planning_interface::MoveGroupInterface::Plan’ has no member named ‘trajectory’; did you mean ‘trajectory_’?
  278 |   visual_tools.publishTrajectoryLine(my_plan.trajectory, joint_model_group);
      |                                              ^~~~~~~~~~
      |                                              trajectory_
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp:354:46: error: ‘struct moveit::planning_interface::MoveGroupInterface::Plan’ has no member named ‘trajectory’; did you mean ‘trajectory_’?
  354 |   visual_tools.publishTrajectoryLine(my_plan.trajectory, joint_model_group);
      |                                              ^~~~~~~~~~
      |                                              trajectory_
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp:406:46: error: ‘struct moveit::planning_interface::MoveGroupInterface::Plan’ has no member named ‘trajectory’; did you mean ‘trajectory_’?
  406 |   visual_tools.publishTrajectoryLine(my_plan.trajectory, joint_model_group);
      |                                              ^~~~~~~~~~
      |                                              trajectory_
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/move_group_interface/src/move_group_interface_tutorial.cpp:461:46: error: ‘struct moveit::planning_interface::MoveGroupInterface::Plan’ has no member named ‘trajectory’; did you mean ‘trajectory_’?
  461 |   visual_tools.publishTrajectoryLine(my_plan.trajectory, joint_model_group);
      |                                              ^~~~~~~~~~
      |                                              trajectory_
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/motion_planning_api/src/motion_planning_api_tutorial.cpp: In function ‘int main(int, char**)’:
/home/user/ROS2/moveit_fork_ws/src/moveit2_tutorials/doc/examples/motion_planning_api/src/motion_planning_api_tutorial.cpp:207:69: error: ‘struct planning_interface::MotionPlanResponse’ has no member named ‘error_code’; did you mean ‘error_code_’?
  207 |       planner_instance->getPlanningContext(planning_scene, req, res.error_code);
      |                                                                     ^~~~~~~~~~
      |                                                                     error_code_

I'm using ROS2 Humble on Ubuntu 22.04

That's an incompatibility between Humble and main. Are you also now using the main branch of moveit2 repo?
Also make sure you have uninstalled any other moveit packages from binaries: sudo apt-get remove ros-humble-moveit*

No, my bad , I was still using the humble branch. Nevertheless, I couldn't install the main branch since I get some errors using the Humble distro. I've seen some people with the same issues, like the last comment in this issue or this issue.
Before I keep trying, do you know if this is possible? I'm following the installation steps.

I'm actually not sure if the main branch of MoveIt works on Humble anymore. You can try by ensuring you have removed all your MoveIt binaries and following the source instructions in a clean build.

@alejomancinelli is this ready for review again, or are you still working on it? If it is ready, re-request and I will test it out again.

Hello @sea-bass. Yes, sorry, I think is ready. I didn't know I had to do a re-request. I will merge with main and do it.

Update: With https://github.com/moveit/moveit2/pull/2943, I could get these tutorials to run correctly on Rolling / main!

@sea-bass I am unable to reproduce the issue you saw with the duplicated time points on rolling, but in my opinion, that is a MoveIt / Pilz issue and shouldn't block the addition of this tutorial.

I'm on Ubuntu 24.04 -- are you as well? Anyway, I found a band-aid fix in https://github.com/moveit/moveit2/pull/2943

I'm on 22.04 host but I run everything in the osrf/ros:rolling-desktop container which is 24.04, so should be similar.

@alejomancinelli thank you again for the awesome new tutorial! 🚀

Hello, I found a way of uing the MoveGroupInterface to greatly simplify the process of creating the action requests. Is that of interest? If it is, I would be happy to create a new pull request with new tutorials and everything. Here is an example

@gaspatxo cool! If you have time, you should put up a PR modifying this existing tutorial with whatever improvements you like.

@alejomancinelli thank you again for the awesome new tutorial! 🚀

@sea-bass thank you and the team for all the feedback! I learn a lot with this.

@gaspatxo interested in the simplified process, found this PR, did you have a change to make this other example?


Burt not showing up as a possibility for the 8:30 spawn

Burt from East Luterra only appears in the upcoming merchants for 9:30, even though he also spawns at 8:30.

The data file seems correct.
https://github.com/Xeio/WanderLost/blob/92ee8e3ce60f813f04fd3b685ba9e6bfd385ce7c/WanderLost/WanderLost/Client/wwwroot/data/merchants.json#L342-L345
Are you saying you see Malone/Oliver/Nox and the other merchants that also spawn at the same time but not Burt?


Add feature to compile .typ file to image to cli

The now cli now compiles .typ files to .png with the flag --image.
Each page/frame is saved individually (output-1.png for the first page for example).
Additional options have been added that sets the pixels per point and background color (including transparent).

I've rewritten this to work with clap now, however I can't seem to import Color from typst in args.rs. So I can't make a parser for the fill option. Could I get some help with this?

args.rs is used both by main.rs and by build.rs. To usetypst there, you would need to copy the Typst dependency in [dependencies] to [build-dependencies].

Closing as this went stale. Feel free to reopen if it is rebased.


chore: downgrade amplify codegen to 4.7.3

Description of changes
Downgrade amplify-codegen to 4.7.3 and @aws-amplify/appsync-modelgen-plugin to 2.8.1

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

In favor of https://github.com/aws-amplify/amplify-cli/pull/13696


Schedule

@jossiwolf @helmuthb feel free to check the schedule out. It needs some minor changes and more speaker data (and the workshops will have to be added), but it's 80% ready and we need to announce already, so I say we roll out and add more things as we get more information.

I'd like to merge this tomorrow (Thursday) morning and kick off the announcements.


B 20012 INT GBLOC Endpoint Followup

B-20012
Summary
Updates the GBLOC Switcher to use a list of distinct GBLOCs from the transportation_offices table.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the office app as an HQ user.
Click around, switch GBLOCs, refresh the page, etc. making sure it functions the same as before.
Open up your DB viewer and get a list of GBLOCs with the following query: SELECT DISTINCT gbloc FROM public.transportation_offices ORDER BY gbloc ASC
Verify the list returned matches the list available in the dropdown.

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 1e5e1c62210fed9cd94fc2dbfdcb3c347bdfc6cc


Non-existing Thunderstore dependency

The current dependency on bbepis-BepInExPack in the Thunderstore version of the mod causes an error when trying to download the mod through r2modmanager as bbepis-BepInExPack does not exist as a package on Thunderstore
To resolve this issue, the dependency string for BepInEx should be changed to denikson-BepInExPack_Valheim-5.4.800 in the manifest.json

Thanks for pointing this out @NeonCarbide, I've updated the dependency and pushed v0.1.2 to Thunderstore. I won't be able to test until tonight but feel free to reopen the issue in case it still isn't working!

Everything works a-ok in 0.1.2, thanks for the quick update ^-^

Awesome! You're more than welcome, enjoy modding or using the library 😄


fix commands not responding with command message

when you do a command like !timeis it replies only with the command output and doesn't show the original command sent to get the output.

it used to show the command !timeis with a second message from the user below with arrow responding to command message with the output.

found the issue

disregard old issue the reason it was set to false is because it will do arrow response to the first message you send after you join

there is also another issue where it will not show your command message if you have already chatted and can send messages without your name appearing

ok never mind there at least 6 different issues with command responses, it's insanely buggy and some issues you cant get to repeat reliably as its just random


Project's landing page should tell more

The https://ij.bazel.build/ landing page has no content.
It should contain similar info as in the GitHub README.

Good idea -- thanks! Added a small description in the landing page.
Note we're not yet in a position to keep the website up to date, let alone spend much time improving its appearance. So apologies for any inaccuracies.


Update README.md to include repo link for Crystal-Lang port

Added link to crystal-lang repo.

Looks like #912 already covers this!


Tests randomly failing in AppVeyor

Regarding #310 and #315,
AppVeyor just randomly fails partway through testing on v10 with no output or reasoning. Try to log any output and it starts working, so it's seemingly impossible to figure out for now. Gonna hit up  @FeodorFitsner to see if he has any ideas.

Is this fixed via https://github.com/nodegit/nodegit/pull/324?

Nope :(
Still an issue.

Trying to ferret the issue out via https://github.com/nodegit/nodegit/pull/352.
It seems the be that the async cloning is killing something somewhere. I was able to repo it once locally and only through Powershell and not cmd.exe or cmder.
Removing just one of the clone tests seems to fix the issue but that's a pretty crappy solution. Any help with this though would be great.

So we decided that for the time being we're going to skip some of the clone tests and get appveyor passing. It's not the best solution but we've spent a lot of time trying to figure out what is actually the issue and it just seems like a timing thing. After #352 is merged I'll close this out.

Hey @FeodorFitsner, is there an environment variable that I can tie into to know that code is being run on AppVeyor?
Alternatively you can help me figure out why the clone tests are dying. :)

10 seconds after I ask I find http://www.appveyor.com/docs/environment-variables

#352 was finalized and we decided to remove node v0.12/0.11 from the test matrix for AppVeyor for the time being since we now support io.js. AppVeyor is passing and now good so I think we can close this issue out now.

This issue is very much alive. We still get it in the clone tests, and I suspect thats what is happening with the status tests in https://github.com/nodegit/nodegit/pull/374.
@FeodorFitsner, any chance you can take a look at this?

Have you tried updating npm to 2.0: http://www.appveyor.com/docs/lang/nodejs-iojs#locking-errors

No, but that doesnt quite seem like the issue. I think the problem may have been something on our side related to slashes. Path module generally handles them, but there were some slipping through in a string passed to fs.exec. Fixing all of that up right now to see if it helps.

Hrm, this is finally occurring outside of appveyor, and happening locally on windows.

Figured it out. A few weird synchronicity issues, amongst other things. Fixed here:
https://github.com/nodegit/nodegit/pull/388


Add cpp-fiabonacci

*Please follow the Contributing Guidelines & naming conventions in the guidelines accordingly. 🙂
Thanks for contributing


Access NSError Object in ErrorView

In order to give the user a more concrete info than just "Something went wrong.", it would be straightforward to have access to the NSError object in the view.

Hi, thanks for the suggestion.
I was thinking about a generic PlaceholderViewProtocol that every placeholder view can optionally adopt. The protocol gets called whenever the view

willAppear
didAppear
willDisappear
didDisappear

Optionally, the current state (including any errors) might be passed with these calls.
Would that be sufficient in your case?

Yes, that would be sufficient.
One more thing: In some of my ViewControllers, I would like the StatefulView not to cover the whole screen. I would like to set an y-offset or a custom constraint. Is there a non-hacky way to achieve that?
Thanks!

Perfect. :+1:
Not sure when I will start to tackle this issue, if you want to hack on it feel free to open a pull request any time.

Regarding your question: A quick solution would be to use a transparent view as one of your placeholder views. However, this will block the UI behind it.
Alternatively, we could look into the possibility to provide a delegate method (on the view or the controller) to return the constraints for specific views.

Any updates on exposing the NSError object inside the error view? I would really like to give a better explanation to the user why the operation failed, instead of a generic message.


fix: General Fixes and Improvements from v<IP_ADDRESS> Fixes Branch

This pull request includes significant updates to the GitHub Actions workflows and Docker build configurations. The changes aim to support new build types, improve version handling, and streamline the Docker build process.
Workflow Enhancements:

.github/workflows/docker-build.yml: Added support for building Docker images with a new version format and additional checks for version inputs. Introduced version-no-v output to handle versions without a leading 'v'. [1] [2] [3] [4] [5] [6] [7]
.github/workflows/release.yml: Added a new job to build Docker images for Langflow with an entrypoint (main-ep). [1] [2]
.github/workflows/release_nightly.yml: Introduced a new job to build nightly Docker images for Langflow with an entrypoint. [1] [2] [3] [4]

Dockerfile Updates:

docker/build_and_push.Dockerfile: Added README.md to the Docker build context to ensure it is included in the build.
docker/build_and_push_ep.Dockerfile: Created a new Dockerfile for building Langflow with an entrypoint, including steps for building dependencies and setting up the runtime environment.

LGTM


Attach version_ to checkpoint path only if version is int

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
N/A Did you make sure to update the docs?
[x] Did you write any new necessary tests?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes #1659.
We should only attach the prefix version_ to the default checkpoint path when the logger's version is an integer.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @yukw777! Thanks for opening this PR.

In the file tests/callbacks/test_callbacks.py:

Line 263:111: E501 line too long (116 > 110 characters)

Do see the Hitchhiker's guide to code style


TFKUBE-226: Add test for remote agents

Can you remove line 4-5 in installer_test.go? Go linter complains because they are unused imports

Thanks for testing. Let's clean up the code in TFKUBE-48 to avoid breaking the code.
This maybe needed for TFKUBE-224 and TFKUBE-225 - the reason is we implemented all together, and split the relevant code to raise different PRs.


31000: Ensure Glue creates and transmits update events

Field
Value

Redmine Ticket Number
31000

App-Dev Road Map #
n/a

Start Date
n/a

Due Date
n/a

App-Dev Priority
UP-Disqueued

Development Story Points
n/a

Peer Review Story Points
n/a

Ticket Author
@kevin-lerer

closing PR this need to be open against MA glue


fix(ts-sdk): fix subpath issue on codegen

@marcosnils raised an issue happening when loading a module as
a subpath of a root directory.
This leaded to an error happening during the diff
failed to diff generated code: select:
cannot diff with different relative paths

This PR fixes that issue by not creating a directory
as generated result code but instead reusing the
codegen dir.
Example now
{
  git(url:"github.com/marcosnils/daggerverse") {
    branch(name:"main") {
     tree {
      directory(path:"gptools") {
        asModule (sourceRootPath:"examples/typescript") {
          id
        }
      }
     }
    }
  }
}

Result:
 dagger query --doc query.gql
{
    "git": {
        "branch": {
            "tree": {
                "directory": {
                    "asModule": {
                        "id": "ChV4eGgzOjZlNzllZjllZDVjNjlmZjUSawoVeHhoMzo0MGQ0OTRkOTFiYjcxYTA5ElIKFXh4aDM6ZWUwZjU0YzY0NmI2MmU3ORIKCgZHaXRSZWYYARoGYnJhbmNoIg4KBG5hbWUSBjoEbWFpbkoVeHhoMzo0MGQ0OTRkOTFiYjcxYTA5EoYBChV4eGgzOjZlNzllZjllZDVjNjlmZjUSbQoVeHhoMzo4NzQwODY2ZTE3MzJhMDcxEgoKBk1vZHVsZRgBGghhc01vZHVsZSInCg5zb3VyY2VSb290UGF0aBIVOhNleGFtcGxlcy90eXBlc2NyaXB0ShV4eGgzOjZlNzllZjllZDVjNjlmZjUSdAoVeHhoMzo4NzQwODY2ZTE3MzJhMDcxElsKFXh4aDM6ZDhhMTgxZTE5ODM5NTFkMBINCglEaXJlY3RvcnkYARoJZGlyZWN0b3J5IhEKBHBhdGgSCToHZ3B0b29sc0oVeHhoMzo4NzQwODY2ZTE3MzJhMDcxElwKFXh4aDM6ZDhhMTgxZTE5ODM5NTFkMBJDChV4eGgzOjQwZDQ5NGQ5MWJiNzFhMDkSDQoJRGlyZWN0b3J5GAEaBHRyZWVKFXh4aDM6ZDhhMTgxZTE5ODM5NTFkMBJ0ChV4eGgzOmVlMGY1NGM2NDZiNjJlNzkSWxIRCg1HaXRSZXBvc2l0b3J5GAEaA2dpdCIqCgN1cmwSIzohZ2l0aHViLmNvbS9tYXJjb3NuaWxzL2RhZ2dlcnZlcnNlShV4eGgzOmVlMGY1NGM2NDZiNjJlNzk="
                    }
                }
            }
        }

Signed-off-by: Tom Chauveau<EMAIL_ADDRESS>

Hm @marcosnils do you have the full error message? Which two paths didn't match?
cc @sipsma, I know you fixed something like this in https://github.com/dagger/dagger/pull/7328, just wanted to check this with you.
I wonder if it might just be worth biting the bullet and removing this limitation with Diff.

Hm @marcosnils do you have the full error message? Which two paths didn't match?

Here's an example
! failed to create module from directory: select: failed to create module: select: failed to update codegen and runtime: failed to diff generated code: select: cannot diff with different relative paths: "gptools" != "/"

Error: make request: input: git.branch.tree.directory.asModule resolve: failed to create module from directory: select: failed to create module: select: failed to update codegen and runtime: failed to diff generated code: select: cannot diff with different relative paths: "gptools" != "/"

Hm, fair enough, looks like a genuine case, and not a typo-y issue like before.
Could we have a test for this? Based on past work, this does seem to be fragile logic, so I'm a bit concerned that we don't seem to have a test that covers this edge case. Do we not have a test for module subpaths in a root directory for typescript? I think this is covered for go.

Could we have a test for this? Based on past work, this does seem to be fragile logic, so I'm a bit concerned that we don't seem to have a test that covers this edge case. Do we not have a test for module subpaths in a root directory for typescript? I think this is covered for go.

We got some tests:

With different root
https://github.com/dagger/dagger/blob/aa9fc534b0c750f71f26ae0f288c6369fa030d06/core/integration/module_typescript_test.go#L34

Related to dependencies integrations
https://github.com/dagger/dagger/blob/aa9fc534b0c750f71f26ae0f288c6369fa030d06/core/integration/module_typescript_test.go#L623

Do you have a precise example in mind of a test that I could write??

Yeah I can see have tests for typescript module loading - but clearly a case is missing, for this bug to not have not been caught by the test - we should add that case.

Yeah I can see have tests for typescript module loading - but clearly a case is missing, for this bug to not have not been caught by the test - we should add that case.

Okay I made a test to repro that case, I'm checking if it works locally


Consider using Enum for Nibbles type

From @pipermerriam:
Currently, the trie implementation uses a bytes type for the sequence of nibbles, with the stated validation rules that each byte may be no larger than 15.  This requirement doesn't appear to actually be enforced and it would be ideal if the type definitions actually constrained the inputs via the type system rather than runtime validation.
I would suggest using an enum.IntEnum for this:
class Nibble(enum.IntEnum):
    NIBBLE_0 = 0x0
    NIBBLE_1 = 0x1
    ...
    NIBBLE_F = 0xf

Nibbles = List[Nibble]

Not only will this give you strong guarantees on anything that needs to take a Nibbles type as an input, but it also lets you have runtime validation for free
def bytes_to_nibbles(value: bytes) -> Nibbles:
    return [Nibble(v) for v in value]  # will raise a TypeError if the value isn't part of the Enum)

Hey, I would like to tackle this problem as my first.

@abdulqgg I have assigned you to this, let us know if you have any questions. Thanks!

@lightclient Where can i find the trie implication that i will need to edit for this issue?

@lightclient Where can i find the trie implication that i will need to edit for this issue?

https://github.com/ethereum/eth1.0-specs/blob/master/src/ethereum/frontier/trie.py
Unless you mean something else?

@SamWilsn @lightclient I think the Nibbles datatype mentioned in the issue will eventually have to serve as the key for various nodes in a trie. I don't think it is right to have the key of a dictionary be a list.
We should however, just implement an explicit byte <= 0xF check on all bytes that are supposed to be nibbles

What is the status?
#281 PR was abandoned, but I could give a hand if everyone is OK with the initial request.


Rustbot isn't responding to label requests

See here:
https://github.com/rust-lang/rust-clippy/issues/5875#issuecomment-670781557
Labeling is enabled for Clippy, but rustbot isn't responding to my label request. Am I doing something wrong?

@rustbot modify labels to question

Looks like it's working here

Whoa, when I asked rustbot to modify the labels, it created a new issue. That, or something's going awry with GitHub.

@Mark-Simulacrum Do you know what's going on here?

I don't, do you have a link to the issue it created?

Yes, it was the original issue I had opened: #753
I'll close #753 so there isn't a duplicate.

Fixed.


Colorbar of the scatter shows wrong label

Sample notebook code:
import holoviews as hv
import numpy as np
hv.extension('matplotlib')

%opts Scatter [color_index=2 colorbar=True]
hv.Scatter(np.random.rand(100, 3), vdims=["y", "c"])

Colorbar should be labeled "c", because that's the name of the column providing values for color. However, the label is "y".

Thanks for the bug report.


LeakCanary reports a memory leak in Session object

From the logs:
08-06 15:15:03.418   8729-11967/LeakCanary﹕ * GC ROOT static com.uservoice.uservoicesdk.Session.instance
08-06 15:15:03.418   8729-11967/LeakCanary﹕ * references com.uservoice.uservoicesdk.Session.context
It's interesting because it doesn't look like the context is being kept in a static object, which would be a source of memory leaks. Please investigate.

The Session is a singleton and is intended to be kept in memory. It holds onto a Context in order to have access to SharedPreferences. Is this a problem?

Yes. Context should not be kept as static member variables. Please see the below stack overflow links.
http://stackoverflow.com/questions/3346080/android-references-to-a-context-and-memory-leaks
http://stackoverflow.com/questions/28457499/using-context-as-a-member-variable-in-all-classes
From: Austin Taylor
Reply-To: uservoice/uservoice-android-sdk
Date: Thursday, August 20, 2015 at 11:58 AM
To: uservoice/uservoice-android-sdk
Cc: Caleb Tseng
Subject: Re: [uservoice-android-sdk] LeakCanary reports a memory leak in Session object (#198)
The Session is a singleton and is intended to be kept in memory. It holds onto a Context in order to have access to SharedPreferences. Is this a problem?
—
Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fuservoice%2Fuservoice-android-sdk%2Fissues%2F198%23issuecomment-133119244&data=01|01|calebt%40microsoft.com|09f3ec87dc954c37e3d508d2a991487c|72f988bf86f141af91ab2d7cd011db47|1&sdata=W%2B%2BnUs7s1YdIlbVl0l116r6yP2Suj7xRJxn9lVVNa6c%3D.

I see. I guess we can just pass the context everywhere. 5629cdc

Yeah…We had to change our code to do this as well to solve our memory leaks. It’s the unfortunate way that Android works, I guess.
From: Austin Taylor<EMAIL_ADDRESS>Sent: Thursday, August 20, 2015 2:30 PM
To: uservoice/uservoice-android-sdk<EMAIL_ADDRESS>Cc: Caleb Tseng<EMAIL_ADDRESS>Subject: Re: [uservoice-android-sdk] LeakCanary reports a memory leak in Session object (#198)
I see. I guess we can just pass the context everywhere. 5629cdchttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fuservoice%2Fuservoice-android-sdk%2Fcommit%2F5629cdc249969fdfbec4701253c9f5db7edca8f2&data=01|01|calebt%40microsoft.com|21594e8401c24415669b08d2a9a68cca|72f988bf86f141af91ab2d7cd011db47|1&sdata=%2F6sb3Sy%2Fsfo5XKSvyQvLLlKV6xLELWId1yHrD7Mm2zk%3D
—
Reply to this email directly or view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fuservoice%2Fuservoice-android-sdk%2Fissues%2F198%23issuecomment-133184515&data=01|01|calebt%40microsoft.com|21594e8401c24415669b08d2a9a68cca|72f988bf86f141af91ab2d7cd011db47|1&sdata=yqfWSUOMCKdQZ1PZOHjtV7m%2BB3BqWK1fo47NM5yjcVI%3D.


Serial Number always returns '42' for Automation Switches

The Serial Number field for all Automation Switches always returns 42 but normal Switches do get their unique number after the SW prefix. Looking at the code I really can't figure out what's causing this.

Found the culprit. Commit 7001109a55eec3ab9807b52cb27d98f9a82045f8 fixes this issue but the released version 3.2.1 doesn't include it.


Messages in a conversation are not loading

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

I can't scroll upwards past these messages, when on relay.cc they are viewable.
Then, reloading the page and now those messages aren't loading.

Expected Behavior
Messages prior to 10:47 in the screenshot load. In relay.cc, the full message history of me and skydao.eth are viewable.
Steps To Reproduce
Only one example for now.

Tried multiple page reloads, but then reloaded an hour later and it was fixed.


Solutions?

Thank you so much for the course!
Is there any link to the canonical solutions?

Just so you know: I've had to make a surprise house-move, but I definitely haven't forgotten your email!

I don't ask for a special personal attention:) I just check 'answers' branch from time to time.


duckstation: 0.pre+date=2021-12-16 -> 0.pre+date=2022-01-18

Motivation for this change
Duckstation development is on indefinite hold (the repository got archived). Even if nothing noteworthy changed, this is going to be the latest version for a while.
Things done

I don't know whether it makes sense to add -DCMAKE_BUILD_TYPE=Release, the build guide uses it.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I like what you got! Good job!


Add new "Architectures" field

Codecov Report

Merging #10 into master will decrease coverage by 4.69%.
The diff coverage is 46.93%.

@@            Coverage Diff            @@
##           master      #10     +/-   ##
=========================================
- Coverage   86.75%   82.05%   -4.7%
=========================================
  Files           7        7
  Lines         370      418     +48
=========================================
+ Hits          321      343     +22
- Misses         25       47     +22
- Partials       24       28      +4

Impacted Files
Coverage Δ

manifest/rfc2822.go
76.03% <46.93%> (-7.48%)
:arrow_down:

Continue to review full report at Codecov.

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

With that latest push we've only got a 3% coverage decrease (@codecov-io didn't update the comment properly :disappointed:).

Ok, with that final push we're down to 2.31%.


bash completion broken with filename choices containing space

Using just master and bash shell, in a folder with
$ ls
 justfile  'justfile (1)'

Trying to autocomplete just -f  offers choices justfile and (1).  It won't autocomplete justfile (1) (neither quoted nor backslash-escaped).  It also won't autocomplete the common prefix justfile.
Autocompleting the just -f '(1)' choice doesn't work either, although I wouldn't expect the resulting command to work since there is no file named (1).

Ahh, that's annoying. The completion scripts continue to be a source of pain 😅


Could you demonstrate how to send a request form central server to charge point

Could you demonstrate how to send a request form central server to charge point
Thanks a lot
Originally posted by @JavanWang in https://github.com/mobilityhouse/ocpp/issues/5#issuecomment-497015625

Sending a request (or Call) to a Charge point works similar as sending a call from a charge point to the central system.
You should use ocpp.v16.ChargePoint.call().
Here can find an example of a Central System that sends a ChangeConfiguration call to a charge point  when the charge points connects.
import asyncio
import websockets
from datetime import datetime

from ocpp.v16 import ChargePoint as cp
from ocpp.v16 import call

class ChargePoint(cp):
    async def change_configuration(self):
        response = await self.call(call.ChangeConfigurationPayload(
            key="HeartbeatInterval",
            value="10"
        ))

        print(response.status)

async def on_connect(websocket, path):
    """ For every new charge point that connects, create a ChargePoint instance
    and start listening for messages.

    """
    charge_point_id = path.strip('/')
    cp = ChargePoint(charge_point_id, websocket)

    await asyncio.gather(cp.start(), cp.change_configuration())

async def main():
    server = await websockets.serve(
        on_connect,
        '<IP_ADDRESS>',
        9000,
        subprotocols=['ocpp1.6']
    )

    await server.wait_closed()

if __name__ == '__main__':
    asyncio.run(main())

Note the call await asyncio.gather(cp.start(), cp.change_configuration()) in def on_connect(). It is here that cp.change_configuration() is called.


linux-raspberrypi: Only deploy cmdline.txt for the main kernel

When multiple kernels are being built, not all of them can deploy the
same file.
This patch assumes the same command line arguments is used by all kernels.
Related to https://github.com/agherzan/meta-raspberrypi/pull/736

Can one of the admins verify this patch?


Emitted C# from TypeSpec for binary return from REST call throws

Describe the issue or request
With TypeSpec like:
@actionSeparator("/")
@route("imageanalysis:segment")
@post
@sharedRoute
@action("segment")
op segmentFromStream is Azure.Core.RpcOperation<
{
  @doc("The image to be analyzed")
  @body
  imageContents: bytes;

  @doc("The format of the image stream")
  @header()
  contentType:"image/jpeg" | "image/png" | "image/bmp" | "image/gif" | "image/tiff" | "application/octet-stream";

  @doc("The type of segmentation to perform")
  @query("mode")
  mode: segmentationMode;

  @doc("Impl Detail")
  @query("overload")
  overload: "stream";
},
{
  @header
  contentType: "image/png";
  @body
  bytes:bytes
} >;

The emitted C# is:
        public virtual Response<BinaryData> SegmentFromStream(SegmentationMode mode, BinaryData imageContents, ContentType contentType, CancellationToken cancellationToken = default)
        {
            Argument.AssertNotNull(imageContents, nameof(imageContents));

            RequestContext context = FromCancellationToken(cancellationToken);
            Response response = SegmentFromStream(mode.ToString(), imageContents, contentType, context);
            return Response.FromValue(response.Content.ToObjectFromJson<BinaryData>(), response);
        }

And the response.Content.ToObjectFromJson<> throws because the response is not JSON...
Describe your ideas for solutions
The "simplest" would be to detect the return type is BinaryData and short circuit conversion to return:
return Response.FromValue(response.Content, response);

duplicate with https://github.com/Azure/autorest.csharp/issues/3682


Create separate file for How to Contribute

Create a separate file for How to contribute. Code for how to contribute is already written in README file, cut it from there and paste it in How to contribute file.

Can you please assign me this issue as a part of Cross Woc. I would be very happy to do it

issue and pr has already been made
issue -> #11 , pr-> #15
your issue means contributing file here. THANK YOU!

Hii @muthuannamalai12 Just a quick message. Half of the work is already done by @Sloth-Panda
So now you just have to remove the How to Contribute part from the README.md file.
Sorry for the inconvenience.

Hii @muthuannamalai12 Just a quick message. Half of the work is already done by @Sloth-Panda
So now you just have to remove the How to Contribute part from the README.md file.
Sorry for the inconvenience.

Ok @rockingrohit9639. No Worries

Issue solved by @muthuannamalai12
PR #17

Issue solved by @muthuannamalai12
PR #17


Enforce ARRAY_SIZE(property) == ARRAY_SIZE(property-names)

This is not enforced by the spec or by dtc, but sounds like something that would improve the dev experience.

What is your expectation for adding an issue here? Do you intend to work on it? Dropping issues here is kind of a dead end. I apply patches people submit for the spec. That's the extent of spec work.
For this issue, I suppose the spec should be explicit about this though that's the whole point of .*-names. Anything the spec says should ultimately be enforced/checked by the tools to be helpful. This could go in either dtc or dtschema. dtc already knows some of the common bindings already. dtschema does too. json-schema may address this issue as constraints depending on other properties' value is on their radar. I would like to define a framework within dt-schema for adding code based checks where we could just drop new python files in to check anything. That's a bit of work. dtb.py already figures out how many entries these bindings have, so we could perhaps add in something there. It's not really setup to properly report errors back if you've noticed the errors with no filenames...

What is your expectation for adding an issue here? Do you intend to work on it? Dropping issues here is kind of a dead end. I apply patches people submit for the spec. That's the extent of spec work.

[...]

This could go in either dtc or dtschema. dtc already knows some of the common bindings already. dtschema does too. json-schema may address this issue as constraints depending on other properties' value is on their radar.

Right. I was not sure where to point this out, I suppose this concerns both the spec and the tools. Probably schema would be the best choice, as it seems to be the one that's responsible for checking anything that's higher level than bare syntax and references.

For this issue, I suppose the spec should be explicit about this though that's the whole point of .*-names. Anything the spec says should ultimately be enforced/checked by the tools to be helpful.

Agreed!


Externalized the hazelcast from the carbon-kernel

Describe the issue:
Remove hazelcast from API-M 3.2.0. As part of it, we externalized the hazelcast from the carbon-kernel.
[1] Mail Thread - Feasibility analysis on fixing Hazelcast vulnerability in API-M 3.2.0

Externalized by https://github.com/wso2/carbon-kernel/pull/3697


Embed json interactive tutorial

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/70934cf53fe934b3e383cdc1e8c8bc9e37e290c7/
Edited pages:

tutorials/demo-json-support-interactive.md
tutorials/katacoda/json-support/finish.md
tutorials/katacoda/json-support/step1.md
tutorials/katacoda/json-support/step2.md
tutorials/katacoda/json-support/step3.md
tutorials/katacoda/json-support/step4.md
tutorials/katacoda/json-support/step5.md
tutorials/katacoda/json-support/step6.md
tutorials/katacoda/json-support/step7.md
tutorials/katacoda/json-support/step8.md
tutorials/katacoda/json-support/step9.md
tutorials/katacoda/learn-cockroachdb-sql/finish.md
v20.1/demo-json-support.md
v20.2/demo-json-support.md


Font Led?

Hello. I can't use Led font, is it normal?  I have it in fonts folder, all the over parameters are OK.
That's very strange, because if i edit the setting directly in chrome console; it's ok!

Isn't it possible to pass it as parameter to fontNumber, fontTitle, fontUnits?

That's very strange, because if i edit the setting directly in chrome console; it's ok!

it's a parameter! and I set it. but it don't work. I don't understand why.  But when i modify it in chrome console, it work!


Integrate Indoor maps

Is there a way to add indoor maps to this wrapper?

Also looking for the same functionality.
Hope this gets imlemented soon.

Hi @JoshuaRoi  and @shmili , Thank you for raising the issue. We have plan to support indoor maps in our roadmap. Will keep you posted on the new functionality.

Any progress on this?


Fix label parse error when / is used

Signed-off-by: kadisi<EMAIL_ADDRESS>fix #2494

Is there a test we can add for this?  I believe the parser is unit tested vary well and it should be simple to write one.

@crosbymichael ok, i will add test  for it

Codecov Report

Merging #2498 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2498      +/-   ##
==========================================
- Coverage   44.76%   44.72%   -0.04%
==========================================
  Files          93       93
  Lines        9555     9562       +7
==========================================
  Hits         4277     4277
- Misses       4585     4592       +7
  Partials      693      693

Flag
Coverage Δ

#linux
48.94% <0%> (-0.04%)
:arrow_down:

#windows
41% <0%> (-0.04%)
:arrow_down:

Impacted Files
Coverage Δ

filters/parser.go
77.05% <0%> (-3.31%)
:arrow_down:

Continue to review full report at Codecov.

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

@kadisi take a look at filters/scanner.go to see if it will also work from isFieldRune. For a change like this it is sometimes a good idea to update the unit tests first, then you can try different solutions easily to see which is best.
Make sure to update the grammar in parser.go as well to include /

Done, @dmcgowan  @crosbymichael PTAL again.

Is there a failure test for the parser to test the boundary case where / is at the beginning or end? The current tests could just pass updating the field rune, but I see you are accounting for more than that.

/retest

You have to quote the value to use a label in a filter expression. How does this change affect the grammar? Please do a proper analysis so we don't get something impossible in there.

@kadisi Since #2494 has been closed, I am going to suggest we close this. If you want to send a PR to update the documentation to show how filters work, that would be most helpful.


chore(package): update file-loader to version 0.10.0

https://greenkeeper.io/

Codecov Report

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

@@            Coverage Diff            @@
##             master     #307   +/-   ##
=========================================
  Coverage          ?   98.83%
=========================================
  Files             ?      119
  Lines             ?     1883
  Branches          ?        0
=========================================
  Hits              ?     1861
  Misses            ?       22
  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 c20222a...36c8719. Read the comment docs.


parse stdout malformed parameters:missing equals when using href=

Here is the troublesome bit of code
`fmt.Println("VaccinateCA")
fmt.Println("---")
for i := 0; i < len(g.Content); i++ {
	if g.Content[i].County == "Contra Costa County" && g.Content[i].Has == 1 {
		fmt.Println("Name: " + g.Content[i].Name)
		fmt.Println("Availabliity Info: ")
		for k := 0; k < len(g.Content[i].Availability); k++ {
			fmt.Println(g.Content[i].Availability[k])
		}
		if len(g.Content[i].ClickURL) != 0 {
			fmt.Println("Make Appointment | href=" + g.Content[i].ClickURL[0])
		}
		fmt.Println("---")
	}
}`

In the terminal it prints
Make Appointment | href=https://www.cvs.com/immunizations/covid-19-vaccine
But when its in Xbar I get the error listed as the title. If I comment out the href part of my code, it works (obviously without the clickable text).

Hi. I'm unable to reproduce this. Using the following script, I get redirected to the CVS site:

@aaronedell what version of xbar are you running?

@aaronedell Are you sure your code is correct? Do g have a proper value? Try removing the | to verify that the URL are indeed fetched as supposed even when running in xbar. (Then you will see the href= argument in the xbar menu)

I mean, I can't be sure its correct but when the same code outputs to the
terminal, the output is correct with the | href=http:... etc. I do know
something is wrong with the way its putting the value in because if I just
hardcode in a URL it works. What happens if it prints | href=null or
something that isn't a proper URL? Mr. Ryer helped me by suggesting I
change the code from fmt.Println("Make Appointment | href="+
g.Content[i].ClickURL[0]) to fmt.Printf("Make Appointment | href=%q\n",
g.Content[i].ClickURL[0])
It works now - but I don't know why. xbar version is 2.0.33
On Mon, Mar 29, 2021 at 2:35 AM Magnus Ihse Bursie @.***>
wrote:

@aaronedell https://github.com/aaronedell Are you sure your code is
correct? Do g have a proper value? Try removing the | to verify that the
URL are indeed fetched as supposed even when running in xbar. (Then you
will see the href= argument in the xbar menu)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/matryer/xbar/issues/667#issuecomment-809231371, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AC2FWJS6MLLZ4EK2YHUKGCDTGBJXFANCNFSM4Z4DLUAQ
.

I think the URL may have contained spaces or something, so quoting it has fixed it. Closing for now. Thanks all.

Looks like a \n issue to me. Echo will add one so my script worked. Printf requires an explicit one. Just a guess!


Error on Orders with missing Shipping Address (Downloadable Products)

I encountered error on running API https://localhost:5001/api/orders/customer/1 with no shipping address for downloadable products. Can you extend the API to include this list -> https://localhost:5001/customer/downloadableproducts?

Yes, currently the shipping address is required even for downloadable products, I will fix that.
As for adding api to query downloadable products, I think it is more suitable for products controller. Something like GET api/products?is_download=true.