For Beginners document and reference in read me added

I added a relative link and reference to the "for beginners" documentation in the read.me file. I also added a step-by-step walk-through for turning any old machine into a CLAW environment.

Sorry...I had to do quite a bit of research and editing to correct this missing link. I believe it's corrected now.

I haven't actually run through this process, but it all looks correct and if we find some corrections later we can handle them then.
👍

@rtilla1 this is good, but I couldn't find your name (or Tulane) on the list of current CLAs.
https://github.com/Islandora/islandora/wiki/Contributor-License-Agreements
Have you signed an individual CLA or has Tulane submitted one? We'll need one before I can merge this, my apologizes I should have caught this earlier.

I have not signed a personal one. I’m surprised Tulane hasn’t submitted one. I’ll check with Jeff Rubin because we definitely should sign one as an institution.
Rachel Tillay
Pronouns: they/them/theirs
Cataloging and Metadata Librarian
Tulane University
rtillay<EMAIL_ADDRESS>T:<PHONE_NUMBER>
From: Jared Whiklo<EMAIL_ADDRESS>Sent: Wednesday, September 19, 2018 4:09 PM
To: Islandora-Devops/claw-playbook<EMAIL_ADDRESS>Cc: Tillay, Rachel M<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Islandora-Devops/claw-playbook] For Beginners document and reference in read me added (#74)
@rtilla1https://github.com/rtilla1 this is good, but I couldn't find your name (or Tulane) on the list of current CLAs.
https://github.com/Islandora/islandora/wiki/Contributor-License-Agreements
Have you signed an individual CLA or has Tulane submitted one? We'll need one before I can merge this, my apologizes I should have caught this earlier.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Islandora-Devops/claw-playbook/pull/74#issuecomment-422959409, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALbYk1jlQBmzS4qIhB5c2kLORjfCfdySks5ucrKDgaJpZM4WGlT3.

@manez @dannylamb what do we do here?
We had a similar situation in Fedora where someone had come and helped update a bunch of dependency versions (so no code changes) and disappeared. We merged it with the plan to add a note to the CLA table indicating this deviance from the norm.
Or we dump it 🤷♂

@rtilla1 is certainly still around, but might not be getting her GitHub notices. I'll email her off-thread and see if she's made any CLA progress.

And it's now official! We have a cCLA from Tulane and @rtilla1 's awesome work is good to go 🎉 🎉 🎉


validates_uniqueness_of activerecord exception : Base table or view not found

I am trying out php active record, it's a great ORM, however I am at a standstill.
I have looked around google, blogs, phpactiverecord documentation as well as statckoverflow for days but have not been able to come across a suitable solution to this problem.
I am able to carry out the basic CRUD (insert,fetch, modify and delete) operations however as soon as i validate an object property using a static $validates_uniqueness_of filter, i get
Fatal error: Uncaught exception 'ActiveRecord\DatabaseException'
With message
exception 'PDOException' with message 'SQLSTATE[42S02]: Base table or view not found: 1146 Table 'test_ar.models' doesn't exist' in C:\wamp\www\test_AR\AR\lib\Connection.php on line 325
Here is the code i used in full.
set_model_directory('model');
        $cfg->set_connections(
                        array(
                         'development' => 'mysql://root:asdf@localhost/test_ar',
                         'test' => 'mysql://username:password@localhost/test_database_name',
                         'production' => 'mysql://username:password@localhost/production_database_name'
                        )
                );
        });

/*
class user extends ActiveRecord\Model
{

  static $validates_presence_of = array(array('username', 'message' => 'Please supply a username')); //this works just fine
  static $validates_uniqueness_of = array(array('username'));//this line causes the PDO exception

}
*/
$user = new user();

user::create((array('username'=>'mike','password'=>'test','created'=>time())));
$user::create(array('username'=>'mike')); //cannot even reach this line because of the exeption

References i have tried/looked at

https://github.com/kla/php-activerecord/issues/274 (though i don't really understand what's going on there)

http://www.phpactiverecord.org/projects/main/wiki/Validations#validates_uniqueness_of

http:// blog.felho.hu/what-is-new-in-php-53-part-2-late-static-binding.html

as well as many others.

Platform and php version

I am using php 5.3.4 and using nightly build (May 8 2013) I have almost failed to get my head around this. Please advise on how to correct this.

Does anyone have an example of this with the multi column unique?
I've put together a stackoverflow for this since its still an issue.  I made sure to give this thread credit.  http://stackoverflow.com/questions/43705005/phpactiverecord-validates-uniqueness-of-not-working/43705326#43705326

On that same stackoverflow post above I created an answer that you can easily drop in your models as a trait.


KOGITO-2313 - Support for node/events metadata attributes

It is now possible to add metadata attributes to all nodes.
JIRA: KOGITO-2313
Referenced Pull Requests:
https://github.com/kiegroup/kie-wb-common/pull/3659
VS Code: VSCode Extension

Not sure why but there is something strange happening with tests for standalone editors. If it is possible could you please check if you can run them locally with these changes I won't be able to get to that earlier than next week..

@Josephblt reviewed the code and I think we should add some unit tests for nodes that newly have advanced section. I think I went over all of the modified tests and did not notice new test cases, just mocks being fixed. Did I miss something?
Other than that is looks good.
Another question. Kinda offtopic, but this is touching its context, so I will ask :D
Would it be possible to add unqiue ID to global variables widget? They share the same widget as ProcessVariables and it makes it hard to find good locators for them in E2E tests.

@Josephblt One more thing I noticed, which I do not think is related but I kind reproduce this on master.
Notice the quick menu being a bit more white when over different node:

On master I see this:

Any ideas why is this present here?

Hey @domhanak
Thanks for the early feedback!
I'm agree on the issues aronud new metadata widget about dirty state and saving/reopening, so plz @Josephblt could you verify and fix? Those look basic usecases we should properly handle. Thx in advance!
About the contextual menu appearance being changed, it's a recent update we just did, by collaborating with UX. See https://issues.redhat.com/browse/KOGITO-4977
Thanks!

@Josephblt any updates here?

Jenkins run cdb
Jenkins run fdb

Hello @romartin, @domhanak. Yes. I fixed all issues here. Updated the PR and added the link with a .vsix for testing. You can go ahead with the testing.

@Josephblt hey could you please rerun the build to see the builds green? Once this is ready to merge, please let me know.
Thanks!

Jenkins do fdb

Hey @domhanak @ederign - you know if the CI integration tests are broken for this repo? Or this this specific PR which is causing issues during integration tests run? Looking at the generated errors, I looks not specific to these changes, and no way to make those work by re-firing jobs. Any ideas, and in case, is somebody is looking at this already? Thx in advance!

@karreiro can you help on this one tomorrow ?

@ederign @romartin, the CI was passing for the latest merged PR [1] and it's still passing for this PR (still open), so I believe the CI integration tests are fine for the repo :-)
[1] Latest merged:

@Josephblt it is safe to ignore the test failure in kie-editors-standalone we need to adjust the test to account for extra property row added by this PR.

Thanks @karreiro @domhanak for claritying.
@Josephblt  please rebase once possible, and I'll merge.
Thanks guys!

Hello @romartin. Rebased. Let's wait for the builds now. Thanks.

@kiegroup/gatekeepers This is good to be merged. Thank you!


Client Sided or Server Sided?

How can I make it so the mass updates to whatever mass the current player is.
For example
My high score is 5405
My current mass: 4544
How can I make mass: 5405 turn into mass: 4544 as if it is the same mass as the player?
great grammar alex well done.

Wwll, this is client sided, but I believe it may be easy for custom clients.

Alright.

In early clients, you can actually press ESC and then the play button, and your high score is updated to your current score. Say my high score was 22500 mass, and my current score is 19000 mass. Pressing ESC then the play button will change my high score from 22500 to 19000.

I know this lol, I just wanna do it as if it updated without pressing ESC.

I know that, but I don't think I'd know how, or bother to try lol

Client of Luka has this Feature built in.


Wine builds access

Hi,
what if I wanted to use one of your Wine build (https://github.com/Kron4ek/Wine-Builds) with Conty? I tried putting in the extracted Wine build folder (wine-6.13-staging-tkg-amd64) in /opt and build the conty image, however I couldn't get access to it with ./conty.sh /opt/wine-6.13-staging-tkg-amd64/bin/wine becaue it looks for it on my local hardrive. So I added a wine.sh in the /usr/bin of the conty.sh because I know conty will find it (inside the wine.sh is /opt/wine-6.13-staging-tkg-amd64/bin/wine "$@") but same thing, /opt/wine-6.13-staging-tkg-amd64/ which is inside the conty container is invisible to it.
So I proceeded differently, I moved wine-6.13-staging-tkg-amd64/ to my local machine's opt folder and kept wine.sh inside conty.sh and it finds it however i get this error which tells me it doesn't find other stuff:
0180:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
So my question is, how would I create a minimal conty.sh image (which I have already done) and put one of your wine builds inside of it and call wine?
Thank you.

Hi,
You can put it to /usr/local/wine-6.13-staging-tkg-amd64. And then execute it like:
./conty.sh /usr/local/wine-6.13-staging-tkg-amd64/bin/wine

Thank you for the reply. I did what you suggested but unfortunately I get the same error message about missing driver :
`Running Conty
wineserver: using server-side synchronization.
002c:fixme:winediag:LdrInitializeThunk Wine TkG (staging) 6.13 is a testing version containing experimental patches.
002c:fixme:winediag:LdrInitializeThunk Please don't report bugs about it on winehq.org and use https://github.com/Frogging-Family/wine-tkg-git/issues instead.
002c:err:wineboot:process_run_key Error running cmd L"C:\windows\system32\winemenubuilder.exe -a -r" (2).
0064:err:module:open_builtin_so_file failed to load .so lib "/usr/local/wine-6.13-staging-tkg-amd64/lib/wine/x86_64-unix/wineusb.sys.so"
0064:err:ntoskrnl:ZwLoadDriver failed to create driver L"\Registry\Machine\System\CurrentControlSet\Services\wineusb": c0000142
0034:fixme:service:scmdatabase_autostart_services Auto-start service L"wineusb" failed to start: 1114
0094:err:plugplay:try_add_device Building report descriptor failed, removing device
0094:err:plugplay:try_add_device Building report descriptor failed, removing device
0094:err:plugplay:try_add_device Building report descriptor failed, removing device
0094:err:plugplay:try_add_device Building report descriptor failed, removing device
0140:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
0140:err:winediag:nodrv_CreateWindow Unknown error (998).
0140:err:systray:initialize_systray Could not create tray window
017c:err:module:open_builtin_so_file failed to load .so lib "/usr/local/wine-6.13-staging-tkg-amd64/lib/wine/i386-unix/l3codeca.acm.so"
0184:err:module:open_builtin_so_file failed to load .so lib "/usr/local/wine-6.13-staging-tkg-amd64/lib/wine/x86_64-unix/l3codeca.acm.so"
0184:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
0184:err:winediag:nodrv_CreateWindow Unknown error (998).`
FWIW, conty works on my machine and I also managed to pack one of the now archived portable wines inside the image and it worked. I don't know why this is not working.

That's odd, it works for me. Maybe there is a regression in 6.13 release. Could you try staging-tkg-6.12 and maybe staging-6.12? No need to rebuild the image, you can just extract the builds to your home directory and then:
./conty.sh ~/wine-6.12-staging-tkg-amd64/bin/wine

Sorry, i don't know how to fix this midi error.

It seems like it's a pulse lib missing or something. I'll investigate, thanks again for your help.

Just to let you know that I took the time to try with wine-6.12-staging-tkg-amd64 and it's not working as opposed to wine staging. I get the :
0184:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.

Seems like some of the additional patches included in staging-tkg builds cause this. However, i can't reproduce this on my end, and since i can't, it's really hard to investigate this.

Ok

I will try to reproduce this on Debian 9, maybe this problem happens only there. You could try to find out the last staging-tkg version that works for you, 6.12 and 6.13 don't work, but maybe some relatively old version like 6.0 will.

I'll let you know when I do it, wine-6.3-5-proton-amd64 also has the same issue for me. However don't waste too much time on this, only when you have time it's not a priority. Thank you again :)

By the way, have you tried to creating a new clean Wine prefix? Like:
export WINEPREFIX=~/somenewprefix
./conty.sh ~/wine-6.12-staging-tkg-amd64/bin/winecfg

Yes, I did it with a new prefix everytime. I just tried wine-6.0-staging-tkg-amd64 and it's working. I'll try others tomorrow.

I tried to run it on Debian 9 now and staging-tkg-6.13 still works fine for me even there.
However, i found out that removing libxrandr or libxrender causes this "Application tried to create a window, but no driver could be loaded." error on new staging-tkg builds, while regular staging builds still work even without these libraries.
Given this, do you have libxrandr and libxrender installed inside the container? You can check like this:
./conty.sh ls /usr/lib | grep 'libXrandr\|libXrender'

The output should be:
libXrandr.so
libXrandr.so.2
libXrandr.so.2.2.0
libXrender.so
libXrender.so.1
libXrender.so.1.3.0

No I don't, and that's probably the problem. I just realized I was working with a conty.sh I had created myself and had removed some packages from the original create-arch-bootstrap.sh script. So that has to be the reason. Thank you so much for helping with this. I consider it solved now. Have a nice day :)


