CUDA: Provide wrappers for all libdevice functions, and fix typing of math function (#4618)

Whilst looking into fixing #4618 I also thought it would be good to add wrappers for all libdevice functions, rather than just a subset needed for the currently-implemented math functions. This PR adds wrappers for all libdevice functions. The implementation is based around a data structure containing all relevant details of each libdevice function in numba/cuda/libdevicefuncs.py, from which the stubs, typing, lowering, and documentation are generated.
The changes in this PR are broadly:

Addition of the definitions of libdevice functions in libdevicefuncs.py
Generation of libdevice stubs using the definitions and function in libdevicefuncs.py, which are in libdevice.py
Add declarations and implementations of stub functions based on the definitions in libdevicefuncs.py.

The typing and lowering of functions where only a single output is produced through the return value are straightforward.
The typing and lowering of functions with multiple return values are a little more complex, because the Python function signature needs to be generated such that the return value is a tuple of all output values, and the arguments are all input values. Output values can be distinguished from input values because outputs are passed as pointers (or are the returned value). To generate the implementation, a temporary value needs to be allocated for each output passed as a pointer, and after the call to the libdevice function the return values need to be packed into a struct or array (depending on whether the return type is a Tuple or UniTuple).

Addition of the typing and lowering registries to the CUDA typing and target contexts.
Move all the math module implementations into numba.cuda.mathimpl (instead of being in libdevice). These new implementations make calls to the libdevice stub functions rather than directly generating calls to libdevice functions.
Move the implementation of lowering math.isinf and math.isnan for integer arguments from cudaimpl into mathimpl, where it is more appropriate.
Update tests in test_math. This consists of:

Adds tests of passing integer arguments to all math functions that are typed as accepting integers, and
Using assert_allclose to provide better feedback when a test fails.

Add tests for libdevice functions. The tests consist of:

One compile test per wrapped function generated programatically, and
One test of each different return type that compares the returned values against reference values computed with NumPy.
It isn't practical at present to numerically test every wrapped function - the aim is that these  combinations of tests demonstrate sufficient confidence in the typing and lowering that at least the stub functions, typing, and lowering are correctly passing values to and returning values from the libdevice functions. Note also that the test_math tests are numerically testing the libdevice wrappers for all the functions that math implementations wrap, since the lowering of the math functions now uses the libdevice wrappers.

Inspection shows that these changes make no difference to the generated PTX, with the exception of additional variants being generated for the new tests that are added with this PR.
Fixes #4618.

The last commit I pushed here was an accident - it should have been on another PR. I will revert it when back to a desktop machine (or leave it in if you like, it does correct a small documentation error).

Moving this into 0.52RC as it fixes #4618, which is in 0.52RC. Will attend to the comments and conflicts tomorrow.

@stuartarchibald Could this please have a buildfarm run prior to review, so if there are any issues I can fix them up before review?

BFID: numba_smoketest_cuda_116

The Linux tests that use CUDA 8 are failing on this one. Is it to be expected?

The Windows builds have all glitched and wouldn't start, so I am giving this another spin at: numba_smoketest_cuda_117.

The Linux tests that use CUDA 8 are failing on this one. Is it to be expected?

The merge of #6127 dropped support for CUDA 8 - I would guess it should be removed from the farm now.

@gmarkall CUDA 8.0 has been dropped

Buildfarm ID: numba_smoketest_cuda_120 for 2e3ed5d. It'll potentially need another run post review fixes but thought I'd get it started.

Buildfarm ID: numba_smoketest_cuda_121 for a752f01 .

Buildfarm ID: numba_smoketest_cuda_121 for a752f01 .

This passed.


Define -march string

This is essentially a collection of all recent change requests for the -march string:

Define the -march string, as it is not exactly the RISC-V ISA naming string, and highlight the lower-case requirement (https://gcc.gnu.org/pipermail/gcc-patches/2022-November/605249.html)
Relax the ISA string order (https://github.com/riscv-non-isa/riscv-toolchain-conventions/issues/11 and https://github.com/riscv-non-isa/riscv-toolchain-conventions/pull/14)
Add custom extensions (https://github.com/riscv-non-isa/riscv-toolchain-conventions/issues/1)
Add profiles support (https://lists.riscv.org/g/sig-toolchains/message/379 and https://github.com/riscv/riscv-profiles/pull/78)

Most of this patch is based on proposals and inputs from Kito and Palmer (see linked resources above).
It might not be perfect, but it could serve as a reasonable initial draft for further discussions.
Comments are welcome!
CC: @palmer-dabbelt @kito-cheng @asb @jrtc27

@palmer-dabbelt commented the following on the GCC mailing list:

IMO trying to handle this with another RISC-V spec is a waste of time:
we've spent many years trying to follow the specs here, it's pretty
clear they're just not meant to be read in that level of detail.  This
sort of problem is all over the place in RISC-V land, moving to a
different spec doesn't fix the problem.

As I wrote in my reply message, I think -march should be documented here (as it is already) because this repo is the common place to document things like that for GCC and LLVM.
Also, I don't think a common -march documentation/specification is a waste of time. But of course, I don't consider my proposal perfect. Technical comments will help to improve it. If someone else wants to drive this issue instead, feel free to do so.

Thank you for the work.
Trying to understand how the interpretation works so maybe I can help on the LLVM side. Is there an existing table for interpretation(s) to the ISA? If so this will help me clarify the confusion that I have to both "dates" and "version number" coexists (based on this PR - 2.2 and 20190608) as an identifier to an interpretation.

Trying to understand how the interpretation works so maybe I can help on the LLVM side. Is there an existing table for interpretation(s) to the ISA? If so this will help me clarify the confusion that I have to both "dates" and "version number" coexists (based on this PR - 2.2 and 20190608) as an identifier to an interpretation.

I tried to cover exactly this by the amount of examples included in the PR.
If there are unclear cases, then I think we should list them in this document anyway.
Regarding the ISA versions: we will just have to support all existing versioning schemes: so versions are well-ordered, but not lexicographic sortable, strings.

I've updated the PR according to the provided feedback.
Thanks for the comments!

Kito mentioned that there should be more details regarding the versioning of extensions. I've added some details about that to the PR.

We should have -march=isa+ext for completeness (yes, _ works in that case, but then you have to care about whether it’s an ISA or profile as your “base”). We also need -march=+foo so you can add to whatever the default is (whether that’s from a flag in user-provided CFLAGS or the system default when not specified). Maybe also -march=-foo, though that gets a bit weird (but that already needs to work in the assembler due to .option arch, so give it the same semantics).

I also worry about what happens if a user gives CFLAGS=“-march=isa -misa-spec=…” and a build system then adds -march=profile given you’ve said it’s an error to combine those.

I do worry that profile strings might be rather cryptic and easily confusable (by humans) with ISA naming strings. If it's agreed this is problematic, it's not clear if it's best handled in the profiles spec or in how the profile names are represented in -march arguments.
i2020u32 is cryptic but perhaps less easily confusable than rvi20u32. As we get closer to 2032, rvi32u32 vs the ISA naming string rv32i... seems potentially confusing. As Andrew argues in https://github.com/riscv/riscv-profiles/issues/82 it's possible that the confusability won't matter as almost nobody is using ISA naming strings.

As mentioned in the profiles ticket, would it be ok, to mention in a note that tools/compilers may impose restrictions about which march string format is accepted (e.g. via prefix or cmdln flag) and that users may look in to the compiler manual for details? So we would just define the formats here and leave some room for further restrictions for tools/compilers (if any).

We should have -march=isa+ext for completeness (yes, _ works in that case, but then you have to care about whether it’s an ISA or profile as your “base”). We also need -march=+foo so you can add to whatever the default is (whether that’s from a flag in user-provided CFLAGS or the system default when not specified). Maybe also -march=-foo, though that gets a bit weird (but that already needs to work in the assembler due to .option arch, so give it the same semantics).

I'm not sure if we should define this here (unless enough people want this). The idea is good, but it feels a bit like an added-value feature of a tool/compiler.

I also worry about what happens if a user gives CFLAGS=“-march=isa -misa-spec=…” and a build system then adds -march=profile given you’ve said it’s an error to combine those.

Same for this. I would let this behavior be defined by the tool as I fear that different tools prefer different behavior (e.g. accepting the last provided flag vs. throwing an error vs. ./configure flag to define the policy...).
I would prefer to restrict the description to a minimum that's usable by a user. Compilers and tools can then derive from here and extend more features.
Would this be ok?

We should have -march=isa+ext for completeness (yes, _ works in that case, but then you have to care about whether it’s an ISA or profile as your “base”). We also need -march=+foo so you can add to whatever the default is (whether that’s from a flag in user-provided CFLAGS or the system default when not specified). Maybe also -march=-foo, though that gets a bit weird (but that already needs to work in the assembler due to .option arch, so give it the same semantics).

I'm not sure if we should define this here (unless enough people want this). The idea is good, but it feels a bit like an added-value feature of a tool/compiler.

I also worry about what happens if a user gives CFLAGS=“-march=isa -misa-spec=…” and a build system then adds -march=profile given you’ve said it’s an error to combine those.

Same for this. I would let this behavior be defined by the tool as I fear that different tools prefer different behavior (e.g. accepting the last provided flag vs. throwing an error vs. ./configure flag to define the policy...).
I would prefer to restrict the description to a minimum that's usable by a user. Compilers and tools can then derive from here and extend more features.
Would this be ok?

If GCC and LLVM do different things for any of these it will be a pain point, so no, I think they really do need to be documented conventions. Otherwise either nobody will use them or they'll use them and be locked into one compiler for no good reason.

Following content has send in another private mail thread, but I think it could be also post here:

I agree the naming of profile can be improved to prevent the confusion, but actually I am less concern about the confusion on the -march string in practice.
Consider the following two cases (and use the most confused possible
name rvi32u32):

-march= with profile name only without extra extension
-march=rvi32u32  - the possible and legal misspell version should
be rv32iu32 which is rv32i base plus U extension with version 32,
that's generally a useless configuration, should got error soon when
trying to build/compile/assemble anything.

-march= with profile name only with extra extensions
-march=rvi32u32+xsffoo profile rvi32u32 with xsffoo vendor
extension, and that's apparently a profile based arch string, since
there is a plus sign there, it's only used in profile style -march
string
Arguable point might has 2:
first is plus sign not included in the ISA string now, but
used/defined in future

Low probability, but I guess we could...try to stop when that happened.
second is we might also want to accept  the plus sign on the
normal ISA-string-like -march option.
This could be prevent when we relax the canonical order
requirement of -march string

So based on the above two situations I think that should not be
confusing (or misuse with misspellings) in practice.

In today's Toolchain SIG call, we agreed to split this PR into multiple ones not to mix different issues and to move this forward:

document current -march string
document -march string relaxation
add custom extensions
add profiles support


parallax images not always filling container

the parallax images dont seem to load correctly on init.
on scroll or resize it gets fixed.
im using angular and init everything after the dom is rendered.
the images are way too big for the page, so that may be the issue
here's an image:

should look like

live:
http://<IP_ADDRESS>/s/posts/2
also there's an issue with mousewheel scrolling, but i think there's already a few issues open regarding that.

it looks like the parallax handler does not get refreshed when you switch tabs.
because of this the image in the 'new' tab has the position of the image in the 'old' tab.
firing a reflow with
$(window).scroll();

in the click handler just before the
e.preventDefauld()

in tabs.js (line 103) seems to do the job

We are currently reworking parallax so hopefully in the next release it will be much better with less bugs


eap73-rhel84-standalone-byos-patch3

Fixed the storage kind issue. Now user can select "StorageV1 or StorageV2" as per desire.

Need to resolve conflict above


Test helper for permitted_attributes method

Hi guys!. I was looking for a test helper for permitted_attributes method in my policies in different contexts, but I haven't found anything. Is there a way to do it or is it a good idea to write something?

I don't know what you mean by this, could you elaborate on your question?


Local assets do not seem to be considered

Hello,
I'm delighted to find out there is a possibility to use the academic theme without having to give away the privacy of my visitors.
However, after installing academic-admin and running academic import --assets, my extension Privacy Badger tells me there still are trackers on my website.
What could have gone wrong?
Thanks

Depending on which features (and thus assets) you are using, your specific set of attributes may not be fully downloaded as the community has not completed the implemention of this tool for all assets. Please refer to the open issues and consider contributing.


build.gradle fixes

Related #13, #15
I forgot one tiny detail for #13 which I fixed in this PR.

I fixed the build error in the latest commit. Rebase your branch and it should pass.

oh crap i just added my changes in :/
time to drop them

This is a go from my side @Flibio


Install SAP NetWeaver in openSUSE

The documentation in some of the steps is not clear.
For example when extracting the rar archive it says
"All files must be stored in the same folder. As this is a multipart archive, you only need to start the extraction process for the first file sap_netweaver_as_abap_750_sp02_ase_dev_edition.part1.rar – the other files will be extracted automatically."
What does "All files must be stored in the same folder" means? Should the be extracted without subdirectories so all files is stored in one map and not in the structure they archived in !!
On mac there is no winrar that extracts all rar files. If you extract each rar file in same catalog it complains that files exists.
Also strange is that all rar files seems to have same files in the
server/tar/x86_64
The extraction rar files maybe seems to be an easy step but it would be good if this step could be explained more exact

@bjwo Can you address?

@carweb99 Thank you for your question. Unfortunately, I don't use a mac, but I was told that you can use the "Unarchiver" tool to extract rar archives on mac.
All downloaded files should be stored in the same folder, then you can start the extraction process for the multipart archive.
Kind regards
Björn


open-source-hack

this issue doesn't match any issues in github (open or closed). This will not be helpful to implement.


Repair cart page including reinstated course api improvements

What are the relevant tickets?
https://github.com/mitodl/mitxonline/issues/1876
https://github.com/mitodl/mitxonline/issues/1869
Description (What does it do?)

Reinstates: https://github.com/mitodl/mitxonline/pull/1872 & https://github.com/mitodl/mitxonline/pull/1877
Fixes an issue when loading the cart page (https://mitxonline.mit.edu/cart/) with an item in the basket.  Issue ref: https://mit-office-of-digital-learning.sentry.io/issues/4472765075/?alert_rule_id=8724694&alert_type=issue&notification_uuid=2bbf21e8-fcf4-46ec-944a-9cc1c91f3c80&project=5864687&referrer=slack

How can this be tested?

Enroll into a course and proceed with certificate enrollment.
When redirected to the cart page, ensure that the cart page loads as expected and there are no errors in the browser console.
Through http://mitxonline.odl.local:8013/cms/, add a flexible price child page to the course page for the course you're enrolled in.
Edit the course page and add the URL of the flexible price page to the URL field under the "price details" section.
Refresh the cart page and ensure that you see a "Need financial assistance?" link.

Additional Context
This was caused because I was passing a course to isFinancialAssistanceAvailable, here: https://github.com/mitodl/mitxonline/blob/c87bdc15e75c6bae728749dedd5a91cae3755c8d/frontend/public/src/containers/pages/checkout/CartPage.js#L141C15-L141C15

Are there no tests for this code? I would have expected this to be caught by a test somewhere.
More forward-thinking, would type checking have helped?

Are there no tests for this code? I would have expected this to be caught by a test somewhere.
More forward-thinking, would type checking have helped?

@pdpinch There are no tests for the cart page.  Typing existed but did not help here.  I can add tests but I don't think it will delay this by a day.


Email for OPMDG group?

Is there a front email address to the whole OPMDG group?
The Debian packaging process would like to have one as upstream address although it does seem to be compulsory.

There's no email for the group, we only get contacted through the repositories.


alert block

Fix #126
If your PR introduces a new block/black varaint, please update the block library https://main--stewart--hlxsites.hlx.page/tools/sidekick/library/library.html
Test URLs:

Before: https://main--stewart--hlxsites.hlx.live/en/customer-type/international-title-services
After: https://alert--stewart--hlxsites.hlx.live/en/customer-type/international-title-services

@badvision quite a bit of iteration here. most of that was me improving the fallback font and font loading. finally got it to something I'm happy with.


DS-3542 Spring security authorizations

Work in progress PR to integrate Spring Security authorizations checks with the new DSpace 7 REST API.
THIS IS NOT READY YET!

For now I put a most of the EPersonRepository integration tests on ignore cause it seems that an EPerson (according to the AuthorizeService) doesn't get READ access to itself. This might be something we should look into.

Closed in favor of https://github.com/DSpace/DSpace/pull/1976


Avoid NPE on packet logging when using xmpp bosh connection.

How does this prevent an NPE?

When connecting using bosh there is no socket, so we cannot obtain the address and we pass null as local and remote address and packet logger throws NPE.
There was a case handling that (new byte[4]), but not for all cases ...


kafka-Serializer/Deserializer config not work if i put it in boostrap.yml,but worked in application.yml

kafka-Serializer/Deserializer config not work if i put it in boostrap.yml,but worked in application.yml
It's very confusing,because other properties all effective except the serde properties.
eg:
spring.kafka.consumer.key-deserializer=org.apache.kafka.common.serialization.StringDeserializer
spring.kafka.consumer.value-deserializer=org.apache.kafka.common.serialization.StringDeserializer
spring.kafka.producer.key-serializer=org.apache.kafka.common.serialization.StringSerializer
spring.kafka.producer.value-serializer=org.apache.kafka.common.serialization.StringSerializer
test case:
https://github.com/celesky/spring-cloud-demo/tree/master/bus-kafka
just run it to reproduce this ~
thanks~

I think you should report this in https://github.com/spring-projects/spring-kafka.  If they determine it is really a problem  with Spring Cloud than we can reopen this.


fix: tests and styles

port to python >=3.8
deprecate <3.8

@afunTW I was just trying to fetch the changes from @a5r0n 's branch hahaha
hi @a5r0n would you be ok to discuss here?

@bluet  i'm not really in the context, it's been a while, but let's give it a try

@a5r0n I revise the CI flow in https://github.com/bluet/proxybroker2/pull/79 and https://github.com/bluet/proxybroker2/pull/80, would you like to rebase and trigger the latest CI again?

Since the PR mainly revise the asyncio break change https://docs.python.org/3/library/asyncio-queue.html#asyncio.Queue, and has been test. Will merge for further maintainese.

Awesome 👍


Replies in E2E rooms sent from riot-web are consistently misformatted

with the reply appended to the quoted text rather than being separated by the end of the blockquote.

there should be a linebreak and unindent between 'enabled?' and 'it should be.

Event body:
{
  "room_id": "!something:matrix.org",
  "type": "m.room.message",
  "content": {
    "msgtype": "m.text",
    "format": "org.matrix.custom.html",
    "body": "> <@Giom:matrix.org> Michael (Delph): is the gdpr enabled?\n\nIt should be but I'm having problems testing it - I haven't got the riot app telling me I need to hit it and I'm having problems joining rooms",
    "formatted_body": "<mx-reply>
In reply to @Giom:matrix.org
Michael (Delph): is the gdpr enabled?
</mx-reply>It should be but I'm having problems testing it - I haven't got the riot app telling me I need to hit it and I'm having problems joining rooms"
  }
}```

Issue fixed by https://github.com/matrix-org/matrix-ios-sdk/pull/534. For encrypted event m.relates_to was not reflected to clear content and so not interpreted as a reply.


Add feature to Lint .editorconfig file

To make the maintainer's life easier, I...

[X] created at least 1 spec to cover my changes,
[X] npm tested my code and it's green like an :green_apple:,
[X] adjusted the readme.md, if it was necessary and
[X] would like to receive get a :beer: or :coffee: after that hard work!

This pull request adds linting feature when editing .editorconfig file.
It solves issue #141

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#141: Notify user about malformed editorconfig-files

IssueHunt has been backed by the following sponsors. Become a sponsor

This is what it looks like in action:

Personally, I'd avoid making linter a package dependency. Simply providing the service should be enough: if users have linter installed, they'll see messages whilst editing .editorconfig files. Those who don't are greeted with a nag screen at startup asking them to install linter, intentions, linter-ui-default, and busy-signal. For a package that only uses the .editorconfig format indirectly, it's pretty overkill and even a little disruptive.
I recommend the following changes. Do this and we can merge. 👍 Don't forget to regenerate the package-lock.json file (run npm i --package-lock-only to update it manually if you need to).
diff --git a/index.js b/index.js
index 1c85cf4..e182e82 100644
--- a/index.js
+++ b/index.js
@@ -388,8 +388,6 @@ module.exports = {
 			this.disposables.dispose();
 		}

-		require('atom-package-deps').install('editorconfig');
-
 		this.disposables = new CompositeDisposable(
 			atom.commands.add('atom-workspace', {
 				'EditorConfig:fix-file': () => {
diff --git a/package.json b/package.json
index 429035e..d420f9c 100644
--- a/package.json
+++ b/package.json
@@ -27,17 +27,12 @@
 	"atomTestRunner": "atom-mocha",
 	"dependencies": {
 		"editorconfig": "^0.15.3",
-		"mapped-disposable": "^1.0.3",
-		"atom-linter": "^10.0.0",
-		"atom-package-deps": "^5.0.0"
+		"mapped-disposable": "^1.0.3"
 	},
 	"devDependencies": {
 		"atom-mocha": "^2.2.0",
 		"xo": "^0.24.0"
 	},
-	"package-deps": [
-		"linter"
-	],
 	"xo": {
 		"globals": [
 			"AtomMocha",


Sharing global object

The documentation shows how to share data between pages:
// In the main process.
global.sharedObject = {
  someProperty: 'default value'
}

but with Typescript version, we get this error:

error TS2339: Property 'sharedObject' does not exist on type 'Global'.

interface SharedObject {
  someProperty: string;
};

declare const sharedObject: SharedObject = { ... }

Does it work for you?

I tried but it doesn't work, it doesn't get the data.

Here a repo where I replicate my error and @theJian solution, but unfortunately not working
https://github.com/Leonardonline/electron-quick-start-typescript

I solved using:
(<any>global).sharedObject = {
  someProperty: string
};

@Leonardonline Thank you. I faced the same issue and your solution helped!

declare namespace global{     let settingsService: SettingsService }
So you can write global.settingService

Create a global.d.ts file in your project.
declare namespace NodeJS{
    export interface Global {
        myVar: number
    }
}

This will merge the base NodeJS declarations with yours, so you can write global.myVar in your code. I couldn't figure out how to specify existing interfeces, so I use : any for my global settingsService

fake...
In the global.d.ts:
declare namespace NodeJS {   export interface Global {     certificateInfo: { trusted: boolean; url: string };   } }
console.log(global.certificateInfo); // got undefined


[TW#20007] What does this error means? core: E () WIFI: TIMQ_NUL

Sometimes I get this error when the ESP32 is trying to connect to a WiFi network, I'm using IDF v2.1.1. After this error code is received the internal WiFi task stops calling event_handler which results in it stops trying to reconnect.

Hi @ali-slim-sst it means there timer Queue is full. In IDF v3.0, the timer is re-designed. Is it possible for you to switch to idfv3.0?

@ali-slim-sst This sounds like the same problem I am experiencing in this thread: https://esp32.com/viewtopic.php?f=2&t=5183 and in this issue: https://github.com/espressif/esp-idf/issues/1797
In short, sometimes SYSTEM_EVENT_STA_DISCONNECTED does not appear, even though esp_wifi_connect() returns ESP_OK.

@XinDeng11 please help to check it.

Hi @ali-slim-sst @vonnieda， there are one case may cause the issue, you are in an environment where have more than two same SSID APs and one of them is you try to connect. However, I am not sure whether it is the scenario I guess, so can you give me more information like the config parameter you set and your environment when you meet the issue, thanks.

Hi,@ali-slim-sst, I mean I need the scan type you set when set station config, because when in all channel scan type, we will scan all match SSID APs information in the list, and try with the best one first, if it fail to connect and we found there are still others in the list, we wouldn't send disconnect while try to connect with the others, and put the one which fail to connect to the blacklist until we connect success or all the APs in the list fail to connect, so I hope to know whether you set to scan all the channel before connect

Hi @XinDeng11 , I'm not setting scan_method as in the WiFi connection example, I'm just setting SSID and password.
So you recommend using WIFI_ALL_CHANNEL_SCAN to overcome the multiple SSID's issue?
Thanks!

@ali-slim-sst No, All channel scan is used to scan the best AP in a multiple SSID's environment. However, only when you in this config may cause don't send disconnect when not connect with the AP, may be you can offer the logs for us when you meet the issue again

@XinDeng11 unfortunately I cannot offer any logs, the test device is on a truck so I cannot log through serial. Is there a way monitor and recover the WiFi task?
Currently I have a separate task to monitor the WiFi while its disconnected from the AP and forcing a software reset if SYSTEM_EVENT_STA_DISCONNECTED not called within a set time frame. However this is bad for my application I prefer more elegant error recovery that doesn't involve reset.

Hi@ali-slim-sst, I found there might be one case like yours, but I need make sure whether you invoked scan API when the STA disconnect and was trying to reconnect?

Hi @ali-slim-sst ,
Is there any update? Could you provide more information to help debug this issue?
Thanks.

Hi @FayeY
I will do more testing in the next few weeks and update this issue.

Any update?

@FayeY sorry I didn't have time to work on that yet.

Closing this, please feel free to reopen if this is still an issue after you testing it and you can supply more details. Thanks.


Security headers are missing

Bug Report
Few security-related headers are missing
Issue Description
CSP policies headers are not here

Missing Headers

Content-Security-PolicyContent Security Policy is an effective measure to protect your site from XSS attacks. By whitelisting sources of approved content, you can prevent the browser from loading malicious assets.Permissions-PolicyPermissions Policy is a new header that allows a site to control which features and APIs can be used in the browser.

Bagisto Version
Latest version
Steps To Reproduce
Generate demo via https://demo.bagisto.com/ and
paste link into these sites
https://cspvalidator.org/
https://securityheaders.com/
Actual Result
Headers are not there

Expected Result
Headers must be there

Hi there,
Yes, the CSP header is missing because it impacts the whole content of the homepage, and I am also aware of that.
I will add this but needs to check the content also as this will block all the dynamic contents mostly.

Hi @AkashThoriya,
Yes, the CSP header is missing because it impacts the whole content of the homepage, and I am also aware of that.
I will add this but needs to check the content also as this will block all the dynamic contents mostly.

Yes, you are right, it impacts the whole content of the homepage.


Include styles needed for stacked icons

Save the user the time of checking the example project to find exactly how to setup the stacked icons.

Thanks @mgmcdermott, merging now


Type annotation on AsyncRunner.run is wrong

onDone does not get an error argument

ah, thanks. I forgot I had an issue!


Make gitbase suitable to run over PGA

Parent objective: #8
Progress: 2/4

[ ] [P0] Make gitbase better suited for rooted repositories
[x] [P0] Make gitbase queries on PGA not crash
[ ] [P1] Instrument gitbase to be able to trace performance
[x] [P1] Improve performance of gitbase over PGA

[X] [P0] Make gitbase better suited for rooted repositories
We can close this one, because now is better suited for rooted repos than 4 months ago.

[X] [P0] Make gitbase queries on PGA not crash
Queries sover PGA will not crash. We added a flag to gitbase to be able to continue if we encounter some git error.

I assume that basic instrumentation is already there, and that performance improved a lot with indexes.

The main objective is not done yet. Should we re-define it again with other more accurate bullet points or keep that here and add new points to the actual ones?

@ajnavarro We have defined another gitbase performance improvement objective for Q3.
@eiso What do you think?

Closing it as done. Opening new gitbase performance improvements OKRs for Q3.


feat: Added publishing of ndeath message on disconnect.

Summary
Publishes a NDEATH message on disconnect.
Why?
The specification states that: "[tck-id-payloads-ndeath-will-message-publisher] An NDEATH message SHOULD be published
by the Edge Node before it intentionally disconnects from the MQTT Server."
Note: I removed the _setup_will() call in the edge node class init, since the self._bd_seq_metric would be off by one when retrieved in the disconnect function.

Merged this into 0.4.0 just now


only parse json files once

Currently we do it once to generate the by_month pages, and then again to generate the atom feed. This is inefficient...

We need to parse all the json file to generate all the html pages, and when we re-parse them again we only look at the last 100 entries, so we don't really care


Remove rubocop

But consistent style!

I'm open to the community's opinion on this one.
In my view rubocop provides little value and significant annoyance. I'd rather the code be clear, concise and expressive than consistent (though consistency is good where it doesn't detract from those other characteristics). I believe these values are better enforce by humans than machines and PR comments are a great vehicle for doing just that.

@pezra as you see fit! I think computers are the perfect entities to nitpick about consistent code style, but that’s just me : )


Starting 1231-domain image fails wih Failed to deploy library Extension-Name: jax-rs, Specification-Version: 2, Implementation-Version: <IP_ADDRESS>, because of conflicting library Manifest values, and library information registered with the server: [Implementation-Version: <IP_ADDRESS> vs. 2.5.1].

Hi,
I'm trying to build simple single server image using https://github.com/oracle/docker-images/tree/master/OracleWebLogic/samples/1213-domain , I've change the dockerfile to use the following image FROM store/oracle/weblogic:<IP_ADDRESS>
Build completes, but during runtime I get following exception and cannot access admin console.
Any hints how to resolve it?
Regards
Greg
<Nov 21, 2019 9:46:44,796 AM GMT>    <Failed to initialize the application "jax-rs [LibSpecVersion=2.0,LibImplVersion=2.5.1]" due to error weblogic.app
lication.library.LibraryDeploymentException: [J2EE:160145]Failed to deploy library Extension-Name: jax-rs, Specification-Version: 2, Implementation-Version: <IP_ADDRESS>, because of conflicting library Manifest values, and library information registered with the server: [Implementation-Version: <IP_ADDRESS> vs. 2.5.1]. Check the library MANIFEST.MF file and correct
version information there to match server settings, or undeploy the misconfigured library.
weblogic.application.library.LibraryDeploymentException: [J2EE:160145]Failed to deploy library Extension-Name: jax-rs, Specification-Version: 2, Implementation-Version: <IP_ADDRESS>, b
ecause of conflicting library Manifest values, and library information registered with the server: [Implementation-Version: <IP_ADDRESS> vs. 2.5.1]. Check the library MANIFEST.MF file
and correct version information there to match server settings, or undeploy the misconfigured library.
at weblogic.application.internal.library.LibraryDeploymentFactory.getLibData(LibraryDeploymentFactory.java:129)
at weblogic.application.internal.library.LibraryDeploymentFactory.createDeployment(LibraryDeploymentFactory.java:50)
at weblogic.application.internal.DeploymentManagerImpl.createDeployment(DeploymentManagerImpl.java:195)
at weblogic.application.internal.DeploymentManagerImpl.access$800(DeploymentManagerImpl.java:61)
at weblogic.application.internal.DeploymentManagerImpl$DeploymentCreatorImpl.createDeployment(DeploymentManagerImpl.java:628)
Truncated. see log file for complete stacktrace

Please use the https://github.com/oracle/docker-images/tree/master/OracleWebLogic/samples/12213-domain to create the domain, 12.1.3 is not a supported version of WebLogic.


Upgrade to Angular 8

Blog post about the new release: https://blog.angular.io/version-8-of-angular-smaller-bundles-cli-apis-and-alignment-with-the-ecosystem-af0261112a27

Done.


Execution in docker image issue vgg16_bn.py

Hey, so I got this working on my mac pretty easily. However I just can't seem to get it to work on a docker image.
It seems to crash when executing this line https://github.com/clovaai/CRAFT-pytorch/blob/master/basenet/vgg16_bn.py#L61
Tried to debug through, but don't really have any experience with pdb so got nowhere there. I suspect my Dockerfile isn't setup right. Did anyone manage to get this working or have similar issues? My dockerfile is below.

RUN pip install -r requirements.txt
COPY . .

CMD ["python", "test.py"]

Do you mind sharing your Dockerfile setup for this?

@loopedice. Sure, it's extremely simple:
FROM python:3.6.8

RUN apt-get -y update
RUN apt-get install -y --fix-missing \
    wget \
    nano \
    htop \
    tmux \
    unzip \
    zip \
    && apt-get clean && rm -rf /tmp/* /var/tmp/*

COPY . /code
WORKDIR /code
RUN pip install -r requirements.txt

CMD python server.py

@lamhoangtung Thanks for that. It still seems to be failing. I will try again later tonight on my windows machine with docker, and see if that doesn't produce a crash.

@lamhoangtung Thanks for that. It still seems to be failing. I will try again later tonight on my windows machine with docker, and see if that doesn't produce a crash like my mac.

Both Mac and linux works fine for me :P

It's so strange 😬 Maybe my docker config has limited resources?

Resolved my issue. Problem was that I was using 0.4.1 and not 0.4.1.post2. So I just went to 1.0.0 due to errors in pip not finding 0.4.1.post2


Add Routes Protection

fixes #79
This PR applies protecting routes by unauthenticated users for security reasons.
https://github.com/user-attachments/assets/2929831e-3b23-4004-ac15-fbe7f8153d13

hey @gupta-ritik please review

hey @gupta-ritik please finalize this PR!!

resolve the conflicts @haseebzaki-07

hey @gupta-ritik resolved the conflicts


Option to not allow guest registrations for event

The organizers of this event need to email all attendees, but many of them registered as guests and now the system doesn't have their email. It's pretty common for events to need to be able to contact all attendees. We need to give organizers control over whether they accept registrations from guest users.

Hmmm I was wrong, even if they registered as guests it should have collected their email. So why are these attendees not receiving the email notification of the update?

It's strange, all these people got registered as BACKER rather than ATTENDEE.
However, looking at Mailgun's logs, it looks that the number of emails and the addresses they were sent to match what we expect. To better investigate, I would need an example of an email that did not get the notification where it should have.


CLI Timing out and failing despite Sandbox being completed

Summary
CLI Timing out and failing despite Sandbox being completed
Steps To Reproduce:
Create a sandbox using the org create command
Expected result
Sandbox is authenticated against the CLI
Actual result
Errors out after sitting in completed for hours

Additional information
Feel free to attach a screenshot.
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):
Latest
SFDX plugin Version(to find the version of the CLI plugin run sfdx plugins --core)
Latest
OS and version
Linux

The error is unfortunate, but it does work (run org:status with -s and -a to finish getting it hooked up)

agreed and we added that retry but its still problematic cause the CI/CD just sits and waits for hours since it thinks it will never finish

@MGarfOppLoans My team is investigating ways in which we can fix the issue. Can you please try the workaround mentioned by @mshanemc and tell us if it authorizes the sandbox to the CLI? Thank you.

We added the retry and sometimes it doesn't work. It seems to be way more stable this past month or so

closing this now that org create sandbox and org resume sandbox are available.


Apple Silicon Not Supported

Mac M1 chipset is not supported due to dependency on Shapely.

Works on my side with M1 Pro chipset.

@Jurisak Awesome! Which version of Shapely are you using?

Thanks @Jurisak - Looks like Shapely fixed it in >=1.8.1 -- PR incoming

It was working with shapely 1.8.0 on my side.
BTW I am running it on python 3.10. You can include it in supported versions.

Nice. I think Shapely was the main requirement that was causing us from supporting 3.10, but it looks like they have fixed it on their end. I just tried on Windows with 3.10.1 and was able to run it as well.

Python 3.10 is now officially supported!


[APIM][3.1.0] A null pointer is thrown when getting a user access token

Description:
After the fix #2179, when a federated claim with no local claim mapping is present as user attributes, a null pointer is thrown when generating a user access token.
This happens because the org.wso2.carbon.apimgt.keymgt.issuers.RoleBasedScopesIssuer#getRolesFromUserAttribute  method is trying to get the claimUri from the local claim, even when the local claim is null.

Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Fixed with https://github.com/wso2-extensions/apim-km-wso2is/pull/32


Can Presto SQL be converted to Hive SQL

Can Presto SQL be converted to Hive SQL？

It is work in progress. Please check out the trino2rel package.


bug(cicd): Change to config repo does not trigger the AWSAccelerator-Pipeline

Describe the bug
For any change to be deploy after a commit pushed to the aws-accelerator-config codecommit repo I have to click on "Release Change"  in console for them to be applied.
To Reproduce
Push updates to the config repo. Nothing will happen in pipeline
Expected behavior
Automatic trigger of AWSAccelerator-Pipeline on aws-accelerator-config codecommit repo commits.
Please complete the following information about the solution:

[x] Version: v1.3.0

[x] Region: eu-west-1

[x] Was the solution modified from the version published on this repository? No

[x] If the answer to the previous question was yes, are the changes available on GitHub? -

[x] Have you checked your service quotas for the sevices this solution uses? Yes

[x] Were there any errors in the CloudWatch Logs? No

Hi @flochaz!
Hope you're doing well.
Thank you for submitting an issue for the LZA. At the moment this is the desired configuration, as many customers prefer to make several, small changes to their configuration repo before triggering the pipeline. Due to the broad permission set which the LZA pipeline has to modify any and all resources across an AWS Organization, the triggering of the pipeline should be a deliberate action.
I think there is a valid argument to make a configurable parameter to automatically trigger the pipeline after new commits, however that may not outweigh the risk of customers unintentionally triggering their LZA pipeline and deploying undesired changes.
If you would like to enable this behavior, you can manually update the Code Pipeline in the AWS Code Pipeline console by doing the following:

Go to the Accelerator Pipeline (not installer).
Edit the Pipeline
Edit the Source stage of the Pipeline
For the Configuration Action Group, point to the proper Code Commit Repo/branch and select 'Done'

I have also included a link to the documentation here:
https://docs.aws.amazon.com/codepipeline/latest/userguide/update-change-detection.html
Since this is currently our intended behavior, I'll go ahead and resolve this issue. Please reach out if you have any further questions or concerns.
Cheers!


Reset exponential backoff after a bump

Describe the bug
Sometimes users will overpay fee because we're not resetting exponential backoff properly after a bump
To Reproduce
Steps to reproduce the behavior:

Go to 'https://connextscan.io/tx/0xfd674bf7bfe0e4b2b813e7136a3b06c1eb250b9f8ebd567db6d9efe5b3f507e7'
The origin sender bumped fee several times and relayerFee paid is more than 6.5 MATIC for arbitrum tx fee.
About 0.5 MATIC should be enough but the user would try to bump it until next execution time is up.

** How to fix **
We should reset a backoff after a bump

done


Remove ASEOpenMMSimulation from examples notebooks

This PR alters the following tutorial notebooks in the examples directory so that they use the ASESimulation class in place of the ASEOpenMMSimulation class when setting up ASE simulations using and OpenMM calculator:

ase_openmm_graphene.ipynb
ase_openmm_nanotube.ipynb

It also fixes the ase_openmm_neuraminidase.ipynb notebook, which was not running properly before.
I have tested these changes locally on my computer, and the notebooks seem to run normally. I have also tested that you can connect to the servers in VR and see what you expect to see using the standalone APK, and everything seems to behave as expected. I have not tested this with PC-VR (as I am unable to), so if someone wanted to check everything works fine for that use case they are welcome to! However I think the tests I have conducted myself suffice.

@Ragzouken is on updating the figure in examples/fundamentals/images that contains ASEOpenMMSimulation (which is the last reference to ASEOpenMMSimulation that isn't the definition or an internal test)

I have updated the diagram to be for ASE instead of ASEOpenMM. I have also removed the simulation type itself and its scant suit of tests.


Why usingCaching() method doesnt working in IE 11

I use usingCaching() to performance the query. Method works great in Chrome.
My Chaching configuration:
 pnp.setup({
      defaultCachingStore: "session", // or "local"
      defaultCachingTimeoutSeconds: 120,
      globalCacheDisable: false // or true to disable caching in case of debugging/testing
    });

Test request time:
var start_time = new Date().getTime();
let empObjects = await pnp.sp.site.rootWeb.lists.getByTitle("myEmps").items.usingCaching().get();
var request_time = new Date().getTime() - start_time;
console.log(request_time);

In Chrome:
First request: 389ms
After reload page: 92ms
In IE11:
First request: 476ms
After reload page: 401ms
What is the reason? Thanks previously!
Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Have you tested the requests in the same window or have you close the window and reopened it?
The reason I'm asking is that IE11 as a confirmed bug in handling local storage in different windows.
https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/110656/

See my updated question. I use Session storage..

Since we don't see on your code what you are actually caching in your session. There are might be other issues that make problems with the sessionStorage too.
https://caniuse.com/#search=sessionstorage
In addition, can you write the global variable sessionStorage in your console?
console.log('Session Storage:', sessionStorage)

To see if something is stored right before you write to the sessionStorage.
The first time you will get an undefined or just the items SharePoint Stores there by default. The second time your entries should be visible.

Based on OP's reaction to last comment, seems like the issue is resolved.
But regardless, the method in question is related to PNPJS, not SharePoint / SPFx so it doesn't belong here, rather belongs in the PnPJS issue list.


Self-host thumbnail images

I was reminded, with the recent issues with class 1, that we are hotlinking to to the thumbnail images for the youtube videos, and they are way too huge. Looking for someone to download them all, resize them to appropriate dimensions, and save them named by their video IDs. Automate this if you can, it's unnecessarily tedious to do by hand!

Fantastic, thank you! :)


Flexbox property 'order' gets different highlight

For some reason, in the theme I'm using, a file blah.scss having style:
input {   order: 1; }
has just the word "order" in bright pink background (white font) -- though only when not selected.
This magenta background is also on a fragment identified with:
&__svg (so, compiles to classname .blah__svg)
A regular 'svg' style is not seen as an error.
A later directive @extend %blah__svg; also does not raise a syntax error highlight.
Looks like your sass highlighter thinks these are invalid. But they are not. The file compiles to css just fine.
'order' is valid if in a flex directive
&__svg is not a selector of an svg tag, but BEM name concatenation

Thanks. This package has been rewritten from scratch and released a Sublime Text 3 only version.
Could you please double check if these issues still happened in the new version?

Looks good. Thanks!


Restructured Paf reviewer roles flow with actual user groups and actual users as approvers

Fixes #3188
Add description here.

@frjo I have fixed that ManyToManyField issue, it was happening because of saving the m2m before saving the instance. I read about it and used wagtail's ParentalManyToManyField and it is working fine.
I am working on the sequential part, but I think in the meantime we can put it on test and test it as parallel flow.

@frjo I have fixed the tests and that demo issue. Actually, that demo issue wasn't an issue with the flow or PAF-related code, it was just a variable('request') overriding in the template https://github.com/HyphaApp/hypha/pull/3249/commits/7489ff77bcd004fd3fbd1aa2526f700cc180b12b . That's why it wasn't happening with all projects.  But it was a little hard to find this small thing.
I think we can push it to the test.  I am working to remove the final approval feature and will push the changes by tomorrow.


⚠ Bofrost.de has degraded performance

In 7897863, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1424 ms

Resolved: Bofrost.de performance has improved in eacecb5 after 7 minutes.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 10:03:29 on 2023.05.31
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 22993
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=22993

Problem has been resolved in 6m 0s at 10:09:29 on 2023.05.31
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 22993


New package: Spinners v0.1.0

Registering package: Spinners
Repository: https://github.com/AshlinHarris/Spinners.jl
Created by: @AshlinHarris
Version: v0.1.0
Commit: 68da587d65f1331e63509e26ebc1397770a5d5fd
Reviewed by: @AshlinHarris
Reference: https://github.com/AshlinHarris/Spinners.jl/issues/1#issuecomment-1226634405
Description: Command line spinners in Julia with decent Unicode support

I recommend [noblock] for this package, even though ProgressMeter.jl already has its own version of spinners, developed initially as an unpublished package called Spinner.jl. I'm not aware of any related package in Julia.
[This thread on the Julia Discourse] gives some technical details, but essentially those spinners are still progress meters, which must be updated at points with the function being measured. In contrast, I'm developing spinners that update independently and only need a signal to terminate after a function call elapses.
I developed a working prototype and added some Unicode support in v0.1. The emphasis of v0.2 will be running the spinner on its own process and adding a macro for ease of use, and I might focus on performance for v0.3.


Create multiple points on a route

Is there any way to create multiple points on a route?

The component is quite simple: uses an HTML page with google maps javascript functions inside and then calls them.
Look at the code where the html file is saved into temp folder and navigate the browser to is, so you can see the output produced by the component.
If you find a google-map javascript example to produce what you need we can expand the javascript code of the component and call the new function to add multiple points on a route...


Adding padding to cart and orders button

Hey @Varinder-Dhillon0  I would like to work on this issue. Please assign it to me

yes @iSubhamMani


Docs mention unfollow_all URL,  which doesn't exist.

Docs here:
https://django-activity-stream.readthedocs.org/en/latest/follow.html
mention a URL for unfollow_all:
curl -X GET http://localhost:8000/activity/unfollow_all/<content_type_id>/<object_id>/?next=/blog/ # Unfollow and redirect
but this URL doesn't exist and isn't listed in urls.py.

fixed in 85bdea0b3dc4251f3b7cf07f8340041165ca8409


ISSUE-740 Streamline initial screen in secure mode

This close #740
currently, we are showing ref screenshot below:-

+1


Problem with content preview

I have problem with html elements in which is markdown text.
If a content looks like this:

## FAQ

Then it is not displayed correctly in preview mode. It just takes css styles from bootstrap...

It is a problem in css styles.


config: implement config.get api

Codecov Report

Merging #201 into 1.1.0-alpha will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           1.1.0-alpha     #201   +/-   ##
============================================
  Coverage        16.81%   16.81%
============================================
  Files               15       15
  Lines             1029     1029
============================================
  Hits               173      173
  Misses             845      845
  Partials            11       11

Impacted Files
Coverage Δ

base/settings/settings.go
0% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Include value generation as part of the schema references

When a new value is generated in LancerFuzz. The exact table and column for which this value is used shall be passed. Then on the backend we check for constraints including types, unique constraints, primary keys, foreign keys etc and try to return a valid output.

The typing part is done with 47a063cdbf3dfe221c4b833dadd5fc30acdd8fac, the rest will not be considered due to the extra amount of logic needs to be implemented + the book-keeping functionality the target DBMS has to support to achieve the goal.


Proof-of-concept: Support OAuth and OIDC auth providers

[!WARNING]
This is just a proof-of-concept. There are a number of TODO's that would need cleaned up before merging.

Ticket
Resolves #235
Changes

Enable Next.js app router
Add next-auth with an example Cognito integration

Testing
http://localhost:3000/user
https://github.com/navapbc/template-application-nextjs/assets/371943/aaa68bac-ae5f-43c9-a980-c2084796440d

@sawyerh you are a 🧙


update index.html

Your changes are not correct again! Please do read the read me and rectify your pull request!
Also there is no contribution in any div on index page!


Hosted gateway full request support

Codecov Report

Merging #102 into master will decrease coverage by 0.34%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   99.87%   99.53%   -0.35%
==========================================
  Files          52       52
  Lines         833      855      +22
==========================================
+ Hits          832      851      +19
- Misses          1        4       +3

Impacted Files
Coverage Δ

src/Services/Adapters/BoletoPaymentAdapter.php
100% <100%> (ø)
:arrow_up:

src/Services/Adapters/CashPaymentAdapter.php
100% <100%> (ø)
:arrow_up:

src/Services/Adapters/BaseAdapter.php
100% <100%> (ø)
:arrow_up:

src/Services/Adapters/RequestAdapter.php
95.31% <92.5%> (-4.69%)
:arrow_down:

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 20ca891...e5e4bc5. Read the comment docs.

WTF @SparK-Cruz ?
https://codecov.io/gh/ebanx/benjamin/pull/102/diff?src=pr&el=tree#diff-c3JjL1NlcnZpY2VzL0FkYXB0ZXJzL1JlcXVlc3RBZGFwdGVyLnBocA==


Omnet++6.0.1 fails configure in M1 mac

Omnet++  6.0.1 crashes
To Reproduce

Download omnet++ 6.0.1
Run source setenv
Run ./configure

Expected behaviour
The code should run with no errors.
Screenshots

Desktop

OS: macOS 12.5.1, without docker, M1
QuISP version 0.2.1
OMNeT++ version 6.0.1
Compiler or other build info Apple clang version 13.1.6

Additional context
The error was caused because it tried to read Version as a C++ file.
It was fixed by adding // at the beginning of Version.

The VSCode devcontainer helps with this problem https://github.com/sfc-aqua/quisp/pull/439


Add Plan run duration to JSON result

Codecov Report

Merging #32 (392644d) into main (54b079b) will increase coverage by 0.30%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   86.64%   86.94%   +0.30%
==========================================
  Files           9        9
  Lines         509      521      +12
==========================================
+ Hits          441      453      +12
  Misses         56       56
  Partials       12       12

Impacted Files
Coverage Δ

internal/storage/tfe/tfe.go
82.73% <100.00%> (+1.33%)
:arrow_up:

internal/workspace/process/result.go
93.51% <100.00%> (+0.12%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Helm hook - Job status - timeoutSeconds too high

Output of helm version:

version.BuildInfo{Version:"v3.8.2", GitCommit:"6e3701edea09e5d55a8ca2aae03a68917630e91b", GitTreeState:"clean", GoVersion:"go1.17.5"}

Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.2", GitCommit:"092fbfbf53427de67cac1e9fa54aaa09a28371d7", GitTreeState:"clean", BuildDate:"2021-06-16T12:59:11Z", GoVersion:"go1.16.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.15+IKS", GitCommit:"bef807c747db0037aebbcbebd12699950a06d1a2", GitTreeState:"clean", BuildDate:"2022-01-20T14:23:00Z", GoVersion:"go1.15.15", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):

IBMCloud Kubernetes (IKS)

hello,
When I use a helm hook (pre-install or post-install) specifying for example an optional wait with timeout of 600s... I noticed that the command to retrieve the status of my job indicated a timeout parameter (timeoutSeconds) that seemed "random":
c0509 13:13:41.985950 2943713 round_trippers.go:466] curl -v -XGET  -H "Accept: application/json" 'https://XXXXXXXXXXXX/apis/batch/v1/namespaces/mynamespace/jobs?allowWatchBookmarks=true&fieldSelector=metadata.name%3Dkafka-connect-dev-1&resourceVersion=69903348&timeoutSeconds=423&watch=true'
423 secondes in this example.
In this case, even if my deployment is done in 100 seconds and I put a timeout of 130 seconds for example .... in the end my deployment is indicated as failed because the command curl to recover job status takes 423 seconds before hand over.
I am therefore obliged to set a timeout greater than 423 seconds for my deployment to end in success. Knowing that the value of timeoutSeconds varies, I have to put a very large timeout to cover the value defined in this variable timeoutSeconds.
Do you know how this timeoutSeconds value is defined? and especially how it would be possible to reduce it?
 helm install kafka-connect-dev-1 ./kafka-connect --wait --timeout 30s  --debug -v 9
apiVersion: batch/v1
kind: Job
metadata:
  name: kafka-connect-dev-1
  annotations:
    helm.sh/hook: pre-install
  labels:
    app: kafka-connect
    release: kafka-connect-dev-1
    helm.sh/chart: kafka-connect-1.0.0
    app.kubernetes.io/version: "3.1.0"
    app.kubernetes.io/name: kafka-connect
    app.kubernetes.io/instance: kafka-connect-dev-1
    app.kubernetes.io/component: kafka-connect
    app.kubernetes.io/managed-by: Helm
spec:
  template:
    metadata:
      name: kafka-connect-dev-1
      labels:
        app: kafka-connect
        release: kafka-connect-dev-1
        helm.sh/chart: kafka-connect-1.0.0
        app.kubernetes.io/version: "3.1.0"
        app.kubernetes.io/name: kafka-connect
        app.kubernetes.io/instance: kafka-connect-dev-1
        app.kubernetes.io/component: kafka-connect
        app.kubernetes.io/managed-by: Helm
    spec:
      imagePullSecrets:
        - name: my-secret-reg
      restartPolicy: Never
      containers:
      - name: kafka-connect-post-install
        image: "private.fr2.icr.io/myregistry/kafka-connect-splunk:2.0.7-3.1.0"
        imagePullPolicy: IfNotPresent
        command: ["/bin/bash"]

Extract of helm command in debug mode showing curl call to get job status
I0509 13:13:41.985950 2943713 round_trippers.go:466] curl -v -XGET  -H "Accept: application/json" 'https://XXXXXXXXXXXX/apis/batch/v1/namespaces/mynamespace/jobs?allowWatchBookmarks=true&fieldSelector=metadata.name%3Dkafka-connect-dev-1&resourceVersion=69903348&timeoutSeconds=423&watch=true'
client.go:557: [debug] Add/Modify event for kafka-connect-dev-1: ADDED
client.go:596: [debug] kafka-connect-dev-1: Jobs active: 0, jobs failed: 0, jobs succeeded: 0
I0509 13:13:41.992798 2943713 round_trippers.go:570] HTTP Statistics: GetConnection 0 ms ServerProcessing 6 ms Duration 6 ms
I0509 13:13:41.992814 2943713 round_trippers.go:577] Response Headers:
I0509 13:13:41.992820 2943713 round_trippers.go:580]     Strict-Transport-Security: max-age=15552000; includeSubDomains
I0509 13:13:41.992825 2943713 round_trippers.go:580]     Vary: Accept-Encoding
I0509 13:13:41.992828 2943713 round_trippers.go:580]     Date: Mon, 09 May 2022 11:13:41 GMT
I0509 13:13:41.992832 2943713 round_trippers.go:580]     X-Content-Type-Options: nosniff
I0509 13:13:41.992842 2943713 round_trippers.go:580]     Set-Cookie: BIGipServer~IKSxXXX~IKSxXXXXXXXXXXXX~P_XXXXXXXXXXXX=121768358.25467.0000; path=/; Httponly; Secure
I0509 13:13:41.992847 2943713 round_trippers.go:580]     X-Xss-Protection: 1; mode=block
I0509 13:13:41.992851 2943713 round_trippers.go:580]     X-Frame-Options: SAMEORIGIN
I0509 13:13:41.992855 2943713 round_trippers.go:580]     X-Content-Security-Policy: default-src 'self'
I0509 13:13:41.992859 2943713 round_trippers.go:580]     Cache-Control: no-cache, private
I0509 13:13:41.992863 2943713 round_trippers.go:580]     Content-Type: application/json

Hi @waewoo. We ask that users please provide a simple sample chart to help reproduce their issue. The output from helm create is a good starting point. Once you can reproduce the issue, that will help the community identify potential fixes and help serve as a test case preventing future code regressions.
I cannot tell what might be the cause of the issue based on the information provided here. Please provide a sample chart and a set of instructions for testing. Optimally, the chart should not involve a complex set of instructions to install, and should be pretty easy to install on a local development cluster such as minikube.
Thanks.


Classic bluetooth reconnect bug (IDFGH-5940)

Environment

Development Kit:  ESP-32 WROOM
Kit version V3
Module or chip used:  ESP32-WROOM-32
IDF version: esp-idf-v4.2.1
Build System: idf.py
Compiler version: esp-2020r3-8.4.0
Operating System: Windows 10
Using an IDE?: Eclipse V4.19.0.v20210303-1800
Power Supply:  External 3.3

Problem Description
I am using classic bluetooth on my device and pairing, disconnect,.. all seem to work fine. For my use case i should be able to remove bonded devices. After removing the bonded devices using: 'esp_bt_gap_remove_bond_device' , the ESP returns 0 as number of bonded devices.
But when i set the bluetooth discovery mode to 0, my mobile phone (android) is still able to reconnect to the ESP. It will get a new key and is able to reconnect.
Expected Behavior
I would expect when removing a bonded bluetooth device, that it can not reconect to the ESP as long as the ESP is not discoverable.
Actual Behavior
The removed bonded device is able to reconnect to the ESP.
Steps to reproduce

Bond a mobile phone to the ESP (SSP)
Remove the bonded device from the ESP using: esp_bt_gap_remove_bond_device
Make sure the ESP is now set to: 'ESP_BT_NON_DISCOVERABLE'
Try to reconnect using a mobile phone

Code to reproduce this issue
I am using the a2dp example provided by esspresif.

Hi, @SimonPVS,
Based on your description about the connection behavior, I am afraid it is just correct.
When your android phone connected once successfully to ESP32, the phone will record some information including peer bluetooth device address. When your phone tried reconnecting to ESP32, it will just connect to the recorded address without an INQUIRY process, so it just connected successfully even if the ESP32 is non-discoverable.
So, removing bonded device locally on ESP32 and set mode NON-DISCOVERABLE will not affect the connection behavior of peer device.
Turn to us anytime.
Thanks

Okay thanks!

Thanks for reporting and sharing the updates, feel free to reopen.


I'm not able to retrieve the data added with the Xamarin.Forms application from the SQL Database.

Hi team,
I moved this from here: https://github.com/MicrosoftDocs/azure-docs.it-it/issues/428
Please check the original comment, the contributor placed some screenshots
@TheKlingon commented 15 hours ago — with docs.microsoft.com
Hi.
I followed the document for the Xamarin.Forms app, but this doesn't receive the data from the database.
I'm searching for a solution, but I'm wondering if there is something wrong into the document.
Thanks.

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

ID: c4467b7b-c082-73d2-5626-038fb12d25a3
Version Independent ID: b9874b73-5524-6238-ea70-a23d8a27aa47
Content: Get started with Xamarin.Forms - Azure Mobile Apps
Content Source: articles/app-service-mobile/app-service-mobile-xamarin-forms-get-started.md
Service: app-service-mobile
GitHub Login: @elamalani
Microsoft Alias: emalani

@CeciAc , Thank you for raising the issue. I will investigate and update you.

@elamalani Request you to please take a look at this issue and document.

@CeciAc Hi I have followed this tutorial. Try to replace the connection string before puslish backend project to azure.

@CeciAc Hi I have followed this tutorial today. Try to replace the connection string before puslish backend project to azure.

Hi.
I already tried to do that, but without success.

@TheKlingon Hi,
In my case, I have created two services: 1. Azure Web App and 2. AzureSQL.

Enable-Migrations
Add-Migration "CreateDoItemTable"
Update-Database

@CeciAc Just wanted to understand if you have further queries.

Hi.
I'll try to solve the issue applying the last advice tomorrow, when I'll return to my home.
Thanks for your patience.
Alain.
Get Outlook for Androidhttps://aka.ms/ghei36

From: DashleenBhandari-MSFT<EMAIL_ADDRESS>Sent: Wednesday, March 25, 2020 3:37:28 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Alain Natalini<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] I'm not able to retrieve the data added with the Xamarin.Forms application from the SQL Database. (#50435)
@CeciAchttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCeciAc&data=02|01||d52869385c264440d4bd08d7d0657650|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637207006496566056&sdata=4NDf2SlmgpOyxBX47HRlXGsTWsqAO8u6JTvboWtoHUc%3D&reserved=0 Just wanted to understand if you have further queries.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F50435%23issuecomment-603606923&data=02|01||d52869385c264440d4bd08d7d0657650|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637207006496566056&sdata=8lshWpJ3hj%2Fsl6pWWkbDFqqlnQ2whrpRUvtGlo%2FzO64%3D&reserved=0, or unsubscribehttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACEDCCYEGV5H6FWB77GWJ53RJFU6RANCNFSM4LOX7PEQ&data=02|01||d52869385c264440d4bd08d7d0657650|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637207006496576045&sdata=kUkczwC6CRYNKCCNFnNAw%2FE6WDmIAVS5iySCku%2FetjQ%3D&reserved=0.

@CeciAc , I have not heard back from you. Please let me know if you need assistance on this issue.

Get Outlook for Androidhttps://aka.ms/ghei36

From: DashleenBhandari-MSFT<EMAIL_ADDRESS>Sent: Friday, March 27, 2020, 5:01 AM
To: MicrosoftDocs/azure-docs
Cc: Alain Natalini; Mention
Subject: Re: [MicrosoftDocs/azure-docs] I'm not able to retrieve the data added with the Xamarin.Forms application from the SQL Database. (#50435)
@CeciAchttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCeciAc&data=02|01||116eee61b4dc47e7ef4a08d7d20397e8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637208785176942921&sdata=nM5IPDod%2FjnrKduCEvImqsz9gmGWJE3IEIdBS2seUNY%3D&reserved=0 , I have not heard back from you. Please let me know if you need assistance on this issue.
—
Hi. It still doesn't work.
I'm doing the entire process from scratch.
If you want, you can close the issue.
If I'll need for further assistance, I'll open it again.
Thank you so much for all.
Alain.
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F50435%23issuecomment-604803098&data=02|01||116eee61b4dc47e7ef4a08d7d20397e8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637208785176952914&sdata=J0W7sfqBhgFXFnGSNsMBQqxmU3misY9%2BgZyN41u3XX0%3D&reserved=0, or unsubscribehttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACEDCC4NEIISQUYTKNPNQQTRJQQLJANCNFSM4LOX7PEQ&data=02|01||116eee61b4dc47e7ef4a08d7d20397e8|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637208785176952914&sdata=iLJco%2BypNOcpr7KchwVk3Nwaunde5nhdkILYwJNgQrs%3D&reserved=0.

Hi. It still doesn't work. I'll try to follow the entire process from scratch. If you want, you can close the issue. If I'll need further assistance about, I'll open it again. Thank you so much for your time and patience.


Setting offset properties to their defaults when setting the transform.

I have a strong suspicion about what effect this change has but I'm not 100% sure. The description should make it clear why a change is being made.


Portable version of MPM

Could you make a portable version of MPM that is pure C/C++ without linking to an FFT library?  Maybe using KissFFT (https://sourceforge.net/projects/kissfft/files/kissfft/v1_3_0), or using time-domain based autocorrelation instead of FFT (not sure which approach would execute the autocorrelation faster)?

https://github.com/sevagh/pitch-detection/blob/master/src/mpm.cpp#L55
Time-domain autocorrelation is slower than using KissFFT but it's the easiest solution for now. I don't have the desire to play with Yet Another FFT Library.


Reload breaks tab navigation

Problem
Reload breaks tab navigation – if exit page is not the start tab page
Expected
The sub tab page is expected to load (works)
Clicking the tab icon should route to the start tab page (fails).
Steps to reproduce:

Open the codepen tab example in debug mode: http://s.codepen.io/ionic/debug/odqCz
Click the About tab
Click the Tab Nav Stack button
Reload the page
The first page of About is now no longer accessible

Hi @jakob-e,
This sort of issue is due to ui-router.  Unfortunately, we aren't making changes to this sort of functionality in V1 at this time unless enough of the community gets behind it.  We're sorry about that.  It's just too risky.  Almost all of this type of issue will be resolved in V2's navigation.
Thanks again for contributing and I'm happy to hear your thoughts on the subject.
Thanks,
Dan


[maketmpl] add CI setup for new templates building tool

we are using golang for the following reasons:

separate it from python dependencies in this repository, which are
used by the templates
it is easier to build a single binary and run it against other
branches in the past; properly packaging a python program with its
dependencies is much harder without introducing additional tools
it is just faster

the full draft PR is here:
https://github.com/anyscale/templates/pull/315
this is just to set up the skeleton.


cleanup/unused: Remove un-used code in codebase

Remove un-used methods and params in code base
cleanup/unused: Remove un-used code in codebase

test-me-please

test-me-please

test-me-please

@aanm @tklauser I think it will skip all packages in pkg as pkg can be imported by other project(s). I assume that only hubble is importing cilium :thinking:, but not 100% sure.
Checking actually binary dependency with different OS/ARCH will be interesting though.

I think it will skip all packages in pkg as pkg can be imported by other project(s).

It should work on all packages, including ones in pkg/ it's not a hard convention like internal/ is. It ignores all exported symbols since those can be directly depended on by any other package in any other repo and are not safe to remove as a result.

retest-runtime

thanks all for review and add your comments, TIL 💯
I didn't check the test until now. 2 failures are related to #14125 and #13839. I am marking this PR ready to merge now, feel free to let me know if you think otherwise.


Fix for unit tests failing.

This PR fixes the ordering of when I run the before start lifecycle hook. I accidentally moved it after start in the DistributedApplication.StartAsync. Most folks don't use this, but we do in our local only CI tests.
This change increases the timeout waiting for the endpoints to be allocated (we need to optimize all this in P4) but also reverts the bug that I introduced.
Microsoft Reviewers: Open in CodeFlow

/backport to release/8.0-preview3


DataFrame styling not consistently preserved

When I save and reopen notebooks with styled dataframes e.g. from https://gsganden.github.io/model_inspector/tune.html#confusion_matrix, the styling is sometimes lost. IDK why.

I figured it out: it won't render on open if the notebook isn't trusted. http://gandenberger.org/2021/08/05/problem-solved-custom-pandas-styling-not-rendering-when-reopening-notebook/


Mre doc edits round 2

Small language edits s
The Quick Start and main documentation have missing links -- these need to be either explained why they don't exist or added @raulraja from this section:
"Freestyle includes ready to go Algebras and Integrations for the most common application concerns:
Ready to use integrations to achieve parallelism through scala.concurrent.Future, Akka Actors and Monix Task.
Ready to use integrations that cover most of the commons applications concerns such as logging, configuration, dependency injection, persistence, etc.
Traditional effects stacks (reader, writer, state, error, option, either)"

Codecov Report

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


⚠ ClamAV has degraded performance

In a6ae292, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 225 ms

Resolved: ClamAV performance has improved in 1f5c3c2.


Would you consider OpenBSD support?

Discussed in https://github.com/slackhq/nebula/discussions/806

Originally posted by ablackcatstail January 14, 2023
I am a huge fan of OpenBSD having learned Unix-like operating systems on it. Given that OpenBSD has good support for the Go compiler, would there be any interest in creating binaries for it? I am certainly no programmer by any means but I would willingly assist in testing and I would supply a public VM for such purposes. Given OpenBSD's emphasis on security, firewalling, and routing (also having a kernel implementation of WireGuard), I think it would make a great addition to the Nebula family. Please let me know if there is interest. Thank you in advance.

PR at #812

#812 merged.


Provide common set of preferences out-of-the-box

We could provide .general, .advanced, .updates and maybe .license:

Since almost everyone would have a General preference tab. I wonder if we should ship with a PreferenableGeneral type thing that has all these fields already filled out. My goal is to require the least amount of boilerplate possible to set up a preferences window. Thoughts?
We could do this for Advanced too. And eventually even ship some localizations for those, so users don't have to care about that either.

(Originally posted by @sindresorhus in https://github.com/sindresorhus/Preferences/pull/6)

I think we should only provide .general and advanced. Most apps only use .general and .advanced, or just .general, with the system icons. There's no common standard/convention for .updates or .license. Most apps I've tried just have update controls in the app menu or in .general. And for license, a lot of apps use and separate window for it.

@issuehunt has funded $60.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.


Add support for DOCTYPE, comments and CDATA section in rwhtml

RwHtml parser should support

DOCTYPE - after directives and before html
comments - at least traditional html ones, serve-side only may be also useful
CDATA section - stuff inside will be html escaped

Done in 7147fb4b6faf22de9572c0f16766241a37b58661


Upgrade the version of CUDA.jl in MadNLPGPU.jl

We can use KrylovPreconditioners.jl with ArgosGPU.jl because MadNLPGPU.jl only supports CUDA v4.x.
KrylovPreconditioners.jl requires v5.1.1.

Closed by #283


Kernel Mode Heap Corruption

Describe the bug
Started getting Kernel Mode Heap Corruption BSOD on Windows 10 today. I was using 5.2.* version and after getting the error I updated to version 5.2.3-latest20210921.1. The error is still happening though.
To Reproduce
Steps to reproduce the behavior:

Go to an existing 'Cluster'
Go to Charts
Click on a Chart (in my case Redis)
Click on Install and select version
A screen will open at the bottom (same place as the terminal window) with options to select namespace, name etc.
See error

Expected behavior
The steps above were what I did the first 2 times the error occurred. After that I upgrade to latest version and saw the issue just navigating in the Lens and trying to click on namespaces.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

Lens Version:

OS: Windows 10
Installation method: chocolatey

Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
Your logs go here...

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
{
  "preferences": {},
  "apiVersion": "v1",
  "kind": "Config",
  "current-context": "redis-lab-npdp1-pks",
  "contexts": [
    {
      "context": {
        "cluster": "redis-lab-npdp1-pks",
        "user": "ddd3e101-2e6d-4faa-ab00-da24cab1847d"
      },
      "name": "redis-lab-npdp1-pks"
    }
  ],
  "clusters": [
    {
      "cluster": {
        "server": "https://<IP_ADDRESS>:8443",
        "certificate-authority-data": "REDACTED"
      },
      "name": "redis-lab-npdp1-pks"
    }
  ],
  "users": [
    {
      "name": "ddd3e101-2e6d-4faa-ab00-da24cab1847d",
      "user": {
        "token": "REDACTED"
      }
    }
  ]
}

Additional context
Never had any issues with Lens until this morning. Thanks for the great tool you guys provide and I hope this bug report can help finding this nasty bug.

What build of windows are you running?

By coincidence my machine got updated this morning. It turns out the issue is no longer happening. I'm sad I haven't saved this information from yesterday but I am happy the issue is gone. I know it's no longer useful but here is the current version anyways. You can close this. Thank you so much!
OS Name:                   Microsoft Windows 10 Enterprise
OS Version:                10.0.18363 N/A Build 18363

I will close this then as it might have been a electron/windows interaction bug.
However, what do you mean by "installed lens via chocolaty"?

I installed with choco install lens provided by https://chocolatey.org/. Just realized this option is not listed in the documentation. I not only installed that way as I also do upgrades from time to time using the same method (choco upgrade lens).


Added redis extension.

lgtm

LGTM, but why is this failing? The VM just shuts down?

It's possibly because uploading the image was too slow. I restarted the build.

Alright, the build is green, so merging.


Make test-migrations-via-try-runtime CI job depend on live-sync-credi…

…tcoin
this should reuse the database which is already on-disk and not cause a second sync, hopefully reducing the overall execution time for this CI pipeline.
Also specify missing --pruning archive option to deal with errors like:
Error = Call(Custom(ErrorObject { code: ServerError(-32000), message: "Client error: UnknownBlock: State already discarded for 0x7a6052272f51d086f3c0f89ced306b845ba93465591026107c6e236174d993d4", data: None }))
Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Successfull test-migrations-via-try-runtime against Mainnet https://github.com/gluwa/creditcoin/actions/runs/6796162156/job/18478386218?pr=1377

Codecov Report

Merging #1377 (aa340a0) into dev (f517168) will decrease coverage by 1.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #1377      +/-   ##
==========================================
- Coverage   71.86%   70.84%   -1.03%
==========================================
  Files         111      111
  Lines       12953    12953
  Branches      128      128
==========================================
- Hits         9309     9176     -133
- Misses       3634     3777     +143
+ Partials       10        0      -10

see 7 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Revert "Move the changes in PR #20241 to 1.1.x"

Reverts ballerina-platform/ballerina-lang#20508

Closing this since the issue which prompted this revert PR actually turned out to be due to something else. And hence, this is not required anymore


Convert use_eval to use send/recv system

Converts the use_eval hook to use a send/recv system.
Example:
fn app(cx: Scope) -> Element {
    // Construct the evaluator
    let eval_provider = dioxus_html::prelude::use_eval(cx);
    let eval = eval_provider(r#"
        dioxus.send("Hi from JS!");
        let msg = await dioxus.recv();
        console.log(msg);
    "#);

    // Run the JS
    eval.run();

    // Send a message to JS
    eval.send("Hi from Rust!".into());

    // Get a receiver channel to receive messages from JS
    let receiver = eval.receiver();
    let future = use_future(cx, (), |_| async move {
        receiver.recv().await.unwrap()
    });

   // Act on received data
    match future.value() {
        Some(v) => {
            // Cleanup any eval artifacts
            eval.done();
            cx.render(rsx!(
                p { "{v}" }
            ))
        }
        _ => {
            cx.render(rsx!(
                p { "hello" }
            ))
        }
    }
}

Todo:

[x] Web
[x] Desktop
[ ] Liveview? #1056

Closes #1026

Will fix the CI issues tomorrow.

Sorry to hijack. I'm using this PR and I am trying to get multiple messages from the channel. Having replicated your example successfully. I set out to tweak it for my use case.
     let mut eval = eval_provider(
         r#"
           window.addEventListener("message", dioxus.send)
    "#,);

Which should in theory attach an event listener to the window. Listening for a postMessage from either a service worker or an iframe. (technically the same context works, but that's not the value add). The issue I'm having is the receiver is falling out of scope. In the example you create a receiver and you use it as a future. If I grok the code this is specifically an UnboundedReciever. Which leads me to believe that if I stuck it in a variable or a use_ref or some other form of maintaining a reference to the channel, I could read from it indefinitely. Which should work perfectly with window.addEventListener.
Am I understanding the potential correctly? How would you recommend I maintain a reference to the channel to get multiple values out of it? And bonus points, if I make an event listener in the JS context in this manner, how could I clean it up?

Yes, I believe that's possible.

I could read from it indefinitely.

It sounds like you want cx.spawn,  cx.spawn_forever or maybe even use_coroutine. You would move the eval into the async block (solving the drop-ing issue) and then call eval.run() on it (so it only runs once and not every time the component re-renders).
In terms of cleaning up, you could have the script wait for a message before running additional code:
let mut eval = eval_provider(r#"
    window.addEventListener("message", dioxus.send);

    // Wait for any message from Rust
    await dioxus.recv();

    // Cleanup code
    window.removeEventListener("message", dioxus.send);
    "#);

// When you want to cleanup the event listener:
eval.send(serde_json::Value::Null);

@dr-oofenshmirtz Yes, I believe that's possible.

I could read from it indefinitely.

It sounds like you want cx.spawn, cx.spawn_forever or maybe even use_coroutine. You would move the eval into the async block (solving the drop-ing issue) and then call eval.run() on it (so it only runs once and not every time the component re-renders).
In terms of cleaning up, you could have the script wait for a message before running additional code:
let mut eval = eval_provider(r#"
    window.addEventListener("message", dioxus.send);

    // Wait for any message from Rust
    await dioxus.recv();

    // Cleanup code
    window.removeEventListener("message", dioxus.send);
    "#);

// When you want to cleanup the event listener:
eval.send(serde_json::Value::Null);

Clever use of await!

This now works with Liveview, and it implements the more efficient non-broadcast method of channels


Bug: When Service Connection is configured at the Management Group Level the pipeline throws an error in the Prod Plan - Feature Branch / Main Branch

Bug: When Service Connection is configured at the Management Group Level the pipeline throws an error in the Prod Plan - Feature Branch / Main Branch
After following the required permissions for the sc-pac-plan

Connection
Stages
PAC-DEV-001
PAC-TEST-001
Tenant 1
Tenant 2

sc-pac-plan
prodPlanFeatureStage  prodPlanMainStage

EPAC Policy ReaderSecurity Reader

Per Azure tenant at your highest Management Group (called rootScope in EPAC vernacular)
- Security Reader and EPAC Policy Reader (custom) or Policy Contributor roles for planning the EPAC prod deployment
And configuring the Service Connection at the Management Group Level

I got this DevOps Error when running the Prod Plan - Feature Branch and Prod Plan - Main Branch Stage in the DevOps Pipeline.
Message: The client '1111111e-b222-c333-d444-ab123456789c' with object id '1111111e-b222-c333-d444-ab123456789c' does not have authorization to perform action 'Microsoft.Management/register/action' over scope '/subscriptions/1111111e-b222-c333-d444-ab123456789c' or the scope is invalid. If access was recently granted, please refresh your credentials.
Command 'az account management-group show --name d-mg-sandboxes --expand --recurse ' command exited with error

It looks like when the Service Connection is configured at the Management Group level there is one Action Permission additionally needed in the custom role for it to work and that is Microsoft.Management/register/action

At least when I added this permission within the Policy Reader Custom Role the error went away.
I've not tested this with a Service Connection on Subscription Scope in DevOps, as I just assumed that the Service Connection should be configured on Management Group scope as we are dealing with multiple subscriptions right?
At least it's not clear if you should configure the Service Connection based on Subscription or Management Group Scope in the documentation.
And to add more context to what this action permission does is that it's allowing the Service Principal to register a resource provider within the Management Group scope.

Did a little bit more Troubleshooting here and I have additional information regarding this, when using Subscription based Service Connection you don't need this permission as the subscription is not registered to Microsoft.Management when it's directly pointing at the subscription and not at the Management Group:

When using a management group-based service connection - the subscription is registered with Microsoft.Management Resouce Provider

Thank you for adding that information. I'll update the documentation and the script to include this. I encountered this recently (last week) with one of my customers as well. Since I hadn't yet time to debug, I wanted to thank you for the root cause analysis.
THANK YOU! I will mark this as a bug.

Thank you for adding that information. I'll update the documentation and the script to include this. I encountered this recently (last week) with one of my customers as well. Since I hadn't yet time to debug, I wanted to thank you for the root cause analysis.
THANK YOU! I will mark this as a bug.

Thank you very much for spending time creating such a great solution, happy to help out, I've contributed and created a PR with the needed changes here for you to review #78
/Haflidi

Sorry, I still have a question about this case.
We also ran into the error.
During our own troubleshooting we tried to keep the permissions as low as possible. Our solution would be to extend the following command with the parameter  -no-register.
Invoke-AzCli account management-group show --name $scopeParam.ManagementGroupName --expand --recurse --no-register
This would allow the implementation of the planning of EPAC prod deployment with only the Reader role.
Is there anything against this solution? Is the authorization required for the registration/activation action for azure.management in EPAC if you have already registered it for the respective scope?


Video's work in debug version but in release version they don't show

The component works just like I want it in the debug version. But as soon as I assembleRelease it only shows a white screen instead of the video. Why doesn't it play in the release version?
I think that somehow when it compiles it can't find the location anymore, but how would I fix that?
EDIT:
The videos are locally loaded with require()

I managed to solve this by putting the videos in res/raw/. I posted my answer here:
http://stackoverflow.com/questions/40975137/videos-work-in-debug-version-but-in-release-version-they-dont-show/40978270#40978270


unable to install azure static web app extension

Does this occur consistently?
Repro steps:

Action: staticWebApps.activate
Error type: string
Error Message: Timed out waiting for authentication provider to register
Version: 0.12.2
OS: darwin
OS Release: 23.4.0
Product: Visual Studio Code
Product Version: 1.89.1
Language: en

Please follow the comment here for a potential fix: https://github.com/microsoft/vscode-pull-request-github/issues/3469#issuecomment-1109584285
If this doesn't help resolve your issue, please provide repro steps.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add autoload and output operation for UPI

What this PR does / why we need it:

Adding integration of UPI interface into standard transformer. To fully support UPI interface in standard transformer, we introduce 3 new operation:

UPIAutoload
UPIPreprocessOutput
UPIPostprocessOutput

UPIAutoload
This operation will load tables and variables from incoming request or model response
UPIPreprocessOutput
This operation will convert tables into UPI PredictValuesRequest. This operation only applicable for preprocess step and upi_v1 protocol
UPIPostprocessOutput
This operation will convert tables into UPI PredictValuesResponse. This operation only applicable for postprocess step and upi_v1 protocol
Other than that, in this PR we introduce SourceType to indicate what is the source type of the jsonpath. There are two source types:

Map. Existing one, jsonpath lookup operation is based on map type object and using existing jsonpath library
Proto. jsonpath lookup operation is based on protobuf.Message object and using protopath library to do the compilation and evaluation

Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

Checklist

[x] Added unit test, integration, and/or e2e tests
[x] Tested locally
[ ] Updated documentation
[ ] Update Swagger spec if the PR introduce API changes
[ ] Regenerated Golang and Python client if the PR introduce API changes

Can you rebase against main branch?

This is great work @tiopramayudi! I leave comments above. One important thing is to understand the performance of the new operations via benchmark. Additionally, do you plan to create end to end test in this PR or separately?

@tiopramayudi You'll need to rebase again against main branch to include fix for e2e testing https://github.com/gojek/merlin/commit/9f10ae6032f1016541b9ae5284054312fb5cde17


Don't render RSS and sitemap for unpublished articles

See https://github.com/RustMagazine/rustmagazine/issues/25

LGTM


Code samples in manual lack imports

First off thank you for creating this project, I'm really excited to use it :)
I did however run into some confusion trying to get setup.
The documentation does not mention the need for imports either in text or in the examples.
As a result I was getting errors like:

"error: not found: value Theme"

and

error: not found: value Helium
laikaTheme := Helium.defaults

All I needed was
import laika.theme.Theme

I think, the earliest examples in the documentation should explicitly show the relevant imports.
Happy to submit a PR if folks think it would be worthwhile.

Welcome, and sorry for the late response. I think this is a perfectly reasonable request, things like this surface because there has been surprisingly little feedback for the manual for a project with 300+ stars. And I rarely have to use my own manual when using Laika, so I did not notice either. 🙂
I'd be happy to accept a PR that addresses this in the way you described: Add imports to code samples that introduce new API and omit them when showing slight variations of the same code. You also don't need to cover the whole manual if if you don't have the time, I'm equally happy to just merge something that gets that improvement started.
I hope you don't mind I renamed the issue. It's not because it sounded somewhat broadly negative 🙂 - but more so that users know what this is actually about.


Adds focus method to VirtualizedSelect

Creates a ref to the internal select component and call focus on that ref

Looks good. Thanks!


Think about assigning DOIs or PURLs to Databus IDs and of migrating a Databus namespace

questions are

for what kind of Databus assets it makes sense to have DOIs (datasets, versions, or even files)
how to attach / assign DOIs (use as a mod post registration)
how to display them on the UI
is there a way to have/register a dedicated DOI handle resolver (that could e.g. be in the databus) for a give DOI prefix
think about (other) strategies for migrating a Databus to a different domain /namespace  e.g. from energy.databus.dbpedia.org to energy.databus.dlr.de (also effect on DOIs)

I checked Datacite. Theoretically, it is just a matter of cost: https://datacite.org/feemodel.html


Fork hbbft to get minimint compiling on M1 Mac

Before this commit I ran into this issue on M1 Mac.
After this commit, the project compiles and the pass on M1 Mac. Needed to fork hbbft in order to upgrade reed-solomon-erasure to 5.0.1 in order to include this commit.
I have not idea if I correctly upgraded the reed-solomon-erasure dependency in hbbfd. Here's the commit where I did it. I just hacked it to get it compiling. No tests fail, but the reordering_attack test doesn't finish. And a suspicious tests/binary_agreement_mitm.proptest-regressions file is saved with the following contents:
# Seeds for failure cases proptest has generated in the past. It is
# automatically read and these particular cases re-run before any
# novel cases are generated.
#
# It is recommended to check this file in to source control so that
# everyone who runs the test benefits from these saved cases.
xs<PHONE_NUMBER> 484839646<PHONE_NUMBER><PHONE_NUMBER> # shrinks to seed = [217, 197, 151, 151, 13, 244, 31, 82, 44, 31, 141, 203, 43, 206, 53, 58]

Before merging we should move the hbbft fork from my personal GitHub account to fedimint org.

Added another commit to hbbft fork.

Is there currently any way to test on GH CI on M1?