TelegramBotClient.SetWebhookAsync not working

When calling SetWebhookASync I don't get any error and still do not recieve any webhook updates...
I read all the guidelines for verified certificates and tested them all (https://core.telegram.org/bots/webhooks) . I know have a valid verfied certificate and still cannot set any web hooks...
Does this works? Someone knows if this is a bug? anyone made webhooks works???
I tried with or without a pem/cer file:

InputFileStream certFile = new InputFileStream(File.OpenRead("c:\\data\\certs\\cert.cer"));

m_client.SetWebhookAsync("https://mydomain:443/telegram/realtimeupdates/", certificate: certFile, allowedUpdates: new UpdateType[0]).Wait();

m_client.SetWebhookAsync("https://mydomain:443/telegram/realtimeupdates/, allowedUpdates: new UpdateType[0]).Wait();

Try making this request in the browser https://api.telegram.org/botYOURTOKEN/getWebhookInfo. What does it return?

thanks @tuscen !
I tried it an got:
{"ok":true,"result":{"url":"https://mydomain:443/telegram/realtimeupdates/Z3_bBAbmEdP3veUDHGv9HSg%3d%3d","has_custom_certificate":false,"pending_update_count":0,"max_connections":40,"allowed_updates":["message","channel_post"]}}
which looks good, but, I still don't get ANY updates to this URL. What am I missing???

You have allowed_updates set to only channel posts. Is that what you wanted?

@tuscen channel post and messages. no? i want to receive user sent messages mainly

I see, I missed the first entry in the array. Are you sure you're sending correct url and path? I couldn't reproduce the problem on my local machine with ngrok. Webhook worked correctly.

@tuscen yes. the url and domain are good and also tested, but i don't seem to get a request from telegram to that url. It's just a regular request right? https/post with application/json content-type right?

Correct, it's always a post request with json payload. What happens when you send some messages and then call getWebhookInfo? Does it return non-zero pending messages count?

@tuscen No, I checked that... it's always 0... really frustrating

That might mean two things: a bug in telegram bot api servers which for some reason not only prevents updates to be sent to your server, but also don't register new updates on their server (which is unlikely, too many problems at once, but who knows), or that messages are sent and your server acknowledges them with 200 OK. Maybe you configured something incorrectly? Could you show me your receiving code and your startup class (I assume you're using ASP.NET Core)? By the way, try testing your webhook locally with ngrok.

@tuscen I'm not using .net core, we're using .NET framework 4+.
And also I can't show you the code because it's complicated and part of a really big platform.
But, I'm testing post messages to the same URL from postman and I see them entering good, but from telegram i can't see them... thanks anyway for your help, is there a way to contact Telegram and ask for support?

In that case check if TLS 1.2 is enabled on your server with appropriate ciphers. Telegram disabled everything older than TLS 1.2.

@tuscen Just checked and it is tls 1.2 with a trusted certificate from telegram's list

@tuscen this is the URL : https://devapi.commbox.xyz/telegram/realtimeupdates/Z3_bBAbmEdP3veUDHGv9HSg%3D%3D
and this is a monitor url:
https://devapi.commbox.xyz/status
if you have any idea i'll be very thankfull

I've checked available ciphers on your domain and the required ones are seems to be enabled. I'm out of ideas what might be wrong.

I dont get updates too, i have following json payload from getWebhookInfo
{
"ok": true,
"result": {
"url": "https://desktop.johmarjac.me:8443/api/update",
"has_custom_certificate": false,
"pending_update_count": 15,
"last_error_date":<PHONE_NUMBER>,
"last_error_message": "Connection timed out",
"max_connections": 40
}
}

I’m closing this issue because it has been inactive for a few months. This probably means that it is not reproducible or it has been fixed in a newer version.
Please reopen if you still encounter this issue with the latest stable version and then please use the issue template.


Fixes 4587: ZeroState fix, Node modules update, affects eslint and fec-config

Summary

Fixes a bug in zero-state not appearing due to a mismatch of react-router-dom version changes.

The front-end now requires docker or podman (and their corresponding desktop applications) to run locally.

Additional information:

While debugging an issue running locally, a bug was found in the frontend-component library that we use here.
Until the above bug is merged/frontend-components can be updated, I'll leave this PR in draft state.

[x] Docker bug merged

Testing steps

Ensure you have podman/podman desktop installed (or orbstack/docker)

Delete current node-modules folder (likely not needed but best practice)

npm install

npm start

Ensure all is working as expected (sanity check)

https://issues.redhat.com/browse/HMS-4587

Hi
tests OK .
Thank you.


ng-required="false" triggers uiDateValidator error

Hi
I'm trying to make an ui-date input conditionally required, buy using ng-required="someVal === true", in conjunction with specify the ui-date options from an object. However, when ng-required="false" a uiDateValidator error is triggered - when it shouldn't be. That field could be left blank.
e.g.
<form name="someForm">
<input type="text" name="date" ng-required="false" ui-date="dateOptions" ng-model="notReqDate">
</form>

where
$scope.dateOptions = {
  dateFormat: 'dd/mm/yy'
};

triggers a someForm.$error.uiDateValidator validation error
Full Example - JS Bin
Is this expected functionality? If so, how do you specify a dateFormat while making a field optional?
Thanks

@theblindfrog thanks for raising this, it wasn't expected functionality, just a case that needed to be tested.


updating reading from json files

added eo:bands to summaries
cleaned up service json file opening

fixed stac to 0.9.0


Required Reason API Privacy manifest declaration/ Privacy-impacting SDK

According to the Apple's documentation, the reason has to be clarified in case of using a required reason API: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api
In addition, FirebaseMessaging is marked as a privacy-impacting SDK by Apple. It must have a privacy manifest + must be signed with a signature.
https://developer.apple.com/support/third-party-SDK-requirements/
Do you have an ETA when an update is available which supports this new requirement?

Ref: https://github.com/firebase/firebase-ios-sdk/issues/11490

Hi @rolfbjarne ,
Now that Apple has set a May 1st deadline on privacy manifests (https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api) - is there plans to update the binding libraries for Firebase etc. before this deadline?

@andrewtechhelp I think this is the plan: https://github.com/dotnet/maui/discussions/20359#discussioncomment-8667387

firebase-ios-sdk has added PrivacyInfo.xcprivacy file to each package.
Is there any plan to update GoogleApisForiOSComponents to reflect PrivacyInfo??


[Feature Request] Add Leetcode Problems in Python

🚀 Feature Request
Is your feature request related to a problem? Please describe.
I can contribute a list of graph, linkedlist and array problems that I currently solve to the repository.
Describe the solution you'd like
I have a list of leetcode problems that I have solved. I can contribute my approach with one test case a driver code and the code complexity
Describe alternatives you've considered
There are not alternatives. I saw that the leetcode currenly has one long readme It would be better for there to be a seperate directory for each language with test cases and driver code and timer as well.
Additional context
None

Would you like to work on this feature?

[ ] Yes

Could you also add the necessary labels as well

Hey @ajay-dhangar this is not completed
Could you let me know why this issue has been closed?


Flutter is not running properly for the new installation or may be after the recent update. ('crumb' is not recognized as an internal or external command, operable program or batch file)

Steps to Reproduce

...
...
...

Logs

Logs

'crumb' is not recognized as an internal or external command,
operable program or batch file.

@sudhanshuy17 Please provide the output of flutter doctor -v.
Also, please check this SO answer if it can help to solve this: https://stackoverflow.com/questions/52411883/crumb-is-not-recognized-as-an-internal-or-external-command-operable-program-o

flutter doctor -v
'crumb' is not recognized as an internal or external command,
operable program or batch file.
[√] Flutter (Channel stable, 3.3.8, on Microsoft Windows [Version 10.0.19044.2311], locale en-US)
• Flutter version 3.3.8 on channel stable at D:\flutter_windows_3.3.8-stable\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 52b3dc25f6 (2 weeks ago), 2022-11-09 12:09:26 +0800
• Engine revision 857bd6b74c
• Dart version 2.18.4
• DevTools version 2.15.0

[!] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
• Android SDK at C:\Users\Sudhanshu\AppData\Local\Android\sdk
• Platform android-32, build-tools 32.0.0
X Android SDK file not found: C:\Users\Sudhanshu\AppData\Local\Android\sdk\build-tools\32.0.0\aapt.
• Try re-installing or updating your Android SDK,
visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.3.6)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.3.32929.385
• Windows 10 SDK version 10.0.19041.0
[√] Android Studio (version 2021.2)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
[√] IntelliJ IDEA Community Edition (version 2021.3)
• IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2021.3
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
[√] VS Code (version 1.73.1)
• VS Code at C:\Users\Sudhanshu\AppData\Local\Programs\Microsoft VS Code
• Flutter extension can be installed from:
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19044.2311]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 107.0.5304.107
• Edge (web)        • edge    • web-javascript • Microsoft Edge 107.0.1418.52
[√] HTTP Host Availability
• All required HTTP hosts are available
! Doctor found issues in 1 category.

@sudhanshuy17 Have you tried solutions in above SO answers? Is there another error message after trying them?

I tried that and so many other things also but noting works


Added & updated some translations

Initial translation for some attributes, changed others. Due to complexity of the Slovenian language, we should probably go thru all of these and make some adjustments with real-world examples. I am setting up a new project right now and will work on forms in about 14 days.

Thank you

You're welcome!


Kanban Board Setup

In this milestone we:

Copied the Kanban board on the Github Project here
There are 2 people in this team @Yacoubou-seidou : Student A  & @MarwanDev : Student B

Hi Team,
You did extremely great on this project! 👏 👏
There are however a few issues that you still need to work on but you are almost there!
🚦 __🏃 🏡 🏃♂ 🏃♀ 🏚 🌴
Highlights 👍
🎲 Created correct number of cards as per the group!
Required Changes ♻

[ ] Cards are not assigned to any member of the group yet. Let's ensure that each card is assigned to a team member accordingly.

Tasks not assigned:

Assigned cards should look like the below, indicated as correct

Cheers and Happy coding!👏 💻 🍷
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.

Hi @MarwanDev and @Yacoubou-seidou,
Approved 🪄
Your project is complete! Congratulations! 🎉
Highlights

Kanban board created ✅
Each task is assigned the correct team member ✅

Optional suggestions
You can also consider the following:
N/A
Cheers and Happy coding!🍻💻
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy, you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.


Update?

When will be the next update?

To clarify, will there be a new release of aq?

