Filter by values

Write a function that takes arguments:

a list of dictionaries; each dictionary has the same set of keys of str type
a list of strings that are valid keys for dictionaries from the list

It should return a list with dictionaries that have no duplicates in given keys.
assert filter_by_values([{"x": 1, "y": 2, "z": 3}, {"x": 0, "y": 2, "z": 3}], ["x"]) == [{"x": 1, "y": 2, "z": 3}, {"x": 0, "y": 2, "z": 3}]
assert filter_by_values([{"x": 1, "y": 2, "z": 3}, {"x": 0, "y": 2, "z": 3}], ["x", "z"]) == [{"x": 1, "y": 2, "z": 3}, {"x": 0, "y": 2, "z": 3}]
assert filter_by_values([{"x": 1, "y": 2, "z": 3}, {"x": 0, "y": 2, "z": 3}], ["y"]) == [{"x": 1, "y": 2, "z": 3}]
assert filter_by_values([{"x": 1, "y": 2, "z": 3}, {"x": 0, "y": 2, "z": 3}], ["y", "z"]) == [{"x": 1, "y": 2, "z": 3}]

Resolved with f6fcf5e9b9e2249c6f09f28b26ddec63844b526a


Simple range select causes infinite loop

To reproduce
select ['a'] as i from range(10) tbl(i);

Environment details
Client: shell.duckdb.org
Version: 0.4.0<EMAIL_ADDRESS>Reproducable outside of WASM?
I have tried this same query on DuckDB CLI (master:latest), it completes just fine there

Current live version<EMAIL_ADDRESS>seems to work as expected (Chrome / Safari / Firefox), could you possibly give it another try?

duckdb> select ['a'] as i from range(10) tbl(i);
┌─────┐
│ i   │
╞═════╡
│ [a] │
│ [a] │
│ [a] │
│ [a] │
│ [a] │
│ [a] │
│ [a] │
│ [a] │
│ [a] │
│ [a] │
└─────┘

👍


Regress certifi version

it needs to be not later than 2020.12.5 to be
compatible with azlurm 3.0.1.

Without this PR i am getting build errors like:
`ERROR: Cannot install azure-slurm==3.0.1 and cyclecloud-scalelib==1.0.0 because these package versions have conflicting dependencies.
The conflict is caused by:
azure-slurm 3.0.1 depends on certifi==2020.12.5
cyclecloud-scalelib 1.0.0 depends on certifi==2022.12.07
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip attempt to solve the dependency conflict`

nevermind, we dont want to regress.


use LongAdder for filter concurrent count

Summary

replace AtomicInteger with LongAdder in BaseFilter

why use LongAdder?
This class [LongAdder] is usually preferable to AtomicLong when
multiple threads update a common sum that
is used for purposes such as collecting statistics,
not for fine-grained synchronization control. Under
 low update contention, the two classes have similar
characteristics. But under high contention,
 expected throughput of this class is significantly
higher, at the expense of higher space consumption.

Javadoc:
https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/util/concurrent/atomic/LongAdder.html
LongAdder microbenchmark results
https://github.com/sullis/microbenchmarks-java/blob/main/README.md

I am moving this PR to DRAFT status.   LongAdder needs more evaluation.


[bug] OpenFunctions-v2:

