Drush is slow on my drupalvm

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 1.9.4
VirtualBox 5.1.22r115126
ansible <IP_ADDRESS>
config file =
configured module search path = Default w/o overrides
python version = 2.7.10 (default, Feb  7 2017, 00:08:15) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.34)]
2015 Macbook Pro i7 2.2GHz
16Gb ram
256gb flash harddrive
Your OS

macOS Sierra

Full console output

Summary
When I run drush cr it takes 58 seconds to run. I have allocated 4gb ram to my virtualbox for this drupalvm.
My database file I did a dump of is 124mb, if that helps.
Thank you for your help.

@hmartens - Can you post your config.yml file (if you have one set up)? I'm most interested in how you have your synced folders configured.
Also, if you open Activity Monitor on your Mac, and run drush cr, is there a lot of CPU activity?
Finally, assuming you have this same website running elsewhere, is drush cr fast in that environment?

Hi Jeff, here's my config.yml. I've removed some of the sensitive info in there.

Acquia Cloud uses Ubuntu 12.04 LTS.
vagrant_user: vagrant
vagrant_synced_folder_default_type: nfs
If you need to run multiple instances of Drupal VM, set a unique hostname,
machine name, and IP address for each instance.
vagrant_hostname: acsf-mysite-vm.dev
vagrant_machine_name: acsf-mysite-vm.dev
vagrant_ip: 192.myip
Memory and CPU to use for this VM.
vagrant_memory: 2048
vagrant_cpus: 2
vagrant_synced_folders:
The first synced folder will be used for the default Drupal installation, if
any of the build_* settings are 'true'. By default the folder is set to
the drupal-vm folder.

local_path: ../../mysite
destination: /var/www/drupalvm
type: nfs
create: true

build_makefile: false
build_composer: false
build_composer_project: false
Set this to 'false' if you don't need to install drupal (using the drupal_*
settings below), but instead copy down a database (e.g. using drush sql-sync).
install_site: false
drupal_core_path: "{{ drupal_composer_install_dir }}"
drupal_mysql_user: mysite_local
drupal_mysql_password: mysite_local
drupal_mysql_database: mysite_new
drupal_core_path: "/var/www/drupalvm/docroot"
drupal_domain: "mydomain-vm.dev"
apache_vhosts:

servername: "{{ drupal_domain }}"
serveralias: "www.{{ drupal_domain }}"
documentroot: "{{ drupal_core_path }}"
extra_parameters: |
ProxyPassMatch ^/(..php(/.)?)$ "fcgi://<IP_ADDRESS>:9000{{ drupal_core_path }}"

servername: "mysite.dd"
documentroot: "{{ drupal_core_path }}"
extra_parameters: |
ProxyPassMatch ^/(..php(/.)?)$ "fcgi://<IP_ADDRESS>:9000{{ drupal_core_path }}"

servername: "adminer.{{ vagrant_hostname }}"
documentroot: "{{ adminer_install_dir }}"
extra_parameters: |
ProxyPassMatch ^/(..php(/.)?)$ "fcgi://<IP_ADDRESS>:9000{{ adminer_install_dir }}"

servername: "xhprof.{{ vagrant_hostname }}"
documentroot: "{{ php_xhprof_html_dir }}"
extra_parameters: |
ProxyPassMatch ^/(..php(/.)?)$ "fcgi://<IP_ADDRESS>:9000{{ php_xhprof_html_dir }}"

servername: "pimpmylog.{{ vagrant_hostname }}"
documentroot: "{{ pimpmylog_install_dir }}"
extra_parameters: |
ProxyPassMatch ^/(..php(/.)?)$ "fcgi://<IP_ADDRESS>:9000{{ pimpmylog_install_dir }}"

servername: "{{ vagrant_ip }}"
serveralias: "dashboard.{{ vagrant_hostname }}"
documentroot: "{{ dashboard_install_dir }}"
extra_parameters: |
ProxyPassMatch ^/(..php(/.)?)$ "fcgi://<IP_ADDRESS>:9000{{ dashboard_install_dir }}"
DirectoryIndex index.html

Acquia Cloud Search defaults to Solr 4.x.
installed_extras:

solr
drupalconsole
xdebug

php_xdebug_default_enable: 1
php_xdebug_idekey: PHPSTORM
I have a process called VBoxHeadless that goes to 85% CPU when I run drush cr
We have this same website running on Acquia and it is much faster there.
Thank you for your help.

XDebug being enabled is definitely causing it to be slower.
Try disabling it and running vagrant provision again.
php_xdebug_default_enable: 0

Hi Jeff, thanks for the superfast response :) I disabled the xdebug and and started vagrant again and then ran vagrant provision. At the end I get an error
fatal: [acsf-mysite-vm.dev]: FAILED! => {"failed": true, "msg": "The conditional check ''{{ item }}' not in '{{ solr_cores_current.content }}'' failed. The error was: Invalid conditional detected: EOL while scanning string literal (<unknown>, line 1)\n\nThe error appears to have been in '/Users/myusr/Workspace/dev/mysite/mysite-vm/drupalvm/provisioning/roles/geerlingguy.solr/tasks/cores.yml': line 8, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Ensure Solr conf directories exist.\n  ^ here\n"}
Should I ignore this as it's just solr that is not on my machine?
Drush cr is much quicker, it's down to 37 seconds ;)
Is there any other way to make it even faster? I'm also runing twig debug. Does this make it slower?
Then I also found that when I click on any link that drupa hasn't built the cache for it "polls" very long before going to the page. On Chrome the progress circle will turn grey for just over 40seconds and then it will turn blue and go to the page.

You could check /var/log/mysql/mysql-slow.log if there were any especially slow database queries

And @oxyc is @oxyc, not Jeff ;)

Thanks Oxyc, I appreciate all your help..both you and Jeff ;)

Happy to help. 40s for a page load definitely sounds excessive. You can probably track down what's causing the holdup using xhprof/tideways or the mysql-slow.log I mentioned. Alternatively the devel module can help with tracking down slow queries if that's the issue.
If something sticks out, let us know and maybe there's a config option that needs tuning.

Closing this issue for now, but please open it back up if you find something that needs further discussion!

agin i am getting slow vagarant drual vm


Error on SD health check (SS)

https://github.com/bithyve/bitcoin-keeper/assets/62699947/eb3da910-78ef-4406-a577-9b4d9c29b8dd

Same on Jade Health check

Verified this issue on keeper app v 1.1.11(297)


Fix missing comma

Prior to this commit temperature method caused a stack trace in the new
gem version due to a missing comma

Sorry I just noticed this PR! Good catch, and thanks for the contribution.


no devices are seen

Describe the bug
title, cant explain it more
To Reproduce

play music on another device
shift+D
observe

Expected behavior
i can actually see devices, including #141
Log and backtrace
none interesting
Screenshots

Environment

Application version: 0.12.0
Application features: alsa-backend,media-control,notify

Additional context
sorry for not being comprehensive

#142 and #141 should be resolved already with spotify connect feature. If u used spotify-tui previously, you should be familiar with the process of creating a Spotify app. Unlike spotify-tui, spotify only needs the client's ID, not your secret.

bru iom stupid

it would be nice if the spotify applications site tab thing was linked in the same area in the readme

https://developer.spotify.com/dashboard/applications i think its this

Seeming to have an issue similar to this, but I have setup spotify connect with no help - the device is connected through my account, not my local network, and web client sees it but spotify_player can only see its own internal device


To change color of footer

Screenshots
If applicable, add screenshots to help explain your problem.
To change color of footer in the login page according to the light and dark mode .

I would like to contribute to this issue

@ankitapuri there is also one more issue text is not in center

and overlapping in mobie view

I would like to work on this issue and I will also resolved these two problems under this issue.
@ankitapuri kindly assign this issue to me .

@ankitapuri there is also one more issue text is not in center

and overlapping in mobie view

I would like to work on this issue and I will also resolved these two problems under this issue.
@ankitapuri kindly assign this issue to me .

I can create a separate issue for the same . but its first cum first serve so @SnehaVeerakumar shall be given a chance .

@ankitapuri should i make a separate issue for this ??

@ankitapuri should i make a separate issue for this ??

sure

I need to change the color of "made with <3" to white when the dark mode is turned on. Am I correct ? @ankitapuri

