[Packaging] Drop Ubuntu 20.10 Groovy Gorilla support

According to

https://wiki.ubuntu.com/Releases
https://releases.ubuntu.com/
https://fridge.ubuntu.com/2021/07/25/ubuntu-20-10-groovy-gorilla-end-of-life-reached-on-july-22-2021/

Ubuntu 20.10 Groovy Gorilla has reached its End of Life at July 22, 2021. Azure CLI should drop its support, like the way we dropped Ubuntu 19.10 Eoan Ermine support (#16306).

This will be in the release notes: https://docs.microsoft.com/en-us/cli/azure/release-notes-azure-cli
When Eoan was dropped (https://github.com/Azure/azure-cli/pull/16306), there was no other official announcement. Users on Groovy can still install Azure CLI, but not the latest version. So this shouldn't be a breaking change.


[UG] Update add and edit command details

Fix #255, #257, #269, #276, and #293
Fix #263

Latest update


Adding documentation to react-router-dom and fixing repo link

Addresses issue #4648, which has 165 thumbs up to date, indicating a
real problem that needs solving.

Thanks, but it's already pointed out in the documentation, which you would need to read to understand how to use the library anyways :smile:

@timdorr where in the documentation is this difference between react-router and react-router-dom called out? I was trying to poke around the documentation and could not find any explanation as good as the one presented in #4648 . For example, I see nothing that calls out that a difference between react-router and react-router-redux is that the former only exports functions, while the latter will actually render DOM onto the page.

@timdorr I just looked through the docs again, and although they correctly advise people to install react-router-dom or an alternative, they definitely do not specifically answer the questions you did in #4648 or clarify what the difference between react-router and react-router-dom is at all.


[5.9] Add ability to set model instance in relationship methods

According to these ideas : laravel/ideas#1652 and #27278
And according to @driesvints answer in laravel/framework#28726, I try submitting my PR here.
I propose to allow passing instance of Model to relationship declaration methods. This can be useful to set a specific configuration (connection, table name, ...) for a given relationship.
Example :
public function superCustomer()
{
    $related = new Customer();
    $related->setTable('super_customers');

    return $this->belongsTo($related, 'id_customer', 'id_customer');
}

I dunno. I don't think I want to go down this road and further complicate the ORM.


net/http: Server panics on nil connection

What version of Go are you using (go version)?
Go 1.14
Does this issue reproduce with the latest release?
Yes, as well as 1.13.8.
What did you do?
When using a custom net.Listener there was a case where a nil net.Conn could be returned, along with a nil error. The listen function doesn't check whether or not the connection coming from Accept is nil: https://golang.org/src/net/http/server.go?s=90476:90522#L2901
This can lead to a hard to track down panic, because if this happens e.g. on program shutdown whether you actually see the panic or not may be timing-dependent.
I should note that the net.Listener documentation does not say that it is required to return a non-nil net.Conn if err is nil.
What did you expect to see?
A proper nil check.
What did you see instead?
A panic.

Go only documents unusual cases.
If a func returns (T, error), that means by default that exactly 1 of T or error must be non-nil. If both can be non-nil or both can be nil, then we document it.

I don't think there's anything to do here. Maybe we could panic with a more helpful string, but I'd still panic.

If a func returns (T, error), that means by default that exactly 1 of T or error must be non-nil. If both can be non-nil or both can be nil, then we document it.

Can I just ask, where is that in the spec? I haven't read the spec lately, it's been a few years, but I don't remember seeing that and some grepping through the spec right now doesn't pull anything up.

It's not in the spec.  The language permits returning multiple non-zero values.  But the convention is that when returning an error the other values are returned as the zero values for their type.  Functions that do not follow that convention are expected to document that fact.  There is no need to document the fact that a function does follow that convention.

There is no need to document the fact that a function does follow that convention.

In fact, it's actively harmful, because then people wonder why some places document it and some don't. We've actually removed redundant documentation in the past because it was causing such confusion elsewhere.

There's a difference between "this is what I see a lot of places" and "this is a convention that is expected to be followed". If this isn't in the spec, you are relying on the user's memory and/or experience and/or for them to never actually just make mistakes and have bugs (as happened in this case) in order to not trip up the stdlib with an iron-clad but undocumented convention you are expected to follow. Maybe I'm alone here but it feels pretty user-hostile to require developers to just "know convention", especially for a language as generally well-specified as Go.
Regardless, I don't even really see what all of that that has to do with this report. My custom listener had a bug and it triggered a panic. Wouldn't it be better to do a nil check and return an error than panic on a nil pointer? Defense in depth and all.

I believe this is a won't do.


Issue Report Regarding Date-Year-Input Preview

The issue primarily revolves around the DateYear component. Specifically, during the generation of the preview response, particularly when the pdf flag is set to true, an anomaly is encountered.
Detailed Description:
Upon inspection, it was observed that within the preview response generation, the value retrieval mechanism relies on the getPDFValue function. Below is a snippet exemplifying its integration:
return React.createElement(textview_1.default, {
  id: props.id,
  item: props.item,
  value: getPDFValue(),
  onRenderMarkdown: props.onRenderMarkdown,
  helpButton: props.helpButton,
  helpElement: props.helpElement
}, props.children);

The getPDFValue function internally invokes the getYear function to fetch the relevant year data. However, a critical issue arises within the getYear function. Despite the conditional check for props.answer being an array, it invariably evaluates to false, leading to an undefined outcome. Consequently, the final response defaults to the 'not found text'.
Below is a segment illustrating the problematic section within the getYear function:
const getYear = () => {
  if (Array.isArray(props.answer)) {
    return props.answer.map(m => {
      return (createDateFromYear(props.item, m))?.getFullYear();
    });
  }
};

Proposed Solution:
To address this issue, I have devised a solution that involves refining the getYear and getPDFValue functions. The modifications aim to enhance the logic flow and ensure accurate retrieval and presentation of the year data.
Here is a synopsis of the adjustments made:
const getYear = (): (number | undefined)[] | undefined | string | number => {
  if (Array.isArray(props.answer)) {
    return props.answer.map((m: QuestionnaireResponseItemAnswer) => createDateFromYear(props.item, m)?.getFullYear());
  }

  if (props.answer?.valueDate) {
    return props.answer?.valueDate
  }

  return answerState;
};

const getPDFValue = (): string | number => {
  const ikkeBesvartText = props.resources?.ikkeBesvart || '';

  const year = getYear();

  if (Array.isArray(year)) {
    return year
      ?.map(m => m?.toString())
      .join(', ') || ikkeBesvartText;
  }

  return year || ikkeBesvartText;
};

The outlined adjustments ensure robust handling of diverse data scenarios, thereby mitigating the issue encountered during the preview response generation.
I trust that this information provides valuable insight into the matter at hand. Please feel free to reach out should further clarification or assistance be required.
Thank you for your attention to this matter.
Best regards,
Astghik

We are currently in the prosess of replacing the old date components with new components from @helsenorge/designsystem-react package

We are now releasing v.16.0.0, here we are using a new date component. let us know if you still are having issues


Add PPE filter checkbox to the facility search page

Overview
Adds a PPE checkbox to the facility search page and connects it to the search API.
Connects #1043
Demo

Testing Instructions
These instructions assume resetdb has been run.

Download and unzip ppe.csv.zip
Log in as<EMAIL_ADDRESS>browse http://localhost:6543/contribute and submit ppe.csv
Fully process the file by running commands on the vagrant VM

./scripts/manage batch_process --list-id 16 --action parse
./scripts/manage batch_process --list-id 16 --action geocode
./scripts/manage batch_process --list-id 16 --action match

Browse http://localhost:6543/facilities?ppe=true and verify that all 5 facilities from the CSV are returned.
Clear the search and run a text search for "all" and verify that there are 11 results. Then check the PPE checkbox and search again. Verify that there is a single result.
Browse http://localhost:8081/api/docs/#!/facilities/facilities_list and verify that the "ppe" field is documented.

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

While fixing up the Swagger docs I realized that we should be conditionally hiding the ppe query argument documentation based on the feature flag. I made this simple change in 4bc25b3 and tested it.

Thanks for the review.


Alter docs for ODBC connection rework

https://github.com/duckdb/duckdb/pull/10692
Fixes #2190

Splendid, thank you!


add verdaccio link to repo

I like to promote all plugins and this one is great
I also added yours to the list https://verdaccio.org/docs/en/next/plugins#storage-plugins

Hi @juanpicado, thanks for the PR, nice to see the plugin on the Verdaccio documentation :)


feat: hrp support uploading file

Main Changes

feat: support uploading file by multipart/form-data
fix: failed to convert postman collection containing multipart/form-data requests to pytest

Codecov Report

Merging #1375 (5944930) into master (c00a2a5) will decrease coverage by 10.39%.
The diff coverage is 16.66%.

@@             Coverage Diff             @@
##           master    #1375       +/-   ##
===========================================
- Coverage   57.65%   47.26%   -10.40%
===========================================
  Files          82       47       -35
  Lines       10116     6661     -3455
===========================================
- Hits         5832     3148     -2684
+ Misses       4014     3239      -775
- Partials      270      274        +4

Flag
Coverage Δ

unittests
47.26% <16.66%> (-10.40%)
:arrow_down:

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

Impacted Files
Coverage Δ

hrp/internal/builtin/function.go
3.33% <0.00%> (-8.44%)
:arrow_down:

hrp/step_request.go
47.02% <17.85%> (-2.16%)
:arrow_down:

hrp/internal/convert/converter_postman.go
67.98% <69.23%> (+4.57%)
:arrow_up:

hrp/boomer.go
66.66% <0.00%> (-14.50%)
:arrow_down:

hrp/step_thinktime.go
69.56% <0.00%> (-6.53%)
:arrow_down:

httprunner/make.py

httprunner/compat.py

httprunner/step_request_test.py

httprunner/builtin/__init__.py

... and 32 more

Continue to review full report at Codecov.

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


listunspent shows zero unspent outputs

I created and sent a transaction with 100 outputs (each is 546 satoshis). After that I run listunspent, it showed these 100 outputs. Then I used omni_send to transfer some coins to another address. After that listunspent shows 0 utxos, even though getbalance is not 0
listunspent 0 99999999 '["n2o5KhdKs6krGuvT5gRVhNnvVhgNGPgRwb"]'
> []

setaccount n2o5KhdKs6krGuvT5gRVhNnvVhgNGPgRwb test
getbalance test
> 0.00958776

Wallet has the private key, so it should be showing all these unspent outputs
dumpprivkey n2o5KhdKs6krGuvT5gRVhNnvVhgNGPgRwb
> cV...Tzr

The problem is I can't reliable send coins several times because each time utxos are reset (even though they are present and belong to the address), omni_send/omni_funded_send gives this error "Error with selected inputs for the send transaction" when listunspent returns empty array.

It's also probably worth noting that I'm doing this on local regtest daemon.


circle icon list isn't accurate

for example, magnify points to "information"

refactored a good chunk of it, moved it to ribbons.
better code as well.


Wrong generation for QRCode

I used this code to generate qrcode, but the result is wrong. How to resolve this problem?

https://github.com/rosskhanas/react-qr-code?tab=readme-ov-file#api


🍒 Cherry pick PR #13028 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/13028 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.2.31-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Canonical link from getSeoMeta is not updated on path change

Thank you for the response @blittle! This makes sense, had missed the implementation of getSeoMeta utility.
Something to improve: Just noticed in the Demo Store that the canonical link from getSeoMeta is not updated on path change. Other fields such as title and description are updated
Originally posted by @nerioslamaj in https://github.com/Shopify/hydrogen/discussions/2280#discussioncomment-9940180

Thanks for raising this bug issue, looking forward to the upgrade


fix(libsinsp): fix sinsp_state_sc_set ppm sc APIs

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug
/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
libsinsp::events::sinsp_state_sc_set returned no results, instead of fixing the bug cut over to a new approach.
We have many more refactors and feature additions planned not just around the ppm sc API, but also the event and syscalls lookup tables ...
Therefore, suggesting to hard-code the sinsp state set based on EF_MODIFIES_STATE flag only in order to reduce the number of moving parts in the interim and introduce stability.
Lastly, we had multiple discussions around getting this fully out of scap anyways, so this is a great interim step towards this step, which will fall in place alongside the already in progress additional refactors.
Updated and added more unit tests.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: incertum
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

In the interest of time and since we work opposite time zones, please feel free to directly push into here @FedeDP @jasondellaluce @Andreagit97 and more 🙏 .

In the interest of time and since we work opposite time zones, please feel free to directly push onto this branch @FedeDP @jasondellaluce @Andreagit97 🙏 .

@FedeDP please also confirm here that all issues addressed in this PR have been fixed in 889, see https://github.com/falcosecurity/libs/pull/889/files#r1113354596, so that this one can be closed out, thanks.

Fix moved to https://github.com/falcosecurity/libs/pull/921, thanks! Closing this out.


Integrate with EU Digital COVID Certificate for vaccination, recovery and test (previously called EU Digital Green Certificate)

Problem description
The EU's proposed Digital Green Certificate (now called EU Digital COVID Certificate) is a personalized proof of COVID-19 health status. In terms of test status it concerns itself with non-infectious individuals.
CWA on the other hand uses pseudoanonymous data exchange regarding test results and shares positive tests pseudoanonymously back to the community of CWA users.
Users are sometimes forced to deal with different systems resulting out of one test because pseudoanonymous and personified data are handled completely separately.
Feature Description
Integration of CWA into the European Union's eHealth : Digital health and care proposed Digital Green Certificate:
"A Digital Green Certificate is a digital proof that a person has been vaccinated against COVID-19, has recovered from COVID-19 or has a negative test result. The Digital Green Certificate is therefore a name used to collectively refer to 3 types of certificates: vaccination, recovery, test."
The feature proposal for CWA and for related processes, such as the 10C testing form, is to revise the test reporting mechanism so that there is one standardized way of reporting the results of a COVID-19 test back to the person being tested, so it can be automatically integrated into CWA and in parallel into a Digital Green Certificate in the case of a negative test.
"On 17 March 2021, the European Commission presented a proposal to create a Digital Green Certificate to facilitate the safe free movement of citizens within the EU during the COVID-19 pandemic.
Digital Green Certificates will be valid in all EU Member State"
Timeline (according to EU documents)

[x] 1 June - EU Gateway (interconnection of national systems) goes live.
[X] 1 - 30 June - Warm-up phase: Member States can launch the certificate on a voluntary basis provided they are ready to issue and verify certificates, and have the necessary legal base in place.
[X] 1 July - The EU Digital COVID Certificate enters into application throughout the EU.
[X] 1 July - 12 August - Phase-in period: if a Member State is not yet ready to issue the new certificate to its citizens, other formats can still be used and should be accepted in other Member States.

Integrated Features

[X] Vaccination certificates (CWA 2.3)
[X] PCR-Test certificates (CWA 2.4)
[X] Rapid-test certificates (CWA 2.4)
[ ] Add multiple certificates (CWA 2.5)
[ ] Recovery certificates (CWA 2.5)

Related Apps

CovPass-App (German language information only)
CovPassCheck-App (German language information only)

Related Issues
Recovery
How do I tell the app that I recovered from COVID-19?

Edit: Based on information from EU Press Room
EU COVID-19 certificate must facilitate free movement without discrimination April 29, 2021 I have changed the name from "Digital Green Certificate" to "COVID-19 certificate".
Edit: See also https://github.com/eu-digital-green-certificates/dgc-overview "Digital Green Certificate: Overview" Open Source Project.
Edit: https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en now named "EU Digital COVID Certificate".
Edit: Add link to https://digitaler-impfnachweis-app.de/

Internal Tracking ID: EXPOSUREAPP-6080

@MikeMcC399 Thanks for proposal. Internal Tracking ID: EXPOSUREAPP-6080

n-tv talks about a planned implementation!
(h/t @vaubeahn 🙂)

Read this optimistic news.
Very encouraging for the propagation and hence effectivity of the CWA.

This issue was mentioned in https://www.focus.de/corona-virus/impfung/corona-impf-nachweis-grosses-update-corona-warn-app-kuenftig-mit-digitalen-impfpass_id_13239972.html

@Ein-Tim
Thanks for closing the loop! The Focus article - Großes Update: Corona-Warn-App künftig mit digitalen Impfpass refers back to this wishlist item #424.
focus.de actually refers to chip.de and the original article is posted under chip.de - Digitaler Impfpass kommt in die Corona-Warn-App.

My understanding of this feature suggestion is that the test result and green-pass status can be fed back into the CWA app where it can be used in determining infection risks. E.g. If I have a green-pass I will not receive a high-risk notification after checking into an event where someone else later tests positive. But the certificate with my personal details is kept in a separate app.
However, from the screenshots and description of the article it implies that the CWA will in future contain personal information:

Ein negatives Ergebnis eines Schnelltestes bleibt 48 Stunden in Corona-Warn-App gespeichert und dient als digitaler Nachweis. Dabei ist der Test personalisiert und es werden Name und Geburtsdatum der getesteten Person angezeigt.

Either the article is wrong, or there is some aspect of the mechanism that I have not understood. This is important to me because currently I show skeptics that there is no personal stored in the CWA, other than what I choose to write in the journal, and even there I don't type in full names.

Either the article is wrong, or there is some aspect of the mechanism that I have not understood. This is important to me because currently I show skeptics that there is no personal stored in the CWA, other than what I choose to write in the journal, and even there I don't type in full names.

Please refer to this talk @MikeMcC399 and I had yesterday: https://github.com/corona-warn-app/cwa-wishlist/issues/489#issuecomment-827574263

The work on this just begun: https://github.com/corona-warn-app/cwa-app-ios/pull/2584

@Ein-Tim

Seems like we'll see an integration in version 2.3.

I hope we'll see a blog entry on https://www.coronawarn.app/en/blog/ if that is what has been decided. @dsarkar did warn though in https://github.com/corona-warn-app/cwa-wishlist/issues/489#issuecomment-827736774 that the situation is "dynamic".

According to https://www.europarl.europa.eu/news/en/press-room/20210422IPR02606/eu-covid-19-certificate-must-facilitate-free-movement-without-discrimination from today, based on the EU parliament's debate yesterday (April 28, 2021) they are now calling it “EU COVID-19 certificate” - instead of "Digital Green Certificate", so I will modify the title and of the main issue.
Trying to interpret the information in the above link, I also come to the same conclusion as @dsarkar that the situation continues to be dynamic!

FYI https://github.com/eu-digital-green-certificates

@dsarkar
Many thanks for the link!
https://github.com/eu-digital-green-certificates/dgc-overview so congratulations to Deutsche Telekom and SAP!
About this Project
The digital green certificate implements the EU trustframework according the specification of the eHealth Network Interoperability Subgroup (https://github.com/ehn-digital-green-development/hcert-spec).
To this end the European Commission has tasked Deutsche Telekom and SAP to develop the Digital Green Certificates Gateway as well as applications that may be used by all countries connected to the gateway. These include a national backend with an issuer component ( Issuance Service, Issuance Web ) and an app to validate Digital Green Certificates ( Verifier Service, Verifier App - Android, Verifier App - iOS ) as well as a wallet app to be used by the citizens of connected countries ( Wallet App - Android, Wallet App - iOS )."

This is also related to
https://www.bundesgesundheitsministerium.de/coronavirus/faq-covid-19-impfung/faq-digitaler-impfnachweis.html
and
https://digitaler-impfnachweis-app.de/ which refers to the CovPass-App from RKI.

https://www.n-tv.de/panorama/Bund-startet-Test-fuer-digitalen-Impfnachweis-article22578846.html reports "Bund startet Test für digitalen Impfnachweis" with CovPass-App.

I wonder why this is not yet labeled as Fix 2.3 @dsarkar
Don't we integrate with the Digital Green Certificate in the next version? Even if it's not yet official, everybody can follow the (nearly finished) work in the prod. repos.

@Ein-Tim

I wonder why this is not yet labeled as Fix 2.3 @dsarkar
Don't we integrate with the Digital Green Certificate in the next version? Even if it's not yet official, everybody can follow the (nearly finished) work in the prod. repos.

On Android, the release_info_strings.xml for release/2.3.x says "You can now add your vaccination certificates in the app and present them via QR code. The app displays full vaccination protection 14 days after the final vaccination."
The Digital Green Certificate is more than just vaccination. It corresponds to the triad of vaccination, negative test or recovery. So I would not expect to see the integration in the 2.3 release, because the release text does not mention a certificate for negative test, nor for recovery.
@heinezen has added the approved label to this issue. The  Android v2.4.0-RC3 release candidate mentions "Green certificate UI skeleton (EXPOSUREAPP-7516)" with PR https://github.com/corona-warn-app/cwa-app-android/pull/3283 so I suggest that 2.4 would be the next release to look at, as pieces are added to work with the Digital Green Certificate.

I wonder why this is not yet labeled as Fix 2.3 @dsarkar
Don't we integrate with the Digital Green Certificate in the next version? Even if it's not yet official, everybody can follow the (nearly finished) work in the prod. repos.

@Ein-Tim As @MikeMcC399 said, more features for the EU COVID certificate are coming in subsequent releases. It's better to think of it as a certificate infrastructure than just one single certificate, although 2.3 will implement the most important parts as far as I know.
It should also be noted that CWA 2.3 (and maybe 2.4) will likely release before the "official" start date of the certificate, so there might be last-minute adjustments based on feedback from the testing and staged rollout phases. Furthermore, backend infrastructure for the health departments is built in parallel in the background. As of now, it is difficult to determine when everything is ready, but the feature releases implementing the certificate will probably come in short succession in June/Juli.

Corona-Warn-App Open Source Team

The original link https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/covid-19-digital-green-certificates_en now redirects to
https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_en where it shows the label:
EU Digital COVID Certificate
The contents there, including the timeline, have also been updated.

@MikeMcC399 Thanks for updating us on that. I put the timeline in your top post, so that everyone knows what's going on.

Corona-Warn-App Open Source Team

According to https://www.n-tv.de/panorama/Den-Digi-Impfpass-gibt-s-beim-Apotheker-article22605257.html "Übertrag aus Impfbuch ab Montag" CovPass-App will be available on Monday.

https://www.n-tv.de/panorama/Digitaler-Corona-Impfpass-ab-Donnerstag-verfuegbar-article22607422.html "Digitaler Corona-Impfpass ab Donnerstag verfügbar" tells us that CWA 2.3.2 was released last night and that Bundesgesundheitsminister Jens Spahn is going to make an announcement tomorrow, Thursday, Jun 10, 2021 regarding the CovPass-App.

CWA Version 2.3 has been released and this version includes the functionality to scan in SARS-CoV-2 vaccination certificates. See https://www.coronawarn.app/en/blog/.
CovPass-App is available now and the FAQs
https://digitaler-impfnachweis-app.apps.public.bfarm.de/faq/ say:
"** Gilt der digitale Impfnachweis nur in Deutschland oder auch in anderen Mitgliedstaaten der Europäischen Union?**
Der digitale Impfnachweis kann nicht nur in Deutschland genutzt werden. Der in Deutschland ausgestellte digitale Impfnachweis kann auch im EU-Ausland als Nachweis für erfolgte Impfungen gelten, wenn der jeweilige EU-Mitgliedstaat digitale Nachweise zulässt. Dasselbe gilt perspektivisch auch für digitale Genesenenzertifikate und Testzertifikate."
So, the first part of this wishlist item, to integrate with the EU Digital COVID Certificate for vaccination, has been fulfilled with the release of CWA 2.3.2.
Outstanding is the functionality for compatibility with the EU Digital COVID Certificate for recovery and test.

"Today, the Presidents of the three EU institutions, the European Parliament, the Council of the EU and the European Commission attended the official signing ceremony for the Regulation on the EU Digital COVID Certificate, marking the end of the legislative process."
Full announcement on: EU clears way for the EU Digital COVID Certificate, June 14, 2021

https://www.n-tv.de/ratgeber/Diese-Erleichterungen-stehen-an-article22651577.html reports on official introduction of cross-border EU Digital COVID Certificates starting tomorrow, July 1, 2021.

This wishlist item is implemented.
See blog announcements https://www.coronawarn.app/en/blog/
https://www.coronawarn.app/en/blog/2021-07-12-cwa-version-2-5/ says:
"Deutsche Telekom and SAP’s project team have released version 2.5 of the Corona-Warn-App, in which users who have recovered from COVID-19 can integrate a digital certificate that serves as official proof for their recovery. Whether vaccinated, recovered, or tested, users can now manage all types of certificates in one place in their Corona-Warn-App. Users can manage certificates both for themselves and for others, such as children."

Thanks @MikeMcC399.


css style of docs/pages/guide/i18n.md on mobile is incorrect

The i18n page's css style is incorrect.
When using https://valaxy.site/, it will be correct after refreshing, but in dev it has no change.

I solved this problem in #115.
Now it looks like this:


Sc file size

Modify the way getting a file's size works, so that if you use the getFileSize method it will populate the File vertex's fileSize attribute from the blob's reported size if the attribute is null.
Modified the getSize method (which always tries to get the size from the blob before checking the attribute) so it doesn't call the getFileSize method (as that could call the blob's size method twice).
I'm not keen on side effects like this change incorporates, but it prevents going to the blob every time you want to know the file's size when calling getFileSize because that can be an expensive/slow activity.
@ato @yetti @josephcurtis @jrixon @sjacob @snezanam @shuangzhou, basically 'at' everyone :-)

