correct erroneous use of rttm in instructions and in tools

rttm is explained in pp. 18+ here: https://web.archive.org/web/20170119114252/http://www.itl.nist.gov/iad/mig/tests/rt/2009/docs/rt09-meeting-eval-plan-v2.pdf
Some errors I noticed in the instructions:

[ ] general example is underinformative
[ ] SAD example contains non-speech
[ ] noisemes examples violates type given subtype

These need to be fixed in the input/output assumptions of the different tools as well

I add an error that I noticed previously :

[ ] Some rttm files (opensmile_sad_DH_0149.rttm) end up to have negative values

And some thoughts to improve the way we're managing all the different formats.

[ ]  RTTM, being the format that we use the most. Writing a script to check if a RTTM file is well-formatted could be useful. Here, I think the minimal specification is the RTTM one. On top of that, we could add our own (light) constraints if necessary (being sorted for instance).
[ ] We loose a lot of information in the conversion process. Even if this information is not used by our models, some users may want to use the VM to convert their own files (without necessary using our models). It might be helpful to write a doc summarizing how to convert from one format into another one ? What information is lost ? What information is preserved ? Minimal requirements to be well-formatted ?. We could refer to it each time we need to convert one format into another one.

This is terrific! Could you please do a few online searches before you
start writing all of that? Just to check if someone has already done this.
If the search turns up empty, I can drop a line to two people who might
know before we put in any more effort.

Yep, indeed. It was not consistent and that led to some errors during the evaluation process.
I just changed those problematic lines, I access to the RTTM fields in a different way (as soon as we can get the t_beg, duration and filename, it works).

Update : After digging into the rttm documentation, we discovered that indeed, the rttm files were white-spaced. Here, a useful example that is easier to understand than the doc :
SPEAKER YOUR_AUDIO_FILENAME 1 5.87 0.370 <NA> <NA> spkr1 <NA>
LEXEME YOUR_AUDIO_FILENAME 1 5.87 0.370 ஹலோ lex spkr1 0.5
SPEAKER YOUR_AUDIO_FILENAME 1 8.78 2.380 <NA> <NA> spkr1 <NA>
LEXEME YOUR_AUDIO_FILENAME 1 8.78 0.300 உம்ம் lex spkr1 0.5
LEXEME YOUR_AUDIO_FILENAME 1 9.08 0.480 அதான் lex spkr1 0.5
LEXEME YOUR_AUDIO_FILENAME 1 9.56 0.510 சரியான lex spkr1 0.5
LEXEME YOUR_AUDIO_FILENAME 1 10.07 0.560 மெசேஜ்டா lex spkr1 0.5
LEXEME YOUR_AUDIO_FILENAME 1 10.63 0.350 சான்ஸே lex spkr1 0.5
LEXEME YOUR_AUDIO_FILENAME 1 10.98 0.180 இல்லயே lex spkr1 0.5

That shows that there is no problem for the transcription field. Our (wrong) hypothesis was that, since there is a string field, the lines should be separated by tabulations because a confusion would appear between a white-space separator, and a "normal" white-space.
Visibly, the format specification ensures that there are no extra white-spaces. All of the white-spaces have to be separators.
I updated all of the concerned repositories to standardize our use of the rttm format. The concerned repo are :

DiViMe
dscore
To-Combo-Sad
tools
varia

Now the output of the models should be all white-spaced. And the pipeline is designed to receive white-spaced reference rttm.

This is great to have RTTM formats be better standardized. I know there is one tool which may still produce improperly formatted RTTM; http://github.com/srvk/TALNet I believe uses double quotes around strings which contain spaces.
Once we start actually using TALNet, we'll have to fix it, probably by stripping quotation marks of the output, and if not in an automated fashion, then manually modifying the .csv file containing the class names, likely replacing space with underscore.

I'd like to add: any RTTM our tools produce will use the Space character to separate fields, but just to err on the side of caution, maybe our tools should all accept Tab or Space, as "whitespace". I'd rather not make a hard requirement/assumption that externally obtained RTTM will conform to space as a field separator (though we could always advertise this as a requirement of the VM, and/or do simple conversion)

Yep, I agree with that. I'm just afraid that some errors may occur while converting tab-separated RTTM to space-separated RTTM (if one field contains more than one space).
If we accept both tab and space, we should maybe add some sanity-check : for instance, check that the number of fields is equal to 9 (or 10), not more, not less.

Upon further discussion, we decide that getting the rttm format perfect is not a priority.
When we do get to it, we should read to get it right 2005 NIST eval plan, appendix C, reproduced next:
The Rich Transcription Time Mark (RTTM) file format (with ".rttm" filename extension) will be used for both the system output and
reference for the SPKR and SAD tasks. A separate RTTM file should be generated for each meeting in the test set. See Appendix A for a
detailed definition of the RTTM format. This appendix explains the application of the RTTM format for the SPKR and SAD tasks.
The RTTM format supports markup of a variety of metadata tasks. However, for RT-05S, only the information required for the SPKR and
SAD tasks should be provided. The RTTM file format provides two types of records related to the speakers: SPKR-INFO records and
SPEAKER records. The SPKR-INFO record for a speaker is associated with all the SPKR records for that speaker by means of matching
values in their name fields. RT-05S participants in the SPKR task will therefore need to output the following: one SPKR-INFO entry per
unique speaker in each source file followed by a SPEAKER entry for each occurrence of a given speaker in the source file.
Participants in the RT-05S evaluation can run their systems on two distinct microphone conditions: multiple distant mics (mdm) and single
distant mic (sdm). The source file name (file field in the RTTM records) to be used is the name of the microphone recording file for the sdm
condition. However, for the distant microphone conditions, the meeting ID (e.g., NIST_20020214-1148 from Section 12.2.2) is to be
used instead of the audio filename.
The SPKR-INFO records (one per speaker) associate the speaker type (adult_male, adult_female, child, or unknown) in the stype
field, with the speaker's name in the name field. There is only one SPKR-INFO record per speaker. The SPKR-INFO records are typically
all put at the beginning of the RTTM file since they have no associated timestamp. For RT-05S, the speaker type will not be evaluated so
there is no need for participants to provide a value other than “unknown” for the stype field.
The SPEAKER records give information about when a speaker is speaking. Each time the speaker (identified by its name) starts speaking,
there is a SPEAKER record that states the time when the speaker began speaking (tbeg) and how long the speaker spoke (tdur).
SPKR example for the sdm condition on a recording named NIST_20020214-1148_d05_NONE.sph:
SPKR-INFO NIST_20020214-1148_d05_NONE.sph 1    unknown speaker2 0.5
SPKR-INFO NIST_20020214-1148_d05_NONE.sph 1    unknown speaker5
.
.
SPEAKER NIST_20020214-1148_d05_NONE.sph 1 4.57 8.70   speaker2 0.63
SPEAKER NIST_20020214-1148_d05_NONE.sph 1 8.54 9.03   speaker5 0.33
SPKR Example for the mdm condition:
SPKR-INFO NIST_20020214-1148 1    unknown speaker3
SPKR-INFO NIST_20020214-1148 1    unknown speaker8
.
.
SPEAKER NIST_20020214-1148 1 0 3.70   speaker3
SPEAKER NIST_20020214-1148 1 10.23 19.47   speaker8 0.86
For the SAD task, only one SPKR-INFO line is required per source file regardless of how many speakers exist in a recording. The stype should be “unknown” and will not be taken into account for scoring
SAD Example for the mdm condition:
SPKR-INFO NIST_20020214-1148 1    unknown speech
.
.
SPEAKER NIST_20020214-1148 1 0 3.70   speech
SPEAKER NIST_20020214-1148 1 10.23 19.47   speech 0.86


Python wrong ELF class: ELFCLASS64

I'm having the same issue as was discussed and solved in this thread here. The error in the other situation was due to the Raspberry Pi KERNEL being upgraded from 32-bit to 64-bit. (I'm still on Raspberry Pi).
Thanks.

Yes, I can confirm. Thanks very much.


PR# 2486 breaks Android builds

Hi, since https://github.com/libuv/libuv/pull/2486/commits/c4a9c2c7daffcad160c87e8528653c103a0d4f0c Android-28+ uses function uv__random_getentropy, but src/unix/random-getentropy.c is not built on Android and resulting library has an undefined reference.
I have tested it with android-ndk toolchain (27 works, 28+ does not link), but it looks like src/unix/random-getentropy.c isn't built by any buildsystem supposedly supporting Android.

Version: 1.33+
Platform: Android

What build system are you using, GYP? CMake?

CMake, with Android-ndk 20

@interj Thanks for the report. Can you check that #2704 fixes the build for you?

It fixes the problem, at least for CMake builds.


Remove redundant social media icons

New Mockup
Please check assets/css/global.css, design-assets folder images, and figma files for updated style guides. Any new pull request should use the updated color, spacing, and sizing variables.
No Bootstrap or jQuery
Please refrain from adding Bootstrap or jQuery to the site. These pr's will be closed. They overwrite global styles. Vanilla CSS and JS is way better now anyways 😊
New Task:
What is the task
Remove the, now redundant, larger social media icons above the footer to go along with the new mockup.
Who is the task for?

[x] I created this task for me to complete, I will create a PR soon.
[ ] I created this task for someone else to claim.
[ ] I think this is a task that multiple people can do

Confirm all of the following:

[x] I have looked at all the issues currently available and this does not exist or overlap with another ticket

⚠ PLEASE ONLY SUBMIT A PR FOR THIS TASK IF THIS TASK WAS AVAILABLE FOR YOU TO CLAIM

I am working on this


How to solve this problem plz help me

Sliders not working in angularjs routing/ng-view

I'm afraid that this issue sounds like a support question and is not actionable. If you believe there is a bug in AngularJS then please create a new issue including steps to reproduce the bug.


Fix bootstrap style path for JS license page

Yes

Sweet, thanks :thumbsup:
That's good enough for me. Merging.


IsControlElement Property of Text Box is false when empty

Describe the bug

The IsControlElement property of the text box control must always be true. However, it is false when it is empty.
Steps to reproduce the bug

Steps to reproduce the behavior:

Add a text box with place holder text and no text.
Run accessibility insights on the app.
The IsControlElement Property is set to false when there is no text entered. This property cannot be set programmatically and must be set by default in the text box control.

Link to code - https://github.com/microsoft/PowerToys/blob/master/src/modules/keyboardmanager/ui/ShortcutControl.cpp#L77-L82
Expected behavior

IsControlElement property must always be true for a text box.
Screenshots

As soon as text is entered into the text box, this error no longer appears, as is clear from the screen shot.

