Add Expires header too

gtmetrix.com complains when no Expires header is sent - even when cache-control is set.
Perhaps we should set both, and not just cache-control?
See https://wordpress.org/support/topic/expires-headers-not-showing/#post-11390627

Direct quote from https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Expires ...

If there is a Cache-Control header with the max-age or s-maxage directive in the response, the Expires header is ignored.

So, it is safe to use both.

ok, thanks :)

Actually, looks like it's just a syntax error in the default cache-control header setting.
This works:
public, max-age=31536000, stale-while-revalidate=604800, stale-if-error=604800

But the plugin will default to max-age:86400 with the : being the bug (also, maybe they should be cached for a year by default?)

By default (when nothing is touched), the Cache-Control header option is set to "Do not set". And this result in the expected (no Cache-Control header). I am happy with this default. If it is set to one year per default, it will take a year to completely undo it. Not setting the header results in browsers caching only a short period, but over time (if resource has not changed) increasingly longer.

I found the bug your referred to. It is the default value that shows up, when you select "Custom". This default value is not used, unless you select Custom and leave it at that value. This should not affect too many. But of course I must fix it! Thanks for reporting!


truncating from usize -> u8 (for example) doesn't actually truncate at runtime

Can be observed in this sample program:
https://github.com/exoticorn/microw8/blob/master/examples/rust/tunnel.rs
Since wasm doesn't give us this "for free", we'll have to add a runtime implementation ourself and call when appropriate.
As a workaround, forcing a store (by going to an intermediate local) should resolve the issue

overflow has the same issue. bleh. we could just insert a store followed by a load to force the wrapping?

Created #45 for overflow

Closed in d8300c8


Helm v3 getting current-context wrong

If I run: kubectl config current-context - I get the correct cluster.
My KUBECONFIG=/home/klavs/.kube/microk8s:/home/klavs/.kube/t-boss-ca:/home/klavs/.kube/p-boa-kube-ca
but helm picks the wrong cluster. Thats VERY dangerous - when your kubectl config current-context says you're working on a different cluster (and my PS1 shows that as the active one as well)..
If I run helm - pointing to the cluster config file I want it to select like this - it works:
KUBECONFIG=/home/klavs/.kube/t-boss-ca helm ls

When I use multiple config files - and then use kubectl to set current-context - it ONLY updates the first file.. as the other files in the KUBECONFIG path - actually had invalid "current-context" - due to that context being renamed (by kubectl).. which means - that the individual files (as being used in my work-around above) - can become invalid - and in that case helm just not say the KUBECONFIG is invalid - it simply tries to connect to localhost:8080 instead..

