Raise an error on sufficiently different insta vs cargo-insta versions

This has two purposes:

By raising an error if major versions differ, it allows eventually upgrading to insta 2 without confusing breaks (we want to add this a while before releasing v2)
It potentially allows for making some changes before a big break, by warning when minor versions differ sufficiently. This could be upgraded to an error. Currently we have a few deprecated items; and I was thinking of trying to unify some of the pending / named snapshot handling, but it becomes burdensome if we need to support the existing approach forever.

...one advantage of gradual changes over a big break is that a big break would cause issues with cargo-insta — unless it worked with both insta 1.0 & 2.0, folks would need to be flipping between versions when working between projects that have vs. haven't upgraded. Whereas most projects can be expected to upgrade minor versions after a while. (For reference, 10 minor version is currently 15 months ago)

WDYT?

But then this just says we break compatibility which means that someone would need to be able to run cargo insta1 and cargo insta2 if they need to support both things on a machine. That seems not ideal?

Very much agree (some of what I meant in the bullet above starting "...one advantage of gradual changes over a big break "....).
Are you thinking that cargo-insta=2 would still support snapshots from cargo-insta=1? That would be a much better UX, at the cost of more work to support.

Even without the warning of v2 vs v1, this raises a warning on sufficiently different minor versions of insta vs. cargo-insta; is that worthwhile? My intention was for that to make it easier to very gradually make changes without folks getting confusing test failures.

I think that cargo-insta should definitely support old and new snapshots.  My eventual goal at the moment is still to add snapshot!(...) macro but I don't want to deprecate all the existing macros for quite a while.  So I fully expect that even if insta2 ends up having a dramatically simpler core crate, cargo-insta just keeps supporting old stuff.  Even just to maybe help with migration etc.
I wonder if the right approach here might be to add a "snapshot version" marker to the snapshot files and to use that to set a lower bound to cargo-insta?

I think that cargo-insta should definitely support old and new snapshots. My eventual goal at the moment is still to add snapshot!(...) macro but I don't want to deprecate all the existing macros for quite a while. So I fully expect that even if insta2 ends up having a dramatically simpler core crate, cargo-insta just keeps supporting old stuff. Even just to maybe help with migration etc.

OK amazing

I wonder if the right approach here might be to add a "snapshot version" marker to the snapshot files and to use that to set a lower bound to cargo-insta?

I thought about this too! I think it's a nice legible approach.
The reason I leaned away is:

How would this update? Most snapshots are compatible for a long long time. So this would require rewriting snapshots to just change the version number every-so-often.
It wouldn't help for inline snapshots
We can generally tell from the snapshot whether it's old without needing a version number. For example, when we first had inline snapshots with leading characters, or the prior metadata format. I've recently added some warnings for old snapshots.

A couple of alternatives:

We could start increasing the major version of insta, even without doing big changes, and have a contract like "no warnings in the final minor version means the next major version will work". I think if we do this then we don't even need a version field, since the version field is the major version of insta in Cargo.toml.
A version in the insta config file (rather than each snapshot) avoiding the problems above.

When a snapshot was created with a version older than that version, it's not guaranteed to work — meaning that the failures would come when updating the config file, rather than when using a more recent version of insta
We could enforce something like "the minor version needs to be within 10 versions of the current insta version", which would allow us to stop supporting very old snapshots.
Relative to the major version changes, this approach arguably works better when there are incompatible changes in a very small number of snapshots. (It has nice aesthetics — it doesn't look like things are changing often to almost everyone...)

If it were just me, I would initially vote to start increasing the major version even without big changes. We could even do that soon: i.e. check we have warnings on everything that's old, confirm that --force-update-snapshots correctly rewrites old formats, check that cargo-insta would be OK on both old & new versions — and then bump the major version. Though probably worth waiting for some changes such as whitespace to land.

How would this update? Most snapshots are compatible for a long long time. So this would require rewriting snapshots to just change the version number every-so-often.

It would really only update in case an insta-2 comes around I believe.  So we would basically be on version 2 for now (since we did one change in the past) until we finally decide to really change things around.
The global version in a config would also be an option but I would need to think about this more.


Refactor Ethos-U codegen tests

The Ethos-U codegen tests had a lot of replicated functionality due to them being developed concurrently. This refactor consolidates those commonalities.

cc @ekalda @lhutton1 @NicolaLancellotti @dchauhan-arm - let me know what you think :)

broadly LGTM! Especially the change to making printing the command stream be optional so it doesn't print a novel in the logs. Just a few minor Q's inline

Thanks @mbaret @ekalda @lhutton1 @NicolaLancellotti @dchauhan-arm !. This is merged now!


Android Basics: Activities and intents

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-activities-intents?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-activities-intents#1
Describe the problem
Add a section to explain how to accept licenses.
Steps to reproduce?

Download Repository
Import it to Android Studio
Run it

Versions
Android Studio version:  4.1.2

@JustSxm
What licenses, could you please provide more explanation?
Also, add a note to how is this related to this codelab.

Hello, the actual problem is gradle and such are not updated when you
download the project so you're prompted to update when you open the
project. If you accept to update you must accept licenses in
AndroidStudio/licenses
On Wed., Mar. 17, 2021, 7:47 p.m. jtavva @.***> wrote:

@JustSxm https://github.com/JustSxm
What licenses, could you please provide more explanation?
Also, add a note to how is this related to this codelab.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google-developer-training/android-basics-kotlin-words-app/issues/62#issuecomment-801513968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKF7TFIGPXNKBWFF3LSKABTTEE5RLANCNFSM4YUA3TBQ
.

@JustSxm
Do you still face this issue?
If yes, could you please provide us with the screenshots, to debug your issue further.  .

Hello, I apologize for not answering this issue. Updating was not necessary, I've updated but it was not needed. Thank you for the help


Looping within the bot.

I've tried a number of ways of looping within the bot but nothing seems to work.
I've tried adding a react event loop within a $discord->on('ready', function ($discord) { but this seem to stop the bot working., ive also tried a simple while(true) loop but this also stops the bot working.
any thoughts? or should I just be looping outside the bot and launching a new instance of the bot each loop to interact with discord.

Could you try explaining more on what you are trying to do? Using the while (true) loop blocks the event loop so the websocket disconnects. If you want jump on our discord server and I can help you.

Same Problem:
Here is an example in javascript, what i mean:
setInverval(() => {
     console.log("This comes all 12 seconds.");
}, 1000 * 12);

If you want to add a timer, use the react event loop timers. You can get the event loop using $discord->getLoop(), so e.g. $discord->getLoop()->addPeriodicTimer(...).

kk

@davidcole1340 maybe pthreads?

Pthreads only works for PHP 7.2< so I don't think it would work

maybe make a version of DiscordPHP that uses pthreads?

No point of developing for a library that isn't maintained anymore, have a look at parallel

Hmm idk


Ability to cancel receive operations

Alternative to https://github.com/Azure/azure-sdk-for-net/pull/19888
Closes https://github.com/Azure/azure-sdk-for-net/issues/19306
All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] I have read the contribution guidelines.
[X] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.
If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version,

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

Addresses https://github.com/Azure/azure-sdk-for-net/issues/17734

/azp run net - servicebus - tests

This is awesome! Thanks @danielmarbach


input descriptor iteration algorithm

as raised in #18

(TODO: unwind this advice to cease. A prover might have reasons to compute all the possible variations and pick the "best" one on the basis of which one is cheaper to present (has less fees associated), which one is more private, which credential they previously used with this verifier, etc. Stopping after the first match is not the business of the data format and is none of the verifier's business. Their requirements just way what is acceptable, not how it's computed.)

currently the algorithm for matching fields with input descriptors stops after the first match. Which field a prover wishes to select may be more nuanced than "pick the first one that matches"

Proposal: add note with pseudo-example to show iteration of the credentials you hold.

Addressed in PR


Allow Splunk/Splunkforwarder to be installed with "package" resource

This is an enhancement request.
For those using Satellite or a package manager with the splunkforwarder in their repos it would be great to be able to install the splunkforwarder with the package resource instead of bundling it with the module/in Puppet repo.
Any reason this is not supported?

hi @inspired, I'm not 100% sure what you mean. The package is currenty installed via package resource. Do you want to manage it outside of this module?

@bastelfreak Yeah, I'd like to install it with a RPM or APT repo instead of bundling it with the module. As far as I can see that's not supported? Correct me if I'm wrong :-)

I'd like this feature too (was about to implement it myself, but haven't gotten around to it). Where possible, I try to add the Splunk packages to an internal repo, so yum and apt are able to just "simply" install them without puppet providing the binary package as a file.

if you set source to '' in the splunk class declaration, then it will assume to use package repos, we do this already internally and it works well for us

I would even be ok with the puppet module package installing the splunkforwarder package, I have requirements that all my packages be managed centrally in a repository owned by a compliance team.  So our firewalls would block access to download the rpms in other ways, but there's an existing repo with splunkforwarder available.

I'm going to add my +1 to this request.  Storing binaries and artifacts alongside source code in git is a really bad idea.  It's highly inefficient for git and wastes storage storage unnecessarily.  I am confused why we're not suggesting the use of the alternative source mechanism by default?
https://github.com/voxpupuli/puppet-splunk/blob/master/manifests/forwarder.pp#L49-L54
My understanding from reading the code is if you set $manage_package_source to false, and then provide a url to $package_source, then this should be doable.


Error opening Electra Desktop (release 1.0.0) on Ubuntu 16.04

After close the error alert, it shows "Starting daemon" forever.

@sandro-csimas It happens when you have the old QT wallet opened at the same time. Can you close all your wallets and try to relaunch only Electra Desktop ?

@ivangabriele, Yes I closed the old QT wallet. Is there any application log to check?

It seems the WalletDaemonState is different of STARTED...
  public get lockState(): WalletLockState {
    if (this.LOCK_STATE === undefined && this.DAEMON_STATE !== WalletDaemonState.STARTED) {
      throw new Error(`ElectraJs.Wallet: You need to #startDaemon in order to know the wallet #lockState.`)
    }
    return this.LOCK_STATE as WalletLockState
  }