I need to change the color of "made with <3" to white when the dark mode is turned on. Am I correct ? @ankitapuri

yes

@ankitapuri  yeah done ! Check my PR please.


 Install/Update dependencies? Y/n: n
[?] Please enter the name of your wireless interface (for the AP): wlan0
[?] Please enter the name of your internet connected interface: eth0
[I] Killing wpa_supplicant on wlan0...
[?] Use SSLSTRIP 2.0? Y/n: n
[?] Capture unencrypted images with DRIFTNET? (ONLY WORKS WITH GUI) y/N: N
[I] Backing up /etc/dnsmasq.conf...
[I] Creating new /etc/dnsmasq.conf...
[I] Deleting old config file...
[I] Writing config file...
[?] Create new HOSTAPD config file at '/etc/hostapd/hostapd.conf' Y/n: Y
[?] Please enter the SSID for the AP: derp
[?] Please enter the channel for the AP: 13
[?] Enable WPA2 encryption? y/N: N
[I] Deleting old config file...
[I] Writing config file...
[I] Configuring AP interface...
[I] Applying iptables rules...
[?] Set speed limit for the clients? Y/n: n
[I] Skipping...
[?] Start WIRESHARK on wlan0? (ONLY WORKS WITH GUI) y/N: N
[?] Capture packets to .pcap with TSHARK? (no gui needed) Y/n: Y
[?] Spoof DNS? Y/n: Y
[?] How many domains do you want to spoof?: 3
[I] Backing up /etc/dnsmasq.conf...
[?] 1. domain to spoof (no need for 'www.'): facebook.com
[?] Fake IP for domain 'facebook.com': <IP_ADDRESS>:8001
[?] 2. domain to spoof (no need for 'www.'): google.com
[?] Fake IP for domain 'google.com': <IP_ADDRESS>:8002
[?] 3. domain to spoof (no need for 'www.'): reddit.com
[?] Fake IP for domain 'reddit.com': <IP_ADDRESS>:8003
[I] Starting DNSMASQ server...

dnsmasq: bad address at line 10 of /etc/dnsmasq.conf
[?] Capture traffic? Y/n: Y
[?] Capture HTTPS traffic too? (Need to install certificate on device) y/N: N
[I] Starting AP on wlan0 in screen terminal...
[I] Starting TSHARK...

Starting MITMPROXY in 5 seconds... (press q and y to exit)

usage: mitmproxy [options]
mitmproxy: error: unrecognized arguments: --follow

[!] Stopping...
No screen session found.
No screen session found.
[I] Stopping DNSMASQ server...
[I] Restoring old dnsmasq.cfg...
[I] Deleting old '/etc/dnsmasq.hosts' file...
[I] Removeing speed limit from wlan0...
[I] Flushing iptables rules...
[I] Traffic have been saved to the file '/home/pi/mitmAP/logs/mitmap-proxy.mitmproxy' and to file '/home/pi/mitmAP/logs/mitmap-tshark.pcap'. View the '.mitmap' file later by 'mitmproxy -r [file]'.
[I] Traffic have been saved to the file '/home/pi/mitmAP/logs/mitmap-tshark.pcap'.

[!] WARNING: If you want to use the AP interface normally, please reboot the PI!

[I] mitmAP stopped.

