PHP stops working from time to time, WPN-XM should keep it always online

PHP stops working quite often. While using other software (eg Winginx) I never had this issue.
WPN-XM should keep PHP always online.
My Environment:
OS: Win 8.1 x64 ru
PHP: 5.6.5 x64 single threaded
WPN-XM: 0.8.0
Error logs: none

This is not helpful. Environment (OS and architecture for example)? Error logs?

Thank you for reporting this issue, Dumitru.
The issue about PHP crashes and instability came up before, see https://github.com/WPN-XM/WPN-XM/issues/202
As mentioned in the posts there, one solution would be to hide the crashes by using services https://github.com/WPN-XM/WPN-XM/issues/165
An PHP crash dump, trace or error log would be nice.

While using other software (eg Winginx) I never had this issue.
Ok, that brings up the question: what do they do differently?
A short look at their configuration doesn't reveal anything.
Maybe they use services to start the PHP daemon or monitor the daemon from the Tray app.
But who knows ... except Alexey, because Winginx is not open-source.

I can not raise the major version to stable, if the stack still has this issue.
This issue must be solved before v1.0.0.
I will put a strong focus on this issue, after finishing the next release.
I'm also looking forward receiving contributions, which tackle this problem.

Ok, i took a look at Winginx again.
They have a tool around called "spawn-fcgi". Originally, this tool comes from the lighttpd web server.
I'm not sure, if PHP runs more stable if daemons are started that way.
http://wiki.habariproject.org/en/Installation_using_Nginx

There is no error in the error log.
At first I thought there is something wrong with my PHP source code, but it crashes while running two different frameworks - one is very complex and the other one is very simple.
I ran same version of PHP as a service using winsw.exe and it still keeps crashing (service just stops without any error in the logs).
I upgraded to PHP 5.6.5, same thing, with WPN-XM and as a service it just stops. Sometimes after 10 min of working, sometimes after 2 hours. The more request I run, the quicker it stops.
I don't know what winginx does, but I notice it spawns three processes for each version of PHP. It might be spawn-fcgi.
No matter what the issue is (and there might be many issues with PHP and extensions), I think WPN-XN should keep PHP always running.
I also think it would be nice to have more PHP processes running, otherwise it gets very slow while developing heavy PHP applications. One of my projects makes a lot of request to PHP at any page load (JS, CSS, Images, AJAX requests all go through PHP) and with one PHP process it load the page in about 10 sec, while with more PHP processes it loads in only 3 sec.
I'll be more then happy to help when I have more free time.

I like the idea of spawn-fcgi.
I'll do some experiments with it later on.
I'm going to run PHP as a service with winsw.exe and spawn-fcgi.exe and see what happens.
In the case it works good, I'll post back my results here.

It looks like spawn-fcgi does its job quite good.
I couldn't run it as a service, I guess because the spawn-fcgi process exists immediately after it spawns PHP child processes.
But I managed to create a start-php.cmd that spawns few PHP processes:
REM start-php.cmd
@echo off
set _dir=%~dp0
set PHPRC=%_dir%\php.ini
spawn-fcgi -f "%_dir%\php-cgi.exe" -a <IP_ADDRESS> -p 9056 -C 6 -F 4 -P "%_dir%..\temp\php.pid"

And here is my stop-php.cmd file:
REM stop-php.cmd
@echo off
taskkill /f /IM php-cgi.exe

My stop-php.cmd is not perfect, cause it kills any php-cgi.exe process.
It is possible to catch all PIDs that spawn-fcgi forks and kill only them when user stops PHP.
I guess Winginx does the same.

Thanks you for testing, Dumitru!
This is very valuable information for me.

I ran same version of PHP as a service using winsw.exe and it still keeps crashing (service just stops without any error in the logs).

This is really odd. Maybe i should request help from the PHP team to help us track this down.
PHP is not stable on Windows - and it's somehow related to sockets/cgi/spawning.

I also think it would be nice to have more PHP processes running

:+1:  This is a feature request. I will open a new issue for this.
I think i will add multiple configurations, so that the user can select, if he wants to run one, two or a pool of php daemons. Nginx would act as a balancer and distribute requests to the pool, like described here: https://github.com/WPN-XM/WPN-XM/issues/202#issuecomment-45089295
This will improve the performance a lot.
But it will not change a thing in regard to stability, because the whole pool crashes.

My stop-php.cmd is not perfect, cause it kills any php-cgi.exe process.

Yes, we do the same currently with the WPN-XM SCP and start/stop scripts.
It's not really good, but works. In the future, i want to do a more fine-grained shutdown, maybe related to the WPN-XM folder. So that the stack is only responsible for the daemons living inside it's own folder structure. I'm open for ideas in this area.

I found the source-code of spawn-fcgi over at https://github.com/lighttpd/spawn-fcgi/blob/master/src/spawn-fcgi.c
The main functions are bind_socket and fcgi_spawn_connection, the rest is argument handling and permissions. I will take a closer look at this, maybe i can incorporate some parts into the SCP.

I'm experiencing random shutdown of php-fpm under Win 8.1 x64 as well, on two different systems - nothing appears in the log, no error message anywhere, the service just shuts down quietly.
I am able to repeat the crashes by just firing concurrent requests at the server, e.g. ab -n 1000 -c 20 will make it crash almost every time - this is with two different codebases on two different systems.
Perhaps an upgrade from 5.6.6 to 5.6.9 is worth attempting first thing? Could be this has been fixed already. Unlikely though - the PHP team tends to not care much about the Windows build...

Unlikely though - the PHP team tends to not care much about the Windows build...

Rumors say so.. but they care - or maybe i'm just an optimist.
The real downside is, that there are only a few guys working "actively" on the Windows parts (Pierre, Anatol).
If someone, properly identifies the issue in a reproducible manner, it won't be a problem to get a fix into the core. I still think this/these issue are CGI related - but that's just guessing around.
If you can get segfaults or a back trace from PHP, so that we can narrow it down to code and open a bugreport over at php.net.
You will need the Debug Pack and a diag tool, like Debug Diagnose.
Referencing: https://bugs.php.net/bugs-generating-backtrace-win32.php
This whole procedure of tracing a PHP crash on Windows is still a high barrier.
Btw: i've already added "LiteRC" installers to test PHP Release Candidates versions.
The next step is to provide the tools to do backtraces for PHP out-of-the-box.
That means to add the matching "Debug Pack", a "Diagnose Tool" and "ab".
Not sure, if i'm able to redistribute the "Microsoft Debug Diagnostic Tools",
but there should be alternatives.

Okay, the plot thickens. This is weird. Really, really weird.
So if I run ab -n 1000 -c 20, fpm shuts down consistently after between 500-519 requests.
But if I run ab -n 1000 -c 1, it shuts down after precisely 500 requests, every time.
It's like there's a hard 500 request limit built into this binary and it just consistently stops when it reaches that number.
I wonder if somebody left a diagnostic statement in there somewhere checking for the number of request to reach 500 or something?? Really that's my best guess ;-)
Why 500? Precisely 500 - every time, consistently repeatable. Why?

You are running into the FCGI_MAX_REQUEST limit.
The PHP FastCGI processes exit after handling 500 requests by default.
It may exit later (at any time), if more requests are made.
So you could end up with "517 requests + hit limit + exit".
This event is an error, which should be logged and is reported as "500 Internal Server Error" to the client.
You should not hit a limit of 500, because of fastcgi_param  PHP_FCGI_MAX_REQUESTS 1000; in nginx.conf. But the directive seems to be ignored, when set from Nginx Config
https://github.com/WPN-XM/WPN-XM/blob/master/configs/nginx.conf#L132

If the directive is set on the ENV: SET PHP_FCGI_MAX_REQUESTS=0
start-wpnxm.bat
:start-php
    echo Starting PHP FastCGI...
    set PHP_FCGI_MAX_REQUESTS=0
    set PHP_FCGI_CHILDREN=2
    %HIDECONSOLE% %cd%\bin\php\php-cgi.exe -b <IP_ADDRESS>:9100 -c %cd%\bin\php\php.ini
    echo.
goto END

index.php
<?php echo 'Hello World!';

> WeigHTTP.exe -n 20000 -c 20 -t 2 http://localhost/index.php

WeigHTTP - Version 0.3 (Win32)
A lightweight and simple webserver benchmarking tool...

Starting benchmark...

Spawning thread #1: 10 concurrent requests, 10000 total requests.
Spawning thread #2: 10 concurrent requests, 10000 total requests.
Progress:  10% done.
Progress:  20% done.
Progress:  30% done.
Progress:  40% done.
Progress:  50% done.
Progress:  60% done.
Progress:  70% done.
Progress:  80% done.
Progress:  90% done.
Progress: 100% done.

Finished in 9 sec, 704 millisec and 0 microsec,
            2061 req/s, 456 kbyte/s.

