Unstable depth values for each frame in D415

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 18.04)

Kernel Version (Linux Only)
5.3.0

Platform
PC

SDK Version
2.29

Language
Python 3

Segment
Others

Issue Description
I am measuring pixel depth with depth units set to 100um(i.e., 0.0001). But, the values keep varying for each frame. It varies by 100 - 1200 microns(.1 - 1.2 mm). I have tried using various combinations of presets as well as post-processing filters, some of them reduced the variation range, but still the values varied. My questions are as follows,

Why does the depth value fluctuate? Do you have any suggestions to make it return constant values?
How to measure depth in 10um(0.00001)? I know we can directly set it in the realsense viewer from FW 05.12.1, but that option is out of scope for me. Is there a way to set it from the python code itself?

Here's some additional information,

Tested on surface: Metal(flat machined surface) | white paper | wood
Light source: Sunlight(indoor) .
I have measured depth from both live feed as well as bag files. In both cases, the values fluctuate.
The distance is measured at close range(400-650 mm).
At depth unit 1000um the values vary between 0 to 2 mm. But, for some filters, a few pixel depth values remain constant for a set of frames. However, I am interested only in depth units of 100um or less.
Camera is fixed, facing down to a table. No moving entities in video. Well lit room.

Thanks.

If you have fluorescent lights such as ceiling strip lights in the location that the camera is used in then the lights may introduce noise into the image.  This is because the gas inside them flickers at frequencies that are hard to see with the human eye.  Using an FPS speed close to the operating frequency of such lights may help to reduce disruption.  For some lights it may be 30 FPS and for others it may be 60 FPS.

If the main light source is sunlight rather than artificial lights then this may be subject to more variation from moment to moment than a controlled artificial light source that produces a fixed, stable range and intensity of light that always projects over the same area.
If the surface being observed is a flat metallic surface though and the surface is reflective, there is the possibility that light may be reflecting off the metal's surface in some areas of it and making those areas more difficult for the camera to analyse for depth detail when the reflection is present.
Applying a physical filter such as a linear polarizer over the left and right camera lenses may help to reduce the negative effect of reflections, as described in Section 4.4 of Intel's white-paper document about optical filters.
https://dev.intelrealsense.com/docs/optical-filters-for-intel-realsense-depth-cameras-d400

You could set the depth units in Python like this:

depth_sensor.set_option(rs.option.depth_units,0.0001)
A script in the link below demonstrates an example of this:
https://github.com/IntelRealSense/librealsense/issues/2870

As far as I know though, the minimum depth scale is still 0.0001.  I ran some tests with the depth unit option in the Viewer of SDK 2.36.0 just now.  Values less than 0.00005 became zero, and values greater than 0.00005 were rounded up automatically to 0.0001.

Thanks @MartyG-RealSense As always you are spot-on!
After going through your suggestion and following the links, I came to know that external factors like light source, depth surface, surface texture, etc., have a significant impact on depth quality. But, there are various options available to address them. However, due to delay in availability of required materials, I have yet to access the performance improvements by changing the external parameters.
Regarding depth_unit, I guess the only way to set it to 0.00001 is by upgrading the FW to 5.12+

I'm glad that I could be of help!  I look forward to your test results when you are able to access required materials.
I tested 0.00001 in the RealSense Viewer with the newest <IP_ADDRESS> firmware and it still reset the depth unit value to 0, so I don't think updating your firmware will make any difference in regard to being able to set a depth unit value lower than 0.0001.

Hi @Dilip-Gurumurthy   Do you have any updates that you can provide yet, please?   Thanks!

Hi @MartyG-RealSense
There is an improvement in the consistency of depth values after following your suggestion.
Here's what I did,

Stick polarizing film on both left and right stereo cameras.
Enclose a non-flickering 9w LED bulb(flicker controlled- doesn't flicker even when capturing in slow motion from camera) with polarizing film in-order to have a polarized light source.
Adjusted the films so that the reflections are completely eliminated.

I ran some experiments with varying presets and post-processing filters. Let's say, I take a hundred readings (with depth units set to 0.0001) of the same point(one pixel or more pixels averaged). Earlier the standard deviation was in the range of 50-300 microns. Now it has come down to 30-160 microns. But, for flat surfaces it is always less than 100 microns with or without filters.
My concern is about the slight variations. The range of fluctuation has definitely reduced but small fluctuation is still an issue for me.

For flat surfaces (especially ones with little or no texture), you could try having the IR Emitter enabled so that the projector protects a pattern of semi-random dots onto surfaces in the scene.  The camera can use these dots as a texture source to analyse them for depth detail.
If the strong lighting in the room is not illuminating the flat surfaces strongly, the dot pattern projection may make a positive distance to the camera's ability to read detail from those surfaces.


Small case l rendered as upper case L in markdown editor

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:38:23.027Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-color
ans
0.4.5

python
don
0.7.0

xml
Dot
1.9.2

VBScript
lug
0.0.2

mssql
ms-
1.1.0

csharp
ms-
1.12.1

Steps to Reproduce:

edit a markdown file (*.md)
_l_ rendered as _L_ (italics)

Reproduces without extensions: No

Can you please provide a screenshot and explain the issue in more detail. Here's what I see:

Hi,
Finally realised that the itallic small cap 'L' looked like a capital 'L':

"editor.fontFamily": "Consolas"


Docker images add support for Linux/ARM

I saw that ArtalkGo has pre-compiled binaries for Linux ARM/ARM64, can you add a Docker image support for ARM/ARM64 architecture to the project, thanks.

好主意。目前我们正在进行单存储库重构。之后，我们将在将来的版本中添加此类支持。

i need armxx64 docker version too,pls support arm

We're currently developing a new version of Artalk with improved performance, but it hasn't been released yet.
The development process has been slow as I've been occupied with school exams recently 😢, and we want to make sure the new version is thoroughly tested for stability before releasing it. Rest assured, future releases will still support the arm64 platform. 😎
I apologize for the delay. The old v2.3.0 backend has now been recompiled and made available on Docker Hub at:
https://hub.docker.com/r/artalk/artalk-go/tags
This Artalk Docker image supports both arm64 and amd64 platforms. 😁
If you have any questions, don't hesitate to reach out. We appreciate your support of Artalk! 😉


[Feature request] Support Facebook FLAVA

Name of the feature
In general, the feature you want added should be supported by HuggingFace's transformers library:

If requesting a model, it must be listed here.
If requesting a pipeline, it must be listed here.
If requesting a task, it must be listed here.

Support Facebook FLAVA.
Reason for request
Why is it important that we add this feature? What is your intended use case? Remember, we are more likely to add support for models/pipelines/tasks that are popular (e.g., many downloads), or contain functionality that does not exist (e.g., new input type).
I would like to use Facebook FLAVA to extract embeddings for image-text pairs, images, and text separately.

Hi there 👋  Unfortunately, FLAVA is not yet supported by the optimum library, which is what we rely on for conversions. Please post a feature request there and once it's implemented, we will be able to add it to transformers.js! 🙌


Update package.json

Update Node engine to be >=14 to support additional & newer Node versions. This should fix a warning about using an unsupported engine in projects using Node 20:
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '^14 || ^16 || ^18' },
npm WARN EBADENGINE   current: { node: 'v20.10.0', npm: '10.2.3' }
npm WARN EBADENGINE }

ty!


Trading bot implementation

Kindly assign this issue to me . I can complete it @Akshat111111

I made a pull request for this issue . Kindly verify the files and allow it to merge .Thank you @Akshat111111


Trouble getting server started

I've created the container with az container create....
I open a bash window, tmux attach to start the server, and get the list of worlds that I've created or copied in.
Select a world, set the options, it flashes up a message I can't read then exits tmux & clears the window.
A few seconds later, tmux reports it has exited with code 137.
Also, I've set environment variables to specify the world & password, but it seems to ignore them?
Setting these in the config file seems to "work" (in that it won't let me open a bash window anymore).
In all cases, I can't connect to the server from Terraria.

I managed to get a screenshot of the error message:


Custom action urls

Not sure if this is already supported:
For create and index my API expects a route like:
/authors/11/books

But for show, delete, update it expects:
/books/44

not
/authors/11/books/44

So is it possible right now to do this? If not please consider adding support for overriding urls for specific actions.

In CoffeeScript:
@app.factory 'Comment', ['RailsResource', (RailsResource) ->
  class Comment extends RailsResource
    @configure name: 'comment'
    @configure url: (context) ->
                      if context.post_id
                        "/posts/#{context.post_id}/comments"
                      else
                        '/comments'
]


Add reverse_mode GET param to trigger -R passed to iperf3

This allows people to test downstream to the exporter host from the iperf server.
Test: docker build -t iperf3_exporter:cooper .
Attempts to fix #8

Ok compiles now. We should be able to test.


[AutoPR authorization/resource-manager] typo: authorization/resource-manager/Microsoft.Authorization

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4886

Build output:

dist/com.microsoft.azure.authorization.v2018_09_01_preview/azure-mgmt-authorization-1.0.0-beta-sources.jar
dist/com.microsoft.azure.authorization.v2018_09_01_preview/azure-mgmt-authorization-1.0.0-beta-javadoc.jar
dist/com.microsoft.azure.authorization.v2018_09_01_preview/azure-mgmt-authorization-1.0.0-beta.pom
dist/com.microsoft.azure.authorization.v2018_09_01_preview/azure-mgmt-authorization-1.0.0-beta.jar

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/2628


removeListener function removes all subscriptions for a given listener and eventName

As per Nodejs EventEmitter documentation for removeListener function:

removeListener will remove, at most, one instance of a listener from the listener array. If any single listener has been added multiple times to the listener array for the specified eventName, then removeListener must be called multiple times to remove each instance.

eventEmitter3 removeListener implementation however seems to be removing all the subscriptions for a given eventName and listener function. Following is a sample code to reproduce the issue.
var emitter = new window.EventEmitter();
var fn = function(){ alert("called");}

//add fn listener twice for event1
emitter.on("event1", fn);
emitter.on("event1", fn);

//try removing one listener
emitter.removeListener("event1",fn);

//expected: fn should get called once
//actual: fn doesn't get called
emitter.emit("event1");

Yes, this is expected, that's how it is implemented in eventemitter3.

I see. Interesting. This is a deviation from the behavior documented on nodejs site here and implemented by eventemitter2 library.

Yes, you can even remove all listeners of a given event if you don't pass a function.
ee.removeListner('event');

is the equivalent to
ee.removeAllListeners('event');

Good to know that. Thanks.
I think the point I am trying to make here is that, although it's a rare edge case,technically it's not a 100% 'drop in replacement for existing EventEmitters' as it does modify an existing behavior :)

