Elm static analysis and dynamic execution (in Elm)

Elm looks very interesting (so much so that I may try porting a polymer framework of components to it), but I'm interested in knowing to what extent I can create functions/values/modules from strings, and parse/analyze elm source-code in Elm natively.
Despite the shortcomings of languages like Javascript and Lua, they have pretty strong facilities for analyzing and executing user-provided code at runtime.  I'm very new to the ML world, but it looks like Haskell can do much of the same.  Is there a way to create runtime usable resources from strings of Elm?  Is there a plan to port the compiler (and type inferencing mechanisms) to Elm?  Is that going to become my 2016 project?

There have been various discussions on the mailing list about both aspects. A recent one is this thread.
Generally, asking that kind of question on the mailing list is likely going to give more useful results (including further pointers to earlier discussions) than an issue here in this repository.

I don't have near-term plans for stuff like Template Haskell, but I am aware of it and things like MetaML and MetaOCaml. I have some very modest "macro" ideas that I think are plausible as well. It'd be interesting to get an idea of what you are interested in specifically to help guide those thoughts though. Do you mind sharing your particular vision of things on the elm-dev mailing list?


TypeError: Path must be a string. Received undefined

run command yarn start after yarn eject get the error：
path.js:28
    throw new TypeError('Path must be a string. Received ' + inspect(path));
    ^

TypeError: Path must be a string. Received undefined
    at assertPath (path.js:28:11)
    at Object.resolve (path.js:211:7)
    at Function.findConfig (C:\Users\xxx\Desktop\my-app\node_modules\react-dev-utils\node_modules\browserslist\node.js:212:17)
    at checkBrowsers (C:\Users\xxx\Desktop\my-app\node_modules\react-dev-utils\browsersHelper.js:24:32)
    at Object.<anonymous> (C:\Users\xxx\Desktop\my-app\scripts\start.js:65:1)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@ChiaJune I didn't test the eject command in this project, the best you can do is fork the project and use modify it from there, you can add the #1 in your fork


How to monitor transcoding start event and transcoding finish event?

Q
A

Bug?
no

New Feature?
no

Version Used
v0.17.0

FFmpeg Version
4.3.2

OS
Windows 10

How to monitor transcoding start event and transcoding finish event?

Hi @etjson,
You can easily monitor the progress of your encoding. Using the percentage of progress, you'll now when it starts or finish.
$video_ffmpeg = $ffmpeg->open( $full_video_path );

$format_x264 = new FFMpeg\Format\Video\X264();
$format_x264->setAudioCodec("libmp3lame");

$format_x264->on('progress', function ($video_ffmpeg, $format_x264, $percentage) use ($my_variable, $my_other_variable) {
    // Do whatever you want here with the percentage of progress...
});

$video_ffmpeg
    ->save($transcoded_file);

Hope this helps.
Romain


IShellItemImageFactory.GetImage should return HRESULT

Actual behavior
Currently IShellItemImageFactory.GetImage returns void.
Expected behavior
According to the official documentation it should return an HRESULT.
Repro steps

NativeMethods.txt content:

IShellItemImageFactory

NativeMethods.json content (if present):

{
  "$schema": "https://aka.ms/CsWin32.schema.json"
}

Any of your own code that should be shared?

Context

CsWin32 version: 0.3.106
Win32Metadata version (if explicitly set by project): not set
Target Framework: net8.0-windows
LangVersion (if explicitly set by project): not set

CsWin32 is doing the default .NET thing: considering PreserveSig to be false by default. If a failing HRESULT were to be returned from this API, an exception would be thrown to your code.
If you really want an HRESULT instead of exceptions, you can add this API to the list of preservesig APIs as described in the readme.


Add timeout for HTTP requests

Currently, we will wait forever for an outbound HTTP request (from us to another service) to return. We should add a configurable timeout, with a sensible default value. treq provides a timeout parameter for us to do this.

@rudigiesler ready for review

@rudigiesler ready for rereview


Setting for the theme

Then also add here the option in the settings for light / dark / automatic theme?

will do

im in process of removing all proprietary theming and having it call system available equivalents

added theme chooser

thanks, however when I select the dark theme from the app settings the navbar and status bar remain black / gray.  This is not true if it is on an automatic theme with system-wide dark mode.

in the green underline why did the information about duration and other things that were present before disappear?  Could you put them back on please?


Add Carto View post-processing task

Create a new post-processing task that will create a view in the Carto database (using the Carto API, not via direct db access) by linking the current validated file to a dataset already on Carto.  The view will be an inner join of the the two tables.
The UI will have 3 dropdowns:

the columns from the current file, allow the users to pick the column to use for the join
the list of datasets already on Carto (which will be kept up-to-date on the server)
the list of columns for the dataset (which will be populated when the user chooses a dataset by querying the columns from the Carto API), which will be used in the join

This will be written in Python, and be external to PLUTO core, but used as an example of a complex rule.

Here's what I'm currently thinking @leejaykar @bettyruth @bb-acarrera

This work has been merged into develop branch as the post process task.


added transports option on client connection

hi, my socket.io server transport option is "websocket" so i can not connect on admin-ui
i added the 2 option to connection params

Hi! That's indeed a known issue of the client: https://github.com/socketio/engine.io-client/issues/575
I'd rather not merge this though, as it disables the HTTP long-polling transport completely.


[WIP] Spike triggered average

Function to plot spike triggered average
Remaining work:

[ ] Documentation and examples
[ ] Tests

Codecov Report

Merging #71 (8ba8d3b) into update-0.1.2 (f4b0afb) will increase coverage by 5.90%.
The diff coverage is 30.43%.

@@               Coverage Diff                @@
##           update-0.1.2      #71      +/-   ##
================================================
+ Coverage         43.08%   48.99%   +5.90%
================================================
  Files                29       32       +3
  Lines               926     1045     +119
  Branches             96      118      +22
================================================
+ Hits                399      512     +113
+ Misses              522      519       -3
- Partials              5       14       +9

Flag
Coverage Δ

unittests
48.99% <30.43%> (+5.90%)
:arrow_up:

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

Impacted Files
Coverage Δ

miv/visualization/spike_triggered_avg.py
27.27% <27.27%> (ø)

miv/visualization/__init__.py
100.00% <100.00%> (ø)

miv/signal/generator/__init__.py
100.00% <0.00%> (ø)

miv/signal/generator/spike_generation.py
77.77% <0.00%> (ø)

miv/io/data.py
50.50% <0.00%> (+20.75%)
:arrow_up:

miv/signal/spike/detection.py
96.00% <0.00%> (+54.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Refactor duplicate code ( verifying_key assignment )

Closes #461

I wonder if it's better to make verifying_key a property and save the initial verifying key to be private (with the _ prefix)

I think it would be better to set verifying_key and signing_key to private ( with _ prefix ), and to access only with get_verifying_key

well we have to consider backwards compatibility. Anyways, that can be left to a future PR

@Andrew-Chen-Wang thanks for merging this!! btw when will the next version be released?


Inheritance and interfaces in modelling

I use a rather large third party library of components and I would like to model its behaviour. To make the work manageable I would like to model methods in the classes highest in the class hierarchy. Additionally it would be very useful to model interfaces. E.g. it would be convenient to model the List interface instead of providing identical models for every list implementation.
-It seems method models are not inherited. Is this correct or did I make a mistake somewhere?
-Are there plans to support modelling of interfaces?

Unfortunately, it does not work right now to model the interfaces in order to model all the implementations at once. This is due to an incomplete support for dynamic dispatch in the analyzer. For example, if a method is of the form:
int foo(List l) {
    return l.lenght();
}

int bar(ArrayList l) {
  return foo(l);
}

Then the model provided for List.length() will not be used in the call to foo(l) in bar.
I am leaving this task opened for now as this are things that we are working on.
However, if you model a method in a class A, then you do not need to model it again in the subclasses of A.

Thank you for your reply and for describing the current situation in the tool! Good to hear that support for modelling interfaces is in the plans

@jeremydubreil Is there an update on this issue of supporting inheritance and interfaces in the Java models?

@pontusbostrom In general, we are trying to move away from having models in source code, i.e. in Java, in favor of having the models builtin the OCaml code. The later approach is more flexible and allows to model the same method differently for different analysis backends.


failImageSource  can not use loacl image

Hello author can I use the local image to load this component  I hope it's compatible  thanks

https://github.com/ascoders/react-native-image-viewer/issues/437#issuecomment-823187948

ok,i got it,thank you


Character class advancement missing features

Describe the bug
Imported characters' classes don't include features in their advancement. This prevents leveling up characters solely in Foundry. If a player attempts this, they will find
To Reproduce
Steps to reproduce the behavior:

Import a character
View their class
View the "Advancement" tab
Compare to equivalent class from the SRD

Expected behavior
The DDB version of character classes' advancement to include character feats.
Screenshots
Two version of the Cleric class: left SRD, right DDB. Notice "Divine Domain", "Channel Divinity", and "Turn Undead" character feats missing.

Environment:

Browser: Chrome Version 120.0.6099.131 (Official Build) (64-bit)
Foundry Version: 11 Stable Build 315
5e System Version: v2.4.1
Module Version: v3.7.6
Does this behaviour occur with other modules disabled: Yes

Additional context

Example character URL: https://www.dndbeyond.com/characters/109960799
No errors in the browser console

In order to link features like this they need to exist in the Compendium, you will need to run the Class and Feature munch feature first in the the DDB Muncher

After running the Class and Feature munch, there are still features missing. For example, Channel Divinity: Turn Undead is missing.

Additionally, Channel Divinity is listed as passive instead of an action.


adding the capability to retry a download.

Downloading large files (e.g. 100M vectors) sometimes fails.
This adds a retry capability.

This approach still has problems. While it attempts to retry - it just makes the download fail multiple times, which causes free disk space to go down to 0. A better approach is needed - where an interrupted download can continue.


validation code and config

Hi,
I want to see my model performance on nuscences val dataset, how can i do it ? it seems that all the config only can train, cannot validate.
thanks!

the dist_test.py will evaluate your model in the end.

hello, tianweiy. I want to monitor the training process, such as do a evaluate for a trained epoch right now. Can i realize it by pass some parameter in train config? Or how can i realize it?

not really sure, but maybe adding a 'val': 1 to here https://github.com/tianweiy/CenterPoint/blob/47d61adacb75ba6ccb49ae666dc257d04e323a2c/configs/nusc/voxelnet/nusc_centerpoint_voxelnet_0075voxel_dcn.py#L231
may do a validation every 1 epoch

ok, I'll try ~. By the way, can you share your experience in training? Only by observing the printed loss or something other? Thanks~

ok, I'll try . By the way, can you share your experience in training? Only by observing the printed loss or something other? Thanks

I never really look at the loss. I do a full run of training and then measure map. This is the only valid way to my knowledge. For faster iteration, I may downsample the dataset

ok, I got it , thanks ~

thanks.
if the dist_test.py is on val dataset.
how can i get the test prediction in order to get nuscnenes tensorboard test dataset rank?

You can change https://github.com/tianweiy/CenterPoint/blob/47d61adacb75ba6ccb49ae666dc257d04e323a2c/configs/nusc/voxelnet/nusc_centerpoint_voxelnet_0075voxel_fix_bn_z.py#L167
to infos_test

thanks~


Vs2017 format

Possible to compile under netstandard2.0
Most of the tests are green (2 failures: System.NullReferenceException instead of System.InvalidOperationException)

Pretty weird that the DataReader is set to null somehow and triggers that NullReferenceException... :thinking:

I agree. I don't really get how that can happen.


How do I grab the checked value on init?

I have some code on changing the checkbox that toggles some UI on/off, that piece is working fine per this code
$('input[name="leave_type[validate_balance]"]').change(function() {
	$('#threshold_options').toggle($(this).prop('checked'));
});

If I try and grab the value on init, it is always false, no matter what's been saved, how do I get the state of the checkbox on first load? Everything else works perfectly.
$('input[name="leave_type[validate_balance]"]').init(function() {
    // Always false
    alert($('input[name="leave_type[validate_balance]"]').prop('checked'));
});

This plugin has not been supported for more than 6 years. What's the point of writing anything here?

Are there other alternative? We are using this plugin a lot


Fungus in Unity3d 2022.1 Ton of Errors due to LeanTween

Describe the bug
Bunch  of errors when installing Fungus to Unity 2022.1. Errors thrown caused by LeanTween.
To Reproduce
Import Fungus package, and see the console log.
Temporary Workaround
Remove the included leantween.dll in Fungus, and download the new updated LeanTween from assets store. The latest version is 2.51
Here's a screenshot of release notes in the latest LeanTween

Note: The newly updated package wasn't based off of the master branch nor any other branches of LeanTween git repo. The creator seems updating it locally before uploading to Assets Store.
Edit: Added release note screenshot

Misread title, it's a 2022 issue, so less urgent so pushed back a bugfix version.


Amazon Linux 2023 - There isn't enough space

I am trying to run swiftly install latest on t3..micro AWS EC2 instance but  I got the error:
Fetching the latest stable Swift release...
Installing Swift 5.10.0
                                                                                Downloading Swift 5.10.0
77% [=====================================================================================================================================----------------------------------------]
Downloaded 451.7 MiB of 579.1 MiB
Error: The operation could not be completed. There isn’t enough space.

I run df -T command:
Filesystem       Type     1K-blocks    Used Available Use% Mounted on
devtmpfs         devtmpfs      4096       0      4096   0% /dev
tmpfs            tmpfs       463368       0    463368   0% /dev/shm
tmpfs            tmpfs       185348     444    184904   1% /run
/dev/nvme0n1p1   xfs        8310764 1635088   6675676  20% /
tmpfs            tmpfs       463372       0    463372   0% /tmp
/dev/nvme0n1p128 vfat         10202    1310      8892  13% /boot/efi
tmpfs            tmpfs        92672       0     92672   0% /run/user/1000

[ec2-user@ip-172-31-40-159 ~]$ swiftly --version
0.2.0

$ uname -a
Linux ip-172-31-40-159.eu-north-1.compute.internal 6.1.79-99.164.amzn2023.x86_64 #1 SMP PREEMPT_DYNAMIC Tue Feb 27 18:02:23 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

The same configuration, with Ubuntu,  works fine. What do I miss?

Swift isn't supported on Amazon Linux 2023 at this point in time. There is ongoing work to get it working.
There is a long thread on the forums about getting it working. See https://forums.swift.org/t/recommended-way-to-develop-for-amazon-linux-2023/69958/36.
Also once it is working Swiftly will probably need to be updated to support it.

I think the reason is your /tmp is too small. Your /tmp looks like 452 MiB and the download failed at 451.7 MiB.
swiftly install downloads a toolchain to /tmp temporarily.
https://github.com/swift-server/swiftly/blob/22ba150d6ea4323b4c7f6f1d04f0eeba4de5004e/Sources/Swiftly/Install.swift#L143

Yes, the /tmp was the primary issue. Secondary is what Adam linked. Thank you.


[3pt] Bug corrections

There's a minor bug which causes that whenever refreshing '/reservations' & 'vehicle details' the information fetched is lost.
https://github.com/VelzckC0D3/Land-Rover-E-Commerce/assets/92229666/fa86b005-1465-414e-a24f-95d6087ce9c8

Here is the P.R with Fix that bug and something makes better optimization:


Dynamic Service Port

I went through the documentation and not sure if it would be possible to let docker select the service port (-P) dynamically and randomly when using docker swarm mode.
client --https://abc.com --> haproxy --> abcServiceDockerContainer-1:33012
client --https://abc.com --> haproxy --> abcServiceDockerContainer-2:33012
client --https://def.com --> haproxy --> defServiceDockerContainer-1:36022
client --https://def.com --> haproxy --> defServiceDockerContainer-2:37022
The idea is when we create the  abcServiceDockerContainer and defServiceDockerContainer services, we would let docker assign service ports (using -P). We don't want to manage the port mappings and would just have Docker pick the ports dynamically/randomly such as 33012, 36022, 37022, etc. Would that be possible?
From the internet in, customers can use different domains or URL to reach a given service -- always using the standard 80 or 443 ports.

Assuming that you are using Docker networking, there is no need to open any ports. If you take a look at the examples from http://proxy.dockerflow.com/swarm-mode-auto/ or http://proxy.dockerflow.com/swarm-mode-stack/, you'll notice that the go-demo  service that not open any ports.
As long as the proxy and a service belong to the same network, the proxy will find its way to forward requests. The only requirement is that the destination service has the proper service labels. One of those labels is the port the service is listening to inside a container.
Can you please join http://slack.devops20toolkit.com/ and ping me? I can walk you through the process.

all good...thanks


sample code - Facebook-events URL is broken

In section:
"
For further reference the Facebook-events sample bot can be used to explore the bot communication with Facebook Messenger.
"
The url https://aka.ms/facebook-events is 404
In fact full url is broken:
https://github.com/Microsoft/BotBuilder-Samples/tree/v4/samples/csharp_dotnetcore/23.facebook-events
And I belive it's somewhere from here:
https://github.com/Microsoft/BotBuilder-Samples/tree/master/samples/csharp_dotnetcore/23.facebook-events

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

ID: acb1736e-f822-2374-fc91-60a13567966e
Version Independent ID: 03a952ae-0752-9aeb-a214-981bbfaf3b91
Content: Connect a bot to Facebook Messenger - Bot Service
Content Source: articles/bot-service-channel-connect-facebook.md
Service: bot-service
Product: bot-framework
GitHub Login: @RobStand
Microsoft Alias: RobStand

@alundiak - fixed. thanks for reporting.


blog: add libp2p Day 2022 blog post

@p-shahi : I'm not sure how to resolve this (maybe compare to another recent PR?), but I don't see a preview of the post when I look at https://bafybeiha2xrtbhgkpr6apsvinz3attzk2skd5xkpbaibrgbtxwhfkn6w5u.on.fleek.co/

@cwaring mentioned that was because the post date was set in the future and the blog post was missing a header image. Both have been added, so the preview is up now.
I'm making some more cosmetic (image formatting) changes but generally this is ready for review (pending incorporation of some of @DannyS03 's feedback)

⚠ this post has 21MB of embedded images, could you please resize and compress them to JPG. You should be able to get this down to < 1.5MB

@emilymvaughan @BigLep I went through a resolved all the comments. I think this is ready to publish. 🤞

🌐 https://blog.ipfs.tech/2022-11-22-libp2p-day-2022-recap/

Thanks for your help on this @cwaring! And thank you to @p-shahi and @BigLep for drafting the post! <3


Can't install the CHIPTool iOS application on our iPhones

Problem
The CHIPTool iOS app cannot be installed on our phones out of the box.
By default xcode gives you an error.

Currently to use a real iPhone we need to make modifications to the CHIPTool xcode project.
Specifically the Bundle-ID needs to be set to something not already claimed by another user and the Development Team needs to be set.

Proposed Solution
There might be a better way to do this?

@woody-apple , I just googled CHIPTool for iOS and found this.  Find a work around?

This video https://www.youtube.com/watch?v=eW1sPzDY3S0 seems to have an IOS app. Is that in this repository somewhere and are there build instructions?
Thanks


Refactor: Move into proper project config

Summary
After some testing it seems we can do what we want.
We should clean things up to start building upon.

Build from an inherited base class instead of def run
Other "standard" bot design patterns
Update web/ against the latest fullstack reference
Proper logging to be shared between the two?

Basic Example
something like this, I think
➜ ntp bytestructexample && mkdir -p src/{web/{lib,domain},bot/{ext},lib/{db,log}} && tree
  bytestructexample
 └─  src
    ├─  bot
    │  └─  {ext}
    ├─  lib
    │  ├─  log
    │  └─  db
    └─  web
       ├─  domain
       └─  lib

Drawbacks and Impact
No response
Unresolved questions
No response

Another major idea architecturally would be to just have the bot made API calls to a Litestar API that can be scaled as needed. This way we could showcase Litestar in action, explore different design patterns, scale better, etc.

We should avoid any inter-domain dependencies in case we separated from the monorepo pattern.


Chatbot made responsive for mobile devices

This is a pull request to resolve the following issue : Chatbot interface not positioned properly in mobile view #73
It has been made responsive and has been tested for various device widths 👍

It seems like not responsive could you please check again,
Add screenshot,
And also u should not make PR from main branch,

Apologies, I fetched upstream after creating a pull request 🙏🏻 I'll create a new pull request without PR from main branch and also add the screenshots

Apologies, I fetched upstream after creating a pull request 🙏🏻 I'll create a new pull request without PR from main branch and also add the screenshots 👍🏻

Don't worry, create new PR🙂


