websocket-client and multiprocessing.

I wanted to ask here just to get some clarification.
I ran into some issues when using the python asyncio based websockets.
I have a program with 2 processes. Each one connects to a websocket. Each websocket needs to be pinged every x seconds to keep the connection open.
The problem is asyncio requires me to await websocket.send("ping") This is where the bottleneck lies, awaiting causes delays in my message processing.
It would be beneficial if I could send these pings in a way that's non blocking such as in a different process, a different thread or some period when nothing else is going on .
Could I use this websocket-client in such a manner?

The problem is asyncio requires me to await websocket.send("ping") This is where the bottleneck lies, awaiting causes delays in my message processing.

Did you benchmark that?
await websocket.send("ping") is very fast; it doesn't even yield to the event loop in regular circumstances (i.e. when send buffers aren't full, a fairly unlikely situation).

a different process, a different thread

"a different task"

The problem is asyncio requires me to await websocket.send("ping") This is where the bottleneck lies, awaiting causes delays in my message processing.

Did you benchmark that?
await websocket.send("ping") is very fast; it doesn't even yield to the event loop in regular circumstances (i.e. when send buffers aren't full, a fairly unlikely situation).

a different process, a different thread

"a different task"

I actually noticed the problem because I need to send a message at least ever 5 seconds. When doing the await call, i noticed that the whole program slows down because when I await send the heartbeat message to the server, the whole asyncio loop is blocked.
I can't run the socket heartbeat in 1 asyncio loop or thread and the receive in another.
Is there a function to check if close() was called on a WebSocketApp instance?

Ah, we're talking about different libraries. You're using websocket-client. Here it's websockets.

a function to check if close() was called on a WebSocketApp instance?

websocket-client does not use await...
Receiving messages is fairly quickly but I need to send a message to the server every x seconds. When i call
await send("keepalive")

it will actually cause the connection to drop, because await the future takes longer than x seconds. I cannot use different asyncio loops to send and receive. So is there any options to use a low priority thread or some other way to avoid having the main loop blocks while sending these keepalive messages?


Push events in the data layer to the presentation layer

What about update events in the data layer and how to notify about them the presentation layer? Let's suppose, that some third party service has updated some user data in the data base.
Is event bus approach good for that?
For example, we will define some event class in the domain layer: UserUpdatedEvent. Some presenter in the presentation can subscribe/unsubscribe for this event on the event bus. Data layer can post the UserUpdatedEvent on event bus ...

+1 on the question.
I would like to add that the current implementation creates a new "UserListSubscriber" for example for each call to the UseCaseLayer.
How would you like to handle common data (let's say that we show the UserList in another screen/widget as well)?
A UserListFetchedEvent could be broadcasted to more than "one" subscriber.

A possibility is to implement an Event Bus or Event Aggregator in the Data Layer. That Bus can be injected via Dagger in a Presenter. That Presenter would then subscribe (and also unsubscribe on destruct) to certain types of Events, which are pushed up from the Data Layer.
Be aware that Event Busses come with their own downsides, as this allows loose coupling between layers and in larger apps can quickly result in event spaghetti. Also, events indicate that something has happened, not that something should happen. So a UserUpdatedEvent means that the Presenter knows a User is updated and make a call to the Repository ( through a UseCase ) to get the new user.

@Trikke, thats where I was going.. so why not make it a standard?
Use UseCases for invoking/triggering actions and subscribe to results/data via bus events?

@Trikke I will absolutely agree with you dude for the spaghetti. Here we have a big event spaghetti that's why in the new version we removed the event bus.

@erikash i wouldn't make it a standard because of my reasons against it. This repository is implemented in such a way now that allows tight coupling between components in layers and the flow of data is also tight and easy to follow from Presentation down to Data and up again.
Event busses are used for a specific set of problems and people are quick to use them to solve any of their problems since they are easy to implement.

So Event busses can be useful in the case of "Data layer lets the Presentation layer know a User was updated in a background service to we can show a toast". But they are not to be used in case of Commands, eg: "User clicks on a button to update a user". As said, events indicate that something has happened, not that something should happen.

@Trikke , I agree with you events should be in the past tense.
You contradicted yourself with the example you provided...
"When a user clicks on a button to update a user" should be handled via a Use Case class.
The response of the updated user should be agnostic to the trigger, whether its a users "click" or a background service or another platform altogether...

Explain why the response of an action should be separated from the action itself?
The way i see it there is a difference between Events and Actions. An Event for me is just something that is thrown up to indicate a state change and notifies listeners, it is usually triggered externally. An Action is something entire different and encapsulates an entire flow of information. In the case of the button click, it is logic to have all components tightly coupled and expect a result back after an Action has happened.
So for a User clicking on a button, you would have :
GUI -> domain -> data -> cloud service -> callback -> data -> domain -> GUI
If you would have a background service that polls you cloud service for updates, then Events makes sense.
Background service -> cloud service -> callback -> push event -> listeners are notified

An action response most of the time would be an "event".
For example, if your product has multiple platforms : web, ios, android.
The used can take most of the actions on any of the platforms, those actions trigger events (state changes) which should be handled in all platforms.
That's why i'm suggesting that all action "events" should be handled the same way...
If you have two handlers for a user update "click" callback from:
GUI -> domain -> data -> cloud service -> callback -> data -> domain -> GUI
and another for a "click" in another platform:
Background service -> cloud service -> callback -> push event -> listeners are notified
Then your risking duplication bugs and other technical difficulties...
Does the things i'm saying make sense?

@erikash i understand what you are trying to say, and in the end it all depends on your implementation and requirements ( and mostly, how you view it ).
But I advocate that there is a difference between Events and Actions; and that a Response is not "an event", but actually just that, a Response. Data that is returned ( be it immediately or later ) because an Action requires it to finish. A Response could hold the same information as an Event, but both are used differently.
I perfectly understand the reasons why you would architect it that way, i'm just concerned that after some time and as the project grows you'll have a hard time to figure out where and when events happened and even who triggered them. ( in my case, the external cloud service triggers it. In your case, it is buttons, external cloud service, anything actually because it is not restrictive and loose )
The architecture in this repository is implemented by Uncle Bob's clean architecture vision, so coupling is different to what you would want to achieve. I do recommend reading up on what Uncle Bob has to say.

You're right.. it all depends on the use case... all I said earlier is that the use case you pointed out is a bad example.
The response callback could be "200", but I suggest that state change handler for UserUpdatedEvent should be the same regardless of  the trigger.
This case is actually very common for applications spanning multiple platforms.. that's why I suggest adding a solution for it in Android-CleanArchitecture.

@erikash You keep commenting that my Use Case is a bad example without ever explicitly explaining why it is so. I feel you are calling it that because it doesn't fit in your architectural plans. In my example there are certainly no duplication bugs or anything. Of course you would never just copy paste code around. Just because you have 2 separate ways to call a Cloud Service to get an updated user, this wouldn't mean that there is duplicate code anywhere. But structuring it like this, with Actions and Events, you are certain that, if something happens, you can more easily debug and find out where the problem came from, be it a button press or an event triggered from a background service. In a completely event-driven situation, this becomes much harder, since your event wouldn't be coupled to any origin.
There is no "solution" required in this repository as you suggested, as the architectural changes you suggest do not fit in the ideologies presented in this repository. I think the author ( @android10 ) can chime in on this.
You are looking to create an "event-driven" architecture, which comes with its own set of problems.

I'm completely agree with @Trikke. Actually, i meant  exactly the same way of the event bus usage. And the example about external service which is scheduled in order to fetch new updates from the server is exactly my case. So, makes sense for me.

@risayew to get back to your original question. From the architectural standpoint presented in this repository, an event bus certainly can be implemented, and most probably in the Data Layer. You could have some kind of background service that would poll your Api from time to time. When an update happens, that background service would have a references to the Event Bus ( or Aggregator ) and post a UserUpdatedEvent to it. Any subscribers on the Event Bus would then be notified. Another discussion is where these subscribers should exist, but that depends on the scope of your application. Usually this will be in the Presentation Layer.
Something of note is that Events indicate that something has happen, but be careful with what data you pass with the Event. Ideally the Event holds minimal immutable data to inform subscribers. In the case of a UserUpdatedEvent, the Event holds the ID of the user that was updated, but not the User Object itself. If you would do this, the subscriber would consume the Event and use that data (eg, show the new User's name in a Toast) but might skip over a lot of Data or Business logic. The subscriber should use the ID passed by the event to get the new User through a Use Case. This way, you pass by your business rules and any Data implementation that might exist. For example, a business rule states that a user with a long name should verify if this name is correct. But if the subscriber in the Presenter consumes an entire User object and just shows the Toast, the business rules that are applied when getting a User through a Use Case are not enforced. ( keep in mind, i'm just making up silly rules here )
I hope i explained the matter clearly.

@Trikke, thank you very much for the explanation. It's great!
In my project i don't pass the data from the data layer to the presentation via the event bus. Only the boolean flag indicating, that information in some table has been updated. Suppose, i have 2 tables: "news" and "infos". My Updater service updates the database periodically. And if, for example, information in the "news" table has been updated, then corresponding event: ContentUpdatedEvent with the flags set to newsUpdated=true, infosUpdated=false will be posted on the event bus. Views, which are interested in this information are subscribed to the event bus. If some view interested in news information, then GetNewsList use case will be activated.
In general, i can say, that i use event bus only in order to pass events from the data layer to the presentation layer. Events, but not the data.

@risayew yup, that's the general idea of it! The Presentation Layer receives the Event and reacts to it by invoking the Domain Layer. Just make sure to keep away from writing code to make an Event happen because you need the logic that is executed as a response to that Event to happen for entirely different reasons than that logic was written.
For example : you have a ContentUpdatedEvent, to which Views react and get the new data through the Domain. Now imaging you make a Button the user can press to immediately try to get new News or Infos. This button should just call trough to the Domain to get the information required. You should not write code that would trigger your Updater, to trigger the ContentUpdatedEvent to get the Views to update. This might seem logical in a way because you are making the system fire Events and reacting to them. But in this instance, you'll end up spending lots of time debugging when your Updater fails and you have no idea where or why it was triggered. (Did the user click that button, did he click it twice, did he click it and close the app, did the update in the background thread fail). Now imagine a much more complex system like this and you have Event Spaghetti.

Nice discussion here! Just my 2 cents here: As @Trikke pointed out, do not confuse Events with Actions. Of course with an EventBus we are totally decoupled, however, I think they work the same way as  GOTO instruction (https://en.wikipedia.org/wiki/Goto), so careful must be taken since we can loose control of our flow and this will bring as a consequence, hard debugging, and for instance difficulties when solving bugs.

@Trikke What if the Domain Layer will receive the evant (ContentUpdatedEvent) and reacts to it by invoking the Use Case (notify Presentation Layer to show progress if needed, than do some business logic, get refreshed data) then update refreshed data in Presentation Layer. In this flow we don't skip business logic and Presentation don't know about push event. It is a good idea or not?

@justplay1 This discussion is months old, so i'd advice going through it entirely to read up on the point. Because what you seem to be suggesting feels wrong. This discussion was about the Data Layer having a background service that would push events up.
A Use Case can't notify the Presentation Layer to show progress, it has no reference to it. So if you would have an event listener in your Domain Layer, the only thing you can do is run business logic. From that layer you cannot access the Presentation Layer, so you cannot inform your user of any updates, or update data in the UI. A listener in the Domain layer would only make sense if it is important to your business logic.
For example : A background service pushes an event to tell us that the app configuration was updated. Domain listens to this, gets the updated configuration and runs the required business logic that could be involved. The user has no need (and can not) to know about this.
But even then, i think there are better ways to accomplish this then with "event-spagetti".

@Trikke , thank you for the explanation


Fix inverse scroll triggering next function on scroll down

Fixes issue #223
Please review & merge ASAP since it's a major bug :pray:

Could we merge this?

Reviewing this today and will release a new version


pull request for e2e test at 01/10/2017 11:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


play, pause, end and complete events

How can I track shaka player's play, pause, end and complete events in javascript.
How can I save bookmark [means already played video history] so that when user comes back again shaka start playing where he/she left earlier.

Hi, you can listen to events directly from the video element:
video.addEventListener('playing', onPlaying);
video.addEventListener('pause', onPause);
video.addEventListener('ended', onEnded);

To save playback position you can save video.currentTime in your application.
The next time you start playback you can pass that time to load(), e.g.,
var player = new shaka.Player(video);
player.load(manifestUri, lastPlaybackPosition);


How to modify http request headers？

Is it possible to  modify the http request headers?

Hi,
For most headers, you can use the OkHttp APIs to modify the request headers. There's a few headers with special treatment, such as accept-encoding and user-agent, if you're seeing an inconsistent behavior between plain OkHttp and this library please provide more details.

With code like below okhttpclient communicates via HTTP/2 even though the google site supports quic.

import android.content.ClipData
import android.content.ClipboardManager
import android.content.Context
import android.os.Bundle
import android.widget.Toast
import androidx.appcompat.app.AppCompatActivity
import com.google.net.cronet.okhttptransport.CronetInterceptor
import okhttp3.*
import org.chromium.net.CronetEngine
import java.io.IOException

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        val engine = CronetEngine.Builder(applicationContext).build()
        val callFactory = OkHttpClient().newBuilder()
            .addInterceptor(CronetInterceptor.newBuilder(engine).build())
            .build()
        val request=Request.Builder()
            .url("https://www.google.com/")
            .header("User-Agent","Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:98.0) Gecko/20100101 Firefox/98.0")
            .header("Accept-Encoding","gzip, deflate, br")
            .header("Accept","text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,*/*;q=0.8")
            .build()
        callFactory.newCall(request).enqueue(object : Callback {
            override fun onFailure(call: Call, e: IOException) {
            }

            override fun onResponse(call: Call, response: Response) {
                runOnUiThread{
                    var contentEncoding=response.headers.get("Content-Encoding")
                    if(contentEncoding.isNullOrEmpty()){
                        contentEncoding="identity"
                    }
                    Toast.makeText(applicationContext,response.protocol.toString()+"\n"+contentEncoding+"\nEncoding:"+response.toString(),
                        Toast.LENGTH_SHORT).show()
                    copyToClipboard(applicationContext,"",response.body!!.string())
                }
            }
        })
    }
    fun copyToClipboard(context: Context, label: String?, text: String?) {
        val clipboardManager: ClipboardManager =
            context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
                ?: return
        clipboardManager.setPrimaryClip(ClipData.newPlainText(label, text))
    }
}