it seems helm ALWAYS chooses the FIRST configfile in KUBECONFIG - and only works on that - completely ignoring the otherfiles - and also ignoring current-context.. :(

@KlavsKlavsen The detection of the Kubernetes configuration happens via imports from packages put out by Kubernetes. With that in mind... can you share the version of Helm you're using. We need that to trace where this issue is happening in either a dependency or our use of a dependency. On a first pass in Helm v3 I didn't see an issue. Doesn't mean it's not there but more information would help me figure out what's going on.

Sorry.. its v3.0.2

I started looking into this issue yesterday and did find some issues with how we loading the kubeconfig.  I'll work on a patch when I find the exact issue.


Hover Effect in buttons of Gymso Website

Can I add Beautiful UI Hover effects in the buttons of Gymso Fitness Website?

absolutely! your assigned and hacktoberfest tags added!

@0silverback0 Please Check #51

closing this issue, nice work!


Check for gem and file in bin/theme

Addresses the TODO in bin/theme.
Did a little refactoring too just to make things look simpler.

Closing, completely forgot I did this in #19.


bug(mat-tabs): MatMenu within a MatTabNavPanel causes ChangedAfterChecked errors

Feature Description

Submitting as a Feature as I'm pretty sure the below scenario isn't officially supported by MDL.

StackBlitz Scenario
When building out child routes accessed via a <nav mat-tab-nav-bar>, it's helpful to provide a button or menu exposing common functionality directly within the tab panel (vs. adding another panel parallel or adjacent to the tab panel). This technically works by linking a mat-tab-link to a menu, but it triggers the following console errors when the menu is opened / closed:

If I change the menu trigger to a simple <a mat-button>, the styling for the button does not conform to the styling of the tabs in the nav panel:

Recommendation would be to adjust styling for mat-button to to conform to tab styling when embedded in a mat-tab-nav-bar:
nav.mat-tab-nav-bar .mat-button {
  height: 48px;
  padding: 0 24px;
  opacity: .6;
  text-align: center;
  display: inline-flex;
  justify-content: center;
  align-items: center;
  white-space: nowrap;
  vertical-align: top;
  text-decoration: none;
  position: relative;
  overflow: hidden;
}

Use Case
No response

Changing to a bug since as far as I'm aware this usage should be supported. Assigning to @andrewseguin to see what he thinks

Is there any update on this, or any work-around?
(This seems to not occur if we have a tabPanel, but if we add a tabPanel then the behavior changes completely.)

Hey! We're still getting this error in Angular 16, just FYI.


Transform machine-learning-productionization

Transform the contents of this folder.
The path is:
open-machine-learning-jupyter-book/machine-learning-productionization
https://github.com/ocademy-ai/machine-learning/tree/main/open-machine-learning-jupyter-book/machine-learning-productionization


Use laptop camera as light sensor

📝 Provide a description of the new feature
Many tablets have a good feature: autobrightness. They use integrated cameras to check lights level.
Every laptop have camera but Windows can't use this cameras as light sensor.
Such functionality doesn't need to everyone so there are no need to integrate this feature in Windows core but this can be realized with PowerToys as a real toy. This toy can auto-balance brightness by checking external light levels.

Most laptops actually can set screen brightness.  I am not sure if they use the actual camera to accomplish that as well versus a light sensor.
Is this for like a desktop scenario?

Not every laptop have such a feature. For example - my 2019's Lenovo Legion doesn't (and his camera not even supported by Windows Hello). Only Night Light, HD Color and scaling.
For such loosers this toy can be a salvation as an external solution for oficially not supported hardware configurations.

This means the camera has to be open either continuously or a short time in regular intervals. Does that not use more power?
Also, (my experience is) the camera usually can only output to one program at a time. What if the camera is already in use for a video call? Or the other way around: if this new function blocks access from other programs?
What about the cam-on indicator light - if present? If it's always on (even though the user is not really using the cam) then the security factor of this light is being undone. A flashing light might be considered annoying or just confusing.
As both of you said, cameras can be quite different in capabilities on different devices, with or without a simple brightness sensor, IR sensor (for Hello) and indicator light.

while interesting idea, i don't seeing the PowerToys team building this idea out.  It has a ton of permutations of complexity.

camera can be in use
hard to adjust
ability to set brightness dependent on HW


Replace CI approvers

thank you @cschwede, it was a pleasure working with you, and thanks for all the help.


Correct author order

Dearest Reviewer,
PR #2696 added some new checks for environment variables for author and
email. After the pull request was merged a comment was left about gits
ordering for looking at the variables. This pull request closes #2705 by
reordering the look up and also addeds CARGO_NAME and CARGO_EMAIL to the
top of the order.
Thanks
Becker

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+ c29be695a13f978a62fa2be1db21fc7a53da7aed
Thanks!

:hourglass: Testing commit c29be69 with merge 7466f45...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


On refresh profile API hangs

When viewing a user's page that is not your own, if the refresh button is clicked the call to profile never returns data, freezing the client unless a timeout is set on the AJAX call (which results in an error message).
Two unhandled errors are visible in the command window:
2016-01-04 13:43:33-0500 [OpenBazaarProtocol (UDP)] Unhandled error in Deferred:
Unhandled error in Deferred:
2016-01-04 13:43:33-0500 [OpenBazaarProtocol (UDP)] Unhandled Error
Traceback (most recent call last):
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 490, in _startRunCallbacks
self._runCallbacks()
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 577, in _runCallbacks
current.result = callback(current.result, *args, **kw)
File "C:\Users\J\ob\OpenBazaar-Server\api\restapi.py", line 83, in get_node
_showImage()
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 1237, in unwindGenerator
return _inlineCallbacks(None, gen, Deferred())
---  ---
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 1099, in _inlineCallbacks
result = g.send(result)
File "C:\Users\J\ob\OpenBazaar-Server\api\restapi.py", line 71, in _showImage
request.finish()
File "C:\python27\lib\site-packages\twisted\web\server.py", line 228, in finish
return http.Request.finish(self)
File "C:\python27\lib\site-packages\twisted\web\http.py", line 930, in finish
"Request.finish called on a request after its connection was lost; "
exceptions.RuntimeError: Request.finish called on a request after its connection was lost; use Request.notifyFinish to keep track of this.
Unhandled Error
Traceback (most recent call last):
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 490, in _startRunCallbacks
self._runCallbacks()
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 577, in _runCallbacks
current.result = callback(current.result, *args, **kw)
File "C:\Users\J\ob\OpenBazaar-Server\api\restapi.py", line 83, in get_node
_showImage()
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 1237, in unwindGenerator
return _inlineCallbacks(None, gen, Deferred())
---  ---
File "C:\python27\lib\site-packages\twisted\internet\defer.py", line 1099, in _inlineCallbacks
result = g.send(result)
File "C:\Users\J\ob\OpenBazaar-Server\api\restapi.py", line 71, in _showImage
request.finish()
File "C:\python27\lib\site-packages\twisted\web\server.py", line 228, in finish
return http.Request.finish(self)
File "C:\python27\lib\site-packages\twisted\web\http.py", line 930, in finish
"Request.finish called on a request after its connection was lost; "
exceptions.RuntimeError: Request.finish called on a request after its connection was lost; use Request.notifyFinish to keep track of this.
2016-01-04 13:44:10-0500 [OpenBazaard] [INFO] bootstrap complete
2016-01-04 13:46:42-0500 [ConnHandler] [INFO] connected to <IP_ADDRESS>:28467

Those errors are saying the UI closed the connection before the server was able to return the data.

The web dev panel shows the request as still open, and not timed out. Is there something else the client might be doing that would tell the server the connection is closed?

I think this has been fixed @cpacia and @jjeffryes  so I'm going to tentatively close this, given that it's 27 days old. Please reopen if this is still a problem.


Enable CodeQL scanning?

@bai thoughts on enabling https://github.com/Shopify/sarama/security/code-scanning? I haven't much personal experience of the CodeQL analysis of Go projects https://github.com/github/codeql-go compared with what we already get from golangci-lint and staticcheck, but it seems like it wouldn't hurt and could have additional useful coverage

This is a great idea! Submitted a PR to test things out - let me know what you think 🙏🏼

Thanks, working well and clean bill of health on master now https://github.com/Shopify/sarama/security/code-scanning


[Feature Request] Adjust GetNoteData to return correct note type with holds

Please Select the game mode your feature request is about.
All Game Modes
Is your feature request related to a problem?
Currently, when using notedata, all holds return "TapNoteSubType_Hold". There is no way to access information about the type of tail a hold or roll has. This information is potentially useful for themes, noteskins, and modfiles.
Describe the solution you'd like
Adjusting the way GetNoteData returns the notedata table to account for newer hold and roll types is an ideal solution.
Describe alternatives you've considered
No response
Additional context
No response

Giving a shot at this, currently I have the output to show this:
return {
	{4.000000,1,"TapNoteSubType::TapNoteSubType_Hold","TapNote_4th",group = "TapNoteGroupType_Normal",length = 4.000000},
	{8.000000,2,"TapNoteSubType::TapNoteSubType_Hold","TapNote_4th",group = "TapNoteGroupType_Mine",length = 4.000000},
	{12.000000,3,"TapNoteSubType::TapNoteSubType_Hold","TapNote_4th",group = "TapNoteGroupType_Normal",length = 4.000000},
	{16.000000,3,"TapNoteType::TapNoteType_LongNoteTail","TapNote_4th",group = "TapNoteGroupType_Lift"},
}

The way it works is that the group item for this data set is present for all the items. However, the change here is that if there is a Lift Hold present, the lift note itself will be added to the table of items as a separate entity to ease parsing.
However, I'm open for suggestions about this implementation.

this looks straightforward to me, and it seems like it keeps in line with previous specs, i agree with it


Review other Well Architected Framework pillars

Operational Excellence
Reliability
Performance

Reliability

Change from 01 EC2 instance in 01 public subnet to ECS cluster in 02 private subnets
Deploy 02 NAT Gateway in 02 public subnets
Scale horizontally to increase aggregate system availability by scaling at task level with ECS
Manage change in automation with CI/CI pipeline

Performance:

Using advanced technologies with ECS
Use serverless architectures


Only image edge tiles displayed

I have been running into this issue frequently, but not reproducibly! The OSD viewer only displays the image edge tiles, but not the internal tiles as shown below. Frequently, the same images are displayed fine when I browse from another computer. I tend to think that it's a setting in my Chrome browser, but I can't quite figure out what affects it. Clearing out the local browser cache and reloading does not seem to have any effect. Any ideas for diagnosing the cause are greatly appreciated!
In the example below I'm loading 6 images and I'm creating a custom layout

I'm finding that if I don't use a fully qualified domain name for the server that hosts the page, the problem is resolved. I don't understand why though!

That is very strange! Do you have a URL I could try?
If you open up the developer console are there any errors?

Unfortunately not. This is happening on our internal network and IS NOT reproducible for everyone. Given that this issue does not happen with all of the users, I have started suspecting network latency and timeout issues. There are no errors reported in the developer tools console, but i'm wondering whether the OSD API has any way of adjusting the timeout for displaying the images. What is the behavior if this timeout is exceeded? Should OSD be able to load slow images from the browser cache next time?

You can adjust the image timeout with the timeout option when you create your viewer. The default is 30 seconds (you specify milliseconds). You should get a message in the console if it's timing out, though. You're not seeing anything in the network tab of the dev inspector either?
OSD just uses the browser's image fetching (unless you specify you want to do it via AJAX), so the standard browser cache does get used as appropriate.


Spout output very banded

Banded output
Is there anyway that i can set the Spout sender to have 32 bit full float precision to avoid having those bands on the output for the recievers? I'm currently using the Texture Capture method and a render texture in HDRP.

I think you can change the format by modifying the following lines:
https://github.com/keijiro/KlakSpout/blob/main/Plugin/Sender.h#L58
https://github.com/keijiro/KlakSpout/blob/main/Packages/jp.keijiro.klak.spout/Runtime/SpoutSender.cs#L43


Cannot build android app after upgrading to Flutter 3.24.1 | Customer IO 1.5.2

exception:
Execution failed for task ':app:mergeExtDexProductionDebug'.
> Could not resolve all files for configuration ':app:productionDebugRuntimeClasspath'.
   > Failed to transform tracking-3.11.2.aar (io.customer.android:tracking:3.11.2) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.category=library, org.gradle.status=release, org.gradle.usage=java-runtime, org.jetbrains.kotlin.platform.type=androidJvm}.
      > Execution failed for DexingNoClasspathTransform: /Users/myUSERNAME/.gradle/caches/transforms-3/d5f3630ab66d2645cc295211aa0befa7/transformed/jetified-tracking-3.11.2-runtime.jar.
         > Error while dexing.

kotlin version: 1.9.22
gradle version: 7.2.2
Customer IO: 1.5.2

Hi @Mo0Khaled,
Thank you for reaching out! Can you try clearing our Gradle cache by running ./gradlew clean and remove any .gradle caches in your project directory if that helps fix the issue.
Could you also share more details, such as detailed error logs or any other information, to help us troubleshoot further?

Hi @Mo0Khaled,
Thank you for reaching out! Can you try clearing our Gradle cache by running ./gradlew clean and remove any .gradle caches in your project directory if that helps fix the issue.
Could you also share more details, such as detailed error logs or any other information, to help us troubleshoot further?

Thank you @ami-aman for your fast replay,
actually I've tried it but didn't work,
there is no error except the one I mentioned above, but let me tell you that the exception disappear and I can run the project if I upgraded to gradle 8.2.1 but this will cause a lot of problems with other plugins, I wonder if there are other solutions for it,
and for anyone who will not have any problems with other plugins you just need to do the following so you can run your project smoothly:

in /project/android/gradle/wrapper/gradle-wrapper.properties
add distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-all.zip

and set the gradle version in the build.gradle to 8.2.1
and you may want to set the Kotlin verision to 1.9.22 or higher.

Thanks for sharing all the helpful details here.
As the next step, I suggest you try updating the Android Gradle version in your settings.gradle file like this:
id "com.android.application" version "7.4.1" apply false

Please ensure you are checking the Android Gradle Plugin version in settings.gradle and not the Gradle version in gradle-wrapper.properties.
Let me know if you run into any further issues!

Closing this issue due to inactivity. Feel free to reopen if required.

Hey @Mo0Khaled  !
We fully support Gradle 8 and above, so upgrading to version 8 or higher should not cause any issues. If you’re facing a specific problem, could you elaborate more on what you’re encountering? We’ll be happy to help troubleshoot or provide guidance based on the issue you’re experiencing.


Add support for verifying dsse-intoto