feat: Add support for Adafruit Neokey Trinkey board

You would also need to add your board to crates.json so it gets picked up by CI

👀


[Stats] Clear all timers on print, since they stopped self-clearing recently.

You may have seen giant swathes of timers being printed to stderr in recent logs that gather stats from swiftc. That is because LLVM changed their TimerGroup semantics. This fixes our use thereof.
(see https://reviews.llvm.org/rL339980 for details; relevant LLVM change was cherry-picked over in apple/swift-llvm@3b20ef35078c815907afd9a2881f21ef925fde3a)

@swift-ci please test and merge

@swift-ci please test and merge

@swift-ci please test and merge


PartyList Vouching

Starting this feature after doing door for a night.  We need this.  Also if we are going to have variable amount of vouches between events there should be a permission/view added for sage and first

Closed in favor of #89


Git clone gets permission denied

Issue summary
When I run this in cmd
git clone --recursive git@github.com:openai/retro.git gym-retro
I get :
'Cloning into 'gym-retro'...
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.'
What might be the issue?
System information

Windows 10
Python 3.5
pip install https://storage.googleapis.com/gym-retro/builds/gym_retro-0.5.2-cp35-cp35m-linux_x86_64.whl

use https instead of git@, as seen in #7.


Sample: Selectable activity

This work is required for Web Chat integration in Emulator.

[ ] If activity can be selected programmatically, will be nice to scroll the activity into view on select

Design is needed for Emulator. To speed things up, I will make this sample first, then align with designer afterward.

Dupe of #1117.


genshin impact- stuck on loading screen

at the start it launches perfectly, till it reaches this point (as displayed in the screenshot). it doesn't load further than the geo icon, ive updated my computer, refreshed everything, cleared my caches and nothing has worked
is there any way to fix this ??

Normally try waiting. First launch can take up to several minutes to load... Otherwise try using your mobile device or any other device to teleport to a less busy location (e.g. mondstadt) so it can facilitate the first load on yaagl.

I would like to know your system settings including:
Mac chip and model
RAM
System (I know of issues with sequoia, i suspect this is the cause of the issue)
There are frequent issues with the updating of mac versions. Usually, I recommand waiting until YAAGL is modified for the newest mac version to then update your mac.
Hope this helped!


Wrong selectbox rendering after reset form with scrollable.js

Wrong rendering applies to Selectboxes with scrollable.js plugin. After reset the form, the height of the select box is 0.
See https://jsfiddle.net/k056xn67/6/

Hi, I'm trying to solve similar issue. Problem is with custom scroll and dropdown using jcf.
Steps to reproduce:

Open dropdown
Click outside of dropdown list to close it
Click "refresh all" link to refresh all jcf elements
Open dropdown again
Now it has 0px width and 0px height

Here is the fiddle: https://jsfiddle.net/kazlepka/tws1bdnh/
I was checking the code and problem is most probably in refreshing instances, even the ones that doesn't exists anymore (scrollable dropdown). It's not in DOM anymore.


Add FAQ entry: (C) copyright symbol

Both (C) and (c) are producing copyright symbol in Metanorma. In AsciiDoctor, I was able to find that only (C) produces this symbol.

When writing (C) in .adoc, expected result is produced, i.e. (C). However, backslash is generated in DOC/PDF in case of writing (c) in .adoc and I wasn't able to find the proper way to generate (c) instead of the copyright symbol.

The difference betweeen Asciisdoctor and Metanorma is because I repair these expansions using the gem sterile (for smart quotes), and sterile also has a more liberal set of symbols.
My generic advice to @manuel489 from yesterday, https://github.com/metanorma/metanorma-standoc/issues/263, also applies here. If you want to prevent a combination of two or more characters from being converted to a single symbol, don't use backslash, which is completely at the mercy of what Asciidoctor has implemented. Instead, interrupt the sequence using the Zero Width Non Join character, U+200C. It is invisible, but it prevents combinations of characters from being recognised as such. So in this case:
(c)
Originally posted by @opoudjis in https://github.com/metanorma/metanorma-nist/issues/197#issuecomment-622185712

Um, guys, I've already done this. https://github.com/metanorma/metanorma.com/pull/252, https://www.metanorma.com/author/ref/asciidoc-tips/#asciidoc_escaping
@ronaldtse, we've been through this several times: I always document new features as I go.


Make use of new library from instructlab/sdg

This PR drops the sdg code from this repo and replaces it with the new
library housed in instructlab/sdg. The first release of the library
will maintain an identical interface as used today to allow for a
quick transition.
Closes #1245
Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

@Mergifyio rebase

@mergifyio rebase

I tried disabling the pip cache to see if a bad cache was the cause of the failure related to the schema package, but that's not it -- same failure

I fixed the problem that was breaking CI. There's a bit more explanation in the fix here: https://github.com/instructlab/sdg/pull/28
I believe this is ready now.


libcaliptra: idev, ldev, dpe commands

This adds support for the following mailbox commands:
OP_GET_IDEV_CSR
OP_GET_LDEV_CERT
OP_INVOKE_DPE_COMMAND
It also normalizes the new command API pattern, which will be necessary in the future to support asynchronous mailbox operations.

Can we get this PR merged? We would like to be able to add some DPE tests that leverage it.
cc @JohnTraverAmd @fdamato @jlmahowa-amd


chore(art taste quiz): Add INTL to Welcome Screen

The type of this PR is: Chore

This PR solves GRO-1317
Description

Adds useTranslation and t function to Welcome Screen and copy text to translation.json file.

LGTM 👍🏾 🥇 looks like an unrelated test is failing on CI though, you might have to run a yarn install or do a  rebase on this pr to get it passing again. Let me know if you have questions about that!

Looks like re-running Circle CI test is what was needed to get the test to pass, so feel free to merge when you're ready 👍🏾

Gonna merge it in!


Generate Rustdoc as part of CI

We had a few Rustdoc errors sneak in recently that could have easily been caught by CI if we'd been checking. This should probably get enabled ASAP.

Closed by #243.


Git commits shows Gogs nickname as first option

Pushing data to gogs records the system user as the commit user and not the user who authenticated against gogs.
i.e. Logged in as root but pushing as a different user would record root as the commit user, this makes it hard to validate the real identity of the person who commits the updates.
Thanks
Dave

Thanks your feedback!

I don't quite get this part. So... push git data, the data already contains the author info, Gogs only reads it. What exactly situation are you describing?

Within the gog's interface I have a number of updates from the user root, which is the system user I ran git push from.  Is there anyway this user could be set to the authenticated i.e. user "dwb" in the example below?
[root@localhost centos]# git push
Username for 'https://hostname': dwb
Password for 'https://dwb@hostname':
Thanks for a great product
Dave

What is the remote address for your git push?

I'm a bit confusing... I think you're doing something wrong..

The commit records the Author "user.name" and that author is uploaded to gogs and not aligned with the authenticated user who pushed it, so its hard to see who carried out the commit if the "user.name" is incorrect, my request is to align the commit with the authenticated user not the Author in the initial commit.  But I definitely could be misunderstanding the process.
Something like this.  https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/
Thanks
Dave

@dwbconsulting so does the GitHub solution(add another e-mail to account) fits your situation?

I would like to have the option to see the user account (gogs login name) as the Author in gogs not the user.name which made the commit in the local repository before the push to gogs.  My number one issue is that I am unable to see who made the push because the user.name can be anything.
Thanks

My number one issue is that I am unable to see who made the push because the user.name can be anything.

It really sounds like a completely new feature. Who pushes commits isn't really a part of Git repo in my understanding of how Git works.

It looks like there is no consistency in gogs when showing that name. In some places it shows the username and in other places it shows the full name. I agree it is better to use always the gogs username when possible. Any one that pushes must be a user of the system so it will mandatory have a username. This would be only not true when you migrate a repository and some commits will belong to users that are not registered. In that case would be fine to show the commit author.
@Unknwon I bet you don't realize about the issue because in your case the gogs nickname and the user.name you have configured on git is the same. But a lot of people has a nickname but in the git configuration they use full name. That is common practice.
For example.
https://try.gogs.io/unknwon/gogs/commits/master
Here gogs is showing full name, the commit author full name, instead of gogs nickname.
In the dashboard the gogs nickname is always used.
I bet in all repository pages it is shown as commit author full name.

On github for example they show github nickname if the email of the commit author (user.email configured on git) match an email of a github account.

@manfer thanks your explanation, I got it now.

Is this the same issue? I committed this as crayfield but it doesn't show it that way in the commit log only in the activity log on the dashboard it will say I "crayfield" pushed to master etc...

@cb88 can you click on the user name? Or it's just plain text "Gogs"?

It's just plain text.

@cb88 thanks for following up!

Does this happen to every repo?
Can you reproduce at https://try.gogs.io ?

Yes all of my repos.

@cb88 thanks!

Can you reproduce at https://try.gogs.io ?

Don't skip my comments, please.
What is version of your Gogs?
Does all commits existed became plain text "Gogs"?

https://try.gogs.io/crayfield/TharBeATest
Yep.. it happened there.

I see... so even though I am pushing as crayfied it's using the username from the git config.
I'm not sure I like that. So if I set the username in my git config to crayfield will they match up?

@cb88 I'm not sure how Git works with local repo config and global config.
You should have configured your profile with git config --global very first time you use Git.
The reason the fake one is because Git requires to have one no matter you use it or not. So this behavior should only happen in a fresh server or linux/unix user account.

yes, but shouldn't that only apply to local repositories? If I commit as an authenticated user... that is the what I really want in the gogs repo. As multiple people could be using this same computer etc...

I'm not sure what do you exactly mean by local repository, but Gogs only setup a fake user field value but Git is the only who uses this value. If you want to argue about why Git does not use this value, Gogs can't help 😂
Also you said that the activity timeline looks OK? Can you take a screenshot because I'm confused now what is OK mean to you...

Ok, so the key is that my git user.name and user.email must match... then gogs knows that the commit matches up with my account (there is a URL to my profile now as you can see on the test repo)

To match you and your account on Gogs, the author email address must match.
However, Gogs is remote to you and in your case it just happens you start Gogs on your local machine, but you cannot expect Gogs to correct who should be the one commit the changes because it does not know what's your git config, Gogs only reads info from Git data.

And I think you mixed concepts about who did the commit of changes and who pushed commits.
Overall, basically, your Git config is wrong and you should set user.name and user.email in global git config.

I think this is already implemented.

It is possible to make as on GitHub?
In a web it is possible to see a сommitter name (username or full name) if an author name (git config user.name) or an e-mail (git config user.email) disperses from a сommitter namer (full name) and a сommiter e-mail.
As a result it was possible to see from under what account it is made commit.


Feature/x86

I don't currently have a way to test this, is it WIP or needs testing?

I don't currently have a way to test this, is it WIP or needs testing?

needs testing. I don't think this is urgent, prob don't need to test until you have a working x86 box

Okay, still standing by on testing.  Guessing beginning of next week


Output when doing shell pipes and redirects

Currently, redirecting or piping output is troublesome as output has headings when using globs. This should only be enabled with a flag so that piping / redir is easier in the default case

v0.2.0


feat(ci): test standalone cluster in main-cron, with e2e test

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] 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.

