TypeError: Cannot read property '3' of null

TypeError: Cannot read property '3' of null
    at Client.unsubscribe (/node_modules/nats/lib/nats.js:740:22)
    at [object Object].action (/src/lib/bus.js:53:24)

Probably user error but would be nice with better error messages.

Happens when trying to unsubscribe a closed instance. Should probably swallow error and warn.

Agree this should be cleaner, will get it into next release.


[Bug] SqlBlockRule NPE Exception

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
0.15

What's Wrong?
We need judge sql whether exist.
What You Expected?
metadata and matchSql can run success.
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

In this pr, I will refactor some null value handling, checkstyle, and ut.


I can't seem to find this file: Pages/Index.cshtml on Windows PC

Problem encountered on https://dotnet.microsoft.com/learn/web/aspnet-hello-world-tutorial/modify
Operating System: windows
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Does it happen for every new project? On multiple machines/VMs?

Did you start the tutorial at the first step? https://dotnet.microsoft.com/learn/web/aspnet-hello-world-tutorial/create
Running dotnet new ... should create all those files.

I am new to this and just going through the tutorial : https://dotnet.microsoft.com/learn/web/aspnet-hello-world-tutorial/modify
Everything has been successful up to the 'edit your code' part.
I tried to find the file on my computer by following the path made in the command prompt however I cannot find it. I then tried to open Notepad++ and save it as a file but I can't even save it as a .cshtml

sorry I didn't mean to close this..

Hi @TheresaPinter , do you know in which folder you ran the dotnet new command? If you navigate to that folder in Explorer, do you see the various files it created? (Various CSHTML files, JSON files, CS files, etc.)

Sorry, am I understanding you correctly: I am looking for this file in a web browser?

I only receive a google search.
Here is an image of what I see on my computer:

I only receive a google search.
Here is an image of what I see on my computer:

@TheresaPinter sorry I meant a file on your computer.
When you ran the dotnet new ..... command at a Command Prompt, it created several folders and files for the project on your hard drive.
It might be easiest to start from scratch:

Open a command prompt window and navigate to some folder (e.g. c:\projects)
Run the dotnet new ..... command per the tutorial
Run start . (note the . at the end of the command) and that will open the folder so you can see the files it created

Hi @TheresaPinter , were you able to get this working?

I think I am going to circle back around to this in about a week or so. I
am just reviewing C# language. Thanks for checking in.
On Mon, Apr 8, 2019 at 6:31 PM Eilon Lipton<EMAIL_ADDRESS>wrote:

Hi @TheresaPinter https://github.com/TheresaPinter , were you able to
get this working?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/2531#issuecomment-481032160, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Akbeo3E5d-KG2TaYS_67_7QVqHdxQIGuks5ve8NBgaJpZM4cV_ci
.

--

@TheresaPinter got it. I'll close this issue for now but if you hit a new issue while doing the tutorial, please file an issue at https://github.com/aspnet/AspNetCore.Docs (if you suspect an issue with the tutorial itself), or https://github.com/aspnet/AspNetCore (if you suspect it's an issue with ASP.NET Core). Thanks!

@TheresaPinter sorry I meant a file on your computer.
When you ran the dotnet new ..... command at a Command Prompt, it created several folders and files for the project on your hard drive.
It might be easiest to start from scratch:

Open a command prompt window and navigate to some folder (e.g. c:\projects)
Run the dotnet new ..... command per the tutorial
Run start . (note the . at the end of the command) and that will open the folder so you can see the files it created

@Eilon Thank you soo much. This worked for me.

Hi folks, I was facing this issue today too. Was looking to see where "Pages/Index.cshtml" was stored after following the tutorial., where the instructions are:
"Open Pages/Index.cshtml in any text editor and replace all of the code with the following, then save the file."
I found the folder on my computer, after changing the directory (cd commands) in my VS Code Terminal.
On a Windows computer, you will also find this by going to: "C:\Users<yourpcusername>\myWebApp\Pages".
Attached are a couple of screenshots to help in case anyone else faces this issue.
I hope that helps!

Hi Guys I can't seem to understand,  the section of the tutorial where it says, " Edit your code
Open Pages/Index.cshtml in any text editor and replace all of the code with the following, then save the file.", where do I even find that file,

@leLeo24 we're going to try to make improvements to that tutorial to make that part easier to understand. But inside the myWebApp folder where the project was created, you should see a Pages subfolder where you should find the Index.cshtml page.
Also, be aware that Windows might hide the file extension. So, if that's the case, the cshtml file is the first one in the screenshot below.

@mairaw Okay thanks, I tried looking for the file and this is what I found, and I clicked on the first file, and it shows some codes, which I don't think is the one, I have attached a screenshot below, the 1st one is the files I found, and the 2nd one is the file when I open it.
And where do I find where the project was created because I just searched for it, in local disk?

@leLeo24 the project is created in whatever folder you ran the command in. Can you show how/where you ran the dotnet new command?

**vulongtran ** commented on 19 Jul 2020

Thank you for your help dude !!!

@mairaw Thank you, your comment was very helpful! I ran into the same issue on mac, I was stuck for a while. It took me 1 hour to finish a 5 mins tutorial.

@TheresaPinter sorry I meant a file on your computer.
When you ran the dotnet new ..... command at a Command Prompt, it created several folders and files for the project on your hard drive.
It might be easiest to start from scratch:

Open a command prompt window and navigate to some folder (e.g. c:\projects)
Run the dotnet new ..... command per the tutorial
Run start . (note the . at the end of the command) and that will open the folder so you can see the files it created

Starting my journey as an asp.net dev. Glad to see I wasn't the only one facing this issue.
found the file through start. Thanks!


Report for (search) query hits

Is your proposal related to a problem?
In order to gain insight into the popular search teams as a content editor and/or marketeer I want to view a report.
Currently the only way is to view this by registering a (django) ModelAdmin config or use the popup via search promotions.
Describe the solution you'd like
An option to add a report with search teams with relevant filters via the settings, just as any other wagtail report.
As reports are not often familiar for content editors it would be nice to switch this feature on/off via django settings.
Working on this
We have capacity to create this report as a pull request. Planned already to implement this within an active project.

Sounds great @robmoorman! I can see why it’d be useful to view this information in a filterable / export-able report.

An option to add a report with search teams with relevant filters via the settings, just as any other wagtail report.
As reports are not often familiar for content editors it would be nice to switch this feature on/off via django settings.

From my perspective I’d expect the presence of the report by default for sites using this app to not be much of an issue, as reports have their own menu item to be hidden inside of. It’d be nice to align with other apps / reports on how exactly this is configurable

Ok great! We will see if we can put a PR in on short notice. I expect it's an easy feature to implement and review as it makes pretty sense what to do here and put in the report.
Looking at other "feature flags" the name of the setting can be something like WAGTAIL_SEARCHPROMOTION_REPORT_ENABLED (default set to True). It's sort of a convention to put the xxx_ENABLED to turn a feature on/off.

Completed in #12341.


Bugfix in _fix_gnuplot_svg_size and extra tp arguments

The width/height were not correctly read in the _fix_gnuplot_svg_size
function - they were strings causing svg.setAttribute calls to crash
when '%dpx' was used. Also, if no args.size are provided, the picture
size needs to be passed as None to this function, or the default size
variables are used.
An additional parameter -tp has been added (also to the --help) to
allow arbitrary parameters to be passed to the tikzpicture call.

Thanks a lot for your PR !

I made a test of this new feature in https://github.com/meduz/ipython_magics/commit/e81455dbe3d8a426182d97de2086b464d3decb41

Great, thanks - I should have added something to the test file. I use vector graphics for conversion from notebook to pdf so svg output is a useful feature. Thanks for publishing tikzmagic, it's very useful! I'll push up any further mods I make.

cool, looking forward to it. I myself use the extension a lot for post-processing figures and having SVG output is very useful.
kudos to http://www2.ipp.mpg.de/~mkraus/python/tikzmagic.py for the idea!


Squirrel SQL array values don't work

returned as nulll
actor.name                                r.roles  movie.title
---------------------------------------------------------------------------------------------
Tom Hanks                                 <null>   That Thing You Do
Tom Hanks                                 <null>   Joe Versus the Volcano
Tom Hanks                                 <null>   Sleepless in Seattle
Tom Hanks                                 <null>   You've Got Mail
Tom Hanks                                 <null>   Charlie Wilson's War
Tom Hanks                                 <null>   The Polar Express

r.roles[0] etc. works

This bug seems related to the SquirrelSQL way to manage java.sql.Array (SQL type = 2003), In the previous version of the driver it used to work because the array was returned as a String so everything worked fine.
I tried a specific approach for SquirrelSQL but I can't find a System.property which is related to it.
Do you have ideas? Personally I prefer to return a java.sql.Array instead of a String (as a default behaviour).


Post release automated changes for containerservice releases

Post release automated changes for azure-arm-containerservice

API change check
API changes are not detected in this pull request.