What happens if the file gets updated/changed?

I've addressed Yetrina's issue by making modifications based on Shuang's suggestion and added a couple of tests to check (FileTest.java).
Note: i cleaned up a couple of tests along the way so they extend the AbstractDatabaseIntegrationTest class to make them faster (WorkTest.java and CopyTest.java). These are unrelated to this change except that i saw them and thought i'd do a quick tidy.
@yetti or anyone ?

Looks good 👍


Merge pull request #1 from adscib/master

upd

I'm sorry, clicked wrong button. Will send you a proper PR soon though. Thanks!


Fixes to the resource page

This is a second part of https://github.com/flattenthecurve/guide/pull/305 (should be only integrated once the other PR is in).
This PR:

switches to lowercase url attribute for resources (to eliminate need for special treatments)
properly escapes all resource attributes & removes some ad-hoc empty lines in the resource files
Converts United States to USA in one of the resource files. [1]

[1] it might be worth creating automatic rule converting the country name to the canonical form when new resource submission is received.

I have reverted the changes to _scripts/import_resources.py so that this can be merged into master independently of the other PR.

@jaysonvirissimo, @emersonthis and/or @mverzilli - please approve.

I added the "on hold" label to signal that this shouldn't be merged until #305 is merged.

I made this actually independent PR that can be merged as-is w/o having to
wait for the other one
On Sun, Apr 5, 2020, 14:22 emersonthis<EMAIL_ADDRESS>wrote:

I added the "on hold" label to signal that this shouldn't be merged until
#305 https://github.com/flattenthecurve/guide/pull/305 is merged.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flattenthecurve/guide/pull/306#issuecomment-609476542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYSCGD6JQFUSXKZJSEG7MLRLDSA7ANCNFSM4MBIDZTQ
.


Publish elemental-operator chart on registry.opensuse.org

Fixed the title (🤦🏻♂ ) Marking as done.
Pushing to .suse.com would be a second step.


When autocrafting, keeps asking for bucket of liquids even when fluid is kept in the system

Issue description: when autocrafting, shouldnt it just pull from the fluid storage ? Keeps asking for buckets of fluids as per the reciepe request ?
What happens: see above
What you expected to happen: see above
Steps to reproduce:

...
Version (make sure you are on the latest version before reporting):

Minecraft: 1.14.4
Forge: 28.1.117
Refined Storage: Latest build

Does this issue occur on a server? [yes/no]
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

ok, i got the idea, but when making the pattern....how do put the liguid form into the pattern (1000 mB of the fluid of your choice) when i place it in the pattern via a bucket, it comes up as fluid in a bucket ?
also Im running 1.14.4, thermal exampsion is not ported over, what machine can i use to fill it with >?

On the left side of the pattern GUI is a button that switches between item and fluid view.
You can create a bucket filling machine using vanilla dispensers in combination with a constructor.  For 1.14 I've gone with silent mechanisms pump instead of the dispenser but it works the same way.