Requests: 20000 total, 20000 started, 20000 done,
          20000 succeeded, 0 failed, 0 errored.

Status codes: 20000 2xx, 0 3xx, 0 4xx, 0 5xx.

Traffic: 4540000 bytes total, 4120000 bytes http, 420000 bytes data.

> WeigHTTP.exe -n 50000 -c 200 -t 2 http://localhost/index.php

WeigHTTP - Version 0.3 (Win32)
A lightweight and simple webserver benchmarking tool...

Starting benchmark...

Spawning thread #1: 100 concurrent requests, 25000 total requests.
Spawning thread #2: 100 concurrent requests, 25000 total requests.
Progress:  10% done.
Progress:  20% done.
Progress:  30% done.
Progress:  40% done.
Progress:  50% done.
Progress:  60% done.
Progress:  70% done.
Progress:  80% done.
Progress:  90% done.
Progress: 100% done.

Finished in 25 sec, 40 millisec and 0 microsec,
            1996 req/s, 442 kbyte/s.

Requests: 50000 total, 50000 started, 50000 done,
          50000 succeeded, 0 failed, 0 errored.

Status codes: 50000 2xx, 0 3xx, 0 4xx, 0 5xx.

Traffic: 11350000 bytes total, 10300000 bytes http, 1050000 bytes data.

But this doesn't change a thing in regard to the two issues (when a limit is set):

when the limit is hit there is no error logged. it's empty.
recycling / respawning never happens.