Passed. Will run it 2 more times to make sure it's stable.


Question: how to create nullable field in schema when database already exists ?

Describe the bug
Adding a new field to my plug-in schema.
After running cli migrate, I get:
error: column "last_api_login" of relation "user" contains null values
I tested input:false, defaultValue:"" etc... but none worked. What's the appropriate way ?
Thank you

Well turned out I had to add required: false, which I assumed was the default


Training for continuous values not working.

Hello,
I implemented the learning for continuous targets (they are between -3.5 and 3.5).
In fast_lsa_t.py I did self.dense = nn.Linear(opt.embed_dim, 1) (because we need 1 continuous output), I removed the softmax in the apc_trainer.py and added MSELoss as loss function. But when I print the predictions and the targets, it seems that the algorithm is not learning.
Is this normal? I have 3000 sentences, is it possible that the algorithm does not have enough data to learn continuous value, or I am probably doing something wrong?
Thank you.

The reason may be that MSELoss is a unusual loss function for classification

Yes, but I have transformed the problem to regression, because I am predicting continuous values.

I mean, I am passing continuous values and I changed the model to output 1 continuous value (no softmax)

Maybe you can push your change to your branch, so I can try to figure out where is wrong if I got time

OK. I created a new fork repo because I have problems creating the pull request: https://github.com/pepi99/pyabsa-regr
Here is a picture with the files that I changed:

OK, I will try to make it work, but it may take a lone time due to my tight schedule

Ok, we can work together if you want! When are you free