rename pytorch_lightning

pytorch_lightning has been renamed to just lightning
https://github.com/Lightning-AI/lightning/discussions/16688

closed by #49


Confirm MouseEvent.MOUSE_OVER and MOUSE_OUT on HTML5/next targets

This may not be enabled or working properly, we need to do testing and confirm that these events are properly implemented

Are there any plans to implement this?


eBPF use cases should be on the home page and have their own pages

should be on the home page and have their own pages

Don't remember what this was about


[Java] Add Opaque canonical extension type

Describe the enhancement requested
Part of https://github.com/apache/arrow/issues/43453
Component(s)
Java

Issue resolved by pull request 43460
https://github.com/apache/arrow/pull/43460


Show list states

Add a state chooser

Data loaded
Loading
No data
No search matches

 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.

Added to (repository list) a playground settings button, bottom right of screen

Added 'data loaded' state

Added no data state and component:

Added no search results state and component:

Example of no data state for pipelines list when inside repos tab view:


Do you have a plan to support LibMan?

Do you have a plan to support LibMan?
BTW: Some libs seems not supported in LibMan e.g. pnotify.buttons

@JeffreySu - none at the moment, but feel free to open up a PR and add support for LibMan if that's something you're interested in :)


Force CamelCase for --device-name

Could we have device-driver-cli force upper camel case for the driver name? Should be doable with the diff below:
diff --git a/cli/Cargo.toml b/cli/Cargo.toml
index 0ca4726..cb5d31c 100644
--- a/cli/Cargo.toml
+++ b/cli/Cargo.toml
@@ -13,6 +13,7 @@ readme = "README.md"

 [dependencies]
 clap = { version = "4.5.20", features = ["derive"] }
+convert_case = "0.6.0"
 device-driver-generation = { version = "1.0.0-rc.1", path = "../generation" }
 prettyplease = { version = "0.2.22", features = ["verbatim"] }
 syn = { version = "2.0" }
diff --git a/cli/src/main.rs b/cli/src/main.rs
index 5e875cf..34ef36a 100644
--- a/cli/src/main.rs
+++ b/cli/src/main.rs
@@ -1,4 +1,5 @@
 use clap::Parser;
+use convert_case::{Case, Casing};
 use std::{ops::Deref, path::PathBuf};

 #[derive(Parser, Debug)]