@ivangabriele, I saw few people in Facebook with the same problem. The way to solve it is by removing the Electra directory content (except the wallet.dat) and letting the new wallet creating the content again.

I removed everthing besides wallet.dat. I still get the message.

my error is a little different
Uncaught Exception:
Error: ElectraJs.Wallet: You need to #startDaemon in order to know the wallet #lockState.
at WalletHard.get lockState [as lockState] (/Applications/Electra Desktop.app/Contents/Resources/app.asar/build/main.js:67565:19)
at EventEmitter.electron_1.ipcMain.on (/Applications/Electra Desktop.app/Contents/Resources/app.asar/build/main.js:83515:53)
at emitOne (events.js:115:13)
at EventEmitter.emit (events.js:210:7)
at WebContents. (/Applications/Electra Desktop.app/Contents/Resources/electron.asar/browser/api/web-contents.js:275:13)
at emitTwo (events.js:125:13)
at WebContents.emit (events.js:213:7)

@sandrocsimas @bak5221
Now we need to establish a 100% reproducible case. On Linux, I think this is related to the manually built QT wallet which, depending on which code it was build from, is incompatible with
Fyi, here are the Linux build instruction for the QT wallet.

@ivangabriele, I reproduced minutes ago with the latest QT wallet (manually built) and the latest Electra Desktop.

Yep so I confirm the issue is on Electra core side.

im still get this message

@bak5221 Please join the #user-support channel on our Discord. There is a workaround to fix that.


Webinterface admininstration part buttons: Manage Printers & Classes broken

Describe the bug
In the webinterface in the Administration tab the buttons Manage Printers and Manage Classes do not work, return 'Not Found'
Steps to reproduce the behavior:

Go to url: localhost:631/admin
Click on 'Manage Printers'
the url  is localhost:631/printers/? and produces 'Not Found'
work around: change the url to: localhost:631/printers
Similar problem with 'Manage Classes', but somehow 'Manage Jobs' works fine non the less.

Expected behavior:
showing the right webpages.

System Information:

ArchLinux
browser Firefox for Arch Linux 126.0.1 (64 bit)
CUPS version 2..4.8

Additional context
See also: https://bbs.archlinux.org/viewtopic.php?id=296377

Fixed in the past by https://github.com/OpenPrinting/cups/commit/3826de00c582 .


Release v1.5.0

Since the downstream v1.5.0 is already out, let's merge this PR for now, and we can add in the changelog in a different patch.


Fix debugging in tachyon-workers.sh

The debug statement placed in tachyon-workers.ssh to state which user
account is being used to SSH to the remote workers was wrapped inside an
if by a later commit so it only fired when using non-background SSH
which doesn't aid debugging.  Moved the echo outside of the offending if statement
Also made minor white space changes for readability

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2855/
Test PASSed.

Since SSH background is in parallel, it will just dump all the connections at once right? Though I suppose seeing all the workers is better than nothing. What issues did you find it useful for debugging?

I guess so, note that the SSH backgrounding does not redirect the output so all the SSH output will go to be intermingled on the console anyway
It's useful for debugging SSH connection issues associated with authentication, two particular examples we've run into:

User account not properly set up for password-less SSH on some worker nodes
Launched such that the user launching the processes is not the same as your user account e.g. launching via sudo

Thanks for the details!
+1 LGTM


Wiki Software For Career Resources

One of the requirements of the initial MVP is to have a wiki type of setup that can be used to post content to the Career Resources, among others.  As I see it, this will likely be the most complex requirement and I would love to start the discussion about how we see this working.
Aside from that, what David and I have spoken about is creating an aggregated feed below the fold on the main page that has a list of most recent blog and wiki posted articles.

1 for the aggregated feed!


Add custom html tags to

It'd be nice if I can pass some tags from larecipe config to , so that I can add some meta tags.
My use-case is that, I have a sandbox website, and I don't want the larecipe docs to be crawled by Google. Even though, I have my robots.txt file properly, it seems like I need to add robots meta tag in the html and unfortunately there is no way to achieve it.
head_meta_tags => [
    env('IS_SANDBOX') ? "<meta name="robots" content="noindex">" : ''
]

I can create this and create a PR - what do you think?

I think you can do that by default laravel supports that. Just override the file here https://github.com/saleem-hadad/larecipe/blob/master/resources/views/default.blade.php and add the headers you like.


Help getting basic example working

Based off of the README I was trying to get the basic example to work. The README is a little outdated, so if you want I can make a PR to update that. However, I am trying to figure out what the correct why to load the .beam file into erl. I moved the .beam file to an ebin directory in my project folder and ran erl -pa ./ebin/. I am able to run c(test_module). But the output I get is:
{error,beam_lib,
       {missing_chunk,'./ebin/test_module.beam',"CInf"}}

