The label-on-left property of w-switch makes it look like always off

This is a functioning example of w-switch without label-on-left:

But with label-on-left, its behavior is really curious: (the v-model value is changing but just cannot be tell from the look)

The browser is Mozill Firefox 87.0b1 (64-bit). And I can reproduce this on Google Chrome 88.0.4324.182 (Official Build) (64-bit).
I am currently using wave-ui@next <EMAIL_ADDRESS>and the examples (see below) in CodePen seems to be using https://unpkg.com/wave-ui@latest, that is<EMAIL_ADDRESS>Anyway, thank you for your hard work on this brilliant project!
A way to reproduce
Go to https://antoniandre.github.io/wave-ui/w-switch. And from any of the examples, go to a corresponding CodePen editor and make one w-switch label-on-left with an arbitrary label.

Thanks for your feedback and for reporting bug @GuDzpoz!
You're right, and your issue report is very clear & helpful 👍
I'll fix this soon and will let you know.

Hi @GuDzpoz, this is now fixed in version 1.34.2!
Please reopen if not solved. :)


Use geometry.hpp

Use geometry.hpp as the internal representation of geometry in tile.cpp and geometry.cpp, at the bare minimum as the interface to polygon clipping and cleaning, and ideally throughout.

Tippecanoe now uses geometry.hpp to communicate with Wagyu. https://github.com/mapbox/tippecanoe/pull/333


Manage spot market bids with Terraform

Right now the Packet spot market bids are just managed by hand. I think it'd be better and more transparent to manage them via terraform, right here.
Here is a starter config:
variable "project_id" {
  default = "86d5d066-b891-4608-af55-a481aa2c0094"
}

resource "packet_spot_market_request" "req" {
  project_id    = var.project_id
  max_bid_price = 1.80
  facilities    = ["ams1", "sjc1", "dfw2", "nrt1", "ewr1"]
  devices_min   = 1
  devices_max   = 1

  instance_parameters {
    hostname         = "c2.large.arm"
    billing_cycle    = "hourly"
    operating_system = "custom_ipxe"
    always_pxe       = true
    plan             = "c2.large.arm"
    ipxe_script_url  = "https://netboot.gsc.io/hydra-aarch64-linux/netboot.ipxe"
    project_ssh_keys = []
    user_ssh_keys    = []
    tags             = [ "hydra" ]
  }
}

resource "packet_spot_market_request" "req2" {
  project_id    = var.project_id
  max_bid_price = 1.70
  facilities    = ["ams1", "sjc1", "dfw2", "nrt1", "ewr1"]
  devices_min   = 1
  devices_max   = 1

  instance_parameters {
    hostname         = "c2.large.arm"
    billing_cycle    = "hourly"
    operating_system = "custom_ipxe"
    always_pxe       = true
    plan             = "c2.large.arm"
    ipxe_script_url  = "https://netboot.gsc.io/hydra-aarch64-linux/netboot.ipxe"
    project_ssh_keys = []
    user_ssh_keys    = []
    tags             = [ "hydra" ]
  }
}

Setting up the shell.nix, adding it to the build pipeline, etc. is still to-do. We should assume the packet API key will come from the environment, though not sure exactly what the environment variable should be named, it is currently named PACKET_TOKEN.

https://github.com/NixOS/nixpkgs/pull/105790

Ideally this would use terraform's for_each support, and accept config in the format of:
aarch64-bids = [
  { bid = 1; count = 1; }
]

like this:
variable "docker_ports" {
  type = list(object({
    internal = number
    external = number
    protocol = string
  }))
  default = [
    {
      internal = 8300
      external = 8300
      protocol = "tcp"
    }
  ]
}

from https://www.terraform.io/docs/configuration/variables.html


deploy base theme

I believe it would be useful to have the base theme on staging along with the remaining hosted portals. It is useful to have a place to show the various layouts and options in a visual manner. It could be done using hp-demo-fungi but it feels a bit convoluted, since they are being developed in the base theme anyway.
Currently the base theme is hosted on netlify

It's now at https://hp-theme.gbif-staging.org/.  If it's released through GitHub, it will be deployed to https://hp-theme.gbif.org/

the site isn't working anymore. https://hp-theme.gbif-staging.org/styling/ is a server 404 page (not the styling existing page it should be) and not the custom 404 page either.
The data page isn't working. Neither with or without the trailing slash

It seems the cache isn't cleared upon deployment?

Is it just the browser cache? I saw the same effect, but refresh fixed it.
The browser caching is just the default; we don't have Cache-Control: must-revalidate or similar.

I agree - I must have fumbled with the cache. I didn't see it earlier today.

404 pages doesn't work for the base theme


Shared state

Hi,
Great work on Marvin, is a great tool.
Is there any way to share state between steps in Marvin, in a similar way to how it is demonstrated in the following user guide (https://acuminous.gitbooks.io/yadda-user-guide/content/en/usage/managing-state.html).
Thanks

Hi,
No... or better not at the moment...
however it's definitely something useful... if you want to give a try at implementing the feature, i would accept a pull request...
Thanks

Hi,
Apologies for just getting back to you.
Okay, I will look at implementing this feature.
Thanks


how to get the custom attribute in product cart

hi ,   i added a custom attribute in products named discount i want operator to be able to set a discount amount for configurable product as long as its not possible to set an static number for it and i want to label that in front end home page in card i wonder to know how can i access that because i am only having product in the card not the relationship which is the attribute . thanks

Hello @farshadff
From the cart, you can get the model of the product and using this one you can get the attribute family for the given product (e.g $product->attribute_family).
Now using this, you can get all attributes for the given family.

@rahulcs0082  thanks for the , saying from cart means user must add it to card first or i get that wrong ??? because i want to show it in home page products and before user adds them in cart . i will try what you said . thanks

Hi @farshadff
Without adding it to the cart, you can achieve this as products are used to show on the home page using the product model and you can get its attribute family and then all attributes for product.

thanks for the help but 1 more question . i have added one attribute named homediscount to the default family and using this in product now i inter to achive it like below :
    @dd($product->attribute_family->homediscount)

it returns null . i have dont any thing wrong ?

Hi @farshadff
use this one -
$product->attribute_family->getCustomAttributesAttribute()
It will return all attributes for the given product.

I hope above reply solved your issue. I am closing this issue for now. Feel free to reopen this issue if needed.

@jitendra-webkul  thanks for the notice i forgot to update post i got the attribute but actually what i was looking for was the value of that . i am trying to get that and post it here so others may use it


Crash when using ExoPlayer-with-RTMP-and-FLV-seek demo

Thanks for your excellent library and demo, it crashes when I start PlayerActivity and then finish it on and on, and the logs are as below,
com.google.android.exoplayer.demo A/libc: Fatal signal 11 (SIGSEGV), code 1, fault addr 0x18 in tid 2134 (RenderThread)
please tell me how to resolve it.
Thanks again!

Hi @dfpro,
Unfourtunately, I cannot say something specific. I need to test and find the bug you reported. For now, as you know,  it is about memory problem. Library tried to access a memory location that is not allocated or freed.
I hope i will find sometime to fix these issues in near future.
Sorry.

Closing issue if you have any question, please let me know
Btw, there are some updates in the repo, these updates may already fix your issue.


Observability task:allocation center related index data improvement investigation and improvement（可观测性任务配置中心相关指标数据完善调研与完善）

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Describe the proposal

En: The configuration center is mainly used to manage the configuration. Here, users should pay more attention to the history of configuration changes and some information interacted with the configuration center. Of course, it is not only these, which need to be researched and analyzed in actual scenarios. Increase the transparency of dubbo3 applications in the field of configuration center by adding configuration changes and buried points for interaction between applications and configuration centers.
中文：配置中心主要是用来管理配置的，这里用户应该比较关注的是配置变更历史时间和与配置中心交互的一些信息，当然不仅仅是这些，需要在实际场景中调研与分析。通过增加配置变更，应用与配置中心交互的埋点来增加dubbo3应用在配置中心这个领域的透明度。

I want to try

name: dubbo.configcenter.updated.total
tags:

ip
hostname
config_center: apollo/nacos/zookeeper
key:
group:
change_type: added/deleted/modified


MessageThread: Auto-scroll to bottom not working on window resize

Description:
I am using the MessageThread component from @azure/communication-react to display a list of messages in a chat. When the browser window is resized, the component does not automatically scroll to the last message. This creates a usability issue as the user has to manually scroll down to see the new messages.
Steps to Reproduce:
Include the MessageThread component in a React application.
Populate the component with a list of messages.
Resize the browser window.
Expected Behavior:
The MessageThread component should automatically scroll to the last message when the window is resized.
Actual Behavior:
The MessageThread component does not scroll to the last message when the window is resized.
Proposed Solution:
Implement a scrollToBottom method in the MessageThread component that can be called manually to force the scroll to the last message. Alternatively, handle the automatic scroll to the bottom when the window is resized.

Thanks for raising the issue, please let me to look into it and I will get you back :)

Hi @ciaoamigoschat
Thanks for the feature request and I appreciated your patient. We had an internal discussion about this, we think the best solution is to have some code on your side to achieve this behaviour rather than supporting it from UI library side, you can do something like this:

const scrollToBottom = useCallback((): void => {
  if (scrollDivRef.current) {
    scrollDivRef.current.scrollIntoView({ block: 'end', behavior: 'smooth' });
  }
}, []);

const scrollDivRef = useRef<null | HTMLDivElement>(null);

return (

    <MessageThread
      userId={userId}
      messages={[ ... ]}
    />

)