Version Info

NuGet package version:

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

Link to issue in PowerToys repo - https://github.com/microsoft/PowerToys/issues/5621
Similar issue created in the accessibility insights repository - https://github.com/microsoft/accessibility-insights-windows/issues/446

Cannot repro so closing.


Documentation of API

Is there going to be any documentation for this PHP API library?
There does not appear to be anything functional or useable and I am wondering if this will ever change.
For example getStartTime()->serializeToJsonString() ?
Where would someone find this ? Are there alternatives ?
Its well known that Googles Docs are worse than any other system for almost any API there is, bar none - so was just wondering if Google or anyone would be addressing this ?

Hi @audas thanks for dropping by. Can you tell us a little more? Which API or library are you trying to use?

Google Speech To Text.
There is literally no documentation on V1 on the API for things like serializeToJsonString()
ALSO
Authorizing in V1 and then using cloud storage bucket for processing.
Honestly - just include examples of authorization instead of links to external sites which offer absolutely no insight into how to accomplish the process.
Whats worse is, as mentioned, using the storage bucket required a DIFFERENT form of authorization.
for storage we need to use this (the alternatives failed)
    $storage = new StorageClient([
        'keyFilePath' => 'client_secret.json',
        'projectId' => 'projectID'
    ]);

However for V1 speech to text this was the only method which worked. These are the SAME credentials
$credentials=json_decode($results['json_files'], true);
$client = new SpeechClient(['credentials' => $credentials]);

I was forced to write the credentials from the database to a file in order for Storage Client to work.
The process of AUTHORIZATION is the most critical aspect of using these libraries and it is literally NOT addressed at all.
Thanks.

Hi @audas
Apologies for the late response on this issue.
I'm not sure when was it published, but the documentation and samples for the Speech to text library can be found here.
In some samples I can see reference to serializeToJsonString calls as well.
As far as credentials are concerned. There are a few clients which support the credentials config param while, some which currently use the keyFile and keyFilePath parameters. This is mentioned in AUTHENTICATION.md for now.
We are in the process of unifying these client options for all the clients. But until that happens, please use the mentioned AUTHENTICATION.md as a reference.
Closing this issue for now, but please feel free to reopen this if you have any more questions.
Thanks.

Ok, so 3 years later - thanks. Had to literally learn new languages and ditch PHP because this was so bad.
Will check out the update....


Check the .gitignore file

Maybe set it up to remove R history files as those really shouldn't be in the repo.

Added .Rhistory to our .gitignore file to resolve this


New command to add target to repo

tufrepo edit <TARGETS> add-target ... works but requires me to know which targets metadata to add things into. this is inconvenient with hash bins.
I would like a way to add the target to the final delegated metadata in the chain. this could be just a flag to add-target, like --delegate that means the target should be added to the final terminating metadata

I ended up re-designing the CLI a bit (because add-target is not really a edit-subcommand anymore). The PR seems to work really nicely but the code is a bit meh still (so is a draft)


Does Sciety have an API that can be used to get review data for a specific article?

Description
We need to display review data with the article data, and as such we need to get that review data from somewhere. Sciety we think should be able to get this data for us, hence we need to investigate the API's they have (if any) and see if they can supply the information that we need.
DoD

[ ] An answer to the question 'can we use Sciety's API to get review information for a specific article'.
[ ] Next steps, raised as issues in the backlog.

It does, here is an example that gets some review information for a specific article.

Following on from my above message, I've just spent some time watching the background video here which gives some more context around the structure of the response from the API. Would advise you both watch it also.

Also need to figure out how we get actual review content

For Biophysics Colab we can get the review content via the hypothesis api which returns a JSON object containing the content of the review and some related data.
for example: a review from our sample paper
This is probably enough for us to get started without worrying about duplicating effort from Sciety as its only a single call and a type to match the JSON output.
If we combine the two calls to docmaps and hypothesis we will have all of the meta information and the content of the review. We will need to speak to the Sciety team about when they will add the Biophysics Colab group to the available docmaps.

I have spoken to a member of the Sciety team and while adding other groups to the docmaps api is on the cards its not an immediate priority so it may be best (as they suggested) to hardcode a response of the docmap call in the code so we can just swap in the fetch when that has been done and still get on with the rest of the tasks.


Convert Create/Edit to Mid Fi (Analyzer Ranges )

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context (wireframes, business rules, etc.)?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a user
I want to view MP Analyzer Ranges  using the mid fidelity design
So that I have a better experience
Acceptance Criteria
Given a user has accessed MP Analyzer Ranges
When it is created or edited
Then it is displayed according to the wireframes
Context
https://xd.adobe.com/view/df8c27d8-1cff-494d-946d-7c8cf1a2bdc3-8adf/
https://xd.adobe.com/view/eb6c3176-9c62-46e0-af54-8be9a2771b7a-be3f/
Testing
Test Case:
Tester:
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Has the user story been accepted by the Product Owner?
[ ] Is the user story ready to be deployed to the test/staging environments?

Modal not built yet. Will address responsiveness when modal is built.


Add walking a directory tree to the standard library

It may be useful for libraries built on top of jk, say to be able to create a ConfigMap object from directory ful lof files.

do we want that?
look at how node does it

We now have std.dir() and std.info().


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app/src/atom.js:55:11)
<unknown> (/Users/stelek/.atom/packages/term2/index.coffee:44:13)

Same deprecation here and the git version of atom.


Fluent UX: Settings

PowerToys should do its best to look like a system level integration. The current settings screen is a close but needs to be refine a bit more.  It looks close but little differences in both look and animations make it clearly not native.

Here's a mock-up using WinUI 3.0

Light theme:

It tries to mimic the default Windows OS settings as much as possible. Respects the selected theme (light/dark) and using the new WinUI 2.3 NumberBox.
Source code in this repo: https://github.com/niels9001/PowerToysUXWinUI3Alpha

Some items will have to be adjusted here but yes, that is awesome!

Here is my thought.  Have a “new look” toggle so we can start migrating new stuff over.
Use WinUI 3, wpf and a xaml island to just start working. @enricogior any reason why win32 would be a must or would wpf be good enough?

the new setting app would be a new app too so we don’t take a hit for the entire app to move to 19h1 min version.

Use WinUI 3, wpf and a xaml island to just start working. @enricogior any reason why win32 would be a must or would wpf be good enough?

Do we still want the settings to be a standalone app? If not Win32 is a must.
If we keep it standalone we may have to redesign the way the app exchange data with the runner.
Some investigation is needed before giving a cost estimate.

Why wouldn’t we want to keep it a stand-alone app?  How would that work if it wasn’t?

Why wouldn’t we want to keep it a stand-alone app?

The only reason why it was a standalone app, was because we didn't know how to run the WebView control in an elevated process. There weren't other reasons.
We may decide to keep it standalone, if that is beneficial, but since we haven't had that discussion yet, I asked.
Currently the runner creates two named pipes that are passed to a class called TwoWayPipeMessageIPC and used to communicate between the runner and the settings app.
When the settings app starts, it calls the runner to get the general settings and each module settings.
When the settings are changed and sent back to the runner, the runner processes the general settings and dispatches the modules settings to each module.

How would that work if it wasn’t?

The runner would open directly the Settings window and not have to use the two-way pipe channel to exchange data and control commands.

Couldn't the WPF app just consume a Win32 dll that has that implemented?  That way the logic for that stays in a single DLL for both runner and settings.
What would the settings app then be run out of?  The Runner?

Couldn't the WPF app just consume a Win32 dll that has that implemented?

Yes, but there is work to be done to make it callable from C#. We should factor the cost of the code implementation and of the testing for this work. Also test the performance impact of opening the settings as a standalone WPF app.

What would the settings app then be run out of? The Runner?

Yes, similarly to how is done now, it's the runner that starts the Settings app.

Looking at @zeealeid's mock

This is functional XAML.

needs more padding on the left but i'm digging it.

Code is from a basic Windows Template Studio generated app but here is the current XAML so it isn't lost
<winui:NavigationView
    x:Name="navigationView"
    IsBackButtonVisible="Collapsed"
    IsBackEnabled="{x:Bind IsBackEnabled, Mode=OneWay}"
    SelectedItem="{x:Bind Selected, Mode=OneWay}"
    ItemInvoked="OnItemInvoked"
    IsSettingsVisible="True"
    IsPaneToggleButtonVisible="False"
    Background="{ThemeResource SystemControlBackgroundAltHighBrush}">

    <winui:NavigationView.PaneHeader>
        <TextBlock Margin="12, 24, 0, 6" Style="{StaticResource SubheaderTextBlockStyle}" FontWeight="Bold">PowerToys</TextBlock>
    </winui:NavigationView.PaneHeader>
    <winui:NavigationView.MenuItems>
        <!--
        TODO WTS: Change the symbols for each item as appropriate for your app
        More on Segoe UI Symbol icons: https://docs.microsoft.com/windows/uwp/style/segoe-ui-symbol-font
        Or to use an IconElement instead of a Symbol see https://github.com/Microsoft/WindowsTemplateStudio/blob/master/docs/projectTypes/navigationpane.md
        Edit String/en-US/Resources.resw: Add a menu item title for each page
        -->
        <winui:NavigationViewItem x:Uid="Shell_Main" Icon="Document" helpers:NavHelper.NavigateTo="views:MainPage" />
        <winui:NavigationViewItem x:Uid="Shell_Main" Icon="Document" helpers:NavHelper.NavigateTo="views:MainPage" />
        <winui:NavigationViewItem x:Uid="Shell_Main" Icon="Document" helpers:NavHelper.NavigateTo="views:MainPage" />
        <winui:NavigationViewItem x:Uid="Shell_Main" Icon="Document" helpers:NavHelper.NavigateTo="views:MainPage" />
    </winui:NavigationView.MenuItems>
    <i:Interaction.Behaviors>
        <behaviors:NavigationViewHeaderBehavior
            DefaultHeader="{x:Bind Selected.Content, Mode=OneWay}">
            <behaviors:NavigationViewHeaderBehavior.DefaultHeaderTemplate>
                <DataTemplate>
                    <Grid Margin="0,24,0, 6">
                        <TextBlock
                            Text="{Binding}"
                            Style="{ThemeResource TitleTextBlockStyle}"
                            FontWeight="Bold"
                            Margin="{StaticResource SmallLeftRightMargin}" />
                    </Grid>
                </DataTemplate>
            </behaviors:NavigationViewHeaderBehavior.DefaultHeaderTemplate>
        </behaviors:NavigationViewHeaderBehavior>
    </i:Interaction.Behaviors>
    <Grid>
        <Frame x:Name="shellFrame" />
    </Grid>