Describe the bug
This is the error I'm seeing when I try to run inference_hosted.py.
Error communicating with OpenAI: HTTPConnectionPool(host='luigi.millennium.berkeley.edu', port=8000)
Is that host down?
If not, how do I access that endpoint?
Screenshots
This is the stacktrace
The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/root/gorilla/openfunctions/inference_hosted.py", line 52, in <module>
    run_conversation()
  File "/root/gorilla/openfunctions/inference_hosted.py", line 39, in run_conversation
    completion = openai.ChatCompletion.create(
  File "/usr/local/lib/python3.10/dist-packages/openai/api_resources/chat_completion.py", line 25, in create
    return super().create(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/openai/api_resources/abstract/engine_api_resource.py", line 155, in create
    response, _, api_key = requestor.request(
  File "/usr/local/lib/python3.10/dist-packages/openai/api_requestor.py", line 289, in request
    result = self.request_raw(
  File "/usr/local/lib/python3.10/dist-packages/openai/api_requestor.py", line 619, in request_raw
    raise error.APIConnectionError(
openai.error.APIConnectionError: Error communicating with OpenAI: HTTPConnectionPool(host='luigi.millennium.berkeley.edu', port=8000): Max retries exceeded with url: /v1/chat/completions (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fd776bd6dd0>: Failed to establish a new connection: [Errno 111] Connection refused'))

**Proposed Solution**
If you want to suggest a proposed solution or an idea for one?

**Additional context**
Add any other context about the problem here.

Thanks for raising this @shraddhasun This should now be fixed?


Created using Colaboratory

Module 02 More Gradient Boosting
Overall: Commit at least part of the assignment.
Feedback:  Push yourself to at least get a baseline score.
Do more feature engineering.

"Talking" through your code is helpful to those reading your code and also your future self.
Use distplots 📊to see what your distributions
Add one feature at a time

Use xgboost for gradient boosting.

Give it a try 🚀

Submit new predictions.

Try to submit so you can see your private vs public score 🎼

Commit your notebook to your fork of the GitHub repo.

✅


Issue: Achieve different authorization in different HTTP HandlerFunc

Description
The current implementation of the JWT middleware applies the same authorization logic to all route functions under a specific route prefix. This approach lacks the flexibility required in production environments where different API endpoints often need different authorization checks.
Current Behavior
The registered Authorizator is called before the execution of any route function that uses the JWT middleware. This results in uniform authorization logic across all functions under the same route prefix.
Desired Behavior
Implement a mechanism to apply different authorization logic to different HTTP handlers, allowing for more granular and specific access control. For example:

For an endpoint that operates on a document, check if the current user has permission to edit that specific document.
For an endpoint that updates a group name, verify if the current user has the appropriate group admin role.

The method I can think of is to construct multiple JWT middleware objects, one for each authorization logic. These JWT objects would have the same Authenticator, PayloadFunc, and IdentityHandler, with only the Authorizator being different. Then, use the corresponding JWT middleware object for specific route functions. I think this approach looks a bit messy. Is there a more elegant way to implement this requirement?

@bodhisatan @justlorain please take a look

A possible solution is to create a simple custom middleware for grouped access control, refer to https://github.com/appleboy/gin-jwt/issues/25:
func GroupAuthorizator(group string) app.HandlerFunc {
	return func(ctx context.Context, c *app.RequestContext) {
		switch group {
		case "product":
			usr, _ := c.Get(JwtMiddleware.IdentityKey)
			fmt.Println(usr.(*models.User).UserName)
			c.Next(ctx)
			return
		default:
			JwtMiddleware.Unauthorized(ctx, c, http.StatusForbidden, JwtMiddleware.HTTPStatusMessageFunc(jwt.ErrForbidden, ctx, c))
		}
	}
}

auth := r.Group("/product", mw.JwtMiddleware.MiddlewareFunc(), mw.GroupAuthorizator("product"))
auth.GET("/get", handler.GetProduct)

However, this approach doesn’t solve the fundamental issue with the JWT middleware. Another solution I thought of is to perform access control inside the Authorizator by using the FullPath or Path method of app.RequestContext:
Authorizator: func(data interface{}, ctx context.Context, c *app.RequestContext) bool {
	group := strings.Split(c.FullPath(), "/")[1]
	switch group {
	case "product":
		if v, ok := data.(*User); ok && v.UserName == "admin" {
			return true
		}
	case "account":
		if v, ok := data.(*User); ok && v.UserName == "account-admin" {
			return true
		}
	}
	return false
},

I hope this helps solve your problem.

BTW, if you want more fine-grained access control, you can consider using the casbin middleware supported by Hertz.


 missing from dbReader.xsl Template

Thanks for this utility. I believe the <note> element needs to be added to the following template in dbReader.xsl.
<!-- TO DO: MAY NEED TO DISTINGUISH section -->
<xsl:template match="*" mode="topic.body.content.in">
<xsl:apply-templates select="ackno|address|anchor|annotation|
bibliolist|blockquote|bridgehead|calloutlist|caution|classsynopsis|
cmdsynopsis|constraintdef|constructorsynopsis|destructorsynopsis|
epigraph|equation|example|fieldsynopsis|figure|formalpara|
funcsynopsis|glosslist|important|indexterm|informalequation|
informalexample|informalfigure|informaltable|itemizedlist|
literallayout|mediaobject|methodsynopsis|orderedlist|
para|procedure|productionset|programlisting|
programlistingco|remark|revhistory|screen|screenco|
screenshot|segmentedlist|sidebar|simpara|simplelist|synopsis|
table|tip|variablelist|warning"
mode="topic.body.child"/>
</xsl:template>

Hi, thanks a lot for the feedback. Probably you are right about the missing element, could you provide any document containing the  element to compare the conversion before and after your proposed change?

Here is a test file that exercises the admonitions. I changed the extension so that it could be uploaded to github
Admonition.adoc.txt

Thanks for your time, I will release soon a new version with the fix.

Sounds good. Thanks

From: jmourelos<EMAIL_ADDRESS>Sent: Monday, October 17, 2016 9:35 AM
To: jmourelos/doconv
Cc: lakeward; Author
Subject: Re: [jmourelos/doconv]  missing from dbReader.xsl Template (#3)
Thanks for your time, I will release soon a new version with the fix.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jmourelos/doconv/issues/3#issuecomment-254243701, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABwuqMlVI_0BxUuUjy9y5IXDeAroU4THks5q05XMgaJpZM4KQWYu.


Switch to using steady_clock for robot control loop

http://www.boost.org/doc/libs/1_53_0/doc/html/chrono/reference.html#chrono.reference.cpp0x.system_clocks_hpp.steady_clock
This avoids issues when the system clock jumps

Also need to pass current time/duration to read functions so they can compute time difference


containerd builds for arm64

There are prebuilt binaries for amd64. As arm64 grows in popularity for building micro Kubernetes clusters, it would be nice to be able to pick arm64 builds of containerd instead of needing to install docker, or spend a lot of time building containerd on arm.

We had been in discussions with using CNCF resources for other architectures (see #634) but seems like there were some hurdles with having to use gitlab to sync with our github repo so remote gitlab-runners on these other architectures could then run CI.
This is similar to the #2996 where we have the same issue for either ppc64 or s390x; to have official release binaries, we need actual hardware access running CI to validate these platforms in the same way that Travis provides for amd64. Of course for ppc64le specifically, it does look like that support did get turned on in upstream travis (it was worked on with IBM for enterprise), so I guess we could add ppc64le today, like Kata containers already did.

It seems there were some hurdles with finding a suitable ARM CI resource to build and test contianerd. I had also try to run build and test of containerd using drone, and then encountered the same problem with https://github.com/containerd/containerd/issues/2901#issuecomment-453782136 due to container privileged problem. So, I think maybe what we really need is an real arm64 resource.
Fortunately, the Openlab can provide ARM resource pool, and these resources can be provided to the open source team, just like the K8S kind project  already did.
I try to complete the POC on containerd arm build. Current the containerd has been built and tested in Openlab arm resource pool successfully, see here, and the build results (including make.txt, make_test.txt, and binaries ) can be found in here.

So do you think it's a possible way to support ARM release? If it is possible, I'm very interested and glad to be a liaison between containerd and Openlab, to complete the containerd ARM build and test stuff on Openlab officially.

@estesp @Yikun @stevvooe @vielmetti Is there some progress outside of this PR? I would love to redeploy my Rock64 cluster with containerd and provide feedback!

@wilmardo For now, The support for containerd arm64 build has been added the OpenLab, there are two way to trigger:

Periodic  job: The periodic job had been enabled, now you can see the periodic containerd-build-arm64 job for containerd/containerd in OpenLab dashboard [1], the arm build job of containerd/containerd master branch will be ran everyday.

PR triggered job: you can also do some test on PR triggered in repo theopenlab/containerd [2], the demo like PR [3],

If we want to move the job from theopenlab/containerd into containerd/containerd officially, I'd like to help it. : )
[1] http://status.openlabtesting.org/builds?job_name=envoy-build-arm64
[2] https://github.com/theopenlab/containerd
[3] https://github.com/theopenlab/containerd/pull/3

@Yikun please add a WIP PR to show what the move the job from theopenlab/containerd into containerd/containerd officially would look like.

@dims thanks for your suggestion. the WIP PR is here: https://github.com/containerd/containerd/pull/3242

I'm not sure if it fully solves the issue, but now that we have the OpenLab CI running, is there any publish step happening there for easy access to binaries? The issue will be that at release time, I assume we won't have a consolidated way to deliver multiple architectures without some extra steps.

Any updates?

In the meantime I found that the Docker-CE repository holds arm64 builds for the four most popular distros. The package is named containerd.io and can be installed separately from Docker.
The only downside is that the version in the repo seems to lack one version behind.
Not that related to this issue but might be useful for someone looking to run containerd on arm64 :)

ah. that looks good. Thanks for the pointer. :)

I see the issue still open -- anyone driving this forward? Is this still gated by HW for the builder?

It looks like there's an issue with the OpenLab builds, they're failing (for at least the last two weeks) due to what appear to be system issues:
2020-07-28 12:41:31.144555 | ubuntu-xenial-arm64 | + apt-get update
2020-07-28 12:41:31.295697 | ubuntu-xenial-arm64 | Err:1 http://mirror.london.linaro-london.openstack.org/ubuntu-ports xenial InRelease
2020-07-28 12:41:31.295873 | ubuntu-xenial-arm64 |   Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.296033 | ubuntu-xenial-arm64 | Err:2 http://mirror.london.linaro-london.openstack.org/ubuntu-ports xenial-updates InRelease
2020-07-28 12:41:31.296156 | ubuntu-xenial-arm64 |   Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.296312 | ubuntu-xenial-arm64 | Err:3 http://mirror.london.linaro-london.openstack.org/ubuntu-ports xenial-backports InRelease
2020-07-28 12:41:31.296430 | ubuntu-xenial-arm64 |   Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.296584 | ubuntu-xenial-arm64 | Err:4 http://mirror.london.linaro-london.openstack.org/ubuntu-ports xenial-security InRelease
2020-07-28 12:41:31.296699 | ubuntu-xenial-arm64 |   Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.320125 | ubuntu-xenial-arm64 | Reading package lists...
2020-07-28 12:41:31.325907 | ubuntu-xenial-arm64 | W: Failed to fetch http://mirror.london.linaro-london.openstack.org/ubuntu-ports/dists/xenial/InRelease  Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.326198 | ubuntu-xenial-arm64 | W: Failed to fetch http://mirror.london.linaro-london.openstack.org/ubuntu-ports/dists/xenial-updates/InRelease  Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.326474 | ubuntu-xenial-arm64 | W: Failed to fetch http://mirror.london.linaro-london.openstack.org/ubuntu-ports/dists/xenial-backports/InRelease  Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.326790 | ubuntu-xenial-arm64 | W: Failed to fetch http://mirror.london.linaro-london.openstack.org/ubuntu-ports/dists/xenial-security/InRelease  Could not resolve 'mirror.london.linaro-london.openstack.org'
2020-07-28 12:41:31.326949 | ubuntu-xenial-arm64 | W: Some index files failed to download. They have been ignored, or old ones used instead.

as the failing host appears to be part of the node configuration.
I raised this at https://github.com/theopenlab/openlab/issues/563, but it seems that GitHub Actions supports arm64, so perhaps we want to migrate the arm64 builds into the existing GH Actions setup?

@TBBle Thanks for your comment, I submit a pr to change the node set to a stable one.

It seems that GitHub Actions supports arm64, so perhaps we want to migrate the arm64 builds into the existing GH Actions setup?

Not yet, the arm64 is only supported by self hosted runner or some 3rd action (like run-on-arch-action, use qemu to create virtual arm64 env). And I remembered the containerd should be built in vitrual mashine or bare rather than container-like ci.
[1] https://github.com/containerd/containerd/pull/4433

@TBBle fixed now. https://github.com/containerd/containerd/pull/4433

Ah yes, my mistake about arm64. I saw a comment that containerd has nightly arm64 binaries, but I realise now they're cross-compiled (#3829). I had incorrectly assumed that that we had arm64 builders that were building those.

The c6g, m6g and r6g are now GA in AWS for some time. Any chance there will be official ARM64 packages released for 1.4.x?


error running ve/bin/pip bin/srv.py

I was following the README :
$ ve/bin/pip bin/srv.py
ERROR: unknown command "bin/srv.py"

However, bin/srv.py exists:
$ tree bin
bin
├── proxy.py
└── srv.py

0 directories, 2 files

How should I proceed further?

@j75 It's a bug in the documentation of course, it should be ve/bin/python not pip

$ ve/bin/python bin/srv.py
Traceback (most recent call last):
  File "bin/srv.py", line 20, in <module>
    import wttr_srv
  File "/home/marian/public_html/wttr.in/ve/local/lib/python2.7/site-packages/gevent/builtins.py", line 96, in __import__
    result = _import(*args, **kwargs)
  File "/home/marian/public_html/wttr.in/lib/wttr_srv.py", line 13, in <module>
    import wttrin_png
  File "/home/marian/public_html/wttr.in/ve/local/lib/python2.7/site-packages/gevent/builtins.py", line 96, in __import__
    result = _import(*args, **kwargs)
  File "/home/marian/public_html/wttr.in/lib/wttrin_png.py", line 22, in <module>
    from PIL import Image, ImageFont, ImageDraw
  File "/home/marian/public_html/wttr.in/ve/local/lib/python2.7/site-packages/gevent/builtins.py", line 96, in __import__
    result = _import(*args, **kwargs)
ImportError: No module named PIL

However I have installed the python-pil package!
$ aptitude search python-pil
i A python-pil                                - Python Imaging Library (Pillow fork)

@j75 Use pip from your virtual environment for that:
ve/bin/pip install Pillow

A small step forward:
$  ve/bin/python bin/srv.py
Traceback (most recent call last):
  File "bin/srv.py", line 20, in <module>
    import wttr_srv
...
ImportError: No module named pyte.screens

Shouldn't be all these dependencies in requirements.txt?

Actually, they all are mentioned in the README.md file:
If you want to get weather reports as PNG files, you'll also need to install:

    PIL
    pyte (>=0.6)
    necessary fonts

Probably you are right, we should add them to requirements.txt, but they are not there by default because not everybody needs PNG output.

Neither do I, but how could I specify that? I am following the docs that states that all I have to do is to run ve/bin/python bin/srv.py - is there any option to pass?
Please note that in the documentation the part referring to the configuration is located after the part discussing server start.

@j75 Yes, you are right. We should fix this part. I hope that now is everything configured correctly and almost up and running. Please ping me via email (igor@chub.in), I will mail you the last part that is not committed into this repository, and after that you will have a running 1:1 copy of wttr.in at your local site.

The last error message I have now is
from globals import GEOLITE, GEOLOCATOR_SERVICE, IP2LCACHE, IP2LOCATION_KEY, NOT_FOUND_LOCATION, \
ImportError: cannot import name IP2LOCATION_KEY

@j75 It is if you have no IP2location key. This is fixed, please update

I did, and here's what I have now:
  File "/home/m.ion/public_html/tmp/wttr.in/ve/local/lib/python2.7/site-packages/maxminddb/__init__.py", line 41, in open_database
    return maxminddb.reader.Reader(database, mode)
  File "/home/m.ion/public_html/tmp/wttr.in/ve/local/lib/python2.7/site-packages/maxminddb/reader.py", line 52, in __init__
    with open(database, 'rb') as db_file:
IOError: [Errno 2] No such file or directory: '/home/m.ion/public_html/tmp/wttr.in/data/GeoLite2-City.mmdb'

I have installed (on my Ubuntu system) the geoip-database package and thus I have a file /usr/share/GeoIP/GeoLiteCity.dat that I gave as a value to the WTTR_GEOLITE environment variable; however I noticed that in your README.md you are using a GeoLite2-City.mmdb file, maybe in a different format, how could I get one like that ?

Yes, sure, you can download it from here:
https://geolite.maxmind.com/download/geoip/database/GeoLite2-City.tar.gz
(details: https://dev.maxmind.com/geoip/geoip2/geolite2/)

Well, it spite of defining WTTR_GEOLITE variable, it still spits out
  File "/home/m.ion/public_html/tmp/wttr.in/ve/local/lib/python2.7/site-packages/maxminddb/reader.py", line 52, in __init__
    with open(database, 'rb') as db_file:
IOError: [Errno 2] No such file or directory: '/home/m.ion/public_html/tmp/wttr.in/data/GeoLite2-City.mmdb'

OK, I moved the file in the data/ folder then I stumble on other errors :
ImportError: No module named dateutil.parser

Then I performed a ve/bin/pip install python-dateutil and I finnaly reached this error:
  File "/home/m.ion/public_html/tmp/wttr.in/lib/weather_data.py", line 7, in <module>
    from globals import WWO_KEY
ImportError: cannot import name WWO_KEY

Yes, that was my fault. Thank you for reporting it.

python-dateutil was missing in requirements.txt
The WTTR_* environment variables were ignored
if WWO_KEY was not configured, it crashed.

All those problems are fixed now. Now you can use the environment variables,
e.g. you can remove the copy of the Geolite.db from data/ and leave it in the configured place.
Ok, now please:

Update
Create ~/.wwo.key with your WWO key in your home directory, Keep in mind that the key should be configured both for wego and for wttr.in itself (~/.wwo.key)

Create ~/.wwo.key => what is the format of this file? just the key (12345...) or a JSON?
Because I still have errors:
Installing setuptools, pkg_resources, pip, wheel...done.
Traceback (most recent call last):
  File "bin/srv.py", line 20, in <module>
    import wttr_srv
...
  File "/home/pula/public_html/tmp/wttr.in/lib/wttr_srv.py", line 29, in <module>
    os.makedirs(os.path.dirname(LOG_FILE))
  File "/home/pula/public_html/tmp/wttr.in/ve/lib/python2.7/os.py", line 150, in makedirs
    makedirs(head, mode)
  File "/home/pula/public_html/tmp/wttr.in/ve/lib/python2.7/os.py", line 157, in makedirs
    mkdir(name, mode)
OSError: [Errno 13] Permission denied: '/wttr.in'

Now it is fine; it just tries to create /wttr.in in the root directory, because it is configured as the work directory for wttr.in. Keep in mind that it is code for a dedicated wttr.in server which processes millions queries daily, and it needs a separate data volumes for the data. On the live server it is mounted into /wttr.in.
Now you have two options:

Create /wttr.in and make chown USER /wttr.in for your USER under which wttr.in runs
Specify other directory as the workdir of wttr.in in globals.py

Yes, the server starts, but performing a GET @ http://<IP_ADDRESS>:8002/ results in an internal error (500), the log saying
2019-05-15 16:07:29,693 Exception on / [GET]
Traceback (most recent call last):
  File "/tmp/pula/ve/local/lib/python2.7/site-packages/flask/app.py", line 2292, in wsgi_app
    response = self.full_dispatch_request()
  File "/tmp/pula/ve/local/lib/python2.7/site-packages/flask/app.py", line 1815, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/tmp/pula/ve/local/lib/python2.7/site-packages/flask/app.py", line 1718, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/tmp/pula/ve/local/lib/python2.7/site-packages/flask/app.py", line 1813, in full_dispatch_request
    rv = self.dispatch_request()
  File "/tmp/pula/ve/local/lib/python2.7/site-packages/flask/app.py", line 1799, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "bin/srv.py", line 50, in wttr
    return wttr_srv.wttr(location, request)
  File "/tmp/pula/lib/wttr_srv.py", line 200, in wttr
    location_processing(location, ip_addr)
  File "/tmp/pula/lib/location.py", line 225, in location_processing
    query_source_location = get_location(ip_addr)
  File "/tmp/pula/lib/location.py", line 142, in get_location
    city, country = ip2location(ip_addr)
  File "/tmp/pula/lib/location.py", line 111, in ip2location
    if ';' in location:
TypeError: argument of type 'NoneType' is not iterable

@j75 I fixed it, but in your case the IP address can't be geolocated, i.e. you will be sent to Oymyakon

try to query it with some city

Yes, it displays
ERROR: location not found: paris,france
2019/05/20 09:30:56 Get http://<IP_ADDRESS>:5001/premium/v1/weather.ashx?key=02fa7c8f8184a27734357dbd2e3cb&q=paris%2Cfrance&format=json&num_of_days=3&tp=3&lang=en: dial tcp <IP_ADDRESS>:5001: connect: connection refused

So, where is this <IP_ADDRESS>:5001/ set and how could it be modified?

Sorry for the delay. It was an extremely busy week, but it is better now.
This time the error message is correct. The problem is that the caching server (that caches all the queries) is not reachable. because you did not start it. Can you please start it first?
$ ve/bin/python ve/bin/proxy.py

As soon as your own server is up and running and functional,
we will update the documentation and fix all the missing parts. Your experience will be invaluable.
Usually people does not install wttr.in on their own, they just use it, but for that rare users that really need to have their own running copy of wttr.in, it is extremely helpful to have up-to-date and correct documentation. Thank you very much for your time and your efforts

Unfortunately, still doesn't work! :worried:
Here's what the proxy displays:
http://api.worldweatheronline.com//premium/v1/weather.ashx?key=xxxxxxx&q=paris%2Cfrance&format=json&num_of_days=3&tp=3&lang=en
[2019-05-23 20:20:50,936] ERROR in app: Exception on /premium/v1/weather.ashx [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 2311, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1834, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1737, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1832, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1818, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "bin/proxy.py", line 193, in proxy
    content = add_translations(response.content, lang)
  File "bin/proxy.py", line 120, in add_translations
    weather_condition = d['data']['current_condition'][0]['weatherDesc'][0]['value']
KeyError: 'current_condition'

My questions are:

wouldn't it be possible to add the we-lang.go file to the project?
the URL worldweatheronline seems bad (2 double slashes,..),
it seems that worldweatheronline does not offer free keys anymore (I used one that I had once, but it is not valid anymore); on the wego project it is advised to use a a forecast.io account

... however, their configuration file (where the weather provider is chosen) seems to be different from the .wegorc you indicate (yours is a JSON, the other is a simple key=value file), could it work? otherwise, what modifications should be done?

wouldn't it be possible to add the we-lang.go file to the project?

I think that what we will do instead (soon), is that it we will use one own rendering engine,
and so this dependency will become obsolete.

the worldweatheronline URL seems bad (2 double slashes,..),

It does not matter in this case. It should work with double slashes too, even if they look ugly/suspicious

it seems that worldweatheronline does not offer free keys anymore

Please use the URL I've sent you in email.

wegorc configuration

On the server the files looks like:
$ cat ~/.wegorc
{
        "APIKey": "XXX",
        "City": "London",
        "Numdays": 3,
        "Imperial": false,
        "Lang": "en"
}

But the values are being overridden by command line options, so it actually does not really matter what you have here.


BTF illegal structure name

Hello,
I've recently been playing around with RedBPF and had cool results - big thanks for this lib!
Originally I worked with RedBPF v2.3.0, but because I was especially excited about some new features (like the ring buffer) you implemented after the latest release I decided to go bleeding edge and work right off the master branch.
I'm obviously aware that this means a much less stable version, but that's fine for me. However, whenever I build & load my probes with any recent master version (tested commit c864a69e4cb3beac1cdf1d01b07cc3ae89b17368 and 975d87ef92d0ba4a8ae604ed5a6287a622701662) I get the following error messages on startup:
ERROR redbpf::btf: illegal structure name: ____btf_map_VARIABLE
ERROR redbpf::btf: illegal structure name: ____btf_map_OUTPUT
ERROR redbpf::btf: illegal structure name: ____btf_map_CONN_MAP
ERROR redbpf::btf: illegal structure name: ____btf_map_PROBES
...

This error seems to appear for every single map I've got in my probes - one per map. The BTF names match my map names.
Funnily enough, all of my maps load up and work just fine though. It's just a lot of nagging error messages on startup. It appears as if cargo-bpf compiles some structures into the elf with names the loader doesn't expect? Not sure what's wrong.
My cargo-bpf is on the same bleeding edge version as my redBPF libs, so I don't think it's a version mismatch issue?
This does not happen when I roll back to redBPF v2.3.0, but I really like the new features ❤

Considering that everything seems to work just fine, I'd rather think that this is a bug on our end. Considerately pinging @rhdxmr who knows more about this code. I know he's busy with other stuff but maybe has some input.

Hello
I'm glad to hear you enjoy redbpf!
I guess this is a bug that was introduced by #298
I'll fix this soon. Thank you for reporting this issue.


not getting started

File: get-started/index.md
I think we missed some steps to get started....
download docker...
open docker, we are now here

no buttons do anything... ok switch to the getting started tutorial,
now he is here:
https://youtu.be/iqqDU2crIEQ
"start with a base image," he says
" we will start with a node application"
and we are here:

ok, is that VS Code ?
what steps are missing here?
open VSCode, open a terminal, ?
run this line: ?
docker run -d -p 80:80 docker/getting-started
opps that does not work.
Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.
Try the new cross-platform PowerShell https://aka.ms/pscore6
PS c:\users\machine\documents\Test1>  docker run -d -p 80:80 docker/getting-started
docker : The term 'docker' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:2

docker run -d -p 80:80 docker/getting-started

 + CategoryInfo          : ObjectNotFound: (docker:String) [], CommandNotFoundException
 + FullyQualifiedErrorId : CommandNotFoundException

so please fill in the gaps in your getting started...
what do I do before I run this line ?
I have restarted the machine once. should I do it twice ?

Still not getting started:
"To run the Quick Start Guide on demand, right-click the Docker icon in the Notifications area (or System tray) to open the Docker Desktop menu and then select Quick Start Guide. "
this doesn't work either

I tried to do a factory reset and it crashed:

now I have uninstalled and reinstalled and it still crashes the same way

Hi @TraversingAU, thanks for sharing the issue.
It seems the Docker UI-based quickstart guide may have not started because of the error.
From the error, it appears that you may need to enable a setting in your device's BIOS. The documentation link will lead you to https://docs.docker.com/desktop/troubleshoot/topics/#virtualization
In step 3, follow the link to https://bce.berkeley.edu/enabling-virtualization-in-your-pc-bios.html
If after following the steps, the error still occurs, please help file a bug report at https://github.com/docker/for-win
You may also seek support at:

Docker Community Forums
Docker Community Slack
If you have a paid Docker subscription, you can use Docker support.

After the error is resolved, you can manually initiate the UI-based quickstart guide. To manually start the quickstart, please see https://docs.docker.com/desktop/get-started/
You must run the steps in the UI-based guide before running the command, docker run -d -p 80:80 docker/getting-started.
I'll look in to incorporating this information into https://docs.docker.com/get-started/


Handled ErrorResponseException

Modified the code to include ErrorResponseException

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/how-to/rate-limit.md
93
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit e66a565:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/rate-limit.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix semantic changes related to numeric literals

Purpose

$title.

Fixes #33316
Fixes #33318
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Example - 1
public function main() {
    byte b = 0x7F & 10;
}

Example - 2
public function main() {
    byte b2 = 10 & 10;
}

We allow Example - 1 and disallow Example - 2 in the current implementation. With this fix, I have disallowed both cases.

Shall we add refracting suggestions as a sperate PR ?


restore starting position at center

It gets tough now to wrestle more out of it but at least the snake starts in the center again...

Wow, I saw it's possible to use 0xF9C for cx for the masking and use the two jas but thought it will be problematic with the div cl and lds and even though I thought it could be fixed with lds dx, it just meant the next dummy instruction will be problematic, but it turns out I should have just switched cl and dl awesome! Totally slipped by me.

Intrested to know what's your background in assembly..?

Wow, I saw it's possible to use 0xF9C for cx for the masking and use the two jas but thought it will be problematic with the div cl and lds and even though I thought it could be fixed with lds dx, it just meant the next dummy instruction will be problematic, but it turns out I should have just switched cl and dl awesome! Totally slipped by me.

Was struggling with that as well, 0xF9C just isn't suitable for dummy instructions. Also losing 0xA for the MUL in input handling had me worried initially.

Intrested to know what's your background in assembly..?

Intrested to know what's your background in assembly..?

Grew up with the C64 in the 80s with BASIC and assembly then moved on to PCs in the 90s with C and assembly.
Got hooked seeing Second Reality by Future Crew and got involved in the demo scene a bit.
Only did programming as a job shortly in the game industry in the late 90s and working on photogrammetry software in the early 2000s (mostly C/C++ but optimized assembly for performance relevant parts).
Now I'm "old", have too much free time on my hands and do things for fun randomly when my interest gets peeked but still there are very few things that give me more joy than trying to optimize stuff in assembly.
I've seen you're quite young - love it that this kind of coding lives on :)

Wow, I saw it's possible to use 0xF9C for cx for the masking and use the two jas but thought it will be problematic with the div cl and lds and even though I thought it could be fixed with lds dx, it just meant the next dummy instruction will be problematic, but it turns out I should have just switched cl and dl awesome! Totally slipped by me.

Was struggling with that as well, 0xF9C just isn't suitable for dummy instructions. Also losing 0xA for the MUL in input handling had me worried initially.

Exactly!

Intrested to know what's your background in assembly..?

Grew up with the C64 in the 80s with BASIC and assembly then moved on to PCs in the 90s with C and assembly. Got hooked seeing Second Reality by Future Crew and got involved in the demo scene a bit. Only did programming as a job shortly in the game industry in the late 90s and working on photogrammetry software in the early 2000s (mostly C/C++ but optimized assembly for performance relevant parts).
Now I'm "old", have too much free time on my hands and do things for fun randomly when my interest gets peeked but still there are very few things that give me more joy than trying to optimize stuff in assembly.
I've seen you're quite young - love it that this kind of coding lives on :)

Never heard of Second Reality will definitely look that up!

I feel very lucky to have you involved, thanks!

DAMN! Just saw Second Reality. What!

DAMN! Just saw Second Reality. What!

Exactly! Imagine how my mind was blown 30 years ago...

I feel very lucky to have you involved, thanks!

Thanks, my pleasure! I'm happy to have stumbled upon your project - in fact a good friend pointed me to it because he knew it would be right up my alley. And for sure it provided me with hours of fun so far.


Working on Python Bindings

Hi,
we are trying to provide Grappa primitives to Python via boost::python. We are apparently stuck on implementing grappa::run(). Our idea was to expose this through a Python method
def run(function_object)

We have been playing with stack.S; first, we #def'd DYNAMIC_LIB and added -fPICto the compiler flags. The library links successfully, and we are now able toimport` it from python, from where it starts successfully (we are able to get the core and locale ids).
However, when we try to export Grappa::run() and call it from within a Python context we are getting a segfault.
In particular, with the default stack configuration and running with mpirun -n 2 python test.py the first core executes the function we provide to the run() routine, while the second crashes. We have no problems when we execute the program from a single process (-n 1)
Can you give us any hints ?
Thanks in advance

Is it possible to provide a backtrace from the point of the segfault? This will provide more information.

Hi, apparently the problem can be tracked down to linking libGrappa.a from a shared library.
This Gist is a minimal example where the problem occurs. An executable that linking a shared library exporting a symbol that links to Grappa will crash at runtime, if run within a mpirun context.
e.g.:  mpirun -np 2 grappa_exe
here is the result of the command:
  mpirun -np 4 --mca param "mpi_abort_print_stack" ./bin/grappa_exe ompi_info --param mpi all

W0710 16:12:57.559685 17392 Communicator.cpp:264] Adjusting to fit in target footprint: 45908326 bytes

I0710 16:12:57.560355 17392 Grappa.cpp:324]

Footprint estimates:

- locale_heap_bytes: 104932147

- global_heap_bytes: 262324224

- total for Grappa:  170516275

  - global_communicator:    33554432

  - global_rdma_aggregator: 2130180

  - global_task_manager:    33554432

I0710 16:12:57.564333 17392 Allocator.hpp:174] Allocator is responsible for addresses from 0 to 0xfa2c000

I0710 16:12:57.564664 17392 GlobalMemory.cpp:56] Initialized GlobalMemory with 262324224 bytes of shared heap.

I0710 16:12:57.564839 17393 GlobalMemory.cpp:56] Initialized GlobalMemory with 262324224 bytes of shared heap.

I0710 16:12:57.565794 17394 GlobalMemory.cpp:56] Initialized GlobalMemory with 262324224 bytes of shared heap.

