URLs in stdout surrounded by '<' and '>' cause problems with some environments.

Quick Summary: URLs such as https://elm-lang.org/0.19.0/init when followed sometimes include trailing greater than character
Ex.

Check out https://elm-lang.org/0.19.0/init for all the answers!

Clicking the corresponding link in the terminal included with Visual Studio Code on Windows 10 leads to the URL https://elm-lang.org/0.19.0/init> which results in a 404 error. This may be a problem with vscode, powershell, or Windows 10 specifically.

Elm: 0.19 bugfix6
Browser: N/A
Operating System: Windows 10

Additional Details
This issue could also be a problem in other places I'm unaware of. A workaround that would likely work is to change the format of the URL in stdout as defined by makeLink to put some white space between the angle brackets and the URL or to use a different format altogether.

This issue is also present when calling elm reactor, since it produces a message like:
Go to <http://localhost:8000> to see your project dashboard.

which I can't open by CTRL+ clicking the link


Adding newest cf-icons

Added

Adds icons for history, table of contents, closing quote, livestream, parents and servicemembers

Nice! Thanks for adding these @KimberlyMunoz!

On IE 6/8/9

IE 6

IE 8

IE 9

IE 10

IE 7 ??? :stuck_out_tongue_winking_eye:

IE 7 is the worst... and busted on my machine, so hold tight.

IE 6 ?!?!?! :rage4:

FTFY, @anselmbradford

Haha @Scotchester that's a collision of painfully unfun and fun old memories..

In all seriousness, we don't support IE6. Don't spend any time on it. And it's debatable how much longer we're going to support IE7.

:+1: Nicely done.

Tagged a release for this so we can pull it into cfgov-refresh... hope that was alright! Edit as needed.


Python 3.9.2 on Debian 11 w/ Git Diff Patch that fixes "Rip DVD as Data" and "= alignment" failures

Is there an existing issue for this?

[X] I have searched the existing issues

I have checked the wiki

[X] I have searched the wiki for this problem

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
Two Issues:

Error when ripping that states "= alignment not allowed in string format specifier"
None Tracks found by Handbrake (sort of) resulting in rip as Data Discs.

Additional issue in that for 1 it said that a trace would follow (in the logs) and didn't.
Additional complexity in that arm_wrapper.sh did properly identify the disc as DVD so /var/logs/arm.log wasn't useful.
To Reproduce
Installed ARM on fresh install of Debian 11. Tried to rip DVDs.
What OS are you seeing the problem on?
Debian 11
ARM Release Version or Git hash (You can find this in the arm settings page)
d33a6c6b3418e01f764a34f5fdcb097d7c50de6d
Expected behavior?
Proper Rip as DVD and without faulting
Relevant log output
I deleted the logs while debugging... Sorry...

Anything else?
Git Diff attached. I MIGHT be able to make a pull request but these are two tiny changes in handbrake.py and my dev environment is borked right now and my repository fork is woefully behind.
The first is the filename line and is the result of trying to format a string as a number.
Fix: Changed ":02" to ":0>2"
Comment: As detailed in this Stackoverflow post. Not sure if there's a safer/better way to handle this but this worked for me.
https://stackoverflow.com/questions/36044676/text-formatting-error-alignment-not-allowed-in-string-format-specifier.
The second is the Regex for result includes an unescaped parenthesis set.
Fix: Removed the "(s)" from the Regex.
ARM Issues 2022-08-26.txt
Code of Conduct

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

Sorry about the late reply.
Please use the main repo https://github.com/automatic-ripping-machine/automatic-ripping-machine


error while executing scripts/test.py

I am getting the following error while running test.py. I have installed all the dependencies.
here is the log:
Checking mygraph submodule ...
Traceback (most recent call last):
File "scripts/test.py", line 13, in
raise RuntimeError("mygraph submodule not avaiable: {}".format(str(e)))
RuntimeError: mygraph submodule not avaiable: /home/raavan/mountedDrives/dynamicGraphs/DynamicTriad-master/core/mygraph.so: undefined symbol: _ZN5boost6python6detail11init_moduleER11PyModuleDefPFvvE

After some googling, I've found a related stackoverflow question here.
It seems to be an incompatibility problem between python versions, that the function boost::python::detail::init_module is defined differently in python 2 and 3.
As for your specific problem, your loader is looking for a python 3 function, which is unusual as the build script requires a python 2.7 environment.
I'm not sure about the exact reason, but my guess is that you are somehow building the module with python 3 headers in combination with python 2 libraries.
Please check if this is the case. (p.s. the key to debugging this problem is to trace the value of the macro  PY_VERSION_HEX during compilation).


Application names should support whitespaces and special characters

If I give --conf spark.app.name="spark pi"  as parameter (a name with a white space), this exception is thrown ( Invalid value: "spark pi-1510599807833-driver": a DNS-1123 subdomain must consist of lower case alphanumeric characters) - I guess some code could replace all whitespaces with some hyphen to make DNS happy - To make it more robust, a more advance normalization (special characters...) should also be done.
Exception in thread "main" io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: POST at: https://<IP_ADDRESS>:6443/api/v1/namespaces/default/pods. Message: Pod "spark pi-1510599807833-driver" is invalid: metadata.name: Invalid value: "spark pi-1510599807833-driver": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*'). Received status: Status(apiVersion=v1, code=422, details=StatusDetails(causes=[StatusCause(field=metadata.name, message=Invalid value: "spark pi-1510599807833-driver": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*'), reason=FieldValueInvalid, additionalProperties={})], group=null, kind=Pod, name=spark pi-1510599807833-driver, retryAfterSeconds=null, uid=null, additionalProperties={}), kind=Status, message=Pod "spark pi-1510599807833-driver" is invalid: metadata.name: Invalid value: "spark pi-1510599807833-driver": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*'), metadata=ListMeta(resourceVersion=null, selfLink=null, additionalProperties={}), reason=Invalid, status=Failure, additionalProperties={}).
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.requestFailure(OperationSupport.java:470)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.assertResponseCode(OperationSupport.java:409)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.handleResponse(OperationSupport.java:379)
	at io.fabric8.kubernetes.client.dsl.base.OperationSupport.handleResponse(OperationSupport.java:343)

Agreed, we should probably do some stronger normalization to ensure using app names as kubernetes resources doesn't cause characters to be reused in places where they're invalid


i can't play with immersive portals mod with optifabric the optifabric is 1.18.2 1.13.0 and the immersive portals mod is 1.3.6

Only the latest version is being supported. If you are using an outdated version, upgrade to the latest version.
Provide instructions for how to reproduce the issue. Include the minecraft version and mod version.
Forge version issues belong to  https://github.com/qouteall/ImmersivePortalsModForForge/issues
If it's a mod compatibility issue, post that mod's link and open another issue on that mod

Optifine/Optifabric is not supported by Immersive Portals. Optifine is closed-source, which means mod compatibility is very difficult to achieve. Use the supported mods Sodium for lag prevention and Iris for shaders (most shaders that work with optifine also work with Iris).


removed all but one apt-get update from roles

It was suggested to only run apt-get update on the first role we run to elevate repetitive actions. I have made the fix. @heckj @benbp

+1

I think it makes more sense to have the update be its own role, rather than just keeping it in the build-essential role.

But yeah, this should shave off several minutes.

+1

:+1:


Bug/circuit gen for variable_gain_amplifier

The basic steps of read_netlist, match_graph, and write_verilog FAIL on variable_gain_amplifier:
It does not matter whether flat is set to 0 or 1.
(general) root@desmond-v01:/dataVolume/variable_gain_amplifier/topology_output#
(general) root@desmond-v01:/dataVolume/variable_gain_amplifier/topology_output# !211
python3 /ALIGN-public/sub_circuit_identification/src/read_netlist.py --dir ./input_circuit/ -f variable_gain_amplifier.sp --subckt variable_gain_amplifier --flat 1
Reading netlist file: variable_gain_amplifier.sp
INFO: PARSING INPUT NETLIST FILE DONE
circuit graph written in dir: circuit_graphs
(general) root@desmond-v01:/dataVolume/variable_gain_amplifier/topology_output# !212
python3 /ALIGN-public/sub_circuit_identification/src/match_graph.py
Start matching graphs
(general) root@desmond-v01:/dataVolume/variable_gain_amplifier/topology_output# !214
python3 /ALIGN-public/sub_circuit_identification/src/write_verilog_lef.py -U_cap 12 -U_mos 12
Traceback (most recent call last):
File "/ALIGN-public/sub_circuit_identification/src/write_verilog_lef.py", line 422, in
ALL_LEF, UNIT_SIZE_MOS, UNIT_SIZE_CAP)
File "/ALIGN-public/sub_circuit_identification/src/write_verilog_lef.py", line 328, in generate_lef
" --length " + str(values['l']))
KeyError: 'w'

fixed in #245

@desmonddak It has been fixed and merged to master, did you get a chance to try it again or are you facing any other issues?


embassy timer impl is very inaccurate

Starting with a fresh (template-generated) project, and adding
embassy-time = { version = "0.1.5", features = ["generic-queue"] } to my dependencies,
and with the following code:
use esp_idf_svc::hal::task::block_on;
use embassy_time::{Duration, Timer};

fn main() {
    esp_idf_svc::sys::link_patches();
    esp_idf_svc::log::EspLogger::initialize_default();

    block_on(async {
        for _ in 0..10 {
            log::info!("tick");
            Timer::after(Duration::from_secs(1)).await;
        }
    });
}

I get the output:
I (571) esp_idf_embassy_time: tick
I (2151) esp_idf_embassy_time: tick
I (5311) esp_idf_embassy_time: tick
I (11621) esp_idf_embassy_time: tick
I (24251) esp_idf_embassy_time: tick
I (49501) esp_idf_embassy_time: tick
I (100011) esp_idf_embassy_time: tick

which is very inaccurate.
The same code produces better results when using the esp-idf-svc/embassy-time-isr-queue feature and removing the embassy-time/generic-queue feature

Should be fine now.

@ivmarkov I hate to bother you with this, but next time please put at least one character into the commit message that isn't the issue number you're fixing. It's pretty annoying to find what has changed :D

