Changing Temperature

I think there is a problem when changing the temperature while being on PermanentOverride . It changes the temp but it changes only for 2 hours(till next setpoint), while being on PermanentOverride in HA. For example. I changed the livingroom to 21 degrees at 10.30 in Ha in permanent override, but when i look in my evohome app it says it wil change only till 12.30, so its not in permanent override anymore. When changing from temporary override to permanent override in Ha it stays the same for 2 hours so it doesnt change to permanent override . U can change from follow the schedule to permanent override. Then the state is correct. Why is temperary override not till next setpoint from follow schedule and only for 2 hours. Can u check this pls. Thk u
Example in log : for permanent override mode temperature schould not be None (will use current target temp).

This is due to a limitation of HA; the long-term solution is to create a custom service call.
I will try to address this, but I do not have a lot of spare capacity at the moment.

@Tacchi1983 There were some bugs found/squashed that may have addressed this issue.
Can you confirm if it is still here with HA v0.88, and evohome_cc v0.9.9?

I believe this is fixed with the latest version of HA (on/before HA v0.100).


Error refreshing the OAuth2 token

Hi, after installation, i receive this error when I try to use this package:
Error refreshing the OAuth2 token, message: '{ "error" : "invalid_grant" }'
I've read on StackOverflow about NTP offset, this is my situation when I do ntpd -p:
time1-st1.apple <IP_ADDRESS>   2 u   7     512    1       167.002   -1.008 1275.79
Where -1.008 is my offset.
Any helps?

I am getting this issue a lot and I can't for the life of me figure out why. I've entered everything as requested; the Google credentials provided me with a password for the private key too but didn't have anywhere to enter this in the package config. Wondering if this could be the reason why I'm getting this error? Checked path to the private key, all fine.
Please advise.


Can't use identifiers program_id or accounts in handler names

The codegen uses these names before invoking the handler. Instead, we should add a __ prefix to all variables used by the codegen so that there's no collision.

Related https://github.com/project-serum/anchor/issues/347.

Fixed in #2464.


Using Nylas Node v3 SDK, create a code sample to read calendar

Using Nylas Node v3 SDK, create a code sample to read calendar

Please assign this to me , I would like to work and learn more

Done! Thanks for taking this on @Surya-V-28. Some further details:

Here are the details to install the Nylas SDK for Nylas v3: https://www.npmjs.com/package/nylas/v/7.0.0-beta.2
Send us an email at devrel @ nylas.com and we can help set up an account for testing the code sample

@Surya-V-28 - did you get a chance to take a look at the issue?

@Surya-V-28 - I haven't heard back from you, so will wait a few days.

@Surya-V-28 - going to re-assign the ticket if you are not able to pick it up by Monday, let us know :)

If this is open for reassignment, I'd be happy to take this on!

@kelbri10 - awesome, just assigned to you 👍

@kelbri10 - did you get a chance to look at this ticket?

@kelbri10 - did you get a chance to look at this ticket?

Hi! Yes I did! I had a couple things come up but I hope to submit tomorrow!


Translate game

We translated the game to Korean and add function to select English or Korean from the first start screen.

:writing_hand:  We like this PR because multi-language support is important to us. :world_map:
We are trying to refactor the project right now to make it easier for PRs like this one to start adding multiple languages to the project.
Please note however that adding multiple languages into C++ projects is quite a challenge. We are thinking of ways to support the feature so that it is easy for anyone to add new languages quickly. :dove:

Currently, we have come to the decision that using hard-coded if() statements blocks would not be wise method of supporting multiple languages.
This is because:

it will be difficult to scale (for 10+ languages)
the code needed will be difficult to support in the projects' lifecycle after long periods of time.

This PR will not be merged and will be closed as it does not fit the answer the project needs right now.
However, it is great that you brought it to our attention! :tada:
It is exciting to see that this game is being played around the world so thank you for your contribution! :world_map:
I have made a new issue (#89) and have used this PR as reference.
It will help in finding a solution to the hurdle, ok?
Thanks! :smile:  :+1:

Thank You for your answer! 😄


[OTHER] small color changes for accessibility

What would you like to share?
Hi! I have some small color changes that can make the application more accessible.
original:

new version:

I changed the original purple color to deep purple and added a slightly lighter purple to make some parts of the text more readable. I also made the gray a bit lighter.
Let me know if you like it!
Additional information
No response

I agree with you. I like it!

Yes Lou! Looking at it now, I see how cool the contrast is. You can go ahead and submit a pull request for this!


[HackaDoc] Enable and view storage metrics - corrected

Enable and view storage metrics - corrected

Hi @MaochenWang, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

Nice job!


[PackageModel] Use separate type for each build setting condition

This is probably a better model than shoving all conditions into
a single struct.

@swift-ci smoke test


Feature: Add ability to compile templates into the CLI

Problem
At the moment we cannot add templates for any of the create command. We are using a makeshift implementation for the v1 flag that we check and then manually swap out the content to write out when the command is executed.
Solution
Add native supported for templating into the CLI by introducing a new folder structure.
How it currently works
The way we create templates in our CLI is by compiling them directly into the package as a const String's. The template map is created by looking at the folder structure and creating a set of modifications that align with that structure. The intention behind this was to ensure that a template can be stored using the final file structure we want it to take up.
Each folder in the root of the lib/src/templates folder is for a specific create command. We have the following at the moment:

app: The template applied when running stacked create app
bottom_sheet
dialog
service
view

This works well but we need it to be more extensible.
Why we need to change
As we're building the FilledStacks academy app in Flutter Web we have found the requirement to have a separate template for creating a web app. The current structure doesn't allow for us to do this. In addition to that we also want to update the view command to take in templates, specifically for creating views that we have templates for.
Things like a list view that already has continuous loading implemented, pull to refresh, etc.
How will we do it
Instead of compiling a single command from the root folders each root folder will serve as an entry for the templates and the folders inside of it will serve as the templates that we can pass in to the cli. This to add a template to the app command we'll have:

app

default
web

This will then add those folder names to the options for the command when running stacked create app --t=web where running without the --t will use the default template.
Rabbit Holes
The functionality below will be excluded from this work

Creating templates that includes pieces of the default template. For now we'll simply duplicate and create templates until it becomes un maintainable
A root template folder without a default folder inside it should use the root as the template and not add --t options.

Technical changes
To complete this functionality we will need to update the compile command to go a level deeper and build up template options using the subdirectories of the root command folder.
When we do that we also need to update the templateMap to account for the templates. So instead of a single surface level mapping of  using the StackedTemplate directly after getting it from the kCompiledStackedTemplates we will instead do an additional index using the --template value passed in. This requires that we update the StackedTemplate to now store a map of templates for the Template types of the command.
In addition to that, to keep things maintainable and have our domain language be solid we need to rename some of the naming from templates to commands since we'll have an actual template parameter now that we need to refer to.

I like the specification of this function. The only thing I would change is to avoid a default folder and instead pass the default template as an option in the arguments. This makes changing the default template super easy and decoupled from the template system.
    argParser.addOption(
      ksTemplate,
      abbr: 't',
      defaultsTo: 'mobile',
    );

@ferrarafer Great suggestion! I like it. Updating the original spec now.

PR in #865


Wrong behavior on empty css file

Hi,
I have a working setup with cssnext-loader, but if I require an empty css file, instead of getting an empty string, I am getting a json object.
The problem is at line 30 in index.js:
if (result.css)

should be
if (typeof result.css !== 'undefined')

because result.css is empty, and that's correct, but the check assume there was an error.
If you want, I can make a PR for that.

cssnext-loader will be deprecated, soon, you should consider using postcss-loader + postcss-cssnext plugin.

Oh. Didn't know that :(

I will keep that issue open so people will see it if they got an issue. Will close if someone fix it or when npm package will be deprecated.


SFTP / FTP module

Develop a system for uploading, downloading, comparing and synchronizing files on an sftp / ftp. The system should also handle code changes to git pushes and pulls.

https://marketplace.visualstudio.com/items?itemName=Natizyskunk.sftp

Software of this magnitude needs a proprietary module for ftp / sftp. I have tried many of these plugins but each has limitations. For example for loading git pulls and for sustainability over time etc ..
I have also seen online that this is a highly requested feature.

This is a perfect plugin for most needs: Name: Deploy (reloaded), Visual Studio Marketplace: https://marketplace.visualstudio.com/items?itemName=mkloubert.vscode-deploy-reloaded
But the latest update dates back to a 22/9/2019.


Update INSTALL.md for local storage data retention

include path to mount to retain created schemas when using local storage

Summary by CodeRabbit

New Features

Added a new configuration option to back up JSON schemas during the Azimutt setup process.

Thanks for the PR :+1:


Managed CockroachDB replication controls

Not related to 19.1. Milestone should be updated at some point.


:bug: rename last submit button in analysis wizard

"Run" instead of "Next", not sure if someone wants to set up the translations for this

Thanks @gitdallas


Incorrect logic for rest CreateAction

The action returns incorrect model because does not work Location header.
We must call Yii::$app->end() when "Location" header is set.
Example: my Fake model has fields(id, name (required), email, created_at, updated_at)
call POST /api/fakes body name: 'Fake name'
Response:
{"id": 1, "name" => "Fake name"}
When I use GET /api/fakes/1
Response:
{"id": 1, "name" => "Fake name", "email": null, "created_at": null, "updated_at": null}

@maxlapko, what is the status of your issue. Have you found a solution yet?

I'm closing the issue unless further information provided.


Explain why  is needed

This may be more obvious when we have an implementation. But, it's not clear immediately why it's necessary, as opposed to making <parallax> elements work within any scrolling container.
If it's a matter of allowing the code to find the nearest scrolling container, that's interesting, as we've also seen similar problems in the virtual scroller design, so that might point to a missing primitive.

FWIW, I believe with Scroll-linked Animations there's no need for a <parallax> element.


Remove mvBoincProjects + more

mvBoincProjects was used for 'list projects' and 'list explainmagnitude' when there was no response from neural network. with list projects it used StructCPID and initialized projects etc. this also led to the spacing in list projects. This part of the code is not needed as everything else is grabbed from the cache. Also note that this is only populated during startup and because of this the project remained in mvBoincProject. Which is another reason why the client required a restart for project to no longer be displayed in list projects or old format of explainmagnitude. When talking about explainmagnitude it used the start up values and would not actually contain updated information about rac over the period of days and more unless a restart occured.
Details of PR

removed mvBoincProjects and used AppCacheFilter. for changes related to list explainmagnitude and list projects.
removed old explainmagnitude format from explainmagnitude to avoid confusion such as in #205 . This also allows them to see neural data only and not old data stored in mvBoincProject
explainmagnitude will only use neural response and if it happens to be empty it will make a call for it and next time they call it the data should be there.
removed old std::string AppCache function that was not in use that also caused conflicts with appcache.h when included in main.cpp
changed ProjectIsValid to use cache
'list projects' now shows the project URL as well and in the case where an extra stats/ is added for NN that is removed so a user can have a direct link to the project website.

Note: @tomasbrod your add TrxID to cache (i reenabled) is affecting the outputs on 'list projects' as well as getlistof and anything related such as beaconreport. I recommend thats disabled again till a better solution for it is in place as displays of numerous information can be a mess for the end-user.

#643 also reflects this and note only reason a project would not show on this is if its not in whitelist anymore or its missing from NN as the poster mentioned some are occasionally gone. also mvBoincProjects wouldn't have the updated information if a new project was taken on.

also go through and make sure nothing is missed thx :)

