Use SAP HANA as the Database for a CAP Java Application

Tutorials: https://developers.sap.com/tutorials/cp-cap-java-hana-db.html
I am stuck on Step 3: Deploy database artifacts to SAP HANA. I am getting an error when creating service Hana hdi-shared bookstore-Hana. it says The plan hdi-shared could not be found for service hanatrial.

do you have any idea how I can fix this error?
you can reach me at<EMAIL_ADDRESS>

Hi all,
Manual creation of SAP HANA database in SAP BTP cockpit could solve the issue, capable to complete this tutorial yesterday:

Logon to SAP BTP Cockpit trial account
Click on Subaccount "trial"
Click on Spaces "dev"
Click on "SAP HANA Cloud"
Click on "Create"
Click on "SAP HANA database"
Click on "SAP HANA Cloud, SAP HANA Database" (other data lake possible too, not tested in this scenario)
Click on "Next step"
Choose Instance name, e.g. "hub"
Choose SAP HANA Database Password
Click on "Next step", click on "Next step", click on "Review and Create", confirm last button before creation db
Wait for database status "running"
Execute on local environment: cf create-service hana hdi-shared my-bookshop-db -> worked for me

Hi all,
Manual creation of SAP HANA database in SAP BTP cockpit could solve "no database available", faced and capable to fix in tutorial Business Application Using CAP for Node.js https://developers.sap.com/tutorials/cap-service-deploy.html yesterday:

Logon to SAP BTP Cockpit trial account
Click on Subaccount "trial"
Click on Spaces "dev"
Click on "SAP HANA Cloud"
Click on "Create"
Click on "SAP HANA database"
Click on "SAP HANA Cloud, SAP HANA Database" (please test data lake too)
Click on "Next step"
Choose Instance name, e.g. "hub"
Choose SAP HANA Database Password
Click on "Next step", click on "Next step", click on "Review and Create", confirm last button before creation db
Wait for database status "running"
Execute on local environment: cf create-service hana hdi-shared my-bookshop-db
Proceed tutorial

Are you using a SAP BTP Trial account as described in the first part of the tutorial: https://developers.sap.com/tutorials/appstudio-onboarding.html ?
You can also try to create your own HANA trial instance in your dev space as described here: https://developers.sap.com/tutorials/hana-cloud-mission-trial-2.html
After that you should be able to create HDI containers with the hana service, like cf create-service hana hdi-shared bookstore-hana.

I just checked and also for me the hanatrial hdi-shared service is no longer offered for a newly created trial account on the US landscape. Looks like creating your own HANA instance is now a prerequisite to create HDI containers in a trial account.
Please try to follow the second tutorial link that I provided and confirm that after following these steps you can run the cds deploy --to hana:bookstore-hana command successfully.

I just checked and also for me the hanatrial hdi-shared service is no longer offered for a newly created trial account on the US landscape. Looks like creating your own HANA instance is now a prerequisite to create HDI containers in a trial account. Please try to follow the second tutorial link that I provided and confirm that after following these steps you can run the cds deploy --to hana:bookstore-hana command successfully.

yes, this is working but with that also getting error which can be solved to give with a different name "cds deploy --to hana:bookstore-hana1"


tidy test folders

We now have a tests folder containing all current unit and system tests, plus a tests-old folder containing as-yet unattended older unit tests, system tests and test cases. All test files are in one of these two folders now.

@mo-lucy-gordon, this needs re-review/approval after merging in the lastest master because a new test had been added since this branch was created, and had to be moved in with the others.


If I want to proxy both the container registry and Hugging Face, how should I set it up?

It seems that only one proxy can be set up now. How do we set up two? We want to proxy both the container registry and Hugging Face.

You can set the hugging face mirror, and then the containerd configuration passes X-Dragonfly-Registry to overwrite the hugging face mirror address.

So that's how it can be done! Thank you!

We are currently optimizing the documents and trying to make it better.
Refer to https://github.com/dragonflyoss/d7y.io/issues/53.

Sorry, I'm a bit confused here. What should I replace https://example.com with? Because I didn't see the relevant configuration in the helm values in this document.

Sorry, I'm a bit confused here. What should I replace https://example.com with? Because I didn't see the relevant configuration in the helm values in this document.

If you have the other LFS repo, replace example url with the other LFS repo address. If using hugging face LFS repo, you can remove the example line.


build: file restructuring

Reorganise file structure
Based on https://www.robinwieruch.de/react-folder-structure/

Currently blocked by Marketplace Incident: https://status.dev.azure.com/_event/302592133


New version: Google.Chrome.Canary version 118.0.5985.0

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Reset PR


Create willenbring-team-types.md

Adding a tea time idea. Will also add an entry in the index file.

I have added the entry in the index file, so this is ready to merge unless other changes are needed.


loader doesn't stop displaying on confirm modal

The loader doesn't stop displaying on the send modal when a wrong password is submitted. It should revert back to the send button when there's an error on the modal. The same thing occurs when the send button is clicked and godcr is connected to the decred network.

This issue should be resolved when Pr #414 is merged.

this issue still occurs but not on the confirmation modal. It occurs on the modal templates when there is an error.
I am working to resolve this.


Add release notes for c4b-azure 0.18.0

Description Of Changes
Add release notes for c4b-azure 0.18.0
Motivation and Context
Publishing release notes for new release.
Testing

[x] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[ ] Minor documentation fix (typos etc.).
[X] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated

Related Issue
N/A

@Windos all fixed up, thanks for catching that one! 💖


overlay setVisible

Why should the overlay  setVisible method be made protected?

setVisible() is not part of the @api. To temporarily hide them, it's better to remove them from the map.


This is not recognized on outputs

I followed the steps in your example everything works except when I try to use this. This always is undefined.
HTML
  <ng-container *mfeOutlet="'targetarea/Settings';outputs: { okEvent: next};">
      </ng-container>

CODE BEHIND
  public next(e) {

    if (e.sucess) {
      that.nameStep = true;
    }
  }

Targetarea/Settings CODE
export class SettingsComponent implements OnInit, AfterViewInit {
  @Output() public okEvent = new EventEmitter();
....

  public continue() {
    const projectTypes = PROJECTTYPES;
    this.okEvent.emit({ sucess: true });
  }

}

Hi dkrunov,
I updated *mfeOutlet as following
<ng-container *mfeOutlet="'targetarea/Settings';outputs: { okEvent: next.bind(this)};">
  </ng-container>

Hi, nice, I add this thing into the documentation, thanks!


ibm-bindingDB

I downloaed ./data folder as 'InterpretableDTIP-master' folder from "https://github.com/IBM/InterpretableDTIP" and tried to run 'ibm_bindingdb.py'.
But, the following error occurs.
Traceback (most recent call last):
  File "ibm_bindingdb.py", line 102, in <module>
    with open("InterpretableDTIP-master/IBM_BindingDBuniprotPdb.txt", 'r') as fp:
FileNotFoundError: [Errno 2] No such file or directory: 'InterpretableDTIP-master/IBM_BindingDBuniprotPdb.txt'

Try putting the text file in the InterpretableDTIP-master folder and run it again.

I am sorry but I can not find the text file "IBM_BindingDBuniprotPdb.txt"

Here is the link to the txt file:
https://github.com/prokia/drugVQA/blob/master/data/IBM_BindingDBuniprotPdb
Just change the extension to txt.

Thanks for sharing!


perf: changing the way to search for .csproj and .sln files to reduce the number of analyzed files

fixes: #45
This changes the behaviour since we would stop looking for .csproj and .sln files for all the directories inside the root directory. I don't know if this is so important but, if it is, we could provide two ways of search (broad and narrow) in some kind of option (I think that narrow would be the better default).
If there are anything that needs change, please let me know.

Thanks for this! I will try to review this soon (not sure I have time today though)

Okay, so I finally got around to think a little bit more about this. I had something else in my mind a couple of days ago so I wanted to think about that a bit more and try it a bit out, however I didn't like what I had in mind, and I think something like this could work.
I would probably need to add another option (either broad search or some other options, but it doesn't need to be a part of this PR as I don't know yet how I want that).

@seblj I’m not sure what your thoughts are on handling dependencies, but ripgrep might be useful for this type of search. If you were considering making the broader search opt-in, it could be worth exploring how we could integrate it.

Yeah that is certainly an option, and it could also be checked to see if ripgrep is installed, and prefer that if they have it, and maybe fallback to something slower🤔 I need to think about this, because ideally I think I would not like it, but I am not necessarily opposed to it

Thanks for this though! I think it is good to merge now as I would believe that not many people (if any) have a weird placement of the solution file in a project, so I can do something locally for myself for now and experiment with some solution on how to integrate it.
Currently I am thinking of either some type of option for a broad search, or some callback to let the user just select a solution file themselves, and implement logic for it however they would like


Crash on 4grid