I will, but in the meantime... :P
[image: For_Daniel.png]
On Sat, Jan 13, 2024 at 4:55 PM Dániel Buga @.***>
wrote:

@ivmarkov https://github.com/ivmarkov I hate to bother you with this,
but next time please put at least one character into the commit message
that isn't the issue number you're fixing. It's pretty annoying to find
what has changed :D
—
Reply to this email directly, view it on GitHub
https://github.com/esp-rs/esp-idf-svc/issues/342#issuecomment-1890479449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAT4TZJJN7BW3PCGIECYTITYOKN4ZAVCNFSM6AAAAABBYANZI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQGQ3TSNBUHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add preprint for  On assisting scientific data curation in collection-based dataflows using labels

http://dx.doi.org/10.1145/2534248.2534249 is not open access.
Could Pinar Alper, Carole Goble or @khalidb provide a preprint?

@khalidb provided postprint, updated eScholar uk-ac-man-scw:236944 (PDF should appear in a few hours)


Filter out automatically created resources

Some resources, like ResourceBindings, might be automatically created by a controller.
We should find a way to avoid extracting them.
Example:
apiVersion: authorization.openshift.io/v1
groupNames: null
kind: RoleBinding
metadata:
  annotations:
    openshift.io/description: Allows deploymentconfigs in this namespace to rollout
      pods in this namespace.  It is auto-managed by a controller; remove subjects
      to disable.

@dmartinol Do you mean resources with  .metadata.ownerReferences block?

Thanks for showing this powerful feature of kustomize, actually I was referring to resources automatically created by some controllers that have no owner field: resources that have an actual owner are automatically removed from the extracted configuration by Konveyor crane.
As you can see in the initial example, it has apiVersion authorization.openshift.io instead of the usual rbac.authorization.k8s.io, and the annotation says:

It is auto-managed by a controller
so we should avoid re-creating it, correct?
The point is how to identify such resources, as there is no other annotation that can immediately mark them. Just looking at the existence of an annotation called openshift.io/description might be misleading, WDYT?

Oh , I See.
In this particular case, i think that we need to identify that the resource provisioned automatically by openshift.
In this particular case, it has to do with the self-provisioning a project on openshift.
So there is an easy solution to address that issue and to identify these resources.

whenever you're  creating a project using the self-provisioning functionality(oc new-project project-name, in this way, unprivileged users granted the authorization to create themselves projects and to be an admin on that specific project, this is granted by adding the user the self-provisioner role), there are basically the following resources on an empty provisioned project:

 oc new-project dummy-project
Now using project "dummy-project" on server "https://api.tran-temenos.kni.syseng.devcluster.openshift.com:6443".

You can add applications to this project with the 'new-app' command. For example, try:

    oc new-app rails-postgresql-example

to build a new example application in Ruby. Or use kubectl to deploy a simple Kubernetes application:

    kubectl create deployment hello-node --image=k8s.gcr.io/serve_hostname

 oc api-resources | grep -v NAME | awk '{ if (($4=="true" || $3=="true") && ($3!=false  )) { print $1 }}' | xargs -i oc get {}

NAME                       DATA   AGE
config-service-cabundle    1      13m
config-trusted-cabundle    1      13m
kube-root-ca.crt           1      13m
openshift-service-ca.crt   1      13m
NAME                       TYPE                                  DATA   AGE
builder-dockercfg-87qxk    kubernetes.io/dockercfg               1      13m
builder-token-2f4sj        kubernetes.io/service-account-token   4      13m
builder-token-5kbpb        kubernetes.io/service-account-token   4      13m
default-dockercfg-lllqk    kubernetes.io/dockercfg               1      13m
default-token-2r52d        kubernetes.io/service-account-token   4      13m
default-token-bcsrt        kubernetes.io/service-account-token   4      13m
deployer-dockercfg-bxzjn   kubernetes.io/dockercfg               1      13m
deployer-token-7pxs4       kubernetes.io/service-account-token   4      13m
deployer-token-9wx98       kubernetes.io/service-account-token   4      13m
pipeline-dockercfg-rm272   kubernetes.io/dockercfg               1      13m
pipeline-token-2tm4d       kubernetes.io/service-account-token   4      13m
pipeline-token-bzlsh       kubernetes.io/service-account-token   4      13m
NAME       SECRETS   AGE
builder    2         13m
default    2         13m
deployer   2         13m
pipeline   2         13m

NAME                        ROLE                                    AGE
admin                       ClusterRole/admin                       14m
openshift-pipelines-edit    ClusterRole/edit                        14m
pipelines-scc-rolebinding   ClusterRole/pipelines-scc-clusterrole   14m
system:deployers            ClusterRole/system:deployer             14m
system:image-builders       ClusterRole/system:image-builder        14m
system:image-pullers        ClusterRole/system:image-puller         14m

All such resources, should be ignored , as they're generated by openshift when a new project is created/self provisioned.

In addition, you can change the template of creating new project(so potentially there are more resources that can be auto-generated) , by following guide in:

https://docs.openshift.com/container-platform/4.9/applications/projects/configuring-project-creation.html#modifying-template-for-new-projects_configuring-project-creation
4.if we take , for example, rolebinding system:deployers, then we can verify that it's present on both namespaces - dummy-project and openshift
[zgrinber@zgrinber docker-containers]$ oc get rolebinding system:deployers -o yaml
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  annotations:
    openshift.io/description: Allows deploymentconfigs in this namespace to rollout
      pods in this namespace.  It is auto-managed by a controller; remove subjects
      to disable.
  creationTimestamp: "2023-01-02T16:13:23Z"
  name: system:deployers
  namespace: dummy-project
  resourceVersion: "220535727"
  uid: 86ecd144-6fe6-4690-a261-4755ed4f9909
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: system:deployer
subjects:
- kind: ServiceAccount
  name: deployer
  namespace: dummy-project
[zgrinber@zgrinber docker-containers]$ oc get rolebinding system:deployers -o yaml -n openshift
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  annotations:
    openshift.io/description: Allows deploymentconfigs in this namespace to rollout
      pods in this namespace.  It is auto-managed by a controller; remove subjects
      to disable.
  creationTimestamp: "2022-04-19T15:39:45Z"
  name: system:deployers
  namespace: openshift
  resourceVersion: "13565"
  uid: 4a058f12-5d01-4a31-86b1-ef6b8e17a4d3
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: system:deployer
subjects:
- kind: ServiceAccount
  name: deployer
  namespace: openshift

So we can infer from that , that one way to determine if to filter out a resource is to check if it's in the above list of a new created project, or to check if it's exists in another namespace(for example openshift namespace).

For the demo:
find an annotation matching against "openshift.io/description: .... It is auto-managed by a controller" and discard it.


[Question] How many time does release build needed on MacOs?

Hi,
cargo tauri build  was hang for about 4 hours for my empty demo(crated by cargo tauri init),  Not sure if this is a normal case.

I notice that rustc is still working