Not sure what I am doing wrong, was wondering if you are able to give me any advice to get this basic example working. Maybe once I get a basic example working I can make a PR to update the README if you are open to it.
Below is my Haskell code:
{-# LANGUAGE OverloadedStrings #-}
module Main where

import qualified Data.ByteString.Lazy as LBS

import Codec.Beam.Instructions (func_info, label, move, return')
import qualified Codec.Beam as Beam

main :: IO ()
main =
  LBS.writeFile "test_module.beam" $
    Beam.encode "test_module"
      [ Beam.export "tuple_of_one" 0
      , Beam.insertModuleInfo
      ]
      [ label $ Beam.Label 1
      , func_info "tuple_of_one" 0
      , label $ Beam.Label 2
      , move (Beam.Tuple [Beam.Integer 1]) (Beam.X 0)
      , return'
      ]

Thank you and sorry if I am overlooking a small detail!

Hi @mattludwigs — I'm glad you reached out. A few things are coming to mind:

You mentioned that you're using c(test_module)., but since the module is already compiled, I usually use l(module).. Could you let me know if using l instead of c that produces the same error?
What Erlang/OTP version are you running? This one is probably a long shot, but I know that v20.3 was released a few weeks ago, and I haven't tested against it yet.

Thank you for the quick response!
Using l worked!  :man_facepalming:
Would you be up for me updating the README for getting the basic example running? I probably won't get around to it until later tonight or tomorrow sometime though.


cgroups are mounted after check initialization

The check can initialize before the cgroup mounts resulting in the following error:
<TIMESTAMP> | ERROR | dd.collector | config(config.py:897) | Unable to initialize check docker
Traceback (most recent call last):
  File "/opt/datadog-agent/agent/config.py", line 889, in load_check_directory
    agentConfig=agentConfig, instances=instances)
  File "/opt/datadog-agent/agent/checks.d/docker.py", line 125, in __init__
    self._mountpoints[metric["cgroup"]] = self._find_cgroup(metric["cgroup"], docker_root)
  File "/opt/datadog-agent/agent/checks.d/docker.py", line 427, in _find_cgroup
    raise Exception("Can't find mounted cgroups. If you run the Agent inside a container,"
Exception: Can't find mounted cgroups. If you run the Agent inside a container, please refer to the documentation.

Good catch, that's definitively a bug.
Can you telle me more about your setup? Which OS? Are you running the Agent on the host directly or the Agent container? What do you use to run the Agent at startup?


LUIS key error is not reverted in Project tree

Create an echo bot
Create a new dialog and add a trigger. Update the name of trigger and add a trigger phrase
The ProjectTree gives an error warning of a missing LUIS key correctly.
Remove the dialog to which we added this trigger

Expected: The LUIS error on the project tree should disappear as the bot does not need LUIS anymore
Actual: The LUIS error still shows up even after deleting the dialog

@srinaath

we could not repro the case that deleting dialog still need LUIS key.
for deleting triggers, the current design is not considering the triggers to tell whether LUIS key needed. The current design is looking for LUISRecoginzer+.lu file not empty.

So i'm taking down the bug label here.

@boydc2014 Deleting  dialog works as expected. I think @srinaath is referring deleting triggers. The last time when I worked on this problem. The logic is what you said above. I checked code and sync with @luhan2017 today. Deleting a onIntent trigger should also deleting corresponding lu intent. The code is already there. But the parameter is not correct. I opened PR for it.


feat(setup): switch to official terramate-setup-action

https://github.com/terramate-io/terramate-action
fixes #10

feels like rate limits. any idea @m4s-b3n

https://github.com/infinite-automations/terramate-all-in-one/actions/runs/9809251203/job/27086774760?pr=11
feels like rate limits. any idea @m4s-b3n

nope, as long as we can not say for sure 🙅


support Spark SST Datasource

Signed-off-by: marsishandsome<EMAIL_ADDRESS>close https://github.com/tikv/migration/issues/37
Add SSTDataSource
subtask of https://github.com/tikv/tikv/issues/11007
In order to support decoding SST files parallelly, a Spark DataSource library SSTDataSource will be implemented in TiKV migration Repo as follows:
class SSTDataSource extends FileDataSourceV2

Users can simplly use the folowing code to decode SST files by Spark:
val path = "hdfs:///path/to/sst/files/"
val keyValueDF = spark.read.format("sst").load(path)

depend on https://github.com/tikv/client-java/pull/284

@liangyouze @daimashusheng @peng1999 @andylokandy @Little-Wallace PTAL


Template of controller --api

Laravel Version: 6.2
PHP Version: 7.4

Description:
So when I use the artisan command for creating a api controller the default response is  \Illuminate\Http\Responsebut for an api like from axios by default you want a \Illuminate\Http\JsonResponse no
?, like response()->json(), is it just a choice ?, should I just change the default template ?
@return \Illuminate\Http\Response instead of \Illuminate\Http\JsonResponse

Steps To Reproduce:
php artisan make:controller API/TestController --api

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


clang::Cursor::num_template_args should return Option

Right now it returns a c_int, but the documentation for clang_Cursor_getNumTemplateArgs says it returns -1 if the cursor's referent is not a template declaration or specialization. We should translate negative values into None and positive values into Some. This will also require fixing up callers.
I can mentor someone who wants to pick up this bug.

Please make a comment here if you intend to work on this issue. Thank you!

Should it return Option instead of Option (as c_int == i32, this is the closest unsigned type) ?
Furthermore, This would be more consistent with other API definitions like Type::num_template_args() returning already Option.

Yeah, both u32 and usize are fine. Type::num_template_args is gone now, but I agree.

Ok, I will take care of this issue then.

Great, thanks!


Screenshots with UI show a black screen.

Description

I use this feature a LOT and have only noticed yesterday while looking in my screenshot folder that a lot of my screenies are just black. Taking screenshots without UI works fine...
Reproduction

Try to take a screenshot of your screen with F2 (or whatever other hotkey you have), look in your screenshot folder and find out your screenies are gonezo
Screenshots

Additional context

I don't take screenshots everyday, but the first day it started happening for me was the 14th of february.

If its any help, when recording with screen capture the first few seconds of a recording are a black screen. Not to mention game capture does not work at all.


AutoLab presentation for Viewers

Will there be an intro course for this series that runs through the Autolab, or are we expected to set it up for viewer during first/last module?

I will be making a video setup guide for students in a couple of weeks after we finish adding a couple more configs

Jason Helmick
Author/Evangelist | Pluralsight
www.pluralsight.com

On Oct 3, 2016, at 2:22 PM, themichaelbender<EMAIL_ADDRESS>wrote:
Will there be an intro course for this series that runs through the Autolab, or are we expected to set it up for viewer during first/last module?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


exploited XSS, via URL strings

I left the demo window open for a bit, and got multiple javascript alerts as a result.
https://gist.github.com/1521661

I noticed this too. I'm not sure where it's not being escaped. Best guess is reading from the log isn't escaped so I'm running that as escaped on the server. Thanks!

In your JavaScript. You're not using Jinja2 filling the page, so no |raw.

How about this? #e4e521084e

I'm not sure, https://github.com/maxcountryman/logmon/pull/2 vaguely seems more sensible to me, but I can't back that up right now.

All right, I'd be happy to merge that if you think that makes more sense. It seems more logical to escape in JS itself but I guess because Flask is what I know I reached for that first.


Refactor maintainer data and types into own file

Describe your changes
Moved maintainer data into it's own file and created a new IMaintainer type which can be used in both About page and Maintainer Card component. Renamed maintainerData to currentMaintainerData in case we have maintainers that leave in the future then a pastMaintainderData array can be added too.

[x] I have performed a self-review of my code.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.
[x] I have checked my code with an automatic accessibility tool such as Axe Dev Tools or Wave
and it shows no errors.

A question. We are using alt={fullName} for the image.
To make a more descriptive ALT text. Isn't it better to have an imageAlt property in the maintainer data object, where we can be more accurate? (i.e. "Maintainer x smiling in front of a dark background")

A question. We are using alt={fullName} for the image.
To make a more descriptive alt text. Isn't it better to have an imageAlt property in the maintainer data object, where we can be more accurate? (i.e. "Maintainer x smiling in front of a dark background")

Hmm, I was wondering about this but I’m not sure how much detail is really needed. But definitely something to consider.


Feature Request: make color configurable

Would it be possible to make the main color of the webinterface configurable somewhere in the deployment?
Use-case: when running multiple instances of k8dash in test and production environments, I would like to make the differences between the environments visually very clear

Changing the global scss variable $color-accent seems to be enough (at least it is for the proposed use case). The question now is how can we make it available at runtime ? I'll try to investigate some solutions ;)

We've recently added a "dark mode" that can be enabled by going to "Profile" and toggling "Use Dark Mode". I know that's not quite what was asked for here, but hopefully it's at least a useable workaround for the time being.


Add explicit --boot-disk-auto-delete to GCE VM creation.

LGTM. Is this not working right now? I thought --boot-disk-auto-delete was the default.

It is the default, so there's no problem right now. @voellm suggested we make it explicit.

Sounds good. Thanks.


Frame Processor Error: Regular javascript function '' cannot be shared. Try decorating the function with the 'worklet' keyword to allow the javascript function to be used as a worklet., js engine: VisionCamer

Hi, thanks very much for this pluggin.
I'm persistently getting:
Frame Processor Error: Regular javascript function '' cannot be shared. Try decorating the function with the 'worklet' keyword to allow the javascript function to be used as a worklet., js engine: VisionCamera
When running the example code on a dev build (expo).
Any pointers would be greatly appreciated.
Tom

What is your frame processor function look like?
I've updated the example to vision camera 3.9.0. You can try to update it and try again.

Thanks, It was the anonymous function that was triggering it I think. I ended up copying your MRZ specific example (https://github.com/tony-xlh/react-native-mrz-scanner) and it compiled. I actually had a lot of trouble getting it to recognise MRZ (perhaps to do with model and custom template). So I've ended up creating my demo another way. Will come back to Dynamasoft if we take the feature into production.

@xulihang Why it is showing license invalid and not recongizing the label if i have tried to clone it as well but it shows invalid license alert
and this output
LOG  frame width:1920
LOG  frame height:1080
LOG  {}


Package rke2: request for improvements and documentation

Issue description
Installing the package does not run rke2 - and there is no config or docs to do so.
IMO it should follow the same pattern as k3s
services.rke2.enable = true;
services.rke2.role = "server";

Steps to reproduce
rke2 server
  environment.systemPackages = [
    pkgs.rke2

[root@nixos:~]# systemctl enable rke2-server.service
Failed to enable unit: Unit file rke2-server.service does not exist.

[root@nixos:~]# systemctl start rke2-server.service
Failed to start rke2-server.service: Unit rke2-server.service not found.

[root@nixos:~]# systemctl enable rke2-agent.service
Failed to enable unit: Unit file rke2-agent.service does not exist.

Starting it with rke2 server it still fails to be available as default
[nix-shell:~]# kubectl get pods
E0417 22:10:03.701350 1986591 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0417 22:10:03.701573 1986591 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0417 22:10:03.704130 1986591 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0417 22:10:03.705225 1986591 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
E0417 22:10:03.706016 1986591 memcache.go:265] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp [::1]:8080: connect: connection refused
The connection to the server localhost:8080 was refused - did you specify the right host or port?

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/fl9z1dgjpqg4ywr85d59hs74v4003s8q-nix-info
copying path '/nix/store/fl9z1dgjpqg4ywr85d59hs74v4003s8q-nix-info' from 'https://cache.nixos.org'...
 - system: `"aarch64-linux"`
 - host os: `Linux 6.6.18-hardened1, NixOS, 24.05 (Uakari), 24.05.20240410.1042fd8`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.19.2`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/root/.nix-defexpr/channels/nixpkgs`

You may be confusing a package with a NixOS module. K3s has a NixOS module.
Rke2 might not have a NixOS module yet. No mention at:

https://github.com/NixOS/nixpkgs/tree/master/nixos/modules/services/cluster

#308090

Should be available soon in a channel near you: https://nixpk.gs/pr-tracker.html?pr=308090


[gui error report] Error: Initializing daemon...

👉 Please describe what you were doing when this error happened.
Specifications

OS: linux
IPFS Desktop Version: 0.23.0
Electron Version: 19.0.9
Chrome Version: 102.0.5005.167

Error
Error: Initializing daemon...
Kubo version: 0.15.0
Repo version: 12
System version: amd64/linux
Golang version: go1.18.5

Error: Your programs version (12) is lower than your repos (13).
Please update ipfs to a version that supports the existing repo, or run
a migration in reverse.

See https://github.com/ipfs/fs-repo-migrations/blob/master/run.md for details.

    at errorTemplate (/opt/IPFS Desktop/resources/app.asar/src/daemon/migration-prompt.js:78:49)
    at Object.loadWindow (/opt/IPFS Desktop/resources/app.asar/src/daemon/migration-prompt.js:129:26)
    at startIpfsWithLogs (/opt/IPFS Desktop/resources/app.asar/src/daemon/daemon.js:191:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async startDaemon (/opt/IPFS Desktop/resources/app.asar/src/daemon/daemon.js:230:21)
    at async startIpfs (/opt/IPFS Desktop/resources/app.asar/src/daemon/index.js:48:17)
    at async setupDaemon (/opt/IPFS Desktop/resources/app.asar/src/daemon/index.js:116:3)
    at async run (/opt/IPFS Desktop/resources/app.asar/src/index.js:85:5)

I have a temporary solution while this gets worked out. Currently using the terminal version just to run the daemon, and then using a plugin to run the desktop version of IPFS.
My use case is reading RSS files and making them accessable as an alternative to port forwarding my local server, which I'd prefer not opening up to the public.

@LWFlouisa thanks for reporting this issue!
If you update IPFS Desktop it should resolve this error.  Your IPFS Desktop version is 0.23.0 but we're on 0.27.2 now, which won't show you the repo migration warning you're currently getting.
The latest repo version is 13 IIRC, and it seems you have ran a version of kubo with that repo version on your machine, which ran the repo migration.
Please let us know if you run into any trouble

Yea I changed out the old dataset and the issue seems to have resolved.


"" not found (check_syntax)

There are quite a few of this error in my project. It probably also causes other dependent issues.
I'll try to add the different variations below.
Example:
./src/ecc/#jcs#bc_dynamic_selections.prog.abap[18, 1]                      - "rsdynsel" not found (check_syntax)  1)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[66, 7]                      - Not a structure, target (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[104, 5]                     - Not a structure, target (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[111, 5]                     - "node" not found (check_syntax) 2)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[112, 7]                     - Not a structure, target (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[217, 5]                     - Not a structure, FieldChain (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[227, 5]                     - Not a structure, FieldChain (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[227, 5]                     - Not a structure, FieldChain (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[237, 5]                     - Not a structure, FieldChain (check_syntax)  ./src/ecc/#jcs#bc_dynamic_selections.prog.abap[237, 5]                     - Not a structure, FieldChain (check_syntax)

rsdynsel exist as TABL in dependency repository

node is defined as RANGE

se16n_seltab exists as TABL in dependency repository
./src/ecc/#jcs#bc_se16n_batch.prog.abap[76, 1]                             - "se16n_seltab" not found (check_syntax)

smpdyntext exists as TABL in dependency repository
 ./src/ecc/#jcs#bc_se16n_batch.prog.abap[280, 1]                            - "smp_dyntxt" not found (check_syntax)

1: is data element DYNSELID and domain DYNSELID also in the dependencies?

Interesting... the DTEL is showing up in dependencies but is not generated as file. (used List Deps with zip).
No domain either.

okay, there is a problem with LIKE typing in abaplint

try again, 2.38.10

Interesting... from example above, the "rsdynsel" issue is gone :-) but the "node" one is still there.
Many of the not found messages are gone.
There is now also trace information available. Including messages like this:
* .INCLU--AP TYPE Unknown type: Table COBL, todo, group named INCLUDE.
I'm attaching the complete run's information.
x.txt

argh, the "trace information" should not be there, probably a console.log somewhere in the code

ahh, its just very long error descriptions :)

if it helps... is there a how-to-debug-abaplint guide?
As you can see, there are still a lot of strange messages in the log.

no, not really,
first I typically try to reproduce the issue on the playground, https://playground.abaplint.org
if it doesnt work, or for reproducing the issue requires multiple files, I make an empty folder and copy the files there. Also recommend the https://marketplace.visualstudio.com/items?itemName=larshp.vscode-abaplint, but note it does not always run the latest version.

Verified.
Created seperate issue #1122 for the ranges issue.


Text omitted for embedded tags

Hello,
Trying to load file :
<root> 	<tag1>Hello <tag2>the text</tag2> World!</tag1> </root>
The code is pretty straightforward:
df=spark.read.format('com.databricks.spark.xml')\
  .option('mode', 'FAILFAST')\
  .option('rowTag', 'root')\
  .load('/mnt/datalake/sample.xml')

df.show()

The text "World!" is skipped, output:
+------+--------+
|  tag1|    tag2|
+------+--------+
|Hello |the text|
+------+--------+
May you advice any workarounds please? Not sure if the root cause is the same as in https://github.com/databricks/spark-xml/issues/428.
Regards,
Radislav

Yeah, this isn't supported. Part of the issue is the semantics: what would the right type for tag1 be? Does it have a single string member with all that text, or two, or an array?
It would be reasonably hard to support, I think, knowing how the code works -- which doesn't mean it shouldn't be. But I also wonder how common this is in the type of XML one normally feeds. That's "HTML-like" XML, whereas usually we see tags with only a value child or only tag children in XML that encodes structured data.
Not quite the same thing as #428 but similar challenges and questions.

Yeah, this isn't supported. Part of the issue is the semantics: what would the right type for tag1 be? Does it have a single string member with all that text, or two, or an array?
It would be reasonably hard to support, I think, knowing how the code works -- which doesn't mean it shouldn't be. But I also wonder how common this is in the type of XML one normally feeds. That's "HTML-like" XML, whereas usually we see tags with only a value child or only tag children in XML that encodes structured data.
Not quite the same thing as #428 but similar challenges and questions.

Part of the issue is the semantics: what would the right type for tag1 be?

Would it be possible to use custom schema as a hint?

how common this is in the type of XML one normally feeds

Not so many. In my particular case it is about 1% of the overall, however in absolute values there are millions of files.

Part of the issue is the semantics: what would the right type for tag1 be?

Would it be possible to use custom schema as a hint?

how common this is in the type of XML one normally feeds

Not so many. In my particular case it is about 1% of the overall, however in absolute values there are millions of files.

No it just doesn't parse this at the moment. You can take a look at the code and see if there is actually a clean way to handle this case.

No it just doesn't parse this at the moment. You can take a look at the code and see if there is actually a clean way to handle this case.

It's already been noted that this isn't supported. I just want to note that this isn't isolated to the element with embedded tags. There can be other fatal side effects, for example:
<Parent>
  <Child1>Child1</Child1>
  <Child2>
    <ElementWithEmbeddedTags>
      BeforeText
      <SomeTag/>
    </ElementWithEmbeddedTags>
  </Child2>
  <Child3>Child3</Child3>
</Parent>

In this case, when ElementWithEmbeddedTags is the child of top level Child2, doing a df.select on any further element, like Child3 will produce null. This isn't the case if ElementWithEmbeddedTags is itself a "top level" element. This is assuming <Parent> is your rowTag.

No, it just doesn't do this. This is 'unusual' for the type of XML that is typically used to encode tabular-like data. The semantics are not clear. It's not crazy or impossible to do, just hard to do, so don't think it's going to happen unless someone comes up with a clean solution


[WIP] feat: 🎸 add dataset viewer

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

see https://github.com/huggingface/moon-landing/issues/3308 and https://github.com/huggingface/datasets/issues/4538

lgtm


Build on ARM

Fixes #85

Hi, could we get this merge? I'm working on a side project utilizing pgvectorscale, but unable to test it locally since the current ARM build of the TimescaleDB Docker image is throwing an error that the extension is not found.

note the run https://github.com/timescale/timescaledb-docker-ha/pull/472 to build into docker.


Absolute Url

Hello everyone how not found any solution to change relative links of the request returns, created a plugin for this available in https://github.com/andrehrf/superagent-absoluteurl would like to ask everyone to look at the code and will improve increasingly =)

You might consider coding this as a superagent plugin that works with request.use(absoluteUrlPlugin) and then submitting a PR to add your plugin to the superagent README.
Aside: I think your module might need to support the obscure HTML <base> tag to correctly handle the (rare) pages that make use of it.
Aside: Your repo has misspelled "examples" in the directory name.
Looks like a neat plugin!

https://gist.github.com/Valeronlol/eac4fca14d485592070904c02629ad33


Ping time is hardcoded and should be a variable

When testing against a server bash performs a ping and the interval is hardcoded.  This should become a variable as some routers may see this as a flood and block the traffic.

https://github.com/covermymeds/dbdeployer/pull/41 resolves this issue. It will go out with the next version in the next couple of days.


March 2020 Release Plan (v1.5.0)

Timeline

March 6, 2020: Code freeze
March 9, 2020 - March 13, 2020: Release Candidate(s) and testing
March 18, 2020: Full Release

Features
Styles

Expressive Theme Foundation (#712 )

React
Production level and accessibility testing performed

Table of Contents (#984 )
Card section - simple (#1147 )
Card section - with images (#1148 )
Lead space (#1149 )
Content block - mixed groups (#1151 )
Content block - segmented (#1152 )
Content block - simple (#1153 )
Content block - with media (#1154 )
Content group - simple (#1156 )
Content group - with cards (#1157  )
Content group - with pictograms (#1158  )
Feature Card (#1138 )

Sub-patterns

Card (#1202 )
Content section (#1198 )
Link List (#1254 )

Services

Service to grab the video metadata (#1065 )

Feature Requests

Nested layout sub-pattern modifications (#1581 )
- Content block - segmented:  the supporting link feature is missing (#1541 )

Defect Fixes

Leadspace: Content not staying aligned to the grid (#1182 )
Leadspace: Top spacing fixes (#1184 )
Leadspace-centered (no image) - Bottom spacing fixes (#1186 )
Leadspace: Button text and icon size (#1187 )
CSS for Content Block Patterns is missing some styles (#1488 )
Content Block Patterns have the same data-autoid value (#1490 )
Content Block Patterns seems to be missing Grid Styles (#1537 )
Content Block Simple pattern does not have caption for image (#1589 )
Content Block Media is not generating all css (#1592 )
Card Section Simple has a generic data-autoid (#1621)
Card Section Simple Pattern seems to be missing CSS (#1624 )
Link List Top and Bottom Spacing (#1667 )
CTA is no longer rendering URL when style is text (#1682 )

The release has been done,
https://github.com/carbon-design-system/ibm-dotcom-library/releases/tag/v1.5.0
Thanks!


Speakers announcement design

In Progress: Notion Page:
https://twitter.com/i/status/1722667870352515315


[BUG][monitoring rebase] alertmanager failed to send alerts in rke2 clusters

Rancher Server Setup

Rancher version: 2.6-head commit ID 4bf459d
Installation option (Docker install/Helm Chart): helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): rke2

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.24
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions: admin

Describe the bug
after installing rancher-monitoring:100.2.0+up40.1.2 and rancher-alerting-drivers:100.0.4, configured alerts are not being sent - error in alertmanager pod:
ts=2023-01-24T19:26:38.004Z caller=dispatch.go:354 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="default/amc-7/prom2sms/webhook[0]: notify retry canceled due to unrecoverable error after 1 attempts: unexpected status code 400: http://rancher-alerting-drivers-sachet.cattle-monitoring-system.svc:9876/alert"
To Reproduce

rke2 cluster (tested on windows & non-windows - occurred on both)
install monitoring
install alerting-drivers
create generic webhook receiver
once alert is created, check alertmanager pod logs

Result
error - alerts not sent
Expected Result
alerts sent
Screenshots

Additional context

error is occurring in rke1 clusters, too. this is from an azure hosted cluster
ts=2023-01-26T16:14:39.168Z caller=dispatch.go:354 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="default/amc-7/prom2sms/webhook[0]: notify retry canceled due to unrecoverable error after 1 attempts: unexpected status code 400: http://rancher-alerting-drivers-sachet.cattle-monitoring-system.svc:9876/alert"

Closing this as a documentation issue
Rancher v2.6-head commit ID b465433
confirmed receiving alerts in azure aks hosted and ubuntu rke2 downstream.
rancher-alerting-drivers:100.0.4
rancher-monitoring:100.2.0+up40.1.2


Need more error information - batch documents as a stream

We recently upgraded an application in production that was using Firestore and now we are seeing errors like the below in the log.  Is there a particular method that would cause this issue?  It is very difficult to troubleshoot without additional information or context.
err: "Invalid serialization: FirestoreErrorPublicGenericDetails { code: "SerializationError" }"
message: "Error occurred while consuming batch documents as a stream."

Doing a bit more research, it looks like the deserialization error is getting hidden because the code is using the function batch instead of batch_with_errors

Hey,
I recommend for stream operations use _with_errors variant since the errors can occur while reading documents and the 'simple' version of function will just log it which can be dangerous in some cases.
So, _with_errors and use relevant try/try_collext is the right way to work with this.

I'll check the display implementation for serialisation errors. Seems it missing some output.

In the Firestore deserialization error, it would be very helpful to have the document ID of the record that is attempting to be deserialized.

This version should help with error messages and details:
https://github.com/abdolence/firestore-rs/releases/tag/v0.39.2

Thank you.  I will let you know if I have more issues.


fix tag error

What changed? Why?
The check for tags in tutorials doesn't have a check to ensure the tags field exists. This change returns false if no tags field exists on the docs metadata
Notes to reviewers
How has it been tested?
Built site locally, confirmed fix works
Does this PR add a new token to the bridge?
no
Are you adding an entry to assets.ts?

[X] No, this PR does not add a new token to the bridge
[ ] Yes, and I've confirmed this token doesn't use a bridge override
[ ] Yes, and I've confirmed this token is an OptimismMintableERC20

If you are adding a token to the bridge, please include evidence of both confirmations above for your reviewers.

Changes are here too:
https://github.com/base-org/web/pull/435


Using VPN Gateway or Equivalent with Service Fabric Mesh

Is it possible for Service Fabric Mesh to use Azure VPN Gateway in order to communicate with an on-prem network?
Some of our services are currently relying on Anti-Corruption Layers.
I've noticed that Service Fabric Mesh's network resource isn't the same as the Virtual Network that you use to setup a VPN Gateway.

"Bring you own Vnet" capability is planned to come in around OCT timeframe. At that time the network resource will have configurations that will allow for this capblity to be requested.
You are right, the current Mesh network resource is not the same as the networking capabilities that  Azure Networking provides.  We are working with Azure networking to enable BYO-VNET integration. The current Mesh networks will also evolve to supprot layer 7 capabilities, like mulitple ingress for services etc. We will also provide an isolated application scoped VNET as well.

That's sounds awesome!
Keep up the good work Chacko


Add dark mode and always use blue as accent color

Fixes #9
Fixes #5
Changes:

introduces light mode
changes dark mode to use blue for navigation badges

ToDo:

[ ] Change header image so that it fits to light mode
[ ] Change navbar logo to have dark text

This is still a work in progress. A few screenshots of what I have so far:

It's great! Especially the background color, which is not completely white and is similar to the color of cream. I think the header photo is good and does not need to be changed, however, forgive me for causing you trouble.

I think that the header image itself can work fine if the gradient overlay is adjusted.  Rather than the top end of the gradient being completely transparent, I would make it partially transparent instead such as rgba(241, 241, 241, 0.63).  The goal is to decrease the contrast between the header and the banner so that it is less hard on the eyes.  Replacing with a light version of the image may also work.
Note: The logo is orange because I inverted the color so have the text be black when reviewing.  Also, this screenshot isn't of your build, but me just updating some of the CSS values in the inspector for a quick feel so some things may not match.

I also think the background color of the header is just a tad too dark.  I think a lighter color like #ededed as I implemented above would help.
Other than that, I think it looks great!  Thank you for the hard work!

Thanks for the feedback @jwmcgettigan!
I've adjusted the header to #ededed and used a darker font color for the logo. I've also created a new light mode version of the search background (see godotengine/godot-design#72).
In addition I made quaternary-low a bit darker and highlight-background brighter to increase the contrast on hovered navigation buttons and filled form input fields for better readability.

From my side this is good to be merged! :tada:

If approved, will merge during Planned Maintenance

At least on my phone, this seems lack in contrast for the active tab, I can't tell at a glance that "New" is selected, unlike the light theme.

I should have clarified, "Latest" is the selected item. "New" is showing the subtle mouse hover effect.


Fix portal blending with portal surfaces

Enables writing to color buffer when rendering the portal surface in FinalisePortalCommand::ExecuteSelf(). Changes glStencilOp to avoid stencil test failing in light pass, and clears stencil buffer in case there were multiple portals on the screen. Fixes #1011.
glStencilOp( GL_KEEP, GL_KEEP, GL_INCR ) was likely intended to prevent incorrect rendering with multiple portals on the screen, but it was preventing the portal surface from rendering.
Here's some screenshots:

The colour is slightly off because for some reason the portal surfaces are rendered twice, so they blend with themselves. The colour is off due to light pass, which may or may not be a bug with the lighting system. Using Nsight confirms that the first draw produces the expected result (this is not the final colour that will be in the frame, but it illustrates the issue):

Awesome ! 😲

The colour is slightly off because for some reason the portal surfaces are rendered twice, so they blend with themselves. Using Nsight confirms that the first draw produces the expected result (this is not the final colour that will be in the frame, but it illustrates the issue):

Apparently that is the light stage. If the output from it is incorrect, then that issue is likely with lighting. It could also be that tremulous was rendering it wrong.

glStencilOp( GL_KEEP, GL_KEEP, GL_DECR ) was likely intended to prevent incorrect rendering with multiple portals on the screen, but it was preventing the portal surface from rendering.

On second thought, that might be for recursive portals. This pr might break them, but we don't seem to have any maps with those, so I'd need a test case map for that.

On second thought, that might be for recursive portals. This pr might break them, but we don't seem to have any maps with those, so I'd need a test case map for that.

spacetracks map, viewpos 128 -1950 602 90 0, you should see the two mirrors at the same time on the left and on the right.

spacetracks map, viewpos 128 -1950 602 90 0, you should see the two mirrors at the same time on the left and on the right.

They don't seem to be in view of each other though? I've just got a test map from @cu-kai and it looks like recursive portals don't even work in 0.54.1.

It probably works, but not entirely. The fact we can see the two mirrors of spacetracks at the same time in 0.54.1 is only possible because I enabled portal recursion some time ago. If we disable recursion, only one portal is displayed at a time and a warning is printed.
I also made a test map and recursion seems to only be half working.

Interesting, I think it might be linked to one of the issues described in https://github.com/DaemonEngine/Daemon/issues/1005#issuecomment-1879504829, since it looks like things that should be drawn through a recursive portal aren't added correctly.

Apparently that is the light stage. If the output from it is incorrect, then that issue is likely with lighting. It could also be that tremulous was rendering it wrong.

For now ignore any lightmap issue, there is an ongoing PR to fix many lightmap issues and trying to fix lightmap issues before merging that other PR is likely to introduce more bugs and create strong merge conflicts:

https://github.com/DaemonEngine/Daemon/pull/992

Also I have some ideas about why there may be lightmap issues with portals, but I will not investigate those ideas before that lightmap PR is merged.
So any lightmap issue remaining when rendering portals should not block the merge of that PR.

Ah, that's good to hear.

Closing this in favour of #1031, which fixes this issue among some other things without breaking recursive portals.


chore: add gpu_fakedev to the extensions list

Add gpu_fakedev to the extensions list

@tkatila, what do you think, does it make sense to mention fakedev in extensions list doc?

Well, there are also items like "Intel FPGA test tool" in the extensions. So GPU fakedev would be fine in that sense. We're also missing the xpumanager-sidecar from the list.

Well, there are also items like "Intel FPGA test tool" in the extensions. So GPU fakedev would be fine in that sense.

Maybe there should be a separate testing tools list?

We're also missing the xpumanager-sidecar from the list.

Oh, that should definitely be in the extension list.

Well, there are also items like "Intel FPGA test tool" in the extensions. So GPU fakedev would be fine in that sense.

Maybe there should be a separate testing tools list?

I don't think that's necessary.
@hugo-syn can you also add xpu-manager-sidecar to the extensions list. Then I'm fine to merge this.

added a TODO in #1045 and closing


FIX: Misconfigured test of unwarping workflow

The test seemed broken after a PR was merged (see #169), but that PR actually improved the implementation.
Here, the coefficients file is rewritten with a newly generated file from TOPUP and after applying TOPUPCoeffReorient.

Okay, test looking better now - https://2316-189236569-gh.circle-artifacts.com/0/tmp/tests/sdcflows/sub-101006/figures/sub-101006_task-rest_dir-LR_desc-corrected_bold.svg


Don't need to link projects in Sample/Example(s)

Many react native libraries has a Sample/Example(s) folder and has iOS/android project in it(e.g. https://github.com/race604/react-native-viewpager), we don't need to link that projects.
Before https://github.com/rnpm/rnpm/commit/3b03abf39ad477554a5aa22f2f30146b1357d77a#diff-2683058bcad92e1272ec697fed36ba90R22 rnpm ignores Test/Example(s) folders, why remove that? I suggest add Test/Example/Sample to ignore lists.

We still have this check here: https://github.com/rnpm/rnpm/commit/3b03abf39ad477554a5aa22f2f30146b1357d77a#diff-2683058bcad92e1272ec697fed36ba90R40

Do you have any issues after that, @cpunion?

@Kureev Thanks. Could you please add sample into it?

Sure. But I'm a bit busy now, can't give you an ETA for this.
Probably, you can find some time to compose a PR? I would be glad to merge it 👍

@Kureev I will try.

Merged, thanks!


ENH: automatic events param in EpochsArray

closes https://github.com/mne-tools/mne-python/issues/3189

oh no, what did I do here...

Time for a cherry pick :)


Authentication Issue: Unable to Fetch FHIR Resources After Deploying DICOM Cast and Syncing Metadata into FHIR Server for Azure

I've deployed DICOM Cast successfully. And I have performed all the steps of Sync Medical Imaging Server for DICOM metadata into FHIR Server for Azure.
I am unable to fetch FHIR resources when I setup authentication in the FHIR service.

As you can see in the below diagram, If I skipped step no 3 then I'm able to fetch FHIR resources.

I cannot fetch FHIR resources when I set up authentication in the FHIR service. It is working fine when I skip that step. Do you have any ideas on why I'm unable to get FHIR resources?

@NancyOsalaniya what type of FHIR service are you using?

@KnicKnic I'm using Azure Health Data Services' FHIR service.


Various fixes, multi threaded and support for HA using DDB

Hi,
I know these are a lot of changes. But this plugin is under some moderate load on a testing environment and is now working great.
The DynamoDB option, is for HA, this can be accomplished also by using EFS. Yet it's probably a better option for multi-threads

As you are using DynamoDB, can you please update Readme.md to have the correct policy for using DynamoDB (update table state_ddb_table).

+1 Please merge this!

Hi,
First of all the DDB option is optional.
I do think it brings plugin stability and speed. Of course there are other options also, like still keeping the state on files that live on an NFS. DDB is faster and lowers the probability of a race condition (thought each thread consumes it's own stream and each stream is kept on a different file).
I did not understand what is the approach on HA made by out_elasticsearch_dynamic, could you detail that?

I did not understand what is the approach on HA made by out_elasticsearch_dynamic, could you detail that?

This approach is similar to out_elasticsearch_dynamic.

+1 any idea when this would be merged


remove index.d.ts

this should be generated from typescript now

We should PR either this or the generated index.d.ts against definitely-typed to make these types widely available. We've been putting that off for far too long. I'll take that on.


Complete Happy Number and Valid Parentheses

I found 3/2=1.5, 3//2=1. So I updated "/" to "//" in previous programs.

lgtm


Failure in test_collections during CI

Saw the following failure during CI for netcoreapp2.1.CPython.test_collections:
FAILED (errors=1)
  X netcoreapp2.1.CPython.test_collections [3s 239ms]
  Error Message:
   Traceback (most recent call last):
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\test\test_collections.py", line 1293, in test_main
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\test\support\__init__.py", line 1812, in run_unittest
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\test\support\__init__.py", line 1787, in _run_suite
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\test\test_collections.py", line 1298, in <module>
test.support.TestFailed: Traceback (most recent call last):
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\test\test_collections.py", line 313, in test_pickle
    q = loads(dumps(p, protocol))
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\pickle.py", line 1558, in _loads
    return _Unpickler(file, fix_imports=fix_imports,
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\pickle.py", line 1041, in load
    dispatch[key[0]](self)
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\pickle.py", line 1336, in load_global
    klass = self.find_class(module, name)
  File "D:\a\ironpython3\ironpython3\Src\StdLib\Lib\pickle.py", line 1386, in find_class
    __import__(module, level=0)
ImportError: No module named 'copy_reg'

Occured again on Windows net46.test_collections.


Question: WinUI3 - Windows.Current is null, then how to set mouse pointer position?

Describe the bug
Maybe similar to this question: https://github.com/microsoft/microsoft-ui-xaml/issues/2609
But here, it is focused on moving mouse pointer to a position.
For UWP, I did like this:
Window.Current.CoreWindow.PointerPosition = new Point(x, (y + 25.0) / scaleMultiplier);
But VS complains that Window.Current is null.
From this issue, https://github.com/microsoft/microsoft-ui-xaml/issues/4152
I see that ProtectedCursor is there, but it is for the shape of mouse pointer.
Then, how can I move mouse pointer?
Steps to reproduce the bug
Steps to reproduce the behavior:

Create new Blank App, Packaged (WinUI in Desktop) app
Open MainWindow.xaml.cs and try get Window.Current

Expected behavior
Window.Current in Blank App, Packaged (WinUI in UWP) returns a window instance
Screenshots
No response
NuGet package version
No response
Windows version
No response
Additional context
No response

CoreWindow is documented to be outright unsupported in desktop applications. So yes, this is expected.
As for moving the mouse pointer, don't forget, you are now in desktop application land. This means that SetCursorPos is available. The .NET framework also has classes like Cursor, or you could just platform invoke SetCursorPos.

@DarranRowe Could you provide me with some sample code to do 'platform invoke' for WinUI (.Net6)?

It is no different from using platform invoke in any other situation.
You first set up the class to reference the native function:
namespace Win32
{
    public sealed class Win32Invoke
    {
        [DllImport("user32.dll")]
        [return: MarshalAs(UnmanagedType.Bool)]
        public static extern bool SetCursorPos(int x, int y);
    }
}

and just call it:
Win32.Win32Invoke.SetCursorPos(0, 0);
There is nothing special about the namespace or class name used here. You can also use using to get rid of the namespace name as you normally can with C#. The function name needs to match the exported if you don't provide the entry point in the DllImport attribute though.


DPicker removes attributes from

On interaction (opening/closing the picker, selecting a date, …), DPicker removes many attributes (such as class or data-*). I want to use classes to style the  field of the DPicker for my current project, but the classes are removed on every change.

Where are you adding classes? Can't you use the given classes instead?

I am adding the classes directly in the HTML to the  element.

Sorry for the long response time \o/.


ISPN-6379 Add event logs for backup sites failures

https://issues.jboss.org/browse/ISPN-6379

updated!

Pushed, thanks.


chore: Add affinity to the ndm-operator deployment

Pull Request template
Why is this PR required? What issue does it fix?:
This PR allows the user of the helm chart to set affinity to the ndm-operator pods.
What this PR does?:
Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Fixes #
[ ] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

PLEASE REMOVE BELOW INFORMATION BEFORE SUBMITTING
The PR title message must follow convention:
<type>(<scope>): <subject>.
Where:
Most common types are:
* feat        - for new features, not a new feature for build script
* fix         - for bug fixes or improvements, not a fix for build script
* chore       - changes not related to production code
* docs        - changes related to documentation
* style       - formatting, missing semi colons, linting fix etc; no significant production code changes
* test        - adding missing tests, refactoring tests; no production code change
* refactor    - refactoring production code, eg. renaming a variable or function name, there should not be any significant production code changes
* cherry-pick - if PR is merged in develop branch and raised to release branch(like v0.4.x)
IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@nearora-msft -- Thank you for this helm chart enhancement. We release a new chart for each new change, and as such need a version bump on the chart 'version'. Your PR needs a change in the https://github.com/openebs/node-disk-manager/blob/develop/deploy/helm/charts/Chart.yaml file, 'version' field. You'd have to bump the version to 2.1.1. The appVersion need not change, as there is no code change.
Also you'd have to sign your commit. We need our DCO!
I'll try to summarise the steps, but feel free to google this :P:

Check if you name and email is set on your git config

     get config --get user.email

If it's not set, set them using git config user.name "<your-name>", git config user.main "<your-email>
Sign your commit:

git commit --amend -s --no-edit


Public/Private properties unsupported

Hi. I have the following code:
export class TimeRule {

	/**
	 * Array with all the bitmasktimes.
	 */
	#bitmasktimes;

	/**
	 * Array with the checked bitmasktimes
	 */
	#checkedBitmasktimes;

	/**
	 * Array of the current intervals
	 */
	#intervals;

	/**
	 * Constructor method
	 *
	 * @param {array} bitmasktimes Object containing all the bitmasktimes
	 * @param {array} checkedBitmasktimes Object containing the checked bitmasktimes
	 * @return {void}
	 */
	constructor(bitmasktimes, checkedBitmasktimes) {
		this.#bitmasktimes = bitmasktimes;
		this.#checkedBitmasktimes = checkedBitmasktimes;
		this.#intervals = [];
	}
}

and when I run the JShint check, I'm getting the following errors:
..\www\js\es6\time_rule.js: line 6, col 3, Unexpected '#'. (E024)
..\www\js\es6\time_rule.js: line 6, col 16, Class properties must be methods. Expected '(' but instead saw ';'. (E054)
..\www\js\es6\time_rule.js: line 26, col 10, Expected an identifier and instead saw '#'. (E030)
..\www\js\es6\time_rule.js: line 26, col 10, Expected an assignment or function call and instead saw an expression. (W030)
..\www\js\es6\time_rule.js: line 26, col 11, Missing semicolon. (E058)
..\www\js\es6\time_rule.js: line 27, col 10, Expected an identifier and instead saw '#'. (E030)
..\www\js\es6\time_rule.js: line 27, col 10, Expected an assignment or function call and instead saw an expression. (W030)
..\www\js\es6\time_rule.js: line 27, col 11, Missing semicolon. (E058)
..\www\js\es6\time_rule.js: line 28, col 10, Expected an identifier and instead saw '#'. (E030)
..\www\js\es6\time_rule.js: line 28, col 10, Expected an assignment or function call and instead saw an expression. (W030)
..\www\js\es6\time_rule.js: line 28, col 11, Missing semicolon. (E058)
..\www\js\es6\time_rule.js: line 28, col 11, 'intervals' is not defined. (W117)

Is there any way to support public/private properties on classes?
I'm mentioning that I've already added esversion: 6 to the rules set.

PRs welcome :)

Any updates when JSHint start supporting private class fields and methods?
Spec: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Classes/Private_class_fields

Ping, still not supported?


ArduinoIDE not compiling

@imbeacon The ArduinoIDE does not compile with the newest version, because PlattformIo does get the right package, but for the v2.9 release in the ThingsBoard Library References the change from #include "PubSubClient.h" to #include "TBPubSubClient.h" does not seem to have been included.
/home/runner/Arduino/libraries/TBPubSubClient/src/TBPubSubClient.cpp:8:10: fatal error: PubSubClient.h: No such file or directory
    8 | #include "PubSubClient.h"
       |          ^~~~~~~~~~~~~~~~

This can be seen if the firmware is downloaded from this page, done when clicking on the version 2.9.0
Perhaps a new version 3.0.0 would need to be created to actually include these changes, because it seems like the version tag does not currently include that change.
Additionally if this is fixed it would be possible to enable the workflows again, once done I will open a small pull request that adjusts the documentation to the newest changes and fixes the workflow files being broken because the dependencies were not updated.

Hi @MathewHDYT ,
Thanks for information, please restart the Arduino IDE, and check for updates for the library. It should be OK in version 2.9.1 of TBPubSubClient.


New pages for marketing website

Page links on dev.
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/solutions/headless-cms/headless-cms-benefits/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/solutions/headless-cms/enterprise-headless-cms/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/solutions/headless-cms/digital-asset-management/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/solutions/headless-cms/commerce/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/solutions/headless-cms/headless-mobile/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/solutions/headless-cms/blogs-editorial/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/localization/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/cdn-hosting/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/digital-governance/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/elastic-scalability/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/headless-cms/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/multi-language/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/seo/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/features/multi-site-management/
https://zesty-dev-website-m3rbwjxm5q-uc.a.run.app/agency-partners/partner-program-directory/

Added optional chaining.


[Project] imodel react hooks

extracted from the it1 monorepo with some changes. Currently used in at least two known applications, which will need to migrate to the new name and public location. (this is a duplicate because I force pushed)

I forgot I have to manually run lint-staged and also send them a PR because lint-staged is broken on msys2. One moment

looks like my package needs an empty cover script, adding that


kie-issues#1019: React-Based DMN Editor: allowedValues not allowed for custom type

Closes: https://github.com/apache/incubator-kie-issues/issues/1019
Description
This PR enables constraints for several cases:

a refecence to struct (complex item definition);

a collection of a reference to struct (complex item definition);

a reference for a data type;

a collection of a reference for a data type;

a collection of a reference for a collection of a data type;

all cases above for the item components of a complex type

Additional fixes:

Changing between two or more data types that had an expression  constriant would not update the expression constraint according.

before:
https://github.com/apache/incubator-kie-tools/assets/24302289/e696532a-07bf-4451-89de-bb0bf4e35c46
after:
https://github.com/apache/incubator-kie-tools/assets/24302289/a40eeda9-e88d-4241-92b3-911fe7983b5c

@jomarko Thanks a lot for your review! I've addressed the first problem on my latest commit. Could you please review again?
Regarding the second problem, it was addressed on https://github.com/apache/incubator-kie-drools/pull/5858 . To check it yourself, you can change the KOGITO_RUNTIME_version env variable on root-env/env/index.js  to<PHONE_NUMBER>8-SNAPSHOT, build the extended-services-java with pnpm build:dev and start it.

@ljmotta thank you for the updates.
01 Clearing the Collection constraint expression
Now works, if the user clears the expression, the toggle group button is not switched to none anymore.
However, there is one interesting issue, in the Expression constraint widget. The monaco suggestions are displayed just every odd hit of the keyboard. Imagine you want to type count word:

c: monaco is shown
o: monaco is hidden
u: monaco is shown
n: monaco is hidden
t: monaco is shown

02 Constraints and runner
Thank you for the guidance of using latest available KOGITO_RUNTIME_version. I can confirm I was able to put a Collection item constraint with extended services connected to the sandbox. However now I see the same behavior if I try to put some Collection constraint expression in a moment, when given data type is already in use (assigned as data type to any node). Is that expected? I mean there is no KOGITO_RUNTIME_version available that would contain a fix for it?

@jomarko I've addressed the issue. It took more than I thought. I've also made a deploy of it: https://ljmotta.github.io/kie-tools-demos/kie-issues-1019/#/


Added JSSATEN

My college is JSS Academy of Technical Education In Noida, India. Please add it.

@JINDALG Pull request merged. Thank you!


Fix broken links

Thanks!  The version thingy got fixed in #160.  By random chance I happened to see #159 first, which does the same thing here.  But thank you regardless.


Tests: Fix user regional settings dependent unit tests

Description
Due to the current CultureInfo() constructor usages, unit tests are prone to fail if a test-running-machine has custom regional settings.
float1.Culture = new CultureInfo("en-US");
CultureInfo constructor has an optional useUserOverride parameter which uses default culture settings of .net if false, and uses Windows user's custom regional settings if true. The problem is, it is true by default. So, all the tests with CultureInfo() constructors depend on potential custom user settings.
Solution
Always set 'useUserOverride' parameter to 'false' with CultureInfo constructors in unit test bodies in order to avoid the problem, and make tests more robust.
float1.Culture = new CultureInfo("en-US", false);

How Has This Been Tested?

Change your regional settings on your local computer (e.g., Short Date format)
Re-run unit tests on your local computer
Observe that user regional settings dependent tests fail. (e.g., BindingConverterTests.DefaultConverterTestWithCustomFormat)
example of a failing test: Expected dt1.Set(newDateTime(2020,11,03)) to be "11/03/2020", but "11-03-2020" differs near "-03" (index 2).

Windows 10 - Regional settings screen (Use this screen to customise settings)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Much appreciated!


Missing "back" (octicon-mail-reply) icon in link to parent directory of repo files

Gitea version (or commit ref): 1.13.0+dev-156-g777015c2c
Git version: N/A
Operating system: N/A
Database (use [x]): N/A

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[X] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
When in a repo's file listing, in a subdirectory, it looks like it was meant to show a back arrow icon next to the ".." at the top of the list to go back to the parent directory.
The code shows it should have octicon-mail-reply (Gitea code: https://github.com/go-gitea/gitea/blob/777015c2c2d2d74d98d15e145f9f8af04f1d9c8f/templates/repo/view_list.tmpl#L39) (icon: https://commons.wikimedia.org/wiki/File:Octicons-mail-reply.svg) but it comes up empty on both my copy of Gitea and on try.gitea.io (e.g. https://try.gitea.io/richmahn/cecil/src/branch/master/Mono)
Screenshots

Looks like they changed the name to just "reply"
https://primer.style/octicons/reply


Colocated jobs creating package name conflicts

It looks like on July 4th the following changes were made:
https://github.com/pivotal-cf/cf-redis-release/commit/e5a2eef7ab664618937202191c7e13696a3625e4
I believe this is the same issue descibed here:
https://github.com/pivotal-cf/cf-redis-release/issues/84
Colocating these jobs causes our deployment to fail just like it did before we updated the include the fix for the above issue. Here is the error we see:
Error: Package name collision detected in instance group 'cf-redis-broker': job 'cf-cli/cf-cli-6-linux' depends on package 'cf-cli/cf-cli-6-linux' with fingerprint 'af6517f138b3ed0584b263f6273f30f22f6e5c9d', job 'cf-redis/broker-registrar' depends on package 'cf-redis/cf-cli-6-linux' with fingerprint '0971d73c486be23f7d87702ddc75f4d6565942f5'. BOSH cannot currently collocate two packages with identical names and different fingerprints or dependencies.

Just thought i would give a heads up since my team happened to notice this.
Keenan

Hi @KeenanHoffman, is this still an issue? We noticed similar behaviour and made a further change that should have corrected this.


ecoMax800 is not connected

Is there an existing issue for this?

[X] I have searched the existing issues

I'm having the following issue:
Здравствуйте. Не удается подключить ecoMax 800R, он поддерживается данной интеграцией?

I have following devices connected:

[ ] ecoMAX 3xx series
[X] ecoMAX 8xx series
[ ] ecoMAX 9xx series
[ ] Expansion module B
[ ] Expansion module C
[ ] ecoSTER 200/ecoSTER Touch
[ ] ecoLAMBDA
[ ] ecoNET 300

I'm connecting to my devices using:
USB to RS-485 adapter
I'm seeing following log messages:
No response
My diagnostics data:
No response
Code of Conduct

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

Hi,
Thank you for the feedback!
Sorry for delayed response, was offline for a couple days.
I checked diagnostics data, that we've collected from different devices and it seems that nobody tried the integration with EM800R device before, so I sadly cannot definitely confirm, that it is supported.
That being said, I've looked at manual for your device and I think it should work fine with this integration. It supports standard Plum modules like ecoLAMBDA and ecoSTER, which is good indicator that your device uses standard EM (ecoMAX) protocol as opposed to newer GM3 (GazModem 3), which we don't support yet.
From the error message, I can see that integration sees the connection, but fails to receive expected response from the EM.
This usually indicates hw problem, so first things to try would be following:

swap D+ and D- wires around
try different rs485 ports (if your device has multiple)
try different USB to RS485 adapter, as FTB6SPL3 serial from your first screenshot, indicates that your adapter is built using fake FTDI chip, which are prone to failures (see first comment here)

Can you write the software version of ecoMAX 800 module?

Can you write the software version of ecoMAX 800 module?

panel v.01.20.22.B0
Module A: v.01.31.23.J1

swap D+ and D- wires around
try different rs485 ports (if your device has multiple)
try different USB to RS485 adapter, as FTB6SPL3 serial from your first screenshot, indicates that your adapter is built using fake FTDI chip, which are prone to failures (see first comment here)

D+ and D- changed, the problem remained. I tried connecting to both G1 and G2 - the result is the same. I'll try to order another adapter.

Can you write the software version of ecoMAX 800 module?

panel v.01.20.22.B0 Module A: v.01.31.23.J1

Unfortunately, this is the last version of the program which does not support econet300.

@Almaz90
Taziff has been very helpful with EM-related information during development of this project, so I think we can trust his information on the issue. I keep this open, just in case you get it working, but if EM doesn't support ecoNET, there's not a lot we can do sadly. Sorry.
@Taziff
Thank you for the info!

Module A program v01 is from 2013r. There is a newer program, but there is a high chance that the module will be damaged during the update.

Taziff has been very helpful with EM-related information during development of this project, so I think we can trust his information on the issue. I keep this open, just in case you get it working, but if EM doesn't support ecoNET, there's not a lot we can do sadly. Sorry.

I ordered USR-TCP232-304. If it doesn't work out with him, then we'll close the issue.

Good afternoon. USR-TCP232-304 did not help, the problem remained. It looks like there really is a problem with the firmware

Good afternoon. USR-TCP232-304 did not help, the problem remained. It looks like there really is a problem with the firmware

Hi,
Thank you very much for heads up, although I'm really sorry to hear that :(
I seems that I'll need to compile at least some sort of (un)supported device list...

There is newer software for this device that supports econet300, but there is a high probability that something will go wrong during the update.

There is newer software for this device that supports econet300, but there is a high probability that something will go wrong during the update.

Is it possible to revive the block later?

Hi same issue.
ecoMAX 800ec panel  v=17.10.01
Log:
ERROR (MainThread) [custom_components.plum_ecomax.config_flow]
Traceback (most recent call last):
File "/usr/local/lib/python3.13/asyncio/tasks.py", line 507, in wait_for
return await fut
^^^^^^^^^
File "/usr/local/lib/python3.13/asyncio/locks.py", line 213, in wait
await fut
asyncio.exceptions.CancelledError
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/plum_ecomax/config_flow.py", line 188, in async_step_identify
await self.identify_task
File "/config/custom_components/plum_ecomax/config_flow.py", line 264, in _async_identify_device
await self.connection.get(DeviceType.ECOMAX, timeout=DEFAULT_TIMEOUT),
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.13/site-packages/pyplumio/helpers/event_manager.py", line 66, in get
await self.wait_for(name, timeout=timeout)
File "/usr/local/lib/python3.13/site-packages/pyplumio/helpers/event_manager.py", line 52, in wait_for
await asyncio.wait_for(self.create_event(name).wait(), timeout=timeout)
File "/usr/local/lib/python3.13/asyncio/tasks.py", line 506, in wait_for
async with timeouts.timeout(timeout):
~~~~~~~~~~~~~~~~^^^^^^^^^
File "/usr/local/lib/python3.13/asyncio/timeouts.py", line 116, in aexit
raise TimeoutError from exc_val
TimeoutError


Added a retry timeout for confirmTransaction and a counter for counting the number of transactions confirmed.

Problem
Currently there is no retry timeout for confirmTransaction so I have added that,
Also added a counter for counting how many transactions are getting confirmed.

@Lioncat2002 Please wait for #9 to merge. This will cause a lot of merge issues.

@Lioncat2002 Please wait for #9 to merge. This will cause a lot of merge issues.

Okies


Travis spend 65% of test time on pip install

Most of the test time is lost on pip install which I believe spend 90% of the time building leidenalg, python-igraph and louvain.
It's not nice to wait that long to see a failing test that could be triggered in 1 minute.
Moreover these libraries are only use in notebooks, I'm not sure they are mandatory for the tests. We could just skip them in the notebook and keep the same coverage.

I address this in #482. We can still tests notebooks on the side but I think I mostly fixed the issue.


Added events in Shop package

Issue Reference
Added events in Shop package

@shivendra-webkul
Please resolve the Conflicts in the PR.


[Backport 2.x] [Tiered Caching] Stats rework (2/4): Gates CacheStatsHolder logic behind FeatureFlags.PLUGGABLE_CACHE setting

Original PR
Description
Part of tiered caching stats. Tiered/pluggable caching is currently an experimental feature with the feature flag PLUGGABLE_CACHE. We want to gate the new CacheStatsHolder logic, which is run in the search path within cache implementations, behind this feature flag in case of any unexpected behavior.
With this PR, if PLUGGABLE_CACHE is true, cache implementations use the existing CacheStatsHolder class to keep track of their stats. If it's false, they instead use a DummyCacheStatsHolder, which keeps track of count only. In this case, the new cache stats API exposed in this PR will return all-zero stats and not support aggregating by any levels.
Because TieredSpilloverCache will store its own stats in https://github.com/opensearch-project/OpenSearch/pull/13236/, NoopCacheStatsHolder will also be used in the individual tiers of the TieredSpilloverCache to avoid storing redundant stats.
Related Issues
Follow up to https://github.com/opensearch-project/OpenSearch/pull/12531.
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
~- [N/A] Public documentation issue/PR created~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@msfroh @dblock @andrross @reta Would anybody be able to approve this backport PR? There are no changes compared to the original PR.


{CosmosDB} fix for the client.CreateDatabaseAsync

Code should be cosmosClient.CreateDatabaseAsync instead of client.CreateDatabaseAsync.
Fixes Azure/azure-sdk-for-net#27135

Refer PR https://github.com/Azure/azure-cosmos-dotnet-v3/pull/3046


Failed validator doesn't get transformed

So I've just setup Dingo 0.9 on a new Project and it doesn't seem to transform failed validations or any exceptions besides ones with status code 500 anymore.
I send a request to an endpoint and if the validation fails I just want to return the errors nicely formatted like documented here https://github.com/dingo/api/wiki/Errors-And-Error-Responses.
$validator = validator()->make($request->all(), $rules]);

if ($validator->fails()) {
    throw new ValidationHttpException($validator->errors()); // also tried StoreResourceFailedException, same result
}

I just get the generic grey "Whoops, looks like something went wrong." error page for all exceptions besides ones that have a status code of 500.
On Dingo 0.8 it seemed to work just fine, now with Laravel 5.1 and Dingo 0.9 it won't handle the Exceptions anymore.
Any ideas what could be the issue that causes the exception not to be transformed?

Can you share more of the stack trace? I'm on the road at the moment
heading home so I'll be able to give you more feedback once I'm in front of
a computer.
On Thu, 23 Jul 2015 21:23 DraperStudio<EMAIL_ADDRESS>wrote:

So I've just setup Dingo 0.9 on a new Project and it doesn't seem to
transform failed validations or any exceptions besides ones with status
code 500 anymore.
I send a request to an endpoint and if the validation fails I just want to
return the errors nicely formatted like documented here
https://github.com/dingo/api/wiki/Errors-And-Error-Responses.
$validator = validator()->make($request->all(), $rules]);
if ($validator->fails()) {
throw new ValidationHttpException($validator->errors()); // also tried StoreResourceFailedException, same result
}
I just get the generic grey "Whoops, looks like something went wrong."
error page for all exceptions besides ones that have a status code of 500.
On Dingo 0.8 it seemed to work just fine, now with Laravel 5.1 and Dingo
0.9 it won't handle the Exceptions anymore.
Any ideas what could be the issue that causes the exception not to be
transformed?
—
Reply to this email directly or view it on GitHub
https://github.com/dingo/api/issues/535.

That is the stack trace https://gist.github.com/DraperStudio/e6643da5fe07a12732cd
The ApiRequestTrait is what you once posted here in an issue.
<?php

namespace App\Traits;

use Dingo\Api\Exception\ValidationHttpException;
use Illuminate\Contracts\Validation\Validator;
use Symfony\Component\HttpKernel\Exception\HttpException;

trait ApiRequestTrait
{
    protected function failedValidation(Validator $validator)
    {
        throw new ValidationHttpException($validator->errors());
    }

    protected function failedAuthorization()
    {
        throw new HttpException(403);
    }
}

<?php

namespace App\Http\Requests;

class StorePostRequest extends ApiRequest
{
    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'title' => 'required',
        ];
    }
}

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;
use App\Traits\ApiRequestTrait;

abstract class ApiRequest extends FormRequest
{
    use ApiRequestTrait;
}

My first thought was that it might couldn't intercept the exception because the FormRequest was causing an issue but Dingo doesn't catch any HttpException. I tried to throw any Exception that is listed in the Wiki and it doesn't seem to catch them.

I followed back to where the Exceptions go and they all go to the report-method in the Handler and that is the end.
Shouldn't they go into the render-method so that a rendered response is created?

There was an issue with the ValidationHttpException which has now been resolved. I think this may have been the problem you were seeing. There is a difference between that exception and the StoreResourceFailedException. Make sure with the StoreResourceFailedException exception (and other resource exceptions) that the first parameter is a message and the second parameter the validation errors.
The ValidationHttpException only takes the validation errors as the first parameter.
Anyhow, try updating and checking now.

Well, did an require ~0.10@dev to get the fix and now authentication is broken. No matter if a route is protected or not I receive an UnauthorizedHttpException for every request.
Has anything been changed to JWT authentication in 0.10?

Also I did apply the ValidationHttpException change manually but I still just get the normal exception page and as I wrote this for 99% of exceptions, except the NotFoundHttpException.

Authentication changed slightly between 0.9 and 0.10. Refer to the documentation, but basically you just need to define the api.auth middleware on the routes and/or groups you want  protected.
As for the exceptions, I'm unable to reproduce what you're seeing. I've copied what you've pasted here and just filled in the gaps and I'm seeing a proper error response. Can you share all relevant code.
Routes, controllers, form request class, etc. Something seems amiss because it's working fine here on a fresh install.

The API Route and Controller-method look like this.
public function index()
{
   // authenticated = app('Dingo\Api\Auth\Auth')->user()
    if ($this->authenticated()) {
        $posts = $this->posts->findAllPaginatedForUser($this->getCurrentUser());
    } else {
        $posts = $this->posts->findAllPaginated();
    }

    // $this->pagination is just a shortcut to Dingo\Api\Routing\Helpers\Response->paginator()
    // https://github.com/DraperStudio/Domesticated/blob/master/src/Traits/TransformerTrait.php
    return $this->paginator($posts);
}
$api->version('v1', function ($api) {
    $api->group(['prefix' => 'posts'], function ($api) {
        $api->get('/', 'PostController@index');
    });
 });

The "Frontend"-route then looks like this.
/**
 * @Get("posts", as="posts")
 */
public function posts()
{
    $posts = $this->api->get('posts'); // sometimes used with ->once($this->getCurrentUser())

    return view('posts')->with('posts', $posts);
}

Thats pretty much it and there is no authentication required for this route, it is optional, and still I receive the error that authentication is required no matter if a route is protected or not which is weird because I did the authentication changes.

By default the app('Dingo\Api\Auth\Auth')->user() will attempt to authenticate. This is mentioned in the "Optional Authentication section.
You can, however, pass false as a parameter to instruct the authenticator to not attempt authentication.
return app('Dingo\Api\Auth\Auth')->user(false);

I've altered the default behaviour of the check method as I think it makes more sense in this case for it to check if authentication has occurred.
So you can use this instead:
return app('Dingo\Api\Auth\Auth')->check();

Did a few changes here and there and now it returns proper responses for all exceptions again.
Thanks for your help and thanks for the change to the check-method.

All good mate glad things are back in the right gear. :+1:


fix: aws-aaos-pull-request

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

changes are done in target.py