Verification should be able to correctly validate a bundle as cryptographically valid (VerificationOptions.empty())
Verifiers may also include signer identity during verification
Verifiers should extract the embedded attestation to do further analysis on the attestation. Sigstore-java does not process those in any way
There is no signing options for DSSE bundles

I'm gonna split this up, make it a little easier to review.


Release fixes

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?
operator limits raised to 300m/100Mi
Revert "feat: use fluentd gracefulReload for configuration updates"
Checklist

[x] Error handling code meets the guideline
[x] Logging code meets the guideline (TODO)
[x] User guide and development docs updated (if needed)

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Add support for logical types in schema-registry-avro

Problem
The @azure/schema-registry-avro library does not allow configuring deserialization for logical types.
Solution
Currently the options parameter passed to the constructor of the AvroSerializer class only supports the property groupName. This parameter could be expanded to support also passing options to use when calling avro.Type.forSchema().
Additional context
In the concrete scenario we are dealing with, the Avro schema contains a date field specified as follows:
{
      name: "startDate",
      type: {
        logicalType: "timestamp-millis",
        type: "long"
      }
}

Calling deserialize() fails because long cannot be deserialized (generates an error in the avsc library stating "Potential precision loss"). Furthermore, the number needs to be deserialized as Date. This is supported by the avsc library as describe here,
However there is no way to pass such options to the avro.Type.forSchema() call perfomrmed in the getSerializerForSchema() function.
For example, the following works:
function getSerializerForSchema(schema) {
  return wrapError(() => avro.Type.forSchema(JSON.parse(schema),
    {
      omitRecordMethods: true,
      logicalTypes: { 'timestamp-millis': DateType }
    }), `Parsing Avro schema failed:\n\n\t${schema}\n\nSee 'cause' for more details.`);
}

where DateType is defined as described here.

Thank you for opening this report!

Calling deserialize() fails because long cannot be deserialized (generates an error in the avsc library stating "Potential precision loss")

I agree this can be frustrating but it is the reality of JS.

Furthermore, the number needs to be deserialized as Date. This is supported by the avsc library as describe here,

You're right that logical types aren't supported at the moment. Let me get back to you on this.

Hi @deyaaeldeen , thanks for your reply. Just wanted to point out that there are ways of configuring support for long in the underlying avsc lib (see here), however these are not accessible through the Azure lib. Being able to pass options to forSchema() would tackle this aspect as well

Thank you for raising the issue. We have released<EMAIL_ADDRESS>with a fix. Please feel free to check it out and let us know if you run into any other issues.

Yes, this now works as expected for dates expressed as longs with logical types


Error : value too large for defined data type caused

I followed the tutorial for installing sysbox, but when I got to this line :
docker run --runtime=sysbox-runc -P -d nestybox/ubuntu-bionic-systemd-docker
I got his error :
docker: Error response from daemon: failed to create shim task: OCI runtime create failed: container_linux.go:425: starting container process caused: process_linux.go:607: container init caused: process_linux.go:578: handleReqOp caused: rootfs_init_linux.go:366: failed to mkdirall /var/lib/sysbox/shiftfs/bec40c1d-957b-43af-99ee-67680a8689ed/var/lib/rancher/rke2: mkdir /var/lib/sysbox/shiftfs/bec40c1d-957b-43af-99ee-67680a8689ed/var/lib/rancher: value too large for defined data type caused: mkdir /var/lib/sysbox/shiftfs/bec40c1d-957b-43af-99ee-67680a8689ed/var/lib/rancher: value too large for defined data type: unknown.
I am running Mint version 21.1 and Docker version 20.10.21, build 20.10.21-0ubuntu1~22.04.2.
My linux kernel is : 5.15.0-67-generic
My shiftfs : 5.15.0-67-generic SMP mod_unload modversions
Do you have any idea why this is happening ?

There seems to be a problem with kernel 5.15 for shiftfs as described here :
https://stackoverflow.com/questions/75343570/sysbox-runc-is-not-working-as-runtime-for-docker-in-ubuntu
Trying to upgrade to newer verison of shiftfs

Solution : disable shifts thanks to https://github.com/nestybox/sysbox/issues/596#issuecomment-1297185361
sudo mkdir -p /etc/systemd/system/sysbox-mgr.service.d && printf '%s\n' '[Service]' 'ExecStart=' 'ExecStart=/usr/bin/sysbox-mgr --disable-shiftfs' | sudo tee /etc/systemd/system/sysbox-mgr.service.d/override.conf && sudo systemctl daemon-reload && sudo systemctl restart sysbox

Hi @Vaunorage, thanks for trying Sysbox and sorry you hit this issue with shiftfs, but glad you were able to find the work-around.


TypeError in yandex music extractor

youtube-dl --version is 2016.07.22, but related file https://github.com/rg3/youtube-dl/commits/master/youtube_dl/extractor/yandexmusic.py was modified before that date.

[ ] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The issue is that youtube-dl exits with TypeError in yandex music extractor:
youtube-dl https://music.yandex.ru/album/1613116
[yandexmusic:album] 1613116: Downloading album JSON
[download] Downloading playlist: Abney Park - Ancient World (2012)
[yandexmusic:album] playlist Abney Park - Ancient World (2012): Collected 15 video ids (downloading 15 of them)
[download] Downloading video 1 of 15
[yandexmusic:track] 14721249: Downloading track JSON
[yandexmusic:track] 14721249: Downloading track location JSON
Traceback (most recent call last):
  File "/usr/lib/python-exec/python2.7/youtube-dl", line 9, in <module>
    load_entry_point('youtube-dl==2016.7.22', 'console_scripts', 'youtube-dl')()
  File "/usr/lib64/python2.7/site-packages/youtube_dl/__init__.py", line 422, in main
    _real_main(argv)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/__init__.py", line 412, in _real_main
    retcode = ydl.download(all_urls)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 1775, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 693, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 854, in process_ie_result
    extra_info=extra)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 746, in process_ie_result
    extra_info=extra_info)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 682, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/common.py", line 347, in extract
    return self._real_extract(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", line 138, in _real_extract
    return self._get_track_info(track)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", line 96, in _get_track_info
    'url': self._get_track_url(track['storageDir'], track['id']),
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", line 78, in _get_track_url
    key = hashlib.md5(('XGRlBW9FXlekgbPrRHuSiA' + data['path'][1:] + data['s']).encode('utf-8')).hexdigest()
TypeError: coercing to Unicode: need string or buffer, list found

There is an empty list under youtube-dl https://music.yandex.ru/album/1456657
It gives another Type Error now:
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", line 88, in _get_track_url
    % (data['host'], key, data['ts'] + data['path'], storage[1]))
TypeError: unsupported operand type(s) for +: 'NoneType' and 'unicode'

@weeshy Please post the full verbose log of the command youtube-dl -v https://music.yandex.ru/album/1456657

I tried to reproduce the error and inserted debugging print() class, but suddenly all become nice again.


Implement to Laravel Project

How to implement this template to Laravel project?
What should I do?
I already copied components, package.json, and some javascripts.
But still not working.

HMMMMMMMMMMMMMMMMMMM

i have used it with laravel. you have to remove all lang attributes from template tags.

i have used it with laravel. you have to remove all lang attributes from template tags.

remove all lang ? just like that ?


fixes README links

Coverage remained the same at 98.064% when pulling 5fb215c08489be35c3f22f728f600f8489876d1b on fixes-readme-links into a30b7934e826d8ae3a3195b54a49ad5ca536d492 on master.


Changelog still shows 1.23.16 as "Not released yet"

This shows up in the VS Code Extensions view Changelog tab.

@gregvanl  Thanks for opening this issue. Unfortunately the changelog shown in the marketplace comes from the commit used to build the shipping package. Since we do not know how long validation will take, we do not update this bit of text until after release. It has already been updated in main - see https://github.com/OmniSharp/omnisharp-vscode/blob/master/CHANGELOG.md#12316-oct-12th-2021


CRLP Rollup field values not cleared on Opportunity Reparenting

Critical Changes
Changes
Issues Closed
Fixes #3752
New Metadata
Deleted Metadata

**lurch: attach W-029419

Tracking W-029419

Thank you both for reviewing and testing do quickly. This is probably ready for merge.


deploy_from_abi_and_bytecode failed

error:

reference:
https://github.com/gakonst/ethers-rs/blob/master/examples/contracts/examples/deploy_from_abi_and_bytecode.rs
version:
ethers = "1.0.2"

is Atoken by any chance just an interface?

no, simple ERC20 Contract

could you please provide the json content and the cargo expand output of the example.

AToken.json.txt

did this ever get resolved?


Subset cmap remapping in buuilder config?