Compiled failed finally...  :(
(signal: 9, SIGKILL: kill)
Error: failed to build app

Same or similar issue here on Ubuntu 21.04. Building in release mode hangs in the building step for a undefinite period of time.
I'm using latest beta.10 version. No web frameworks (so no Vue, Angular or similar) are used but part of the code uses WASM (if this is somehow relevant).
Any ideas what could be the reason?

Can you guys share your tauri info output? Specifically I'd like to know the distDir on your tauri.conf.json file.

Hi Lucas,
after your comment, I was able to fix the issue. In my case, I was executing the npm run tauri build command from parent directory of src-tauri (in which index.html is located). But it looks like that distDir was looking for index.html not relative to src-tauri but relative to my current directory. After changing distDir to ./, everything started working.
Not sure if I was using tauri wrong - if yes sorry for confusion!
Sergej

distDir is relative to src-tauri, but you can't use ./ since that'll increase your bundle size (including target folder etc).


Keogram li

new parameter -x (expands image to get the proportions of source)
new parameter -c (show channel infos - mean value)
text with black background
show date
calculate mean value only in the inner circle (mask) - without steetlights, cars,...

./keogram_li -d images/20211031 -e jpg -o test_li.jpg -x -c

daylight saving:

without changes:

There shouldn't need to be commas between the three values. All the scripts that call keogram put the color values in quotes so getopt sees it as one argument that happens to have two spaces in it. Do you have different scripts that don't put quotes around the color numbers?

yesterday I'have seen the problem of optarg with this parameter. Only >"255< was delivered as argument.
the change to 255,255,255 was a quick and dirty hack
have you checked the functionality of optarg with this parameter ?

Please add the "x" and "c" options to "usage_and_exit()".

Done.
-x | --image-expand : expand image to get the proportions of source
-c | --channel-info : show channel infos - mean value of R/G/B

@AndreasLMeg
First off, thanks for submitting this PR.  I assume it implements your nice changes to keogram?  I seem to recall you had an "addRows" argument, but that's not in this PR.  Did you rename it?
Here's the command that executes on my Pi using the current software on Thomas' GitHub:
'/home/pi/allsky/keogram' -d '/home/pi/allsky/images/20211031' -e jpg -o '/home/pi/allsky/images/20211031/keogram/keogram-20211031.jpg' --rotate 0 --font-size 1.0 --font-line 1 --font-color '255 255 255'

We added a variable to config.sh to allow the user to easily pass arguments to keogram, it's called "KEOGRAM_EXTRA_PARAMETERS" (different name than you have but it does the same thing).  Here's what I have:
KEOGRAM_EXTRA_PARAMETERS="--rotate 0 --font-size 1.0 --font-line 1 --font-color '255 255 255'"
What does your keogram command look like?

@AndreasLMeg, another option for the ' ' versus ',' is to allow either.  It'll only add a couple lines of code.

Setting nchan to 3 eliminates the crashes when there are some images with 1 channel and some with 3, but if all the images are mono it will "fix" them all to color. I have a version of keogram.cpp that looks at the first image and uses it to determine the "nchan". I'll do a PR for that after your PR is merged.

my suggestion: a parameter f
-f | --fixed-channel-number  : define number of channels 0=auto, 1=mono, 3=rgb (0=auto)
pi@allsky:~/allsky_keogram_li/src $  ./keogram -d ~/allsky/images/20211031 -e jpg -C "255 255 255" -o test_li.jpg -x -c
OpenCV Error: Assertion failed (channels() == CV_MAT_CN(dtype)) in copyTo, file /build/opencv-L65chJ/opencv-3.2.0+dfsg/modules/core/src/copy.cpp, line 259
terminate called after throwing an instance of 'cv::Exception'
what():  /build/opencv-L65chJ/opencv-3.2.0+dfsg/modules/core/src/copy.cpp:259: error: (-215) channels() == CV_MAT_CN(dtype) in function copyTo
Abgebrochen
and now with the parameter -f 3
pi@allsky:~/allsky_keogram_li/src $  ./keogram -d ~/allsky/images/20211031 -e jpg -C "255 255 255" -o test_li.jpg -x -c -f 3
pi@allsky:~/allsky_keogram_li/src $

@AndreasLMeg, Once I do the PR to fix the crashes when there are both 1-channel and 3-channel images, the "auto" mode would then be based on the 1st image, which is what my fix looks at to determine what to use.

@AndreasLMeg, Once I do the PR to fix the crashes when there are both 1-channel and 3-channel images, the "auto" mode would then be based on the 1st image, which is what my fix looks at to determine what to use.

Unfortunately, the RaspHQ camera is always overexposed = WHITE (sunrise / sunset)
This means that the first pictures will always be mono. But I want RGB
I.e. your change will unfortunately not work for me
By the way: With the exposure control, this will no longer happen in the future

I'm still trying to find the problem ...
here is the version 255,255,255

/home/pi/allsky/keogram -d /home/pi/allsky/images/20211031/ -e jpg -o /home/pi/allsky/images/20211031/keogram/keogram-20211031.jpg --font-size 2.0 --font-line 2 --font-color 255,255,255 -x -c
optarg:<255,255,255>

and here "255 255 255" - You can see linux bash makes the Problem: changes "255 255 255" to  '"255' 255 '255"'

/home/pi/allsky/keogram -d /home/pi/allsky/images/20211031/ -e jpg -o /home/pi/allsky/images/20211031/keogram/keogram-20211031.jpg --font-size 2.0 --font-line 2 --font-color '"255' 255 '255"' -x -c
optarg:<"255>

@AndreasLMeg, when the HD camera outputs a white frame does it chang the number of channels to 1?  The code is checking for imagesrc.channels().  If that's true, I could easily change it to look at the middle picture.  I currently just have "0" hard coded for the file number.
Are you running the latrst files from Thomas?  If so, run scripts/generateForDay.sk -k 20211031.
If you don't have the latest code, look in that script to see what it's doing and try the same on your Pi.
It's strange that bash is adding the single quotes.  I wonder if that is country-dependant.

@AndreasLMeg, another option for the ' ' versus ',' is to allow either. It'll only add a couple lines of code.

I think that's the best solution
I haven't found the cause yet, but the problem already happens during import
source "$ {ALLSKY_CONFIG} /config.sh"

@EricClaeys: That's enough adjustments for today - good starry night

@AndreasLMeg, I will be interested to see what the problem is since we have a lot of people in different countries. No one else has mentioned that issue though, and quite a few are using the newer config.sh.

@AndreasLMeg, Chris originally allowed only hex numbers but I got him to also allow 0-255 since not everyone is a programmer who understands hex, and the capture programs accept 0-255 so that's what people are used to.  I think the capture programs should also allow both.

I have now done various tests and have come to the result: It must be the keogram itself. And I can only imagine the new option handling as the cause. But I'm not very familiar with that - so I keep my hands off it
I suspect that this option "--font-color '255 255 255'" is rarely used. Can you do a test?
here is the keogram of last night (with activ gain control for RaspiHQ)

@AndreasLMeg, how about if I merge your approved change in, then post my PR that eliminates the crashes when there are images of different channel numbers?  I will also add in the ability to do either " " or "," separators for the font color.
I use "--font-color '255 255 255'" in my KEOGRAM_EXTRA_PARAMETERS variable since I find dark blue on black hard to see.

@EricClaeys: That would be nice of you. But only if the ideas fit into Allsky's concept and it is an improvement.
I still think the idea with the "finishline" is good. But I'll make that available later.
For me, keogram is always the best way to get a quick overview of the night, which is why improvements are very important to me.
Thank you very much for your efforts.


SA1005 Fix All in Solution appears to throw an exception

I'm working with Newtonsoft.Json.Portable.sln from the recently added churn tests, and ran StyleCopTester with /id:SA1005 /fixall. I haven't yet tracked down the cause.
The fix 'SA1005CodeFixProvider' threw an exception after 34ms:
System.ArgumentException: The changes must be ordered and not overlapping.
Parameter name: changes
   at Microsoft.CodeAnalysis.Text.SourceText.WithChanges(IEnumerable`1 changes)
   at StyleCop.Analyzers.SpacingRules.SA1005CodeFixProvider.FixAll.<FixAllInDocumentAsync>d__5.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCop.Analyzers.CodeFixes\SpacingRules\SA1005CodeFixProvider.cs:line 106
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at StyleCop.Analyzers.Helpers.DocumentBasedFixAllProvider.<GetSolutionFixesAsync>d__5.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCop.Analyzers.CodeFixes\Helpers\DocumentBasedFixAllProvider.cs:line 107
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.<ComputeOperationsAsync>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.<GetOperationsCoreAsync>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.CodeActions.CodeAction.<GetOperationsAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at StyleCopTester.CodeFixEquivalenceGroup.<GetOperationsAsync>d__23.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCopTester\CodeFixEquivalenceGroup.cs:line 141
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at StyleCopTester.Program.<TestFixAllAsync>d__3.MoveNext() in J:\dev\github\sharwell\StyleCopAnalyzers\StyleCop.Analyzers\StyleCopTester\Program.cs:line 198

This was caused by a bug in #1983, and was corrected in 5fa8d695.


Version main to pre-release 0.3.0a0

Modulus Pull Request
Closes: https://github.com/NVIDIA/modulus-launch/issues/74
Description

Updating main branch to a higher pre-release version. This will make sure install from source always has priority over
Checklist

[ x ] I am familiar with the Contributing Guidelines.
[ x ] New or existing tests cover these changes.
[ x ] The documentation is up to date with these changes.
[ x ] The CHANGELOG.md is
up to date with these changes.

Dependencies

/blossom-ci


fix: Fix object() validator bug

@allcontributors please add @wisnie for bug

@allcontributors please add @wisnie for bug

Bundle size comparison

size comparison

browser
uncompressed 5.49 kB  ▶ +1 B (+0.02% :small_red_triangle:) gzipped 2 kB  ▶ +10 B (+0.50% :small_red_triangle:)

main
uncompressed 5.37 kB  ▶ +1 B (+0.02% :small_red_triangle:) gzipped 1.92 kB  ▶ +8 B (+0.42% :small_red_triangle:)

module
uncompressed 5.26 kB  ▶ +1 B (+0.02% :small_red_triangle:) gzipped 1.92 kB  ▶ +10 B (+0.52% :small_red_triangle:)

Speed comparison
PR

library
relative speed
operations per second
avg. operation time

@typeofweb/schema@pr-branch
+16.08%
(1,679,783 rps)
(avg: 595ns)

@typeofweb/schema@base-branch
0%
(1,447,130 rps)
(avg: 691ns)

<EMAIL_ADDRESS>-91.33%
(125,413 rps)
(avg: 7μs)

<EMAIL_ADDRESS>-94.4%
(81,068 rps)
(avg: 12μs)

<EMAIL_ADDRESS>-96.17%
(55,431 rps)
(avg: 18μs)

<EMAIL_ADDRESS>-98.4%
(23,085 rps)
(avg: 43μs)

Bundle size comparison

size comparison

browser
uncompressed 5.49 kB  ▶ +1 B (+0.02% :small_red_triangle:) gzipped 2 kB  ▶ +10 B (+0.50% :small_red_triangle:)

main
uncompressed 5.37 kB  ▶ +1 B (+0.02% :small_red_triangle:) gzipped 1.92 kB  ▶ +8 B (+0.42% :small_red_triangle:)

module
uncompressed 5.26 kB  ▶ +1 B (+0.02% :small_red_triangle:) gzipped 1.92 kB  ▶ +10 B (+0.52% :small_red_triangle:)

Speed comparison
PR

library
relative speed
operations per second
avg. operation time

@typeofweb/schema@pr-branch
+16.08%
(1,679,783 rps)
(avg: 595ns)

@typeofweb/schema@base-branch
0%
(1,447,130 rps)
(avg: 691ns)

<EMAIL_ADDRESS>-91.33%
(125,413 rps)
(avg: 7μs)

<EMAIL_ADDRESS>-94.4%
(81,068 rps)
(avg: 12μs)

<EMAIL_ADDRESS>-96.17%
(55,431 rps)
(avg: 18μs)

<EMAIL_ADDRESS>-98.4%
(23,085 rps)
(avg: 43μs)


Contact Us

[ ] Khine Zar Thwe for General cases and Donations
[ ] U Yawatta for Donation and Student, School infos
[ ] Hnin Oo Wai Hlaing and () for donations and account
[ ] ChanMyaeSanHlaing and Sis Min Maw for

@yeemon04  Hi Mama! Please reference this infos  for Contact Us Page


Add buffer and time-to-start metrics

app-side PR https://github.com/lbryio/lbry-desktop/pull/3877

👍


Swipe to send page up/down does not work

Even though I have checked "Page up/down gesture" in the settings, swiping on the left side of the screen does not send page up/down keys. It only scrolls the terminal, like the rest of the screen.
This is with the current newest master (c9682d3). The error was introduced by #277, as stated by @rhansby in #270.

This occurs in v1.9.0 which auto-updated onto my Nexus 5x today.  After realizing the gesture no longer worked, I enabled it in settings.  Even after an app exit (and Force Stop), the setting did not take effect.

Ah, this only works on pre-Honeycomb devices currently because the TerminalTextViewOverlay used for text selection is causing the onScroll to bail out early.

If it ends up being difficult to fix this, I would be more than happy to have a switch to go back to the old behavior regarding scrolling and copy/paste (my naive impression of what TerminalTextViewOverlay is used for).  I use this heavily for irssi and as a result I copy very rarely, but page up and down constantly.


Update BDD rules with multi language support

Related #646

@bhirsz Can you explain what should be done here?

We have some rules regarding BDD syntax. It only supports english names, and we would need to support translated keywords like "Jeżeli" "Und"..

Are they all available through RF API?

No.. its bit more complicated with BDD. We need to load the language - user need to provide langage shebang or use --langauge option . Then we can use robot API to load the possible BDD keywords and use for rules. I did it on the translate transformer in Robotidy: https://github.com/MarketSquare/robotframework-tidy/blob/main/robotidy/transformers/Translate.py
If we have some rules that rely on keyword name, we would also need to give BDD support for it (ie Then Return From Keyword?).


Allow the GM to drag lights from a character's inventory onto a scene

The GM should also be able to add a dropped light source back to a character with a selector to choose who to give it to on the Light actor sheet.


Adaptive EntityDecoder[Json] for circe

After doing some benchmarks I've decided to make an instance that combines performance of decoding via ByteBuffer for small payloads and via jawn-fs2 for big payloads.
Default cutoff value of 100000 was chosen by doing benchmarks with different cutoff value. Mesages with Content-Length lower than cutoff are handled by ByteBuffer parser. Everything higher than cutoff or when Content-Length is unknown is handled by incremental decoder.
[info] Benchmark                          (approxContentLength)  (cutoff)  Mode  Cnt        Score       Error  Units
[info] CirceJsonBench.decode_adaptive                      1100    100000  avgt   20    28218.335 ±   147.533  ns/op
[info] CirceJsonBench.decode_adaptive                      5000    100000  avgt   20    76649.070 ±  1259.016  ns/op
[info] CirceJsonBench.decode_adaptive                     10000    100000  avgt   20   123584.282 ±   704.567  ns/op
[info] CirceJsonBench.decode_adaptive                    100000    100000  avgt   20  1013073.677 ± 24022.673  ns/op
[info] CirceJsonBench.decode_adaptive                    500000    100000  avgt   20  5211421.634 ± 68338.766  ns/op
[info] CirceJsonBench.decode_byte_buffer                   1100    100000  avgt   20    28265.613 ±    94.894  ns/op
[info] CirceJsonBench.decode_byte_buffer                   5000    100000  avgt   20    75760.460 ±   887.846  ns/op
[info] CirceJsonBench.decode_byte_buffer                  10000    100000  avgt   20   123096.025 ±   667.088  ns/op
[info] CirceJsonBench.decode_byte_buffer                 100000    100000  avgt   20  1088689.815 ± 11907.722  ns/op
[info] CirceJsonBench.decode_byte_buffer                 500000    100000  avgt   20  5534396.999 ± 42943.154  ns/op
[info] CirceJsonBench.decode_incremental                   1100    100000  avgt   20    55218.931 ±   736.221  ns/op
[info] CirceJsonBench.decode_incremental                   5000    100000  avgt   20    95511.884 ±  1571.320  ns/op
[info] CirceJsonBench.decode_incremental                  10000    100000  avgt   20   140359.831 ±  2344.243  ns/op
[info] CirceJsonBench.decode_incremental                 100000    100000  avgt   20  1008533.778 ± 18852.809  ns/op
[info] CirceJsonBench.decode_incremental                 500000    100000  avgt   20  5153683.990 ± 49192.662  ns/op

jsonOf refers to jsonDecoder, which means users of it are stuck with adaptive decoding at our default cutoff

how about making jsonDecoder abstract and specialize it to jsonDecoderAdaptive(cutoff = 100000) in package object (and CirceInstances object?)

Yeah, the abstract idea is similar to how we handled the default pretty printer.
Summoning the implicit EntityDecoder[Json] wouldn't work well with the jsonOf[A] syntax.

I'll do the backporting a bit later this week.

I've made jsonDecoder abstract


[Model Serving]: Add authentication for external route

Feature description
Follow up of #651, add authentication for the external route (Mocks)

Add ability to enable token authorization.
Add ability to generate token with custom name (Mocks).
Bind the token to the model server.

Close this in favor of #642


ABORT when creating an association in rtLinux

There is an ABORT chunk sent out immediately by client when it receive INIT_ACK from server. This problem was only found on rtLinux OS, but return to normal when we remove RT patch from kernal.
the wireshark trace is like this:

ps: the verification tag in ABORT is the same as INIT_ACK. So it looks like the abort chunk is sent by usrsctp. But i can't find where it is from. The procedure is keep on and the COOKIE_ECHO is sent out as normal.

It would be easier to look at the .pcap file. Are you using two different physical machines. If you see such an ABORT, it normally comes from having the kernel stack enabled in addition to using a userland stack.  I'm not sure about the two INIT-ACK chunks. Are they a timer based retransmission? A .pcap file would help here.

sorry for my unclear description. I think two INIT_ACK chunks is due to repeat capturing, since the peer is run in VM_Ware. If I capture pcap in VM_Ware, we could see only one INIT_ACK.

Are you running rtLinux as a VM in WMWare? Please provide a capture from within the VM.

The rtLinux is running at client. The server is running on VMWare. This is the log captured within the VMWare.

usrsctp_conn_abort.zip

The client uses usrsctp, runs on the host OS and uses <IP_ADDRESS>.
The server uses a Linux kernel stack, runs in the VM and uses <IP_ADDRESS>.
What happens is that whenever you sent an SCTP packet from VM towards the host, some instance sends a packet with an ABORT chunk towards the the VM. So I guess that you have either on your host another SCTP stack running (could be a kernel stack, could be another instance of usrsctp) or you are using a NAT between the VM and the Host. In the first case, ensure you are running only a single instance of SCTP. What is the host OS? If it is Linux, you can check with lsmod what kernel modules are loaded. If SCTP is listed there, you have your other stack. To avoid NAT interactions, please use a bridged interface.

yes. It shows sctp is loaded in the host OS by the command of lsmod. So does it means the kernal sctp service is also working? how can i remove it? when i use rmmod ,it shows sctp is in use. Thank you very much.

Yes, the Linux kernel implementation is also active. I'm not a Linux expert, but at some point of time you could enforce the unload. The other possibility is to reboot the Linux system and avoid that the kernel extension is loaded. How you achieve this, depend on the reason why it was loaded.
I don't think I can help there a lot, since this is a linux admin problem for the host OS. So I'm closing this.


HelenTraining does not work

DlibDotNet in src is too old.
So examples program try to build src/DlibDotNet and occurs the gap of interface issue.
Therefore, HelenTraining example should refer nuget package.

README.md indicates how to refer nuget pacakge.
So close.


Solana Block Missing message field

Solana firehose protobuf defines a few fields in the TransactionStatusMeta that are not autogenerated in the substreams build.rs file.
Those fields are:

loaded_writeable_addresses
loaded_readable_addresses
address_table_lookups
program_id

Hi @dmelotik,
The fields address_table_lookups and program_id are not part of the TransactionStatusMeta proto message. You can check here.
Also, the fields for loaded_writeable_addresses and loaded_readable_addresses were added after the spkg containing  the block was generated. The spkg was generated on June 3 and the proto was updated on Sept 29th

Hi @Eduard-Voiculescu thanks for the response! That was a mistake, address_table_lookup is in the Message and program_id is in TokenBalance. Either way it is the same issue.
What is the timeline on being able to get these in the firehose solana stream?

Thanks @Eduard-Voiculescu !

@jubeless was our data plucker from bigtable not updated to the latest .proto perhaps? We could rerun and overwrite some of the blocks that don't have the latest data. Let's discuss.


error: Undefined name 'navigator'

I have a StatefulWidget for page in flutter. On the press of a button I call the following method:
IconButton(
           icon: Icon(Icons.photo),
           iconSize: 25.0,
           color: Theme.of(context).primaryColor,
           onPressed: (){
             Navigator.push(
               context,
               MaterialPageRoute(builder: (context) => CallPage(title:contact)),
             );
           },
         ),

CallPage is another statefulWidget from which I am trying to create local and remote RTCVideoRenderers.
I use the following code to get the userMedia: video and audio from the navigator.
_getUserMedia() async {
   final Map<String, dynamic> mediaConstraints = {
     'audio': false,
     'video': {'facingMode': 'user'}
   };

   if(await Permission.camera.request().isGranted) {
     if(await Permission.microphone.request().isGranted){
       MediaStream _localStream = await navigator.getUserMedia(mediaConstraints);
       _localRenderer.srcObject = _localStream;
       return _localStream;
     }
   }
 }

Here, the Dart Analysis is throwing an error:

error: Undefined name 'navigator'. (undefined_identifier at [chatapp] lib/home/call_page.dart:99)

Wher line 99 calls navigator.getUserMedia.
When I try to run the application it says this:

Error: The getter 'navigator' isn't defined for the class '_MyHomePageState'.

_MyHomePageState is the state which extends my StatefulWidget.
 This exact app was working a few minutes ago. Suddenly it stopped working.
I have tried deleting the build directory, running flutter clean, restarting the project, everything.
What am I doing wrong?

Class name navigator was changed to mediaDevices. in flutter-webrtc package just 1 day ago!.


/apps/rich-api - Providing a Rich List Index

What would you like to be added:
Providing a Rich List index & endpoint (top 1000) service. This service doesn't need to be live but we need to be able to provide a token-rich list index.
/triage accepted
/assign @ivan-zynesis
/area apps
/priority important-soon

# Jellyfish
## packages
- ### jellyfish-ops-adapter?
  - #### QueueModuleInterface
  - #### PersistentStorageInterface
  - #### RichListServer (with 2 of above injected)
    - create('channel', LIFO)
    - publish('channel', 'msg')
    - read('channel')
    - address watcher -> push address to queue
    - daemon: read from queue + existing -> update new rich list
- ### ocean-api-client
  - #### api/rich-list
    - /GET list

## apps
- ocean-api
  - controllers/rich-list
  - modules
    - QueueModule
    - PersistentStorageModule
    - RichListServerModule

 controller

# Jellyfish-ops?
- QueueModule: implement with SQS
- PersistentStorageModule: even simple text could work</br>
  (if only for rich list storage purpose, eg daemon run at daily/hourly basis only)


Type '({ gl }: { gl: any; }) => void' has no properties in common with type 'Partial'

In this code:
 gl={({ gl }) => {
        gl.toneMapping = THREE.ACESFilmicToneMapping;
        gl.outputEncoding = THREE.sRGBEncoding;
      }}

Typescript raises this error:

Type '({ gl }: { gl: any; }) => void' has no properties in common with type 'Partial'

Should this line:
https://github.com/react-spring/react-three-fiber/blob/dfe596ed07b7ca54de60fee101481a8f8a1dae5c/src/canvas.tsx#L94
be
gl?: (gl: Partial<THREE.WebGLRendererParameters>) => void

?

No, gl just takes props, you mean onCreated. Anyway, you could just write: <Canvas sRGB ... />

The above code is taken from the example in the repo :P https://codesandbox.io/embed/r3f-montage-jz9l97qn89

oops. a typo. :-) it does nothing, it's trying to spread a function and goes about its way, good that typescript complained!


Fix recursive filename after several rebuilds in development

Fixes #30

@arthurbergmz Actually there was a legitimate bug that was reported in #30.
This fixes the issue. Basically, the issue is that options.filename was being rewritten using manifest.file, which had the public path appended on each iteration. By using output, the issue goes away.

Do not merge yet, looking at the tests.

Testes were fixed!


the "last edit"-date is missing

the filters work perfect, but now the date in the content-footer is missing.
I'm not sure whether it has to do with #215, #221 or something different, but a completely unqualified guess from my side in that maybe this block needs a rework:
https://github.com/RyanNoelk/OpenEats/blob/master/frontend/modules/recipe/components/RecipeScheme.js#L179-L183

The API is no longer returning date information about the recipes; even though the database has the data. I'm taking a look.

This be fixed now.


Header and payload packet generation and parsing

Using randomly generated tests, header and payload packets successfully make it through the pipeline!

r? @oconnor663

Ok, think I hit all the comments above

@oconnor663 okay, I finished changing it from outputting msgpack'd stuff into outputting actual objects that we can pass to the msgpack stream. ready for review again

Ah sorry, meant to conclude with: :ship:!!!


v7.3.83 of Docker Image fails to run on ARMv6

Hi, thanks for a great project. Saved me a lot of headache. FYI, per https://github.com/chopeen/unifi-docker-raspi#supported-models, images from https://hub.docker.com/r/chopeen/unifi-docker-raspi/tags should be supported on ARMv6.

v7.1.65 - works fine!
v7.3.83 - is not supported. cat ./log/server.log errors out with something like 'VM is only supported on ARMv7+ VFP', because it auto selects openjdk-jre-11, instead of openjdk-jre-8, that is supposedly installed by Dockerfile instructions.

Thanks for submitting the bug!
My Raspberry was down for a while and today I finally turned it on, but could not start the Controller. It is your bug that made me realize I saw VM is only supported on ARMv7+ VFP before and was trying to solved it in #18.
I removed v7.3.83 from Docker Hub and for now we need to rely on the old version I'm afraid.


chore: namespaces filtering

Support to filter namespaces by specifying --app-namespace (multiple times if necessary) or adding app_namespace in configuration file. By default all namespaces will be watched.
What's more, after this PR is merged, e2e test cases can be run in parallel.
Closing #139

LGTM

LGTM


HBASE-23613 ProcedureExecutor check StuckWorkers blocked by DeadServe…

…rMetricRegionChore

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 45s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  1s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
6m 35s
master passed

+1 :green_heart:
compile
1m  7s
master passed

+1 :green_heart:
checkstyle
1m 36s
master passed

+1 :green_heart:
shadedjars
5m 31s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 44s
master passed

+0 :ok:
spotbugs
5m  3s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m  0s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
6m  4s
the patch passed

+1 :green_heart:
compile
1m  8s
the patch passed

+1 :green_heart:
javac
1m  8s
the patch passed

-1 :x:
checkstyle
1m 32s
hbase-server: The patch generated 1 new + 9 unchanged - 0 fixed = 10 total (was 9)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
5m 26s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
16m 42s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 37s
the patch passed

+1 :green_heart:
findbugs
4m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
166m 25s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 34s
The patch does not generate ASF License warnings.

230m 34s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/960

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux dcb084b57a05 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-960/out/precommit/personality/provided.sh

git revision
master / 4b6ce0fcbf

Default Java
1.8.0_181

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/1/artifact/out/diff-checkstyle-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/1/testReport/

Max. process+thread count
4630 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 46s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
6m 54s
master passed

+1 :green_heart:
compile
1m  5s
master passed

+1 :green_heart:
checkstyle
1m 33s
master passed

+1 :green_heart:
shadedjars
5m 39s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 41s
master passed

+0 :ok:
spotbugs
4m 32s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 31s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 58s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

+1 :green_heart:
checkstyle
1m 16s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 40s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 57s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 36s
the patch passed

+1 :green_heart:
findbugs
4m 26s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
167m  1s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 35s
The patch does not generate ASF License warnings.

228m 10s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/960

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 9412526c0836 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-960/out/precommit/personality/provided.sh

git revision
master / fc15ea7546

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/2/testReport/

Max. process+thread count
4609 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 32s
master passed

+1 :green_heart:
compile
0m 58s
master passed

+1 :green_heart:
checkstyle
1m 17s
master passed

+1 :green_heart:
shadedjars
4m 41s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

+0 :ok:
spotbugs
4m 19s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 16s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 55s
the patch passed

+1 :green_heart:
compile
0m 55s
the patch passed

+1 :green_heart:
javac
0m 55s
the patch passed

+1 :green_heart:
checkstyle
1m 15s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 39s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 50s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 34s
the patch passed

+1 :green_heart:
findbugs
4m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
167m  1s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 36s
The patch does not generate ASF License warnings.

224m 18s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/960

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 2fb1660aa563 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-960/out/precommit/personality/provided.sh

git revision
master / fc15ea7546

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/3/testReport/

Max. process+thread count
4682 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-960/3/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Working way to install for rails 7?

I have been struggling trying top get this installed and working with rails 7. Not sure if this gem is being worked on still, but was wondering if anyone could list the steps they have taken? I tried the methods listed in #153  and am getting errors on loading the javascript stating:
script: Expected a JavaScript module script but the server responded with a MIME type of "text/coffeescript". Strict MIME type checking is enforced for module scripts per HTML spec.

Wow i got it working. What I did was add this to importmap.rb
pin_all_from "app/javascript/recurring-select", under: "recurring-select"
Added the javascript files from #153 at the bottom under app/javascript/recurring-select. I had to change the jquery reference at the top of recurring_select_dialogue.js:
Old:
var $, RecurringSelectDialog;

$ = require("jquery");

New:
var RecurringSelectDialog;

import $ from "jquery";

Then in application.js I added  import "recurring-select".
In application.tailwind.css I added @import "recurring_select/recurring_select"; and oput the two files referenced in that location.
Seems to be working! I'll need to go through and tweak the styles to bring over to tailwind.

jQuery loads a bit differently in my Rails 7 app, so i followed your same instructions:

importmap.rb pin_all_from "app/javascript/recurring-select", under: "recurring-select"
pasted 2 JS files inside new app/javascript/recurring-select directory
pasted 2 CSS files inside new app/stylesheets/recurring_select directory

except inside the 2 JS files i replaced:
import $ from "jquery";

with:
var $ = jQuery;

thanks for sharing this!


Linked list add object for user defined functions

#4812 introduced the requirement for user defined comparison/new/delete callbacks for LinkedList values.
This pull implements that:

breaks ABI of wLinkedList (now opaque) and wLinkedListNode (now internal)
Adds LinkedList_Object to get a pointer to wObject to set the user defined callbacks
Modifies typedef void* (*OBJECT_NEW_FN)(void); to typedef void* (*OBJECT_NEW_FN)(); to allow new functions to have arguments

@hardening The only breaking points are the now opaque type and removed internal structs.
They shouldn't have been used anyway (and the LinkedList was not in heavy use)


Make detrending polynomial

🚀 Feature Request
There is a need to make detrending polynomial instead of linear.
Motivation
It will make possible to capture more difficult trends (e.g. quadratic).
Proposal

Add poly_degree parameter to _OneSegmentLinearTrendBaseTransform, LinearTrendTransform and TheilSenTrendTransform that indicates the degree of the polynom to fit during detrending. Default value should be 1 and not change anything.

Test cases

Add new fixtures with polynomial trend (degree=2) for one segment and many segments.
Test that fit_transform for one-segment model works with poly_degree=2 on new fixture.
Test that fit_transform for all-segments model works with poly_degree=2 on new fixture.
Test that inverse_transform for one-segment model works with poly_degree=2 on new fixture.
Test that inverse_transform for all-segments model works with poly_degree=2 on new fixture.

Alternatives
No response
Additional context
No response
Checklist

[ ] I discussed this issue with ETNA Team

Probably should use this https://scikit-learn.org/stable/modules/generated/sklearn.preprocessing.PolynomialFeatures.html


update dependencies to v5

Running into this issue:
https://github.com/godbus/dbus/issues/176
And this is the commit created after this fork to fix it:
https://github.com/godbus/dbus/pull/177/files

Sorry I didn't see this! Looks good!


Support API key or Token in Web

In backend of translation service, API key or Token is supported. It controls who can access translation interface and the limit for translation.
However, in Web browser, users cannot input API key or Token now.
API key or Token can be set in the same UI for translation endpoint just like mobile APP.

Done.


no progress output when launched from launchd

I have skicka being kicked off at system startup (before any user login) via launchd.
To be specific, I have launchd calling a separate bash script, and it is the bash script that calls skicka.
I am not seeing any progress update output in console.log (especially system.log).  Specifically, I am not seeing any of the progress update information that is normally present at the command line.

This is not an issue... i had to redirect with 2> instead of >
I am able to get all output piped to a file, no problemo.


fix bug: texture is used but not defined.

fix bug: texture is used but not defined. cannot load textures.

@vtxf thanks for the pull request!
I noticed that CHANGES.md has not been updated. If this change updates the Cesium API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and bump this pull request so we know it was updated. For more info, see the Pull Request Guidelines.
I am a bot who helps you make Cesium awesome! Thanks again.

@vtxf thanks for the pull request!
I noticed that a file in ThirdParty/ or Source/ThirdParty/ has been added or modified. Please verify that it has a section in LICENSE.md and that its license information is up to date with this new version.  Once you do, please confirm by commenting on this pull request.
I am a bot who helps you make Cesium awesome! Thanks again.

Thanks for the fix @vtxf. Can you complete a CLA for us to merge this?
Do you have the model that caused the bug? I'm curious why it would have happened, it almost seems like a model is invalid in some way.
Also missing in the globalMapping object are images, samplers, and scenes. Could you add those as well?


Feature/CSI plugin move to shiny UBI-minimal images

[in progress]
Moving to shiny UBI base images instead of centos7.
csi node
New base image is registry.access.redhat.com/ubi8/ubi-minimal:8.0
csi controller
New base image is registry.access.redhat.com/ubi8/python-36:1 which already include python3 and pip3.
RedHat certification docker image alignments:

Adding the following labels to the dockerfiles:
name, vendor, version, release, summary, description +
In addition Openshift specific labels: io.k8s.display-name, io.k8s.description, io.openshift.tags
TBD - /license directory with relevant license file
TBD - how to set user\group in this ubi python.

More
UBI is the new Redhat\IBM thin base image for containers, many advantages for using UBI, one of them is a small footprint (just 101MB).

@matt-levan please please give it another review, and let me know if I missed something to pass the redhat image certification process.
@rivkap - please review the text in the Dockerfiles.

Please see my comments


Slight issue in the "Simple Code" example

I think there's a small error in the example you provided. You have
loss = loss_fn(image=var)["loss"]

I think this needs to be the following in order to run, though:
loss = loss_fn.image_loss(image=var)["loss"]

Also, thank you for this resource - this library will be very useful!

@waltergerych yes, thanks! Public API was changed, but we forget to change README too. We will fix it soon.

Done


Blur does not disappear after login

On the right side:

That's actually not a bug, you can't add new columns at the moment because the feature isn't developed.

Ah, got it :) thanks for the clarification. But perhaps consider greying out the button (it can be done quite easily via css, with filter: saturate(0);), and adding a tooltip explaining why it's disabled, rather than using a blur. IMHO that would communicate the disabled status more effectively.

@waldyrious But it's already gray in the screenshot you posted.

I meant the entire object, including the black tooltip. I was suggesting a native tooltip (using the title attribute) that would appear when the user hovers over that element, explaining that the feature is not available yet. I get that this is a minor UI concern, but blurring doesn't communicate "intentionally disabled" to me, and left me confused about whether it could be a bug.
Now, whether resolving this potential for confusion is something that's worth prioritizing, considering the current state of the rest of the project, its a decision I leave to you :)

@waldyrious I agree with your point!
I removed the blur and added "soon":

Does it make sense to you now?

Yeah, that's perfectly clear :) thanks!

It's live (modulo caches)!


Emit utility to wrap a Resource trait implementation as a ResourceRef in Rust

This comes up in extendable resource implementations, such as ClientSuppliers from the MPL. Is an Into<...> impl possible here too?

Resolved by #635


PSX Branch needs syncing with PSXPC

PSX branch is severely out of date, must update/add all missing PSX PSYQ SDK calls.

Root of the problem is unimplemented CD wrapper/SDK calls. Results in data not being read from disc i.e (GAMEWAD header) resulting in a crash.

Now up to date.


disallowNewlineBeforeBlockStatements: check if prevToken is defined so t...

...hat files starting with block statement don't throw error
Fixes: #887

Coverage remained the same when pulling 08b0b9b785a73801da7be90d3ae3021e297f36dc on tteltrab:fix-lone-block-statement-break into 4d90c261c9323293e8098d01209cecb26ab4fa20 on jscs-dev:master.

ah, cool.  didn't know about those, but will do - could use an update.  And the strict type check is probably overkill, wasn't sure which way was preferred.

Coverage decreased (-0.0%) when pulling 93e0352750f29928ead230efd23924dd8de61120 on tteltrab:fix-lone-block-statement-break into 4d90c261c9323293e8098d01209cecb26ab4fa20 on jscs-dev:master.

@tteltrab Definitely ;-D

Pull request is #888 I believe.


Create worker that periodically runs job to classify images

as systemd service

As the aim for the gallery is to be running in an isolated Docker container, using a systemd service as indexer is probably not the best approach.
Instead, we could have the indexing integrated in the Flask app, where a new process could be spawned for doing the indexing. The indexing could be triggered by a button in the front end or a call to the REST API. This would also make it easier to show the current indexing status in the front end and possibly temporarily disable some features (like manually updating labels, in order to not get into concurrency problems when writing to the database).


Running webdriver-manager update before e2e tests

Fixes https://github.com/blackbaud/skyux-cli/issues/105

Codecov Report

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

@@          Coverage Diff          @@
##           master    #75   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          22     22
  Lines         506    506
  Branches       57     57
=====================================
  Hits          506    506

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 da6b215...7fb798b. Read the comment docs.


List the PackageIdentifier of the package to be added to automation
Ryujinx.Ryujinx.Ava
Details about the application
https://github.com/Ryujinx/release-channel-master/releases

I tried it myself, but every time I get an error when using https://github.com/vedantmgoyal2009/vedantmgoyal2009/blob/main/winget-pkgs-automation/New-PackageJson.ps1:
--- Generated package json with GitHub defaults ---
NOTE: You can modify the package json file manually if the defaults are not suitable for the package.
Out-File: C:\Users\<user_name>\Downloads\New-PackageJson.ps1:403
Line |
 403 |  … Package -Depth 7 | Out-File -Encoding UTF8 -FilePath $PackageJsonPath
     |                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Could not find a part of the path 'C:\Users\<user_name>\Downloads\packages\r\ryujinx.ryujinx.ava.json'.
JSON file created: C:\Users\<user_name>\Downloads\.\packages\r\ryujinx.ryujinx.ava.json

----- Test package -----

Do you want to test the package?


DnD with auto sorting - equivalent of a VSCode sidebar

Hi!
I'd like to copy drag & drop UI from VSCode (without choosing leaf order) like below:

When I drag File4.txt from FolderB to files in FolderA the files 1, 2, 3 and FolderA all have hovered state. I can't find a way to do that in Arborist.
The main problem is that I can't get any information about dragging over or between the leafs.

When the cursor is active between the files I can't get its state from the node component.
I managed to highlight all nested children when hovering over the parent internal node (FolderA) using the following code:
let parent = props.node.parent;
while (parent) {
	if (parent.willReceiveDrop) {
		return true;
        }
	parent = parent.parent;
}
return props.node.willReceiveDrop;

In the docs I read that <RowRenderer> is responsible for attaching the drop ref but I couldn't figure anything out.
Since the readme states that we can build the equivalent of a VSCode sidebar could you provide a sample or correct the description if it's impossible?

Hey there! You're right. I was not able to re-create VSCode with the current version, despite my marketing claims :)
I've get a branch started to make the changes necessary to do what you want here as well as add a VSCode demo to the demo website.
Take a look: https://github.com/brimdata/react-arborist/pull/179

I finally got the time to check it out in version 3.4.0 and I can confirm it works flawlessly :)
Thanks!

Fantastic!


Request for Geo world bubble map with enhancement

Is your feature request related to a problem? Please describe.
We need Geo map with below functionality

Need a zoom in/out with focus on some region
onclick on region point map should be drill down with more point on same region with some location zooming
Please refer to below screen record for more understanding

Screen Recording 2024-01-12 at 10.07.05 AM.mov…

https://github.com/kyndryl-design-system/shidoka-charts/assets/18259137/d8fe3df5-6dd9-4d44-9079-89ded2b53e44

The third party chartjs geo plugin is not supported by the third party chartjs zoom plugin, so this feature is not possible.
Two options:

You can hook into click events on the chartjs instance to render a new chart based on the clicked region.
You need to use an alternate solution with full mapping capabilities such as Google Maps or similar.


Preparing release branch for azure-cosmos v4.63.1.

API change check
API changes are not detected in this pull request.

Split-based tests for incremental CFP tests are timing out which pass locally. Overriding this test result.


fix(nuxt): do not try auto-install outside of a Nuxt context

🔗 Linked issue
resolves https://github.com/nuxt/test-utils/issues/618
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
When the component loader plugin is running outside of a Nuxt context (i.e. in @nuxt/test-utils), we don't want to prompt to auto-install. This is true either if options.test is true or if Nuxt isn't accessible.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@758b8be). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #24605   +/-   ##
=======================================
  Coverage        ?   58.65%
=======================================
  Files           ?        5
  Lines           ?      861
  Branches        ?       46
=======================================
  Hits            ?      505
  Misses          ?      356
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add attribute to disable animation on desktop

It looks like scrollreveal added the desktop option in 4.0.0.  You'll also need to update package.json's scrollreveal dependency to ^4.0.2 for the modifier to have any effect.
Also, please run npm/yarn run build after you've made the changes to update dist/ and then push.
Thanks!

Thanks!


File Explorer add-on preview. md file garbled

Microsoft PowerToys version
0.770
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
FancyZones Editor
Steps to reproduce

✔ Expected Behavior
预览文件时不会乱码，可以设置自动编码或自定义编码
❌ Actual Behavior
No response
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/24812


Warning when trying to set region: latitude with a variable

Hi guys!
When trying to set region latitude with a variable, my iOS simulator is showing a warning message saying that the required propType : latitude was not found. But actually the map is working!
Is that ok?
Thanks!

What are the params you are using? Do you have a sample code? Is latitude the only missing proptype?

I have a fetch getting data from a json and set state consulta receiving this json object.
Then in render i have =
var consulta = this.state.consulta;
<MapView
style = {styles.map}
region={{
latitude: consulta.Latitude,
longitude: consulta.Longitude,
latitudeDelta: 0.005,
longitudeDelta:  0.010
}}
/>

Actually the message is: Warning: Failed propType: Required prop 'region.latitude' was not specified in 'MapView'. Check render method.
This happens only with region.latitude and initialRegion.latitude so far.

Just tested here. With MapView.Marker coordinate also is trowing the warning on latitude.

should it be consulta.latitude instead of consulta.Latitude ?

in our JSON is Latitude.
The Map is working... It's getting the latitude value on console too.
But keeps the warning message.

Is it rendered multiple times? Does it has the latitude on the first rendering?

@jrichardlai You are right. Its rendering twice. The fetch is done on componentDidMount().
Sorry about that and thanks a lot for the attention!


Improve error handling for local execution

Tracking issue
NA
Why are the changes needed?
Improve error messages
What changes were proposed in this pull request?

Remove duplicate error message
Remove traceback from user_entry_point

How was this patch tested?
pyflyte run pyflyte run train_wf.py wf

Before:

After:

pyflyte --verbose run pyflyte run train_wf.py wf

Before:

After:

Check all the applicable boxes

[x] I updated the documentation accordingly.
[x] All new and existing tests passed.
[x] All commits are signed-off.

thank you for doing this, i was about to fix this annoying thing


Github Actions

Run at least one test when deployed to main.

Running smoothly, testing to make sure it builds correctly


Allows dittybopper to be deployed in user-specified namespaces

Description
Allow dittydopper to be deployed in user-specific namespaces.
Signed-off-by: Krishna Harsha Voora<EMAIL_ADDRESS>

@smalleni @mukrishn ptal

I would rather default it to dittybopper ns and make it overrid'able if they need to be on user-specified ns, but is there any benefit on making this on a different ns ?

Yes, the default is always dittybopper ns and the existing logic is also inline to be overridden w/ a user-specified ns, however, the deployment fails as below given.
This PR appreciates the inline logic and allows it to be deployed in non-dittybopper ns as well
Using k8s command: oc
Using namespace: harsha
Using default grafana password: admin

Getting environment vars...
Prometheus URL is: https://prometheus-k8s-openshift-monitoring.apps.xxxxxx/
Prometheus bearer token collected.

Creating namespace...
namespace/harsha created

Deploying Grafana...
Error from server (NotFound): error when creating "STDIN": namespaces "dittybopper" not found
Error from server (NotFound): error when creating "STDIN": namespaces "dittybopper" not found
Error from server (NotFound): error when creating "STDIN": namespaces "dittybopper" not found
Error from server (NotFound): error when creating "STDIN": namespaces "dittybopper" not found
Error from server (NotFound): error when creating "STDIN": namespaces "dittybopper" not found
[root@krvoora dittybopper]#

Ahh got it, it was there already but templates have hard coded it. Also make sure you apply this on the user specified project.

Also make sure you apply this on the user specified project.

Thanks for the pointer, I pushed those changes as of now, can you ptal?

user_ns.log
default_ns.log


Retry the macOS builds in case of notarize failures

The notarize operation is flaky so I'm wrapping it in a retry

Retry appears to be working, gonna merge https://github.com/wavetermdev/waveterm/actions/runs/12208580986


[terra-status-view] Update docs to use mdx syntax

Summary
updated docs to use MDX syntax.

The type of "variant" prop is not displaying array of strings correctly as seen here.
As per this discussion, one solution is to convert the proptypes here as strings.


Bump testcontainers-scala from 0.41.0 to 0.41.4

:mag: Description
Issue References 🔗

This pull request fixes #
Describe Your Solution 🔧
Bump testcontainers-scala from 0.41.0 (released in Aug 2023) to 0.41.4 (released in Jun 2024)
Types of changes :bookmark:

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

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[ ] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (c6212d0) to head (06551cb).
Report is 1 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6745   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         684     684
  Lines       42281   42281
  Branches     5768    5768
======================================
  Misses      42281   42281

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Put all lakeFS samples in a repo along with canonical Docker Compose

Iceberg have a very nice way of making a demo environment and notebooks available to users. GH repo includes the samples as notebooks and the Docker Compose to run them. The Notebooks are all automagically tested too to make sure they still work.
I wonder if we should move to this kind of direction with https://github.com/treeverse/lakeFS-samples/ plus Everything Bagel, and then double down on it.
The wrinkle in that would be lakeFS Cloud and there that sits in the user's journey (or where we want it to sit in the user's journey). Ideally we'd offer the notebook execution within lakeFS Cloud too, and then the experience locally or cloud would be smooth and excellent :slightly_smiling_face: This would take the user well beyond a static markdown file to copy and paste bleeding chunks from.  (also related to my comment here regarding execution environment)

Move Everything Bagel to lakeFS-samples repo, strip back image to
* Minio
* lakeFS w/DuckDB (Docker image tag)
* Jupyter w/PySpark

These are the samples that don't have hooks or airflow deps and so should hopefully just go straight into the notebook folder:

[ ] lakeFS samples 06: Airflow
[ ] lakeFS samples 02: DAIS 2022 Chaos Engineering
[ ] lakeFS samples 03: Delta Lake Demo
[ ] lakeFS samples 03: Spark Demo
[ ] lakeFS samples 05: Dev/Test with branches
[ ] lakeFS samples 18: Import from Multiple Buckets
[ ] lakeFS samples 03: RBAC Demo
[ ] lakeFS samples 03: Reprocess Data Demo
[ ] lakeFS samples 07: ML Reproducibility
[ ] lakeFS samples 08: Data Lineage
[ ] lakeFS samples 09: Delta Diff
[ ] lakeFS samples 10: ML Experimentation
[ ] lakeFS samples 11: Version Control of multi-buckets pipelines
[ ] lakeFS samples 12: Labelbox Integration
[ ] lakeFS samples 14: Repo migration
[ ] lakeFS samples 19: Dagster Integration
[ ] lakeFS samples 01: Delta Lake - Multi-table txns

TODO:

[ ] Update https://docs.lakefs.io/quickstart/iso_env.html to use lakeFS-samples
[ ] Migrate samples that use hooks

[ ] Update https://docs.lakefs.io/use_cases/cicd_for_data.html
[ ] lakeFS samples 03: Hooks Schema Validation
[ ] lakeFS samples 04: DQ checks with hooks
[ ] lakeFS samples 13: Schema validation with Lua Hooks

[ ] Migrate samples that use airflow

[ ] lakeFS samples 03: Hooks Airflow Demo
[ ] lakeFS samples 03: Airflow DAG Versioning
[ ] lakeFS samples 03: Airflow Demo Existing DAG
[ ] lakeFS samples 03: Airflow Demo New DAG

[ ] Write new blog showing off lakeFS-samples
[ ] Remove/deprecate EB from lakefs repo


feat: drop dpx 0.0.6 support \n\n BREAKING CHANGE: DPH Service updated

Data Product Hub Service SDK updates
PR summary
Data Product Hub Service SDK updates
PR Checklist
Please make sure that your PR fulfills the following requirements:

[X] The commit message follows the Angular Commit Message Guidelines.
[X] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] New tests
[x] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Other information

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


actually use scope for canBeDefault

This is a minor bug fix. It looks like the canBeDefault method wasn't using the scope that was being passed in. The property doesn't exist on the global scope, so the function always failed.

Force-pushed with a signed commit.

Thanks for this!


Skip matplotlib-dependent tests if it's not installed

Closes #214. A pretty simple fix :)

Thank you @StanczakDominik
closes #226


chore: update badges in README

Detail

The badges we had in the README which referred to our Python versions and formatter were out of date

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

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 1db9932bff1c91ac14650639c08099de53410fdf
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 1db9932bff1c91ac14650639c08099de53410fdf
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Navigation bar dives into the status bar when pushed from UIImagePickerController

Steps to reproduce.

Download CLImageEditor-0.2.4.zip
Open Demo/CLIMageEditorDemo/CLImageEditorDemo.xcodeproj
Run on iPhone X Simulator
Tap 'New' button on Tabbar
Select 'Photo Library' on ActionSheet
Select photo on UIImagePickerController
Select toolbar button (e.g. Filter button) on CLImagEditor screen

Then navigation bar dives into the status bar.
So I can't tap Back button or OK button.
This occurs from 0.2.4. There is no problem with 0.2.3.

I found the workaround for this issue.
CLImageEditor0.2.4 works fine with modal screen transition.
So, when we transit from UIImagePickerController to CLImageEditor, instead of a push transition using pushViewController, modal screen transition using presentViewController, all the subsequent operations will work well.
Specifically, in the ViewController.m file of CLImageEditorDemo project,
change the code in the didFinishPickingMediaWithInfo method from pushViewController to presentViewController as follows.
- (void)imagePickerController:(UIImagePickerController *)picker didFinishPickingMediaWithInfo:(NSDictionary *)info
{
    UIImage *image = [info objectForKey:UIImagePickerControllerOriginalImage];

    CLImageEditor *editor = [[CLImageEditor alloc] initWithImage:image];
    editor.delegate = self;

//    [picker pushViewController:editor animated:YES];
    [picker presentViewController:editor animated:YES completion:nil];
}

Then, instead of closing CLImageEditor in the didFinishEditingWithImage method, close UIImagePickerController by specifying self as follows.
- (void)imageEditor:(CLImageEditor *)editor didFinishEditingWithImage:(UIImage *)image
{
    _imageView.image = image;
    [self refreshImageView];

//    [editor dismissViewControllerAnimated:YES completion:nil];
    [self dismissViewControllerAnimated:YES completion:nil];
}

In addition to that, adding the implementation of the imageEditorDidCancel method as follows can avoid the issue of #97.
- (void)imageEditorDidCancel:(CLImageEditor *)editor
{
    [self dismissViewControllerAnimated:YES completion:nil];
}


It probably also increases the justification of https://github.com/google/horologist/issues/1221, as we don't have a known date for AGP screenshotTest to save us.
WHY
Resolve https://github.com/robolectric/robolectric/issues/8592
HOW
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

Too big to iterate on, going to land since there are no main code changes.  Only tests.  Will work on two issues and feedback from a previous PR.


Passing Slice to BufferedIO#read terminates at 8192nd character

Hi all,
I'm new to Crystal and I really like it. Great work!
I have a question about BufferedIO. If I call BufferedIO#read without slice, it read whole buffer. On the other hand, passing Slice to BufferedIO#read makes different behavior: terminated at 8192nd character even if I pass an adequate size of Slice (FYI, 8192 is the same as BufferedIO::BUFFER_SIZE).
text = "a" * 20_000 + "b"

io = BufferedIO.new(StringIO.new(text))
puts io.read[-1] # => b

io = BufferedIO.new(StringIO.new(text))
slice = Slice(UInt8).new(30000)
count = io.read(slice, 30000)
_text = String.new(slice)
puts _text[-1]    # => a
puts _text.length # => 8192
puts count        # => 8192

May I ask whether it is intentional or not?

This was definitely something to improve, thank you for letting us know!

Great! Many thanks!


Add confirmation message in panel for no errors case

Fix #13. What do you think? @jdan

I'll check this out later in the afternoon, but what happens with the Landmarks plugin?

@jdan ,this is just another check in shared info-panel to check for 0 errors case and add a confirmation message in error tab when there are no errors. This will work with all the plugins since info-panel is shared among all plugins. I had added Landmarks plugin image in issue #13 just as an example. Thanks!

@ankitaggarwal011 my concern is that Landmarks will never have errors though, as it's a plugin that reports errors. We'll have others like this like a color blindness simulator.

I understand your concern now. But still I think, even in cases such as Landmarks, it's better than displaying a confusing blank panel (current scenario). This doesn't require a lot changes in plugin architecture.
I can think of a few solutions for this problem:

Don't display panels for Landmarks and similar plugins. This can be done adding a new variable to check for non-error plugins.
Add a new variable to hide/show panel with each plugin settings.

Though, both of these solutions would require changing in the plugin architecture.
What do you think? @jdan.

Landmarks won't display a blank panel, only Headings does - and that's because the summary is empty, but still triggers the info panel. Errors aren't the issue here - our conversation seems to have forked a little.
A fix for #13 involves simply fixing the logic to not render an info panel if summary is empty. I think we should deal with the "confirmation if no errors" part separately, since it requires some more thought. Does that make sense?
So I'll close this PR because I'm reluctant to merge anything dealing with "confirmation" right now.

@jdan , Yeah sure, no worries.
Yeah, I guess the discussion has diverged a bit, but the issue is still isn't solved yet. You may want to look at it again later. Maybe you can keep it at a lower priority is issue list.
But I want to state that there is one core issue at hand here:

One where there is no confirmation in case of 0 errors. This is the case for all of the plugins, not just Headings plugin. In landmarks and similar plugins, nothing is shown on the screen for no errors and for heading and similar plugins, empty summary is shown for no errors.

I was trying to address this core issue by showing a confirmation of 0 errors in all the cases.
There are also better ways to address this issue by generalizing plugin creation as I mentioned in the last comment. I believe you should incorporate an endpoint in plugin creation regarding this before you move on to the standard release as it could be an important issue as your plugin list grows.
Thanks for looking into it!


Question: Can I not use a custom emoji?

Hi, so I tried putting <:dnd:802482588463005706> as the emoji for the starboard, which is a custom emoji in that server, and when I react to it, nothing happens, does this package not support custom emojis?

After some testing, it seems that the latest update doesn't support the use of custom emojis. I'll work on fixing this and keep you updated ;)

Fixed with https://github.com/SimonLeclere/discord-starboards/commit/f8b26286c0df06a7f65347bb0f5758c94f9e9ab3

Thanks, it now accepts custom emojis, but is it possible to replace this text with an image of the emoji?

Normally it already does, are you sure you are using the latest version of discord-starboards?

Normally it already does, are you sure you are using the latest version of discord-starboards?

Yes.

The emoji is also from that server

Oh wait, I was in the wrong file sorry


Synapse import tool probably doesn't work for URL previewed images

actually, no, should be fine. The export tool doesn't though.


The requested URL /v1/projects/myproject/location/global/prometheus/api/v1/rules was not found on this server.

We are getting a 404 with the message below when we click the Explore menu on Grafana.
The requested URL /v1/projects/myscopingproject/location/global/prometheus/api/v1/rules was not found on this server.

Cluster: 1.22.8-gke.202
The SA used by the GKE nodes has Monitoring Editor.
When I tested the Managed Prometheus in the GCP console, the "up" showed up just fine across all clusters/projects being monitored by my scoping project.
I deployed the prometheus UI and pointed the project id in the deployment.yaml in my Scoping Project.
Data source is working when I added it in Grafana but the 404 error comes up when I navigate to the Explore button menu on Grafana 9.0.2. (deployed as a chart as well in the same scoping project GKE cluster and monitoring namespace)
Aside from these 404 error, im also getting errors when I navigate to Alerting menu on Grafana.

Logs from Grafana
{"kubernetes":{…}, "log":"logger=context traceID=xxxx xxxx xxxx xxxxxxxx xxxx xxxx xxxx userId=1 orgId=1  t=2022-07-13T07:19:27.956940454Z level=info msg="Request Completed" method=GET path=/api/datasources/1/resources//api/v1/query_exemplars status=404 remote_addr=<IP_ADDRESS> time_ms=32 duration=32.845834ms size=1703 referer="https://grafana.dev.online/explore?orgId=1&left=%7B%22datasource%22:%22Google%20Managed%20Prometheus%22,%22queries%22:%5B%7B%22refId%22:%22A%22%7D%5D,%22range%22:%7B%22from%22:%22now-1h%22,%22to%22:%22now%22%7D%7D" traceID=xxxx xxxx xxxx xxxxxxxx xxxx xxxx xxxx"}

path=/api/datasources/1/resources/api/v1/rules status=404

My clusters are all private the api for monitoring is enabled for all GKE clusters, all service accts attached to the node pools has the Monitoring Editor permissions and I am not seeing any pod errors or warnings in my gmp-system workloads.
Workload Identity is Disabled.
Errors when I inspect

Also getting these errors when trying this dashboard which is working in a self managed prometheus.

What could be wrong here?

errors in explore, i just made to an HTML for readability

The /alerts endpoint is not implemented, by design.
When using a regular Prometheus server, your scrape configs, data, queries, and alerts are all colocated on the same server. However, we've decoupled all these in GMP - scrape configs live on the managed collectors, data lives in Monarch and queries are executed against Monarch, and rules live on the rule-evaluator components that live within your clusters (which executes queries against Monarch and writes back results).
The GMP data source that Grafana uses is hooked up to Monarch so that it can execute queries against the global data store. However, as rules are not installed into Monarch, the /alerts endpoint wouldn't have any data even if implemented.
You can get alerts by running kubectl describe Rules/ClusterRules/GlobalRules in each namespace where you have them installed.
I agree that a global status page showing what rules you have and their current state would be useful - we'll chew this over.

I think that should be written on the documentation.
On Thu, Jul 14, 2022 at 12:45 AM Lee Yanco @.***> wrote:

The /alerts endpoint is not implemented, by design.
When using a regular Prometheus server, your scrape configs, data,
queries, and alerts are all colocated on the same server. However, we've
decoupled all these in GMP - scrape configs live on the managed collectors,
data lives in Monarch and queries are executed against Monarch, and rules
live on the rule-evaluator components that live within your clusters (which
executes queries against Monarch and writes back results).
The GMP data source that Grafana uses is hooked up to Monarch so that it
can execute queries against the global data store. However, as rules are
not installed into Monarch, the /alerts endpoint wouldn't have any data
even if implemented.
You can get alerts by running kubectl describe
Rules/ClusterRules/GlobalRules in each namespace where you have them
installed.
I agree that a global status page showing what rules you have and their
current state would be useful - we'll chew this over.
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/prometheus-engine/issues/278#issuecomment-1183453024,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFYVSGPEP4WB45HWHCPYNY3VT3XC7ANCNFSM53NNFNPQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Sent from Gmail via iPhone

Why am I getting errors in the /rules? I didnt get this part.
On Thu, Jul 14, 2022 at 5:02 AM Lucky G @.***> wrote:

I think that should be written on the documentation.
On Thu, Jul 14, 2022 at 12:45 AM Lee Yanco @.***>
wrote:

The /alerts endpoint is not implemented, by design.
When using a regular Prometheus server, your scrape configs, data,
queries, and alerts are all colocated on the same server. However, we've
decoupled all these in GMP - scrape configs live on the managed collectors,
data lives in Monarch and queries are executed against Monarch, and rules
live on the rule-evaluator components that live within your clusters (which
executes queries against Monarch and writes back results).
The GMP data source that Grafana uses is hooked up to Monarch so that it
can execute queries against the global data store. However, as rules are
not installed into Monarch, the /alerts endpoint wouldn't have any data
even if implemented.
You can get alerts by running kubectl describe
Rules/ClusterRules/GlobalRules in each namespace where you have them
installed.
I agree that a global status page showing what rules you have and their
current state would be useful - we'll chew this over.
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/prometheus-engine/issues/278#issuecomment-1183453024,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFYVSGPEP4WB45HWHCPYNY3VT3XC7ANCNFSM53NNFNPQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Sent from Gmail via iPhone

--
Sent from Gmail via iPhone

Re: /rules, the instance of Grafana that runs queries against Monarch is not hooked up to anything that runs rules. There is no way to get rules into Grafana right now.
Re: the 400, Can you hover over the ! and tell me what the error returned by Grafana is?

@lyanco
This is the 400 error in the dashboard.

For the rules, does it mean we wont be able to navigate in the Explore menu within Grafana if we use GMP as datasource? Because currently, this is what we're getting if we click the Explore menu.

Both /rules and /query_exemplars are returning 404
@lyanco