I have changed some stuff here (float(item.polarity) and float(item[label_name]), because otherwise it is looking for discrete values and changing my polarities. Maybe it messes something with the clusters, IDK..

I am really sorry that i can not proceed on this issue due to my schedule, and this is not a widely-needed demand, so I will close this issue.


Cannot read property 'getCurrentLocation' of undefined

I'm trying to setup a router using V3 and following the basic example of the documentation and I have the following error:
Cannot read property 'getCurrentLocation' of undefined
Here is my whole app:
import * as React from 'react';
import * as ReactDOM from 'react-dom';
import {Router, Route} from 'react-router';

class A extends React.Component {
    render() {
        return 'A';
    }
}

class B extends React.Component {
    render() {
        return 'B';
    }
}

ReactDOM.render(
    <Router>
        <Route path="/" component={A}>
            <Route path="callback" component={B}/>
        </Route>
    </Router>,
    document.getElementById('app')
);

Inspired from the route configuration doc:
https://github.com/ReactTraining/react-router/blob/v3/docs/guides/RouteConfiguration.md

The code in that guide is not complete. You need to pass a history prop to the router.


UI Does not work with the Material design theme.

Bummer

May just be styling issues, the functionality seems to work fine actually, after trial and error I was messing with the CSS and overwrote some styles using Flexbox, and i was about to get it to properly display. The main issue is that elements collapse on each other. I lost track of time and accidentally closed Atom, so my changes didn't save, (i was just using the built in DevTools), if I have time, i'll try to replicate it tomorrow.

What exactly is not working ?


multilingual available with Business subscriptions

Hello,
I see the instructions from this article are valid for both Business and Enterprise subscriptions.
however here:
https://docs.microsoft.com/en-us/office365/servicedescriptions/office-applications-service-description/office-applications-service-description
it lists the business subscriptions as not including a so-called "multilingual user interface".
can you confirm that we do have proofing tools etc also with Business subscribers?
or what does that "multilingual user interface" stand for?

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

ID: faba8b5e-afa4-9040-d577-104923c72ad9
Version Independent ID: 67d9044e-a683-75c1-2aaf-3a8bd06f3552
Content: Overview of deploying languages for Microsoft 365 Apps
Content Source: DeployOffice/overview-deploying-languages-microsoft-365-apps.md
Service: o365-proplus-itpro
GitHub Login: @jwhit-MSFT
Microsoft Alias: jwhit

hello everyone, any news on this question?

Checking again on the status here...


Could not read VTU file with appended data when header is encoded separately

I was playing around with vtu meshes with different appended data formats, and it seems, that meshio assumes the header to be encoded with the following data (uncompressed, base64) in the appended section (maybe even in data-array sections). So I couldn't load the following mesh:
example
Isn't that file written correctly? Paraview was able to read it.

I guess we could check the string of the appended data, if there are one or two = after the header.

The difference in encoding is demonstrated here:
import base64
import numpy as np

vtu_to_numpy_type = {
    "Float32": np.dtype(np.float32),
    "Float64": np.dtype(np.float64),
    "Int8": np.dtype(np.int8),
    "Int16": np.dtype(np.int16),
    "Int32": np.dtype(np.int32),
    "Int64": np.dtype(np.int64),
    "UInt8": np.dtype(np.uint8),
    "UInt16": np.dtype(np.uint16),
    "UInt32": np.dtype(np.uint32),
    "UInt64": np.dtype(np.uint64),
}
numpy_to_vtu_type = {v: k for k, v in vtu_to_numpy_type.items()}

header_type = "UInt32"
data = np.arange(2, dtype=np.float64)

data_bytes = data.tostring()
# collect header
header = np.array(
    len(data_bytes), dtype=vtu_to_numpy_type[header_type]
)
print(base64.b64encode(header.tostring() + data_bytes).decode())
print(base64.b64encode(header.tostring()).decode() + base64.b64encode(data_bytes).decode())

Which gives:
EAAAAAAAAAAAAAAAAAAAAAAA8D8=
EAAAAA==AAAAAAAAAAAAAAAAAADwPw==

where the first one is in the way meshio exports data.

So if header_type == "UInt32" we should check if the 7. and 8. character is = and if header_type == "UInt32" we should check if the 12. character is = to determine if the header was encoded separately.

I can't recall too well, but I thought the =s where just blanks to fill the block if the data is not enough. So if there was more header data, there'd be no =. Could this be?

Yep, they are added to make the last encoded block contain four Base64 characters.
When the header is encoded separately, they are added to make it a 8-character or 12-character string.

Okay, yes. So does paraview open both variants, header + data encoded together and encoded separately?

Yes, I can open both.

The format (page 16) isn't too clear about it. Well, if paraview can open it, so should meshio. Do you want to prepare a PR? A comment here and there would be in order, perhaps with a link to this report.

@MuellerSeb I might have a fix for this. However, the mesh you attached does lead to another problem with meshio, since it seams to implement an unsupported cell type (11). Were you able to read it after manually fixing the strings?


Refactor foundation.dart to be dart:ui agnostic

(Related to https://github.com/flutter/flutter/issues/29234)
There are two dependencies on dart:ui types within foundation.dart:

The VoidCallback type, which is exported from basic_types.dart
Window, which is part of a larger flutter/ui specific dependency on the binding class.

To handle the former case:

Introduce a VoidCallback type into the dart_sdk into dart:core. This would allow to remove the re-export from basic_types without it becoming a breaking change. dartui would still be able to use VoidCallback.

Increasing the API surface of dart:core is probably not desirable in general, though the contract of VoidCallback is fairly trivial. There is also an existing VoidCallback type (https://github.com/dart-lang/sdk/blob/edca6169c1c040702acde237e605be0bfe037326/sdk/lib/web_sql/dart2js/web_sql_dart2js.dart#L131) which seems to indicate that Flutter is not the only team that found it useful.

Add a new VoidCallback type to basic_types that mirrors the implementation in dart:ui and re-export that one instead.

This may cause issues in places where users import both dart:ui and package:flutter/foundation.dart. I'm less clear on what the failure mode is here. Depending on how severe it may be a breaking change

Rename VoidCallback to _VoidCallback in dart:ui and introduce a new VoidCallback type in basic_types.dart.

This is a breaking change in the case where someone is using VoidCallback by importing dart:ui and not package:flutter/foundation.dart. Because Function types are mostly structural, there are no type issues.
To handle the latter case:

Lift binding.dart out of foundation, and into a new layer which handles integration of dart:ui and foundation.

The binding class itself may be breaking the flutter/foundation contract since it is highly dependent on dart:ui via the Window, and conceptually via the flutter service extensions. This might be a breaking change, depending on how this Binding class is currently being used. To make it less breaking, we could leave an abstract interface behind.

Introduce a new library called foundation_core.dart in flutter/lib which re-exports all of foundation besides the binding.

This doesn't solve the underlying problem completely, but it is fully non-breaking and provides a library which meets the guarantees of foundation.
cc @jacob314

cc @Hixie @goderbauer for a framework perspective

For the VoidCallback case, my preference would be option #1 or #3 as they minimize confusion from the Analyzer and users about which VoidCallback type to use. For example, would be nice for users to not get code completion suggestions to import VoidCallback from dart:ui.

I prefer option 1, but it has not gotten much support from the Dart-side: https://github.com/dart-lang/sdk/issues/27791

There should not be any need to export typedefs, their name and the library that defines them are irrelevant.
Replace export 'dart:ui' show VoidCallback; with typedef VoidCallback = void Function();

We'd rather have only one place that defines VoidCallback because otherwise the documentation can be quite confusing (not to mention that we'd need to duplicate all the docs, etc).


how to add a New chart type

Is your feature request related to a problem? Please describe.
https://Github.com/airbnb/superset/pull/3013
Describe the solution you'd like
I want to add chart type use echart, then i follow https://Github.com/airbnb/superset/pull/3013, but it did'nt work. some body can help me?
Describe alternatives you've considered
https://Github.com/airbnb/superset/pull/3013

Please fill out the full issue template so that the project maintainers can provide assistance.

ref:
https://github.com/apache/incubator-superset/pull/8229
https://github.com/apache-superset/superset-ui-plugins/pull/202


Add more info and dependencies to pyproject.toml.

Probably should remove --no-index from pip install.

@messense Thanks! Like this: https://github.com/pymc-devs/nutpie/pull/31 ?

This info is in the pyproject on main now


Unfortunately all 120 could not be downloaded because some images were not downloadable. 0 is all we got for this search filter!

Hi, thanks for the wonder script.
I met a problem when downloading image if more than 100.
if the --limit is not specificed or less than 100, everything goes OK.
However, if the limit was set more that 100, it always gave ''
0 is all we got for this search filter!
chromedriver 2.37 is installed.
Full log:
`python google_images_download/google_images_download.py -k 'clouds overcast sky' --image_directory 'overcast'  -l 120 -cd /usr/bin/chromedriver
Item no.: 1 --> Item name = clouds overcast sky
Evaluating...
Getting you a lot of images. This may take a few moments...
Reached end of Page.
Starting Download...
Unfortunately all 120 could not be downloaded because some images were not downloadable. 0 is all we got for this search filter!
Errors: 0
`
When I debug the code, it went to no_link branch in _get_next_item() and _get_all_items().

ps. I am behind a proxy. However when the limit is less than 100, the script works. So I guess it is not the reason.

I finally solve the problem by adding
from pyvirtualdisplay import Display display = Display(visible=0, size=(800, 600)) display.start() and removing the options.add_argument("--headless")


Added support for 'length' attribute for plugins.

This feature allows one to truncate the line length of any line (title or menu bar item) to a specified length, per line.
I have also implemented a default line length of 20 characters, to prevent scripts from outputting an unnecessarily long title, or menu bar item, such as in #124.
One may override the default line length by using the length attribute.
Example:
'This is a very long string of text that needs to be longer than 20 characters | length=100'
||
'This is a very long string that can be any length so we will disable the truncation | length=-1'

I will also have to submit a PR for the plugins repo to list this attribute.

Why not just truncate in your shell script?
$ echo "this is a long string" | cut -c1-10
this is a

That's nice !
Should we provide a way to see the full text though ? Maybe by hovering over the menu item ?
What happens if we pass length=0 ? Could that make is so that the text is not truncated ?

@jcaille I figure some people may have a need for a zero-length string, so making length=0 would prevent users from doing that.
We definitely need a way to view the full length though. I think that hovering is our safest bet. Scrolling would be tough because our script should ideally be stateless,
@0wnrepo You're probably right. Especially if BitBar ever gets a config file, and this ends up as one of the config variables.

@jcaille So having a tooltip provide the full string doesn't look terrible. I was also thinking that we could do what most native menu's do, and provide more detail (i.e. the full string) when you press and hold the option key.
This is a 100 character tooltip truncated to the default 20 characters:

This is a 300 character tooltip truncated to the default 20 characters:

When it's truncated, we should add …, don't you think?

Heh, probably. Otherwise people may not realize why their output is missing.

So the option key modifier works well, as long as your string isn't ridiculously long...It seems like anything over ~200 characters gets automatically truncated. So we may be better off with the tooltip, obviously it's up to the community to decide.
No option modifier:

With option modifier:

(Also, I implemented the ellipsis per request.)

I think the tooltip is a good idea - maybe we can push the default limit up a bit?
We should keep the option alternative as something people can provide for themselves later. That'd be cool.

I agree. I've found, even with some simple scripts, that 20 characters is a bit too small. Are you referring to passing an option param after the string? Or were you just referring to people who want that, can implement it themselves?

(see my edit)

Yes. But that interface might not work - because it's essentially a different option completely. Needs some thought.

I think the length keyword is a good idea, but I would personally remove the default for the following reason

It does not encourage plugin writer to be succinct.
The cut might be weirdly placed, or cut off important information.
It might be seen as a bug by new developers who might wonder why their text is being truncated (and maintaining option is not easily discoverable)
It might break existing plugins
Bitbar can display errors inline (allowing to easily see what went wrong). Truncating those errors make it less obvious.

Just keeping the length parameter available would allow plugins that might output very long text to toggle the option.

@jcaille makes some good points.

Fair enough. I didn't consider the issue with errors. But being considered a bug by anyone is definitely not ideal. That's good enough of an argument for me. I'll kill the default, and implement the tooltips.

I pushed a new commit. It includes the following:

Default limit is gone. You must specify length=some_positive_integer for any truncation to occur.

length=0 will simply leave an ellipsis with a tooltip of the full message.

An ellipsis will be appended to any truncated string.
A tooltip will be added to any truncated menu item, that will display the full string.

I tested this param with multiple different cases:

[x] With params color, size, font (all possible combinations)
[x] With param bash
[x] With param dropdown=false

All mentioned tests worked flawlessly.

Excellent - I wonder if you could update the README too?

Absolutely. I didn't realize that the API section was moved here.

Nice work :)

... while you're in there, could you stick this link on the shebang word? https://en.wikipedia.org/wiki/Shebang_(Unix)

Haha certainly

Alright, README has been updated with the shebang link and the necessary length documentation.

Excellent. Nice work. Thanks to everyone for their input.

Is there a way to pad a string using the length parameter?
The output from the ping plugin has different widths, depending on the ping time - I would love to have a fixed width - say 10 characters so that the output doesn't "jump" when it's getting longer/shorter. Or is there a way to do it with bash? I tried using printf (printf "%10s" "$MSG") but this didn't work. It works in the terminal but somehow the leading spaces are removed.

@mm2270 Please open a new issue.

Will do. Thanks.


COM interfaces are not declared with the partial keyword

0.1.506-beta
I wanted to try something out by writing this:
namespace Windows.Win32.UI.Shell
{
    [CoClass(typeof(ShellLink))]
    internal partial interface IShellLinkW { }
}

But the IShellLinkW.g.cs declaration does not use the partial keyword:
[Guid("000214F9-0000-0000-C000-000000000046"),InterfaceType(ComInterfaceType.InterfaceIsIUnknown),ComImport()]internal interface IShellLinkW
{

Using partial on a COM interface would lead to undefined member ordering inside the interface, which would be disastrous. I get it in your case you just want to add attributes, but that's a very narrow case for using partial and IMO the risk that folks might use it for other purposes and end up corrupting the vtable would be huge.
Let's keep your other issue to track adding the attributes you want on the interface within the generated code.

I think the entire risk could be mitigated with a new source generator error if any user-defined parts contain (non-type) members. But maybe the use case of adding attributes is too niche to he worth it.

Honestly, a C# compiler-level warning when methods are added to two parts of a partial interface that includes interop attributes would probably be appropriate, IMO. If the compiler emits one (maybe it already does), then I wouldn't object to this suggestion. Can you test whether it already does, and if it doesn't, maybe file a bug at https://github.com/dotnet/csharplang?

There is no warning (sharplab).
Since this affects existing C#, I expect they will only want to add this in a warning wave. Things like this tend to be designed at dotnet/roslyn rather than dotnet/csharplang, which is good news because only a few things that require a spec change ever ship and it's usually years out.

I checked on the repo contribution Discord channel and Fred confirmed that this would probably be a feature request for the Roslyn repo. https://github.com/dotnet/roslyn/issues/55006

@kant2002 does your COM Wrapper work in winforms need a partial interface at all? I believe winforms is going to use cswin32 without marshalling, so the interfaces will not be generated? Just making sure this doesn't end up blocking your work.

I’m was not able to properly check current state of CsWin32 , but in general I never use partials for COM interface declaration so I’m fine with that. What’s important is that DllImport gives IntPtr for interface parameters. All I need is control of how COM proxy is created.
With DllImport + IntPtr parameter I do that manually. With LibraryImport + IComInterface I think it’s possible to provide marshalers so I can control that part.
Need more time to properly get context to answer you more targeted

Thanks, we should know more when they start to look at it.

Quick glance
From what I see on feature/wincs32

DllImport and not LibraryImport (given how DllImport user, I think switch to LibraryImport when allowMarshaling = false is quite possible. Without that it indirectly affects WinForms or any app who want trim.
comInterop.useIntPtrForComOutPointers is interesting. With comInterop.useIntPtrForComOutPointers = true I see how current codegen can be used almost identically for current approach, but then we need additional comInterop.useIntPtrForComInPointers, or is there assumption that IntPtr returned by ComWrappers.GetOrCreateComInterfaceForObject should be blindly converted to, for example, winmdroot.System.Com.IUnknown*?
struct + Vtbl as interfaces. I do remember that I was suggested by Jan Kotas or Aaron that instead of creatin VTBL structures I directly operate on pointers. Otherwise it produce a lot of metadata. Maybe recent changes in P7 by @MichalStrechovsky trim that out, but otherwise it looks like additional bloat.

RCW
Currently RCW build like this.
public HRESULT Init(IntPtr hwndEdit, IEnumString punkACL, IntPtr pwszRegKeyPath, IntPtr pwszQuickComplete)
{
	var punkACLPtr = WinFormsComWrappers.Instance.GetOrCreateComInterfaceForObject(punkACL, CreateComInterfaceFlags.None);
	return ((delegate* unmanaged<IntPtr, IntPtr, IntPtr, IntPtr, IntPtr, HRESULT>)(*(*(void***)_wrappedInstance + 3)))
			(_wrappedInstance, hwndEdit, punkACLPtr, pwszRegKeyPath, pwszQuickComplete);
}

With current autogenerated structs
IAutoComplete2* instance;
public HRESULT Init(IntPtr hwndEdit, IEnumString punkACL, IntPtr pwszRegKeyPath, IntPtr pwszQuickComplete)
{
	var punkACLPtr = WinFormsComWrappers.Instance.GetOrCreateComInterfaceForObject(punkACL, CreateComInterfaceFlags.None);
	return instance->Init(hwndEdit, (winmdroot.System.Com.IUnknown*)punkACLPtr, (winmdroot.Foundation.PCWSTR)pwszRegKeyPath, (winmdroot.Foundation.PCWSTR)pwszQuickComplete);
}

or
IAutoComplete2* instance;
public HRESULT Init(IntPtr hwndEdit, IEnumString punkACL, string pwszRegKeyPath, string pwszQuickComplete)
{
	var punkACLPtr = WinFormsComWrappers.Instance.GetOrCreateComInterfaceForObject(punkACL, CreateComInterfaceFlags.None);
	return instance->Init(hwndEdit, (winmdroot.System.Com.IUnknown*)punkACLPtr, pwszRegKeyPath, pwszQuickComplete);
}

Still require manual passing all parameters. AddRef/Release/QI function is pointless because they never would be used.
CCW
Currently CCW build like this
[UnmanagedCallersOnly]
private static int Seek(IntPtr thisPtr, long dlibMove, SeekOrigin dwOrigin, ulong* plibNewPosition)
{
	try
	{
		Interop.Ole32.IStream instance = ComInterfaceDispatch.GetInstance<Interop.Ole32.IStream>((ComInterfaceDispatch*)thisPtr);
		instance.Seek(dlibMove, dwOrigin, plibNewPosition);
		return S_OK;
	}
	catch (Exception ex)
	{
		Debug.WriteLine(ex);
		return ex.HResult;
	}
}

if allowMarshaling = true then this can be used like this. Also here the reason why partial is not needed.
[UnmanagedCallersOnly]
private static int Seek(IntPtr thisPtr, long dlibMove, winmdroot.System.Com.STREAM_SEEK dwOrigin, ulong* plibNewPosition)
{
	try
	{
		Windows.Win32.System.Com.IStream instance = ComInterfaceDispatch.GetInstance<Windows.Win32.System.Com.IStream>((ComInterfaceDispatch*)thisPtr);
		instance.Seek(dlibMove, dwOrigin, plibNewPosition);
		return S_OK;
	}
	catch (Exception ex)
	{
		Debug.WriteLine(ex);
		return ex.HResult;
	}
}

not much of a difference.
Additional comments
So basically for one scenario I need structs, for another interface.
Maybe instead of structs if allowMarshaling = true, can be generated just Vtbl part that way I can write code for RCW like this.
IAutoComplete2Vtbl** instance;
public HRESULT Init(IntPtr hwndEdit, IEnumString punkACL, IntPtr pwszRegKeyPath, IntPtr pwszQuickComplete)
{
	var punkACLPtr = WinFormsComWrappers.Instance.GetOrCreateComInterfaceForObject(punkACL, CreateComInterfaceFlags.None);
	return (*instance)->Init((IntPtr)instance, hwndEdit, (winmdroot.System.Com.IUnknown*)punkACLPtr, (winmdroot.Foundation.PCWSTR)pwszRegKeyPath, (winmdroot.Foundation.PCWSTR)pwszQuickComplete);
}

I may screw with * here, but that's for illustrative purposes for now.

@kant2002, thank you for your write-up.

Taking IntPtr for input parameters in lieu of COM interfaces should be possible.  If that's important to you, can you file a new issue to track that? Please comment on #669 to carry on this discussion.
Yes, the struct vtbl we generate today isn't particularly 'trimming friendly', and we expect to switch to the pointers+casting approach you refer to in the future. This is now tracked by #668.

I think I'll need more time to study the rest of your comments.

I re-read this whole thread. I'm not clear on whether the partial modifier added to the interface or the struct code for a COM interface is useful or not, as it seems @kant2002 was making the case both for and against. As that seems to be the last request uniquely described by this issue, can anyone clarify on requirements here? Otherwise, I'll close the issue as Won't Fix in a few days. Of course feedback after that on this issue would be welcome and we could reactivate if needed.

@kant2002 what changes in .NET8? Do ComWrappers get easier?

Interop team plans to add source generators which would works with LibraryImportGenerator . I do not sure if happens for sure, but plans are really here.

Could you please link the associated issue?

Do not know the issue, but plans are present https://twitter.com/mstrehovsky/status/1573203145236582403?s=46&t=mSvL8EQaVWZs8crDDX4qtg Interop team usually rock solid when they implement something.

I'll close this based on this feedback. We can reactivate if anyone decides this is important.


Embeddings: Expose the prompt templates found in config_sentence_transformers.json, when available

Feature request
Many embeddings models are trained with task-specific prefixes, and these prefixes are often described in a file named config_sentence_transformers.json (example - see the 'prompts' object). It would be nice if the content of that object was exposed in some way.
Motivation
Without this, there's no way for me to configure which task-specific model I'd like to use at runtime and get the performance out of that model that I need. It limits the set of models that my code can run against, as I need to go and find the task-specific prefixes out of band.
I'm currently working on a simple text classifier that uses sentence embeddings models, but I've also encountered this while working on a RAG pipeline. I'd like to be able to initialize a feature-extraction pipeline, and inspect it to determine whether I need to add a task-specific prefix to the text that I'm about to embed with the pipeline.
Alternatively, I'd like it if the task-specific prefix were applied automatically by the pipeline itself, however I imagine that this would require the user to identify the task to the pipeline on creation, which may be a more complicated change than allowing me to inspect the config.
Your contribution
If the maintainers would provide some guidance for how this feature should be implemented, I'd be happy to submit a PR.

Hi there 👋 Since this is related to sentence-transformers and not transformers, I believe this feature is out of scope for transformers.js. Maybe it could be useful when sentence-transformers.js comes around though? 😉


Documentation: deployment guide for self-hosted

We need to document how to deploy with kubectl flannel.yml with step-by-step instructions including how your Kubernetes cluster needs to be brought up.
cc @tomdee @mikedanese

Need this! Before the official documentation is out, is there any link/doc I can read to set this thing up?

@tomdee Would something using kubeadm + the requirements for Flannel + the creation of resources from the latest version of the flannel.yml be good for you?


Can’t install pycrypto

Use ‘pip install crypto’ will stuck in ‘Building wheel for pycrypto’

Pycrypto needs a c compiler that can produce executables for Python to use. For now, only pure Python modules can be installed. Modules containing native components needs to be bundled with the app.


Update version of vendored dependency: packaging

Summary of changes

Update version of vendored dependency: packaging
Remove no longer necessary vendored dependency: pyparsing
Adapt existing tests and implementation to deal with the fact that `packaging>=23' is more strict
As a side-effect of running tox -e vendor jaraco.context also got updated (since it was not a very different release I kept this update)

I manually reverted all the other vendored files that got accidentally updated

Related: https://github.com/pypa/setuptools/discussions/3722
(For anyone reviewing the PR, you can ignore all the */_vendor/* but vendored.txt files, since those were changed by an automate process).
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d/.
(See documentation for details)

Thanks a lot for doing this!

Can we please have a release with these changes? It blocks downstreams from updating python-packaging.

Releasing as v67.


Support AWS Access Keys in retry_join

Background:
The Consul auto-join feature supports access-keys to discover nodes it should join. Ideally you should be using a role that has the describe-instance permission attached to it. This works great if your cluster is entirely within a single account.
However, there is currently no way use the auto join feature across AWS accounts with IAM roles. The solution for now is to use a AWS access keys to be able to describe instances in a different account.
Problem:
The run-consul script does not support access keys.
https://github.com/hashicorp/terraform-aws-consul/blob/db3d81e030f03da3832e3865806193400a7b576a/modules/run-consul/run-consul#L253
Solution
Add the ability to set both access_key_id and secret_access_key to the retry_join block as mentioned here: https://www.consul.io/docs/agent/cloud-auto-join.html#amazon-ec2

Since this isn't a super common use case, is it something you could add to your own config by overriding the configuration?


Crashing on setting _cancelledOnCompletedSource_ClearedOnSet to nil.

The following code often crashes on Signal.

  TOCFutureSource* futureResultSource = [TOCFutureSource futureSourceUntil:untilCancelledToken];

    PacketHandlerBlock packetHandler = ^(id packet) {
        if (![futureResultSource trySetResult:packet]) {;
            errorHandler([IgnoredPacketFailure new:@"Received another packet before relay socket events redirected to new handler."], packet, false);
        }
    };

Crash stack traces are showing that it crashes inside TOCFutureSource _tryComplete:succeeded:
Thread 2 name:
Thread 2 Crashed:
0   libobjc.A.dylib               	0x00000001938f0174 objc_release + 20 (objc-object.h:419)
1   Signal                        	0x0000000100289be0 -[TOCFutureSource _tryComplete:succeeded:] + 84 (TOCFutureAndSource.m:386)
2   Signal                        	0x0000000100289c8c -[TOCFutureSource trySetResult:] + 144 (TOCFutureAndSource.m:396)
3   Signal                        	0x000000010002ef38 __102+[CallConnectUtil_Server asyncFirstPacketReceivedAfterStartingSocket:untilCancelled:withErrorHandler:]_block_invoke + 52 (CallConnectUtil_Server.m:183)
4   Signal                        	0x00000001000d53e0 -[PacketHandler handlePacket:] + 28 (PacketHandler.m:21)
5   Signal                        	0x00000001000817d0 -[UdpSocket onReceivedData:withEventType:from:] + 428 (UdpSocket.m:114)
6   Signal                        	0x00000001000815dc onReceivedData + 140 (UdpSocket.m:92)
7   CoreFoundation                	0x000000018206f594 __CFSocketPerformV0 + 768 (CFSocket.c:2862)
8   CoreFoundation                	0x000000018206c23c __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 20 (CFRunLoop.c:1763)
9   CoreFoundation                	0x000000018206b4e0 __CFRunLoopDoSources0 + 260 (CFRunLoop.c:1809)
10  CoreFoundation                	0x0000000182069590 __CFRunLoopRun + 708 (CFRunLoop.c:2526)
11  CoreFoundation                	0x0000000181f952d0 CFRunLoopRunSpecific + 392 (CFRunLoop.c:2795)
12  Foundation                    	0x0000000182ecd4c4 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 312 (NSRunLoop.m:366)
13  Foundation                    	0x0000000182eefa5c -[NSRunLoop(NSRunLoop) runUntilDate:] + 104 (NSRunLoop.m:407)
14  Signal                        	0x0000000100084fc0 -[RunningThreadRunLoopPair runLoopUntilCancelled] + 212 (ThreadManager.m:33)
15  Foundation                    	0x0000000182fb5db4 __NSThread__main__ + 1068 (NSThread.m:1113)
16  libsystem_pthread.dylib       	0x000000019410bdc4 _pthread_body + 160 (pthread.c:644)
17  libsystem_pthread.dylib       	0x000000019410bd20 _pthread_start + 156 (pthread.c:680)
18  libsystem_pthread.dylib       	0x0000000194108ef4 thread_start + 0 (pthread_asm.s:188)

which refers to
-(bool) _tryComplete:(id)value succeeded:(bool)succeeded {
    bool didSet = [future _ForSource_tryComplete:value succeeded:succeeded];
    if (!didSet) return false;

    [_cancelledOnCompletedSource_ClearedOnSet cancel];
    _cancelledOnCompletedSource_ClearedOnSet = nil;
    return true;
}

My assumption is that _cancelledOnCompletedSource_ClearedOnSet is already deallocated at that point. Thoughts @Strilanc?

I'm not sure how it could be deallocated at that point.
When a new future source is created, a new cancel token source is created and assigned to the future source's _cancelledOnCompletedSource_ClearedOnSet field. Nothing else holds that reference. In order for it to be deallocated, either one of the completion methods (which clear the field) needs to be called, or the future source needs to be deallocated. Unless the future source is losing references in callbacks in a re-entrant way, and that makes the reference count hit zero despite the existence of a reference on the stack, I'm befuddled.
Maybe an ARC flag is set incorrectly somewhere?
I think I need more context about the failure to solve this. Are you able to trigger this failure at will? If so then the first order of business is to barf NSLogs all over the place, between every line of the _tryComplete: method and also in the dealloc methods of TOCFutureSource, TOCFuture, TOCCancelToken, and TOCCancelTokenSource.
-(bool) _tryComplete:(id)value succeeded:(bool)succeeded {
    NSLog(@"[%@ _tryComplete:%@ succeeded:$@] 1", self, value, @(succeeded));
    bool didSet = [future _ForSource_tryComplete:value succeeded:succeeded];
    NSLog(@"[%@ _tryComplete:%@ succeeded:$@] 2", self, value, @(succeeded));
    if (!didSet) return false;
    NSLog(@"[%@ _tryComplete:%@ succeeded:$@] 3", self, value, @(succeeded));

    [_cancelledOnCompletedSource_ClearedOnSet cancel];
    NSLog(@"[%@ _tryComplete:%@ succeeded:$@] 4", self, value, @(succeeded));
    _cancelledOnCompletedSource_ClearedOnSet = nil;
    NSLog(@"[%@ _tryComplete:%@ succeeded:$@] 5", self, value, @(succeeded));
    return true;
}

That will tell us if the dealloc is somehow happening during the method. Note that, if this is an ARC-related bug, introducing code using "self" may mask it, so it's also interesting if it becomes impossible to repro when the logging is there.

No, I'm sadly unable to trigger this at will. I'm unsure what triggers it but it crash logs like the one above are present in very large numbers, all having a pretty similar trace. The fact that the trace reveals that objc_release is crashing the app, my assumption would be that  _cancelledOnCompletedSource_ClearedOnSet is deallocated before reaching that point.

This seems to be a quite common crash.
@Strilanc: Any ideas on a fix?

 Simulator may not be able to trigger it.".
What other properties does the crash have? Did it only start happening recently? Does it only happen on a certain version of iOS, or on a certain type of iphone?
Do you happen have debug code lying around to put signal into a connect/disconnect loop?

@Strilanc: No, as far as the crash logs I have are going, it's been there since v1. But at the time it was not looked into it because other crashes happened more frequently.
The concurrency clearly adds to the difficulty of debugging.

Is there any other debug information, like stack traces of concurrently running threads or the types of the last couple packets / next couple packets?

I just emailed you a few symbolicated crash logs.

I didn't find anything helpful in the concurrently running threads. Did you?

Not yet, but I also haven't run anything yet.
I went to start testing, but Signal kept crashing instead of running. I tracked it down to a bunch of interesting ways that Signal trashes its own data when you run the tests.
Hopefully this isn't too hard to reproduce.

This is still happening in the wild, though I've never reproduced it.
One hint is that we're not using the vanilla TOCFuture init, but rather
TOCFutureSource *futureResultSource = [TOCFutureSource futureSourceUntil:untilCancelledToken];

Which is invoked through our retriable future operation. We're using futures throughout the app, but this is the only invocation leading to crashes.
   TOCUntilOperation operation = ^(TOCCancelToken *internalUntilCancelledToken) {
      return [self asyncAttemptResolveThenConnectToUdpRelayDescribedBy:sessionDescriptor
                                                        untilCancelled:internalUntilCancelledToken
                                                      withErrorHandler:callController.errorHandler];
    };

    TOCFuture *futureRelayedUdpSocket = [TOCFuture retry:[TOCFuture operationTry:operation]
                                              upToNTimes:MAX_TRY_COUNT
                                         withBaseTimeout:BASE_TIMEOUT_SECONDS
                                          andRetryFactor:RETRY_TIMEOUT_FACTOR
                                          untilCancelled:[callController untilCancelledToken]];

+ (TOCFuture *)retry:(TOCUntilOperation)operation
          upToNTimes:(NSUInteger)maxTryCount
     withBaseTimeout:(NSTimeInterval)baseTimeout
      andRetryFactor:(NSTimeInterval)timeoutRetryFactor
      untilCancelled:(TOCCancelToken *)untilCancelledToken {
    ows_require(operation != nil);
    ows_require(maxTryCount >= 0);
    ows_require(baseTimeout >= 0);
    ows_require(timeoutRetryFactor >= 0);

    if (maxTryCount == 0)
        return TOCFuture.futureWithTimeoutFailure;

    TOCFuture *futureResult =
        [TOCFuture futureFromUntilOperation:operation withOperationTimeout:baseTimeout until:untilCancelledToken];

    return [futureResult catchTry:^(id error) {
      bool operationCancelled     = untilCancelledToken.isAlreadyCancelled;
      bool operationDidNotTimeout = !futureResult.hasFailedWithTimeout;
      if (operationCancelled || operationDidNotTimeout) {
          return [TOCFuture futureWithFailure:error];
      }

      return [self retry:operation
               upToNTimes:maxTryCount - 1
          withBaseTimeout:baseTimeout * timeoutRetryFactor
           andRetryFactor:timeoutRetryFactor
           untilCancelled:untilCancelledToken];
    }];
}

Thanks, that's useful information. I'll take another stab at reproducing the issue.
A self-contained repro would be ideal, of course. Concurrency bugs are extremely difficult to reproduce, because they can depend on context like "this ran on the networking thread" or "the user was using processor version Y". It's hard to know what to try next, when staring a blank wall of "works in every situation I've tried".

It's hard to know what to try next, when staring a blank wall of "works in every situation I've tried".

Tell me about it.
I'm working on reproducing and I'll give you an update if I succeed.

@michaelkirk I noticed you forked the code. Let me know if you have basic question of how something works that's holding you up. Or if you figure out how to repro this thing!

Obtuse question, but in the first place, why are we nilling that reference explicitly vs just letting it happen on dealloc?

@michaelkirk I don't remember a reason for it to be necessary for correctness, as opposed to saving a bit of memory.
However, I do find it mentally convenient to nil. We're in a context where it's possible to create cycles (e.g. future sources containing callbacks to complete each other), and niling the references means I don't need to carefully consider if it might be keeping the source that holds it alive.

@michaelkirk Although, looking at it now, I'm a bit worried that when there's an exception it won't be cleared and that could cause a memory leak if the case wasn't carefully reasoned through at the time.


[DAPPS] Make tx notifications progress visible for an additional couple of seconds after reaching 100%

The tx notifications progress should be visible for an additional couple of seconds after reaching 100%.
Currently, it disappears even before 100%

Steps:

go to any DAPP
make a tx
wait while progress is finished

checked


Don't hardcode fixed size for GetMemory in ChunkedTests

Pass the length from GetMemory to the receiver so it knows what chunk size to expect.
Should fix the ChunksWithGetMemoryWithInitialFlushWorks issue in the dep updates https://github.com/aspnet/AspNetCore/pull/7640
/cc @jkotalik @Tratcher

Superseded by https://github.com/aspnet/AspNetCore/pull/7671


Webcomponent Wrapper (re-)mount fix

In https://github.com/cloudera/hue/pull/3102 I provided a fix for the broken unmount behaviour and mentioned that we might also have an issue in the mount case. Now we ran into exactly this issue. This PR includes the fix for the case where you create a custom element, attach it to the DOM, remove and then re-add it. In our application we have situations where the host app (also Vue) decides to briefly remove a web component from the DOM and reattaches it a few moments later.
I've isolated the issue in this Stackblitz. The problem seems to be, that it's not possible to unmount and then re-mount a Vue 3 Application. Once it's unmounted you have to recreate it.
What changes were proposed in this pull request?

connectedCallback now always creates a Vue instance and mounts it. Before creation was done once in the constructor.

How was this patch tested?
Here you can find a side by side comparison of the current and patched version of the webcomponent wrapper. In the current version it's not possible to reattach the component to the DOM once it has been removed.
https://stackblitz.com/edit/vitejs-vite-syzsx9?file=src%2Fmain.js

@naltatis Thanks for contributing, we will go through your PR shortly.

Will be done soon :)
https://github.com/cloudera/hue/pull/3133

@naltatis Version 5.0.0 published

Nice! Will you also publish vue3-webcomponent-wrapper?

@naltatis Its published.

Great. Thanks so much @sreenaths 🙌

@naltatis Thank you so much for the PRs 🙌

@naltatis I have invited you as a maintainer of vue3-webcomponent-wrapper npm package. Feel free to publish future changes.

Thanks 🙏


Add a Zephyr example.

Arduino example compiles with warning...
/home/zak/Development/Arduino/libraries/notecard-env-var-manager/examples/arduino/arduino.ino: In function 'void envVarManagerCb(const char*, const char*, void*)':
/home/zak/Development/Arduino/libraries/notecard-env-var-manager/examples/arduino/arduino.ino:40:16: warning: 'char* strncpy(char*, const char*, size_t)' specified bound 16 equals destination size [-Wstringop-truncation]
   40 |         strncpy(cache->valueB, val, sizeof(cache->valueB));
      |         ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/zak/Development/Arduino/libraries/notecard-env-var-manager/examples/arduino/arduino.ino:37:16: warning: 'char* strncpy(char*, const char*, size_t)' specified bound 16 equals destination size [-Wstringop-truncation]
   37 |         strncpy(cache->valueA, val, sizeof(cache->valueA));
      |         ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

We can clear the warning by substracting 1 from the strncpy() size.
Example:
strncpy(cache->valueB, val, sizeof(cache->valueB)-1);

https://stackoverflow.com/questions/50198319/gcc-8-wstringop-truncation-what-is-the-good-practice

We also need to find a new home for the Zephyr example. The Arduino IDE will try to compile any folder it finds under examples/ folder.
/home/zak/Development/GitHub/blues/notecard-env-var-manager/examples/zephyr/build/zephyr/misc/generated/syscalls_links/include_zephyr_linker_common-rom (No such file or directory)
Invalid library found in /home/zak/Development/Arduino/libraries/notecard-env-var-manager: no headers files (.h) found in /home/zak/Development/Arduino/libraries/notecard-env-var-manager

Not seeing that on my end. I launch Arduino IDE, open examples/arduino/arduino.ino, and it builds without errors. Are you doing something different from that?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.02%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
68

Relevant Lines:
68

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.02%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
68

Relevant Lines:
68

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.02%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
68

Relevant Lines:
68

💛  - Coveralls


:stable tag is not working anymore

Describe the bug
stable tag is not working anymore after update to the new 0.22.5 version from 0.22.4 version
Steps to reproduce
use the :stable tag pull down the latest :stable the page is not working anymore.
The version of Memos you're using.
0.22.4
Screenshots or additional context
not open page anymore.

2024/09/06 04:22:42 ERROR failed to migrate error="strconv.Atoi: parsing "00_reaction.sql": invalid syntax\nfailed to convert patch version to int: 00_reaction.sql\ngithub.com/usememos/memos/store.(*Store).getSchemaVersionOfMigrateScript\n\tgithub.com/usememos/memos/store/migrator.go:233\ngithub.com/usememos/memos/store.(*Store).normalizedMigrationHistoryList\n\tgithub.com/usememos/memos/store/migrator.go:272\ngithub.com/usememos/memos/store.(*Store).preMigrate\n\tgithub.com/usememos/memos/store/migrator.go:151\ngithub.com/usememos/memos/store.(*Store).Migrate\n\tgithub.com/usememos/memos/store/migrator.go:37\nmain.init.func1\n\t./main.go:61\ngithub.com/spf13/cobra.(*Command).execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:989\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1041\nmain.main\n\t./main.go:171\nruntime.main\n\truntime/proc.go:271\nruntime.goexit\n\truntime/asm_amd64.s:1695\nfailed to get schema version of migrate script\ngithub.com/usememos/memos/store.(*Store).normalizedMigrationHistoryList\n\tgithub.com/usememos/memos/store/migrator.go:274\ngithub.com/usememos/memos/store.(*Store).preMigrate\n\tgithub.com/usememos/memos/store/migrator.go:151\ngithub.com/usememos/memos/store.(*Store).Migrate\n\tgithub.com/usememos/memos/store/migrator.go:37\nmain.init.func1\n\t./main.go:61\ngithub.com/spf13/cobra.(*Command).execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:989\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1041\nmain.main\n\t./main.go:171\nruntime.main\n\truntime/proc.go:271\nruntime.goexit\n\truntime/asm_amd64.s:1695\nfailed to normalize migration history list\ngithub.com/usememos/memos/store.(*Store).preMigrate\n\tgithub.com/usememos/memos/store/migrator.go:152\ngithub.com/usememos/memos/store.(*Store).Migrate\n\tgithub.com/usememos/memos/store/migrator.go:37\nmain.init.func1\n\t./main.go:61\ngithub.com/spf13/cobra.(*Command).execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:989\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1041\nmain.main\n\t./main.go:171\nruntime.main\n\truntime/proc.go:271\nruntime.goexit\n\truntime/asm_amd64.s:1695\nfailed to pre-migrate\ngithub.com/usememos/memos/store.(*Store).Migrate\n\tgithub.com/usememos/memos/store/migrator.go:38\nmain.init.func1\n\t./main.go:61\ngithub.com/spf13/cobra.(*Command).execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:989\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\tgithub.com/spf13/cobra@v1.8.1/command.go:1041\nmain.main\n\t./main.go:171\nruntime.main\n\truntime/proc.go:271\nruntime.goexit\n\truntime/asm_amd64.s:1695"

See the following pr for more info
https://github.com/usememos/memos/pull/3873

This issue probably shouldn't be closed until the fix makes it to the stable tag? That's what the install instructions guide you to use, so onboarding process is now completely dead for postgres users.


x/build/internal/task: provide a release archive with all the syso files, or release the files separately

What version of Go are you using (go version)?
1.17 SDK, via Bazel's rules_go: https://github.com/bazelbuild/rules_go.
Does this issue reproduce with the latest release?
Yes, the latest release (1.18.4  from go.dev/dl) has that issue.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/blorente/Library/Caches/go-build"
GOENV="/Users/blorente/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/blorente/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/blorente/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/homebrew/Cellar/go/1.18.4/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/homebrew/Cellar/go/1.18.4/libexec/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.18.4"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/0b/d4kmqgkx0w5bs1qb8cktwfpw0000gn/T/go-build1763548739=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I tried to build a binary for linux using the musl C toolchain.
rules_go tries to build the standard library by downloading a copy of the sdk in the host architecture (e.g., it would be https://go.dev/dl/go1.18.4.darwin-amd64.tar.gz for my architecture), and then doing roughly:

Create a new dir, let's call it foo.
Copy src and pkg there.
Set foo as the new GOROOT, and set GOOS and GOARCH to the target platform (in this case, GOOS=linux, GOARCH=x86_64
Build the standard library by calling go install.

This builds correctly with musl.
What did you expect to see?
However, if I try to link a binary that uses the race detector to this standard library, it will fail to find libc-specific symbols from the thread sanitiser. This issue has been solved in the precompiled syso files for race, in src/runtime/race.
And, in fact, if reproduce the process but manually place race_{GOOS}_{GOARCH}.syso in src/runtime/race, the syso is packed into the race.a archive, and the binary links successfully.
Since the go sdk is supposed to cross-compile, I would have expected this operation to succeed.
What did you see instead?
Since this tool strips the syso for platforms other than the host, the syso can't end up in the .a archive for the package.
Further info
I'm preparing to contribute a fix to rules_go, which involves getting a hold of the sysos somehow. My question is:
What would be the recommended way to get those syso files?
Should I download the src sdk (e.g. https://go.dev/dl/go1.18.4.src.tar.gz) and incur the bandwidth penalty? Should I download the binaries directly from the GitHub mirror, or should I do something else?

CC @golang/release.

Sorry not answering before. There is indeed no more to do, thanks for the investigation.


Socket Fix

Fixed our socket issue, it was an issue with where the connection is
being pointed. Look in client/eventSources/gameState for where the
change is located. Kind of just a hotfix really since I don't know why
pointing it there over '/' makes it attach to the socket properly.

This fixes #13


Feature Table

The feature table lists that Windows 10 E3 is not supported for ASR rules, however this article indicates that it does, just with limited capabilities for reporting, and not all rules are available.
https://docs.microsoft.com/en-us/windows/security/threat-protection/windows-defender-exploit-guard/attack-surface-reduction-rules-in-windows-10-enterprise-e3

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

ID: 84941b99-5200-b99d-f4d9-767da4241566
Version Independent ID: bab895c3-46d0-f530-dbf5-ebb7292f3769
Content: Use Windows Defender Exploit Guard to protect your network
Content Source: windows/security/threat-protection/windows-defender-exploit-guard/windows-defender-exploit-guard.md
Product: w10
GitHub Login: @andreabichsel
Microsoft Alias: v-anbic

I apologize for the ambiguity here. The product team wants to make it clear that ASR rules work far better with the complete Windows Defender Advanced Threat Protection available with E5. However, a limited set of ASR rules are available with an E3 license, but monitoring and reporting will have to be a diy solution in this case.


Extensions Issue

Type: Bug
Greetings,
Getting "Error while fetching extensions. XHR failed."
VS Code version: Code 1.90.2 (5437499feb04f7a586f677b155b039bc2b3669eb, 2024-06-18T22:34:26.404Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Hi @hassaanmshaikh
Thanks for reaching out. If you're using a VPN client or some local firewall, please adjusting the firewall settings to allow these requests. If the error is still persisting, it will help us to investigate, if you can share the network log ( command: Developer: Open Logs Folder).
Regards,
VS Marketplace Team

Can you elaborate on the local firewall settings. My vpn is not connected, and in regards to the firewall, I have visual studio in my allowed apps list.

I had tried everything online regarding the firewall and could not solve this issue. Finally was able to solve it with these steps:

CMD → echo %http_proxy%
%http_proxy% (no proxy).
http:// local:3000 (there is a proxy).
Removed any Proxy Server under System Environment Variables → Issue Resolved.

Thanks for the confirmation and sharing the details.


feat: refactor string package remove-first

Part of https://github.com/stdlib-js/stdlib/issues/1062
Description

What is the purpose of this pull request?

This pull request:

Refactors the string package @stdlib/string/remove-first
Adds @stdlib/string/base/remove-first
Adds @stdlib/string/base/remove-first-code-point
Adds @stdlib/string/base/remove-first-grapheme-cluster

Related Issues

Does this pull request have any related issues?

This pull request:

Is part of https://github.com/stdlib-js/stdlib/issues/1062

Questions

Any questions for reviewers of this pull request?

The linter doesn't like the identifier name removeFirstGraphemeCluster because it exceeds 25 characters, should we supress this rule, or there's a shorter name we want to adopt for that package?

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@steff456 for removeFirstGraphemeCluster, just disable the line rule on a per line basis where the variable is first declared. E.g.,
var removeFirstGraphemeCluster = require( '<pkg>' ); // eslint-disable-line id-length

In Markdown, you'll need to add the following before each JavaScript block.
<!-- eslint-disable id-length -->

@steff456 for removeFirstGraphemeCluster, just disable the line rule on a per line basis where the variable is first declared. E.g.,
var removeFirstGraphemeCluster = require( '<pkg>' ); // eslint-disable-line id-length

In Markdown, you'll need to add the following before each JavaScript block.
<!-- eslint-disable id-length -->


issu in start apllication

Hey, thanks for contributing to Koel! To save time for both of us, please make sure these checkboxes are checked before submitting the issue:

[x] You have read and followed closely the Wiki, Upgrade Guide, as well as Troubleshooting
[x] The issue has not been reported before
[x] This is not a "how to install on Windows" or "why is my npm messed up" question
[x] You're a cool person

All checked? Now also make sure your issue

[ ]

Is associated with a version. Or better yet, a commit.

[ ] Is as detailed as possible (ahem... OS, browser, steps to reproduce, maybe?)
[x] Includes the error output if it's a bug/error report ("Whoops!" is not very helpful, you know)
[x] Is in English, 因为我不说中文。

i have this problem in start app