Documentation outdated

How to categorize this issue?

/area documentation
/kind bug
/priority normal
What happened:
The component descriptor in  https://github.com/gardener/landscaper/blob/master/docs/tutorials/01-create-simple-blueprint.md doesn't contain a repositoryContexts. The deployment is not working.
What you expected to happen:
Documentation is correct
How to reproduce it (as minimally and precisely as possible):
See above
Anything else we need to know?:
Environment:

Landscaper version:
Kubernetes version (use kubectl version):
Others:

Only the first component descriptor that just includes the helm chart does not contain a repository context.
The idea here was that the first component descriptor is not finished yet (the blueprint is still missing).
So no unfinished component descriptor is uploaded.
The complete component descriptor is defined in the upload section which then also contains the repository context.
See https://github.com/gardener/landscaper/blob/master/docs/tutorials/01-create-simple-blueprint.md#remote-upload
There is also a note that says that the repository context and the blueprint have to be added:

Maybe we should make that explanation more prominent.
e.g. ⚠ Note that the repository context as well as the blueprint resource should be added to the component descriptor


Copy environment variables

If I set environment variables like SYMFONY__RABBITMQ_HOST on my server (or in this case docker container, but whatever), then add a rabbitmq-cli-consumer program to my supervisord config, supervisor is kind enough to copy the environment into the process that runs rabbitmq-cli-consumer, but alas when the consumer runs the symfony command, that environment isn't present, so it fails. I hope that makes sense :smile:
Ideally rabbitmq-cli-consumer would copy its environment into the shell running the configured command. Alternatively being able to set environment variables on the command line (or from a config file) would also be ok.

I'll take this into consideration for v2, but can't promise it'll be in the first version. I do see the use case though, so I'll add it to the feature list.


Updated addressable dependency

Fixes https://github.com/fastlane/fastlane/issues/4069

:+1: :ship: it


Strip trailing whitespace on each row in pip list

Like pip, we should strip the trailing whitespace in each row.

This also breaks tests if your editor strips trailing whitespace today, which keeps biting me :joy:


BlueLeft moving to side

Overview
BlueLeft doesn't move to the side properly
Solution
Possibly raise the block, check that mecanum wheels aren't slipping.
BlueLeft current code for reference

Fixed with gyro code - 0803f306bcfe878f9cb7bc0c54cce23d588ba9eb


I want to have inlined Aliases (with CVE) when I use the packages API endpoint

When I use the packages API endpoint for instance at https://public.vulnerablecode.io/api/packages/ I want to see the aliases for each reported vulnerabilities.
Today I only get the VCID and this would demand to make a second query on each package details to get this.

In ORT we have the same issue. Here we use the packages/bulk_search API to query vulnerability information for multiple packages.
I am currently working on a PR against ORT to support aliases. Can we expect that the information will be present in the result of the bulk_search in the following format:
[
  {
    "name": "some-package",
    "namespace": "some-namespace",
    "purl"<EMAIL_ADDRESS>    "qualifiers": {},
    "resolved_vulnerabilities": [],
    "subpath": "",
    "type": "maven",
    "version": "1.0",
    "url": "http://testserver/api/packages/3467",
    "unresolved_vulnerabilities": [
      {
        "references": [
          {
            "reference_id": "",
            "scores": [],
            "source": "",
            "url": "https://some.url"
          }
        ],
        "aliases": [
          {
            "alias": "CVE-yyy"
          },
          {
            "alias": "GHSA-zzz"
          }
        ],
        "url": "http://testserver/api/vulnerabilities/60",
        "vulnerability_id": "VCID-xxx"
      }
    ]
  }
]

@oheger-bosch with the latest RC we now have aliases at package level for example this came from the bulk search:
[{'url': 'http://public.vulnerablecode.io/api/packages/97100', 'purl'<EMAIL_ADDRESS>'type': 'npm', 'namespace': '', 'name': 'grunt-radical', 'version': '0.0.14', 'qualifiers': {}, 'subpath': '', 'affected_by_vulnerabilities': [{'url': 'http://public.vulnerablecode.io/api/vulnerabilities/6777', 'vulnerability_id': 'VCID-d1c1-43rx-aaas', 'summary': "Improper Neutralization of Special Elements used in an OS Command ('OS Command Injection') in grunt-radical.", 'references': [{'reference_url': 'https://www.npmjs.com/advisories/935', 'reference_id': '', 'scores': [], 'url': 'https://www.npmjs.com/advisories/935'}, {'reference_url': 'https://github.com/advisories/GHSA-4627-w373-375v', 'reference_id': 'GHSA-4627-w373-375v', 'scores': [], 'url': 'https://github.com/advisories/GHSA-4627-w373-375v'}], 'fixed_packages': [], 'aliases': ['GHSA-4627-w373-375v', 'GMS-2020-283']}], 'fixing_vulnerabilities': [], 'unresolved_vulnerabilities': [{'url': 'http://public.vulnerablecode.io/api/vulnerabilities/6777', 'vulnerability_id': 'VCID-d1c1-43rx-aaas', 'summary': "Improper Neutralization of Special Elements used in an OS Command ('OS Command Injection') in grunt-radical.", 'references': [{'reference_url': 'https://www.npmjs.com/advisories/935', 'reference_id': '', 'scores': [], 'url': 'https://www.npmjs.com/advisories/935'}, {'reference_url': 'https://github.com/advisories/GHSA-4627-w373-375v', 'reference_id': 'GHSA-4627-w373-375v', 'scores': [], 'url': 'https://github.com/advisories/GHSA-4627-w373-375v'}], 'fixed_packages': [], 'aliases': ['GHSA-4627-w373-375v', 'GMS-2020-283']}]}]

@TG1999, thank you for the update. For ORT, I have created https://github.com/oss-review-toolkit/ort/pull/6478 which adds support for reading aliases, so that instead of the VCID a more meaningful alias is used as identifier for the vulnerability created by ORT.
Can you confirm that the representation of aliases in the responses remains stable, so that the PR can be merged as is?

Can you confirm that the representation of aliases in the responses remains stable
@oheger-bosch for now we are planning to keep aliases in the response if we intend to refactor the response in the future we will let you know .


HARVESTER: Cloud provider enhancement

Summary
Fixes #

Occurred changes and/or fixed issues

https://github.com/harvester/harvester/issues/1580
https://github.com/harvester/harvester/issues/1458

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

@n313893254   Please resolve the conflict.


Replace all uses of failure with std::Error

We are not actually using any cool features of failure, backtrace collection can slow us down and, today, implementing std::error::Error is as simple as impl Error for MyType.
We should remove failure dependecy and just replace it with Box<dyn std::Error>. For bail macro, we can do just Err("error message")? (there's From<&str> for Box<dyn std::Error>).
Here's how I've done this transition for another project recently: https://gitlab.com/jD91mZM2/rnix/merge_requests/13

today, implementing std::error::Error is as simple as impl Error for MyType

Well, there's also cause() and fmt.
I also like https://docs.rs/snafu/0.4.1/snafu/ as an alternative to failure.

Snafu is cool, but we don't actually do that much error handing, so I'd prefer to stick with std::Error for now.
Fixing this issue should also close https://github.com/rust-analyzer/rust-analyzer/issues/1331

I'd like to take this! ☺

go for it! It maybe makes to not do this all at once, and instead send a spearate PR for crate


Adding analytics events

This is exact duplicate of https://github.com/artsy/force/pull/2594 just to check if the build is going to be more cooperative.

Did not help:( closing


kube-scheduler config file api version

the api version componentconfig/v1alpha1 has been deprecated since 1.9 is not supported in 1.13, which should be changed to apiVersion: kubescheduler.config.k8s.io/v1alpha1

Just here to correctly indent the fix:
cat <<EOF | sudo tee /etc/kubernetes/config/kube-scheduler.yaml
apiVersion: kubescheduler.config.k8s.io/v1alpha1
kind: KubeSchedulerConfiguration
clientConnection:
  kubeconfig: "/var/lib/kubernetes/kube-scheduler.kubeconfig"
leaderElection:
  leaderElect: true
EOF

Then verify with
systemctl status kube-scheduler
and see that the status has moved to Active: active (running) (it auto-restarts every 5 seconds).
Make sure to do this on all (3) controllers.
Run
kubectl get componentstatuses --kubeconfig admin.kubeconfig
to verify that the scheduler is reporting Healthy as well.

as per https://github.com/kubernetes/kubernetes/pull/66916 , using a config file is still alpha.
changed the service file to use the same options as in the config file

[Unit]
Description=Kubernetes Scheduler
Documentation=https://github.com/kubernetes/kubernetes
[Service]
ExecStart=/usr/local/bin/kube-scheduler
--leader-elect=true
--kubeconfig=/var/lib/kubernetes/kube-scheduler.kubeconfig
--v=2
Restart=on-failure
RestartSec=5
[Install]
WantedBy=multi-user.target

Fixed on master.

For the future readers, the new version is currently: kubescheduler.config.k8s.io/v1beta1
Example:
apiVersion: kubescheduler.config.k8s.io/v1beta1
kind: KubeSchedulerConfiguration
clientConnection:
  kubeconfig: "/var/lib/kubernetes/kube-scheduler.kubeconfig"
leaderElection:
  leaderElect: true

/ref https://kubernetes.io/docs/reference/scheduling/config/

For the future readers, the new version is currently: kubescheduler.config.k8s.io/v1beta1
Example:
apiVersion: kubescheduler.config.k8s.io/v1beta1
kind: KubeSchedulerConfiguration
clientConnection:
  kubeconfig: "/var/lib/kubernetes/kube-scheduler.kubeconfig"
leaderElection:
  leaderElect: true

/ref https://kubernetes.io/docs/reference/scheduling/config/

Whether KubeSchedulerConfiguration only enable it in 1.19, I found some information on v1.18 already started to use this, but I through the kubectl api-resources did not find this kind of resources

@JulienBreux I am still seeing the same Issue. My yaml for kube-scheduler looks like this
apiVersion: kubescheduler.config.k8s.io/v1beta1
kind: KubeSchedulerConfiguration
clientConnection:
kubeconfig: "/var/lib/kubernetes/kube-scheduler.kubeconfig"
leaderElection:
leaderElect: true
After reloading and restarting I am still not able to restart the scheduler
● kube-scheduler.service - Kubernetes Scheduler
Loaded: loaded (/etc/systemd/system/kube-scheduler.service; enabled; vendor preset: enabled)
Active: activating (auto-restart) (Result: exit-code) since Thu 2022-01-20 11:42:38 UTC; 1s ago
Docs: https://github.com/kubernetes/kubernetes
Process: 9675 ExecStart=/usr/local/bin/kube-scheduler --config=/etc/kubernetes/config/kube-scheduler.yaml --v=2 (code=exited, status=255)
Main PID: 9675 (code=exited, status=255)
kube-scheduler[9675]: I0120 11:42:38.664840    9675 flags.go:27] FLAG: --scheduler-name="default-scheduler"
kube-scheduler[9675]: I0120 11:42:38.664847    9675 flags.go:27] FLAG: --stderrthreshold="2"
kube-scheduler[9675]: I0120 11:42:38.664853    9675 flags.go:27] FLAG: --use-legacy-policy-config="false"
kube-scheduler[9675]: I0120 11:42:38.664860    9675 flags.go:27] FLAG: --v="2"
kube-scheduler[9675]: I0120 11:42:38.664866    9675 flags.go:27] FLAG: --version="false"
kube-scheduler[9675]: I0120 11:42:38.664878    9675 flags.go:27] FLAG: --vmodule=""
kube-scheduler[9675]: F0120 11:42:38.665137    9675 helpers.go:119] error: no kind "KubeSchedulerConfiguration" is registered for version "kubescheduler.config.k8s.io/v1beta1"
systemd[1]: kube-scheduler.service: Main process exited, code=exited, status=255/n/a
systemd[1]: kube-scheduler.service: Unit entered failed state.
systemd[1]: kube-scheduler.service: Failed with result 'exit-code'.
My Kuber version is
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.2", GitCommit:"81753b10df112992bf51bbc2c2f85208aad78335", GitTreeState:"clean", BuildDate:"2018-04-27T09:22:21Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Ran into this same error today.
The apiVersion is now v1beta2
Example:
apiVersion: kubescheduler.config.k8s.io/v1beta2 kind: KubeSchedulerConfiguration clientConnection:   kubeconfig: "/var/lib/kubernetes/kube-scheduler.kubeconfig" leaderElection:   leaderElect: true