@khaledhosny I occasionally encounter situations in which one or more characters would be better mapped to a different GID in a subset font. Is there a way this could be conveniently supported via the builder config file, e.g. as a mapping list of codepoints to glyph names? [I presume, since this would usually be at the subset level, that the required glyph name would be post .ren renaming, as per the subsetting itself.]

I think we can do that in the post processing step after subsetting. Do you have a preferred way to do this mapping? I guess an external file would be best. I guess something like this:
0xNNNN glyphname

It would then loop over cmap subtables and check if 0xNNNN exists in the subtable and update the mapping, should it also add a new mapping of none exists?

An external file is probably right for this. Yes, it would be good to be able to add new mappings, although I can’t think of a specific use case at the moment.


Sign Firefox Add-on

The Firefox add-on needs to be signed so it can continue to be installed in upcoming versions of Firefox.
https://blog.mozilla.org/addons/2015/02/10/extension-signing-safer-experience/
https://support.mozilla.org/en-US/kb/add-on-signing-in-firefox
https://wiki.mozilla.org/Addons/Extension_Signing#Timeline

Thanks for the heads-up. Something important missing last I looked was an API for the extension signing service. The build and deployment for Passcards is entirely automated. For Chrome, there is a script which publishes to the store using the Chrome Extension store APIs.
I think I read somewhere that AdBlock Plus is currently using Selenium or a similar browser testing/automation tool for this. Not pretty, but it works.

I don't know of any API that would allow the process to be entirely automated but the review process is also slightly different than Chrome if I understand correctly. Every update of a Firefox add-on goes through a manual review process so it is likely to always be a few days behind an automated Chrome extension release anyway. I would be willing to post new versions to Mozilla within a one or two days of a change being made.

I don't know of any API that would allow the process to be entirely automated
but the review process is also slightly different than Chrome if I understand correctly

As I understand it, the manual review process in the past was partly down to Firefox addons essentially having unfettered access to the browser and also I think partly down to different expectations about what was an acceptable speed for delivery of updates. Chrome changed those expectations and Mozilla are adapting.
Also, Passcards' build and deployment is entirely automated - I push changes to master and the Firefox and Chrome extensions, web app and command-line app are all published a short a few minutes later. Together with the entire thing being written in JS and sharing as much code as possible between versions, this is important to make it possible to ship a useful app with far fewer resources than commercial alternatives.

I understand that everything is automated now but until Mozilla creates this API, updates to the Firefox add-on are going to be significantly behind that of the Chrome add-on. I think you should at least publish it manually for now because in one month no one will be able to install Passcards (anyone who runs a nightly version of Firefox, like me, hasn't been able to install Passcards for a while).
Again, I would be willing to upload new versions of Passcards to AMO for you if you would like.

@aaronraimist - I agree and thanks for the offer. A first step will be to get the extension to a point where it validates at least the checks required for automated signing on AMO, which are less strict than those required for manual review.
I've not submitted a recent build to AMO myself so I don't know how many issues that will flag up.

See this discourse post for more information from Andreas Wagner: https://discourse.mozilla-community.org/t/upload-an-add-on-update-via-api/5034
It lists a Mozilla bug and some GitHub issues so you can track the progress of implementing the API.
For now you can upload it manually or use something like this: https://github.com/minj/foxtrick/blob/master/maintainer/dist/amo_upload.py (found in https://github.com/mozilla/shumway/issues/2329) or use the Adblock Plus code: https://adblockplus.org/blog/adblock-plus-for-firefox-and-element-hiding-helper-development-builds-now-hosted-on-amo that you mentioned earlier.

@robertknight Have you submitted a build to AMO yet? If you would like, I can submit a build and set you as the owner so the review can get started.

@aaronraimist - I haven't submitted a build to AMO yet and if you could that would be much appreciated. Thanks.

@robertknight I submitted an initial build to AMO. It is currently 12 of 12 in the queue for a preliminary review. I'll post my changes to a fork of this code soon.

Your version was rejected because of the following problems:

1)  Evaluation of strings as code can lead to security vulnerabilities and performance issues, even in the most innocuous of circumstances. Please avoid using `eval` and the `Function` constructor when at all possible.

Alternatives are available for most use cases. See https://developer.mozilla.org/en-US/Add-ons/Overlay_Extensions/XUL_School/Appendix_C:_Avoid_using_eval_in_Add-ons for more information.

2) In order to prevent vulnerabilities, the `setTimeout` and `setInterval` functions should be called only with function expressions as their first argument. If you are passing a function which is not being correctly detected as such, please consider passing a closure or arrow function, which in turn calls the original function.

Please fix them and submit again. Thank you.

Sighs - To the best of my knowledge, all of the warnings about setTimeout() are spurious. That is, in the context of Chrome or a normal web page, CSP could be used to make any uses of setTimeout(<code>) or eval() fail as runtime errors instead of being security hazards and indeed that is how Chrome deals with the issue. Is it possible to do the same for addons or any idea if that is in the pipeline as part of web extensions?

I should add that all of the flagged code runs in the context of a popup panel which appears when clicking on the app's icon - so it doesn't need access to any Firefox-specific APIs, except the message port that is used to communicate with the addon's main source file in order to determine the current tab's URL and route messages between the tab's content script and the panel.

https://blog.mozilla.org/addons/2015/11/20/signing-api-now-available/


Pie chart labels

Pie chart labels don't change with the CSS used for bar charts. I don't see any other CSS to change. I have 2 charts on the same page. The bar chart labels changed with the alpha setting on ct-label and color property.
.ct-label {
fill: rgba(0, 0, 0, 0.4);
color: rgba(0, 0, 0, 1);
font-size: 0.75rem;
line-height: 1; }

Pie chart labels are created with a text tag and bar chart labels are created with a span. So you need to use fill with pie chart labels and color with bar chart labels.


GM color constants in RGB instead of BGR

The native GMS colors are defined in the BGR format and thus, should set the optional colourIsGameMakerBGR in scribble_add_colour().
https://github.com/GameMakerDiscord/scribble/blob/60151c89a756edc942b76a24869c88e91adffc40/scripts/scribble_init_start/scribble_init_start.gml#L63-L80

Whooooops. Fixed in 4.5.1.


retrieve executed agreements from e-signature backend

at present, after PDFs are generated, we inject them into echosign, and hope for the best.
it would be good to continue to monitor the progress of the execution, so that we can show what's going on, in the v2 dashboard.
when the documents are executed, we should download them to the "add to folder" path; if "add to folder" is not defined, then save them to the folder where the originally generated PDFs were stored.
also, update the various date fields in the deal sheet which indicate when the agreement was dated. right now there are formulas for =TODAY() which is not quite right.

This is good. In the V2 dashboards, showing the progress, I assume these are high-level events shown. Are there plans for overall lower-level logging of key events along the process? Like a log exposed for internal or 3rd party auditing purposes.

Nm. I'm looking through the code and see it.


Update instructions due to the Bionic to Focal migration

This PR accommodates the recent changes:

https://github.com/bitcoin/bitcoin/pull/21036
https://github.com/bitcoin/bitcoin/pull/21130

ACK f3bccb822af522ca25901886a287aec1ac435073 - I guess we are not needing instructions for building < 0.17

ACK
Could apply the same changes here: https://github.com/bitcoin-core/docs/blob/master/gitian-building/gitian-building-setup-gitian-fedora.md#setting-up-the-gitian-image ?

Could apply the same changes here: https://github.com/bitcoin-core/docs/blob/master/gitian-building/gitian-building-setup-gitian-fedora.md#setting-up-the-gitian-image ?

Done.

ACK 6d330caf2dd7c51d042e806166a79832aa68e174


Create shock tube calculator

Current iteration was sufficient for my final exam, but it is not feature complete


[$500] Remove members out of a workspace are not crossed out in offline mode

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: 1.4.43-3
Reproducible in staging?: y
Reproducible in production?: n
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: @hoangzinh
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1708519808525439
Action Performed:

Go to Settings -> Workspace -> Open any WS -> Members
Invite several members
Turn off the internet connection
Remove WS members

Expected Result:
Removed members should be strike out
Actual Result:
Notice that members are disappearing instead of crossed out.
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/13808d1d-d266-4010-b937-e97999501713
https://github.com/Expensify/App/assets/38435837/cc74e740-ceaf-4a2c-95f6-fc31108f76d7

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016f83f55225363e92
Upwork Job ID:<PHONE_NUMBER>388736000
Last Price Increase: 2024-02-22

I'm seeing something different now, there's a sort of placeholder div that remains. Still a bug nonetheless, going to check the current changes to see what caused this.
https://github.com/Expensify/App/assets/2229301/ee99cfb9-2494-4155-84a3-0571c2d4df4b

Yeah good catch, tested locally and it works well. Feel free to submit a PR plese @mkhutornyi.