As you can see, all we need to do is to create a
 wrapper with a ref scrollDivRef, then you can have some listeners to wait for window size changes and call scrollToBottom()
in this recording, I made the leave call button to call scrollToBottom()
feel free to let me know if you have any other questions!

Hi @ciaoamigoschat
Thanks for providing your code snippets, it's very useful for me to debug and understand the issue you are facing. I was able to reproduce with the code you provided, when you have 'maxHeight' defined, the entire div is already in max height and there's nothing to scroll. What you want to do is to scroll the the content within the div therefore you need to have ref defined one layer down. Here's an example:
return (

          <MessageThread
            userId={userId}
            messages={[ ...  ]}
          />

      <button id="chat-thread-button" style={{ display: 'block', margin: '30px auto' }} onClick={scrollToBottom}>
        Scroll to Bottom
      </button>

);

Noticing how we have the maxHeight on a base view called view-wrapper and have the ref defined in a sub view called chat-thread-content-view
Let's see it in action:
Give it a try and let me know if that works for you : )

Hi @ciaoamigoschat it has been a while since we hear back from you so I will go ahead to close this issue for now, please reopen this issue at any time if you still have questions :)


CI actions are duplicated for pull requests

The current GitHub actions settings are doubling the CI work by running on the push and pull_request events. See here for an example. The runs on the left nav bar are identical.
One solution is to limit the push event to the master branch.

Good point! Since we only merge the PR if they pass the tests we don't need to re-run them for the push! We should fix this in several repositories.


Node compatibility issues

I didn't find any nod version requirements and that confused me a little when i was migrating my existing project to use Jest

Hmm, honestly I would prefer not to add this message to the Jest code. We used to have it but I don't really like having those deprecation notices in node. I feel like 0.12 and 0.10 are "pretty much dead". Can we only add it to the troubleshooting guide?

removed it :)

@facebook-github-bot shipit


Fix attributes to use node object

Changed the attributes to use the node[] object.

This cookbook is deprecated, use poise-python instead.


Logger outputs all logs to stderr

The current logger output all the log levels to stderr
Please change the logger so everything lower than WARNING goes to stdout instead than stderr.
Thanks <3

This doesn't seem like a bug to me - most cli apps do not split logs between the two outputs. Logs are not intended to be user facing generally, and should not go to stdout.


Spring boot upgrade

Close https://github.com/jhipster/jhipster-lite/issues/6959

Following https://spring.io/security/cve-2023-34035 I think it's the good way to fix it


Please add Cython support

According to https://github.com/microsoft/vscode-python/discussions/20199

Thanks for the feature request! We are going to give the community 60 days from when this issue was created to provide 7 👍 upvotes on the opening comment to gauge general interest in this idea. If there's enough upvotes then we will consider this feature request in our future planning. If there's unfortunately not enough upvotes then we will close this issue.

Could you please clarify what you mean by, "Cython support". Are you just after syntax highlighting?

I am using Language-Cython, this extension already provide syntax highlighting, it would be great if you can provide auto completion and go to definition in addition.

I am using Language-Cython, this extension already provide syntax highlighting, it would be great if you can provide auto completion and go to definition in addition.

Hello, mate, what do you think about this? @janosh

It seems that community does not interest about Cython support, I'll close this issue.
I'm using ktnrg45.vscode-cython BTW, this extension provides some features I mentioned above.


[botskills] CLI operation should fail if it can't find LU file

When --luisFolder is passed to the CLI it uses this to find the LU files. If the CLI can't find the required LU file it should terminate the CLI and not progress.
Right now it continues through the rest of the steps and results in a skill being added but no dispatch intent added for the Skill.
dispatch add --type file --name newsSkill --filePath C:\Users\darrenj\source\repos\4.4\skills\src\csharp\experimental\newsskill\Deployment\Resources\LU\news.luis --intentName newsSkill --dataFolder C:\Users\darrenj\source\repos\build\djvabuilddemo2\djvabuilddemo2\djvabuilddemo2\Deployment\Resources\Dispatch\en --dispatch C:\Users\darrenj\source\repos\build\djvabuilddemo2\djvabuilddemo2\djvabuilddemo2\Deployment\Resources\Dispatch\en\djvabuilddemo2en_Dispatch.dispatch
C:\Users\darrenj\source\repos\4.4\skills\src\csharp\experimental\newsskill\Deployment\Resources\LU\news.luis does not exist.  Please check the file path.

Thanks for reporting this issue! We actually have a few more in radar, I'll be creating an issue with a list of all known issues for tracking them, so you are aware of what is being work on.


[KYUUBI #5030] Support get query id in Spark engine

Why are the changes needed?

This PR aims to support getQueryId in Spark engine. It get sparl.sql.execution.id by adding a Listener.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

why not use operationId directly?

why not use operationId directly?

I apologize for the late reply due to the busy past, It might be better to agree with the executionId on the engine side, for example, to see Spark UI job information by this id.

That means a query without execution or failed has no id?

Thanks, merged to master


Compatibility with Next 12

Is this plugin compatible with Next 12?
I've tried integrating it with:

<EMAIL_ADDRESS><EMAIL_ADDRESS>
and every time I try to start my cypress component tests I get:
ERROR in ./cypress/support/component.ts
Module build failed (from ../node_modules/next/dist/build/webpack/loaders/next-swc-loader.js):
Error: failed to process failed to invoke plugin: Failed to create plugin instance

Caused by:
    Error while importing "env"."__set_transform_plugin_core_pkg_diagnostics": unknown import. Expected Function(FunctionType { params: [I32, I32], results: [] })

ERROR in ./src/components/molecules/RouteSelector/RouteSelector.spec.cy.tsx
Module build failed (from ../node_modules/next/dist/build/webpack/loaders/next-swc-loader.js):
Error: failed to process failed to invoke plugin: Failed to create plugin instance

Caused by:
    Error while importing "env"."__set_transform_plugin_core_pkg_diagnostics": unknown import. Expected Function(FunctionType { params: [I32, I32], results: [] })

You may try old versions instead. This plugin tries to support forward compat to the newer versions of swc, so latest version does not gaurantee to work with old versions of swc.


Inputs randomly ignored if controller is connected to computer

Godot version
v3.4.stable.official [206ba70f4]
System information
OS: Windows 10, cpu: i5-7600k, gpu: nvidia gtx1060 6GB, rendering: GLES3
Issue description
Input gets randomly ignored if controller is plugged in and button will require being released and pressed again to work correctly until it occurs again. In case project was not started with controller plugged in the issue will only appear after pressing buttons on controller. When using only controller the issue will be always present.
Steps to reproduce
Was able to reproduce it in platformer example by making a long straight surface and moving character around.
Minimal reproduction project
No response

Related to https://github.com/godotengine/godot/issues/36396. Can you reproduce this after disabling Vsync in the Project Settings?

yes it still happens both in example and my project(completely stops in my project but that is more likely due to my coding so I doubt it is related to this issue besides the momentary skip that appears in platfromer example where it seems to lose input for just a few frames)

update on the issue, tried to use joystick on controller and it works correctly, only button type inputs seem to cause the issue (keyboard and d-pad)

@nikso14 Please upload a minimal reproduction project to make this easier to troubleshoot.

2d plat kinematic.zip
bit different behavior than in my project (goes directly into idle state in my project) but it still skips inputs with controller plugged in and works fine when not

Found a band-aid type of solution, unbinding joystick from action fixes the issue, was probably causing minor inputs and even if those inputs weren't past the deadzone it seems that godot was picking them up.


Update to latest GitHub library

Hi @mlschechter , go-github recently launched v64.
Can you upgrade your PR?. I'm happy to merge it once that's done.

Hi @mlschechter , go-github recently launched v64.
Can you upgrade your PR?. I'm happy to merge it once that's done.

@migueleliasweb - Thanks for the heads-up! I've updated the dependency and pushed to my branch.
Please let me know if you need anything else!


Option to indent with tabs

Thanks for a superb plugin!
What our team is looking for is an option to select either to indent with spaces or tabs.
I am fully aware of the Spaces vs Tabs war, but the option to select one of them would be awesomeness!
Thanks!

Prettier currently does not support tabs. But we will provide an option for that once :question: prettier supports it.
There is an issue about that in prettier/prettier#908

Closed in favour of prettier issue

There is also https://github.com/passionkind/prettier-vscode-with-tabs

@rhengles I see a fork without any change. Do I miss something?

now there are changes... 🙈

@esbenp please let me know in case you want me to change anything to clearly point out that you're the original author

It is completely fine with me :-)


[Feature] Modify file object (name) before upload to the server

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
I'm trying to add timestamp to the filename, but I can't imagine how to make it work with JQuery FilePond. I do need this because I can't modify the file.source after the file uploaded to replace File to a URL of the uploaded file.
Describe the solution you'd like
I'd like to be able to change the name of loading file (or any other property) in the File object OR be able to replace processed file with URL from backend.
Describe alternatives you've considered
I've tried to use oninit, beforeAddFile, onload (in process) to replace file that is being uploaded.

https://pqina.nl/filepond/docs/api/plugins/file-rename/ ?

This makes sense, thanks! Thought that plugin is for users to rename file before upload. My bad!

But I have to mention, that this worked for the processing file, but when I'm clicking on a file to delete it, I receive in onremovefile old name of the file.

I believe the file is renamed before upload.


esp_log_level_set() can can create multiple inconsistent entries in cache (IDFGH-6947)

Environment

Module or chip used: ESP32-WROVER
IDF version :v4.4-12-ga0404a9ed2

Build System:         idf.py