@@ -35,19 +36,19 @@ fn main() {
     let output = match extension.deref() {
         "json" => device_driver_generation::transform_json(
             &manifest_contents,
-            &args.device_name.to_string(),
+            &args.device_name.to_case(Case::UpperCamel),
         ),
         "yaml" => device_driver_generation::transform_yaml(
             &manifest_contents,
-            &args.device_name.to_string(),
+            &args.device_name.to_case(Case::UpperCamel),
         ),
         "toml" => device_driver_generation::transform_toml(
             &manifest_contents,
-            &args.device_name.to_string(),
+            &args.device_name.to_case(Case::UpperCamel),
         ),
         "dsl" => device_driver_generation::transform_dsl(
             syn::parse_str(&manifest_contents).expect("Could not (syn) parse the DSL"),
-            &args.device_name.to_string(),
+            &args.device_name.to_case(Case::UpperCamel),
         ),
         unknown => panic!("Unknown manifest file extension: '{unknown}'. Only 'dsl', 'json', 'yaml' and 'toml' are allowed."),
     };

I can send it as a PR if @diondokter agrees with the suggestion.

Thanks for bringing it up!


mx:DateValidator not working

Project Details are :
https://paste.apache.org/cpo8y
Test case is available on VDI.
Go to Feasibility from Select in Menu
Go to Product Grouping in Menu
File : com/dbz/modules/FS/dbzProductGroupingFeasilbilitySetup
In this setup we are using dbzSetupList
Path : DBIZCOMLIB\src\com\dbz\modules\components\dbzSetupList

I do not understand what problem is being described here.  I cannot tell from the attached image.  Please provide more detailed steps and what components are involved and what is expected and what is actually happening.  I do not know how to get the validators to run or where to set breakpoints.
I did notice the DateValidators are binding to HostFrm which is an Object and not some explicit type, so there is a good chance that the production version will not work as the minifier may rename those properties.

@aharui
the point where we called SetupList.HostFrm = this;
It was executed after the execution of line
<mx:DateValidator id="vldDate"  source = "{HostFrm.setupColumnDate}"  inputFormat="{parentMxmlDocument.parentMxmlDocument.parentMxmlDocument.appParams.DATE_FORMAT}" property="text"/>
In flex SetupList.HostFrm = this; was executed before
<mx:DateValidator id="vldDate"  source = "{HostFrm.setupColumnDate}"  inputFormat="{parentMxmlDocument.parentMxmlDocument.parentMxmlDocument.appParams.DATE_FORMAT}" property="text"/>


Don't try to stretch canvas if it's provided by the user

Currently we're stretching all canvas to parent size if width and height are not given, but it doesn't make sense to do that for user provided canvas DOM since they might already setup their ideal canvas size with html or css.

Hmm, I think there is a chance. I'll think more on this


CS line not being pulled low (EPD 2.9)

I'm using the EPD_Waveshare_2_9.ino example sketch and defined my pins as
#define CS 0
#define RST 15
#define DC 2
#define BUSY 16

I'm using I2C so I couldn't use pins 4&5.  Here is what I see on the logic analyzer:

If I unplug CS, it drops low, and the screen updates as expected in the example sketch.
I'm using a Huzzah Feather, per the pinout, it does not have an internal pullup.
Do I need to change any other definitions to use alternate pins?

Have you tried a different pin for CS ? I use gpio15 without any problems, Also, note on nearly all boards GPIO-0 has to be pulled high to ensure the ESP enters the correct boot mode, usually with a 10K resistor. The Huzzah has a pull-up on GPIO-0 as expected.
You can redefine the I2C pins provided they are not set by one of the libraries. Wire.begin(SDApin,SCLpin); Maybe rearrange your circuit.
Have you tried under programme control setting GPIO-0 low in a short no-frills simple programme that sets the pin low?
From the Huzzah notes:
'GPIO #0, which does not have an internal pullup, and is also connected to both a mini tactile switch and red LED. This pin is used by the ESP8266 to determine when to boot into the bootloader. If the pin is held low during power-up it will start bootloading! That said, you can always use it as an output, and blink the red LED.'
So does the RED led blink when the screen is being updated (i.e. CS is active)?


New Documents to add

November 30th Minutes.docx
New Minute file

November 30th Minutes.docx

New Minute file

I don't think that's very new...


Bug: Support more Types for Properties

Currently the Property case class only supports String values, however this does not work in cases such as Booleans, i.e. checked = true is not the same as checked = 'true'.
The underlying snabbdom lib accepts Any type, so we just need to enrich the API a little to support a few basic types:

Double
Int (not strictly necessary but...)
String
Boolean
JS Functions. (Functions.fun1 for example?)

The only slight complication to this is that it affects the code gen stuff that happens at build time.

This won't work in all cases, but for Booleans you can set a boolean property flag by doing this (because JavaScript has funny notions of 'truth'), where p is some predicate:
property("checked", if p then "true" else "")


Move updating the MSGraph DB to the SelectGuidancePlugin

We should move updating the MSGraph DB to the SelectGuidancePlugin. Right now, we update the DB on each Dev Proxy start, while the database is only used by the SelectGuidancePlugin, which is disabled by default. We should update the DB on the plugin registration instead which will eliminate the overhead for folks who don't use Dev Proxy with Microsoft Graph.

Great idea and will help with perf on start-up for non M365 scenarios 🚀


Use the latest build scripts for building php 7.2 in appveyor

This change is

Coverage remained the same at 73.228% when pulling 4d9cc3c15f222db90baf484043fc3dcc82fa17fa on yitam:php72 into 16e0b8961fd0bfa862a0d6df36258a2cfa5b4501 on Microsoft:dev.

Codecov Report

Merging #757 into dev will decrease coverage by 0.55%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              dev    #757      +/-   ##
=========================================
- Coverage   78.85%   78.3%   -0.56%
=========================================
  Files          25      25
  Lines        7157    7157
=========================================
- Hits         5644    5604      -40
- Misses       1513    1553      +40

Impacted Files
Coverage Δ

...php/x86/php-7.1.16-src/ext/pdo_sqlsrv/pdo_util.cpp

...-7.1.16-src/ext/pdo_sqlsrv/shared/core_results.cpp

...6/php-7.1.16-src/ext/sqlsrv/shared/core_stream.cpp

...rojects/php/x86/php-7.1.16-src/ext/sqlsrv/conn.cpp

.../php-7.1.16-src/ext/sqlsrv/shared/core_results.cpp

...x86/php-7.1.16-src/ext/sqlsrv/shared/core_util.cpp

...cts/php/x86/php-7.1.16-src/ext/sqlsrv/php_sqlsrv.h

...x86/php-7.1.16-src/ext/sqlsrv/shared/core_sqlsrv.h

...p/x86/php-7.1.16-src/ext/pdo_sqlsrv/pdo_parser.cpp

...php-7.1.16-src/ext/pdo_sqlsrv/shared/core_conn.cpp

... and 40 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 16e0b89...4d9cc3c. Read the comment docs.


PLAYWRIGHT_RESTART_DISCONNECTED_BROWSER not working on local browser

The handler is not allowing enough time for the new browser to launch after a crash.
Sample spider adapted from #167.
# crash.py
import os
from signal import SIGKILL

import psutil
import scrapy

class CrashSpider(scrapy.Spider):
    name = "crash"
    custom_settings = {
        "TWISTED_REACTOR": "twisted.internet.asyncioreactor.AsyncioSelectorReactor",
        "DOWNLOAD_HANDLERS": {
            "https": "scrapy_playwright.handler.ScrapyPlaywrightDownloadHandler",
        },
    }

    def start_requests(self):
        yield scrapy.Request("https://httpbin.org/get", meta={"playwright": True})

    def parse(self, response):
        print("request:{}".format(response.request.url))
        for proc in psutil.process_iter(["pid", "name"]):
            if proc.info["name"] == "chrome":
                os.kill(proc.info["pid"], SIGKILL)
        yield scrapy.Request("https://httpbin.org/headers", meta={"playwright": True})

$ scrapy runspider crash.py

(...)
2024-07-16 14:55:09 [scrapy.core.engine] INFO: Spider opened
2024-07-16 14:55:09 [scrapy.extensions.logstats] INFO: Crawled 0 pages (at 0 pages/min), scraped 0 items (at 0 items/min)
2024-07-16 14:55:09 [scrapy.extensions.telnet] INFO: Telnet console listening on <IP_ADDRESS>:6023
2024-07-16 14:55:09 [scrapy-playwright] INFO: Starting download handler
2024-07-16 14:55:14 [scrapy-playwright] INFO: Launching browser chromium
2024-07-16 14:55:14 [scrapy-playwright] INFO: Browser chromium launched
2024-07-16 14:55:14 [scrapy-playwright] DEBUG: Browser context started: 'default' (persistent=False, remote=False)
2024-07-16 14:55:14 [scrapy-playwright] DEBUG: [Context=default] New page created, page count is 1 (1 for all contexts)
2024-07-16 14:55:14 [scrapy-playwright] DEBUG: [Context=default] Request: <GET https://httpbin.org/get> (resource type: document)
2024-07-16 14:55:14 [scrapy-playwright] DEBUG: [Context=default] Response: <200 https://httpbin.org/get>
2024-07-16 14:55:14 [scrapy.core.engine] DEBUG: Crawled (200) <GET https://httpbin.org/get> (referer: None) ['playwright']
Response: <200 https://httpbin.org/get>
2024-07-16 14:55:14 [scrapy-playwright] DEBUG: Browser context closed: 'default' (persistent=False, remote=False)
2024-07-16 14:55:14 [scrapy-playwright] DEBUG: Browser disconnected
2024-07-16 14:55:15 [scrapy.core.scraper] ERROR: Error downloading <GET https://httpbin.org/headers>
Traceback (most recent call last):
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/twisted/internet/defer.py", line 1996, in _inlineCallbacks
    result = context.run(
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/twisted/python/failure.py", line 519, in throwExceptionIntoGenerator
    return g.throw(self.type, self.value, self.tb)
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/scrapy/core/downloader/middleware.py", line 54, in process_request
    return (yield download_func(request=request, spider=spider))
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/twisted/internet/defer.py", line 1248, in adapt
    extracted: _SelfResultT | Failure = result.result()
  File "/.../scrapy_playwright/handler.py", line 358, in _download_request
    page = await self._create_page(request=request, spider=spider)
  File "/.../scrapy_playwright/handler.py", line 286, in _create_page
    page = await ctx_wrapper.context.new_page()
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/playwright/async_api/_generated.py", line 12379, in new_page
    return mapping.from_impl(await self._impl_obj.new_page())
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/playwright/_impl/_browser_context.py", line 294, in new_page
    return from_channel(await self._channel.send("newPage"))
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/playwright/_impl/_connection.py", line 59, in send
    return await self._connection.wrap_api_call(
  File "/.../venv-scrapy-playwright/lib/python3.10/site-packages/playwright/_impl/_connection.py", line 514, in wrap_api_call
    raise rewrite_error(error, f"{parsed_st['apiName']}: {error}") from None
playwright._impl._errors.TargetClosedError: BrowserContext.new_page: Target page, context or browser has been closed
Browser logs:

<launching> /home/eugenio/.cache/ms-playwright/chromium-1117/chrome-linux/chrome --disable-field-trial-config --disable-background-networking --enable-features=NetworkService,NetworkServiceInProcess --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=ImprovedCookieControls,LazyFrameLoading,GlobalMediaControls,DestroyProfileOnBrowserClose,MediaRouter,DialMediaRouteProvider,AcceptCHFrame,AutoExpandDetailsElement,CertificateTransparencyComponentUpdater,AvoidUnnecessaryBeforeUnloadCheckSync,Translate,HttpsUpgrades,PaintHolding --allow-pre-commit-input --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --force-color-profile=srgb --metrics-recording-only --no-first-run --enable-automation --password-store=basic --use-mock-keychain --no-service-autorun --export-tagged-pdf --disable-search-engine-choice-screen --headless --hide-scrollbars --mute-audio --blink-settings=primaryHoverType=2,availableHoverTypes=2,primaryPointerType=4,availablePointerTypes=4 --no-sandbox --user-data-dir=/tmp/playwright_chromiumdev_profile-XXXXXXTy2tU6 --remote-debugging-pipe --no-startup-window
<launched> pid=59155
[pid=59155][err] [0716/145514.301003:INFO:config_dir_policy_loader.cc(118)] Skipping mandatory platform policies because no policy file was found at: /etc/chromium/policies/managed
[pid=59155][err] [0716/145514.301041:INFO:config_dir_policy_loader.cc(118)] Skipping recommended platform policies because no policy file was found at: /etc/chromium/policies/recommended
[pid=59155][err] [0716/145514.308584:WARNING:bluez_dbus_manager.cc(248)] Floss manager not present, cannot set Floss enable/disable.
[pid=59155][err] [0716/145514.343012:WARNING:sandbox_linux.cc(436)] InitializeSandbox() called with multiple threads in process gpu-process.
2024-07-16 14:55:15 [scrapy.core.engine] INFO: Closing spider (finished)
(...)

$ scrapy version -v
Scrapy       : 2.11.1
lxml         : <IP_ADDRESS>
libxml2      : 2.12.3
cssselect    : 1.2.0
parsel       : 1.8.1
w3lib        : 2.1.2
Twisted      : 23.10.0
Python       : 3.10.12 (main, Mar 22 2024, 16:50:05) [GCC 11.4.0]
pyOpenSSL    : 24.0.0 (OpenSSL 3.2.1 30 Jan 2024)
cryptography : 42.0.5
Platform     : Linux-6.5.0-41-generic-x86_64-with-glibc2.35

$ python -c "import scrapy_playwright; print(scrapy_playwright.__version__)"
0.0.39

I don't think this can be handled with locking or other synchronization primitives, as the browser crash could happen at any time. Retrying seems like the most sensible way.

Hi @elacuesta  in relation to my issue here: https://github.com/scrapy-plugins/scrapy-playwright/issues/294
I think the update you made to have the browser restarted worked for me! However I have this retry middleware enabled. I just find it weird that when the browser crashes, it does show up again but it seems that the retry middleware I've made doesn't necessarily retry that specific request anymore. I'm not sure why , it could be my middleware but just letting you know just in case. Let me know if you need additional info. Thank you.


Invalid username in register page

After entering invalid username and password in login page  and entering register page , the state of error is maintained and invalid username and password is displayed above register button

Assigned! If you any doubts contact us on discord!


pkg/operator: Cleanup the isReportFinished helper function

Update the pkg/operator/reports.go isReportFinished helper function, adding more general comments and deflating some of the if/else if/else conditionals to single if statements for better readability.

/retest

/retest


User Registration and Authentication

Allow users to create accounts and log in securely to access the platform's features.
Implement authentication mechanisms to ensure user data privacy and security.

I would work on this using firebases please assign it to me.

This issue has already been assigned to someone else. Please try selecting a different issue.

I would work on this using firebases please assign it to me.

already assigned to someone else


Will you release the training codes?

You can train our model with EDSR framework, any details are welcome to discuss with me


I don't trust you

I don't trust you! You only have thirteen (13) GitHub stars. Is this a malware?

No and you can look at the source code yourself to verify.

I don't care


Allow empty objects

The following is valid HCL:
variable "foo" {
  type = map
  default = {
    # explanatory comment
  }
}

Update the grammar to accept objects that contain newlines but no keys

Hi! I previously filed this PR against upstream python-hcl2 before I realized Checkov depends on this one instead. It looks like this fork has diverged quite a bit from upstream. Is this basically a separate project now, or is the plan to eventually merge your changes?

cc @arielkru (whenever you're able to take a look)

Thanks @santoshankr !
We currently maintain this fork as a separated project, so we can handle issues quickly in a way that fits best checkov users and the community, without depending on python-hcl2


Redirect location should also return json if you pass json header

If you make a request to httpstat.us/308 and you get a location back, right now it's httpstat.us/ which cannot return json.
I propose a version of this that sets Location to another status code, so that clients can test that their redirect logic keeps the correct headers.
Created a draft PR that just returns to /200, but could be configurable with a request param or header: https://github.com/aaronpowell/httpstatus/pull/112

Sounds like a good idea, let me put some notes on the PR

Closing as custom response headers can be provided, which would solve this.


[DEV-1927] add wallet apiKey

Release Checklist
Pre-Release

[ ] Check migrations

No database related infos (sqldb-xxx)
Impact on GS (new/removed columns)

[ ] Check for linter errors (in PR)
[ ] Test basic user operations

Login/logout
Registration
Update user data
Create/delete payment routes

Post-Release

Test basic user operations
Monitor application insights log

Add migration


⚠ Bofrost.de has degraded performance

In e70cb87, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1845 ms

Resolved: Bofrost.de performance has improved in 3ff10aa.


[Triton] InitLoadMultiple - ACE file at index 1 has incompatible metadata

Hello there. We use project acoustic (plugin 2022.1.410 ) with UE 5.1.1.
We try implement InitLoadMultiple function from triton with array of FTritonUnrealIOHook for loading more .ace files at once, but it's not work for us, we got error:
LogAcousticsRuntime: Error: [TritonRuntime::EncodedMapData::EncodedMapDataPrivate::Init, Ln474] ACE file at index 1 has incompatible metadata.
Moreover, if there is only one file in the array, everything will work like a normal call InitLoad()
Could you explain how we can set up our .ace files so they can loaded by InitLoadMultiple?

The incompatible metadata error occurs when the two .ace files have different global parameters. For example, if one .ace file was baked at the "coarse" setting and the other at the "fine" setting, you won't be able to load them both at once.

Thanks for you asnwer, we will check it & write result here

Thanks again for your reply, we re-baked the acoustic date with the same settings and everything worked as expected!


Update submodule and version 2.21.0

This PR has the following changes:

Updates the crypto submodule
Add ChangeLog entries for the changes brought in
Bumps the version to 2.21.0
Updates the release data in the ChangeLog header

All the steps in PR-HEAD passed successfully, only the result analysis step failed. We don't need the analysis right now and there isn't any tests in that step.


Add missing build step for "add-layer"

Add one missing step for adding the demo layer to the build
Signed-off-by: Philipp Ahmann<EMAIL_ADDRESS>

Current layer design is supported agl feature by templates/feature/elisa-cluster-demo.
In this case, not need to "bitbake-layers add-layer".
Can you tell me in which environment (jj? master?) need it?

Current layer design is supported agl feature by templates/feature/elisa-cluster-demo.
In this case, not need to "bitbake-layers add-layer".
Can you tell me in which environment (jj? master?) need it?

I thought so too based on your commit messages, and the template, but when I tried to build, it did not work unless I manually add-layer it.
I'll give it a try once more in the evening and let you know what exactly it complains about if I don't add the layers by hand

I thought so too based on your commit messages, and the template, but when I tried to build, it did not work unless I manually add-layer it.
I'll give it a try once more in the evening and let you know what exactly it complains about if I don't add the layers by hand

Gave it another try, whatever went wrong the first time around, a missing bitbake-layers add-layer was not the issue, it now builds just fine without it.
We can discard this pull request in my opinion, sorry for the confusion :P

Gave it another try, whatever went wrong the first time around, a missing bitbake-layers add-layer was not the issue, it now builds just fine without it.
We can discard this pull request in my opinion, sorry for the confusion :P

Thank you Jochen.
I try to clean build yesterday.  In this timing, meta-elisa automagical include was available too.

Thank you Jochen.
I try to clean build yesterday.  In this timing, meta-elisa automagical include was available too.

Thanks for clarification. It seems no action is required. I will close this PR.

Thanks for clarification. It seems no action is required. I will close this PR.


HHH-16830: apply filters to find() method

https://hibernate.atlassian.net/browse/HHH-16830

Good stuff. I am awaiting this feature with great anticipation! Thanks for the hard work.


Prevent upscaled SVG from exceeding Image bounds

Also expose Image MAX_WIDTH and MAX_HEIGHT.
Fixes #24455.

Cherry-picked for 3.0.7.


MRG: Set subject when simulating source

@nfoti I think removing the projections is a reasonable thing to do. Would this docstring have saved you a bit of annoyance when using simulate_evoked?
Note I also added auto stc.subject setting in simulate*_stc.

[Current coverage][cc-pull] is 86.23%

Merging [#3316][cc-pull] into [master][cc-base-branch] will decrease coverage by <.01%

@@             master      #3316   diff @@
==========================================
  Files           336        336
  Lines         57278      57282     +4
  Methods           0          0
  Messages          0          0
  Branches       8716       8716
==========================================
+ Hits          49397      49399     +2
- Misses         5215       5217     +2
  Partials       2666       2666

[][cc-pull]

Powered by Codecov. Last updated by [8a7921e...d066d68][cc-compare]
[cc-base-branch]: https://codecov.io/gh/mne-tools/mne-python/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/mne-tools/mne-python/compare/8a7921ed7d9c89887ac707608b1871e638adbf1a...d066d68312fa1624d1ae147d953530ece7a79a10
[cc-pull]: https://codecov.io/gh/mne-tools/mne-python/pull/3316?src=pr

Yes, that docstring is very helpful. Thanks.

Cool. @agramfort feel free to merge if you're happy.
BTW @nfoti this also includes a change such that after doing stc = simulate_sparse_stc(...) you shouldn't need to do stc.subject =  as it will be set automatically.

thx @Eric89GXL


Observation with India phone number parsing

`var region = 'IN';
var phone = '4155216849';
var phoneNumber = phoneUtil.parse(phone, region);
console.log(phone + ' isValid ' + phoneUtil.isValidNumber(phoneNumber));
console.log(phone + ' number type ' + phoneUtil.getNumberType(phoneNumber));
console.log(phoneUtil.format(phoneNumber, PNF.INTERNATIONAL));`
Output:
4155216849 isValid true

4155216849 number type 0

+91 4155 216 849

I tried dialing this number in India and seems its not a valid number in India.
Few references:
https://en.wikipedia.org/wiki/Telephone_numbers_in_India
https://en.wikipedia.org/wiki/Mobile_telephone_numbering_in_India

@saketanisha
We don't support single numbers per say , and validation of the number is based on the ranges/prefixes and not the actual validity of number itself. As range 415 is valid range in India so the number<PHONE_NUMBER> is returned as valid.
Thanks

I believe 415XXXXXXX is not a valid series in India. Can you please provide a valid number in Inida starting with 415.

Here is the numbering scheme from Department of Telecommunication. I spent a good amount of time matching the various combination of the number but don;t find any match. I believe the 91 reg ex needs a fresh look.


Combining flip-angle maps from different DREAM voltages

In PR #9, it was discovered that the scanner already outputs both the FA and RefV maps for DREAM scans in this comment, and that we can simply use those for the dataset. However, it was stated earlier in the discussion in this comment that the FA maps from different voltages should be combined to minimize bias.
At the moment, the bids_conversion.py script simply takes the DREAM scans (FA map and RefV) from the optimal reference voltage (x1.0), instead of combining different voltages into a single FA map. Is this an issue? Should we modify the script so that it combines the DREAM scans?

No, it is better if the notebook retrieves all 3 scans (0.66*refV, refV, 1.5refV) and combines them.

I apologize if this is obvious, but how would we go about combining these scans? This is something that is not clear to me, and I would appreciate some clarification; my understanding was that we would combine all 3 FAmap NIFTI scans into a single NIFTI file, and put this in the dataset. I am beginning to understand now though that you may have meant that we should simply take the average of the data we get in all 3 voltages when we process the data in the analysis code? So we don't actually combine the scans themselves, but the results acquired from the scans...?
Assuming you do mean that we should combine the 3 NIFTI scans from different voltages into a single NIFTI scan: As you recently stated in this comment from the original bids_conversion.py PR, the 0.66x and 1.5x scans are not currently included in the dataset.  Should I include them all in the dataset and do the combining in the analysis code, or handle this "combining" in the bids_conversion.py script and have it already be combined in the OpenNeuro dataset? For the same reason @jcohenadad recommended we put the processing code to generate the FAmap DREAM scans in the analysis code instead of in a pre-processing script here, my vote would be to put it in the analysis script.
If we put the FAmap DREAM scans for all 3 voltages in the OpenNeuro dataset however, how should we name these files to be BIDS compliant? The 1.0x FAmap scan is currently named sub-SUBJECT_acq-famp_TB1DREAM, so maybe sub-SUBJECT_acq-famp-0.66_TB1DREAM and sub-SUBJECT_acq-famp-1.5_TB1DREAM? Maybe @jcohenadad can pitch in on this?

The 3 DREAM scans should be part of the OpenNeuro dataset. We will combine them into one scan using the approach described here https://github.com/spinal-cord-7t/coil-qc-code/issues/7#issuecomment-1939582828 in the analysis script.
I'm not sure about the labeling, and what is and isn't accepted. I'll try to see if I can find examples.

Based on the acq- entry: https://bids-specification.readthedocs.io/en/stable/appendices/entities.html#acq I think what you proposed is ok.

I'm not sure yet if we need them or not. It might be that we have all the info we need from the flip angle maps, I'll consult with others. In the meantime add them, and we can remove them later if we don't need them.

This has been handled by 8522cc5. Can we close this issue?


feature request: center, print immediately

Pleas make center button or even auto center check box. at some point. Niimbot original software is slow to use with all bell &whistlers. I like to see web app is fast to use so that use can just click sticker area, write text and press print.

I need some clarification. What centering do you mean? Vertical or horizontal? Relative to what? Vertical or horizontal?
Or you just want to put object on the label center?

Thanks. Those buttons are what I mean.  I just wondering that 90% of all labels are just one text at label.
What you think about idea that by default sticker contain one text box with auto center/auto resize enabled. Then if user want modify layout them can disable those features by clicking property icons at object editor.
That will help that at most cases user need just click that text and edit it and hit print. If printer is not connected print will pop up that connect dialog. If printer is connected it just print it without asking anything. Minimum amout of clicks when you just want basic sticker.

Okay, I'll think about how to make it suitable for most of users.
For now, I've made new text to be centered.

Added centered text as new label template.

Added print without preview button.


Font buffer size is being calculated incorrectly

I believe the following lines should not take into account the number of elements in the TextVertex struct, but rather simply store the current number of tVerts.
https://github.com/Illation/ETEngine/blob/13f203b6731ca9c61f724234e1b30c06d48655a2/source/Engine/GraphicsHelper/TextRenderer.cpp#L168
https://github.com/Illation/ETEngine/blob/13f203b6731ca9c61f724234e1b30c06d48655a2/source/Engine/GraphicsHelper/TextRenderer.cpp#L209
E.g, the correct code would be
pFont->m_BufferStart = (int32)tVerts.size();

and
pFont->m_BufferSize = ((int32)tVerts.size()) - pFont->m_BufferStart;

The old text renderer was removed in favour of GUI rendering through RmlUI and imgui


ON-15296: Add unit test for conditional node labelling

The addKmmLabelsToNodes() method should not add the SFC KMM label to the node with the Onload KMM label with the value not matching the requested Onload CR's version. This is to not interfere with the upgrade workflow.

It's a bit of a corner case that this PR is trying to unit-test!


Q: why not use GroundingDINO as git submodule?

hi,
I'm wondering why not just use GroundingDINO(https://github.com/IDEA-Research/GroundingDINO) as submodule in this project?
is there difference btw the GroundingDINO repo and GroundingDINO directory here?  are they synced?
thanks a lot!

hi,
I'm wondering why not just use GroundingDINO(https://github.com/IDEA-Research/GroundingDINO) as submodule in this project? is there difference btw the GroundingDINO repo and GroundingDINO directory here? are they synced?
thanks a lot!

There's almost no difference, we only modified some visualization function under /Grounded-Segment-Anything/GroundingDINO, we just only forgot to make it as a submodule lol

wonderful work! thanks!


Chapter 1, example 09, book version doesn't print the count

The book version (page 18) that I have is:
#include <stdio.h>

main()
{
    long nc;

    nc = 0;
    while(getchar() != EOF)
        ++nc;
   printf("%ld\n",nc);
}

That makes printf() call outside the while loop. I don't know how to trigger EOF without terminating the program, effectively not getting the character count printed.

Check if maybe there's an errata.
Check Polish translation.

ctrl+d


Method Chaining

Feature Suggestion (Polished 1.93, Styled-Components 3.3.3)
Introduce ability to chain polished methods.
Current
const wrapped = setSaturation(0.5, lighten(0.45, 'red'))
Proposed
const chained = polished('red').lighten(0.45).setSaturation(0.5)
Happy to work on a PR if this doesn't already exist in some form and people think it will be useful.

@seanemmer Our preferred method for this is via composition, which all our color functions currently support.
@nikgraf Covers how that all works in this talk: https://www.youtube.com/watch?v=qJgff2spvzM
For your example it would look something like this:
import { compose } from 'ramda' // Replace with any compose() function of your choice
import { lighten, SetSaturation } from 'polished'

// Create setTone() helper
const setTone = compose(lighten(0.45), setSaturation(0.5))
const newRed = setTone('red')

We made a specific decision to go this route over the model other libraries use that require you to instantiate a color object in order to modify it.


Decals often break when playerstatus goes funky

AFAIK this shouldn't happen but it occasionally does on live:

I guess this is caused by players being connected before the system is initialized? I'm not all that sure though


[Freiburg] Data

http://www.freiburg.de/pb/,Lde/231355.html

done in #121  :)

Done in #122  (jetzt wirklich)  :)

Gemerged.

@kawie https://geoportal.freiburg.de/cgi-bin/mapserv643?map=/gisdata/gdm_pls/gdm_plslive.map&request=getfeature&service=wfs&version=1.1.0&typename=pls&outputformat=geojson&srsname=epsg:4326 currently only returns an empty feature collection. Is there an alternative source (I couldn't find one)?

According to http://www.freiburg.de/pb/,Lde/231355.html there is a server maintenance which causes the missing data. I'll close this issue once data is available again.


Removing mix.lock from maru_reload breaks build

Steps to reproduce:

clone the repo
cd maru_reload
rm mix.lock
mix deps.get
iex -S mix

Result:
** (Mix) Could not start application exfswatch: exited in: ExFSWatch.start(:normal, [])
    ** (EXIT) an exception was raised:
        ** (UndefinedFunctionError) function :fsevents.find_executable/0 is undefined (module :fsevents is not available)
            :fsevents.find_executable()
            (exfswatch) lib/exfswatch.ex:19: ExFSWatch.start/2
            (kernel) application_master.erl:273: :application_master.start_it_old/4

Disclaimer: I'm an Elixir noob so I might be doing something wrong here.

I'm sorry that's a bug of exfswatch.
I'm trying to fix it and build a new version.
You can add such line to your deps to fix it temporary {:fs, github: "synrc/fs", override: true}


Qt6

same PR but to develop instead of master

I had to do a rebase

It is clean now, please review

I'm having a very weird error now
  File "/usr/lib/python3.10/site-packages/videomorph/forms/videomorph.py", line 103, in _setup_model
    self.library.setup_converter(
  File "/usr/lib/python3.10/site-packages/videomorph/converter/converter.py", line 37, in setup_converter
    self._process.setProcessChannelMode(process_channel)
TypeError: setProcessChannelMode(self, mode: QProcess.ProcessChannelMode): argument 1 has unexpected type 'ProcessChannelMode'

May someone investigate


Introduce quotas per project

It does happen once in a while that training participants happen to scale to, say, 2000 pods just for the giggles and that will leave no resources on the machine to fix the situation. So for self protection, we should create quotas that limit the number of pods per project.

CC: @marekjelen ^^^

We need either to
a) Limit project per user [1]
b) Disable self-provisioning of projects [2]
c) Setup quota per project created by the tool [3]
or
a) Setup multi-project quota [4]
[1] https://docs.openshift.org/latest/admin_guide/managing_projects.html#limit-projects-per-user
[2] https://docs.openshift.org/latest/admin_guide/managing_projects.html#disabling-self-provisioning
[3] https://docs.openshift.org/latest/admin_guide/quota.html
[4] https://docs.openshift.org/latest/admin_guide/multiproject_quota.html


Standalone LSP Server in Rust

I came across this repo based on the announcement in Nu's changelog blog and I was keen to integrate that into my Neovim setup. However, I noticed that the LSP Server is integrated into this plugin and is based on Javascript.
I'm and others (see here) not a fan of tooling written in Javascript. Therefore, I'm wondering if it is possible to create a standalone LSP, written with the aid of lsp_server or tower-lsp, in its own repo.

Hey @schrieveslaach. We took the quickest route to get to something that provides more functionality than we currently have. I, for one, would love to see a standalone LSP, like you've mentioned. I kind of see this as a steppingstone to that. I am not well versed in LSPs, so I'd have to rely on others to lead the way but we'd welcome it.

This morning I was thinking about it and I was wondering if Nushell itself should provide an LSP backend. Maybe some sub command, such as nu language-server or something like that.
I'm imaging if this is provided and I use ctrl+o (opens your editor of choice to edit line input) my editor could start nu language-server --use-current-scope to get completions that take the current scope into account. For example, if I defined a new variable in the scope that pointed to a file and after that I typed ls into the terminal, it would be really great if the editor can suggest the variable as an argument to the ls command.
@fdncred, what do you think of integrating a language server into Nushell itself?

I think the dataframe support was removed from nu itself by default due to payload size
On my system, nu is already 50MB compared to 2MB for bash, so having a full-blown LSP implementation might bloat the binary again
That said, it's also possible that there's so much shared code between normal usage of nu and the LSP (parser, etc) that wrapping an HTTP service around it doesn't actually add much more to the final payload :shrug:

@schrieveslaach I think having a stand-alone LSP is a good idea and I'd like to see it.
@jokeyrhyme dataframe support wasn't removed from nushell, it's feature-gated now, so, only the people who want it can compile/install with it.

@fdncred that's exactly what I said

dataframe support was removed from nu itself by default

FWIW this is also blocking us from adding the nu LSP to Zed.

FWIW this is also blocking us from adding the nu LSP to Zed. I might take a crack at helping out at some later point :)

We'd love to have your help @mikayla-maki. We're currently using a LSP-lite strategy by calling into nushell with --ide-blah parameters. Zed could use this strategy as well, but it sounds like it's not what you're looking for.

Oh, so your server is implemented in nu itself? That's pretty sweet. Does that mean we just have to write a little wrapper for forwarding the LSP messages to the nu command?

Ooh, that does seem to be how it's implemented 👀 https://github.com/nushell/vscode-nushell-lang/blob/main/server/src/server.ts#L259-L277, very cool! I might try to do that tomorrow then, it's not too dissimilar from a side project I'm working on :)

yup, if you do nu --help, you'll see 5 or so --ide-something parameters. Those were all written to support our vscode integration, but it just spits out json, so it should be pretty easy to consume.

@fdncred, if the CLI options are already present, it should be possible to add an LSP crate on top of it. Or am I missing something?

I think it's just a case of there being so much work to do and only so many maintainers/volunteers
On the plus side, there's a decent amount of prior art, including this: https://github.com/IWANABETHATGUY/tower-lsp-boilerplate

@fdncred, if the CLI options are already present, it should be possible to add an LSP crate on top of it. Or am I missing something?

No, you're not missing anything really. I'm not sure our --ide-blah parameters exactly match to any LSP as I've never written an LSP. I think @jokeyrhyme is right about maintainers and time. There's also a component of, "I love to write X so I'm going to keep writing X and not Y". Open source is that way, of course.
We're definitely up for someone taking this code and building an LSP into nushell and I'm sure we'd help with it.

This is really ugly code to start with, but I have hover working and should have the rest working over the next couple of hours/days depending on unexpected distractions: https://github.com/jokeyrhyme/nuls
Very happy to accept contributions, and also to donate this to the nushell project should the official folks want it

@jokeyrhyme This is cool. Can you list some steps to try things out in vscode? :)