Hi! Ran into an issue with the 4 photos 2x2 grid option.
1,2 and 3 execute without issue, option 4 causes Oopsie
Logs
pi@raspberrypi:~/pibooth-1.0.1 $ pibooth
[ INFO    ] pibooth           : Starting the photo booth application...
[ WARNING ] gphoto2           : (b'_get_config [config.c:7471]') b"Type of property 'ISO Auto Hi Limit' expected: 0x0001 got: 0x0002"
[ WARNING ] gphoto2           : (b'_get_config [config.c:7471]') b"Type of property 'Active D-Lighting' expected: 0x0001 got: 0x0002"
[ WARNING ] gphoto2           : (b'_get_config [config.c:7471]') b"Type of property 'High ISO Noise Reduction' expected: 0x0001 got: 0x0002"
[ WARNING ] gphoto2           : (b'_get_config [config.c:7471]') b"Type of property 'Flash Exposure Compensation' expected: 0x0002 got: 0x0001"
[ WARNING ] pibooth           : Invalid value '100' for option iso (possible choices: ['100', '125', '160', '200', '250', '320', '400', '500', '640', '800', '1000', '1250', '1600', '2000', '2500', '3200', '4000', '5000', '6400'])
[ WARNING ] pibooth           : Invalid value 'Carte mémoire' for option capturetarget (possible choices: ['Internal RAM', 'Memory card'])
[ INFO    ] pibooth           : Show picture choice (nothing selected)
[ INFO    ] pibooth           : Show picture choice (4 pictures selected)
[ INFO    ] pibooth           : Start new pictures sequence
[ INFO    ] pibooth           : Capturing new preview image
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Take picture and save it in /home/pi/Pictures/pibooth/2019-02-15-21-36-59/ptb000.jpg
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'camera_nikon_capture [library.c:3093]') b'Parentobject of newobject 0x2b000008 is 0x0 now?'
[ INFO    ] pibooth           : Capturing new preview image
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Take picture and save it in /home/pi/Pictures/pibooth/2019-02-15-21-36-59/ptb001.jpg
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x9201 receiving resp failed: Invalid Status (0xa004)'
[ WARNING ] gphoto2           : (b'camera_nikon_capture [library.c:3093]') b'Parentobject of newobject 0x2b000009 is 0x0 now?'
[ INFO    ] pibooth           : Capturing new preview image
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Take picture and save it in /home/pi/Pictures/pibooth/2019-02-15-21-36-59/ptb002.jpg
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x9201 receiving resp failed: Invalid Status (0xa004)'
[ WARNING ] gphoto2           : (b'camera_nikon_capture [library.c:3093]') b'Parentobject of newobject 0x2b00000a is 0x0 now?'
[ INFO    ] pibooth           : Capturing new preview image
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x90c8 receiving resp failed: PTP Device Busy (0x2019)'
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Capturing new preview image
[ INFO    ] pibooth           : Take picture and save it in /home/pi/Pictures/pibooth/2019-02-15-21-36-59/ptb003.jpg
[ WARNING ] gphoto2           : (b'ptp_usb_getresp [usb.c:465]') b'PTP_OC 0x9201 receiving resp failed: Invalid Status (0xa004)'
[ WARNING ] gphoto2           : (b'camera_nikon_capture [library.c:3093]') b'Parentobject of newobject 0x2b00000b is 0x0 now?'
[ INFO    ] pibooth           : Creating merged picture
[ WARNING ] gphoto2           : (b'gp_context_error') b'You need to specify a folder starting with /store_xxxxxxxxx/'
[ WARNING ] gphoto2           : (b'gp_camera_file_get [gphoto2-camera.c:1693]') b"'gp_filesystem_get_file (camera->fs, folder, file, type, camera_file, context)' failed: -1"
[ ERROR   ] pibooth           : [-1] Unspecified error

I got the same issue when there were not enough space on the internal memory of the camera, hence the setting to use the memory card instead of the internal memory. Have you tried to change the line as explained in the other issue thread?
I hope this helps, and I'm sorry I could not give you a better answer.

I've been a bit scared to use the sd, considering the meltdown of the last.
On this run of testing (literally done this for 14 hours today...) I went back to a clean version of 1.0.1.
I'll try adjusting the error from Carte memoire to Memory card tomorrow.
[ WARNING ] pibooth : Invalid value 'Carte mémoire' for option capturetarget (possible choices: ['Internal RAM', 'Memory card'])
Thanks for your help!

The configuration of the DSL camera seems to be language dependent, will be hard to fix that (doing a mapping for each config key and supported language?)

Fix pushed to translate DSLR config values in appropriate language


Footer rework.

Is your feature request related to a problem? Please describe.
Footer needs to be change. Add and replace content.
Describe the solution you'd like
Try to replace content with icons. eg. Icon with total contributors.
Additional notes
Utilize the footer space. Make it minimal.

You can work on it @adsingh14 and if it looks good I'll consider it.

Again, I pass it to another new comer.

I will take this one!

@adsingh14 I can't really pass this one to anyone else as it is your idea and it is still vague so it's not 100% approved. Either you work on this or close this issue.