I0710 16:12:57.566788 17395 GlobalMemory.cpp:56] Initialized GlobalMemory with 262324224 bytes of shared heap.

I0710 16:12:57.573757 17392 Grappa.cpp:629]

-------------------------

Shared memory breakdown:

  node total:                   1.95451 GB

  locale shared heap total:     0.977257 GB

  locale shared heap per core:  0.244314 GB

  communicator per core:        0.03125 GB

  tasks per core:               0.0156631 GB

  global heap per core:         0.0610771 GB

  aggregator per core:          0.00247955 GB

  shared_pool current per core: 4.76837e-07 GB

  shared_pool max per core:     0.0305393 GB

  free per locale:              0.539333 GB

  free per core:                0.134833 GB

-------------------------

Exiting due to signal 11 with siginfo 0x40001ba428f0 and payload 0x40001ba427c0

Exiting due to signal 11 with siginfo 0x40001ba588f0 and payload 0x40001ba587c0

-------------------------------------------------------

Primary job  terminated normally, but 1 process returned

a non-zero exit code.. Per user-direction, the job has been aborted.

-------------------------------------------------------

Exiting due to signal 11 with siginfo 0x40001ba4d8f0 and payload 0x40001ba4d7c0

--------------------------------------------------------------------------

mpirun detected that one or more processes exited with non-zero status, thus causing