Yeah, it needs a README
I've been testing it with https://helix-editor.com/ where you only have to give the editor the path to the language server executable and everything just works
I'll see if I can figure out VSCode

Okay, it's weirdly quite a bit more involved to get VSCode working
It'll probably have to be a fork of this extension, or a new extension: https://code.visualstudio.com/api/language-extensions/language-server-extension-guide

I've updated the README.md: https://github.com/jokeyrhyme/nuls#readme
There's a whole bunch of other stuff to do, so I won't be focusing on getting nuls working with Visual Studio Code for now (you already have this extension) :)
Eventually, someone (maybe me) will raise a PR against this official Visual Studio Code extension to make it use nuls instead directly calling nu --ide-...

Thanks for updating it and the info!

Looking forward to integrating nuls once it's ready @jokeyrhyme! :D

I wonder if it's better to have separate nu and nuls executables or if nuls should be integrated into the nu binary?

Yeah, I do wonder about this, having it all in one executable could be one way to do it
Alternatively, it might eventually be appropriate to have an architecture similar to rustc versus rust-analyzer, where there might be some code that is shared between them, some functionality that is implemented differently to optimise for interactivity, and some functionality that is definitely only in one and not the other
For example, the language server has a JSONRPC implementation and often a TCP server, things that might not be useful to most users of nu (and might never be in the feature set of nu), whilst nu can actually execute/evaluate code which is something the language server might never need to do
It probably also depends on how large the nu executable is allowed to become?