You are correct, but It has been like this as long as I can remember.
Maybe it makes sense to document this deviation.

I agree.

Want to send a pr?

Sorry for the delay. Thanks for the closure.

No problem.


Improve dummy-blocks for better test data

This is adds a couple of functions that insert various types of dummy blocks (series, text, title, video).
I'm sure there are a lot of things that can be improved upon, as I am still pretty new to rust. Some things might look weird to seasoned rust developers, so I left some comments trying to explain what the code is meant to do/what I think it does.
Closes #601 when eventually completed.

Admin UI CI check fails as the API is different on master. This would be solved by rebasing but since this is a mixed-author PR and I don't want to touch Oles commits, lets just merge. There is no real problem anyway.


Remote port forwarding

Hi!
Kindly direct how to implement server to client port forwarding.
For example:
Remote client with ip x.x.x.x connect to server y.y.y.y:zzzz and this ports forwards to websoket client with u.u.u.u:ssss?

Currently not supported.  However you can do this:

setup a ssh tunnel from client to server via wstunnel, i.e. client:2222 wstunnel to server:22
then you can do reverse tunnel via SSH.

Thank you.
I do it in this way now, but i’d like to do in via websocket.
Thank you.

9 янв. 2015 г., в 6:36, M H Zed<EMAIL_ADDRESS><EMAIL_ADDRESS>написал(а):
Currently not supported. However you can do this:

setup a ssh tunnel from client to server via wstunnel, i.e. client:2222 wstunnel to server:22
then you can do reverse tunnel via SSH.

—
Reply to this email directly or view it on GitHub https://github.com/mhzed/wstunnel/issues/1#issuecomment-69281176.


Update fs2-io, fs2-reactive-streams to 2.4.0

Updates

co.fs2:fs2-io
co.fs2:fs2-reactive-streams

from 2.3.0 to 2.4.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

labels: library-update, semver-minor

Superseded by #3484.


Makefile: Use 'GO_CMD' instead of 'go'

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
I modified Makefile to use 'GO_CMD' instead of 'go'.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Uhm.... the rule doesn't seem to be working https://github.com/kubernetes-sigs/kueue/blob/main/OWNERS

@alculquicondor Is this related?

WARNING: The approve plugin https://github.com/kubernetes/test-infra/issues/7690. Until that is fixed, filters should only be used for the labels key (as shown in the above example).

https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md#filters

Yeah... that looks like it.

Yeah... that looks like it.