the job to be terminated. The first process to do so was:

  Process name: [[37729,1],1]

  Exit code:    1

--------------------------------------------------------------------------

Let us know if you can give further instructions to provide you with a more detailed trace.
it is also worth mentioning that running the same command line through gdb --args ... wiil result in a successful run (on a single host).

Thanks for the concise example!
Grappa currently makes the assumption that any code called through delegate operations is located at the same addresses on all cores/processes, and this is not true in general for code in shared libraries. Disabling ASLR on your machine might help; I'm not sure.
To make this work properly, we'll either need to find a way to force the Grappa shared library and any code it calls to be loaded at a particular address, or to change delegate ops to index into the global offset table for the shared library instead of passing an address directly. This seems doable, but I'm at a conference right now and won't have time to think about it in detail until next week.

A promising first step: GNU ld has an argument -Wl,-Ttext-segment can be used to specify a default load address for dynamic libraries. This appears to preempt ASLR for the library. So far I've only tried it with a very simple bit of code; the next step is to see if I can try it with your example.
Can you show me how you built your example code? I want to make sure I do the same thing.

Hi,
you can compile the example with the following lines (adjusting the cli params according to your system):
g++ -g -fPIC --std=c++11 -shared -L../grappa_dev/build/Make+Release/system/ -lGrappa -I../grappa_dev/system/tasks -I../grappa_dev/system -I../grappa_dev/build/Make+Release/third-party/include -I/usr/include/mpich -o libpygrappa.so libpygrappa.cpp

g++ -g -fPIC --std=c++11 -L/lib/x86_64-linux-gnu/ -L../grappa_dev/build/Make+Release/third-party/lib/ -L../grappa_dev/build/Make+Release/system/ -L./ -I../grappa_dev/system/tasks -I../grappa_dev/system -I../grappa_dev/build/Make+Release/third-party/include -I/usr/include/mpich -o grappa_exe grappa_exe.cpp -pthread -lpygrappa -lGrappa -lgflags -lglog -lmpi -lc -lz -lm -lutil -ldl -lrt -lboost_system

mpirun -n 2 LD_LIBRARY_PATH=.:../grappa_dev/build/Make+Release/third-party/lib/ ./grappa_exe

we were also able to carry on with our experiments by temporarily disabling ASLR on our testing VMs (although this cannot be considered a definitive solution)

I'm starting to think that any non-c++ bindings would be better off starting from a thin C-compatible layer, but I'm struggling to think how this would work.

I can easily see a CGlobalAddress being an alias for GlobalAddress<unsigned char> and a generic global_malloc(size) (remote_malloc is already unaware of the size of T)
I can easily see how to implement forall() using a C function pointer