alibi's version is  incompatible in gcr.io/kfserving/alibi-explainer:v0.3.0

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
I deploy kfserving v0.3.0 in my k8s cluster and run test explanation/alibi/imagenet. After execute test_imagenet.py, I get response_code 500.
explainer server pod logs is:
[I 200707 20:36:08 __main__:116] Extra args: {'batch_size': 25}
[I 200707 20:36:08 storage:35] Copying contents of /mnt/models to local
[I 200707 20:36:08 __main__:125] Loading Alibi model
[I 200707 20:36:08 explainer:47] Predict URL set to imagenet-predictor-default.kfserving-test
[I 200707 20:36:08 kfserver:88] Registering model: imagenet
[I 200707 20:36:08 kfserver:77] Listening on port 8080
[I 200707 20:36:08 kfserver:79] Will fork 0 workers
[I 200707 20:36:08 process:126] Starting 32 processes
[I 200708 05:00:10 anchor_images:35] Calling explain on image of shape ((1, 299, 299, 3),)
[E 200708 05:00:13 web:1792] Uncaught exception POST /v1/models/imagenet:explain (<IP_ADDRESS>)
    HTTPServerRequest(protocol='http', host='imagenet-explainer-default.kfserving-test.svc.cluster.local', method='POST', uri='/v1/models/imagenet:explain', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
    Traceback (most recent call last):
      File "/usr/local/lib/python3.7/site-packages/tornado/web.py", line 1701, in _execute
        result = method(*self.path_args, **self.path_kwargs)
      File "/kfserving/kfserving/handlers/http.py", line 61, in post
        response = model.explain(request)
      File "/alibiexplainer/alibiexplainer/explainer.py", line 74, in explain
        explanation = self.wrapper.explain(request["instances"])
      File "/alibiexplainer/alibiexplainer/anchor_images.py", line 45, in explain
        anchor_exp = self.anchors_image.explain(arr[0], **self.kwargs)
      File "/usr/local/lib/python3.7/site-packages/alibi/explainers/anchor_image.py", line 365, in explain
        self.instance_label = self.predictor(image[np.newaxis, ...])[0]
    AttributeError: 'AnchorImage' object has no attribute 'predictor'
[E 200708 05:00:13 web:2250] 500 POST /v1/models/imagenet:explain (<IP_ADDRESS>) 3161.48ms

What did you expect to happen:
Get correct result of explainer
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
I found alibi verion is 0.3.3.dev0 in gcr.io/kfserving/alibi-explainer:v0.3.0.
In alibi-0.3.3.dev0, AnchorImage object has attribute predictor instead of predict_fn, while kfserving alibi explainer use predict_fn.
Environment:

Istio Version:
Knative Version:
KFServing Version:
Kubeflow version:
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

@quanguachong how did you install 0.3? Looks it is using the wrong image, should be using https://github.com/kubeflow/kfserving/blob/master/install/v0.3.0/kfserving.yaml#L6874

@yuzisun You're right, thank you. I install 0.3 with master's ./install/v0.3.0 . As the version name is v0.3.0, I thought they are the same.

@yuzisun Thank you, you're right. I install kfserinvg with branch v0.3.0: https://github.com/kubeflow/kfserving/blob/v0.3.0/install/v0.3.0/kfserving.yaml#L6923 whose explainer image is not compatible with alibi-explainer-sample in branch v0.3.0: https://github.com/kubeflow/kfserving/tree/v0.3.0/python/alibiexplainer.


Form Sample Ribbon Header - only displays title in side panel but not when opened from embedded list

In testing out the ribbon header sample I noticed that the title displays nicely in header if the form is directly opened from list and opens in side panel. However when form is opened through direct link in browser or from an embedded list in a page, the title no longer appears since it no longer is a form, but a page
Ribbon Header Sample
What Should Happen/ looks like in form on side panel

What Actually Happens/ looks like in direct link (title appears in breadcrumb above

Is there a way to make the title display in header even in item "page"?

Hi @VictKat .
I tried it on my end and the ribbon shows the Title column value correctly.

Is the Title column displayed in the body of the form?


cmd/go: go clean should not accept flags like -modcache with packages

What version of Go are you using (go version)?

$ go version
go version devel go1.19-818ffd4247 Tue Jul 5 15:19:44 2022 +0100 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/mvdan/.cache/go-build"
GOENV="/home/mvdan/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/mvdan/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/mvdan/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/mvdan/tip"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/mvdan/tip/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="devel go1.19-818ffd4247 Tue Jul 5 15:19:44 2022 +0100"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/mvdan/src/garble/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1584385538=/tmp/go-build -gno-record-gcc-switches"
GOROOT/bin/go version: go version devel go1.19-818ffd4247 Tue Jul 5 15:19:44 2022 +0100 linux/amd64
GOROOT/bin/go tool compile -V: compile version devel go1.19-818ffd4247 Tue Jul 5 15:19:44 2022 +0100
uname -sr: Linux 5.18.9-arch1-1
/usr/lib/libc.so.6: GNU C Library (GNU libc) stable release version 2.35.
gdb --version: GNU gdb (GDB) 12.1

What did you do?
I wanted to simulate fetching a module from proxy.golang.org, and since it is already in my GOMODCACHE, I wanted to delete it from there. So I naively ran:
go clean -modcache some/module/path

What did you expect to see?
I expected only a few files/directories to be deleted from GOMODCACHE.
What did you see instead?
It deleted my entire GOMODCACHE; presumably, the command is equivalent to:
go clean -modcache
go clean some/module/path

Given that flags like -cache, -modcache, -testcache, and -fuzzcache clear the entire cache directories in question, they should reject package arguments, because otherwise they are very easy to be misused or misunderstood.
It is possible that this might break a few users, but I generally doubt it will be many. I haven't seen any uses of go clean pkg in the wild in years. If anyone wants the current behavior of go clean -modcache pkg, they can always do go clean -modcache; go clean pkg as shown above, which is also more explicit.

Doesn't seem to be too uncommon, usually with ./...
https://sourcegraph.com/search?q=context:global+/go+clean(+-[a-z]%2B)*(+-(cache|modcache|testcache|fuzzcache))%2B(+-[a-z]%2B)*+[a-z\.]%2B/&patternType=regexp

I agree that it's better to report an error since this is confusing and the cost of the mistake can be large,
even though this behavior is documented.
Related https://github.com/golang/go/issues/32976, which is marked as duplicate of https://github.com/golang/go/issues/28835 even though the motivation of the proposal and the proposed command line structure is different.

Thanks @seankhliao; the same search also works on github: https://cs.github.com/?scopeName=All+repos&scope=&q=%2Fgo+clean(+-[a-z]%2B)*(+-(cache|modcache|testcache|fuzzcache))%2B(+-[a-z]%2B)*+[a-z\.]%2B%2F
I'm willing to bet that the majority of those lines are buggy. The way they are written, they look like the intent is to e.g. remove the cached test results for a few packages.

Related #32976, which is marked as duplicate of #28835 even though the motivation of the proposal and the proposed command line structure is different.

Good call out, I did not notice that proposal - cc @bcmills. I think this change could be a stepping stone towards that proposal; any people who currently use go clean -flag pkg and who want #28835's behavior will then come show their support for the proposal. Any people who just wanted go clean -flag, as per the current behavior, can simply adjust their commands.

@bcmills suggests that I should turn this into a proposal, because it's a CLI change that could potentially break some users. See my reasoning above as to why I think the breakage will be minimal, though.

Reporting an error sounds fine to me.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

I don't think that's a good suggestion.
I just want to clean my cgo package, go clean -cache my/cgo/package It's what I want.
It worked well in the past.
When I upgraded to 1.20, It say "go: clean -cache cannot be used with package arguments".
Now I have to run go clean -cache to clean all  packages, and then rebuild cost more time.
#53725 cause the trouble.

@penggy please read the original post carefully - go clean -cache some/pkg used to clean the entire build cache, not just that one package's build cache entries. It never did what you thought it did, which is why I raised this issue.

@mvdan I already read the original post carefully, I am sure go clean -cache my/cgo/package only clean my cgo package, not the entire build cache in my environment.
One more word, I use go vendor not go mod to manage packages.

@mvdan 😄
Thank you for reply.
I have misunderstanding on go clean.

https://go.dev/cl/425874 was merged in 2022. I think this proposal was implemented in Go 1.20?


Proposal: Allow ARG before FROM (Dockerfile templates)

Currently it doesn't seem possible to use the ARG statement before FROM in a Dockerfile:
Sending build context to Docker daemon 2.598 MB
Step 1/3 : ARG foo
Please provide a source image with `from` prior to commit

I've noticed a pattern with a number of Dockerfiles where they are using a template Dockerfile, and then parsing it with some shell scripts to generate the specific Dockerfiles for each image: eg. The file for ubuntu, alpine, etc.
I was thinking it might be interesting to solve this problem by using the build args to make a sort of 'Dockerfile template'.
Below is a contrived example:
# Dockerfile
ARG myfrom
ARG myplatform
FROM ${myfrom}

COPY scripts/ /scripts/

RUN /scripts/foo.${myplatform}.sh

CMD ["/bin/sh"]

I could then pass through the build args like:
# Alpine
docker build --build-arg myfrom="usualoma/ruby-with-therubyracer:2.3.0-alpine" --build-arg myplatform=alpine .

# Ubuntu
docker build --build-arg myfrom="ruby:2.3" --build-arg myplatform=debian .

There seem to be other options currently available, but require 3rd party tools, eg:

https://www.npmjs.com/package/dockerfile-template

It would also be pretty useful if we were able to RUN a build script without having to copy it into the container first. Eg. RUN-FROMCONTEXT foo.sh.

Work in progress: https://github.com/docker/docker/pull/31352
let me close this issue

Oh, must have missed that. Thanks!

rory@ubuntu:~/git/broker-hdsm$ make build
Building PIP package...
/usr/src/broker-hdsm
Building Docker image...
Sending build context to Docker daemon 8.777 MB
Step 1/12 : ARG BASE_IMAGE
Please provide a source image with `from` prior to commit
Makefile:84: recipe for target 'build' failed
make: *** [build] Error 1

@roryodonnell that's expected; Docker 1.13 is really old, and has reached end of life in March of last year. This feature is available in Docker 17.05 and up so any current version of Docker should have this feature.

FWIW this was introduced in 17050-ce-2017-05-04.


Error while trying to setup

I'm new to using docker and I'm getting the below error after running sudo docker build -t virmine . I'm on Ubuntu 16.04 and I would appreciate any assistance from you in this regard
Step 11/36 : RUN git clone https://github.com/najoshi/sickle
---> Running in 06b159905807
Cloning into 'sickle'...
Removing intermediate container 06b159905807
---> 8ca4d070bf47
Step 12/36 : RUN cd sickle && make
---> Running in c9e53538d60d
gcc -Wall -pedantic -DVERSION=1.33 -O3 -c src/sliding.c
gcc -Wall -pedantic -DVERSION=1.33 -O3 -c src/trim_single.c
src/trim_single.c: In function 'single_main':
src/trim_single.c:85:50: warning: statement will never be executed [-Wswitch-unreachable]
if (single_long_options[option_index].flag != 0)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
gcc -Wall -pedantic -DVERSION=1.33 -O3 -c src/trim_paired.c
src/trim_paired.c: In function 'paired_main':
src/trim_paired.c:130:50: warning: statement will never be executed [-Wswitch-unreachable]
if (paired_long_options[option_index].flag != 0)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
gcc -Wall -pedantic -DVERSION=1.33 -O3 -c src/sickle.c
gcc -Wall -pedantic -DVERSION=1.33 -O3 -c src/print_record.c
gcc -Wall -pedantic -DVERSION=1.33  -O3 sliding.o trim_single.o trim_paired.o sickle.o print_record.o -o sickle -lz
Removing intermediate container c9e53538d60d
---> 454ccf629cfa
Step 13/36 : RUN cd megahit && make
---> Running in 49a807869085
make: *** No targets specified and no makefile found.  Stop.

Thank you for letting me know, I have pushed a change that should fix this.
git pull the repo and everything should run fine.

Thank you very much. Does virMine take gzipped read flles?

Yes. It accepts gzipped files. The first step, read QC, will unzip them and QC them.

On Apr 16, 2019, at 2:01 AM, Anto007<EMAIL_ADDRESS>wrote:
Thank you very much. Does virMine take gzipped read flles?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/thatzopoulos/virMine/issues/1#issuecomment-483536683, or mute the thread https://github.com/notifications/unsubscribe-auth/AJn5E43tzusD62SRmLuALgqtofVjCvwKks5vhXVRgaJpZM4cvXvg.


feat: (L1) Unify crates under same directory

Motivation

Reduce the ammount of crate folders in the project
Description

After trying to make different changes to unify the "storage" and "trie" files in one, we end with a circular dependecy. Just noticed that everywhere we import .cores, we also import .storage, we could merge .cores into .storage

crates/storage/storage/ looks a bit redundant What do you think about leaving the storage directory as is and only adding trie as a module. (This is how the crate layout looked like before promoting trie to a separate crate)

I agree, just added that in this pr as it had a problem with dependencies and needed to move the core crate also to the storage


Save new Apps

The problem
With Version 0.0.10 when I want to change the text of an existing app the Save Option does not show up. I have to add a new app in order to be able to change an existing app.
Version of nodejs
16.19.1
Version of ioBroker js-controller
4.0.24
Version of Adapter
0.0.10
Operating system running ioBroker
Debian
Checklist of files to include below

[X] iobroker.current.log (Please share the logs in debug mode)
[ ] Contents of the JavaScript browser console (always include in cases of issues with the user interface)

Additional information & file uploads
Let me know if you need more.

There is a new validator which just allows lower case letters in the app name and the app name must be unique


Helm chart repository URL has changed

https://github.com/Azure/secrets-store-csi-driver-provider-azure/tree/master/charts/csi-secrets-store-provider-azure

The helm chart repository URL has changed from https://raw.githubusercontent.com/Azure/secrets-store-csi-driver-provider-azure/master/charts to https://azure.github.io/secrets-store-csi-driver-provider-azure/charts.

@sethmanheim - Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Export validator, stake, and lockup files

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

#34  👈
#27 : 1 other dependent PR (#28 )
main

This stack of pull requests is managed by Graphite.


Fix error when a language skips a certain version in interop test

On master, running
tools/interop_matrix/run_interop_matrix_tests.py --release=v1.0.1 --language all appears to result in AttributeError: 'list' object has no attribute 'keys'.
I was able to run that without script errors after this change.

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[trickle] No significant performance differences

Corrupt JSON data (indicates timeout or crash):
    bm_fullstack_unary_ping_pong.BM_UnaryPingPong_InProcessCHTTP2_NoOpMutator_NoOpMutator__0_32768.opt.old: 1

[microbenchmarks] No significant performance differences

[microbenchmarks] No significant performance differences

[microbenchmarks] No significant performance differences

test failures:

linux artifact build failed with a kokoro exception: hudson.AbortException: Queue task was cancelled
asan c++: https://github.com/grpc/grpc/issues/13443
basic tests c++ linux opt: https://github.com/grpc/grpc/issues/10092
basic tests macos: https://github.com/grpc/grpc/issues/13446, https://github.com/grpc/grpc/issues/13516
basic tests multi lang linux: https://github.com/grpc/grpc/issues/13404,
basic tests windows dbg: https://github.com/grpc/grpc/issues/13516
tsan C++: https://github.com/grpc/grpc/issues/13478


Blue button color too bright/intense

The shade of blue of the blue buttons on the site is very intense and, for me at least, is painful. This might be an accessibility issue for other people, too. The background colour is a much nicer shade so maybe the buttons could be changed to that same shade?

Looks good to me, thank you!


[typescript] dom definitions not up to date

From @yukulele on October 5, 2017 12:7

VSCode Version: 1.16.1
OS Version: macOS Sierra 10.12.6

Steps to Reproduce:

Create this code in typescript file

const elm = document.createElement('main')

elm is typed as HTMLElement instead of HTMLMainElement

lib.dom.d.ts is not up to date
how to update it without wait for vscode update ?

Reproduces without extensions: Yes
Copied from original issue: Microsoft/vscode#35620

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

Can I fix this one as my first contribution to TypeScript?

@jamOne- just make a PR https://github.com/Microsoft/TSJS-lib-generator

I already know it, but I don't think this change is that simple.
As far as I understand this generator code, to generate interface HTMLElementTagNameMap in dom.generated.d.ts only browser.webidl.xml interfaces are used, but HTMLMainElement is in addedTypes.json file so I have to change the generator code, or am I wrong?

Pretty sure you can just add overload to addedTypes.json.

I am not sure I understand you. Does generating interface HTMLElementTagNameMap take into account addedTypes.json?

Please, have a look at the on-going progress before proceeding: https://github.com/Microsoft/TypeScript/issues/17828

The following elements are affected:   abbr address article aside b bdi bdo cite code dd dfn dt em figcaption figure footer header hgroup html i kbd main mark nav noscript rp rt ruby s samp section small strong sub summary sup u var wbr


feat: Support animation style prop from IonContent

Feature Request
Ionic version:

[x] 4.x
Describe the Feature Request
Please support animation for IonContent element.
Describe Preferred Solution
Include new prop to support animation style.
Describe Alternatives
Adding the value via TS, but it's not working. e.g:
const contentElement = (this.content['el'] as HTMLElement);
const innerScroll = contentElement.shadowRoot.querySelector('.inner-scroll') as HTMLElement;
innerScroll.style.animation = 'background 30s ease infinite';

Related Code
ion-content {
  --background: linear-gradient(124deg, #ff2400, #e81d1d, #e8b71d, #e3e81d, #1de840, #1ddde8, #2b1de8, #dd00f3, #dd00f3);
  --animation: background 30s ease infinite;
}

@keyframes background {
    0%{background-position:0% 50%}
    50%{background-position:100% 50%}
    100%{background-position:0% 50%}
}

Thanks in advance!

this work for me, put this in your page component
@ViewChild(IonContent,{static:false}) set content(content){
    if(content){
      let shadow = content.el.shadowRoot;
      let animationStyle = new CSSStyleSheet();
      (<any>animationStyle).replace(`
      :host{
        --animation-duration: 0s;
          --animation-timing-function: ease;
          --animation-delay: 0s;
          --animation-iteration-count: 1;
          --animation-direction: normal;
          --animation-fill-mode: none;
          --animation-play-state: running;
          --animation-name: none;
      }
      .inner-scroll{
        animation-duration: var(--animation-duration);
          animation-timing-function: var(--animation-timing-function);
          animation-delay: var(--animation-delay);
          animation-iteration-count: var(--animation-iteration-count);
          animation-direction: var(--animation-direction);
          animation-fill-mode: var(--animation-fill-mode);
          animation-play-state: var(--animation-play-state);
          animation-name: var(--animation-name);
      }
      `);
      shadow.adoptedStyleSheets = shadow.adoptedStyleSheets.concat(animationStyle);
    }
  }

then in your page.sccs add this for example
ion-content{
  --animation-name:perro
}

and the same for the rest of properties that you need

OMG, very interesting! @Isaac-cura I want to avoid adding styles from code, to be able to override the styles of the components from the styles of that page, but thanks for your excellent example! <3

Thanks for the issue. I am going to close this as we are going to be rolling out a broader animation solution in the next major release of Ionic Framework that should account for this use case. Thanks!

Ohh interesting, thanks!


Correctly transform touch_up position

See issue #4477

I notice you mention in the linked issue that it might have some issues - did you work out what was going on there?

@inclement Yes, the FileChooserListView somehow changes (can't really describe: Some clicks seem to go missing, opens wrong folders, slower). See this Gist: https://gist.github.com/johanneshk/5bfa9a3983e81a20ed16ed1ff0d80257#file-main2-py
This PR seems to fix the initial behavior, but for some reason the FilechooserListView is broken (at least for me?). I don't know why, but that's what I'm experiencing...

Could someone confirm my Edit above? Plus maybe the behavior in general? If this is correct, I could go ahead and look into FileChooserListView and why it behaves differently.
Thanks!

At a first glance you're correct. Because _do_touch_up is "called" from on_scroll_stop which receives touches in parent coordinates, not window coordinates. So to_local rather than to_widget should be used. That is your reasoning, right?
The problem with this though is that it's "scheduled", not called from on_scroll_stop. So in between the scheduling, when it's in parent coordinates, and the execution; the touch is likely to have been converted back to window coordinates. That's because he touch starts and ends in window coordinates.
At least this is how I understand this.

@matham  your explanation makes perfect sense. Plus the first comment in '_do_touch_up' is 'touch is in window coords'. With the pop statements you added this now seems to be true in all cases.
So please don't merge this PR but rather #4508 .


When creating / destroying multiple records it sometimes hangs on creation / destruction

I have a terraform script that is creating four different classes of machines, with multiple in each class.
I have four different sections like this:
resource "windns" "dns_redis" {
  count  = var.vm_count_redis
  record_name   = "${var.vm_name_redis}-${element(random_id.vm_name_redis.*.hex, count.index)}"
  record_type = "A"
  zone_name = var.vm_domain
  ipv4address = element(phpipam_first_free_address.newip_redis, count.index).ip_address
}

Each section uses different variables for anywhere there is _redis (_logstash, etc.)
When I run it, creating six machines, it hangs on one of the creation. When I look in DNS, I can see it has created the record, but it never completes from Terraform's perspective, so it hangs there.
The same is true on destruction. It hangs on a single element, even though it has deleted it already. If I re-run the destroy, it errors with Remove-DnsServerResourceRecord: Failed to get elk-logstash-fdf7b7 record
Not sure what is causing it, but if I can do any more debug to help, let me know!

@rfranks-securenet Can you replicate this at will? Can you try it out with the --parallelism flag, and set it to 1 or something lower (https://www.terraform.io/docs/commands/apply.html#parallelism-n)?
I'm pretty sure I ran into this in our environment as well, and it was something to do with the amount of parallelism - shelling out and remoting into the downstream DNS server is such a hack, but unfortunately all we can really do with no API. :(

I appreciate that shelling out and so on is a hack, but I've written some code to implement lockfiles which works around this issue, without having to run terraform with parallelism set low.

@rfranks-securenet I'll take a look. My fingers are crossed that Hashicorp will release an official WinDNS provider since they just released an AD provider, so I can sunset this one. :)

@rfranks-securenet release cut with your update! thanks mate!


Ionic/Angular Router fix for dotnet backend or universal backend

We want to avoid this (index.html# prefix):
http://localhost:5050/index.html#/admin

This feature will be released in Ionic v4
https://github.com/ionic-team/ionic/issues/10565#issuecomment-355055546
Closing this as this will require a migration to the next version after it gets released


Add documentation for singleton modules

Original PR: https://github.com/area17/twill/pull/1178

A starting effort was made here: https://github.com/area17/twill/pull/1329
But I think we can elaborate on that to include usecases.


#240 Support explicit block mappings & non-scalar nodes as mapping keys

In this PR, explicit block mappings and sequence/mapping keys have been supported in the deserialization feature.
Also, new unit test cases have been added to ensure the newly supported features work as expected.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

coverage: 100.0%. remained the same
when pulling 3bcb93d7738245995b5d990ddf3e6aff5af1d9b7 on feature/240_support_explicit_mapping_keys
into ffe16ae924211f111a2bf94f14a8b57ac60149b9 on develop.


SSL/HTTPS Support

Passing our user credentials to the npm server in cleartext over HTTP poses a security concern for us. Also, any npm modules downloaded from the server are transmitted over HTTP.
Would like to see support for SSL/HTTPS so that we could address these security concerns. Is this on the roadmap?

Terminate SSL before the app. This configuration should not be a part of the app itself.


test(client): add integration test for methods "create" and "delete" about commit

Pull Request Test Coverage Report for Build 898068062

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.001%

Totals

Change from base Build 897809189:
0.0%

Covered Lines:
4502

Relevant Lines:
5424

💛  - Coveralls


Add localised keys in src/locales/skSk.mjs

I’ve just translated all the strings from enUs.mjs into Slovakian.
Currently, there are some differences between enUs.mjs and skSk.mjs:

key name (enUs.mjs)
key name (skSk.mjs)
notes

"blessedHrzonataMartyr"
"blessedHroznataMartyr"
typo in enUs.mjs

"jesusChristTheEternalHighPriest"

duplicate (same key name)

"ourLadyOfFatima"

duplicate (same key name)

"ourLadyOfLourdes"

duplicate (same key name)

"ourLadyOfLujanPatronessOfArgentina"

duplicate (same key name)

"ourLadyOfMountCarmel"

duplicate (same key name)

"peterAndPaulApostles"

duplicate (same key name)

"presentationOfTheBlessedVirginMary"

duplicate (same key name)

"saintAndrewTheApostle"

duplicate (same key name)

"saintAugustineOfCanterburyBishop"

duplicate (same key name)

"saintBonifaceBishopAndMartyr"

duplicate (same key name)

"saintCasimir"

duplicate (same key name)

"saintCyrilOfJerusalemBishopAndDoctor"

duplicate (same key name)

"saintElizabethOfHungary"

duplicate (same key name)

"saintElizabethOfPortugal"

duplicate (same key name)

"saintFabianPopeAndMartyrAndSaintSebastianPopeAndMartyr"
"saintFabianPopeAndMartyrAndSaintSebastianMartyr"
while @pejulian  change the value, but he forgot about the key name, hence the difference

"saintFrancisXavierPriest"

duplicate (same key name)

"saintGregoryTheGreatPopeAndDoctor"

duplicate (same key name)

"saintHelena"
I added this key; the celebration is to be added yet

"saintHenryBishopAndMartyr"

duplicate (same key name)

"saintJohnOfKantyPriest"

duplicate (same key name)

"saintMariaGorettiVirginAndMartyr"

duplicate (same key name)

"saintPatrickBishop"

duplicate (same key name)

"saintPeterCanisiusPriestAndDoctor"

duplicate (same key name)

"saintPiusVPope"

duplicate (same key name)

"saintRaymondOfPenyafortPriest"

duplicate (same key name)

"saintsFelipeDeJesusPaulMikiAndCompanionsMartyrs"
I left this key because of the unresolved issue #24

"saintsPedroBautistaPaulMikiAndCompanionsMartyrs"
I left this key because of the unresolved issue #24

"saintsPhilipAndJamesApostles"

duplicate (same key name)

"saintStanislausBishopAndMartyr"

duplicate (same key name)

"saintStephenOfHungary"

duplicate (same key name)

"saintTeresaBenedictaOfTheCrossEdithSteinVirginAndMartyr"

duplicate (same key name)

"saintTeresaOfJesusVirginAndDoctorOfTheChurch"

duplicate (same key name)

"saintThomasBecketBishopAndMartyr"

duplicate (same key name)

"saintThomasTheApostle"

duplicate (same key name)

"saintVincentDeaconAndMartyr"

duplicate (same key name)

"visitationOfTheBlessedVirginMary"

duplicate (same key name)

Then, I’ve put there some comments:

all celebrations (except for those from General Roman Calendar) has a comment of calendar (country) name (e.g. unitedStates);
duplicate keys with different names are notes as duplicate;
joint celebrations (those that needs to be separated, not all) are noted with to be separated;
celebrations that have a different name in some countries, are noted with localised {key}.[calendarName] (e.g., localised {key}.slovakia)
sometimes I put there some other comments (like notes for myself in regard of the skSk.mjs translation).

I did my best in translation name of the saints, blessed and celebrations which are not yet translated into Slovakian. I might have made some mistakes.
The keys are not yet sorted (issue #18).

By the way, I used the following one-liner for the key name diff (you need to be in the src/locales directory):
grep "[ ]*\".*\"\: \"" enUs.mjs | sed 's/^[ ]*\(".*"\):.*/\1/g' - | sort > ~/en.diff && grep "[ ]*\".*\"\: \"" skSk.mjs | sed 's/^[ ]*\(".*"\):.*/\1/g' - | sort > ~/sk.diff && diff -y --suppress-common-lines ~/{en,sk}.diff | less && rm ~/{en,sk}.diff
And for output of keys and comments from a locale file (again, cd src/locales):
grep "[ ]*\".*\"\: \".*\"[,]*[ ]*//" skSk.mjs | sed 's/: \".*\"[,]*//g' -

Note that before merging this PR, (1) all other locales must change (correct) there keys and (2) also the calendars must be updated to use the new strings.

A big thank you for this great job of translation! 👍
Also when it's going to be ready, could you squash all your commits into one ?
So as much as we can we keep a clean commit history.

Okay, I’ll squash ’em all. :) … However, your link does not work, though I know how to do it.

Link fixed 😺, thanks!

I was to eager to merge this PR, however I have forgotten to push my final commit from my computer to GitHub. Now, it is in my fork, however, what should I do?

Should I revert this merge, and there remerge it with the missing commit?
Should I create a new PR and then merge it?


test(semantic): add scope tests for IIFEs

DO NOT MERGE THIS PR UNTIL THE ASSOCIATED ISSUE HAS BEEN RESOLVED!
Reproduction for the bug described in #4452. Part of #4445

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4451  👈
#4450
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite

@DonIsaac Do these tests pass now? If so, is there any harm in merging this PR?


"only sequential writes supported" error in case of zero data/file hole

'cp' might skip over the zero data/file hole when copying a file to goofys,
causing "only sequential writes supported" error because offset != fh.nextWriteOffset.
Fix the issue by filling the range [offset, fh.nextWriteOffset) with zero.

there's WIP work here as well: https://github.com/kahing/goofys/commits/sparse
also depends on https://github.com/jacobsa/fuse/pull/66 which cp may need depends on where the sparse region is


Potential off by one in ChannelEvent Channel field

I think there's an off by one error in the channel for ChannelEvent objects. I know drums are always supposed to be on channel 10 but in the lib it comes back as 9. Cross referenced this with my DAW

Also I see this:
    public void setChannel(int c)
    {
        if(c < 0)
        {
            c = 0;
        }
        else if(c > 15)
        {
            c = 15;
        }
        mChannel = c;
    }

the reason I'm not so adamant about this being a bug is because maybe the goal is to reflect the data and not the actual number the users would see when interacting with channels.


how to use mui by vsto in ms office addin?

Hi,
i tried to use the mui in an office addin and vsto.
As I know are the restrictions from vsto that I can't use wpf windows directly. The only way to use wpf is to use a wpf control in a wpf host element on a winforms window.
I tried to use the modernWindow or a wpf window in the host element and it is not working.
I am now at the point that I have no idea what is to change to use mui as a wpf control ?!
Have anyone another experience to solve this scenario?
Thanks for advice!
Freddy

Hi,
I am launching WPF components from Excel, but could you explain more what you mean by "directly"?
I've never tried with ModernUI but it was my goal to do so at some point, and I didn't expect particular problems since I can already  do it with WPF.

Hi,
as "directly" I mean, that I can start an MUI app or window in the same way as from an winforms app.
Now I can handle hosted WPF controls but I have no idea what I can do to use mui.

I'm try the some. I just can't get the VS to resolve the StaticResources. Style="{StaticResource ContentRoot}"> => "The resource "xxx" could not be resolved.
Did anyone succeed integrating MUI in a VSTO?


Occasionally getting error "Could not connect to server: Connection refused"

We are occasionally getting the connection refused error in production. We get this error 2-3 times a day. Our app is deployed in App Engine. I couldn't find anything helpful in the logs. We are using psycopg2 v 2.8.5 binary as postgresql engine.
could not connect to server: Connection refused
Is the server running locally and accepting
connections on Unix domain socket "/cloudsql/xxxxxxxx:us-central1:xxxxx-production-1/.s.PGSQL.5432"?

Are you using connection pooling as part of your setup? Mind sharing the application code that shows how you're creating connections?

No. We aren't using connection pools. We are using Django framework.
DATABASES = {
'default': {
'ENGINE': 'django.db.backends.postgresql_psycopg2',
    'ENGINE': 'django.db.backends.postgresql',
    'OPTIONS': {
        'options': '-c search_path="schemaname"'
    },
    'NAME': 'dbname',
    'USER': 'username',
    'PASSWORD': 'xxxxxxx',
    'HOST': f'/cloudsql/xxxxxxxx:us-central1:xxxx-production-1',
    'PORT': '5432',
}

}

Hi @Tech2pr,
Opening and closing connections can have some additional overhead, your application may be experiencing the error when it is seeing higher database traffic hitting Django's limit for number of connections.
One thing you can try is setting the CONN_MAX_AGE param in your config. This will allow persistent database connections (similar to connection pooling) that connections can be re-used. This might help reduce the burden of opening and closing each connection since the default for CONN_MAX_AGE is 0.
Example:
DATABASES = {
'default': {
'ENGINE': 'django.db.backends.postgresql',
'CONN_MAX_AGE': 60,
'OPTIONS': {
'options': '-c search_path="schemaname"'
},
# ...

Also when you mention looking at the logs are you specifically talking about the App Engine logs? Sometimes the App Engine logs can provide some more detailed errors than say Cloud Logging logs.
Let me know if the above helps at all, thanks for raising this question!
Have a great day.

Thank you @jackwotherspoon. We have implemented the suggestion and we will monitor for the same error in the upcoming days.

How's the updated deployment looking @Tech2pr?

I’m going to close this since it’s been awhile now. Feel free to add additional info here and re-open if necessary.


Missing module error in Federated Learning with NVFlare tutorial

Discussed in https://github.com/Project-MONAI/tutorials/discussions/376

Originally posted by sorensenmarius September 28, 2021
I have followed each step of the federated learning with NVFlare without Docker tutorial, and have run into a problem when I want to start the server and the clients. When I do the "Run federated learning" step in the README I am getting this error from sub_start.sh:
/tutorials/federated_learning/nvflare/nvflare_example/nvflare_monai/bin/python3: No module named nvflare.private.fed.app.client.client_train
and
/tutorials/federated_learning/nvflare/nvflare_example/nvflare_monai/bin/python3: No module named nvflare.private.fed.app.server.server_train
I have tried redoing the whole installation and manually installing nvflare with both pip and pip3, however these tell me that all requirements are satisfied.
Has anyone run into this problem? I would love some input on how to move forward with this.

thanks, this is addressed by https://github.com/Project-MONAI/tutorials/pull/291.

@TaocsZhang I am facing the same issue, how you got any resolution to it ?
I am facing something like below :
The following files are not secure content.
fed_server.json
start fl because process of 2026 does not exist
System is in trouble and unable to start the task!!!!!
Thanks.

@Anand-afk  I solved this problem by using the same version of nvflare in every environment.
In my case, my provision tool works in nvflare==2.0.16
my federated leaning server works in nvflare==2.1.2
source code
I found that signature_filename is signature.json in 2.1.2
In 2.0.16, the signature_filename  is signature.pkl


bug onDetachedFromWindow

if load image with ImageLoader，and the task is loading ,i finish acitivty who with PhotoView  and set the listener with loading like below:
ImageLoader.getInstance().displayImage(url, new ImageView(), new SimpleImageLoadingListener() {                       public void onLoadingComplete(String imageUri, View view, Bitmap loadedImage) {                 view.setOnLongClickListener(listener)//crash with NullPointException;             }  });
before loading complete if the view is detached from window.
i think onDetachedFromWindow should not set mAttacher = null

Is this within a fragment? If so, you should check if (getView() != null) before setting the on long click listener

it is caused by that i use it in ViewPager within an activity, and reuse the PhotoView object which like the item reuse in listview

This is a leak if you are still referencing a view after an async operation completes, you need to check and see if the UI is still relevant before modifying it


Add /marvin delegate @[...]

https://github.com/NixOS/nixpkgs/pull/92243#issuecomment-653746032
(Aside: marvin could apply those same heuristics exposed in the comment on git logs to assign in the first place.)
Enables reliable identification of #44
I would argue a best-effort manual delegation by the delegating reviewer (which is marvin enabled, thus trustworthy) could be opt-out for the targeted reviewer. That implied those reviewers need not necesarily be in marvin's internal maintainer list.

Marvin doesn't need to know who is currently "responsible", it only needs to know that someone is. It might be nice to know who is, but that requires more explicit commands. The goal is to get out of everyone's way as much as possible and make a normal PR workflow possible.
The problem in https://github.com/NixOS/nixpkgs/pull/92243#issuecomment-653746032 has been resolved by #58.

Aside: marvin could apply those same heuristics exposed in the comment on git logs to assign in the first place.

That's not possible / desirable because of two reasons:

most people will not have signed up as a marvin reviewer / will not be available. I want people to actually respond to review requests, and that works best when they actively signed up for it. Repeatedly "nagging" the same few "domain experts" will just lead to them ignoring the requests
it further centralizes knowledge. If a random "shepherd" is selected instead, they can use their own judgement to delegate review and they will also learn from the feedback of the new reviewer.

Very true arguments. Closing this as errata.


add woo

Token Address: 0x4691937a7508860F876c9c0a2a617E7d9E945D4B
Token Name: Wootrade Network
Token Decimals: 18
Token Symbol: Woo
Official homepage: https://woo.network/
Etherscan: https://etherscan.io/token/0x4691937a7508860f876c9c0a2a617e7d9e945d4b
Twitter: https://twitter.com/wootraderS
Coinmarketcap page of token: https://coinmarketcap.com/currencies/wootrade/

Hi, @KennethAshley - thanks for your contribution. Your SVG image looks like is broken + has 100Kb size. Regular SVG logo has < 5Kb - could you try to fix it, please?


update FileEnumerator.cpp

fixes #3
this patch turns the throw in FileEnumerator into a return to allow the process to continue with other paths

Try that on for size?

Will review this tonight, am running a little behind thanks to BUILD this week. Thanks a lot for the edits thus far! 👾

I'll be watching Build remotely :-) yey for streaming video :-D


CSharp and Java versions for the-state-machine stack. Update TS and Python Stack to CDK 1.82.0. Documentation issues.

Good evening. In this PR I created the C# and Java version and made some changes in TS/Python stacks. These changes include a refactored piece of code due to being obsolete/deprecated.
1 - TS and Python versions updated to CDK 1.82.0.
2 - Fixed some comments. Just to keep the things organized.
Let me know if I need to change anything.
Ty

The step function definitions fluctuated A LOT last year, I think it is stable now though. I do know that this change breaks the typescript unit tests since the step function definition is different but testing it post deploy the behaviour is the same so the unit test just needs updated for the new definition from this new lambdaInvoke method

The step function definitions fluctuated A LOT last year, I think it is stable now though. I do know that this change breaks the typescript unit tests since the step function definition is different but testing it post deploy the behaviour is the same so the unit test just needs updated for the new definition from this new lambdaInvoke method

Hmm you're right! Tbh, when I refactor or add new languages to the stacks, I redeploy all stacks in all languages and check if it is working as expected, but I don't run the ts tests, that's a failure, my bad! I will fix and update the pr.

Hmm you're right! Tbh, when I refactor or add new languages to the stacks, I redeploy all stacks in all languages and check if it is working as expected, but I don't run the ts tests, that's a failure, my bad! I will fix and update the pr.

This made me wonder why my github action didn't fail which runs all the ts unit tests and cdk synths all the python versions. There was a bug in the ts script in that it was running all the tests but the exit on error flag wasn't set so when tests failed it reported that the build still passed. I just fixed that and some broken tests with my latest PR so some of the patterns you have updated might have had broken ts tests that I have now fixed. The test in this pr aside, it's worth checking if you run the ts tests in any of your branches and they fail if I have already fixed it as all tests now pass

This made me wonder why my github action didn't fail which runs all the ts unit tests and cdk synths all the python versions. There was a bug in the ts script in that it was running all the tests but the exit on error flag wasn't set so when tests failed it reported that the build still passed. I just fixed that and some broken tests with my latest PR so some of the patterns you have updated might have had broken ts tests that I have now fixed. The test in this pr aside, it's worth checking if you run the ts tests in any of your branches and they fail if I have already fixed it as all tests now pass

Hmm tomorrow I will check this PR and yours considerations!  At a glance I saw that the unit tests did not break, even changing constructor, but I need to analyze in detail.

Hmm tomorrow I will check this PR and yours considerations!  At a glance I saw that the unit tests did not break, even changing constructor, but I need to analyze in detail.

All you need to do is on your branch cd into the-state-machine/typescript, run npm i then npm run build && npm run test you should see a test failure:

All you need to do is on your branch cd into the-state-machine/typescript, run npm i then npm run build && npm run test you should see a test failure:

Yep, my tests were cached and jest couldn't test again, i have no idea why but I cleaned up here and now it's working.
Once the lambda invokation has changed, now the synth includes a new "ErrorEquals" parameters with lambda exceptions and, because of this, the previous test failed.
Now I think it is ok.

Yep, my tests were cached and jest couldn't test again, i have no idea why but I cleaned up here and now it's working.
Once the lambda invokation has changed, now the synth includes a new "ErrorEquals" parameters with lambda exceptions and, because of this, the previous test failed.
Now I think it is ok.


Update python runtime to push a master tag to Docker from Travis build

Update the Travis build to push a tag "master" to Docker when the master branch is updated.
This will make it easier to pull the image that aligns with the latest changes on the Master branch.
Reference can be taken from: apache/incubator-openwhisk-runtime-java#37

This is already done with latest


Using cast() on pointer variable throws a compiler error due to seeing the expression as a value type.

When using cast<T>() on a pointer variable, the compiler will see the pointer variable as a value variable. You can reproduce this bug by writing the following code:
type Vector2 : struct {
    x: int;
    y: int;
}

fn main(): int {
    v2: Vector2;

    v2ptr := &v2;
    ptrint := cast<int>(v2ptr);
    printf("v2 @ 0x%X\n", ptrint);
    return 0;
}

When compiled the compiler will spit out the following error:
RuntimeError:
    v2ptr := &v2;
    ptrint := cast<int>(v2ptr);
~~~~~~~~~~~~~~~~~~~~~~~~^
main.sat (10:25): Cannot cast expression of type 'Vector2' to 'int'.

The expected behavior is a successful conversion of pointer to integer and no compiler error.

As of the current commit this issue is no longer present. The above code now compiles correctly.


Change in clean energy building patterns

Sub-questions

[ ] What counties are expected to see the greatest growth (% and raw MWs) in clean energy development over the next few years? What about by 2030? 2040?
[ ] What about states? RTOs?
[ ] Which of these states have local control laws?

@bendnorman @TrentonBush is this still relevant or can we close?


What depends on without version

Follow up to #114. Fixes #110.

Wow thank you! This will be such a help!


Adding ViMusic app as vi_music.svg

Is this the original icon?

Yes, that one.

Ah, no, if you find the previous one more suitable then let it be that way :)
The reason I thought this one looks better is because of its simplicity, but reviewing it again now makes me think it is, way too simple.
Let the previous icon remain and wait for @medtty to comment.

I'll be closing this PR. The current icon has been agreed upon as more suitable. Thanks for contributing


Allow $user to be used as a context variable in sql queries

$user should be available to SQL queries as the OUser document representing the current logged in user.
select name from $user
select from posts where user = $user

Thx


Change Trace Number and Original Trace Number to string for legacy systems

Reference closed issues #236
A quick run of https://github.com/alecthomas/gometalinter (which can be ran for free on PR's via https://golangci.com/) gives the following error.
This would be a breaking change as int -> int64 isn't automatic.
I'm also running this on a 32-bit machine (yea, I know...), so that might be it.

UGH! Can we chat about this over a hangout. Not sure how I missed this issue.

Sure thing. I'm free today.

FYI, this was on an old 32bit system of mine.


dbContext..Update() is creating new records

When i do something like:
var e = new Entity();
e.Id = 100;
e.Name = "Name 100";
dbContext.Entities.Update(e);
dbContext.SaveChanges();

a new record is created.
Is this the correct behaviour?

@DiegoZoracKy it should attempt to update an existing record, not insert a new one. Can you share a full code listing that we can use to reproduce the issue? (i.e. the code for Entity and your context)

@rowanmiller i've created something simple to show the case. See:
using Microsoft.AspNet.Mvc;
using System.Linq;
using Microsoft.Data.Entity;
using Newtonsoft.Json.Linq;

public class TestUpdate
{
	public int Id { get; set; }
	public string Name { get; set; }
}

public class MyContext : DbContext
{
	public DbSet<TestUpdate> TestUpdates { get; set; }
}

[Route("api/[controller]")]
public class TestController : Controller
{
	private MyContext dbContext;

	public TestController(MyContext context)
	{
		dbContext = context;
	}

	[HttpGet]
	public IActionResult Get()
	{
		var e = new TestUpdate();
		e.Name = "Name 100";
		dbContext.TestUpdates.Update(e);
		dbContext.SaveChanges();

		return new ObjectResult(dbContext.TestUpdates.AsNoTracking().ToList());
	}
}

Everytime you hit api/test (GET) a new record will be saved.

@rowanmiller  i've found what it seems to be another weird behavior on Update method. Let's suppose that in my previous example, AnotherClass exists on the project and it makes a reference to TestUpdate, like:
	public class AnotherClass
	{
    	public int Id { get; set; }
    	public virtual List<TestUpdate> TestUpdate { get; set; }
	}

Thinking about SQLServer, this will create a column AnotherClassId on the TestUpdate Table, with a reference to records on AnotherClass, right?
The problem is that when i do .Update on TestUpdate, this reference is being set to NULL.

@DiegoZoracKy regarding the first issue you reported...
This situation is arising because TestUpdate.Id is assigned a value of 0. The fact that we are implying this means it is added (even though you called Update) is a bug and we will fix it. That said, do you actually have an existing row in the database with an Id of zero... or is that just a coding error where you are not setting the key property?
If having a key value of zero is a legitimate requirement, then you can workaround this until we fix the bug by explicitly setting the state (rather than calling Update).
dbContext.Entry(e).State = EntityState.Modified;

@DiegoZoracKy can you open a new issue for the second case you mentioned (and include code that demonstrates the issue again).

@rowanmillerNo, id 0 is not a requirement of my software.  I've created this simple example for you to see that the Update method is creating new records. Good that it is a Bug and not its correct behavior.
I will create a new issue for sure, within some hours. This current issue is something easy to be avoided, just bringing a performance penalty by checking if the record exists for real before update it. But this last one can be a huge problem.

@DiegoZoracKy just to be clear, it only inserts a row if the key is set to zero. If it is set to anything else then it will correctly update it.
i.e. this code
var e = new TestUpdate();
e.Name = "Name 100";
e.Id = 5;
dbContext.TestUpdates.Update(e);
dbContext.SaveChanges();

Produces this SQL
exec sp_executesql N'SET NOCOUNT OFF;
UPDATE [TestUpdate] SET [Name] = @p1
WHERE [Id] = @p0;
SELECT @@ROWCOUNT;
',N'@p0 int,@p1 nvarchar(4000)',@p0=5,@p1=N'Name 100'

Not saying that it's not a bug... but just that you'll only see the insert behavior if you are attempting to update an entity with a key set to zero.

@rowanmiller i just tried setting a value to Id, like that:
var e = new TestUpdate();
        e.Id = 100;
        e.Name = "Name 100";
        dbContext.TestUpdates.Update(e);
        dbContext.SaveChanges();

And a new record was created. But for this test i used inMemory (EntityFramework.InMemory: 7.0.0-rc1-final) instead of SQL Server.

@rowanmiller And, a new issue about the reference being lost during Update, that i commented above: #3948

@DiegoZoracKy I was able to reproduce that issue with InMemory. It's different from the original bug on this issue (which is already fixed), so I opened https://github.com/aspnet/EntityFramework/issues/3953 for the new issue.

Thanks @rowanmiller. I will track this new issue.

Re-opening as we still need this to track the issue where the entity being passed to Update is marked as added if it has no key value assigned.


Increase max image size for gif

Raises the maximum size of gifs allowed to 15MB per Twitter's documentaion.
This resolves https://github.com/tweepy/tweepy/issues/1336
Note: I set the max size to 15360KB (15 MB). I noticed that the max size for images was slightly less than 5 MB though (4883KB). I couldn't find the reason for that, but if there is one I can revise the size for gifs as well.

I noticed this PR because a similar issue #1359 was just brought up.

@achasveachas The changes look good, except for the number. If you can adjust that then I'd approve. See notes below.
The function you modified does a call to _pack_image which does a calc given a value - max_size * 1024 https://github.com/tweepy/tweepy/blob/3733fd6/tweepy/api.py#L1364. And so using max_size as 4883 gets to 5,000,192 - which is slightly over 5MB and not under as you mentioned.
Reverse engineering that:
5MB
= 5*000*1000/1024
= 4882.8125
~= 4883

Therefore:
15MB
= 15*1000*1000/1024
= 14648.4375
~= 14648  # rounded up to be safe

Therefore 14648 is your magic number ⭐  🚀

I see you have a doc reference already, but here's another extra reference I found: https://developer.twitter.com/en/docs/media/upload-media/uploading-media/media-best-practices

Image size <= 5 MB, animated GIF size <= 15 MB

@MichaelCurrin that fix is in.

I added file_type as an optional argument to API._pack_image. Now it will only determine the file_type in cases where it isn't passed in.

Thanks!


Replace Transducers.jl by OhMyThreads.jl

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

Project coverage is 84.92%. Comparing base (ca2426d) to head (c1a7720).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   84.92%   84.92%
=======================================
  Files          26       26
  Lines         670      670
=======================================
  Hits          569      569
  Misses        101      101

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


Add type augmentation test

Plugins of Vue.js usually add some static or instance properties/methods to Vue and some new options to component options.
I think it is good to provide how to augment their typings for the plugin developers.
I add the test code of such augmentation by this PR and it would be the example for it.

Badly need this! The example test helps a lot!


vive nested folders: clicked folder stays focused and prevents other presses

when I call
inputObject.pressed(true)
the folder which i opened stays highlighted and prevents any further clicks to other folders and/or buttons.^
detaled information:
i am creating my gui recursively like in the nested example:
objects.forEach(function (element) {
      this.addGUIElement(this.gui,element);
    }, this);

    var inputObject = dat.GUIVR.addInputObject(this.gamepad.object);
    this.scene.add(inputObject);

    window.addEventListener('viveControllerThumbpadPressed', (e) => {
      inputObject.pressed(true);
    });

 addGUIElement(folder, subtree) {
    if (subtree.children) {
      console.log('creating subfolder %s', subtree.label);
      const newFolder = dat.GUIVR.create(subtree.label || '');
      newFolder.name= subtree.label;
      subtree.children.forEach(function (element) {
        this.addGUIElement(newFolder, element);

      }, this);
      folder.addFolder(newFolder);

    } else {
      var setting = {
        travel: () => {
          console.log(subtree.label + 'pressed');
        }
      }
      var btn1 = folder.add(setting, 'travel');
      btn1.name(subtree.label);
      console.log('created button %s', subtree.label);
      return;
    }
  }

I'll try to remember to check that soon.  Maybe ping me if you don't hear anything by mid next week.  You might also want to try with my fork https://github.com/xinaesthete/dat.guiVR/tree/ImageButton is a branch with a few new (not well documented) features added but it should do the standard stuff fine and may well have bug fixes relevant to this, I don't remember off the top of my head.
Anyway, that's the version I was using when I was last actively working with this and I certainly had some nested folders behaving reasonably with Vive.

@xinaesthete Actually, your fork was the first one I tried out due to the ImageButton functionality. I would love to use it, but it's not possible without any documentation.
For Example, I can't use the addFolder function because it is missing in your branch. If you can show me an example of how you used it, this would be great!

@ManuelGraf https://github.com/xinaesthete/dat.guiVR/blob/ImageButton/examples/imagebutton.html is all working here (non-Vive example, but Vive stuff should be working as well).  That example doesn't cover quite 100% of new features, doesn't have much in the way of comments, and the interface for some of the new stuff is liable to change at some point, but it is using addFolder and image buttons / grids as well as colour pickers and text box input.

I've included that example in the examples index page as "Simple ImageButton Example".

@ManuelGraf is it possible that you accidentally checked out the master branch of my fork, rather than ImageButton?  That will be behind.


toposort models and chunk big model files

Issue #, if available:
https://github.com/aws/aws-sdk-js-v3/issues/1460
Related: https://github.com/aws/aws-sdk-js-v3/pull/1510
Description of changes:
This change exports shapes in topological order, and chunk them into models/models_0.ts, models/models_1.ts, models/models_2.ts..., models/models_x.ts. A models/index.ts will re-export shapes from each chunks.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks like the TopologicalIndex is not available in Travis environment.

Re-ran Travis builds as 1.0.11 was released including the TopologicalIndex.


Double Tap Zoom #HelpWanted

Hi,
thanks for the beautiful plugin. By default - with a double-tap I get a soft-zoom, and with another one I get a deeper-zoom. An extra double-tap will reset the zoom to 0. I would like to have directly the deeper-zoom with a single double-tap, and a reset if I repeat the action.
How I can achieve that? Thanks in advance!

You want to customize the scale state cycle.
The default one is:
PhotoViewScaleState defaultScaleStateCycle(PhotoViewScaleState actual) {
  switch (actual) {
    case PhotoViewScaleState.initial:
      return PhotoViewScaleState.covering;
    case PhotoViewScaleState.covering:
      return PhotoViewScaleState.originalSize;
    case PhotoViewScaleState.originalSize:
      return PhotoViewScaleState.initial;
    case PhotoViewScaleState.zoomedIn:
    case PhotoViewScaleState.zoomedOut:
      return PhotoViewScaleState.initial;
    default:
      return PhotoViewScaleState.initial;
  }
}

To achieve what you want, you need to create you own cycle. I guess this works for you:
PhotoViewScaleState myScaleStateCycle(PhotoViewScaleState actual) {
  switch (actual) {
    case PhotoViewScaleState.initial:
      return PhotoViewScaleState.originalSize;
    default:
      return PhotoViewScaleState.initial;
  }
}

and then pass your cycle function to PhotoView as the scaleStateCycle option.

Works perfectly as expected, thanks!


Add --registry option

Coverage remained the same at 91.429% when pulling 26389229ee20dc17c700f1c5b82329d5dce9f7f9 on feature/add-registry-option into a1decf49224d28116144764752fbb49ff484da39 on master.

Coverage remained the same at 91.429% when pulling 26389229ee20dc17c700f1c5b82329d5dce9f7f9 on feature/add-registry-option into a1decf49224d28116144764752fbb49ff484da39 on master.


Investigate the need of Request\RestRequest, Request\RestRequestFactory, Request\RestRequestProxy

Do we need those classes to generate routes?
Blocks https://github.com/ongr-io/ApiBundle/issues/94

IMO, when we simplified the BATCH command, I cannot find the reason which we need to use these classes anymore. I removed the proxy class in my work; and the near future, I think I will remove the Factory class too.

Good, I also thought that we dont need those clases, just created this issue as a double check.


[BUG-6543][Master] process instance state is always running when failure task

close #6543 process instance state is always running when failure task exists

Codecov Report

Merging #6547 (b4c2c65) into dev (ba4cb9d) will decrease coverage by 0.01%.
The diff coverage is 2.50%.

@@             Coverage Diff              @@
##                dev    #6547      +/-   ##
============================================
- Coverage     38.41%   38.40%   -0.02%
- Complexity     3205     3207       +2
============================================
  Files           645      645
  Lines         25819    25840      +21
  Branches       2792     2797       +5
============================================
+ Hits           9919     9923       +4
- Misses        14991    15006      +15
- Partials        909      911       +2

Impacted Files
Coverage Δ

...ache/dolphinscheduler/dao/entity/TaskInstance.java
64.19% <0.00%> (-3.78%)
:arrow_down:

...uler/server/master/runner/EventExecuteService.java
0.00% <0.00%> (ø)

.../server/master/runner/StateWheelExecuteThread.java
0.00% <0.00%> (ø)

...er/server/master/runner/WorkflowExecuteThread.java
0.00% <0.00%> (ø)

...lphinscheduler/service/process/ProcessService.java
37.14% <0.00%> (-0.08%)
:arrow_down:

...r/service/queue/PeerTaskInstancePriorityQueue.java
73.91% <33.33%> (+10.27%)
:arrow_up:

...er/master/dispatch/host/assign/RandomSelector.java
83.33% <0.00%> (+5.55%)
:arrow_up:

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 ba4cb9d...b4c2c65. Read the comment docs.


A tool to produce a CSV linking collection members with file paths

This tool creates a .csv file listing the members of a collection, their associated member status, LID, their associated VID, and the attributed path.
The tool should be called pds4_create_collection_member_index.py
The following specifications for this module:

Input argument should be described in comment (or docstring?) as the path containing the collection_*.xml file.  Default should be the current directory.

Using lxml, find the filename of the collection product (call it collection_product_filename) at Product_Collection:File_Area_Inventory:File:file_name

Open the file collection_product_filename, and read it.  Each line is comma-separated, with two elements (we should ask Mitch whether it’s okay to assume this in all cases).  The first is the member_status (either P for primary or S for secondary).  The second is the LIDVID, but parse this separately into the LID and the VID.  Create a dictionary collection_member_index in which the key is the LID and the value is a list with three items, namely the VID, the member_status, and a blank string that you will later replace with the path.

Use os.walk to look through all subdirectories for files whose extension is .lblx or .xml

Using lxml, find the LID of each file found.  If it matches a key in the dictionary, then put its path into the corresponding place.  If it does not match any key in the dictionary, then create a line of output saying "PDS4 label found but not a member of this bundle: <filepath>, <lid>."

If there are any remaining entries in your dictionary whose paths have not been filled, throw an error.

Write the dictionary collection_member_index into a file collection_member_index.csv

To clarify: The tool that answers this issue should be given a single collection_*.xml file and should create a CSV in the same directory with the characteristics described.  A different issue will cover the task of running this tool recursively.


How do I connect my Twitch account?

I want to stream with this mod but I have no idea how to connect my Twitch account and it doesn't seem like it works whenever I go online.

Use StreamCore, it’s a mod library and it will show you how to set it up here


Do not treat IP addresses as numbers

Single or double quotes are optional in most cases, such as when using IP addresses. Unfortunately, the extension thinks the IP address is a number when no quotes are surrounded:

Just a heads up that this will appear fixed - but that's because I've temporarily simplified the syntax highlighter. Still needs to be solved permanently.


Delete build directories

Addresses this card: https://trello.com/c/JzehzCa7/925-make-the-mcrouter-cookbook-delete-its-own-build-directory-correctly-this-time

@eherot I feel like there should be some tests to assert that those folders are, in fact, not present.

You mean like this one? https://github.com/evertrue/mcrouter-cookbook/blob/master/test/integration/default/serverspec/default_spec.rb#L63

Yup! :+1:

@jeffbyrnes is this :+1: ?

@eherot yep, I think this is good to go. Thanks for fixing this up!


Using "existing" for parent resources fails with BadRequest

Bicep version
Bicep CLI version 0.3.126 (a5e4c2e567)
Describe the bug
The expectation people have is that an "existing" resource reference is interchangeable with a full resource definition, much like how a C/C++ function declaration is always valid as a substitute for a function definition.
With Bicep, I get errors even for very basic templates that I expect to work when using the "existing" keyword.
To Reproduce
Sample script:
param VMName string

resource vm 'Microsoft.Compute/virtualMachines@2020-12-01' existing = {
  name: VMName
}

resource vmDependency 'Microsoft.Compute/virtualMachines/extensions@2020-12-01' = {
  parent: vm
  location: vm.location
  name: 'DAExtension'
  properties:{
    publisher: 'Microsoft.Azure.Monitoring.DependencyAgent'
    type: 'DependencyAgentWindows'
    typeHandlerVersion: '9.5'
    autoUpgradeMinorVersion: true

  }
}

Outputs this error:
New-AzResourceGroupDeployment: 3:37:06 PM - Error: Code=InvalidTemplate; Message=Deployment template validation failed: 'The template resource 'YourVMNameHere/DAExtension' at line '13' and column '9' is not valid: The template function 'reference' is not expected at this location. Please see https://aka.ms/arm-template-expressions for usage
details.. Please see https://aka.ms/arm-template-expressions for usage details.'.

New-AzResourceGroupDeployment: The deployment validation failed

Update: this seems to be caused by the "vm.location" expression, which isn't technically an issue with "exists", but instead is an issue that would be triggered by usages of "exists" that people expect ought to work...

Related to #1988


Aggregate data for display

Instead of displaying all data points, aggregate the data for specifid periods (e.g. daily or every 5 seconds) using particular aggregation function, e.g. average or median.

Hi @sergeychernyshev,
I was wondering if you had any plans to implement this. We've been collecting lots of data for the last three years but the web interface is way too slow when loading it. I thought that aggregation might fix the problem and I've found this ticket from 2012.
The other alternative I imagine would be to drop data older than, say, 3 months, but we would like to keep it if possible.
Thanks!

Well, deletion of old data is available now.
Not sure about aggregation though - it would change quite a lot of display logic. Are you grabbing more than one datapoint per day per URL?

Actually - yes, we were collecting one datapoint per URL every half an hour! But from your comment I'm starting to think now that this is really overkill... 😅
To be fair, this is an old box that was manually build around 2012. We didn't have any other good monitoring tools back then, so we used showslow to kinda fulfil that role. We have those tools now, so I've trying to replace this server with fancy a new one that it's completely automated.
I'm very happy to know that showslow supports automatic deletion of old data. I will enable this on the new machine!


Executing workflows in the UI unexpectedly uses the namespace last selected in the UI

Checklist

[x] Double-checked my configuration.
[ ] Tested using the latest version.  -- looking to upgrade soon
[x] Used the Emissary executor.

Summary
The issue can be summarised as 3 distinct behaviours:

Executing workflows in the UI unexpectedly uses the namespace last selected in the UI, e.g set by filtering workflow list in the by the namespace field (possibly introduced in #2441).

There seems to be no way to specify a namespace in the UI for a ClusterWorkflowTemplate without a namespace defined in the manifest yaml (at least I could not find one, please correct me if I'm wrong). So, the problem described in (1) above seems to be the only workaround for this missing feature (see step-by-step instructions below for an example).

Submitting a workflow from the "Workflows" UI page does allow selecting the workflow namespace, but this action also seems to "select a namespace" same as filtering by namespace in the UI described in (1).

I think this behaviour in (3) may be desired. But, I think that it, along with the feature mentioned in (1) should probably be documented as well. Not sure about (2), how come we're not allowing to set namespace in CWTs? Please let me know if anything in the descriptions seems wrong.
We're running argo workflows version 3.2.4
Diagnostics
Workflow yaml (copy pasted from the docs, not tested this particular workflow myself, but should work the same I reckon):
apiVersion: argoproj.io/v1alpha1
kind: ClusterWorkflowTemplate
metadata:
  name: cluster-workflow-template-whalesay-template
spec:
  templates:
  - name: whalesay-template
    inputs:
      parameters:
      - name: message
    container:
      image: docker/whalesay
      command: [cowsay]
      args: ["{{inputs.parameters.message}}"]

Step-by-step instructions:

Have a ClusterWorkflowTemplate with no namespace defined, let's call it "my-workflow".
Use the Workflows -> +Submit workflow action to create the "argosay" example workflow, in the default namespace
Use the Cluster Workflow Templates -> Create New Cluster Workflow Template action to create an instance of "my-workflow" template. Result: it runs in the default namespace.
Repeat step 2, but use workflows namespace
Repeat step 3. Result: it runs in the workflows namespace.

Additional discussion is in the thread here:
https://cloud-native.slack.com/archives/C01QW9QSSSK/p1660849688488899
NOTE: will add logs when I'm back in office.
# Logs from the workflow controller:
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

# If the workflow's pods have not been created, you can skip the rest of the diagnostics.

# The workflow's pods that are problematic:
kubectl get pod -o yaml -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

# Logs from in your workflow's wait container, something like:
kubectl logs -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

Message from the maintainers:
Impacted by this regression? Give it a 👍. We prioritise the issues with the most 👍.

@dpadhiar can you do the initial triage?

I've no comment on this issue. Someone needs to start up the UI and start debugging.

Hey @alexec  @asfaltboy  can you please assign me this project,I would like to work on this.


Disable sign up per oauth source

Is your feature request related to a problem?
In #1620, new users from Oauth are also rejected if sign-up is disabled. However, the other way should also be useful, e.g. when you are running a private Gitea instance, and want to allow any user there to also be able to use memos.
Describe the solution you'd like
I propose to enable/disable sign-up separately for each auth source, the current option then only disables sign-up from the form.
Additional context
No response

You can fill the identifier filter in SSO setting. It will only allow those user who passes the filter checks. https://github.com/usememos/memos/blob/main/server/auth.go#L88

Thanks for your reply!
I think I need to clarify: I want to prevent anyone to register an account with the built-in register form, but allow new users from SSO sources, not about preventing certain users from an SSO source. In the current release version 0.12.2, this is possible, but after #1620, this seems to be impossible. Or maybe I misunderstood something about #1620 and there is nothing to worry about?

Removed the disable sign up in UI check for SSO https://github.com/usememos/memos/pull/1684

I noticed that an SSO user sign up works only if the general "Allow user signup" is enabled. This allows also normal user (non-sso) to sign up.
How to allow only SSO user to sign up?

Need the feature only sso user to sign up too.


Limit group by to MaxGroupByPoints (currently 100,000)

Fixes #1992
Also fixes a bug in results.Error().  It wasn't looking into the slice of series to see if those also had an error.  I needed to fix that to make my tests work otherwise I couldn't see the error.

One possible enhancement we could make to this is to either use the limit or the end time of the query.

Made the test @corylanou requested, but now the tests are failing. Looking into it.

I see the original implementation of Aggregated did a walk, so IsRawQuery is not an exact copy.

@otoolep yeah, IsRawQuery should actually be doing the same thing Aggregated() is doing, but inverse :)

D'oh @pauldix of course, reverse it.

LGTM :ship:

@otoolep Super big awesome thank you for picking this up!


Icon.ToolbarAndroid: logo property does not work

Assuming not an issue as related pr indicates now working