Ah, right, there's a few things that may come into play here:

In order to use QUIC you need to enable it on your Cronet Engine.
You should provide QUIC hints to let Cronet know that a particular host supports QUIC.
If you don't we'll pick that information up from alt-svc headers and will use quic the next time you connect to the host. However, note that due to connection pooling the reconnect might not happen immediately so subsequent request to the same host might still use h2.
Either way, we also suggest that you enable on-disk cache to persist "these servers support QUIC" signals.

Unrelated to QUIC support - you shouldn't set Accept-Encoding manually. Cronet sets these automatically depending on you Cronet engine configuration (e.g. enableBrotli) and decodes the server response transparently.

Enabling QUIC and setting QUIC Hints solved this problem.
I also want to use QUIC on non-Android OS such as Linux. Can this library be used on Linux and Windows?

Caveat to that: The only officially supported platform is Android.

On Linux and Windows, it seems difficult to use this library. So I will use this only on android.
Thank you for your advice.


train config for retrieval

Hi, amazing work.
When you have the time, could you release the config file for finetuning on MSRVTT with Xretrieval.py Thanks.

Same request


Lost-HDD ?

(1) I am not very techsavvy.
(2) Some time ago, I noticed my raspiblitz was not active on the internet. IIRC, it said "logs full"
I tried to reboot, but then got a "lost-HDD" message, which is what I now see on boot.
I've tried to reflash and replace the SSD, but that's not the issue (I think).
(3) So, do I have any software fixes for this, or is the HDD somehow bricked?
(4) Recent logs, when I tried to reformat the HDD:
> Linux raspberrypi 5.15.32-v8+ #1538 SMP PREEMPT Thu Mar 31 19:40:39 BST 2022 aarch64