We need to fix The whole of OWNER... :(


Convert strings to unicde in post_save voting signal.

Small PEP8 fixes.

r+

Please fix commit message typo

Coverage decreased (-0.01%) to 91.58% when pulling 8f664b59893bb4c247144d84b4c93cffcbd9dbbe on akatsoulas:post_save_unicode into f45e27b7bda0d72ccbe8ed711a34c3d78e9549ec on mozilla:master.


In the Curl command it seems to lag or something else

Windows Terminal version
1.18.2822.0
Windows build number
10.0.22621.2506
Other Software
OS Build Version: 10.0.22621.2506.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.23
Steps to reproduce
Type this in Administrator: Windows Power Shell
" curl github.com "
Expected Behavior
It should at least be a response.
Actual Behavior
Still no response.

Your instructions don't involve telnet. Is there something missing?

Can you check again?

What should I check? Your issue description hasn't changed. It just says:

Type this in Administrator: Windows Power Shell
" curl github.com "

There's no mention of telnet. Did you mean curl?

Ah I see you changed the title... In that case my next question would be:

Does typing the command lag, or does it only lag after you press Enter?
Does it lag less when you type curl.exe github.com (= with .exe)?

It lags after I press Enter. It doesn't lag when I type .exe after curl. The response of curl github.com still didn't come.

I'm confident to say that this also not a Windows Terminal issue, unfortunately.
A terminal is basically like a browser: We only do text rendering and the UI around the text, but we don't really have anything to do with application-specific issues like that. It's almost assuredly not us failing you to show a response, but rather curl not actually printing one in the first place. And we basically don't have any control over what curl does.
 I'd also never use curl, but only curl.exe, because as I said in the other issue, curl is an alias for Invoke-WebRequest, which is not the same as the true curl. The true curl is vastly superior and much faster, so I'd always use that.
If you want to make sure that it's really not due to Windows Terminal, you can try this:

Press Win+R
Type conhost powershell.exe and press Enter
Type curl.exe github.com and press Enter

If it also lags there then this is not a Windows Terminal issue. If it doesn't lag however, then please try it again in Windows Terminal. If it only lags in Windows Terminal and doesn't lag in "conhost", then please reopen this issue.

Oh, okay.

The response of curl github.com still didn't come.

You know what, I think I just realized what you meant with "response didn't come"... You didn't mean that curl.exe hangs right? You just meant that there's no visible output?
Try typing this:
curl.exe -v github.com

-v stands for verbose (= detailed) mode. That way, curl will tell you exactly what it does under the hood. You'll then see:
*   Trying <IP_ADDRESS>:80...
* Connected to github.com (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: github.com
> User-Agent: curl/8.0.1
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Content-Length: 0
< Location: https://github.com/
<
* Connection #0 to host github.com left intact

In other words, you're being redirected. There's two solutions to this:

Use the proper, redirected URL: curl.exe https://github.com/
Use -L which stands for "follow redirects" (The curl documentation is super helpful!)

I'd suggest doing the former in this case, because that way you only do 1 request instead of 2.

I'm confident to say that this also not a Windows Terminal issue, unfortunately.
A terminal is basically like a browser: We only do text rendering and the UI around the text, but we don't really have anything to do with application-specific issues like that. It's almost assuredly not us failing you to show a response, but rather curl not actually printing one in the first place. And we basically don't have any control over what curl does.
 I'd also never use curl, but only curl.exe, because as I said in the other issue, curl is an alias for Invoke-WebRequest, which is not the same as the true curl. The true curl is vastly superior and much faster, so I'd always use that.
If you want to make sure that it's really not due to Windows Terminal, you can try this:

Press Win+R
Type conhost powershell.exe and press Enter
Type curl.exe github.com and press Enter

If it also lags there then this is not a Windows Terminal issue. If it doesn't lag however, then please try it again in Windows Terminal. If it only lags in Windows Terminal and doesn't lag in "conhost", then please reopen this issue.

It lagged.

The response of curl github.com still didn't come.

You know what, I think I just realized what you meant with "response didn't come"... You didn't mean that curl.exe hangs right? You just meant that there's no visible output?
Try typing this:
curl.exe -v github.com

-v stands for verbose (= detailed) mode. That way, curl will tell you exactly what it does under the hood. You'll then see:
*   Trying <IP_ADDRESS>:80...
* Connected to github.com (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: github.com
> User-Agent: curl/8.0.1
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Content-Length: 0
< Location: https://github.com/
<
* Connection #0 to host github.com left intact

In other words, you're being redirected and there's no other response from github (= no text). There's two solutions to this:

Use the proper, redirected URL: curl.exe https://github.com/
Use -L which stands for "follow redirects" (The curl documentation is super helpful!)

I'd suggest doing the former in this case, because that way you only do 1 request instead of 2.

Can you tell me the final URL? Yes, I meant that, and the response which was shown above was correct.

It's here
< Location: https://github.com/

So it's https://github.com/

Huh, can you explain this?


Fix binary package, ask user to verify ulib

Motivation
Since #2267 , CI has failed to build (Linux) binary packages for F*:

The .checked files for the standard library are not part of the binary package, so some tests have to rebuild it. There, some modules such as FStar.Math.Lemmas fail to verify because they need to be verified with different (Z3) parameters than the rest of the library. So, the standard library needs to be reverified before testing.
A straightforward solution would be to include the *.checked files for the standard library into the binary package. However, this would make it grow beyond 200 MB, which is too much to upload to GitHub FStarLang/binaries. So, we need to ask the user to verify the standard library themselves.
However, with the *.cm* files for the standard library (packages fstarlib , fstar-tactics-lib and fstar-compiler-lib) not part of the binary package, trying to verify the Steel standard library makes F* segfault. (Cf. #2209.) Thus, we need to compile those packages and include them in the binary package.

This solution
To solve these two issues, this PR:

Builds and includes the OCaml *.cm* files for the fstarlib, fstar-tactics-lib and fstar-compiler-lib into the binary package.

When testing the binary package, reverifies ulib before doing any tests.

Proposes a new Dockerfile, .docker/package.Dockerfile, to test the binary package on Linux with guaranteed isolation, in the sense that the Dockerfile contains the following several disjoint subimages:

an image with all opam dependencies and build-only dependencies and the F* sources to build the binary package
an image with only the freshly built F* binary package and the opam dependencies of fstarlib, fstar-tactics-lib and fstar-compiler-lib to test the binary package for compilation of OCaml code generated by F*
another such image, but disjoint to test rebuilding fstarlib and fstar-tactics-lib
a no-OCaml image to test the freshly built binary package without OCaml

Although not used by CI yet, I claim that this Dockerfile may be a viable alternative to .scripts/process_build.sh on Linux.

Accordingly updates INSTALL.md with new usage instructions for the binary package: we now ask the user to verify ulib themselves, but not recompile fstarlib or fstar-tactics-lib. (fstar-compiler-lib cannot be rebuilt with the binary package alone, since it needs the F* sources.)

Bumps the F* version number to <IP_ADDRESS>~dev to account for those new instructions in a "before/after" fashion.

With this PR, I successfully produced, tested and uploaded a F* binary package for Linux: https://github.com/FStarLang/binaries/blob/master/weekly/fstar_202104220032_13e688dda1_Linux_x86_64.tar.gz

Based on this PR, I also managed to build, test and upload a Windows binary package (using the existing .scripts/process_build.sh, given Docker Windows images with Cygwin have no longer been working since moby/moby#41058)

Would it make sense to produce an F* nuget package from a binary package now that the windows build works again? I've been doing this for my own purposes and it makes it really easy to start using F* as part of a bigger F# project (you just reference a nuget and add fst(i) files to your project).
I have some powershell scripts for this but I would be happy to rework them so that they fit into your CI pipeline if you think it's worth doing.

Thank you @mateuszbujalski for your proposal. At this point I would suggest to open a separate pull request (or issue) to discuss the specific topic of releasing a NuGet package for F*. Since I believe none of us among F* developers is really familiar with NuGet packages, we would welcome more detailed explanations there. Thank you in advance!

@tahina-pro Sure, I didn't mean doing it as part of this PR obviously :) I'll open an issue and describe my proposal in more detail there.
Thanks a lot for fixing the weekly binary packages for windows! :)

After discussions with @nikswamy , @aseemr and others, we finally decided to include the .checked files of the standard library into the binary packages, so that they are ready to use with no further installation step for the user.
This will solve #2068: ulib will no longer be reverified as part of tests/ or examples/.
As a consequence, due to size issues, we can no longer push F* binary packages to FStarLang/binaries. We will use GitHub Releases instead. I changed .scripts/process_build.sh accordingly and I successfully tested it on Windows; I also added a Dockerized test via .scripts/process_build_linux_docker.sh for Linux, which I also successfully tested. With both of those scripts, I managed to automatically push a release on my F* fork.   So I plan to adapt CI to run those two scripts automatically very soon.
The automated release process is based on Satsuki, which is a Python library to use the GitHub API via command line. To avoid installation/CI issues, I Dockerized that step with a platform-independent Dockerfile at .docker/release.Dockerfile. This is the same approach as the one I chose for EverParse releases.

Since we are going to push weekly packages to GitHub Releases, we will finally switch F* version numbers to YYYY.MM.DD instead of 0.x.y.z. However, I have not reflected that in the version.txt file. In practice, when building a binary package, I automatically generate a tag (if not already there) and temporarily overwrite the contents of version.txt with that version number, which is then picked up by make package. However, I do not push that modified version.txt to master, for two reasons mainly related to CI:

master is protected against direct writes, so I would need to go through a separate CI branch through several steps just to merge that single file
GitHub personal authentication tokens are enough to push releases via Satsuki, but if I were to push a change to version.txt, then I would need another authentication method, such as a SSH key.

I finally managed to automatically push a GitHub release with both Linux and Windows binary packages, all through a CI task.

At this point, none of those binary packages have any html documentation included, since I am no longer running madoko to generate them.

that's great ... Thanks Tahina!
Did you do the Linux package from an Azure VM finally?

I tried, but I failed because such an Azure VM did not have enough memory. Instead, I pushed change 38d014d19696306328daba02f023bc9c35065185 to accommodate older versions of git such as the one on our current CI machine.

Finally, from now on, the version number will be updated every night at the same time as hints and the snapshot, and a release is pushed with the version number recorded in version.txt.


explicit windows platform support

https://github.com/roapi/roapi/issues/65 fixed.

Thank you @tempbottle for the fix, looks good!


Update list

Will this work for private repositories as well? Harvestcraft is.

a bit unrelated, up to you - would be nice if list was sorted alphabetically

List is particularly sorted to my own tracking list of project Migrations. Can do the sorting though. No stragglers left


Entitlement refactoring

The refactoring in this pull request are for

api keys for activations/namespaces (first commit)
enabling packaged actions in rules (second commit)
bulk authorization checks for sequences (second commit)

Also fixed a related authorization issue from issue #1470 (third commit)

LGTM

PG1 797 approved

I dropped the second commit and will submit that as part of a separate entitlement related PR that completes the checks for referenced entities. This keep this pr more focused on the IAM separation.
New pg pending.

new pg2/546 approved.
this should be merged after #1486.

Merged as https://github.com/openwhisk/openwhisk/commit/7976c8d00377a9ca0ec2c64355339c74401014f5.


400 response from airbrake

Airbrake errors are throwing a 400 error in mail_to_hip_chat.
The test email works perfectly.

Thanks for letting me know. I'll take a look.

So, I'm not working somewhere using HipChat now. So it's a little more work to take a look at this. Sorry to leave the issue open so long. Were you able to get any further with it?


Method autocompletion for given argument type

If you are not familiar with common packages functionality, you may waste some time searching the function needed for the particular object type.
In OO languages you can simply look for methods in a pop-up list when you type a dot after an object. For now I don’t know how to do similar thing in Atom/Juno - there are pop-ups only for method names, not for the first argument type.
There is methodswith(typeof(x)) function for a complete list of methods with an argument of given type, but it is too long list and should be filtered with a substring, or sorted by occurrence rate (or both sorted and filtered).
Since there is no dot-method syntax in Julia, it can be probably bound to some hotkey after entering a variable name.

This definitely would be very useful to have, but right now we could only make this work in a very limited number of cases since we basically only have type information for global objects. Once static analysis (with type propagation) is a bit further along this might actually be useful though.

Indeed, this is a part of static analysis.
In general, it would be great to have more static type-related info about variables and methods used, because every time I look into some unfamiliar module, I can't say what method is used on what data type, I can't go to method definition, and this leads to poor understanding of what happens.


Failing To Start Up A Native Base Project

Hello.. Hope Your All fine. I'm trying to start up a project of react native with native base but when i follow the procedures of the documentation on the native base website... but still failed.. because after finishing the procedures and try to run the app using expo from my computer to my phone i get an error that {"Error: Missing app.json. See https://docs.expo.io/ "
Couldn't start project. Please fix the errors and restart the project."} any help on this please.. or any procedures u can give for me to make the app work on expo.. thanks

@jeromesenyonga Share the project structure, I want to know the files and folders present
Also share app.json screenshot

@jeromesenyonga Any updates?

Hello @SupriyaKalghatgi  this it

Looking at your folder structure, I am not sure if this is React Native or Expo app
I would like you to go through docs https://docs.nativebase.io/docs/GetStarted.html

did you try creating a fresh project?
If yes, then share screenshot of folder structure, package.json, app.json

Expo version in package.json and SDK version in app.json should be same

I tried creating a new CRNA project, it had expo 27
Added NativeBase and tried https://docs.nativebase.io/Components.html#anatomy-headref
Worked fine

Any help needed? Can you be a little quick?

Hello, @SupriyaKalghatgi Could you please if you don't mind list for me the steps you followed creating a new CRNA project and then adding in NativeBase Please In comments without directing me to links... i think there it will be more better. Thanks

npm install -g create-react-native-app
create-react-native-app my-app
cd my-app
yarn add native-base --save
npm install @expo/vector-icons --save

NativeBase use some custom fonts that can be loaded using loadAsync function. Check out this expo link.
async componentWillMount() {
await Expo.Font.loadAsync({
'Roboto': require('native-base/Fonts/Roboto.ttf'),
'Roboto_medium': require('native-base/Fonts/Roboto_medium.ttf'),
});
}
You can also refer NativeBase KitchenSink to know how to add these fonts https://github.com/GeekyAnts/NativeBase-KitchenSink/blob/CRNA/src/boot/setup.js
Then run the project

Hello.. @SupriyaKalghatgi Thank You So much... the project now runs very well though this is the warning i'm trying to get in the terminal....

Just asking if it wont bring errors...

Check if you can run this snippet https://docs.nativebase.io/Components.html#anatomy-headref
And please be quick

I will be closing this today


Repo Housekeeping

Things to implement / include in CI prior to 1.0.0 release:

[x] linting - flake8 / black
[ ] mypy (requires #33)
[x] isort
[x] tox
[x] bumpversion
[x] pre-commit
[x] Keep a Changelog

This is mostly finished, #33 covers the last task.


feat: add execution analytics

also ensure JobID is called job_id in all events for easier sql

My overall preference as discussed before is to have a single self-contained event that includes job information, and summary of execution stats related the job, such as execution count, state counts, reasons for failures or rejection, number of nodes, ...

To clarify, how would you envision this information being collected when a job terminates execution?
Would the approach be to query the BucketJobHistory bucket when a job reaches a terminal state—extracting events related to the job and its executions into a single structure? Assuming that’s the intended outcome, how would you prefer to gather fields not present in the job history (like type, count, task engine/publisher/inputs, etc.)? Should we also query the BucketJobs bucket to enrich the event with that information? Similarly, for execution fields not present in the execution history, would querying BucketJobExecutions also be needed?

Unfortunately that is not what is delivered here as we are publishing a dedicated event per execution, which is nice to have, but the job event now doesn't have reasons why the job actually failed. The message field in the job is not the actual reason as discussed.
... please create a backlog item to enrich the job event.

I understand this PR may have missed your expectations, and I want to ensure we are aligned moving forward. Given the questions raised above, I think it would be really helpful if you could provide more specific direction or open an issue outlining the desired structure, fields, and means of collecting the data. This would allow us to hit the mark more accurately and meet the goals you've outlined.

lets focus on getting this PR out and then we can discuss how to aggregate execution stats in the job event next week

Sounds good, I believe I have addressed all feedback.


smslib lightweight?

smslib is a great looking tool, however, for my needs its kinda slow to connect i believe this is due to all of the unnecessary stuff it does, like creating a http server, and a dozen different threads and having code run on those threads. these things seem unnecessary to me in most cases. could a version of smslib be distributed with these things stripped out? maybe it is due to the singleton nature of this application, and maybe its just not possible.
i mention this because i am looking for a library to send sms messages as part of a larger application, my application will be sharing the gsm modem with other applications and having smslib running efficiently means keeping the connection open. i cant keep other applications from connecting to it.

Hi,
Do you mean that you need to frequently start/stop smslib, so you require fast startup times?
I don't know how much faster it could be... The weak link here is the modem communication which is tens of times slower than all the threads you are mentioning.

yes, but at the same time i may not need to send a text for days or weeks.
Looking at the documentation for v4 and v3 there are a hell of a lot of dependencies which really shouldn't be necessary for sending AT commands over TCP/IP or communicating through a web service. Do you agree?
What i am interested in is a core library. Something that will allow me to just send texts through a java library.
This is because for the application i am building i do not need the HTTP server i also don't need an always listening service. I need a minimal solution, an example might be;
    SmsLib example = new SmsLib(host, port);
    example.setNumber(number);
    example.setMessage(message);
    example.send();
    example.close();

+1


Lualine does not redraw when activating caps lock in cmdline.

...thus label [CAPS LOCK] appears only when any character has been typed.
Possible solution:
autocmd CmdlineEnter * redraws
autocmd CmdlineChanged * redraws

No autocmd, just needed to call redraws on activate


Merchant Purchased Item Not Transferred

Describe the bug
When a player attempts to purchase any item from a Merchant, the correct currency amount is subtracted but the item is not added to their inventory, and is also not subtracted from Merchant's inventory.
To Reproduce
Steps to reproduce the behavior:

DM creates a Merchant with some items given some currency value
Player is given "Observer" access to merchant and successfully accesses token
Player attempts to buy an item using either Buy Item or Buy Stack
Correct currency is subtracted from the player's inventory, but item is not added to player's inventory or subtracted from Merchant's inventory

Expected behavior
Currency is subtracted from player's inventory, item(s) are subtracted from Merchant sheet, and item(s) are added to player's inventory
Screenshots

Desktop (please complete the following information):

OS: MacOS Big Sur (DM/Player), Windows 10 (Player)
Browser: Chrome
Foundry Version/Build: 9.249
System: DnD5e 1.5.6
Module Version: 3.4.1

Additional context
Does not seem to matter what type of item is purchased, whether item is custom or default, or whether the player sheet is locked or unlocked. Does still successfully give an error if not enough funds are present.

I'm experiencing the same problem. My differences are:
Windows 10 both GM and players.
Opera, Chrome, Firefox browsers
It's been happening for the last 3 Foundry released versions (all 9.x)


Plastic BRDF

Similar to Mitsuba's rough plastic.

Looking pretty good! Just a few minor things to fix and it's good to merge.
I like the fact that the internal scattering parameter is a float, not a boolean.

Make sure to update the test scenes as well.

Should be good to merge now.

Thank you!


Unarmed Vs Fleshless only uses one attack type (2 unused due to duplicate label).

Description of Problem or Question
Unarmed Vs Fleshless only uses one attack type (2 unused due to duplicate label).
Valkyrie Version
2.4.01

I've had a few reports of strange Skeleton behavour, where when players select spell attacks it comes up as a heavy weapon attack, etc.

#1158, fixed in 2.4.01.


Architecture diagram is not showing in Perspective UI

Hello,
After scanning the resources, when an individual resource is clicked the diagram is flashing in the screen and vanished.
Thanks.

Do you have any filters enabled? This sounds similar to an issue here: https://github.com/awslabs/aws-perspective/issues/46. There will be a fix in the uocoming v1.1.0 release.

Do you have any filters enabled? This sounds similar to an issue here: https://github.com/awslabs/aws-perspective/issues/46. There will be a fix in the uocoming v1.1.0 release.

Do we have any info about  v1.1.0  timelines?

Do we have any info about  v1.1.0  timelines?

Yes filters are enabled for few resources. After removing the filters, it seems working now.

Yes filters are enabled for few resources. After removing the filters, it seems working now.

Thank you, as I said this will be fixed in the new release, which should be released this month. I'm going to close this for now as we're tracking this bug in #46. If you look in the staged column of our public roadmap you can see what fixes are due to be fixed:
https://github.com/awslabs/aws-perspective/projects/2

Thank you, as I said this will be fixed in the new release, which should be released this month. I'm going to close this for now as we're tracking this bug in #46. If you look in the staged column of our public roadmap you can see what fixes are due to be fixed:
https://github.com/awslabs/aws-perspective/projects/2


feat: add support for lts alias

Description:
This PR adds support for lts alias, which is more clear than lts/*.
Related issue:
https://github.com/actions/setup-node/issues/996
Check list:

[ *] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

💎💎

Upd


etwlogrus: Fix event logging not including fields

Signed-off-by: Kevin Parsons<EMAIL_ADDRESS>

Bizarre--I saw that bug and thought I fixed it, but somehow that got lost. Thanks.


koordlet: delete deprecated featuregate performance collector

Signed-off-by: songtao98<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Feature gate PerformanceCollector is deprecated now and is already noticed to the community that we will delete it in version v1.1.0. This PR delete it in source code.
In the charts file of v1.1.0 I already replace it with CPICollector.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/approve

/approve

/milestone v1.1


Split each component color in a file

Do a refactor to split each component color option in a file like I did with tabs. This will make easier to create the lite version too.

+1 love this buddy

@petehouston the version 0.2.3 that I released today all the scss files are separated

Check the master branch

closed on b3db8a4770ace76a7ac733dcdbe68a37445e9d12


The testing should include multistream

The testing should include multiple streams

@efajardo do you have a corresponding VMU run?

@efajardo do you have a corresponding VMU run?

@brianhlin nop but we found out that 5.1.0 broke multistream tpc that was working on 5.0.2 so I wanted to add this so this is caught.

@brianhlin nop but we found out that 5.1.0 broke multistream tpc that was working on 5.0.2 so I wanted to add this so this is caught.

Things look good in this VMU Tests @brianhlin

Things look good in this VMU Tests @brianhlin

Hey @efajardo I spot-checked a few of the outputs and didn't see any X-Number-Of-Streams headers so I think something's up

Hey @efajardo I spot-checked a few of the outputs and didn't see any X-Number-Of-Streams headers so I think something's up

The new VMU tests show the errors we found out on 5.1.0.rc4
osgtest: 2021-01-05 17:10:06: test_465_xrootd_tpc.py:test_02_initate_tpc:57: runuser vdttest -c "curl -A Test -vk -X COPY -H 'Authorization: Bearer MDAyOGxvY2F0aW9uIE9TR19URVNUX1hST09URF9TVEFOREFMT05FCjAwMzRpZGVudGlmaWVyIDk0NWYxNTk1LWIxYTMtNGRlYy1iYWZlLWNiYmNkNzVkODAzOAowMDE1Y2lkIG5hbWU6dmR0dGVzdAowMDJkY2lkIGFjdGl2aXR5OlJFQURfTUVUQURBVEEsRE9XTkxPQUQsTElTVAowMDE1Y2lkIGFjdGl2aXR5OkFMTAowMDM3Y2lkIHBhdGg6L3Vzci9zaGFyZS9vc2ctdGVzdC90ZXN0X2dyaWRmdHBfZGF0YS50eHQKMDAyNGNpZCBiZWZvcmU6MjAyMS0wMS0wNVQyMzozMDowNVoKMDAyZnNpZ25hdHVyZSC71RSEGDO1fsZnDeFEP55uyHVdTNEk7QwshTI9L-rQvgo' -H 'Source: https://osg-test.localdomain:9001/usr/share/osg-test/test_gridftp_data.txt' -H 'Overwrite: T' -H 'X-Number-Of-Streams: 2' -H 'Copy-Header:  Authorization: Bearer MDAyOGxvY2F0aW9uIE9TR19URVNUX1hST09URF9TVEFOREFMT05FCjAwMzRpZGVudGlmaWVyIDg3ODhiZjAyLTdkNDgtNGYyOC1iNDE2LTlhMTJmNDE3YjQ3ZAowMDE1Y2lkIG5hbWU6dmR0dGVzdAowMDUyY2lkIGFjdGl2aXR5OlJFQURfTUVUQURBVEEsVVBMT0FELERPV05MT0FELERFTEVURSxNQU5BR0UsVVBEQVRFX01FVEFEQVRBLExJU1QKMDAxNWNpZCBhY3Rpdml0eTpBTEwKMDAyY2NpZCBwYXRoOi90bXAvdGVzdF9ncmlkZnRwX2RhdGFfdHBjLnR4dAowMDI0Y2lkIGJlZm9yZToyMDIxLTAxLTA1VDIzOjMwOjA1WgowMDJmc2lnbmF0dXJlIHK8xGS4SxKqivv5QERTWSSTCeN9a7xNf4RwxF-TE1g0Cg' https://osg-test.localdomain:9002/tmp/test_gridftp_data_tpc.txt" >>> 0 2404 0
STDOUT:{
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0* About to connect() to osg-test.localdomain port 9002 (#0)
*   Trying <IP_ADDRESS>...
* Connected to osg-test.localdomain (<IP_ADDRESS>) port 9002 (#0)
* Initializing NSS with certpath: sql:/etc/pki/nssdb
* skipping SSL peer certificate verification
* NSS: client certificate not found (nickname not specified)
* SSL connection using TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384

The new VMU tests show the errors we found out on 5.1.0.rc4
osgtest: 2021-01-05 17:10:06: test_465_xrootd_tpc.py:test_02_initate_tpc:57: runuser vdttest -c "curl -A Test -vk -X COPY -H 'Authorization: Bearer MDAyOGxvY2F0aW9uIE9TR19URVNUX1hST09URF9TVEFOREFMT05FCjAwMzRpZGVudGlmaWVyIDk0NWYxNTk1LWIxYTMtNGRlYy1iYWZlLWNiYmNkNzVkODAzOAowMDE1Y2lkIG5hbWU6dmR0dGVzdAowMDJkY2lkIGFjdGl2aXR5OlJFQURfTUVUQURBVEEsRE9XTkxPQUQsTElTVAowMDE1Y2lkIGFjdGl2aXR5OkFMTAowMDM3Y2lkIHBhdGg6L3Vzci9zaGFyZS9vc2ctdGVzdC90ZXN0X2dyaWRmdHBfZGF0YS50eHQKMDAyNGNpZCBiZWZvcmU6MjAyMS0wMS0wNVQyMzozMDowNVoKMDAyZnNpZ25hdHVyZSC71RSEGDO1fsZnDeFEP55uyHVdTNEk7QwshTI9L-rQvgo' -H 'Source: https://osg-test.localdomain:9001/usr/share/osg-test/test_gridftp_data.txt' -H 'Overwrite: T' -H 'X-Number-Of-Streams: 2' -H 'Copy-Header:  Authorization: Bearer MDAyOGxvY2F0aW9uIE9TR19URVNUX1hST09URF9TVEFOREFMT05FCjAwMzRpZGVudGlmaWVyIDg3ODhiZjAyLTdkNDgtNGYyOC1iNDE2LTlhMTJmNDE3YjQ3ZAowMDE1Y2lkIG5hbWU6dmR0dGVzdAowMDUyY2lkIGFjdGl2aXR5OlJFQURfTUVUQURBVEEsVVBMT0FELERPV05MT0FELERFTEVURSxNQU5BR0UsVVBEQVRFX01FVEFEQVRBLExJU1QKMDAxNWNpZCBhY3Rpdml0eTpBTEwKMDAyY2NpZCBwYXRoOi90bXAvdGVzdF9ncmlkZnRwX2RhdGFfdHBjLnR4dAowMDI0Y2lkIGJlZm9yZToyMDIxLTAxLTA1VDIzOjMwOjA1WgowMDJmc2lnbmF0dXJlIHK8xGS4SxKqivv5QERTWSSTCeN9a7xNf4RwxF-TE1g0Cg' https://osg-test.localdomain:9002/tmp/test_gridftp_data_tpc.txt" >>> 0 2404 0
STDOUT:{
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0* About to connect() to osg-test.localdomain port 9002 (#0)
*   Trying <IP_ADDRESS>...
* Connected to osg-test.localdomain (<IP_ADDRESS>) port 9002 (#0)
* Initializing NSS with certpath: sql:/etc/pki/nssdb
* skipping SSL peer certificate verification
* NSS: client certificate not found (nickname not specified)
* SSL connection using TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384

It doesn't look like 5.1.0 got installed, just 4.12.6. Does this bug also affect the 4.12 series?

It doesn't look like 5.1.0 got installed, just 4.12.6. Does this bug also affect the 4.12 series?

We found it on the 5.1.0 series:
https://github.com/xrootd/xrootd/issues/1378
But yea I guess it is also in 4.12 but in 5.0.2 the problem is gone

We found it on the 5.1.0 series:
https://github.com/xrootd/xrootd/issues/1378
But yea I guess it is also in 4.12 but in 5.0.2 the problem is gone


Duplicate guarded transition allowed in Umple

class CarTransmission {
Boolean driveSelected;
state {
neutral {
selectReverse -> reverse;
selectDrive -> drive;
selectFirst -> first;
selectSecond -> second;
}
reverse {
  selectNeutral -> neutral;
}

drive {
  selectNeutral -> neutral;
  selectFirst -> first;
  selectSecond -> second;

  first {
    reachSecondSpeed [driveSelected]  -> second;
    reachSecondSpeed [driveSelected]  -> second;  //duplicate guarded transition
  }
  second {  }
}

}
}

I think we need to build a truth table for conditions and make sure they are not the same. This concept can be more complicated when we have fix values participated in the boolean expression or one of the conditions are reduced.
I believe this algorithm is really important for Umple in order to guarantee non-deterministic state machines. I have done a small search (as part of my own work) but couldn't find a concrete algorithm for this purpose.

Right. So duplicate guarded transitions are OK as long as there are no situations where two of the guards can be true at the same time. If two guards are identical, as in the example at the start of this issue then there should be a warning. Then there should be a different warning if they are not the same but are not mutually exclusive. There should be no warning if they are mutually exclusive.

A warning is sufficient for the case presented in the issue. However, I have implemented a temporary solution to prevent such duplicates in the list returned by "*.getAllTransitions()" as it guarantees correct output for nuXmv pending the time the issue is resolved.

Consider the following simple example:
class A{
	Boolean x;
	Boolean y;
	status{
		on{
			turnOff[x<y] ->off; //n1
			turnOff[y>y] ->off; //n2
			turnOff[x<y] -> on; //n3
		}
		off{
			turnOn[x>y] -> on; //f1
			turnOn[y<=x] -> on; //f2
			turnOn[y==x] -> off;//f3
		}
	}
}

There is no conflict between n1 and n2 because they go to the same state. Therefore, there is no worry. We may consider to just warn the designer about his/her design.
There is conflict obviously between (n1 and n3)  and (n2 and n3). I think we should generate an error because it makes Umple state machines non-deterministic, except if would like to support non-deterministic state machines. In that case, we must provide a mechanism to deal those cases, such as prioritization.
There is another non-deterministic case between f2 and f3 .

Is there a final verdict on whether we should be generating an error or just a warning in the cases outlined in Vahdat's last post?

If there is no guard, it is a warning if there are two identical events. We get:
45 Warning on line 10 : Transition xxxxx will be ignored due to a unguarded duplicate event.
If there are guards and both event and guards are identical, then logically it should also be a warning. The duplicate is ignored. It should work like error 45 does now. Message would be
Transition xxxxx will be ignored to to duplicate events with identical guards.

In PR https://github.com/umple/umple/pull/938, I added logic to ignore transitions that have the same event and equivalent guard as another transition. The warning message displayed is "Guarded Transition xxxxx will be ignored due to a duplicate event with an equivalent guard".
Also works for auto transitions that have equivalent guards.
The logic for determining whether two guards are equivalent will do most basic comparison between the guards. For example, two guards like [(x + y) < z] and [z >= (y+x)] would be confirmed as equivalent. Also, works for more complicated guards that use negation and constraintBody's (parenthesis -- OPEN_ROUND_BRACKET [[constraint]] CLOSE_ROUND_BRACKET). See PR

@tlaport4  thank.
Consider the following example
class A{
  sm{
    on{
      [x]-> off;
      -> on;
    }
    off{

    }

  }
}

There are two warning messages for one transition.
Warning 54 on line 7 of file "test.ump":
Transition  will be ignored due to an unguarded duplicate event
Warning 55 on line 7 of file "test.ump":
Transition  in super-state will be ignored due to unguarded duplicate events in all sub-states

Check all locations and test cases related to warning 54 and see if your algorithm can satisfy them. Then, you can keep yours and delete others. It's better to keep one which is more comprehensive.

Another example
class A{
  sm{
    on{
      [x>y]-> off;
      [y<x]-> on;
    }
    off{  }
  }

Hmmm yeah that's right.
When I wrote I for some reason I thought the equivalent of > would be <=, but that's the opposite. I'll quickly update it.

I updated the CR and all the tests just passed.

You algorithm cannot detect the issue related to f2 and f3 in the above example

Those aren't necessarily equivalent though. They're only equivalent when x = y. What should it do? Should either one be ignored?

This is a dynamic behavior and from my perspective, it must be notified to the modeler. However, maybe @TimLethbridge  and @opeyemiAdesina want to comment more about this case.
This clearly shows that the state machine has this potential to be in two states. However, it might never happen because of a restriction on the values of x and y (e.g. OCL constraint). For example, someone might define a constraint in which it is said x and y must not be equal.

This is a case where one guard is a sub-case of another. We can't easily compute all these cases logically (i.e. analyse all truth tables). In this case one could compute easily that one is a sub-case, but cases get complex very quickly.  I say we leave this unless we are going to go 'whole-hog' and try to compute all the cases that are computable statically (this is a static case).
I don't call this non-determinism by the way. In Umple, order of declaration determines which case applies. It is only nondeterministic if it can change. But it is under-specified what happens in the presence of traits, and with mixins the order can be hard to be sure about.

Maybe this issue can be closed, as it is partly solved. But more testing would be useful


RHOAIENG-11046 - Add support for AMD GPU image

Issue link

What changes have been made

Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

Hello @anishasthana can you please help us to find out what is the right image?

@jiripetrlik Yes that's preferable to default to quay.io/modh/ray:2.35.0-py39-rocm61 as @ChughShilpa suggested.


TypeError: Cannot read property 'destroy' of undefined

On model Customer.js:
module.exports = {
	attributes: {
		name: { type: 'string', required: true, size: 100 },
		phone: { type: 'string', unique: true, required: true, size: 15 },
		//Relations
		orders: { collection: 'order', via: 'customer' }
	},
	setPhonePrefix: function setPhonePrefix(values, prefix) {
		var regexp = new RegExp('^\\' + (prefix || "+995") + '.*$');
		values.phone && !regexp.test(values.phone)
						&& (values.phone = "+995" + values.phone);
		return values;
	},
	beforeCreate: function(values, cb) {
		this.setPhonePrefix(values);
		cb();
	},
	beforeUpdate: function(values, cb) {
		this.setPhonePrefix(values);
		cb();
	},
	beforeFind: function(query, cb) {
		this.setPhonePrefix(query);
		cb();
	}
};

On test:
/* global describe, Customer */
var chai = require('chai'),
				assert = chai.assert,
				searchCriteria = { phone: '555555555' },
customerToCreate = { name: 'san', phone: '555555555' };

before(function (done) {
	Customer.destroy({ }, function (err) {
		done(err);
	});
});

after(function (done) {
	Customer.destroy({ }, function (err) {
		done(err);
	});
});

describe('Customer', function () {

	it('#findOrCreate()', function (done) {
		Customer.setPhonePrefix(searchCriteria);
		Customer.findOrCreate({ where: searchCriteria }, customerToCreate)
						.then(function (foundOrCreated) {
							assert.propertyVal(foundOrCreated, 'name', "san", "Name of customer");
							assert.propertyVal(foundOrCreated, 'phone', "+995555555555", "Phone of customer");
							Customer.find({ where: searchCriteria })
											.then(function (result) {
												assert.equal(result.length, 1);
												var customer = result[0];
												assert.propertyVal(customer, 'name', "san", "Name of customer");
												assert.propertyVal(customer, 'phone', "+995555555555", "Phone of customer");
done();
											})
											.catch(done);
						}).catch(done);
	});

});

I don't know how you're running your tests, but I don't have any reason to believe that Customer should exist at that point.  Maybe you're using sails and are used to models being made globals.

Yup, seems Customer isn't being correctly set as a global.

Well It actually is set as global correctly. Because the error happens for line 14.
Sorry that I didn't mention before: my test environment is initialized as required, i thought it's obviously that I must recheck all other possibilities before creating the issue here.
An here is the initialization of my tests (there is another dozen of them that works perfectly) - bootstrap.test.js:
var Sails = require('sails'),
				mocha = require('mocha'),
				tk = require('timekeeper'),
				sails;

var time = new Date('2015-05-15T01:01:01');
tk.freeze(time);

before(function (done) {
	Sails.lift({
		models: {
			connection: 'localDiskDb',
			migrate: 'drop'
		}
	}, function (err, server) {
		sails = server;
		sails.log.verbose("Sails loaded?", err, server);
		if (err) return done(err);
		done(err, sails);
	});
});

after(function (done) {
	// here you can clear fixtures, etc.
	sails.lower(done);
});

Can we kindly reopen/review this issue?

Perhaps sails are being lowered before your after test callback is running.  I don't think waterline is unsetting your global Customers variable, or setting it to undefined.  Mind logging at the beginnig of all four before and after callbacks to determine the order in which they're run?

Thanks, @devinivy.
You are right.
Here's log:
bootstrap.test before Sails.lift callback
Customer.test before
Customer.test before destroy callback
ok 1 OrderController #createOrder()
Customer.test #findOrCreate()
Customer.test #findOrCreate() callback
Customer.test #findOrCreate() find callback
//Other tests ommited
bootstrap.test after
bootstrap.test after sails.lower callback undefined [ undefined, undefined, undefined ]
Customer.test after
not ok 25  "after all" hook
  TypeError: Cannot read property 'destroy' of undefined
      at Context.<anonymous> (D:\System\Dropbox\in2circle\Development_I2C\sadili.ge\test\unit\models\Customer.test.js:17:11)
      at Context.<anonymous> (D:\System\Dropbox\in2circle\Development_I2C\sadili.ge\test\unit\controllers\OrderController.test.js:38:2)
      at D:\System\Dropbox\in2circle\Development_I2C\sadili.ge\test\bootstrap.test.js:43:3
      at net.js:1419:10

I don't think there are any changes we can make in waterline to help you out here– you may be better off opening an issue in the sails repo, since it seems to have to do with the order of operations with sails test bootstrapping.  What do you think?  Good luck!

Thanks. Sails is OK too.
So thanks for your time.
I think I should close it now.


Include errors in sync events

When we run the sync, we log but otherwise ignore error results from kubectl apply. The excuse is that in many cases it will be to do with the ordering -- creating a resource before its namespace has been created, for example. But this misses some important cases that we do want to know about, like having an invalid manifest.
I suggest we include these "genuine" errors in the sync event sent upstream, so they can be included in notifications. (If we can be clever about not including ordering problems, either by filtering them out, or #602, so much the better).

At the moment my thoughts are that namespaces should be created first, and all errors logged.

I had another idea: we could annotate resources with the last sync error (if the resource exists), and show those in UI/CLI.

I like it

we could annotate resources with the last sync error

The trouble is not all resources appear in the CLI/UI. So we'd need a special means of displaying these; maybe as part of a special sync status view.

Does #970 resolve this?

Does #970 resolve this?

Yup. Shall make a comment there.


Missing dependency com.blackberry.bdp.common:bdp-common:jar:0.0.9

Hi,
I have an issue while compilng Krackle : the 0.8.0 version needs com.blackberry.bdp.common:bdp-common:jar:0.0.9 but I don't know how to get it.
Could you please tell me if this library is available ?
Regards,
Christophe

Hi Christophe!
Please check this out... We're not yet pushing our artifacts to maven central so in the meantime you'll need to obtain the specific BlackBerry projects from Github and build them locally installing them in your local maven repo.
https://github.com/blackberry/BB-BigData-Common-Utils


A tutorial on how to use VS Code to debug Lambda functions in a CDK app

PR with the post: https://github.com/cdk-dev/website/pull/68
Let me know if it looks okay!

#68 was merged almost 1yr ago


figures in white vs ggdark

FWCP special request.  see last years report for examples

Does this also include the fish sampling plot for pscis crossing 125000? That plot is built with the fpr function fpr_plot_fish_box() so we'd have to add a parameter to the function in fpr for themes. The rest of the figures in the report were straightforward to change.

https://github.com/NewGraphEnvironment/fpr/issues/35


Implement some observability/monitoring part-2

As part of our observability and monitoring effort, we want to set up the monitoring for the following components apart from this issue:

Explore and set up the monitoring for backend APIs. This should ping the APIs every 5 minutes and post a notification on Slack if the APIs are down with a relevant response.
Slack bot monitoring
Page on GitHub on what kind of observation metrics/actions are needed.

More details around this can be found https://github.com/astronomer/ask-astro/issues/39#issuecomment-1782573622

Created a postman collection to monitor the APIs: postman collection

Working on making this part of monitoring DAG instead

PR will be created today


Is kontra.assets.loadImage() deprecated?

First of all, amazing framework! I plan to use it to teach a class on JavaScript to people who are new to coding.
Anyways, I was trying to run some of the examples and I got the error kontra.assets.loadImage is not a function a couple times. However, when I replaced the call with kontra.assets.load everything worked!
I'm pretty sure it's because the API changed at some point, but I couldn't find any mention of it.

Thanks for the kind remarks. And yes, sorry, kontra.loadAssets was deprecated in favor of using konra.assets.load. I was in such a rush to get the next version out for js13k that I didn't make great release notes. Let me do so now in the 3.0 release.

Alright, added the very long list of deprecations. Hopefully that clears things up.


irmin-pack: replace inode's Map by a list of pairs

New tentative for #1989

Codecov Report

Merging #2042 (ad273c5) into main (b5d5e3f) will decrease coverage by 0.01%.
The diff coverage is 63.82%.

@@            Coverage Diff             @@
##             main    #2042      +/-   ##
==========================================
- Coverage   64.48%   64.47%   -0.02%
==========================================
  Files         130      130
  Lines       15604    15650      +46
==========================================
+ Hits        10063    10091      +28
- Misses       5541     5559      +18

Impacted Files
Coverage Δ

src/irmin-pack/inode.ml
78.48% <0.00%> (-0.11%)
:arrow_down:

src/irmin/import.ml
69.89% <65.21%> (-4.58%)
:arrow_down:

src/irmin-fs/unix/irmin_fs_unix.ml
67.09% <0.00%> (-0.65%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

This seems to improve performance pretty significantly compared to main:
 |                          |     0     |       1
 | --                       | --        | --
 | -- main metrics --       |           |
 | CPU time elapsed         |     0m47s |     0m39s  83%
 | Wall time elapsed        |     0m47s |     0m39s  83%
 | TZ-transactions per sec  |   253.723 |   304.650 120%
 | TZ-operations per sec    |  2632.493 |  3160.876 120%
 | Context.set per sec      | 19018.674 | 22836.024 120%
 | tail latency (1)         |   0.954 s |   0.723 s  76%
 |                          |           |
 | -- resource usage --     |           |
 | disk IO (total)          |           |
 |   IOPS (op/sec)          |    43_802 |    52_314 119%
 |   throughput (bytes/sec) |   4.954 M |   5.937 M 120%
 |   total (bytes)          |   0.233 G |   0.233 G 100%
 | disk IO (read)           |           |
 |   IOPS (op/sec)          |    42_364 |    50_587 119%
 |   throughput (bytes/sec) |   2.097 M |   2.505 M 119%
 |   total (bytes)          |   0.099 G |   0.098 G  99%
 | disk IO (write)          |           |
 |   IOPS (op/sec)          |     1_438 |     1_727 120%
 |   throughput (bytes/sec) |   2.857 M |   3.432 M 120%
 |   total (bytes)          |   0.135 G |   0.135 G 100%
 |                          |           |
 | max memory usage         |   0.561 G |   0.561 G 100%
 | mean CPU usage           |      100% |      100%

Full comparison: stepmap.log

Nice! That looks promising! I'm running a lib_context benchmark for this PR to see how it performs in that context.

The results of running the lib_context benchmark is less conclusive, in particular I'm looking at the increased tail latency. I'll run another one to see if it is a consistent result.
 |                          |   base    |    stepmap
 | --                       | --        | --
 | -- main metrics --       |           |
 | CPU time elapsed         |   192m21s |   189m07s  98%
 | Wall time elapsed        |   231m55s |   228m33s  99%
 | TZ-transactions per sec  |  1370.961 |  1394.399 102%
 | TZ-operations per sec    |  2241.601 |  2279.923 102%
 | Context.add per sec      | 18272.123 | 18584.501 102%
 | tail latency (1)         |   0.640 s |   0.946 s 148%
 |                          |           |
 | -- resource usage --     |           |
 | disk IO (total)          |           |
 |   IOPS (op/sec)          |   246_585 |   250_802 102%
 |   throughput (bytes/sec) |  18.820 M |  19.142 M 102%
 |   total (bytes)          | 217.206 G | 217.207 G 100%
 | disk IO (read)           |           |
 |   IOPS (op/sec)          |   246_510 |   250_726 102%
 |   throughput (bytes/sec) |  13.870 M |  14.107 M 102%
 |   total (bytes)          | 160.071 G | 160.073 G 100%
 | disk IO (write)          |           |
 |   IOPS (op/sec)          |        76 |        77 102%
 |   throughput (bytes/sec) |   4.950 M |   5.035 M 102%
 |   total (bytes)          |  57.134 G |  57.135 G 100%
 |                          |           |
 | max memory usage (bytes) |   0.577 G |   0.565 G  98%
 | mean CPU usage           |       83% |       83%

The lib-context code might need a similar update to the one I've done for the schema changes (to stop sorting the list of children with a weird and useless ordering)

A second run (b) brought this branch more in line with main for tail latency (which is good to see!). I can also run another one with an updated schema to see what change that might have.
 |                          |   base    |       a        |       b
 | --                       | --        | --             | --
 | -- main metrics --       |           |                |
 | CPU time elapsed         |   192m46s |   189m07s  98% |   190m26s  99%
 | Wall time elapsed        |   232m39s |   228m33s  98% |   230m19s  99%
 | TZ-transactions per sec  |  1368.073 |  1394.399 102% |  1384.737 101%
 | TZ-operations per sec    |  2236.879 |  2279.923 102% |  2264.125 101%
 | Context.add per sec      | 18233.635 | 18584.501 102% | 18455.726 101%
 | tail latency (1)         |   0.695 s |   0.946 s 136% |   0.673 s  97%
 |                          |           |                |
 | -- resource usage --     |           |                |
 | disk IO (total)          |           |                |
 |   IOPS (op/sec)          |   246_068 |   250_802 102% |   249_068 101%
 |   throughput (bytes/sec) |  18.781 M |  19.142 M 102% |  19.010 M 101%
 |   total (bytes)          | 217.208 G | 217.207 G 100% | 217.210 G 100%
 | disk IO (read)           |           |                |
 |   IOPS (op/sec)          |   245_993 |   250_726 102% |   248_992 101%
 |   throughput (bytes/sec) |  13.840 M |  14.107 M 102% |  14.009 M 101%
 |   total (bytes)          | 160.073 G | 160.073 G 100% | 160.075 G 100%
 | disk IO (write)          |           |                |
 |   IOPS (op/sec)          |        75 |        77 102% |        76 101%
 |   throughput (bytes/sec) |   4.940 M |   5.035 M 102% |   5.000 M 101%
 |   total (bytes)          |  57.135 G |  57.135 G 100% |  57.135 G 100%
 |                          |           |                |
 | max memory usage (bytes) |   0.576 G |   0.565 G  98% |   0.584 G 101%
 | mean CPU usage           |       83% |       83%      |       83%

Here is one run of tezos patched with https://github.com/mirage/irmin/pull/2014 and this PR. Higher tail latency again, but results mostly seem inconclusive to me.
 |                          |   base    |    stepmap     | stepmap_tezos_order
 | --                       | --        | --             | --
 | -- main metrics --       |           |                |
 | CPU time elapsed         |   192m46s |   190m26s  99% |        191m09s  99%
 | Wall time elapsed        |   232m39s |   230m19s  99% |        231m01s  99%
 | TZ-transactions per sec  |  1368.073 |  1384.737 101% |       1379.620 101%
 | TZ-operations per sec    |  2236.879 |  2264.125 101% |       2255.758 101%
 | Context.add per sec      | 18233.635 | 18455.726 101% |      18387.522 101%
 | tail latency (1)         |   0.695 s |   0.673 s  97% |        0.853 s 123%
 |                          |           |                |
 | -- resource usage --     |           |                |
 | disk IO (total)          |           |                |
 |   IOPS (op/sec)          |   246_068 |   249_068 101% |        248_146 101%
 |   throughput (bytes/sec) |  18.781 M |  19.010 M 101% |       18.939 M 101%
 |   total (bytes)          | 217.208 G | 217.210 G 100% |      217.208 G 100%
 | disk IO (read)           |           |                |
 |   IOPS (op/sec)          |   245_993 |   248_992 101% |        248_070 101%
 |   throughput (bytes/sec) |  13.840 M |  14.009 M 101% |       13.957 M 101%
 |   total (bytes)          | 160.073 G | 160.075 G 100% |      160.074 G 100%
 | disk IO (write)          |           |                |
 |   IOPS (op/sec)          |        75 |        76 101% |             76 101%
 |   throughput (bytes/sec) |   4.940 M |   5.000 M 101% |        4.982 M 101%
 |   total (bytes)          |  57.135 G |  57.135 G 100% |       57.134 G 100%
 |                          |           |                |
 | max memory usage (bytes) |   0.576 G |   0.584 G 101% |        0.561 G  97%
 | mean CPU usage           |       83% |       83%      |            83%

I'm surprised that this PR doesn't have a bigger effect on allocated words:
 |                                            |                     |      total       | avg per block
 | gc.minor_words allocated                   | main_220810         |  2356.498 G      |  16.252 M
 |                                            | stepmap             |  2334.767 G  99% |  16.102 M  99%
 |                                            | stepmap_tezos_order |  2328.504 G  99% |  16.059 M  99%
 | gc.major_words allocated                   | main_220810         |   205.634 G      |   1.418 M
 |                                            | stepmap             |   201.136 G  98% |   1.387 M  98%
 |                                            | stepmap_tezos_order |   201.135 G  98% |   1.387 M  98%

Maybe Craig's Small_list on https://gitlab.com/tezos/tezos/-/merge_requests/3526 could help, but I'm not even sure.


Use TYPE (instead of USER_TYPE) as USER property

It looks like the new USER property TYPE = PERSON | SERVICE | LEGACY_SERVICE | NULL
have been added to titan, but as a non-existing property called USER_TYPE.
I would expect this YAML to work:
users:
  - name: some_user
    owner: USERADMIN
    email<EMAIL_ADDRESS>    user: PERSON

I try to stay 1:1 to the original SQL when possible. Since "type" has semantic meaning in Python I made a judgement call to use "user_type" in this case. I can make this more clear in the docs.

What I noticed, when I used the user_type as a YMAL argument, like this:
users:
  - name: some_user
    owner: USERADMIN
    email<EMAIL_ADDRESS>    user_type: PERSON

... was that this was the SQL sent to Snowflake:
ALTER USER SOME_USER SET user_type = 'PERSON'

So, I guess some last-mile mapping from the in-parameter to the generated SQL-parameter is missing then.

That's a bug, I'll get it fixed.

@teej I ran into this bug as well!

Fixed in v0.9


Dobe/projectionvisitor

Coverage increased (+0.01%) to 81.52% when pulling 74ebcf62a5d391afbc46fad77ba12f0e4d74f05c on dobe/projectionvisitor into e09835b948b4fcdb9f4b28a24bafd4a5aef5639c on master.

Changes Unknown when pulling 78e2c185095eabcbba83767e7de92a191f1adb73 on dobe/projectionvisitor into ** on master**.

Changes Unknown when pulling 78e2c185095eabcbba83767e7de92a191f1adb73 on dobe/projectionvisitor into ** on master**.

otherwise ack

I would like to get 30mg adderall and 40mg merhadone


[snapshot] iPhone 15 and new iPads have unsupported screen sizes

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
After using snapshot to take screenshots, deliver fails with size validation for new devices

iPad (10th generation)
iPad Air (5th generation)
iPad mini (6th generation)
iPhone 15 Pro
iPhone 15

Similar to https://github.com/fastlane/fastlane/pull/21393 we need to handle newer devices
Command executed
fastlane deliver
Complete output when running fastlane, including the stack trace and command used

[21:32:01]: 🚫 Invalid screenshots were detected! Here are the reasons:
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad (10th generation)-0Launch.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad (10th generation)-1RecentEvents.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad (10th generation)-2Calendar.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad (10th generation)-3CalendarSettings.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad (10th generation)-4TopiclessEvents.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad Air (5th generation)-0Launch.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad Air (5th generation)-1RecentEvents.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad Air (5th generation)-2Calendar.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad Air (5th generation)-3CalendarSettings.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad Air (5th generation)-4TopiclessEvents.png - Invalid screen size (Actual size is 1640x2360. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad mini (6th generation)-0Launch.png - Invalid screen size (Actual size is 1488x2266. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad mini (6th generation)-1RecentEvents.png - Invalid screen size (Actual size is 1488x2266. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad mini (6th generation)-2Calendar.png - Invalid screen size (Actual size is 1488x2266. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad mini (6th generation)-3CalendarSettings.png - Invalid screen size (Actual size is 1488x2266. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPad mini (6th generation)-4TopiclessEvents.png - Invalid screen size (Actual size is 1488x2266. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15 Pro-0Launch.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15 Pro-1RecentEvents.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15 Pro-2Calendar.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15 Pro-3CalendarSettings.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15 Pro-4TopiclessEvents.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15-0Launch.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15-1RecentEvents.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15-2Calendar.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15-3CalendarSettings.png - Invalid screen size (Actual size is 1178x2556. See the specifications to fix https://help.apple.com/app-store-connect/#/devd274dd925)
[21:32:01]: 🚫 Error: ./fastlane/screenshots/en-US/iPhone 15-4TopiclessEvents.png - Invalid screen size (Actual size is 1178x2556.


Fix request in _filterCachedFastbootCookies

@marcoow You forgot something. :smile:

I can make those changes.

This library will also probably need to be updated to accommodate https://github.com/ember-fastboot/ember-cli-fastboot/pull/171

Changes are updated. It looks like I'll be submitting a newer PR once ember-cli-fastboot is updated again to use the new fastboot request protocol.


Feature request: Issue tracking on osf.io

I think osf.io is a very cool project with a lot of great features supporting scientific workflows.
However, one area in which I feel it falls short, for some use-cases, is in project management. In particular, I am thinking about collaborating at a distance, where keeping track of important tasks right on the osf project page could significantly lower the barrier to participation in e.g. theoretical fields, where there is no fundamental need for researchers to all be in the same place.
As a starting point, I think it would be awesome if osf.io had an issue-tracking system somewhat like Github's! I realise github is integrated already, so perhaps an initial implementation could be a wrapper around github's issue-tracker? It would be great if an interested scientist visiting the project page would see a set of tasks they could contribute to.
I expect this is something the osf team have thought about, but even if you have reached a conclusion I would still appreciate hearing your thoughts about it.

Hi @amilsted,
Thank you for your feedback and support! We keep track of feature requests in an internal tracker to help inform future development, and I'll add a note there for you.
As a workaround, you can do simple task management in the wiki through lists and "@" mentions. You can have a wiki page dedicated to each collaborator's set of tasks, and add to them as needed.
I hope this helps! I'll let you if/when this feature becomes available in the future.


Add support for exact median aggregate function

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Needed for h20 benchmark group by query 6.
Describe the solution you'd like
Add support for median aggregate function
Describe alternatives you've considered
We have approx_median but that would be cheating.
Additional context
None

So this is a bit more involved than I had hoped.
Per the discussion in https://github.com/apache/arrow-datafusion/issues/1486#issuecomment-1016877360
TLDR is I think median is quite complicated to get both correct and high performance.
The key problem is that you basically need to have buffered the entire input stream before you know what the output is
Starting with an approximation of median would be fine and I suspect other users of DataFusion would find it valuable.
To implement an exact median, you could implement an Aggregator like we have for other aggregates that buffers up all the input, sorts it,  and produces the median value. It would have terrible memory consumption but is probably reasonably straightforward to implement using existing compute kernels as @matthewmturner suggests
If you wanted to try and reuse existing operators (e.g. LogicalPlan and ExecutionPlan), I think that would be tough.
The mapping of SQL --> LogicalPlan is formulaic but non trivial. All aggregate functions are treated the same (create the same LogicalPlan nodes, with different expr lists, so if we special case median with a special  operator we'll have to sort out how to handle queries like SELECT sum(foo), median(foo) from bar GROUP BY baz which I think will be complicated
Another  challenge with median is that there is no way to partially aggregate intermediate results. DataFusion currently makes this pattern (which is useful when calculating sums, for example, because you can calculate partial sums and then sum them together)
                        ┌──────────┐
                        │          │
                        │Aggregator│
                        │ (Final)  │
                        │          │
                        │          │
                        └──────────┘
                              ▲
                              │
      ┌────────────────┬──────┴────────────────────┐
      │                │                           │
      │                │                           │
      │                │                           │
┌──────────┐     ┌──────────┐                ┌──────────┐
│          │     │          │                │          │
│Aggregator│     │Aggregator│                │Aggregator│
│(Partial) │     │(Partial) │         ...    │(Partial) │
│          │     │          │                │          │
│          │     │          │                │          │
└──────────┘     └──────────┘                └──────────┘
      ▲                ▲                           ▲
      │                │                           │
      └────────────────┴──────┬────────────────────┘
                              │
                              │
                        ┌───────────┐
                        │           │
                        │Repartition│
                        │           │
                        │           │
                        └───────────┘
                              ▲
                              │
                              │
                        ┌───────────┐
                        │Input      │
                        │           │
                        └───────────┘

I think that the first step here would be to change the Accumular::state function to return ArrayRef instead of ScalarValue.

I am making good progress on this and will have a PR up in the next week or two.

Thanks @jonmmease. I just submitted a PR with the functionality. I need to clean this up, but I think it is ready for testing.


:pdf-page-margin: attribute no longer works correctly

Attribute :pdf-page-margin: no longer works as expected as of asciidoctor-pdf version 1.5.0.rc.1.
Example document:
= Test document
:pdf-page-margin: [0.7in, 0.98in, 0.7in, 0.98in]

Lorem culpa reiciendis quos expedita similique! Harum facilis nemo quae aspernatur laudantium, sapiente. Inventore ad iusto impedit enim exercitationem accusamus Cupiditate ex corporis exercitationem odio ut Labore totam expedita numquam.

Resulting PDF document has 0 left margin, as shown in the following screenshot:

$ asciidoctor --version
Asciidoctor 2.0.10 [https://asciidoctor.org]
Runtime Environment (ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-linux]) (lc:UTF-8 fs:UTF-8 in:UTF-8 ex:UTF-8)

$ asciidoctor-pdf --version
Asciidoctor PDF 1.5.0.rc.1 using Asciidoctor 2.0.10 [https://asciidoctor.org]
Runtime Environment (ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-linux]) (lc:UTF-8 fs:UTF-8 in:UTF-8 ex:UTF-8)

Yep, there's a bug in the page margin parsing code, and we're missing a test for it. I'll get it fixed in the next release candidate. Thanks for reporting!


how to get date mask

how to get date mask in "yyyy-mm-dd" format?

mask : {dddd}-{dd}-{dd}
maskTemplate : yyyy-mm-dd
Thx,
Artem

how can i do 12 month limit?
or 30 days limit?

New version supported separate blocks editing. You can check block text and output filtered text.

@artemkrachulov How can i do that? I can find the block index and see how many elements there are in it, but can`t see the text it contains.


RPC authorization - service name with first letter capitalized

Hello,
After upgrading to 1.1.0, if the RPC service name starts with uppercase, I believe it's gonna lead to an inconsitency between the way the interface will save the RPC authorization information and the way zf-mvc-auth will check ACL.
The interface will save the authorization information using the service name with the first letter capitalized, eg:
'TestAPI\\V1\\Rpc\\Test\\Controller' => array(
    'actions' => array(
        'Test' => array(),
    ),
)

This way, when the acl rules are build, the resource name is gonna be TestAPI\V1\Rpc\Test\Controller::Teste
For the other side the acl is verified based on the ID returned by: ZF\MvcAuth\Authorization\DefaultResourceResolverListener::buildResourceString ... which consider the real action name (first letter in lowercase TestAPI\V1\Rpc\Test\Controller::teste).
This will result in a wrong acl check.
If the service name starts with lowercase everything works as expected.

Fixed: https://github.com/zfcampus/zf-mvc-auth/pull/68


Some keywords becomes colorless in the recent versions

This is my favorite theme and I have used it for about a year! Thanks for your great work.
But in the recent versions, some keywords becomes colorless:

The left side is v6.0.4. And the right side is v4.2.0. LanguageID: typescript
I really like the borderless look in the new versions but this issue prevents me to switch to it.
Is there anything I can do to fix this? Thanks!

That's intentional, the syntax highlighting logic has been redesigned since v5.0.0 #10.
But thanks for your reminding, the original syntax highlighting logic should not be discarded hastily. I've added a new configuration option to enable the original syntax highlighting logic 2318e30.

It's renamed as gruvboxMaterial.colorfulSyntax now.

Thanks for the quick reply!  I have checked it out and I love it. Thank you!


chore(main): release 0.10.0

:robot: I have created a release beep boop
0.10.0 (2024-11-08)
Features

refac (a7f33fb)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.10.0
:sunflower:


Event exporter can't collect event on too many events.

Related issue: https://github.com/opsgenie/kubernetes-event-exporter/issues/159
Latest disscuss: https://github.com/resmoio/kubernetes-event-exporter/issues/16
Fixed commit: https://github.com/resmoio/kubernetes-event-exporter/commit/3cd9dd2c0db6df00197326e125fe041026b87a51

We should use image: docker.io/resmoio/kubernetes-event-exporter:v1.0
And add config:
kubeQPS: 60
kubeBurst: 60

Because of problem: starting container process caused "chdir to cwd (\"/home/nonroot\") set in config.json failed: permission denied": unknown
I revert commit: https://github.com/opsgenie/kubernetes-event-exporter/pull/187/commits/f0dc4143c75d049ae1670199e4d48a8d321ed4bc . Then build a new image to resolve, tag is: v1.0-kubegems

60 qps still seems small, maybe scale to 300.


Add cn_prefix and o to keypair requests (RBAC)

@puja, @marians, and I discussed this last Thursday, that it was becoming important in the short term to be able to set the Common Name of certificates we generate during key pair creation. Since it allows for setting more granular RBAC permissions per certificate instead of granting full admin rights each time.
This PR proposes a way to achieve that for the end user.
Implementation considerations:
We want to allow only a prefix of the Common Name of the certificate to be defined by the user. This lets us avoid collisions with other k8s components, as well as ensures users won't be able to make certificates that have no chance of working, since the K8s API will reject certificates that aren't a subdomain of the clusterdomain.
To avoid collisions with other k8s components, user certificate's common names should be a subdomain of user.clusterdomain like: oliverponder_giantswarm_io.user.clusterdomain
While we're at it, it would also be nice to be able to set the O of the certificate subject.
Relevant info:
https://kubernetes.io/docs/admin/authentication/#x509-client-certs

@xh3b4sd did you work mostly on the keypair creation and vault integration? Can you shed some light on if this spec seems achievable from that point of view?

I aded @rossf7 as he has recently migrated Vault to the newest version which supports this functionality (the Os)

If necessary the cn_prefix can be used in kubernetesd when creating the cert TPOs.

I'm a bit confused about the changes needed to certificatetpr and kubernetesd
I don't think that'll be needed. If my mental model is right, kubernetesd is making the cluster, and the certificates for components of the cluster right?
The cn_prefix shouldn't appear in any of those certificates. The cn_prefix part is something the user can choose to set per certificate that he creates and is intended to identify a single user.
As far as I knew the user made keypairs aren't TPO's. I might be wrong though, so please correct me if I misunderstand the situation.

@oponder @rossf7 there's two things we need, and a lot of the requirements and changes are similar:

User should be able to get <username>.user.clusterdomain as CN and any O they need.
k8sd and and cluster-service should be able to generate certs with any O (we especially need system:masters and things like system:node, system::kube-controller-manager etc) as well as CNs that describe the component the cert is used by. For latter we can either use <componentname>.clusterdomain (like we already have with api.) or we use <componentname>.component.clusterdomain or something else for the component part in that domain.

So somewhere in the chain it should be possible to choose any CN and any O. The limitation to user subdomain can be pretty far up the stack (e.g. API + UIs only).

@oponder you're right the user key pairs aren't TPOs. I got sidetracked because there is a lot of overlap as @puja108 says.
We will need the changes to certctl to set the o for both of these requirements. As it's our library for integrating with Vault.

@puja: understood. But I was talking about and responding to ross's use of cn_prefix specifically.
For what you're talking about, "the ability to choose any O and CN" would probably land in certctl since that is what makes the api call to vault.

Cool, i think we're on the same page :D

@puja: for the stuff about k8sd and changes to the certs that components are getting should be moved to a separate issue I think. Though its good to keep in mind this requirement. We shouldn't try to satisfy that here.
As long as certctl can set O, we should be good to iterate further on changes needed in k8sd and the operators. Could make an issue to describe/track those changes further?

Here's my view of the required changes and what components will be affected.

api

add CNPrefix and Organization to the request struct in the keypair creator endpoint and
service. Pipe it through to the call to cluster-service using clusterclient
Add logic for setting the default value of CNPrefix in case the user left it blank.

clusterclient

add CNPrefix and Organization to the create keypair request struct to be able to achieve point 1

cluster-service

use what was received for CNPrefix and prepend it to what was there previously for CommonName. Add Organization

certctl

add Organization as a setting in the call it makes to the Vault API. CommonName is already there.

I think I can handle all the changes, if it's just a matter of making the right call to vault from certctl.

@oponder LGTM
Agree, that the k8sd functionality is a different ticket and doesn't go over API.
point 4 might be tricky and you might need to sync with @rossf7 there, IIRC Os in Vault belong to the "role", and currently we're using a single role in Vault, so current structure is:
PKI - role - all certs. It might be that we need to create a role per O (or actually combination of Os) in which all certs that need that (combination of) O get created. So it would be:
PKI - role1 (O: foo, bar) - certs with O foo, bar
PKI - role2 (O: foo) - certs with O foo
...

@puja108 Thanks for the pointer on the roles. I agree point 4 is tricky and I think needs more certctl changes.
@oponder Here is where the certctl PKI service creates the single role. So we'd need to be able to create multiple roles and maybe add roles to an existing PKI?
https://github.com/giantswarm/certctl/blob/master/service/pki/service.go#L233
Hope that helps let me know if you want to sync more.

Yeah I might need to do the certctl part in pair programming, or otherwise get some help with it.
But from what I'm reading, and from what I see in the code @rossf7 linked, it is possible to create role on the fly.
So I'm thinking I'd add similar code to what is in the pki service to the cert-signer service. Where it'll try to create a role if it doesn't exist yet, and then use that role to issue the certificate.
(https://github.com/giantswarm/certctl/blob/master/service/cert-signer/cert_signer.go#L54)

Looks like you guys have it sorted. My two cents. I noticed we run Vault 0.6.4 on lycan. We have to make sure Vault is up to date with these requirements in all installations. Maybe SIG updates or SIG host cluster can organize this.

I was thinking today about a critical path, and as customers would like to have this rather sooner than later, we could start from the bottom and enable it in certctl first, so that we can at least already manually create the keypairs for them until they get the functionality to do it themselves. @oponder WDYT?

Yup, im starting with certctl. Was hacking on it today a bit already, and will continue hacking tonight. I think I can have something up for review by tmw morning.

I've change it from the key for setting the organizations subject from o to certificate_organizations and key cn_prefix as is.
@marians fyi: if you're around and want to give a final look at this spec, I'd appreciate it. I think you have good reasoning about the names of keys for things in the API.
In short summary: we're adding cn_prefix and certificate_organizations fields to the key pair create request.


add damelin.me domain

@damelin.me support

@teb0ho I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host damelin.me not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Update SBT, Scala, and other versions

Codecov Report

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

@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   97.13%   97.13%
=======================================
  Files          28       28
  Lines         839      839
  Branches       51       51
=======================================
  Hits          815      815
  Misses         24       24

Continue to review full report at Codecov.

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


Filename image.jpg.jpg

Should not stutter the extension

occurs with photo from WhatsApp
and from acrobat reader: document.pdf.pdf
but not with photo from Gallery, resized and renamed by Cool Maze