</winui:NavigationView>

Windows 10 X is altering the font styles used in it's settings app - so hopefully those changes come to Windows 10 and can be coordinated with the Powertoys UI

Dependence: WinUI 3

@crutkas
Here is a list of what is done and what is still on the way:

Task
Status

General Settings
Done

Power Rename
Done

Power Launcher
Done

File Explorer Preview
Done

Fancy Zones
On The way

Image Resizer
On The Way

Keyboard Remmaper

Replace Two Way IPC with File Trigger change communication

Moved status to the top

@crutkas Do we have a list of contributors per module somewhere? We could start adding those I guess.

I was planning to start adding those in Friday but the feature branch now is slightly busted after we merged KBM and setting v2.

@mdtauk hopefully they start using WinUI for the Windows shell. They've missed so many opportunities. Maybe they're waiting for WinUI 3.0.

Mildly unrelated, but i don't see an issue for Launcher Fluent UX - is this not on the cards for this release, or are we just not there yet?

Mildly unrelated, but i don't see an issue for Launcher Fluent UX - is this not on the cards for this release, or are we just not there yet?

Being designed out of the box like that :)

Awesome! Can’t wait to use it! From: Clint RutkasSent: Friday, May 8, 2020 10:13 AMTo: microsoft/PowerToysCc: Aaron McElwee; CommentSubject: Re: [microsoft/PowerToys] Fluent UX: Settings (#889) Mildly unrelated, but i don't see an issue for Launcher Fluent UX - is this not on the cards for this release, or are we just not there yet?Being designed out of the box like that :)—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.

The new Settings are now available in 0.18 https://github.com/microsoft/PowerToys/releases on Windows 10 version 1903 and higher.


[Feature][parameter] Add workflow_instance_id/workflow_id/URI parameter

Search before asking

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

Description
Add workflow_instance_id or workflow_id or URI as global parameter
Use case
We use Dolphin Scheduler as task schduler for data process, and develop a additional bussiness data management system and a lightly task record manager。
we'd like to relate [ouer task] to [Dolphin Scheduler Task] by URI。
assume Add workflow_instance_id or workflow_id or URI as global parameter, on each workfow instance create by crontab, in workflow instance we persist it toOUER database
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@piscesvivian Could u plz give a more detailed explanation, better with examples, where u want this workflow_instance_id/workflow_id/URI? BTW, if it is only for ur company's commercial feature, the community is likely to decline this request.

Can I set a global paramter like ${workflow_instance_id} or Custom param ${task_id}/${task_instance_id}  in workflow?
After crontab Task online, is there a way to get instance every time?
can you add more parameters?
All built-in parameter in docs here:
https://dolphinscheduler.apache.org/zh-cn/docs/3.1.4/guide/parameter/built-in
That's 3 in total.


[Monaco] Add monaco.editor.registerEditorOpener method to be able to intercept editor open operations

Fixes https://github.com/microsoft/monaco-editor/issues/2000
By default the "go to defintion" action in the Monaco editor does nothing if the definition is in another model, or opens a peek window when multiple definitions are found. This default is a good thing because just changing the underlying model would be very detrimental when embedding the editor. However, there are use-cases where you maybe want to control what happens when you go to a definition. For example, when we embed the Monaco editor we would like to open a new tab to the source code containing the definition for certain models but not all of them.
This PR adds a new function monaco.editor.registerEditorOpener that lets the embedder intercept such open operations and handle those themselves. By using a boolean return value the embedder can indicate if the resource was handled or the editor should fallback to the default. The function returns an IDisposable instance that can be used to unregister the handler again. This interception can be tested in the code below by e.g. Ctrl-clicking on the alert function. By intercepting the open operation you can also choose to change the underlying editor model yourself if that makes sense in your application (see linked issue).
Edit: I decided to remove a second commit because it didn't work properly in some cases. This reduces the PR to just the new method to intercept the editor open operation.
Monaco Playground Code:
const text = `function hello() {
	alert('Hello world!');
	Date.now();
}`;

// Hover on each property to see its docs!
const editor = monaco.editor.create(document.getElementById("container"), {
	value: text,
	language: "javascript",
	automaticLayout: true,
});

monaco.editor.registerEditorOpener({
	openCodeEditor(source, resource, selectionOrPosition) {
		console.log(source, resource, selectionOrPosition);
		if (resource.path === "/lib.dom.d.ts") {
			// simulate openening a new browser tab for our own type (open definition of alert)
			console.log(`Opened definition for ${resource}`);

			// alternatively set model directly in the editor if you have your own tab/navigation implementation
			// const model = monaco.editor.getModel(resource);
			// editor.setModel(model);
			// if (monaco.Range.isIRange(selectionOrPosition)) {
			// 	editor.revealRangeInCenterIfOutsideViewport(selectionOrPosition);
			// 	editor.setSelection(selectionOrPosition);
			// } else {
			// 	editor.revealPositionInCenterIfOutsideViewport(selectionOrPosition);
			// 	editor.setPosition(selectionOrPosition);
			// }

			return true;
		}
		return false;
	}
});

Thanks for the PR, i really need this :)

@hediet Could this be considered for one of the next Monaco versions? Thank you!

Addressed review and rebased onto main.

Thanks!

Addressed review and rebased onto main.

Generally, please don't rebase in PRs, as that resets the review.
(we can also just squash the PR when merging)

This seems stuck, do I need to do something?


Partially implement blueflood backend

@jeeyoungk

Mostly nitty things, but looks pretty good. I'll take another round of look afterwards.
Thanks!

LGTM


Update composer to support guzzlehttp/psr7 > 2.0

Trying to install the sdk in Magento 2.4.6 but I am getting version 0.6.0 which lacks a lot of classes.
When trying to in stall version 19.1.0 I am getting a conflicts with guzzlehttps/psr7:
[root@localhost htdocs]# php81 composer.phar require benmorel/ebay-sdk-php:19.1.0
./composer.json has been updated
Running composer update benmorel/ebay-sdk-php
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires benmorel/ebay-sdk-php 19.1.0 -> satisfiable by benmorel/ebay-sdk-php[19.1.0].
    - benmorel/ebay-sdk-php 19.1.0 requires guzzlehttp/psr7 ^1.0 -> found guzzlehttp/psr7[1.0.0, ..., 1.9.1] but the package is fixed to 2.6.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Please merge the PR so I can try reinstalling.
Thanks

...and of course publish a new version on packagist. Thanks.

Hi, sorry for the delay. I reverted the change of package name (!), and rebased. Many tests are failing, due to functions being removed in v2 of guzzlehttps/psr7.
I'll try to fix this.

Released in 19.2.0.


How can I fix the node when I use  the sigma.plugins.dragNodes plugin

I use the sigma.plugins.dragNodes . But I want some node is fixed ,and the node can't  drag, Please help me ,thanks!

@AVermeij