The programs included with the Debian GNU/Linux system are free software;
the exact distribution terms for each program are described in the
individual files in /usr/share/doc/*/copyright.

Debian GNU/Linux comes with ABSOLUTELY NO WARRANTY, to the extent
permitted by applicable law.
Last login: Sat Sep  3 13:33:57 2022 from <IP_ADDRESS>
Starting SSH user interface ... (please wait)
# start ssh menu loop
# starting ssh menu loop ...

># controlSetupDialog.sh
mkdir: cannot create directory '/var/cache/raspiblitz/temp': File exists
# Starting basic setup dialog ...

>Running Format: (ext4) ()
># DATA DRIVE - FORMATTING to EXT4 layout (old)
># missing valid third parameter as the device (like 'sda')
># run 'status' to see candidate devices
>FAIL ON FORMATTING THE DRIVE:
>missing parameter
>Please report as issue on the raspiblitz github.
># menu loop received exit code 1 --> exit to terminal

***********************************
* RaspiBlitz Commandline
* Here be dragons .. have fun :)
***********************************
Your setup is not finished.
For setup logs: cat raspiblitz.log
or call the command 'debug' to see bigger report.
Blitz command line options: blitzhelp
Back to menus use command: raspiblitz

admin@<IP_ADDRESS>:~ ₿ Connection to <IP_ADDRESS> closed by remote host.

Now it says
>RaspiBlitz 1.8.0 / noHDD / <IP_ADDRESS>

                                                       │ Waiting for HDD/SSD ...         │
                                                       │ Please connect a >=1TB          │
                                                       │ HDD or SSD to the device.

So do I need to get a new SSD? Or is there a way to check/fix it when it's "not detected"?

Try again with a fresh flashed sd card - if also second try fails your HDD/SSD seems not working in your setup.

Thanks for the reply @rootzoll , but I've indeed swapped for newly flashed cards 3x. No luck :(

Hi, I've the same issue. I wanted to install version 1.9 and since then I had this problem

Same problem here. For me it appeared during updating from 1.9 to 1.10
HDD just worked fine. SD flashed and got this lost HDD error.

OK so the problem is that hddCandidate is empty .. can be seen here that the second brackets are empty
>Running Format: (ext4) ()
Improved the HDD/SSD detection .. try if the following RaspberryPi v1.11.0RC1 image (Download under Artifacts) fixes this:
https://github.com/raspiblitz/raspiblitz/actions/runs/7201966440

Thanks for the suggestion but I gave up long ago and threw the whole rig away.

I do have the same issue with lost hdd message and

Running Format: (ext4) ()

..while the SSD is connected to an USB 3.0 port of my Raspi 4.
I am struggling to find the download of the suggested RaspberryPi v1.11.0RC1 under artifacts. I would willingly try the image.
Please advice @rootzoll. Thank you

After login at github the download link appeared... 👍 let you know the result after flashing the 1.11.0RC1

Still does not work for me with my Crucial BX500 2.5 SSD 1 TB while it is connected at an USB 3.0 port. When I use other Unix based OS it does work with this disc on those ports. I go ahead with the minimal image on an USB 2.0 port and might try later with next release.

Giving again some feedback on this issue :

Node lost power
After reboot got into "emergency mode"
Get into "emergency mode" after every reboot from now on
Re-flashed SD card
Now getting the "lost HDD"-issue


Tuple 2i test

Test logic looks good to me.
In a lot of cases I ask people to switch to using prepared statements instead of .format, but this is quite readable as is, probably more readable than with prepared statements, because you don't have to refer back to a bunch of prepared statements when reading one of the loops. The downside is that there are multiple sites to change if the schema under test changes. So, I don't have strong feelings, but that's what went through my head when I reviewed this.
Other than my little nits, +1, looks good!


Any Update for windows 10 with Unity 2018 ?

as title said . have you ever update it for the newest system and unity?
it seems that it can not work fine in that environment

I just ran his sample scene in 2018.2.13f1, works fine. What version are you using?


fix: update grid gutters to match design

https://openedx.atlassian.net/browse/PAR-442
From 30px gutters to 24px

Coverage remained the same at 89.75% when pulling 3134449f78a16044c1a01972b363118874712db8 on abutterworth/grid-gutters into df31d7bc394046c5ca5bf3f23bf3b2958a0f581a on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enketo submissions don't work for forms with space in formid that are in projects with project-level encryption

+4da6d37bb30dd44716ef68f279669672d279189a server (v1.1.1-138-g4da6d37)

In a project with project-level encryption, create a form with a formid with spaces (e.g. Geotrace and geoshape
Try to make an Enketo submission (e.g. here)
See Submission service on data server not found. (404)

I'm pretty sure this is related to the spaces in formid ~and that it's a regression~. Another interesting thing about this form is that it has a blank version but I've tried another form with a blank version without problem.
I get the same behavior with a published form on v1.1.1 but on a draft on v1.1.1 I get a different error message "Unexpected form id value Geotrace_and_shape; did not match the form ID in the URL"

Likely root cause of https://forum.getodk.org/t/odk-central-change-to-encrypted-form-once-already-started-data-collection-breaks-submissions/33082

i wrote a test that does all of the above except use enketo, and it passes. so it must be something enketo is doing. but i can't debug it without access to that server to see the enketo logs.
    it.only('should not fail given encryption and spaces in the name', testService((service, { all }) =>
      service.login('alice', (asAlice) =>
        asAlice.post('/v1/projects/1/key')
          .send({ passphrase: 'test' })
          .expect(200)
          .then(() => asAlice.post('/v1/projects/1/forms?publish=true')
            .set('Content-Type', 'application/xml')
            .send(testData.forms.simple.replace('id="simple"', 'id="sim ple"'))
            .expect(200))
            .then(() => asAlice.get('/v1/projects/1/forms/sim%20ple')
              .expect(200)
              .then(({ body }) => asAlice.post('/v1/projects/1/submission')
                .set('X-OpenRosa-Version', '1.0')
                .attach('xml_submission_file', Buffer.from(testData.instances.simple.one
                  .replace('id="simple"', `id="sim ple" version="${body.version}"`)), { filename: 'data.xml' })
                .expect(201))))));

my best guess enketo is overzealously encoding spaces somewhere?

@lognaturel does this line mean anything to you?
https://github.com/enketo/enketo-express/blob/3f353ea44934045e185db71e26ec331146ec2610/public/js/src/module/controller-webform.js#L268

and in particular why it differs from line 261 above which references form.instanceID. i would bet the difference here is the problem.
because the instanceID is taken as xml attribute text, i would not expect to see it url-escaped ever.

Line doesn't mean anything to me but I agree it looks like a very likely culprit. It's definitely something Enketo-specific. I can't see anything obviously related in issues for either enketo-express or enketo-core.
I looked at logs and all that's in there since the last restart is ERROR: Submissions element(s) not supported. They're not verbose logs.
From nginx: "POST /v1/projects/275/submission HTTP/1.1" 404 183 "https://test.getodk.cloud/-/mNI13cGzUkxcIfvQOgheKuWvocUjauz"
Nothing useful in the redis DB I don't think:
<IP_ADDRESS>:6379> hgetall id:mNI13cGzUkxcIfvQOgheKuWvocUjauz
 1) "openRosaServer"
 2) "https://test.getodk.cloud/v1/projects/275"
 3) "openRosaId"
 4) "Geotrace and shape"
 5) "submissions"
 6) "0"
 7) "launchDate"
 8) "2021-04-26T18:10:37.413Z"
 9) "active"
10) "true"
11) "theme"
12) ""
13) "lastAccessed"
14) "2021-04-26T20:59:06.759Z"

You can ssh<EMAIL_ADDRESS>if you want more.

i logged on but.. couldn't find where anything is. the thing i would like to see is enketo trying to talk to central, which i don't see in here.

There's no logging to that effect other than at the nginx layer. So I can't think of a way to figure out what formID Enketo is trying to use without doing it locally.

Closing in favor of enketo/enketo-express#252 since there's no action to take on the Central end.


I am not able to configure the CSS properly.

I am not able to configure the CSS properly, I am getting an error as if I have not configure the tags in the module, can you update the read me with the proper configurations?

Would you minding pasting the error message here? Thanks


Invoice image corresponding to sample.json file

Hey there,
Thanks for open sourcing this!
I have a question regarding the data. Is it possible for your to provide the image of the invoice corresponding to the sample.json file please?
It would be very helpful to understand how the information on the image corresponds to the information inside the .json file.
Thanks in advance!

I do actually! and it's one of my own so I can share it. Here you go
faktura-1.pdf


Tracking Outputs vs Detection Outputs

While using ByteTrack as a tracker, the visualized bounding boxes sometimes do not precisely match the detection results. Are there any methods to assign tracking IDs to detections without altering the bounding box coordinates? I am asking this question because the tracking bounding boxes are sometimes too wide for my specific case.

are sloved this problem


update alert.html

Opening this issue will trigger GitHub Actions to refresh all notebooks.

I don't understand this issue


Snowflake integration tests

Describe your changes

Add Snowflake configuration & fixtures into integration testing
Make test table name with random hash string during every testing session (Reason below)
Modify all test cases in integration testing support dynamic table name

The reason we need the dynamic table name is now we use one single snowflake connection endpoint to perform the test.
To prevent the race condition or any data consistency issue during the testing, each test session should have standalone environment: that's why we logically add the hashing to each test table.
Example table name: taxi -> taxi-abc123 (6 chars uuid)
Example case: Two people may push the commit to github at same time, those intergration testing might conflict to each other, since they are sharing the same database, same table name on snowflake

By the way, now the developer needs to add snowflake credential in the local environment variable:
(Please ask me $$$)
conda env config vars set SF_USERNAME=$$$
conda env config vars set SF_PASSWORD=$$$

Issue number
Closes https://github.com/ploomber/jupysql/issues/247
Checklist before requesting a review

[ ] Performed a self-review of my code
[ ] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added documentation in the docstring and changelog (when needed)

:books: Documentation preview :books:: https://jupysql--336.org.readthedocs.build/en/336/

@edublancas Would you check again why docs/readthedocs.org:jupysql  fail again?

doc building has been fixed in master, please rebase

Yeah...I am fixing those

@idomic Ready for review

I don't see any docs for this in the developer guide: https://jupysql.ploomber.io/en/latest/community/developer-guide.html

what happens if I want to run them locally? Looks like I'd  need to set some environment variables

I don't see any docs for this in the developer guide: https://jupysql.ploomber.io/en/latest/community/developer-guide.html

Good point. We should mention how to test this, system environment variables has to be set up in order to perform snowflake testing on local


Add a CONTRIBUTING.md to the Jackson repo

Guidelines 👇🏼
If you would like to be rewarded by BoxyHQ for your participation in contributing to Hacktoberfest, please register for our Hacktoberfest event.

Description: The project should include a CONTRIBUTING.md file to provide guidelines for contributions and make it easier for users to understand how to contribute effectively.
You can reference the CONTRIBUTING.md guide from the SaaS Starter Kit but the setup steps are not the same.

Assign to me

Hi @NathanTarbert , kindly assign this to me.
If there is any other useful information you can provide, I will appreciate it.

Can you please assign it to me

I am interested in taking this up, if it is still available

Hi @ukrocks007 l, Can you please assign this work to me?


Typo pg_audit should be pgaudit

There is a typo.
Current;
Search for the pg_audit parameters.
Should be:
Search for the pgaudit parameters.
[Enter feedback here]

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

ID: 4e1c3072-ed87-965f-0b05-098cb2c83d41
Version Independent ID: de27de2d-3867-218d-909d-d4100b35f176
Content: Audit logging - Azure Database for PostgreSQL - Flexible Server
Content Source: articles/postgresql/flexible-server/concepts-audit.md
Service: postgresql
Sub-service: flexible-server
GitHub Login: @GennadNY
Microsoft Alias: gennadyk

@haveyoumetcp Thank you for your feedback. We will review this and make an update if appropriate.

@haveyoumetcp I have updated the doc page to change pg_audit to pgaudit. Changes should be live soon when this will be approved by the correct team.
Thank you for raising this into our attention. https://github.com/MicrosoftDocs/azure-docs-pr/pull/250154


A non-numeric value encountered

I did a fresh install and I am having an issue login into the admin panel, it gives me this error when I try to log in.
app / Http / Middleware/ SetLocale.php: 25
error on this line -> return $next($request);
A non-numeric value encountered
Please help, Thank you.

Do you have a full stack trace?

Do you have a full stack trace?

Does this help?

No. Can you share the stack trace on Flare (share button at the top)?

No. Can you share the stack trace on Flare (share button at the top)?

Sorry, here is the link
https://flareapp.io/share/87noqjM5#F68

You seem to have an invalid setting for post_max_size or upload_max_filesize. Can you please check what are the values for them?

post_max_size

I got it fixed, Thank you


Autorest generated dec sig

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-autorest

Show changes
@azure-tools/typespec-autorest - internal ✏

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


create text file with the original and simulation times for each radar file

This could be written outright and made available as a text file in a format something like this:
KLOT20240716_003414  --  2024/08/26 22:04:14
KLOT20240716_003947  --  2024/08/26 22:09:47
KLOT20240716_004521  --  2024/08/26 22:15:21
etc..
A basic example is here ... https://rssic.nws.noaa.gov/assets/file_times.txt

File times text file is now created for each instance but it's unclear whether it will be overwritten when multiple radars are in the simulation. Will need to check this.


tracker: add a warn log for deadline exceeded query

What is changed and how it works?

Issue Number: ref #15566
What's Changed:

Add a warning log when a query is timeout. This can help make some issue investigation easier.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
None

@bufferflies @overvenus @tonyxuqqi  PTAL

/hold

/unhold

/merge

LGTM

/unhold


Paying with wallet that has zero tft or less amount of tft, should return meaningful message

Branches
libs, threebot: unstable_reservation_payments
other repos: unstable
Environment
Devnet
Scenario
1- create wallet with zero tft
2- create reservation
3- user should pay for this reservation
Actual Result
JSX> res = zos.billing.payout_farmers(wallet2, registered_reservation)
Mon 06 17:56:03 sal/zosv2/billing.py -  35 - payout_farmers                     : EXCEPTION:
    AttributeError("'Billing' object has no attribute '_log_error'",)
--TRACEBACK------------------
<stdin> in <module>
    1
/sandbox/code/github/threefoldtech/jumpscaleX_libs/JumpscaleLibs/sal/zosv2/billing.py in payout_farmers
    35    self._log_error(e)
-----------------------------

'Billing' object has no attribute '_log_error'
JSX> res = zos.billing.payout_farmers(wallet2, registered_reservation)
         res                  ResourceWarning

Expected Result
Paying with wallet that has zero tft or less amount of tft than required registered reservation amount , should  return message implying that  user's wallet need to be filled up before proceeding

@john-kheir can you pull and try again? :-)

working fine now


Add command line tool to generate JWT for use by bots

Edit: Creating a command line tool to generate JWTs should be enough for now.
Previous title:

Idea: Add support for multiple authentication schemes at the same time

If we have bots creating annotations to mark certain mappings, we possibly need other ways to authenticate besides JWT/login-server. Ideally, that bot just uses a persistent token to authenticate. I was also thinking that we could possibly allow users to create/remove their own auth tokens for applications like that, although that's something we probably won't need for a while.
Internally, jskos-server could keep a table mapping auth tokens to user objects.

I don't fully see the need for many multiple authentication schemes but auth tokens make sense for the bot account. But we don't need a users to create auth tokens, in lack of a use case. These tokens can be managed via command line as well. The bot is running locally on the same machine as jskos-server anyway.

Okay, my suggestion: Add a command line script to create a valid JWT that can be used for authentication. That way, we don't need to change anything in the code because JWTs are already fully supported.
If we need more than that, we can add it later.

This could support a command line client to access JSKOS API with commands such as
jskos-api voc                                           # list all vocabularies
jskos-api data uri=http://dewey.info/class/612.112/e23/ # get item
jskos-api add $ITEM                                     # entity type guessed from `type`
jskos-api edit $ITEM                                    # entity URI in `uri`
jskos-api delete $ITEM
...

The API endpoint and JWT token are read from a config file ./jskos-api.json:
{
  "api": "http://bartoc.org/api/",
  "token": "...."
}

This could support a command line client to access JSKOS API with commands such as
jskos-api voc                                           # list all vocabularies
jskos-api data uri=http://dewey.info/class/612.112/e23/ # get item
jskos-api add $ITEM                                     # entity type guessed from `type`
jskos-api edit $ITEM                                    # entity URI in `uri`
jskos-api delete $ITEM
...

The API endpoint and JWT token are read from a config file ./jskos-api.json:
{
  "api": "http://bartoc.org/api/",
  "token": "...."
}


Alert: TargetDown in metallb-system

Alert TargetDown firing in metallb-system namespace
This is an automated issue created by the monitoring system. Please do not edit this message.
Alertmanager URL: https://alertmanager.ankhmorpork.thaum.xyz
Issue was last updated at 2023-03-07 14:38:19.451754361 +0000 UTC m=+1851.287716182.
Common Labels

alertname
TargetDown

cluster
ankhmorpork

job
metallb-system/metallb

namespace
metallb-system

prometheus
monitoring/k8s

severity
warning

Common Annotations

description
25% of the metallb-system/metallb/ targets in metallb-system namespace are down.

runbook_url
https://runbooks.prometheus-operator.dev/runbooks/general/targetdown

summary
One or more targets are unreachable.

Alerts

StartsAt
Links

2023-03-07 14:22:48.612 +0000 UTC
GeneratorURL

(DO NOT MODIFY: d8c63b9c2dca8d23d05cb93eb129b404aea4bd0740a981f8162a64c4615dd023 )

Spam. Increased threshold to 50% with https://github.com/thaum-xyz/ankhmorpork/commit/e16ee72ceb072d58917b542abef1d1a2e8d82fdc


PDF file download problem

Describe the bug
I'll report this as an Angular UI issue to start the discussion, but aspects of the REST API are also involved.
Currently, when the user choses to download a PDF file (and probably other file formats that can be rendered inline in the browser) the  bitstream download page is opened followed by a hard redirect to the REST API content endpoint. In the case of a PDF, the REST API response Content-Disposition header usually indicates an inline resource and the PDF file is rendered in the browser window.
There are 2 problems with this.

Two clicks in the browser navbar are required to return to the DSpace Item page; the first back click results in the PDF file being rendered again a second time.
If the bitstream has access restrictions, the first click instead results in a WhiteLabel page indicating a 401 error.

One possible solution for #1 is to open the bitstream download page in a new browser tab. This can be done easily a themed version of file-section-component.html by adding isBlank=true to  attributes of theds-download-link element. We might want to consider making this change in the default component.
Unfortunately, if bitstream access is restricted, opening the download page in a new tab also results in #2 (the 401 error ) so more work would be required to make this solution work.
There may be other, better ways to solve the navigation problem.
To Reproduce
Steps to reproduce the behavior:

Click to download a PDF file in the simple Item view
Once open, use the browser back button to navigate back to the item
Try the same thing again on a bitstream with restricted access and you should see a 401 error.

Expected behavior
The download page serves a useful purpose so ideally everything would remain as is with the exception of fixing the 2-click return problem for inline files. Alternately, opening the download page in a new tab an acceptable compromise once the authorization error is fixed. There may be other solutions.
Related work
Link to any related tickets or PRs here.

Re: the "usually" above:  inline vs. attachment is affected by the value of webui.content_disposition_threshold.  Testing with a sufficiently large bitstream will result in different browser behavior.

This is related to #1004, which is the original creation of the Bitstream download page.
@artlowel : Pinging you as I know your team built the bitstream download page in #1004, so I wonder if you'd have thoughts on how to improve the usability per this ticket.
In the meantime, I'll also pull this over to our 7.4 board, as I feel this might be nice to improve if there's a quick solution available. If not, we may need to consider moving it to 7.5

Quick note: my description of the "2-click" issue is based on the Chrome browser; the behavior might be a bit different in Firefox and other browsers.
fyi, I'm currently following @mwoodiupui's suggestion and setting the backend's webui.content_disposition_threshold to zero, which forces the attachment header in all cases.  This seems to work well enough if a solution doesn't find its way into in 7.4. (Honestly, downloading as an attachment is an easier sell to users at this university, especially because 95% of the PDF files here are access restricted and the authorization error is a bit of a blemish when exposed to users. From what I've seen so far, browsers make it pretty easy to open and view  the downloaded file.)

Closing, this is no longer reproducible in 7.5.  I believe it was fixed by the fix for #1809


[Question] Cuda Graphs support?

A handy tool provided in Torch that I think would make a great addition to the bindings is the CUDAGraph support.
You can see the header file here.
What are your thoughts on this?

Thanks for the suggestion, here is a small attempt to expose this api, #632. Would you mind trying it out to see if it would work for your use cases?

Ah actually just adding some example at the moment, it seems that this will require being able to specify some cuda stream to work, so never mind for now I'll add support for specifying the cuda stream and will come back when the basic example runs.

Amazing. Think these bits are kind of crucial to expose for those really highly performance based use cases so your work is appreciated. Drop another message here and I'll try to apply the proposed API into our use case.

You should be able to try out #632 right now if you manually install libtorch (I think it's using 1.13 and not 2.0 on this branch though).
Re merging the branch, this is a bit more tricky than what I thought because of https://github.com/pytorch/pytorch/issues/47743 , the issue being that the header files in the libtorch cuda binary packages are not self contained so if there is no local cuda install, compiling this branch will not work. I'm not really sure how to get around this at the moment - vendoring the cuda headers doesn't sound great, maybe this could be put under a feature flag for now and be only available for users that have a proper libtorch install but that doesn't sound great neither.

Apologies for the long absence. Priorities changed a lot and only now can I return to trying out tch again. I tried out candle where you're also a contributor so thank you for that too!
I think with respect to this, the API example you have looks very nice to work with. I think since this was opened a lot of things about the way tch is installed including the automated version detection and installation/downloading was added. I think it makes the idea of vendoring these headers more pallet-able?


Add async support to invoker

Signed-off-by: Bora M. Alper<EMAIL_ADDRESS>

Main changes:

invoker now invokes each workflow serially (i.e. one by one) rather than in parallel (i.e. round-robin). This is because:

the TimeseriesDB does not support multiple different workflows running at the same time.
the previous solution was unclear to begin with... At each "tick", it invoked a different URL, why?

Truncate the output files before writing them to prevent garbled results.
Do not wait for the first tick! Previously, with default parameters rps=1 time=5s, our real RPS was never greater than 0.80 because we were waiting for the first tick for our first call. Now, for the same default parameters, we make 5 calls (instead of 4) and can reach 100% RPS!


Make plugin compatible with macos or restrict it just to linux distro

Hello,
I ran into a problem with SED when executing the sdk-refresh-completion-files command on MacOs (10.14.5).
This is the following error I'm getting:
➜ ~ sdk-refresh-completion-files sed: illegal option -- r usage: sed script [-Ealn] [-i extension] [file ...]

From what I know sed is a bit different for Mac because of the BSD difference.
Could you change the usage of sed to grep or any other text tool that would be compatible with OSX and Linux ?

Hello,
I don't have a lot of time right now to take a look at this.
However I'll try as soon as I can (Maybe this week-end). If you come across a fix, don't hesitate to fork and PR !
Thanks for the heads-up.

HI.
I have released a new version you can use to correct the sed -r error.
However there might be other problems with sed expressions, so i am not sure if this is completely corrected.
Tell me if this fixes you problem. Don't hesitate to give me some more feedback on this issue and I'll try to correct them along the way.
Thanks in advance.

Has your problem been fixed ?

Hello, did my update fix your problem ?


Prevent a panic when a query simultaneously finishes and is killed at the same time

There is a strange race condition where a query can be killed and finish
at approximately the same time. If this happens, the query gets
retrieved by the killing task, the query gets closed by the normal
processing thread, and then the killing task attempts to kill the query
afterwards. Since the close doesn't mark the query as already killed
(since it's not killed, just merely unused), the killing thread attempts
to close the channel again.
Mark the query as killed whenever it is closed to prevent a double close
from happening. This should never cause the status to be erroneously
reported since the query status is removed from the query table within
the same lock scope.
Backport of #9220.

[x] Rebased/mergable
[x] Tests pass
[x] CHANGELOG.md updated

Is 1.4.3 merge this feature?


Bosh deploy with an error

Hi I tried to run Bosh deploy after bosh create release & bosh upload release. I have verified the release to be accurate with bosh verify and bosh releases.
The problem comes after, when I attempt to bosh deploy, the following error returned. I have no idea what went wrong with the deployment file if it was supposed to be generated and merged with spiff.
[root@vmstack-01 cf-release]# bosh deploy

Processing deployment manifest
------------------------------
Getting deployment properties from director...
Compiling deployment manifest...
Please review all changes carefully

Deploying
---------
Deployment name: `cf-deployment.yml'
Director name: `microbosh-centos'
Are you sure you want to deploy? (type 'yes' to continue): yes

Director task 8
  Started unknown
  Started unknown > Binding deployment. Done (00:00:00)

Error 40001: Required property `range' was not specified in object ({"cloud_properties"=>nil, "static"=>["<IP_ADDRESS> - <IP_ADDRESS>"]})

Task 8 error

For a more detailed error report, run: bosh task 8 --debug

Please advise.

Looks like this is mistake in your deployment manifest. Take a look at your "networks" section, network with type "manual" (or without type – network without type is a manual by default) – there's a missing required "range" argument in one of subnets. See http://bosh.io/docs/networks.html#manual for details.

@uzzz Right, I fixed that. It is working now, but more errors popping up.


Fix workflows binding to create a workflow without arguments

Fixes WOR-317.
With this PR, 2 bugs were fixed:

Fix workflows binding to create a workflow without arguments. If no argument were passed, the wrangler local dev would fail. Now we validate that the arguments exist, and if not, manage the creation of the necessary variables.
Fix instance.id is now available synchronously, as in production

Co-authored-by: Sidhartha Chatterjee<EMAIL_ADDRESS>

Tests

[ ] TODO (before merge)
[ ] Tests included
[x] Tests not necessary because: workflows-shared is managed by the workflows team

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[ ] I don't know
[ ] Required
[x] Not required because: workflows does not have them

Public documentation

[ ] TODO (before merge)
[ ] Cloudflare docs PR(s):
[x] Documentation not necessary because: no modifications to the feature usage

I'm a little confused on what the second change around the id is for - could you clarify? 🙂

I am sorry for the noise. I identified a miss match between wrangler local dev and production, regarding the way you could access instance.id.
This commits were me trying to debug, until I learned how to do local builds with fixtures.
This last commit address this issue.

@penalosa can you take a look at this? This PR is really improve for our customers, as currently wrangler local dev is not working properly.

@bruxodasilva could you add a regression test to the workflows fixture test (fixtures/workflow)?

Looks good. Thank you!


Current highlighted line on Terminal not visible

First of all, I love this theme; so hope this can be sorted!
You can see here that the current line doesn't show when using Nord.

If I switch to a default theme such as Solarized it displays fine:

In addition, I noticed the issue in Neovim at first when cycling through search selections. In that case, when pressing n to move to the next match, the selected match would disappear too:

Is this is known issue? Is there any setting I can change in iTerm to fix (running 3.3.4)?

Hi @benfrain :wave:, thanks for your contribution :+1:
Are you running nvim within a multiplexer like tmux? These setups require some additional configurations in order to make colors work correctly.
I'll try to reproduce the problem with the current line, but I've also never saw that the current line has a special highlighting. I'll need to check if this is maybe a feature added in later versions.

Hi! 👋
No tmux. Just Neovim direct from iterm.

After further investigation, I'm not sure this is to do with Nord iterm2.
I noticed that I don't have this issue using iterm2 on my work computer. So I copied the term config json and applied it to my problematic setup and it now works as expected. There is something in the iterm setup that causes this issue.
Here are the working and non-working iterm json configs:
Working: https://gist.github.com/benfrain/11c7b4c6e7cde22348f6f5e6aca64861
Non-working: https://gist.github.com/benfrain/c18fd9bba2dbd56cdbb4c5ef5d25e614
Hopefully that may help reveal the cause of this issue.

First of all, good that the problem is solved :smile:
Thanks for the feedback and the Gists, I've compared both configurations and found 2 possible options that might cause the problem where all of them are included in the non-working set:
Cursor Guide
:link: Line 478
     "Use Bright Bold": true,
+    "Use Cursor Guide": true,
     "Use Italic Font": true,

I guess this option has the highest probability to be the root cause: The Cursor Guide option

The cursor guide is a horizontal rule that indicates the vertical position of the cursor. You can adjust its color, including alpha value, to make it more visible against your background color.

This is exactly what's visible on your screenshots and I guess it causes problems due to its rendering style using a color value with a alpha layer for opacity.
Also see the documentation section about Cursor Finery that can be enabled when using the block cursor style.
Underline Color
:link: Line 482
+    "Use Underline Color": false,
     "Vertical Spacing": 1,

It is possible that the Underline Color option also causes problems in combination with the Cursor Guide option when rendering underlined elements.

If enabled, this color will be used for all underlining, independent of the color that underlined characters have themselves.

:arrow_right: You could try to play around with both options to see if this causes the problem again, but I'll also try to reproduce it to maybe include it in the docs/FAQ of this theme.
I close this isseu for now since the problem is resolved, but feel free to post your findings :wink:


Fix fd_write with offset

https://github.com/wasmerio/wasmer-js/issues/175#issuecomment-572494101:

I'm not 100% sure if this is correct in the general case, but this change fixes the problem (both in the reproduction and my project)

See issue for complete description and example, closes #175

@mischnic Thank you very much for the PR! It is super appreciated! 😄 Don't know why it didn't occur to me I could ask you to open a PR, but I am super glad you did!
I re-ran the tests because they timedout, and once they pass I will merge this 😄 Thanks again!

@mischnic Got the tests to re-run, so now it is in master 😄
I'll go ahead and release this next week, as we will be releasing something else then 👍


Master is currently failing due to files not meeting the Spring Java Formatting

Describe the bug
Currently multiple files, across multiple modules are not formatted correctly resulting in the current CI Build failing, and not able to run a successful build locally
The affected files are,
affected_unformatted_files.txt
Sample
N/A

affected_unformatted_files.txt
Forgot to pull master, updated with correct files that are affected.


adds color for CumulusCloud

signed CLA

Thanks for the contribution @lyqh-ctx! I pushed some small fixes in https://github.com/CesiumGS/cesium/pull/9877/commits/ab777b3351f630c45dd7aae9949a2f61be4edfe6.


Refactor casts

According to issue #295.

Do you think that you have fixed all c-style casts? I just would like to know if #295 can be closed by this pull request.

I will give it another look to be sure, I think i missed some (const char*) casts, and I might take a look at compiling it with gcc option  -Wold-style-cast.

Good idea with with -Wold-style-cast. It is probably the best way to detect all such casts.

Found two more, will fix them soon.

According to Travis build there is another one:
[ 76%] Building CXX object CMakeFiles/KVEngine.dir/sources/representation/View2.cpp.o

/home/travis/build/griefly/griefly/sources/representation/View2.cpp: In member function ‘bool View2::FramesetState::IsTransp(int, int, int, int)’:

/home/travis/build/griefly/griefly/sources/representation/View2.cpp:44:38: warning: use of old-style cast [-Wold-style-cast]

     if (shift >= (int)GetMetadata()->dirs)

                                      ^

I'll get on that.

That should be it.


docstring(Algebra/DirectSum/Basic): correct docstring

The docstring for docs#DirectSum indicated DirectSum β instead of DirectSum ι β.

bors r+


Unregister entity modal goes full screen when width < 960 px

Expected Behavior

The unregister entity popup is a modal that pops above content in all situations when attempting to perform the unregister function. It centers in the middle of the screen and lets the user click it.
Current Behavior

I had my screen moderately sized, and clicked the unregstier entity by using the vertical dots link at far top right of an entity, and discovered below 960px the modal goes full screen:

Possible Solution

I think going full screen is good, but maybe 960px width is too large for this flex to trigger?
Steps to Reproduce

Make your Backstage window less than full screen
Open an entity
Click the two dots at top right to trigger an Unregsiter entity, then click unregister
See a modal which is centered in middle of screen.

Context

This may have been entirely intentional, controlled globally, or something else, but it just surprised me and given the small size of the content in the modal, just seems very large to be breaking to flex at that size; open to counter opinions on it.
Your Environment

NodeJS Version (v12): 12
Operating System and Version (e.g. Ubuntu 14.04):
Browser Information: Microsoft Edge (Chrome 8x)

This behavior does not seem to be what Material-ui recommends as I don't see any breakpoint on https://material-ui.com/components/dialogs/
So my recommendation would be to remove the behavior to be consistant with material-ui. what do you think?

#4729 missed the "closes" keyword in the ref to this issue... ;)

indeed


Project doesn't see slides. Help needed.

This is my datasets.json file
{
 "dataset2": {
    "slides": "D:/Belgeler/SlideFlow/slides",
    "roi": "dataset/slides2.svs/rois",
    "tiles": "D:/Belgeler/SlideFlow/tiles",
    "tfrecords": "D:/Belgeler/SlideFlow/tfrecords"
  }
}

and this is settings.json file
{
 "name": "MyProject",
 "annotations": "D:/Belgeler/SlideFlow/annotation.csv",
 "dataset_config": "./datasets.json",
 "sources": [
  "dataset2"
 ],
 "models_dir": "./models",
 "eval_dir": "./eval",
 "slideflow_version": "2.3.1"
}

When I load the project on studio no slides are detected. I can't extract tiles either so I'm stuck.
I'm on windows 10 and also slides in  /SlideFlow/slides directory are in .svs format. I'm using vips version 8.15.2

I'm coming across the same issue.
I'm able to extract tiles from the TCGA_LUNG dataset, but not when I apply the code to my own dataset.

I apologize it took so long to address this issue. If you are still encountering the problem, I can assist with figuring out the root cause.
As a disclaimer, Windows is not officially supported as our team has not had capacity to extend testing to this platform. So there may be unexpected issues. With that said, I have run Slideflow on Windows previously so it should, in theory, be able to work.
First thing to check is the annotations file - can you share the first couple of lines from this file here, as well as the directory layout where your slides are located?


Ibm s390x arch

We are running ZLinux using the S390X chipset architecture. The only JVM that runs on this architecture is the official IBM SDK compiled specifically for the S390X chipset.
The change I made checks is the architecture starts with s390, and if so, sets the IBM SDK as the default.

Hi @zamedic, thanks for the contribution! I have no way to test this (no mainframes laying around) but it looks simple enough.
Can you fix the foodcritic issue? Also please revert the version number change, that happens as part of the release process.

@erichelgeson Thank you. I have made the changes as requested.
The new Food Critic errors, are those related to anything I have done?

I rebased & squashed your commits and they pass. I see you merged master in at some point, which maybe is what caused the issue. v1.44.0 is on it's way to be published via travis in a few min.
810da6cc1a45ef3fb4d5c7f1bd2275fd6d7492b7

I've also setup a cron job in travis if no builds have happened in 24 hours to build to catch these issues when chef-dk updates (not wait for someone to open a PR and have unexpected issues :)


Should we use PlugIn or Plugin

I'm currently not sure whether to use PlugIn or Plugin. In the past I've preferred using PlugIn.

"Plugin" has become a standalone word. It consists of two words, but it is used as one word today. I would prefer "Plugin". Wordpress, mozilla and many others call it Plugin.

I agree.


haskell-packages: use older version of apply-refact with ghc-8.0.x

Version 0.4 of apply-refact doesn't compile with GHC 8.0.x. This patch
updates configurations-ghc-8.0.x to use apply-refact version <IP_ADDRESS>.
Fix #26895
Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Please do not edit hackage-packages.nix. That is an auto-generated file and all changes you make here will be lost. Note the comment at the very top that says so.

@peti One could modify the script generating the auto-generated file to include a comment # this is an auto-generated file by <name of script> do not modify on every line.  If we would standardize on the specific string being used, we could even automatically reject PRs containing such modifications.
(Clearly, it wasn't clear enough, because the universe just created a person who didn't follow your scheme.)

I should have noticed the comment at the top of hackage-packages.nix.
Now, I am wondering if this PR makes sense or not. @peti, I have just read one of your reply on another thread saying that it doesn't make sense to put the 60,000 versions of hackage packages into Nixpkgs. You recommended to go with stack build to save performances of nix-env and I understand your point of you. Moreover, apply-refact==<IP_ADDRESS> only works with GHC 8.0.x. Older and newer versions of GHC are not supported. It means, I have to add two lines in configuration-hackage2nix.yaml if we want the support to be full.
Is it worth for Nixpkgs community to apply these changes. What is your opinion, should we mark the PR as abandoned?

The needs of the Nix community are very diverse and it is very hard to predict what exactly other users will or won't find useful. Personally, I believe that we should support one compiler version at the time -- which is currently GHC 8.0.2 -- and that all packages in haskellPackages (which uses that compiler) should build. So, if you make a PR that modifies configuration-hackage2nix.yaml so that apply-refact == 0.3.* becomes the default version, that sounds like a good idea to me. Now, you can also add overrides for other compilers that switch their respective package set to a different version if you want to, but my impression is that those package sets aren't used much and therefore the effort might be wasted in the sense that nobody will ever try to install that package built with, say, ghc 7.10.x. Support for GHC 8.2.x might be more interesting, because at the time LTS 9.x comes out, that is going to become our new default compiler.

Personally, I believe that we should support one compiler version at the time -- which is currently GHC 8.0.2 -- and that all packages in haskellPackages (which uses that compiler) should build.

OK, I will do so for GHC 8.0.2. Just two questions:

Where is the soft that takes the configuration-hackage2nix.yaml and produces the hackage-packages.nix? I just want to be sure that I am applying the correct modification to configuration-hackage2nix.yaml.
Do you know when haskellPackages will use GHC 8.2.x?

Updates are run by a cron job that executes the following script: https://github.com/NixOS/cabal2nix/blob/master/update-nixpkgs.sh.
Nixpkgs will switch to GHC 8.2.x when LTS 9.x comes out. Nobody knows when that will be. Experience suggests that the switch happens some 3-4 months after the compiler itself has been released.

Thanks for the answer. I updated the configuration-hackage2nix.yaml and tested everything works fine by executing update-nixpkgs.sh and testing #26895 with the updated hackage-packages.nix. Let me know if you want me to add/amend something.

I'll merge this PR later today. There is an other change I'd like to see completed first.


question about to use this code with Windows Forms

Hi,
Is is not an issue just a question.
You did a huge work to convert to Xamarin Android several useful apps from pure Android.
Is it possible (and how to do that) to use this code (and others you did) with Xamarin Forms and Xaml ?
Thanks

You can always create a native renderer, but in general this is a StackOverflow question and doesn't belong here.

Sorry to post my question here but I wanted also to know if you have these codes with Windows.Forms and if I had not missed anything. Now I know that you have not.


Silently ignore cargo::rustc-check-cfg to avoid MSRV annoyance when stabilizing -Zcheck-cfg

This PR, removes the warning when trying to use cargo::rustc-check-cfg on stable or nightly (without the nightly-only -Zcheck-cfg flag) to avoid MSRV annoyance when stabilizing -Zcheck-cfg.
See this Zulip thread for more information and context.
cc @ehuss

:umbrella: The latest upstream changes (presumably #13409) made this pull request unmergeable. Please resolve the merge conflicts.

@bors r+

(Oh, I think homu is down. Will probably need to kick this once it is fixed.)

@bors r+

:pushpin: Commit da5900740e9b59ce4521a5083ace80dc4eaf8ca1 has been approved by ehuss
It is now in the queue for this repository.

:hourglass: Testing commit da5900740e9b59ce4521a5083ace80dc4eaf8ca1 with merge a3ba4ee28a3823b4ab31d3477332174f94a9c95f...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing a3ba4ee28a3823b4ab31d3477332174f94a9c95f to master...


Added moving car

*Fixed #654
Description
It is a moving car animation effect using html, css, javascript.
Fixes:  #654

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:

[x] I have taken help from some online resourses and mentioned in PR
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Video
https://user-images.githubusercontent.com/85051124/168085526-46823301-65e5-4891-9657-48820d3d563f.mp4

@DEEPALI-SUK  This pr was not merged in projectKart so I have made the pr here.


⚠ Jenkins Plugins Artifactory has degraded performance

In 9d7b16a, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1299 ms

Resolved: Jenkins Plugins Artifactory performance has improved in f8bb694 after 19 minutes.


ZK LeaderInitiator - Implement SmartLifecycle

Depends on #13

Removed extra synchronization. Rebased and merged as 137a8b2.


Question about getting usage from stream response

Hello!
I have a question about and would appreciate any help.
According to Anthropic's docs the message delta is supposed to have a usage with the message_delta event.
event: message_delta data: {"type": "message_delta", "delta": {"stop_reason": "end_turn", "stop_sequence":null, "usage":{"output_tokens": 15}}}
When I  look the usage (on .messageDelta type) it appears to always be nil.
Any ideas why that would happen?

Hi @hyperlink. Thanks for the feedback.

When I look the usage (on .messageDelta type) it appears to always be nil.

I have checked your point, and the Anthropic documentation explains that the response to a message_delta event has the following type.
event: message_delta
data: {"type": "message_delta", "delta": {"stop_reason": "end_turn", "stop_sequence":null, "usage":{"output_tokens": 15}}}

However, when I checked the actual response, I found the following
event: message_delta
data: {"type": "message_delta", "delta": {"stop_reason": "end_turn", "stop_sequence":null}, "usage":{"output_tokens": 15}}

The difference between the two is whether usage is under delta or not. Perhaps the documentation is wrong, but we are reserving our response at this time to contact Anthropic.

I reported the issue to Anthropic's support. I have it fixed locally and I'll try to create a PR.

Fixed by #8


Use github arm64 runners

github now offer arm64 runners.
The arm64 image is a plain ubuntu 22.04 server with nothing installed (no docker).
Fixes  #7026

Thanks for the PR @seanyoung. Were you able to test the docker changes?

I haven't tested it, I was hoping CI would that for me.

I am trying to test that at least the docker install works, but it's stuck waiting, not sure if that's expected...it did trigger with the correct runner, but maybe it's stuck because it's on my fork siladu/besu/actions/runs/8978675974/job/24659493468

Only github repos/organizations signed up to the private beta will have them available, so it won't work on your fork.
Is there a way we can test this in hyperledger/besu?

Thanks for the PR @seanyoung. Were you able to test the docker changes?

I haven't tested it, I was hoping CI would do that for me.

I am trying to test that at least the docker install works, but it's stuck waiting, not sure if that's expected...it did trigger with the correct runner, but maybe it's stuck because it's on my fork siladu/besu/actions/runs/8978675974/job/24659493468

Only github repos/organizations signed up to the private beta will have them available, so it won't work on your fork.
Is there a way we can test this in hyperledger/besu?

@seanyoung Short of creating your own personal runner for this PR to point to, I can't think of one. Will merge and revert if it doesn't work. Not a big deal if we miss some intermediate commits on the develop tag.

@seanyoung It did indeed fail: https://github.com/hyperledger/besu/actions/runs/9026355992/job/24803532413

PR to revert: https://github.com/hyperledger/besu/pull/7087


[5.4] PhpRedis method compatibility

This PR covers most method signature differences. https://github.com/laravel/framework/pull/17196 should be merged as well.

Why not just using @method docblock signatures in the class?


Improved speed for semantic locators

Optimization for searching via aria-labelledby attribute (searching for elements only with id attribute).
Optimization for locating field by label (searching for elements only with for attribute)
Replaces #1941

The fix doesn't solve the performance issues for me.


TypeError [ERR_INVALID_PROTOCOL]: Protocol "http:" not supported. Expected "https:"

Error Message
env

Node 14
testing react application using msw, msw/node.

messages
    TypeError [ERR_INVALID_PROTOCOL]: Protocol "http:" not supported. Expected "https:"

      at Object.proxiedOriginalRequest (node_modules/node-request-interceptor/lib/http/override.js:55:36)
      at ClientRequestOverride.<anonymous> (node_modules/node-request-interceptor/lib/http/ClientRequest/ClientRequestOverride.js:244:39)
      at step (node_modules/node-request-interceptor/lib/http/ClientRequest/ClientRequestOverride.js:33:23)
      at Object.next (node_modules/node-request-interceptor/lib/http/ClientRequest/ClientRequestOverride.js:14:53)
      at fulfilled (node_modules/node-request-interceptor/lib/http/ClientRequest/ClientRequestOverride.js:5:58)

Why this happen?
This happens when mock https request.
This error occurs because there is no cert property in options.
so options.protocol always set as http:
https://github.com/mswjs/node-request-interceptor/blob/master/src/utils/getUrlByRequestOptions.ts#L22-L23
    console.log(options); // No 'cert' property in `options` object.

    // Assume HTTPS if using an SSL certificate.
    options.protocol = options.cert ? 'https:' : DEFAULT_PROTOCOL

Attachment
console.log(options) result

using fetch('https://...') => log: https-log.txt
using fetch('http://...') => log: http-log.txt

Possible Solution
I think, options.protocol value should be decided by options.uri.protocol === 'https:'
Example, Reproduce
https://github.com/gwanduke/example-node-request-interceptor-issue-45
App.js
...
    fetch('https://example.com')
...

App.test.js
setupServer(
  rest.get('https://example.com', (req, res, ctx) => {
    return res(
      ctx.json({})
    );
  })
);

test('render', async () => {
    render(<App />);
});

i had mistake

Hey, @gwanduke 👋 Could you please mention what was the root cause of this for other people to follow? Thanks.

I forget to use listen when using msw library
after write this code, it works well.
beforeAll(() => {
  server.listen();
});

(https://mswjs.io/docs/api/setup-server/listen)
I must have been mistaken while debugging yesterday.
Thanks.


401 Client Error

I have surfaced the issue when creating calendar event, 401 Client Error: Unauthorized for url:https://yourdomain.com/EWS/Exchange.asmx
Has anyone experienced anything like this?!

Same issue here ...


Add Ryan to zenodo file

@ryla5068  will you please add yourself to our zenodo.json file as a contributor.
https://github.com/earthlab/matplotcheck/blob/master/.zenodo.json
do you have an orcid id for that? it's ideal if you do. you will then be a part of the citation for this tool.

this was merged! closing.


How do you explicitly fail a test?

I would love to see something like
fail()

I found here that you can just do a
XCTFail (format…)

but that's not every Expecta-like.

I've taken a stab at this, opinions? https://github.com/specta/expecta/issues/96

OK, this is in a new release has been pushed, give it a test run please.

failureWithError(NSError *) would be nice!


[Request] Add those Games to the TitleDB.

Sega Master System Injects (Forward to Picodrive).zip
Super Nintendo Injects (Forward to Snes9x for 3ds).zip
I Tested All those Games on my 3ds & they all Work.

Emulator injections are officially considered piracy as the emulator they're injected into is copyright to Nintendo, even when the ROM itself being used is public domain (which I don't suspect any of these are anyway.)
Basically, if there's an injected game on the site and it's brought to my attention, I'm obligated to remove them. I'm certainly not about to manually add a bunch of extra ones, as I'd get sued and the service itself would likely have to be shut down.


Could not load file or assembly Microsoft.Extensions.CommandLineUtils

I have installed the latest version of Power Query SDK.
I have checked existing resources, including the common issues and the release notes.
I have searched for similar issues.
Power Query SDK
0.2.3
Regression From
No response
Platform
Windows
Architecture
x64
OS Version
Windows 10
VSCode version
1.78.2
PQSdkTool Path
C:$HOME.vscode\extensions\powerquery.vscode-powerquery-sdk-0.2.3-win32-x64.nuget\Microsoft.PowerQuery.SdkTools.2.114.4\tools\
Bug Description
PQTest.Console.Program.Main
Various processes throw the error that it cannot find the dll. Clean install of laptop, Visual Code and dotnet SDKs
Full error
[Error]	[Task exited abnormally] pqtest info pid(20668) exit(3762504530) stderr:
Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.Extensions.CommandLineUtils, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. The system cannot find the file specified.
at PQTest.Console.Program.Main(String[] args)
Installed sdks and runtimes
6.0.408 [C:\Program Files\dotnet\sdk]
7.0.203 [C:\Program Files\dotnet\sdk]
7.0.302 [C:\Program Files\dotnet\sdk]
Microsoft.AspNetCore.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.0-preview.4.23259.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Steps to Reproduce
Install extension and create a new project.

This works in 0.2.2
Missing DLLs in the
.vscode\extensions\powerquery.vscode-powerquery-sdk-0.2.2-win32-x64.nuget\Microsoft.PowerQuery.SdkTools.2.114.4\tools
folder in 0.2.3

This works in 0.2.2
Missing DLLs in the
.vscode\extensions\powerquery.vscode-powerquery-sdk-0.2.2-win32-x64.nuget\Microsoft.PowerQuery.SdkTools.2.114.4\tools
folder in 0.2.3

Was this fixed?  Still seeing this issue in 0.2.3.  We had to downgrade to 0.2.2, run the ask install, then re-upgrade to 0.2.3.


Align gateway request properties with Mutiny HTTP client and Defaults

What this PR does:
Align gateway request properties with Mutiny HTTP client and Defaults
Which issue(s) this PR fixes:
Fixes Jira C2-3550, Jira C2-3538
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Merge it first and will solve the tests later


how to add custom field for response json outside data

Hi all.,
i have problem to add custom field like:
cols: [], data: []
how to add cols outside data using transformer?
thanks

Cant younput them in meta key?
Sent from my iPhone

On 27 Mar 2016, at 15:59, Ardani Rohman<EMAIL_ADDRESS>wrote:
Hi all.,
i have problem to add custom field like:
cols: [],
data: []
how to add cols outside data using transformer?
thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

hi @catalinux .
i cant because i want create structure like this
{
  "cols" : {
    "userId": {
      "index": 1,
      "type": "number",
      "unique": true,
    },
    "name": {
      "index": 2,
      "type": "string"
    }
  },
  "rows": [
    {
      "userId": 1,
      "name": "Batman"
    },
    {
      "userId": 2,
      "name": "Superman"
    }
  ]
}

any suggest for me?

If this is the "main data" of your end pointl you should put it in data key.
Depending on your semantic, it should be a collection in your data or a item. Your data looks to me to be more like an item.
Sent from my iPhone

On 27 Mar 2016, at 17:59, Ardani Rohman<EMAIL_ADDRESS>wrote:
hi @catalinux .
i cant because i want create structure like this
{
"cols" : {
"userId": {
"index": 1,
"type": "number",
"unique": true,
},
"name": {
"index": 2,
"type": "string"
}
},
"rows": [
{
"userId": 1,
"name": "Batman"
},
{
"userId": 2,
"name": "Superman"
}
]
}
any suggest for me?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

I don't think any of the Fractal serializers offer the ability to merge data with the top level out of the box? Probably something you need to look at implementing yourself on the Fractal side of things.


Need EAN 8 Support

Hi, thank you for Great work!
It is possible to add a EAN 8 format support?
Thanks a lot.

Going to take a look at it soon

Great, thank you!

Now added EAN8 support to the master branch. Going to make some tests before compiling it as a release but you can do it yourself if you want to use it now.

Thanks a lot.
Will check tomorrow.
ср, 20 янв. 2016, 15:49, Johan Lindell<EMAIL_ADDRESS>
Now added EAN8 support to the master branch. Going to make some tests
before compiling it as a release but you can do it yourself if you want to
use it now.
—
Reply to this email directly or view it on GitHub
https://github.com/lindell/JsBarcode/issues/39#issuecomment-173209924.

The EAN8 support is now in the latest release.
Also added it to the generator

Awesome!
We will use this feature here : http://provectapos.com

Nice :smile: always nice knowing where the code will be used :+1:


Called save and restore with the correct context

To match the other plugin calls, I believe these methods should also use the chart node as the context.

Thanks for the PR!  Looks good!


267 possible to share like wishlist

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #267
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Please, elaborate the commit message. For example, when I am deleting a file we must mention the reason why we are doing it.

We need not multiple controller for a specific model. For example, all the related to subscription may reside in SubscriptionController.  There may have may methods like getSubscription, createSubscription, updateSubscription, deleteSubscription so on and on.
Most of the project nowadays follows MVC pattern. Model, View, Controller, Database related task may go to the model classes.

Please reread the previous reviews. We must try to strictly follow SOLID principles.


Allow .svg file to be in the repositories

Allow .svg file to be in the repositories.

@olenagerasimova , please review

@ChGen thanks, is it ok that all checks failed?

@olenagerasimova , I don't remember this check statuses before. I remember it get deployed only after merge to master.

@olenagerasimova , I removed some folders with incorrect permissions and check passed.

@ChGen great, merging