@jakoch got it, thanks! added a global environment variable PHP_FCGI_MAX_REQUESTS=0 and it's working now. (this being a dev server, I don't worry about memory fragmentation etc.)
Off-topic, but my other big problem with this new stack is this issue which I suspect could be either the Windows build of PHP 5.6 or xdebug. (or php-code-coverage itself of course, I'm waiting to hear back from Sebastian Bergmann.)

Glad to hear that it's working now!
I will add the directive to the SCP start, too.
Mem-wise: yes, that's why i initially set 1000.
Anyway: disabling max requests is not that bad for a "dev" stack.
--
In regard to your "php-code-coverage" issue:
From the stack-perspective i can only advise to (a) check the components xdebug and php for compatibility and (b) apply some performance tweaks to PHP and Xdebug.
PHP: By default the OpCode Cache is disabled for PHP56 (https://github.com/WPN-XM/WPN-XM/blob/master/configs/php/php56.ini-dev#L1887). If not already done, set opcache.enable=1 in php.ini and tweak some of the other values to get a speed-up. Not sure, if this helps.
Xdebug: You can also play around with XDEBUG_CC_UNUSED, XDEBUG_CC_DEAD_CODE.

On the code coverage issue, I downgraded to xdebug 2.2.7 (latest 2.2 release) and the performance problem is gone. (XAMPP still distributes their stack with xdebug 2.2, I wonder if this is the reason.)

Glad you solved that issue, too.
Maybe related: http://bugs.xdebug.org/view.php?id=1123


datmo task run creates commits to the master

datmo task run created commits to the user's branch without creating the datmo refs. Had to change the code in create_refs to reset to a previous commit id and create the refs from the latest commit id. Resolved.

fixed with PR #115


PHPUnit 5.7 doesn't support PHP 7.4

Recent build breakages have highlighted that the mocking system of PHPUnit 5.7 (well, phpunit-mock-objects 3.4) uses an API that was documented as deprecated in PHP 7.1 and started throwing deprecation errors in PHP 7.1
PHPUnit 8 solves that, but we're not using that in SilverStripe 4, and although we might be able to get away with using it in SS 4.5+, now that we've dropped php 5.6 / 7.10, it likely has breaking API changes that would force SS4 test suites to be rewritten, which wouldn't be okay.
Possible solutions:

Patch phpunit-mock-objects, hopefully via a PR to the upstream project, or by creating a fork of our own
Drop support for php 7.4, meaning that SS4 has an externally driven end-date of 6 Dec 2021

I'm not happy about the 2nd as this caused us a lot of pain with SS3. So I think we need to patch php-mock-objects. Ideally we can bump up to php-mock-objects 4, 5, or 6 breaking the APIs we use and Sebastian will accept a PR to those more recent versions and produce a stable tag. But failing that we will probably want to publish a fork.

The mock-objects repo is readonly and so we'd need to publish a fork to address this.

We may need to disable the php7.4 build until this ticket is addressed, then, @maxime-rainville...

We'd probably need to run a fork of it, since PHPUnit 5.x has been out of support for almost 18 months now

#9063 should still run the 7.4 build, but allow it to fail without failing the entire build.

Forking mock-objects feels dirty, but I don't see any other viable alternative.

I've set up https://github.com/sminnee/phpunit-mock-objects and it's got a branch with the fix in place; should I publish sminnee/phpunit-mock-objects that provides phpunit/phpunit-mock-objects:3.4.5? I'm happy to be "the maintainer", such as it is.

Before we continue, is this something that every module and project will need to opt in to using when they require phpunit 5.7? If so, that seems totally un-ideal to me. We may be able to hack in a class_alias() based replacement for a shim somewhere in framework instead?

Wouldn't we better to have it under the open sausage org? That way if we need to update it, we won't have to annoy you all the time.

If you want to take responsibility for it then it should be github.com/silverstripe, IMO. Team repos should never publish anything.

Or it could be maxime. But I can give you commit rights on the sminnee repo if you want

Actually is this something we'll expect regular dev to use as well? Presumably, many outside project use a set up relatively similar to ours for testing, so they'll run in the same issue.

Before we continue, is this something that every module and project will need to opt in to using when they require phpunit 5.7? If so, that seems totally un-ideal to me. We may be able to hack in a class_alias() based replacement for a shim somewhere in framework instead?

It only applies to projects using the PHPUnit mock system that want to run on PHP 7.4.

It only applies to projects using the PHPUnit mock system that want to run on PHP 7.4.

Just checking - by this do you mean any user code that uses $this->createMock() in a unit test?

Or $this->getMockBuilder() as EmailTest does for example, but yeah that's the idea.

I'd say that'd be pretty widespread in the community. Let's see what the author replies to your email with, and if there's no interest then I think we should try a hacky shim in framework rather than a package fork that everybody would need to remember to install.

Noting that this is the crux of @sminnee's patch, for reference: https://github.com/sminnee/phpunit-mock-objects/commit/64650615dbde4041483cab2609293c3da3bd7042


JVFloatLabeledTextView placeholder text appears in 2 line when it is used in tableviewCell.

JVFloatLabeledTextView placeholder text appears in 2 line when it is used in tableviewCell
Initially it works fine. when we remove the text from textview it becomes 2 lines, and after that never resets itself.
PFA  of issue below :

I cannot reproduce. Please provide example code to demonstrate the issue if it still occurs.


[Bug] When "Vague Details" is enabled for saves, the numbers are still visible

I've set saves to have vague descriptions, but the numbers are still visible. Possibly a bug, given that if I try to set misinformation for them, it uses the proper Ex/Hi/Mod/Lo/Ter dropdown rather than a numeric field.
Module v<IP_ADDRESS> (Edit: Still appears to be happening after I updated to v<IP_ADDRESS>)
PF2e v6.2.2
Foundry v12.331

Just leaving a note here that I've fixed this issue and it will be included in the next little release. I just want to bring out a few small features aswell 😄

I've just released v<IP_ADDRESS>. It fixes the vague saves again 😁


device does not use trusted execution

'device does not use trusted execution environment' error in discover Readers stripe, why this error is happening

Hi @salarmiirzaeie, can you please file provide some background information about your issue here using the bug report template? This will help us understand what you're running into here.

Closing this due to inactivity, feel free to open a new one using the bug report template if you are still using this issue


VS code typescript errors

We have written the needed data into your clipboard because it was too large to send. Please paste.

@alland67  You missed to copy the actual data into this issue report.

/needsMoreInfo

Please remove it immediately and make it your top priority. To me this is almost equally offensive as a swastika.


CO-2034_Mis-calculation_of_Enrollment_Flow_Approvers_priviledges

Calculation of Approver's permissions during an Enrolment Flow are buggy

@boshrin concluded the changes discussed during the Dev call.


adapt vernacular names

brought union list species common names in line with iasregulation.be; added missing dutch common names; corrected types

Thanks @timadriaens, I am runnig the mapping and building the website.


+$+Full VIDEOS Dolphin Ayan Video Original Viral Video on Social X Twitter #1

05 minutes ago — [アニプレックス] While several avenues exist to view the highly praised film Dolphin Ayan online streaming.
Full VIDEOS Dolphin Ayan Video Original Viral Video on Twitter
𝖶𝖺𝗍𝖼𝗁 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤►𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤►𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
Dolphin Ayan L.eaked Video New collections of Dolphin Ayan L.eaked Video now being a creator on Fanfix uploading adult contents. Social media star Dolphin Ayan L.eaked Video been posting short Videoos and naughty pics on Tiktok platform for a while now.
The purported leak has stirred a maelanage of reactions, from disbelief to morbid curiosity, marking yet another chapter in the saga of celebrity scandals that have dotted the landscape of pop culture. However, this isn't the first rodeo for Dolphin Ayan L.eaked Videowhen it comes to rumors of a sex tape. In March 2024, similar whispers emerged, only to be debunked as baseless. The rapper, known for hits like In Ha Mood vehemently denied the rumors, critiquing the eagerness of some to believe in such falsehoods.
On Monday March 18, formerly known as Twitter, became abuzz with speculation that a L.eaked sex tape featuring Dolphin Ayan L.eaked Videowas readily available online. However, instead of the, there are dozens of pages promising to release the Videoo in exchange for interaction with their post.
The recurrent theme of L.eaked tapes and the subsequent fallout serves as a reminder of the fragility of reputation in the digital era. As the lines between private and public life continue to blur, celebrities like Dolphin Ayan L.eaked Videofind themselves at the mercy of internet chatter, where a rumor can ignite a firestorm of speculation and judgment.
In the ever evolving landscape of celebrity culture, the Ishowspeedscandal underscores the relentless pursuit of sensationalism, a pursuit that often comes at the expense of truth and dignity. As we navigate the complexities of the digital age, the line between entertainment and exploitation remains perilously thin.
As the situation unfolds, the truth remains shrouded in mystery, leaving the public to ponder the authenticity of the rumors. In a world where fame and infamy are two sides of the same coin, the saga of Ishowspeedis a testament to the power of social media to shape narratives and challenge the boundaries of privacy and consent.


 I do this in the middleware by updating request.path, however I also need some way to update request.headers or request.accept (these are both currently properties)
I propose either changing request.accept / request.headers to attributes or adding a request.set_header(name, value) method
Happy to pick this up if it's :ok_hand:

Hi @fraser-lowndes !
If really needed, one could just change that as nothing is truly private in Python. FWIW, particularly so after merging https://github.com/falconry/falcon/commit/793f30491dca86e62530ff1fb4126e9201e3ff12 . However, I am afraid the API was designed with treating these attributes as read-only in mind (hence a fat warning in the mentioned commit). Based on this, at least personally I am not completely convinced it is the direction we want to go.
However, at the same time we are striving after making Falcon as useful as possible. Getting back to your case, could this possibly be solved in an alternative way?

Do you have control over the method handling the modified request? In this case, could you consider setting custom attributes on req.context to convey the same information? NB that we have not updated the docs yet, but Falcon 2.0 would support both setting attributes, and using the mapping interface (which would be considered deprecated though), on the context object.
Alternatively, if you know you are employing req.accept to handle this, be it your own or 3rd party component, you may consider subclassing Request to override the accept property in a way that it is possible to set it, storing the information in a new attribute or context attribute, and supplying the API with your custom request_type.

Would the above make sense wrt your use case?
cheers
Vytas

Hi @vytas7, thanks for the suggestions. Could you go over the reasoning for designing the request object as primarily read-only? I haven't used this framework much, but from my perspective it seems like shooting yourself in the foot as you lose a lot of ability for powerful routing / middleware.

Hi again @fraser-lowndes ,
I will let the original creator and project lead @kgriffs chime in on the original design.
That design makes sense to me as well: the incoming request is treated as input data, to which the response is rendered. Many popular web frameworks and servers, including the ubiquitous Nginx, adopt variations of it. You may of course ask about path, but that could be treated as internal redirect/rewriting request, again, found in many popular servers. Routing in Falcon is, at the time of writing, based solely on path.
What is more, permitting to set anything would likely increase maintenance burden and inflate code base, since we cache various derived properties in internal attributes etc, i.e. the design assumptions allows us to make optimizations and simplifications.
Furthermore, looking into the future (see also here: https://github.com/falconry/falcon/issues/1394), we may want to speed up Falcon even further by providing alternative implementation of critical code paths (that would very likely include the most of API, Request and Response classes) in Cython, Rust, or whatever the future brings us. I am very obviously just speculating now, but it is not unreasonable that we will want to keep much of the request data and property parsing inside lower level languages, if we choose to go that way. Supporting fully writable request interface might add further burden in that case.
I will nevertheless label your issue a proposal to gather more feedback from other maintainers and our community.
Getting back to your task at hand once again, a yet another way could be doing something along the lines:
class Resource:

    def on_get(self, req, resp):
        if self._accepts_image(req.accept):
            self._render_as_image(req, resp, ...)
        else:
            self._render_as_something_else(req, resp, ...)

    def on_get_image(self, req, resp):
        self._render_as_image(req, resp, ...)

api = falcon.API()
resource = Resource()

api.add_route('resource/1', resource)
api.add_route('image/resource/1', resource, suffix='image')

Here we are using alternate routes to the same resource, where one of them is suffixed: https://falcon.readthedocs.io/en/latest/api/api.html#falcon.API.add_route
Maybe that looks more elegant than my previous suggestions?

Thanks for the code example @vytas7, I didn't know about the suffix parameter. That's a good enough compromise. I would have liked the generic middleware, so I can keep my routes cleaner and have a single method (e.g. on_get) for determining the appropriate content type, but I appreciate that the speed and simplicity of falcon comes with some constraints / compromises. I'd like to leave this issue open to gather any other feedback, though my specific issue is resolved so I will leave it to your discretion how long this is left open. Thank you very much for your assistance.

That's true, it is not in any stable release yet :grimacing:
However, we hope to get Falcon 2.0 beta out the door really soon, some time next week.
For the time being, you could install Git master directly, for instance, by using
pip install git+https://github.com/falconry/falcon

Okay, in the meantime I've changed the image URL to /resources/1.png which allows me to check the ending of path and go from there. I wish I'd have thought of this first, much simpler than any middleware or routing..

Casual encounters

@tiffchad I've just seen your comment, really curious what that's about... You created an account just to comment on this issue with that cryptic message? Weird. I'm gonna be trying to figure this out for ages...


Problem with the code "examples/graph_saint.py"

🐛 Bug

I tried to run the "examples/graph_saint.py", but the code stayed at the line 16:
loader = GraphSAINTRandomWalkSampler(data, batch_size=6000, walk_length=2,
                                     num_steps=5, sample_coverage=1000,
                                     save_dir=dataset.processed_dir,
                                     num_workers=4)"

The terminal displayed this:
Compute GraphSAINT normalization:   0%|   |

And I downloaded the office code, "https://github.com/GraphSAINT/GraphSAINT", it works well and I got results in few seconds.
I wonder if there is any problem here or maybe I am wrong.
Thanks a lot.
To Reproduce
cd <dir>;  python graph_saint.py

Environment

OS:  Linux
Python version: 3.7.7
PyTorch version: 1.5.0
CUDA/cuDNN version: 10.1
GCC version: 4.8.5
Any other relevant information:

Can you try setting num_workers to 0? There might be a problem with threading that I need to look into.

Yes, setting num_workers to 0 works.


hci_spi_st: send ACI config synchronously

There's no reason to send these async, and since the reply isn't waited for anywhere, this can cause the HCI buffer pool to be exhausted and bt_enable() to fail.

More reviews needed here.


fix(i18n): update Chinese translation of responsive web design projects

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

LGTM. Thanks for all your efforts!
This is ready for merging 🎉 @raisedadead

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Unknown error when trying to set up openmediavault intergration

Here is the log
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/aiohttp/web_protocol.py", line 422, in _handle_request
resp = await self._request_handler(request)
File "/usr/local/lib/python3.9/site-packages/aiohttp/web_app.py", line 499, in _handle
resp = await handler(request)
File "/usr/local/lib/python3.9/site-packages/aiohttp/web_middlewares.py", line 119, in impl
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 98, in forwarded_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 24, in request_context_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 78, in ban_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 138, in auth_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 135, in handle
result = await result
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 157, in post
return await super().post(request, flow_id)
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 62, in wrapper
result = await method(view, request, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 246, in async_configure
result = await self._async_handle_step(flow, cur_step["step_id"], user_input)
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 320, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
File "/config/custom_components/openmediavault/config_flow.py", line 75, in async_step_user
if not await self.hass.async_add_executor_job(api.connect):
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/openmediavault/omv_api.py", line 128, in connect
cookies = load_cookies(self._cookie_jar_file)
File "/config/custom_components/openmediavault/omv_api.py", line 23, in load_cookies
return pickle_load(f)
EOFError: Ran out of input

looks like cookie corruption.
delete file .omv_cookies.json in your /config directory

Thanks that helped but here is the new error
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 304, in async_setup
result = await component.async_setup_entry(hass, self)  # type: ignore
File "/config/custom_components/openmediavault/init.py", line 23, in async_setup_entry
await controller.async_hwinfo_update()
File "/config/custom_components/openmediavault/omv_controller.py", line 112, in async_hwinfo_update
await self.hass.async_add_executor_job(self.get_hwinfo)
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/openmediavault/omv_controller.py", line 171, in get_hwinfo
tmp = self.data["hwinfo"]["uptime"].split(" ")
AttributeError: 'float' object has no attribute 'split'

that could mean that wrong data is received from OMV.
are you using OMV5? OMV6 wont work.
Also check OMV permissions for user you are using.

Yeah OMV 6 so guess will have to wait for update huh. Thanks for the swift replies it worked great when I had OMV 5 installed.


Brick Road Indicator For Settings Page

Details

Fixed Issues

https://github.com/Expensify/Expensify/issues/218719
Tests

Dev testing:
Update the default props of policyMembers on InitialSettingsPage.js
    policyMembers: {
        'policyMemberList_<policyIDYouOwnHere>': {'email': {role:'bla', errors: []}, 'email2': {role:'bla', errors: [{1231312313: 'Unable to do X'}]}},
    },

Notice that when you open settings, the policy has a red dot (screenshot below)
Now remove those local changes and test the following:
The onyx data key we're using here does not exist yet so this should not have any change on the functionality of the app

[ ] Open up settings
[ ] Click on a workspace
[ ] Verify that there are no errors
[ ] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The Contributor+ will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

QA Steps

[ ] Verify that no errors appear in the JS console
[ ] Open up settings
[ ] Click on a workspace
[ ] Make sure you don't run into any errors

Screenshots

Web

Mobile Web

Desktop

(When I was testing the error state)

iOS

Android

Details

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @iwiznia in version: 1.1.87-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
failure ❌

🚀 Deployed to production by @yuwenmemon in version: 1.1.87-9 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


*: Protocol Buffer docs auto-generate script

It's experimental. One can run './scripts/genproto.sh -g'
Fix https://github.com/coreos/etcd/issues/5100.

Wrote a simple CLI https://github.com/coreos/prodoc.
Considered https://github.com/estan/protoc-gen-doc but this doesn't support any proto files with import "gogoproto/gogo.proto";.
If I have more time, I will replace the parser with actual golang parser https://github.com/golang/protobuf.

Can you show us the generate markdown output?

@xiang90 Here's the sample markdown output https://github.com/gyuho/etcd/blob/proto_gen/Documentation/api_etcdserverpb.md.

@xiang90 Fixed up. Changed the order and print Emtpy ... for 0 method. https://github.com/gyuho/etcd/blob/proto_gen/Documentation/api_etcdserverpb.md. PTAL.

@xiang90 @heyitsanthony Just updated it https://github.com/gyuho/etcd/blob/proto_gen/Documentation/api_protocol_buffer.md.

All docs in one markdown
only expose public message (not including InternalRaftMessage, etc)

PTAL.
Thanks.

hold on... lease.proto missing.

Ok now ready for review.
https://github.com/gyuho/etcd/blob/proto_gen/Documentation/api_protocol_buffer.md
PTAL.

LGTM. Defer to @heyitsanthony

lgtm

@xiang90 Changed file name to api_reference_v3.md. Merging in greens. Thanks.


Support topic resize

PubSub Lite has announced support for increasing topic partitions of an existing topic. Currently PSL Spark connector doesn't support it. This is being worked on and will be included in next release soon.

This has been fixed in https://github.com/googleapis/java-pubsublite-spark/pull/115, and will be included in next release.


Enable auto-cancelation in Travis

Not a code issue, but I feel here is the right place to say it.
There is a new feature in Travis that will help reducing time getting results from there.
Maybe we should enable it.
https://blog.travis-ci.com/2017-03-22-introducing-auto-cancellation

Already taken care of :-) Thanks @chimeno!


react-zpl-viewer sample

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
no

What's in this Pull Request?
New sample that allows users to choose a text file containing ZPL, the webpart then can show an image of the what the label would look like when printed.
Guidance

thx @zachroberts8668, nice neat sample on using external API - really clean implementation.


Beginner-Challenge-4

The Pull Request for the fourth Beginner Challenge.

how are you doing beginner/06
where do we have to upload gist file ?

Gist files can be created by going to gist.github.com You have to create a .md file as specified in beginner/06 in your-username-details/ branch and then push it


[Feature]: Add following pointer on home page

Is your feature request related to a problem? Please describe.
Use of following pointer with name will improve the UI.
Describe the solution you'd like
Use https://ui.aceternity.com/components/following-pointer

Alternatives
No response
Additional Context
No response
Are you interested in contributing?

[ ] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
gssoc, swoc, kwoc, dwoc, tublian

Hi, i would like to work on it, please assign it to me.

I would like to work on this issue please assign me this issue.

hi @pranshugupta54  I have completed the task please assign this to me

Kindly read Issue Guidelines and PR Guidelines

Hi Pranshu, I would like to work on section 1 of the homepage to implement this.


#87 Adding Faker to the Tests

Have completed the classes till \RestAssured.Net.Tests\ResponseBodyJsonVerificationTests.cs
Modified/ Added models to accommodate data variations
Added variable for the base url of mock server

This looks pretty good so far, @guneshpatil

@basdijkstra I this I am done adding to the PR. Faker has been added to most classes.
Please let me know if you feel any other classes can use Faker as well. thank you!


Is this code really can run?

I don’t know where can I get the dataset in your code
Follow the steps in this readme can’t get dataset, too.
I don’t know where can I get the dataset and how to deal with the dataset
I’m really interested in this paper

im sorry i found it‘s my fault～sorry！！！！！！


[stable/postgresql] how add extension ?

Describe the bug
I want to use postgresql as external database for github. In this case the postgresql db need pg_trgm extension.
I'm not sure it's a bug or not, I can not be in the right wayto use pg_trgm extension
to do that I add
    init_postgres.sh: |
          #!/bin/bash
          export PGPASSWORD=$(cat /opt/bitnami/postgresql/secrets/postgresql-password);
          /opt/bitnami/postgresql/bin/psql -U gitlab -c 'CREATE EXTENSION IF NOT EXISTS pg_trgm;'

Version of Helm and Kubernetes:
Kubernetes 1.14.1
Helm 2.14.0
Which chart:
stable/postgresql
version 5.3.11
What happened:
ERROR: permission denied to create extension "pg_trgm"
HINT : must be superuser to create this extension
What you expected to happen:
I expect that the extension is accepted

Hi, @obeyler.
That should work. However looking at this:
ERROR: permission denied to create extension "pg_trgm"
HINT : must be superuser to create this extension

It looks like you need to be superuser in order to create the extension. The superuser is the postgres user. Have you tried running that command as postgres?
Regards,
Alejandro

Hy @alemorcuq As you said I use now postgres user to fix this issue. I'm not completely satisfied as my user is postgres instead of gitlab but it's working.

Hi, @obeyler.
I'm not a PostgreSQL so I'm not really sure what else can be done here: if PostgreSQL wants you to be a superuser to create an extension then you have to be a superuser to create the extension. I've found this after a quick search, it may be useful for you: https://dba.stackexchange.com/questions/175319/postgresql-enabling-extensions-without-super-user
On the other hand, I think you could still be using the database as gitlab: you just need the postgres user to create the extension. Another option would be making the gitlab user a superuser, but I wouldn't recommend using the database as a superuser for security reasons.
Hope this helps.
Regards,
Alejandro


Add project metadata to the gemspec

Add project metadata to the gemspec file. This'll allow people to more easily access the source code, raise issues and read the changelog. These bug_tracker_uri, changelog_uri, documentation_uri, wiki_uri and source_code_uri links will appear on the rubygems page at https://rubygems.org/gems/zendesk_api after the next release.

@orien, could you please commit the change to the ruby doc and rebase on master? Thanks!

@nogates: How does that look?

@nogates: How does that look?

It looks great :) Spec live tests are passing. Merging!


Can not find eosio account for local dev

I run eos with docker, after I import the private key of eosio I can not see the eosio account:

cleos get accounts only works if history_plugin was enabled prior to the transactions that create the account.  If in a multinode network, this can be achieved by replaying and resynching with another node.  If a single node testnet, either create a new account or start over.

I changed the command inside Docker/nodeosd.sh:
from:
exec /opt/eosio/bin/nodeos  $CONFIG_DIR "$@"
to:
exec /opt/eosio/bin/nodeos --delete-all-blocks $CONFIG_DIR "$@"
Then added eosio::history_plugin into config.ini:
plugin = eosio::history_plugin
Then run docker-compose up -d again and it works.


Small testing fixes

end2end test deadlines may complete before checking IsCancelled
=> don't expect it to be false in these cases
add exponential backoff to port_posix
ensure run_tests rebuilds targets with a regex I commonly use

LGTM

need a copyright fix?

clang-format?

sigh done

:)


chore(release): merge dev into main branch

this PR is needed?

you may use RC.0 to test the dev branch, then merge dev to main

you may use RC.0 to test the dev branch, then merge dev to main

Sure.


First release created on GitHub

Tested by team and closed off. New ticket created to resolve conflict.


Results on postprocessed or raw output data?

Hi,
thanks four you excellent work.
The codebase does some postprocessing after validation to improve the results. Are your results based on the postprocessed results or without postprocessing?
Thank you!

Hi @Leonngm ,
Thank you for your interest in our work.
The results we report for UNETR++, as well as all other methods, are after post-processing.
Best regards,
Abdelrahman.

All the reported results are based on post-processing results to compare with other methods consistently.


Feature request: Allow changing the latency/samples_ahead in the settings menu

Partially addressed in 821def4. More to come.


Webm playback  issue

At the end of any video, it won't loop with the error "cant play this video". Even though I tap on ok, the message spams forever until my device lags, forcing me to close it. My device is a HTC One A9 on 6.0.1 (htc unlocked)

I know that this happens, though I'm currently not sure why.
Might be because I played with the code a bit, but can't see the Problem :/

Issue is confirmed resolved in latest version.


refactor: Refactor RunTaskModal children components

No jira.
This aims to split SystemsSelect component into smaller components (TaskName, TaskDescription), that would have their own separated state and business logic.
How to test
The modal should work as before the PR. The tests must pass.

Codecov Report
Attention: Patch coverage is 92.85714% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 81.87%.  Consider uploading reports for the commit c0868ae to get more accurate results

Files
Patch %
Lines

src/SmartComponents/RunTaskModal/TaskName.js
89.47%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
+ Coverage   81.82%   81.87%   +0.05%
==========================================
  Files          75       77       +2
  Lines        1623     1628       +5
  Branches      522      522
==========================================
+ Hits         1328     1333       +5
  Misses        258      258
  Partials       37       37

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


Show LO snackbar message for restored engagement

Jira issue:
https://glia.atlassian.net/browse/MOB-3444
What was solved?
Show LO snackbar message for restored engagement to align behaviour with Android platform.
Release notes:

[ ] Feature
[x] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Is the feature sufficiently tested? All tests fixed? Necessary unit, acceptance, snapshots added? Check that at least new public classes & methods are covered with unit tests
[ ] Did you add logging beneficial for troubleshooting of customer issues?
[ ] Did you add new logging? We would like the logging between platforms to be similar. Refer to Logging from iOS SDKs → Things to consider for newly added logs in Confluence for more information.

Screenshots:

Is it possible to add unit test?

Added unit test 44c5f524b39deaada24f39429277c719cfedadd9.

!squash


fix: generation of memoized class member calls

fix: generating memoized class member calls (on non-static members) should still use the class for python code generation
added testcase

Fixes the (last) issue in #954

:tada: This PR is included in version 0.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.10.0
GitHub release

Your semantic-release bot :package::rocket:


feat: use sub-command for plugins

BREAKING CHANGE: the CLI command and arguments was changed
See the discussion: https://github.com/Checkmarx/2ms/discussions/20#discussioncomment-5647413
I think we don't need to support running 2ms for multiple plugins at once. It is a rare case and it is confusing the command line arguments.
Instead, I'm proposing using SubCommand for each plugin.

From @bryantschuck: I agree, I have never been a fan of running multiple plugins per execution so that works for me


[TW#27204] heap assert failed when call heap_caps_print_heap_info(MALLOC_CAP_SPIRAM)

I am trying to build app: I2S + DMA with BLE. As for now its not complicated app yet, simple camera app in one task and simple BLE app in other task. If i run only camera task or only ble task it works fine, but when i am trying to run both i am getting assert multi_heap error. Usually i can find the reason, but this time i am getting this error when i am trying to print heap info from SPIRAM with this command: 		heap_caps_print_heap_info(MALLOC_CAP_SPIRAM); even before any task is running, literally its first command in app_main. This is log:
ets Jun  8 2016 00:22:57

rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0018,len:4
load:0x3fff001c,len:6212
load:0x40078000,len:10084
ho 0 tail 12 room 4
load:0x40080400,len:6612
entry 0x40080764
I (30) boot: ESP-IDF v3.2-dev-1429-g7efbe8178-dirty 2nd stage bootloader
I (30) boot: compile time 13:39:45
I (31) boot: Enabling RNG early entropy source...
I (37) boot: SPI Speed      : 40MHz
I (41) boot: SPI Mode       : DIO
I (45) boot: SPI Flash Size : 4MB
I (49) boot: Partition Table:
I (53) boot: ## Label            Usage          Type ST Offset   Length
I (60) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (67) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (75) boot:  2 factory          factory app      00 00 00010000 00330000
I (82) boot: End of partition table
I (86) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x533f0 (340976) map
I (215) esp_image: segment 1: paddr=0x00063418 vaddr=0x3ff80000 size=0x00000 (     0) load
I (215) esp_image: segment 2: paddr=0x00063420 vaddr=0x3ff80000 size=0x00000 (     0) load
I (221) esp_image: segment 3: paddr=0x00063428 vaddr=0x3ffbdb60 size=0x02d84 ( 11652) load
I (235) esp_image: segment 4: paddr=0x000661b4 vaddr=0x3ffc08e4 size=0x00000 (     0) load
I (239) esp_image: segment 5: paddr=0x000661bc vaddr=0x40080000 size=0x00400 (  1024) load
0x40080000: _WindowOverflow4 at D:/msys32/home/imper/esp/esp-idf/components/freertos/xtensa_vectors.S:1685

I (249) esp_image: segment 6: paddr=0x000665c4 vaddr=0x40080400 size=0x09a4c ( 39500) load
I (274) esp_image: segment 7: paddr=0x00070018 vaddr=0x400d0018 size=0xa2f58 (667480) map
0x400d0018: _flash_cache_start at ??:?

I (508) esp_image: segment 8: paddr=0x00112f78 vaddr=0x40089e4c size=0x0dea4 ( 56996) load
0x40089e4c: r_lld_evt_schedule at ??:?

I (532) esp_image: segment 9: paddr=0x00120e24 vaddr=0x400c0000 size=0x00000 (     0) load
I (532) esp_image: segment 10: paddr=0x00120e2c vaddr=0x50000000 size=0x00000 (     0) load
I (538) esp_image: segment 11: paddr=0x00120e34 vaddr=0x50000000 size=0x00000 (     0) load
I (561) boot: Loaded app from partition at offset 0x10000
I (561) boot: Disabling RNG early entropy source...
I (563) spiram: Found 64MBit SPI RAM device
I (566) spiram: SPI RAM mode: flash 40m sram 40m
I (572) spiram: PSRAM initialized, cache is in low/high (2-core) mode.
I (579) cpu_start: Pro cpu up.
I (582) cpu_start: Starting app cpu, entry point is 0x40081548
0x40081548: call_start_cpu1 at D:/msys32/home/imper/esp/esp-idf/components/esp32/cpu_start.c:245

I (0) cpu_start: App cpu up.
I (1464) spiram: SPI SRAM memory test OK

0x40080000: _WindowOverflow4 at D:/msys32/home/imper/esp/esp-idf/components/freertos/xtensa_vectors.S:1685

D (1534) memory_layout: Reserved memory range 0x40080000 - 0x40097cf0
0x40080000: _WindowOverflow4 at D:/msys32/home/imper/esp/esp-idf/components/freertos/xtensa_vectors.S:1685

I (1748) heap_init: Initializing. RAM available for dynamic allocation:
D (1755) heap_init: New heap initialised at 0x3ffaff10
I (1760) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
D (1766) heap_init: New heap initialised at 0x3ffb6388
I (1771) heap_init: At 3FFB6388 len 00001C78 (7 KiB): DRAM
D (1778) heap_init: New heap initialised at 0x3ffb9a20
I (1783) heap_init: At 3FFB9A20 len 00004108 (16 KiB): DRAM
I (1789) heap_init: At 3FFBDB5C len 00000004 (0 KiB): DRAM
D (1795) heap_init: New heap initialised at 0x3ffc3848
I (1800) heap_init: At 3FFC3848 len 0001C7B8 (113 KiB): DRAM
I (1807) heap_init: At 3FFE0440 len 00003BC0 (14 KiB): D/IRAM
I (1813) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
D (1820) heap_init: New heap initialised at 0x40097cf0
I (1825) heap_init: At 40097CF0 len 00008310 (32 KiB): IRAM
I (1831) cpu_start: Pro cpu start user code
I (1836) spiram: Adding pool of 4089K of external SPI memory to heap allocator
D (1875) clk: RTC_SLOW_CLK calibration value: 15264218
D (92) intr_alloc: Connected src 46 to int 2 (cpu 0)
D (93) intr_alloc: Connected src 57 to int 3 (cpu 0)
D (93) intr_alloc: Connected src 24 to int 9 (cpu 0)
I (97) cpu_start: Starting scheduler on PRO CPU.
D (0) intr_alloc: Connected src 25 to int 2 (cpu 1)
I (4) cpu_start: Starting scheduler on APP CPU.
D (121) heap_init: New heap initialised at 0x3ffe0440
D (127) heap_init: New heap initialised at 0x3ffe4350
I (132) spiram: Reserving pool of 32K of internal memory for DMA/internal allocations
D (140) spiram: Allocating block of size 32768 bytes
Heap summary for capabilities 0x00000400:
assertion "next > (intptr_t)block" failed: file "D:/msys32/home/imper/esp/esp-idf/components/heap/multi_heap
.c", line 124, function: get_next_block
abort() was called at PC 0x400d7867 on core 0
0x400d7867: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newl
ib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c:63 (discriminator 8)

Backtrace: 0x400950d0:0x3ffbb070 0x40095301:0x3ffbb090 0x400d7867:0x3ffbb0b0 0x40094e16:0x3ffbb0e0 0x400d703
a:0x3ffbb100 0x400d747f:0x3ffbb140 0x400d306a:0x3ffbb170 0x40092a79:0x3ffbb190
0x400950d0: invoke_abort at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:680

0x40095301: abort at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:680

0x400d7867: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newl
ib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c:63 (discriminator 8)

0x40094e16: get_next_block at D:/msys32/home/imper/esp/esp-idf/components/heap/multi_heap.c:377
 (inlined by) multi_heap_get_info_impl at D:/msys32/home/imper/esp/esp-idf/components/heap/multi_heap.c:724

0x400d703a: heap_caps_print_heap_info at D:/msys32/home/imper/esp/esp-idf/components/heap/heap_caps.c:389

0x400d747f: app_main at D:/esp32_camera/ble_project/ESP32_CAMERA_QR/main/app_main.cpp:95

0x400d306a: main_task at D:/msys32/home/imper/esp/esp-idf/components/esp32/cpu_start.c:493

0x40092a79: vPortTaskWrapper at D:/msys32/home/imper/esp/esp-idf/components/freertos/port.c:435

CPU halted.

Hi @chegewara ,
This looks like a bug, the assert indicates heap corruption somehow. Changing the malloc() threshold maybe works around it by causing something that was (incorrectly) allocated in SPIRAM to be allocated into internal RAM.
Can you please try making it crash again and then try calling heap_caps_dump(MALLOC_CAP_SPIRAM); instead of heap_caps_print_info(), and show us what it dumps before it crashes?
Angus

I have to change the story a little bit. All i described is correct except that issue "has been fixed" when i changed this parameter in menuconfig:
 [*] Allow .bss segment placed in external memory not Maximum malloc() size, in bytes, to always put in internal memory (i changed both but Maximum malloc size is not important).
Logs from heap_caps_dump:
Heap start 0x3f806bf4 end 0x3fbffff4
First free block 0x3f806bf9
Block 0x3f806bf4 data size 0x00000000 bytes next block 0x3f806bf8 FREE. Next free 0x3f806bf9
assertion "next > (intptr_t)block" failed: file "D:/msys32/home/imper/esp/esp-idf/components/heap/multi_heap.c", line 124, function: get_next_block
abort() was called at PC 0x400d77ff on core 0
0x400d77ff: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c:63 (discriminator 8)

Backtrace: 0x400951b8:0x3ffbb090 0x400953e9:0x3ffbb0b0 0x400d77ff:0x3ffbb0d0 0x40094de7:0x3ffbb100 0x400d7033:0x3ffbb120 0x400d7432:0x3ffbb140 0x400d304e:0x3ffbb170 0x40092a91:0x3ffbb190
0x400951b8: invoke_abort at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:680

0x400953e9: abort at D:/msys32/home/imper/esp/esp-idf/components/esp32/panic.c:680

0x400d77ff: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c:63 (discriminator 8)

0x40094de7: get_next_block at D:/msys32/home/imper/esp/esp-idf/components/heap/multi_heap.c:124
 (inlined by) multi_heap_dump at D:/msys32/home/imper/esp/esp-idf/components/heap/multi_heap.c:689

0x400d7033: heap_caps_dump at D:/msys32/home/imper/esp/esp-idf/components/heap/heap_caps.c:441

@chegewara So it crashes when "Allow .bss segment placed in external memory" is disabled, but enabling this option it doesn't crash. Is that correct?
Any chance that you are able to send us the .elf file for this app, please?

@projectgus Enabling this option causing crash, with disabled is all ok.

Hi @chegewara ,
Sorry I never followed up on this. If you still have the app and the ELF file, please share them. Thanks.
Angus

Sorry, i dont have any resources now. I will close it and reopen when i have any news.


UHF-3479: Installed read only role.

Waiting for updated Helfi TPR module and Helfi platform config module.

This was merged in a different PR.


In batch mode, it would help if we can specify the crop size in pixels for easier workflow for stacking

I typically use batch mode to pre-process my scans for stacking. However, I noticed the crop factors provided do not result in the same image dimensions. This becomes a pain point since I would like to stack these images to get better detail. One suggestion is to allow to define a pixel count for the length of the crop so that all cropped images can be processed easier by AS3 or AS4.

You can already do this, by using a script. For example, the following script will perform auto-cropping then stacking of images automatically and generate several images (eclipse, doppler, ...):
dopplerShift=3.1
cropFactor = 1.2
alignment=64 # This makes sure that autocrop aligns widths on a multiple of 64

[outputs]
base=autocrop2(rl_decon(avg(range(-1;1;.5)));cropFactor;alignment)
red=autocrop2(rl_decon(img(-dopplerShift));cropFactor;alignment)
blue=autocrop2(rl_decon(img(dopplerShift));cropFactor;alignment)
cont=autocrop2(continuum();cropFactor;alignment)

[[batch]]
stacked=ellipse_fit(stack(base))
stacked_red=ellipse_fit(stack(red))
stacked_blue=ellipse_fit(stack(blue))
stacked_cont=linear_stretch(pow(stack(cont);1.8))
cst=sharpen(asinh_stretch(linear_stretch(pow(stacked;1.8));200;2))
eclipse=asinh_stretch(disk_fill(stacked);200;8)
doppler_base=linear_stretch(saturate(rgb(stacked_red;avg(stacked_red,stacked_blue);stacked_blue);1.1))

[outputs]
continuums=draw_text(draw_obs_details(stacked_cont);2400; 64; "Continuum")
jsolex=draw_obs_details(auto_contrast(stacked;1.5))
compo=draw_obs_details(max(eclipse;cst))
doppler=draw_obs_details(doppler_base)
eclipse_doppler=draw_obs_details(linear_stretch(disk_fill(doppler_base)))

Please refer to the docs for a list of possible options, in particular take a look at the autocrop2 function.

and here's a video with an explanation: https://youtu.be/xxs2blZSJkY

Closing. Feel free to reopen if you think I misunderstood.


can't override instance methods with Inheritance

in my use case I have several Her model so I created the following super class
  class HerModel
    include Her::JsonApi::Model

    def created_at
      @created_at ||= Time.zone.parse(super)
    end

  end

and classes like this
class CollectorContent < HerModel
  use_api COLLECTOR_CONTENTS_WS_API
  collection_path "collector/:collector_id/collector_contents"
  type :collector_contents
end

In order to have the CollectorContent created_at attribute overridden I need to define it in every  child class:
class CollectorContent < HerModel
  use_api COLLECTOR_CONTENTS_WS_API
  collection_path "collector/:collector_id/collector_contents"
  type :collector_contents

  def created_at
    @created_at ||= Time.zone.parse(super)
  end

end

am I missing anything?

faced with the same situation, is there any information?

@masciugo @evgeniy-kunitsa is this peculiar to the created_at attribute, or any attribute you define in the base class?
Does it work if you define it in a mixin instead?

@edtjones, no, in my case I have base class:
module Api
 class Organization
   include Her::Model

   def suborganizations
     @suborganizations ||= self[:suborganizations].map { |org| Organization.new(org) }
   end
 end
end

And child class:
module External
  class Organization < Api::Organization
  end
end

I try to do smth like this:
Api::Organization.new(suborganizations: [{ uid: 123123 }]).suborganizations
=> [#<Api::Organization(organizations) uid=123123>]

External::Organization.new(suborganizations: [{ uid: 123123 }]).suborganizations
=> [{"uid"=>123123}]

Why in second case I got an array of hashes instead of array of organizations?
FYI: ruby 2.5.3, rails 5.2.2


Added account settings section where a user can swap the send/new lin…

…e behaviour. By experience I have noticed that most of the users tend to like the opposite behaviour.

Thanks @amaddio for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

Swapping Eric for PM review since he owns end user features.

Thanks @amyblais, in the future can we please make sure to add the "Editor Review" label to PRs to catch things such as the inconsistencies notes above (ie the menu icon and the formatting of the setting breadcrumbs)?

Hi @amaddio, the changes suggested above by esethna will also need to be edited on this pull request. Let me know if you have any questions.

@esethna amaddio has applied the changes you requested, can you help give this a second review?


moveit_cpp_tutorial-4] [INFO] [1675937187.657785499] [moveit_ros.current_state_monitor]: Didn't received robot state (joint angles) with recent timestamp within 1.000000 seconds. [moveit_cpp_tutorial-4] Check clock synchronization if your are running ROS across multiple machines! [moveit_cpp_tutorial-4] [ERROR] [1675937187.657812738] [move_group_interface]: Failed to fetch current robot state

I tried to add constraint motion in ros2 moveit cpp code, but  got the above error. some github solutions are suggested to remap the joint_state topic. But ut doesn't work for me. Anyone know how to solve this issue

@akhilkp777 If you install moveit from apt, you can try the following fix
Execute following command
sudo gedit /opt/ros/humble/lib/python3.10/site-packages/moveit_configs_utils/launches.py

Add  "use_sim_time": True, on line after
"monitor_dynamics": False,

So final code will be
    move_group_configuration = {
        "publish_robot_description_semantic": True,
        "allow_trajectory_execution": LaunchConfiguration("allow_trajectory_execution"),
        # Note: Wrapping the following values is necessary so that the parameter value can be the empty string
        "capabilities": ParameterValue(
            LaunchConfiguration("capabilities"), value_type=str
        ),
        "disable_capabilities": ParameterValue(
            LaunchConfiguration("disable_capabilities"), value_type=str
        ),
        # Publish the planning scene of the physical robot so that rviz plugin can know actual robot
        "publish_planning_scene": should_publish,
        "publish_geometry_updates": should_publish,
        "publish_state_updates": should_publish,
        "publish_transforms_updates": should_publish,
        "monitor_dynamics": False,
        "use_sim_time": True,
    }

Save it and try execute the code
Reference:
If you setup moveit from source code, i think this fix is already done, if not you can add this into same file.

@qboticslabs @akhilkp777  GitHub question, translated into English: @qboticslabs @akhilkp777 I have encountered the same issue, and I have set "use_sim_time": True in both the /opt/ros/humble/lib/python3.10/site-packages/moveit_configs_utils/launches.py file and the moveit.launch.py file. In Rviz, MoveIt can plan and execute, but when using a C++ program, it reports the following error. Could you please provide assistance? Thank you.
ros2 launch moveit2_tutorials move_group_interface_tutorial.launch.py
[INFO] [launch]: All log files can be found below /home/robot/.ros/log/2023-04-06-15-38-16-583006-c-80920
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [move_group_interface_tutorial-1]: process started with pid [80921]
[move_group_interface_tutorial-1] [INFO] [1680766696.896307773] [moveit_rdf_loader.rdf_loader]: Loaded robot model in 0.0278734 seconds
[move_group_interface_tutorial-1] [INFO] [1680766696.896364812] [moveit_robot_model.robot_model]: Loading robot model 'panda'...
[move_group_interface_tutorial-1] [INFO] [1680766696.930198795] [move_group_interface]: Ready to take commands for planning group panda_arm.
[move_group_interface_tutorial-1] [INFO] [1680766696.942789451] [moveit_ros.current_state_monitor]: Listening to joint states on topic 'joint_states'
[move_group_interface_tutorial-1] [WARN] [1680766696.944043430] [moveit_ros.current_state_monitor]: Unable to update multi-DOF joint 'virtual_joint':Failure to lookup transform between 'world'and 'panda_link0' with TF exception: "panda_link0" passed to lookupTransform argument source_frame does not exist.
[move_group_interface_tutorial-1] [INFO] [1680766697.942866961] [moveit_ros.current_state_monitor]: Didn't received robot state (joint angles) with recent timestamp within 1.000000 seconds.
[move_group_interface_tutorial-1] Check clock synchronization if your are running ROS across multiple machines!
[move_group_interface_tutorial-1] [ERROR] [1680766697.942905819] [move_group_interface]: Failed to fetch current robot state
[move_group_interface_tutorial-1] Stack trace (most recent call last):
[move_group_interface_tutorial-1] #4    Object "", at 0xffffffffffffffff, in
[move_group_interface_tutorial-1] #3    Object "/home/robot/ws_moveit2/install/moveit2_tutorials/lib/moveit2_tutorials/move_group_interface_tutorial", at 0x56052e7debe4, in _start
[move_group_interface_tutorial-1] #2    Source "../csu/libc-start.c", line 392, in __libc_start_main_impl [0x7faeff1c1e3f]
[move_group_interface_tutorial-1] #1    Source "../sysdeps/nptl/libc_start_call_main.h", line 58, in __libc_start_call_main [0x7faeff1c1d8f]
[move_group_interface_tutorial-1] #0    Object "/home/robot/ws_moveit2/install/moveit2_tutorials/lib/moveit2_tutorials/move_group_interface_tutorial", at 0x56052e7dbb67, in main
[move_group_interface_tutorial-1] Segmentation fault (Address not mapped to object [(nil)])
[ERROR] [move_group_interface_tutorial-1]: process has died [pid 80921, exit code -11, cmd '/home/robot/ws_moveit2/install/moveit2_tutorials/lib/moveit2_tutorials/move_group_interface_tutorial --ros-args -r _node:=move_group_interface_tutorial --params-file /tmp/launch_params_gklulxa2 --params-file /tmp/launch_params_nqt__koh --params-file /tmp/launch_params_hr4b0gq'].

const moveit::core::JointModelGroup* joint_model_group =
move_group.getCurrentState()->getJointModelGroup(PLANNING_GROUP);
log:
[move_group_interface_tutorial-1] [INFO] [1680775155.255027553] [moveit_ros.current_state_monitor]: Didn't receive robot state (joint angles) with recent timestamp within 1.000000 seconds. Requested time<PHONE_NUMBER>.254896, but latest received state has time 454.390000.

some additional information added as follows:
/gazebo:
enable_performance_metrics
publish_rate
qos_overrides./clock.subscription.depth
qos_overrides./clock.subscription.durability
qos_overrides./clock.subscription.history
qos_overrides./clock.subscription.reliability
qos_overrides./parameter_events.publisher.depth
qos_overrides./parameter_events.publisher.durability
qos_overrides./parameter_events.publisher.history
qos_overrides./parameter_events.publisher.reliability
use_sim_time
ros2 param get /gazebo use_sim_time
Boolean value is: True

I have already resolved this problem, thank you everyone. by adding {"use_sim_time": True}
from launch import LaunchDescription
from launch_ros.actions import Node
from moveit_configs_utils import MoveItConfigsBuilder
def generate_launch_description():
moveit_config = MoveItConfigsBuilder("moveit_resources_panda").to_moveit_configs()
# MoveGroupInterface demo executable
move_group_demo = Node(
    name="move_group_interface_tutorial",
    package="moveit2_tutorials",
    executable="move_group_interface_tutorial",
    output="screen",
    parameters=[
        moveit_config.robot_description,
        moveit_config.robot_description_semantic,
        moveit_config.robot_description_kinematics,
        {"use_sim_time": True},
    ],
)

return LaunchDescription([move_group_demo])

@akhilkp777 does use_sim_time also fix this issue for you?

I've encountered the same issue without simulation in Gazebo involved. Setting use_sim_time:=True for all the nodes fixes the issue.


Add disable_search attribute

This will avoid querying chef servers on infrastructure where it is not
relevant (for those using allowed_hosts directly for instance)
Change-Id: Idf42aeaa85b2e7cb2894b118aedc49ee36813ec5

@tas50 would you have feedback on that PR?

We should be removing the Chef solo specific behaviour here. Instead of augmenting it. Seeing as search actually works now.

Hey @kamaradclimber do you still want this PR or are we good to close it?

Honestly i've used a fork for a long time now and now going away from nrpe
so feel free to close the pr
Le sam. 4 août 2018 à 20:55, Dan Webb<EMAIL_ADDRESS>a écrit :

Hey @kamaradclimber https://github.com/kamaradclimber do you still want
this PR or are we good to close it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sous-chefs/nrpe/pull/77#issuecomment-410470319, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAeu8eluhzIIP5KVEmR6aVO0516p1pfXks5uNe4CgaJpZM4TBdBi
.

OK thanks!


Map not displaying in mobile view

Overview
In the EM test site and on the setting pages of users who have an EM set up, there is no map displayign on the map tab. See screen recording:
https://images.zenhubusercontent.com/159699912/db1f7b98-b8e7-457f-9f11-d7e0b7504ed8/screen_recording_2022_10_20_at_9_56_43_am.mov
Expected Behavior
Map should display similar to the map that displays when you enter "&embed=1" at the end of URL
Map that displays shows the settings that the contributor has configured in their settings
Actual Behavior
What happens instead of the expected behavior?
Steps to Reproduce
Go to: https://open-apparel-registry.github.io/embeded-map-test-fixed-size.html?host=prd.fb84e0f7529f2737.openapparel.org&contributors=540
See that map tab is blank.

I changed the title of this issue to "Map not displaying in mobile view" because it is not just a problem with embedded maps. I was able to reproduce it in the regular mobile view.

I can't replicate this on production, so it's likely one of the few PRs we've merged over the last week. #2250 seems a likely culprit.

I can replicate this in production
https://user-images.githubusercontent.com/17363/197285591-b78d978a-bf36-4bdc-9044-0e0c1fc4653b.mp4

More specifically is looks like the issue is the "middle" width between 600px and 1280px
https://user-images.githubusercontent.com/17363/197286378-e5a4d619-fc53-4572-a910-0212b8dd87a6.mp4

I started from ogr/master and worked backwards until the map was visible at middle widths. It appears that the introduction of the new footer in #2189 is when the map stopped appearing at middle widths.


Parallel lost after update

Just updated XGBoost from 06.xx to 08.62.1
Lost parallel working. Before worked fine, because of an issue with prediction did the update.
My computer
R version 3.4.3 (2017-11-30) -- "Kite-Eating Tree"
Copyright (C) 2017 The R Foundation for Statistical Computing
Platform: x86_64-apple-darwin15.6.0 (64-bit)

sessionInfo()
R version 3.4.3 (2017-11-30)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS  10.14.2

Matrix products: default
BLAS: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.4/Resources/lib/libRlapack.dylib
locale:
[1] de_DE.UTF-8/de_DE.UTF-8/de_DE.UTF-8/C/de_DE.UTF-8/de_DE.UTF-8
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] compiler_3.4.3 tools_3.4.3    yaml_2.1.16

“Lost parallel working”
Can you be more specific?

Normally I work on my MacBook. Because I am not able to install openMP, I today used my iMac. When I build a model with "old" version I had all 4 cores running parallel, but when I finished calculation prediction was not possible for gblinear. Thus I updated XGBoost from within Rstudio. Next run of modeling is still running :-(. It is now late in the evening, have to look after it tomorrow evening.
BTW: any tips how to install OpenMP on a new MacBook ? Tried it today several times, took about 3 hours without success. My Macbook has a 6 core processor (2018), my iMac only 4 cores (2013).
Peter

@phahn57 The issue is that the default compiler on Macbook doesn't support OpenMP. To enable OpenMP, you should use Homebrew GCC compiler instead.
Follow these steps on your Macbook (tested on my Macbook Pro):

Install Homebrew from https://brew.sh/.
In the command line, run brew update.
Install Homebrew GCC by running brew install gcc@8.
Install CMake by running brew install cmake
Now download code for XGBoost 0.82: git clone --recursive https://github.com/dmlc/xgboost -b release_0.82.
Compile XGBoost with the following commands:

cd xgboost
mkdir build
cd build
CC=gcc-8 CXX=g++-8 cmake .. -DR_LIB=ON
make -j$(nproc) install

Now you should have access to XGBoost from the Rstudio. This time, you should be able to use all cores on your machine.

Thanks. Amazing fast


Switch from testnet to mainnet

Anyone point me to a document that explains clearly. How to switch from testnet to mainnet and how to check if my node is running on testnet and mainnet

run cleos get info
check chain_id was aca376f206b8fc25a6ed44dbdc66547c36c6c33e3a119ffbeaef943642f0e906

i get his response chain_id is different

{
"server_version": "40a20769",
"chain_id": "cf057bbfb72640471fd910bcb67639c22df9f92470936cddc1ade0e2f2e7dc4f",
"head_block_num": 2538894,
"last_irreversible_block_num": 2538893,
"last_irreversible_block_id": "0026bd8d6f1f8e8f7ea47f11502d488d2a0a86a04c8bd89d0503bc11fc77e33b",
"head_block_id": "0026bd8e4fb6af2f7e914de0464ee2e06733ffb50487337d46e5deb7985deea0",
"head_block_time": "2018-08-12T13:15:26.500",
"head_block_producer": "eosio",
"virtual_block_cpu_limit": 200000000,
"virtual_block_net_limit":<PHONE_NUMBER>,
"block_cpu_limit": 199900,
"block_net_limit": 1048576
}

This is what I got from mainnet using  ./cleos -u https://eu1.eosdac.io:443 get info
{
"server_version": "2d3bb003",
"chain_id": "aca376f206b8fc25a6ed44dbdc66547c36c6c33e3a119ffbeaef943642f0e906",
"head_block_num": 10841760,
"last_irreversible_block_num": 10841428,
"last_irreversible_block_id": "00a56d54dc95ac21360b7114cc6f515e2122f59123c18987184d8cf95a2e8062",
"head_block_id": "00a56ea014e38ab89fa5bf3793e86021cfe5a8539eeb7141b285e0683dff162c",
"head_block_time": "2018-08-13T01:51:58.500",
"head_block_producer": "eosswedenorg",
"virtual_block_cpu_limit": 200000000,
"virtual_block_net_limit":<PHONE_NUMBER>,
"block_cpu_limit": 199900,
"block_net_limit": 1048576
}

@taokayan with your command it means, We are connected to main net via this URL https://eu1.eosdac.io:443 and this is not our own node. Its third party node?

Will raise this as required update to documentation.


"editor.tabsize" does not support per-language configuration.

"editor.tabsize" does not support per-language configuration.
Originally posted by @BenjaVR in https://github.com/microsoft/vscode/issues/23933#issuecomment-291849166

@zicjin originally posted by me?

@zicjin What are the steps to reproduce?

Obviously this feature is not currently supported

@zicjin you have to use editor.tabSize instead of editor.tabsize.


Error: undeclared identifier: 'shallowCopy'`` on all examples

All the examples I try give the following error:
neo/dense.nim(90, 14) Error: undeclared identifier: 'shallowCopy'
I am using Nim Compiler Version 1.9.1 .

As someone who has the same problem, an easy solucion is write --gc:refc as option, something like nim c --gc:refc -r main.nim

How feasible would it be for a Nim newcomer to implement the shallow pragma workaround? I'm hoping to use Nim for a WebAssembly project, which doesn't seem compatible with --gc:refc.

It would amount to marking data as shallow in places like here and here, and similarly in cudadense etc, then removing shallowCopy where it appears, running tests and checking what breaks. When something breaks, it will be probably because there was an implicit copy of data somewhere; in thit case, make the copy explicit.
Test coverage is good enough that I think if tests pass, everything is fine.


[cleanup][dependency] Remove use of JUnit

Signed-off-by: Zixuan Liu<EMAIL_ADDRESS>Motivation
Using TestNG instead of Junit.
Documentation

[x] doc-not-needed

When this PR merges into the master branch, then we should quickly merge https://github.com/apache/pulsar/pull/15616 to avoid the using JUnit.

/pulsarbot rerun-failure-checks


[Do not merge] Transfer api ga/nitegupt

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

Hi @nitegupt, as this PR is pending for a long time, can we close it if it is not need any more


Page number font size is incorrect

#set page(paper: "a4", numbering: "1")

#align(center, text(100pt)[
  Big size
])

// something

#pagebreak

Normal size

This produces the following two pages:

The expected result is that the page number's style is the same for both pages (normal size).
N.b. if the // something line is uncommented, then the word something appears in normal size and the page number also returns to the normal size.

confirmed:
#set page(numbering: "1")
#text(99pt)[big size]
#pagebreak()
normal size

looks like an easy fix, I'd like to knock this one off tomorrow, if nobody minds?

I have a workaround that seems to work.  If I add a zero width space, it seems to correct the header and footer issue.
//-----------------------------------------------------------------------------
// Formal Title Page
//-----------------------------------------------------------------------------
#sym.zws // Hack to workaround known issue: https://github.com/typst/typst/issues/241
#set text(size: 26pt)
#align(center)[*#doc_full_title \ Revision: #doc_version*]
#set text(size: 11pt)
#pagebreak()

If the workaround line is removed, the header and footer pick up the format of the text there, font size, strong emphasis, and so forth.

I've also found a workaround that helped in my case. I've placed an invisible rectangle below the heading, which corrected the formatting in the footer
= Heading
#rect(stroke: 0em)
#pagebreak()
= Heading 2
...