For the node which you do not want users to be able to drag, you can set a property fixed to true. In sigma.plugins.dragNodes.js (line 156), you can then change the condition from
if (_node && size > 1) {
to
if (_node && !_node.fixed && size > 1) {
to make sure that nodes which have their fixed property set to true can never be dragged.

@AVermeij  it works thanks.


rt-virtual tag

This creates support for a <rt-virtual> tag whose root won't be rendered on the DOM.
Meant to be used with rt-if, rt-repeat, etc... An example:

  <rt-virtual rt-repeat="n in [1,2,3]">

{n}

{n*2}

  </virtual>

generates

1

2

2

4

3

6

+1

+1

Great work, thanks for this pull request. It solves a lot of problems very elegantly and removes the need for other suggested features like rt-repeat-inside

this needs to be added to the docs, but unfortunately I'm not good at writing in english.

@avi  - just one problem I noticed while developing the same functionality as @nippur72 . This adds elemens inside rt-virtual as a collection without unique keys which means React will rerender all of them whenever one of them changes.

@antitoxic that's true but React will re-render child items only if one of them is added or removed, not just changed. So in most of cases that shouldn't be a concern (bad reconciliation occurs only if you have <rt-virtual> containing a rt-if as direct child).

It would be interesting to see how does babel render JSX with if statements.

@antitoxic I don't know much of JSX, but I guess you have to put your ifs by yourself, there is no if-feature in JSX (as far as I know):
return a > b ?
hello
 : null

Regarding the issue, I will experiment by adding a key to all direct children of an <rt-virtual>.

I made it in #98, refer to it for details.
I wonder if we should apply the same technique to rt-if, giving an unique key by default.


Upgrade dependencies to latest versions

Basic information

nrich version:
1.1.1
Module:
Project

Additional information

Enhancement description

Upgrade dependencies of all libraries to latest versions, possibly refactor code to match the new versions used (remove deprecated code etc).
Current behaviour

Used dependencies are outdated.
Wanted behaviour

The dependencies used are the latest versions.
Possible workarounds

Fixed in https://github.com/croz-ltd/nrich/pull/23


Asserts directly without checking before

if + NSAssert is redundant in this case, we can directly assert the condition.

@gobetti the pre-checks are essential because some build configurations will strip NSAssert's and thus the logic will be allowed to run.
While it may not be ideal to have silent failures it does prevent crashes.


error with D10.2.3

I believe this is not related to the library, but with your local computer trying to access the OpenAI API. From your screenshot, my guess is that you are using an old Windows version?
OpenAI seems to use Let's Encrypt certificate, so I guess you are hitting this issue:
https://letsencrypt.org/docs/dst-root-ca-x3-expiration-september-2021/

Sorry, this https://github.com/fpiette/DelphiChatGPT project works like a charm.
And there is one more both work fine

The mentioned project uses a different library to perform HTTP requests.
I don't know what is the "one more" you mentioned.
This library doesn't do anything regarding HTTP request. It just uses the built-in Delphi classes, which in turn uses WinHTTP as far as I know.
Maybe you are hitting the TLS 1.2 issue then, you can try this:
https://support.microsoft.com/en-us/topic/update-to-enable-tls-1-1-and-tls-1-2-as-default-secure-protocols-in-winhttp-in-windows-c4bd73d2-31d7-761e-0178-11268bb10392

Estou com o mesmo problema. Windows 8.1 atualizado para Windows server 2012 R2.

@mikelustosa Have you tried the fixes suggested in the link I posted?

@mikelustosa Have you tried the fixes suggested in the link I posted?

Yes. Not work. 😕

So, you installed the root certificate manually, and also enabled TLS 1.2 and TLS 1.3, and neither work?
What is the exact error message you get?

I Made my own. I took 2 projects one that works and the other that
does not (certificate problem) but has a great screen and merged them

There is now an option to use Indy to connect to OpenAI API. You can switch to it using a code like this:
uses {...}, OpenApiRest, OpenApiIndy, IdHTTP;

procedure TMainDataModule.IndyClientCreated(Client: TIdHTTP);
begin
  // Set extra Client properties here
end;

procedure TMainDataModule.DataModuleCreate(Sender: TObject);
var
  Factory: TIndyRestRequestFactory;
begin
  // At the beginning of your application, set the DefaultRequestFactory
  Factory := TIndyRestRequestFactory.Create;
  DefaultRequestFactory := Factory;
  Factory.OnClientCreated := IndyClientCreated;
end;

More info in this link: https://github.com/landgraf-dev/openapi-delphi-generator#client-compatibility
This change will probably allow the client to work better on old Windows versions which don't support the latest ciphers used by the API server.


Setting up NavigationView's back behavior is cumbersome

When clicking the back button in NV, the NV gets an event fired and can navigate its content Frame backwards. However it is currently app to the NV to figure out how to select the right NavigationViewItem (and not cause a Frame navigation)
So: Setting up a NavigationView to sync the selected item when going backwards is cumbersome, it should be something native in the framework. Getting the syntax right for C++ is hard.

@chigy FYI


Fuzz testing

seeding random values
should we stop after first fail
should we print all failures
configuring value range
from what range should we generate values by default


Trouble Following Install Docs

Hi there! I'm having trouble following the installation docs located at https://deis.com/docs/workflow/quickstart/provider/aws/install-aws/. The first problem I encountered was a missing namespace:
› helmc install workflow-v2.0.0
---> Running `kubectl create -f` ...
Error from server: error when retrieving current configuration of:
&{0xc8201280a0 0xc8200a20e0  deis STDIN 0xc82069bc20 0xc82069b950 }
from server for: "STDIN": namespaces "deis" not found

No big deal. I created the namespace with a kubectl create command.
Running the command again, it makes it further but crashes with a different error:
› helmc install workflow-v2.0.0
---> Running `kubectl create -f` ...
namespace "deis" configured

secret "builder-ssh-private-keys" created

secret "builder-key-auth" created

secret "django-secret-key" created

secret "database-creds" created

secret "objectstorage-keyfile" created

secret "deis-router-dhparam" created

serviceaccount "deis-builder" created

serviceaccount "deis-controller" created

serviceaccount "deis-database" created

serviceaccount "deis-logger-fluentd" created

serviceaccount "deis-logger" created

serviceaccount "deis-minio" created

serviceaccount "deis-monitor-telegraf" created

serviceaccount "deis-registry" created

serviceaccount "deis-router" created

serviceaccount "deis-workflow-manager" created

service "deis-builder" created

service "deis-controller" created

service "deis-database" created

service "deis-logger" created

service "deis-minio" created

service "deis-monitor-grafana" created

service "deis-monitor-influxapi" created

service "deis-monitor-influxui" created

service "deis-monitor-stdout" created

Error from server: error when retrieving current configuration of:
&{0xc82000e230 0xc820110310 deis deis-registry STDIN 0xc820463080 0xc820227b80 }
from server for: "STDIN": services "deis-registry" not found

[ERROR] Failed to upload manifests: exit status 1

BTW I had initially tried the installation following the Vagrant instructions, then retried here with the AWS instructions. I had the same issues with both.

Can you confirm if your kubernetes server and kubectl are both on version 1.2...I had seen similar error because of using a old client/server

Ah - my client is version 1.1.8, while my server is version 1.2.4. I'll update my client and see if that fixes the issue (I'm sure it will).
Thanks @kmala!

Yes, that fixed my second problem. I'm tearing down my cluster and starting up a new one to verify that this also fixes the first (the missing namespace).

Yup. The namespace was created this time around. Thanks again, @kmala. At the least, anyone else hitting this problem can find an easy solution 😃.

Had same issue.. my kubectl also was at an older version than my server. Downloading the kubectl with version same as server fixed problem


add and run ruff-format

as black was removed in this commit, I believe we should replace it with ruff-format

Perfect!


Replace grey rectangle in multiplicity tree & enabling of assignment mode from multiplicity tree

replacement of grey rectangle around multiplicity tree (as highlighting indicator) by controlled opacity and stroke width values
enabling of assignment mode by clicking on a multiplicity tree
fixed a problem with tree width and height
simplifications

Thanks !


Is this detectable in any way?

Hey, I know it might be a bit of a silly question but I was wondering if this plugin sends anything to the SE servers that might trigger a flag on their end? I'm always a little hesitant with using new plugins.
Thanks!

Yes, they will ban you

It depends on the features that you use.
Basic boss modules should not be detectable, they only read stuff. Features marked as 'experimental' in config might be theoretically detectable - these are all disabled by default.
In any case, using the plugin should never result in game sending something that it couldn't with some specific player actions (of course, barring some unknown bugs).

Just don't tell anyone you're using the plugin and your chances of being caught is 0.


When I want to use the snapshot to synchronize to the latest block, the data is always at the initial height of the snapshot and cannot be synchronized

Node version: v1.8.7
P2P address is updated
The following is a screenshot of the synchronization log:

why？

It is hard to tell without seeing more of the log and possibly enabling debug level logging. Do you only have the two eoshubipool.com peers? If so it looks like those two peers are not accepting any more connections.


windows

how to install in windows ?

Follow the instructions. They are the same for windows and linux.


MM-13844 Show rendered emojis in the text composition areas: DRAFT / WIP

Summary
Show rendered emojis in the text composition areas.
Emojis turn into rendered emojis in the text area, when

They are selected in the emoji picker
They are selected from the emoji autocomplete
They are typed in full with the trailing colon (ie :smile:)
They are typed with the literal shorthands (ie :))

Once converted to an emoji, pressing backspace removes the emoji from the text area.
DRAFT / WIP
I am posting because we (@lieut-data and I) thought it would be good to keep everyone updated. This is a draft/WIP/feature branch because this feature requires a change that affects a large portion of the app (replacing textarea with a contentEditable box via the QuillJs package). There are about 40 TODOs still outstanding, so a lot of work left to do. Also, testing.
Right now emojis are rendered as described above, but only in the main textarea. Have to still implement for all other textareas, modals, rhs, etc. Have to implement edit post as well.
I'd appreciate feedback on whether you see anything that could bite us later on. This will require a /lot/ of testing before it's ready to be merged.
Ticket Link
#10163
Jira ticket

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Closing and reopening this to get Jenkins to pick up the branch properly

Spintmint test server creation failed. See the logs for more information.

/home/ubuntu/jenkins/workspace/mw_mattermost-webapp_PR-2634/components/quill_editor/quill_editor.jsx

  121:28  error  'oldContents' is defined but never used           no-unused-vars
  121:41  error  'source' is defined but never used                no-unused-vars
  124:54  error  Block must not be padded by blank lines           padded-blocks
  127:13  error  Unexpected console statement                      no-console
  127:25  error  Strings must use singlequote                      quotes
  127:75  error  Strings must use singlequote                      quotes
  132:9   error  Unexpected console statement                      no-console
  132:21  error  Strings must use singlequote                      quotes
  132:61  error  Strings must use singlequote                      quotes
  186:14  error  'leaf' is never reassigned. Use 'const' instead   prefer-const
  204:13  error  'keepPretext' is assigned a value but never used  no-unused-vars
✖ 11 problems (11 errors, 0 warnings)
  5 errors and 0 warnings potentially fixable with the `--fix` option.

@cpanato as expected -- I think most of those are in console statements I'm using to debug. Will remove as I keep going. :)

but if you or @hmhealey need a test server the build must pass in order to generate it

Oh, I see -- I don't think we should test this then. It's not ready, and I have to rewrite the existing tests anyway (and write new ones). It will definitely fail at the moment.

Most of it is working now. Except:

the edit channel title and purpose are broken on master at the moment
the search bar
still need to work on tests, so @hmhealey could you take this off test server for the moment? Thanks!

Apologies for rebasing onto master -- by the time I remembered this was public, it was too far gone. I'll try not to do it again.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spintmint test server creation failed. See the logs for more information.

/home/ubuntu/jenkins/workspace/mw_mattermost-webapp_PR-2634/components/textbox_ql/textbox_ql.jsx

  9:8  error  'QuillEditor' is defined but never used  no-unused-vars

Nice work, @cpoile! Looks like the build is passing -- worth moving this out of draft status so we can setup a test server for UX testing?

@lieut-data There are still a lot of little things that I know won't pass UX:

the input box doesn't focus on a mouse click until the cursor is actually in the input area (compare to master where the cursor changes immediately on crossing the enclosing div's border)
I haven't implemented character limit yet (eg for edit channel purpose modal)
I need to recalculate the size of the editor (for channel purpose modal)
scroll bars don't work yet
cutting and pasting is a little wonky
I haven't finished the markdown->quill transformation functions yet (so editing a post will show markdown, not emojis)

Given those, is it worth moving it out of draft? Your call.

I /suspect/ that's already way better than anyone had expected from a proof-of-concept, so it would be great to validate the fundamental changes here even if there are rough edges.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-036a6d7a76370f356.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-036a6d7a76370f356

@esethna, as discussed above and on the community server, this is not "fully ready" for review: there are lots of known rough edges. However, I think it's almost certainly at a point where we should vet the fundamental changes here (e.g. replacing our text editor with Quill) to make sure we're all pulling in the same direction. Might you give the test server a whirl and provide some initial feedback as such?

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0dea7e7986b655722.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0dea7e7986b655722

Awesome, thanks Eric! I fixed those issues (and found a new one). I'm not sure how to force spinmint to start a new server...
About QA: I would bet QA would flag all the things I've mentioned in the todo list. So maybe we should hold off till I have those polished out?
I've added a running todo list a few posts back. Let me know if I should put it somewhere else.

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0e0c843f99e53db4f.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0e0c843f99e53db4f

Thanks @cpoile, ca we please move the to do list to the PR description? Agree we can hold off on QA testing until the list is complete

Spinmint test running for more than 7 days. This test server was terminated.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@cpoile What are the next steps on this? Will you continue to work on this?

@hanzei yes, it's on my backlog, will probably resume it next week.

Closing for now, will reopen if/when we need to.


Questionnaire Links

As a user, when viewing a questionnaire, I would like to navigate to the questionnaire's links, allowing me to print the questionnaire, view user guides or see the license in more detail.

May make sense to add after #21 is in place.


