unable to connect

hello, I have growatt local running now some time for 2 invertors. after a update from raspbarry pi 3 to 4 i can not connect any new invertors any more. 1 is running and 1 is failing. when i try to reconnect it again i get "unknow error occurred".
When i remove the one that is working and reinstal everything i get this error on both of them. The only way to get 1 working back is to do a restore of a backup.
I there a way to change the modbus adres without reinstaling the invertor

Hello,
There is currently no option to change this from the front end of HA. This would require a Options Flow.
What you can do is go to the folder of home assistant and open .storage/core.config_entries
Search for growatt_local and update the address value in the configuration.
Are both inverters connected on a single serial port or do you have two serial ports connected to your raspberry pi?
Did you also check the home assistant logs?

Hello Wouter,
THANX!!!!. this did solve my problem.  Now it is connected again.
I checked my home assistant logs. thats way i wanted to change the port adres. Why i can not connect any new ( removed) invertors any more i dont know. i only get the "unknown error occurred"
I have the 2 inverters on saparete modbus to usb dongels. This was already the best option for the rasp.3. I changed to the rasp.4 because the port numbers kept changing after every restart. so i had to reinstal the invertors everytime i did a restart. ( i have 3 modbus connections. and it going to be more). with the rasp.4 it is not a problem any more.
Many thanx for the nice addon!


Create guide for GKE Engine operator

It is not necessary. Let's do it later.

https://issues.apache.org/jira/browse/AIRFLOW-6290?jql=project %3D AIRFLOW AND component %3D documentation


createIndexesTest is unstable

I've seen the test failing in separate random occasions and had to re-start CI to make it pass.
The failure is:
FAIL  packages/graphback-runtime-mongodb/tests/createIndexesTest.ts
  ● MongoDB indexing › can create relation indexes

    MongoError: ns does not exist: test.comment

      at MessageStream.messageHandler (node_modules/mongodb/lib/cmap/connection.js:266:20)
      at processIncomingData (node_modules/mongodb/lib/cmap/message_stream.js:144:12)
      at MessageStream._write (node_modules/mongodb/lib/cmap/message_stream.js:42:5)

See https://app.circleci.com/pipelines/github/aerogear/graphback/7359/workflows/259093cd-6792-4e23-bb39-b7632159eb02/jobs/7506 for example

/cc @craicoverflow, @wtrocki
Automatically generated comment to notify maintainers


SOCRATES: automatically use atmospheric bulk properties

At the moment, we have to set atmospheric properties such as gravity through the rad_ccf file in Socrates. As these can change with time in the coupled model, we should either load these through rad_ccf (possibly like the method in Cl_run_cdf) or edit the model so that they're in a namelist.

Initial conditions homogenised throughout PROTEUS.


To Date in Right Showing the Wrong Date

Hi please see my code--showing a date ten days from now
https://codesandbox.io/s/13om50k2p4

That is odd, in the meantime as a workaround, you can add the mix and max values in your state to the XAxis. Which fixes the issue.
<XAxis min={min} max={max}>

P.s. As an aside, I think you are computing the initial now data incorrectly,
instead of
state = {
  now: (Math.round(Date.now()) * 1000)
}

I think you mean
state = {
  now: Math.floor(Date.now() / 1000)
}

absolutely on point.  thanks so much.


Android - Chat - Compose box is moving up when tap on it

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch the app
Log in with expensifail account
Select any chat
Tap on compose box

Expected Result:
Compose box is not moving up when tap on it
Actual Result:
Compose box is moving up when tap on it
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/180885917-f4214c85-77ff-4892-bf49-c8d4cffc9c10.mp4
https://user-images.githubusercontent.com/93399543/180887204-600ebbde-f68f-42f7-aed9-b32358643fa0.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Seems possibly related to the KeyboardAvoidingView stuff. @jasperhuangg ?

I am having a hard time testing that theory because reverting this PR comes with conflicts

Thanks for the updates @marcaaron. I think @jasperhuangg comes back from ooo tomorrow and will be able to assist us in reverting this?

This seems to have been fixed by this revert. Closing.

ah nice was just about to do this 🎉


Open Service Broker recommended, but is no longer maintained

Under the section Open Service Broker for Azure, when clicking the first link, you're taken to this page.
The first line of the readme is:

This project is no longer being maintained by Microsoft. Please refer to Azure/azure-service-operator for an alternate implementation.

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

ID: 702c42c6-2f4c-6f58-6f7e-09d5d1545705
Version Independent ID: 346b5b91-d9c0-4b13-0897-e8155f476fae
Content: Connect to Azure Kubernetes Service - Azure Database for PostgreSQL - Single Server
Content Source: articles/postgresql/concepts-aks.md
Service: postgresql
GitHub Login: @rachel-msft
Microsoft Alias: raagyema

@keyz182 Thank you for the feedback. We are actively investigating and will get back to you soon.

@rachel-msft any tips?

Yes, I'm reviewing

Thanks @keyz182 for flagging this. I've checked in a documentation update to remove this. It will be published tomorrow.

#please-close


Add module with technical indicators

These are my initial ideas, these are mostly ponderings about the purpose & design (not so much on the code)

Accept numpy arrays as argument. Accepting & unpacking pandas types with a decorator can indeed also be nice, and we can then also accept native lists there too? Internally  would make it based on numpy arrays. Most algorithms don't need/use pandas indices and columns names in algorithms and so we shouldn't pass them / expect them to be passed. I personally prefer data in numpy objects when making models, and I wouldn't like to be forces to convert them to pandas before being able to compute indicators.

Merge it in with samcoml-transform. We have 10+ indicator there already, we also have a design there like generic window functions, and support for transform that have different number of output columns than input columns. Maybe that design is not fully optimal yet but then it can be polished.

because our submodules don't have much coupling, we can think of it as a standalone module (and do so for all our submodule ideas). There are many indicator libraries on pypi, why not use those? What would set it apart? The samcoml-transform came about because it has some functions that are both useful and unique like moving_quantile, moving_slope, and multiasset ideas. When Andrejs was implementing Boruta he mentioned that there was already a mature implementation available, and he had a point IMO.

If we add it, then I would suggest we commit to the goal of making it mature enough to cover all common indicators and most use cases? Also add indicators that are not available in popular libraries (and that we use ourselves) as a justification for not using something that's already available (or alternatively donate those to a popular indicator library and become a co-author there). We should also make the interface user-friendly for potential uses by adding numpy support.
If such a commitment is too much work & deviation from the actual goal (a project on top of Factset data) then we can also decide to using available 3rd party indicators for that project, or move a limited number or new indicators into an optional submodule (or separate main module?) and not fully replicates alternatives?


Added additional audio filter

Added echo audio filter
Added lavadsp-extended dependencies source: https://github.com/rohank05/lavadsp-extended
Edited IMPLEMENTATION.md to add echo example

Maybe you could create your own lavalink plugin using the lavalink plugin-api with your custom filters? That way the lavalink doesn't need an extra dependency which may need regular updates (if you plan to add more filters idk), and if someone doesn't want more filters, just don't use your plugin.

Ahh i didn't realise


[stable/kong] replicaset service account error running with db-less config

Describe the bug
So, i'm trying to install kong in my cluster, but when i set database=off in my values.yaml, ReplicaSet doesn't start the pods because apparently it needs a serviceAccount, which is configured to be created automatically,
Version of Helm and Kubernetes:
Helm Version:
Client: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}

Kubernetes Version: 1.13.6-gke.0
Which chart: kong-0.11.2
What happened:
Events:
  Type     Reason        Age               From                   Message
  ----     ------        ----              ----                   -------
  Warning  FailedCreate  2s (x11 over 7s)  replicaset-controller  Error creating: pods "kong-kong-747b884764-" is forbidden: error looking up service account api-gateway/kong-kong: serviceaccount "kong-kong" not found

What you expected to happen: Kong starts using DB-less config
How to reproduce it (as minimally and precisely as possible):
helm upgrade --install kong stable/kong --set env.database=off --set postgresql.enabled=false

Anything else we need to know:
I'm not installing Kong as IngressController in this case, i just want it to be an application inside my cluster.

I am experiencing the same issue. I am also trying to use dbless configuration. I had to manually create the ServiceAccount that the helm chart was expecting to get a successful deployment. I am using minikube.
Helm Chart:
xkong                   1               Thu Aug 15 16:49:50 2019        DEPLOYED        kong-0.11.2             1.1             default
Helm version:
Client: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"} Server: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
minikube:
minikube version: v1.3.1 commit: ca60a424ce69a4d79f502650199ca2b52f29e631
Should the Helm Chart be creating the ServiceAccount?


Date format doesn't change when we change language

Environment Details
Spartacus: Release 1.0 + hotfix
Browser: Safari
Device: mac
Happens on: https://dev-com-19.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD/
Steps to Reproduce
Login
Place an order
Go to MyAccount -> Order History
Change the language to DE
Actual Result
The date format doesn't change.
It's still July 3, 2019.
It should be something like 3 Juli 2019 (used to be like that before)

Still reproducible on dev2, dev17 & dev19.

Was able to reproduce the bug on 17 & 19 but this was working on my local. Maybe someone can test again on 17 & 19 once it gets updated.

It works on my local as well.

I don't see a fix yet.
Switching language from English to German still leaves date in the same format.
Frontend dev-com6. Backend: dev-com19.

I know it works locally. But can we please investigate why it doesn't work on the server?
Maybe it's a config issue?
But we need to solve this, because otherwise, the e2e test checking the date format when we change the language will fail.

I was able to repro!

Basically this is cause by the fact that we only provide one local in the Spartacus install script. If this is required to be fixed on our servers I can add the german local.

PR is 3 in the Spartacus-installation repo.

Approved.  Works locally.  We just need to redeploy on the dev servers for the fix to be applied.

QA Steps:

Tested on dev2. Works now!


EventChannel missing and duplicating messages.