For v1.25:
apiVersion: kubescheduler.config.k8s.io/v1

or
apiVersion: kubescheduler.config.k8s.io/v1beta3

https://kubernetes.io/docs/reference/scheduling/config/

For v1.25:
apiVersion: kubescheduler.config.k8s.io/v1

or
apiVersion: kubescheduler.config.k8s.io/v1beta3

https://kubernetes.io/docs/reference/scheduling/config/

Thank you bro you saved my day!


Remove unused opencv linking

Opencv is not used in any cpp file, and it's also not specified in the package.xml. After removing, the package still builds in a fresh docker image.

Thanks! Your PR was merged in 16cbbaada48ed886a239a4e7ff0c302b00b87588


Add aliases to options

The --restart option should also be available as --background and --fork to make it sort of easier to remember.
Not totally sure about this.

I think for --background alias more appropriate --daemon :)


Add an option to make all calls via the full qualified sub name

Add an option to make all calls via the full qualified sub name.  If the code for a package calls a local sub, it just uses localsub(args).  However, if another module, like a test driver, monkeypatches the sub with like:
MyPackage.localsub = sub { 0 };

this new version won't be called since we don't call MyPackage.localsub within MyPackage, we just call localsub.  Add an option to always call thru the fully qualified name for (rare) cases like this one.

-f is available

Implemented -f and -F in v1.027.  -f is now the default if -M is used or implied.


"ConnectionTerminated error_code:ErrorCodes.PROTOCOL_ERROR" for some sites

Problem Description
When connecting to some sites through mitmproxy, I can't connect, instead getting ConnectionTerminated error_code:ErrorCodes.PROTOCOL_ERROR
Steps to reproduce the behavior:

Run mitmdump with LANG=en_US.UTF-8 mitmdump --listen-host <IP_ADDRESS> -p 2011  -s history.py --set stream_large_bodies="1m" --set stream_websockets=true
history.py:

from mitmproxy import http
from datetime import datetime
from bs4 import BeautifulSoup

def responseheaders(flow):
    if not flow.request.headers['accept'].startswith('text/html,application/xhtml+xml,application/xml'):
        flow.response.stream = True

def response(flow: http.HTTPFlow):
    if flow.request.headers['accept'].startswith('text/html,application/xhtml+xml,application/xml'):
        try:
            title=BeautifulSoup(flow.response.content).title.string
        except:
            return
        file=str(datetime.fromtimestamp(int(flow.response.timestamp_start))).split(' ')[0]
        with open('../logs/ '[:-1]+file,'a+') as f:
            f.write(str(datetime.fromtimestamp(int(flow.response.timestamp_start)))+ ' '+title+' '+str(flow.request.url)+'\n\n')

Try to connect to https://vladtalks.tech/vpn/how-to-turn-off-vpn-on-mac
Fail with HTTP/2 connection closed: <ConnectionTerminated error_code:ErrorCodes.PROTOCOL_ERROR, last_stream_id:0, additional_data:None>

System Information
Paste the output of "mitmproxy --version" here.
Mitmproxy: 7.0.4
Python:    3.9.9
OpenSSL:   OpenSSL 1.1.1l  24 Aug 2021
Platform:  Linux-5.15.5-1-ARCH-aarch64-with-glibc2.32

This is solved with --no-http2

For the record, the problem here is a bug in the HTTP/2 implementation of the website in question (vladtalks.tech).
Upon connect, mitmproxy sends an initial HTTP/2 settings frame with ENABLE_CONNECT_PROTOCOL=0. vladtalks.tech does not understand that and closes the connection with a protocol error. If they would follow RFC 7540, they would ignore the setting instead:

An endpoint that receives a SETTINGS frame with any unknown or unsupported identifier MUST ignore that setting.

As long as this only affects isolated websites, we are not going to work around their implementation bugs.

So what does --no-http2 do?

It disables HTTP/2, which makes mitmproxy fall back to HTTP/1 (where the bug is not present).

Update: it seems that vladtalks.tech fixed its problem, so the flag is no longer needed.


It crashes when you use grinderhead to make iron dust

crash-2024-07-15_15.13.34-client.txt
version：1.20.1 fabric
version：SimpelAdd-fabric-1.20.1-1.109

This is now Fixed in Version 1.110. Thank you For the Bug Report.

ok


Update CMakeLists.txt for MSVC linker errors

Added /FORCE:MULTIPLE for linker errors on Windows

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


"Unhelpful" button

Using the method from #82 to increment a counter, but additionally showing a popup asking the reader how we could have done better and posting what they say to a Discord channel #criticism in Rob's Discord along with a link to the answer they're complaining about via Stampy.

From duplicate #289:

Would allow for better understanding of which answers need extra attention in particular.
On click the button should probably prompt for a comment by the user on how exactly the answer was unhelpful, and then remind them that they can also go into google docs to leave more granular comments about specific parts of the text if they so wish.


[Request] Enable the syntax "NL" in GUI buttons

I'm discovering that making larger buttons where one word appears below another word in GUI buttons is extremely difficult. The biggest suggestion I've gotten is to make new GuiTextCtrl children of the button, but honestly getting it to resize properly is extremely difficult, and you have to add a lot more code simply to get a line break.
The other Torque Syntax options ( @ , TAB, and SPC ) all appear to work when processed in this manner, but the NL only displays a square where the line break should be. I'm hoping this request isn't too hard (Especially since in my web search I discovered it was typically much easier to do a line break in competing engines).

@capnlove is correct. GuiMLTextCtrl is intended to be used for mult-line text. If you need multi-line text in your buttons, simply create a new GuiMLTextCtrl and parent it to the button control.
The reason other syntax options (such as TAB and SPC) work with GuiTextCtrl is because the control's "text" field gets processed by the console system, which manually inserts spaces accordingly.
You could rewrite GuiTextCtrl to support newline commands but there's really no need. I recommend closing this issue.

Recommendation accepted.


FR: Separate store and display formats support.

Hi,
Would be great to have an option to store date in one format (my db format YYYY-MM-DD, mysql) but display the picker to the user in the local format 'M/D/YY' (US). And when he changes the date I want input.val() become YYYY-MM-DD again (so I can save it to the db).
Thanks for the great lib!

:+1:
We were looking to use this as a drop-in polyfill for the native data input type, but that requires ISO 8601 dates. And those look terrible when displaying them to users.

there's an option for extraFormats which allows users to input d/t without being so strict. You could also listen for the change event and format that value to however you need it.

extraFormats is useful if we want to accept an alternate format from the server or user and submit the displayed format (as generated by bootstrap-datetimepicker) back, and it's currently how we are getting around the issue. But it's not actually a solution.
With extraFormats, you can allow for multiple formats to be accepted  by the user (or the default value), but format still determines what is displayed to the user and sent to the server.
I'm looking for a serverFormat (or displayFormat) option that controls how the value is sent to the server, so the user can see their localized times but the server still gets ISO 8601 (or some other format) sent to it. This would also allow the user to modify the value of the displayed date to match what they are looking for, and have the converted format sent to the server.
Right now we would have to do this with a hidden input that is sent to the server, but this breaks our original plan of using this only for when the browser didn't support the native date (Firefox/Edge) and datetime (everyone) input formats.

Yes, exactly, it's just separating localized/displayed (which is different from country to country) date format from internal date format (which is database specific). Of course this can be worked around either by reformatting the date on the client or on the server, but this is very inconvenient.