Inconsistent or not documented "close" interface

First discussed here
When the queue is not empty and a close handler was added the context is passed instead error.
In all cases of close the user expects an error.

Definitely.
Would you be able to send a PR?

What's your expectation?

I will prepare a PR

Released with https://github.com/mcollina/avvio/releases/tag/v5.4.2


Remove options to disable kube-vip

Details

While I like the option to disable it there's a lot of issues and logic that I really don't want to maintain.
If the configure script was more robost and used jinja2 or gotmpl to create user specific files I'd consider keeping this option but using yq and it's annoying version sprawl is very unfriendly.
On a single node cluster kube-vip can still be used, and if that's not desirable there can be documentation written somewhere to explain how to use the node IP instead.

Done in https://github.com/onedr0p/flux-cluster-template/commit/1134b2927b940cb4de9b9b744dbe89046b5912d6


MF-1340 - Add CLI config TOML file

Pull request title should be MF-XXX - description or NOISSUE - description where XXX is ID of issue that this PR relate to.
Please review the CONTRIBUTING.md file for detailed contributing guidelines.
What does this do?
Creates a TOML file for CLI
Which issue(s) does this PR fix/relate to?
#1340
List any changes that modify/break current functionality
N/A
Have you included tests for your changes?
No
Did you document any new/modified functionality?
No
Notes

@WashingtonKK please refer to the correct originating issue

There is no support in the CLI for parsing most of the config parameters in this PR
https://github.com/mainflux/mainflux/blob/aee0081864f9654f0eeb2c08a36ed5350ce04372/cli/config.go#L15-L20
.
I also don't think this PR fully addresses the requirements in #1340

What do you recommend?
Also, @dborovcanin , what do you think?

There is no support in the CLI for parsing most of the config parameters in this PR
https://github.com/mainflux/mainflux/blob/aee0081864f9654f0eeb2c08a36ed5350ce04372/cli/config.go#L15-L20
.
I also don't think this PR fully addresses the requirements in #1340

What do you recommend?
Also, @dborovcanin , what do you think?

Addressed in PR review.

@WashingtonKK , Configurations in config.toml are not getting configured in SDK,
I tried with users URL it doesn't works

@WashingtonKK , Configurations in config.toml are not getting configured in SDK, I tried with users URL it doesn't works

@arvindh123 , I have fixed this

@SammyOina please review and test this one

I have resolved the comments.


Add state transition context

We will need some sort of Context for a variety of state transition optimization we will want, and this also is a nice place to pass around some configuration data that is a bit unwieldy to pass around in other ways (e.g. thru compile-time constants)

i'll copy the commit message that i think summarizes the need here:
add Context type to state transition

lay ground work for future optimizations
pass around "compile-time" constants that are prohibitive to use
as const generics

as the situation around compile-time constant inference improves, we can shrink the Context over time... for now this should be sufficient


[Feature request] Select several packages

It would be very handy to be able to (optionally) select several packages to install/uninstall at once instead of having to go back and forth between Pacseek and yay one package at a time.

Not sure if it's worth to implement.
So not a priority for me atm to be honest.

Not sure if it's worth to implement. So not a priority for me atm to be honest.

really hope that queuing is possible in the future


[Website] can't be reached over IPv6

Your website can't be opened over IPv6 connections. See: https://www.ssllabs.com/ssltest/analyze.html?d=feedi.me

2001-08d8-100f-f000-0000-0000-0000-0271.elastic-ssl.ui-r.com
Failed to communicate with the secure server
static.<IP_ADDRESS>.clients.your-server.de
Ready

7 months later and problem still exist.
Looks like no interest / project inactive

changed to api.feedirss.com https://www.ssllabs.com/ssltest/analyze.html?d=api.feedirss.com it works now!


Add A real  It's not a bug, it's a feature  feature

Similar to the famous Nuclear Gandhi bug in Civilization II, we can introduce:
Bug: If the snake gets too long, its length inverts and its head is now at the end of its body (instead of the tail being at the end).
Feature: This feature of moving "head-last" allows the rest of the snake's body to overflow through the canvas, thus it easier to handle and it the player will have an easy time staying alive.
If we include some signs and references to Integer Overflow and Nuclear Gandhi it can add to the game design.
Originally posted by @Sushant-Padha in https://github.com/legosiv/valiant-valkyries/issues/13#issuecomment-1198464187

Todo:

switch the head and tail images
change collision detection so the user only loses when the snakes entire body is outside not just the "front"

We could have each one design their own bug, we introduce a timer/event, when event gets triggered a random buggish feature occurs.


Move plugin index

As long as our plugins are all in /plugins/, the index of all plugins should be at /plugins/

Only those with write access to this repository can merge pull requests.

How did merging this break the build?


SQLite Unknown column type timestamp

Unknown column type: timestamp  table name: user column name: created_at
Error occurred in typeorm-model-generator.
<EMAIL_ADDRESS><EMAIL_ADDRESS>

What is type of this column(shown in some gui/from statements which creates it)?
I don't see timestamp in types supported by sqlite databases in typeorm https://github.com/typeorm/typeorm/blob/f2ba9012fe4e851bc667dfdfedc3fd4af665d52b/src/driver/sqlite-abstract/AbstractSqliteDriver.ts#L77-L109

ouch, you are right. It was my mistake to write timestamp instead datetime in my migrations). I was testing the library trying re-create the DB.


Upgrade deps (Gatsby 5.11) and minor improvements

Upgraded all the dependencies I could. Noteworthy:

Gatsby was upgraded from 5.7 to 5.11