i think either one or both of #205 and #643 could be closed.


Revisit 5.1.0 does not contain Runspace.DefaultRunspace.CreateNestedPipeline()

This is a continuation of #42.
I'm in the middle of researching the ability to migrate from a DotNet Framework 4.6 class library to DotNetStandard 2.0, in order to support both PowerShell 5.x and PowerShellCore.  I have existing code that writes to the Verbose stream from within my class library that needs to be captured via console redirect, and uses the CreateNestedPipeline API to achieve that.  Without this support, or alternative, this is a blocker for me.  Here is a snippet of a method within a static class I have within my library which is used within a PowerShell module (PSM1):
public static void Verbose(string msg, string source)
{

    if (Namspace.StaticClass1.ShowConsoleVerboseOutput || Namspace.StaticClass1.EnableVerbose )
    {

        var rs = Runspace.DefaultRunspace
            .CreateNestedPipeline("Write-Verbose \"[" + source + "] " + msg.Replace("\"", "\"\"") + "\" -v", false)
            .Invoke();

    }

}

What is the alternative?
Please see this Stackoverflow thread on the topic, and proposed working code.
Originally posted by @ChrisLynchHPE in https://github.com/PowerShell/PowerShellStandard/issues/42#issuecomment-516255508

After some further research, I came across this blog post on how to access the current PowerShell runspace, in order to then execute a command/script.  So, I have adjusted my code to the following:
public static void Verbose(string msg, string source)
{

    if (Namspace.StaticClass1.ShowConsoleVerboseOutput || Namspace.StaticClass1.EnableVerbose )
    {

        using (PowerShell initialPowerShell = PowerShell.Create(RunspaceMode.CurrentRunspace))
        {

            initialPowerShell.Commands.AddScript("Write-Verbose \"[" + source + "] " + msg.Replace("\"", "\"\"") + "\" -v");
            initialPowerShell.Invoke();

        }

    }

}

Which is working in PowerShell 5.1 and PowerShell Core 6.2.2 for Windows, and PowerShellCore 6.2.2 for Linux (Debian/Ubuntu).
Unless the developers have any different comments or input, I would consider this closed.

Yes this is the correct API. My only suggestion is to use the AddCommand method instead of AddScript as it's a bit more readable:
initialPowerShell
    .AddCommand("Write-Verbose")
    .AddParameter("Message", $"[{source}] {msg}")
    .AddParameter("Verbose");

Which is the same as:
Write-Verbose -Message "[source] foo" -Verbose


chore: update playwright to 1.12.3

updates to latest playwright

:grey_exclamation: Build Aborted

Either there was a build timeout or someone aborted the build.'}

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #329 opened

Start Time: 2021-06-30T14:55:55.306+0000

Duration: 30 min 58 sec

Commit: b19986c4f20c3874ef9e66caa0843df1d51f3312

Test stats :test_tube:

Test
Results

Failed
0

Passed
113

Skipped
0

Total
113

Trends :test_tube:

Log output
Expand to view the last 100 lines of log output