Do you see a way to fix that regression? If you do, that's great and let's fix it too, otherwise we can put a comment in the PR to fix it later. Not a huge deal since that PR is not on staging yet.

Works for me.

This is known https://github.com/Expensify/App/pull/37074#issuecomment-1959456354 and will be fixed when we update the remove functionality next

Removing blocker label, since this is known.

@mkhutornyi I've already addressed this issue in my PR regarding removing members on the members page, as stated in the mentioned comment: https://github.com/Expensify/App/pull/37199 - it's still in draft state but it will be published tomorrow.

Should we close seeing as this is going to be fixed here https://github.com/Expensify/App/pull/37074#issuecomment-1959456354 and we don't have a universal solution identified yet - What do you think @parasharrajat ?

Ideally, can we fix this in OfflineWithFeedback? So that function as child would work with strikethrough. I'm concerned similar issue might happen in the future. This is already 2nd bug. 1st bug was #35702

Yeah - I'm working on this right now. Thanks for pointing this out.

done

Thanks @parasharrajat, let's close. I don't think any payment is due as nothing was ever merged but please let me know if anyone disagrees


Nuclear counts loading

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #646.
How did you implement your changes
Update logic to only load nuclear image data if nuclear_counts=True.

Looks good. Will you test out the segmentation notebook, deleting the feature_1 tiffs that are generated by the deepcell call, to make sure there aren't any other hidden places where the nuclear images are assumed to exist

The last few code sections generating and saving the cell table ran without issue!


Add title to HTML

Right now, the HTML does not contain a <title>. I guess we just call it GitZip?

Thanks!


Add Yarn files to .gitignore

Summary:
After running yarn install in the root directory of a React Native project, cached files from Yarn appear when checking Git changes, which can be a bit annoying. Let's ignore them! :)
This commit references #42313.

Changelog:
[GENERAL] [ADDED] - Add Yarn files to .gitignore in react-native
Test Plan:
yarn cached files should ignore after exec pod install

@zhongwuzw How are you configuring Yarn here? Yarn should not generate any v3/v4 files in .yarn/ when it detects a Yarn Classic (1.x) project such as this repository. (Happy to be corrected on if this behaviour has changed.)

@huntie I don't change any configs in my react-native repo, maybe because my yarn version is v3?

Are you instead aiming to ignore .yarn/ files in a newly created React Native project via the Community CLI? (cc @szymonrybczak)

No, I did some work in react-native repo , not a rn project via cli.

@zhongwuzw Thanks for clarifying!
Interesting. Can you share more about how you installed Yarn on your system?
On my Mac, I've used brew install yarn, which is currently 1.22.19 globally, and works with this repo. (Then, what I mentioned was the other way round: If I cd into a project specifying Yarn 4 (containing .yarnrc.yml), Corepack/my Yarn 1.x binary will select that newer Yarn version for that other project.)
I tried out a Corepack install (i.e. following modern Yarn's current install process). If I spin up a fresh GitHub Codespace and try this, it also resolves to Yarn 1.x:

The main thing to note here is this repository requires Yarn 1.x. I'll open a separate PR which helps programatically manage this via Corepack. And perhaps we should also update our contributing docs.

@huntie Sorry I forgot how to install and upgrade my yarn on my Mac :) , and I don't use any yml to specify yarn's version.

@zhongwuzw Okay 🙂 Curious if you could try again with the change from https://github.com/facebook/react-native/pull/47091.

@zhongwuzw Can you also check and remove any .yarnrc[.yml] in the project dir and all parent directories?
Annoyingly, if globally configured (example is contrived), Yarn can self-upgrade itself in the repo, ignoring "packageManager" and even "engines" -_-.

Do you get any similar Yarn output saying it's migrating your project?
Also can you try freshly cloning facebook/react-native in a separate directory?

Either way, we require Yarn 1.x in this repo, so I won't be able to accept this .gitignore change, which will mask an incorrect install.


Errors when trying to get a token price history from polygon

I'm trying to read a price history from polygon and I just get an error, I doesn't matter with contract address I pass.
Tried already by sdk and api directly.
Tried with the other blockchains and is working, expect polygon.
Example:
    "jsonrpc": "2.0",
    "method": "ankr_getTokenPriceHistory",
    "params": {
        "blockchain": "polygon",
        "contractAddress": "0x0000000000000000000000000000000000001010",
        "toTimestamp":<PHONE_NUMBER>,
        "interval": 6000,
        "limit": 8
    },
    "id": 1
}

Result:
{
    "jsonrpc": "2.0",
    "id": 137,
    "error": {
        "code": -32005,
        "message": "Not found: no prices for token=0x0000000000000000000000000000000000001010"
    }
}

Hi, thanks a lot for feedback and sorry for not replying for so long. There's an issue with this special polygon case, to be fixed soon. Until then, please try to call for a native price:
{
    "jsonrpc": "2.0",
    "method": "ankr_getTokenPrice",
    "params": {
        "blockchain": "polygon"
    },
    "id": 1
}


Adding draft actions for each type

Coverage decreased (-0.06%) to 94.539% when pulling c818f9a0ede87db113d99772b7c1e616129de40d on reshma-actions-draft into 9afab3ed9affa70108bb0770d24eb8e55962331c on main.


id counter -> totalInstances

when creating the data object for each instance, a local counter is incremented. This works, if the plugin is initialized on a class selector. but if you initialize the plugin on multiple elements with id selector, each element has the same id (mCSB_idx) created by the plugin.
In my opinion, the following would be an improvement: on plugin init, count the number of elements which have an id like mCSB and set that number to totalInstances.
Would it be possible to change that? Or should I make a Pull-Request?

... so on line 374 in the uncompressed file, I would define the variable as follows:
totalInstances=$('.mCSB_container').length


Improved error message

As i was setting coolify on my system for the first time, I didn't understand why it wasn't working on localhost
apparently, Ubuntu desktop doesn't come with openssh-server installed :)
Changed the error message to include a bit of info on that possible error.

Thanks for the PR. I will close this and instead add a link to the docs where it is explained what and how to set on your server.


theme variant for AccountSelector

          Ohh I see! That's not good because we are changing the whole css template just for a button. We must fix that in ui-kit (with variants like in buttons) and then use it here.

I'll take care of that!
Originally posted by @mazmassa in https://github.com/massalabs/thyra-plugin-wallet/pull/277#discussion_r1211388288

merged


[Refactor] Remove before copy libs of be

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

When you building your project with starrocks_be running, maybe you just forget to stop it, and it will fail with the following error msg
cp: cannot create regular file ‘xxx/starrocks/output/be/lib/starrocks_be’: Text file busy

Solution: remove the old libs before copy just as fe does

run starrocks_fe_unittest

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)


Sending too much personal information in User-Anget. Risky

>>> r.request.headers["User-Agent"]
'python-requests/2.x.x CPython/x.x.x Linux/3.xx.xx-generic'