how did you use the pump from Silent Mech's ? I placed it, put a crafter to it, put my liquid patterns in there and for some reason it keeps pulling water into it.....can you show me or explain how you set this up, please....


Make it simpler to get localTempDir

job.tempDir or something would be nice.

FYI job.fileStore.localTempDir tells you the sandbox for the job. You can use that till this is implemented

Addressed with #1940


Mobile menu opens when going Back

Describe the bug
The mobile menu opens when using browser Back button in some cases, which I found unexpected.
To Reproduce
Steps to reproduce the behavior:

Go to blowfish.page on mobile
Open menu and tap a page link like Docs or Shortcodes
Tap browser back button

Expected behavior
I expected the menu to be closed when returning to previous page.
If this is a feature, is it configurable to disable?
Desktop:

OS: Android
Browser: Chrome
Version: 127

Hugo & Blowfish versions
Latest on blowfish.page.

Replicated on mobile Firefox and all major desktop browsers (when dropdowns exist) as well.


IModule::getDependencyFilePath returns relative path for shaders in cwd

For hot reload in SGL, we are pulling on all file dependencies of a slang module to find all the directories we need to watch for changes.
The IModule::getDependencyFilePath gives up a bit of a mixed bag of results.
It returns:

Absolute paths for most of the files (what we'd expect)
Relative paths for shaders that are in the current working directory (didn't expect that)
Strings like cbe84a99d1f8c9bdc0f6cc4ffa3b612576d8bd1f, probably for string modules (as opposed to file modules), would expect a nullptr here as there is no FilePath.

Could you please shed some light on the expected behavior? From our side, the ideal behavior would be nullptr for modules with no file, and absolute filepath for all file based modules, but even a more detailed description of the possible outputs would help catch it all.
We do have a workaround, if the path is not absolute, we try to look for the returned string as a file in cwd, and if it is not there, we assume it is a string based module and skip, but we're not sure we have all the possible options covered.

In Slang, every module is associated with a file path, that file path can be a real physical path, or an imaginary one.
Slang cannot tell if a path is real or not. After all, when you load a module from source, you also get to specify the file path for the source that slang will use to identify this module. It makes sense for slang to return  back whatever is being passed to the filepath parameter at loadModuleFromSource when you query file dependencies.
It is therefore the application's responsibility if they want to make sure a file path slang returns is an actual file.

In addtion, slang allows applications to pass in a custom IFileSystem interface to handle all file load/store requests. The path can be completely arbitrary/imaginary as long as the user provided file system handler can deal with them. There isn't a canonical way in Slang codebase to tell what accounts a valid path and what does not.

Because of the restricted view of the file system through the IFileSystem interface, it is also not possible for slang to canonical all paths into absolute paths. It is dependent on where the path comes from and slang itself has no knowledge to convert the path names.

I am closing this issue now, since I don't see any good actionable items. I think your workaround makes sense and feels the right way to address the issue.

I am gonna ping this again, since I was digging a bit deeper after I did my workaround.
My problem comes from IncludeSystem::findFile.
It first tries to find the file relative to the current path, and if it can, it just returns it like that.
If it cannot, it will then try to prepend search paths to it, turning the path into absolute path.
In either case, it then goes and converts to absolute path for uniqueIdentity, but when asking for Module::getDependencyFilePath I get the "found path" which is "either relative or absolute, depending on the cwd" and not the uniqueIndentity, which is always absolute.
I don't think the result of foundPath should be change based on cwd. I'd just convert the path to absolute path always, even when found without using the search path (e.g., in STL I'd just call std::filesystem::absolute on the result).
However, I do not know how is IncludeSystem related to the IFileSystem and how much could this change break the alternative implementations.


wrong import of HBW-LC-RGBWW-IN6-DR

I agree to the following

[X] I have read the documentation right now
[X] I have read the FAQs right now
[X] I am aware of the latest release notes
[X] I am aware that an integration that was installed via HACS also needs to be updated via HACS
[X] The Backend (CCU/Homegear/...) is working as expected, devices are taught in and are controllable by its own UI.
[X] I am running the latest version of the custom_component (and Home Assistant)

The problem
I need help with the HBW-LC-RGBWW-IN6-DR. I have two of this and on RaspberryMatic I don’t have any problems.
I don´t use the HBW-LC-RGBWW-IN6-DR for RGB strips. I´m useing it vor 3 white strips.
On this page is written, you can use it for 2 RGB-strips, 1 RGBWW or 3 CW+WW-strips like 6 canal mode.
https://haus-bus.secure-stores.de/?showProduct=9
I´m use it with 3 CW+WW-stripe like 6 canal mode.
My problem is now that I have only two channels (1-2) like the picture below.

And four canal (4-6) looks like this picture.

If canal 1or 2 is on and I put canal 3 on, than 2 chance to off.
I need all canals like first picture.
So, is it possible to fix the define of HBW-LC-RGBWW-IN6-DR? If not, how I can do it?
What version of HomematicIP (local) has the issue?
1.54.0
What was the last working version of HomematicIP (local)?
not work
What type of installation are you running?
Home Assistant OS
What type of installation are you running for your homematic backend?
RaspberryMatic Standalone
Which version of your homematic backend are you running?
<IP_ADDRESS>31130
What hardware are you running for your system?
PI3
Which config details do you use

[ ] TLS
[ ] callback data (see)

Which interfaces do you use?

[X] Homematic IP
[X] Homematic (Bidcos-RF)
[X] Groups (Heating-Group)
[X] BidCos-Wired (HM-Wired)

Diagnostics information (no logs here)
config_entry-homematicip_local-8d3ece07cc2647596aa32487aa59b9a1.json.txt
Log file extract. Anything in the logs that might be useful for us? The log (Setting/System/Logs -> load full log) is the best source to support trouble shooting!
No response
Additional information
No response

I'm sorry, but I have to say that I have little motivation to read about these homebrew devices. The homebrew devices are very special, differ from the normal eQ3 approach, and are only used by a few users.
Unfortunately I can't do anything with the information from the link.
For me it is not clear from the existing device descriptions how one should be able to switch between the operating modes.
Either you can follow up with the information, or there is someone else who can do it.
I will therefore close the issue and convert it into a discussion/feature request.


Feature Request: Support for TmDb watch_region/with_watch_providers

It would be really great if the new filters are added.
https://developers.themoviedb.org/3/discover/tv-discover

Commit linked to this issue (62532535c9d6d6b2d686716614c36e05c78efc0e) does not resolve issue. Remaining/new issues are:

All with_watch_providers or without_watch_providers entries fail, as they are never referenced in the _tmdb function of modules/builder.py
with_watch_monetization_types cannot be used without also specifying a with_watch_providers entry, due to the requirement for with_watch_monetization_types to also have watch_region defined, which then hits the rule  that watch_region must have with_watch_providers defined.
Similar to number 2. Without_watch_providers cannot be used because the watch_region rule only checks for with_watch_providers to be defined.

These issues persist with the latest develop commit - 7fac92652c350ae5a49c1ea455d86a243e8571a1
My suggested fix would be, in modules/builder.py to change lines 1026-1030 to:
elif discover_attr == "watch_region":
    if "with_watch_providers" in dict_data or "without_watch_providers" in dict_data or "with_watch_monetization_types" in dict_data:
        new_dictionary[discover_final] = discover_data
    else:
        raise Failed(f"Collection Error: {method_name} {discover_final} attribute: must be used with either with_watch_providers, without_watch_providers, or with_watch_monetization_types")

and to add the following items to the array on line 1048
"with_watch_providers", "without_watch_providers"

I can make a pull request doing this if you would like. Just let me know.

ok this one should work lol


DO NOT MERGE TO MASTER - Enable using ReactiveUI with WinUI3-UWP

What kind of change does this PR introduce?
Allows use of ReactiveUI on a UWP project that uses WinUI3 (preview) for the UI.   While WinUI3 probably works as-is on a desktop app using WinUI3, WinUI3 for UWP changes a lot of namespaces for things built into ReactiveUI.
System.Windows.Input -> Microsoft.UI.Xaml.Input
System.ComponentModel -> Microsoft.UI.Xaml.Data
That means INotifyPropertyChanged didn't work as-is.  ICommand (ReactiveCommand) also didn't work.  In fact, INotifyPropertyChanging doesn't currently exist for UWP+WinUI3.
This PR should update all of the namespaces for UWP without touching the other builds by adding a WINUI3UWP compiler constant.
What is the current behavior?
ReactiveUI can be installed, but you can't use any XAML controls and ReactiveObject is not reactive.
What is the new behavior?
The controls work (RoutedViewHost), ReactiveCommand works with controls, and ReactiveObjects will yield property changed notifications again.
What might this PR break?
Definitely will break any normal UWP projects.  You can only use this with WinUI3 UWP projects.  Other projects should work fine as the compiler constant only applies to uap projects.
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

Dispatcher seems to not be working correctly as RxApp.MainThreadScheduler doesn't seem to work.  I need to fix that, too.

Dispatcher seems to not be working correctly as RxApp.MainThreadScheduler doesn't seem to work.  I need to fix that, too.

Dispatcher was working fine.  In having to use EventRegistrationTokenTable<PropertyChangedEventHandler> to add/remove events manually, I forgot to invoke the event using InvocationList...

Dispatcher was working fine.  In having to use EventRegistrationTokenTable<PropertyChangedEventHandler> to add/remove events manually, I forgot to invoke the event using InvocationList...

I had to modify DynamicData, too.  RoutedState depends on DD and DD uses a lot of INotifyPropertyChanged. 👎    I'm going to push a PR of DD to that repo just so people can find it.

I had to modify DynamicData, too.  RoutedState depends on DD and DD uses a lot of INotifyPropertyChanged. 👎    I'm going to push a PR of DD to that repo just so people can find it.

Dispatcher seems to not be working correctly as RxApp.MainThreadScheduler doesn't seem to work. I need to fix that, too.

Just wondering if you need to use DispatcherQueue.
Also, I am not sure for the reasons you referenced DD?

I was incorrect that the dispatcher wasn't working. If you read the rest of the comments, there were other places in reactive ui that depended in INPC. Also, Routing in reactive ui depends on DD and that uses a lot of INPC, too.

Could this be the solution for this inpc issue?
https://github.com/microsoft/microsoft-ui-xaml/issues/1556

We integrated WinUI through another PR, thanks for all the help with this one @limefrogyank


Strip EXIF -- doesn't appear to be working

When using the orientation: true , the GM code also strips the EXIF data.  However, it appears to not strip it.
Is there a way to strip the exif data completely? I added the below code in ImageResizer.js just to force it, however the exif data is still associated to the newly resized/saved images:
            var img = gm(image.data).geometry(this.options.size.toString());
            // if ( "orientation" in this.options ) {
                console.log("strip exif");
                // img = img.autoOrient();
                img = img.strip();
            // }

No errors occur however the EXIF data isn't stripped yet the new images are infact created.

Cancel :) It is :)


add methods for setting more MCF sections

Still need methods for misc properties in:

contact
distribution
metadata
identification

This list of sections to add provided by @empavia

Broad title: <gmd:fileIdentifier>
Ability to add contact information: <gmd:organisationName>, <gmd:positionName>, <gmd:electronicMailAddress>
Optional descriptions of data in MD_DataIdentification: <gmd:alternateTitle>, <gmd:edition>
Purpose section in addition to abstract: <gmd:purpose>
Sections to define constraints in resourceConstraints such as in MD_Constraints and MD_LegalConstraints
Lineage sections: <gmd:lineage>. Need to include lineage information for source data and to describe pre-processing. At a minimum, it would include: <gmd:LI_Lineage>, <gmd:LI_ProcessStep>, and <gmd:LI_ProcessStep>

We added these sections to the best that we could without relying on the ISO19115 2014 updates, which are not yet included in pygeometa.


How to use state parameter for callback

Hi,
As per the example from this module, I saw people use state parameter.
@app.get("/callback")
async def callback(code: str):
    token, refresh_token = await discord.get_access_token(code)
    return {
        "access_token": token,
        "refresh_token": refresh_token
    }

Could you please elaborate? It will be helpful.
Thanks,
@rudSarkar

I implmented a Feature to generate a OAuth2 Link with a State Parameter in #7.
To implement this in a FastAPI Project you probably need a Session System to save State Token(https://discord.com/developers/docs/topics/oauth2#state-and-security) but I'm not sure how implement this correctly.

I am closing this Issue as you implemented the state param.


Issue with installModule for nuxtjs/i18n Failing When Used with Multiple Nuxt Modules

Environment Details

Operating System: Darwin
Node Version: v22.0.0
Nuxt Version: 3.13.2
CLI Version: 3.15.0
Nitro Version: 2.9.7
Package Manager<EMAIL_ADDRESS>Modules & Configurations:

Runtime Modules:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules: None specified.

Steps to Reproduce
Access the reproduction environment on Stackblitz: Stackblitz Project
Bug Description
While configuring nuxtjs/i18n in module.ts as per the official documentation, I used the following setup:
await installModule('@nuxtjs/i18n', {
  langDir: resolve('./lang'),
  locales: [
    { code: 'en', file: resolve('./lang/en.json') },
    { code: 'fr', file: resolve('./lang/fr.json') },
  ]
});

In nuxt.config.ts, when specifying the modules like this:
modules: [
  'module-1', // working
  '@nuxtjs/i18n'
]

it works as expected. However, when arranging the modules in other combinations, the functionality breaks as shown below:

Configuration 1:
modules: [
  'module-1', // working
  'module-2', // not working
  '@nuxtjs/i18n'
]

Configuration 2:
modules: [
  'module-2', // working
  'module-1', // not working
  '@nuxtjs/i18n'
]

Additional Context
I am looking for guidance on achieving the intended configuration or an alternative solution.
Logs
No logs generated in this scenario.

This is main issues I have with installModule, a module really only runs its setup once regardless of how many other modules try to install it using installModule. The same happens for layers installing the same module, but layer options are stored and can be retrieved/merged by the module that's being installed.
@danielroe is there any way to make installModule work when it's called in multiple places? The options are not stored someplace like they are for layers nuxt.options._layers right?

Second module won't be called/merged since the first one will register that module. you should use i18n:registerModule module hook:
https://stackblitz.com/edit/github-dtrh2g-p8sges?file=nuxt.config.ts,node_modules%2F.pnpm%2F%40nuxt%2Bkit%403.13.2_magicast%400.3.5_rollup%404.24.3%2Fnode_modules%2F%40nuxt%2Fkit%2Fdist%2Findex.mjs,package.json,node_modules%2F%40nuxt%2Fkit%2Fdist%2Findex.mjs,modules%2Fmodule-2%2Fmodule.ts,modules%2Fmodule-1%2Fmodule.ts,modules%2Fmodule-2%2Fruntime%2Fcomponent-2.vue

installModule is about installing a module, we cannot use it to "merge" i18n options since the module will be installed only once and next calls will be discarded when Nuxt (Nuxt Kit) installing it and already found.
You can also use layer + module registering i18n module options: i18n will merge them for you.

installModule is about installing a module, we cannot use it to "merge" i18n options since the module will be installed only once and next calls will be discarded when Nuxt (Nuxt Kit) installing it and already found.
You can also use layer + module registering i18n module options: i18n will merge them for you.

I've previously tried your approach and was able to resolve the issue to some extent. However, without installing the @nuxtjs/i18n module, I'm still unable to resolve useI18n in #imports within src/runtime, which remains inconvenient 😅. This limitation also affects testing with Vitest, where useI18n isn’t recognized.
I'm wondering if the @nuxtjs/i18n team might have a solution to address this issue in a way that also supports multi-module compatibility.

What would probably work best for your use case is to use installModule without passing options just to ensure the module is being installed and use the i18n:registerModule to register the locales.

can you check if present in .nuxt/imports.d.ts?

What would probably work best for your use case is to use installModule without passing options just to ensure the module is being installed and use the i18n:registerModule to register the locales.

Thanks I followed your config and it worked and achieved the desired goal 🥰, so I will close this issue


Issues when requiring 'rb-readline'

I was not able to make it work as the README file states.
Looking at the code at:

lib/rb-readline.rb
lib/rbreadline.rb
lib/readline.rb

I have made my way with this fragment of code:
# This will remove old Readline if it's already defined
require 'rb-readline'

# This will load RbReadline for real
require 'rbreadline'

# This will load Readline that relies on RbReadline
require 'readline'

The README says we should do
require 'readline'

I think it is a little confusing to ask someone to require the gem with "another" name.
And the chances of name collision in bug situations are greater.
IMHO, the 'canonical' way of requiring rb-readline should be simple as:
require 'rb-readlne'

What are the maintainers opinion on this?
If you agree I'll try to find some time to offer a patch for that (don't promise, but I can try).
But, if somebody wants to give a try sooner, feel free to do it.

Requiring RbReadline is a tricky business. I've been looking around for my various experiments with loading the library in a clean way, but alas they seem to have disappeared into the ether along with my memory of the problems I encountered.
I do remember that because Readline is part of the stdlib, this is a very awkward library to require. Sorry this isn't more help.


core/finality-grandpa: Gossip synced blocks via neighbor packets

Goal
In order to participate in the Grandpa gossip process a Grandpa voter needs
to know:

Which blocks are relevant for the current round.

Where to retrieve these relevant blocks from.

Grandpa voters already exchange neighbor packets with each other in an
epidemic fashion. This pull request extends the neighbor packet by a set of
block hashes which:

The node has imported.

Have been mentioned by Grandpa prevote or precommit messages.

Within the current Grandpa round.

Listening to these extended neighbor packets a node can pass the information
down to the network sync service enabled through
https://github.com/paritytech/substrate/pull/3633.
This is part two of https://github.com/paritytech/substrate/issues/3629.
Possible (current) solution
In order for a component within core/finality-grandpa/src to be able to
achieve the above goal it would need to:
(1) Receive prevote and precommit messages to keep track of the mentioned
block hashes.
(2) Somehow know whether a given block is available locally (is synced).
(3) Have access to a sink to send out the neighbor packets.
This pull request uses
core/finality-grandpa/src/communication/gossip::GossipValidator as it has
access to (1) and (3) and is already the component sending out
NeighborPackets. We pass down a Client (stripped down version via
BlockStatus trait) through NetworkBridge into the GossipValidator thus
also achieving (2).
Pull request status
For now this pull request is only a draft. I am opening it up at this point
to ease architectural discussions and to ask questions marked with TODO.
What do people think of the approach of extending the GossipValidator? Does
this stretch the responsibilities of a network validator a bit too much given
that the new logic is proactive?

Closing in favor of https://github.com/paritytech/substrate/pull/3800. I will follow up on the comments in future pull requests. Thanks for the help @andresilva.


#87 InlineInterfaceRefactor method refactoring

#87 refactoring

Hi @Vichukano, thank you for raising this pull request. I aim to review this soon.

Great stuff! I've made a couple of formatting tweaks. This is looking good to go.
Thank you for your contribution to Astra, @Vichukano! Merging now.


Remove global functions

There are many functions which accept TTDevice/PCIDevice as a first argument -> These should be moved as private members of these classes.
There are also some functions which should be private members of tt_SiliconDriver.
There are also some functions which can go to a new file "system_utils.h" and be implemented as global there, so that its clear they aren't tied to any specific class.
Revisit all global functions.

Worked on this in several PRs such as #253 #149, but most of what was captured by this issue has been fixed since


Ensure star type assertion is handled properly

Is your feature request related to a problem? Please describe.
Suppose we have a structure declared in the contract:
type (
	UserKey struct {
		Address interop.Hash160
		Data    []byte
		Role    int
	}
)

And suppose we have the following method:
func GetUserKey(ctx storage.Context, idKey []byte) *UserKey {
	data := storage.Get(ctx, idKey)
	if data != nil {
		return std.Deserialize(data.([]byte)).(*UserKey)
	}

	return nil
}

The result of compilation is invalid script:
    compile.go:55:
        	Error Trace:	/home/anna/Documents/GitProjects/nspcc-dev/neo-go/pkg/neotest/compile.go:55
        	            				/home/anna/Documents/GitProjects/nspcc-dev/console/internal/contract/usermanagement/usermanagement_contract_test.go:17
        	            				/home/anna/Documents/GitProjects/nspcc-dev/console/internal/contract/usermanagement/usermanagement_contract_test.go:24
        	Error:      	Received unexpected error:
        	            	using type ANY is incorrect at offset 275

The wrong line is the conversion of structure got after std.Deserialize to Any (which is the pointer to UserKey).
Describe the solution you'd like
The desired result is to convert the structure got after std.Deserialize directly to the UserKey structure if it's not nil. The case is that this code is very similar to the one that we have in our interop package:
https://github.com/nspcc-dev/neo-go/blob/9a270ae30cb889ecbc346efc2efa1d1b6c408abd/pkg/interop/native/ledger/ledger.go#L42-L44
However, the interop API works fine.

Interops are inlined and IIRC there is some special code to handle interop data types, but in general your example should work, so it's just a bug in pointer handling.

The workaround from https://github.com/nspcc-dev/neo-go/issues/3580#issuecomment-2341916070 doesn't really work, btw (nothing is copied, NULLs are left in the struct). Field-by-field assignments do work.


error occurs when i run elg.py

OS: ubuntu 18.04
Python: 3.6.7
Tensorflow: 1.12.0
opencv: 4.0.0
Thanks for your work! When I run the elg_train.py,  it works perfect util step (about) 075000.
when i meet this error, sometimes, it can work continue or just stop after work 4-6 steps. Looking forward to your reply.
`
10/05 16:14 INFO 0083137> heatmaps_mse = 0.00106249, radius_mse = 1.75501e-07
10/05 16:15 INFO 0083139> heatmaps_mse = 0.00100996, radius_mse = 1.32564e-07
Exception in thread preprocess_UnityEyes_4:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib/python3.6/threading.py", line 864, in run
self._target(self._args, **self._kwargs)
File "/home/vision_rd/huhuaidi/project/GazeML/src/core/data_source.py", line 245, in preprocess_job
preprocessed_entry_dict = self.preprocess_entry(raw_entry)
File "/home/vision_rd/huhuaidi/project/GazeML/src/datasources/unityeyes.py", line 238, in preprocess_entry
thickness=int(6line_rand_nums[j + 4]), lineType=cv.LINE_AA)
cv2.error: OpenCV(4.0.0) /io/opencv/modules/imgproc/src/drawing.cpp:1811: error: (-215:Assertion failed) 0 < thickness && thickness <= MAX_THICKNESS in function 'line'
10/05 16:15 INFO 0083141> heatmaps_mse = 0.00116619, radius_mse = 1.57683e-07
10/05 16:15 INFO 0083143> heatmaps_mse = 0.000970301, radius_mse = 1.92502e-07
10/05 16:15 INFO 0083145> heatmaps_mse = 0.00101215, radius_mse = 1.51191e-07
`

Hi could you check if the issue still exists with branch cvlineaug_fix (https://github.com/swook/GazeML/tree/cvlineaug_fix)?
Thanks!

Hi could you check if the issue still exists with branch cvlineaug_fix (https://github.com/swook/GazeML/tree/cvlineaug_fix)?
Thanks!

Hi could you check if the issue still exists with branch cvlineaug_fix (https://github.com/swook/GazeML/tree/cvlineaug_fix)?
Thanks!

Thanks for reply ! I print the value of 6*line_rand_nums[j + 4]。When value < 1, it reports an error。I just change the unityeyes.py（lines 237)
t = int(6*line_rand_nums[j + 4]) if t < 1 :     t = 1 eye = cv.line(eye, (lx0, ly0), (lx1, ly1),                               color=(line_colour, line_colour, line_colour),                               thicknesss = t, lineType = cv.LINE_AA)
Then, it can work without error.  Apparently, your code is more pithy .


CodeQL Fails due to Go Version in Go Mod, for this repo, and many others using go-makefile-maker

I know I've surfaced this issue before, and was told we would continue forward like this. We continue to see variations of issues with it. It might be time to correct? We require CodeQL

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

nevermind, it's just warnings. I thought for a second it became off, if it's just warnings and still running, this isn't valid. Closing.


Fix varnish version in images

See #480; this was a bug caused by Appstream pkg for varnish overriding the custom one from the Package Cloud repo with specific versions resulting in all the Varnish images having the default 4.0.x version from EL 8 rpms.
After these changes, the built images have the correct varnish versions once more:
$ docker run --rm -it wardenenv/varnish:6.5 varnishd -V
varnishd (varnish-6.5.2 revision e7233b0ad2639043341819d19a8d2e418e94ce1b)
Copyright (c) 2006 Verdens Gang AS
Copyright (c) 2006-2020 Varnish Software

$ docker run --rm -it wardenenv/varnish:6.4 varnishd -V
varnishd (varnish-6.4.0 revision 13f137934ec1cf14af66baf7896311115ee35598)
Copyright (c) 2006 Verdens Gang AS
Copyright (c) 2006-2020 Varnish Software AS

$ docker run --rm -it wardenenv/varnish:6.0 varnishd -V
varnishd (varnish-6.0.10 revision 9a7da4ff4c0c824af33e230740a11e99fdca23d9)
Copyright (c) 2006 Verdens Gang AS
Copyright (c) 2006-2021 Varnish Software

$ docker run --rm -it wardenenv/varnish:4.1 varnishd -V
varnishd (varnish-4.1.11 revision 61367ed17d08a9ef80a2d42dc84caef79cdeee7a)
Copyright (c) 2006 Verdens Gang AS
Copyright (c) 2006-2019 Varnish Software AS

@davidalger great! Thank you!

@davidalger I'd like to prepare arm versions of all the images, based on what's available in this repo and in https://github.com/drpayyne/warden-multi-arch
Will you accept it if I'll do a PR?


DarkMode - simplified implementation.

📑 Description
Simplified implementation. Uses dynamic variable instead of querying document for elements.
✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x] All the tests have passed
[x] My pull request is based on the latest commit/version

Thank you for your PR.


Request for Advice

A request for advice rather than an actual issue...
I'm using a Midea M-Thermal ASHP (air-to-water) system in the UK with a separate Midea M-Thermal control unit.  I'd like to read data from that system into Home Assistant via ESPHome via an ESP32 Devkit board.  I came across your project and wondered if I could use or adapt it for my needs.  Only looking to read & display the Modbus data right now as I already have a management system,
After checking that you have a custom board available, but that it has no stock I thought I would load your yaml onto my ESP32 WROOM devkitC and see if I could get any data.  Using pins 1 and 3 for TX/RX I connected into H1 and H2 on the controller,  I then watched to see what comes in, but it seems that regardless I don't get anything useful, mostly command timeouts and CRC check failures. Swapping wires doesn't help and it's even questionable if I'm actually getting ANY real data at all as I've got similar output just running it on my desk unconnected to the Midea controller.
I have tried changing pins used, and also the modbus address from 0x0, 0x1, 0x16 etc.  While I have some coding experience and a little prior use of ESP32 and other micro-controllers I'm totally new to Modbus and have no understanding of UART.
My real questions are:
Can I even use a plain ESP32 to read in data using your config?
Why are there only 2 connections at the controller board (H1/H1) but your yaml references TX, RX and a Modbus pin?
Am I completely misunderstanding the connections into the MCU or the way this functions?
Hope you can find time to help.

I've got the same Heatpump as you, and I also just wired in an existing ESP32, mainly because they don't ship the premade stuff to the UK.
You've most likely got a wiring problem. Can you show how you wired the ESP32 to the RS485 controller, and how you wired that to the heatpump controller?

I think you've already started to answer my question, so thanks.
As a newbie to Modbus, UART I didn't see that I needed to also involve an RS485 controller board.
I'm guessing something like this? : https://www.aliexpress.com/item/1005001621798947.html

Ah, yes. The board this project sells is a custom job which is an ESP32 & RS485 controller on the same PCB.
Any should work, I bought this one from Amazon because it was quicker: https://www.amazon.co.uk/gp/product/B07B667STP/

Great thanks again, I'll get somehing ordered. Any advice then on connections from new RS485 to ESP32 board as they will be separate?  I'll probably also order a couple of larger breadboards anyway to help.

The Amazon one comes with a cable which you can use to easily connect the two boards together via the headers. I used this to help with the pinout: https://embeddedthere.com/how-to-interface-esp32-with-rs485-modbus-sensors-with-example-code/
If you can't get it working let me know and I'll help.
Just make sure you feed each board the correct voltage.

I've got the same Heatpump as you, and I also just wired in an existing ESP32, mainly because they don't ship the premade stuff to the UK.

We do ship to the UK, we’ve already done that a few times.

I think we have the same setup as you described, with an ext- and internal unit. My advise is to do a good check on the power level and if it’s AC or DC. It’s probably AC, but newer units can use DC. Our solution handles both, so if you are into electronics, it should nog be a problem!

Closing the issue, since it seems to be solved

I've got the same Heatpump as you, and I also just wired in an existing ESP32, mainly because they don't ship the premade stuff to the UK.

We do ship to the UK, we’ve already done that a few times.

Apologies for the misinformation @Mosibi, I would have sworn you didn't as I looked at ordering one ages ago! I may still as I've not yet gotten around to making mine look pretty.

Apologies for the misinformation @Mosibi, I would have sworn you didn't as I looked at ordering one ages ago! I may still as I've not yet gotten around to making mine look pretty.

In the very first beginning the UK was indeed not in our "countries-we-ship-to" list, due to very high shipping costs, but luckily we do now for a while with very decent shipping rates

I realise this is a closed issue but I wanted to say I have a basic Midea M-Thermal Heat Pump monitoring solution running now, thanks to @Mosibi and @timmccor for effort and advice.Now I have to tune out noise and figure out how to also drop the occasional out-of-range or false temp readings that sometimes come in.


Allow setting detectBrowserLanguage.cookieDomain by runtimeConfig

Describe the feature
Heyho
Currently we can specify the cookieDomain in the nuxt.config.ts to bake it into the NuxtI18nOptions, or leave it null so it defaults to the host.
Problem
Sometimes, however, if it is used in several different deployments, it may be necessary to adapt the domain for the cookie accordingly. But right now, it is not possible to customise the domain for different deployments.
Solution
Introduce an environment variable, similar to baseUrl, to change the value for cookieDomain during runtime.
Suggestion: NUXT_PUBLIC_I18N_COOKIE_DOMAIN
The icing on the cake would be that the option accepts a function with the context as a parameter that returns either the domain string or null.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide (The contribution guideline of nuxt-modules/i18n is compliant with Nuxt too).
[X] Check existing discussions and issues.

I have worked on runtimeConfig support for the detectBrowserLanguage a while ago (https://github.com/nuxt-modules/i18n/pull/2576) but unfortunately it's blocked until the next nitropack release which includes https://github.com/unjs/nitro/pull/1831. I'll update/sync the feature PR soon, I think the next nitropack release will be sometime this month.

The icing on the cake would be that the option accepts a function with the context as a parameter that returns either the domain string or null.

This wouldn't be possible using runtimeConfig as functions are not serializable, I'm also a bit hesitant to add function support in the options as the same support for baseUrl has been a source of issues, though it's not out of the question if it is a common request.

I have worked on runtimeConfig support for the detectBrowserLanguage a while ago (#2576) ...

Oh awesome. I quickly skimmed through that.
That would be even better than just handling individual options of detectBrowserLanguage.
If I understand correctly, this would allow:
export default defineNuxtConfig({
  runtimeConfig: {
    public: {
      i18n: {
        detectBrowserLanguage: {
          cookieDomain: null,
        },
      },
    },
  },
})

Which makes it overwritable by NUXT_PUBLIC_I18N_DETECT_BROWSER_LANGUAGE_COOKIE_DOMAIN.
I guess?


The bot works with my test server, but not on my Main server

Are both servers hosted on the same hardware/same location?

Does your host allow out-going connections? aka is their firewall blocking outgoing requests..

No, it doesn't. Can I know how to allow it for a VibeGames server?
On Mon, Mar 21, 2022 at 1:08 AM TheONLYGod1 @.***>
wrote:

Does your host allow out-going connections? aka is their firewall blocking
outgoing requests..
—
Reply to this email directly, view it on GitHub
https://github.com/TheONLYGod1/FivemServerStatus/issues/11#issuecomment-1073319807,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUQY3OQBXKRIX5RH7RZKC3VA55EXANCNFSM5QZHWEVA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

You'd have to ask your host to allow outgoing requests