Version 5.12 was a no go, unfortunately. They introduced hosting adapters which supersedes gatsby-plugin-netlify, but it does not look ready at all (lots of regressions and it doesn't work with Gatsby esm configs). Tracking https://github.com/gatsbyjs/gatsby/issues/38542 and https://github.com/gatsbyjs/gatsby/issues/38606

Prettier v3 .format() is async now, so I refactored the tags-transforms script accordingly
Can't upgrade to remark-gfm 4 at the moment, I assume there is work to do in gatsby-plugin-mdx still. I saw some babel-loader errors when trying to parse markdown files.

And the minor improvements:

Added links to guide card images in /guides
Added a missing React key in the showcase form

Wow, thank you @fturmel! @runemadsen @fdoflorenzano I've looked this through and am going to merge, but wanted to tag you in just as an FYI.


Vickel: Watch ahrensdaniel

Vickel requests the watch of the watch_keyword ahrensdaniel. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ahrensdaniel has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Unable to write ADF files with verify left on

Hello all, Newbie to all this but I have an issue when writing ADF files to disk.
I have a Geaseweazle v4.1 (updated to firmware 1.5) and a "new" old stock Sony MPF920 3.5 floppy drive (looks mint condition with no signs of use and quiet in operation). Using FluxMyFluffyFloppy (v5.0.3).
Was able to format and read disks fine but had trouble getting anything ADF file to write until I selected the "No verify" option. Here is what I get with the "No verify" option off:
C:\FluxMyFluffyFloppy-master>"C:\greaseweazle-1.16.2\gw.exe" write --retries=3 "I:\AmigaTestKit-1.21\AmigaTestKit.adf"
Writing c=0-79:h=0-1
Format amiga.amigados
T0.0: Writing Track (Flux: 200.0ms period, 219.9 ms total, Terminate at index)
T0.0: Writing Track (Verify Failure: Retry #1)
T0.0: Writing Track (Verify Failure: Retry #2)
T0.0: Writing Track (Verify Failure: Retry #3)
** FATAL ERROR:
Failed to verify Track 0.0
Enabling "no verify" and the disk appear to write ok but concerned there may an issue with the drives write quality. Is this normal behaviour, have I missed an option for this drive or does it sound like a bad drive?
Any ideas?

If the drive can write, and it can read, then verified write should not be a problem. Have you tried other disks? Are you sure the write did actually write?

Tried a variety of disk old and new including a box of new sealed HD disks. Same result for all.
Disks are writing although I'm getting the occasional read error when I try them on my Amiga A1200, although that could be the age of the internal drive. Without verifying the write I'm not sure if the read errors are coming from the write or the read phase.

If you're using HD disk be sure to cover the HD hole. Otherwise you can expect read errors on a high density drive (double density Amiga drives don't look for the HD hole).

Perfect, that's got it working correctly now.
Many thanks


Still need godotsteam.sh

Describe the bug
GodotSteam does not launch scenes if it is not run through the godotsteam.sh script.
To Reproduce
Steps to reproduce the behavior:

Create a new project
Run linux-4-editor.x86_64
Launch any scene

Expected behavior
The scene does not launch
Desktop (please complete the following information):

OS: Ubuntu
Version 22.04.2 LTS

Version of Godot & GodotSteam:
Godot 4.0 - Steamworks 1.55 - GodotSteam 4.2
Fix:
Run GodotSteam 4.2 with the godotsteam.sh script provided in older versions.

Hmm, so interestingly enough for Godot 3.x builds, this fix works.  For 4.x, it does not.  Also, how libsteam_api.so is linked in is completely different:
3.x - libsteam_api.so => /home/gp/Programs/Godot/./libsteam_api.so (0x00007f8fd4c59000)
4.x - libsteam_api.so => ./libsteam_api.so (0x00007fc4a6d1a000)

One using a relative path and one using absolute path.
@mikix Any idea why that would be?  The same change was applied to both branches with, at least for me, totally different outcomes.
@JonasMht Can you also try the newest Godot 3.5.2 version of GodotSteam from the release section to see if you get a similar outcome?  I don't need the wrapper for that one but I do for the Godot 4.x version.

@Gramps Running 3.5.2 without the wrapper works as intended on my side.

Hmm. Thanks for testing and reporting!  Not sure why this worked for 3.x but not 4.x but good to have confirmation.  Maybe @mikix has some ideas.  I will look into it more this afternoon!

No immediate ideas, but maybe other build script changes are conflicting with what my change tried to do. I might not be able to look at it today.

All good, just tagging you in to see if you might know something.  I will check on it a little later today and see what I can find!

OK, after getting up, eating, and sitting down at my desk, I figured out what happened!  These 4.x builds were made manually since Sapphire hasn't quite finished the Github Action stuff.  I never updated the SCsub file locally which is probably why this didn't work for the 4.x version.  I am now compiling with the updated SCsub file and will reissue the 4.x release.  That should fix this...

🙂 No worries.

@Gramps Just tried the new build, it works fine :)

Excellent!  Then 100% my fault and all is well now!  Thank you for reporting it and testing the new build!


Design contribution

HI @ushiushix  I am a designer and intarest about project.
i want to contribute to your project by designing a unique logo.
if you have any requirements then let me know.


publish V2 throws error locally

Describe the bug
when running the publish command mentioned in docs: https://dendron.so/notes/861e4e48-dcc5-4813-a695-8940ba6e64d3.html#setup,
To Reproduce

build site using instructions in docs

Expected behavior
site should be built locally and available on localhost:8080
Screenshots

.
Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser: firefox
Dendron: 0.22.0
Nodejs: v14.15

This is one fo the known regressions from the eleventy publishing - we use
prism behind the scenes and it doesn't support all syntaxes. Will take a
look at adding dot.
On Tue, Dec 29, 2020 at 5:29 PM Devashish<EMAIL_ADDRESS>wrote:

Describe the bug
when running the publish command mentioned in docs:
https://dendron.so/notes/861e4e48-dcc5-4813-a695-8940ba6e64d3.html#setup,
To Reproduce

build site using instructions in docs
https://dendron.so/notes/861e4e48-dcc5-4813-a695-8940ba6e64d3.html#setup

Expected behavior
site should be built locally and available on localhost:8080
Screenshots
[image: image]
https://user-images.githubusercontent.com/1110988/103324183-1f8ec000-49fb-11eb-87ed-b098faeaa5a1.png
.
Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser: firefox
Dendron: 0.22.0
Nodejs: v14.15

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/issues/434, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPIPQY5QL62MFVAOTTKJ3SXJ67RANCNFSM4VNZO4ZQ
.

makes sense. looking forward to the fix. i'll revert to using publishing v1 for the time being. thanks!

makes sense. looking forward to the fix. i'll revert to using publishing v1 for the time being. thanks!

is there a way to reduce the compile time for v1 publishing on local system. Its ridiculously long for me right now ~30 mins using bundle exec jekyll serve --incremental command for about 500 notes

is there a way to reduce the compile time for v1 publishing on local system. Its ridiculously long for me right now ~30 mins using bundle exec jekyll serve --incremental command for about 500 notes

The long compile time was the reason why we're switching to eleventy. I've
noticed the same for http://kevinslin.com/ (around 450 pages). The new 11ty
publishing publishes everything in ~11s. We will have the prism thing fixed
by Monday so you should be able to switch over at that point
On Sat, Jan 2, 2021 at 2:34 PM Devashish<EMAIL_ADDRESS>wrote:

is there a way to reduce the compile time for v1 publishing on local
system. Its ridiculously long for me right now ~30 mins using bundle exec
jekyll serve --incremental command for about 500 notes
[image: image]
https://user-images.githubusercontent.com/1110988/103467782-5d475d80-4d07-11eb-960f-fecece7961ef.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/issues/434#issuecomment-753538248,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPIPVCKDKWGEGIB5FM7ZDSX6NOHANCNFSM4VNZO4ZQ
.

The long compile time was the reason why we're switching to eleventy. I've
noticed the same for http://kevinslin.com/ (around 450 pages). The new 11ty
publishing publishes everything in ~11s. We will have the prism thing fixed
by Monday so you should be able to switch over at that point
On Sat, Jan 2, 2021 at 2:34 PM Devashish<EMAIL_ADDRESS>wrote:

is there a way to reduce the compile time for v1 publishing on local
system. Its ridiculously long for me right now ~30 mins using bundle exec
jekyll serve --incremental command for about 500 notes
[image: image]
https://user-images.githubusercontent.com/1110988/103467782-5d475d80-4d07-11eb-960f-fecece7961ef.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/issues/434#issuecomment-753538248,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPIPVCKDKWGEGIB5FM7ZDSX6NOHANCNFSM4VNZO4ZQ
.

this is fixed with 0.23.0 release.
thanks @kevins8 for the quick response!!

this is fixed with 0.23.0 release.
thanks @kevins8 for the quick response!!


Change blending options in attempt to fix overlapping billboard fringes.

If this works, it should fix #5007.  Hopefully it whacks this mole without other ones popping up.
To see old behavior, compare GeoJSON in 1.27 to GeoJSON in 1.30.  The newer version is on the right:

This branch offers this:

If you look closely at the 1.27 shot, you can see in the "rounded" corners of the billboard where the old bug was "punching holes" in the ones behind, making the corners appear more squared off.  In the 1.30 shot, everything is messed up, but the "punching holes" bug at least is gone.
On this branch, this demo appears more orderly.  There's a slight issue remaining, where low-alpha fringes of the inset icons punch holes in the background color of the same icon.  This may actually be a bug with the alpha values coming out of the PinBuilder, since the output billboard should be calling for opaque fragments there in the interior, regardless of the inset graphic.
Anyway, I tested this with label backgrounds, and with non-backgrounded labels.  It seems to mostly work.  There's some strangeness where a non-background label that sticks into terrain will have its opaque fragments occluded at a slightly different depth from the translucent fragments.  But mostly I think this is a big improvement.

Here's an example of what I'm talking about with labels sticking into mountains.  I know the text looks crowded and bad, but there are not supposed to be any holes in the glyphs.

This branch large label demo

Cesium 1.27 is too old to load this format of gist.  Cesium 1.28 needs to be manually set to "Natural Earth II" base map, but then it works, and it looks perfect:

Cesium 1.28 large label demo

Cesium 1.30 exhibits the label depth strangeness, which means it is not introduced by this branch:

Cesium 1.30 large label demo

So, it looks like this is a separate, but likely related, issue.

So, it looks like this is a separate, but likely related, issue.

Agreed.  Can you submit a separate issue for this?

Also, update CHANGES.md.

New issue #5083 submitted for the label depth problem mentioned above.

Does anyone else want to look at this?  @bagnell?

Updated.

OK with me.  Does anyone else want to test this out?
I labeled this next release so we get it into master soon.

Now that #5099 was merged to master, and I merged master in here, this fix is looking better than any of the screenshots posted originally.

This looks good to me.

This is ready.


Amend version on installation page

closes #85

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   88.28%   88.28%
=======================================
  Files           8        8
  Lines         111      111
  Branches        3        3
=======================================
  Hits           98       98
  Misses         13       13

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 9374573...b513d43. Read the comment docs.

👍

@jcranky please approve

Thanks @juanjovazquez !

@etorreborre You're welcome. Just a minor thing to motivate myself for getting deeper into grafter ;-)


ttx: out of range error converting Neue Haas Grotesk TTF

Source of the TTF files:
https://github.com/cid-harvard/atlas-economic-complexity/find/06ee6ee01fc692a983321aceea8325b8834260c2
search for ".ttf"
This zip will last 14 days (feel free to edit/remove the link out when you've downloaded): https://transfer.sh/tsaQu/nhg.zip
Generally the error is:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/fontTools/ttLib/tables/ttProgram.py", line 235, in toXML
    assembly = self.getAssembly()
  File "/usr/local/lib/python2.7/site-packages/fontTools/ttLib/tables/ttProgram.py", line 229, in getAssembly
    self._disassemble(preserve=preserve)
  File "/usr/local/lib/python2.7/site-packages/fontTools/ttLib/tables/ttProgram.py", line 461, in _disassemble
    value = (bytecode[i] << 8) | bytecode[i+1]
IndexError: array index out of range

ERROR: An exception occurred during the decompilation of glyph program:

Terminal output:
https://gist.github.com/gingerbeardman/d22ea80c3c4aae5353a930040522cd73
ttx --version
3.20.1
ProductName:	Mac OS X
ProductVersion:	10.13.2
BuildVersion:	17C88

hm.. the hinting data in those fonts seem to be broken. Where did they come from?

from the cid-harvard GitHub project I linked 😨

the error is saying that our ttProgram module's opcodeDict is missing the opcode with value 64.
Now, that op code is indeed missing if I pretty-print that dictionary:
[...]
62: ('MIAP', 1, 62, 'MoveIndirectAbsPt'),
 63: ('MIAP', 1, 62, 'MoveIndirectAbsPt'),
 66: ('WS', 0, 0, 'WriteStore'),
 67: ('RS', 0, 0, 'ReadStore')
[...]

I don't know...

ignore what I wrote, sorry... 0x40 (or 64 in decimal) is the opcode of NPUSHB (the KeyError is normal, as those special opcodes are handled separately).
The real issue is the IndexError at line 455, which happens probably because the instruction stream for that glyf was truncated somehow, as the NPUSHB is requesting to read more values than are in the bytecode array.
Yes, definitely a broken font.

Ah, sorry for wasting time. I should have checked...
Font Book and ftxvalidator also complain.

Here's a FontVal page for one of the files.
A bunch of warnings but only one error, usWeightClass not valid value.

Font Validator Report - ed18aa48-557e-4d1f-a53c-58399a7c1bc4.ttf
Show	 Info	 Pass	 Warning	 Error
Run DateTime:	13 December 2017 16:23
Machine Name:	MacBookPro-Matt
Font file:	/Users/matt/Downloads/2017-12-13/nhg/ed18aa48-557e-4d1f-a53c-58399a7c1bc4.ttf
Error code	Message	Details
Index: 0, NeueHaasGroteskDisp W01, Version 1.01, 06/05/2014
Table Tag	Table Offset	Table Length	Table Checksum
OS/2	0x0000010c	0x00000060	0x65be6180
VDMX	0x0000016c	0x000005e0	0x6e07758a
cmap	0x0000074c	0x0000017a	0x9aa8ba11
cvt	0x000008c8	0x000001e2	0x08370950
fpgm	0x00000aac	0x00000aa2	0x630e9dd9
gasp	0x00001550	0x00000008	0xffff0004
glyf	0x00001558	0x00007aa0	0x8289b696
head	0x00008ff8	0x00000036	0xfc4ffd8e
hhea	0x00009030	0x00000024	0x07e40460
hmtx	0x00009054	0x00000370	0xba7c20a2
kern	0x000093c4	0x0000b444	0xecf7ffdc
loca	0x00014808	0x00000374	0x00322710
maxp	0x00014b7c	0x00000020	0x03340110
name	0x00014b9c	0x0000167d	0x997e85d2
post	0x0001621c	0x00000020	0xff880014
prep	0x0001623c	0x00000257	0xa3e08d9d
General Font Information
Error code	Message	Details
W0022	Recommended table is missing	hdmx
W0022	Recommended table is missing	DSIG
W0020	Tables are not in optimal order	table 'OS/2' precedes table 'head'
OS/2
Error code	Message	Details
W2106	The version number is valid, but less than 5	3
E2133	The usWeightClass is not a valid value	450
W2101	A CodePage bit is set in ulCodePageRange, but the font is missing some of the printable characters from that codepage	bit #4, Turkish (missing chars: U011E U0130 U015E U011F U015F)
W2101	A CodePage bit is set in ulCodePageRange, but the font is missing some of the printable characters from that codepage	bit #19, Korean Wansung (17003 missing, first ten missing chars are: UAC02 UAC03 UAC05 UAC06 UAC0B UAC0C UAC0D UAC0E UAC0F UAC18)
W2101	A CodePage bit is set in ulCodePageRange, but the font is missing some of the printable characters from that codepage	bit #21, Korean Johab (17019 missing, first ten missing chars are: U11A8 U11A9 U11AA U11AB U11AC U11AD U11AE U11AF U11B0 U11B1)
W2101	A CodePage bit is set in ulCodePageRange, but the font is missing some of the printable characters from that codepage	bit #29, Mac character set (23 missing, first ten missing chars are: U2260 U221E U2264 U2265 U2202 U2211 U220F U03C0 U222B U2126)
W2101	A CodePage bit is set in ulCodePageRange, but the font is missing some of the printable characters from that codepage	bit #56, IBM Turkish (34 missing, first ten missing chars are: U0130 U015E U015F U011E U011F U2591 U2592 U2593 U2502 U2524)
W2101	A CodePage bit is set in ulCodePageRange, but the font is missing some of the printable characters from that codepage	bit #62, WE/Latin 1 (30 missing, first ten missing chars are: U2591 U2592 U2593 U2502 U2524 U2563 U2551 U2557 U255D U2510)
VDMX
Error code	Message	Details
W2500	The VDMX data doesn't match the data calculated by the rasterizer	The differences were smaller than the tolerance, so they may well be valid if the VDMX was hand-tuned
cmap
Error code	Message	Details
W0302	The table does not contain any Apple subtables
W0301	Apple logo mapping test not performed, cmap 1,0 not present
cvt
Error code	Message	Details
fpgm
Error code	Message	Details
gasp
Error code	Message	Details
glyf
Error code	Message	Details
W1111	Duplicated knots	Glyph index 8
W1111	Duplicated knots	Glyph index 9
W1111	Duplicated knots	Glyph index 23
W1111	Duplicated knots	Glyph index 91
W1111	Duplicated knots	Glyph index 113
W1112	Not all extremes are marked with the on-curve control points	Glyph index 199
head
Error code	Message	Details
W1302	Non-linear scaling flag (bit 4) is set, but hdmx table is not present
W1303	Non-linear scaling flag (bit 4) is set, but LTSH table is not present
W1313	The unitsPerEm value is not a power of two	1000
hhea
Error code	Message	Details
hmtx
Error code	Message	Details
kern
Error code	Message	Details
loca
Error code	Message	Details
maxp
Error code	Message	Details
name
Error code	Message	Details
W2004	The description strings are inconsistent	(1,0,0,10)='Designed by Christian Schwartz 2004?2010. Originally designed 1957-1961 by Max Miedinger with art direction by Eduard Hoffmann. Released as Neue Haas Grotesk by the Haas'sche Schriftgiesserei, and then revised and released as Helvetica by Linotype AG. Revival originally commissioned in 2004 by Mark Porter at The Guardian. Completed in 2010 for Richard Turley at Bloomberg Businessweek. Thinnest weight designed by Berton Hasebe.', (3,1,1033,10)='Designed by Christian Schwartz 20042010. Originally designed 1957-1961 by Max Miedinger with art direction by Eduard Hoffmann. Released as Neue Haas Grotesk by the Haas'sche Schriftgiesserei, and then revised and released as Helvetica by Linotype AG. Revival originally commissioned in 2004 by Mark Porter at The Guardian. Completed in 2010 for Richard Turley at Bloomberg Businessweek. Thinnest weight designed by Berton Hasebe.'
post
Error code	Message	Details
W2304	Apple recommends against using post table format 3 under most circumstances
prep
Error code	Message	Details
Rasterization Test, BW
Error code	Message	Details
Rasterization Test, Grayscale
Error code	Message	Details
Rasterization Test, ClearType
Error code	Message	Details
Note: The Font Validator's helpfile contains detailed information about each error message. The latest OpenType specification is available at the Microsoft Typography website.

FontVal.pdf


mkgmap: 4432 -> 4555

Motivation for this change

Changelog (svn log -r 4432:4555)
enable tests
add mkgmap-splitter

Things done

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

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)

$ nix path-info -Sh ./result
/nix/store/0rk5280sd8ayj3jv2x904iqv24jhpvkk-mkgmap-4432	 221.0M
$ nix path-info -Sh ./result
/nix/store/gvxgr8l7r9cqbncdp3f4xmgvk6ni371x-mkgmap-4555	 220.0M
$ nix path-info -Sh ./result
/nix/store/v3gdks6bff4f9q8gx7dr11ri1z9yij9f-splitter-597	 218.3M

[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/225

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/243


advanced_demo.py closes immediatly

I'm not sure if if supposed to be entering a loop or what, but when I run the basic example it closes immediately with no error output.
Not sure if this would be a better issue here or at https://github.com/littlevgl/lv_micropython

@traverseda advanced_demo.py is not supposed to close immediately.
It is supposed to display an interactive GUI with two pages, a few buttons etc.
It is working for me on both unix and ESP32 port, so I'm not sure what the problem is on your side.
It would help if you could provide more information:

What micropython port are you using?
How did you build it? What make command did you use?
Were there any warnings or other problems during the build?
Did you get the micropython prompt? Did you try to use it?
How did you load advanced_demo.py on your port? On the unix port the simplest thing is to copy it, click CTRL+e, paste the script, then click CTRL+d to run the script you pasted.
On ESP32 it's better to first write the script to the file system then import it from there.

I'm not sure if if supposed to be entering a loop or what, but when I run the basic example it closes immediately with no error output.
Not sure if this would be a better issue here or at https://github.com/littlevgl/lv_micropython

@traverseda  Try to add the while True: pass at the end of file.

@traverseda Try to add the while True: pass at the end of file.

@fanday I don't think this is the issue. The REPL (interactive prompt) should remain open and responsive to user input. If it closes, there is some problem.
Also worth running it with gdb and see if there was some exception thrown, invalid memory access etc.

@traverseda Try to add the while True: pass at the end of file.

@fanday I don't think this is the issue. The REPL (interactive prompt) should remain open and responsive to user input. If it closes, there is some problem.
Also worth running it with gdb and see if there was some exception thrown, invalid memory access etc.

@amirgon Somebody always try run the script as this:./micropython advanced_demo.py.And it seems nothing on the screen.

@amirgon Somebody always try run the script as this:./micropython advanced_demo.py.And it seems nothing on the screen.

@fanday You can't run the script like that, it will not work. It should run in micropython with lvgl module, not as a standalone Python script.
If someone tries to run it like this, he would get import error immediately since lvgl module is missing on regular Python.

@amirgon Somebody always try run the script as this:./micropython advanced_demo.py.And it seems nothing on the screen.

@fanday You can't run the script like that, it will not work. It should run in micropython with lvgl module, not as a standalone Python script.
If someone tries to run it like this, he would get import error immediately since lvgl module is missing on regular Python.

@amirgon Yes,you are right.I can run it after build the lv_mpy.

I can confirm that running the repl and typing import advanced_demo works fine, but trying to run it like a normal python script using micropython advanced_demo.py causes it to close immediately.

@traverseda Try to add the while True: pass at the end of file.

@fanday you were right.
The REPL is not run by default when a script is given as a parameter.
Fixed on 50ebf70.
Closing.


Refactor internal connection code.

Changed: Improve internal gRPC connection management for better error propagation.
Fixes #223
Major rewrite of how gRPC connections are managed internally. This new design allows exceptions like InterruptedException to be propagated even when they occur in the connection manager thread.
The public client method shutdown changed. It used to be void but now it's CompletableFuture<Void>. Technically, it shouldn't break user code unless they have enabled very pedantic compiler options.
However I deprecated the Endpoint type as using InetSocketAddress across the board was simpler and removed unnecessary allocations when converting Endpoint to InetSocketAddress.
Technically those breaking changes, while minor, warrant upping the major release number.

Before merging, let's discuss how this should be mentioned in the release notes. /cc @ylorph

Request review from Claude as we would like someone strong in Java to look at this.


Cython crashes when trying to build extension from init.py

I've noticed this on some occasions.
When Cython tries to build an extension module with distutils, it crashes and spits out an error like this.
Here is me trying to compile a test __init__ file from the command line.
C:\Users\You>cythonize -i -3 __init__.py
running build_ext
building 'You.__init__' extension
creating C:\Users\tmpo_kxxv5_\Release
creating C:\Users\tmpo_kxxv5_\Release\Users
creating C:\Users\tmpo_kxxv5_\Release\Users\You
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\x86_amd64\cl.exe /c /
nologo /Ox /W3 /GL /DNDEBUG /MD -Ic:\python37\include -Ic:\python37\include "-IC
:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\INCLUDE" "-IC:\Program Fil
es (x86)\Microsoft Visual Studio 14.0\VC\ATLMFC\INCLUDE" "-IC:\Program Files (x8
6)\Windows Kits\10\include\10.0.17763.0\ucrt" "-IC:\Program Files (x86)\Windows
Kits\NETFXSDK\4.6.1\include\um" "-IC:\Program Files (x86)\Windows Kits\10\includ
e\10.0.17763.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17
763.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\winrt"
/TcC:\Users\You\__init__.c /FoC:\Users\tmpo_kxxv5_\Release\Users\You\__init__.ob
j
__init__.c
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\x86_amd64\link.exe /n
ologo /INCREMENTAL:NO /LTCG /DLL /MANIFEST:EMBED,ID=2 /MANIFESTUAC:NO /LIBPATH:c
:\python37\libs /LIBPATH:c:\python37\PCbuild\amd64 "/LIBPATH:C:\Program Files (x
86)\Microsoft Visual Studio 14.0\VC\LIB\amd64" "/LIBPATH:C:\Program Files (x86)\
Microsoft Visual Studio 14.0\VC\ATLMFC\LIB\amd64" "/LIBPATH:C:\Program Files (x8
6)\Windows Kits\10\lib\10.0.17763.0\ucrt\x64" "/LIBPATH:C:\Program Files (x86)\W
indows Kits\NETFXSDK\4.6.1\lib\um\x64" "/LIBPATH:C:\Program Files (x86)\Windows
Kits\10\lib\10.0.17763.0\um\x64" /EXPORT:PyInit___init__ C:\Users\tmpo_kxxv5_\Re
lease\Users\You\__init__.obj /OUT:C:\Users\You\__init__.cp37-win_amd64.pyd /IMPL
IB:C:\Users\tmpo_kxxv5_\Release\Users\You\__init__.cp37-win_amd64.lib

LINK : error LNK2001: unresolved external symbol PyInit___init__
C:\Users\tmpo_kxxv5_\Release\Users\You\__init__.cp37-win_amd64.lib : fatal error
 LNK1120: 1 unresolved externals

error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\BIN\\
x86_amd64\\link.exe' failed with exit status 1120

Not only that, but it creates pyd fieles when then crash the program if something tries to import them because there empty.
I believe the error is caused by PyInit___init__, which probably is aukward enough to cause the compiler to throw an error.
The contents of the init file is:
'''testing file'''
pass

This was resolved in b43bf13574669dbb10e1ac4c992f9fc66132b6f4.

(I don't think there was a ticket for this yet, so I'm using this one as a reference in the changelog.)

Thanks!
I guess I need to download Cython from Github and install that if I want to compipe init

@Keithcat1 where did you download Cython from?

Must've been from pip.
I got my current cython version from the command:
"pip install cython==3.0a5"
Anyway, it works now, so closing.

I still see essentially the same issue posted above on the latest version of cython (3.0a6) when attempting to create a bdist_wheel. Here is an example of how my setup script refers to all python scripts within each package directory and subdirectory. These directories each contain empty init.py files, which seem to be the cause of this.
ext_modules=cythonize(
[
Extension("mypkg.", ["mypkg/.py"]),
Extension("mypkg.subpkg*", ["mypkg/subpkg/*.py"]),
],
Worth noting that this same setup script builds without issue on Mac and Linux. This appears to be a Windows-only issue. Strangely,  I'm able to repeatedly run the setup script in an attempt to build the wheel and eventually after a dozen or so failures with the LNK1120 error, the .whl is finally built.

For reference: The relevant Python bug report - this is really an issue with distutils rather than Cython.

@justinvirtualitics this shouldn't happen any more with the 3.0 alpha releases. Are you sure it also appears in a clean checkout?
Apart from that, is there a reason why you would want to compile empty __init__.py files, or is it more for the convenience of using a simple "compile all" path expression?

@scoder as far as I can tell, imports will fail if init.c is not present

even after updating to the latest cython, the old *.c files must be deleted and regenerated

even after updating to the latest cython, the old *.c files must be deleted and regenerated

That is correct. Cython does not attempt to identify the Cython version that generated a .c file when it determines whether the generated file is up-to-date. That seems a reasonable choice that avoids guesses about the users' intent. If you want a clean state, clean up your workspace. If you want Cython to regenerate everything, pass -f.


Custom subtitle outline width

Use case description
Subtitle edge (text outline) is an important part of accessibility to make text readable. However the current implementation hardcodes this to twoDpInPx and on some devices like some TV screens, this might be very small and make text unreadable. There is currently no API surface nor internal method accessible via reflection to change the subtitle outline width.
Proposed solutions/Alternatives considered

Allow for some sort of modification of this via CaptionStyleCompat. I know this might not be possible due to android.view.accessibility.CaptioningManager.CaptionStyle. But honestly, this restriction is makes no sense if I as an app developer want to deliver an specific experience.
Change this to Sp. Why is it even in Dp to begin with? The android docs specifically calls Sp when text is involved. https://developer.android.com/training/multiscreen/screendensities
Allow for cues to modify this so we can modify the text style in SubtitleParser.
Allow for custom subtitle painters/subtitle views. Right now the subtitle view is directly set to R.id.exo_subtitles and is a final SubtitleView, meaning that there is no possible way to modify the subtitle rendering after parsing it.
Insert a hidden reflection method/mark as non final. Even with reflection this property is currently impossible to modify as it is marked as private + final. Just adding a hidden method or marking this as non final would fix this. It is impossible to modify this one variable without forking the entire repository.

Note, this is a resubmission of https://github.com/google/ExoPlayer/issues/10507 but exoplayer has been marked as deprecated.

Thanks - I think my comment here about the 'right' way to implement a fully custom outline (by adding it to Cue) probably still stands (i.e. your 3) suggestion): https://github.com/google/ExoPlayer/issues/10507#issuecomment-1215155994
We don't have plans to do this soon, but would consider a high quality PR implementing this (including rendering support in both CanvasSubtitleOutput and WebViewSubtitleOutput).

2. Change this to Sp. Why is it even in Dp to begin with? The android docs specifically calls Sp when text is involved. https://developer.android.com/training/multiscreen/screendensities

This is interesting - looking at the blame layer, it's been this way for a long time. Since this code was first published to GitHub in 2017, but internally I can see this line was added in the very first version of the SubtitleView file in 2014. So it's not something we've deliberately decided to do recently.
However, the link you posted doesn't seem to explicitly say that "text related styling" should use sp, just text size itself (which isn't quite what we're discussing here):

When defining text sizes, you can instead use scalable pixels (sp) as your units. The sp unit is the same size as a dp, by default, but it resizes based on the user's preferred text size. Never use sp for layout sizes.


Not working anymore?

Does it still work for sending apn? not working anymore for me.

Hello. As mentioned in another issue, none of the original developers/maintainers use this tool any longer, and so we've had no time to keep atop it. I know Apple changed the protocols a few years back, but this tooling still uses the original APNS protocols, so it would not surprise me if this no longer works.


more type safe dual signature, closes #2967

Type

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Ad discussed in #2967, the dual type signature could be improved to avoid inferring an (...args: any[]) => any function type, which is assignable to  every function signature you define for your dual api, and instead be precise over the implementation signature used. Of course the rest of the type signature would still be unsafe, but that's unavoidable.
This should help prevent errors when the type signature of the implementation used for the dual api is different to the one used as the final signature of such dual api.
NOTICE:
There are a couple of tests failing, regarding the schema package, which should not be related to this change. In fact, if you run the tests on the main this pr is based on, those tests keep failing.
This PR should be type level only. I added dtslint tests for the dual signature type checks I did in the first place that brought this change to exist. This should help to assert those cases are preserved in the future.
Related

Closes #2967

Imho seems to be over complexifying usage, also it would be a breaking change that potentially breaks user code, not sure about it

Imho seems to be over complexifying usage, also it would be a breaking change that potentially breaks user code, not sure about it

Usage should be the same in the "common case" scenario. There are a couple of instances where it can't infer automatically the whole type you'd like your dual signature to be, and require you to type the Other and Impl parts of it as generic types. This, btw, remove the need to type the implementation internally though, so it's more like moving and typing more directly than just inferring.
It's true some user code could be breaking due to this change, at least at the type level, since this PR is all about types, even if it's just about reordering types so to put generics in place.

This would be a major change, the changeset will need adjusting.
dual was also made quite "relaxed" on purpose, so I'm on fence about this change.

This would be a major change, the changeset will need adjusting.
dual was also made quite "relaxed" on purpose, so I'm on fence about this change.

Just to have a clear view on this so that everyone is aligned, the "new" signature would be as relaxed as the old one. You can always tell ts your dual signature is something else other than the actual implementation, which is always the case, no matter how strict you write the dual signature. This only enforces the implementation you write don't end up being treated as an "any" function early on without you noticing.


Bump Python to python3.12

SUMMARY
python3.12
ISSUE TYPE

Python version update

ACTUAL RESULTS

creating Lambda Function (stop_rds-schedule): operation error Lambda: CreateFunction, https response error StatusCode: 400, RequestID: b7bb373a-862e-40c4-b5ad-131e73e4450a, InvalidParameterValueException: The runtime parameter of python3.7 is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (python3.12) while creating or updating functions.

Jacalab, the current master code seems to be using python3.10, just check the main.tf. Are you using a recent release?


Name of link from another package is not showed

Links in my package point to ["@deep-foundation/core", "Any"], I see this link, I see its type, I see its id, but I do not see its name. Before in the previous versions I have been seeing its name
This behavior occurs in the latest dev monorepository in gitpod

https://github.com/deep-foundation/deepcase/issues/189 may have a clearer/better description.


Disallow uploading mods to the mod browser with the default build.txt description or no homepage field

See title.

Do you mean requiring one of these, or both? Because I would definitely disagree with always requiring a homepage. I have a few utility mods that really don't need a homepage, and requiring me to supply one would just be annoying. It's not like I can't just supply an empty webpage afterall.

If the URL is invalid, the mod is likely also invalid.
It's trivial to supply a quick TCF thread or similar page for at least some communication or information point relevant to the mod.
There's a strong correlation between a zero-effort mod and not having a homepage.

I'm with hamstar here, simple mods may be entirely self-describing but we'd like a place to send people for discussion/support.  Multiple mods can all have the same homepage. I really shouldn't suggest this but, there's no real way to distinguish google.com from an actual mod page so ¯_(ツ)_/¯

First of all, there is no 'default' description and homepage field. We added a homepage field to be entirely optional as a useful link that can be clicked (as opposed to in the description). I don't think we should restrict at all based on these criteria. That there's a strong correlation between a zero-effort mod and it not having a homepage is not really relevant, bad mods don't get many downloads anyway.

It's not an exact science to tell a bad mod by its title/thumbnail alone, just as judging a book by its cover makes little sense.
Though to be fair, that's why I made a tagging feature with Mod Helpers...

What people do, such as judging a book by its cover, is not up to us. I do not believe we should be restricting what people get to see based on our own beliefs, whether we think it is right or wrong that they did or didn't put any time into their description.

A non-mod skeleton description and the existence of a homepage field are not really matters of subjectivity...

I don't see how this would do anything positive. All it does is discourage people who are new, they don't want to make a thread for some 1 item mod they've made that they just want to be able to share with friends in a way that's easy to keep up to date.
What's so hard about just not downloading a mod if you don't know what it does?

Thankyou for your input @Luiafk, I think you raise an important point. Thankyou for adding a different perspective/use-case. I think the mod-browser as an easy mod sharing medium is quite valuable for new people. I previously had only considered it as a repository for 'ready-to-use' mods

Mods can be shared for that use by private file exchanges or hosting a server. Any other use involves everyone else browsing through a tedious list judging books by their covers. If a mod wants to be junk, it should have some formal information accounting for it as junk.

Well ignoring the broken mod browser scroll bar issue, are we talking about scrolling 'a bit' more, or 'a lot' more? If anything, what 'upsets' me is having to consider the matter; not what consideration to apply.

At this point the discussion is best moved to discord. We need to list the pros and cons and consider implementation details before coming to a decision

Almost like... giving mods tags. XD