Would love to see a new release soon.

Hi guys.
I am so busy, sorry that maintenance of this repository was not adequate. However, development continues, of course.
As you know, Facebook and Tencent released a very strong bot with AlphaGo reimplementation. This method requires a lot of resources, perhaps it will still take some time to catch up even with a smart distributed system like Leela Zero.
AQ is currently being developed with a completely different approach, and it is necessary to review all existing codes. Hopefully, the next version (probably 3.0.0) is going to release around this summer, so please wait for a while.

i love to hear new version of aq will be released this summer.
thank to ymg thanks a lot.

I'm looking forward to see it.
Cheers~

nice jobs in
2018 Tencent World AI Weiqi Competition
and
2018 Chinese weiqi congress

This issue mentions an older version, or hasn't been discussed in a long time, so we close it.


Restart icon is not themeable

Applies To

[x] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Nit: toolbar is contributing an SVG that is not themeable and stands out in themes:

Suggestion to remove this icon path:
https://github.com/microsoft/vscode-jupyter/blob/4a13fca8f5293815e34be98b5f5a0465a007dcdf/package.json#L602-L605
and replace with using native icons:
"icon": "$(restart)"

This also allows product icon themes to customize this icon & color:

Version: 1.94.2
Commit: 384ff7382de624fb94dbaf6da11977bba1ecd427
Date: 2024-10-09T16:08:44.566Z
Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Code/1.94.2 Chrome/124.0.6367.243 Electron/30.5.1 Safari/537.36
VS Code Version
1.94.2
Jupyter Extension Version
v2024.9.1
Jupyter logs
No response
Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
None

potentially related when label strategy is dynamic:


docs:translate service and pv recommendation to en

What type of PR is this?
doc
What this PR does / why we need it:
1 translate service and pv recommendation to en
2 Add pv and service recommendation in how-to-optimize-your-application-resource
3 update doc link

Fixes #
Special notes for your reviewer:

@liu-song thanks for your works.


Python versions

Removal of end of life (or soon to be ) python versions from the tests.yml
Removal of numpy from requirements (installed by eccodes)

Changes now included in thread bug fix branch. Closing.


Nodes are not being imported/created when running trados:import

Hello, I am mantaining a Neos 7 website, which has been heavily customized, and I now need to import some XML files created by the Flownative Trados package. Because of the customizations I am not even sure that this depends on the Flownative package or on Neos itself.
I have been investigating this issue for a few days but I am unable to find a solution.
Example of command that I run with a success response:
php -d memory_limit=-1 flow trados:import --filename "trados_de.xml" --target-language "de_DE"
Some nodes get translated, some others do not.
The ones that get translated are already present in the neos_contentrepository_domain_model_nodedata table, the ones that do not get translated are missing from that table.
If I add the records manually, I can see they get translated correctly.
It appears like the nodes copies do not get inserted in the table when importing the XML files.
I have made several tries, even making smaller XML files to see if there was a problem with the characters or encoding, or maybe with the XML files themselves, but it seems not the case.
Now, of course since the website is a customized version, I have no idea if there are other modifications that could be creating some bugs.
Previously, the Flownative.Neos.Trados package had been modified by other developers. It was giving me some errors when running the import command, so I put the latest available files from this repository.
Is there maybe a way to manually create the missing nodes, or am I missing some steps? Could it depend on configuration files?
This is an example of XML with only 2 nodes, one being imported while the other not:
<?xml version="1.0" encoding="utf-8"?>
<content name="Test" sitePackageKey="Test.Test" workspace="live" sourceLanguage="en_INT">
 <nodes formatVersion="1.0">
  <node identifier="a14fdbc8-202d-4929-b3fd-d42332430cbb" nodeName="node-cblvpg097e5q2">
   <variant nodeType="Test.Test.NodeTypes:Content.Text">
    <properties>
     <text type="string"><![CDATA[This translation is NOT imported]]></text>
    </properties>
   </variant>
  </node>
  <node identifier="10c7c5bc-a87f-4737-b10c-ac22c1b558f3" nodeName="node-xordsf22vyhgj">
   <variant nodeType="Test.Test.NodeTypes:Content.Text">
    <properties>
     <text type="string"><![CDATA[This translation is imported correctly]]></text>
    </properties>
   </variant>
  </node>
 </nodes>
</content>

I will post here the debug I did and the solutions I applied. I hope this can be helpful to myself from the future and to any other developer with similar issues. I had multiple issues with importing the XML files.

I encountered a duplicate entry on unique index error. There were also 2 nodes with same ID (idk how) and I had to remove both of them (not only one) to make it work

The ImportService.php code was customized by a previous (before me) developer and it raised some conflicts with Neos 7.3, or so I believe. To solve this, after many attempts, I used the latest release:
https://github.com/flownative/neos-trados/commit/0122e19a8b21d6427dd36415d44931aea519bfe0

A few nodes in the XML ended up as "not found" giving me a few errors when trying to import the node

I will try to post a PR with the solution that worked for me. On the other hand, it will probably still contain bugs. I believe it could give you some issues if you try to import the same file on the same workspace, it could raise a duplicate entry error.
Following is a more detailed explanation of my debug:
The previous developer added these lines in the ImportService.php file because the Nodes were not being imported as expected. I believe this code was probably written with the help of someone from Neos, but it is just my guess:
/** @var \Neos\ContentRepository\Domain\Model\Node */
$node = $this->contentContext->getNodeByIdentifier($this->currentNodeIdentifier);

$dimensionsForCurrentNodeVariant = [
    $this->languageDimension => [$this->targetLanguage]
];

/** @var \Neos\Neos\Domain\Service\ContentContext */
$targetContentContextForCurrentNodeVariant = $this->contentContextFactory->create([
    'workspaceName' => $this->targetWorkspace->getName(),
    'dimensions' => $dimensionsForCurrentNodeVariant,
    'targetDimensions' => $targetDimensions,
    'invisibleContentShown' => true,
]);

$currentNodeVariant = $node->createVariantForContext($targetContentContext);
$this->persistenceManager->persistAll();

This was added in 2022. I updated the ImportService.php to the latest available in this repository, which does not have those lines. The duplicate key error was gone but the nodes were not being imported yet.
I have now added these lines back into the latest version of ImportService.php and then I also had to add a few more checks in case of not found nodes:
$node = $this->contentContext->getNodeByIdentifier($this->currentNodeIdentifier);
if ($node == null) {
	# A node might have been deleted since the last Trados export
	echo '[Warning] Node ID not found: <' . $this->currentNodeIdentifier . ">\n";
	return;
}

The [Warning] I added will appear in the command line when running the import command. This was useful to me to know what nodes where not being found.
Example of response I got:
[Warning] Node ID not found: <4bdd61f2-9d59-4c80-bde2-17c61f469e38>
[Warning] Node ID not found: <1021eb1e-5a3c-44d4-b9c8-2e0573e2020a>
[Warning] Node ID not found: <7b244060-20bb-492a-a941-6f88ec319866>
[Warning] Node ID not found: <50eb248d-839b-4733-8c4a-2e440e90b56d>

In the end, my guess is that probably, if there is a fallback language and a child language, the ImportService is unable to determine which dimension to assign the translation to.

SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '46b38f926c10e0b961bf8567c0f0729c-de-review-5225d824dc2e2135ad...' for key 'UNIQ_CE6515692DBEC7578D94001992F8FB01'

This is the settings of the index:

I believe this happens only upon node creation, because if I run this:
SELECT * FROM neos_contentrepository_domain_model_nodedata   where pathhash = '46b38f926c10e0b961bf8567c0f0729c' and dimensionshash LIKE '%5225d824dc2e2135ad%';
I get 0 records. So I guess this happens at import runtime.
I believe I might have also run into something similar to this problem:
https://github.com/kdambekalns/neos-development-collection/blob/7.3/Neos.ContentRepository/Classes/Domain/Model/Node.php
Lines 1714-1718:
// ACTUAL BEHAVIOR
// - the pasted node is not shown, but is still in the database.
// - it can happen that the node *is* shown, if it is inserted above the removed node. Still, we have an invariant violation nevertheless.
// - this can also trigger problems when **publishing** the not-rendered-anymore-node (UniqueConstraint errors in the database) - this is
//   how we actually found the error.

Another issue I encountered. The duplicate key refers to a node with this identifier:
98b57826-97e2-4aa5-adf9-35cdbcc156f7
This was used twice (for two different nodes) inside the XML files. No idea why. For some reasons tho, even deleting the duplicate from the XML file still triggered the error. So, I had to delete both nodes with that identifier.

This is the working fix in my case, probably not 100% correct but at least I managed to do what was needed in the best way I could:
if ($currentNodeVariant === null) {
    //return;

    /** @var \Neos\ContentRepository\Domain\Model\Node */
    $node = $this->contentContext->getNodeByIdentifier($this->currentNodeIdentifier);
    if ($node == null) {
        # A node might have been deleted since the last Trados export
        echo '[Warning] Node ID not found: <' . $this->currentNodeIdentifier . ">\n";
        return;
    }

    $dimensionsForCurrentNodeVariant = [
        $this->languageDimension => [$this->targetLanguage]
    ];

    /** @var \Neos\Neos\Domain\Service\ContentContext */
    $targetContentContextForCurrentNodeVariant = $this->contentContextFactory->create([
        'workspaceName' => $this->targetWorkspace->getName(),
        'dimensions' => $dimensionsForCurrentNodeVariant,
        'targetDimensions' => $targetDimensions,
        'invisibleContentShown' => true,
    ]);

    //$currentNodeVariant = $node->createVariantForContext($targetContentContext);
    $currentNodeVariant = $node->createVariantForContext($targetContentContextForCurrentNodeVariant);
    $this->persistenceManager->persistAll();
}

The warning is useful to know what nodes are missing but of course I am not sure it is a good thing to put it there - it was just a quick solution