I'm just guessing, but the way it's implemented now, you have to have both of them anyway. So, why not put them together. Plus, if they're together, it's more responsive because you don't have to shell out to call nushell with parameters, you could just call those functions directly. I'm fine with this continuing on as is but I think eventually, we may want to merge them together. Perhaps adding this as a nuls crate in nushell?

I definitely do want to explore having the language server import/use the Rust code from nushell directly rather than starting a new process over and over
It'd also mean being able to bypass the current round-trip through JSON

I although thought about implementing the language server in nushell itself, which would be kinda' awesome :)

I also thought about writing the language server in the nushell language itself, which would be kinda' awesome :)

LOL! That would be awesome!


[ChatGPT] - Problem when "Use Full Text Input" is activated

Extension
https://www.raycast.com/abielzulio/chatgpt
Description
Hi,
Better to show you a video :  Uploading Enregistrement de l’écran 2024-03-21 à 09.59.45.mov…
As you can see, the shortcut work well, but you can't see the result without returning back 2 times
Thank you for your help !
Steps To Reproduce
I've both activated : Auto-load Text and Use Full Text Input
I've defined a shortcut to use the text I've selected to push it directly to Ask ChatGPT, I see that it's working background but can't see the result
Current Behaviour
I need to go back 2 times to be able to see the result of ChatGPT
Expected Behaviour
I think the result have to be shown when ready ?

https://we.tl/t-OsRpdx0WsC

https://github.com/raycast/extensions/assets/20685961/6fa6aea0-c088-4f0f-aabd-21be3b2f55a5

@pernielsentikaer Does there have a window's focus hook?

No there is no such hook @bxb100


List available IPT