so again, if you had serverFormat or something similar how is that different from getting the change event or accessing .data(...').date()?

No difference, just shortcut method.

It's important to note that bootstrap-datetimepicker isn't always used in a JS-only environment, where the data is retrieved from the form ahead of time and sent to the server outside of the standard HTML form submission flow.

how is that different from getting the change event or accessing .data(...').date()?

Right now, even when using the change event or .data('').date() to get the current date, it doesn't change the fact that once the form data is submitted to the server it uses the display format.
The difference here is that the server would be receiving a specific format, possibly different from the format displayed to the user. In order to do this right now one would have to hook into the submit event emitted by the form and normalize the date sent to the server before the data was sent.

Yes, that's how I'm doing it now, I hook in on the submit event.

I had already tried with extraFormats, but doesn't work!
Here my Option:
$('.datepicker').datetimepicker({
locale: 'de',
format: 'YYYY-MM-DD LT',
extraFormats:['DD.MM.YYYY LT'],
});
MY HTTP-POST REGUEST
myArray[von]:2015-06-25 23:00 <-- why?
If extraFormats would have worked, would have to request it to be so:
myArray[von]:25.06.2015 23:00
Yeah, well, what's wrong?

@Fruchtgummi extraFormats allows for multiple formats to be valid from the user. It will not change the format you get on post.
@afanasy it wouldn't be a shortcut method, you'd just be looking for a different function name
@kevin-brown I use this picker in my .Net MVC apps all the time.
You'll have to hook into the submit event, set the primary format to the way you need it to be server side, or you'll have to convert it server side to the format you need. You're asking me to force a hook into a form submit event.
I don't know what else to tell you.

There should be another way, other than hooking in the form submit.

I was looking for the same requested feature and found this thread. Here's the use case for us. We store all dates in ISO format. We want to display in a variety of human readable formats which i'm clear the format option allows us to do. However, the datepicker instance doesn't both persistently store the converted ISO value as the actual input value and make both the display and submit value formats available to the DOM for calculations and printing.

I'm facing the same issue. You don't need to listen to the form submit event; see jQueryUI altField and [altFormat](http://api.jqueryui.com/datepicker/#option-altFormat]. Could this be implemented?

Thanks @MatTheCat , altFormat is exactly what is needed!

Just to be clear, altFormat requires altField. The trick is to instantiate the datepicker on a visible field (which name attribute is optional) and to make altField reference a hidden input which will be sent to the server.

Hm, with altField it's still nice, but not as attractive, since you'll need to create additional field to manage this. But it's better than form submit hook probably.

I think it's better too, and the main point is that it can be managed by the datepicker.

Adding to use cases;
I display the dates in localized format, so there are utf8 strings and all. And these values get posted to server when the users submits the form. I manually have to parse these utf8 display strings into dates and believe me, its a real pain.
However, if; "displayFormat" and "valueFormat" parameters were different, i would simply set valueFormat to iso8601 standart and displayFormat to my localized moment strings.
I hope i am clear.
Thanks.

@1dot44mb, the problem is that there is no easy way to implement this, without hooking into form submit. Otherwise the use case seems to be very frequent.

@1dot44mb I don't want to touch your form anymore then I have. If you want to post a different format then the input, grab the .date() from the picker and format it however you'd like with a form submit hook.
It shouldn't be up to this library to do that for you.

You don't need to listen to the form submit event; see jQueryUI altField and altFormat. Could this be implemented?

Once again, but why? Why do I need to duplicate functions that already exist? If you want a different format to a different field, listen to the change event and set the input value to whatever you want.


Remove all derivations of Typeable

Description
Include a description for your changes, including the motivation
behind them.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: It works.

- [ ] I updated the CHANGES.md file
- [ ] I updated the XMonad.Doc.Extending file (if appropriate)

seding is exactly what i did but im not a regex master or sed master so some things turned out fucked up, it didnt work when i put \n in the replace expression.

I will fix these in a minute.

Thank you for the contribution!  I just fixed a few outliers that were still in there, squashed everything down and gave the commit message some more love

No, I thank you.


🐛 Frame data is wrong on special device

What were you trying to do?
i want to use frame processor
Reproduceable Code
const frameProcessor = useFrameProcessor((frame) => {
    "worklet";
   console.log(frame)
  }, []);

What happened instead?

i save frame to file, and it is how it look like
Relevant log output
No response
Device
vsmart joy 4
VisionCamera Version
"react-native-vision-camera": "^2.12.2"
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Incomplete issue. There is literally nothing for me to reproduce, and I have no idea what you're doing.

i save frame to file, and it is how it look like

this does not tell me anything. I cannot reproduce "save frame to file", no code was shared.

fun Image.toBitmap(): Bitmap {
val yBuffer = planes[0].buffer // Y
val vuBuffer = planes[2].buffer // VU
val ySize = yBuffer.remaining()
val vuSize = vuBuffer.remaining()
val nv21 = ByteArray(ySize + vuSize)
yBuffer.get(nv21, 0, ySize)
vuBuffer.get(nv21, ySize, vuSize)
val yuvImage = YuvImage(nv21, ImageFormat.NV21, this.width, this.height, null)
val out = ByteArrayOutputStream()
yuvImage.compressToJpeg(Rect(0, 0, yuvImage.width, yuvImage.height), 100, out)
val imageBytes = out.toByteArray()
return BitmapFactory.decodeByteArray(imageBytes, 0, imageBytes.size)
}

private fun  saveBitmapToFile(bit: Bitmap){
try {
val fileName = Date().time
val fileOutputStream: FileOutputStream =
this.context.openFileOutput(fileName.toString()+".jpg", Context.MODE_PRIVATE)
bit.compress(Bitmap.CompressFormat.JPEG, 100, fileOutputStream)
fileOutputStream.close()
} catch (e: java.lang.Exception) {
e.printStackTrace()
Log.e("error",e.message.toString())
}
}

override fun callback(image: ImageProxy, params: Array): Any? {
Log.e("rotationDegrees", image.imageInfo.rotationDegrees.toString())
try {
@SuppressLint("UnsafeOptInUsageError")
val bit = image.image!!.toBitmap()
saveBitmapToFile(bit)

@mrousavy
i'm sorry for missing information
it only happen on vsmart joy 4

other devices is still working correctly

@mrousavy  could you reopen it?

hm, are we sure that this is a YUV buffer?

Try this one:
private Bitmap toBitmap(Image image) {
    Image.Plane[] planes = image.getPlanes();
    ByteBuffer yBuffer = planes[0].getBuffer();
    ByteBuffer uBuffer = planes[1].getBuffer();
    ByteBuffer vBuffer = planes[2].getBuffer();

    int ySize = yBuffer.remaining();
    int uSize = uBuffer.remaining();
    int vSize = vBuffer.remaining();

    byte[] nv21 = new byte[ySize + uSize + vSize];
    //U and V are swapped
    yBuffer.get(nv21, 0, ySize);
    vBuffer.get(nv21, ySize, vSize);
    uBuffer.get(nv21, ySize + vSize, uSize);

    YuvImage yuvImage = new YuvImage(nv21, ImageFormat.NV21, image.getWidth(), image.getHeight(), null);
    ByteArrayOutputStream out = new ByteArrayOutputStream();
    yuvImage.compressToJpeg(new Rect(0, 0, yuvImage.getWidth(), yuvImage.getHeight()), 75, out);

    byte[] imageBytes = out.toByteArray();
    return BitmapFactory.decodeByteArray(imageBytes, 0, imageBytes.length);
}

i'm sorry for my late response
after few days of trying, i figure out that: i only work when i comment all function imageAnalysisBuilder.setTargetAspectRatio(aspectRatio) (if no format is set)
and imageAnalysisBuilder.setTargetResolution(format.videoSize) (if format is set)
and it only work if i set setTargetResolution(480, 640) at the line https://github.com/mrousavy/react-native-vision-camera/blob/65d4d46f6ab4e04dca15c15b8cf9c07a17f97d72/android/src/main/java/com/mrousavy/camera/CameraView.kt#L396
other value is not work.
Do you have any suggestion to figure out this?

const item = cameraDevice.formats.find((item) => {         return (           item.videoWidth > 600 &&           item.videoWidth < 700 &&           item.videoHeight > 400 &&           item.videoHeight < 500         );       });
      setCameraConfig({         preset: undefined,         format: { ...item, videoWidth: 480, videoHeight: 640 },       });
my workaround is fixed resolution for only this device model.

it only appear on special device. so i close it now

I am getting same issue with Xiaomi mi A2 lite and also was getting same issue with emulator too (but not always)

this issue should be open. If I use workaround suggested above, than I sacrifice other devices, which do not support suggested resolution :|

This is now fixed in V3! 🥳


Card name field missing on confirmation page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation (hyperlinked to channel name):
Action Performed:
Prerequisite:
Members and company cards added in workspace

Go to staging.new.expensify.com
Choose the workspace from settings
Initiate the "assign card" flow
Observe the confirmation page

Expected Result:
Card name field should appear on confirmation page
Actual Result:
Card name field missing on confirmation page
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
![image - 2024-10-31T152811 232](https://github.com/user-attachments/assets/3b06c599-3773-47c6-b103-90ba42e2e628)

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

BZ team member, Callstack will work on this issue, please wait for them to comment before moving the issue forward.

Handling this as part of the project I have worked on

Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue.

Currently the assign card command does jot allow to use custom name, we will need to update it as such

Updates: I've prepared FE Draft PR. I just need to test it over BE updates when they are ready and add screenshots/recordings.

The PR is ready for the review!

@allgandalf It was a part of the design doc that we missed I guess.

Waiting for the checklist, then we can close this issue as we will handle regression testing with @joekaufmanexpensify and payments for the project altogether

@allgandalf it sounds like we should have a regression test for this, can you please add the regression test to the template?
If you disagree, let me know

Oh hmm so we aren't handling payments here either? Ok, no prob if they are being managed under a different project.


Add a buildah-build target.

@dgoodwin with a newer builder like buildah we should be able to go back to a single multi-stage Dockerfile. We can do that later though.
/lgtm


Runtime Exception on multiple file upload

Error
====
Fatal error: Uncaught exception 'RuntimeException' with message 'SplFileInfo::__construct() expects parameter 1 to be string, array given' in D:\xampp\htdocs\test\Uploading\libs\vendor\codeguy\upload\src\Upload\File.php:146 Stack trace: #0 D:\xampp\htdocs\test\Uploading\libs\vendor\codeguy\upload\src\Upload\File.php(146): SplFileInfo->__construct(Array) #1 D:\xampp\htdocs\test\Uploading\upload.php(19): Upload\File->__construct('uploadFile', Object(Upload\Storage\FileSystem)) #2 {main} thrown in D:\xampp\htdocs\test\Uploading\libs\vendor\codeguy\upload\src\Upload\File.php on line 146

Frontend Code
============
<form action="upload.php" method="post" enctype="multipart/form-data" name="form1" id="form1">

multiple Files Upload

Select file
<input name="uploadFile[]" type="file" id="ufile[]" size="50" />

Select file
<input name="uploadFile[]" type="file" id="ufile[]" size="50" />

Select file
<input name="uploadFile[]" type="file" id="ufile[]" size="50" />

<input type="submit" name="Submit" value="Upload" />

</form>

Backend Code
============
$storage = new \Upload\Storage\FileSystem('uploads');
$file = new \Upload\File('uploadFile',$storage);

I recently had the same error. The problem is that when I install the package through composer require codeguy/upload, composer is installing an old version. I checked my composer.json and the version constraint was ^1.3, which appears to not support multiple file uploads. I changed it to dev-master and it works. I'm not quite sure why this is happening, though.

I recently had the same error. The problem is that when I install the package through composer require codeguy/upload, composer is installing an old version. I checked my composer.json and the version constraint was ^1.3, which appears to not support multiple file uploads. I changed it to dev-master and it works. I'm not quite sure why this is happening, though.

I got same issue. Is this package still support?

@khoanguyen2610
In your composer.json
"codeguy/upload": "dev-master"
composer update


Onion messaging + bolt 12 offers

This branch builds on earlier work by @carlaKC. The branch imports the experimental library boltnd and uses it to add support for bolt12 to LND.
Since the original work is over 2 years old now, I've updated both this branch and the experimental library  boltnd to work with the latest versions of lnd, lndclient and lightning-onion. Since route blinding is now supported natively, we don't have to depend on custom forks of those libraries.
CAVEAT
This is by no means production ready.
Also, any changes to the bolt12 spec from the past 2 years have not been implemented. The current branch only "fixes" the original code to work with the current state of lnd and related libraries.
I opened up this draft PR to sollicit early feedback.
Building LND with bolt12 support
This branch introduces the bolt12 build flag, enabling support for bolt12. It introduces two Makefile targets that work with this new flag:

itest-b12
build-itest-b12

Todos

[ ] Update to the latest state of bolt12 spec
[ ] Decide on integrating the boltnd code in lnd or keep it as a external library. Originally, the code is largely implemented in the external boltnd library,  to keep a clean separation of experimental code. But when the experimental label isn't applicable anymore, we need to decide on how to go forward.
[ ] Rate limiting

[ ] Initial token bucket rate limiting
[ ] Back propagation

If this is just a POC to show that we can still use my old code, ignore the below.
If this is actually an approach being considered I have some thoughts.

If the plan it to implement BOLTs externally going forward, I think that the design should be more carefully thought out (eg, importing lndclient is going to be an absolute disaster for dependencies), likely taking an approach similar to lightning-onion.
The ability to run an external daemon with full onion messaging (that has actually been reviewed!!!) and bolt 12 send support already exists in LNDK, so I feel that the most impactful use of time would be to start updating and reviewing the OM code properly (either in a LND pr, or in a lightning-messaging repo).


JDBC IAM Auth to Redshift does not work with the "Profile" connection attribute.

Driver version
<IP_ADDRESS>
Redshift version
PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.26742
Client Operating System
Amazon Linux 2
I am using the AMI - amzn2-ami-hvm-2.0.20210126.0-x86_64-gp2
output of uname -a is:
Linux ip-172-31-90-114.ec2.internal 4.14.214-160.339.amzn2.x86_64 #1 SMP Sun Jan 10 05:53:05 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
JAVA/JVM version
openjdk version "11.0.9" 2020-10-20 LTS
OpenJDK Runtime Environment 18.9 (build 11.0.9+11-LTS)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.9+11-LTS, mixed mode, sharing)
Table schema
Not Applicable to this issue
Problem description
I am trying to use JDBC to connect to a Redshift cluster using IAM Authentication. The code is given below. The code works when the EC2 on which I am running this has an IAM role attached which has permissions for GetClusterCredentials. But when I run the code using the "Profile" JDBC connection attribute after configure an IAM user using the aws configure --profile profilename option providing a static set of access key and secret access id, it does not work. I get a JDBC error given below.

Expected behaviour: I expect the connection to succeed and print the result of the query SELECT count(*) as numcols from information_schema.columns
Actual behaviour: I get an error that says Exception in thread "main" com.amazon.redshift.util.RedshiftException: Something unusual has occurred to cause the driver to fail. Please report this exception.
Error message/stack trace:

Exception in thread "main" com.amazon.redshift.util.RedshiftException: Something unusual has occurred to cause the driver to fail. Please report this exception.
        at com.amazon.redshift.Driver.connect(Driver.java:316)
        at java.sql.DriverManager.getConnection(DriverManager.java:664)
        at java.sql.DriverManager.getConnection(DriverManager.java:208)
        at com.mycompany.app.App.main(App.java:41)
Caused by: com.amazonaws.SdkClientException: Unable to load AWS credentials from any provider in the chain
        at com.amazonaws.auth.AWSCredentialsProviderChain.getCredentials(AWSCredentialsProviderChain.java:131)
        at com.amazon.redshift.core.IamHelper.setIAMCredentials(IamHelper.java:448)
        at com.amazon.redshift.core.IamHelper.setIAMProperties(IamHelper.java:304)
        at com.amazon.redshift.jdbc.RedshiftConnectionImpl.<init>(RedshiftConnectionImpl.java:244)
        at com.amazon.redshift.Driver.makeConnection(Driver.java:466)
        at com.amazon.redshift.Driver.connect(Driver.java:279)
        ... 3 more

Any other details that can be helpful:
I tried the same on an Ubuntu machine too, but that too did not work. I got the same error.

JDBC trace logs
I am not sure how to enable this trace. I tried searching for this but could not find a clear answer. Can someone provide instructions on how to do this?
Reproduction code
package com.mycompany.app;

import java.sql.*;
import java.util.Properties;
import java.util.*;

public class App {

   public static void main(String []args) throws Exception {

        Scanner sc= new Scanner(System.in);

        System.out.print("Enter the dbname: ");
        String db= sc.nextLine();
        System.out.print("Enter the DB user you want to login as: ");
        String user= sc.nextLine();
        System.out.print("Enter the user profile you want to use [Hit enter for default]: ");
        String profile= sc.nextLine();
        System.out.print("Enter the value of AutoCreate parameter: ");
        String autocreate= sc.nextLine();
        if (profile.length() == 0) {
            profile = "default";
        }

        Class.forName("com.amazon.redshift.jdbc42.Driver");

        System.out.println("Connecting with profile: "+profile);

        //Please change the cluster endpoint here
        String url = "jdbc:redshift:iam://redshift-cluster-1.abcdefghijklm123.us-east-1.redshift.amazonaws.com:5439/"+db;
        Properties properties = new Properties();
        properties.setProperty("DbUser", user);
        properties.setProperty("AutoCreate", autocreate);
        properties.setProperty("Profile", profile);
        properties.setProperty("Trace", "on");
        Connection connection = DriverManager.getConnection(url, properties);

        System.out.println("Using IAM");
        System.out.println("Creating statement...");
        Statement stmt = connection.createStatement();
        String sql;

        //1st query
        sql = "SELECT count(*) as numcols from information_schema.columns";
        ResultSet rs = stmt.executeQuery(sql);

        //STEP 5: Extract data from result set
        while(rs.next()){
         //Retrieve by column name
         int numcols  = rs.getInt("numcols");
         //Display values
         System.out.println("Count: " + numcols);
        }
   }
}

In the URL put: "LogLevel=6;Logpath=/some_exist_dir
LogPath must be exist. Then run your test and it should create redshift*.log in that directory.
In your case URL become:
String url = "jdbc:redshift:iam://redshift-cluster-1.abcdefghijklm123.us-east-1.redshift.amazonaws.com:5439/"+db + ";LogLevel=6;Logpath=/some_exist_dir";

Also can you try lower case profile as prop name?
properties.setProperty("profile", profile);

Thanks @iggarish. Changing the case of the Profile connection attribute solved the issue. And also thank you for showing how to enable trace log. Should this issue still be marked as a bug since the documentation uses upper case P for profile? I am ok with closing this if you think this is not a bug, please let me know.

Keep the issue open. I will check case sensitivity of prop name in the code and see it's ok to change code otherwise we will change the doc.

Change the code to support "profile" as well "Profile" as key in connection property. Fix will available in next release i.e. <IP_ADDRESS>.


build: add code coverage in GitHub action

Closing this PR due to no-activity. Please re-open when this is ready for review.


Add login link template override for unverified & verified verbiage

Add the ability to override the verbiage that appears above the login link on a per app or appContext basis.  This will, have the ability to support simple html element like <b>, <strong>, <em>, <i>, <u>
For now the ability to subject line is there, but hidden behind a display: none class on the form row above the unverifed field

unverified

verified

 Can we have the wording changed on the login email?


[NotMapped] property adding to script unexpectedly

should not see [NotMapped] property in script

Pretty sure this would be fixed by
https://github.com/adamfoneil/Models/issues/5
but I don't have bandwidth for this -- just manually exclude it for now


[Extensibility Request] report 99001017 "Calculate Plan - Plan. Wksh." - InitializeRequest

Most of our customers are running this plan during the night hours using the job queue and task scheduler. We are not able to set all the parameters available in the request page. Could you provide below mentioned new procedure to accommodate that:
procedure InitializeRequest(NewMPS: Boolean; NewMRP: Boolean; NewUseForecast: Code[10]; NewExcludeForecastBefore: Date; NewNoPlanningResiliency: Boolean)
begin
    //A108F0061.Event20.00.ns
    MPS := NewMPS;
    MRP := NewMRP;
    UseForecast := NewUseForecast;
    ExcludeForecastBefore := NewExcludeForecastBefore;
    NoPlanningResiliency := NewNoPlanningResiliency;
    //A108F0061.Event20.00.ne
end;

Suggested procedure does not have parameters for FromDate, ToDate and RespectPlanningParm. Should we add all parameters to new procedure?

That is also fine by me, but would that not result into a 'breaking' change? That is why I added a new procedure with the same name;
procedure InitializeRequest(NewFromDate: Date; NewToDate: Date; NewRespectPlanningParm: Boolean)
does not work because MPS and MRP are from a manufacturing setup instead of another setup we have per day in our addon.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Level 200: Automated Deployment of VPC

Stack Creation Failed using file :https://www.wellarchitectedlabs.com/Common/Create_VPC_Stack/Code/vpc-alb-app-db.yaml
Below is the error:
Cannot render the template because of an error.: YAMLException: unknown tag !<!Cidr> at line 238, column 115: ... ubnets, !Ref VPCSubnetIPv4Size ]] ^

Thank you @AKANKSHAKSB for your feedback.
I am unable to reproduce this issue.  Can you please provide more details about how you attempted to create the stack?  Was this in the AWS Console?  At what point did it give you this error?
Can you please attach a copy of the YAML template you used? (so we can check if it somehow has become corrupted)?

Ah, I figured it out!
After loading the template, please be sure to click on NEXT and do NOT click on View in Designer

Exactly it is the issue with View in Designer.
Usually templates get rendered in the designer without any error but sometimes they don't like this one.But anyways thanks,if we skip view in designer and directly got to Next ,stack is created successfully.


added login routes, wrote login handlebars, edited user model to enco…

…de the password

Home
Login
Edit
Profile


Can't get $API in vg-player-ready="onPlayerReady($API)"

Hi. I have an issue in videogular and i don't know is it my fault or internal issue.
Heres some code

.... Rest of video container
ngApp.controller('VideoController', function($scope,  $sce) {
angular.extend(this, $controller('BaseController', {$scope: $scope}));
$scope.onPlayerReady = function(API){
console.log(API);
this.API = API;
};
});
The API argument passed to function is always undefined. Rest of argument also.
Do you have any idea how to fix that?
Thanks.

Which version of Videogular are you using? It should be 1.0.0.
Also, it would be great if you can post an example at codepen or jsfiddle.

Closing issue after 2 weeks without info.


Vertical Nav icon size dont change

I'm trying to change the size of clr-icon inside Vertical Nav using size attribute.
Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
Be able to change the icon size by the size attribute.
Actual behavior
Size attribute don't change de size of clr-icon.
Reproduction of behavior
https://plnkr.co/edit/HKIn8hiNBf9WB3egr7Qt?p=preview
Environment details

Angular version: 4.3.6

Clarity version: 0.10.7

OS and version: Ubuntu 16.04

Browser: [ tested on Chrome and Firefox ]

@williangd: I would recommend that you create a separate class and apply the styles to that. Please refer to this plnkr: https://plnkr.co/edit/qBTft97DXV8G8bYYBWww?p=preview (See: app.component.css). Your styles aren't being applied because the flex property is not being overridden.
Please note that we don't recommend that you do this because as the size of the icons in the vertical nav is configured to work best in both expanded and collapsed state. If you do override the styles you would need to make sure that things are aligned properly in all states.
Closing the issue based on this. Feel free to reopen the issue if you have more questions regarding this.


Refacto prerequisites

The files debian.yml and centos.yml in the task directory were merged into os_prerequisites.yml file to prevent repetitive code
The task that install unzip is modified to be installed whatever the os by using the following module :
- name: Install unzip   package:     name: unzip     state: present
Since the name of the java package is different on debian os and on centos we'll install it accordingly on the two systems by using the condition when

Done

Conflict handled


Immediate error about conditional binary operator expected

https://gist.github.com/lholmquist/ac30beacc8e25da9b21edd5b767811a9
For me, it happened on master,   ran make test and immediately got this

I really don't remember if my test was yesterday or today.
https://github.com/bucharest-gold/entente/issues/126#issuecomment-278314323
/me assuming yesterday but with your feedback, now I have no idea :D

closing,  I didn't have the latest master commit


Authentication fails

I tried a plain copy from your template with my own .env credentials. I made them as follows (fake handle & app-password of course
BSKY_PASSWORD=here-goes-app-pw
BSKY_HANDLE=myFabHandle
But I get an error when I build and run
XRPCError: Invalid identifier or password
at ServiceClient.call (/node_modules/@atproto/api/dist/index.js:7099:15)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async BskyAgent.login (/node_modules/@atproto/api/dist/index.js:14759:19)
at async Bot.run (file:///dist/lib/bot.js:35:9)
at async file:///dist/index.js:3:14 {
status: 401,
error: 'AuthenticationRequired',
success: false
I changed the handle and password to a string, added them directly in config.ts line 14 and 15 but no luck. I tried the actual password but still no change in error. Might there be a change with the @atproto API? Anything else I can test?

Hi Frank,
I received the same error but solved it by deleting the @ from the handle in .env, online username.bsky.social
Maybe this helps.

Thanks @olfrygt your solution work=s perfectly :)


APP: multiplayer pet duplicates

not sure on exact repro, this happened when I joined a parcel that already had someone in it with a pet following them.

Probably the pet is not being detected as a dupe when sending status.

Probably the pet is not being detected as a dupe when sending status.

Fixed.

Fixed.


Include links of imprints into link replacement

Short description

This PR adds ImprintPageTranslation into the target models of replace_links so links in imprints can be replaced too by the "Search&Replace" on the uppr right corner of the broken link list.
Proposed changes

Add ImprintPageTranslation into the list models in the function replace_links

Side effects

Should we probaly think of centralised check for such process of adding/removing a model to/from our link features? Somthing like, listing models in one single list (as an environment variable, for exmaple) and we add a check whether all models of the list are included at the points where you need to add changes while removing/adding a model to the link check feature.

Resolved issues

Fixes: #2982

Pull Request Review Guidelines

@david-venhoff Thank you for review and idea 😸 I'd suggest to do it in another PR to stay in the policy "one PR, one purpose".


Line Graph with DateTime Crashing

Hi,
I am trying to make a Line Graph using DateTime values as the X-Axis. Unfortunately this is making NeoDash crash with the error:

Uncaught TypeError: Cannot read properties of null (reading 'getTime')

As an easy way of testing, I tried also making a 1-point LineGraph with the following code:
return datetime("2019-06-01T18:40:32.142+0100") as Date, 1 as Value
Unfortunately this results in the same error.
I would be very grateful for any hints on what I might be doing wrong.
Cheers,
Miguel

Hi @mbernardes!
Thanks for sharing this issue. In fact, there was a regression introduced recently not caught on our tests.
This issue will be closed with https://github.com/neo4j-labs/neodash/pull/245 on 2.1.10 that's going to be release later today.
Thanks, Miguel
Cheers,
Harold


Goddamn updates breaking shit....

Need to add .layer { background: transparent !important; } as the .layer update finally made it to the stable version.

where do i add it?
thanks

nevermind, just paste it into the "custom css" tab and it's done!
hope this helps :)

Fixed


More array ports

More changes to array ports schema: Float and Int types.

@barbieri suggested it and @cmarcelo also mentioned the 32 array ports for operations like addition, multiplication, etc. I sent an email to the list about this saying that I would proceed with the changes if no one else had a different opinion. But, yes, I agree the 32 is arbitrary, but If I'm a Soletta user I'd prefer the 32 ports instead of having to cascade several nodes to have a simple arithmetic operation done. Besides, we can get back here in the future and change it again when dynamic arrays is supported.

It was discussed in the devapp mailing list, which brings up the

It was brought up in the devapp-list, which brings up the soletta-dev that we should be using for these things (unless it's something that can't be discussed in public).
32 is arbitrary, but seems large enough to be sufficient for now. About dynamic array ports, are people seeing a need to have that in the near future? It would require some major reworking of the way we do things right now.

@lgywata I see that you updated the tests, remember to also check the samples affected by these changes.

@anselmolsm, you're right. Thanks!

v2 sent with samples included. #220


⚠ ClamAV has degraded performance

In 47fcc07, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 431 ms

Resolved: ClamAV performance has improved in 949c10b.


Download quality not matching hi-res preview

Prerequisites

[ ] Did you read FAQ section? Yes
[ ] Did you test with the latest releases or commit ? Yes

Description
Not sure what to do exactly, because this issue is present in both manual and script-assisted downloads. Basically, the downloaded image does not match the clarity of the hi-res preview (when you click on the image to maximize on pixiv.net itself). https://www.pixiv.net/en/artworks/72829758 <-- this is a good example. When you maximize the image, it shows a very crisp quality, but when you actually download the image it's quite pixelated.
Steps to Reproduce

First, view preview quality by maximizing image on pixiv's website.
Next, download image either manually or with PixivUtil2.
Compare the two (the preview is much nicer).

Expected behavior: Downloaded image would match the preview quality
Actual behavior:  Download quality was noticeably worse
Versions
20200125

get the latest version, also compare the file size and urls is the same.

download not match with image on pixiv look here
https://imgur.com/a/mEHPsft

Perhaps this is a better indicator of what I mean:
Look at the quality of this screenshot. The lines are crisp and clear, well-defined, etc. This is with the pixiv image expanded and zoomed in on (for some reason the enhanced quality only shows up then).

Now, look at the file that actually gets downloaded. It's easy to see the discrepancy.

can you tell the image url then? because it doesn't do any conversion or modification.
do you set downloadResized = True in the config.ini?

Look at the quality of this screenshot. The lines are crisp and clear, well-defined, etc. This is with the pixiv image expanded and zoomed in on (for some reason the enhanced quality only shows up then).

By default, Chrome applies color management based on your system ICC profile, which could cause a difference in appearance. If you don't have a standard gamut monitor calibrated to sRGB, something like that could happen.

Now, look at the file that actually gets downloaded.

There is no problem with this image, it matches the file size & file hash of the original image. If you view it color managed in the same way, it should look identical.

Ah, thank you, that makes sense Cyberbeing. I didn't know that was a thing.


[Fix-7753][Master] Failed to pass global parameters between parent and sub process

Purpose of the pull request
close #7753

Codecov Report

Merging #7755 (e68c10b) into dev (d7d13f7) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #7755      +/-   ##
============================================
- Coverage     40.89%   40.85%   -0.04%
+ Complexity     3668     3664       -4
============================================
  Files           642      642
  Lines         26706    26707       +1
  Branches       2997     2997
============================================
- Hits          10922    10912      -10
- Misses        14736    14743       +7
- Partials       1048     1052       +4

Impacted Files
Coverage Δ

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

...olphinscheduler/rpc/remote/NettyClientHandler.java
41.86% <0.00%> (-9.31%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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


Featurerequest: Open door command

Hello, I use the nuki hub to integrate a NUKI Smart Lock 3.0 in homeassistant. First of all, thank you for the development of the bridge. The installation worked without any problems.
To be able to use the nuki hub fully, I would be happy if it could be extended with the possibility to open the door.
Greetings
Andreas

It's already there, I refer you to the thread in the home assistant forum:
https://community.home-assistant.io/t/nuki-hub-firmware-for-the-esp32/417572/137?u=technyon

Oh yeah, thank you very much. The lock.open service works!


Add wildcard breakpoint setting that triggers ALL exceptions/errors

Currently, there is no way to trigger a breakpoint on custom php Exceptions. You can't even trigger on predefined Exceptions such as ErrorException.
I think it would be a nice feature to allow us to add custom Exception Breakpoints in the settings. That way we could watch for certain custom Exceptions.
In the meanwhile, I added a wildcard * exception breakpoint that people can use.
Here is it's dbgp output: breakpoint_set -i 18 -t "exception" -x "*".
Here is what it looks like:

LGTM. Thanks.


[8.x] Fix batch add method doc block

Add mixed parameter type to the batch add method for Batchable jobs, allowing singlular jobs to be passed. This already works (because the method wraps the given jobs in a Collection) but this change fixes warnings in IDEs and static analyzers
Before:
    public function handle()
    {
        $this->batch()->add([
            new ProcessPodcast()
        ]);
    }

After:
    public function handle()
    {
        $this->batch()->add(new ProcessPodcast());
    }

I believe we intended that an array or collection should always be passed.

I suggest just placing the single job in an array.


docker workflow: update target

Signed-off-by: Maryam Tahhan<EMAIL_ADDRESS>

depends on https://github.com/CloudNativeDataPlane/cndp/pull/265

Let's do this in a staged manner... I will restore the old docker build target in the other pull request and when that passes we can update the target

after we update the target we can push another pull request then that removes the old target from the CNDP work base...


[🐛 BUG?]: MaxConcurrentActivityExecutionSize ignored.

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
MaxConcurrentActivityExecutionSize seems to be ignored, it's supposed to decrease the https://docs.temporal.io/docs/operation/how-to-tune-workers/#metrics task_slots_available metric, but it doesn't. On go SDK it's working.
In both go workers and PHP workers the default is 1000, but if I set it to 1 on PHP the metric stays 1000, when I set it to 1 on pure GO worker, it goes to 1 as it should.
I'm sure from the logs, RR is getting the right value.
Any ideas guys? Maybe I'm setting up something wrong? :thinking:
Version
2.8.4
Relevant log output
2022-03-22T19:12:33.968Z	DEBUG	temporal    	received message	{"data": "\n\ufffd&*\ufffd&\n\ufffd\r\n\u0016\n\u0008encoding\u0012\njson/plain\u0012\ufffd\u000c{\"TaskQueue\":\"test_queue\",\"Options\":{\"MaxConcurrentActivityExecutionSize\":1,\
"WorkerActivitiesPerSecond\":0.0,\"MaxConcurrentLocalActivityExecutionSize\":0,\"WorkerLocalActivitiesPerSecond\":0.0,\"TaskQueueActivitiesPerSecond\":0.0,\"MaxConcurrentActivityTaskPollers\":0,\"MaxConcurrentWorkflowTaskExecutionSize\":0,\"MaxConcurrentWorkflowTaskPollers\":0,\"StickyScheduleToStartTimeout\":null,\"WorkerStopTimeout\":null,\"EnableSessionWorker\":false,\"SessionResourceID\":null,\"MaxConcurrentSessionExecutionSize\":1000},\"Workflows\":[....]
{\"TaskQueue\":\"test_conversion\",\"Options\":{\"MaxConcurrentActivityExecutionSize\":1,\
"WorkerActivitiesPerSecond\":0.0,\"MaxConcurrentLocalActivityExecutionSize\":0,\"WorkerLocalActivitiesPerSecond\":0.0,\"TaskQueueActivitiesPerSecond\":0.0,\"MaxConcurrentActivityTaskPollers\":0,\"MaxConcurrentWorkflowTaskExecutionSize\":0,\"MaxConcurrentWorkflowTaskPollers\":0,\"StickyScheduleToStartTimeout\":null,\"WorkerStopTimeout\":null,\"EnableSessionWorker\":false,\"SessionResourceID\":null,\"MaxConcurrentSessionExecutionSize\":1000},\"Workflows\":[{\"Name\":\"ConverterWorkflow\",\"Queries\":[],\"Signals\":[]}],\"Activities\":[....]

Hey @Zylius 👋🏻 . I'll verify and fix it in the next release.

Hey @Zylius. I tried to verify that, but as far as I see, I definitely can manipulate this argument:

PHP side:

As you may see on the first screenshot, the ActivityExecutionSize is 10 as I set and WorkflowTaskExecutionSize because 1 WF was in progress to grab the metric.

It would be best to wait for a little because this is not an immediate metric.

Hmm I'll try to do it just like you did, maybe my code was wrong :thinking:

I can confirm this was an issue on our end :( terribly sorry, closing the issue.


[TTL] - Auto refresh expiring time when read operation is performed.

The current behavior of the cache will not extend expire time of the result when a read operation is performed for a given resource.
Current behavior

Do a query
Cache is created at 60s and starts counting down
After 20s Do the same query again
Cached data is reused but the expiry time was not extended - so it's 40s  now.

TODO:
As an option for a Query and Aggregation, would be good to create a param that can be passed next to the TTL value, to
optionally change the behavior above to work like

Do a query
Cache is created at 60s and starts counting down
After 20s Do the same query again
Cached data is reused and expiry tam was refreshed - so it's 60s  now.

So for example it can be called like Model.find({}).cacheQuery({ttl: 60, refreshExpiryTimeOnRead: true })
Of course, param name might be different :)
Acceptance criteria:

Should work for both strategies
Should be optional
Should be unit tested

Hi @arqo123, great project! I'd like to try to add this enhancement

Hello @elonyavist! Sure, feel free to ask me anything below in the ticket. I would try to respond ASAP :)

Perfect!! I'm working on it :)

@elonyavist  i've updated repo with husky, prettier and more linting rules. Please merge changes or rebase your code.

@arqo123 looking at the documentation (redisClient and keyv) seems doesn't exits a proper method to refresh the TTL. I think, if the use use the new parameter, we have to set again the value with the same key and the TTL

@elonyavist  In the case of keyv - yes. Inside it's just a normal JS map.
But in the case of redis please just call expire() method on the client.
Please check how it's done here https://github.com/arqo123/speedgoose/blob/master/src/cachingStrategies/redisStrategy.ts#L30

@arqo123  I'm trying to declare this method:
public abstract refreshTtlForCachedResult(namespace: string, key: string, ttl:number, value:CachedResult): Promise<void>;
into the CommonCacheStrategyAbstract and into the executeQueryWithCache add something like that:
 if(context.shouldRefreshTtlOnRead){ await refreshTtlForCachedResult(context.cacheKey, context.ttl, cachedValue)}
what do you think?
Otherwise into the getResultsFromCache we have to pass ttl and the new shouldRefreshTtlOnRead params

@elonyavist

@arqo123 I'm trying to declare this method: public abstract refreshTtlForCachedResult(namespace: string, key: string, ttl:number, value:CachedResult): Promise<void>; into the CommonCacheStrategyAbstract and into the executeQueryWithCache add something like that:  if(context.shouldRefreshTtlOnRead){ await refreshTtlForCachedResult(context.cacheKey, context.ttl, cachedValue)}

Yes, that's correct :)

perfect! Thx


OrientationBuilder inside SingleChildScrollView always returns Orientation.portrait.

Steps to Reproduce
The title is self-explanatory.

Here is a link to a gist that reproduce the issue.
Expected results:

Actual results:

flutter doctor -v
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.22000.434], locale en-US)
    • Flutter version 2.8.1 at C:\Users\Brown Technology\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (5 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\Sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Users\Brown Technology\AppData\Local\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.8)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.32002.261
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.63.2)
    • VS Code at C:\Users\Brown Technology\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.32.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.434]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 97.0.4692.71
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 97.0.1072.55

• No issues found!

Hi @psyonixFx, I believe this is the expected behavior.
The OrientationBuilder docs state
Builds a widget tree that can depend on the parent widget's orientation (distinct from the device orientation).

Note that it doesn't return the orientation of the device.
I believe why it is that way with SingleChildScrollView is because the default scrollDirection is Axis.vertical. If you change the scrollDirection to Axis.horizontal, it will return InnerOrientation will be Orientation.landscape.
If you want the actual device orientation, you may use https://api.flutter.dev/flutter/widgets/MediaQueryData/orientation.html

My aim wasn't the device DeviceOrientation like in package:flutter/services.dart, all i wanted is  a shortcut for ( is width is bigger than hight etc... ) for my flutter web project.
Didn't know about the MediaQuery.of(context).orientation but it seems like the key for my problem. Thanks @danagbemava-nc .

Given your last comment, I feel it is safe to close this issue.
Closing as solved.
Thank you


Change installation instructions in readme

The installation instructions in the readme are still about dep. The project has moved to go modules. I need to change it.

Fixed in b355e5a2c90466c2c6111d49d5eb1fbbb3a6a683


Created the Backend Flask API

PR to initiailize the backend for the playgorund

Thanks for the PR, nice start!


Added yt support

The yt library uses Matplotlib to render one or more plots per object to PNG.  Although yt objects have a _repr_html_ and can thus be displayed without any special support, the enclosing Bokeh Div does not resize appropriately to the size of the contents, which is very inconvenient.  This PR adds a pane.YT class that is the same as pane.HTML except that it calculates the total height (of all stacked PNGs) and the maximum width (of all stacked PNGs) and uses those to size the Div.  This implementation is somewhat inefficient, in that it renders the images to PNG just to get their sizes, but doing so seemed simpler than trying to parse them out of the HTML representation to find out their sizes.  Example:
try:
    import yt
except:
    !conda install -c conda-forge yt

import os
file = "enzo_tiny_cosmology"
if not os.path.exists(file):
    !curl -sSO http://yt-project.org/data/$file.tar.gz
    !tar xzf $file.tar.gz

import yt

import panel as pp
from panel import pane
pp.extension()

ds = yt.load("enzo_tiny_cosmology/DD0046/DD0046")
p = yt.ProjectionPlot(ds, "z", ["density", "temperature"], weight_field="density")

pp.Row(p)

I don't know how to get a tiny YT data file in order to add more official tests than this.

Given that a yt object can return multiple plots and that stacking them as above could be confusing, one could consider returning a pp.Column object instead of a single pane, which would be trivial to implement given the code in this PR that already effectively generates a list of PNGs.  But that would be a new behavior for Pane, so I'll leave such an implementation up to @philippjfr if he prefers it.  I think yt users can simply define only a single plot per yt object if they want to use the current version in a more reasonable way.

Codecov Report

Merging #28 into master will decrease coverage by 0.43%.
The diff coverage is 29.41%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage   92.75%   92.32%   -0.44%
==========================================
  Files          21       21
  Lines        2457     2474      +17
==========================================
+ Hits         2279     2284       +5
- Misses        178      190      +12

Impacted Files
Coverage Δ

panel/pane.py
81.81% <29.41%> (-3.07%)
:arrow_down:

Continue to review full report at Codecov.

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


[KYUUBI #4247][FOLLOWUP] Fix build/mvn version comparison

Why are the changes needed?

build/mvn should download the maven when the local mvn version does not match, this corrects the change in #4261
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

cc @edddddy FYI

Thanks, merged to master


Feature requests to gord.

What do you want
Image support
Voice support
Slowmode countdown
A way to turn off tags completely (or once) on replies
Theme support (half done)
Mod support
If not voice support ability to see voice channels and who are in them
AUR and GURU

Why
More usable
More accessible

Implementation hints
you can prolly do image support thru viu

Ok hot thx.
 Also you can adjust a gifs framerate by using -f (--frame-rate).

big chungus nutsack among sus

big chungus nutsack among sus

dank can you not pls

You can do scaling through viu, with -h (--height) and -w (--width) flags. Also you can adjust a gifs framerate by using -f (--frame-rate).

yep looks easy enough from the help menu so just keeping aspect ratio will be the fun bit

You just need to basically;
Get images size (width and height)
Pass them to viu using the flags i mentioned
then display it at position that the message is at

oh shit actually u know what. i can set a limit for height and width, then whichever goes more over the limit just pass that, you can omit one

Yep.
also a new idea/fix: make it so it would not allow you to try sending messages on locked channels

Also stage channels?

Those are technically voice channels ¯_(ツ)_/¯

yeah will display as part of voice channels and display an icon or something

:+1:

@cainy-a i already made a gord-bin package on the AUR anyway, but a makefile would be nice for compiling from source

ok cool aur package thx.
Also im gonna be using viu for images, which picks the best method for your terminal automatically :+1:

Is voice support still in the pipeline?
Also, thank you for making an arm64 build

Voice support, if it gets implemented, is a way off to be honest. It will involve implementing audio playback and recording on all platforms, and then somehow hooking that up to WebRTC and all while hoping Discord doesn't ban us for getting it a bit wrong.

Would it be possible to add a setting for leaving emotes in their colon form? like :emotename: instead of the actual emote. Use case here is in terminals that don't support full unicode. (eg the linux console, the tty version, can only do 512 glyphs as far as I can tell)
At the moment on said terminals emotes turn into just sort of a unicode mess and not only are not the emote, but are also not distinguishable from other emotes.
Unfortunately I don't know any go so I can't really add to this project.
Using the AUR gord-git.
This project is great. Thank you!

Yeah this  sounds like a good idea

Another suggestion: can we get something like the default ctrl-k channel searcher? I use it a lot and it would be great to have. Might be a pain to implement, though, so I understand if it is a low priority.

Sounds like a good idea


App can not run unless you update your google play services? Ask a Question

When I try to sign in using firebase and run app on emulator then I notify me to update you google play services.Now what should I do to fix this?

@Naveed-anwar123 emulator images come with a version of Google Play services baked in, but most emulators cannot auto update.  You need to download the latest emulator image and then re-create your emulator.
There are also now some emulators that support the full Play Store.  On those images the update prompt will actually work.


Optionally separate Tasks` stderr from stdout.

Optionally separate Tasks` stderr from stdout.
Fixes a serious problem where spurious output from xcrun breaks swift's discovery of libarclite.
Also applied small amounts of clang-format.
rdar://problem/28573949

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test


GEODE-4622 Update library dependencies

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[X] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[X] Is your initial contribution a single, squashed commit?

[X] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[X] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Testing notes:

log4j 2.10.0 seems to break servlet session replication tests.
fast-classpath-scanner gives different results than previously.
shiro-core now depends on other shiro libraries (all ASL 2.0).
swagger adds these libs:

byte-buddy (ASL)
javassist (MPL / LGPL / ASL)
reflections (WTFPL // approved by ASF)

clean precheckin

To be clear, you're not intending to update log4j and fast-classpath-scanner now?

Correct.  log4j2 2.10.0 has a problem with Tomcat which shows up in our session replication tests.
The new version of fast-classpath-scanner has different behavior which means we'll need to change our usage if we want to update.

I'm actually about to update fast-classpath-scanner for another issue.


Add support mini-media-player card and tracker card

Please add support mini media player card
https://github.com/kalkih/mini-media-player
And also
https://github.com/custom-cards/tracker-card

Custom UI components can't e supported for now

Other Android app will also not supported custom UI components


Failing metrics tests in test_namespace_cache

Environment info

NooBaa Version: master branch (commit b5a1664c52)
Platform: Kubernetes 1.14.1 | minikube 1.1.1 | OpenShift 4.1 | other: specify

Actual behavior

When running unit tests in test_namespace_cache tests, some metrics tests are failing

Expected behavior

These tests should pass

Steps to reproduce

Uncomment the lines starting with validate_metric in test_namespace_cache.js
Run unit tests within that class

More information - Screenshots / Logs / Other output

Thanks @mcrepeau
Just adding a reference here to your PR that this was created from - #6181


Open API: Changed RoomKeys to RoomKey

Changed:

putRoomKeysBySessionId to putRoomKeyBySessionId
getRoomKeysBySessionId to getRoomKeyBySessionId
deleteRoomKeysBySessionId to deleteRoomKeysBySessionId

in matrix-doc/data/api/client-server/key_backup.yaml.
Fixes: #3486
Signed-off-by: ankur12-1610<EMAIL_ADDRESS>
Preview: https://pr3500--matrix-org-previews.netlify.app

@Dominaezzz kindly verify the changes :)

If I knew issues I created would be attended to so quickly I'd create a whole lot more haha.

If I knew issues I created would be attended to so quickly I'd create a whole lot more haha.

:smile: thanks!

Thanks for your contribution. Also, congratulations on getting number 3500.

Thanks, I guess I was lucky 😅


Freezing of desktop while using VS Code for even 5 minute

VSCode Version:1.46
OS Version: Pop!_OS 20.04 LTS x86_64

Steps to Reproduce:
1.Force Shutdown

Does this issue occur when all extensions are disabled?: Yes/No
Yes

/needsMoreInfo
Please use this guide to provide more information: https://github.com/Microsoft/vscode/wiki/Performance-Issues. Thanks.


SharePoint configuration page, invalid lookbook

It appears that the lookbook link on the following page is no longer valid:
https://github.com/OfficeDev/microsoft-teams-apps-newemployeeonboarding/wiki/SharePoint-configuration
Clicking https://lookbook.microsoft.com/details/75e60a32-9849-4ed4-b83e-b2b08983ad19 redirects you to https://lookbook.microsoft.com/.
It appears that https://lookbook.microsoft.com/details/388a159f-475b-4557-9088-c9073a26c576 may be the new version.

Do we know if this is the right Lookbook link?

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!


[Draft] Filter out reports based on the active workspace

Filter out reports based on the active workspace with the following criteria. Display only the filtered reports in LHN.

Reports that belong to the active workspace
DMs and GroupDMs from members on the active workspace (there is a caveat on this requirements. See this part and this part of the design doc for detailed requirements)

The list of reports that are displayed on the current LHN (we're changing the design of LHN) are generated in this part of the code. Filter out reports based on the active workspace.
When the active workspace is Expensify/All, all reports across workspaces should be displayed.
Relevant design doc section

Yes because this issue is related to Wave 8 Ideal Nav, which is still in detailed design doc phase. We'll probably delegate this task to an expert agency. When it's time to pay for C+ review, I gonna assign a Bug Zero team back to this issue 🙇

Weekly Update: No update yet.

Weekly Update: Thank you @hayata-suenaga!

I'm going to work on this task

Because of the requirement change (instead of storing the active workspace ID, we decided to store the URL of the last visited path), we need some adjustment in term of implementation for this issue.
@WojtekBoman, are you working on this right now?

The new backend API has been deployed to the staging environment. @WojtekBoman can use the new API to test their PR.
I also commented about the new API in the PR.

This is completed and merged to the feature branch, great work!


Add a test to verify the violation badge

Ref: https://issues.redhat.com/browse/ACM-15541

/cc @mprahl @dhaiducek @JustinKuli


Added a getUniqueId helper

This takes some prefix and some JSON and makes sure the id will be unique if it is added to the JSON.
See https://github.com/AnalyticalGraphicsInc/gltf-pipeline/pull/113

Coverage increased (+0.04%) to 93.752% when pulling ea035985775387e59c367a7b4155bed4dbd90adb on getUniqueId into 0edf83efd1752a2d547b4e5b668daeb027e1fa17 on master.

This doesn't need to search the entire glTF JSON.  It only needs to search top-level objects with ids.
For example, see https://github.com/KhronosGroup/glTF/blob/master/specification/README.md#concepts

IDs for top-level glTF dictionary objects (accessors, animations, buffers, bufferViews, cameras, images, materials, meshes, nodes, programs, samplers, scenes, shaders, skins, techniques, and textures) are in the same namespace and are unique.

Coverage increased (+0.05%) to 93.754% when pulling 20c552a2793814afea498f01de09678164618ba4 on getUniqueId into 0edf83efd1752a2d547b4e5b668daeb027e1fa17 on master.

Updated.

Coverage increased (+0.08%) to 93.783% when pulling 62b96d2c750648639734164a8568aa77b8f44345 on getUniqueId into 0edf83efd1752a2d547b4e5b668daeb027e1fa17 on master.

Updated.

Coverage increased (+0.08%) to 93.783% when pulling afd97ca4eae3fc1a58235d362d6efafd14c27117 on getUniqueId into 0edf83efd1752a2d547b4e5b668daeb027e1fa17 on master.

Updated. Thanks @lasalvavida!

Coverage decreased (-0.03%) to 93.68% when pulling cd3c284c4d2b1bbe0c54fe3e7843247dc6fc032e on getUniqueId into 0edf83efd1752a2d547b4e5b668daeb027e1fa17 on master.

This looks great, thanks @likangning93!


Reduce the default size

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #801 into stable will decrease coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           stable     #801      +/-   ##
==========================================
- Coverage    55.5%   55.39%   -0.11%
==========================================
  Files         232      232
  Lines       23625    23595      -30
==========================================
- Hits        13112    13071      -41
- Misses       9313     9323      +10
- Partials     1200     1201       +1

Impacted Files
Coverage Δ

pkg/network/connection.go
29.11% <100%> (-1.29%)
:arrow_down:

pkg/stream/sofarpc/stream.go
43.62% <100%> (ø)
:arrow_up:

pkg/log/logger.go
55% <100%> (-1.67%)
:arrow_down:

pkg/network/filtermanager.go
16.94% <100%> (ø)
:arrow_up:

pkg/config/configmanager.go
42.1% <0%> (-9.41%)
:arrow_down:

pkg/config/dumper.go
33.33% <0%> (-8.78%)
:arrow_down:

pkg/router/base_rule.go
70.96% <0%> (-5.08%)
:arrow_down:

pkg/api/v2/types.go
71.42% <0%> (ø)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


HighVoltage::PageFinder#clean_path should raise InvalidPageIdError when blank

When HighVoltage encounters a page_id with invalid characters, it strips those invalid characters and tries to calculate a file path from the result. If the resulting path is blank, HighVoltage will attempt to render a template at pages/. This causes an ActionView::MissingTemplate in production as a 500 error.
For example:
pf = HighVoltage::PageFinder.new('關於我們/合作伙伴')
pf.find # => "pages/"
pf.send(:clean_id) # => "/"
pf.send(:clean_path) # => ""

Thanks @hube


Handle incoming webhooks

Adds support for handling webhooks of the types INCOMING_TOKEN, INCOMING_ETHER, MODULE_TRANSACTION, OUTGOING_TOKEN and OUTGOING_ETHER
CacheHooksService now handles PENDING_MULTISIG_TRANSACTION and NEW_CONFIRMATION events in addition to the events aforementioned

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

54 of 54 (100.0%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 92.196%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
4117

Relevant Lines:
4359

💛  - Coveralls


Add yarn verify script

Basically the same static analysis as the github workflow, but in the form of a yarn script for more convenient local use.
If there's already an easy way to do this let me know, but I didn't see anything along these lines in the README or package.json.

I think it would be better if the verify scripts in each subdirectory did all the work for that subproject, and then in the root directory it checked everything. To me that would be the ideal version. But I like this better than the status quo. Maybe someone can improve it later.


Buttons and TextureButtons positioned in Origin when migrating project from Godot 3.5 to Godot 4

Godot version
3.5
System information
Garuda Linux, 5900HX, 6800M
Issue description
I tried migrating a copy of my project from Godot 3.5 to Godot 4.0 both Alpha 16 and git and the Buttons and Texture Button were converted to be placed at the origin of the root scene. A completely 2D game.
Steps to reproduce
Place a Button or Texture Button somewhere in a scene with Godot 3.5. Open in Godot 4 and the object is misplaced.
Minimal reproduction project
I don't think a project is required but if needed I will provide one.

Yep, seconding.

That's because properties were renamed. Use project converter (it will be easier in alpha 17, with #64927)


[Feature] MCAS Module - Link user to the M365 User page

Get-MCASInvestigationScore -> Link to Security User Page (https://security.microsoft.com/user?aad=AADUSERID)

Added tenant info to base module to assist in building these links via f00e0b7cbea95d0627226ecbdeb40f194f4b90d5

Added comments in eee29767c032e350dbb27cf43d20310658553953


gRPC segfaults while updating node resources

Dec  1 09:41:00 primary DEBUG sf-resources[3315361] Disk free; blobs=85228068864; events=85228068864;
    image_cache=85228068864; instances=85228068864; sfroot=85228068864; uploads=85228068864
...
Dec  1 09:41:09 primary kernel: [ 8252.854681] sf-resources[3315361]: segfault at 18 ip 00007fe91a1b27e5
    sp 00007ffd22523860 error 4 in cygrpc.cpython-310-x86_64-linux-gnu.so[7fe919f75000+953000]
Dec  1 09:41:09 primary kernel: [ 8252.854720] Code: 8d 54 24 40 48 89 14 24 48 89 10 e9 c1 fe ff ff 90 0f 1f
    40 00 41 57 41 56 41 55 41 54 55 48 89 fd 53 48 83 ec 48 89 74 24 04 <48> 8b 5f 18 48 8d 05 50 9f 9f 00
    0f b6 40 10 84 c0 0f 85 6d 1b de
...
Dec  1 09:41:14 primary WARNING sf-main-v0.8.0[10877] resources process has exited
Dec  1 09:41:14 primary WARNING sf-main-v0.8.0[10877] resources is dead; exit=-11; pid=3315361
Dec  1 09:41:14 primary INFO sf-main-v0.8.0[10877] Started resources; pid=1433717

I can't find any relevant bugs in a search, apart from vague accusations that threading doesn't work well with gRPC. This code does not use threading.

So far I have observed this in:

Ubuntu 22.04 resources daemon while putting  new node metrics job.

This seems to be much more common when the CI cluster is under load, but I don't know why that would cause a segfault?


[NTR] Neuroendocrine cell of prostate epithelium

Note - please check that the term does not already exist (check OLS: https://www.ebi.ac.uk/ols/ontologies/cl)
Preferred term label:
Neuroendocrine cell of prostate epithelium
Synonyms
Definition (free text, please give PubMed ID in format PMID:XXXXXX)
Neuroendocrine cell (CL:0000165) in the prostate

PMID:33842835
PMID:31390564
PMID:32497356

Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols/ontologies/cl):
CL:0002231
What is the anatomical structure that the cell is a part of? Please check Uberon: https://www.ebi.ac.uk/ols/ontologies/uberon
UBERON:0000428
Your nano-attribution (ORCID)
https://orcid.org/0000-0001-7772-9578
Any additional notes or concerns

@GHAStVHenry PMID:31390564 (https://www.ncbi.nlm.nih.gov/labs/pmc/articles/PMC6710009/)  does not seem to mential neuroendocrine cells. Am I missing anything?

Prostate neuroendocrine are rare and sometimes difficult to identify unless you are specifically looking for them. That paper did not identify them as it wasn't important for proving their hypothesis. They are usually referred to in publications in reference to prostate cancer, which can trans-differentiate into neuroendocrine-like prostate cancer... but they are present in the normal prostate and is very well accepted in the urologic field as being present.
The review @ @pmc8023015 describes them as:
NE cells are scattered among the more abundant luminal and basal cells, representing only ∼1% of the epithelial population.11, 13, 21 Because of their rarity in benign prostate and most prostate tumors, they cannot be appreciated on standard H&E stained sections. However, through the use of electron microscopy, they can be identified and differentiated from luminal and basal cells by their elongated cell bodies and intracytoplasmic dense core secretory granules.21 Furthermore, electron microscopy can differentiate between two morphologic subtypes of NE cells: open-type and closed-type.24 Open-type cells possess long microvilli which reach the lumen and can detect changes in pH as well as respond to other chemical stimuli in luminal secretions. Closed-type cells possess dendritic-like processes and receive stimuli from nerve endings, blood vessels, and smooth muscle cells. A more practical way to highlight NE cells in the prostatic epithelium involves the use of immunohistochemistry (IHC) (Fig. 1) and commonly used markers include chromogranin A (CgA),11, 13, 21 synaptophysin (SYP),11, 13, 21 neuron-specific enolase (NSE),11, 13, 21 neural cell adhesion molecule (NCAM or CD56),11, 13, 21 forkhead-box A2 (FOXA2),25 and CXC chemokine receptor 2 (CXCR2).26 In addition to being negative for AR and PSA, they are also negative for Ki67 because they are non-proliferative, post-mitotic cells.11, 13, 21, 26

...as a matter of fact, it even describes two types (open-type and closed-type) which I have never heard about.

@GHAStVHenry  thanks! I am working on this ticket, but it has been a while since my last CL edit, the review might take a while, but it should be moving forward.

That's awesome, thanks @lubianat, while this is important...
I'm really would love to have terms #1133 #1134... I have some curation processes that could use terms for them... are you working on those tickets too?

I've seen them, I plan is to pick up on those two after this one is complete (unless someone happens to work on them first)

sounds good... this is the "easiest" one as it is an "old" cell type... the other 2 are "new" so there are fewer references

@lubianat assigning to you as I see you made a PR. Thanks.
As a reminder, I removed the automated assignments;
There are 4 NTRs by the same submitter, so it'd make sense to address them all at the same time: https://github.com/obophenotype/cell-ontology/issues/1133 (urethra), https://github.com/obophenotype/cell-ontology/issues/1134 (urethra), https://github.com/obophenotype/cell-ontology/issues/1135 (prostate) and https://github.com/obophenotype/cell-ontology/issues/1171 (bronchiole).


Translations update from Hosted Weblate

Translations update from Hosted Weblate for Halma/Halma.
Current translation status:

Thank you so much for your support, @bittin! 😘


XCTest.framework/XCTest' does not contain bitcode

Using Kiwi 3.0.0
When trying to run the Unit Tests on a real device, Xcode trows an error saying You must rebuild it with bitcode enabled. Running the tests on simulator works fine.
Nimble had a similar issue. They way I solved this is setting the ENABLE_BITCODE to NO using cocoapods
if ['Kiwi'].include? target.name
	target.build_configurations.each do |config|
		config.build_settings['ENABLE_BITCODE'] = 'NO'
       end
end

ld: '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/Frameworks/XCTest.framework/XCTest' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. file '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/Frameworks/XCTest.framework/XCTest' for architecture arm64
Is Bitcode actually really required?

to make PatrickDotStar's answer for clear
add the following to the Podfile
post_install do |installer|
  installer.pods_project.targets.each do |target|
    if ['Kiwi'].include? target.name
        target.build_configurations.each do |config|
            config.build_settings['ENABLE_BITCODE'] = 'NO'
        end
    end
  end
end

go to your target name => build setting=> EnableBitcode
change the value to no


MVVMified with MahApps UI

Organized code into MVVM structure using MVVM Light so that expansion is easier. Retained existing functionality for now, although I'll need to add back the progress label or a progress bar.

Closing, forgot to include JSON export.


Fixed color of list_tile icon

Description
Actually ListTile replaces the icon theme color of the app to place a fixed color, with this change, I can set the color of the trailing icon of the ListTile by the Theme.
Related Issues
I made this PR by a problem that I had, but I found this other issue that can be related to this: #11593
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[x] No, no existing tests failed, so this is not a breaking change.

Thanks for the contribution. Sorry it took a while for me to get to this.
It looks great, and is a nice feature. Unfortunately it would break several of our internal apps that depend on the current behavior. We can update these apps to work around it, but we would rather hold off until we have some of the larger theme changes that we have planned at the same time (see Material Theme System Updates for more info).  When we get to the list tile we can incorporate this type of change.
For now though we need to hold off on this fix.