I have a small update about this issue. I think I found the reason for the duplicate entry violation I was getting.
I am importing an XML file for a new language (pt_BR) using with this command:
php -d memory_limit=-1 flow trados:import --filename "trados_import_ptbr_20240118.xml" --target-language "pt_BR" --workspace "pt-br-review"
The XML file tho had a "misleading" header of this kind:
<content name="XXXX" sitePackageKey="YYYY" workspace="live" sourceLanguage="en_INT" >
It seems that even if I specify the workspace in the command line, the import process will still use the workspace from the XML file. In other words, in my case, it is comparing the nodes with the "live" workspace instead of the new workspace I am creating.
I am not sure why it happens only on some specific nodes (it happened with 2 nodes in particular) but, after specifying the correct workspace in the file also, and not only in the command line, it seems to work well.
So if anyone is encountering this problem, make sure to check the name of the workspace you are using is the same both in the XML file and in the command line. Or, I guess, there is no need to specify it in the command line.
Now, my new XML line is this one:
<content name="XXXX" sitePackageKey="YYYY" workspace="pt-br-review3" sourceLanguage="en_INT" targetLanguage="pt_BR">
Remember to NOT use underscores in the workspace name, as this gave me some problems in the past.


Unconscious Command+Shift+R reloads Hyper breaking all ongoing processes

[X] I am on the latest Hyper.app version
[X] I have searched the issues of this repo and believe that this is not a duplicate

Mac OS X El Capitan:
Hyper.app 2.0.0:
Link of a Gist with the contents of your .hyper.js:
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app:

Issue

window:reloadFull": "command+shift+r" i find myself constantly doing this to reload the browser until unconsciously do it on Hyper and kills all my processes and must restart again.
A popup warning "Do you want to do a full reload" would be good to prevent this.

Maybe better suited as a plugin? 🤔 I would not want a warning every-time to reload, it's really useful when debugging plugins/themes 🤔 Maybe if we could only show a dialog if there's a process running in any of the tabs?

Yeah, same here. Processes keep running and need to be killed, whereas the info in the terminal is gone. This needs improvement. A warning would be sufficient I think. There's still CTRL-R that can be kept without warning as it keeps things running.


webtv.senato.it

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running youtube-dl version 2021.05.16
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that none of provided URLs violate any copyrights
[X] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://webtv.senato.it/Leg18/webtv_aula?seduta_assemblea=21401
Single video: https://webtv.senato.it/Leg18/webtv_aula_hq?seduta_assemblea=21401

Description
The video is segmented and loaded via JavaScript using jwplayer. Look for the script tag with id video_playlist. It contains a line like this: playlist: "https://prd-senato.spectar.tv/canale/assemblea/legislatura/18/seduta/329/formato/flvplayer/quality/hq". There is an XML file containing a playlist with multiple videos, in different formats.

@frafra The playlist seems to be a JWPlayer XML playlist. I'm having issues finding documentation at the moment and youtube-dl sadly lacks a parser as part of its standard library. JWPlayer is currently under a non-commercial Creative Commons license, which clashes with the Unlicense, meaning I can't use their code, and I'm struggling to find when JWPlayer was still under an open-source license. I'll see what I can do tomorrow.

@frafra I've ultimately decided that I'm going to poorly parse the XML myself in order for us to get what we really want: those juicy HLS playlists. Before I go any further, can I have some more example URLs so that I can make sure I parse the URLs correctly?

Sure! Here are some other links:

https://webtv.senato.it/4621?video_evento=209001
https://webtv.camera.it/evento/18413

IIRC ytdl already uses XML (or at least HTML) parsing through stock modules, so you won't need to poorly parse it, and it's hugely discouraged to even try to.

@rautamiekka

IIRC ytdl already uses XML (or at least HTML) parsing through stock modules, so you won't need to poorly parse it, and it's hugely discouraged to even try to.

I know, I was more talking about how I'm going to take a lot of shortcuts in parsing the data structure itself and make a lot of assumptions compared to an official JWPlayer parser.

Is issue still open?


Implementation of a logging function that include file paths an line numbers for easier debugging

fixes #44

This is deprecated for #55


The addition of mobile website viewing capability is required.

It is necessary to fix the error that causes a white stripe to appear on the right side when opening on a mobile phone.

The ability to view the website from a mobile phone has been implemented in this commit. The issue can be closed.


Can we remove usingEvents prop in Day.js?

For example:
if there is no events in 2016 October, the eventIndicatorFiller view won't be there, the height of day is 65, then if we change date to 2016 Nov, let's say there are events in this month, then the eventIndicatorFiller view will be render , the height of day will be 70 (if the eventIndicatorFiller view height is 5), it will cause page flash for row height change.
Can we remove the usingEvents props? instead we can always keep the view there, we can show the indicator based on the events, but i don't want the page shake

@christopherdro

Mind submitting a PR with these changes?

@christopherdro no problem, have done it.  please check out PR

Updated via https://github.com/christopherdro/react-native-calendar/pull/77


Define the lifetime of a blob URL created inside a service worker

Right now in Gecko at least we tie the lifetime of such URLs to the Worker or SharedWorker that created them, but doing the same (as in, invalidating the blob URL when the service worker goes away) seems wrong, since that can happen at any time at UA's discretion.
How should we handle this?
One option that comes to my mind right now is keeping those URLs valid until the service worker that registered them is unregistered.

I think Gecko's current behaviour is correct. Yes, it may get GC'd when the SW closes, but same with variables. If you want a long term way to store a response, the caches api is the way to go right?
Tieing it to registration feels wrong as it would need to survive browser shut down.
What is the current behaviour breaking? I'm wondering if a way to access items in the cache api by url may be a better solution.

We should maybe restrict this API inside service workers since it's a giant memory leak. And only allow the one-time-minting variant.

Is it a giant memory leak if the data is GC'd when the SW terminates?

Depends on whether service workers will forever terminate quickly. It seems better to simply not offer APIs that are known to be problematic.

@annevk I really don't think this is a problem any more than it is on pages. Service workers are generally not alive longer than pages.

Right, that argument was made, and the argument against was that it might not remain that way. I guess the other question is why we need blob URLs in service workers since we can just use Request/Response objects.

Need a resolution. Would like to discuss at the f2f.

Implementor feedback from Chrome: if we continue to allow blob URLs in service workers, tieing the lifetime to the service worker (i.e., the execution context that created it, not the registration) sounds good. That's what Chrome does already.

F2F resolution: remove createObjectURL

@annevk Is it better to make URL.createObjectURL(blob), URL.createFor(blob) and URL.revokeObjectURL(url) explicitly throw in File API spec?

@jungkees no, I would get the File API specification to use [Exposed=] similar to how XMLHttpRequest does that for e.g., responseXML, to hide these from service workers.

Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1264182

Closing with https://github.com/w3c/FileAPI/pull/31 merged.

Blink tracking bug: https://bugs.chromium.org/p/chromium/issues/detail?id=604951
I filed https://github.com/w3c/media-source/issues/67 since MediaSource has createObjectURL as well.
MediaStream had it as well at some point but it's been removed from the spec.


Tabs in Gift Card admin are non functional

Describe the bug
When clicking on "Redeemed" or "Pending", nothing is happening.
Screenshots

Works as expected on local/staging. May be an issue specific to opencollectiveinc

Will check

Current status: I tried to reproduce on Ubuntu 18.04 for opencollectiveinc with both Firefox 66.0.1 and Chromium 73.0.3683.75, using both François and Pia's accounts and wan't able to reproduce it.

Can't reproduce anymore. Closing.


made SelectionArea alignment consistent between web and other platform

Currently, when text is placed inside a SelectionArea widget that's nested within a Column widget, it results in misalignment, causing the text to appear centered instead of aligned as intended.
This was originally #149552 but had issues with my branch.
List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.
Fix intended for #148934
Also fixes #149426
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@Renzo-Olivares thank you for the review in #149552, i made all changes you suggested.

Can you also have a look at #121053, please? I think this would likely fix it too.

@kaboc thanks, i tried your sample code and indeed, this seems to be fixing your issue too.

Great! Thank you, too.

@LimaneGaya FYI, a word like "Fixes" is necessary in front of each issue number to auto-close those issues when the changes in this PR is merged. The linked issues should be displayed in the "Development" section in the right column of this page.

Since there is no particular reason for alignment should it just be removed along with fit?


symlink error check

On Windows the docker-compose build would fail and try to run sudo to create the link when it already exists on a windows machine.

Thank you @Kaffiend ! Is this good to go at this point or does anything else need to be done?

No problem @NathanWalker Yessir, It fixed the docker build on both my windows machines, and my Linux box appears to be unaffected as well.