This page should list available IPT and have a link to start a new instance of each.
There are no requirements to show an information page for the IPT, so clicking on one will launch the IPT flow.

Here is the link to the plugin: https://www.npmjs.com/package/@warp-works/warpjs-ipt-plugin


BEP-0004

@tngTUDOR, following successful testing of https://github.com/brightway-lca/brightway-documentation/issues/13, I updated the PR. Ready for review now.


Adding support for dark mode

Codecov Report

Merging #93 (b54b714) into master (8f64183) will increase coverage by 79.57%.
The diff coverage is 58.82%.

:exclamation: Current head b54b714 differs from pull request most recent head 0d7e410. Consider uploading reports for the commit 0d7e410 to get more accurate results

@@             Coverage Diff             @@
##           master      #93       +/-   ##
===========================================
+ Coverage        0   79.57%   +79.57%
===========================================
  Files           0      114      +114
  Lines           0    10459    +10459
  Branches        0     1061     +1061
===========================================
+ Hits            0     8323     +8323
- Misses          0     2136     +2136

Flag
Coverage Δ

unittests
79.57% <58.82%> (?)

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

Impacted Files
Coverage Δ

packages/lib/components/common/theme-provider.tsx
9.55% <0.00%> (ø)

.../lib/components/auto-suggest/auto-suggest-menu.tsx
97.50% <100.00%> (ø)

packages/lib/components/dropdown/dropdown-menu.tsx
100.00% <100.00%> (ø)

packages/lib/components/tags/tag-item.tsx
100.00% <0.00%> (ø)

packages/lib/components/common/withOverlay.tsx
95.65% <0.00%> (ø)

packages/lib/components/list/list-item-content.tsx
100.00% <0.00%> (ø)

packages/lib/icons/arrow-right.tsx
38.88% <0.00%> (ø)

packages/lib/icons/more-vertical.tsx
38.88% <0.00%> (ø)

... and 107 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 8f64183...0d7e410. Read the comment docs.


Styles apply on next render when RTL toggle button is clicked

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I'm using MUI v4.0
When I click toggle button, select input styles apply on second render.

Expected behavior 🤔
No response
Steps to reproduce 🕹
Steps:
Just click toggle button a couple times.
Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Hi, you're describing a problem that occurs in v4. We are currently actively maintaining v5 only. If a similar problem exists in v5 as well, we can take a look at it.
I'm not also not able to verify the issue you described as the Codesandbox you linked does not work (throws Cannot read properties of undefined (reading 'button') immediately).


Bug with multi server - they start listen port and get connection by another

Example:

Start workerman Websocket server on port 8081 - all ok

Connect to server via another application (not workerman, client on local machine at c# language) - all ok

Start another workerman Websocket  server on 8082 - all ok

Server point 1 (8081)  create workerman AsyncTcpConnection to server from pint 3 (with port 8082)  - all ok (doesnt matter wich server connect AsyncTcpConnection  first)

Check after this 8081 port - not ok:

Master and Worker of server from point 3 (with 8082 port) start to listen 8081 port too (as Unix console say at the picture)

additional this server  from point 3 (port 8082) start to listen my client (what connect only to 8081 server from point 1) (as Lunix console say - you can see t the picture)

if i stop server from poit 1 (port 8081)  server from point 3 still Listen the port 8081 and client from pint 2 now have Estabilished connection with server 3 (port 8082)

Without point 2 situatian is same (server  8082 start to listen 8081 after 8082  get AsyncTcpConnection  from 8081  or  another side of AsyncTcpConnection   will - same result )

Well the problem is when start two workerman server and some one (doesnt matter who) will open AsyncTcpConnection to another - two servers will start to listen the port of server what start firstly

As i check it is happen when i try use "exec"  as command to start  one Werkerman from another Workerman

look like doesnt matter what i try execute from "exec"  command from workerman - new process open and listen the port already (i try exec filу with "Hello World" code and process already in Unix termonal listen the port)

look like it is problem in php. After "stream_socket_server" (in Worker.php) if you will try "fork" or "exec" or "shell_exec"  command to open new proccess - it will listen the port too .
But it not open another server - it still one. I was try fork and "unlisten"  it in new process but it unlisen in both process :)
Any idea how start new server from current in new process ?

Is the event extension installed?

Is the event extension installed?

yes - Swoole. I try open create Timer with "exec"  command to open new process with new workerman but result is same - new process will listen port too after it open (before eny code)

Workerman cannot change the features of PHP, such as the process inheritance listening socket when executing exec.
As for the fork problem, some extensions do have such problems, for example, the event extension will affect the behavior of the other processes when you fork process and perform certain operations.

Workerman cannot change the features of PHP, such as the process inheritance listening socket when executing exec. As for the fork problem, some extensions do have such problems, for example, the event extension will affect the behavior of the other processes when you fork process and perform certain operations.

i see...Is it possible save connection after Worker reload if it reloadable? Because if PHP work so NEW worker process (what will fork from master)  will have same socket connection as he have before

or maybe if save connections addres (ip and port)   possible use stream_socket_client  for get resource for new worker

PHP can not do that

but php already save the same socket  at multi process, i think it is real...because then i disconnect from one process another still get connection of client (but client not connect exactly to this process, he connect to process what was close)

at this pictires - it is two different workerman master +1 worker have same client to connect o same socket

may be if master process can collect connections too (as worker do)  and after reload worker fork byself with these connections...

i will create experiment

PHP can not do that

well it is 2 way but it possible

use https://www.php.net/manual/en/function.socket-wsaprotocol-info-export.php   but it only for windows
use https://www.php.net/manual/en/class.ffi.php  and find resource of socket in memory (as you can see it stiil alive at another process)  but need write code in "C"  language

Well for this reason i think i will use Gataway in my project :)

and normaly open another workerman from current workerman only via http , because as i say open via fork exec system shell - will get port's listening in another process - php work like this


i cant create react app need help

debdeb@Dave MINGW64 ~/Desktop/react sample1
$ npx create-react-app reactsample1
Creating a new React app in C:\Users\USER\Desktop\react sample1\reactsample1.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts...
npm ERR! path C:\Users\USER\Desktop\react sample1\reactsample1\node_modules\handlebars\bin\handlebars
npm ERR! code ENOENT
npm ERR! errno -4058
npm ERR! syscall chmod
npm ERR! enoent ENOENT: no such file or directory, chmod 'C:\Users\USER\Desktop\react sample1\reactsample1\node_modules\handlebars\bin\handlebars'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR!      C:\Users\USER\AppData\Roaming\npm-cache_logs\2019-02-23T01_23_05_972Z-debug.log
Aborting installation.
npm install --save --save-exact --loglevel error react react-dom react-scripts has failed.
Deleting generated file... node_modules
Deleting generated file... package.json
Deleting reactsample1/ from C:\Users\USER\Desktop\react sample1
Done.

Can you give the version of Windows you are running and/or version of Node.js?


Add synchronizer option to only keep complete games

To prevent having archives only containing the cue file or a common data/audio track shared by multiple games.

Add ability to add MIA lists, which are probably related.


[>>>>>>>>>>                                        ] 1022/5000, 1.5 task/s, elapsed: 673s, ETA:  2621s已杀死

测试模型到一半出现被杀死的情况，请问出现这个问题的原因是什么？该如何解决？

Can you provide more details, such as the environments, the version of mmdet and mmcv,  the command you run etc.
Without more details, if your system is Linux, I guess your process was killed by the system because it used too much memory and CPU. You can run the below command to check if the system killed your process :
sudo egrep -i -r 'killed process' /var/log

Linux detects changes in memory, and if it finds a process that is using too much memory, it is killed to protect the system. A possible solution is increasing the virtual memory.
Please give me more details if my guesses are not correct.

大佬你好，感谢你的解答。确实如你所说的，因为内存的原因出现了以上的错误，听从了你的建议增加了虚拟内存解决了这个问题。谢谢！


Datasplitter

See the Contribution Guidelines for a comprehensive guide on contributing to this project.
The short version of the Contribution Guidelines is:

PRs must have associated tests for new features
New features must be well documented
Code must adhere to the linting guidelines (PEP8, pycodestyle); don't worry, the CI will check this automatically.

Issues fixed:

List fixed issues here (e.g., #1)

Changes proposed in this pull request:

List your changes here!

Just to confirm beep/prediction/init.py is designed to be empty

Yeh it is


Issue 317: Graph naming convention

I see two main issues with the proposed regex matching:

It will fail for alphabetic characters outside the basic Latin alphabet. I think it's important to support a far broader sweep of Unicode code points, to allow internationalised graph content.

This includes "non-English" characters, such as å, ä, ö, ü etc.

It doesn't permit backtick specification of non-alphabetic strings:

Non-alphabetic characters, including numbers, symbols and whitespace characters, can be used in names, but must be escaped using backticks. For example: ^n, 1first, $$n, and my variable has spaces. Database names are an exception and may include dots without the need for escaping. For example: naming a database foo.bar.baz is perfectly valid.

Have quoted those side-comments from the Neo4J docs: https://neo4j.com/docs/cypher-manual/current/syntax/naming/

Your concern with item 1 is understandable. @rafsun42 can you address this and see if those can be added?
However, item 2, is due to potential security holes that allowing those characters could enable. For now, this restriction needs to stay in place.

@rafsun42 have the changes for item 1 of @TropicalPenguin request been added? Also, can you rebase this pull request off the current master.

@jrgemignani I added the changes for the item 1.

Looks much more comprehensive; thanks!

Also, my build returned these errors -
src/backend/utils/name_validation.c:57:36: warning: universal character names are only valid in C++ and C99 [enabled by default]
src/backend/utils/name_validation.c:57:36: warning: universal character names are only valid in C++ and C99 [enabled by default]
src/backend/utils/name_validation.c:57:36: warning: universal character names are only valid in C++ and C99 [enabled by default]
src/backend/utils/name_validation.c:57:36: warning: universal character names are only valid in C++ and C99 [enabled by default]
src/backend/utils/name_validation.c:57:36: warning: universal character names are only valid in C++ and C99 [enabled by default]

Oid create_label(args) should implement the methods is_valid_vertex_label(label_name_str) and is_valid_edge_label(label_name_str) since many internal functions use create_label(args) to create vertices and edges.

@jrgemignani I updated the code the follow the conventions and also implemented changes suggested by @susano0.

@rafsun42 Only 2 minor adjustments need to be done. First, it needs to be rebased off of the master. Second, there are a few "white" spaces that need to be removed -

@jrgemignani Updated.

@jrgemignani Updated.

I looked and the whitespace is still there in src/backend/utils/name_validation.c

@jrgemignani Updated.

I looked and the whitespace is still there in src/backend/utils/name_validation.c

@jrgemignani I checked, but I could not see the whitespaces in name_validation.c.
However, there were some white spaces in the test sql files, which I thought were outside the coding conventions. Regardless, I updated them.

@rafsun42 Are you using vi or vim? If so -
[jgemignani@cent7nuc7i5bnk POSTGRESQL]$ cat ~jgemignani/.vimrc
highlight ExtraWhitespace ctermbg=red guibg=red
match ExtraWhitespace /\s\+$/
let w:m2=matchadd('ErrorMsg', '\%>80v.\+', -1)
[jgemignani@cent7nuc7i5bnk POSTGRESQL]$

@rafsun42 Are you using vi or vim? If so -
[jgemignani@cent7nuc7i5bnk POSTGRESQL]$ cat ~jgemignani/.vimrc
highlight ExtraWhitespace ctermbg=red guibg=red
match ExtraWhitespace /\s\+$/
let w:m2=matchadd('ErrorMsg', '\%>80v.\+', -1)
[jgemignani@cent7nuc7i5bnk POSTGRESQL]$

@jrgemignani I applied your vim config. I could verify red highlight works by adding extra space. But, name_validation.c has no highlight. This is what the bottom part of the file looks like.

@rafsun42 Are you using vi or vim? If so -
[jgemignani@cent7nuc7i5bnk POSTGRESQL]$ cat ~jgemignani/.vimrc
highlight ExtraWhitespace ctermbg=red guibg=red
match ExtraWhitespace /\s\+$/
let w:m2=matchadd('ErrorMsg', '\%>80v.\+', -1)
[jgemignani@cent7nuc7i5bnk POSTGRESQL]$

@jrgemignani I applied your vim config. I could verify red highlight works by adding extra space. But, name_validation.c has no highlight. This is what the bottom part of the file looks like.

Okay, I will try again. Maybe there was some weird glitch.

@rafsun42 looks good!


Support icon field under [package] in Ballerina.toml

Description:
As the first part of https://github.com/ballerina-platform/ballerina-spec/issues/988 we should $subject in order to be able to specify an image to be used as an icon in GUI based registries.
Syntax:
Ballerina.toml
[package]
org = "ballerinax"
name = "app1"
version = "1.0.0"
icon = "app1.svg"

icon image should be added to the /docs directory inside bala.

@azinneera @hevayo We only allow .svg files here?


Removal of ES/Solr code paths

Codecov Report

Merging #1951 (3dd6c34) into master (5af657d) will increase coverage by 1.40%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1951      +/-   ##
============================================
+ Coverage     58.70%   60.10%   +1.40%
+ Complexity     1079     1061      -18
============================================
  Files           180      178       -2
  Lines         10429     9886     -543
  Branches       1430     1370      -60
============================================
- Hits           6122     5942     -180
+ Misses         3786     3439     -347
+ Partials        521      505      -16

Impacted Files
Coverage Δ

...s/TweetLowerCaseEntityPreservingFilterFactory.java
0.00% <ø> (ø)

src/main/java/io/anserini/index/IndexArgs.java
100.00% <ø> (ø)

.../main/java/io/anserini/rerank/ScoredDocuments.java
27.27% <ø> (-9.10%)
:arrow_down:

...main/java/io/anserini/search/SearchCollection.java
41.88% <ø> (ø)

...c/main/java/io/anserini/index/IndexCollection.java
59.02% <100.00%> (+13.98%)
:arrow_up:

...erini/index/generator/WashingtonPostGenerator.java
0.00% <0.00%> (-34.49%)
: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 5af657d...3dd6c34. Read the comment docs.


Update API Collection and Sample Code color

Update API Collection and Sample Code color
Bug/issue #, if applicable: 91451310
Summary
New color for hero section of API Collection and Sample Code
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[ ] Added tests - NA
[x] Ran npm test, and it succeeded
[ ] Updated documentation if necessary - NA

@swift-ci test


gitlab-mr-discussion fail to get diff

Hello,
I'm currently trying to get reviewdog to work with my local Gitlab instance.
However I get the following error:
reviewdog: fail to get diff: GET https://my.local.instance/api/v4/projects/haynes/webgoat/merge_requests/3: 404 {message: 404 Project Not Found}
This seems to be because reviewdos does not urlencode the namespace.
When I change the URL like this:
https://my.local.instance/api/v4/projects/haynes%2Fwebgoat/merge_requests/3
So when I change the / in the project path to %2F (an URL encoded /) I get the expected result
See also:
https://docs.gitlab.com/ee/api/rest/#namespaced-path-encoding
Any help with this would be greatly appreciated.
Tested with reviewdog version: 0.17.1
Gitlab Version: 16.8.1
edit:
After investigating a bit more I think this line is what's causing the issue:
https://github.com/reviewdog/reviewdog/blob/master/service/gitlab/gitlab_mr_discussion.go#L55
It probably should be projects: url.QueryEscape(owner + "/" + repo),
I didn't verify that assumption yet though.

I've created a draft PR to address this. Still need to build it locally and test it against my gitlab instance. It's been quite a while since I worked with go so I need to figure out how to release binaries again.

hmm closed the PR again after testing the changes locally. I'm not sure what's happening here.
The go-gitlab module already applies a method to urlescape the path. I don't know why that doesn't seem to trigger in my case.


🏗 Remove "Cast" heading when no cast is available

If no cast is available for a given show, the entire cast section should not be displayed, including the heading.
Screenshot

Closed by #1104


Edit button in documentation takes to broken link

https://github.com/sonarwhal/sonarwhal/blob/master/docs/user-guide/rules/rule-content-type.md
should take to
https://github.com/sonarwhal/sonarwhal/blob/master/packages/rule-content-type/README.md

There might be some issues because not all the documentation pages are in the same place (nor are they all README.md)


v.<IP_ADDRESS> Source code does not compile

The source code provided for release v.<IP_ADDRESS> (https://github.com/aspnetboilerplate/module-zero/archive/v<IP_ADDRESS>.zip) does not compile. Found errors on Zero and then in Zero.EntityFramework.

We can compile it. What errors are you getting?

I'm getting a  "'Abp.Zero.Configuration.AbpZeroConfig.EntityTypes.get' must declare a body because it is not marked abstract or extern. Automatically implemented properties must define both get and set accessors." at Abp.Zero\Zero\Configuration\AbpZeroConfig.cs, line 22
public IAbpZeroEntityTypes EntityTypes { get; }
If I corret to:
public IAbpZeroEntityTypes EntityTypes { get; set; }
then I get an error "Invalid expression term '['" in Abp.Zero.EntityFrameworkError.AbpZeroDbMigrator.cs, line 34.
 var args = new DbPerTenantConnectionStringResolveArgs(null, MultiTenancySides.Host)             {                 ["DbContextType"] = typeof(TDbContext)             };

I suppose you are using Visual Studio 2013 (C# 5) and we used C# 6 features (with VS 2015). I just tried in VS 2013 and got the same error. I will revert to C# 5.


Updated README.md with pull request instructions

i have added pull req section

@Charul00 please make the proper PR request and add issue number

@Charul00 please make the proper PR request and add issue number

ok

Pull Request Section
Title: Pull Request is Missing from the README.md Table of Contents
Description
This pull request aims to add a dedicated section for pull requests in the README file, ensuring it is included in the table of contents. The link to this section will be made clickable for easy navigation.
Related Issue
Fixes: #18
Changes Made

Added a "Pull Request" section in the README.md file.
Ensured that the links within the README are clickable, enhancing user experience and accessibility.

Checklist

[ ] I have tested my code and it works as expected.
[ ] I have added/updated documentation where necessary.
[ ] I have included tests for new features.
[ ] I have followed the project's coding standards.

Screenshots (if applicable)

@Charul00 please make the proper PR request and add issue number

yeah i made this check please

@Charul00 we have already a md file of Pull request only you want to add the path to access it

@Charul00 we have already a md file of Pull request only you want to add the path to access it

do i have to add path of my pr ?

@Charul00 please join this link for better communication https://discord.gg/w4nKprr5

@Charul00 your code is having conflicts

@Charul00 your code is having conflicts

ok
i resolve asap


Added a couple more words

Most of them pertain to me and server but can be useful elsewhere

's won't work, because only space is a word divider. If you remove that one I'll merge it in!


source code can still be seen using a bytecode viewer

the classes get obfuscated, but when looking at the obfuscated jar's subclasses through a bytecode viewer, (JByteEdit) it appears as regular non-obfuscated code

this guy is based

Can you share console output?


Could not find a version that satisfies the requirement openvino==2022.3.0

Hello， Could you help to check this issue when I install on Win11?
I run the below command in step 2 of windows installation part, it report the version error message of python module .
openvino-ai-plugins-gimp\install.bat
Thanks.
**** openvino-ai-plugins-gimp Setup started ****
Requirement already satisfied: virtualenv in c:\users\jiaqi\appdata\local\programs\python\python311\lib\site-packages (20.21.0)
Requirement already satisfied: distlib<1,>=0.3.6 in c:\users\jiaqi\appdata\local\programs\python\python311\lib\site-packages (from virtualenv) (0.3.6)
Requirement already satisfied: filelock<4,>=3.4.1 in c:\users\jiaqi\appdata\local\programs\python\python311\lib\site-packages (from virtualenv) (3.12.0)
Requirement already satisfied: platformdirs<4,>=2.4 in c:\users\jiaqi\appdata\local\programs\python\python311\lib\site-packages (from virtualenv) (3.2.0)
created virtual environment CPython3.11.3.final.0-64 in 211ms
  creator CPython3Windows(dest=D:\project\gimpenv3, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, setuptools=bundle, wheel=bundle, via=copy, app_data_dir=C:\Users\jiaqi\AppData\Local\pypa\virtualenv)
    added seed packages: pip==23.0.1, setuptools==67.4.0, wheel==0.38.4
  activators BashActivator,BatchActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator
ERROR: Could not find a version that satisfies the requirement openvino==2022.3.0 (from versions: none)
ERROR: No matching distribution found for openvino==2022.3.0

[notice] A new release of pip is available: 23.0.1 -> 23.1
[notice] To update, run: D:\project\gimpenv3\Scripts\python.exe -m pip install --upgrade pip
Processing d:\project\openvino-ai-plugins-gimp
  Preparing metadata (setup.py) ... done
Collecting numpy
  Using cached numpy-1.24.2-cp311-cp311-win_amd64.whl (14.8 MB)
Collecting scipy
  Using cached scipy-1.10.1-cp311-cp311-win_amd64.whl (42.2 MB)
Collecting typing
  Using cached typing-<IP_ADDRESS>.tar.gz (78 kB)
  Preparing metadata (setup.py) ... done
Collecting gdown
  Using cached gdown-4.7.1-py3-none-any.whl (15 kB)
Collecting requests
  Using cached requests-2.28.2-py3-none-any.whl (62 kB)
Collecting opencv-python<=4.3
  Using cached opencv_python-<IP_ADDRESS>-cp36-abi3-win_amd64.whl (31.3 MB)
Collecting scikit-image
  Using cached scikit_image-0.20.0-cp311-cp311-win_amd64.whl (23.7 MB)
Collecting timm==0.4.5
  Using cached timm-0.4.5-py3-none-any.whl (287 kB)
Collecting transformers
  Using cached transformers-4.28.1-py3-none-any.whl (7.0 MB)
Collecting diffusers
  Using cached diffusers-0.15.1-py3-none-any.whl (851 kB)
ERROR: Ignored the following versions that require a different python version: 1.21.2 Requires-Python >=3.7,<3.11; 1.21.3 Requires-Python >=3.7,<3.11; 1.21.4 Requires-Python >=3.7,<3.11; 1.21.5 Requires-Python >=3.7,<3.11; 1.21.6 Requires-Python >=3.7,<3.11; 1.6.2 Requires-Python >=3.7,<3.10; 1.6.3 Requires-Python >=3.7,<3.10; 1.7.0 Requires-Python >=3.7,<3.10; 1.7.0rc1 Requires-Python >=3.7,<3.10; 1.7.0rc2 Requires-Python >=3.7,<3.10; 1.7.1 Requires-Python >=3.7,<3.10; 1.7.2 Requires-Python >=3.7,<3.11; 1.7.3 Requires-Python >=3.7,<3.11; 1.8.0 Requires-Python >=3.8,<3.11; 1.8.0rc1 Requires-Python >=3.8,<3.11; 1.8.0rc2 Requires-Python >=3.8,<3.11; 1.8.0rc3 Requires-Python >=3.8,<3.11; 1.8.0rc4 Requires-Python >=3.8,<3.11; 1.8.1 Requires-Python >=3.8,<3.11; <IP_ADDRESS> Requires-Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, <3.5; <IP_ADDRESS> Requires-Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, <3.5
ERROR: Could not find a version that satisfies the requirement openvino==2022.3.0 (from gimpopenvino) (from versions: none)
ERROR: No matching distribution found for openvino==2022.3.0

[notice] A new release of pip is available: 23.0.1 -> 23.1
[notice] To update, run: D:\project\gimpenv3\Scripts\python.exe -m pip install --upgrade pip
*** openvino-ai-plugins-gimp Installed ***
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'gimpopenvino'
**** openvino-ai-plugins-gimp Setup Ended ****

Hi @Be27f , looks like you are using python 3.11. I don't think there is (yet) an openvino pip distribution for that version yet. Can you try with python 3.10?

@RyanMetcalfeInt8  yes , you are right.  The issue disappear after downgrade to 3.10.
Thank you so much for your help.
BR.


no weather entities created

I installed via HACS and set API key. only activated weather - but here is no entitie created nor a lovelace card. only the default weather forecast from hass shows up.
any suggestion how to debug?

found this in log:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/pirateweather/weather.py", line 181, in async_setup_entry
pw_weather = PirateWeather(name, unique_id, forecast_mode, weather_coordinator, outputRound)
File "/config/custom_components/pirateweather/weather.py", line 219, in init
self._ds_currently = self._weather_coordinator.data.currently()
AttributeError: 'NoneType' object has no attribute 'currently'

Hi,
Thanks for using this integration, and apologies for the issue here! The error message was key here, and let me work out the cause of this bug- it comes from a race condition when the integration comes up before the internet does.
As a workaround, if you select "Reload" from the integration UI, everything should come back online. I'll push a more permanent fix as soon as I work out how to get around this!

reloaded - no change - same error message.
i use core in docker - maybe this has influence?

I also had the same issue, on Hass OS (no docker).
I solved my issue by going into the api page and "subscribing" (i should have read the docs more closely but its all here https://pirateweather.net/getting-started)

Thanks for posting your solution @googanhiem, I suspect that's the issue here as well.
I'll be pushing out an update to this integration today that makes this error (and resolution) much more visible, which should hopefully help fix these issues in the future

ah - ok. thats definitly not self explanatory.
so i reinstalled and put like described for the days:  0-7, but got error message below. even changing it later (to 0,1,2,3,4,5,6) within the integration setup didn't  change the error message.
Suggest to make the description more clear, add sanity check and prefill suggestion as reference.
Error setting up entry PirateWeather for pirateweather
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 372, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/pirateweather/__init__.py", line 86, in async_setup_entry
    forecast_days = [int(i) for i in forecast_days]
  File "/config/custom_components/pirateweather/__init__.py", line 86, in <listcomp>
    forecast_days = [int(i) for i in forecast_days]
ValueError: invalid literal for int() with base 10: '0-7'

That's a great suggestion, and just pushed that change in the latest version (v1.02)! There's now a line of text underneath the box that clarifies the syntax, and the error messages should be clearer when something is wrong.
To fix this issue, you may have to remove and add the integration in again unfortunately. Since it keeps hitting the error when loading the config, I don't know if it will be able to accept the corrected syntax

This integration looks great and thank you for the hard work.
However, I'm having a similar issue with no sensors created although I have subscribed to the API.
My steps:

Installing on Home Assistant Supervised core-2023.1.2, supervisor-2022.12.1, Debian 11
Confirm pirateweather.net account info. APIs subscribed and authorized:

Install HACS PW 1.02 integration. Restart HA.
Add and configure HA PW integration. Don't really understand the "forecast sensors in CSV" bit yet but 0,1,2,3,4,5,6 seems reasonable to start...

Click Submit. "Success! Created configuration for PirateWeather."
Result: 1 entity available: weather.pirateweather. HA Weather Card works successfully. No sensors have been created.

Enable debug logging on HA PW integration and restart HA.
Check /usr/share/hassio/homeassistant/home-assistant.log. Strangely(?), no information from PW except the usual HA complaint about an untested integration.
Lot of data in the weather.pirateweather entity "Info" tab but still no sensor entities...

Any help appreciated and please pardon if I'm overlooking something obvious. Thanks in advance.

Since this change has been pushed out and seems to resolve things, I'm going to go ahead and close this issue. Definitely feel free to either re-open or create a new issue if anything comes up though!


AsyncScalr setServiceThreadCount potential resource leak

Allowing custom ExecutorServices set on the AsyncScalr class was not a good design choice.
Currently the impl of setServiceThreadCount leaks any replaced ThreadPool when it isn't of type ThreadPoolExecutor.

Removed.
As mentioned in other AsyncScalr bugs, the class was rewritten with subclasses in-mind. Much easier to use and customize now.


Add ExternalDNS

ExternalDNS would give us a Kubernetes-native way of managing DNS. Right now we need to go into our Route53 console to add/remove DNS entries.

Linked Jira ticket: https://jira.lsstcorp.org/browse/DM-21293


Custom Functions Tutorial needs to be updated

Article URL
https://docs.microsoft.com/en-us/office/dev/add-ins/tutorials/excel-tutorial-create-custom-functions?tabs=excel-windows
Issue

Tutorial talks about calling stock prices.
We need to update to reflect new stock end point and API key is required so the function calls continue to work as the underlying API changes to reflect new model

We should update:
var url = "https://api.iextrading.com/1.0/stock/" + ticker + "/price";
with
//Note: you will need to register for an account to get a token on https://iexcloud.io/
var url = "https://cloud.iexapis.com/stable/stock/" + ticker + "/quote/latestPrice?token=<YOUR_TOKEN_HERE>"

API key fix is in and a PR is being prepped with some samples that use an alternative API. Thank you for reporting this and using the p1 tag - that helps us keep track of what's the most important fix. :) Apologies that this fix went in later than we anticipated.


Synchronization

Hi, we will use 6 cameras to record the mouse behaviors.
our camera resolution is up to 5120*5120 with 60 fps.
but considering the calculation ability and data storage and transfer, we want to reduce the resolution and frame rate.
I would like to ask for your suggestion about the most optimal parameters for the recording and network training.
besides, I wonder about the effect of timestamp deviation. if we cannot trigger at the beginning of recording but collect the timestamp. I found there is ~15ms deviation between the same frame across different cameras. please kindly advise if the deviation of frames has an effect on the network training and prediction.
Thank you very much.

5120*5120 is pretty large, and unfortunately much of the additional resolution (depending on how large your FOV is) will be discounted when sampling into 3D volumes. However, 60 fps is good, and I would recommend stay at that framerate if possible. How large is your arena? We normally use at least 1024 pixels to cover an 8 inch arena, and this works well for us.
~15 ms deviation is not ideal, but it's also not the end of the world, as long as you are sure there is not going to be drift over the course of the recording. You'll have to make use of the data_frame arrays inside label3d_dannce.mat to encode the temporal offsets in each camera. That is, at absolute time t (aka a single index into the data_frame and data_sampleID arrays), data_frame will contain the corresponding frame index for each camera. For example, if Camera1 is 15 ms (~1 frame) ahead of Camera2, then sync{1}.data_frame[0] == 1 and sync{2}.data_frame[0] == 2. You can calculate these data_frame arrays from the recorded timestamps using dannce/utils/preprocessData.m once the timestamps are formatted as in dannce/utils/exampleMatchedframes.mat.

Closing for now. Please re-open if you want to continue the discussion about this.


Idea: Add Type Marker (à la tern.js)

Hi @nmn,
I was playing with atom plugins and flow when I found your package, great work!
I was thinking that a really good addition would be to display an overlay as you provide function parameters so you would receive feedback of each type.

If you are wondering how are they doing this, you can have a look here
What do you think?

flow does not support this kind of feedback. very sad, since this is totally a basic function.

Hi @aight8,
The info is there... Have a look to what I get using Nuclide

I also check the command output looking for the function signature and is there:
{
  "name": "componentDidUpdate",
  "type": "",
  "func_details": {
    "return_type": "void",
    "params": [
      {
        "name": "nextProps",
        "type": ""
      },
      {
        "name": "nextState",
        "type": "void"
      },
      {
        "name": "nextContext",
        "type": "any"
      },
      {
        "name": "component",
        "type": "any"
      }
    ]
  },
  "path": "/tmp/flow/flowlib_30940de9/react.js",
  "line": 38,
  "endline": 38,
  "start": 3,
  "end": 96
}

Is it possible to return this information also if the start/end point is in building up function?
function method(a: number, b: string) {}

Request autocompletion on cursors position (some use cases):
method(|)
method(|, "bla")
method(123, |)
I don't know exactly the behaviour of typescript - but typescript AC in vscode works exactly how you except it. In flow autocompletion it's much more a hope if it can analyse the building-up code correctly and suggest something useful.
As addition I don't see any active argument marker, but this is the smallest problem if above example not work? Any idea?

Looks interesting. I'll work on it some time. No promises on timeline though.


Remove Multi CPU Core link from README

The "Multi CPU Core Support" link in the README.md points to a section removed in commit 70db0d1.
Remove the dead link.

Pull Request Test Coverage Report for Build 192

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

Totals

Change from base Build 190:
0.0%

Covered Lines:
640

Relevant Lines:
656

💛  - Coveralls


[bug] Listing display whitespace

The listing display is buggy, I've tested it on Chrome, Opera and Safari:

I'm guessing they only need to update to the latest version of Monocular, because hub.kubeapps.com don't seem to have the same issue.

Related to #212

This was fixed in upstream Monocular. Since the Helm Hub is now redirecting to the Artifact Hub, I'm closing this issue as it's no longer relevant.