[2021-06-30T15:08:02.644Z] Merging remotes/origin/master commit f5edccc292b22992c3119fab00b6d923bc2b02aa into PR head commit b19986c4f20c3874ef9e66caa0843df1d51f3312
[2021-06-30T15:08:02.749Z] Merge succeeded, producing b19986c4f20c3874ef9e66caa0843df1d51f3312
[2021-06-30T15:08:02.750Z] Checking out Revision b19986c4f20c3874ef9e66caa0843df1d51f3312 (PR-329)
[2021-06-30T15:08:02.652Z]  > git config core.sparsecheckout # timeout=10
[2021-06-30T15:08:02.660Z]  > git checkout -f b19986c4f20c3874ef9e66caa0843df1d51f3312 # timeout=15
[2021-06-30T15:08:02.699Z]  > git remote # timeout=10
[2021-06-30T15:08:02.708Z]  > git config --get remote.origin.url # timeout=10
[2021-06-30T15:08:02.728Z] using GIT_SSH to set credentials GitHub user @elasticmachine SSH key
[2021-06-30T15:08:02.732Z]  > git merge f5edccc292b22992c3119fab00b6d923bc2b02aa # timeout=10
[2021-06-30T15:08:02.743Z]  > git rev-parse HEAD^{commit} # timeout=10
[2021-06-30T15:08:02.753Z]  > git config core.sparsecheckout # timeout=10
[2021-06-30T15:08:02.756Z]  > git checkout -f b19986c4f20c3874ef9e66caa0843df1d51f3312 # timeout=15
[2021-06-30T15:08:06.523Z] Commit message: "chore: update playwright to 1.12.3"
[2021-06-30T15:08:06.523Z] Cleaning workspace
[2021-06-30T15:08:06.648Z]  > git --version # timeout=10
[2021-06-30T15:08:06.666Z]  > git --version # 'git version 2.17.1'
[2021-06-30T15:08:06.681Z] fatal: bad object b04e5c07ed96968b50b98aafc151e4fb3139c029
[2021-06-30T15:08:06.526Z]  > git rev-parse --verify HEAD # timeout=10
[2021-06-30T15:08:06.552Z] Resetting working tree
[2021-06-30T15:08:06.552Z]  > git reset --hard # timeout=10
[2021-06-30T15:08:06.584Z]  > git clean -fdx # timeout=10
[2021-06-30T15:08:08.752Z] Running in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/src/github.com/elastic/synthetics
[2021-06-30T15:08:23.712Z] + set +x
[2021-06-30T15:08:23.716Z] => Downloading nvm from git to '/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/.nvm'
[2021-06-30T15:08:23.716Z]
=> Initialized empty Git repository in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/.nvm/.git/
[2021-06-30T15:08:24.287Z] From https://github.com/nvm-sh/nvm
[2021-06-30T15:08:24.287Z]  * [new tag]         v0.37.1    -> v0.37.1
[2021-06-30T15:08:24.287Z] => Compressing and cleaning up git repository
[2021-06-30T15:08:24.287Z]
[2021-06-30T15:08:24.287Z] => Profile not found. Tried ~/.bashrc, ~/.bash_profile, ~/.zshrc, and ~/.profile.
[2021-06-30T15:08:24.287Z] => Create one of them and run this script again
[2021-06-30T15:08:24.287Z]    OR
[2021-06-30T15:08:24.287Z] => Append the following lines to the correct file yourself:
[2021-06-30T15:08:24.287Z]
[2021-06-30T15:08:24.287Z] export NVM_DIR="$HOME/.nvm"
[2021-06-30T15:08:24.287Z] [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
[2021-06-30T15:08:24.287Z]
[2021-06-30T15:08:26.197Z] => Close and reopen your terminal to start using nvm or run the following to use it now:
[2021-06-30T15:08:26.197Z]
[2021-06-30T15:08:26.197Z] export NVM_DIR="$HOME/.nvm"
[2021-06-30T15:08:26.197Z] [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
[2021-06-30T15:08:26.197Z] Found '/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/src/github.com/elastic/synthetics/.nvmrc' with version <12.18.4>
[2021-06-30T15:08:26.768Z] v12.18.4 is already installed.
[2021-06-30T15:08:27.031Z] Now using node v12.18.4 (npm v6.14.6)
[2021-06-30T15:08:27.031Z] /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329
[2021-06-30T15:08:27.172Z] Running in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/src/github.com/elastic/synthetics
[2021-06-30T15:08:27.493Z] + chmod -R ugo+rw examples
[2021-06-30T15:08:27.521Z] Running in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/src/github.com/elastic/synthetics/__tests__/e2e
[2021-06-30T15:08:27.850Z] + npm run test
[2021-06-30T15:08:27.850Z]
[2021-06-30T15:08:27.850Z] > e2e-synthetics@ test /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/src/github.com/elastic/synthetics/__tests__/e2e
[2021-06-30T15:08:27.850Z] > npm i && npm run setup &&  bash scripts/test.sh
[2021-06-30T15:08:27.850Z]
[2021-06-30T15:08:31.146Z] added 3 packages from 4 contributors and audited 178 packages in 2.719s
[2021-06-30T15:08:33.693Z]
[2021-06-30T15:08:33.693Z] 66 packages are looking for funding
[2021-06-30T15:08:33.693Z]   run `npm fund` for details
[2021-06-30T15:08:33.693Z]
[2021-06-30T15:08:33.693Z] found 0 vulnerabilities
[2021-06-30T15:08:33.693Z]
[2021-06-30T15:08:33.954Z]
[2021-06-30T15:08:33.954Z] > e2e-synthetics@ setup /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329/src/github.com/elastic/synthetics/__tests__/e2e
[2021-06-30T15:08:33.954Z] > bash scripts/setup.sh
[2021-06-30T15:08:33.954Z]
[2021-06-30T15:08:33.954Z]
[2021-06-30T15:08:33.954Z] Temporary folder
[2021-06-30T15:08:33.954Z] Temporary files will be stored in: ./tmp
[2021-06-30T15:08:33.954Z]
[2021-06-30T15:08:33.954Z] Starting elasticsearch , kibana and synthetics docker
[2021-06-30T15:08:33.954Z]
[2021-06-30T15:08:33.954Z]
[2021-06-30T15:08:33.954Z] Waiting for synthetics docker to start...
[2021-06-30T15:08:34.215Z] Error: No such object: synthetics
[2021-06-30T15:08:44.264Z] Error: No such object: synthetics
[2021-06-30T15:08:54.263Z] Error: No such object: synthetics
[2021-06-30T15:09:06.482Z] Error: No such object: synthetics
[2021-06-30T15:09:14.681Z] Error: No such object: synthetics
[2021-06-30T15:09:24.673Z] Error: No such object: synthetics
[2021-06-30T15:09:34.671Z] Error: No such object: synthetics
[2021-06-30T15:09:44.711Z] ✅ Setup completed successfully. Running e2e tests...
[2021-06-30T15:09:46.668Z] Waiting for Elastic Search  to start
[2021-06-30T15:10:33.572Z] Waiting for Synthetics to send data to ES for test monitor
[2021-06-30T15:25:25.723Z] Cancelling nested steps due to timeout
[2021-06-30T15:25:25.728Z] Sending interrupt signal to process
[2021-06-30T15:25:50.005Z] After 20s process did not stop
[2021-06-30T15:25:51.317Z] Post stage
[2021-06-30T15:25:51.352Z] Archiving artifacts
[2021-06-30T15:25:51.531Z] Recording test results
[2021-06-30T15:25:51.681Z] None of the test reports contained any result
[2021-06-30T15:25:51.690Z] [Checks API] No suitable checks publisher found.
[2021-06-30T15:25:51.955Z] Stage "Release" skipped due to earlier failure(s)
[2021-06-30T15:25:51.991Z] Stage "Release" skipped due to earlier failure(s)
[2021-06-30T15:25:52.089Z] Stage "Release" skipped due to earlier failure(s)
[2021-06-30T15:25:52.456Z] Running on Jenkins in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-329
[2021-06-30T15:25:52.535Z] [INFO] getVaultSecret: Getting secrets
[2021-06-30T15:25:52.584Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2021-06-30T15:25:53.432Z] + chmod 755 generate-build-data.sh
[2021-06-30T15:25:53.432Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-329/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-329/runs/1 ABORTED 1797854
[2021-06-30T15:25:53.432Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-329/runs/1/steps/?limit=10000 -o steps-info.json
[2021-06-30T15:25:53.432Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-329/runs/1/tests/?status=FAILED -o tests-errors.json


Bungeecord MySQL error?

I already searched all over the internet to fix this but cannot solve it still.
this is the error:
luckperms - Failed to validate connection me.lucko.luckperms.lib.mysql.jdbc.JDBC4Connection@5da79bc4 (No operations allowed after connection closed.)
I hope someone can help me
The credentials are correct, I changed the wait_timeout to 1800 already so I'm clueless now.

What are you hosting and controlling your server with (e.g. multicraft/vps etc)

I host it on a Contabo VPS and I use pterodactyl as panel. Also I use MariaDB for Mysql driver

Only bungee.

Try this: https://github.com/lucko/LuckPerms/wiki/Storage-system-errors

@BrainStone Did that already sorry...

@ICEconchy  The bungeecord error has been resolved now my spigot server has this error:
(Connection.setNetworkTimeout cannot be called on a closed connection)
It is just weird that only luckperms has issues with the MySQL/MariaDB and the other plugins are working just fine....

Brainstone was correct - please read the details on this page. This isn't a LP issue.
https://github.com/lucko/LuckPerms/wiki/Storage-system-errors#mysql-no-operations-allowed-after-connection-closed-error

Alright so could you tell me why no other plugin on that server has problems with MariaDB and luckperms is the only one with this error?

Probably because the other plugins you have don't use a connection pool like LuckPerms does.

I looked at this link: https://github.com/lucko/LuckPerms/wiki/Storage-system-errors#mysql-no-operations-allowed-after-connection-closed-error
My error isnt stated there, I got this error in my console: luckperms - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@6eb81946 (Connection.setNetworkTimeout cannot be called on a closed connection)

Yes it is - you error is stated there.

me.lucko.luckperms.lib.hikari.pool.PoolBase - luckperms - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@xxxxxxx (xxx cannot be called on a closed connection)


Remove navbarPage from front page, since you can't navigate it.

Just a matter of preference
I left a title commented in the code, just in case you prefer it to say "R-Ladies", "R-Ladies Today" or something else.
Thank you for such a lovely Dashboard! 💜

Thank you!


Create a framework for troubleshooting

Create Troubleshooter which can perform troubleshooting of an environment which is not working

Reworked in #127


Transfer Subscription payments to a non Admin person

[Enter feedback here]

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

ID: 805a04c5-96c4-ee47-7746-3105e0b7fc45
Version Independent ID: 9efa7447-6731-6435-d4fc-8ba81344e85a
Content: Understand subscriptions and licenses in Microsoft 365 for business
Content Source: microsoft-365/commerce/licenses/subscriptions-and-licenses.md
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

I am admin and have been using my card in the company to pay for subscription.  Now I want to transfer that to our accountants card, but they are not Admin, is there a way to do this, I have searched and I cannot find any answer.

@CharlesYeomans Thank you for your feedback. You cannot assign Subscription payments to a non-Admin person. In order to handle payments for a subscription one needs to be at least a Billing administrator. more details about the role can be found here - https://learn.microsoft.com/en-us/microsoft-365/admin/add-users/about-admin-roles?view=o365-worldwide#commonly-used-microsoft-365-admin-center-roles
If you have trouble assigning the role, I'd request you to contact support via the need help section in the admin center, so that they can assist you in fixing it.
Hope this helps!
Thanks
Sri

Thanks
Actually I finally figured it out, but didn't know how to cancel my feedback.
Sorry for wasting you time.
Brilliant that you respond so well, thanks
Charles
From: Sriraman M S @.>
Sent: Tuesday, 22 November 2022 20:05
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: Charles Yeomans @.>; Mention @.>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] Transfer Subscription payments to a non Admin person (Issue #10311)
@CharlesYeomanshttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FCharlesYeomans&data=05|01||fa9123ed25e248b82eda08dacc81be78|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638047154889195496|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4DHmRKAwuhY5%2B45uvPk3YWMCXJ5JXPTpIWOxFWy2Yvc%3D&reserved=0 Thank you for your feedback. You cannot assign Subscription payments to a non-Admin person. In order to handle payments for a subscription one needs to be at least a Billing administrator. more details about the role can be found here - https://learn.microsoft.com/en-us/microsoft-365/admin/add-users/about-admin-roles?view=o365-worldwide#commonly-used-microsoft-365-admin-center-roleshttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Flearn.microsoft.com%2Fen-us%2Fmicrosoft-365%2Fadmin%2Fadd-users%2Fabout-admin-roles%3Fview%3Do365-worldwide%23commonly-used-microsoft-365-admin-center-roles&data=05|01||fa9123ed25e248b82eda08dacc81be78|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638047154889195496|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=VtRt9R7i2CF0EV8fSiSLSzUiyve%2F%2FPbt8JX4Am8PdLY%3D&reserved=0
If you have trouble assigning the role, I'd request you to contact support via the need help section in the admin center, so that they can assist you in fixing it.
Hope this helps!
Thanks
Sri

Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-docs%2Fissues%2F10311%23issuecomment-1323561423&data=05|01||fa9123ed25e248b82eda08dacc81be78|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638047154889195496|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=jRqPst%2BCU%2BCcaCqJo53Ikc%2BrcBBb2hcIah6nRExwstk%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAINGGECS5IAM6FVHSGOIRX3WJSZFXANCNFSM6AAAAAASEEI2OI&data=05|01||fa9123ed25e248b82eda08dacc81be78|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638047154889195496|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Nszwlv%2FLSM2EzR9mtWickjsuaBt8C4v%2FcL0viciPSIU%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Multilingual support: Add hreflang HTML tags

This implements Google's advice for webmasters:

https://developers.google.com/search/docs/specialty/international/localized-versions

I adapted the code from here:

https://gohugo.io/methods/page/alltranslations/

@regisphilibert Does it look good to you?


Refactor: Add extern (D) overloadApply and use lambda

This would have been a lot easier to review, as usual, if you hadn't mixed in 1000 other refactoring changes at the same time.

Auto-merge toggled on


Converting Results and Questions into Word or PDF or Excel

Are you requesting a feature, reporting a bug or ask a question?
Question Feature
What is the current behavior?
We would like to download the user response into Word or PDF or Excel questions and along with user response.
Like we need same as is along with Download (options: PDF, Word, Excel) http://surveyjs.org/examples/jquery/survey-displaymode.html
by clicking on download user should get the above link output in  PDF, Word, Excel.
What is the expected behavior?
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser:
browser version:
surveyjs platform (angular or react or jquery or knockout or vue):
surveyjs version:

@krishnathamisetty I am not sure what do you mean to save results in pdf/excel/word. Do you mean all survey results or just for one user?
We have http://www.dxsurvey.com service. You may use it. It is free for now, . You may export all data to Excel. Here is the public result page of one our surveys: https://dxsurvey.com/Results/Survey/d5220f76-4802-40cf-ad67-61d7e55608e5. There is the Export button and export to Excel in the popup menu.
Anyway, surveyjs library gives you result as Json you are free to do with it what you want. The integration with dxsurvey.com is easy and straight forward. Of course you are free to use any other services or create your own.
Thank you,
Andrew

Not sure if this is what you're after, but if you want an offline solution, I'm using docxtemplater for one of my projects with surveyjs. https://docxtemplater.readthedocs.io/en/latest/index.html
You just feed it the Json and a pre-defined word template with the questions and response tags. Then when you "Download" after the survey is completed it will give you the generated word document with responses.

Hello,
I don't see the javascript/code part of the export button. How can I use it ?

Or Is there any features that I can print the result of my questions and it's answers?

@SaharDastitash Here is the example of exporting to PDF
What do you mean by "export button"?
Thank you,
Andrew


Add support for Azure

add support for Azure

https://azure.microsoft.com/en-us/blog/announcing-the-preview-of-java-support-for-azure-functions/

Wait for https://github.com/Azure/azure-functions-java-worker/issues/16 to be resolved

Wait for https://github.com/Azure/azure-functions-java-worker/issues/37

not worth the effort


[MLIR][TESTS] test_conv_igemm_mlir* take too much time

In FP32 gfx908 test, each takes >30 minutes so the overall testing time > 40 minutes. This is too much for Smoke testing. Recommendation:

Create Full MLIR test stage and move these tests there.
Create test_conv_igemm_mlir*_small and use these in Smoke.

It should contain ~10% from the existing test_conv_igemm_mlir*
Tip: Move (not copy) test cases from test_conv_igemm_mlir* to *_small to avoid duplication of test cases

Expected time saving is ~27 minutes -> overall testing time should be ~13 mins.
Alternative, simpler approach: Split 2 existing test_conv_igemm_mlir* tests to 8. Use CTEST_PARALLEL_LEVEL=8 in Jenkinsfile for MLIR tests. Expected time saving is ~22 minutes -> overall testing time should become ~18 minutes. Potential problem: too many parallel threads may lead to resource starvation.

Thanks for bring this up. It does look like we might be missing the build type in top level cmake, which is causing the excessive amount of test time in those stages. Refer to https://github.com/ROCmSoftwarePlatform/llvm-project-private/issues/292.
With this new information, I'm not sure some of the above suggestions still applies. Looks to me CTEST_PARALLEL_LEVEL=8 is the only relevant one if test_conv_igemm_mlir is 10x faster, for example.

MIOpen builds Release builds by default. I think that CI spends time for computation of reference convolution data, for validation. Reducing the number of test cases would linearly reduce the testing time.

MIOpen builds Release builds by default.

That's right, think I was wrong to assume it was debug build.

CI spends time for computation of reference convolution data

I can confirm this when I tested it locally. Now is there a variable like MIOPEN_DRIVER_USE_GPU_REFERENCE=1 that works for bin/test_conv2d? @carlushuang. The variable saved me ton of time in local validation, it seems useful if we can make gpu validation by default on in MIOpen CI in general.

@jerryyin

The variable saved me ton of time in local validation

Just in case -- I still use the caching feature of the MIOpenDriver -- MIOpenDriver ... -C ...

it seems useful if we can make gpu validation by default on in MIOpen CI in general.

Yes, but there are some doubts. And IIRC it only works in the driver, please refer to discussion at #583

@atamazov I actually never knew the -C feature. Since the day MIOPEN_DRIVER_USE_GPU_REFERENCE becomes available I always use it and it is always fast a reliable for me to validate.

Shall we try to add it to the test_conv*? Let's open a ticket if you would like to proceed.

I thought this was what @carlushuang planned to do when we talked couple of months ago, so I want to double check with him. But yeah, I don't see a reason to not use such a convenient/efficient feature in CI.

Alternative, simpler approach: Split 2 existing test_conv_igemm_mlir* tests to 8. Use CTEST_PARALLEL_LEVEL=8 in Jenkinsfile for MLIR tests. Expected time saving is ~22 minutes -> overall testing time should become ~18 minutes. Potential problem: too many parallel threads may lead to resource starvation.

This won't work (let me omit explanations for now).

Closing the ticket as corresponding PR merged. Each individual MLIR smoke test stage now only test 3 configs and finish in around 15 mins.


pin datadog version

Updated to original source to get https://github.com/miketheman/heroku-buildpack-datadog/pull/30 in.

@mezis you might want to take a look at this


Multi-language support for user interface

Hi
I am from China, can you add multi-language support to this application?
I can help you translate simplified Chinese.
Thank you for your work. ：-）

Hello,
As of now, it is too difficult to separate the text from the UI to add translation support. While I may consider this for a future update, I don't have much plans to do so.

Thanks for your reply!
Multi-language support allows the application to have more users. Hope to add multi-language support in future versions.
If you need a Chinese translation, you can send a message to me. I will assist you to complete the Chinese translation content.
Thank u!


When the global file is deleted mark all files as changed

Fixes #36728
It would be easier to look at the change commit wise since lot of baselines have changed because of added affectsGlobalScope file info property
4da9fc5 adds test that shows the current behavior of the issue
f34d481 fix and updates the above test to show its fixed
e282da1 fixes all the rest baselines

ping @andrewbranch


[LLM Runtime] Streaming-LLM based on shift RoPE

Type of Change: feature
API change: add switch --shift-roped-k to the C++ API
Description
Support shifting RoPE-ed K-cache after the KV-cache is filled up for llama.
Per-model changes:

n_ctx is now a field of model_context rathe then model class themselves, together with the added n_keep and shift_roped_k. It is also more semantic as n_ctx is read from args rather than model files.
Added an extra argument n_total for xxx_model_eval_internal, which provides the number of tokens evaluated so far (including evicted tokens if there is any). Note: n_past is used as the offset to which the KV is cached to even before this PR
in xxx_utils.cpp, the template argument of the unique_ptr in model_load_internal are corrected to the derived classes
Time are calculated outside of the xxx_utils.cpp

Note: The version for fused-attention will be implemented in a later PR
Expected Behavior & Potential Risk
N/A
How has this PR been tested?

 She wanted to go to places and meet new people, and have fun.
Then one day, her parents took the little girl on an adventure, that she would never forget. They drove their tiny car, through a dark tunnel, where nothing could be seen, but the stars above. Afterwards, as the days went by, the little girl thought of this place, and how she felt there, inside of it. It was not a place to get lost, or to find treasure. The child wondered at its vastness, with so many people there, and yet no one ever spoke, or even moved very much, except those who were in service.
She did know, though,

Dependency Change?
No

https://inteltf-jenk.sh.intel.com/job/ITREX-cpp-graph-extension/159

update extension test: https://inteltf-jenk.sh.intel.com/job/ITREX-cpp-graph-extension/164/artifact/report.html


made corrections and update article

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[ ] Have you used the correct folder and file structure? - see Contribution Guidelines

[ ] Is your article properly formatted in Markdown? - see Github Markdown guide

[ ] Have you used our preferred heading structure? - H3 (###) and up

[ ] Does your article follow any of the suggested structure formats? - see example formats

[ ] Is your article over 750 words? - Tool: Wordcounter.net

[ ] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[ ] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[ ] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[ ] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[ ] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[ ] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[ ] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[ ] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[ ] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[ ] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #issue-number

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@aidelojep
The PR seems to be merge conflict free - the PR has entered our queue and will be reviewed as soon as our team is finished up with the ones before it. 👍

@hectorkambow Thanks a bunch! Always a pleasure working with you and learning from you too :)

Thanks @Hector..it's always a pleasure to work with you and learn from you
too. :)
On Thu, Dec 2, 2021 at 6:46 PM hectorkambow @.***>
wrote:

@aidelojep https://github.com/aidelojep
The PR seems to be merge conflict free - the PR has entered our queue and
will be reviewed as soon as our team is finished up with the ones before
it. 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/5340#issuecomment-984855151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXGABL4WZNPVE5W4RCRXMDUO6WIPANCNFSM5JGV4YPQ
.

Hello @aidelojep
Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
How To Tune Recommendation  Engine To Forecast Future Trends #5340 Copyleaks Report.pdf

Thank you. I failed to avoid self-plagiarism; I have worked on something
similar before, so I didn't change my referencing. I would work on the
changes as soon as I can.
Many thanks,
Peter.
On Wed, Dec 8, 2021 at 5:31 PM Peter Kayere @.***>
wrote:

Closed #5340
https://github.com/section-engineering-education/engineering-education/pull/5340
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/5340#event-5735016189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXGABL2AHPXCRRT5GORCJDUP6B57ANCNFSM5JGV4YPQ
.

@Kayere @hectorkambow  please note that the self-plagiarism has been corrected.

Hi Peter,
I have made some changes to the article to resolve the self-plagiarism
issue.
Best regards,
Peter L. Aideloje.
On Wed, Dec 8, 2021 at 5:31 PM Peter Kayere @.***>
wrote:

Closed #5340
https://github.com/section-engineering-education/engineering-education/pull/5340
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/5340#event-5735016189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXGABL2AHPXCRRT5GORCJDUP6B57ANCNFSM5JGV4YPQ
.


Does the alarm have documentation?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[1 ] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question
See the alarm function, but do not know how to use
--using Google Translate

Please look your application.yml in your collector config directory. The following settings are alarm related. And I think it is easy to understand.
# alarm threshold
    applicationApdexThreshold: 2000
    serviceErrorRateThreshold: 10.00
    serviceAverageResponseTimeThreshold: 2000
    instanceErrorRateThreshold: 10.00
    instanceAverageResponseTimeThreshold: 2000
    applicationErrorRateThreshold: 10.00
    applicationAverageResponseTimeThreshold: 2000

3Q


Pboros main

Codecov Report

Merging #2 (45e4791) into main (a0325e2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   51.36%   51.36%
=======================================
  Files         114      114
  Lines        5403     5403
=======================================
  Hits         2775     2775
  Misses       2216     2216
  Partials      412      412

Continue to review full report at Codecov.

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


Bug in text formatting

When formatting text(italics, bold, etc),  if I click anywhere in the blue highlighted box, the typed message gets deleted.

Shifted from electron repo : https://github.com/zulip/zulip-electron/issues/232

@timabbott

@zulipbot claim


[KYUUBI #3069][DOC] Add Iceberg connector doc for Spark SQL Engine

Why are the changes needed?
Add Iceberg connector doc for Spark SQL Engine
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

since the doc build fails recently due to upstream breaking changes #3116, can we add a screenshot here to help verify?

Codecov Report

Merging #3115 (119be81) into master (f1312ea) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3115      +/-   ##
============================================
- Coverage     51.34%   51.32%   -0.02%
  Complexity        6        6
============================================
  Files           458      458
  Lines         25388    25388
  Branches       3536     3536
============================================
- Hits          13035    13031       -4
  Misses        11113    11113
- Partials       1240     1244       +4

Impacted Files
Coverage Δ

...g/apache/kyuubi/operation/BatchJobSubmission.scala
77.63% <0.00%> (-1.25%)
:arrow_down:

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
77.27% <0.00%> (-1.14%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
83.12% <0.00%> (-0.63%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us.

Thanks, merging to master


add links, description and name to coin serializer and model class

Extending Client class to allow fetching Coin data by id.
Extending Coin class to deserialize links, description, type and name from JSON response.

Hi @smogm,
First of all, I would like to say: Welcome here and thanks for contributing to our client repo!
Can you please update tests as well?


C# Align to Color Frame and Finding Depth

I am using the following code, I have seen python code and C++ and in all the example code. align() returns FrameSet i.e people are able to get depth frame and color frame. In C# it is returning a processed frame of type Frame. I want to call getdistance(x,y) function but I can't use it. How I calculate distance (depth) using color frame coordinate?
using(var ctx = new Context())
{
var devices = ctx.QueryDevices();
                    Console.WriteLine("There are {0} connected RealSense devices.", devices.Count);
                    if (devices.Count == 0) return 1;
                    var dev = devices[0];

                    Console.WriteLine("\nUsing device 0, an {0}", dev.Info[CameraInfo.Name]);
                    Console.WriteLine("    Serial number: {0}", dev.Info[CameraInfo.SerialNumber]);
                    Console.WriteLine("    Firmware version: {0}", dev.Info[CameraInfo.FirmwareVersion]);
             }

          var cfg = new Config();
            cfg.EnableStream(Intel.RealSense.Stream.Depth, desiredFrameWidht, desiredFrameHeight);
            cfg.EnableStream(Intel.RealSense.Stream.Color, desiredFrameWidht, desiredFrameHeight, Format.Bgr8);

            Console.WriteLine("Starting Realsense pipeline");
            var pipe = new Pipeline();
            var pp = pipe.Start(cfg);

            Console.WriteLine("After pipeline start");
            Colorizer colorizer = new Colorizer();
            Align align = new Align(Intel.RealSense.Stream.Color)

for (;;)
{
                using (var frames = pipe.WaitForFrames())
                using (var colorframe = frames.ColorFrame)
                using (var depthframe=frames.DepthFrame)
                {

                    var aligned_frame = align.Process(frames);
                    var colorized_aligned_frame = colorizer.Process(aligned_frame);

                       OpenCvSharp.Mat frame=new OpenCvSharp.Mat(desiredFrameHeight,desiredFrameWidht,OpenCvSharp.MatType.CV_8UC3,colorframe.Data,0);
                    OpenCvSharp.Mat dframe = new OpenCvSharp.Mat(desiredFrameHeight, desiredFrameWidht, OpenCvSharp.MatType.CV_8UC3, colorized_aligned_frame.Data, 0);

                    // give a frame to the worker
                    worker.work(frame,dframe);

The C# documentation has sample code for getting distance from the depth frame with GetDistance.  I wonder if you would get the result that you needed if you placed this distance measuring code after the alignment of depth to color.
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/csharp#hello-world

Issue is after we do alignment
var aligned_frame = align.Process(frames);
returned object is of type Frame, to call GetDistance() frame has to be DepthFrame. It gives error at compile time. No way I get DepthFrame type after align.Process in C#  and I have searched all C++ and Python example and it appears interface is different. I have a working code in Python but in C# I am can't move forward. following code worked well in python
Get frameset of color and depth
    frames = pipeline.wait_for_frames()
    # frames.get_depth_frame() is a 640x360 depth image

    # Align the depth frame to color frame
    aligned_frames = align.process(frames)

    # Get aligned frames
    aligned_depth_frame = aligned_frames.get_depth_frame() # aligned_depth_frame is a 640x480 depth image
    color_frame = aligned_frames.get_color_frame()

    # Validate that both frames are valid
    if not aligned_depth_frame or not color_frame:
        continue

    depth_image = np.asanyarray(aligned_depth_frame.get_data())
    color_image = np.asanyarray(color_frame.get_data())

I have seen a few reports in the past 18 months about users experiencing problems with aligning depth to color in C#.  Here's an example case that illustrates this.
https://github.com/IntelRealSense/librealsense/issues/4719

I have seen a few reports in the past 18 months about users experiencing problems with aligning depth to color in C#. Here's an example case that illustrates this.
#4719

I have reference the code and I have a simple code not using call backs, I have used the code mentioned in #4719. it gives me both aligned blocks but they are still not proper ColorFrame and DepthFrame (They are frames).   I need to call this AllocatedComposite() as mentioned in the code but I am not sure where to get the FrameSource object.  I just created one to try crashes. Any idea? I am just wonder where this FrameSource is coming from.
                   var aligned_frames=frames.ApplyFilter(align);
                    aligned_frames = aligned_frames.ApplyFilter(colorizer);
                    var aligned_frameset = aligned_frames.As<FrameSet>();

                    var colorFrame = aligned_frameset[Intel.RealSense.Stream.Color, Format.Bgr8];
                    var AligneddDepth = aligned_frameset[Intel.RealSense.Stream.Depth, Format.Rgb8];

This is still open, see my last reply

I considered this question further but it is outside of my programming knowledge, so one of the Intel RealSense team members on this forum will be better equipped to answer it.  I'll add a couple of tags to bring it to their attention.
@ev-mp  @dorodnic

Same problem here.
I found these lines in Align.cs
[Obsolete("This method is obsolete. Use Process with DisposeWith method instead")]
public FrameSet Process(FrameSet original, FramesReleaser releaser)
{
    return Process(original).As<FrameSet>().DisposeWith(releaser);
}

so i tried
```c#
Align align = new Align(Stream.Color);
FrameSet alignedframeset= align.Process(frameset).As<FrameSet>().DisposeWith(frameset);

It works, but after several frames. I got an exception: “Frame didn't arrived within 5000”

@mjyn   I hope the link below will be of help to you in finding working C# align code.
https://github.com/IntelRealSense/librealsense/issues/4719#issuecomment-526046153

@mjyn  To help avoid the frames din't arrive within 5000 error, C# users should ensure that their code is wrapped in a Using block as well as using dispose (as you have done).
https://github.com/IntelRealSense/librealsense/issues/2845#issuecomment-444448923

@mjyn To help avoid the "frames didn't arrive within 5000" error, C# users should ensure that their code is wrapped in a Using block as well as using dispose (as you have done).
#2845 (comment)

Seems like the Frame object before calling As should be disposed to avoid the exception.
using (var frames = pipeline.WaitForFrames())
{
    Align align = new Align(Stream.Color).DisposeWith(frames);
    Frame aligned = align.Process(frames).DisposeWith(frames);
    FrameSet alignedframeset = aligned.As<FrameSet>().DisposeWith(frames);
    var colorFrame = alignedframeset.ColorFrame.DisposeWith(alignedframeset);
    var depthFrame = alignedframeset.DepthFrame.DisposeWith(alignedframeset);
}

My code works now, thank you.

@mjyn   Great news, thanks for the update and for sharing your C# code with the community.   :)

Hi,
Will you be needing further help with this? If we don’t hear from you in 7 days, this issue will be closed.
Thanks


Update instructionset.py

definition of add() needs a return

Summary
Details and comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for your interest. If you're interested in contributing, please have a look at the issues tagged "good first issue" for an issue that doesn't have anyone else working on it.
This method doesn't need a return, and in general, we're unlikely to accept an external PR that isn't closing an issue (a bug or feature request) that a Terra maintainer has commented on, saying it is looking for community contributions.


Repetitive code to check for null/undefined Immutable objects

I find myself often checking whether variables are null or undefined before converting immutable objects to JS:
let category = page.get('category')
if (category !== null) {
    category = category.toJS()
}

I think shorthand for this would save a lot of lines of code. Something like:
const category = page.getJS('category')

Thoughts?

Closing this aging issue.
I'd like to avoid relying on toJS any more since it's a performance pitfall. Pro tip if you're doing lots of accessing, getIn can help access a chain of keys


Only minigpt4 use λLaux loss?

Only minigpt4 use λLaux loss in the source code?

Yes. In our experiment, we found that the MiniGPT, even after fine-tuning with style-consistent data, still exhibits a degree of degeneracy. Therefore, we introduced auxiliary task to further alleviate this degeneration phenomenon. For more powerful models, such as LLaVA and InstructBLIP, the preference fine-tuning does not affect the generation of the model, so auxiliary tasks were not employed. Details can be found in Sec5.2 in paper, where λ is set to 0 for LLaVA and InstructBLIP.


picky about the command line interface

Dear Ned,
I really love the idea and the concept - all the time and work you put into cog.
After I heard about it on Python-Bytes I was planning to use and distribute it through different parts of my professional life and personal projects.
What I saw that made me itch to support you was the command line interface.
I really think there is a great standard with the short / long option style like -c / --crc
If you then give it a broad help keyword - including those /? from windows the command line interface is really a self-documenting thing.
At the moment you mainly use short options.
Would you be interested in a suggestion from me in form of a pull request?
Would it be fine with you to add click as a dependency? (this is nice but not necessary)
Sincerly
Felix

Felix, thanks for enthusiasm!
Cog remains a back-burner project for me, though I still use it when needed.  I'm not opposed to adding long-option support, or a click dependency.  Honestly a big problem with cog is that the coding style is really archaic.  When I go into the code to make a change, I am tempted to change everything, which I have to resist!

which I have to resist!
Because of your time?

I would love to support you here.
I really love the concept and want to use it heavily with our coding projects.
I even have a small idea around a Qt and QtCreator project, where I need cog-superpowers.
I will come back to you with a suggestion...

suggestion:
cog [OPTIONS] [INFILE | @FILELIST] ...

INFILE is the name of an input file, '-' will read from stdin.
FILELIST is the name of a text file containing file names or
    other @FILELISTs.

OPTIONS:
    -c / --crc            Checksum the output to protect it against accidental change.
    -d / --delete         Delete the generator code from the output file.
    -D name=val /
    -DEFINE name=val      Define a global string available to your generator code.
    -e / --error          Warn if a file has no cog code in it.
    -I PATH /
    --INCLUDE PATH        Add PATH to the list of directories for data files and modules.
    -n ENCODING /
    --encoding ENCODING   Use ENCODING when reading and writing files.
    -o OUTNAME  /
    -out OUTNAME          Write the output to OUTNAME.
    -p PROLOGUE /
    -prepend PROLOGUE     Prepend the generator source with PROLOGUE. Useful to insert an
                          import line. Example: -p "import math"
    -r / --replace        Replace the input file with the output.
    -s STRING /
    --suffix STRING       Suffix all generated output lines with STRING.
    -U / --use-lf         Write the output with Unix newlines (only LF line-endings).
    -w CMD /
    --make-writeable CMD  Use CMD if the output file needs to be made writable.
                          A %s in the CMD will be filled with the filename.
    -x / --excise         Excise all the generated output without running the generators.
    -z / --end-eof        The end-output marker can be omitted, and is assumed at eof.
    -v / --version        Print the version of cog and exit.
    --verbosity=VERBOSITY
                Control the amount of output. 2 (the default) lists all files,
                1 lists only changed files, 0 lists no files.
    --markers='START END END-OUTPUT'
                The patterns surrounding cog inline instructions. Should
                include three values separated by spaces, the start, end,
                and end-output markers. Defaults to '[[[cog ]]] [[[end]]]'.
    -h / --help
    /? / -?     Print this help.

next action: design the interface in click (maybe first outside the cog-project)

Looks like in some places you used a single dash where you meant double (DEFINE, out, prepend).  Honestly, -D, -I, and -o are so common, I might skip the double-dash version.  If you want to keep them, use "--define" and "--include", the all-uppercase looks odd.
Thanks!

Ok, no define, include and out.
You are right - uppercase long names is very strange
Thanks for the fast replies.

WIP: first draft with click
Click has soooo many more possibilities.
TODO: look if TEXT and INTEGER could be renamed
TODO: insert all default values in documentation
TODO: look at click.File and click.Folder

  Console interface for cog_2020

  FILENAMES       is [IN_FILE | @IN_FILE_LIST] ...

  IN_FILE         is the name of an input file,                 '-' will
  read from STDIN. @IN_FILE_LIST   is the name of a text file containing
  IN_FILE names                 (it also can contain other @FILE_LIST names)

Options:
  --version                  Show the version and exit.
  -c, --crc / --no-crc       Checksum the output to protect it against
                             accidental change. [default: True]
  -d, --delete TEXT          Delete the generator code from the output file.
  -r, --replace TEXT         Replace the input file with the output.
  -e, --warn TEXT            Warn if a file has no cog code in it. [default:
                             True]
  -I, --path TEXT            Add PATH to the list of directories for data
                             files and modules.
  -D, --define TEXT          Define a global string available to your
                             generator code. (name=value)
  -p, --prepend TEXT         Prepend the generator source with PROLOGUE.
                             Useful to insert an import line.
                                     Example:
                             -p "import math"
  -s, --suffix TEXT          Suffix all generated output lines with STRING.
  -U, --use-lf TEXT          Write the output with Unix newlines (only LF
                             line-endings).
  -z, --end-eof TEXT         The end - output marker can be omitted, and is
                             assumed at eof.
  -x, --excise TEXT          Excise all the generated output without running
                             the generators.
  -n, --encoding TEXT        Use ENCODING when reading and writing files.
  -w, --make-writeable TEXT  Use CMD if the output file needs to be made
                             writable.`
                             A %s in the CMD will be filled with
                             the filename.
  -o, --outfile TEXT         Write the output to OUT_NAME. [default: IN_FILE]
  --verbosity INTEGER        Control the amount of output. 2 (the default)
                             lists all files,
                             lists only changed files, 0
                             lists no files.
  --markers TEXT             START END END-OUTPUT
                             The patterns surrounding cog
                             inline instructions.
                             Should include three values
                             separated by spaces, the start, end, and end -
                             output markers.Defaults to
                                     default =
                             "[[[cog ]]] [[[end]]]"
  --help                     Show this message and exit.

I am bit hung up about the test strategy for the command line interface.
@nedbat : What would you see as a acceptable strategy


check if authors (listwork) are registerd in listperson

@BOberreither  checked this with a little script, that I just run once (so I didn't automate this …)
331 authors where found, 13 are missing:

pmb2823
pmb26198
pmb7334
pmb10117
pmb53330
pmb10928
pmb2914
pmb10574
pmb14049
pmb2563
pmb10809
pmb13791
pmb2150


Tabs not cover all header space

I'm on Chrome 79.0.3945.79
custom-header 1.0.6
homeassistant 0.103.0
This is how lovelace appears with custom-header

...and this is how lovelace appears without custom-header...

As you can see, on the right there is a lot of space unused using custom-header

This is my configuration
custom_header:
  exceptions:
    - conditions:
        user: anna
      config:
        default_tab: anna
        disable_sidebar: true
        hide_config: true
        hide_help: true
        hide_raw: true
        hide_unused: true
        show_tabs: 'anna,clima,finestre'
        voice_hide: true

This is an intentional feature and was in CCH as well.  When the header compacts on scroll the buttons slide down so you have access to the buttons no matter where you are on the page. If you'd like to see more tabs you can also use chevrons: false in order to remove the scrolling arrows as you can still scroll through tabs with a swipe.
https://community.home-assistant.io/t/custom-header/155399/30

You could get even more tabs on screen by adding the voice and menu buttons to the overflow menu in the options button


Evaluating Pangenie vcf output with Truvari

Hi,
Thank you for developing PanGenie! Recently, I am trying to evaluate the output vcf of PanGenie with Truvari, but it comes up that the entries which have multiple sequence (separated by comma) in the ALT field are misclassified as false positives by Truvari. Is there any way or criteria to choose only one sequence from the ALT field, or find the most possible variant that corresponds to such an entry?
For example, given a record like this:

chr1	899960	.	TGGGAGGGGTCCGCGCGTCCGCAGTGGGGATGTGCTGCGGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGA	TGGGAGGGGTCCACGCGTCCGCAGTGGGGCTGCCGGGAGGGGGGCCGGGTCCACAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCGGGAAGGGGGGGGCCGGGTCCGCAGTGGGGCTGTGCTGCCGGGA,TGGGAGGGGTCCGCGCGTCCGCAGTGGGGCTGCCGGGAGGGGGGCCGGGTCCACAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCGGGAAGGGGGGGGCCGGGTCCGCAGTGGGGCTGTGCTGCCGGGA,TGGGAGGGGTCCGCGCGTCCGCAGTGGGGCTGTGCTGCGGGAAG,CCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCGGGAAGGGGGGGGCCGGGTCCGCAGTGGGGATGTGCTGCCGGGA,TGGGAGGGGTCCGCGCGTCCGCAGTGGGGCTGTGCTGCGGGAAGAGGGGGGCCGGGTCCGCAGTGGGGATGTGCTGCCGGGA,CGGGGA,CGGGGAGGGGGGCGCGGGTCCGTAGTGGGGATGTGCTGCGGGAAGGGGGGGGCCGGGTCCACAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCCGGGAGGGGGGCGCGGGTCCGCAGTGGGGATGTGCTGCGGGAA	.	PASS	AF=0.05,0.05,0.15,0.05,0.1,0.15,0.05;UK=301;AK=52,112,95,44,47,70,54,97;MA=8;ID=chr1-899972-SNV-0-1:chr1-899987-DEL-0-5:chr1-899998-SNV-0-1:chr1-900009-DEL-1-1:chr1-900018-SNV-0-1:chr1-900112-SNV-0-1:chr1-900115-SNV-0-1:chr1-900123-SNV-0-1:chr1-900124-INS-0-1:chr1-900141-SNV-0-1,chr1-899987-DEL-0-5:chr1-899998-SNV-0-1:chr1-900009-DEL-1-1:chr1-900018-SNV-0-1:chr1-900112-SNV-0-1:chr1-900115-SNV-0-1:chr1-900123-SNV-0-1:chr1-900124-INS-0-1:chr1-900141-SNV-0-1,chr1-899989-SNV-0-1:chr1-900001-SNV-0-1:chr1-900003-DEL-0-151,chr1-899955-DEL-0-113:chr1-900112-SNV-0-1:chr1-900115-SNV-0-1:chr1-900123-SNV-0-1:chr1-900124-INS-0-1,chr1-899989-SNV-0-1:chr1-900000-INS-0-2:chr1-900009-DEL-2-115,chr1-899955-DEL-1-189:chr1-900150-SNV-0-1,chr1-899955-DEL-2-37:chr1-900019-SNV-0-1:chr1-900036-SNV-0-1:chr1-900039-SNV-0-1:chr1-900047-SNV-0-1:chr1-900048-INS-0-1:chr1-900056-SNV-0-1:chr1-900150-SNV-0-1:chr1-900153-SNV-0-1	GT:GQ:GL:KC	3/3:10000:-82.8146,-359.1,-264.5,-291,-281.3,-185.3,-93.41,-258.7,-197.2,0,-138.8,-309.4,-199.5,-35.39,-57.8,-169.7,-335,-273.5,-101.3,-122.4,-72.88,-187.1,-362.2,-315.3,-117.7,-191.2,-194,-120.8,-247.5,-409.4,-342.2,-178.1,-270.1,-210.3,-201.2,-202.4:52

how should I interpret it and convert it (if possible) to a simple variant record that only has one sequence in both REF and ALT?
Thank you very much
Best

The most likely alleles the sample carries for that variant are defined by the genotype (here 3/3). I guess what you want to do is create a biallelic version of the VCF? How was the input VCF that you used for PanGenie created?


Add "building OKR" to PM's responsibility

What
Title says it all.
Why
Product Managers in Increments are responsible for building OKRs for their teams' products and it's one of the most important responsibilities.

@takoratta Review please :pray:

LGTM

Thanks :heart:


[Bug]: Elements on editor watch window overlap

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Elements on the watch window overlap on smaller resolution. We can instead move all operations to the second line and have only function name appear on the first line

Steps To Reproduce

Keep entity explorer and console open when on JS editor and vary their sizes to observe how the watch window elements behave

Environment
Release
Version
Cloud

Another example of an overlap

@ramsaptami this is the older designs for the JS editor which is being completely revamped. This issues will be resolved with the new JS editor designs.
New Designs
https://www.figma.com/file/sT3HR1VSWOTtPfh5vlpU8x/?node-id=0%3A1

should become irrelevant after #10387


Capella processing

Partially to initiate a discussion.
Basically use a single pipeline for withdrawal processing to simplify the flow and reduce the need for full validator scans during epoch processing.
This should make the feature much more testable amongst other benefits, and makes withdrawal of partial or full amounts an equal priority based on your position in the validator list.
The timing remains approximately equal to previous, except there may have previously been 4 exits per epoch to add that are now just done during sweeping, where previously they were done as a priority.

For the record, the better testability of this approach is because the spec suggests client's optimise by tracking validators which become eligible for withdrawal to avoid having to scan the whole validator set. But then any reference tests which focus specifically on a spec method to do the full iteration and identify newly withdrawable validators wouldn't test that optimisation. To test properly you'd need to do the full epoch processing to ensure the caching code was actually used.

An alternative approach would be to process withdrawals while processing effective balance changes on the same loop, but assessing the safety of that may be tricky

This is based on an old commit in dev. I reworked it to be on recent dev (which had a number of capella cleanups) in #3042

Ok appreciate your help.  Strange anomalies occur constantly
Sent from Outlook for iOShttps://aka.ms/o0ukef

From: Danny Ryan @.>
Sent: Monday, October 17, 2022 8:03:18 AM
To: ethereum/consensus-specs @.>
Cc: Subscribed @.***>
Subject: Re: [ethereum/consensus-specs] Capella processing (PR #3032)
Closed #3032https://github.com/ethereum/consensus-specs/pull/3032.
—
Reply to this email directly, view it on GitHubhttps://github.com/ethereum/consensus-specs/pull/3032#event-7596945610, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV4C5HQHWND42VHCFIEDT4TWDRURNANCNFSM6AAAAAARDPCKPY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Vim9: trying to redefine a function-local function is silently ignored

Describe the bug
Trying to redefine a function-local function is silently ignored.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    def Outer()
        def Inner()
            echo 'first'
        enddef
        def Inner()
            echo 'second'
        enddef
        Inner()
    enddef
    Outer()
EOF
)

Outer() outputs first.
Expected behavior
Outer() raises an error when it tries to redefine the Inner() function.
Environment

Vim version: 8.2 Included patches: 1-1328
OS: Ubuntu 16.04.6 LTS
Terminal: XTerm(358)

Additional context
The same issue applies to a function local to a block:
vim9script
def Outer()
    if 1
        def Inner()
            echo 'first'
        enddef
        def Inner()
            echo 'second'
        enddef
        Inner()
    endif
enddef
Outer()

first

And to an imported function:
vim9script

mkdir('/tmp/import', 'p')

let lines =<< trim END
    vim9script
    export def Func()
        echo 'first'
    enddef
END
writefile(lines, '/tmp/import/a.vim')

lines =<< trim END
    vim9script
    export def Func()
        echo 'second'
    enddef
END
writefile(lines, '/tmp/import/b.vim')

set rtp+=/tmp
import Func from 'a.vim'
import Func from 'b.vim'
Func()

first

This is inconsistent with functions which are local to a script:
vim9script
def Func()
    echo 'first'
enddef
def Func()
    echo 'second'
enddef

E122: Function <SNR>1_Func already exists, add ! to replace it

Notice how an error message is correctly raised (although a bit misleading).

The issue is specific to :def functions inside a :def function.
It does not affect :fu functions inside a :fu function:
vim9script
fu Outer()
    fu Inner()
        echo 'first'
    endfu
    fu Inner()
        echo 'second'
    endfu
    call Inner()
endfu
Outer()

E122: Function Inner already exists, add ! to replace it

And it does not affect :def functions inside a :fu function:
vim9script
fu Outer()
    def Inner()
        echo 'first'
    enddef
    def Inner()
        echo 'second'
    enddef
    call Inner()
endfu
Outer()

E122: Function Inner already exists, add ! to replace it

Fixed by 8.2.1349.


GH-43743: [CI][Docs] Ensure creating build directory

Rationale for this change
It's used as a volume. If it doesn't exist, docker compose reports an error:
Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /home/runner/work/crossbow/crossbow/build/

What changes are included in this PR?

Create build directory
Move required -v $PWD/build/:/build/ to docs/github.linux.yml

Are these changes tested?
Yes.
Are there any user-facing changes?
No.

@github-actions crossbow submit test-debian-12-docs preview-docs

Can we merge this?

I'll merge this to fix nightly docs update.
If there are any problems, let's fix it as a follow-up task.


Support checking repository on push to branch (not via PR)

Owners can push directly to master/other branches:

We need the permissions and workflow figured out before a release is public, as it's not clear exactly how to add more permissions to an integration at a later stage.
Note many people would push to their repository then make a PR, this workflow should ideally not cause GopherCI to run on push and then on PR creation.

We'll need to also support sending an email notification to the user on failure, but it's not clear to me exactly how to get the user's permission. I presume only members of the user/organisation can push directly to a branch, and in which case I may or may not have access to their private email address.

Additionally, gopherCI seems to only build a PR on the first opening of it, not for subsequent commits on the same PR (not sure if I'm misreading this though)

Yeah and I'm tracking that in https://github.com/bradleyfalzon/gopherci/issues/24 it's a really simple fix so will address soon.

This is now being blocked by https://github.com/google/go-github/pull/533 but I don't think I'll have an implementation before that's merged in anyway.

Note many people would push to their repository then make a PR, this workflow should ideally not cause GopherCI to run on push and then on PR creation.

Hehe, Travis does that and it's indeed suboptimal. But the fact they haven't resolved it makes me unsure if it's viable to solve it well (maybe it is, maybe it isn't).
Are you planning on using commit statuses to report CI result on pushed commits? I guess so based on google/go-github#533. That's what Travis does too, and I think it'll be great. Just wanted to give the approach a 👍.

Correct, that's what the go-github PR is for, just working out how to best make https://github.com/bradleyfalzon/gopherci/blob/master/internal/github/handlers.go#L67 a little more generic (move to event method which accepts statuses url, analyser.config and installationID), so a small refactor is needed but I think it's all relatively straight forward.
To begin with, I'll just hit the statuses API and in the near future, we can organise email notifications (and other notifications customisation in general), but the first step is having us check the push request.


Yin-yang can't ignore // aka comments in the config of vscode

Describe the bug
Yin-yang doesn't switch a theme of vscode. The error is
KeyError: 'autogenerated'
    self.set_theme(theme)
    raise ValueError(f'Theme \"{theme}\" is invalid')
  File "/opt/yin-yang/src/plugins/vscode.py", line 98, in set_theme
ValueError: Theme "" is invalid
    raise e
  File "/opt/yin-yang/src/plugins/vscode.py", line 86, in set_theme
    settings = json.load(sett)
               ^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/json/__init__.py", line 293, in load
    return loads(fp.read(),
           ^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/json/decoder.py", line 353, in raw_decode
    obj, end = self.scan_once(s, idx)
               ^^^^^^^^^^^^^^^^^^^^^^
json.decoder.JSONDecodeError: Expecting property name enclosed in double quotes: line 7 column 5 (char 214)

I found out it that yin-yang couldn't ignore // aka comments in the config of vscode. If I remove one comment it fails with another one. If I remove all comments, it works.
Enabled plugins
What plugins did you have enabled? If not relevant, you can remove this.

[ ] Atom
[x] Brave
[x] Colors
[ ] Custom Script
[x] Firefox
[ ] Gedit
[x] GTK
[x] Konsole
[ ] Kvantum
[ ] Only Office
[x] System
[x] VSCode
[ ] Wallpaper

Affected versions

Yin-Yang version: 3.2
VScode: 1.79.2
Python version:  3.11.3
Qt version: 5.15.9

Thats a known issue, see #120


New colours!

Fixes a11y issues with colour palette (figma)
Moves to preset themes for more control over the default variations
Supports a preset theme = none in case consumers want more granular control by supplying custom CSS variables

:tada: This PR is included in version 1.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Migrate from checks to commit statuses

What has been done

Migrated from checks to commit statuses

How to test

Merge PR
Leave a comment on PR and check action build result
Confirm last PR commit status updated

/robin test commit status


BUG: Cannot seek vector iterator after end

Hi @mlivesu,
I was able to compile the main branch but the library crashes on a simple test.

These 2 cubes are saved as obj Input.zip and successfully loaded:
    std::vector<double> in_coords, bool_coords;
    std::vector<uint> in_tris, bool_tris;
    std::vector<uint> in_labels;
    std::vector<std::bitset<NBIT>> bool_labels;
    std::vector<std::string> files;

    files.emplace_back("c:\\1.obj");
    files.emplace_back("c:\\2.obj");

    loadMultipleFiles(files, in_coords, in_tris, in_labels);

when I try performing a simple UNION operation,
booleanPipeline(in_coords, in_tris, in_labels, UNION, bool_coords, bool_tris, bool_labels);

the app crashes with the following error:

I have stepped through the execution logic and was able to pin-point it to this line:
https://github.com/gcherchi/InteractiveAndRobustMeshBooleans/blob/e14f8d3daa7129b8f03f1b6c93689dabe9360165/arrangements/code/triangulation.cpp#L423-L426
I'm working on Windows 10, with VS 2022. After cloning, the library returned 100s of errors and wouldn't compile.
I forced it to work with in Cmake:
add_definitions(-DNOMINMAX)
add_definitions(-DWIN32_LEAN_AND_MEAN)

Any ideas how to fix this issue?

Dear Mariusz,
I checked your meshes.
The problem is that they are quadrilateral meshes.
As we explain in our paper, our code works with triangle meshes.


Fix build times

Investigate why build times are much higher in VES than they are when building through the terminal.
Might have to do with file watchers, output parsing or such?

Note: build times seemingly have improved with the latest Theia updates.


chore/readme_models

Describe Your Changes

Fixes Issues

Closes #
Closes #

Self Checklist

[ ] Added relevant comments, esp in complex areas
[ ] Updated docs (for bug fixes / features)
[ ] Created issues for follow-up changes or refactoring needed

Stale and closing


add read path from object input

Relates #1260
Implements

add block to read path from input object

@devonfw/cobigen

@jan-vcapgemini please add this properly to the correct milestones / project and add proper labels before merging


Speed up streaming output by buffering till newline

Inspired by https://github.com/karpathy/llama2.c/pull/193, see that for details.
Performance difference between buffering beyond newline isn't measurable, but this is much simpler. Current implementation only supports 0 and 1 tokens, again for simplicity. Buffering all output can be achieved by redirecting to a file. Unclear if the number of tokens other than 0, 1, inf is useful.

Is this even faster / does it matter? I don't really think so?

In the PR this was inspired by people report up to 50-80% speedups. I get a couple of percent speedup. Presumably the effect is larger on fast machines (GPUs/etc) with small models.


[VPR] 7.05 Support

Pull request type

[x ] This is new functionality or an addition to existing functionality
[ x] This is a patch bump

Pull request details

[x ] The goal of this PR is detailed below:

Splits Procs into 3 catergories. Finishers, Starters and Ogcds.

[x ] I have updated the data files for all potency changes for this patch
[ x] This is the last PR after all analysis changes for this job have been merged, and now the job is ready to be marked supported for this patch

Testing / Validation
I can't test anything until ACT is updated. Do not merge until this is tested. Best I can tell you is that it compiles and doesn't implode at the moment.
Job Maintenance

[x ] I am active on the xivanalysis Discord and part of the job maintainers group for the job(s) in this PR
[ ] I have collaborated with one or more job maintainers for the job(s) in this PR while developing it
[ ] I prefer to receive any communications through GitHub only

http://localhost:3000/fflogs/rkY4NzRjvW16LZGm/19/11 (m1s log)
Good news, it works. I just have to fix merge conflicts now,

@AzariahAeroborn requesting a 2nd readthru, it should be good to land now.
I reverted back to ids because the lint tests failed to due your Action: SHARED.UNKNOWN change.
I did have to edit coreProcs to have a ReactNode  for a timelineLabel to allow implementors to change the procs group label, but it's a minor 2 liner.


[Compute] Fix #27446: az vm encryption enable: Fix using incorrect client when --key-encryption-key is specified

Related command

az vm encryption enable
Description

Fix #27446
The command accepts --key-encryption-key to encrypt the disk encryption key.
However, when converting the key name to a URL, a CertificateClient is used instead of a KeyClient. which causing  an error because the CertificateClient object is meant for managing certificates, not keys.
The bug was introduced in #26991 , which updated azure-keyvault package to azure-keyvault-key/azure-keyvault-secret/azure-keyvault-certificate.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


Change onSuccess ticksDelta from Int to Long

If the delta is more than 24 days, we cap it because it's Int's max value. But we shouldn't restrict it here.

@artem-zinnatullin does this require a major version change since the listener interface argument type changed?

Technically, yes, however we haven't shipped v1.0.0 yet, it was not deployed to Maven Central back then and we haven't announced the library yet.
What we can do is remove that git tag and set it once you're ready to actually ship v1.0.0

Technically, yes, however we haven't shipped v1.0.0 yet, it was not deployed to Maven Central back then and we haven't announced the library yet.
What we can do is remove that git tag and set it once you're ready to actually ship v1.0.0

sounds good!


Read a paragraph of highlighted text or a PDF page

I really like the bilingual reading function of this extension. There are almost no tools that support this, but unfortunately it can only read one sentence. Hopefully the team can soon develop the function to read a whole page or several pages. page, a paragraph is selected.
In addition, it is recommended to set separate reading speed levels for the source language and translation language.

to fix your trouble check this solution click
maybe this will solve your problem.

I share your opinion.
I hope the functionality to read paragraphs or entire texts can highlight the sentence being read. Also, it would be nice to have a "read selected text aloud" option in the right-click menu.


Just wanted to say thank you.

Thank you for sharing this awsome project. I have ordered the parts that is required to build this project.
Can it learn long-ir codes. Some of the aircon models like Daikin use long IR codes. Does it work ?

@kakopappa
Yes, it does work:)
Any type of infrared radiation in the 38 kHz band can be used.


can't not copy hidden or not display element's content.

Closing since it didn't follow the issue template.


use distroless nonroot

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sadysnaat
To complete the pull request process, please assign abhinavnagaraj
You can assign the PR to them by writing /assign @abhinavnagaraj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


⚠ Bofrost.at has degraded performance

In a1ed314, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1209 ms

Resolved: Bofrost.at performance has improved in 7cd03f1 after 11 minutes.