I'm still concerned as to why this section of install.js did not catch on the process platform and re-symlink with the correct OS command.
when i run node -p process.plaform i get the expected 'win32' return. I also tried adding the dedicated osmodule and usingos.platform()which returns the same thing and it still tries to usesudo` perhaps someone more familiar can spot why its doing that. That is what the underlying cause is. What i added I would only consider a work around TBH.

Configuring...
Attempting to Symlink ../src/client/app ../nativescript/src/app/
Symlink error:  { Error: EEXIST: file already exists, symlink '/home/app/angular-seed/src/client/app' -> '/home/app/angular-seed/nativescript/src/app'
    at Error (native)
    at Object.fs.symlinkSync (fs.js:1054:18)
    at createSymLink (/home/app/angular-seed/tools/install.js:110:8)
    at Object.<anonymous> (/home/app/angular-seed/tools/install.js:41:5)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.runMain (module.js:590:10)
  errno: -17,
  code: 'EEXIST',
  syscall: 'symlink',
  path: '/home/app/angular-seed/src/client/app',
  dest: '/home/app/angular-seed/nativescript/src/app' }
linux  <----process.platform logged out with error

Ok so this is from within the docker container which is linux. This tells me that the issue is sudo is not included with the container distro. And what i added fixed my issues because the link is already there and we were only looking for any error in general to recreate the link. So if we want to always recreate the link we need to make sure sudo is included in the distro the container is running.

added a message to appear after the "Attempting to create a symlink.." message, if the link already exists. This is as far as i go on this issue though. adding sudo to the container distro is more than I can get into right now, I'm not entirely sure its even needed. But this PR is good to go for fixing the docker-compose build

Excellent thank you 👍


Tooling BOSH UAA dependency causes random pipeline failures

Tooling and Primary BOSH use UAA for authentication, which will randomly fail due to UAA authentication failures (despite using the same credentials).
Staging and Production do not leverage UAA and are not experiencing this issue.

This is likely because we are not authenticating correctly in our pipeline tasks with the bosh v2 cli.
If you are using the bosh v2 cli, and UAA you MUST authenticate with a client credentials not user / pass as we currently do.

OK, let's switch to client credentials like you said--but I still don't understand why auth works most of the time and fails apparently at random.

Resolved by authenticating to opslogin with client credentials, h/t @cnelson!


update deployment: document Crusher deployment

Update deployment documentation to include Crusher deployments of E4S 22.05.
Fixes https://github.com/E4S-Project/e4s/issues/82

Looks good to me. I suggest we consolidate the Paratools support disclaimer with a separate PR.

I just removed the duplication of this here :)


Support Save and Retrieve Memory from Redis

Right now to create an agent that has memory, we do
memory = ConversationBufferMemory(memory_key=str('chat_history'))
Then pass it to the agent
agent_executor = AgentExecutor.from_agent_and_tools(agent=agent, tools=tools, verbose=True, memory=memory)
However, the memory will be gone if the server restarts. I learned that memory can be dumped as json, but to avoid losing messages, we have to dump to json every time a message comes in, which is not a good solution.
Can we add support for storing conversation buffer memory in redis and when constructing the agent pass the redis instance to the agent executor?
Let me know if there is an existing solution for this. Appreciate the help!

This has already been added in v0.0.126, see https://github.com/hwchase17/langchain/pull/2122

Thanks @zoltan-fedor


Support Linode v4 API

Linode has publicly released a preview of their new, RESTful API: announcement, docs. There hasn't been any deprecation timeline announced for the previous API, so things this isn't urgent.

https://github.com/linode/python-linode-api/issues/79


[Bug]: Torus Transaction Error - gasPrice but also included maxFeePerGas, these cannot be mixed

Current Behavior
When using the latest version of Onboard 1.37.1, using Torus, and connecting to Polygon chain 137 and try to perform a transaction, I get the below error:
Uncaught (in promise) Error: Invalid transaction params: specified gasPrice but also included maxFeePerGas, these cannot be mixed
    at getEthJsonRpcError (chunk-vendors.d7195693.js:64:300066)

I've also filed a bug with Torus if the issue is on their side: https://github.com/torusresearch/torus-embed/issues/410
Expected Behavior
no error should occur
Steps To Reproduce

Select Torus from Onboard wallet selection and use a Polygon chain (137)
Perform any transaction that requires gas
see browser console log for error

Onboard Version
1.37.1
Node Version
v14.18.3
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Anything else?
No response

@jadbox Is it safe to say that this is a torus issue and can be closed?

I believe so. Closing


merge package template from Kronos-Tools/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #221   +/-   ##
=======================================
  Coverage   97.54%   97.54%
=======================================
  Files           1        1
  Lines         122      122
=======================================
  Hits          119      119
  Misses          3        3

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 4e8b5cc...37ff9e7. Read the comment docs.


Own Pathfinder setup does not show Location of my character ingame

Have the same version running, but on the official site it works.
any ideas?

i have quite some of these in my esi.error.client.log:
Thu, 12 Oct 2017 23:44:43 +0000 [<IP_ADDRESS>] HTTP 404: 'Not Found' | url: POST 'https://esi.tech.ccp.is/v2/universe/names/?datasource=tranquility' | body: stdClass Object
(
[error] => Ensure all IDs are valid before resolving.
)

Sometimes I see myself online on the map for a few seconds ... then offline again.
Already changed API Key, truncated DB etc ... really have no idea what could be wrong

is this maybe related to:
https://github.com/ccpgames/esi-issues/issues/600
https://forums.eveonline.com/t/api-erorr-ensure-all-ids-are-valid-before-resolving/31799/2
but then i wonder why only my installation has the problems?

Was exactly the problem i had. Works again :)


Modified Technique and Sub Technique Module

Mantis #0770019 - Update the fields that need to be displayed in Alert SVT to 'default grid column'

Issue: In Alert SVT, the correlated Techniques or Subtechniques are not displaying any column
Resolution:
Resolution.docx
Please refer to attached

Looks good to me. Thanks Rohit.

I see field names change. Any PB impact due to this?

I see field names has changed. Any PB impact due to this?

Yes, The name impacts playbook that links Technique to Alert record. However, Under Mantis #0773714 we will removing these playbooks. The relavant changes has been made in PBs and will be uploaded in MITRE ATT&CK Threat Hunting - 1.0.0 SP

What all unit testing was carried out to test this change
Please refer  Mantis ID - 0770019 Unit Test Case Document.docx


Possibility to copy video format in conversion process?

I would like to convert the audio format from any -> aac.
I don't really need to convert the video since I don't care about what format it is.
Is there any possibility to simply copy the video, and only convert the audio track in one video file?

Please review the readme. This is the primary function of the script so yes it can do this


Introduction of nullable type attribute

this resolves #26 and is consistent with the suggested solution in this comment

introducing the nullable type attribute
adding an example of how to use nullable
adding a rendering folder with an example of how nullable should be used within tools

This is a duplicate of #37 which was destroyed by a bad rebased. The code is the same.

Just one thing to fix and this is ready to go.

Good catch removing now.

PR accepted. Will be merged when nullable support is implemented in snowcrash and drafter.

@pksunkara Would you like me to create a ticket for snowcrash and drafter to implement support for this and possibly take a swing and a pull request?

@obihann Definitely. All contributions are welcome. :smile:

K I'll get started on that probably in the next day or two, it would be awesome if we can get a solution hammered out rather quickly so I can make use of this new attribute in my current documentation.

Hey @obihann just out of the curiosity are you primarily using Apiary.io or OSS tools (protagonist / drafter / aglio)? The reason I am asking is that the propagation of such a change to Apiary.io will take a little bit longer...

@zdne  OSS tools, specifically drafter and dredd. I don't have any plans to move to Apiary.io in the near future.

@pksunkara now that drafter and snowcrash have been updated we should be safe to merge this.

This is not working yet. So, we need to fix apiaryio/drafter#121 first before merging this. If you want to use nullable you can start using drafter master branch.

I can look into apiaryio/drafter#121 if you like to speed this along, but your right since snowcrash and drafter are now using nullable this is less of an issue. I do have to look into dredd now so I may make a new issue on that project to support nullable shortly.

With apiaryio/drafter#121 now closed is it possible for this to be merged in?

Once the master branch is released, I will merge this in.


Tools to profile performance / benchmark

yo
We need a benchmark suite for python-prompt-toolkit and downstream users.  Making a benchmark suite may be a lot of work, so far I haven't found a coveralls-type service that can give us that kind of data.
Once we're at the pyvim level it gets kind of tricky to figure where the bottlenecks come from.
In lieu of having a benchmark suit, for live profiling, I've jotted down some notes that gets you surprisingly deep live trace of calls. https://github.com/jonathanslenders/python-prompt-toolkit/wiki/Profiling

It'd be cool to see if we could take some of this and use it to our advantage.  Also maybe we can port them from urwid to ppt.
Taking a deeper look into it.

Thanks. Actually, I've been profiling a lot during the last year. The built-in profile module from Python and runsnake have helped me a lot. I'll have a look at this tool later on, but I think that most of the bottlenecks are already as optimized as possible.


Feat add ramsete controller to drive train

this is a little explanation of what is a Ramsete controller and how is used
https://docs.wpilib.org/en/stable/docs/software/advanced-controls/trajectories/ramsete.html
this is a example to how we can use the Ramsete controller in the code in GitHub
https://github.com/wpilibsuite/allwpilib/blob/main/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/RamseteCommand.java#L154
when this is done we will be able to control our drive train using  a Ramset controller that accepts a path as a argument

its a example for a team
https://github.com/team8/FRC-2022-Public/blob/main/src/main/java/com/palyrobotics/frc2022/subsystems/controllers/drive/RamseteDriveController.java#L17

All variables are set up, we are just trying to get it to run now. Next step: Update motor safety.

motor saftey done. make sure to add ALL values for ramsete controllers :>


add docstring for learning

Basically, Core Learning Objects are pretty much ready, just want to eliminate kwargs wherever possible.
The Learning Utilities are still missing several docstrings on initilizations for classes and some function parameters.

Fixed! Merge to docstrings branch if LGTY. Thanks.


docker run success，but can't connect web ui

root@unraid:/mnt/user/appdata/hajimari# docker run -d --name=hajimari -v /mnt/user/appdata/hajimari:/config -p 12345:3000 ghcr.io/toboshii/hajimari:v0.2.0
c7a1926825a451ed19dd08d9f21dae43754ae8e745b097f0a27047687e69a19d
root@unraid:/mnt/user/appdata/hajimari# docker logs c7a1926825a4
time="2022-08-29T07:59:07Z" level=info msg="Listening on :3000\n" _source="/build/main.go:69"
time="2022-08-29T07:59:14Z" level=fatal msg="Cannot get kubernetes config: stat /home/hajimari/.kube/config: no such file or directory" _source="kube/client.go:56"

Hi @xinmans, thanks for your issue.
At this point in time Hajimari is Kubernetes specific and will not run without a valid kubeconfig. If your intention is to run it locally in docker but connected to a remote k8s cluster (for development for example) you'll need to mount a valid kubeconfig at /home/hajimari/.kube/config.
If you are instead looking for something that can run in docker alone, I'm afraid you'll have to look at some other options like flame, heimdall, or dashy.


Coercion

Are there any plans on the horizon to "coerce" documents into a valid format?
It would be nice if {"should_be_number": "82"} actually validated against a schema with {"type": "number"}.  It would be even better if I could get my hands on a document that looks like {"should_be_a_number":  82}.
Would such a patch be welcome? Or is it against the philosophy of this library?

Coercion is a thing that https://github.com/Julian/Seep is designed for (it lacks someone who wants to drive it though at the moment). A patch there would certainly be welcome.

A patch here or a patch there?
On Fri, Jan 30, 2015 at 12:19 AM, Julian Berman<EMAIL_ADDRESS>wrote:

Coercion is a thing that https://github.com/Julian/Seep is designed for
(it lacks someone who wants to drive it though at the moment). A patch
there would certainly be welcome.
—
Reply to this email directly or view it on GitHub
https://github.com/Julian/jsonschema/issues/195#issuecomment-72053519.


Update embedded-hal-async to v1.0.0-rc.1

@Dirbaio : does this mean I need to update the embedded-hal-async version in lora-phy version 1 and 2?

yes, there's a new version out today :)
This PR is for lora-phy v1, you probably want to do it on v2 too.


Use distro repos in cloudapi

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

I'm wondering if it would be worth to extend some of the existing functional test cases to also cover this feature, or at least adding a new unit test for it.

Added a unit test to check it. I had to hard-code a path traversal so I could load the repos, it seems stable, but let me know if there's a better way.


hField not updated on data change

Hi,
I'm currently working on a school project where I need a heightfield, that changes dynamically during runtime. The end goal is to model a type of changeable terrain.
I'm currently stuck on how exactly to effectively change the heightfield data while running the viewer. In the code below I would like to set half of the heightfield to height 2, as soon as the user presses the 'h' key. While the data does change, I was able to confirm this while debugging, the rendered model does not and both the regular visualization and wireframe stay the same.
Here is the full code I have at the moment. Any input is appreciated!
import mujoco
import mujoco.viewer as v
import time
import keyboard
import numpy as np

xml = """
<mujoco>
    <asset>
        <hfield name="terrain"  nrow="16" ncol="16" size="1 1 1 1"/>
    </asset>
  <worldbody>
    <light name="top" pos="0 0 1"/>
      <geom name="red_sphere"  size=".2 .2 .2" rgba="1 0 0 1" pos="0 0 2"/>
        <geom name="heightfield" type="hfield" hfield="terrain"
     />
  </worldbody>
</mujoco>
"""

model = mujoco.MjModel.from_xml_string(xml)

data = mujoco.MjData(model)

with v.launch_passive(model, data) as v:
  start = time.time()

  while v.is_running() and time.time() - start < 30:

    if keyboard.read_key() == "h":
      num_rows = model.hfield_nrow[0]
      num_cols = model.hfield_ncol[0]
      new_hfield = np.ones((num_rows * num_cols))
      new_hfield[:128] = 2.
      model.hfield_data = new_hfield

    step_start = time.time()
    mujoco.mj_step(model, data)
    v.sync()

    time_until_next_step = model.opt.timestep - (time.time() - step_start)
    if time_until_next_step > 0:
      time.sleep(time_until_next_step)

You need to call mjr_uploadHField each time the height field is modified in order for it to be picked up by the renderer. Note that this can be expensive since it's a GPU data movement.


Spark 2.3 is now out... Will Dr.elephant support it out of the box?

https://spark.apache.org/releases/spark-release-2-3-0.html
So ive been reading about how linkedin has been using a modified SHS to pull from 1.X 2.X...
But if you were to install Spark 2.3, will DR.elephant just work with it out of the box?
Has anyone tried Spark 2.3 and DR elephant yet?

I think Spark 2.3 is a requirement to analyze Spark 2.x jobs, as per https://github.com/linkedin/dr-elephant/issues/327 .. or at least you would need to build a custom SHS as it was a prerequisite - see https://issues.apache.org/jira/browse/SPARK-18085 for details.

SPARK-18085 brings the LevelDB storage for Spark History Server(SHS), this would help Dr. Elephant to gather metrics as it improves SHS overall performance. If you have some former versions of SHS which kept all data in-memory, Dr. Elephant can still gather metrics from the former version of SHS, as the Rest Fetcher in Dr. Elephant calls the same RestAPIs. If you don't have a large amount of applications per day, former version SHS might still work for you.
But since we added new metrics in Spark(code change in Executor, Driver, SHS as described in ticket SPARK-23206), you might not be able to get some new metrics that we added. The reason that you need a custom SHS is our PR for these new metrics is not yet getting merged. We are targeting Spark 2.3.1 and Spark 2.4. If you have your own SHS without all these, Dr. Elephant will not be able to gather some new metrics.
Internally we are using Spark 2.3 Spark History Server with the above PR applied. There are some other patches for some SHS issues, but those will not be blockers for using Dr. Elephant.

Thank you @zhouyejoe for this information - really helpful as we will be checking out Dr. Elephant soon too.

@zhouyejoe  does your team/Linkedin have any plans to open source the Spark 2.3/4 patches you mentioned above?

@zhouyejoe does your team/Linkedin have any plans to open source the Spark 2.3/4 patches you mentioned above?

Those PRs has been merged into Spark trunk, please take a look at the comments in https://issues.apache.org/jira/browse/SPARK-23206 for more details.


Increases packages presubmit resources

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


[question] Re-Interview controller node results in node to become Unknown

I'm setting up a new z-wave network, and I'm trying to do it right (or as right as possible) this time.
I bought a Silicon Labs SLUSB7000A UZB-7, and after the initial Firmware issues, I got the controller working correctly.
When I re-interview the controller node, It becomes "unknown".
Controller node debug info correct:
{
  "id": 1,
  "deviceId": "0-4-4",
  "manufacturer": "Silicon Labs",
  "manufacturerId": 0,
  "productType": 4,
  "productId": 4,
  "name": "silabs_zst10_700_00",
  "loc": "",
  "values": [],
  "groups": [],
  "neighbors": [
    2,
    3
  ],
  "ready": true,
  "available": true,
  "hassDevices": {},
  "failed": false,
  "lastActive":<PHONE_NUMBER>449,
  "supportsBeaming": true,
  "supportsSecurity": false,
  "isSecure": false,
  "keepAwake": false,
  "maxBaudRate": null,
  "isRouting": true,
  "isFrequentListening": false,
  "isListening": true,
  "inited": true,
  "hexId": "0x0000-0x0004-0x0004",
  "dbLink": "https://devices.zwave-js.io/?jumpTo=0x0000:0x0004:0x0004:0.0",
  "productLabel": "ZST10-700",
  "productDescription": "700 Series-based Controller",
  "protocolVersion": 3,
  "nodeType": 0,
  "endpointsCount": 0,
  "isControllerNode": true,
  "dataRate": 100000,
  "deviceClass": {
    "basic": 2,
    "generic": 2,
    "specific": 1
  },
  "status": "Alive",
  "interviewStage": "Complete",
  "_name": "silabs_zst10_700_00"
}

Controller node debug info after re-interview:
{
  "id": 1,
  "deviceId": "NaN-NaN-NaN",
  "manufacturer": "Unknown manufacturer 0xXXXX",
  "name": "silabs_zst10_700_00",
  "loc": "",
  "values": [],
  "groups": [],
  "neighbors": [
    2,
    3
  ],
  "ready": true,
  "available": true,
  "hassDevices": {},
  "failed": false,
  "lastActive":<PHONE_NUMBER>402,
  "supportsBeaming": true,
  "supportsSecurity": false,
  "isSecure": false,
  "keepAwake": false,
  "maxBaudRate": null,
  "isRouting": true,
  "isFrequentListening": false,
  "isListening": true,
  "inited": true,
  "hexId": "0xXXXX-0xXXXX-0xXXXX",
  "dbLink": "https://devices.zwave-js.io/?jumpTo=0xXXXX:0xXXXX:0xXXXX:0.0",
  "productLabel": "Unknown product 0xXXXX",
  "productDescription": "0xXXXX",
  "protocolVersion": 3,
  "nodeType": 0,
  "endpointsCount": 0,
  "isControllerNode": true,
  "dataRate": 100000,
  "deviceClass": {
    "basic": 2,
    "generic": 2,
    "specific": 1
  },
  "status": "Alive",
  "interviewStage": "Complete",
  "_name": "silabs_zst10_700_00"
}

Here's the logs I captured:
zwavejs_1.log.gz
The only way to go back to a 'sane' situation is to stop the container, plug out the USB stick, wait for a moment, plug it back in and start the container.

What version of zwave-js are you running?

The manufacturer info for the stick is queried differently, which is not repeated with a normal "interview".
Basically, re-interviewing a controller should neither be necessary nor possible.

No, it's okay - we've got this one.


chore(ui): Simplified object viewer

Description

Simplified object viewer buttons to use one button for collapse / expand.
Show / hide functionality for clicking the header to hide/show the object viewer.
Adjusted the sizing of the header for the object viewer.
Adjusted padding for the object viewer in the calls panel.

Screenshot

Before
After

Updated preview:
https://beta.wandb.ai/?betaVersion=2d72424eebbc1a0442a57445f93b361949321550

I think this makes sense, I like the icon change. One micro tioli style suggestion.
Note: This will change our users experience of the object flow, they might go looking for this button we have removed.
Also, not your problem, but the object viewer not auto resizing after collapse is annoying !

For sure - I thought it was an acceptable level of moving the cheese since we use the title collapse elsewhere for the entire viewer, and the collapse/expand buttons being separate buttons when they're really a toggle was kind of an anti-pattern and was cluttering it up.
Yeah I saw the height re-size as well but I thought it may be better for a separate PR cause I know there was some stuff around setting the explicit height and virtualization to take into account.


docker DB image not used

fresh git clone AGiXT
run ./AGiXT.sh
respond Y to "Do you want to use postgres?"
Visit our documentation at https://AGiXT.com
Welcome to the AGiXT Environment Setup!
Do you want AGiXT to automatically update when launched? AGiXT Hubs always update automatically. (Y for yes, N for No): N
Do you want to set an API key for AGiXT? (Y for yes, N for No): n
Do you have your own AGiXT Hub fork that you would like to install with? (Y for yes, N for No): n
Enter the number of AGiXT workers to run with, default is 4: 10
Do you intend to run local models? (Y for yes, N for No): y
Do you want to use a local text generation web UI? (Only works with NVIDIA currently) (Y for yes, N for No): y
Enter your CUDA version, you can find it here: https://developer.nvidia.com/cuda-gpus (Example: RTX3000-5000 series are version 7.5): 8.6
Do you want to use postgres? (Y for yes, N for No and to use file structure instead): y
Do you want to use an existing postgres database? (Y for yes, N for No and to create a new one automatically): n
Enter postgres password:

After this, check .env file and it's like this:
TORCH_CUDA_ARCH_LIST=8.6
CLI_ARGS='--listen --api --chat'
DB_CONNECTED=false
AGIXT_AUTO_UPDATE=false
AGIXT_HUB=AGiXT/light-hub
AGIXT_URI=http://localhost:7437
AGIXT_API_KEY=
UVICORN_WORKERS=10
GITHUB_USER=
GITHUB_TOKEN=
POSTGRES_SERVER=db
POSTGRES_PORT=5432
POSTGRES_DB=postgres
POSTGRES_USER=postgres
POSTGRES_PASSWORD=xxx

In AGiXT.sh I see DB_CONNECTED is tested for "true" on line 224:
  if [[ "$DB_CONNECTED" == "true" ]]; then
    docker-compose -f docker-compose-postgres-local-nvidia.yml up

Beeing false in .env file it doesn't go for the correct .yml file.

This appears resolved now.  Thank you!


add .hh .cc alternate

I am not planning to add default mappings for every possible naming convention that is out there, please configure this in your own configuration.


Build Quiz endpoint

Create API endpoint that lists all of the questions and their answer for a given quiz.
/quiz/:slug

This is done


Links to social are now clickable outside icon and text too

Bruh, Please submit more substantial pr's!


A new time entry is created when clicking "Continue"

This bug essentially makes the app useless, as I'll have to add up time entries for all the times I've started/stopped or toggled between tasks.

Hi there,
The reason the app started behaving this way is because we removed the "Save duration only" mode from Toggl. The mode was replaced with time entry grouping. You can read more about the feature here
https://support.toggl.com/grouped-mode/
You can turn this option on from your Profile (https://toggl.com/app/profile). Just tick the "Group similar time entries" option.
The grouped mode make your Timer screen look similar to the way it looked with the old "Save duration only" mode.

@vedr4n I'm aware of the new grouping feature, and it works fine in the browser view (it was already checked in my profile settings). However, it doesn't work in the desktop app, which I almost only use. I've checked the version (screenshot above) and tried closing the app, logging out, everything I could think of, but no grouping...

@raahede Try unsetting the grouping option inside the profile and save the changes. Then re-enable the option and save the changes again.
You might need to restart the desktop app after that but the grouping should start working.

Ah, yes. When in doubt, restart :)
Atm we're not sure why some people need to reset the option in order to get grouping to work. Good news is that most desktop users aren't affected.

👍

I was affected on 7.4.38 (New x64 install on Win 10). Reset profile> group setting worked. What kind of survey was made to determine which users affected? Maybe users who were affected gave up and uninstalled. That's what I did -- couldn't deal with 50 individual time entries per day. Luckily found the github project and decided to search issues and found nothing; luckily decided to search closed issues. maybe some|most users figure it out and don't say anything. anyway, cool timer.


feature request: Support variable

When testing prompts, we often need to evaluate how the same prompt template performs with different input values. Currently, this requires creating multiple separate test cases or manually modifying the input values, which is time-consuming and makes it difficult to maintain consistency in testing.
If support one variable with multiple values, (switch on at one time), it would be better.

btw, thanks for working on this project, it's very useful.

@daimajia @daimajia Hi! Thank you for the feature request.
I think what you're looking for might be similar to dataset functionality, where you can test the same prompt template with multiple input values. You can see an example of how this works in this video: https://www.youtube.com/watch?v=5h53mhWFhwU
However, to make sure I fully understand your specific needs:

Are you looking for a way to define a single prompt template and then test it with multiple variable values in one go?
Would you want to see all results simultaneously, or test them one at a time?

If you could describe your ideal workflow, it would help us better understand your requirements and ensure we implement the feature in a way that best serves your needs.


Remove CatalogSourceLister usage from resolver/cache.

The cache hardcodes the use of a CatalogSourceLister (imported from a
non-resolver package in the OLM module) in order to assign source
priorities based on the spec.priority field of CatalogSource. Not all
cache sources map to a CatalogSource, so this can instead accept a
priority provider interface and the resolver -> non-resolver imports
can be removed.

Ah, it looks like approval-acts-as-lgtm doesn't work on this repository.
/lgtm

/hold cancel

/lgtm


Set visual 2D bounds from code

What

Closes #683

Usage:
rrb.Spatial2DView(visual_bounds=rrb.VisualBounds(min=[0.0, 0.0], max=[10.0, 20.0]))

WIP State
Everything works, except for logging bounds from Python (like the snippet above). It fails with some deserialization error.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[ ] I've included a screenshot or gif (if applicable)
[ ] I have tested the web demo (if applicable):

Using examples from latest main build: rerun.io/viewer
Using full set of examples from nightly build: rerun.io/viewer

[ ] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[ ] If applicable, add a new check to the release checklist!

PR Build Summary
Recent benchmark results
Wasm size tracking

To run all checks from main, comment on the PR with @rerun-bot full-check.

$ pixi run py-build
$ _RERUN_TEST_FORCE_SAVE=foo.rrd pixi run python docs/snippets/all/annotation_context_rects.py
$ pixi run rerun print foo.rrd
…


Implement vocabulary value selectors in item’s metadata edit form

References

Fixes #1758

Description
Work in progress
This PR use the existent DsDynamicOneboxComponent and DsDynamicScrollableDropdownComponent components to be able to edit metadata controlled by vocabularies in item's metadata edit form in the same way that is done in submission form.
Instructions for Reviewers
TODO: Please add a more detailed description of the changes made by your PR. At a minimum, providing a bulleted list of changes in your PR is helpful to reviewers.
List of changes in this PR:

Use components DsDynamicOneboxComponent and DsDynamicScrollableDropdownComponent in item's metadata edit form for fields controlled by vocabularies:

DsDynamicOneboxComponent: is used for vocabularies that use a autocomplete functionality and for hierarchical vocabularies
DsDynamicScrollableDropdownComponent: is used for vocabularies configured with value-pairs element in input-forms.xml (dc.type, dc.language.iso, etc.)

Add a function to the Vocabulary data service to retrieve whether a metadata uses controlled vocabularies
Add a field to edit the authority key in a similar way to previous versions
Modify the AuthorityConfidenceStateDirective
...

Checklist

[ ] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[ ] My PR passes ESLint validation using yarn lint
[ ] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[ ] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[ ] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[ ] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[ ] If my PR fixes an issue ticket, I've linked them together.

@toniprieto this is very cool. I tested your changes on DSpace 7.6.1 and it Just Works™. The technique of using the mechanism as in the submission form is clever because it doesn't require any extra configuration. We have about ten fields that use controlled vocabularies. I tested editing and saving on several fields that have a mix of value-pairs and external XML controlled vocabularies. Works great!

Just want to add my feedback, not a full review, but I've tried out this PR and it is working great with some new external data lookups (via choiceauthority) and traditional CV in various presentations

Testing this again via a squashed version of this commit (I think the final pull request should be the finished version of this, without the intermediate work, and without the whitespace additions).
Still working well on DSpace 7.6.2-SNAPSHOT.

I think this is ready for review. I have squashed/reorganized the commits and updated the PR description with more information.

I rebased this patch on DSpace 7.6.1 and it works well. I don't have any more feedback on the functionality, so I will let someone with Angular experience comment on the implementation.
@tdonohue since this technically restores functionality that was in DSpace 6 I think we should consider porting this to dspace-7_x. On that note, there might be some documentation updates needed.

@alanorth : Per a decision by Steering, DSpace 7.6.x will have no new features added in future releases (as it is in maintenance mode).  So, this new feature will only be possible to add to 8.x.

@tdonohue Thanks for the detailed feedback!
Yes, I also think they are bugs, I will try to fix them these days. Regarding the first one, although it is done on purpose, I now think it is better if it is also implemented for the new fields. I'll work on that.
I think some of these bugs are backend issues and can possibly be reproduced in the current submission form as well. For example, the SolrAuthority that is used for the ORCID authority should not index/return virtual authorities, or the issue 4 which is partly due to a poor implementation of the method bestMatch() of the Sherpa Romeo authorities. I will look at it in more detail and make new PRs for the backend if I confirm that it is the problem.

@toniprieto : Just wanted to make sure you are aware that our 8.0 feature PR merger deadline is coming up next week on Friday, Feb 23.  I'd love to see this PR included in 8.0, if you can find time to get those bugs fixed.  (Extensions are possible if the PR is really close, but ideally we'd get this re-reviewed and merged in the next week.)

@tdonohue I have updated the PR, the 4 issues should be addressed:

The vocabulary selector is now implemented when a new value is added.
This is a issue related with the behavior of the ORCID SolrAuthority. With this kind of authorities, when a new value is added the consumer authority automatically assign a new authority key with confidence equals to 0 (NOVALUE). I suppose that you have this consumer configured in:

event.dispatcher.default.consumers = versioning, discovery, eperson, submissionconfig, qaeventsdelete, authority

I have added a change to hide the authority key when the confidence value is equals to 0 (NOVALUE) or -1 (UNSET) to address this issue.
This authority key is displayed in the input field if you edit this field:

Additional note: To make the value of the confidence field more apparent while editing, I have made a change to the Confidence directive to allow its use in a different "mode" that uses fontawesome icons, which should be more identifiable. I have tried to limit this change to a single commit in case it turns out to be odd and needs to be reverted (https://github.com/DSpace/dspace-angular/pull/2653/commits/5d2977a16d151d45b405930e1552c4833c69cd24)

I think that the main problem of this issue is that virtual metadata should not be indexed in Solr Authority Core. I have submitted a Backend PR to avoid this: DSpace/DSpace#9326
And this issue is related with a simple implementation of the getBestMatch() of Sherpa Romeo Authorities. It should be solved by this backend PR: DSpace/DSpace#9327

@tdonohue Additional note, I'm investigating another issue that can be reproduced with:

Add a new field and select the dc.publisher
Enter a word to get suggestions. For example "Mechanics"
Wait for the suggestions to appear, and do not select any
With the suggestions popup open, click on the "Confirm button"
The field is added empty (But ideally, I should have added "Mechanics")

I think it is a bug in the DsDynamicOneboxComponent component that can also be reproduced in the submission form. I am trying to fix it."

Hi @tdonohue,
The first issue should be resolved.
About the second, I have done a change to prevent the green circle from being displayed when the current confidence value is not ACCEPTED. The behavior has improved, although it is not perfect. The authority key still appears, but it is because the SolrAuthority plugin used for the dc.contributor.author field always adds them when saving a value without authority, although with an invalid confidence.
An e2e test has failed, i don't know if it is a random failure.

Adding the needs documentation flag temporarily to remind us that the docs may need to be updated for 8.0 regarding this feature.

@tdonohue I have updated the submission settings section of the "User Interface Configuration" to update the icon configuration modified by this PR: https://wiki.lyrasis.org/display/DSDOC7x/User+Interface+Configuration#UserInterfaceConfiguration-SubmissionSettings
And I have added a new section "Add or edit authority controlled metadata fields" to the Edit Metadata page: https://wiki.lyrasis.org/display/DSDOC8x/Edit+Metadata
Feel free to rephrase it if it can be explained better. Thanks!


Add go mod tidiness CI check

Description
This PR introduces a GitHub Action check to ensure the go.mod/go.sum files are correctly tidied with go mod tidy. Additionally, it bump the golangci-lint version.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[x] Checking the outcome of the CI run

/rebase

/test

/rebase

/test

/test

/test

/merge


refactor getContractByteCode to set cost based on the sdk assessment

Description:
refactor getContractByteCode to set cost based on the sdk assessment using getCost method
Related issue(s):
Fixes #1523
Notes for reviewer:
Did not include a test case, since if every tests we already have continues to work as expected, there should not be an issue, also, testing this overlaps with testing the sdk itself, which is outside the relay and is already tested by the sdk test suites
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Patch coverage: 26.66% and project coverage change: -0.30 :warning:

Comparison is base (313d410) 78.01% compared to head (d2efd53) 77.71%.

:exclamation: Current head d2efd53 differs from pull request most recent head 4c97109. Consider uploading reports for the commit 4c97109 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1537      +/-   ##
==========================================
- Coverage   78.01%   77.71%   -0.30%
==========================================
  Files          32       32
  Lines        2583     2594      +11
  Branches      537      540       +3
==========================================
+ Hits         2015     2016       +1
- Misses        402      412      +10
  Partials      166      166

Impacted Files
Coverage Δ

packages/relay/src/lib/clients/sdkClient.ts
20.45% <0.00%> (-0.38%)
:arrow_down:

packages/relay/src/lib/constants.ts
90.32% <ø> (ø)

packages/relay/src/lib/eth.ts
84.36% <40.00%> (-0.60%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Demo page does not exist

@ChrisCavs Did you delete it by mistake?

I believe it is an issue with github pages.  I’m sure it will be resolved soon

It was actually an issue with the repository being private after they re-configured the rights of non-pro users.  thanks for opening this


Delete a repository from Azure Container Registry

It is not currently possible to delete an image from a repository in Azure Container Registry, either through the UI or the CLI tool -- other than manually looking at all of the digests for all of the layers of a specific image, and deleting those through the blob filesystem (this is an unreasonable method of deletion, prone to human error).
The registry API supports deleting images (all image layers): https://docs.docker.com/registry/spec/api/#deleting-an-image
az acr repository should support deleting an image repository from a specific registry, perhaps in the format of:
az acr repository delete --name MyRegistry --repo someRepo/someImage

Currently ACR doesn't have the delete API enabled yet.
The current implementation of DELETE API may not be what customer expects. The blob delete only deletes the link to the real image blob (since the real blob can still be referenced by other images). Thus it won’t reduce the storage usage much, and it may leave some garbage around.

Delete manifest (image). It will delete the specific version of the manifest of the image.
Delete blob. It will delete the reference to the blob from the specified image, but not the blob data itself.

ACR was considering to turn it on, but didn't since not sure if it’s the desired behavior…Besides, we have been thinking of the alternative where we do garbage collection of unreferenced blob when delete. This is still under design…
Would like to hear more feedback before we act on it…

Would recommend closing this and tracking the issue azure/acr#33 since this is a service implementation limitation and not a azure CLI issue.

Would recommend closing this and tracking the issue Azure/acr#33 since this is a service implementation limitation and not a azure CLI issue.

Creating an issue for ACR is appreciated, but IMO that doesn't invalidate the need for the implementation in the CLI.

@bddenhartog Yes that makes sense. Let's keep this open for the CLI side.

More feedback:  From my point of view the delete should be available for the user.  I know that the current implementation from the Docker API will only delete the reference and not the blob data (layers).  But it would be useful when the list of images becomes large and you would like to remove images tags that are no longer used or needed.

quote @prelution:

I know that the current implementation from the Docker API will only delete the reference and not the blob data

Yes, but Microsoft's implementation should account for this by using the reference to gather and delete all layers that aren't being used by other images. This can be accomplished by comparing the hashes of the layers from other images currently on the host.


[CHORE] Remove enum imports daft core

Overview

removed all instances of use MyEnum::*; from daft-core
using that pattern has a huge chance of hard-to-catch bugs being created in the future

Run the following grep in the daft-core subdir:
rg "use [\w]*::\*;"

You'll notice that no instances (other than use super::*) appear. This is as expected.


Data synchronisation of Demand and Supply

As of now, there is a multiple teams working on demand and supply applications but nothing is in a usable state to centrally manage. A common consensus in Supply data is required to solve this issue. It is a high need to use a common data structure by whoever working on the system so that everyone could be on the same page. Attached the data structure to use so that we can slowly eliminate free-form entry of items in every system. A crowdsourced IT support team will work on data sanitisation and management. It will be helpful if everyone could use the attached data structure for items or a method to sync with that structure.
Details of Relief Materials (1).xlsx

@deepusnath - I want to focus on medical as well. Can you also add a sheet for medicines ?

@bobinson IMA was against a collection of Medicines by the public.  Hope you are in connect with them on that.  Here is the categorisation of medicines
https://docs.google.com/spreadsheets/d/1_j0O_9KdVsdkZPK5qx4q7QEaSiP9aaQ_vcwk7wVkuBo

ack


leave out templates for training

#73  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Angryrou and the rest of your teammates on  Graphite


Make the crafter consume power

Description
Simple fix to consume power, this will need to get rewritten as a task at some point.
Fixes issue(s): #195
Todo
Checklist:

[ ] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Yeah this is fine for now, personally I dont think this will be balanced in any way, but as long as its not free then i think its fine :P


version attribute

Add a __version__ to the top level module and/or IngresDialect class.
No other dialects do this but would make it easier to introspect the dialect directly (than system metadata).

Added via https://github.com/ActianCorp/ingres_sa_dialect/pull/18

I don't see where this was implemented/added, please link to line number before closing.

Possibly implement similar to example.

Verifying this works as implemented via #25.
(.venv) C:\work\saDialect\gh10>python -V
Python 3.10.7

(.venv) C:\work\saDialect\gh10>pip show sqlalchemy-ingres
Name: sqlalchemy-ingres
Version: 0.0.4.dev1
Summary: SQLAlchemy dialect for Actian databases; Actian Data Platform (nee Avalanche), Actian X, Ingres, and Vector
Home-page: https://github.com/ActianCorp/sqlalchemy-ingres
Author: Chris Clark
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: c:\work\sadialect\gh10\.venv\lib\site-packages
Requires: sqlalchemy
Required-by:

(.venv) C:\work\saDialect\gh10>python
Python 3.10.7 (tags/v3.10.7:6cc6b13, Sep  5 2022, 14:08:36) [MSC v.1933 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import sqlalchemy_ingres
>>> print(sqlalchemy_ingres.__version__)
0.0.4.dev1
>>> quit()

(.venv) C:\work\saDialect\gh10>

@clach04 Is this okay to close?


docs: fix command typo in 02-add-the-graphql-schema.mdx

@AndersDJohnson: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


Serialize DocumentReadyState

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[X] I have read the CONTRIBUTING document
[X] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[X] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[ ] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[X] I have added tests to cover my changes
[X] All new and existing tests passed

Description
Serializes the DocumentReadyState enum for document.readyState based on the DomNameAttribute.OfficialName value if available.
Serializing other enum types has been left open, as too many tests/intended functionality breaks otherwise.
The serialized values are only available when AngleSharp v1.0.2 or greater is installed, otherwise previous behaviour is preserved.
Addresses #86

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Wayne Sebbens seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


EFI from latest one is crashing on sonoma 14.6 (probably sequoia beta 8 also)

So i downloaded your latest EFI (1.4.0), it booted fine on ventura without any issues, but when i updated to sonoma 14.6, it crashed the system after loading bar appeared for 2 - 3 secs (and instantly when booting recovery for fresh install, which it doesn't have issues on venture image).
i'm not sure about sequoia beta 8, it might be my fault.
edit: i'm using EFI without heliport

Same here

I cannot update to Sonoma from Ventura. I installed RestrictEvents kext but that made no difference. The system just reboots after selecting "macos installer" from the opencore boot picker screen. :-S


big scores get truncated due to "HI" score display

if you want big scores ten's of thousands or millions can't see score after high score is set

I think we need to overhaul the whole score/timer/lives UI. Maybe we should remove the Hi score and come up with a scheme for long numbers... Luckily we have the luxury of knowing exactly how many characters we are capable of displaying.


[Bug]: DwCA-Import - imports existing CEs as new records

Steps to reproduce the bug
Imported 20 brand new CO records.
All CE records linked to new COs already exist in the database.
All CE records instead imported as brand new records with 1 use. CEs are not linked to any collecting event or GA data (see below screenshot).

I used the following fields for the CE record import since all collecting event and GA data already exists:
-fieldNumber
-TW:Namespace:FieldNumber
-TW:CollectingEvent:verbatim_field_number
-TW:DataAttribute:CollectingEvent:VerbatimCollectionCode

Tested on sandwich first and it worked fine – all CEs linked to existing collecting event and GA data. did not work on production version.

Screenshot

Expected behavior
When I import CO records using existing CE records, I would expect them to link to all of the existing CE data instead of creating a new CE record.
If I Unify the 2 CE records as a temporary solution (screenshot 1), both field numbers show on Edit (screenshot 2) until I delete one of the identifiers (screenshot 3 and 4). Might need to submit a different ticket for this.
Additional Screenshots
SCREENSHOT 1

SCREENSHOT 2

SCREENSHOT 3 & 4

Environment
Production
Sandbox Used
No response
Version
v0.45.0
Browser Used
Chrome Version 130.0.6723.92

@LocoDelAssembly I'm not sure if we presently match on FieldNumber, probably only EventID?

The importer matches by identifier field of the Identifier (both, eventID and FieldNumber).
Sandwich:
irb(main):005> Identifier::Local::FieldNumber.where(cached: "BLF19068", project_id: 69)
=>
[#<Identifier::Local::FieldNumber:0x00007fc1a2dbcbd8
  id: 2802131,
  identifier: "19068", # <<< HERE
  type: "Identifier::Local::FieldNumber",
  created_at: Sat, 02 Apr 2022 20:14:18.895292000 UTC +00:00,
  updated_at: Sat, 02 Apr 2022 20:14:18.895292000 UTC +00:00,
  project_id: 69,
  cached: "BLF19068",
  identifier_object_id: 778200,
  identifier_object_type: "CollectingEvent",
  relation: nil,
  position: 1,
  cached_numeric_identifier: nil>]

Production:
irb(main):014> Identifier::Local::FieldNumber.where(cached: "BLF19068")
=>
[#<Identifier::Local::FieldNumber:0x00007f0a6f5b9110
  id: 4446960,
  identifier: "BLF19068", # <<< HERE
  type: "Identifier::Local::FieldNumber",
  created_at: Wed, 22 Nov 2023 00:48:51.624364000 UTC +00:00,
  updated_at: Mon, 04 Nov 2024 22:33:43.235800000 UTC +00:00,
  namespace_id: 1016,
  project_id: 76,
  cached: "BLF19068",
  identifier_object_id: 553640,
  identifier_object_type: "CollectingEvent",
  relation: nil,
  position: 1,
  cached_numeric_identifier: nil>]

The dataset (assuming it is "Technomyrmex import4"), has non-prefixed values as fieldNumber. Not sure if data above  was the initial state since it looks unification tool was already used.
Probably a virtual vs non-virtual namespace? I suppose that it is not actually possible to either create a duplicate FieldNumber on same Namespace (accounting for identifier field only, not cached which is what it is actually displayed), nor having the same identifier on two CEs.
If the importer actually missed an existing CE, it should have failed to create a new one because it would have attempted to duplicate identifiers and would have failed with "already taken".

I did fix add the correct CEs and deleted the duplicates with no associated CO record. And yes, the non-prefixed fieldNumber in the Technomyrmex import4 is how I imported them. It worked on a previous import that way.

Experimented with a backup prior to this import.
Before import:
irb(main):013> Identifier.joins(:namespace).where("cached like '%06693%'").merge(Namespace.where(short_name: 'BLF_AntWeb'))
=> []

After import of second record:
irb(main):014> Identifier.joins(:namespace).where("cached like '%06693%'").merge(Namespace.where(short_name: 'BLF_AntWeb'))
=>
[#<Identifier::Local::FieldNumber:0x00007f25f2323358
  id: 6769612,
  identifier: "06693",
  type: "Identifier::Local::FieldNumber",
  created_at: Tue, 05 Nov 2024 18:13:07.463666000 UTC +00:00,
  updated_at: Tue, 05 Nov 2024 18:13:07.463666000 UTC +00:00,
  namespace_id: 917,
  created_by_id: 3,
  updated_by_id: 3,
  project_id: 76,
  cached: "BLF06693",
  identifier_object_id: 735688,
  identifier_object_type: "CollectingEvent",
  relation: nil,
  position: 1,
  cached_numeric_identifier: 6693.0>]

When searching by cached (all namespaces):
irb(main):020> Identifier.where(project_id: 76).where(cached: 'BLF06693')
=>
[#<Identifier::Local::FieldNumber:0x00007f25e7787440
  id: 4292804,
  identifier: "BLF06693",
  type: "Identifier::Local::FieldNumber",
  created_at: Tue, 21 Nov 2023 18:49:53.694374000 UTC +00:00,
  updated_at: Tue, 21 Nov 2023 18:49:53.694374000 UTC +00:00,
  namespace_id: 1016,
  created_by_id: 2478,
  updated_by_id: 2478,
  project_id: 76,
  cached: "BLF06693",
  identifier_object_id: 533914,
  identifier_object_type: "CollectingEvent",
  relation: nil,
  position: 1,
  cached_numeric_identifier: nil>,
 #<Identifier::Local::FieldNumber:0x00007f25e7787300
  id: 4113688,
  identifier: "BLF06693",
  type: "Identifier::Local::FieldNumber",
  created_at: Tue, 21 Nov 2023 07:40:14.091645000 UTC +00:00,
  updated_at: Tue, 21 Nov 2023 07:40:14.091645000 UTC +00:00,
  namespace_id: 1014,
  created_by_id: 2478,
  updated_by_id: 2478,
  project_id: 76,
  cached: "BLF06693",
  identifier_object_id: 516759,
  identifier_object_type: "CollectingEvent",
  relation: nil,
  position: 1,
  cached_numeric_identifier: nil>,
 #<Identifier::Local::FieldNumber:0x00007f25e77871c0 #### JUST CREATED BY IMPORT (SAME AS CODE BLOCK ABOVE)
  id: 6769612,
  identifier: "06693",
  type: "Identifier::Local::FieldNumber",
  created_at: Tue, 05 Nov 2024 18:13:07.463666000 UTC +00:00,
  updated_at: Tue, 05 Nov 2024 18:13:07.463666000 UTC +00:00,
  namespace_id: 917,
  created_by_id: 3,
  updated_by_id: 3,
  project_id: 76,
  cached: "BLF06693",
  identifier_object_id: 735688,
  identifier_object_type: "CollectingEvent",
  relation: nil,
  position: 1,
  cached_numeric_identifier: 6693.0>]
irb(main):021>

If the importer was supposed to re-use CE with id=533914, there were two problems, one is that Identifier::FieldNumber identifier is fully qualified because the namespace is virtual (see below), so since the fieldNumber is without prefix it doesn't match and second, it was expected to be found in BLF_AntWeb namespace, but instead it was in an importer auto-generated namespace (from a much earlier import).
One thing that confuses me however, is that existing identifiers are in namespaces that were meant to be used for eventID. Was a script run at some point to re-type them as FieldNumber @mjy? The auto-generated namespaces were created before FieldNumber existed.
irb(main):024> Identifier.where(project_id: 76).where(cached: 'BLF06693').map(&:namespace)
=>
[#<Namespace:0x00007f25e7789ec0
  id: 1016,
  institution: nil,
  name: "eventID namespace for \"specimen formicidae\" dataset in \"AntWeb\" project [d8386021]",
  short_name: "eventID-d8386021",
  created_at: Tue, 21 Nov 2023 16:00:18.539462000 UTC +00:00,
  updated_at: Tue, 10 Sep 2024 21:56:20.426410000 UTC +00:00,
  created_by_id: 2478,
  updated_by_id: 2483,
  verbatim_short_name: "eventID",
  delimiter: ":",
  is_virtual: true>,
 #<Namespace:0x00007f25e7789d80
  id: 1014,
  institution: nil,
  name: "eventID namespace for \"specimen create names\" dataset in \"AntWeb\" project [2a48b12b]",
  short_name: "eventID-2a48b12b",
  created_at: Tue, 21 Nov 2023 07:23:22.255120000 UTC +00:00,
  updated_at: Tue, 10 Sep 2024 21:51:47.133804000 UTC +00:00,
  created_by_id: 2478,
  updated_by_id: 2483,
  verbatim_short_name: "eventID",
  delimiter: ":",
  is_virtual: true>,
 #<Namespace:0x00007f25e7789c40
  id: 917,
  institution: "Brian L. Fisher",
  name: "Brian L. Fisher [AntWeb]",
  short_name: "BLF_AntWeb",
  created_at: Mon, 20 Nov 2023 23:29:41.348662000 UTC +00:00,
  updated_at: Mon, 20 Nov 2023 23:29:41.348662000 UTC +00:00,
  created_by_id: 2478,
  updated_by_id: 2478,
  verbatim_short_name: "BLF",
  delimiter: "NONE",
  is_virtual: false>]

Update: Regarding "If I Unify the 2 CE records as a temporary solution (screenshot 1), both field numbers show on Edit (screenshot 2) until I delete one of the identifiers (screenshot 3 and 4). Might need to submit a different ticket for this."
I think when I delete one of the identifiers it also deletes CO information associated with that CE. The specimens are there, such as the 258 COs associated with CE BLF27044

... but in the CE record it only shows 50 COs associated. Is there some way to revert back to the 3 identifiers to get the data back? I fear this has happened to other CEs that I wasn't aware of.

After further testing... I think the Add/Current only shows the first 50 specimens. A little misleading since there are 259 (I added one)/


contracts: Add test to verify unique trie ids

I was thinking about removing the AccountCounter storage item because I thought we could handle this in memory. After giving it more though I came to the conclusion that we still need it.
Instead of removing I did the following:

Wrote down my thoughts as a doc comment on the AccountCounter so future generations understand what it is for.
Added a test to verify these comments: That we have unique trie ids.
Renamed account_counter to trie_id in exec.rs to be consistent there. The name AccountCounter is there for historical reasons only (renaming a storage item requires a migration).

The name AccountCounter is there for historical reasons only (renaming a storage item requires a migration).

You could use this if you want: https://github.com/paritytech/substrate/pull/9016
Would also just say to pull off the band-aid. Migration is simple

bot merge


Framebuffer deleted before attempting to bind

Hi! Thanks for all of the hard work!
I am having an issue when version 2.0.3 is injected with a very basic example. Note that this error doesn't pop up when running in a browser that doesn't require polyfilling.
Here is the error message:
WebGL warning: bindFramebuffer: Object 'fb' is already deleted.
This originates with my call to this.gl.bindFramebuffer(); and seems to be triggered at the self.realBindFramebuffer.call() calls in both CardboardDistorter.prototype.submitFrame and CardboardDistorter.prototype.patch.
I am using the default config for the polyfill. Any help would be greatly appreciated!

Thanks for filing an issue @stalgiag; do you have a link to reproduce this error?

This link works in Chrome 83 for Android; but not in older versions of Chrome or any version of Firefox. It also does not work with iOS but I don't have a good way to debug to say for certain it is the same error message.
Note that there may be a DPDB warning (dependent on device). This has never been breaking before, so I assume this isn't the problem. Also this link worked with the older version of the polyfill with both Chrome and Firefox on Android and iOS.
Thank you for the help!

This might be more helpful. Here is another test that does not have any code of my own. This is the current Immersive Web Immersive VR test with the most recent build of this polyfill loaded behind a navigator.xr check instead of a url flag. I am seeing the same behavior on firefox for android and chrome+firefox for android. Whereas with past versions of the polyfill this example worked.
Thank you for all of the help!


POSTS not REFLECTING

Describe the bug
A clear and concise description of what the bug is.
So I just added a new post on my wiki and its not reflecting My Wiki
Added wiki on GitHub
I thought the issue was the way I had named the file and renaming it doesn't fix it.
Further digging in I can see we made changes on the _config.yml file. Here is the diff and I am not sure can that be the root cause?

I have fixed it.

Hi sorry I completely missed it. What wad the solution eventually?

So I checked your recent diffs and noticed my blogs config scope path wasn't enclosed with two quotes. Weird bug.
But now its not updating the new post here

Latest blogs post Windows config not seen above ...


Update README.md

add building part

Actually the guide to build the binary is written in _docs/BUILD.md

Actually the guide to build the binary is written in _docs/BUILD.md

oh, haven't noticed
then close it


Add deduplicator class

Related to #98
Task: keep tabs on the input stream and implement different deduplication policies

There is a deduplicator class in the _deduplicate.py module, but it implements only the naive deduplication policy so far. Could be extended in the scope of the pipline milestone.

Closing to keep an overview. If functionality is to be extended, will create a new issue with a better title.