The --follow argument caused problems before as well, so I removed it now.
Try it again now please, and tell me if it works.
(By the way, it's not possible to add ports to DNS spoofing)


Those logs of otel-collector was always [severity: "ERROR"]

I used the below config to deploy OpenTelemetryCollector CRD.  But, I checked the logs of otel-collector  in GKE workloads page, then logs' severity was "ERROR", like the printscreen png.
Would I miss any config of OpenTelemetryCollector CRD ? Thank you.
apiVersion: opentelemetry.io/v1alpha1
kind: OpenTelemetryCollector
metadata:
  name: otel
spec:
  image: otel/opentelemetry-collector-contrib:latest
  config: |
    receivers:
      otlp:
        protocols:
          grpc:
          http:
    processors:
      memory_limiter:
        check_interval: 1s
        limit_percentage: 65
        spike_limit_percentage: 20

    exporters:
      googlecloud:
      logging:
        loglevel: debug

    service:
      pipelines:
        traces:
          receivers: [otlp]
          processors: [memory_limiter]
          exporters: [logging, googlecloud]
        metrics:
          receivers: [otlp]
          processors: [memory_limiter]
          exporters: [googlecloud]
        logs:
          receivers: [otlp]
          processors: [memory_limiter]
          exporters: [googlecloud]

It's the issue of Opentelemetry Collector, not this sample project.


Improve syntax for comments inside hex strings

Before and after the change:

Let me know if you prefer not to highlight the { and }. If so, I can add a region before that match.


Use different icons for "Maximize" and "Restore"

Is your feature request related to a problem? Please describe.
When the Preferences > Advanced > Hide App Menu (Windows & Linux Only) is selected, the menu bar will not really hide (I know it is a bug and will be fixed in the next release), but the customized buttons for minimizing, maximizing and closing window appear (as is shown in the following screenshot).

The buttons work perfectly, but the button for "Maximize" and "Restore" (i.e. the middle of the three buttons) does not change its appearance when the window changes between maximized window and a window of normal size.
Describe the solution you'd like
Given that the button used currently causes user to think of "Maximize", I think it may be good to use another icon for the action "Restore", just as VS Code, another app based on Electron, does:

Describe alternatives you've considered
None. (I think it is easy to use another icon for the action "Restore".)
Additional context
None.
PS: I think it will be better to provide a Chinese feature request template, just like bug reports.

Maintaining the maximized state of the window is a bit cumbersome, and the state is easily inconsistent with the actual situation, such as manually adjusting the size of the form.
I don't want to implement this feature now.
Providing the Chinese version of the bug report issue template is a helpless, : (

That's right. Thank you anyway for your effort devoted in the development of Motrix 🙂


Fix: Keycloak Realm Configuration

Added MICRO_OAUTH2_%s_REALM key to config in order to be able to set Realm name;

Fixed in #2


Add example of typeahead functionality

🙋 Documentation Request
The docs say that the typeahead feature is supported. It would be nice to see an example of that.
🧢 Your Company/Team
No response

In the docs, typeahead is referring to typing typing Pa and having focus move to the Panda item in the collection:

You may be thinking of the feature of having an item auto-filled into the a Combobox/Searchautocomplete as you type. There is an issue for that at https://github.com/adobe/react-spectrum/issues/878. I believe it wasn't included in Combobox's initial release due to issues with screen readers.


Confusing part about 'Sydney, NSW'

This paragraph is confusing as a customer because it's unclear if this is meant seriously or if the reason for this is known but not part of this guide as it would be irrelevant:

Interestingly, if “Sydney, NSW” has a comma, the DLP policy is not triggered. I have no idea why a comma makes any difference here, nor why other cities and states in Australia aren't included in the keywords for the Australian driver's license information type, but there you go. So, what can we do about it? There's a couple of options.

If the reason is unknown, this should be investigated with the product team. Same with why no other cities and states are included.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ce36344d-67bf-68cb-5004-9ac6f27ebf51
Version Independent ID: 89a7d7e7-c643-eb06-d844-b97b80633b68
Content: Create, test, and tune a DLP policy - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/create-test-tune-dlp-policy.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@GiantCrocodile  - Thank you for your comment. I have removed this text from this topic.

So has this issue been fixed @chrfox or is it still live in the product?


Cmake error (Note: Recompile with -Xlint:unchecked for details.)

Hello,
I'm following the notes for building TDLIB for Java & windows.
On the last cmake command (after, cmake -A x64 -DCMAKE_INSTALL_PREFIX:PATH=../../../tdlib -DCMAKE_TOOLCHAIN_FILE:FILEPATH=../../../vcpkg/scripts/buildsystems/vcpkg.cmake -DTd_DIR:PATH=$(Resolve-Path ../td/lib/cmake/Td) .. )
cmake --build . --target install --config Release
I get one error and i'd like to ask how to fix it or if I can continue anyway
PS D:\TDLib\td\example\java\build> cmake --build . --target install --config Release
MSBuild version 17.7.2+d6990bcfa for .NET Framework
1>Checking Build System
Generating Java TDLib API source files
Write file D:/TDLib/td/example/java/org/drinkless/tdlib/TdApi.java
Building Custom Rule D:/TDLib/td/example/java/CMakeLists.txt
Building Java code
D:\TDLib\td\example\java\org\drinkless\tdlib\example\Example.java:306: error: incompatible types: Ok cannot be converted to Error
if (Client.execute(new TdApi.SetLogStream(new TdApi.LogStreamFile("tdlib.log", 1 << 27, false))) instanceof TdApi.Error) {
^
Note: D:\TDLib\td\example\java\org\drinkless\tdlib\Client.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(249,5): error MSB8066: Custom build for 'D:\TDLib\td\example\java\build\CMakeFiles\c245905e49752fb61022f5298b7f86ab\build_java.rule;D:\TDLib\td\example\java\CMa
keLists.txt' exited with code 1. [D:\TDLib\td\example\java\build\build_java.vcxproj]
Generating Javadoc documentation
Loading source files for package org.drinkless.tdlib...
Constructing Javadoc information...
Creating destination directory: "D:/TDLib/td/tdlib/bin/../docs"
Building index for all the packages and classes...
Thank you very much

The Java example was broken in the last commit. This was fixed in TDLib 1.8.21, which will be released soon. For now, I would recommend to revert the last master commit and build the example without it.

محمد الحمامي
في السبت، 4 نونبر 2023 9:19 م Aliaksei Levin @.***> كتب:

The Java example was broken in the last commit. This was fixed in TDLib
1.8.21, which will be released soon. For now, I would recommend to revert
the last master commit and build the example without it.
—
Reply to this email directly, view it on GitHub
https://github.com/tdlib/td/issues/2663#issuecomment-1793512932, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BAK5F73VM43PF4HC2TVWQ5TYC2BMVAVCNFSM6AAAAAA65XCGCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGUYTEOJTGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

The Java example was broken in the last commit. This was fixed in TDLib 1.8.21, which will be released soon. For now, I would recommend to revert the last master commit and build the example without it.

First, thanks for your reply Levlam
I'm a student and to be honest, I have almost not experience using Git. Do you mean, revert to this one?
D:\TDLib\td>git log
commit dd77e4628f1a65f332f0cb81c82a19c0fcfa40c2 (HEAD -> master, origin/master, origin/HEAD)
Author: Vyacheslav<EMAIL_ADDRESS>Date:   Wed Oct 25 00:52:34 2023 +0600
if yes, should I use this command? --> git revert HEAD
And finally, after doing this, should I redo all steps on ' https://tdlib.github.io/td/build.html?language=Java' ?
Thank you very much

if yes, should I use this command? --> git revert HEAD

Yes. After this the example will build without issues.

it seems ok now
**PS D:\TDLib\td\example\java\build> cmake --build . --target install --config Release
MSBuild version 17.7.2+d6990bcfa for .NET Framework
1>Checking Build System
Generating Java TDLib API source files
Building Custom Rule D:/TDLib/td/example/java/CMakeLists.txt
Building Java code
Building Custom Rule D:/TDLib/td/example/java/CMakeLists.txt
Generating Javadoc documentation
Loading source files for package org.drinkless.tdlib...
Constructing Javadoc information...
Building index for all the packages and classes...
Standard Doclet version 20.0.2+9-78
Building tree for all the packages and classes...
Generating D:\TDLib\td\tdlib\bin..\docs\org\drinkless\tdlib\Client.html...**
Thank you very much


Python Analzer in VS2019 hugs up all the CPU and memory.

Describe the bug
Using python projects in VS 2019 has become almost impossible. It spawns way too many python analyzer processes, which consume 100% CPU and crashes eventually as it runs out of memory.
See the attached snapshot of system resources being consumed.
Steps to Reproduce

Open VS 2019 solution with multiple python projects
Edit any python source code file
See CPU and memory consumption grow instantaneously

Additional context and screenshots

Configuration information (If you are providing a diagnostics file (see below), skip this section)
VS Version: VS Pro 2019 v16.3.6
PTVS version: 16.3.19252.1
Python version: 2.7
Diagnostics file
Diagnostic Info 2020-03-06T163138.txt

Thanks for the feedback.
We're currently working on a new Python language server which should have better performance, accuracy and memory usage.
We'll integrate it into VS as part of https://github.com/microsoft/PTVS/issues/5210, so you can follow that issue to be notified when that is done. It's still months away, unfortunately.

Is there a way to disable Python Background Analysis until #5210 is ready?

PUBLIC community feedback: https://developercommunity.visualstudio.com/content/problem/932644/visual-studio-2019-запускает-множество-процессов.html


Implementation method of struct inside function scope breaks code analyze

Code analyze works when I implement method outside function scope:

But when I try implement method inside function scope it breaks code analyze. Return type of is_even() displayed
as {unknown} instead bool

In both cases code compile and run successfully
Code
// struct SomeStruct(i32);

// impl SomeStruct {
//     fn is_even(&self) -> bool {
//         self.0 % 2 == 0
//     }
// }

// fn main() {
//     let o = SomeStruct(3);
//     let is_even = o.is_even();
//     println!("{:?} is even? {}", o.0, is_even);
// }

// -------------

fn main() {
    struct SomeStruct(i32);

    impl SomeStruct {
        fn is_even(&self) -> bool {
            self.0 % 2 == 0
        }
    }

    let o = SomeStruct(3);
    let is_even = o.is_even();
    println!("{:?} is even? {}", o.0, is_even);
}

Confirmed. I was sure there's already an issue for this, but I can't find it right now.

https://github.com/rust-analyzer/rust-analyzer/blob/2c0f433fd2e838ae181f87019b6f1fefe33c6f54/crates/hir_ty/src/infer/expr.rs#L985-L995
I found that it cannot lookup the method called is_even


Logs in OpenShift platform unauthorized

When viewing logs for Pods where previously we saw log messages we now see the JSON below:
https://console-openshift-console.apps.odh-cl1.apps.os-climate.org/k8s/ns/sandbox/pods/physrisk-api2-42-vjmtd/logs
{
"kind": "Status",
"apiVersion": "v1",
"metadata": {},
"status": "Failure",
"message": "the server has asked for the client to provide credentials ( pods/log physrisk-api2-42-vjmtd)",
"reason": "Unauthorized",
"details": {
"name": "physrisk-api2-42-vjmtd",
"kind": "pods/log"
},
"code": 401
}

Hi @MichaelTiemannOSC, @redmikhail,
would you know who I can ask to help with this?
FYI @mvarfima
Thanks,
Joe

Also added @ModeSevenIndustrialSolutions

@joemoorhouse One of the Openshift nodes due to the issue with underlying AWS EC2 instance was only partially available. While pods on this node have continue to run access to logs was not working . It was corrected and logs should be available. Please validate and let me know if issue still exist

Thanks for that Mikhail. Indeed fixed the issue!
On Fri, 26 Jan 2024, 03:28 Mikhail Mikhailitchenko, <
@.***> wrote:

@joemoorhouse https://github.com/joemoorhouse One of the Openshift
nodes due to the issue with underlying AWS EC2 instance was only partially
available. While pods on this node have continue to run access to logs was
not working . It was corrected and logs should be available. Please
validate and let me know if issue still exist
—
Reply to this email directly, view it on GitHub
https://github.com/os-climate/os_c_data_commons/issues/367#issuecomment-1911389908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABG5YQGDBVAD25F6C6HTOCLYQMPGDAVCNFSM6AAAAABAZTNGESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJRGM4DSOJQHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


There should be an ability to log-in with Google, Facebook, Twitter etc.

This is a wiki application feature, not wiki module one.

https://github.com/Nodge/yii-eauth


fix disable cache for 2.2

Addresses https://issues.redhat.com/browse/ACM-6981

Closing this PR as the issue has been resolved in MCE 2.2.8.


Sass source map for dev

#5707

cc @Fabianopb
According to https://github.com/webpack-contrib/sass-loader#source-maps it didn't say it was needed on style-loader.  this set of changes on dev is consistent with the config on prod, I would imagine that it should be added two both configs if deemed necessary.

It's been many months, but I had in my mind that it wasn't necessary to enable sourceMap in style-loader because I remember seeing this documentation that @bugzpodder referenced here.
But, not sure now, as @ianschmitz noticed from the style-loader docs, it seems to be required.
Apparently we have conflicting information, should we ask someone from webpack-contrib to confirm?


Nothing that I run executes inside the container. https://docs.microsoft.com/en-us/azure/devops/pipelines/process/container-phases?view=azure-devops&tabs=yaml

Thank you for your feedback!
The issues in this repo are for documentation feedback.
If you are submitting your documentation feedback issue directly through GitHub, please provide the URL of the article so we can route the feedback to the correct author.
Please report Azure DevOps or TFS software bugs, feedback, or questions in the Azure DevOps Developer Community.

@tomingoglia -- please provide more context around the general task.

Hi @WilliamAntonRohm...sorry about the message above.
Below is the page to the docs:
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/container-phases?view=azure-devops&tabs=yaml
It says that you can run commands  inside the container. Would this be the same as docker exec ...?
So...for example, if we wanted to execute an ls or do a cp to copy files to the image. I can't seem to run any docker command or get anything to run in the context of the container that is running. The container does load as expected based on the logs.
Tom

@tomingoglia -- Tom, thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


creating dictionary

i am using globalize 1.1.1 .
creating dictionary by this code : localization.loadMessages({de: { "name": "Vorname"}})
and use it by this code :Globalize.formatMessage("name")
and got this error Globalize.formatMessage is not a function.
am i using wrong version or its what? it is so confusing that in every version your names is going to change and when we are using them we got so many names that are not clear .

Did you load the message module? Please see our examples as a baseline https://github.com/globalizejs/globalize/tree/master/examples.
If you still think there's an issue, please provide a way to reproduce it and I will reopen. Thanks


zabbix::agent doesn't pass $zabbix_version to zabbix::repo

Using this module out of the box to set up zabbix in our environment, we installed 2.4 on all of our up to date distributions, but when adding in a check to send 2.2 out to ubuntu Lucid hosts, it seems when using in our node definition:
if ( $lsbdistcodename == 'lucid' ) {
    $zabbix_version = '2.2'
  } else {
    $zabbix_version = '2.4'
  }

class { 'zabbix::agent':
      server         => '<IP_ADDRESS>',
      listenip       => '<IP_ADDRESS>',
      hostname       => $::fqdn,
      zabbix_version => $zabbix_version
    }

the 2.2 string is never actually sent to zabbix::repo to set up the repository.
Using notify, I traced $zabbix_version from our node definition, into zabbix::agent, but when zabbix::repo is called it uses the default value defined in zabbix::params regardless of $zabbix_version.
I was able to work around this in our environment by using the resource-style declaration in zabbix::agent
# Check if manage_repo is true.
  if $manage_repo {
    class {'zabbix::repo': zabbix_version => $zabbix_version }
    Package['zabbix-agent'] {require => Class['zabbix::repo']}
  }

Edit: Nevermind, this has already been fixed in the latest release.


[SPARK-47257][SQL] Assign names to error classes _LEGACY_ERROR_TEMP_105[3-4] and _LEGACY_ERROR_TEMP_1113

What changes were proposed in this pull request?

This PR renames a few error classes related to usage of ALTER COLUMN errors:

_LEGACY_ERROR_TEMP_1053 => UNSUPPORTED_FEATURE.TABLE_OPERATION

_LEGACY_ERROR_TEMP_1054 => UNRESOLVED_COLUMN.WITH_SUGGESTION_AND_TABLE

_LEGACY_ERROR_TEMP_1113 => UNRESOLVED_COLUMN.WITH_SUGGESTION_AND_TABLE

Also, this PR changes tests in corresponding test suite to use checkError() method which checks the error class name, context, error message parameters, etc.
Why are the changes needed?

Proper error names and messages improve user experience with Spark SQL.
Does this PR introduce any user-facing change?

Yes, this PR changes user-facing error class and message.
How was this patch tested?

By running tests from PlanResolutionSuite, DataSourceV2SQLSuite and JDBCTableCatalogSuite, and tests from SQLQueryTestSuite with change-column.sql.
Was this patch authored or co-authored using generative AI tooling?

No.

cc @MaxGekk @panbingkun FYI

cc @MaxGekk @panbingkun FYI

also cc @cloud-fan

friendly ping @cloud-fan  do you have any comments on this pr?

cc @yaooqinn  when you have time~

Provide a query context otherwise it seems like bad impacts on user experience.

Update it, I add a new function to keep the context.

+1, LGTM. Merging to master.
Thank you, @wayneguow and @LuciferYang @panbingkun for review.


First run takes long time on startup.

Since the last update, we have discovered that the startup of the container the first time takes several minutes, compared to starting later on which takes seconds. This have been tested on newly started, clean amazon instances.
Commands run:
docker run -d -p 49160:22 -p 49161:1521 wnameless/oracle-xe-11g
docker exec -it <name> bash

When trying to access the db with
sqlplus system/oracle

we get the following error messages
SQL*Plus: Release <IP_ADDRESS>.0 Production on Thu May 7 07:43:10 2015

Copyright (c) 1982, 2011, Oracle.  All rights reserved.

ERROR:
ORA-01033: ORACLE initialization or shutdown in progress
Process ID: 0
Session ID: 0 Serial number: 0

First startup
Thu May 07 00:00:45 2015
ALTER DATABASE OPEN
Thu May 07 00:06:16 2015
Thread 1 opened at log sequence 3
Current log# 1 seq# 3 mem# 0: /u01/app/oracle/fast_recovery_area/XE/onlinelog/o1_mf_1_bbqv44l7_.log
Successful open of redo thread 1
Thu May 07 00:06:16 2015
SMON: enabling cache recovery
Thu May 07 00:06:17 2015
[100] Successfully onlined Undo Tablespace 2.
Undo initialization finished serial:0 start:173426 end:173646 diff:220 (2 seconds)
Verifying file header compatibility for 11g tablespace encryption..
Verifying 11g file header compatibility for tablespace encryption completed
SMON: enabling tx recovery
Database Characterset is AL32UTF8
Opening with Resource Manager plan: INTERNAL_PLAN_XE
Thu May 07 00:06:17 2015
Starting background
process VKRM
Thu May 07 00:06:17 2015
VKRM started with pid=23, OS id=125
replication_dependency_tracking turned off (no async multimaster replication found)
Starting background process QMNC
Thu May 07 00:06:18 2015
QMNC started with pid=24, OS id=127
Completed: ALTER DATABASE OPEN
Thu May 07 00:06:18 2015
Second startup
Thu May 07 00:10:41 2015
ALTER DATABASE OPEN
Thu May 07 00:10:49 2015
Thread 1 opened at log sequence 3
Current log# 1 seq# 3 mem# 0: /u01/app/oracle/fast_recovery_area/XE/onlinelog/o1_mf_1_bbqv44l7_.log
Successful open of redo thread 1
Thu May 07 00:10:50 2015
SMON: enabling cache recovery
[92] Successfully onlined Undo Tablespace 2.
Undo initialization finished serial:0 start:446776 end:446796 diff:20 (0 seconds)
Verifying file header compatibility for 11g tablespace encryption..
Verifying 11g file header compatibility for tablespace encryption completed
SMON: enabling tx recovery
Thu May 07 00:10:54 2015
Database Characterset is AL32UTF8
Opening with Resource Manager plan: INTERNAL_PLAN_XE
Thu May 07 00:10:54 2015
Starting background p
rocess VKRM
Thu May 07 00:10:54 2015
VKRM started with pid=23, OS id=95
replication_dependency_tracking turned off (no async multimaster replication found)
Starting background process QMNC
Thu May 07 00:10:54 2015
QMNC started with pid=24, OS id=97
Completed: ALTER DATABASE OPEN

I confirm this bug, 1st run take several minutes

Confirmed as well.  Is there any workaround for this?  Using this container in a Continuous Integration environment and starting a new one up for each build to test deployment.  The long startup time is not desirable.
It may just be due to Oracle and having a new hostname and having some additional work to do.  I have tried other Oracle Docker containers with the same issue.

I'm not going to say it's going to be infinitely faster, but this seems to be more of an issue with Oracle's startup process. I found this link, which sped up my program's startup significantly. I just created my own dockerfile and added the following line:

COPY startdb.sh. /u01/app/oracle/product/11.2.0/xe/config/scripts/startdb.sh

Where in my startdb.sh I simply moved line 38 in the startdb.sh to be after the following if statement.
Line 38 is this:
$SQLPLUS -s /nolog @$ORACLE_HOME/config/scripts/startdb.sql > /dev/null 2>&1
Or you can copy the following script if you want.
#!/bin/bash
#
#       svaggu 09/28/05 -  Creation
#       svaggu 11/09/05 -  dba groupd check is added
#

xsetroot -cursor_name watch
case $PATH in
    "") PATH=/bin:/usr/bin:/sbin:/etc
        export PATH ;;
esac

export ORACLE_HOME=/u01/app/oracle/product/11.2.0/xe
export ORACLE_SID=XE
LSNR=$ORACLE_HOME/bin/lsnrctl
SQLPLUS=$ORACLE_HOME/bin/sqlplus
LOG="$ORACLE_HOME_LISTNER/listener.log"
user=`/usr/bin/whoami`
group=`/usr/bin/groups $user | grep -i dba`

if test -z "$group"
then
        if [ -f /usr/bin/zenity ]
        then
                /usr/bin/zenity --error --text="$user must be in the DBA OS group to start the database."
                exit 1
        elif [ -f /usr/bin/kdialog ]
        then
                /usr/bin/kdialog --error "$user must be in the DBA OS group to start the database."
                exit 1
        elif [ -f /usr/bin/xterm ]
        then
                /usr/bin/xterm -T "Error" -n "Error" -hold -e "echo $user must be in the DBA OS group to start the database."
                exit 1
        fi
else
# Starting Oracle Database 11g Express Edition instance and Listener
        if [ ! `ps -ef | grep tns | cut -f1 -d" " | grep -q oracle` ]
        then
                $LSNR start > /dev/null 2>&1
        else
                echo ""
        fi
        $SQLPLUS -s /nolog @$ORACLE_HOME/config/scripts/startdb.sql > /dev/null 2>&1
fi

xsetroot -cursor_name left_ptr

Same problem.


Cannot override status bar background color for each page route

Hi,
I facing an issue when changing status bar background color dynamically for each page/scaffold. Status bar color change into undesired brightness. But sometimes it works well when the app state is restarted. I've tried many solutions but it didn't work well. Here is the solution I have tried :

setSystemUIOverlayStyle

SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle.light.copyWith(
   statusBarColor: Colors.black.withOpacity(0.3),
));

AnnotatedRegion

AnnotatedRegion<SystemUiOverlayStyle>(
    value: SystemUiOverlayStyle(statusBarColor: Colors.red)
    child: MyBodyWidget(),
)

flutter_statusbarcolor (https://pub.dev/packages/flutter_statusbarcolor)

FlutterStatusbarcolor.setStatusBarColor(
      Colors.black.withOpacity(0.3),
);
FlutterStatusbarcolor.setStatusBarWhiteForeground(true);

Is there another solution for this problem?
Thank you.
Edit:
Tested on :  Galaxy S7 edge (sdk 26) & Mi A2 (sdk 28)
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.9.1+hotfix.2, on Microsoft Windows [Version 10.0.18362.356], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
[√] Android Studio (version 3.5)
[√] VS Code (version 1.38.1)
[√] Connected device (2 available)

• No issues found!

I am facing the exact same problem - did you manage to get around this/ find a solution?

@samfrank Currently no. I'm still finding the solution.

@samuelg123 @samfrank

try to use getWindow().xx

@dev-zl I tested your solution but still not work. I think its problem lies in the framework. So when I change to another route, status bar icon brightness change itself. Sometimes the background color could also disappear. Thanks

I have the same issue but with status bar color. The status bar was initially transparent; i tried to override it in a route but it would reset itself quickly. Interestingly, overriding status bar icon brightness always works for me, but never status bar color.

Hi @samuelg123
Can you confirm that this problem still exists on the latest master or stable and provide a minimal reproducible code sample and flutter doctor -v?
Thank you


Update aws-java-sdk-dynamodb to 1.12.746 in scala-steward-dependencies

About this PR
📦 Updates com.amazonaws:aws-java-sdk-dynamodb from 1.12.744 to 1.12.746
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #242.


ignore private 'Changed' methods in codegen

From robert.w.hartung:
For me I get the 404 errors at a different point:
I have this code:
  void _onActiveTabChanged(CustomEvent ev) {
    // ...
  }
  @override
  void attached() {
    super.attached();
    $['dock-center'].on['active-tab-changed'].listen(_onActiveTabChanged);
  }
The editor doesnt show any errors/warnings for this. But I get an build error:
Build error:Build completed with 1 errors.
Transform ScriptCompactor on gui|web/gui.html threw error: Bad state: invalid symbol name: "_onActiveTabChanged"
Can someone explain why this happens? Should I create another bug?

This issue has been moved to dart-lang/polymer-dart#451.


Eloquent count returning all times 1 when is spitted

Laravel Version: 9.28.0
PHP Version: 8.1.9
Database Driver & Version:

Description:
Eloquent count not working, when using Eloquent builder spitted.
So right now we need fetch all models, even millions, and after that we count all returned models ;/ Sounds like memory leak and incorrect value.
Steps To Reproduce:
$query = SomeModel::query()
    ->leftJoin()
    ->where()
    ->when()
    ->groupBy();

$a = $query->get()->count(); // return good value
$b = $query->count(); // return wrong value, always 1

Do you have data or a repo to reproduce this?

Sounds like memory leak

Why? What hints you in this direction? I am genuinely curious.
This is known behavior.
All aggregates methods on the query builder (count, min, max, sum, avg) return only the first result (see the $results[0] below)
https://github.com/laravel/framework/blob/004163a06fccd0e95fcd7f78989b6744b8f6325f/src/Illuminate/Database/Query/Builder.php#L3076-L3086
There were several similar issues, I will point to two lengthy ones, with deeper discussion on the rationale on this is would be an expected behavior.

https://github.com/laravel/framework/issues/28931
https://github.com/laravel/ideas/issues/1693

As always, if you believe this behavior should be changed, you could try a PR. But I would prepare a very lengthy defense on why this should be changed as this "issue" (or non-issue) comes up every once in a while.
P.S. actually it doesn't always return 1, if you order by a different column where the first result is different than 1, it will always return that. What happens is that it always returns the first record.

Since the same questions appears multiple times, it could be a good idea to push a PR for countGroupsBy and countDistinctBy().

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

i feel this IS a bug tho. no idea how returning N1 from a group by is considered the "right" answer. if anything return an iterator or sum of all the values or something.
"illuminate/database": "^9.38"
but anyways.. a potential solution.. since this totaly stumped me till i checked the query log ($queries = \App\DB::getQueryLog();:
$record_count = \App\DB::table( "fake" );
$record_count->fromSub($records->select(\App\DB::raw(1)),"query");
$record_count->count();

create a "fake" query builder
add a sub "from" from (...) query
set the "sub query" to select 1 instead of returning huge column data. not sure if this is needed but in my mind it seems like a good idea
do the normal ->count()

results in a sub query ie
select count(*) as aggregate from (select 1 from table group by column1) as `query`

(PLEASE NOTE: this is JUST a demonstration query. please dont be all "but you could just do $records->count() and remove the group by", thats not the point, my queries are far more complex - this cause after much searching ive come to realize some people are... interesting.. a lot of the answers are "change your query")

As always, if you believe this behavior should be changed, you could try a PR. But I would prepare a very lengthy defense on why this should be changed as this "issue" (or non-issue) comes up every once in a while.

As said before, If you do believe this should be changed, please send a PR. Up until now, all the discussions about it lead to the conclusion no change is needed.
Also, what I pointed out is that all aggregates return only the first row from the result set - by pointing out where in the code one could check it out - and that this is a known behavior. I understand the cause of frustration, but I also do understand the arguments around the current stance. I didn't make any value judgment around the matter.
PRs are always welcome, and if they provide enough explanation on the matter, they are actually reviewed. One option could be @DarkGhostHunter suggestion to add an alternative method implementation to cover the alternative outcome some people think is expected.

other than a quick mention about potential performance (subjective?) what are the main reasons for this being an issue (or non issue as has been said). setting up a query builder and using groupBy (which is perfectly valid) and doing a count() and getting the first records aggregate seems very very strange to me. (return $result[0] - therby basicaly "throwing away" a bunch of info that the code originally requested)
surely "this is my query, i want to know how many records it returns, so i use ->count()" is a valid expectation. instead.. "this is my query, i want to know how many records it returns.. o i cant unless i do a count(...->get()) and deal with that resource issue" or use the above.
is there maybe an issue that makes it super hard to do in the code base? surely it shouldnt break any backwards compatibility.
im not being facetious with these posts. im just genuinely curious why some people feel that doing return $result[0] is considered "working as intended". ("we are too busy and its been like this a long time, we dont feel like changing it" is a perfectly acceptable response as well)

It's hard to fix issue, because till now bunch of developers write tons of work around code and each one spend a lot of time figuring out what happened inside framework.
I think, good behavior is to fix that issues, even if they changing some "crazy" logic and bring that fix with new release. Then developers can read about breaking changes and have clear situation what have to working in what organized way.


Added missing letter on extension

@jlattimer : Thanks for your contribution! The author(s) have been notified to review your proposed change.


How can I use .env dotenv_config_path to deploy multiple products with the same source code.

I have two products using the same source code.
The only difference in front-end is titles and other texts. So I want to config these info with .env.
Then I find that, since it's staticly served, I need different builds for different products.
So I looked at .env. dotenv_config_path looks like sth I'm looking for.
Now the problem is CRA's config/env script doesn't allow me to choose a .env config file.

You can't I suggest you create a .env.production.app1 and an .env.production.app2, then add .env.production to your .gitignore file.
Make your build script cp .env.production.app{1,2} .env.production before calling npm run build.


[Question] Add data to db manually

Hello, my friend and I are developing a workout tracker for personal purposes and I was worndering if there's a way to enter data manually that persists in the app. Let me explain myself:
We would like to already have some, for instance, exercices and pre-defined workouts, so, when the user downloads the app for the first time, it does not face an empty application.
So, is there a way to reproduce what has been explained with the package?
Much appreciated!
Edit: The right term would be a way of pre-populate the database.

I've found the Post-migration post in the documentation that explains how to. Thanks anyway!


[Formatter] Comment gets added in weird place when between decorator and scalar statement

@format("x")
/**
 * Desc is NOT working
 */
scalar bar extends string;

Formats to
@format("x")
scalar /**
 * Desc is NOT working
 */
bar extends string;

Pretty sure this formatting issue happened before on other types and needs the same special  handling for scalars
Playground Link

est: 2


Get pixels in First?Third Person view

Hello
I am looking at capturing the buffer of pixels displayed in a FirstPersonView - so that I can apply some machine learning algorithm onto them.
Is there a way do to that?

Yes, you'd use normal DOM APIs, e.g. if you have access to the canvas or gl context, then you can do canvas.toDataURL() or gl.canvas.toDataURL().
Unless you are passing those in, you can get those in different ways e.g. by defining the onWebGLInitialized callback.


iOS Screenshots for Version: 2.2.0

iOS Screenshots for Version: 2.2.0

@Ein-Tim Thanks. Checking.

@Ein-Tim that one looks to me like the new ones. Fine, no?

R 2.2 screenshots

R 2.1 screenshots

LGMT ...

ios-de.pdf

@dsarkar
True! Sorry, I didn't mean outdated but that there are duplicates and other strange things.
Section: "Contact Journal"

[ ] Remove onboarding images (aren't shown for Event-Registration feature)

Section "Check In"

[ ] Rename German section to "Check-in" to match app text

Section: "Test registration & Key sharing"

[ ] Remove image 4
[ ] Remove image 7
[ ] Move image 10 to first place
[ ] Remove image 11
[ ] Add an image showing an positive test

Section: "Test result home"

[ ] Fix all images showing the same

Section: "Rapid antigen test"

[ ] Remove Android screenshots
[ ] Replace image 7 with current implementation

If you want I can open a new issue for this.

@Ein-Tim OK, thanks for detailed list!

@Ein-Tim I am editing your post, hope you don't mind.

@dsarkar
Not at all!

Can this branch be deleted, please?
It's bit confusing if we have more than one screenshots branch for each OS at the same time.
Thank you!

@Ein-Tim Thanks.


Switch consensus crates to new futures

I had this PR almost ready for a long time, was just waiting for #3117 to be merged.
Switches consensus/* to new futures.
Most notably, however, start_aura and start_babe still return an old future (by building a new future and putting the compatibility layer around), so that we avoid breaking everything.
cc #3099

I'd prefer to merge the BABE PRs first, if that's OK.


ansible venv not in path and playbook fails with "boto3 required for this module" error

PYTHON3_MODULE_PATHS was never updated to include the ansible venv path when we moved to python3 (CentOS8) and it can't find the python modules.
I'm not sure if adding "/var/lib/manageiq/venv/lib/python3.6/site-packages" to the path is the only thing needed, so opening an issue.
cc @NickLaMuro

Possibly related, but I'm starting to think (maybe not for jansa), if we should just completely remove any out of the box python modules, and enforce that the user uses a proper requirements file in their playbooks.

For now, this should be addressed here:
https://github.com/ManageIQ/manageiq/pull/20322


Fix bug during response evaluation

Proposed changes:

fixes https://github.com/RasaHQ/rasa/issues/5916

If the test data is missing a response that is present in the training data and during testing this response is predicted, the evaluation failed. This is not fixed.
Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

@tabergma Tried reproducing the functionality, attaching a screenshot -

I am thinking if we should refactor our train test split function for response selector data to always include all responses both in train and test splits. That way, if a user uses rasa data split, they'll never run into this problem. What do you think?

Was thinking the same, but I guess we should create a separate issue for that. Would nevertheless keep this "fix" as it also might happen, that users split the data on their own and miss some labels in the test set. What do you think?

Makes sense, do we need a changelog entry for it?

I don't think a changelog entry is needed as the feature is not yet released. Will create a separate issue for the train/test split.


Add support for few more git operation on repo

fetch
checkout
rebase

Thanks for your PR! Please solve the conflict we have right now.


feat: experiment with no-lock page cache

just an experiment - basically, build a kind of RwLock into the page cache, which owns all the data
and only gives out weak pointers

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

#68  👈
#67
master

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

Join @rphmeier and the rest of your teammates on  Graphite

outdated by #71 #72


Add training pipeline

Adds all the code from the repo used to train the models exposed in this repo.
The pipeline, configurations and datasets may vary from the ones used in the final models

LGTM. I will merge this PR and refactor it where I see necessary, to make it more publishworthy.


Uri::__toString() can yield malformed URIs

Bug Report

Q
A

Version(s)
Current git 3.3.x

Summary
The Uri class is able to parse malformed URIs but this results in Uri::__toString() generating a malformed URI. Attempting to pass that URI back into Uri will yield a InvalidArgumentException.
Current behavior
Certain malformed URIs do not round-trip through \Laminas\Diactoros\Uri.
How to reproduce
<?php

require('vendor/autoload.php');

$value = 'http://invalid:%20https://example.com';
$uri = new \Laminas\Diactoros\Uri($value);
$uri2 = new \Laminas\Diactoros\Uri($uri->__toString()); // Exception is thrown here.

Expected behavior
Either both constructors throw, or neither.

Note: This issue also exists in guzzlehttp/psr7 and was reported at guzzle/psr7#583.

$value = 'http://invalid:%20https://example.com';

PHP's function parse_url can not handle this wrong URL: https://www.php.net/manual/function.parse-url.php#refsect1-function.parse-url-notes
array(3) {
  'scheme' =>
  string(4) "http"
  'host' =>
  string(16) "invalid:%20https"
  'path' =>
  string(13) "//example.com"
}

https://github.com/laminas/laminas-diactoros/blob/4c6ae0026430244f10e9f59e6c31b605c55ce359/src/Uri.php#L80-L87
https://github.com/laminas/laminas-diactoros/blob/4c6ae0026430244f10e9f59e6c31b605c55ce359/src/Uri.php#L384-L392

Thanks, @TimWolla for cross posting.
Lets see how guzzle will handle this, after my latest hassle with php-http/discovery I would prefer to keep this in-sync.
We might also want to cross-post this to https://github.com/php-http/psr7-integration-tests (which is used by diactoros as well).

I introduce basic host validation with the linked PR but anything more comprehensive would need to be done via integration tests.
The changes I introduced need to be elevated to the integration tests as well to ensure it is handled across all implementations.


User group functionality phase-2

Phase-2 should include the following features as it relates to groups:

[ ] Start a discussion
[ ] Delete a discussion
[ ] Comment on a recent activity or discussion.
[ ] Get recent activity list (additional items): comments, discussion posts, user joined the group.
[ ] Dynamic/on demand loading in Find Groups and Recent Activity.
[ ] Ability for members outside of a group to share resources with it.
[ ] Add labeling/favorite functionality to resource listing of Group page.

No longer relevant.


Скласти інтро

Скласти інтро (вступ), де в 2-4 реченнях описати, що замовник робить, що подобається робити, що хочеться робити краще, що важливо робити, тощо.

― I do not compete with others, the only competition that challenges me is myself. As a fighter, friend and an employee I am known as contemplative, tortured and driven.

Résumé
― I'm the most successful person ever to run for the presidency, by far. Nobody's ever been more successful than me. I'm the most successful person ever to run. Ross Perot isn't successful like me. Romney ― I have a Gucci store that's worth more than Romney. Sorry losers and haters, but my IQ is one of the highest ― and you all know it!


Improve performance on new crawls

If a crawl already exists in dgraph, the response back from the API is quick, but new crawls don't return until that crawl has been stored in the database.
The process of writing to the database should be spawned into a concurrent process and be managed separately, and the API should return the nested page object regardless of whether or not it has been written yet.

Fixed in 041f6447ef243430f92ffd49eb9d9ed75b95deb4


GitLab support and non-public GitHub/GitLab instances

It would be really nice if eget could support GitLab repositories in addition to GitHub.
In addition, it would be nice if support for private GitHub/GitLab instances can be used as well.
My proposal for these features is two fold. First, refactor the code so it uses repository provider modules that contain types which satisfy a generic Provider interface with differnet functions needed by eget. The codebase would then consist of:

A Provider interface with well-defined functions for all operations needed by eget from each provider.

For example, to list releases, to find and download assets, etc.

A GitHubProvider concrete type that satisfies Provider containing all the existing code for the GitHub API.
A GitLabProvider concrete type that satisfies Provider containing new code for the GitLab API.
Any future provider implementation, for example for Gitea as requested in #61.

Second, define a syntax for the target argument in eget so a provider and private domain can be specified. For this, the syntax proposed in #61 looks like a good start: [domain/]user/repo. However there are some considerations to address:

Not all providers are limited to user/repo. GitLab for example supports nested groups such as org/group/repo.

For this, I would propose to simply generalize the syntax to [domain/]path.
If no domain is specified, default to public GitHub to preserve current behavior.

Just by allowing domain there is the ambiguity of which provider to use for the requests.

For public instances, each Provider can implement a PublicDomains() function and do a simple check.
For private instances, there are some options to specify the provider to use.

eget could implement an optional --provider flag (and/or environment variable).
The provider can be incorporated in the target syntax: [provider:[domain/]]path
If no provider is specified, default to using domain to detect a public instance.

If you are willing to accept a contribution for all the above, I'm happy to work on it. I would just need your thoughts on the points above so the implementation goes the way you prefer.

I'd be happy to have support for GitLab or other providers. The code is currently organized in a pretty modular way -- it should be possible to add this without too much change. In particular, currently there are various types of Finders, which return all the URLs for assets. A Finder is constructed from the getFinder function, which is given the target string and uses that to determine what Finder to use and constructs it.
I think adding GitLab support would mostly just consist of creating a new GitlabAssetFinder that can find assets given a repository, and possibly other information (similar to the current GithubAssetFinder). Then the getFinder function would have to be updated to use the GitlabAssetFinder if it determines that the domain is a GitLab domain. It could also take into account a custom provide (I'm not sure if a flag or the provider:... syntax is better currently).
I think this is a good starting point. If this is easy to implement, then we can think about generalizing getFinder so that each Finder can provide information about whether it matches a domain or not, moving the logic in getFinder to each individual Finder and hopefully simplifying things.
Let me know what you think.

Ah interesting, I have not dive very mch yet into the codebase, but what you describe sounds good to start with indeed.
Let me try a POC for a set of GitLabXYZ() functions and a simple dispatcher in getFinder() to see how it goes. I will report back here with the results. This is also a good way to get more familiar with your code for me.
I will start with public instance support (similar to existing GitHub support) and then we can discuss target syntax changes or another mean to select a provider for custom private instances.
I think, maybe for later, it would be a good idea to separate github, gitlab, anything-else code into their own module directories/files so the code is easier to maintain if more providers are added later.

@zyedidia I created an initial POC implementation based on your comments.
See the changes here: https://github.com/hhromic/eget/compare/master...feat-gitlab
In general, I tried to keep the code similar to the existing codebase, with as few changes as possible.
With this experience I got much more familiar with the source code of Eget.

Like for GitHub, I implemented support for providing GitLab access tokens.
Like for GitHub, I implemented both, an assets finder and a source tarball finder.
Like for GitHub, I implemented support for the --pre-release option.

However, it's not 100% clear to me how it really works. I have not tested this very much.

Given that GitLab can directly find releases by tag, I did not need to implement FindMatch().

While it was possible to just implement Gitlab-related functions and plumbing as suggested, the code turned out quite ugly because Eget is strongly designed/implemented around Github as the sole provider. I would really suggest to do a code refactoring to abstract GitHub into a provider interface with clearly separated implementations.
Let me know what do you think!


[NEW ICON] Git Light

Skill Name
Git Light
Why?
Skill-icons' way themed light Git icon.
Reference Image

Currently, themed icons are only used when the logo does not provide a background or is not visible on both black and white. Thank you for the suggestion, though!


Ability to embed in site?

Does umami allow the functionality to embed the dashboard into a node.js express basic javascript backend and front end site?

You can embed anything using an iframe.


Turn notifications_total into notifications_sizes

Instead of reporting just the number of notifications sent and received through a counter, this PR now also reports their sizes in bytes through a histogram.
Consequently I renamed notifications_total to notifications_sizes.

benchmark test failing. please merge master.


Need Option to generate Yaml From the routes

Hi Would like to have an option in vertx to generate the Swagger documentation from the routes of a running HTTP server.
It would be great to have this functionality in vertx and improve adaptability and discovery

Duplicate of #137


German translation

Adds a German translation 👍

Two quick questions:

Does translating the little "v" used in between the Title ID and version to "=" mean something?
I noticed that the hint text "Version" for the version text entry field isn't translated, and there's no note on it. Is that intentional? I thought it could be, but you left notes on all of the other things that weren't changed for the translation so I wasn't sure.

Other than that, looks good! Thanks for the translations!

= seems to just insert the original text as translation, since some words aren't different (e.g Version is the same in German and English)

Ah okay, that makes sense. In that case, LGTM! Thanks again!

No worries!

@yeah-its-gloria Hey, just wanted to ask if you'd be willing to update your translations. I've finalized string changes for v1.3.0, so it'll be ready for release once all of the existing translations are up to date. Thanks!

for sure, I'll get onto that in a few days

Sounds good, thank you!


Create own type for InitallyUnowned to match the GObject type hierarchy

Create own type for InitallyUnowned to match the GObject type hierarchy

This is now part of #52 .


doc: reorganize the API reference of Wasm extensions

Signed-off-by: Takeshi Yoneda<EMAIL_ADDRESS>Reorganized the Wasm extensions references, and tried to make it look better and put all the Wasm extensions in a top-level section in the Extensions page. In order to achieve this, this commits separated out the "Wasm Service" proto from the common configurations to the proto named service.proto
Before

After

cc @PiotrSikora @lizan

Is there any workaround for moving message to another proto without breaking change? (I guess there isn't).
Having WasmService definition in a separate file, not with the common configurations, seems necessary for me to better integrate with extension annotations.

OK I see that we can use multiple protodoc-title annotations in a single proto. Will resolve the breaking change.

I tried public imports but it looks like PVG does not support them (validate.pb.h does not include the corresponding validation headers) and results in a breaking change.

According to https://github.com/envoyproxy/envoy/pull/10743#issuecomment-612931049, we don't need to maintain backward compatibility with PGV, so I think I'm going to go with using import public. Will do the follow up in a few days.

@phlax @PiotrSikora PTAL! I updated the PR description what I did in this change.

rendered docs are here https://storage.googleapis.com/envoy-pr/15360/docs/index.html

/retest

/retest

@phlax Thanks for taking a look. Unfortunately I cannot work on this PR anymore for now since the mv of existing proto files seem much more complex than I thought.
Will come back here when I get more cycles. Closing for now.


⚠ Chatwoot has degraded performance

In d2ac2c4, Chatwoot (https://app.chatwoot.com) experienced degraded performance:

HTTP code: 200
Response time: 25118 ms

Resolved: Chatwoot performance has improved in 4ead03e.


Added implementation and test for freeze and unfreeze.

Provides some enforcement for immutability.

Closing in favor of discussion at #53


Benefits?

Hey there!
I was just wondering what the benefits of using Lantern over Sponge are. I don't mean to sound rude, so I hope I don't come off like that.
Thanks :)

Lantern provides more flexibility in some areas for some things that is hard for the Sponge API & Implementation to achieve. There is also some performance benefits to a custom implementation that doesn't rely on the base Minecraft Vanilla Server in some areas too, but it's not too significant. Mainly flexibility I guess. It also adds another way to grow the SpongeAPI and benefit all implementations. 😄

#2

Wouldn't the biggest value be performance of Minecraft servers?  One of the biggest problems in the modded Minecraft community is the single-thread bottleneck of the server architecture.

Unfortunately, lantern is not designed to be forge mod compatible.


feat: added rose-pine colorscheme

Initial module for rose-pine colorscheme. Building currently fails and the
highlights group may need be changed. Based on the configuration found here.

Ok, found it. The mkLuaPlugin adds the plugin to nixneovim.plugins and not to nixneovim.colorschemes.
I think it is easier to just use the mkLuaPlugin function for color schemes, for now.
However, I found there are some other changes I have to make before I can merge this. I will give you detailed feedback later, when that is done.

Thanks, looking forward to it.
On Sun, Apr 23, 2023 at 3:00 PM jooooscha @.***> wrote:

Ok, found it. The mkLuaPlugin adds the plugin to nixneovim.plugins and
not to nixneovim.colorschemes.
I think it is easier to just use the mkLuaPlugin function for color
schemes, for now.
However, I found there are some other changes I have to make before I can
merge this. I will give you detailed feedback later, when that is done.
—
Reply to this email directly, view it on GitHub
https://github.com/NixNeovim/NixNeovim/pull/40#issuecomment-1519133418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTIR3PF6FF4TLFMBC7IORDXCV35RANCNFSM6AAAAAAXG5E6QE
.
You are receiving this because you authored the thread.Message ID:
@.***>

ref #39

I have implemented the necessary changes.
I will add you some comments on the code to help you implement this.
Please also look at the gruvbox-baby module for comparison.
If you are not interested in implementing this, anymore. Please drop me a note.

If you want, you can also add a small test. If you don't know how or don't have the time, drop me a note and I can help you with that

Thanks. Looks good 👍


Rake tasks with arguments

This is really cool! Thanks for developing.
For rake tasks that support arguments, is there a way to enter those in? So I can rake db:reset RAILS_ENV=test for example?

Unfortunately the current architecture of the base build package does not allow adding custom parameters to the executed commands easily. The problem is that the tasks (and their respective commands) are collected when an Atom project (directory) in opened. You cannot modify the commands later... :confused:
Here are some possible workarounds:

Define a new task (alias) for the needed arguments, e.g. add db:reset:test task which will run
db:reset using the test environment.
Set the Rails environment via the system environment variable, on Linux start Atom using command RAILS_ENV=test atom /your/project/directory. This works only for parameters which can be passed also via environment.
Set up a custom build command.

I'm closing the issue as it cannot be easily implemented in a build plugin. Feel free to report an issue for the base build package...


[ACR] 'az acr agentpool': New CLI testing

Description of PR (Mandatory)
Adding CLI testing for new feature Agent pool. Temporary in euap because of blocked released.
Testing Guide
(Example commands with explanations)
History Notes:
Adding CLI testing for new feature Agent pool

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

add to S168

Some checks were not successful

hi @rosanch could you pls fix CI failure ? thanks.

hi @rosanch any update?

@yonzhan  I honestly have no idea why the test is failing. It does work in my machine, and now it's not even able to create VNET, I'm really lost here.

hi @qwordy could you pls help @rosanch on the CI failure?

@qwordy  I commented all parts of the tests related to an issue. I really don't what the problem is. If I uncomment it works in my local environment but there're some issues when doing the testing here.

@qwordy I commented all parts of the tests related to an issue. I really don't what the problem is. If I uncomment it works in my local environment but there're some issues when doing the testing here.

@rosanch did you pass the playback test after you pass live test locally?

@fengzhou-msft I only did the azdev live test of this test.

@fengzhou-msft I only did the azdev live test of this test.

Can you merge latest dev branch and run azdev test acr to test with recording files locally?

@fengzhou-msft azdev test acr works without issues in my machine as well

@fengzhou-msft azdev test acr works without issues in my machine as well

Can you uncomment the tests to try the CI again?

@fengzhou-msft  Done. Still, for whatever reason, it can't create a vnet

/azp run

@MyronFanQiu can you take a look at the CI error for network? The tests passed locally, but failed in ADO.

@rosanch Hi. We bump the network sdk last month. Please do the following three things.

Merge latest dev into your branch
azdev setup
rerun your test in live mode to get new recording file

thanks @fengzhou-msft  it seems it finally worked

/azp run


Update overview.md

SEV-SNP CVMs are now GA and also offer encryption of data while in use

@msmbaldwin,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


vkdt: 0.7.0 -> 0.9.0

Updates vkdt to version 0.9.0, including setup for building rust parts of the code.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Both binaries start, but I ran into issues getting the GUI to work under wayland. Reported upstream at https://github.com/hanatos/vkdt/issues/142

hey, amazing! thank you for taking the time to do this.

Both binaries start, but I ran into issues getting the GUI to work under wayland. Reported upstream at hanatos/vkdt#142

When you say "both binaries start" do you mean vkdt and vkdt-cli? We used to get several binaries, one of them was vkdt-wayland (for wayalnd) and vkdt (for xorg). Not sure why the vkdt-wayland binary is no longer produce.
My other question is can we use the Cargo.lock file in src/pipe/modules/i-raw/rawloader-c/Cargo.lock seen here or is that lock file not sufficient?

When you say "both binaries start" do you mean vkdt and vkdt-cli?

Exactly. I also tested out the vkdt-wayland package which seems to be automatically created.

My other question is can we use the Cargo.lock file in src/pipe/modules/i-raw/rawloader-c/Cargo.lock seen here or is that lock file not sufficient?

I tried using the commited Cargo.lock with [fetchCargoTarball], but that did not work due to the git dependency. Therefore, I switched to importCargoLock, but that requires the Cargo.lock in nixpkgs, as far as I can tell.

Alright, cool. Do we get this merged, since it seems to work if the GPU drivers work?


[Suggestion] Don't run the command while any text is selected on the page

I have a suggestion for improvement.
The blockIfAnyTextboxSelected attribute - if true, it does not run the command as long as any text box is selected on the page (input[type=text], [contexteditable=true], textarea).
<Keydown blockIfAnyTextboxSelected={true} on:key={ ({detail})=> detail==="w"&& fun() } />

The point is that the command should not be invoked when typing in a text field.
But input input[type=number] would not be taken into account by blockIfAnyTextboxSelected - or block it in a separate way, maybe with a separate attribute...
Without it, you have to prepare a complex checking function and add it to the command.

Another improvement would be to make a 'detail' attribute which causes the command to only run if detail matches:
<Keydown detail="w" on:key={()=> fun() } />

or
<Keydown details={["w","s"]} on:key={()=> fun() } />

...thanks to this, the function itself can be simplified - as you can see.
You can think of better names for blockIfAnyTextboxSelected and detail/details. :)
What do you think?

@lukaszpolowczyk I think that's a great suggestion.
I've added a pauseOnInput prop which will not capture input events if set to true.
See the example in the documentation: https://metonym.github.io/svelte-keydown/#pause-on-input
Published to NPM in v0.3.0