@xirzec why is the bot not doing auto merge now :(

@xirzec why is the bot not doing auto merge now :(

I think the bot has been decommissioned :(


Feature to hide the app from the home launcher

just mentioning something you might be interested in:
https://github.com/rkkr/simple-keyboard/issues/213#issuecomment-695903400


Encrypt/Openssl Unset Difference

Zend Framework 2.4.6-2.5.3, Zend\Filter\Encrypt\Openssl.php
I believe the unset should read unset($options['compression']); , lines 90-93.
if (array_key_exists('compression', $options)) {
    $this->setCompression($options['compression']);
    unset($options['compress']);
}

The code above I believe results in 'Undefined index: compression' later in code.  Current workaround is to use ...->setCompression($string) .

@m1st0 please open the issue to zendframework/zend-filter, thanks!

Sounds good. Thanks.
On Sat, Jul 9, 2016, 6:25 AM Enrico Zimuel<EMAIL_ADDRESS>wrote:

@m1st0 https://github.com/m1st0 please open the issue to
zendframework/zend-filter https://github.com/zendframework/zend-filter,
thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-crypt/issues/31#issuecomment-231534312,
or mute the thread
https://github.com/notifications/unsubscribe/AGFAeCQV44Sm__5V-EE0GxA1l_KhYkDqks5qT6FJgaJpZM4JIf3m
.

Moved to zendframework/zend-filter.


Issue No. #8, Basic Portfolio Website files added.

Hello @Ayushparikh-code sir, The Issue Number #8 which was assigned to me has been solved. The files for Basic Portfolio Website has been added. You can Merge this to the main branch of the repository.
Thank You.

@Ayushparikh-code Thank you for the valuable suggestions will do it as soon as possible


Update to senzingdata-v3

See https://github.com/Senzing/kubernetes-demo/issues/107
Examples:
resolver/CHANGELOG.md:- Updated to senzingdata-v2
resolver/README.md:        senzingdata-v2-${SENZING_VERSION_SENZINGDATA_BUILD}
resolver/docs/alternate-installations.md:    export SENZING_DEB_SENZING_DATA=~/Downloads/senzingdata-v2_2.0.0-2_amd64.deb

See https://github.com/Senzing/docker-compose-demo/issues/253

Pull request:

https://github.com/Senzing/resolver/pull/80


Add usage and telemetry funcs

Also filter out call home 413 error from being logged.

@chb0github ☝🏼

I have read the CLA Document and I hereby sign the CLA


Too many applications in a category freezing gnomepie

I found gnomepie after reading an article by noobslab.   Really nice but when I open the games category (or any other category with a lot of applicatioins) gnomepie freezes, high cpu usage and graphics lockup.  I am on Arch Linux with gnome 3.14.2

Thank you for the report.
Gnome-Pie is not designed to display many items. If there are, let's says, more than 15 items per pie it's hardly usable anymore. In such a case you might consider designing an alternative menu structure with less items per category. (for example create sub-categories for each type of game you have)
Nevertheless Gnome-Pie should not crash, so that is obviously a bug. How many apps do you have in this category?

70+ I think.  Maybe a feature request but how about multiple rings?

This has been fixed in 7912369d1c684c369d770f198cbedb18216224c4. There is a setting for maximum slices per pie in the settings menu. If there are more slices, you can scroll through them with your mouse wheel!


Change this dict to an OrderedDict

Summary comments are unordered. I think this change was left out in
https://github.com/markstory/lint-review/commit/377512d8444ae8e73366d186bef370444b027772
When tracking down this problem and not being able to find it, @zoidbergwill managed to guess what it was:

Fixes https://github.com/markstory/lint-review/issues/88

Thanks!

I tagged 0.7.2 with this change in it.


doesn't download covers.

why break something that worked before?

Fixed in the latest release, not sure how this got broken. Sorry.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1934

Questions ?
Please refer to the Backport tool documentation


Issue with tf_record.py

I got this error :
File "generate_tfrecord.py", line 92
writer.write(tf_example.SerializeToString())
^
IndentationError: unexpected indent
###The command i used :

python generate_tfrecord.py --csv_input=images/train_labels.csv --image_dir=images/train --output_path=train.record
I don´t understand why , i havent edited the code , and the indentation is not bad , any idea??

try to delete every linebreak, space or tab - basically type these two lines by hand.
I had this several times with python.


feat(transformer): add "_jsxFileName" variable in jsx source plugin

#3000  👈
main

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

Join @Boshen and the rest of your teammates on  Graphite

Merge activity

Apr 16, 5:40 AM EDT: @Boshen merged this pull request with Graphite.


Add context menu for removing taskmap edges

Right or left clicking edges on task map now opens context menu where you can delete the relation.

Added all the suggestions. Thanks for the help on the styles, that's a really good solution 🙏


A little confusion about the linear layer in the decoder

Hi, thank you for sharing such an excellent work. But there is one detail that makes me curious and confused. When the first layer feature of the encoder passes through the linear layer of the decoder, why the number of output channels is adjusted to the special 48 and not other sizes?

Mainly for efficiency. DeepLabV3+ also uses similar setting.


async response proposal

This is a standard async response proposal that servers can implement.

The status codes in the request's response are independent of the configuration state.
They exist to give the user an indication of the state of their request.
For example the server can be applying a configuration but a subsequent transmit request should fail with a 409 error indicating the server is busy with a previous request.
To reiterate everything in the API/schemas is optional - an implementation can choose what to support, so in the case of POST /config response codes athena can support 204, 400, 409 and indicate in /results/capabilities that 202, 300 is not supported (ie no progress polling, use /results to determine when to issue the next request).

By modeling with openapi it implies using rest(http) but it shouldn't constrain an implementation - ie athena is implementing the model using go on the server and python on the client.
The use of error codes is used to differentiate between responses to a request but it isn't the only thing  that differentiates between the responses.
For example - set_config returns two successful responses - in python I would call set_config and receive a success response object (modeled as a 204 no content response) or an async response object (modeled as a 202 content response).
It also returns an error response object (400) or a conflict response object (409 - ie you tried to set config while an existing set_config is being done).
The code generator if its any good should generate a docstring for the set_config method (the information should come from the response schema objects) or if its not then the implementation will have a samples library or their own documentation.
def set_config():
    """The set config operation

    Returns:
    - obj(SuccessNoContent)
    - obj(SuccessAccepted): poll using progress_url, cancel using cancellation_url
    Raises:
    - obj(BadRequestError)
    - obj(RequestConflictError)
    """"

I still haven't finished the /results/state responses - those will come later as required.
I will be posting sample code later today.

sample code using the proposed model

# configuration definition
config = [
    ports = [
        {
            'name': 'p1'
        }
    ]
    flows = [
        {
            'name': 'f1',
        }
    ]
]

# set the traffic generator configuration state
config_state = {
    'choice': 'config_state',
    'config_state': {
        'state': 'update',
        'config': config
    }
}
# any 409, 590 responses will be raised as exceptions
api.set_state(config_state)

# set the traffic generator flow state
flow_transmit_state = {
    'choice': 'flow_transmit_state',
    'flow_transmit_state': {
        'state': 'start',
        'flow_names': [config['flows'][0]['name']]
    }
}
# any 409, 590 responses will be raised as exceptions
api.set_state(flow_transmit_state)

@winstonliu1111 @ashutshkumr @ankur-sheth  just waiting for your reviews before I merge.


Demonstrate IMDS timeout workaround in docs

A couple of users have asked us to export the internal API DefaultAzureCredential uses to impose a timeout on managed identity authentication, to avoid a long timeout when IMDS is unavailable (#19699, #19721). We don't want to do that because it means adding a redundant, esoteric cancellation mechanism that conflicts with cancellation via context.Context. This example shows how to reimplement this DefaultAzureCredential behavior with Context and a small wrapper.
I'm also tempted to replace the internal API with this wrapper. That would steer future readers of the source in the right direction, and I believe it would simplify some of the implementation as well. Thoughts?

Well, the wrapper got a little more complicated when I put it to internal use. One behavior we have internally is that if managed identity authentication succeeds, DefaultAzureCredential stops applying this special timeout, to prevent future failures due to transient latency. I updated the example to behave the same; it's a bit ugly because GetToken() has a value receiver 🙄


Feature requests: DCD & DSR events, DTR toggle

It'd be useful (to me) to be able to attach callbacks to events that get fired off when the state of the DCD and DSR pins change (with the argument supplied to the callback being the current state of the pin in question.)  Being able to raise and lower DTR would also be helpful.

This has been closed by DCD & DSR have not been addressed. Is there any way to read these values with this library?

Thank you. For my purposes, I have a communication bus and need a way for any node on it to send a simple HIGH/LOW signal back to the host outside of the communication channel. The easiest way would be to attach that to the DCD or DSC line on an FTDI chip (FT232RL).

A possible temporary work around: I haven't tried this yet, but it might be possible with a combination of node-serialport and ioctl. I assume it would look something like this:
const SerialPort = require("serialport");
const ioctl = require('ioctl');

// From ioctl.h
const TIOCMGET	= 0x741d;
const TIOCM_CAR	= 0x100;

// Connect
var port = new SerialPort("/dev/tty-usbserial1");

// Get DCD value
var buff = new Buffer(1);
ioctl(port.fd, TIOCMGET, buff);
var dcd = buff[0] & TIOCM_CAR;

I have some good news, and some bad news.
Good news
I was able to add support for this feature in a forked repo! (forked commit)
There's not a PR since I haven't written tests yet. I'm debating writing tests, based on the bad news...
Bad news
It seems like the standard FTDI driver (VCP serial port driver) does not make these values available.  (at least that's the answer I got here). So for most USB to Serial adapters, this update won't provide anything useful.
Solution?
As far as I can tell, the only way to get these values from an FTDI chip is by using the D2XX driver. Then these values can be accessed directly from their C API. Unfortunately, it doesn't seem like you can access the device as a serial port when you're using this method. So this solution is moot for node-serialport. However, there is a solution for those who are stuck in this situation...
The node-ftdi module provides bindings to the D2XX APIs and I've submitted a PR to add bindings to make the modem status values available.
Conclusion?
So, I'm not sure where that leaves this feature request. The forked commit I made would probably be useful to real serial ports, but I'm not sure how common that would be.

Well, that sounds promising. Thanks for your efforts - I'll try this out soon.  The "real serial ports" scenario probably isn't all that common, but it's the one I'm interested in.
At a glance the main problem with D2XX or node-ftdi would be the need to 'find' the device rather than access it via a name ('COM4') or path ('/dev/ttyUSBx', etc.), or otherwise figure out how to address it.  Past that point the D2XX API seems to have everything that's needed.
Perhaps use of D2XX could be a config toggle for those who know that they need it.  (Not that I want to complicate things.  I can see why you're debating going further with this.)

A nice thing about D2XX addressing, is when you set the VID/PID on the chip (via FT_PROG), you can target your device directly, instead of asking the user to select from the list of serial devices. I'm not entirely sure how to pick a good VID/PID though. For now I'm using the FTDI default VID and custom PID.
I haven't tested reading/writing to the device with node-ftdi yet.

More good news. It looks like the problem in my earlier message, isn't really a problem. It turns out that the stock FTDI drivers on OS X are buggy and do not provide access to the control signals (original post). Downloading the drivers from FTDI fixes the problem...mostly.
I can now get the accurate signal status from a simple python script I wrote and the ruby script from the fore mentioned post. However, my updates to node-serialport still don't seem to read them. That's okay, though, this gives me hope.

My fork of the code is now working on OS X, Windows (and I assume Linux)!

I can handle firing the event, but how is the status read? I thought your
prior note said that you can't.
On Wed, Oct 5, 2016 at 5:49 PM, Francis Gulotta<EMAIL_ADDRESS>wrote:

@neilger https://github.com/neilger we can support querying for the
status but we don't currently have a way to support firing events when the
status changes.

In pyserial I do this by waiting:
     while (s.getDSR() != True):
        time.sleep(0.001)

That's the simple functionality I'd like to replicate.
On Wed, Oct 5, 2016 at 5:54 PM, Francis Gulotta<EMAIL_ADDRESS>wrote:

It's possible for us to support polling, we don't currently. As far as I
can tell we don't know how to support getting events when it changes.

No, sorry about the back and forth with the discussion above. That code should work for any standard serial port.
There were some problems with the driver on my side that initially led me down a path of a custom D2XX implementation. However, when I realized that I just needed to upgrade my driver, I went back to implementing something more standard. I've tested this code with FTDI devices using the standard driver on Mac and Windows.


Feed

Describe your suggested feature
Feed now need close tab for refresh how about add refresh button or pull down for refresh feed
Other details
as described
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.8.2.
[X] I will fill out all of the requested information in this form.

Create an issue for TachiyomiSY instead


@conda-forge-admin, please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pypyr-feedstock#12.


Servicebus messages cannot be completed or abandoned outside the receiver context?

Package Name: servicebus
Package Version: 0.50.3
Operating System: OSX
Python Version: 3.7

Describe the bug

Package Name: servicebus
Package Version: 0.50.3
Operating System: OSX
Python Version: 3.7

Describe the bug
Not sure if it is by design I cannot find anything helpful in the docs. Here it is:

I have a function that receives messages using a queue receiver as a context like

with queue.get_receiver(mode=ReceiveSettleMode.PeekLock) as receiver:
    msg_batch = receiver.fetch_next(max_batch_size=1, timeout=queue_read_timeout)

then msg_batch is retuned by the function and processing happen.
at the end I want to complete or abandon messages.

Problem is that I keep getting an exception MessageSettleFailed calling both message.abandon and message.complete
NULL message_receiver' (b'/Users/runner/runners/2.171.1/work/1/s/src/vendor/azure-uamqp-c/src/message_receiver.c':b'messagereceiver_send_message_disposition':494)

Looks like the receive is not set. Does this mean that I need to keep the processing in the receiver context? Or have a global receiver? With the old control client I could simply rely on the service bus client...

Hello again @andrea-cassioli-maersk  :)
So your assessment is pretty on-the-mark.  In 0.50 you won't be able to settle a message outside of the receiver context.  While I don't have deep familiarity with the prior historical version I would postulate it worked there because it was almost purely REST based, which may have let operations happen with fewer constraints than in 0.50 and beyond when we utilized the AMQP channel.
To give some good news however; allowing settlement-after-client-close is a topic we've discussed and intend to release with the 7.0 GA.   I need to check where we are in that, since some components have already been emplaced, and will loop back here.

OK, thanks. The overall logic makes sense (I get it that the message needs to retain some kind of connection object reference).
I was surprise of such a change moving from 0.20 to 0.50, as they are minors. Moreover, I could not find any specific mention in the doc and the example are so basic!
On top of that, the error I got was pretty obscure and I could have a guess only because it printed out something like Receiver NULL returned in an error message from AMQP C code.
Anyway, that's it I guess. I will close the issue.

Doc gap is totally understood; an unfortunate bit of debt from the history of how these libraries grew more informally in the beginning.  I've added an issue to create subsections in our GA version migration guide to specify coming from .2 vs .5 to hopefully mitigate this sort of confusion for the future.

Hey @andrea-cassioli-maersk ; Wanted to leave a followup here regarding settlement-out-of-context:
The b7/b8 transition contained substantial architect feedback, with their goal being to move settlement methods from the message entities onto the receiver, in order to allow those messages to be treated as pure data-objects and make the "operations requiring a receiver connection" clearly linked to the receiver, and, as such, tied to the receiver's lifecycle.
To be precise, this has the result that one will have to settle within the receiver lifespan.  Don't hesitate to shout (and you have my email if need be, since my github notifications are a brick wall) if this seems utterly insane to you; just wanted to make sure I left the direction this had settled and the reasoning here for posterity.  (the migration guide is, per our discussions, of course still being updated accordingly)


METAMODEL-231

Reproduced METAMODEL-231 with an integration test as well as tested with Neo4j Sandbox (http://neo4j.com/sandbox).

Cool, thanks Tomasz! :) @ClaudiaPHI maybe you can give it a test run in DataCleaner before we merge it (if you have time)?

Yes. I will do it asp. :-)

:+1:
I have tested it and the relationships look nice now.

Movie table:

Person table:

Great, I'll merge it in. Thanks to both of you for fixing and testing :-)


Rename configuring-a-custom-domain-for-your-github-pages-site.md to s…

…ina-kafi.com

Why:

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

initial

initial


PostLogoutRedirectURI does not work for Organizational Accounts

It does not seem to work for accounts in the Organizational Directory.
In particular it gets stuck on this screen after you've selected the account to sign out with:

Reading up on other solutions on the internet I noted that usually you need to send an id_token_hint to get this to work, but still no success. Below is the code added to the OpenIdConnect Notifications which does successfully add a non-null idTokenHint
private Task OnSecurityTokenValidated(SecurityTokenValidatedNotification<OpenIdConnectMessage, OpenIdConnectAuthenticationOptions> n)
        {
            var id = n.AuthenticationTicket.Identity;
            n.AuthenticationTicket.Identity.AddClaim(new Claim("id_token", n.ProtocolMessage.IdToken));
            n.AuthenticationTicket = new AuthenticationTicket(
              id,
              n.AuthenticationTicket.Properties);
            return Task.CompletedTask;
        }

       private  Task OnRedirectToIdentityProvider(RedirectToIdentityProviderNotification<OpenIdConnectMessage, OpenIdConnectAuthenticationOptions> n)
        {
            if (n.ProtocolMessage.RequestType == OpenIdConnectRequestType.Logout)
            {
                var id_token_claim = n.OwinContext.Authentication.User.Claims.FirstOrDefault(x => x.Type == "id_token");
                if (id_token_claim != null)
                {
                    n.ProtocolMessage.IdTokenHint = id_token_claim.Value;
                }
            }
            return Task.CompletedTask;
        }

I find it strange that it doesn't work, because on other projects I've done when using MSAL.js for client side apps it does work. So it should be able to work somehow.

I resolved this and posted the solution on one of my old stackoverflow posts here: https://stackoverflow.com/questions/47419128/post-logout-redirect-uri-does-not-redirect-when-authenticating-using-azure/65368995#65368995


Update gh build action dependencies

Changed

Updated GH build action dependencies

Codecov Report

Merging #122 (dbd68b0) into master (3dc29a8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head dbd68b0 differs from pull request most recent head 9e9647e. Consider uploading reports for the commit 9e9647e to get more accurate results

@@           Coverage Diff           @@
##           master     #122   +/-   ##
=======================================
  Coverage   96.20%   96.20%
=======================================
  Files          94       94
  Lines        1607     1608    +1
  Branches      215      215
=======================================
+ Hits         1546     1547    +1
  Misses         58       58
  Partials        3        3

Flag
Coverage Δ

cuaktask
99.25% <ø> (ø)

iocuak
94.14% <100.00%> (+<0.01%)
:arrow_up:

porygon
98.58% <ø> (ø)

porygon-typeorm
98.90% <ø> (ø)

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

Impacted Files
Coverage Δ

...cuaktask/ContainerInstanceServiceImplementation.ts
100.00% <100.00%> (ø)

...ask/models/cuaktask/CreateInstanceDependentTask.ts
100.00% <100.00%> (ø)

...s/cuaktask/GetInstanceDependenciesDependentTask.ts
100.00% <100.00%> (ø)

...tanceTask/modules/TaskBuilderWithNoDependencies.ts
92.00% <100.00%> (ø)

...c/createInstanceTask/models/domain/TaskKindType.ts
100.00% <0.00%> (ø)

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 6f5297b...9e9647e. Read the comment docs.


CP-2682: release to worker and app tier

Untested currently.

@Azrael808 I made the changes suggested above. Please double check if they make sense.

@bru these totally make sense! Might be why my worker deploy failed on Tuesday 😄


add DecodeWithFormat function

The standard go image library provides the format return value when calling Decode.
The imaging library provided the FormatFromFilename to get the format of the image (needed for encoding into desired image format).
Given that for performing image manipulations the image has to be loaded in memory,  the format comes for free when calling img.Decode.
I did not modify the imaging.Decode functions to not break existing users of this library.

Coverage remained the same at 100.0% when pulling 89923699af688e948a0eb884884e075a46be7df1 on labkode:decode-with-format-return into 5e63c9a565d0a3260d23d060a795783dab90fefe on disintegration:master.

Coverage remained the same at 100.0% when pulling 89923699af688e948a0eb884884e075a46be7df1 on labkode:decode-with-format-return into 5e63c9a565d0a3260d23d060a795783dab90fefe on disintegration:master.

Coverage remained the same at 100.0% when pulling 89923699af688e948a0eb884884e075a46be7df1 on labkode:decode-with-format-return into 5e63c9a565d0a3260d23d060a795783dab90fefe on disintegration:master.

Coverage remained the same at 100.0% when pulling 89923699af688e948a0eb884884e075a46be7df1 on labkode:decode-with-format-return into 5e63c9a565d0a3260d23d060a795783dab90fefe on disintegration:master.

This function doesn't add anything to image.Decode. Imaging is not supposed to be a replacement for the standard image package. I think it's better to use image.Decode directly in this case.
As for the current version of imaging.Decode, I've added it for symmetry with imaging.Encode and at the time, it also converted image to NRGBA format after decoding. I've removed the conversion later, but the function itself is left for backward compatibility.


[BUG/Help] <求助>

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
can i run ‘web_demo.py’, throw a err :Failed to load cpm_kernels:[WinError 267] 目录名称无效。: 'D:\Program\anaconda3\Scripts\conda.exe'
Expected Behavior
No response
Steps To Reproduce
Failed to load cpm_kernels:[WinError 267] 目录名称无效。: 'D:\Program\anaconda3\Scripts\conda.exe'
Environment
- OS:Win11
- Python:3.10
- Transformers:4.27.1
- PyTorch:1.11.0+cu115
- CUDA Support (`python -c "import torch; print(torch.cuda.is_available())"`) :true

Anything else?
No response

The conda and pytroch environments cannot exist at the same time, otherwise an error will be reported, and suddenly there is no problem when I try to uninstall anaconda

If you want to use virtual environments, please refer to this document   Windows部署文档


Threaded Listening within Client Class

I'm writing up a program that allows clients to send messages and interact with fb chat via input. I want to be able to listen() and run onMessage() in the background (in a different thread), while also interacting in my 'interactiveMode' loop, but i can't seem to get the threads to work how i thought they would.
My code at the moment inside my Bot(Client) class:
def onMessage(self, author_id=None, message_object=None, thread_id=None,
              thread_type=ThreadType.USER, **kwargs):

    print("new message received!")

def interactiveMode(self):
        listenerThread = threading.Thread(target=self.listen())
        listenerThread.start()

        while 1:
          # ... <code here>...

at the moment after I initialise the client and run client.interactiveMode(), the rest of interactiveMode code after the threading for self.listen() never runs and i'm stuck in self.listen(). my interactiveMode function works as expected by itself, however i would like the additional feature of printing to terminal when a new message comes in. Thanks!

Your error is probably in threading.Thread(target=self.listen()) - it should be threading.Thread(target=self.listen) (without parentheses).
But I must warn you that fbchat is not thread safe!!! All sorts of wierd stuff can happen if you try this, so beware: You're on your own if you do this, the functionality you're trying to implement is not supported!

 All good, thanks for the help

No problem 👍


iOS not discovering homebridge-lifx-lan

Hey all,
I have homebridge running (I think) successfully. Terminal has loaded the plugin and has produced pin.
However on iOS using the HomeKit.app it can't seem to find my lifx.
Anyone got any suggestions?

Hi ! Can you give me a hand? I have a lot of problems.


Don't crash when the RGB script to be parsed is empty. Fixes #1185

See description in #1185

Thanks :+1:


Instant Email Notifications for Direct Messages

@Zaydme to add details on how to query for new DMs to send emails for.

Email notifications for DMs can be achieved by querying the threads collection,

looking for participants.id equal to user.id (participants is an array of objects, probably not the most efficient)

in the matching participant

WHERE newMessages (Number) is greater than 0
AND lastAccess (Date) is older than whatever the "instant" emails delay is.
IF status is "accepted", just send normal email about new message.
IF status is "pending",  send email about "New message request".
IF status is "blocked" OR "ignored" OR "archived" don't send any email.

in the unmatching participant, (which is the sender in this case, since only 2 participants in the array) you can get:

id (Ref: User)
name and photo (which should be up to date)
type can be one of these [ "Company", "Community", "Government", "Health care provider", "Individual", "Non-profit", "Other", "Startup", "University", ]

notes

if you want the message content, you can query the messages collection for the last message with the threadId equal to the thread._id.
you can also find the message status, which can be one of ["deleted", "edited", "sent"].
Some messages, especially the first one have a postRef, which is the embed post in the message.

so when the status is "pending" and you send an email about "New message request" you will need to include the postRef.title (you can also get the id, objective, content andcreatedAt (which is the post creation date, not the Message document creation date))

Currently in progress.

https://featurenotifications.fightpandemics.xyz/post/5fb46a8ae50b38001183f07a
I didnt get any email for pending status. Just got one email for message thing after clicking on Accept button though send many messages. Please check.

No timestamp in this messages and no email also generated for these messages

Email generated for only single message

@shrutigupta1981 I can't really say if this is expected behavior without knowing the exact reproduction steps. but I will point out that the receiving user's inbox should not be open in your other browser session when you send a direct message to that user. Otherwise the user will be considered "online", and they will not get any email. I would recommend the following test cases be followed, and if the actual result deviates from expected, please note it in this ticket. Thanks!
Case 1 - two users who have never communicated with each other - message from profile page

Open two separate browser sessions for two different users, and make sure that the receiving user is not on the DM inbox page.
From one of the users, send a message to the other user from their profile page (not a post)
Wait at least 10 minutes

Expected: The receiving user receives an email with the subject and body of <user name> sent you a direct message request
Case 2 - two users who have messaged each other at least once - reply back in the inbox

Open two separate browser sessions for two different users, and make sure that the receiving user is not on the DM inbox page.
From one of the users, go to the DM room in the inbox for the receiving user, and send a reply back to the user.
Wait at least 10 minutes

Expected: The receiving user receives an email with the subject <user name> sent you a direct message, and the body should contain the direct message text.
Case 3 - two users who have never communicated with each other - message from post

Open two separate browser sessions for two different users, and make sure that the receiving user is not on the DM inbox page.
From one of the users, send a message to the other user from one of their posts
Wait at least 10 minutes

Expected: The receiving user receives an email with the subject and body of <user name> sent you a direct message request, and the body should contain the title of the post.


is this project dead?

it hasn't been update for a long time

https://github.com/jonataslaw/getx/pull/2008 💁🏼💁🏼

oh, i see


Coolant and waste flow monitor valve indicator change

Have the valve indicators change from OPEN to CLOSE when that specific valve is closed. Also maybe change the default, white if valve is open, to green, and when it's closed have it change to yellow or some other color. The image below is what I was kinda thinking (I used photoshop)

I appreciate the suggestion, but I'm going to add a bit of context first. This is going to be a bit verbose of an explanation of design, but here it goes!
A lot of design decisions I took were based around a few self-defined rules, which include these relevant ones below:

Keep the coordinator displays static, like actual control room panels. This means things are always in the same place and always visible (no drop-down menus, no tabs, etc).
Attempt to follow closely actual guidelines and real control rooms. This strongly influences many things, among them are terminology and indicator color selection.

Number 1 is why the text of indicators lights never change, which is why OPEN is what I settled with. There are however the state indicators I have that are used for multi-block states (reactor, boiler, turbine, matrix, SPS, etc) and the unit waste production on the main view screen. Aesthetically I generally prefer indicator lights, but an opened/closed pair of lights would have taken too much space. Putting state indicators here is something I am open to, so I'll draft out an idea when I get the chance (probably after the next update). The reason I am suggesting state indicators is partially because of the next half of this explanation:
Number 2 applies to color selection here. I tried to stick indicators specifically close to this international atomic energy agency document https://inis.iaea.org/collection/NCLCollectionStore/_Public/27/002/27002051.pdf (which is part of many links here https://github.com/MikaylaFischler/cc-mek-scada/wiki/Design-and-Regulatory-References):

Red: immediate action required by the operator to correct the fault, with the means of action
being in the control room
Yellow: action required, the time delay for action being defined by the nature of the fault and
the location of controls available to the operator
White: information indicating a change of state or an automatic action being taken
Green: automatic action being taken by the safety systems, Operators must immediately verify
that the action requested by the system was done.

Red and yellow are the ones I've tried most to keep assigned to or associated with the above roles. In a recent update I had changed a few indicators from being red/green to off/green, since the red wasn't actually indicating a problem (units not ready as one example). Green has been my more flexible color, to indicate both connections and safe states. The front panels on computers themselves are a different story (which is going to make #340 a bit more complex), as these and other visual guidelines are only being applied to the main control room view. The front panels on computers are modeled instead after industrial automation hardware.
So, yellow here would imply a problem, rather than a functional state (open vs closed). I had debated for a while what colors and what text I would use here, and this is what came of that. Not ideal, I agree, but it seemed to be the clearest.
In short, white is the only color I can use here (if I want to follow the guidelines I've been following) to indicate a neutral action having been taken (valve state change), unless I use a state indicator rather than an indicator light.
tl;dr I'll draft up a state indicator in the place of the open indicator and we can see how it ends up looking. It'll come down to the balancing act between being realistic yet also intuitive (alongside every other design choice I've made 😅).


[Backtracing] Improve warning message about thread suspension failure.

In the event that the backtracer fails to suspend a thread, it emits a warning message.  Unfortunately the exact meaning of this message isn't obvious and it doesn't make clear that it's a secondary problem (that is, you've already crashed at this point).
Update the message to make things clearer.
rdar://116593541

@swift-ci Please smoke test


Obtaining the line number

If I discover a problem with a node (e.g. a property is not set) I would like to generate an error message with the line number from the RDL file. Is that information available?

Since you simply want to print messages to the user, I recommend using the existing message handler. This will print messages in a format that is consistent with the rest of the compiler, and also include source context. I assume this will do what you need implicitly.
Here is an example that uses the atxmega_spi.rdl demo file:
from systemrdl import RDLCompiler
rdlc = RDLCompiler()
rdlc.compile_file("atxmega_spi.rdl")
root = rdlc.elaborate()

# Print messages without any context
rdlc.msg.warning("Warning Message")
rdlc.msg.error("Error Message")

# Print messages with source context
data_if_field = root.top.find_by_path("STATUS.IF")
rdlc.msg.warning(
    "Field definition is here",
    data_if_field.inst.def_src_ref
)
rdlc.msg.warning(
    "Field instance is here",
    data_if_field.inst.inst_src_ref
)
rdlc.msg.warning(
    "Assignment of the 'hw' property is here",
    data_if_field.inst.property_src_ref['hw']
)

Running the above script produces this output:

I'll add the RDLCompiler.msg attribute to the documentation. That really ought to be part of the public API.

Updated docs:

https://systemrdl-compiler.readthedocs.io/en/latest/api/compiler.html#systemrdl.RDLCompiler.msg
https://systemrdl-compiler.readthedocs.io/en/latest/api/messages.html#systemrdl.messages.MessageHandler
https://systemrdl-compiler.readthedocs.io/en/latest/api/component.html#systemrdl.component.Component.inst_src_ref

Just what I needed. Thank you very much.


Improve UI of grading page

General improvements of the UI of ag-grid:

Applied theme where there was none previously

Shifted some columns for better grouping

Auto-size the columns on window resize

Before:

After:

Pull Request Test Coverage Report for Build 1855

1 of 15 (6.67%)  changed or added relevant lines in 1 file are covered.
23 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 27.589%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/academy/grading/index.tsx
1
15
6.67%

Files with Coverage Reduction
New Missed Lines
%

src/components/academy/grading/index.tsx
23
19.77%

Totals

Change from base Build 1854:
-0.009%

Covered Lines:
1402

Relevant Lines:
4388

💛  - Coveralls

Do consider uploading a screenshot of the before / after change so its easier for everyone to see the effect of the UI changes.

Added. You can check it out and if there are any other suggestions please voice them out

@rrtheonlyone what platform is that? it looks fine on firefox and chrome though.

Possible to centralise the text in each cell? (vertically centre)

I think #670 is a better PR, closing this PR.


detect openrc configuration files locations at runtime

arch linux , systemd-free.org and manjaro use different locations for openrc.
Using SYSCONFDIR and BINDIR ,  we can set the correct location at build time.
Packages providing openrc service files for arch linux can not be used on systemd-free.org , manjaro  and vice versa.
If the locations needed for services could be detected at openrc runtime, it should be possible to create  packages that work on all systems that share a package manager / package build system..
Is there a way to achieve this ?

I don't quite understand your question. Can you elaborate a bit more on what these distros are doing differently?

@williamh
Arch guys use SYSCONFDIR=/etc/openrc, whereas manjaro use default /etc, which makes in fcat arch and manjaro openrc packages incompatible.
I have made ages ago the proposal to use default SYSCONFDIR=/etc on arch as well, but no success.
It supposedly clashed with systemd, which it does not.
In my view, its a packaging issue.

I don't see how this has anything to do with run-time vs build-time configuration.  In order to be able to use service files from the various projects simultaneously, OpenRC would need to be able to use multiple configuration directories regardless of when they are configured.

My opening post was indeed way to vague and i wasted your times.
Attempt to clarify :
Last weekend i was working on upgrading arch linux openrc pacakge from 0.22.3 to 0.24 .
While working on the new opentmpfiles packages I used gentoo ebuild and manjaro PKGBUILD as examples.
I noticed a few things :
init.d service files are hardcoded to use #!/sbin/openrc-run
gentoo ebuild does use this, both manjaro and arch linux PKGBUILDs change the initd file to use #!/usr/bin/openrc-run
gentoo ebuild and manjaro PKGBUILD put service files in /etc/conf.d and /etc/init.d , while arch linux PKGBUILD puts them in /etc/openrc/conf.d & /etc/openrc/init.d .
I looked at this as a packager and thought how a template package usable for all openrc distros might look.
At buildtime of a servicefile package two data are needed :
the location of openrc-run and that for the configuration folders
This could be figured out by checking the existence of command & folders in often used locations, but that is prone to errors.
Otoh, if openrc is running it knows the exact values for these 2 locations.
Upon installation , openrc could put a script exporting 2 variables (ex: openrc_sbindir and openrc_confdir ) in a location like /etc/.profile.d so it will b eexecuted every boot.
A template for a servicefile package in pseudocode would then look something like this :
IF NOT openrc_sbindir THEN echo "openrc not installed, aborting"

IF openrc_sbindir NEQ "/sbin/openrc-run" THEN adjust initd-file
copy confd-file to openrc_confdir/conf.d/
copy initd-file to openrc_confdir/init.d/
IF  auto-enabling_is_wanted THEN
openrc_sbindir/rc-update add <service> <runlevel>

I lack the xp with and insight in openrc to know if this is a good solution, but it does solve the issues i noticed.
@udeved :
In my opinion this is  a genuine issue and solving it will improve openrc.
please treat it like that.
Links
https://gitweb.gentoo.org/repo/gentoo.git/tree/sys-apps/opentmpfiles/opentmpfiles-0.1.3.ebuild
https://github.com/manjaro/packages-openrc/blob/master/opentmpfiles/PKGBUILD
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=opentmpfiles

So you want to be able to query build-time configuration settings at run-time in order to use them in a PKGBUILD so that the PKGBUILD, not the built package, can be shared across distro's?

@LW-archlinux
I find it just a bit sledge hammer to solve this problem. If we could only agree to use default /etc, our packages would be fully compatible.

@andrewgregory : that is indeed the idea.
If setup correctly, it could work with rpm source packages , all distros that support the gentoo ebuild format and such. Maybe debian/ubuntu based distros and BSDs would also benefit.
@udeved :
compatibility with other init systems is the most important factor imo.
On arch linux i can have runit, systemd, openrc and sysvinit installed at the same time .
Choosing which is started is done through init= lines in the bootloader.
I boot into openrc 99% of the time, but do have a few usecases where i need to boot into one of the others. So i do have  a personal interest in keeping openrc on archlinux compatible with other init systems.

Its a matter of taste I think,
I would maintain, a user should decide, which init is being installed, a decision between choices the user has to make.

@LW-archlinux
I do also believe something like this could be beneficial to make setup a little bit easier. It would make less work for packagers.
@udeved
I ran into the same problem with init.d being in /etc. There was a conflict with other init systems if I had more than one installed at a time. I started work on a new distro based on linux from scratch that uses pacman for the package manager and sysvinit/openrc as it's primary init system. It does also have lfs bootscripts in it which conflict with /etc/init.d, so in my distribution the solution was to move openrc into /etc/openrc/init.d otherwise I couldn't have both lfs bootscripts and openrc installed at the same time. Thanks to the separation I now have sysvinit, lfs bootscripts, s6, and runit all working together for init and I have openrc, lfs bootscripts, s6 scripts, and runit scripts to handle services, supervision, and rc. They all can coexist if I stick openrc in "/etc/openrc/init.d". You are right about init.d not conflicting with systemd, but it does conflict with arch bootscripts and lfs bootscripts and pretty much any other boot method that uses init.d as it's location to store scripts.

@cromerc
Yes, I know, my point is, you also can't have systemd and eudev installed at same time, its a choice to make.

@LW-archlinux any service scripts written outside of openrc should have a shebang line that looks like #!/sbin/openrc-run in order to have the widest compatibility. Think of this as how we write shell scripts; we don't put sh in /usr/bin and request that everyone change all of their shell scripts to have a #!/usr/bin/sh shebang.
Regarding the location of service scripts (/etc/init.d), I think the cost of changing the default would be very high at this point; it would require a lot of work for users and packagers, so I would have to preserve the default as-is.
I would like other input before I make a decision on what to do with this, I am hesitant to change the location of service scripts.

@williamh
As for shebang line, default #!/sbin/openrc-run doesn't work on arch, the service won't be recognized, because of /usr merge.
If there was to change something in locations, it should probably be two new variables $SYSCONFDIR/$INITD and SYSCONFDIR/$CONFD ? This way, SYSCONFDIR would still point to /etc by default.

The shebang line doesn't work on the distro I am making either because of the merged /usr folder. I had to rewrite them to #!/usr/bin/openrc-run otherwise openrc looses track of the services.
Apparently openrc can't handle the symlinks correctly which forces us to change the shebang line.

My understanding of the usr merge is that /bin, /sbin and /usr/sbin all are symbolic links to /usr/bin, correct?

Thats correct, however, if we don't sed the shebang, the service won't be detected by openrc.
I painfully found out once, when I forgot to sed older version of the udev scripts.
Btw, any chance the get merged in openrc as optional make?

Correct. But openrc fails to follow the symlinks. So as a work around we were forced to change the shebang lines to /usr/bin/openrc-run
If we don't change them the service dissapears completely from openrc and stops running.

I need to know more about what breaks if symlinks are used.
The shebang lines are not being interpreted by OpenRC; those are interpreted by the kernel.

What ends up happening is that if I type the command rc-status, the service has disappeared completely it is invisible. It does not show started, stopped, crashed or any other status. The service also is not running in the background, so either it never started or it stopped immediately after being run. If you want to try it out you can test this with this ISO: https://sourceforge.net/projects/archopenrc/files/arch-openrc/archlinux-openrc-2017.03.01-x86_64.iso/download
When the ISO boots you can change the shebang lines in the service scripts to play around and witness the problem.

Can you link to the PKGBUILD that built the openrc on that ISO?

https://github.com/arch-openrc/arch-openrc/blob/master/openrc/PKGBUILD

Hmmm I just tried it again, this time the service didn't crash and is still running the background. But it still went invisible and is not showing up in rc-status. I tested by changing the shebang line on sshd. Maybe it is just certain scripts that crash not all of them.

If you have the /usr merge symlinks in place, you don'5t need to set prefix or sbindir in your pkgbuild.

Without the prefix and sbin options the package is invalid and can't be installed on arch linux because of a /bin and /sbin conflict. Should I just manually move the stuff from bin and sbin into /usr/bin during the packaging then?

At present, gendepends.sh checks that the shebang line of the initscript matches @SBINDIR@/openrc-run (or runscript, but that's deprecated), and if not, it gets skipped, and not added to the deptree; rc-status then skips it because it's not in the deptree.  That's why things disappear when you change their shebang (and services that have needed dependencies will crash because those dependencies aren't started).  A fix that wouldn't require manually moving things around during package() in the PKGBUILD would be to set up the /usr merge symlinks in the pkgdir before make install, and then remove them after.  Then remove the setting of PREFIX, SYSCONFDIR, and SBINDIR, and you should probably set LIBEXECDIR to /lib/rc (If /lib64 is a thing on amd64, use that instead of /lib on amd64).
We'll probably remove the shebang check from gendepends.sh, but that's still being discussed at this moment.

Alright, I will start repackaging everything with those changes. Thanks.

Thanks a bunch, everything works fine now with the standard #!/sbin/openrc-run shebang line. I rebuilt all my packages. The big problem now is the Arch Linux AUR... hopefully everyone will update their openrc scripts.


Mention send_wrapper in the migration guide

Based on @konstin's comment in Gitter.

(Sorry for the force-pushes; I tweaked the wording and commit message shortly after opening the PR.)

Hmm... :thinking:
SendWrapper is internally a pair of Box<T> and ThreadId.
I don't want to recommend it as a thread-safe alternative of T since it requires redundant Box.

Ok :+1:

While it gives a small performance hit, it is imho still preferable to unsafe impl Send which can cause nasty bugs if used incorrectly.

This change showed both possibilities, so it's definitely helpful when the contents are not threadsafe after all.

I would also be in favour of encouraging users to use send_wrapper instead of unsafe impl if they don't know what they're doing.
As a few of us are in favour of this, reopening.

I agree that we provide no good interface for a class that we don't want to send.
So, as a builtin alternative of send-wrapper, I opened #1009.

Closed in favor of #1009


Feature request: support for brotli

Any plans to add brotli to the mix? It's the most efficient code to date, and widely supported in all modern clients.
Thanks in advance!

Seem like brotliCompressSync hasn't been implemented on bun yet so I'm looking for a workaround rn

Yes, it might take using an external binary but iDK if bun supports that type of module installation. I never could install anything that needs a compiling phase.
Thanks for looking into it though. Let me know if I can help in anything towards this goal.

Yes, it might take using an external binary but iDK if bun supports that type of module installation. I never could install anything that needs a compiling phase. Thanks for looking into it though. Let me know if I can help in anything towards this goal.

Aight, I've already created a feature request on bun rn, https://github.com/oven-sh/bun/issues/6152


OLS-1074: konflux - udpate fbc-4.17 pipelines

Description
This PR completes https://github.com/openshift/lightspeed-operator/pull/442
add on-cel-expression filters to avoid excessive build on unrelated files.
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue # OLS-1074
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

/lgtm
/approve


Use TextDocumentWillSaveEvent to make changes to files being saved

This PR fixes https://github.com/Lonefy/vscode-JS-CSS-HTML-formatter/issues/53 by not applying formatting edits with the active editor but via the on-will-save API. This is required because documents that are not open or not open in the active editor can be saved and get scrambled with the existing logic.

Thanks!


Search Bar, User Dashboard, and basic Home Page

All of these pages need to be integrated with backend
User Dashboard

Search Results

Home Page

I tested @AbdulTur 's changes (adding the search API call) and it works fine for me. I did some testing on the itinerary creation and it didn't create any API request. Is this expected? I'm not sure the level of functionality we are going for here. All of the other pages LGTM though. If someone can confirm this itinerary creation behaviour I can approve

I tested @AbdulTur 's changes (adding the search API call) and it works fine for me. I did some testing on the itinerary creation and it didn't create any API request. Is this expected? I'm not sure the level of functionality we are going for here. All of the other pages LGTM though. If someone can confirm this itinerary creation behaviour I can approve

Itinerary is not implemented yet, i tested the search with the test query u provided

Yeah itinerary is not implemented but pretty sure Jordyn is doing that on her branch


New week, new test-fixture version

It's a brave new world out there.
@justinfagnani @cdata

Tests are green now, merging!


added check for running meetings

After accidentally killing a live server with an ansible run (limited to the wrong pattern) I decided to add a check for running sessions to my old playbook for BBB2.2.
Now I've added that check here.

Yes, this should be optional but maybe enabled by default?
Before this, I had a way to only run "restart" etc. Handlers when I had a variable set - and of course at some point I just kept the "-e bbb_doit=yes" in the command line at all times, so the accident had to happen...
Do you have a suggestion how that variable should be called? Would "bbb_check_for_sessions" fit the bill?

bbb_check_for_running_meetings sounds better to me. A session is not as clear to me as compared to a "running" or "ongoing" meeting.

Ok, I added that variable with the better name now.

I found another issue: "geerlingguy.docker" and "geerlingguy.nodejs" are executed before this check, because Ansible loads all roles including dependencies defined in meta/main.yml and runs those first... I guess that would mean refactoring to "include_role", so the other roles are executed at a defined time instead of the current behaviour.

I found another issue: "geerlingguy.docker" and "geerlingguy.nodejs" are executed before this check, because Ansible loads all roles including dependencies defined in meta/main.yml and runs those first... I guess that would mean refactoring to "include_role", so the other roles are executed at a defined time instead of the current behaviour.

I am pretty sure my checkvars tasks is getting executed before anything else is done, to provide a fail fast intro instead of failing mid runtime.
https://github.com/ebbba-org/ansible-role-bigbluebutton/blob/90f41a0d39c579e92bdbefc36c78e8bc0cb13275/tasks/checkvars.yml
Maybe you can try adding your check there in a similar fashion?

@BrutalBirdie the way the prerequisites are included, makes them run before anything of this role. So gerlingguy.docker or gerlingguy.nodejs might change something on the target and perhaps disrupt running BBB meetings. And I've already inserted "check-for-sessions.yml" right after check_vars (which still runs after docker and nodejs roles)...
The other issue:
The task won't fail in case you're running the playbook in "check" mode (-C) or if the "uri" request does not complete at all (e.g. when there's no BBB installed yet) - look here: https://github.com/ebbba-org/ansible-role-bigbluebutton/blob/ac33d8a90c7f0c55c40477474790180b070a53f0/tasks/check-for-sessions.yml#L15

@BrutalBirdie the way the prerequisites are included, makes them run before anything of this role. So gerlingguy.docker or gerlingguy.nodejs might change something on the target and perhaps disrupt running BBB meetings.
The other issue:

According to Stackoverflow there are two ways to do this: Either I remove this from "tasks/main.yml" and document how people could add this to the "pre_tasks" section in their Playbooks. But at that point a lot of the needed variables aren't defined yet, so not really.
Or we'd need to move the geerlingguy-Roles to "tasks/main.yml" via "include_role" (Ansible >=2.2) or "import_role" (Ansible >=2.4). Means quite a change in the playbook and perhaps dropping support for ancient Ansible versions - can I request that?


Feature request: Headers

Extend API to be able to send headers in NATS messages. This would be very useful in scenarios where you have an access token or similar auth in headers.

Hello, right now I am on holiday, once I return to work I will triage and study the issue properly.
Thank you very much :-)


Add the nav links for 3.0 docs and update the error nav links for docs 2.4

Can one of the admins verify this patch?


Improve design in getting rider details form page

Ride.html page contains rider details form after the animation.
It is essential to re-design the page as users are not able to see the riders details form . To know that there is a form user needs to scroll down . To improve the user experience and greater understandability of information that page holds it is necessary to change.

covers up the riders details form

I want to work on this . @sanjay-kv @rakeshkushwaha332 @akbatra567 @KiranBaliga @07sumit1002

@Anushree-241 sorry I am not the project owner or PA. Please contact Sumit.


chore/webpack chunks

Add Command icon
Experimenting with webpack chunks

What does it do?
Describe the technical changes you did.
Why is it needed?
Describe the issue you are solving.
How to test it?
Provide information about the environment and the path to verify the behaviour.
Related issue(s)/PR(s)
Let us know if this is related to any issue/pull request

@joshuaellis I'd like to have your opinion on this PR please

I'm not sure chunking would help? Isn't the issue here that when we build the library we wrap up Flex as a production ready file that includes Box as it's dep, but isn't it too early to be doing this? It then causes Box to be included multiple times.
This could always be to do with the fact we're releasing a UMD bundle. I think to solve the issue we probably want to be releasing a ESM, CJS and UMD version of the library.
The only example I can think of that I know works (doesn't duplicate it's imports from it's own lib) is https://github.com/pmndrs/drei where we import OrbitControls into the Stage component, when you build both the ESM & CJS version are using their respective versions of import to bring in the OrbitControls, so I assume when people build their packages / apps / libraries only one version of the OrbitControls is there.

We've merged a beta for using vite so i think we can close this.


Session cookie(s) not removed after logout with MVC hybrid client

If i logout using my MVC client, the browser gets correctly redirected to the /endsession endpoint, which in turn calls the Logout on the AccountController.
    [HttpPost]
    [ValidateAntiForgeryToken]
    public async Task<IActionResult> Logout(LogoutInputModel model)
    {
      // build a model so the logged out page knows what to display
      var vm = await BuildLoggedOutViewModelAsync(model.LogoutId);

      if (User?.Identity.IsAuthenticated == true)
      {
        // delete local authentication cookie
        await HttpContext.SignOutAsync();
      }
     return View("LoggedOut", vm);
    }

However, after returning to the MVC client the idsrv.session cookie and .AspNetCore.Identity.Application cookie are still present in the browser. Isn't the HttpContext.SignOutAsync() responsible for removing those local cookies ?
How should i remove those cookies to prevent auto-logins ?
Thanks in advance.

Isn't the HttpContext.SignOutAsync() responsible for removing those local cookies ?

It clears the default signout scheme. Sounds like there might be some misconfig in your ASP.NET authentication schemes.

All set on this issue -- can we close?

Hi @brockallen, please don't close yet. I'm currently having this issue. How do I resolve should it be a misconfiguration in the authentication scheme?
"Sounds like there might be some misconfig in your ASP.NET authentication schemes."

Hello @cxheun ,
Any Update on it? I'm facing same issue. i cannot logout from my MVC client application.

For what it's worth, I'm having the same problem. Logout works just fine, but the cookie remains. I haven't been able to get rid of it.

I'm having the same issue. In the client, the logout appears successful, but the Identity.Application cookie lingers, so if I navigate back to an authorized page, I'm automatically signed-in again. Manually deleting the cookie solves the problem. The server logs indicate a successful result from EndSession.
This occurs when running the AspIdentity quickstart, modified slightly to run in Asp.Net Core 2.1 (the only real difference is a route hack to strip out the "/Identity" area routes).

I found what fixes it on my end, though I'm not sure why my previous code, copied from the samples/forums, didn't work and do precisely the same thing. Here's the logout code from my account controller in my MVC 5 client app:
       [HttpPost]         [ValidateAntiForgeryToken]         public IActionResult Logout()         {             return SignOut("Cookies", "oidc");         }
That makes it work as expected. I'm still using an implicit model, but perhaps that can help you somehow as well? I find this whole project a wonderful contribution but very confusing to use. I expect that's a by-product of the whole authentication/authorization business being so complicated these days.

@mhxjbw even your suggestion didn't work for me, for some reason.
Here is what DID work for me: if you are using the Quickstart UI AND leveraging ASP.NET Core Identity, you'll want to swap a line in the Logout action method of the IdentityServer implementation:
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Logout(LogoutFormModel model)
{
    // build a model so the logged out page knows what to display
    var vm = await BuildLoggedOutViewModelAsync(model.LogoutId);

    if (User?.Identity.IsAuthenticated == true)
    {
        // delete local authentication cookie
        //await HttpContext.SignOutAsync(); // <-- replace this line
        await _signInManager.SignOutAsync(); // <-- with this one

        // raise the logout event
        await _events.RaiseAsync(new UserLogoutSuccessEvent(User.GetSubjectId(), User.GetDisplayName()));
    }

    return View("LoggedOut", vm);
}

If you're NOT using the SignInManager w/ ASP.NET Identity, I think I can still shed some light. It seems that the problem is stemming from the fact that the await HttpContext.SignOutAsync(); call within IS4's Logout action is not properly targeting IdentityServer's authentication scheme: "Identity.Application". If you manually supply that scheme, it works:
await HttpContext.SignOutAsync("Identity.Application");

Yes, when you're using ASP.NET Identity they wire up internally their own cookie scheme, so you need to use their API to revoke their cookie as well.

_signInManager.SignOutAsync()

What is "_signInManager"?

@jayoma Where is _event coming from? What is it?
IE: await _events.RaiseAsync(new UserLogoutSuccessEvent(User.GetSubjectId(), User.GetDisplayName()));


chore: add pageId and applicationSlug to all traces

Description
Add pageId and applicationSlug as common attributes to all spans
Fixes #Issue Number
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!WARNING]
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/11679770316
Commit: de6069395a4e0c6a92ad1604742f8e525b47ad28
Cypress dashboard.
Tags: @tag.Sanity
Spec:
It seems like no tests ran 😔. We are not able to recognize it, please check workflow here.
Tue, 05 Nov 2024 07:55:42 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true


Bunched up import/usings vs new line for each package

Looks like there isn't a common practice for this in the org yet, it was brought up in https://github.com/SciML/DiffEqBayes.jl/pull/262

https://github.com/SciML/SciMLStyle/commit/60e2ba9fdc83aea3a82887fdc319c624e525ac71


How it's possible to catch onDrawerOpen?

Hi!
I have drawer which you can swipe on every screen. But on some screens I have Modal DropDowns which if you leave dropdown open and then swipe to open drawer then drawer is under dropdown.
So how it's possible to catch onDrawerOpen where I can just close all dropdowns.
Thanks!

You mean this? https://github.com/kmagiera/react-native-gesture-handler/blob/master/DrawerLayout.js#L35

This issue seems to be solved.
@greenzeal
If you still face the problem, reopen the issue


In "Tasks" page, add "Version" column to the "Task collections" table

Let's have these columns:

Timestamp
Package
Version  (currently missing column)
Status
Options (currently missing column title)

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.

If a user specifies a version number, that is shown in the Version column. Otherwise, unspecified is used as placeholder.
The server does not provide any info about version, or the package version from which the tasks are imported. At least not in the tests I made.


Add check for capitalization for reserved powershell words

One thing i would like to see is rules that check case of reserved powershell words like function, if, switch, class, etc.
A setting should be available to enforce all lower, upper and proper case. Ideally this would drive a correction that could be used by VSCode to auto correct.

As Powershell isn't a case sensitive language, it would be good to provide the case formatting options for the developers preference.
Something along the lines of:
//Changes the case for powershell reserved words (if, else, elseif, function, switch, param etc...) to either UPPER, lower or First (capitalises the first letter)
"powershell.codeFormatting.reservedWordCase": "lower", "upper", "first"
//Changes the case for powershell variables declared in the global or script scope to either UPPER, lower or FirstUpper (PascalCase) or firstLower (camelCase)
"powershell.codeFormatting.globalVariableCase": "lower", "upper", "FirstUpper", "firstLower"
//Changes the case for powershell variables declared in the local or private scope (within functions) to either UPPER, lower or FirstUpper (PascalCase) or firstLower (camelCase)
"powershell.codeFormatting.privateVariableCase": "lower", "upper", "FirstUpper", "firstLower"
//Autocorrects the case for recognised cmdlets and functions
"powershell.codeFormatting.cmdletCase": true, false
//Autocorrects the case for recognised parameters of cmdlets and functions
"powershell.codeFormatting.parameterCase": true, false

My 2c: I think "lower" should be the default, and "pascal" (or "proper") should be the only other option for keywords (and only because of the C# programmers who write the documentation). PSScriptAnalyzer is the defacto standard-bearer for PowerShell best practices, and I am reasonably sure the community wants to discourage the use of ALL CAPS or camelCase or even PascalCase keywords in PowerShell. Newbies will discover some annoyed veterans if they start capitalizing keywords such as ForEach...
I also really like the idea of fixing the case on variables, but perhaps that should be a separate issue? I think you'd need to separate not just "global" but "constant" and "parameter" variables -- and you don't have a good way of doing that. I think a good first step would be to have the current capitalization fixer fix the variables based on "follow first use" -- make the case of each variable consistent throughout the file, without worrying about what kind of variable it is.
If we fix variable case, should we consider also enforcing the use of ALL_CAPS for environment variables, for compatibility reasons?


DRAFT - Kubecost - Initial Commit

This PR adds the initial Kubecost lab docs to the Azure Hackfest repo.

This should be a draft, we aren't ready for review/merge yet.

Ready for Review!


Not working in a mountable rails engine

Rails : 4.0.1
engine: mountable
add clearance to Gemfile
and run
rails g clearance:install
/home/kkeys/.rvm/gems/ruby-2.0.0-p247/gems/clearance-1.1.0/lib/generators/clearance/install/install_generator.rb:16:in `inject_clearance_into_application_controller': uninitialized constant Clearance::Generators::InstallGenerator::ApplicationController (NameError)

@derekprior  - When I followed the instructions, I get "Did you mean? _current_user".


Bi 12475 image rendering functionality how the image is displayed and available for download to the user on the web

Adding Show Image Button for customer to download single submission.
Replacing apiClient with privateApiClient

Closing the pull request as I need to fix few typo error.


Problem with sidekia-status middleware after upgrading to rails 6

I upgraded a rails app from 5.2 to 6.1.4
After the upgrade, I got an ArgumentError whenever I run my tests.
The problem appeared to be in the client middleware after checking the logs.
I upgraded my sidekiq gems but nothing have changed.
Here are the logs:
Traceback (most recent call last):
       16: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/rspec-core-3.10.1/lib/rspec/core/hooks.rb:486:in `block in run'
       15: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/rspec-core-3.10.1/lib/rspec/core/example.rb:465:in `block in with_around_example_hooks'
       14: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/rspec-core-3.10.1/lib/rspec/core/example.rb:508:in `block in with_around_and_singleton_context_hooks'
       13: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/rspec-core-3.10.1/lib/rspec/core/example.rb:262:in `block in run'
       12: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/rspec-core-3.10.1/lib/rspec/core/example.rb:262:in `instance_exec'
       11: from /Users/heron/combostrike/backoffice/spec/workers/offer_tool/structure_update_worker_spec.rb:39:in `block (3 levels) in <top (required)>'
       10: from (irb):1:in `block (3 levels) in <top (required)>'
        9: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-6.2.2/lib/sidekiq/worker.rb:200:in `perform_async'
        8: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-6.2.2/lib/sidekiq/worker.rb:240:in `client_push'
        7: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-6.2.2/lib/sidekiq/client.rb:71:in `push'
        6: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-6.2.2/lib/sidekiq/client.rb:217:in `process_single'
        5: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-6.2.2/lib/sidekiq/middleware/chain.rb:143:in `invoke'
        4: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-6.2.2/lib/sidekiq/middleware/chain.rb:140:in `block in invoke'
        3: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-status-2.0.2/lib/sidekiq-status/client_middleware.rb:37:in `call'
        2: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-status-2.0.2/lib/sidekiq-status/client_middleware.rb:37:in `new'
        1: from /Users/heron/.rbenv/versions/2.5.3/lib/ruby/gems/2.5.0/gems/sidekiq-status-2.0.2/lib/sidekiq-status/client_middleware.rb:37:in `initialize'
ArgumentError (wrong number of arguments (given 2, expected 0))

I think #4 is your issue and #5 is the fix for it.

I believe this should be fixed in v2.1.0 via https://github.com/kenaniah/sidekiq-status/pull/9. Sidekiq 6.2.2 included an internal API change that impacted this gem.  Please feel free to re-open if you still are experiencing issues with the updated version.


RuntimeError: "length_2D" must be smaller or equal to image dimensions after resampling.

Hello,
So I get the following error when I am trying to process an image with version 4.0 of the code. This only occurs with the BF and SEM models.
File "/home/msavage/anaconda3/envs/axonDeepSegEnv1/lib/python3.8/site- packages/ivadomed/loader/mri2d_segmentation_dataset.py", line 152, in prepare_indices
  raise RuntimeError('"length_2D" must be smaller or equal to image dimensions after resampling.')
RuntimeError: "length_2D" must be smaller or equal to image dimensions after resampling.
The image in question is 2160 x 1440 pixels at a resolution of 0.0064 micron per pixel.
The system running this package is Ubuntu 20.04
Any help would be greatly appreciated.
Best,
Michael

@mariehbourget I think this might be an issue that we used to solve by tweaking the resampling resolution, but now with ivadomed we don't have that parameter anymore. The issue stems from the fact that we cant create the disired patch size because his image resolutions are much smaller than the ones used by the model, right? Does IVADOMED have a solution to this problem, like we used to have in ADS?

Sorry, I meant that we were able to tweek the resolution as such that it did not match the "actual" resolution. This comes back to the idea that I think we need a "zoom" feature to disconnect the patche size vs resolution issue, as often when we were making the users tweak the resolution we were really just making them change the patch sizes, practically

@GrimmSnark, here is the explanation/solution, so you can use the BF and SEM models on your images.
The TEM model works at a resolution of 0.01 micrometer per pixel, since this is close to your image resolution of 0.0064 microns/pixels, you don't get the error with the TEM model.
However, the BF and SEM models works at a resolution of 0.1 micrometer per pixel.
The error you get is due to the minimum patch size of 512x512 of the BF model and 256x256 of the SEM model after resampling to their resolution of 0.1 micrometer per pixel.
The resampling in ADS goes like this:
dimension of the image (in pixels) * pixel size (in microns/pixels) / resolution of the model (in microns/pixel) and the result must be greater than the patch size (in pixels).
For example, with your image of 2160x1140, pixel_size=0.0064 and model_resolution=0.1, you will have the following result (on the smallest dimension of 1440):
1440*0.0064/0.1 = 92
To apply the SEM model, you need to artificially increase the size of the image with padding (i.e. add empty pixels around the borders) so the smaller dimension is 4000 (or more) to have a result equal of greater than 256.
4000*0.0064/0.1 = 256
On the BF model, you will need to pad to a minimum of 8000 pixels to have a result equal of greater than 512.
8000*0.0064/0.1 = 512
Please let us know if that resolve the issue.
Don't hesitate to contact us if your have questions, we will be happy to help!
@mathieuboudreau, there is a warning in our documentation about this issue but it is in the notebooks section and not very detailed. That seems like something that should be part of the Existing models section, and maybe with the above details to help our users, what do you think?

@GrimmSnark would it be possible for you to share a sample image, so that we could try on our side a solution?

Hello,
Thank you for the very helpful suggestions. I have not had the opportunity to try the padded images but I think that will solve the issue. Regardless, attached is a sample image which causes the aforementioned error. The resolution for this image is resolution of 0.0064 micron per pixel.
LM_58_G.zip
All the best,
Michael

Thanks @GrimmSnark !
@mariehbourget and I played around a bit with some of the ADS settings on your dataset, and here are some reflections//results:

Because of your datasets contrast, we saw better results using the TEM model.
Using the TEM model instead of the SEM model changes our internal resolution during segmentation, so there may be less of a need to do padding to avoid the error you were getting with SEM.
Using the TEM model with your true resolution of 0.0064 microns, we saw better results in segmenting large axons (at the cost of missing small axons):

By using the TEM model but changing the input resolution in ADS to 0.0134 (which essentially, what it will be doing is "zooming" in the patches of the images during the segmentation to your image), we saw better results for small axons (at the cost of worse segmentations on your larger axons):

Applying padding did not provide good results in our tests.

I think the consensus internally in our group is that if we were in your place, it would be better to use a resolution 0.0134 during the segmentation phase only in ADS (note that when creating morphology files, you should use your true resolution value to get accurate values), and then manually correct the segmentation of the large axons using the FSLeyes plugin we provide. Manually fixing/manually segmenting a few large axons is much less time consuming than fixing/manually segmenting many small axons, in our experience.
Hope some of this information is helpful!

Thank you very much for all your advice, the labelling is working much better now!


 He should say something either way. Something that sounds accusatory and stumps you.

This is now implemented but testing may be a bit of a pain to hit all the table entries, especially the >21 failures.
The best way to do so is to define a test macro e.e and so forth and have a debug command to say, this is how many failures.


Query output namespace format

Current hosted SPRK and SPR (possibly changing to SPRV) as TTL yield query results like:
https://raw.githubusercontent.com/mathurma/KGQA/main/resources/SPRK.ttlSwanton
rather than:
http://www.w3.org/1999/02/22-rdf-syntax-ns#type
Does it have to do with the content-type of the URI according to the server (see urlopen request)?

Solved with a simple octothorpe.


Missing curly bracket in connection example for Knowledge store

The example for the connection string when using Knowldege store is missing one curly bracket at the start of "storage-account-name":
"knowledgeStore": {   "storageConnectionString": "ResourceId=/subscriptions/{subscription-ID}/resourceGroups/{resource-group-name}/providers/Microsoft.Storage/storageAccounts/storage-account-name};",

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

ID: 3a54c00f-9e78-d4d9-cc62-1ec2b05e320e
Version Independent ID: a0a19016-ccc1-43f9-7ac7-317f4ebd8b1c
Content: Connect using a managed identity - Azure Cognitive Search
Content Source: articles/search/search-howto-managed-identities-data-sources.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@FreBon Thanks for the feedback! We are currently investigating and will update you shortly.

@FreBron Thanks for the heads up, this is now fixed.  It will be in the published docs by tomorrow morning.
#please-close


Any chance of compatibility with cibuild?

There's any chance of compatibility with cibuild/nightly?

Avalonia 11.0.999-cibuild******-beta from https://nuget.avaloniaui.net/repository/avalonia-all/index.json which is compatible with nativeaot. but I kinda found a way to.

Oh, I'm glad to hear that. It would be great if you could describe the way, maybe someone else will need it, @MrakDev