I think requests is sending too much info.
The bot name `"python-requests" would be okay.
But my Python version which may be vulnerable ?
My Linux kernel version which may be vulnerable?
I consider this a big problem. Many people using requests may not be aware of this and inadvertently sending this info.
Please just change the the User-Agent to something like
"python-requests" or "Requests"

@Logmytech We did. Note our changelog, which says that we updated in 2.8.0, released 7 months ago. =)


The client cannot sync after changing EXTERNAL_LISTEN_HOSTS

Have you read a contributing guide?

[X] I have read CONTRIBUTING.md
[X] I have searched the existing issues and didn't find any that were similar
[x] I have considered creating a pull request with fixes instead of a bug report and want to proceed

Current Behavior
After changing the EXTERNAL_LISTEN_HOSTS in .env.override and restart the docker compose, the client (which is login before) cannot sync and stuck on “preparing”.
re-logout / login not help.
My current workaround:
delete all folder in data Directory and re-sync all data.
Expected Behavior
The client can sync after upload new client.yml and without fetch all data again.
Steps To Reproduce

edit EXTERNAL_LISTEN_HOSTS in .env.override
restart the docker compose make restart
export the new client.yml
upload new client.yml in client
enter the key
the client (which is login before) cannot sync and stuck on “preparing”

Environment
- OS: Docker-in-Docker-DinD v26.0.0 under unRaid 6.9.2
- Version: v4.0.2

Anything else?
No response

can't reproduce
cat .env.override
EXTERNAL_LISTEN_HOSTS="<IP_ADDRESS>"

cat you show me your configuration client file? path "./etc/client.yml"

The port <IP_ADDRESS> is tailscale assign (another container in this Docker-in-Docker-DinD)
Also notice that I do not quote the ip, like this:
EXTERNAL_LISTEN_HOSTS=<IP_ADDRESS>
Could the problem be here?
etc/client.yml
id: 6631310efe1cab0019a5343e
networkId: N4GKRjMCc9Gs7AodExgyZE4SfhVAZyWrcZtVqw5eL3FKdpkd
nodes:
  - addresses:
      - any-sync-node-1:1001
      - quic://any-sync-node-1:1011
      - <IP_ADDRESS>:1001
      - quic://<IP_ADDRESS>:1011
    peerId: 12D3KooWDJaL77GcLtskmE3L4spiqbk4SjPqzK91s32UuBqekNDw
    types:
      - tree
  - addresses:
      - any-sync-node-2:1002
      - quic://any-sync-node-2:1012
      - <IP_ADDRESS>:1002
      - quic://<IP_ADDRESS>:1012
    peerId: 12D3KooWDgKEe3SJt5EzhhCcY4VWSnE81LuzVRNnhPdPvqknRUcZ
    types:
      - tree
  - addresses:
      - any-sync-node-3:1003
      - quic://any-sync-node-3:1013
      - <IP_ADDRESS>:1003
      - quic://<IP_ADDRESS>:1013
    peerId: 12D3KooWMHqUAdqMeRiKrjNvyE2dVvzzYUsn4SYXzcjnruN8QVrz
    types:
      - tree
  - addresses:
      - any-sync-coordinator:1004
      - quic://any-sync-coordinator:1014
      - <IP_ADDRESS>:1004
      - quic://<IP_ADDRESS>:1014
    peerId: 12D3KooWHofxyQLSR3KMbR7oZGigyrCxHvyRHiRCBC3cuTumDRTQ
    types:
      - coordinator
  - addresses:
      - any-sync-filenode:1005
      - quic://any-sync-filenode:1015
      - <IP_ADDRESS>:1005
      - quic://<IP_ADDRESS>:1015
    peerId: 12D3KooWRo4eBuzwqhYE59GvSsjt61HYVFHrNiuGTW6vuTPzq48G
    types:
      - file
  - addresses:
      - any-sync-consensusnode:1006
      - quic://any-sync-consensusnode:1016
      - <IP_ADDRESS>:1006
      - quic://<IP_ADDRESS>:1016
    peerId: 12D3KooWCXq9NPu6wTHegg5oW7VXqtpqk63w5q3obyqgrotyZXEz
    types:
      - consensus

Are you sure that the IP address <IP_ADDRESS> is accessible to the client?
Could you please check the port availability using the command:
telnet <IP_ADDRESS> 1001

He @fb929,
I have the same issue just with another ip adress. I tried telnet myip 1001 with the following output.
telnet myip 1001
Trying myip...
Connected to rockpi-4bplus.tail2e082.ts.net.
Escape character is '^]'.
Connection closed by foreign host.

Hope this helps further :-)
Maybe it's a problem using tailscale at the first place since it depends on the fact that every single client has to be connected to the tailscale client.

We haven't tested the product for compatibility with Tailscale.
First of all, you need to check the availability of all any-sync-* daemons.
Take the list of nodes.addresses from the file etc/client.yml and check your external IP addresses/ports using Telnet.
Secondly, try different clients. Android or iOS may have issues with synchronization over VPN, but the desktop client definitely supports synchronization in a VPN network.

Is it normal that there are IP adresses assigned to the containers. Isn't this a problem since the containers cannot be reached if the IP adress does not correspond to the IP Adress of the VPN Network?

For this, we use port forwarding.
The "Docker Host" system, needs to be in a network that is accessible to the clients.
And IP address "Docker Host" in this network should be added to EXTERNAL_LISTEN_HOSTS.

Ah I see the problem. Maybe... Is it correct that the EXTERNAL_LISTEN_HOST does not correspond to the ip address were the docker instance is running but rather the ip of the client which is trying to sync with the host running the docker instance? Or do I need to add both the ip addresses of the the host running the docker instance and the client trying to sync to the .env.override file? It is kind of weirdly formulated in the wiki.

EXTERNAL_LISTEN_HOST (and EXTERNAL_LISTEN_HOSTS) can be either a hostname or an IP address.
Your clients (desktop, android, ios) will resolve this host to an IP and will attempt to connect with the self-hosted any-sync-* daemons.
In a simple situation, this IP address is present on the docker-host. And the scheme in this case is as follows: client -> EXTERNAL_LISTEN_HOST -> internal docker network -> any-sync-XXX container.
In more complex cases, such as when using a network load balancer or DNAT, this IP may not be present on the docker-host system.
However, in any case, you must ensure that the client can establish a connection to EXTERNAL_LISTEN_HOST:PORT and that it reaches the appropriate any-sync-* daemon.

Ok yeah so i did it right the first time. It is still not working. Is there like a log file or something that I could send you? It still says "No Connection".


communication Modelica ↔ Unity

At UdK Berlin (@UdK-VPT, @jraedler, @nytschgeusen) we work on visualization of building simulation results and for this, we are interested in communication between Modelica and Unity. This communication could be sent via UDP, using the MDD UDP sockets. A similar approach was mentioned in section 3.1 and figure 2 of Yamaura et al. 2016.
Yamaura et al did not mention what serialiser/deserialiser was used, and they did not mention how the UDP socket and serialiser/deserialiser were implemented on Unity side. As Unity supports C#, the approach we currently investigate is to use SWIG to generate a C# (and Python) wrapper for these two packages.
Is this SWIG wrapper something that we should contribute back to you in the long term?
Are there any recommendations what we should or should not do to make integration easier?
I have seen issue #112, we are not yet fully sure whether it is the best approach to bring your serialiser to the Unity-CSharp world or whether there are some more widely used serialisers (maybe XDR) that could be integrated in MDD.
Any feedback welcome.

Since you interested in a communication between Modelica and Unity for visualization, this issue might also be interesting for @tbellmann, who had thoughts whether one should work at a dedicated visualization interface for Modelica (he is one of the authors of the DLR Visualization library, http://www.dlr.de/rmc/rm/Portaldata/52/Resources/Software/MotSimVR/DLR_Visualization_Library.pdf). There are also visualization efforts within the OpenModelica project (@vwaurich, @sjoelund), so maybe the Modelica community could benefit from the definition and development of a common interface that can be used (and extended) by different visualization solutions.
The serial packager in MDD was developed having a rather simple communication use-case in mind: periodic sending or receiving of fixed size packages which map to fixed size input and output signals in the Modelica model. This fits well for certain application.
There have been discussions to integrate more flexible approaches.  However, there are currently no concrete develop efforts in this direction. Having an additional serialiser, which can be used optionally (instead of the current one), would be fine.  I expect that there is not a single solution which is perfect for all use-cases, so it would be nice if we could provide some choice.
The initial idea was that one should be able to use different serializers with the same communication block. However, if one reuses the interface of the communication blocks (Blocks.Interfaces.PackageIn and PackageOut)  the design of an alternative serialiser is subject to the constraints of these interfaces. This constrains the design space severely. To escape that constraints, one might need clones of the communication blocks with a different interface. This is not nice, but possibly a necessary trade-off. On the other hand, if one only uses functions (forgets about the things below subpackage Blocks), there are less constraints for adding a new serialiser.
Whatever you decide for, it would be great if you could bring your solution to the attention of the MDD project. Maybe we can integrate it directly into MDD, or maybe we can simply reference to some project of yours (e.g., the SWIG wrappers could be an own project that we reference in the documentation, or they could be shipped with the MDD library). We would need to figure out what works best, once there is something available.
In any case, thanks for the information and for considering to contribute back to MDD.

Thanks for that comprehensive reply & new input.

Yes, initially the ExternalDevices library included both aspects (visualization and device drivers). These two aspects were separated, leading to two internal DLR libraries. Later, one of these libraries evolved to the commercial DLR Visualization library, the other to the open-source MDD library (first open-source release was in August, 2012).

Our code is here:
https://github.com/jraedler/Modelica_DeviceDrivers/tree/swig
It works for us, so we can exchange data to and from Modelica, Python and CSharp.
Before sending a Pull Request, we should probably do some cleanup.

Very nice :-)


Exporting to Excel gives an error when opening the file

When downloading to excel from the module, I get an error when opening the file. If I choose to accept the content of the file is correct. Can I change/update the script to remove the error message?

If I remember correctly we haven't found a way to work around this issue. The contents of the file is a HTML table, but Excel interprets it correctly nonetheless.

Its quite involved to generate real Excel data.  It would probably be best to integrate with a 3rd party javascript lib, e.g. https://www.travismclarke.com/tableexport/ (that's just one I found from google)  Not sure how difficult it would be to integrate.

Here's a proof of concept of using TableExport: http://jsfiddle.net/highcharts/hr7jLwda/
Does the produced Excel file look good to you?
If this is working, we can consider providing a hook for it in the export-csv plugin (which will soon become an integrated Highcharts module). We can plug it into our own file download functionality so we won't depend on FileSaver.js.

I'm still getting the error, BUT I am using a newer version of Excel which is looking for xlsx type.  it should work for older versions of Excel that are expecting xls.
I have updated the example here for xlsx: http://jsfiddle.net/ahdavis/hr7jLwda/3/
I added the xlsx.core.js include at the top and changed the format to xlsx on line 9.  However, highcharts is still sending an extension of xls instead of xlsx.  I think if this could be changed, then it would work for xlsx, and then you would need an additional Download XLSX option.

Thanks for contributing - the path to core.xlsx.js was wrong, but once fixing that, the extension also became correct. I added a separate menu item for XLSX: http://jsfiddle.net/highcharts/hr7jLwda/

Great, xlsx works for me now!  Thank you!


Lazy-load server data with ec2 instance info

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
This PR changes the way data is loaded for the ServerSettings admin component. Whenever a panel showing the details about a server is expanded, the UI will now make a call to a new endpoint in datatools-server to load the full information about the server which could include data about ec2 instances. This is done to avoid making unnecessary calls to AWS until that data is actually needed.
This PR requires using the code in https://github.com/ibi-group/datatools-server/pull/343.

Codecov Report

Merging #615 into dev will decrease coverage by 27.79%.
The diff coverage is 5.55%.

@@             Coverage Diff             @@
##              dev     #615       +/-   ##
===========================================
- Coverage   43.45%   15.66%   -27.80%
===========================================
  Files         323      323
  Lines       17589    16439     -1150
  Branches     5363     4992      -371
===========================================
- Hits         7644     2575     -5069
- Misses       8670    11845     +3175
- Partials     1275     2019      +744

Flag
Coverage Δ

#end_to_end_tests
?

#unit_tests
15.66% <5.55%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lib/admin/components/ServerSettings.js
0.00% <0.00%> (-32.49%)
:arrow_down:

lib/admin/reducers/servers.js
37.50% <0.00%> (-27.21%)
:arrow_down:

lib/manager/components/CollapsiblePanel.js
33.33% <0.00%> (-58.67%)
:arrow_down:

lib/admin/actions/admin.js
18.55% <14.28%> (-40.79%)
:arrow_down:

...nager/components/validation/ServicePerModeChart.js
0.00% <0.00%> (-85.90%)
:arrow_down:

lib/manager/components/validation/TripsChart.js
0.00% <0.00%> (-80.40%)
:arrow_down:

lib/common/util/map-keys.js
25.00% <0.00%> (-75.00%)
:arrow_down:

lib/manager/components/HomeProjectDropdown.js
0.00% <0.00%> (-72.73%)
:arrow_down:

lib/editor/util/objects.js
14.89% <0.00%> (-72.35%)
:arrow_down:

lib/editor/components/map/TextPath.js
0.00% <0.00%> (-70.00%)
:arrow_down:

... and 270 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 71eb526...9e02775. Read the comment docs.

Good catch. I just pushed a change that addresses this.


unknown video.

Hi all,
I was trying to see what formats the video has and I got this :-
$ youtube-dl --verbose -F S9HT1mab2I4
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '-F', 'S9HT1mab2I4']
[debug] Encodings: locale 'UTF-8', fs 'UTF-8', out 'UTF-8', pref: 'UTF-8'
[debug] youtube-dl version 2014.02.17
[debug] Python version 2.7.6 - Linux-3.13-1-amd64-x86_64-with-debian-jessie-sid
[debug] Proxy map: {}
[youtube] Setting language
[youtube] S9HT1mab2I4: Downloading webpage
[youtube] S9HT1mab2I4: Downloading video info webpage
[youtube] S9HT1mab2I4: Extracting video information
[youtube] RTMP download detected
[info] Available formats for S9HT1mab2I4:
format code extension resolution  note
_rtmp       unknown_videounknown

Any help would be appreciated.

rtmpe downloads are not supported, see https://github.com/rg3/youtube-dl/issues/343 for more information.


fix: typo in GHAS4AZDO dependency-scanning

Fixing a "typo" in the troubleshooting part of the GitHub Advanced Security (for Azure Devops) Dependency Scanning docs page.

#sign-off


Enable DebugMode

This PR fixes #385 by enabling the DebugMode even when the app is compiled as RELEASE and adding some documentation for it.
Also the Hotkey for enabling/disabling the DebugMode is manageable by the user.
Things to discuss before merging

[ ] Should all the text be localized? (e.g. ViewPort, PlacedObjects)
[ ] Is the documentation correct/complete?

The translations are not necessary IMHO, but since you already added some translations, you might as well add translations to debug texts as well.

Because it is intended to use for troubleshooting, I omitted the extra translations.


Long Article Link lists should be truncated

As illustrated by the large number of links added by a recent user of the site, both the view at article/show and the article/edit view would benefit from truncating theee long lists with a link to 'view all lnks'
How does wikipedia deal with this?
How do we decide which links to show above the fold?
How do we display to full list?

Wikipedia articles have footnotes, which are not quite the same as links
here, since the links may not relate to any of the text sections on the
article.
I think that a gallery of links may serve us better than just a plain list
at the bottom. In addition, I think that we should show when the link has
been added to the article.
On Sun, Feb 28, 2016, 1:13 PM campgurus<EMAIL_ADDRESS>wrote:

As illustrated by the large number of links added by a recent user of the
site, both the view at article/show and the article/edit view would benefit
from truncating theee long lists with a link to 'view all lnks'
How does wikipedia deal with this?
How do we decide which links to show above the fold?
How do we display to full list?
—
Reply to this email directly or view it on GitHub
https://github.com/BOWiki/BOW/issues/365.

so if there are less than 10 links, how does "show more" and "show less" pan out? It doesn't seem like that link would be needed in that place.

yes.  I will fix that first and then see if we still have a problem.

This is on production. Closing.


IOS Only: RNVectorIconsManager not available, did you add the library to your project and link with libRNVectorIcons.a? Try running react-native link react-native-vector-icons and recompiling.

This happens on IOS when calling getImageSource or getImageSourceSync, would like to call those two without running react-native link because this command is marked as deprecated and will be removed soon. I am using react-native 0.68.2 and react-native-vector-icons 9.2.0.

Same here! Running React-Native 0.69.4

It has happened to me on RN 0.69.5. I've solved it by removing react-native.config.js which was created while following the steps for iOS and running pod-install (npx pod-install) again.

same problem.
"react-native": "0.69.5",
"react-native-vector-icons": "^9.2.0",

Facing same problem and without following changes in react-native.config.js I'm getting duplicate file error during build.
module.exports = {
   ...
  dependencies: {
    'react-native-vector-icons': {
      platforms: {
        ios: null,
      },
    },
  },
};

It has happened to me on RN 0.69.5. I've solved it by removing react-native.config.js which was created while following the steps for iOS and running pod-install (npx pod-install) again.

yarn add<EMAIL_ADDRESS>
You can try this version.

@kanzitelli Thx u

Doesn't removing the config file lines then mean the fonts end up in the bundle (which you don't want)?

My code was calling the loadFont() method
import MaterialCommunityIcons from "react-native-vector-icons/MaterialCommunityIcons" MaterialCommunityIcons.loadFont()
This turned out to be unnecessary for our use case, so the fix was to delete that 2nd line
import MaterialCommunityIcons from "react-native-vector-icons/MaterialCommunityIcons"

Is it fixed?

We’ve just released a major new version with significant changes and improvements. If you’re still experiencing this issue after upgrading, please open a new issue with updated details. We're closing all the very old issues to get on top of things. Thanks for your patience!


Streaming support is genious. Any xml writing library doing the same?

Just curious: I'm using exceljs to read a csv and write it to a xlsx-stream with nodejs. Takes less than 20MB for 500k rows.
I need XML-Support too. But I can't find any xml writing library that supports committing single elements and writing them to disk (which leads to over 2GB memory consumption).
Does the author of exceljs or anyone else know a xml-library that has the same genious streaming support as exceljs?

Hey,
Out of question.. May i get some example to write xls/xlsx file on nodejs? I was trying to do that with documentation but i just got zero byte files.

You mean a simple example with File I/O?
var Excel = require('exceljs');
var workbook = new Excel.Workbook();

var sheet = workbook.addWorksheet('My Sheet');
var cell = sheet.getCell('A1');
cell.value = 'Hello World';

workbook.xlsx.writeFile('testWorkbook.xlsx')
    .then(function() {
        console.log('File written');
    });

Actually my problem was little different. Now i have fixed. I just need to use "tempfile" library to create a file first, write on it after.. Thanks


Minor, add MySQL dependency for DebugTomcat

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Codecov Report

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

@@            Coverage Diff            @@
##             master     #682   +/-   ##
=========================================
  Coverage     25.73%   25.73%
  Complexity     6010     6010
=========================================
  Files          1384     1384
  Lines         82384    82384
  Branches      11551    11551
=========================================
  Hits          21202    21202
+ Misses        59139    59138    -1
- Partials       2043     2044    +1

Impacted Files
Coverage Δ
Complexity Δ

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
69.6% <0%> (-0.61%)
7% <0%> (ø)

...he/kylin/job/impl/threadpool/DefaultScheduler.java
76.74% <0%> (+2.32%)
12% <0%> (ø)
: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 30ff09f...50d89ed. Read the comment docs.

Pull Request Test Coverage Report for Build 4598

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.001%) to 28.22%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
1
78.12%

Totals

Change from base Build 4597:
0.001%

Covered Lines:
23249

Relevant Lines:
82384

💛  - Coveralls


prometheus_2: 2.8.1 -> 2.9.1

Motivation for this change
https://github.com/prometheus/prometheus/releases
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I am using 2.9.1 and it seems to work fine.

Closed in favor of https://github.com/NixOS/nixpkgs/pull/61006


[Question] Adding a publisher on the beginning of every method.

Would it make sense to add a publisher on the beginning of every method, no matter if it's external or not?
Here's the general idea:
global vars:
   SomeInt: Integer

procedure DoSomething(var Variable1, Variable2) output : Boolean
{
   OnBeforeDoSomething(var Variable1, Variable2, SomeInt, var output , IsHandled);
   IF NOT IsHandled THEN BEGIN
      IF SomeInt > 5 THEN BEGIN
         Variable1.Field := 'Value';
         output := true;
      END;
   END;
   EXIT(output);
}

This would allow us to integrate any of our custom processes into existing ones seamlessly.
Sometimes I'll want the DoSomething procedure to do nothing, sometimes I'll want to have my own check against the value of SomeInt. Whatever a developer could thing of as far as DoSomething goes they could implement. This isn't to say that this would be a replacement for other publishers throughout the procedure, but I believe it is important to be able to override a procedure - just like you can in most other programming languages.

We have no plans to follow this way.


After updating, it failed to load plugin

Sorry as I commit many issues :P I'm using the insider desktop version of Obsidian, but I couldn't install the plugin after upgrading it.

Sanity checks

[v ] I am using the latest version of the plugin
[v ] I am using the latest version of Obsidian
[ v] I have read the README

Diagnositics

Component
Version

Obsidian
.......

AnkiBridge
.......

Anki
.......

AnkiConnect
.......

Description
A clear and concise description of what the bug is.
Reproduce

Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Whoops. This is definitely my bad. I changed something around in the build system. I can reproduce and will aim to fix this asap.

This should be fixed in version 0.6.14, which is now available.


WIP feature(examples): Linearlite multiple projects

To do:

[x] UI for new projects (currently its using a Browser prompt input)
[x] Show/hide the sync button when a project is synced (Requires sync status api)
[ ] Unsync project button (Requires an unsung api)

@samwillis updated the project to be ready for the unsub and status APIs
I also updated the load-data script to also use a projects.json file and assign each issue to a random project - I know that the issues don't match the project names but it makes it easier to get a good feel of multi-project syncing

I also updated the load-data script to also use a projects.json file and assign each issue to a random project - I know that the issues don't match the project names but it makes it easier to get a good feel of multi-project syncing

Sounds good for now. We should make a script that produces a dump from a public GutHub using its api. Then we can have multiple "real" projects in there.

@samwillis @kevin-dp could you have a quick look at this PR so we merge Linearlite multiproject? I want to deploy it with 0.12.1 but I'd rather merge the multiproject work first and update the version as soon as we release


fix(curriculum): Add default white bg to tree DS image for better view

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

In the Data Structures: Add a New Element to a Binary Search Tree lesson, there is this image:

But when the user's NIGHT MODE is ON, this image/png makes it hard to see/read the image and it looks like this.

So I've just added a background property for the default transparent image with #f5f6f7, and it looks way better:

Also, this doesn't have any affect on users learning with LIGHT THEME as the bg is same.

@Sky020 any updates?

@cdadityang Thank you for your contribution to the page! :+1:
We're happy to accept these changes, and look forward to future contributions. :memo:

@ojeytonwilliams Cool. Thanks


Header updates - breaking changes

Description
Bringing together a series of breaking changes from the following PRs:

[x] https://github.com/nhsuk/nhsuk-frontend/pull/986
[x] https://github.com/nhsuk/nhsuk-frontend/pull/996
[x] https://github.com/nhsuk/nhsuk-frontend/pull/1047
[x] https://github.com/nhsuk/nhsuk-frontend/pull/1059
[ ] https://github.com/nhsuk/nhsuk-frontend/pull/1067
[ ] https://github.com/nhsuk/nhsuk-frontend/pull/1063
[ ] https://github.com/nhsuk/nhsuk-frontend/pull/1073

Checklist

[ ] Tested against our testing policy (Resolution, Browser & Accessibility)
[ ] Follows our coding standards and style guide
[ ] CHANGELOG entry

@paulrobertlloyd found an issue with your logo SVG change (https://github.com/nhsuk/nhsuk-frontend/pull/1047), when printing the NHS logo no longer shows. Not looked into the code but I assume it's just missing a rule to change it's colour from white to blue on print?

@paulrobertlloyd could we avoid force pushing to this branch for a bit? Makes it harder to maintain the other pull requests into this branch.

Potential new issue to include in the PR: Making the header work without JS enabled and progressively enhancing the responsive layout via JS.
Currently the responsive layout without JS is a bit broken, with the nav links overflowing rather than wrapping.


Add some properties for WebWindow.

Working items

[x] Clean up code.
[x] Size, Width, Height
[x] Location, Left, Top
[x] Topmost
[x] Icon from file
[x] Screen size & DPI
[ ] Maximizable & Minimizable
[x] Resizable
[x] Resize & move event
[x] Register WebWindow singleton

Related issues
#35

@SteveSandersonMS Using auto charset I could remove the multi-byte<=>wide-char string convert methods, and reduce some allocations. I think it belongs to the cleanup, which is my first working item. Please tell me if you want a separate pull request.

This is looking great! Thanks for working on this.
I’m fine with charset cleanup either being in the same PR or a different one.

@SteveSandersonMS I've added the features planned above. However, I don't have a Mac with me now. Could you help me test it? The test page has been added to testassets/MyBlazorApp.
Also there's a remaining issue I haven't figure out the reason: on Windows, if you call StateHasChanged in the handler of SizeChanged or LocationChanged, the window will block.

My Edge Dev updated automatically and it deprecated an API the project uses. I had to use newer API to test it.

This is looking great, @Berrysoft! If you could let me know when you consider this done and ready to merge, I'll be happy to proceed with it.

@SteveSandersonMS I consider it done now. I will keep investigating better work around for the thread problem, but I don't think it is easy to solve at once.

ping @SteveSandersonMS

Thanks, will take a look!

Thanks again @Berrysoft for this! It's looking great!
I've tried it out and encountered a few issues on macOS:

In the MyBlazorApp app, the "Window size" and "Window location" values don't update when you drag the window around

Also the "Window location" values show as negative numbers, though maybe that aspect of it isn't a problem

Also in this app, trying to modify the "Window size" or "Window position" values (e.g., by clicking the "up" arrow next to the widths or heights) causes strange behavior:

The window's left-hand edge moves to the right when the window width is increased, and its bottom moves upwards when the window height is increased. It seems like some size/position calculations are not being done quite correctly.
When changing size or position by editing the text boxes, the console output produces the message NSWindow drag regions should only be invalidated on the Main Thread! This will throw an exception in the future (see below for full stack trace)

I've added some fixes for this which I will include when merging this PR.

Merged. Thanks again @Berrysoft!

That's OK, and thank you for your help and fixes!


fix(SharedMethods): change FindEvenInactive[Component(s)/GameObject] to search in all loaded scenes - fixes #1580

Placing the SDKManger in any other then the active scene led to multiple errors.
FindEvenInactiveComponent, FindEvenInactiveComponents and FindEvenInactiveGameObject only searched in the currently active scene, even when searching from another scene. Now it will search in all currently loaded scenes.

There were the following issues with your Pull Request

Commit: f0465cd3ce8ffac979848eb95d117f76b89f746d
Your commit message body contains a line that is longer than 72 characters
Your subject line is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

There were the following issues with your Pull Request

Commit: f0465cd3ce8ffac979848eb95d117f76b89f746d
Your commit message body contains a line that is longer than 72 characters
Your subject line is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com