Steps to reproduce
I am attempting to listen to platform events via an EventChannel and found that Flutter was dropping some messages or duplicating others. I think this is because the fast (ish) message rate of 1000/s - although each message is small. This issue doesn't occur in my Windows platform implementation, only on Android.
I'm not super familiar with Kotlin so this may be my error on the native side though.
Thanks for your help 🙂
Expected results
I would expect my event channel listener (eventChannel.recieveBroadcastStream().listen(print)) to have all the events from the platform side, in order, with no duplicates. Some delay would be expected though
Actual results
I get the following (or similar) log output:
System.out is the native counter.
Notice this is missing events 1, 5 - 13, 19 - 24  etc on the flutter side, and events 15, 16, 17, etc are duplicated.
I/System.out( 5708): 1
.
.
.
I/System.out( 5708): 143
I/System.out( 5708): 144
I/System.out( 5708): 145
I/flutter ( 5708): 2
I/System.out( 5708): 146
I/System.out( 5708): 147
I/System.out( 5708): 148
I/flutter ( 5708): 3
I/System.out( 5708): 149
I/flutter ( 5708): 4
I/System.out( 5708): 150
I/flutter ( 5708): 15
I/flutter ( 5708): 15
I/System.out( 5708): 151
I/flutter ( 5708): 16
I/flutter ( 5708): 16
I/System.out( 5708): 152
I/flutter ( 5708): 16
I/flutter ( 5708): 16
I/System.out( 5708): 153
I/flutter ( 5708): 16
I/System.out( 5708): 154
I/flutter ( 5708): 17
I/flutter ( 5708): 17
I/System.out( 5708): 155
I/flutter ( 5708): 18
I/System.out( 5708): 156
I/flutter ( 5708): 18
I/flutter ( 5708): 18
I/flutter ( 5708): 25
I/flutter ( 5708): 26
I/flutter ( 5708): 26
I/flutter ( 5708): 27
I/System.out( 5708): 157
I/flutter ( 5708): 27
I/System.out( 5708): 158
I/flutter ( 5708): 27
I/flutter ( 5708): 28
I/System.out( 5708): 159
I/flutter ( 5708): 28
I/flutter ( 5708): 29
I/System.out( 5708): 160
I/flutter ( 5708): 29
I/System.out( 5708): 161
I/flutter ( 5708): 37
I/System.out( 5708): 162
I/flutter ( 5708): 37
I/flutter ( 5708): 38
I/System.out( 5708): 163
I/flutter ( 5708): 38
I/flutter ( 5708): 38
I/System.out( 5708): 164
I/flutter ( 5708): 38
I/flutter ( 5708): 38
I/System.out( 5708): 165
.
.
.

Code sample
Native plugin:
class TestAndroidPlugin : FlutterPlugin, MethodChannel.MethodCallHandler {
   private var testEventSink: EventSink? = null