forall(CGlobalAddress base, int nelems, void (*callback)(*void)) or void (*callback)(*unsigned char)
`forall(int start, int nelems, void (*callback)(int))

but I am already seeing problems with implementing call; how could the result be fetched back from the callee? what should the return type of the callback be? Should it be a function of a given size N (the number of bytes that should be fetched back) ? Should it always return a single char value? What would be the cost for multiple fetches through call? Should I re-implement call from scratch passing N around?

@bholt did something like this for the Alembic project---he can point you at the code. But he may also have a better idea than the byte-oriented interface too....

For the compiler stuff I did a while back I needed to be able to invoke Grappa from within LLVM, which is, as in Python, much easier to do from a simpler C interface. The C versions of all the functions are in Primitive.hpp (which is only in my compiler branch that we never merged in because it's very far from production-ready). You should be able to lift most of those definitions for your C-interface to Grappa. Of particular note are the various grappa_on_* functions. These explicitly instantiate versions of the delegate::call with different "payload" sizes. The way call works in C++ is that you pass the lambda, which has a statically-sized capture and a function, so we send exactly the right amount of data. For the C interface I just had to pass the function pointer and pointer to the data explicitly and choose the function that most closely matches the size (e.g. variant grappa_on_32_0 sends 32 bytes of data, receives 0 bytes on return...)
All that said, I'm concerned about the idea of calling such low-level Grappa functionality from Python. If you're imagining invoking something like delegate::call via the foreign function interfaces all the time, won't it perform extremely poorly? Seems like all the performance improvement to be had from using Grappa will be long-lost waiting for Python to execute these C calls. If you're going to JIT these from Python and just need a way to get at the underlying functions then maybe it still could stand a chance. But in general I'd expect that a better approach would be to look at finding ways to invoke Grappa at a much coarser granularity. A good example of this is the "Knowledge Discovery Toolkit" which is a Python frontend for the high-performance Combinatorial BLAS, or for that matter NumPy is another example where higher-performance C code is invoked to do batch operations over large arrays. Obviously this approach would require work implementing such a library in Grappa (in C++) and then defining the interface to Python at that higher level. But it seems more likely to pay off in the end.
If you'd like to better explain your imagined use case for the Python bindings maybe we can provide more specific advice.


added RobotString class for working with system properties and string…

…s and created class for working with custom keywords of WebDriver

Example of how I can use CustomRobotDriverElement in my test-framework:
https://github.com/YauheniPo/RobotFramework_GUI_Tests/blob/master/src/main/java/popot/robot/keyword/Element.java
https://github.com/YauheniPo/RobotFramework_GUI_Tests/blob/master/src/main/java/popot/robot/keyword/CustomRobotElement.java


Add an NPBD example

With @emilyzakem and @seamanticscience !

Check it out!
https://user-images.githubusercontent.com/15271942/229952827-bd467cf3-6c9a-427b-8c4b-4702bad54a94.mp4
I added a sinking velocity to detritus here:
https://github.com/glwagner/PeachySea.jl/blob/461aa8a2240a7137ea7a154035347916ab09017e/examples/nutrients_plankton_bacteria_detritus.jl#L34-L39
Since main is now protected I'll need an approval from either @seamanticscience or @emilyzakem  to merge --- so let me know what you think.

A few notes:

N is a dangerous symbol to use, since we often take this to mean any Number of things --- Nx or Nt for "number of x points" or "number or time steps"; or even N² ie buoyancy frequency. If possible we may want to call "Nutrients" by another name...
We don't seem to have any issue resolving the dynamics with a 10 minute time step, so this example / parameters at least does not motivate implicit time stepping.

This looks fantastic! I know it was done relatively quickly, but we should definitely normalize having doc strings for functions dispatched on different tracers - otherwise have to search through the input params to figure out what each one is working on...obvious if you know what you're looking for but could be more explicit, I think.

Any ideas on how to make the code more "self-documenting"? If we use verbose names for tracers, ie detritus rather than D it might help.

Any ideas on how to make the code more "self-documenting"? If we use verbose names for tracers, ie detritus rather than D it might help.
I propose that we work on documentation in a future PR, when we implement this example in the source code (it's a bit fruitless here since its just an example, and we don't have docs yet)...

Agreed, let's revisit this - I'm not opposed to the ::Val{} tags/tracer names being initials/acronyms or whatever, but I think having something like:
@inline function biogeochemical_drift_velocity(bgc::NPBD, ::Val{:D})
#
# Sinking velocity of Detritus pool
#
    u = ZeroField()
    v = ZeroField()
    w = ConstantField(- bgc.detritus_sinking_speed)
    return (; u, v, w)
end

Would make it more obvious and readable.

I wonder if there's a better letter than D. Often detritus refers to just
sinking particulates. But if we resolve multiple  organic matter pools,
some of which are dissolved (thus no sinking), is there a better name? "OM"?
On Thu, Apr 6, 2023, 1:59 PM Jonathan Lauderdale @.***>
wrote:

Any ideas on how to make the code more "self-documenting"? If we use
verbose names for tracers, ie detritus rather than D it might help.
I propose that we work on documentation in a future PR, when we implement
this example in the source code (it's a bit fruitless here since its just
an example, and we don't have docs yet)...
Agreed, let's revisit this - I'm not opposed to the ::Val{} tags/tracer
names being initials/acronyms or whatever, but I think having something
like:
@inline function biogeochemical_drift_velocity(bgc::NPBD, ::Val{:D})

Sinking velocity of Detritus pool

u = ZeroField()
v = ZeroField()
w = ConstantField(- bgc.detritus_sinking_speed)
return (; u, v, w)

end
Would make it more obvious and readable.
—
Reply to this email directly, view it on GitHub
https://github.com/glwagner/PeachySea.jl/pull/3#issuecomment-1499623627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWPMDTVDCNCPLMFZCVBKE3W74VC7ANCNFSM6AAAAAAWTFWA4I
.
You are receiving this because you were mentioned.Message ID:
@.***>

Any ideas on how to make the code more "self-documenting"? If we use verbose names for tracers, ie detritus rather than D it might help.
I propose that we work on documentation in a future PR, when we implement this example in the source code (it's a bit fruitless here since its just an example, and we don't have docs yet)...

Agreed, let's revisit this - I'm not opposed to the ::Val{} tags/tracer names being initials/acronyms or whatever, but I think having something like:
@inline function biogeochemical_drift_velocity(bgc::NPBD, ::Val{:D})
#
# Sinking velocity of Detritus pool
#
    u = ZeroField()
    v = ZeroField()
    w = ConstantField(- bgc.detritus_sinking_speed)
    return (; u, v, w)
end

Would make it more obvious and readable.

Julia docstrings are strings above the function, so I'd recommend something more like
"""
    biogeochemical_drift_velocity(bgc::NPBD, ::Val{:D})

Return the drift velocity for the detritus tracer `D`, which sinks vertically at `bgc.detritus_sinking_speed`.
"""
@inline function biogeochemical_drift_velocity(bgc::NPBD, ::Val{:D})
    u = ZeroField()
    v = ZeroField()
    w = ConstantField(- bgc.detritus_sinking_speed)
    return (; u, v, w)
end

check this out
https://docs.julialang.org/en/v1/manual/documentation/#Writing-Documentation

I wonder if there's a better letter than D. Often detritus refers to just sinking particulates. But if we resolve multiple organic matter pools, some of which are dissolved (thus no sinking), is there a better name? "OM"?
…
On Thu, Apr 6, 2023, 1:59 PM Jonathan Lauderdale @.> wrote: Any ideas on how to make the code more "self-documenting"? If we use verbose names for tracers, ie detritus rather than D it might help. I propose that we work on documentation in a future PR, when we implement this example in the source code (it's a bit fruitless here since its just an example, and we don't have docs yet)... Agreed, let's revisit this - I'm not opposed to the ::Val{} tags/tracer names being initials/acronyms or whatever, but I think having something like: @inline function biogeochemical_drift_velocity(bgc::NPBD, ::Val{:D}) # # Sinking velocity of Detritus pool # u = ZeroField() v = ZeroField() w = ConstantField(- bgc.detritus_sinking_speed) return (; u, v, w) end Would make it more obvious and readable. — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHWPMDTVDCNCPLMFZCVBKE3W74VC7ANCNFSM6AAAAAAWTFWA4I . You are receiving this because you were mentioned.Message ID: @.>

Nice! Or maybe organic_matter. Verbose tracer names is not a terrible idea I think! Let's think about it since we do use the short mathematical notation for other tracers / velocities, like u, v, w, b for buoyancy, etc.


Header Feature

Creating a new header to the web page.
Please assign this to me

go ahead @archanagandhi


sign with Developer ID Certificate

Trying to get hyperdaemon notarized, I got multiple issues like this:
"issues": [
  {
    "severity": "error",
    "code": null,
    "path": "Hyper_Daemon.zip/Hyper Daemon.app/Contents/Resources/app.asar.unpacked/node_modules/fuse-shared-library-darwin/osxfuse/osxfuse.fs/Contents/Extensions/10.6/osxfuse.kext/Contents/MacOS/osxfuse",
    "message": "The binary is not signed with a valid Developer ID certificate.",
    "docUrl": null,
    "architecture": null
  }
]

Notarization is required for electron apps because without it you have to go to Security Preferences to allow launching the app.
Do you have a Developer ID certificate? I'm going to work on a PR that will automate this.

actually, this might be an issue with my code signing setup. I'll close for now unless something changes, since I saw that this fetches the official osxfuse binaries


.gitattributes: Fix syntax for directory

The .gitattributes syntax for the Linguist override is incorrect. Selecting all in a directory requires /**. I got confused between the different syntaxes git and GitHub have for directories.
I tested it locally this time by installing Linguist:
$ gem install github-linguist
$ github-linguist
57.00%  178040     Python
37.30%  116499     C++
2.89%   9037       Shell
2.81%   8762       CMake

Fixes: 546421f31dea (".gitattributes: Ignore generated SMT files in GitHub statistics")

Finally!

Glorious indeed :)


Using more than 30 bits for scale?

Hi. I'm trying to use fixed point on an int64_t type, with 32 bits for the fractional part, so I tried to declare like this:
typedef cnl::fixed_point<std::int64_t, -32> real_t;
but I'm getting the error "attempted operation will result in overflow", with a bunch of templates like "cnl::scale<31,2,S,void>" (with S=int).  Is this this the right approach? Thanks

Can you give examples of the code you've written which results in the error? If you are you using ints (including literals such as 0) then you cannot shift left by >30 bits without overflowing the entire value. You will want to use std::int64_t instead and wrap your literals in  INT64_C, e.g. INT64_C(0).

It looks like it's a problem with the comparison, this code triggers the error:
#include "cnl/fixed_point.h"

typedef cnl::fixed_point<std::int64_t, -32> real_t;

int main(int argc, char** argv) {

	real_t n = 5.0;
	if (n > 0) ;
	return (int)n;
};

due to the "n > 0" part. Isn't there an operator>(int) ?

There is a comparison operator but it first converts 0 to cnl::fixed_point<int, 0> because that's the equivalent fixed-point type for int and 0 is an int.
It then scales the value to match real_t which means converting it to cnl::fixed_point<int, -32>. As you might imagine, that means shifting an int by 32 bits which entirely flushes the values.
The simple fix, as I mentioned is to replace 0 with INT64_C(0) or 0LL: https://www.godbolt.org/z/y8d4HV

Explained in FAQ in #924


Is --privileged flag now required?

I am trying to install the newer docker versions of Community Server for months now with no success. But now I see something new that concerns me. In the new installation instructions found in your website: https://helpcenter.onlyoffice.com/installation/groups-install-docker.aspx, I see that the new installation command is this:
sudo docker run --net onlyoffice -i -t -d --privileged --restart=always --name onlyoffice-community-server -p 80:80 -p 443:443 -p 5222:5222
Is the --privileged flag now required? Every article I've read about this flag said what a bad idea is to use it. It is considered a security risk. Please, try to avoid it if it's possible.

Hello @N3tFX yes, --privileged flag now is required. It needs to our services works well (on systemd).

Hello @N3tFX yes, --privileged flag now is required. It needs to our services works well (on systemd).

Oh, no. That's sad news. If this is the final decision from the developers and can't technically be changed, we can close this issue.

Oh, no. That's sad news. If this is the final decision from the developers and can't technically be changed, we can close this issue.

Effectively you are eliminating the ability to run on Docker Swarm now (as the privileged option is not available there). It may be deprecated, but for convenient PaaS such as Caprover that still rely on this functionality, running onlyoffice community is not possible anymore.

This issue was closed due to no response.


Unable to upload ZIP.

When downloading the zip, and trying to import, I get the following error message:
I am new and completely ignore how this works, I just wanted to use the themes. Am I doing it wrong?

Which theme have you tried to use exactly? The official ghost themes are directly available from the Settings area, where you just need to click on "Use <theme_name>". No need to upload any zip file.

Hey @GreivinV, the native comments feature was added in Ghost 5.9.0. If you Ghost site is older than this, try updating Ghost first, then reinstall the theme.


ctrl+shift+_ is not emmitted

the key sequence for undo in emacs is ctrl + shift + - (- becomes _ on shift)
this sequence is not sent to the terminal, i.e. nothing happens.

+1
is there a definitive list of key sequences that are emitted?
I'm noticing that screen key presses do not emit either (control-shift-A or control-shift+"). It's unclear what the expectation is and if we should just be using a lower level library than what term.js aims to provide.


CORGI-346 fix unprocessed relations loading

This reduces the number of builds we try to process in a day while still staying below our max memory for Redis of 1,200mb. I've also adjust the number of slow workers to 10 to include the throughput.
When redis was crashing last week with a memory limit of 500mb I saw that most redis keys where 500k keys in the slow list. We need to avoid trying to load so many tasks in one hit. I calculated if we load less than 150,000 builds a day, and process 10 builds a minute we should be able to avoid evicting messages from redis.
If we do end up evicting messages from redis SoftwareBuild won't be created and we can try to process them again the next day. After this is merged and we try to reload all builds we should keep an eye on the number of keys in the Redis slow list to make sure it's not not getting too close to the limit.

@RedHatProductSecurity/corgi-devs please revew

Looking at the relations table, we have almost 200,000 unique builds we need to process in total. Given we need to process both slow_fetch_brew_build, and slow_software_composition_analysis tasks on each build that doubles the number of tasks to 400,000. If we leave room for an extra 100,000 tasks, our capacity should only need to be 500,000, which we know we can fit in 500mb. If already know we have trouble at 500mb, so I suggest we go a little higher to 1200mb If we leave 20% overhead to the Redis pod, it's pod limit should be set to 1.5gb. That is less than the 3gb suggested by Justin, but should give us plenty of room to fit all the tasks from the relations table, while still leaving an extra 1.5gb in the OpenShift project to allocate to slow workers.
Increasing slow workers increases our throughput. I calculate we can process 144,000 tasks a day with 10 workers. That is an average of 1 task a minute. If we leave room for 24,000 ah-hoc tasks in a day, we should set our MAX_BUILDS_TO_PROCESS setting here to 120,000. If we want to disable this feature in future, just set it to 0 in the djnago config or associated environment variable.
Unfortunately restarting Redis does not help, the tasks are read from the persistent storage and all loaded into memory again on restart. To be able to use a restart to clear the memory we'd have to remove the persistent storage volume altogether. I think with this setup it's unlikely we'll have any keys (tasks) evicted, but if they are they should slow_* tasks backed up on the slow queue, which have the oldest created time, and never been accessed. They should be picked up again on the next load from the relations table, because we order by created_date when fetching them.
We should be able to get through the entire relations table in 5 days with this setup.


Created two files to deploy the app on heroku

Pls except it

I am cancelling my request. I want to update my repo first.

I am cancelling my request. I want to update my repo first.


Add simple integration test

The config is updated:
Remove [chainNode] from obd/config/conf.ini
add [chainNode] to obd/tracker/config/conf.ini
Tracker has been moved to obd/tracker/ as a independ side project of obd.


CreateUserAsync doesn't return the idToken & refreshToken. Am I missing something?

Sign up with email / password
But the REST API returns it. For now, to get the tokens I am just logging on automatically after the registration is done. Is this expected behavior? Thanks! :)

Yes, this is the expected behavior. Admin SDK does not support user authentication operations (sign in or sign up). The CreateUserAsync() API in this library is meant to create user accounts from an administrative or other backend environment, without a direct involvement of the user. The REST API we use for this purpose (POST /account) allows creating user accounts without any parameters, and does not send back ID token since that doesn't make sense in this context.


Use arm template parameters as sole input for live test environment variables

For sovereign cloud testing, many cloud-specific arm template parameters and environment variables get passed in from an external keyvault source. This makes them hard to manage and track. I plan to move these values into source control, and as a pre-requisite I am consolidating as many values into the arm template parameters as possible (as opposed to a mismash of parameters and env var declarations).

/azp run java - keyvault - tests


[bug]: "Unable to get file attributes for dSYM file at path" if iOS/macOS Product Name contains space

Is there an existing issue for this?

[X] I have searched the existing issues.

CLI Version
0.3.0-dev.18
Firebase Tools version
13.0.1
Flutter Doctor Output
[✓] Flutter (Channel stable, 3.16.3, on macOS 13.6 22G120 darwin-arm64 (Rosetta), locale en-DE)
    • Flutter version 3.16.3 on channel stable at /Users/nils/fvm/versions/3.16.3
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0366e0a3f (3 days ago), 2023-12-05 19:46:39 -0800
    • Engine revision 54a7145303
    • Dart version 3.2.3
    • DevTools version 2.28.4

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/nils/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Community Edition (version 2023.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.3)
    • IntelliJ at /Users/nils/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/222.4345.14/IntelliJ
      IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.85.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] VS Code (version 1.82.0-insider)
    • VS Code at /Applications/Visual Studio Code - Insiders.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (3 available)
    • Nils’s iPhone (mobile) • 00008110-00042D6011D3801E • ios            • iOS 17.1.2 21B101
    • macOS (desktop)        • macos                     • darwin-arm64   • macOS 13.6 22G120 darwin-arm64 (Rosetta)
    • Chrome (web)           • chrome                    • web-javascript • Google Chrome 119.0.6045.199

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Description
I set up my macOS app with flutterfire configure. However, I'm unable to build my app because I get the following error message:
Unable to get file attributes for dSYM file at path "/Users/nils/Projects/sharezone/sharezone-app/app/build/macos/Build/Products/Debug-dev/dev Sharezone.app.dSYM/Contents/Resources/DWARF"

When I rename my Product Name to "devSharezone", it works.
Steps to reproduce

Create a new Flutter macOS (or iOS) app
Open XCode
Go Build Settings
Search "Product Name"
Enter "dev Sharezone" (ensure you have a space in the name)

Expected behavior
The build should work.
Screenshots

In this screenshot you can see it generated the "dev Sharezone.DSYM" but tries to do something in "dev". This is the Finder at the location build/macos/Build/Products/Debug-dev.
Additional context and comments
No response

@nilsreichardt - could you give me steps for a repro, please? I'm going to get a release out this week so would be nice to get this fix in for it 👍

I have already included steps to reproduce in the issue. Let me know if this isn't clear enough - in that case I would make it more detailed 👍

@nilsreichardt  It isn't clear to me. I have no idea the name of the project or what setup made this occur.

Hmm, I can't reproduce it in a small project. I'm only able to reproduce it in our Sharezone repository when replacing dev_Sharezone with dev Sharezone as "Product Name" in Xcode.

Since I can't reproduce it in a fresh app (flutter create name), I'm going to close this issue.

Getting the same problem after adding crahlytics.

I am facing this issue right now. Can't build my App in dev flavor. Production flavor works fine.

I don't have a space in the path or the name.

I don't have a space in the path or the name, but i just noticed that the path is messed up because of the prefix name of the App when in debug mode.
build/ios/Debug-dev-iphoneos/[DEV]/Contents/Resources/DWARF.
Thanks a lot for the hint!

In what way the path is messed up? What's wrong with the path? @flavioa44


Begin improving API for the TCB

Introduced BlockInfo which contains general block and executed block metadata (BlockMetadata is already a type) that will eventually be used to fill out the fields in LedgerInfo without relying on querying storage
Migrate VoteData to use BlockInfo instead of the partial data it currently has
Clean up some code to unify common patterns

Substantial unit and integration tests exist to ensure correctness.

:umbrella: The latest upstream changes (presumably c08b344aed9085db40ca654fc88083406873f71b) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra delegate+

:v: @davidiw can now approve this pull request

@bors-libra r+

:pushpin: Commit 2a84111 has been approved by davidiw

:hourglass: Testing commit 2a84111c40001262b471fdb83878b45a34208971 with merge d529ef6c2f061e513cbb5f3fef79487d99d969d3...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: davidiw
Pushing d529ef6c2f061e513cbb5f3fef79487d99d969d3 to master...


Typo in cyclic dependencies wiki

In https://github.com/google/guice/wiki/CyclicDependencies
instead of
privat final FooView view;
it should say
private final FooView view;
Thanks!

Fixed thanks!


Cache:clear command throws InvalidArgumentException after updating to Symfony 3.2

Hello,
I'm having a problem after updating to Symfony 3.2. When I run the cache:clear or cache:warmup command our application throws a InvailidArgumentException 'The class or interface "services" does not exist'.
I think it's related to this new feature : https://github.com/symfony/symfony/pull/19507
This new cache warmer receives the AsynchronousBundle serialization.yml and failed to get metadata for it.
php bin/console cache:warmup -vvv

 // Warming up the cache for the dev environment with debug true

  [Symfony\Component\Serializer\Exception\InvalidArgumentException]
  The class or interface "services" does not exist.

Greetings
David

@Davidmattei thanks for taking the time sumbit this pull request. But i'm not able to reproduce this error which serializer did you implement or use? I'm using the SimpleBus\Serialization\ObjectSerializer and that seems fine. Could you provider steps te reproduce or a sample project so i could check this out and test it?

@cmodijk just found the mistake after creating a blank new Symfony project with the following bundles installed :
new JMS\SerializerBundle\JMSSerializerBundle(),
new SimpleBus\AsynchronousBundle\SimpleBusAsynchronousBundle(),
new SimpleBus\JMSSerializerBundleBridge\SimpleBusJMSSerializerBundleBridgeBundle()
This works fine, but in our project we also enabled the serializer option for the frameworkBundle.
app/config.yml
framework:
serializer:      ~
I think this creates a conflict ?

@Davidmattei thanks for checking i will try to test that tomorrow

So symfony reads the [serialization.yml](https://github.com/SimpleBus/AsynchronousBundle/blob/master/src/Resources/config/serialization.yml) and thinks that is configuration for there serialzer i find that strange is there any documentation at the Symfony that defines they Resources/config/serialization.yml path as a configuration file for there serialzer? I'm looking for it but can't find it.

I have a fix ready only problem is that travis is failing i'm first have to fix that before i'm able to merge it and make a new release for you.

@cmodijk thanks for the quick and good support !

@Davidmattei No problem i just tagged the 2.3.0 version with the fix


Amazing work. Are the models going to be made available?

Are the pretrained models for base, large, and xl going to be made available for public use like LongT5? Thanks!

@jmzeng Hi Jiaming
I'm actually not one of the authors. Afaict, google brain will probably no longer open source and may even restrict their researchers from publishing, given chatgpt


Imlement Streaming Responses

ClaudeRequest should have a stream: Option<bool> in order to stream responses from Claude.

not a high priority


Missing Matplotlib in latest version of Python

Installing with Python 3.11.3 and the latest gmsafir and gmsh. Tried to start gmsafir.py, but would close straight away.
Seems Matplotlib was missing , had to install it manually, then it was all fine, but could cause confusion for other users.

thanks for your feedback. We need indeed to update the install guide version

I can help with that if you like, checking readability,grammar etc, Might be easier for me to check it as I have a bit more distance and can see it with new eyes.

should be ok now in the new install guide version


Editor inspector - issue when selecting things from the 'Resources' tab

There seems to be an issue when selecting something such as a '.scene' file, '.xml' files, etc from the 'Resources' tab. The editor inspector shows the properties for the selected file from the 'Resources' tab but only for a split second until it disappears again, making it impossible to be able to properties for the selected file. I've attached a quick video so you can see what I mean.
https://github.com/rbfx/rbfx/assets/94867124/cee006ca-c4f3-43fd-991f-ba98859fcff0
I'm using "rebelfork-sdk-Windows-msvc-dbg-dll-x64-latest" from Releases.

Does inspector work if you click on scene nodes?
I cannot repro this issue for whatever reason.

Does inspector work if you click on scene nodes? I cannot repro this issue for whatever reason.

The inspector works fine if I click on scene nodes, it's only when clicking on resources that the inspector won't show the properties for the selected resource.

This issue has now been fixed since the last release. Closing.

I'm sorry, I've re-opened this because it hasn't actually been fixed. I now know how you can re-produce the issue. In the Hierarchy, disable "Show Components" and then click on a resource from the "Resources" tab and you'll see the problem happening. I didn't realise it only happened when disabling "Show Components" in the hierarchy.

Thanks, I can reproduce this issue now.

Thanks, I can reproduce this issue now.

 :)


Docs not readable

Documentation is not readable because of broken layout, see screenshot:

@COLABORATI what browser, resolution, etc are you running?

Chromium Browser oin Xubuntu 16.04 on a FullHD Screen. It would be nice if that did not play any role (what usually is the case).

Is your chrome window full screen when this reproduces or is it smaller? I can get a similar look to your screen shot when a shrink down my browser window. Just want to confirm I'm reproducing it the same way as you

When the size of the characters is increased by using the "A" button at the header, this reproduces with any window size of my chrome browser in macOS. Inserting the following CSS code shows a horizontal scroll bar, but I don't know this is the best way to solve this issue.
.markdown-section table {
    display: block;
}

In the Gitbook issue page, they say that the CSS works for online help, but not in PDF.
Tables that are too wide are cut off #1626

Docs have been completely re-written since this time and moved from GitBook to Docusaurus. This is no longer applicable


Fix up the port requirements table

Description

Typha port is only required when Typha is enabled
Group Calico networking options together
Make the intro text more explicit

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

@caseydavenport It is overall very nice except perhaps some minute details, but is there any way we can merge targetPort PR first and then rebase this one off of master LMK

@emanic rebased

Calico networking, cluster of more than 50 nodes

In fact, Typha is required in the configuration that I had described.....

Nah, this isn't true. It's not required, and it's only recommended when using Kubernetes API datastore which wasn't explained in the table and I don't think it should be - we can't say "When using Calico on a cluster of over 50 nodes and configured to use the kubernetes api datastore" whenever we really mean "When typha is enabled". It's not accurate and way too verbose.

This seems fine but I prefer to use consistent terminology vis a vis Typha

Agreed, but that terminology should be "When typha is enabled".

Let us just merge it and we can discuss more later then

Let us just merge it and we can discuss more later then

I'd rather discuss first and get agreement. I also have a few other changes I've realized I'd like to make.


Feature: #182, YAML Config File Generation

What is this PR bringing?
As discussed in #182, this PR will bring the option to generate resource config files in YAML format.

[ ] Dev testing
[ ] Unit tests

Hi @cchunduri , very good work on providing PR. only few minor comments needs to be addressed.
Can you also update the ReadMe file with this change.

Hi @cchunduri
Can you please add a couple of tests (one for Maven and one for Gradle) to make sure that the application configuration is proper and can be successfully bootstrapped?


Standardize ABA RTN (Routing transit number)

Motivation:
Multiple ACH Record Formats require ABA RTN (Routing Transit Number)) commonly referred to as a Routing Number or MICR number for sending and receiving funds.
The ACH record format specification for a routing number is slightly different in each record type.

FileHeader requires 10 digits with a zero left padding in the tenth digit.  (bTTTTAAAAC) for Immediate Origin and Immediate Destination
Entry Details require 8 digits (TTTTAAAA) as the Receiving DFI Identification and 1 digit (C) as the Check Digit.
ADV Entry Detail Records require 9 digits (TTTTAAAAC) as Advice Routing Number and 8 digits (TTTTAAAA) as Routing Number of ACH Operator.
Addenda Record(Returns, dishonored Returns, contested dishonored Returns, COR, refused COR) require 8 digits (TTTTAAAA) as the Original Receiving DFI Identification
Company/Batch Header requires 8 digits (TTTTAAAA) as the Originating DFI Identification.
This proposal is to normalize each of Get/Set functions across all record types. Allowing a consistent API for populating the ACH record. Additionally, when consumers are asked for there routing number, lower left corner of a check, it is this ABA RTN format that they will supply.

The new data type will consistently be as follows:

A string of exactly 9 characters
Consisting of numbers [0-9]
The number is of the form XXXXYYYYC where XXXX is Federal Reserve Routing Symbol, YYYY is ABA Institution Identifier, and C is the Check Digit

Impact:
This will be a breaking change to the current API and documentation. Development should be on a .2 branch of the library. In many cases, functions require an int and will now require a string.
Implementation:
Where possible the RTN number will be exported rather than the underlying NACHA field specification. The library will convert, format, and validate the inputted RTN into the expected field level values for the record type.
Function calls should handle strings of length 8,9,10 where possible:

8 digits(TTTTAAAA)
9 digits (TTTTAAAAC) * expected value
10 digits (bTTTTAAAAC)

Not much utility, but for what it is worth, the first two digits of the nine digit RTN must be in the ranges 00 through 12, 21 through 32, 61 through 72, or 80.

@chris-griffin I wouldn't have added that to the validator but will. We're trying to make the API language interface so that they can just send an RTN and we will handle the rest. Appreciate you chiming in!


How is min_depth and depth_interval determined?

Hello,
Thanks a lot for your excellent contribution! It shows very accurate depth in high confidence regions.
On the application of other datasets (LLFF & Mip 360 V2) I had to determine min_depth depth_interval as well as depth_max etc.
Can you please share how are those values applied in your practice? I see that you put the value in the **_cam.txt but i cannot locate the process to produce it.

Hello, the depth range is determined by the sparse points reconstructed through COLMAP. You can refer to https://github.com/FangjinhuaWang/PatchmatchNet/blob/main/colmap_input.py.


UItextview, Uitextfield, UiSlide do not conform to ValidatableInterfaceelement

I just upgraded to xcode 9.3.
Am getting non conformance to ValidatableInterfacElement protocol
specifically the method:
public func validate<R: ValidationRule>(rule r: R) -> ValidationResult where R.InputType == InputType {
let result = Validator.validate(input: inputValue, rule: r)
if let h = validationHandler { h(result) }
return result
}
i copied and pasted it into the UITextView,UISlider,UITextfield extensions and it will go away.
Is this because you have an extension in an extension?
Should just compile no problems from pods but it does not.

See https://github.com/adamwaite/Validator/issues/93


[stable/spring-cloud-data-flow] Remove support for rabbitmq-ha

Is this a new chart
No

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
rabbitmq chart supports replication since version 0.7.1, there is no longer a need to maintain two dependency in this chart.
Removed rabbitmq-HA suuport.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @sabbyanandan

/assign

before ripping things like this out, we should deprecate them first.
can you provide URL(s) to documentation indicating rabbitmq-ha is deprecated in favor of rabbitmq chart ver 6+? we would need to provide solid references in a deprecation notice and assure people its a 100% drop in replacement, along with references to any migration needs.
if this is to be deprecated, the actual removal would likely occur when we move away from helm 2 and fully helm 3

Hi,
Yes, deprecation notes will be important and moving from helm 2 and fully helm 3 is a good time to remove this.
As of now, the owner of rabbitmq-ha chart has no intention of migrating the chart from helm-stable and will be letting it deprecated with this repo, which will be in the start of May. I'll update when there's more info.


2nd pipeline for MNIST classification, loading, saving and freezing pipelines"

This pull request introduces 1 alert when merging 1de59d7f0793cd6526eeb4c4738389f415e3362e into bfd5768ae842c5ec1e6e388ee89b0d6771c2a3af - view on LGTM.com
new alerts:

1 for Unused import

Comment posted by LGTM.com

This pull request introduces 1 alert when merging a3f764ea00cd9a16721375b361f093caf1eab762 into bfd5768ae842c5ec1e6e388ee89b0d6771c2a3af - view on LGTM.com
new alerts:

1 for Unused import

Comment posted by LGTM.com


Fix VEP column not found bug

Description:
This PR is to fix the intermittent error that was breaking the VEP tool jobs.
The problem is something to do with the following line:
https://github.com/Ensembl/public-plugins/blob/release/100/tools/modules/EnsEMBL/Web/Job/VEP.pm#L155
Modifying the value of $param somehow gets temporarily changed in the stored config.
To fix this, I'm just cloning the $param defined in the foreach to $param_clone and using it instead.
Example ticket that was throwing this error:
http://debug.ensembl.org/Homo_sapiens/Tools/VEP/View?db=core;tl=9QezRSx98mS7heaW-6524731
Related JIRA:
https://www.ebi.ac.uk/panda/jira/browse/ENSWEB-5658

Hi @msmanoj sorry this change doesnt make sense, my $param should always be the same as param_clone so no need to reinitialise $param_clone. it might be something else the issue. we probably need to look into it together if you already have your sandbox setup for it.

my $param should always be the same

Hi @ens-ma7 , I thought the same but for some reason the actual value stored in the conf itself was getting changed temporarily. I tried changing the value of param to 123 and I when I logged the following value:
https://github.com/Ensembl/public-plugins/pull/376/files#diff-3e2121382b5955bcf6c5543e4d9c7231R131
for the next job, the value for param was 123. I can reproduce this on the debug site. Cloning the $param seems to be fixing this.

Sorry @msmanoj  i still dont agree, you might be right but i am not convinced thats the correct solution, it might be something else. its probably best to have a chat about this.

Having gone through what the issue is, i understood where the problem is which is a nasty perl object referencing issue. I suggested a solution which we were not able to test. @msmanoj maybe you can try the temporary solution first and if it works push this instead but if it doesnt i am happy for you to push the temporary fix.

see if something like this works:
sub _configure_plugins {	sub _configure_plugins {
  my $self = shift;
  my $job_data = shift;
my $pl = [];

  # get plugin config into a hash keyed on key	  # get plugin config into a hash keyed on key
  $pl = $self->hub->species_defs->multi_val('ENSEMBL_VEP_PLUGIN_CONFIG');

see if something like this works:
sub _configure_plugins {
  my $self = shift;
  my $job_data = shift;
my $pl = [];

  # get plugin config into a hash keyed on key
  $pl = $self->hub->species_defs->multi_val('ENSEMBL_VEP_PLUGIN_CONFIG');

Hi @ens-ma7 , I tried what you have suggested above and I was still able to reproduce the bug. I guess the current solution I got in this PR will be a good fix for now.

Hi @ens-st3 , am I okay to get this merged?

I think this PR is safe to merge.
The @{$plugin->{params} gets updated when we change $param. And because of shared memory somehow, it randomly is re-used by subsequent requests.
Having a copy of $param will stop it


Bug fix hash join share hash map result not stable

Proposed changes

Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


[SPARK-42057][CONNECT][PYTHON] Migrate Spark Connect Column errors into error class

What changes were proposed in this pull request?
This PR proposes to migrate Spark Connect Column errors into error class.
Why are the changes needed?
Leveraging PySpark error framework.
Does this PR introduce any user-facing change?
No, it's error message improvements.
How was this patch tested?
The existing CI should pass

Merged to master.


assimp: avoid work on #526 with update (resolves #526)

Rather than solving #526 in assimp 3.1.1, this commit updates to the latest assimp version 3.3.1 and applies a backported patch, as all packages then compile on x86 and mips.
I tested with bitbake core-image-ros-world and these two build configurations (x86 and mips):
Build Configuration:
BB_VERSION        = "1.35.0"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "ubuntu-14.04"
TARGET_SYS        = "i586-oe-linux"
MACHINE           = "qemux86"
DISTRO            = "nodistro"
DISTRO_VERSION    = "nodistro.0"
TUNE_FEATURES     = "m32 i586"
TARGET_FPU        = ""
meta              = "HEAD:82ea78bd2f03b6ba4d720595d3a3fbd96e0232f9"
meta-oe
meta-multimedia
meta-python       = "master:ac2a6d2b5d69937577effcb8bb2149651bac0176"
meta-ros          = "assimp:9ff518495caffe7cb6abe461fc6fa0af755bac84"

Build Configuration:
BB_VERSION        = "1.35.0"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "ubuntu-14.04"
TARGET_SYS        = "mips-oe-linux"
MACHINE           = "qemumips"
DISTRO            = "nodistro"
DISTRO_VERSION    = "nodistro.0"
TUNE_FEATURES     = "o32 bigendian fpu-hard mips32r2"
TARGET_FPU        = ""
meta              = "HEAD:82ea78bd2f03b6ba4d720595d3a3fbd96e0232f9"
meta-oe
meta-multimedia
meta-python       = "master:ac2a6d2b5d69937577effcb8bb2149651bac0176"
meta-ros          = "assimp:9ff518495caffe7cb6abe461fc6fa0af755bac84"
[...]
Summary: 1 task failed:
  /home/lukas/work/repositories/github.com/bmwcarit/meta-ros/recipes-ros/mavros/mavros_0.18.7.bb:do_compile
Summary: There was 1 ERROR message shown, returning a non-zero exit code.

There are no build errors with x86, and only one known unrelated build error on mips. The issue #526 is resolved on mips.

bitbake core-image-ros-world was successful with x86 and arm, i.e.,
with these two build configurations:
Build Configuration:
BB_VERSION        = "1.35.0"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "ubuntu-14.04"
TARGET_SYS        = "i586-oe-linux"
MACHINE           = "qemux86"
DISTRO            = "nodistro"
DISTRO_VERSION    = "nodistro.0"
TUNE_FEATURES     = "m32 i586"
TARGET_FPU        = ""
meta              = "master:dd75644db2f662f0d88529a068bbfb599c9790a9"
meta-oe
meta-multimedia
meta-python       = "master:e5681e1dca96d3f313b3b2b346789f22483d7881"
meta-ros          = "assimp:25d97e4bfeb4e2f8b1c12f7ba0a1edc06b65131d"

Build Configuration:
BB_VERSION        = "1.35.0"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "ubuntu-14.04"
TARGET_SYS        = "arm-oe-linux-gnueabi"
MACHINE           = "qemuarm"
DISTRO            = "nodistro"
DISTRO_VERSION    = "nodistro.0"
TUNE_FEATURES     = "arm armv5 thumb dsp"
TARGET_FPU        = "soft"
meta              = "master:dd75644db2f662f0d88529a068bbfb599c9790a9"
meta-oe
meta-multimedia
meta-python       = "master:e5681e1dca96d3f313b3b2b346789f22483d7881"
meta-ros          = "assimp:25d97e4bfeb4e2f8b1c12f7ba0a1edc06b65131d"


Can .value be removed?

Per a discussion, .value is annoying at scale.  Can it be removed?  This is a complicated question, but now is the time to ask it.

It can. But at what cost?


Creating Tooltip using mouserover in xaxis labels/Yaxis labels

Hi Team,
In my current project requirement, i need to populate tooltip  while mouserover in the spots of xaxis labels/Yaxis labels
whether is there any possibility or workaround for it?

You can use the onHover option as demonstrated by the fiddle of @etimberg in #9602 (change onClick to onHover) and then use programatic tooltip to trigger it: https://www.chartjs.org/docs/master/developers/api.html#setactiveelements-activeelements
example of programatic tooltip and hover: https://www.chartjs.org/docs/master/samples/advanced/programmatic-events.html


utf-8 <=> utf-16 conversion needs to be built-in

Currently there is no easy way (that I can find) to convert back and forth between UTF-8 and UTF-16 -encoded strings.
I'm doing this as a workaround: https://github.com/gmlewis/moonbit-pdk/blob/master/pdk/string.mbt
and this: https://github.com/gmlewis/moonbit-pdk/blob/a9777b8b71ff1cf2e77a3cdf95244197d24343fd/pdk/host.mbt#L20-L30
but would like to replace these with standard library calls.

Anyone who is designing a new programming language should watch this:
https://www.youtube.com/watch?v=Ri2NMnSQo4o
and ideally avoid UTF-16 entirely.
Rust and Go both use UTF-8 for a good reason.
If UTF-16 can't be avoided, then maybe a new StringUtf8 type would be nice to have so that users can avoid String as much as possible and only use the UTF-8 variant.

It would be beneficial to include an API that facilitates encoding strings in UTF-8 and writing them to a Buffer, essentially adding Buffer::write_string_utf8.
This enhancement should be straightforward to implement leveraging the @string.String::as_iter function, which returns an Iter[Char]. However, it appears that Buffer is defined within the builtin package, which currently limits its use of @string.String::as_iter.

In the meantime, @peter-jerry-ye pointed me to encoder and decoder here: https://github.com/peter-jerry-ye/jstream
Thank you, @peter-jerry-ye !

Anyone who is designing a new programming language should watch this: https://www.youtube.com/watch?v=Ri2NMnSQo4o and ideally avoid UTF-16 entirely. Rust and Go both use UTF-8 for a good reason. If UTF-16 can't be avoided, then maybe a new type would be nice to have so that users can avoid as much as possible and only use the UTF-8 variant.StringUtf8``String

I think supporting UTF-16 is necessary. In fact, MoonBit was originally UTF-8, but later switched to UTF-16. Because an important backend for our compilation is Wasm/JavaScript, and Wasm's String proposal (including its integration with JavaScript before this experience) and JavaScript's String are both based on UTF-16, which is why we use UTF-16.
But on the other hand, I fully support the conversion method of UTF8 <=> UTF16.


Add division model k-calibration

Add the simpler momentum calibration method proposed by @balerion.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 73 (8.22%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.1%) to 88.934%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sed/core/processor.py
2
15
13.33%

sed/calibrator/momentum.py
4
58
6.9%

Totals

Change from base Build<PHONE_NUMBER>:
-1.1%

Covered Lines:
5015

Relevant Lines:
5639

💛  - Coveralls

Change naming to reflect the actual behaviour.
This is a distortion correction, not  a k calibration method

@balerion would you like to generate the parameters for it? This PR is stuck otherwise


fix: Include Types in NPM bundle (when published)

I just updated to last version (5.4.0) and seen the types folder wasn't here in node_modules.
This PR is kind of a fix of my last merged PR about adding a TypeScript declaration file (#133).
I never published a NPM package so I was missing things around the package.json to make the types available in the bundle published.

Thank you @kevinmarrec


[14.0] ISPN-15144 Protostream 4.6.5.Final

https://issues.redhat.com/browse/ISPN-15144

is there a PR for main branch?

I wanted to get this out of the way first. If you don't want to investigate your changes on 4.7.x for main, I can create it

ready to merge

Merged, thank you

I'm not closing the Jira since maybe we want to have it in the main branch as well?