Compiler version : xtensa-esp32-elf-gcc (crosstool-NG esp-2021r2-patch3) 8.4.0
Operating System:     Windows
(Windows only) environment type: wsl
Using an IDE?: No

Problem Description
Calling esp_log_level_set() can create multiple cache entries in s_log_cache resulting in inconsistent behavior.
Expected Behavior
Log level to match what is set
Actual Behavior
Log level may be an older value.
Steps to reproduce
    {
        char string1[20];
        char string2[20];
        sprintf( string1, "WEB");
        sprintf( string2, "WEB");
        esp_log_level_set( string1, 1);
        printf("Level%s = %d\r\n",  string1, esp_log_level_get(string1));
        esp_log_level_set( string2, 2);
        printf("Level%s = %d\r\n",  string1, esp_log_level_get(string1));
        esp_log_level_set( string1, 3);
        printf("Level%s = %d\r\n",  string1, esp_log_level_get(string1));
        printf("Level%s = %d\r\n",  string2, esp_log_level_get(string2));
    }

Output:
LevelWEB = 1
LevelWEB = 1 (should be 2)
LevelWEB = 1 (should be 3)
LevelWEB = 3

The problem is get_cached_log_level() is checking pointers rather than comparing strings
static inline bool get_cached_log_level(const char *tag, esp_log_level_t *level)
{
    // Look for `tag` in cache
    uint32_t i;
    for (i = 0; i < s_log_cache_entry_count; ++i) {
#ifdef LOG_BUILTIN_CHECKS
        assert(i == 0 || s_log_cache[(i - 1) / 2].generation < s_log_cache[i].generation);
#endif
        if (s_log_cache[i].tag == tag) {
            break;
        }
    }

This should be a (strncmp( s_log_cache[i].tag, tag) ==0 ) like the rest of log.c
GDB output
(gdb) p s_log_cache
$1 =   {{...
 ...
  {tag = 0x3f427b28 "WEB", level = 3, generation = 151},
...
  {tag = 0x3ffe3b04 "WEB", level = 1, generation = 5788},
...
  {tag = 0x3ffe3b18 "WEB", level = 3, generation = 5789},
 ...
  {tag = 0x3f8474c0 "WEB", level = 3, generation = 5790},
...
}

Thanks for bringing this issue up @pickaxprograms!
I think the way the code was supposed to work was:

whenever s_log_level_get_and_unlock is called, look if the cache contains the same tag pointer (get_cached_log_level, which is fast)

if not found, fall back to strcmp-ing the tag with every entry in s_log_tags (get_uncached_log_level, which is slower)
also add the tag to cache via add_to_cache. This should create one entry for each pointer, but since different pointers may point to the same string (in strcmp sense), there could be duplicates. All of them should have the same log level value, though.

whenever esp_log_level_set is called, update the log level in s_log_tags and also update any value in cache which matches (in strcmp sense) the provided tag. So even if the pointer passed to esp_log_level_set is not the same as previously recorded in the cache, the log level for the same tags in cache should still be updated.

Since you observe this issue, there's obviously either a flaw in the above logic or the code doesn't actually do what I think it should. We'll have to look into the code in a bit more detail and try to find the issue.
That said on the two issues you note:

a tag may point to a string on the stack; after the string goes out of scope the tag in the cache would point to some invalid memory
multiple source files may use the same tag, resulting in duplicate cache entries (Note, according to my understanding this should still work correctly, in the sense that esp_log_level_set should update all such entries; that might be something to fix if it doesn't work)

I think both are not likely to happen if you consider regular ESP_LOG usage, which is:

to define TAG statically in each translation unit. Usually you'd have a unique tag for each file / module.
pass the static TAG to ESP_LOGX macros.

Another thing, I recall seeing a comment by someone who pointed out that the whole tag cache solution in esp_log is probably a premature optimization. I can't find the comment right now to give you a link. Indeed in case when we log at least something, a few extra hundred cycles doing strcmp have very little performance impact compared to the thousands of cycles we spend waiting for UART to emit the log output. So perhaps the smarter thing to do would be to remove all the complex cache logic and leave a simple linked list of log levels in place, with strcmp to find find the right tag... Given that most users wouldn't be calling esp_log_level_set too many times, the list is unlikely to get very long.

I uncovered this by wanting to have a command line debug routine to get/set the log level for a passed in tag. When esp_log_level_get() is called with a previously unknown tag, it is added to the s_log_cache with just a pointer to the string that was passed in. If that memory goes out of scope, then s_log_cache will be pointing to memory that is something completely different.

could this possibly cause system crashes? I face crashes that happen immediately on boot, but only if I enable esp_logX statements by calling esp_log_level_set. The dump contains alot of lines, implicating locks and vfs and uart and so on. it makes it very hard to debug when the debug statements themselves are triggering a crash (or causing a double crash). I can't upload logs right now becuase it's on another pc.

The master now has new log Kconfigs that can be used for this use case - https://github.com/espressif/esp-idf/blob/master/components/log/Kconfig.level_settings#L30-L72. The cache can be disabled and use only the linked list. This approach compares the whole strings of log tags for finding the log level.

The master now has new log Kconfigs that can be used for this use case - https://github.com/espressif/esp-idf/blob/master/components/log/Kconfig.level_settings#L30-L72. The cache can be disabled and use only the linked list. This approach compares the whole strings of log tags for finding the log level.

Sounds good, does this apply to version 4.4.2?

No, according to our policy, we backport only fixes not new features. Sorry.


TTL field under Answer is not coming correctly

In dns.Msg I am interested in the Answer part. The TTL value in Answer is always coming as 5 so I suspected an error and was debugging the msg before unpacking the fields and also did a packet capture to compare them.
Added a print in github.com/miekg/dns/msg_helpers.go as follows:
func unpackHeader(msg []byte, off int) (rr RR_Header, off1 int, truncmsg []byte, err error) {
hdr := RR_Header{}
if off == len(msg) {
return hdr, off, msg, nil
}
fmt.Printf("\nInitial Offset: %v Msg: %v", off, msg[off:])

Output:
Initial Offset: 32 Msg: [192 12 0 1 0 1 0 0 0 5 0 4 172 217 194 104 192 12 0 1 ....]
From packet capture:
0000   c0 0c 00 01 00 01 00 00 00 f5 00 04 ac d9 c2 68
www.google.com: type A, class IN, addr <IP_ADDRESS>
Name: www.google.com                                                 -> c0 0c   (192 12)
Type: A (Host Address) (1)                                             -> 0 1        (00 01)
Class: IN (0x0001)                                                           -> 0 1        (00 01)
Time to live: 245                                                              -> 0 0 0 5 (00 00 00 f5) --> ((wrong))
Data length: 4                                                                  -> 0 4        (00 04)
Address: <IP_ADDRESS>                                               -> 172 217 194 104 (ac d9 c2 68)
The pointed time to live field msg value is wrong from packet capture. Please point to any root cause if already known or correct me if I am wrong.

Is this coming up using dns.Client or dns.Server? Do you have a simple reproduction we can test or steps to reproduce (including what DNS resolver or resolver software you're using)?

This is with dns.Client. Please find the setup used at https://github.com/rathega/testdns.git. Follow readme file to repro. Also Added print to display Dns Answer fields along with TTL field.

Tried few things and found this. Looks like the behaviour is not because of the library but because of the platform.
(for any website the same value 5 comes irrespective of wifi network connected to and saw the same behaviour in other Mac machines as well!)
On my Mac, I got the following values for nslookup:
RATHEGS-M-C3XA:app rathegs$ nslookup -debug google.com <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

QUESTIONS:
google.com, type = A, class = IN
ANSWERS:
->  google.com
internet address = <IP_ADDRESS>
**ttl = 5**
AUTHORITY RECORDS:
ADDITIONAL RECORDS:

Non-authoritative answer:
Name:	google.com
Address: <IP_ADDRESS>
On a Lenovo Thinkpad getting ttl values like expected:
C:\Users\rakbansa>nslookup -qa=A -debug google.com <IP_ADDRESS>
Got answer:
HEADER:
opcode = QUERY, id = 3, rcode = NOERROR
header flags:  response, want recursion, recursion avail.
questions = 1,  answers = 1,  authority records = 0,  additional = 0
QUESTIONS:
    google.com, type = A, class = IN
ANSWERS:
->  google.com
    internet address = <IP_ADDRESS>
    **ttl = 286 (4 mins 46 secs)**

So looks to be a different issue and not related to library. Please add in your comments if any and I will close this issue.


OpenTofu Registry

Thanks for this, it's exactly what I need.
Can you please submit to OpenTofu Registry as well as Terraform Registry?

https://github.com/opentofu/registry/issues/1305


Fix 6366 text changes existing screens

Description
Text changes on existing screens
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6366
Screenshots

Labels?


Different limit policy for major and minor versions and Intermediate versions

Hi,
i've added some functionality you may like:
# number of minor versions to be kept or 0 to keep all
maxVersionPolicy.maxMinorVersions=5
# if not zero will be kept intermediate minor versions wich are multiple of the specified value (should be lesser than maxMinorVersions) or 0 to disable
# 	eg. if we have versions from 1.0 to 1.4 (with maxMinorVersions=5 and keepIntermediateMinorVersions=5) and we create vesion 1.5, 1.1 will be deleted instead of 1.0
maxVersionPolicy.keepIntermediateMinorVersions=5
# number of major versions to be kept or 0 to keep all
maxVersionPolicy.maxMajorVersions=5
# like keepIntermediateMinorVersions but for major ones
maxVersionPolicy.keepIntermediateMajorVersions=5

Please take a look; and feel free to merge or cherry pick
I've changed the moduleId, so, if you pick that change remember to first uninstall the previous module before installing this it they will conflict

I am going to bring the add-on to work against 5.1. At the moment these changes conflict with what is in master.  Can you take a look to see what the conflicts are?  Then I can look at merging it in.


Pawn files mismatched as SourcePawn files

I have recently added some of my old plugins I wrote for AMX Mod X (scripting framework used for Half-Life 1/goldsrc engine-based games). The languages these are written in is Pawn, which was later adapted by SourceMod for the Source engine, to make SourcePawn.
Historically, we released the source code for our plugins in files ending with the .sma extension. "sma" is short for "small", which was the original name of the language before "Pawn". Therefore, we sometimes label our language as "Pawn/Small" - the current definition in the linguist language file is "PAWN".
The 2 documented guides for writing Pawn also includes .pawn and .p as possible file extensions for Pawn files. Therefore,
Recommended file extensions

.sma
.p
.pawn

List of repos with sample Pawn files
(I've manually overridden the language through .gitattributes)

https://github.com/ShrmnK/amx_ns_glower
https://github.com/ShrmnK/amx_ns_elechive
https://github.com/ShrmnK/amx_ns_swear
https://github.com/ShrmnK/amx_ns_snipermod

.sma is already recognized as SourcePawn.
There are only 102 .pawn files so that will probably not be enough to add it to Linguist.
.p is already an OpenEdge ABL extension. To add .p to Pawn, you'll have to show that there are a good number of repositories using them. We'll also have to check the detection accuracy. Please see the contribution guidelines on how to add a new extension.

Just gonna note, there are "7,445 code results" for .pwn which is a PAWN file. .pwn is the general extension used for Pawn in SA-MP. We use .pwn for the main scripts and .inc for the includes. The y_testing include in your SourcePawn samples is a SA-MP designated include.

@Crayder .pwn files are already recognized as PAWN files, it is not working for you?

I wasn't saying that was the problem, I was just pointing that out.

Oki :-)


Installing an extension with "requires"

I am not able to construct the "SELECT pgtle.install_extension()" function while specifying the requires clause as a text array. I keep getting all sorts of syntax errors.
Can you provide an example of a simple function call that requires pg_stat_statements?
Might also be useful to add this as an example since all the examples only include the first 3 mandatory parameters.

@MichaelDBA I wanted to let you know that I did reproduce the issue and believe this is a bug.

Great, I mean not great :)

Is there a timeline when this will be fixed?

@MichaelDBA I had to stare at this for a bit because my original guess wasn't the case.
Originally, I thought we did not have the appropriate context switching between file-based/TLE extensions for walking the dependency tree. It turns out we have that.
The problem was actually much simpler, albeit subtler. The way we were copying the file names into the requirements tree was copying some garbage, so the requirement check would work intermittently. This matched the behavior I was seeing. The associated PR fixes this.
In terms of timeline, I'm confident this will be in the next TLE release.

Thanks!


OverscrollIndicator doesn't flatten out when it goes away (it scrolls away instead)

Correct.  The curvature of the circle/arc is supposed to change.  It's pinned at the corners and just flattens into a line, instead of our implementation where it scrolls away.

Fixed.


Allow to use packages like Microsoft.Net.Compilers.Toolset

Currently, the package could use only compilers that rely on .NET Framework 4.7.2 and has tools/csc.exe file like Microsoft.Net.Compilers. This PR enables the package to use compiler packages like Microsoft.Net.Compilers.Toolset which contains .NET Core binaries.
dotnet still is not required to be installed on user machine. Binaries will be downloaded into the Library.
Also, using Microsoft.Net.Compilers.Toolset with .NET Core could fix bug https://github.com/mob-sakai/CSharpCompilerSettingsForUnity/issues/2

Wow! :+1:


fix(runtime-core): unable to set object to dom props

This PR is an alternative to https://github.com/vuejs/vue-next/pull/1092. https://github.com/vuejs/vue-next/pull/1092 probably is quite expensive to do, since a new element is created per prop just to check if that prop is string assignable.
This PR removes casting nullish values to '' for props which I personally think will make more sense.
fix #1049

To me this makes the most sense, the null casting to an empty string seems to be a convenience layer only.
However, this casting can be done in userland when desired, which firstly fixes #1049 and as the alternatives are computational heavy we can remove from the framework.


Duplicate watches still happen

It is not at all infrequent for two people to !!/watch something more or less at the same time in chat, and have two separate branches be created, tested, and merged without conflict but resulting in a CI failure after the merge.
The run-time checking against duplicates doesn't help at all here, because it only examines the current branch, and arguably couldn't know whether another branch is also being created at some point during the time it is processing the command.

Example just now:
https://chat.stackexchange.com/transcript/message/44901622#44901622
I believe all the recent "Duplicate fix" commits by myself are caused by incidents of this type.

Could be fixed with a lock - only let one operation on each list happen at a time

Or store a global list of pending watches and refuse to watch if the item is on the list (this has the benefit of getting cleared on reboot, when the changes have been successfully pushed)

Can be easily solved by checking duplicates in branch master instead of deploy.


Modified: README.md improvements. renamed png files

modified: README to display themes under the heading of "HTML Reader" or "LATEX Document"
fixed: README typos and text flow improvement
fixed: academy link in README

hmmm. yes. yes. looks good.


fix cloudfiles checksum

rackspacecloud_files resource is currently using two different hash functions for the current file and new file so the compare is always different.
This makes them the same so the compare works.

I would have pushed us to use sha 256 since Chef's file resource uses that, but I noticed this comment:
# Use md5 checksums because CloudFiles etag is md5

So it sounds like we should just make sure we use md5 everywhere. This still might surprise folks expecting us to follow more Chef-like conventions.


linux_xanmod, linux_xanmod_latest: 2024-08-29

Description of changes
main variant tested
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I can do create a manual Backport in the evening unless someone else wants to do that.


Switch CBuilder to return assets and dependencies

Thanks for your contribution! Please replace this text with a description of what this PR is changing or adding and why, list any relevant issues, and review the contribution guidelines below.

[ ] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

coverage: 90.815%. remained the same
when pulling 0f0a9b8855982b2ac87553c3fe722932c89788f8 on switchNativeToolchainToLink
into 1a6faf502c01c598ce8ed6c77ea22c29774dbf34 on main.

Closing out old stale PRs.


Create dynamic dimensions tokens

This PR adds new dynamic dimension sizes to our global tokens and replaces all typographic and typography-dependent sizes (i.e. paddings, icon sizes) by the new dynamic ones, expressed in em.
A mdx page to viz the new global tokens has also been added, which entailed having to restructure section 06 of the Storybook doc/src folder.

All necessary changes seem to be correctly reflected on Storybook as intended :D


upload folder, some files are missing

I just uploaded a folder with
gdrive-cli upload -f name\ folder/ -p<PHONE_NUMBER>abcdef
Of the 7 files in folder, 2 are missing.
Is there a way to understand why? A log file, a verbose mode?
thanks

I also had this, but the files would spontaneously appear after a while. May be an issue with GDrive itself? http://stackoverflow.com/questions/13010191/uploaded-file-sometimes-doesnt-appear-on-the-google-drive-web-page

I can confirm , this is an issue with google drive itself , not gdrive.


[clkmgr/prim] Make frequency measurement disable more robust

handles part of #13489

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

@matutem can you take this and have a look with your test? hopefully the enable / disable is more robust now, and the assertions also go away.


Broadcaster should provide account sequence by default

See this comment for full details.
After sdk v0.47.1 it is required to provide the account sequence.
We have a temprorary hack to add it in ibc-go, but this makes the most sense to be included in this function.

Closing via: https://github.com/strangelove-ventures/interchaintest/pull/533


Reaction should have key and label to allow mutable text on tooltip to support i18n

Is your feature request related to a problem? Please describe.
Currently, the reactions in ReactionBarSelector use a label in the tooltip. If I want to use i18n, it is impossible to have a mutable 'label' and a 'key' to determine in a selection event which emoji was selected.
Now:
   const reactions: Reaction[] = React.useMemo(
        () => [
          {
            label: 'woozy',
            node:
🥴
,
          },
          {
            label: 'happy',
            node:
🙂
,
          },
          {
            label: 'exploding-head',
            node:
🤯
,
          },
        ],
        []
      )

So, the content of 'label' is displayed, and I need to display a different text depending on the language, and I need a label to be a key.
Describe the solution you'd like
A reaction should have this structure, e.g.:
  const reactions: Reaction[] = React.useMemo(
      () => [
        {
          key: 'woozy',
          label: i18n.translate('woozy', i18n.language)
          node:
🥴
,
        },
        {
          key: 'happy',
          label: i18n.translate('happy', i18n.language)
          node:
🙂
,
        },
        {
          key: 'exploding-head',
          label: i18n.translate('exploding-head', i18n.language)
          node:
🤯
,
        },
      ],
      [i18n.language]
   )

And onSelect should get key as a param and not label.

Hi @mastepanoski. Thank you for reporting the issue and for using the library.
This is a great idea. I can make an update to the package by the end of the week.

This was published in v0.9.0. Thank you!

Please let me know if there are any issues.

Perfect. Ok, I will let you know.


Python to PyPI

Will pythonic version of this library be published on pyPI?

I'm not really a python guy.  A PR is always welcome :smile:

+1 for this. As long as there's no disagreement with it, I'd be happy to help prep this! Is there an account management system to do this? We'd need a libcsp PyPI.org account, of course.
Is the Python code versioned alongside the main libcsp, or should it get its own version numbers?

I think we should aim for a Native Python implementation of CSP instead.
Lots of what is going on inside CSP, the router, the many network interfaces, the low level OS stuff, is really gone when inside python context. Most likely the main interface used with  python is going to be ZMQ which is already native python as well.

I think this is good enough for now. Really targeting ease of use for ground stations would be huge

A more immediate alternative could be to create a pyproject.toml file, allowing the repository itself to be pip3 installed.
I have tried this myself with the Space Inventor repository PyCSH, although here we mostly use an ./install script to pip3 install a local build. But a GitHub URL can also be used.

That is indeed the first step!

I've made a PR for the pyproject.toml part:
https://github.com/libcsp/libcsp/pull/714
I took a creative liberty by bundling libcsp.so into the pip wheel,
but I think this will be greatly appreciated when users can install directly from PyPI without needing to install libcsp on their system first.

Would be awesome to get a backport to v1.6 for this, and have that lineage published to PyPI!

I think you can configure the newest commit to CSP 1.6, right?

I'm just not too sure if all the other changes in that PR make sense to include in the backport; can you advise? Are any of them breaking changes?

I don't know whether the bindings API has seen breaking changes since 1.6 (outside of this PR), where the .pyi file would need to be backported.
I will probably have to admit to one breaking change. I've left a comment on the PR mentioning that marking pycsp_buffer_get() with METH_NOARGS is a breaking change, as some users may still call it with the unused size argument.
So I will probably change the function to handle that one argument specifically, letting other invalid calls raise exceptions.
I don't intend to make any breaking changes by bundling libcsp.so, as symbols from either the existing namespace or system libraries should take priority when importing/linking. However I am still testing this, in the meantime I would love some discussion as to what others think of this approach.


Line graph isn't fix it the chart in sometime

Hello,
I have a line chart and I want it will display on the control with default zoom level, no user's interaction.
But sometimes, the chart seems bigger than the control so it will be cut off and this is not good.

Can anyone give me some idea to fix it?
Thank you

give some line width it will work.
Line line = new Line(values);
line.setColor(Color.parseColor("#0abe51")).setCubic(true);
    line.setStrokeWidth(2);

https://github.com/lecho/hellocharts-android/issues/446
I fixed it here


Upgrade packages to address Vanta vulnerabilities

Description of the change
Addresses this vulnerability:

Also bump a few other packages to match outline2.0
Type of change

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

Related issues

Fix #1

Checklists
Development

[ ] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[ ] All tests related to the changed code pass in development

Code review

[ ]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

LGTM. Will there be a companion PR at some point to replace all usages of env-utils in outline2.0 with this specific version? That would be really helpful, I think.

Yep, I'll work on that as part of my PR to resolve the Vanta vulnerabilities.


Vertical scrolling is not working on iOS 13.1

Describe the bug
After updating my iPad Pro to version 13.1 vertical scrolling is not working when touching the timeline. If I scroll/touch on the left sidebar scrolling works as before.
Tested in safari and chrome on iOS 13.1
To Reproduce

Open timeline in safari or chrome on iOS 13.1
Touch on the timeline (not the sidebar) and try to scroll up or down
Timeline is not moving

Expected behavior
Timeline moves up/down on touch scroll
Library Version
Latest
tablet :

OS: iOS 13.1
Browser chrome or safari
Version latest

update:
sorry I talking about iOS 13.1

@lars-ewers sorry dude I don't own an ipad. I can't debug this

I just found out, if you disable "touch-action: none" scrolling is working as expected.
But in this case you are not able to move Items around on a touch device
btw: scrolling in x direction is working fine..
Anyone else experiencing this behavior?

@lars-ewers I have no clue sorry

I also have this issue. I can scroll using the right scrollbar if i press on it. but other than that I cant scroll normally by swiping over the screen.
Any solutions yet?
Also iOS 13.1

Hi, no solution so far.
In my case scrolling on mac works as expected. But still touch scrolling on i0S only works on the left sidebar.

I probably had the same issue, vertical scroll (touch) on left sidebar normal, on timeline it gave heavy flickering both in chrome and safari on iOS 14.2. Found a temp-fix, but with this, the sidebar scroll doesn't work.
Added this to my CSS:
.react-calendar-timeline .rct-scroll {
overflow-x: hidden !important;
}


I am Django Developer

What is your used technologies on the projects?

We primarily use nano-vector database and networkx for data storage and retrieval. The rest of the project mainly involves communication with the LLM.


[JSONRPC] Separate data fetching from RPC Server implementation

Motivation
Streaming RPC uses many of the same data fetching as the json RPC server: by separating out pure data fetching vs server implementation, these can be reused (and also tested much more easily)
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
This is a refactor; existing tests

/land


Added support for Orion Browser

Added new icon, edited the workflow to add Orion as a new Webkit browser, and tweaked focus-tab-webkit to support multiple browsers.
There are a lot of changes in info.plist, which I think is auto-generated by Alfred based on the Workflow visual editor. Hopefully those changes are good.
Please take a look and let me know what you think.

Solution for #38


cLvgl: add template to run lvgl widget demo

Provides a template to run LVGL widget demo.

Please I have a question. Right now, the template is almost ready. The only last problem I have is that I must embed in a way LVGL repository. I don't think it is a good idea to embed it in the repository and I'd like to know the best way to get this library available to build the application. Any idea ?

Hey @EDGEMTech-GabrielC I addressed some of the comments above on https://github.com/microhobby/vscode-torizon-templates/tree/add-cLvgl-template , so take a look and if these make sense for you please add the commits on your branch, or give me the access to push it to your repo. Thanks!

I pushed the changes you mentioned, I also gave you access to the repository if you want to change something else.

Hello @EDGEMTech-GabrielC ,
I tried to run the Local AMD64, which runs the project for local debugging on my host, but got the following error:
Error: cannot open framebuffer device: Permission denied

As the template uses framebuffer, should it be able to have local debugging?

@andreriesco this LGTM

Actually needs some small fixes and bumps of tags. I will add the commits with the fixes

Actually needs some small fixes and bumps of tags. I will add the commits with the fixes
Could we accept this and then you open a new MR with these small fixes?

I think it's better to wait for the Torizon 7 to be merged into dev before merging this one, to make it available on Torizon 7

I think it's better to wait for the Torizon 7 to be merged into dev before merging this one, to make it available on Torizon 7

Ok then.

Hello @kisvegabor,
Torizon 7 is already available for the OS part. About the IDE templates for Torizon 7, the PR is currently in development, but the release will be at most next month.

Got it, thank you!

Hello @EDGEMTech-GabrielC @kisvegabor ,
I have tested it on Torizon OS 7 with the bumped tags and it's working both for arm32 and arm64.
However, I am having one last issue. I need to something printed on the Debug Console for the automated tests. I tried adding this line printf("Hello Torizon");, but it's not printing it to the Debug Console. Do you guys have an idea on what might be the issue?

Great news!

I tried adding this line printf("Hello Torizon");, but it's not printing it to the Debug Console. Do you guys have an idea on what might be the issue?

@EDGEMTech-GabrielC can comment on it in January.


test(frontend): test two phase approx percentile with group key is banned (#18085)

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Closes https://github.com/risingwavelabs/risingwave/pull/18085.
Cherry pick.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Need one more review


included app banners &  modified proj-details section

Could you just give us a screenshot of the changes like before and after. Then it could be easier to merge the changes :)

Added proper Footer section and Add-button alignments.


using ImageMagick

I was just wondering why image-magick (maybe through gm node module) wasn't used for this project since it's built in with Amazon linux image that lambda runs? That will avoid the extra complexities of having to build this in a docker container..

Literally no good reason. We picked this module for arbitrary reasons when we put this together during a hackathon. If there's a better option, patches welcome! 😄


Improve email change flow UI: Added explanation, moved confirmation notice

Changes done:

Email Change Modal
Added an explanatory line at the top of the modal: "You will receive a confirmation email at the new address you enter."
Confirmation UI
Moved the confirmation notice from the section heading to a position just below the email field, keeping the original text unchanged.

Fixes parts of: #31975

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Closing this PR, as it does not implement the desired changes. Both of the lines of text should be plain text (not banners), and the location of the confirmation text is wrong. Thanks for giving it a try, @kartikbhtt7 !


ollama cannot running

What is the issue?
ollama list
Error: something went wrong, please see the ollama server logs for details
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
0.1.46

What are the logs contents? (check ~/.ollama/logs/server.log)

I re-installed ollama.app, now it's working fine.

Happy to hear the reinstall cleared up whatever went wrong.


Make it possible to query a type literal/class reference

TL;DR do we want a ClassReference.uri field? or maybe ClassReference.name to be a QualifiedName?

I'm currently building a macro that will take in a specification as an argument and generate a client/server based on this specification
the way it's going to be used will be something along the lines of:
@Client(BaseService)
class BaseClient {}

which generates the following:
{
  "type": "ConstructorInvocation",
  "value": {
    "type": { ... },
    "constructor": { ... },
    "arguments": [
      {
        "type": "PositionalArgument",
        "value": {
          "expression": {
            "type": "TypeLiteral",
            "value": {
              "typeAnnotation": {
                "type": "NamedTypeAnnotation",
                "value": {
                  "reference": {
                    "type": "ClassReference",
                    "value": {
                      "name": "BaseService"
                    }
                  },
                  "typeArguments": []
                }
              }
            }
          }
        }
      }
    ]
  }
}

As you can see, having a name only is not enough to query BaseService since at that moment BaseService is not loaded/resolved (builder.model.node['types'])
"types": {
    "named": {
      "dart:core#Object": { ... },
      "dart:core#Null": { ... },
      "dart:async#Future": { ... },
      "package:bar/test.dart#BaseClient": { ... }
  }

I do actually have a local fix for that, that adds uri to ClassReference (I'm not sure but I feel like it makes the most sense to put that information there):
                  "reference": {
                    "type": "ClassReference",
                    "value": {
                      "name": "BaseService",
                      "uri": "package:bar/test.dart"
                    }
                  },

and I'd love to submit a PR, but I'm not sure if that's how the model should look like? another approach would be to make name a qualified name, like so:
                  "reference": {
                    "type": "ClassReference",
                    "value": {
                      "name": "package:bar/test.dart#BaseService"
                    }
                  },

So my question is, do we want a uri field, name to become a QualifiedName or maybe something else?
Thanks in advance!

Thanks for filing.
I think we're not ready to get into this level of detail here yet, there are bigger questions about the macro metadata model that we need to figure out first.


Introducing new code that is potentially thread unsafe

Signed-off-by: Nikunj Vaidya Macbook<EMAIL_ADDRESS>

Closing the request without a merge.


Make Windows crash dump checks asynchronous.

What does this PR do?
This PR modifies the Windows crash dump checks to be asynchronous.
Motivation
WIndows crash dumps may be very large (example: 128 GB) and it may take a significant while (e.g. minutes) for the Windbg subsystem to process a dump. The checks for crashes (from agent to system probe) will often timeout before the processing completes. This PR modifies system probe's crash reporting endpoint to return a "busy" status while the crash dump is still being process. Once the processing and report is ready, the endpoint returns a "success" status with the original crash information.
Describe how to test/QA your changes
A. Manual

In a Windows VM, make sure there are no existing crash dumps (e.g. C:\Windows\MEMORY.DMP)
Start system probe.
Query the endpoint with:  NamedPipeCmd.exe -method GET -path /windows_crash_detection/check -quiet.  It should return something like: {"statuscode":0,"errstring":"","filename":"","dumptype":1,"datestring":"","offender":"","buckcheckcode":""}
Configure Windows to generate full crash dumps.
Use a tool (e.g. NotMyFault.exe) to cause a bugcheck, which should generate a crash dump (e.g. C:\Windows\MEMORY.DMP)
Wait for the VM to restart.
Start system probe.
Query the endpoint with:  NamedPipeCmd.exe -method GET -path /windows_crash_detection/check -quiet.  It should respond with status 1, like the following: {"statuscode":1,"errstring":"","filename":"C:\\Windows\\MEMORY.DMP","dumptype":1,"datestring":"","offender":"","buckcheckcode":""}
Wait for a minute or so and query the endpoint again.  It should respond with status 0, like the following: {"statuscode":0,"errstring":"","filename":"C:\\Windows\\MEMORY.DMP","dumptype":1,"datestring":"Tue Oct  8 11:53:30.883 2024 (UTC - 7:00)","offender":"myfault+0x163e","buckcheckcode":"000000C2"}

B. Run the following unit tests: wincrash_detect_windows_test.go, crashparse_test.go.
Possible Drawbacks / Trade-offs
The caller on the crash reporting endpoint previously received a full response with single query. as long the crash dump was not too large.  Now the caller must make at least 2 queries because the first one always returns "busy" status.

/merge


Support for ipycytoscape in Google Colab

Hello,
I've been hoping to use ipycytoscape in Google Colab for a while, and it looks like it's finally possible now that Colab has started supporting custom widgets. Specifically, as @blois explained, "Colab recently started exposing Jupyter Comms channels so custom visualizations can use them for communicating with the kernel (see https://colab.sandbox.google.com/notebooks/snippets/advanced_outputs.ipynb#scrollTo=YOd4khppTskD)." However, because the required jupyter-cytoscape extension is written for JupyterLab, some changes need to be made for it to work with Colab. I'm not sure exactly what those are, but I'm hoping that between the ipycytoscape team and @blois we can figure them out, possibly resulting in a new extension colab-cytoscape.
More info on portable widgets for Colab here
Related to colabtools/issues/1524. Possibly tangentially related to #80.

Hi @rchurt, I agree that this would be great! Unfortunately, my reading of the linked issues is that this is more of a google colab problem than an ipycytoscape problem. I think the suggestion in those threads is that with the new colab support for comms you can work around colab not supporting widgets by injecting javascript via IPython.display.javascript. However, I think you'd need to inject quite a lot of javascript and I don't think there's a clean way to do that.
I definitely could be wrong/misreading things here though. I'd be happy to be shown how this is in fact possible.

It depends on how much Javascript is needed- some libraries will directly inject the javascript using IPython.display.javascript as you mention- it may not be elegant but it does ensure the notebook is self-contained and can be viewed by all users at a later time. Other libraries such as Bokeh and Plotly use CDNs to host versioned copies of the libraries. They typically offer some configurability to allow loading resources locally for users sensitive about remote resources and CDNs for users who may want their notebooks to be more easily viewed later.

jupyterlab extensions that dont rely on jupyterlab-specific windowing or apis should work out of the box, which includes most ipywidgets based libraries.

e.g. ipycytoscape exports jupyter notebook compatible js assets here: https://github.com/QuantStack/ipycytoscape/blob/master/webpack.config.js#L19

in general though, google colab is a proprietary product and if the onus is really on google to support what the open source community deems as standards

I am definitely open to suggestions on improving the situation- some background can be found at https://github.com/nteract/nes/tree/master/portable-widgets.
I'm not sure what the standards are here- I don't think there's any feasible way to make JupyterLab extensions work outside of JupyterLab. If there is no desire to support anything other than JupyterLab then that is fine. You can see my goals in the background section of the above link and I do believe that those goals would be beneficial to everyone.
Also take a look at https://github.com/Quansight-Labs/jupyter-output-spec which I've been helping out on as well and is an approach for the issue. I did a prototype implementation of this for Jupyter Lab at https://github.com/blois/js-module-renderer.
The basic model that we've been using is to display visualizations with HTML/JS then if comms are available to enable bi-directional kernel communication. The comms APIs in Colab are intended to be similar to those in Jupyter Notebook to allow more portability, but unfortunately they cannot be identical since there is a lot of code which looks for window.Jupyter and when that is defined will make broad assumptions about executing within Jupyter Notebook.

Thanks for all the helpful discussion! I think this link from @blois describes the problem well--ipycytoscape folks, what's your current approach to offering support for multiple platforms (nteract, voila, nbconvert, etc.)?

I'm not sure what the standards are here- I don't think there's any feasible way to make JupyterLab extensions work outside of JupyterLab.

@blois
You mean that extensions that run in the notebook can be used in colab but the ones that only run on jupyterlab can't?
My take on this is that I'd like to have ipycytoscape supported to colab because I think it'd be beneficial for the community but I have to agree with @timkpaine, colab is a proprietary product and the onus to support it belongs to google. Also, if I was to make these changes I'd have to learn a few things about the low level functioning of Jupyter, so it wouldn't be trivial for me.
This seems very interesting: https://github.com/blois/js-module-renderer
What's the state of it? Does it  convert some specific kinds of extensions automatically? Or does it always require human input? Also, this is converting to other rendering engines or just colab?

what's your current approach to offering support for multiple platforms (nteract, voila, nbconvert, etc.)?

@rchurt
Never tried with nteract nor Jupyter sphinx, but ipycytoscape works fine with voila and nbconvert because I tested it.
If either you or @blois are interested in making this port you're welcome to ask me questions here or in the gitter chat.

Hey @marimeireles, I am the original author of https://github.com/Quansight-Labs/jupyter-output-spec which @blois's work builds on.
If you are interested in helping move this forward, I am happy to try to explain here the state of things or get on a call. I think it would do a lot to help the whole Jupyter community by improving interop between the frontends and making them less siloed. However, I don't have the bandwidth right now to work on it alone.

Nteract, voila, Sphinx, nbconvert,  jupyter, and jupyterlab all work out of the box with the widget cookiecutter assuming you don't need to do anything super custom. I would say this issue should probably be closed as it doesn't make too much sense to build custom functionality here instead of in the cookiecutter that we all rely on underneath.

https://github.com/jupyter-widgets/widget-ts-cookiecutter
https://github.com/jupyter-widgets/widget-cookiecutter

I would say this issue should probably be closed as it doesn't make too much sense to build custom functionality here instead of in the cookiecutter that we all rely on underneath.

I very much agree with this. Unless @marimeireles disagrees I think we should close this and move this discussion over to the jupyter-widgets organization. As for Ipycytoscape support, I feel comfortable saying that it will continue to be updated to reflect any changes to the cookiecutter any solution will ultimately end up here as well.

If you are interested in helping move this forward, I am happy to try to explain here the state of things or get on a call. I think it would do a lot to help the whole Jupyter community by improving interop between the frontends and making them less siloed. However, I don't have the bandwidth right now to work on it alone.

I'm all for more widgets in more places! @saulshanabrook  I think a good place to discuss that may be the jupyter-widgets meetings: https://github.com/jupyter-widgets/team-compass#bi-weekly-team-meetings-every-other-week

I feel comfortable saying that it will continue to be updated to reflect any changes to the cookiecutter, so any solution will ultimately end up here as well.

Yes, seems like a good one :)


NioDatagramAcceptorExIT.shouldConnect not all expectations were satisfied

Log file
Failed tests:
  NioDatagramAcceptorExIT.shouldConnect not all expectations were satisfied
expectations:
  ! expected once, never invoked: ioHandler.sessionCreated(instance of org.kaazing.mina.core.session.IoSessionEx)
  ! expected once, never invoked: ioHandler.sessionOpened(instance of org.kaazing.mina.core.session.IoSessionEx)
  ! expected once, never invoked: ioHandler.messageReceived(instance of org.kaazing.mina.core.session.IoSessionEx, <HeapBuffer[pos=0 lim=4 cap=4: 74 65 78 74]>)
  ! expected once, never invoked: ioHandler.sessionClosed(instance of org.kaazing.mina.core.session.IoSessionEx)
what happened before this: nothing!

Tests run: 39, Failures: 1, Errors: 0, Skipped: 2

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-failsafe-plugin:2.18.1:verify (default) on project mina.netty: There are test failures.
[ERROR]
[ERROR] Please refer to /home/travis/build/kaazing/gateway/mina.netty/target/failsafe-reports for the individual test results.
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <goals> -rf :mina.netty
travis_time:end:2681fc52:start=1433864909750788196,finish=1433865021367869126,duration=111617080930

The command "mvn -q clean verify" exited with 1.

Done. Your build exited with 1.

Closing this issue, as it is a duplicate of #153


[FluentDatePicker] No Date Formatting Option found

Hi, This is Yahya Patel. Just dived int Blazor a week ago and got in love with Fluent UI 💙💙💙. I am finding a way to change date format which currently is MM/dd/yyyy but I need to change it dd/MM/yyyy. Could anyone help? Thanks

Hi Yahya,
Welcome to Blazor and the Fluent UI library.
The component follows the settings of the client (Culture) it is running on. Is your local setting MM/dd/yy?
If you want to change it for the browser, you need to change the locale/Culture for the page/site. See https://learn.microsoft.com/en-us/aspnet/core/fundamentals/localization?view=aspnetcore-70 for more information

Just checked it, it's possible but I want just one date picker to be in different format. How could I do that?

You can force the culture of this component using the Culture property.
Example for setting the format and language English-UnitedState :
<FluentDatePicker @bind-Value="@SelectedValue" Culture="@(System.Globalization.CultureInfo.GetCultureInfo("en-us"))" />

@dvoituron Forcing the culture with Culture property doesn't seem to help with the date format, it only affects calendar (starting day of the week).
For example, if culture is set to sr, date should be in dd/MM/yyyy format:
<FluentDatePicker<EMAIL_ADDRESS>    Culture="@(System.Globalization.CultureInfo.GetCultureInfo("sr"))" />

However, the date is still in the MM/dd/yyyy format because en-US culture is set in the browser:

It should be possible to force the date format in the date picker independent of the browser culture.

@dvoituron Forcing the culture with Culture property doesn't seem to help with the date format, it only affects calendar (starting day of the week). For example, if culture is set to sr, date should be in dd/MM/yyyy format:
<FluentDatePicker<EMAIL_ADDRESS>    Culture="@(System.Globalization.CultureInfo.GetCultureInfo("sr"))" />

However, the date is still in the MM/dd/yyyy format because en-US culture is set in the browser:

It should be possible to force the date format in the date picker independent of the browser culture.

I tried Germany (de) culture and it is showing the correct dd.MM.yyyy format, so it looks like it is not working for Serbian (sr) culture. Weird, but this will do the job.

@jova1111 I've tried this sample and the result seems fine.
<FluentDatePicker AriaLabel="To" @bind-Value="@SelectedValue" Culture="@System.Globalization.CultureInfo.GetCultureInfo("sr")" />


Provide pinvoke/golang-like interop with C

Goal:
C
// gcc -shared -fPIC library.c -o locationOnDisk.so
int my_c_function(char* bla){
      printf(bla);
      fflush(0);
}

Dart
@CLibrary("locationOnDisk.so")
external int my_c_function(String bla);

void main() {
   my_c_function("HI"); // works!
}

When the dartvm starts up / encounters an external function it should look for a matching function that it can dynamically load from a .so on disk. Ideally the native .so should not need to include any dart specific headers or libraries, it should be pure C.
Please advise on whether this is wanted and how it should be implemented.

dart:ffi provides interop with C.


Saf variable names

Displays netCDF4.Variable long_name instead of the key.

@eardkdw it looks like #205 has resulted in a minor conflict, I can attempt to resolve it if you like?

@eardkdw I like the idea of a State hash function

@eardkdw this looks good, I'll merge it as soon as the CI checks pass

Let's me use lru_cache on the (now very expensive) saf.image function.
Still gets called four times, mind. Better than eleven, though.
On Tue, 12 Nov 2019, 10:47 andrewgryan<EMAIL_ADDRESS>wrote:

@eardkdw https://github.com/eardkdw I like the idea of a State hash
function
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/informatics-lab/forest/pull/206?email_source=notifications&email_token=ABQLMNA6UNPVAVHZLPBMQDLQTKCVDA5CNFSM4JKY5PVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDZ2YIY#issuecomment-552840227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQLMNEI54Z2LX3LKCHKPMTQTKCVDANCNFSM4JKY5PVA
.

Are you finished working on it? Is it ready to merge?

It should be mergeable now, yes.


Empty list in query params

Hi! I noticed a bug where streamlit-permalink is unable to distinguish between "the chosen value is an empty list" and "there is no chosen value" for list-based widgets like multiselect. This is fine so long as the default value is also an empty list, but causes issues with a non-empty default value.
For example,
import streamlit as st
import streamlit_permalink as stp

got = stp.multiselect("Multiselect", ["A", "B"], ["A"], url_key="multiselect")
st.write("Got:", got)

I open the app at http://localhost:8501/ and see A selected. If I manually click on B the URL changes to http://localhost:8501/?multiselect=A&multiselect=B as expected. If I reload the page my selections will be preserved.
However, if I then clear the multiselect (such that nothing is selected) the URL goes back to http://localhost:8501/. If I reload the app at this point it'll go back to the default entry ["A"], losing my selection.
I believe we need something like multiselect=_EMPTY_LIST in the query params to address this. I'm happy to contribute the fix for this, and wanted to get your feedback before putting up a PR.

Sounds good!

@franekp https://github.com/franekp/streamlit-permalink/pull/9 is ready for your review when you have time, thanks!

Done. New version here: https://pypi.org/project/streamlit-permalink/0.5.2/


dspyComponent module only allows for a dspy.Predict module as input

I was following the llamaindex cookbook and reached an error for the code
dspy_component = DSPyComponent(dspy.ChainOfThought(GenerateAnswer))

The error trace showed that there was a pydantic validation error for "predict_module". That's because the DSPyComponent object only accepts a dspy.Predict(signature) input instead of a ChainOfThought object as listed in the cookbook.
I didn't have any errors running this cell before until I upgraded my dspy-ai version. Perhaps loosen the allowable classes for QueryComponent (the parent of DSPyComponent)? Or is the team's recommendation to use dspy.Predict instead?
Validation error trace
---------------------------------------------------------------------------
ValidationError                           Traceback (most recent call last)
Cell In[38], line 1
----> 1 DSPyComponent(dspy.ChainOfThought(GenerateAnswer))

File /opt/anaconda3/envs/llm/lib/python3.12/site-packages/dspy/predict/llamaindex.py:185, in DSPyComponent.__init__(self, predict_module)
    180 def __init__(
    181     self,
    182     predict_module: dspy.Predict,
    183 ) -> None:
    184     """Initialize."""
--> 185     return super().__init__(
    186         predict_module=predict_module,
    187         predict_template=signature_to_template(predict_module.signature),
    188     )

File /opt/anaconda3/envs/llm/lib/python3.12/site-packages/pydantic/v1/main.py:341, in BaseModel.__init__(__pydantic_self__, **data)
    339 values, fields_set, validation_error = validate_model(__pydantic_self__.__class__, data)
    340 if validation_error:
--> 341     raise validation_error
    342 try:
    343     object_setattr(__pydantic_self__, '__dict__', values)

ValidationError: 1 validation error for DSPyComponent
predict_module
  instance of Predict expected (type=type_error.arbitrary_type; expected_arbitrary_type=Predict)

Ah good catch. It looks like this some recent updates (switching CoT to become a module) broke compatibility with the llamaindex integraton.
Maybe we should pin the notebook to an old DSPy version?

Sure! I was on dspy-ai 2.4.11 before this I think - the update just before!

what is the status of this? If this isn't getting fixed, why leave the tutorial up so people can waste their time?

Hi, can someone please update this? Or please make sure to remove the tutorial/example notebook


Add in projects and workspaces

In order to make this project a little more complete, I'd like to get the projects. In order to get the projects, I will need to get the workspaces.
The API end points below will help you get both the workspaces and the projects:
￼https://github.com/toggl/toggl_api_docs/blob/master/chapters/workspaces.md

closed with release 0.1.5


improve .ci/go-lint.sh to run over each repository

.ci/go-lint.sh should be used to check each clear containers repositories

Related (probably replaced by) #99


Upgrade example app to RN 0.50.3

New:

Uses pods
Has README

@devfd Hey, could you reach out to me on twitter? DMs are open. I could help a bit with this project since I've been using it for a while.

Are all maintainers completely disappeared? I don't want to start working on my own fork if not necessary. If you're a maintainer please contact me on twitter and we can discuss what you'd like to do with this project. DMs are open. :)

Working on upgrading example app to 0.55. Stay tuned. :)


New mysql schema supports utf8mb4 on selected columns

Support for 4-byte utf8 in the title, author, and content columns of the items table and title column in the feeds table.
Note this does not cover any other column. Charset was customised on a column by column basis as it avoids truncating columns (where there is a unique index) that would potentially affect portability with other database drivers.
I believe the columns chosen would likely represent the set of columns where data holding 4-byte utf8 may be found, happy to add more to the PR on your advice @fguillot.

LGTM, thanks!


GetCurrentFrame for example after button got pressed

Hey,
Thanks for this library, got it running after some fiddeling with the startup stuff.
Have you thought about implementing a method to get the current frame, so you don't have to check every time the OnFrame-method is called, if a button got pressed?
Thank you in advance!

Hi @DaveHOnCode,
I found some time to implement your request. I've tested it on my machine and it worked.
You can update the NuGet-package in your project to the latest Version 2.1.1.
Hope the features works as you imagined :)


Liquid Level Sensor (Ultrasonic)

Need an Implementation of an DYP-A02YY UART Ultrasonic Sensor for reading the Level of a Tank.

Where can I find an example Project for implementing my own Sensor?

The tutorials and projects pages have a variety of examples, including temperature sensors.  The ultrasonic sensor you note above is specialized and will require its own coding, or a separate library, to read.  However, if you have that code (or library) you can integrate it into HomeSpan following any of the many examples that use the loop() method of a Service (such as the Temperature Sensor).

Thats not my Problem: I can read out that sensor. What I need is an Implementation in Apple Home Kit, to show the Data (Fill Level) like in an Battery App (something like that).

Apple HomeKit does not have a Water Level Service, and unfortunately Apple does not have a mechanisms for extending or making any changed to the Home App.  You'll need to repurpose one of the existing Services.  The easiest is probably to use the Temperature Sensor, but instead of temperature you can map water level percentage to 0 degrees - 100 degrees.  The Temperature Sensor Tutorial takes you through all the steps.
There is a Water Level Characteristic, but its part of the Humidifier/Dehumidifier Service.  That Service has a lot of additional components and would might bury the display of the Water Level on the setting screen, but if your sensor is part of a larger appliance, some of the Humidifier/Dehumidifier features may be of use.

THX a lot. WaterLevel is presented in a Sub-Menu of Humidifier Service. That doesnt help. I would like an Icon with a "batterylevel"-like design

Isn't this issue way outside of the scope for this project?
Nevertheless; As far as I know there are no dynamic icons like that in the HomeApp. You could use Current Relative Humidity, which is presented as a sensor (in the round circles above the accessories in each room) with a percentage value.


Conflict with lib.dom.d.ts when using runes

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I'm using eslint-plugin-svelte. (*.svelte file linting does not work with the parser alone. You should also use eslint-plugin-svelte with it.)
[X] I'm sure the problem is a parser problem. (If you are not sure, search for the issue in eslint-plugin-svelte repo and open the issue in eslint-plugin-svelte repo if there is no solution.
[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?

9.9.0

What version of eslint-plugin-svelte and  svelte-eslint-parser are you using?

<EMAIL_ADDRESS>
What did you do?
https://sveltejs.github.io/svelte-eslint-parser/virtual-script-code/#eJyrVkrOT0lVslKyKU4uyiwoUchJzEu3jVEqKY5RsovJy0ktUahWyEvMTVWotYKyrBSKS4oy89IVahVsFVQKivILijU0Y/Ji8mz0IWYA9cH0KNUCAEUdIQ4=
<script lang="ts">
let { name }: { name: string } = $props()

</script>
{ name }

What did you expect to happen?
Name should be able to be defined and not conflict with globals
What actually happened?
It conflicts with the DOM lib globals
Link to GitHub Repo with Minimal Reproducible Example
https://sveltejs.github.io/svelte-eslint-parser/virtual-script-code/#eJyrVkrOT0lVslKyKU4uyiwoUchJzEu3jVEqKY5RsovJy0ktUahWyEvMTVWotYKyrBSKS4oy89IVahVsFVQKivILijU0Y/Ji8mz0IWYA9cH0KNUCAEUdIQ4=
Additional comments
No response

Adding an export {} to the end of the virtual script code resolves this issue, as it forces typescript to treat the module as an actual module


Building instructions are hard to follow :)

Hello sir! I just purchased an S1 without doing a good enough job reading all the information about it, so am now very disappointed that I can't use it for anything meaningful except running from the app...
Looks like you are building exactly what I am looking for. At least if this code can achieve the result in the Demo Video linked from the readme.
However it is hard to follow building instructions that don't exist 😉
I tried to run the robomaster-control app (I am on an M1 Mac so using the CGO_ENABLED=1 GOARCH=amd64 go build command)
This gets me to
panic: Could not load Unity Bridge library at "./lib/darwin/amd64/unitybridge.bundle/Contents/MacOS/unitybridge": dlopen(unitybridge, 0x0002): tried: 'unitybridge' (no such file), '/System/Volumes/Preboot/Cryptexes/OSunitybridge' (no such file), '/usr/lib/unitybridge' (no such file, not in dyld cache), 'unitybridge' (no such file)

I saw some posts from you from a couple of years ago (not related to this repo) talking about building a Wine wrapper. Does that apply here as well? Or where could I get this file?

By the way, I will happily assist you writing some documentation if I get things working.

Update: After some more research I learned that the unitybridge thing was part of the now discontinued Windows/Mac app for the Robomaster. I managed to lay my hands on a copy of the Mac app, even if DJI are no longer linking to it from the downloads page. So now things work. Not sure how to write instructions on how to build this though...


look into a more personalized dashboard

I think a github-like dashboard would be cool. This means:
You can follow another user.
If you follow another user, you are able to see his activities in your dashboard/feed. These activities include:

Created a new hobbit
Created a new record


📝 Getting Ready for migration - Audit Rules backlog

To prepare for a migration from Gatsby to Next, we should go through and mark issues as won't do to clean up the repo.
Tasks
### Tasks
- [ ] Clean up SSW Rules Issues

Closing - We will be staying on Gatsby for this move to TinaCMS.


API feature request to NOT modify the file after TextEditorEdit#setEndOfLine is called

VSCode Version: 1.10.2
OS Version: Windows 10

Steps to Reproduce:

Using the API, call TextEditorEdit#setEndOfLine
Notice that the file is now marked dirty.

This would be OK for most purposes, but for the purpose of editorconfig-vscode, we don't wish to modify the file (marking it dirty) on open.
This feature request is requesting an option I can send along, like so:
setEndOfLine(eol, { modify: false });

This would be backwards compatible.
Related issues:

https://github.com/Microsoft/vscode/issues/9494
https://github.com/editorconfig/editorconfig-vscode/issues/57 and the host of duplicates filed (see the bottom of the page).

@jedmao Are you saying the file should be immediately saved to disk after that? If you change it (that includes changing the EOL-sequence) the file will be dirty because it got modified. Please clarify. Thanks

@jrieken no. I'm saying that { modify: false } would ensure the file is NOT saved. Basically, allowing me to set my end of line "mode" w/o modifying the file. This way, someone can have their .editorconfig file setup with end_of_line = lf and they will see LF in the status bar whenever they open a file (w/o modfiying the file on open).
I'm saying please give me a way to set the end of line sequence w/o saving or modifying the file.

@jrieken the files.eol user/workspace setting works in such a way that when you open a file the EOL mode is selected for you w/o modifying the file. I'm basically asking for the ability to do the same thing w/ the extension API, also NOT modifying the file, because this is a file open operation.

I'm saying please give me a way to set the end of line sequence w/o saving or modifying the file.

Isn't that then a no-op? Do you wanna make the new eol-sequence only effective when other changes happen in that file? And then for the whole file or just the affected lines? Today the status bar reflects the 'is' not the 'wants'.

Today the status bar reflects the 'is' not the 'wants'.

@jrieken that's not actually true. I created a file with mixed line endings that looks like this:
foo\r\n
bar\n

Then, I opened the file in vscode w/o extensions enabled. In the status bar, it reflects my user settings:
"files.eol": "\r\n"

I see CRLF in the status bar, but the file is not marked dirty. This is because on file open, vscode's status bar reflects the 'wants', not the 'is'.
If I then manually change the EOL sequence, the file is marked dirty.

vscode's status bar reflects the 'wants', not the 'is'.

We take the predominant line ending occurring in the file because we cannot show this per-line in the status bar. Add one more line with \n and the status bar will show it like that, not the preference.

Interesting! Well, I guess I'll have to set the EOL sequence just before file save then.

Yes, be a onWillSaveTextDocument-listener and make you changes before the file is being saved.

@jrieken you're not joking that it's bumpy! I have an implementation here, but it doesn't work for the very reasons you mentioned above, that it takes the predominant line ending on file open as the EOL setting.
As such, this test fails:
test('end_of_line = lf', async () => {
	const savedText = await withSetting(
		'end_of_line',
		'lf',
		{
			contents: 'bar\r\n'
		}
	).saveText('foo\r\n');
	assert.strictEqual(savedText, 'foo\nbar\n',
		'editor fails to convert CRLF line endings into LF on save');
});

As you can see, the initial contents have \r\n as the line ending, so CRLF will be the initial EOL setting on file open. Even if I programmatically change those CRLFs into LFs onWillSaveTextDocument it just changes them right back on save.

@jrieken I got a before-save implementation working just fine. The only problem is that I have to find the editor for the text document that's going to be saved first and then await the setEndOfLine to complete before doing the rest. Seems to work!
See https://github.com/editorconfig/editorconfig-vscode/pull/116 for more context.


Implement validation for user  information struct

The user information struct defined here, has many Optional fields (defined with Option<T>) and a few fixed ones,

Implement validation for all the fixed and Optional fields using the validator crate which is already added to the project.

Write unit test at the base of the module

May I try to implement this?

Yes, please.
Thanks for reaching out to contribute

@redredacted
How is it going 😄

fixed!