    override fun onAttachedToEngine(flutterPluginBinding: FlutterPlugin.FlutterPluginBinding) {
        channel = MethodChannel(flutterPluginBinding.binaryMessenger, methodChannel)
        channel.setMethodCallHandler(this)

        EventChannel(
            flutterPluginBinding.binaryMessenger,
            testEventChannel
        ).setStreamHandler(
            object : EventChannel.StreamHandler {
                override fun onListen(arguments: Any?, events: EventSink?) {
                    testEventSink = events
                }

                override fun onCancel(arguments: Any?) {
                    testEventSink = null
                }
            }
        )
    }

// Started by a MethodChannel call - not shown for brevity.
private fun startCounter() {
        thread {
            val mainLoopHandler = Handler(Looper.getMainLooper())
            var i = 0
            while (true) {
                i++
                mainLoopHandler.post {
                    testEventSink!!.success(i)
                }
                println(i)
                Thread.sleep(1)
            }
        }

}

Flutter-side listener
    _testStreamEventChannel.receiveBroadcastStream().listen(print));

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
flutter doctor -v
[!] Flutter (Channel [user-branch], 3.22.3, on Microsoft Windows [Version 10.0.22631.4317], locale en-GB)
    ! Flutter version 3.22.3 on channel [user-branch] at C:\tools\flutter
      Currently on an unknown channel. Run `flutter channel` to switch to an official channel.
      If that doesn't fix the issue, reinstall Flutter by following instructions at https://flutter.dev/docs/get-started/install.
    ! Upstream repository unknown source is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to unknown source to dismiss this error.
    • Framework revision b0850beeb2 (4 months ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\lbound\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.11.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.11.35312.102
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2024.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] VS Code, 64-bit edition (version 1.95.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.100.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22631.4317]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 121.0.6167.86
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 130.0.2849.68

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.```

</details>

Your Kotlin lambda is capturing the variable i (not its value) in a lamba, and using that variable from two different threads. There is no guarantee that the sending of the current value of i on the main thread is interleaved with the incrementing of i.
If you print i inside your lambda, do you see different values than what you are current printing in Dart?
This looks like a bug in the Kotlin code provided in the repo case, rather than something related to event channels.

Ah yes, that explains the weird ordering/duplication - I'm not too familiar with Kotlin/Java lambdas/threading.
I've fixed that now and I get the values as expected but there seems to be a minimum delay between subsequent event of ~1ms on the flutter side between subsequent events no matter how much I reduce the Thread.sleep() delay.
I imagine this is partially due to a combination of the Android UI event loop, the serialisation/deserialisation, and other flutter engine behind the scenes magic.
Is there any way to reduce this per-event delay?

That codepath includes multiple asynchronous steps and multiple thread hops; there is no way to guarantee sub-millisecond accuracy under those conditions.
Closing as the originally reported issue is not related to Flutter code.


A question about broker fakfa

hope use https://github.com/bsm/sarama-cluster instead of https://github.com/Shopify/sarama， which support consumer group。

Please ask questions in the Slack http://slack.micro.mu/. This is also not the relevant repo.If you would like to use sarama-cluster, write a plugin.


Element size argument in TTL create tensor API

Hey,
In the official TTL documents it's shown that user could create TTL_tensor with providing:
1- the pointer of the allocated memory
2- TTL_shape
3- TTL_layout.
(without providing the element_size of the element type).

Issue : In the compilation of the file below, when I'm providing the element_size=4 ( e.g.  sizeof(int32_t) ) in creating the different TTL tensors I'm checking my output and I'm getting really what I've expected.
However, when not providing the element size I'm getting 1/4 of the output's elements as I've expected and 3/4 total different junk values. So, I'm assuming the DMA transaction didn’t go well.
This is the imp.cl code which I'm trying to compile:
#include "TTL.h"
#include "memrefs.h"
#include "ocl_defs.h"

void _imp_addition_i32___kernel(Memref3D_I32_G* v0, Memref3D_I32_G* v1, Memref3D_I32_G* v2,
                                Memref3D_I32_L* b0, Memref3D_I32_L* b1, Memref3D_I32_L* b2,
                                Memref3D_I32_L* b3){

  // /******************* import v0 -> b0 ***********************/
  TTL_event_t event0 = TTL_get_event();
  TTL_layout_t layout0 = TTL_create_layout(v0->strides[1]/*row_spacing*/, v0->strides[0]/*plane_spacing*/);
  TTL_shape_t shape0 = TTL_create_shape(v0->sizes[0] /*width*/, v0->sizes[1] /*height*/, v0->sizes[2] /*depth*/);

  const TTL_const_ext_tensor_t ext_tensor0 = TTL_create_const_ext_tensor(v0->aligned, shape0, layout0, 4/*elem_size*/);
  const TTL_int_tensor_t int_tensor0 = TTL_create_int_tensor(b0->aligned, shape0, layout0, 4/*elem_size*/);

  TTL_import(int_tensor0, ext_tensor0, &event0);
  TTL_wait(1 /*num_events*/, &event0);

  // /******************* import v1 -> b1 ***********************/
  TTL_event_t event1 = TTL_get_event();
  TTL_layout_t layout1 = TTL_create_layout(v1->strides[1]/*row_spacing*/, v1->strides[0]/*plane_spacing*/);
  TTL_shape_t shape1 = TTL_create_shape(v1->sizes[0] /*width*/, v1->sizes[1] /*height*/, v1->sizes[2] /*depth*/);

  const TTL_const_ext_tensor_t ext_tensor1 = TTL_create_const_ext_tensor(v1->aligned, shape1, layout1, 4/*elem_size*/);
  const TTL_int_tensor_t int_tensor1 = TTL_create_int_tensor(b1->aligned, shape1, layout1, 4/*elem_size*/);

  TTL_import(int_tensor1, ext_tensor1, &event1);
  TTL_wait(1 /*num_events*/, &event1);

  // /******************* export b2 -> v2 ***********************/
  TTL_event_t event2 = TTL_get_event();
  TTL_layout_t layout2 = TTL_create_layout(v2->strides[1]/*row_spacing*/, v2->strides[0]/*plane_spacing*/);
  TTL_shape_t shape2 = TTL_create_shape(v2->sizes[0] /*width*/, v2->sizes[1] /*height*/, v2->sizes[2] /*depth*/);

  const TTL_ext_tensor_t ext_tensor2 = TTL_create_ext_tensor(v2->aligned, shape2, layout2, 4/*elem_size*/);
  const TTL_const_int_tensor_t int_tensor2 = TTL_create_const_int_tensor(b2->aligned, shape2, layout2, 4/*elem_size*/);

  TTL_export(int_tensor2, ext_tensor2, &event2);
  TTL_wait(1 /*num_events*/, &event2);
  return;
}

And in the 'memrefs.h' header declared the memref types: Memref3D_I32_G, Memref3D_I32_L in this way:
typedef struct __attribute__((__packed__))  Memref3D_I32_L
{
 __local void *allocated;
 __local int32_t *aligned;
  int offset;
  int sizes[dims];
  int strides[dims];
}Memref3D_I32_L;

The Memref3D_I32_G is declared also the same but with __local/_global difference.
Would be glad for your help/support.
Thanks,
Amir Bishara

We have a problem of sorts here - I broke a large patch into smaller patches as I should have done, and broke this feature. It is correct when the whole stack lands - and fixing it is actually difficult here because of the way macros are used.
Can you try with this https://github.com/KhronosGroup/OpenCL-TTL/pull/14 and see how that works for you.


Changing the carousel active setting resets the slide to first

Feature request is related to

[x] embla-carousel (core package)

Embla Carousel version

embla-carousel-react: 8.0.0-rc10,

Describe the feature request
I have a state, which starts as true, controlling the active option of my carousel.
When scrolling my carousel to a slide other than the initial slide and changing the value of this state to false, my carousel goes back to the first slide
When I change my state value to true again, it returns my carousel to the slide it was before the first state change occurred
CodeSandbox

The link to a CodeSandbox that demonstrates the behavior clearly.

Steps to reproduce

Create a carousel and pass a boolean state that controls the active option
Add a button that when clicked changes this state to false
Scroll the carousel to the second slide
Click on the button created to update the state of the carousel and note that the carousel that was on the second slide returned to the first slide

Expected behavior

When deactivating the carousel on the second slide, it should not go back to the first slide, but should be deactivated on the slide that I stopped

Additional context
- Video showing the behavior

Hi @hmartiins,
Thanks for your bug report. I wouldn’t call this a bug though because it isn’t obvious to me that it should work the way you describe. I think you could argue for both sides.
The active option is often used together with the breakpoints option. And devs use different setups based on breakpoints which often include grouping slides differently and changing slide sizes. Preserving the index in these cases might cause confusion, because the preserved snap index on a certain breakpoint might not correspond to the index with the same slides on a different breakpoint.
Have you tried tracking the index yourself and setting the startIndex?
Best,
David

Hi @davidjerleke!
I understand what you mean, after creating this issue I took a look at the library code and understood what you meant.
However, in my understanding, when changing the state of the active option, it should not reset the carousel to the first slide. But really, this isn't a bug according to the code.
It would be cool and very useful if there was an option in the embla api that we could change the value of active without the carousel going back to the first slide.

Hi @hmartiins,
Thank you for your input. Many devs use this option to disable the carousel entirely on desktops and activate the carousel on smaller screens, for example mobile phones. In these cases, removing any styles applied by Embla is necessary. This is how it works today and in practice this means resetting to the first slide.
So if we’re to introduce something it should be an additional feature and not replace the current behavior.
For example, one way would be to change the active option to accept string values:
type ActiveOptionType = boolean | 'freeze';

true // Carousel is active
false // Carousel is basically destroyed but listening for active option to change
'freeze' // Preserve last scroll location

Is this close to what you meant?
Best,
David

Awesome! 🤩
Yes, it was exactly what I had in mind, and I believe it solves the case I mentioned.

@hmartiins,
Thank you for your swift response. I want to be clear so you have the right expectations:
I have a lot to do before the stable v8 release so new features will not be prioritized until v8 is released. This means that you have to use the workaround for now if you still want to use this library.
The specification for this feature is not a breaking change so it can easily be introduced after the major v8 release.
In the meantime, if you can think of a better term than freeze, please let me know.
Best,
David

@davidjerleke I couldn't think of any better terms, I think freeze suits it well.

I have trouble with this bug too. Can't add to slide list new slides in start of array. My carousel go to 1 slide and it looks like a bug. Maybe @davidjerleke  you  can create patch and pr with your decide how to prevent moving to slide 1?

@Lemaro86 thanks for chiming in.
First of all, this isn’t a bug. It’s a feature request.
Secondly, reading your description of the problem you’re most likely doing something wrong and it doesn’t seem related to this.
Lastly, what do you mean with this:

Maybe @davidjerleke you can create patch and pr with your decide how to prevent moving to slide 1?

?

@Lemaro86 thanks for chiming in.
First of all, this isn’t a bug. It’s a feature request.
Secondly, when reading your description of the problem you’re most likely doing something wrong and it doesn’t seem to be a bug or related to this.
Lastly, what do you mean with this:

Maybe @davidjerleke you can create patch and pr with your decide how to prevent moving to slide 1?

?

Thank you for so fast response. I am sorry. I mean @hmartiins. I'll edit my message. If it feature in what version you wanna to add it in roadmap?

Hi @hmartiins,
Thank you for your input. Many devs use this option to disable the carousel entirely on desktops and activate the carousel on smaller screens, for example mobile phones. In these cases, removing any styles applied by Embla is necessary. This is how it works today and in practice this means resetting to the first slide.
So if we’re to introduce something it should be an additional feature and not replace the current behavior.
For example, one way would be to change the active option to accept string values:
type ActiveOptionType = boolean | 'freeze';

true // Carousel is active
false // Carousel is basically destroyed but listening for active option to change
'freeze' // Preserve last scroll location

Is this close to what you meant?
Best, David

I would love to have this feature too. I keep forgetting that setting active to false completely destroys the carousel—in my mind the words ‘active’ and ‘inactive’ are more of a temporary nature. But yes, the wording is debatable. 🤠
Another suggestion would be to introduce an additional option, something like listening, which determines whether the carousel will (listen and) react to any clicking, dragging or swiping.

@omakise you can preserve the index before applying display: none and restore it when the carousel gets visible with the startIndex option.

Unlikely to have time to look into this feature request. Closing for now.


[CIRCT] Need to add entry component name to calyx.program.

calyx.program now expects the name of the entry component. Examples found here.
Concretely, this is just:
calyx.program {                           // Before
calyx.program "<insert-program-name>" {   // After

Any thoughts on this one @rachitnigam ?
When we go from MLIR --> native, we lose this information. Some options:

Add some comment when emitting that remarks on which component is the entry-point. Gross.
We could add some key value pair to FUD, e.g. -entrypoint "main".
Is there any benefit to marking the entry-point component in Calyx?

component main<"entrypoint"=1>() -> () { ... }

#708 mentions that we need a toplevel annotation on components to work with simulation. Maybe we can reuse that annotation? If there isn't any component with that, we can default to main. How does that sound?

I think fixing this with fud is fine, but do you foresee this as a re-occurring issue?
Specifically, the frontend to Calyx defines a specific component as the entry-point, and the backend needs to determine which component is the entry-point. To me, this should be information kept at the program level rather than the driver level.


chore(main): release bikeweek-client 0.4.1

:robot: I have created a release beep boop
0.4.1 (2024-03-21)
Bug Fixes

deploy now uses dist instead of deploy (8cf9ad4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/madisonbikes/bikeweek-client/releases/tag/bikeweek-client-v0.4.1 :sunflower:


Fix cookies permission logic

We should have cookies permissions enabled for all sui domains and subdomains, including Sui.io, docs.sui.io and explorer.xxx.sui.io
Once a user accepts or choose setting, the permissions popup should not appear when they visit other subdomains. Currently sui.io and docs.sui.io is still triggering permissions despite saving settings.

5/12 couldn't reproduce for sui.io and docs.sui.io; next to implement cookie to explorer


question about graphcam

I read your source code, and have a question about the graphcam.
The code "for index_ in range(3):",I wonder whether the number 3 is fixed or determined by the classes in the classification, for example, for a 5-class task, whether the code should be changed to for index_ in range(5):?
if graphcam_flag:
print('GraphCAM enabled')
p = F.softmax(out)
torch.save(p, 'graphcam/prob.pt')
index = np.argmax(out.cpu().data.numpy(), axis=-1)
        **for index_ in range(3):**
            one_hot = np.zeros((1, out.size()[-1]), dtype=np.float32)
            one_hot[0, index_] = out[0][index_]
            one_hot_vector = one_hot
            one_hot = torch.from_numpy(one_hot).requires_grad_(True)
            one_hot = torch.sum(one_hot.cuda() * out)       #!!!!!!!!!!!!!!!!!!!!out-->p
            self.transformer.zero_grad()
            one_hot.backward(retain_graph=True)

            kwargs = {"alpha": 1}
            cam = self.transformer.relprop(torch.tensor(one_hot_vector).to(X.device), method="transformer_attribution", is_ablation=False,
                                        start_layer=0, **kwargs)

            torch.save(cam, 'graphcam/cam_{}.pt'.format(index_))

    return pred,labels,loss

Hi did you figure out graphcam, may I know where you find this part code in the folder?
Your reply is appreciated.

Agreed. I think that instead of 3 it should be num_classes = p.size(1). If it's more convenient for @GSWS or @vkola-lab I could put in a PR to adjust this, as if it isn't changed I'll likely end up forking it for my own use.
https://github.com/vkola-lab/tmi2022/blob/6d72bfa44c40bd29289d555ffc1f77ffa7f4ce8e/models/GraphTransformer.py#L79-L98

Pull request approved. Thank you for your valuable contribution.


practice

<...describe the change...>

[ ] Change in CHANGELOG.md described

Could you explain how the template itself benefits from the change? - Currently, I don't see the real benefit. So I just close this PR. Feel free to reopen as soon as you have additional details.


App Services Based Lab Scenario (Part 2)

This PR will present a new lab scenario based around App Services instead of Cloud Services

Mod 9 (Automation) has been updated to use both the xPlat CLI and PowerShell

Added MCT-related files

Added MCT-related files


No longer works on Windows 10

Latest update no longer works on Windows 10.
Files:
script-opt > file_browser.conf (Only edited root=C:/,E:/,G:/,R:/)
script-opt > file-browser-keybinds.json
scripts > file-browser.lua
scripts > scroll-list.lua
scripts > user-input.lua
scripts > user-input-module.lua

All of these are the latest version
Pressing TAB (TAB script-binding browse-files in input.conf) does nothing.
When running in Powershell mpv.com TAB does nothing and nothing shows in Powershell

Using file-browser.lua from no-scroll-list tree works.

Can you post a log

[file_browser]
[file_browser] stack traceback:
[file_browser]  [string "C:/Users/iay/Desktop/mpv/scripts/file-browser..."]:1178: in main chunk
[file_browser]  [C]: at 0x007b2ff0
[file_browser]  [C]: at 0x007b1a00
[file_browser] Lua error: cannot open C:\Users\iay\AppData\Roaming/mpv/script-modules/user-input-module.lua: Invalid argument

Ah right, woops

From: twophyro<EMAIL_ADDRESS>Sent: Thursday, February 4, 2021 9:57:43 AM
To: CogentRedTester/mpv-file-browser<EMAIL_ADDRESS>Cc: CogentRedTester<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [CogentRedTester/mpv-file-browser] No longer works on Windows 10 (#24)
[file_browser]
[file_browser] stack traceback:
[file_browser] [string "C:/Users/iay/Desktop/mpv/scripts/file-browser..."]:1178: in main chunk
[file_browser] [C]: at 0x007b2ff0
[file_browser] [C]: at 0x007b1a00
[file_browser] Lua error: cannot open C:\Users\iay\AppData\Roaming/mpv/script-modules/user-input-module.lua: Invalid argument
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/CogentRedTester/mpv-file-browser/issues/24#issuecomment-772897882, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANUOTXAZ5LPYI35Z3P4KAYLS5HLW7ANCNFSM4XBID2RQ.

user-input-module needs to go into  ~~/script-modules/.
The here is that it wasn't supposed to be compulsory.

Would it be possible to move it in to file-browser.conf where the user gets to define where their MPV installation is, or is that too clumsy?

I can allow people to modify the path if you like, I moved the directory to ~~/script-modules/ because I didn't want mpv to also be loading modules that it wasn't going to use..

Also the crashing should now be fixed after the latest commit

It works now 👍

You can now modify the module directory.

I don't know how you want to continue with this but looking at the code it's able to load file-browser-keybinds.json fine even though it looks similar in file-browser.lua

The keybinds json file is only loaded when custom_keybinds is enabled in file_browser.conf, so it's fine. In fact, if custom_keybinds is enabled and the file is not in the correct place then the code is specifically designed to throw an error letting you know that the keybinds failed to load.

Thanks for the help 👍


Update dependency versions in Pytorch

How Has This Been Tested?

Deploy live-build
Go to JupyterHub Route
View packages in spawner UI for Pytorch image

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] For commits that came from upstream, [UPSTREAM] has been prepended to the commit message.
[x] JIRA link(s):  https://issues.redhat.com/browse/RHODS-2978
[x] The Jira story is acked.
[x] Live build image: quay.io/modh/rhods-operator-live-catalog:1.14.0-rhods-2978
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.
[ ] QE contact acknowledges that this has been tested and is approved for merge.

Tested with live build, can be merged after https://github.com/red-hat-data-services/ods-ci/pull/470


Change puppet deprecated setting

Hi,
I've removed some deprecated settings in Puppet. This makes the Puppet a little smoother.
Changes should be pretty straight forward.
Cheers

I've merged in your changes...thank you for the contribution!

Glad I could help!


Swift 4 新特性 - 柏荧的博客 | BY Blog

http://qiubaiying.top/2017/09/11/Swift-4-新特性/
Every failure is leading towards success.

good！


Custom Images not showing unless in a module folder

Describe the bug
When an item like a weapon or spell is using a custom image it will only show if it is in a folder in the modules section.
Versions:

Token Action HUD Core: Current as of 12/15/2024
Token Action HUD SWADE: Current as of 12/15/2024
Swade System: Current as of 12/15/2024
Foundry VTT: Current Stable 12 as of 12/15/2024
Browser: I'm using the local application but a player tested it and they were using some version of Chrome

Additional context
To give context to the image, each item has a custom image the blunderbuss with an image is from when I made a custom module to test if putting it in a module fixed it. This started a few months ago (or maybe a year it is a little fuzzy) and it wasn't all at once. Different file types went at different times.

Hello @Drone4416
This isn't happening to me. Could you please check the console and look for errors.
How to access the console in: chrome, Firefox

Okay feeling very dumb now, I apparently put spaces in the folder name and that is what was causing the issue. Foundry could see it and not have issues but any module dealing with it had issues. I changed the filename and it started working.

Do you have spaces between names or after names? like my folder name or is like myfoldername

It was the spaces between that was the issue. I renamed the folder and ran a macro to change all the links over and that fixed it.


task is not showning in widget list, when a due date is set

Hi there,
at first, thanks for this nice app.
When you create a new task and you set a due date and time, the task is not showing in the widget list.
If you go to the app itself, the created task is showing, but not in the widget.
Android 8.1, Patchlevel Juli, BQ Aquaris Stock Rom 1.3.
Would be great it can be fixed.
TIA Micha

Did you also set a start date? At present the widget only shows tasks which have started already (i.e. have no start date or a start date in the past).

Tests today doen't show this behaviour. Seems that I can create a task with a due date without a start date.


add methods getTiles and setTiles to mimick maplibre-gl-js interface

This adds the methods setTiles and getTiles to VectorSource to mimick the maplibre-gl-js interface
introduced in https://github.com/mapbox/mapbox-gl-js/pull/8048. This is useful to change parametrized data sources without having to load a complete new style.
Todo:

[ ] To completely port the maplibre-gl-js interface, also setUrl should be added.
[ ] Is the additional call to observer->onSourceLoaded correct/necessary?
[ ] There is a race and the map sometimes still requests old tiles even after setTile is called.

Any input especially about the race is appreciated.

We use this patch in production for over a year now. I just finally got around to update to latest maplibre-native opengl-2 branch and it still works as expected. Moreover I can no longer reproduce the race and I doubt it was a problem with this patch anyway.
I no longer thinks we should block this on an implementation of setUrl. That is a separate issue and a bit more complex, so I'd rather just get this in for now.
From my reading of the code the call to observer->onSourceLoaded is not necessary for Core funktionality, as onSourceLoaded and onSourceChanged ist handled exactly the same as onSourceLoaded in https://github.com/maplibre/maplibre-native/blob/main/src/mbgl/style/style_impl.cpp#L314. I'm happy to leave it in or take it out whatever the semantics of an external Observer are supposed to be.
So this ready for reviewing from my point of view.

Nice @xabbu42! Please remove the draft status in that case

I'll have to look in the failed checks tomorrow.

Sorry for the long silence. I'll be away for the next 3 weeks but I plan to get back to this after that.

The checks run through before the merge and it looks like the failures now have nothing to do with my changes. @louwers I hope I addressed your requests adequately (I'm not familiar with doxygen or GoogleTest). So this should be ready to merge now.

@xabbu42 Seems like it runs through cleanly now. Let me have another look.


Does partition_size get passed correctly always?

@qubixes I managed to create a reprex, let's discuss this!
It seems from the toml the partition_size does not necessarily get imposed properly, see a reprex below:
import tempfile

import polars as pl
from metasyn import MetaFrame
from metasyn.config import MetaConfig

config_toml_11 = """
dist_providers = ["builtin", "metasyn-disclosure"]

[privacy]
name = "disclosure"
parameters.partition_size = 11
"""
config_11 = tempfile.NamedTemporaryFile("w", suffix=".toml", delete=False)
config_11.write(config_toml_11)
config_11.close()

config_toml_55 = """
dist_providers = ["builtin", "metasyn-disclosure"]

[privacy]
name = "disclosure"
parameters.partition_size = 55
"""
config_55 = tempfile.NamedTemporaryFile("w", suffix=".toml", delete=False)
config_55.write(config_toml_55)
config_55.close()

df = pl.DataFrame({"x1": range(30)})
mf_no = MetaFrame.fit_dataframe(df)
mf_11 = MetaFrame.fit_dataframe(df, var_specs=MetaConfig.from_toml(config_11.name))
mf_55 = MetaFrame.fit_dataframe(df, var_specs=MetaConfig.from_toml(config_55.name))

print(mf_no)
print(mf_11)
print(mf_55) # ??? why does 55 work & give same result as 11?? it should not!

Even though this info is correct:
mf_55[0].creation_method

{'created_by': 'metasyn', 'privacy': {'name': 'disclosure', 'parameters': {'partition_size': 55}}}

This should be fixed now. If it isn't, reopen the issue.


Update organizing.md

Add a section about sponsorship and legal entities.

:+1:

:+1:
I think that ask for things or services instead of money is a good idea everywhere. Saves a lot of time and hassle.

:+1:
It is always easy to have a request on facilitation approved as opposed to money. So we need to major on services as Erick has put it.

@friedger Looks fine. Will be merging this.


func new in VSCode has weird menu issue

When running func new in the VS Code terminal, the alignment is off by 2 when selecting options.

so this is actually fixed on vscode side in the latest insider build (1.10.0). I'm closing this as I don't think there is something for us to do here.


Importing same class that is in 2 different files does not identify it as same class

Lets say I have a class called AbstractClass:
// animal.ts
export abstract class Animal {
   public abstract talk(): String;
}

// cat.ts
import {Animal} from "./animal";

export class Cat extends Animal {
    public talk(): String {
        return "meow";
   }
}

// dog.ts
import {Animal} from "./animal";

export class Dog extends Animal {
   public talk(): String {
       return "whoof whoof";
   }
}

Now I have another module that has dependency on that class.
import {Animal} from "animal-module/animal";

// play-with-animal.ts
export class PlayWithAnimal {
    private animal: Animal;

    constructor(animal: Animal) {
        this.animal = animal;
    }

   public makeAnimalTalk(): void {
        console.log(this.animal.talk());
   }
}

// animal.component.ts
import {PlayWithAnimal} from "play-with-animal-module/play-with-animal.ts"
import {Cat} from "animal-module/cat";

someFunction() {
    let cat: Cat = new Cat();
    let playWithAnimal: PlayWithAnimal = new PlayWithAnimal(cat);

   // here I get error saying:
  // TS2345: Argument of type 'Cat' is not assignable to parameter of type 'Animal'. 'Animal' is not a class derived from 'Animal'.
}

Here is how the folder structure is. I am using node to resolve the modules.
- app
   animal.component.ts
   // app has dependency of animal-module
- node_modules
   - animal-module
        animal.ts
        cat.ts
        dog.ts
   - play-with-animal-module
      play-with-animal.ts
      // talking-animal-module has dependency of animal-module
      - node_modules
         - animal-module
            animal.ts
            cat.ts
            dog.ts

My finding so far is that, as in the dog.ts or cat.ts, I have imported the Animal as import {Animal} from "./animal";. Notice the relative path.
However in PlayWithAnimal or AppComponent the module is resolved using the node resolution. Somehow typescript thinks that they are separate classes (eventhough) they are same as they are being loaded from different file paths.
If I wrap Animal in a exports module 'SomeModule' and update the code to use SomeModule.Animal then it works fine even if I load the class from 2 different files. However if I do so then I lose ability to import multiple classes like import {Animal, SomeotherClass}.
So my question is:

Is this by design or its an issue?
If this is by design then how are we meant to resolve the modular dependencies?

Thanks.

@adikari there are several errors in the code you posted.
import "PlayWithAnimal" from "play-with-animal-module/play-with-animal.ts"
import "Cat" from "animal-module/cat";

should be
import { PlayWithAnimal } from "play-with-animal-module/play-with-animal";
import { Cat } from "animal-module/cat";

Only a default export can be imported without destructing or spread syntax, and even then, the quotes around the import names are invalid syntax.
The error message

// TS2345: Argument of type 'Cat' is not assignable to parameter of type 'Animal'. 'Animal' is not a class derived from 'Animal'.

is a bit strange though.
There might be a deeper problem here, but the import syntax is definitely wrong.

Classes with private or protected members are compared nominally. i.e. they have to be the exact same class. The idea is that these classes have declared a dependnecy on privatly held fileds, and you can not just substitute something that "looks" like them in place.
If you have two modules that look like one-another, they are not the same module. consider the case where package.json in play-with-animal-module specifies a different version for animal-module than that of your app; at runtime, you really have two modules. and as per your class definition (having a private member) that is not safe to pass one to the other.
so the solution, either do not use privates, or make your public API surface only interfaces, the other alternative, make sure that there is only one play-with-animal-module. this should be the default with npm v3 and later, as all dependencies are flattened.

@aluanhaddad My bad. Its just the typo while i was typing in the markdown editor.

@mhegazy That does make sense. However I am not very clear about,

Classes with private or protected members are compared nominally

Also using interface is not an options and the parent class must be an abstract class as it has more functionality in it that I have not mentioned in the sample code.
So you reckon npm v3 is a solution to this problem where all dependencies will be flattened based upon the version?

Also using interface is not an options and the parent class must be an abstract class as it has more functionality in it that I have not mentioned in the sample code.

Well then interfaces would not work for your use-case. In general types in TS are compared structurally, i.e. two types are comparable iff they have the same shape. classes/interfaces with private/protected properties are an exception.
class C {
    a: number;
}

var c: C = { a: 1 };  // OK, {a:number} and C are compared structurally

class D {
   private a: number;
}

var d: D = { a: 1 };  // Error, {a:number} and D can not be compared, as D has a private and only instances of D can be used

So you reckon npm v3 is a solution to this problem where all dependencies will be flattened based upon the version?

yes. that assuming that there are no version conflicts between the two importing packages.

On a related note, https://github.com/Microsoft/TypeScript/pull/8486 should handle the npm link scenarios where is really one file on disk but linked from multiple sources.


Does not compile with Gradle

$ gradle assemble
Creating properties on demand (a.k.a. dynamic properties) has been deprecated and is scheduled to be removed in Gradle 2.0. Please read http://gradle.org/docs/current/dsl/org.gradle.api.plugins.ExtraPropertiesExtension.html for information on the replacement for dynamic properties.
Deprecated dynamic property: "artifactId" on "project ':crouton-library'", value: "crouton".
FAILURE: Build failed with an exception.

Where:
Build file '/Users/code/studio/libraries/Crouton/library/build.gradle' line: 96

What went wrong:
A problem occurred evaluating project ':crouton-library'.

No such property: repositoryUrl for class: org.gradle.api.publication.maven.internal.ant.DefaultGroovyMavenDeployer
Possible solutions: repository

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 3.749 secs

See README for further info.


missing treebank info on two Greek words

go to line 46 of https://texts-test.alpheios.net/text/urn:cts:greekLit:tlg0012.tlg001.alpheios-text-grc1/passage/1.1-1.30,
both words δ᾽   and ἄρ᾽ are in the tree but we do not show any treebank icons in the pop-up. this is also on texts.alpheios.net

this is actually a really good find.
There are a couple of different issues here.
The treebank is annotated with the part of speech as 'particle'.  This is what morpheus reports for this, but at some point the treebanking guidelines used by Perseus/Perseids changed to have particle tagged as adverb.  Any trees created on Perseids after that change was made will tag particle as adverbs and not offer the ability to tag them as particles.  The trees we use on some of the alpheios texts predate this change, and arethusa doesn't know how to interpret the 'g' postag.  So it doesn't return any part of speech for these.
We could  fix our Arethusa setup so that it can map the 'g' to adverb, but that raises another question -- in that case we would end up getting an entirely new lexeme added in Alpheios, for which we would not have any match in the morpheus output and we would end up duplicating it for the adverb.
We can see this in @vgorman1 's trees e.g. at
https://vgorman1.github.io/151-student-practice/1_6_adjective_practice.html

I'd like to know what @vgorman1 and @rgorman think is the ideal situation here. Should Alpheios always be reporting Greek particles as adverbs?  Is there ever a particle that isn't an adverb?

this is actually a really good find.
There are a couple of different issues here.
The treebank is annotated with the part of speech as 'particle'.  This is what morpheus reports for this, but at some point the treebanking guidelines used by Perseus/Perseids changed to have particle tagged as adverb.  Any trees created on Perseids after that change was made will tag particle as adverbs and not offer the ability to tag them as particles.  The trees we use on some of the alpheios texts predate this change, and arethusa doesn't know how to interpret the 'g' postag.  So it doesn't return any part of speech for these.
We could  fix our Arethusa setup so that it can map the 'g' to adverb, but that raises another question -- in that case we would end up getting an entirely new lexeme added in Alpheios, for which we would not have any match in the morpheus output and we would end up duplicating it for the adverb.
We can see this in @vgorman1 's trees e.g. at
https://vgorman1.github.io/151-student-practice/1_6_adjective_practice.html

I'd like to know what @vgorman1 and @rgorman think is the ideal situation here. Should Alpheios always be reporting Greek particles as adverbs?  Is there ever a particle that isn't an adverb?

I would think tagged all “g”s to become adverbs would be a simple solution.  A particle is, by definition, a subgrouping of the category adverb.
Vanessa B. Gorman
Professor of Ancient History
Department of History
619 Oldfather Hall
University of Nebraska-Lincoln
Lincoln, NE 68588-0327
https://vgorman1.github.io/
From: Bridget Almas<EMAIL_ADDRESS>Sent: Monday, January 11, 2021 3:00 PM
To: alpheios-project/alpheios-trees<EMAIL_ADDRESS>Cc: Vanessa Gorman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [alpheios-project/alpheios-trees] missing treebank info on two Greek words (#13)
Non-NU Email

this is actually a really good find.
There are a couple of different issues here.
The treebank is annotated with the part of speech as 'particle'. This is what morpheus reports for this, but at some point the treebanking guidelines used by Perseus/Perseids changed to have particle tagged as adverb. Any trees created on Perseids after that change was made will tag particle as adverbs and not offer the ability to tag them as particles. The trees we use on some of the alpheios texts predate this change, and arethusa doesn't know how to interpret the 'g' postag. So it doesn't return any part of speech for these.
We could fix our Arethusa setup so that it can map the 'g' to adverb, but that raises another question -- in that case we would end up getting an entirely new lexeme added in Alpheios, for which we would not have any match in the morpheus output and we would end up duplicating it for the adverb.
We can see this in @vgorman1https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vgorman1&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=XJGmp9ZAh-yRZkqNhAclBIEAVQ0jXeaSmD6OAJhjofg&e= 's trees e.g. at
https://vgorman1.github.io/151-student-practice/1_6_adjective_practice.htmlhttps://urldefense.proofpoint.com/v2/url?u=https-3A__vgorman1.github.io_151-2Dstudent-2Dpractice_1-5F6-5Fadjective-5Fpractice.html&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=n72i5Zv2w5lf-Gwd5vRjIa-SbBCJBOaGyM25K0bdaiQ&e=
[d5f6799c-8f3f-48b2-82ce-f4ef60eb9256]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_644870_104237152-2Da3966e00-2D5425-2D11eb-2D8e56-2D5290655ccd2c.png&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=41qwJSzgdvyyAfKOorKeRDiD7Og-sXNYluyJ3DcyS3g&e=
I'd like to know what @vgorman1https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vgorman1&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=XJGmp9ZAh-yRZkqNhAclBIEAVQ0jXeaSmD6OAJhjofg&e= and @rgormanhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=VBHujbec1Z69sQRr0reyh4k9nufCqnTgswxUnew4_ZU&e= think is the ideal situation here. Should Alpheios always be reporting Greek particles as adverbs? Is there ever a particle that isn't an adverb?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_alpheios-2Dproject_alpheios-2Dtrees_issues_13-23issuecomment-2D758219884&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=iUqIGYfGq6svhvoUmG7XAcXmNeVG3YK-hECl8Bv0f8s&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ACPTZUQOWWV5HGO6OLQVXV3SZNRGBANCNFSM4V55E6XQ&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=15kEqCZB1QNz0OhdUZoo0s6Opr85yrNFhA-ybyZVjd4&e=.

I would think tagged all “g”s to become adverbs would be a simple solution.  A particle is, by definition, a subgrouping of the category adverb.
Vanessa B. Gorman
Professor of Ancient History
Department of History
619 Oldfather Hall
University of Nebraska-Lincoln
Lincoln, NE 68588-0327
https://vgorman1.github.io/
From: Bridget Almas<EMAIL_ADDRESS>Sent: Monday, January 11, 2021 3:00 PM
To: alpheios-project/alpheios-trees<EMAIL_ADDRESS>Cc: Vanessa Gorman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [alpheios-project/alpheios-trees] missing treebank info on two Greek words (#13)
Non-NU Email

this is actually a really good find.
There are a couple of different issues here.
The treebank is annotated with the part of speech as 'particle'. This is what morpheus reports for this, but at some point the treebanking guidelines used by Perseus/Perseids changed to have particle tagged as adverb. Any trees created on Perseids after that change was made will tag particle as adverbs and not offer the ability to tag them as particles. The trees we use on some of the alpheios texts predate this change, and arethusa doesn't know how to interpret the 'g' postag. So it doesn't return any part of speech for these.
We could fix our Arethusa setup so that it can map the 'g' to adverb, but that raises another question -- in that case we would end up getting an entirely new lexeme added in Alpheios, for which we would not have any match in the morpheus output and we would end up duplicating it for the adverb.
We can see this in @vgorman1https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vgorman1&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=XJGmp9ZAh-yRZkqNhAclBIEAVQ0jXeaSmD6OAJhjofg&e= 's trees e.g. at
https://vgorman1.github.io/151-student-practice/1_6_adjective_practice.htmlhttps://urldefense.proofpoint.com/v2/url?u=https-3A__vgorman1.github.io_151-2Dstudent-2Dpractice_1-5F6-5Fadjective-5Fpractice.html&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=n72i5Zv2w5lf-Gwd5vRjIa-SbBCJBOaGyM25K0bdaiQ&e=
[d5f6799c-8f3f-48b2-82ce-f4ef60eb9256]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_644870_104237152-2Da3966e00-2D5425-2D11eb-2D8e56-2D5290655ccd2c.png&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=41qwJSzgdvyyAfKOorKeRDiD7Og-sXNYluyJ3DcyS3g&e=
I'd like to know what @vgorman1https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vgorman1&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=XJGmp9ZAh-yRZkqNhAclBIEAVQ0jXeaSmD6OAJhjofg&e= and @rgormanhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rgorman&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=VBHujbec1Z69sQRr0reyh4k9nufCqnTgswxUnew4_ZU&e= think is the ideal situation here. Should Alpheios always be reporting Greek particles as adverbs? Is there ever a particle that isn't an adverb?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_alpheios-2Dproject_alpheios-2Dtrees_issues_13-23issuecomment-2D758219884&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=iUqIGYfGq6svhvoUmG7XAcXmNeVG3YK-hECl8Bv0f8s&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ACPTZUQOWWV5HGO6OLQVXV3SZNRGBANCNFSM4V55E6XQ&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=2H4UzsQahEdH_c1kLxyGAg&m=T6gVfk79lHmUx5ZEAoO0Hk3dOsVAV7BHqSiyG4dEdGg&s=15kEqCZB1QNz0OhdUZoo0s6Opr85yrNFhA-ybyZVjd4&e=.


[PATCH] Set Icon size limits,  Small refactor to shorten the code and ease maintenance

This changes come from the review of the #66 pull request.
Proposed changes:

Set a limit on the custom size accepted for the icon to 150px for the width and 300px for the height, for design purposes;
Shorten the code to set the custom size using Outlets for the related constraints instead of identifiers;
Set a constraint to avoid icon overlapping other elements without using the pop-up height;

looks great! nice work


PgRowPublisherOperation should consider Subscriber demand

PgRowPublisherOperation accepts Flow.Subscriber that requests a number of data signals n (typically rows) via Subscription.request(n).  PgRowPublisherOperation should consider demand to:

Determine a reasonable fetch size (probably a good suggestion to introduce fetchSize(long rows) on RowPublisherOperation in ADBA spec)
Request only n rows from a cursor (when using cursored fetching)
Request nextn rows as soon as Subscriber requests additional rows

Typically, it's hard to fetch exactly n rows, but the point here is to fetch more or less n rows to satisfy demand and pre-buffer some additional rows (smart prefetching) to reduce latency between I/O and processing.

I have thought some more on this:

Specifying a request limit will block the sending of other queries while this on is in flight, as adding a limit to the Execute message of the wire protocol puts the protocol in a "chatty" mode where you can't send and receive asynchronously, maybe not a bit problem but it might impact speed and will require some redesign of the network layer to expose this information to it.
The SubmissionPublisher class has an estimateMinimumDemand() function that looks like it would be suitable for this usecase.
There has been quite some debate about the need for backpressure on the jdbc-spec-discuss mailing list and my gut feeling is that there will be more feedback on that subject, the topic is quite complex.

Based on the above points I still thinks this is a valid feature request, but as it's nontrivial to implement and the specification might change again I will focus on lower hanging fruit for now.

I'm not entirely familiar with the Postgres wire protocol, but I know from older SQL server versions that a single transport connection was bound to single query execution and didn't allow for interleaved SQL batches

This is the case with the Postgres protocol too, a connection cannot be multiplexed. See https://www.postgresql.org/docs/current/protocol.html.


philips_js not showing sources

I cannot view or select sources with philips_js.
I've tried all API's (1-6) but none work.
I think this might be a bug.

Hi there @solidssss,
Could you please provide some more information that allows somebody to reproduce your issue?
For example the version of Home Assistant used, product used, what you've tried already, how does your config look like, and is there anything visible in the logs that might be of help?
Thanks! 👍

Keep in mind that some Philips models don't have the option to give the sources or select them directly.
Even a rather new model I have which uses API 6.
Does the MyRemote app on your phone allow selecting sources directly? Or does it open the menu, and only allow you to select a source by browsing to it?

Hi there @solidssss,
Could you please provide some more information that allows somebody to reproduce your issue?
For example the version of Home Assistant used, product used, what you've tried already, how does your config look like, and is there anything visible in the logs that might be of help?
Thanks! 👍

The device is a 65PUS6703/12. I have Home Assistant 0.103.5.
My config:
`media_player:

platform: philips_js
host: <IP_ADDRESS>
name: TV-65
api_version: 6 `
As stated earlier, I have tried changing api versions from 1 to 6  but still cannot choose input sources.

Keep in mind that some Philips models don't have the option to give the sources or select them directly.
Even a rather new model I have which uses API 6.
Does the MyRemote app on your phone allow selecting sources directly? Or does it open the menu, and only allow you to select a source by browsing to it?

I can change sources in the Myremote app if I press "Sources", then browing with the touch bar below. The app says that is has API: 6.
In my case it seems that I have a model that can't do it.
Thanks for the help

Do you get a choice of sources or is the list empty?

The list is empty.

For v6 of the API, there is currently no support for source selection. It aught to give you channels thou if you are on the tv activity.
I won't be able to fix this since I don't have any such tv.

For v6 of the API, there is currently no support for source selection. It aught to give you channels thou if you are on the tv activity.
I won't be able to fix this since I don't have any such tv.

Alright, thanks for letting me know

@cyberjunky is this something you could help out with?

PS. There is also this: https://community.home-assistant.io/t/philips-android-tv-component/17749
They seem to have some stuff in a better state for v6

I followed  and could control my TV with it. It should work to make a script in home assistant to send keys and change source input. It's not an easy way to change source input, but it would work.

Thanks, the topic is about android tvs' using pairing and using port 1926 instead of non pairing and using port 1925, so I stopped reading through it, it's quite long
But your link https://community.home-assistant.io/t/philips-android-tv-component/17749/256 is also for non-android.. ;)


Pressing ESC closes parent modal form?

Hello,
I put datepicker onto react-responsive-modal - and ESC on opened datepicker closes not only the datepicker but also modal - which is wrong. I do see in docs on keyboard support "Enter & Esc calls preventDefault", and I see it in sources - but modal is closed. Please advise what I'm doing wrong. Thanks!
Test.jsx:
import React, { Component } from "react";
import Modal from "react-responsive-modal";
import DatePicker from "react-datepicker";
import "react-datepicker/dist/react-datepicker.css";
import "./Test.css";

class Test extends Component {
  constructor(props) {
    super(props);

    this.state = { isOpen: false, startDate: new Date(), endDate: new Date() };
  }

  onClick = () => {
    this.setState({ isOpen: !this.state.open });
  };

  onClose = () => {
    this.setState({ isOpen: false });
  };

  changeStartDate = date => {
    this.setState({
      startDate: date
    });
  };

  changeEndDate = date => {
    this.setState({
      endDate: date
    });
  };

  render() {
    return (
      <>
        <button onClick={this.onClick}>Press me</button>
        <Modal
          open={this.state.isOpen}
          onClose={this.onClose}
          classNames={{ modal: "modal-class" }}
          center={true}
          onEscKeyDown={this.onEscKeyDown}
        >
          <input type="text"></input>
          <DatePicker
            selected={this.state.startDate}
            onChange={this.changeStartDate}
          />
          <DatePicker
            selected={this.state.endDate}
            onChange={this.changeEndDate}
          />
        </Modal>
      </>
    );
  }
}

export default Test;

Test.css:
.modal-class {
  border-radius: 5px;
  padding: 20px;
  width: 600px;
  height: 400px;
}

Does someone found a solution to fix this issue?

Does someone found a solution to fix this issue?

As a hack in modal's onClose I do the following check:
onClose = (event: any) => {
if (
event.target.id !== 'idStartDate' &&
event.target.id !== 'idEndDate'
) {
this.setState({ isOpened: false });
}
};
But this is very dirty actually.

I think event. stopPropagation() needs to be added.
https://github.com/Hacker0x01/react-datepicker/blob/f8fc33c8669dc5b26921a14528154cce948be9dc/src/index.jsx#L645-L647
https://github.com/Hacker0x01/react-datepicker/blob/f8fc33c8669dc5b26921a14528154cce948be9dc/src/index.jsx#L667-L669


add: yashSoni9000

Description

added my profile to public/data and to profielist.json file
Related Issues

Changes Proposed

Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@yashSoni9000 Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


feat: macOs

pinpointPy on macOs

Codecov Report

Merging #295 (6998100) into dev (de0924c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #295   +/-   ##
=======================================
  Coverage   74.57%   74.57%
=======================================
  Files          20       20
  Lines         889      889
=======================================
  Hits          663      663
  Misses        226      226

Impacted Files
Coverage Δ

common/src/Cache/SharedObj.c
73.46% <ø> (ø)

common/src/Logging.c
86.66% <ø> (ø)

common/src/ConnectionPool/TransLayer.cpp
81.37% <100.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 de0924c...6998100. Read the comment docs.

Codecov Report

Merging #295 (6998100) into dev (de0924c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #295   +/-   ##
=======================================
  Coverage   74.57%   74.57%
=======================================
  Files          20       20
  Lines         889      889
=======================================
  Hits          663      663
  Misses        226      226

Impacted Files
Coverage Δ

common/src/Cache/SharedObj.c
73.46% <ø> (ø)

common/src/Logging.c
86.66% <ø> (ø)

common/src/ConnectionPool/TransLayer.cpp
81.37% <100.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 de0924c...6998100. Read the comment docs.


Remove -ffast-math compile option when hardware platform is ARM.

softmax-output-imagenet-test and softmax-output-smoketest test failed on arm platform.
-ffast-math compile option can result in incorrect output for programs that depend on
an exact implementation of IEEE or ISO rules/specifications for math functions.

I presume other architectures might be affected as well. Better disable -ffast-math on all architectures.

good suggestion.


update action status types

@apdavison I'm adding the interlex IDs where it is available, added synonyms and adding 'inactive' as additional type.

@apdavison the suggested interlex link was discussed with @tgbugs but I can remove it until the ontology issue is solved.
I agree with your suggestion to split 'inactive' to 'paused', 'on hold' and 'pending'. But I also think we should keep 'inactive' as a broader term in addition.
I don't see why we should stick to only the schema.org action status types when they are clearly missing other action states (e.g. for inactive states). We still keep the interoperability with the one that they have.
Note that ActionStatusType may not only be needed for computational operations but can also be used in other contexts where action states need to be defined (e.g. actions in a behavioral task for example). I would not like to introduce an additional terminology for these types but rather have one (where maybe some terms are computer specific and others e.g. behavioral specific which should then be stated in the definition of the term).

"pending" would be useful. Otherwise, ok to merge.

@apdavison I agreed with all of your suggestions and made the changes. Please double check again


ipcache: avoid memory allocations when not running in debug mode

Read per commit basis.
This will probably be deprecated once we move out of logrus.

This change is

test-me-please

Coverage decreased (-0.09%) to 42.014% when pulling b50c07d06874dc2ee1c0baf0a11b2d7d844fefe5 on pr/optimize-ipcache into 79b4eba1b2cf0b471699c1c53712a8e7a982db4c on master.

Checkout from Git failed on Jenkins: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/12952/execution/node/7/log/

test-me-please


Support for methods public "sabka Bhai" private "mera Bhai" and protected "apna Bhai"

Methods can easily be called easily like in other programming languages
Also as stated by others package support is much appreciated.
Make this a full fledged programing language which Indian coders will use at their jobs.

No current plans to add this feature in this language as this will lead to more complexities in this toy language.


concurrent read/write support

Hi,
I was wondering if you had plans to implement concurrent reading/writing for raster datasets. As far as I can understand from the documentation, that should be achieved by using GDALDatasetHGDALOpenShared(const char*, GDALAccess) instead of GDALDatasetHGDALOpen(const char*, GDALAccess)  . See https://gdal.org/api/raster_c_api.html#_CPPv414GDALOpenSharedPKc10GDALAccess
Thanks in advance.

@geohardtke Could you elaborate more on "concurrent r/w" ?  For instance, with multiple threads, OpenShared has no effect, and is equivalent to Open (as mentioned int he doc link you've mentioned above).  See quoted text below.

Starting with GDAL 1.6.0, if GDALOpenShared() is called on the same pszFilename from two different threads, a different GDALDataset object will be returned as it is not safe to use the same dataset from different threads, unless the user does explicitly use mutexes in its code.

Also, in this crate, we allow Sending a dataset handle to another thread, so to my understanding, OpenShared is not compatible with our usage.

@rmanoka Thank you very much for your answer, I'm glad to hear that what i need is possible in rust, I guess I'm still new to rust and have a lot to learn. I often deal with raster inputs/outputs that are much larger than memory and I'd like to be able to process the inputs blockwise in parallel and write the results to a compressed output raster. I've been doing that using rasterio in python (as shown here: https://rasterio.readthedocs.io/en/latest/topics/concurrency.html) but would really like to understand how to implement somethng simillar in rust. If it is not too much asking, would you mind sharing an example code snipet?  thanks again! cheers

@geohardtke The GDAL library doesn't let you read using the same handle from different threads.  You should open the same dataset in each of the threads.  For chunking, I would suggest phrasing your processing logic as a "map-reduce" pattern, and using a library such as rayon.  Please see this gist for a sample.

Excellen! Thank you very much for your time @rmanoka !


Issue with Modako docs: Title is misspelled

"Modako" is a misspelling of "Madoko".

https://www.markdownguide.org/tools/modako/
https://www.markdownguide.org/tools/

Oof, thanks! 😳 Fixed in 72a38e8 and b46639c.


Allocation problem solution using CPP and Python

The Allocation problem of Google Kickstart 2020 Round A was solved using CPP and Python.

Give a start if this repo was useful!!


[NodesGroup] Bring to the FN from NBitcoin

This should be applied styling and documentation
Need to change to use the asyncloop
Rename this component to avoid collisions with NBitcoin
Remove not used methods

sorry for asking here... is there a problem accessing slack? it's not loading for me on pc nor mobile

Another one is to replace all the NBitcoin trace with the FN trace (ILogger)

 [NStratis] Bring over and refactor NodeGroup

Refactored and moved over with #740


Sensor Update in Video Analyzer

It would be super useful to have Video Analyzer update a sensor, based on the message prompt - of course besides its current operation of describing the video scene.
Use case : because Frigate keeps generating far too many false positives I feed my Frigate Alerts into Video Analyzer to verify if they really contain images of a person. It would be very useful if then Video Analyzer updates some sensor based upon this verification, besides describing the scene.
This sensor can then be used to steer the further downstream process ...

I think it would make sense to enable video input for the existing data_analyzer. There is a bug in data_analyzer at the moment that needs to be fixed first, but I'll try to include this in the next update.
(Might be a while since I don't have a lot of time right now)

I guess you are the best judge of that. I only hope it is clear that I am looking for a solution to get both things done with just ONE call to llmvision, i.e. setting a sensor AND getting the scene description ...

I agree this would be super useful. Sometimes the notifications are too long to display on iOS and I get sad when I can't read the end. Would also be useful to keep track of the last video or images in camera mode so you can validate that the prompt actually describes what's happening.

I did find a hacky workaround by "taking control" of the blueprint and using an input_text helper to capture the LLM response at the end of the automation, but it's not ideal as you need to create 1 for every camera manually and then go diving into yaml to set it at notification time. Also truncates at 100 characters for some reason even though I've set all my input_texts to max of 255.
In snapshot mode you can grab the image file from local/llmvision/camera_0.jpg and use as a picture card, but Lovelace wants to cache it in the browser (works okay on iOS mobile though) so it doesn't refresh.

For now, use two calls: One to get the description of the video and a second one to update the sensor. Data analyzer should be fixed in v1.3.5.


Ci integrations cli architecture improvements

Fixes #167
Checklist

[x] Documented
[x] Unit tested

New package structure looks great - thanks @vladlaba @dmytroSolidSoftware!


Fix some BotW logic

Adds Kokiri Sword to Dead Hand specifically rather than the well as a whole
Adds Din's Fire option for the map chest

CI was added after this, can you add an empty commit (git commit --allow-empty) to trigger tests?

Thanks 👍


Roll Flutter Engine from 599dbd7382ec to bdf66af6ff0b (1 revision)

https://github.com/flutter/engine/compare/599dbd7382ec...bdf66af6ff0b
2024-03-14<EMAIL_ADDRESS>Roll Dart SDK from abc8988e9dac to b1e31a41fb17 (1 revision) (flutter/engine#51403)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Transactions are not showing in the donation account.

I sent some sats to the donation account. Sending was successful but I do not see the amount on receiver side.
I refreshed it agin still not received the amount.Also I kill the app and tried but didn't work.

Verified this issue on staging app v2.0.5(381)


[BACKLOG-9928] Fixing missing types.

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f core/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/database/Database.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.database.Database

Branch Change: -.13%:small_red_triangle_down:
Complexity Change: -.07%:small_red_triangle_down:
Instruction Change: -.04%:small_red_triangle_down:
Line Change: -.03%:small_red_triangle_down:


[Outlook] (preview) Event-based activation deployment guidance

Docs Build status updates of commit 354109e:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 354109e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/outlook-deploy-event-based-add-ins.png
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9fa5b96:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/outlook-deploy-event-based-add-ins.png
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


⚠ Bofrost.fr has degraded performance

In 19775d8, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 940 ms

Resolved: Bofrost.fr performance has improved in 280ed16 after 15 minutes.


DropModal animation looks a bit odd

The 'Boron is amazing' text seems to flicker as it appears, I am not sure if you can see it in the gif below but if you try it in your browser it should be pretty obvious:

+1

As a side note I'd suggest using actual videos rather than GIFs for things like this. Video files end up smaller, and GIFs usually aren't sufficiently high fidelity for bug reports :)

Good tip @Daniel15, thanks :)

@AnSavvides could you replace the gif with a video when you get a chance.


Update Snakefile

Description of proposed changes
What is the goal of this pull request? What does this pull request change?
Related issue(s)

Fixes #
Related to #
Testing
What steps should be taken to test the changes you've proposed?
If you added or changed behavior in the codebase, did you update the tests, or do you need help with this?
Thank you for contributing to Nextstrain!

This is as intended, if there's a bug, open an issue and I'll try to change docs/fix it ;)


ci: log rustc version in github workflows

I want to be able to confirm that the rust-toolchain.toml is getting correctly picked up by CI. It is importaint that we are using a pinned rustc version rather than the latest stable or nightly to make usre our
MSRV doesn't slip.

The installed rust version is in the log of the setup-rust step. Is there any concern that might not be enough/accurate?

Oh yeah, you're right, my mistake


Support for JSON Schema Validation in Prisma Models for Relationnal databases

Problem
Currently, when working with JSON fields in Prisma with a relationnal database, it is not possible to automatically generate a schema or enforce structure within these fields. This limitation makes it challenging to validate or document the expected structure of the data stored in JSON fields, potentially leading to inconsistent or invalid data.
Suggested solution
Introduce a feature that allows developers to define and enforce a JSON schema directly within the Prisma schema file. For example:
model Example {
  id   Int    @id @default(autoincrement())
  data Json   @jsonSchema("path/to/schema")
}

The @jsonSchema directive would reference a schema definition (either inline or external), ensuring that any JSON data conforms to the specified structure.
Alternatives

Manually validating JSON data in application code, which increases development overhead and risk of errors.

Using a separate database-level constraint for JSON validation (e.g., in PostgreSQL), which may not be portable across different databases.

Additional context

This feature could leverage existing JSON schema libraries for validation (e.g., AJV, Zod).
Example use case: enforcing a consistent structure for settings, configurations, or nested data in applications.
Similar functionality exists in some ORMs and database systems, but having this directly in Prisma would improve developer experience and productivity.

Another option would be to make it possible to use composite types in Prisma Schema (currently only used for nested documents in MongoDB) to be used to describe the types of Json fields, and generate the JSON schema from them in the Prisma Client generator.


Reduce amount done per function

It goes without saying that this is low priority, for now.
Some functions, such as update_salinity mapping, are doing far too much leg work for a single function. Because of this, they require numerous local variable definitions, and require a huge amount branches (nested for and if statements).
Not only does this make them somewhat difficult to test (as the testing suite to hit all possible branches would be huge), it makes them difficult to maintain for anyone approaching the code without a huge amount of background knowledge.
I propose a reasonably simple, but potentially time consuming refactor whereby we split these routines into even smaller chunks, separating out data manipulation areas (ie converting from 0 < x < 360 latitude to -180 < x < 180) and sections after "if" and "for" statements into their own easily testable functions, if they aren't already.
This will

Allow testing to be even more thorough and effective
Make code reading and navigating far simpler
Make changes easier to implement in the long term
Allow for more accurate and precise documentation

The update_salinity_mapping has been refactored to improve the redability of this code.
The refactoring includes:
-moving some of the code out into smaller functions
-reduce the number of variables beeing passed using the dictionaries
More refactoring of other codes associated with update salinity mapping still beed to be done.


refactor: replace react-helmet with gatsby head api

Changes:

...

Type of change

[ ] Bug fix
[ ] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

meta keywords are missing, please check it out, thanks :pray:


add static analyzers.

Q
A

Branch?
v2.0(master)

Bug fix?
No

New feature?
No

Deprecations?
No

Internal Changed?
Yes

Removed
No

Tests pass?
Yes

Doc pr?
No

The submission adds static code analysis and fixes the problems found in static code analysis. Some of the code is not fixed, but error suppression is performed. These problems will be fixed in subsequent pull requests.
After that, you need to check all the code to remove unnecessary content such as: comments.
Incompatible changes:
Enumeration: Change DebugLevels to DebugLevel
Enumeration: Change PadTypes to PadType
App.Handler behavior changes, will not throw an exception
Because of the violation: CA1717

Codecov Report

Merging #237 into master will decrease coverage by 0.34%.
The diff coverage is 97.5%.

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   95.85%   95.51%   -0.35%
==========================================
  Files          30       30
  Lines        2656     2674      +18
==========================================
+ Hits         2546     2554       +8
- Misses        110      120      +10

Impacted Files
Coverage Δ

src/CatLib.Core/Support/Extension/IntExtension.cs
100% <ø> (ø)
:arrow_up:

...c/CatLib.Core/Support/Extension/StringExtension.cs
100% <ø> (ø)
:arrow_up:

src/CatLib.Core/CatLib/ApplicationEvents.cs
100% <ø> (ø)
:arrow_up:

src/CatLib.Core/CatLib/Extension/TypeExtend.cs
100% <ø> (ø)
:arrow_up:

...rc/CatLib.Core/Support/Container/ExtendBindData.cs
100% <ø> (ø)
:arrow_up:

src/CatLib.Core/Support/Guard.cs
84.21% <ø> (ø)
:arrow_up:

src/CatLib.Core/CatLib/Events/GlobalDispatcher.cs
5.55% <ø> (ø)
:arrow_up:

...atLib.Core/CatLib/Events/CatLibDispatcherExtend.cs
90% <100%> (ø)

src/CatLib.Core/Support/Container/MethodBind.cs
100% <100%> (ø)
:arrow_up:

src/CatLib.Core/CatLib/PriorityAttribute.cs
100% <100%> (ø)
:arrow_up:

... and 21 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8829f3c...3669991. Read the comment docs.


Add Support for .mjs .cjs files being picked up as .js files

good point, will add

Piggybacking on this issue.
Similarly, TypeScript added support for .mts and .cts.
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-4-7.html#new-file-extensions
If needed, I can open a new issue to track it separately.

Implemented at<EMAIL_ADDRESS>


Performance

My website reaches 100 everywhere in lighthouse, when I disable giscus.
Is there any way to do one or more of the following?

Run giscus in a different thread
Make giscus caching more agressive
Disable console output of giscus
Make giscus prerender on server
Make giscus load faster

Thank you

Do you have lazy-loading enabled in Giscus (Load the comments lazily)? This will probably help a lot if not done already

This is still an issue:
The main thread was blocked for 1176 ms
When files block the main thread, users are unable to interact with the page content. Typically, parsing and executing large JavaScript files, as well as running long JavaScript tasks can block the main thread and should be avoided. These files had high thread blocking times:

115 ms: https://giscus.app/client.js
91 ms: https://giscus.app/_next/static/chunks/framework-821422f1c2a18a27.js
124 ms: https://giscus.app/_next/static/chunks/main-150a3b60f55bb087.js
1066 ms: https://giscus.app/_next/static/chunks/pages/_app-ca480f785f30358e.js

That is with lazy loading enabled.


Add a trace to list which filters are contained in a channel stack

@drfloob

Please also add the new tracer to doc/environment_variables.md.


Create the targeted element before the script

This doesn't solve the issue with iframe resizing, though it's certainly something that should be done to ensure the element targeted for vega exists.

This looks good to me. However, this changes the output HTML which breaks the test. Can you fix it in StaticHTMLRendererSpec? Thanks.

Codecov Report

Merging #108 into master will decrease coverage by 29.29%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   70.25%   40.95%   -29.3%
==========================================
  Files          27       28       +1
  Lines        1785     3064    +1279
  Branches        6        3       -3
==========================================
+ Hits         1254     1255       +1
- Misses        531     1809    +1278

Impacted Files
Coverage Δ

...c/main/scala/vegas/render/StaticHTMLRenderer.scala
100% <100%> (ø)
:arrow_up:

core/src/main/scala/vegas/spec/Spec.scala
0% <0%> (-69.43%)
:arrow_down:

spec/src/main/scala/vegas/spec/Spec.scala
69.42% <0%> (ø)

...os/src/main/scala/vegas/macros/AliasWithLens.scala
100% <0%> (+100%)
:arrow_up:

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 03286dd...4c554d7. Read the comment docs.

All set, want me to squash this down to one commit?

@rgbkrk Thanks for this PR. Merged into master. Github can squash when merging :)


Updates DataFrame.to_hdf docstring for returned values

This PR updates the DataFrame.to_hdf docstring returned values. Currently, it states None is returned if compute == True, however a list of file names for each partition in the DataFrame is what is returned when compute is True. Reference:
https://github.com/dask/dask/blob/a25c47b3a003ba2aff2cd803f49812ae69ff43bc/dask/dataframe/io/hdf.py#L221-L226

Thanks @jrbourbeau !


[otbn] Alter BN.SID encoding to keep register files consistent

BN.SID is the only instruction (across base and bignum ISAs) that uses different bits for a register compared to all other instructions that have consistent use of the grs1,grs2,grd/wrs1,wrs2,wrd  fields. Being the sole exception for this is a minor pain for the decoder and I wonder if we should alter the encoding so it remains consistent across all instructions?
For instance we could place grs2 in BN.SID where it is now for all the other instruction (bits [24:20]) and move the offset bits and the increment control to bits [11:7].
This does give two seperate kinds of immediate to deal with but we've already got multiple immediate types so less of an issue to deal with compared to introducing multiple sources for a register index.

That sounds like a good idea to me. I guess the tradeoff is just where we put the mux latency: if you reckon that we can absorb it more easily on the immediates, let's do that. Would you like me to push up a patch to the YAML?

Actually thinking about this more it makes sense to do the above altering for both BN.SID and BN.LID. The reason being the D for BN.LID isn't actually a destination register, it's an index into the GPR to give us the WPR write register index. So what is labeled as the grd field in BN.LID makes sense to move to  the grs2 location as it will feed into the grs2 address port on the register file.
This has the pleasing side effect of making the location of the offset and increment control bits consistent again (though not consistent with BN.MOVR.
I'm happy to patch the YAML. Think I'm best doing that locally whilst I work on the decoder changes in case I come across anything else that's awkward, then I can do it all in one PR for the YAML (which I'll submit seperately to the RTL PR).

CC @wallento

And indeed for BN.MOVR the same thing makes sense, the grd is actually doing a lookup on the GPR register file so could also move to the grs2 field. Then the increment control fields will be consistent (as much as they can be anyway, as the BN.LID/BN.SID increment increment a memory address for one whilst the BN.MOVR increments always increment a register index).


Question about configuring padding for code screenshots

I am using a code screenshot plugin in nvim and I’m not sure how to configure the padding for the code screenshot content. Sometimes I want to set the padding to be smaller, or even completely remove it. Is there a way to achieve this configuration? Thank you for your help!

@try-to-fly Hi there, sorry CodeSnap doesn't have such options to config padding.


Segfault for almost any keybase command

my log id: 06af0b5a78b2e577ab87de1c

can't see anything in the logs and no other complaints
this is likely some issue with your local machine
try to reinstall the app?