Anf 7051 2020 05 01 api version swagger linting issues kpi

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

This adds API version 2020-05-01 that is mentioned in NetAppFiles ARM Manifest but was not officially released. The additions are therefore functionally the same as 2020-06-01. The PR addresses this Swagger Completeness issue
https://msazure.visualstudio.com/Swagger KPIs/_workitems/edit/8188089
SDK will not need to be generated from this as all changes here are represented in the latest API version 2020-06-01
Note 2020-05-01 API-Version will be in another PR.
Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run unifiedPipeline

/azp run automation - sdk


Add print test cases

In order to be able to read sink in test cases, I changed Output to hold sink as a Box<&mut dyn Write>
I didn't know if this is a good approach so I wanted you to see this before I add more tests.

Thank you! I'll continue to work on the other tests.


why split android and ios versions?

I saw that each field type has two different versions. One for ios and one for android (for example InputField.ios.js and InputField.android.js).
What is the reason for this? That seems to go completely against the philosophy of react-native (having a unified version of an app between different platforms) and is a potential source of inconsistencies (like 'right Icon' that appear on the same lime as the textfield on ios but on a different one on android due to different default stylesheets between the two classes...)
To me, it would make more sense to have android and ios share as much code as possible (same default stylesheets and templates...)

The fields have version for android and a version for ios just for styling purposes but the underlying component is unified.

Well it would make more sense to have the same default styling between both ios and android (and to provide android or ios specific styling in a platform conditional stylesheet that can be passed to the 'style' tag of the component if someone really wants to have a different style on the two platforms).
Because here the problem is that the default styling is inconsistent between the two platforms. So if you extend the default styling (by passing a new style to the style tag of your component), your component would inherit different parent style depending on which platform you are on...

That's right, I'm also thinking to implement the same style for both.
The problem is that android and ios have a different style and unless
you're making a sort of game (and I would suggest to create custom
components for that) having the same style will easily make your form look
different from the whole ecosystem.
I'm about to submit another update that will fix the android
inconsistencies and will also expose the components to allow a complete
customization of the fields.
On Tue, Aug 23, 2016, 5:22 PM ramsestom<EMAIL_ADDRESS>wrote:

Well it would make more sense to have the same default styling between
both ios and android (and to provide android or ios specific styling in a
platform conditional stylesheet that can be passed to the 'style' tag of
the component if someone really wants to have a different style on the two
platforms).
Because here the problem is that the default styling is inconsistent
between the two platforms. So if you extend the default styling (by passing
a new style to the style tag of your component), your component would
inherit different parent style depending on which platform you are on...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MichaelCereda/react-native-form-generator/issues/54#issuecomment-241883319,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGjfk76VtCezAvBvNxAufXq51PiuSpKks5qi2SlgaJpZM4JrTDo
.

--
Michael Cereda
http://michaelcereda.com

The problem is that android and ios have a different style and unless
you're making a sort of game (and I would suggest to create custom
components for that) having the same style will easily make your form look
different from the whole ecosystem.

I think you should just propose ios or android styles as themes (with stylesheets), that the user can choose or not to apply to the component, and keep the default styling identical between both platforms.
Having a platform specific theme as stylesheets doesn't prevent from customizing these styles by overriding them when needed. And, at least, the user can choose or not to apply platform dependent styling and you won't have anymore inconsistencies when overriding the default styling....


WorkShop isn't displayed in "Кабінет надавача послуг" after it's creation

Environment: https://oos.dmytrominochkin.cloud/#/
Reproducible: always
Preconditions

Follow the link https://oos.dmytrominochkin.cloud/#/
Log into OoS website as SP
Select «Мої гуртки» in the menu bar in the top right corner.
Workshop is already created

Steps to reproduce

Open the link: https://oos.dmytrominochkin.cloud/#/personal-cabinet/provider/workshops
Open DevTools (F12)
Refresh the page

Actual result
UI: The workshops don't displayed in the "Кабінет надавача послуг"

Back-end:
Request URL: https://api.oos.dmytrominochkin.cloud/api/v1/Workshop/GetWorkshopProviderViewCardsByProviderId/00000000-0000-0000-0000-000000000000?From=0&Size=12
Request Method: GET
Status Code: 400
Response: Provider id is empty.
Expected result
The Worlshop is displayed in "Кабінет користувача".
Request without the error
User story and test case links
https://jira.softserve.academy/browse/OS-351
Labels to be added
Severity:major, Type ("Functional").

@val-ukr  bug is not reproducible

Log as provider
Логін<EMAIL_ADDRESS>Пароль: A12345a@
Go to personal cabinet -> My Workshops -> click on add new workshop

fill in information about the new workshop  and submit

Workshop as added to teh personal cabinet

Логін<EMAIL_ADDRESS>Пароль: Asdf1234!

The bag was fixed, I created a new account


psql: send transaction state to clients

Certain clients (such as lib/pq) require the changed transaction state to be pushed to the client via the readyForQuery message, when the client issues a BEGIN or COMMIT.

@chaudum did we have some case where something in the clients doesn't work because of this? I'd like to pick this up again eventually if this was the case.

@mfussenegger lib/pq does not work because of the missing transaction state:
	if cn.txnStatus != txnStatusIdleInTransaction {
		cn.bad = true
		return nil, fmt.Errorf("unexpected transaction status %v", cn.txnStatus)
	}


Roncy3

What the program do ?
swap 2 numbers
In what programming language it is written?
python

resolve your conflict


The host id and container id are the wrong way round.

For example when mapping a single group, for example guest's 1000 to the host's 1001.
It produces this for the .conf:
# Generated with Proxmox LXC idmap helper created by Stanisław Nieradko
lxc.idmap: u 0<PHONE_NUMBER>6
lxc.idmap: g 0<PHONE_NUMBER>
lxc.idmap: g 1001 1000 1
lxc.idmap: g 1002<PHONE_NUMBER>4

The correct .conf is actually:
lxc.idmap: u 0<PHONE_NUMBER>6
lxc.idmap: g 0<PHONE_NUMBER>
lxc.idmap: g 1000 1001 1
lxc.idmap: g 1001<PHONE_NUMBER>5

The subgid and subuid are correct however.
I have tested this.

I created a PR for this. Feel free to use https://jo-me.github.io/proxmox-idmap-helper in the meantime.

Thanks, let me know if you need any assistance, i can't promise anything but I'll do my best.


Weird behaviour in RSI moment indicator on development branch

I have one question about the development branch. I am currently plugging it into our project I have noticed that the rsi momentum indicator does not seem to behave like the one defined on the main branch. to be more specific, it seems to me that it is not returning nan values for the first 14 rows of data when we use the default length=14 parameter. There is only a nan value in the first row.
Screen:

When I run the same code using the latest release of 0.3.14b0 the expected nan values are there for the first 14 rows.
Screen:

Is this a bug on the development branch or is this logic something to be expected in future releases?

Hello @einarjohnson,
RSI works as intended with the prenans. There are no plans to alter this behavior.

Unfortunately, I am unable to replicate you results with the development branch. What is your code to reproduce the resultant output?
Kind Regards,
KJ

@einarjohnson
Oh I see... you are not using TA Lib. Looks like you are correct with the lack of prenans for the Pandas TA RSI version.

You are correct that it should have prenans there. I'll add it to the todo list. Thanks for the heads up.

hi @twopirllc,
thanks for such a lighting fast response. :)
i think the issue boils down to the fact the development branch is using rma as the default ma instead of sma which is on the main branch.
If I set the mamode="sma" parameter like so
import pandas_ta as pta
df[DataframeColumns.RSI] = pta.rsi(
    df[DataframeColumns.CLOSE], length=self.settings.rsi_default_length, mamode="sma"
)

the prenans are returned as expected.

@einarjohnson
Oh yeah! It has been awhile since I touch rsi. Seems I chose to default to TradingView's rma calculation if TA Lib is not installed.

Interesting. SMA seems to be the default one when I use TW.

maybe this is a user setting or something that is bound to my account, could very well be since I use TW so much and I might have changed this at some stage (i can't remember doing that though)
Anyways, your help and input is very much appreciated and thanks for this brilliant library, it has proven most useful for our project.

@einarjohnson
Yes it is easy to forget changing settings over time.

Anyways, your help and input is very much appreciated and thanks for this brilliant library, it has proven most useful for our project.

Sponsorship or donations help massively to keep this project alive. 😎
Buy me a Coffee
Thanks!

🤣

apologies for lateness here, been up to my eyeballs with life/work/kids/etc. the coffee has been delivered. <3

I know how it is. Much appreciated. Thank you!


Fix for #17: Check for JAVA_HOME set in Xcode custom paths

Fixed https://github.com/Kotlin/kdoctor/issues/17

Fixed https://github.com/Kotlin/kdoctor/issues/17


fix all clippy warnings

Describe the feature
there are clippy warnings:
cargo +nightly clippy --all --all-features

TODO
fix all obvious clippy warnings
Additional context
No response

I don't see any clippy warnings, was this addressed?

oh my bad they're not warning but suggestions:
cargo +nightly clippy --fix --all-features -Z unstable-options

I can try this one

oh my bad they're not warning but suggestions:
cargo +nightly clippy --fix --all-features -Z unstable-options

I just tried doing this but not getting any suggestions, not sure if I am doing it right. Do you have any suggestions?

ah sorry, forgot to add the --all argument as well...
it should be
cargo +nightly clippy --fix --all --all-features -Z unstable-options


feat: Introduce Copy Button at Top of Code Blocks

Description
This pull request introduces a copy button at the top of each code block. Users can easily copy the code by clicking the button, providing a convenient and accessible way to copy code snippets.
Changes Made

Added a copy button (with an SVG icon) at the top of each code block.
Improved the layout to ensure the button remains fixed at the top even during scrolling.
Enhanced the styling for a better user experience.

Screen Shots

Resolved:
#8 Make code block code easily copyable
#36 Added Js for fixing this issue

status?

status?

I'm pretty busy these days and it's not unusual for issues and PRs to be a few weeks before I get another burst of time. Just setting realistic expectations :). That said, I'm trying to carve out a little time more regularly, but I can't say when exactly I'll get to this one.


[auto-bump] dependency by max-hoffman

:coffee: An Automated Dependency Version Bump PR :crown:
Initial Changes
The initial changes contained in this PR were produced by `go get`ing the dependency.
```
$ cd ./go
$ go get github.com/dolthub//go@
$ go mod tidy
```
Before Merging
This PR must have passing CI and a review before merging.
After Merging
An automatic PR will be opened against the LD repo that bumps the bounties version there.

This PR has been superseded by https://github.com/dolthub/bounties/pull/2746

This PR has been superseded by https://github.com/dolthub/bounties/pull/2747


CB-21006 - DataHub - Flow changes to support mounting Ephemeral volumes

DataHub - Flow changes to support mounting Ephemeral volumes & modify config for log and local directories for services on ClouderaManager

1 small ask. Furthermore, there are still some earlier comments open for CoreVerticalScaleActions and CoreVerticalScaleService.pollClouderaManagerServices().

@lajosrodek please do a final review. I have changed the polling to com.dyngr.Polling, also the passing of variables in Actions


dbm2135 Assignment Week 2

Completed week 2 assignment by Dorothy MacAusland

Correct Week 2 Assignment file is notebooks/week-2/dbm2135 Assignment 2 Complete.ipynb

Good work!


Login redirects to broken URL

I am running this bot on Glitch and the both the login button and links take me to a bad URL.
There are no errors in the logs and I have not changed anything in the dashboard.js file.
Images for context:

I can tell that the client id is in the wrong place in the url but everything is in the correct order in dashboard.js

Can you please provide the following settings from the config.js file:

dashboard.secure
dashboard.domain

secure: 'true',
domain: process.env.DOMAIN,
ENV file:
DOMAIN=https://dzshpee.glitch.me
I don't see how this affects the link though.

Oh I feel so dumb now. The config enabled https and I included https in the link.


Deno usage?

Is it ok to use this in deno, just like this?
import Enum from 'https://cdn.jsdelivr.net/gh/adrai/enum/lib/enum.js'

@ry @bartlomieju @piscisaureus can you give some feedback?
I've no experience with deno and neither with TypeScript.

https://github.com/denoland/deno_website2/pull/316

ok, so now like this ;-)
import Enum from 'https://deno.land/x/enum/index.js'


[chore] Move AWS ECS, EKS and Log attributes to the registry

Related to https://github.com/open-telemetry/semantic-conventions/issues/407
Changes
Move AWS ECS, EKS and Log to the attributes registry
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[ ] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[ ] schema-next.yaml updated with changes to existing conventions.

Let's have a single aws file in the registry. See related comment in https://github.com/open-telemetry/semantic-conventions/pull/850/files#r1542415558.

Let's have a single aws file in the registry. See related comment in #850 (files).

@AlexanderWert fixed!


Feature/fiber

add fiber v2

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.81%. Comparing base (94b8b9f) to head (cb5f0a2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage   83.81%   83.81%
=======================================
  Files          19       19
  Lines         791      791
=======================================
  Hits          663      663
  Misses        102      102
  Partials       26       26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

LGTM

@NameHaibinZhang Don't forget to update README.md in the root directory

@NameHaibinZhang Don't forget to update README.md in the root directory

this change is changed


How to enable Fast Multi-Frame Metric Reconstruction display in rviz?

I launch roslaunch kimera_vio_ros kimera_ros_euroc.launch

How to enable Fast Multi-Frame Metric Reconstruction display in rviz?
Is it possible to make Slow Global Metric Dense Reconstruction without semantic information?

I did not have time to implement this one.
Yes, but I need to document how to do that (let me do this in Kimera-Semantics repo though).

For 2. Checkout https://github.com/MIT-SPARK/Kimera-Semantics/issues/7


Move environment functionality to utils

Since multiple ignition libraries make use of setting/getting environment variables, it makes more sense to locate it here.

Codecov Report

Merging #30 (db81dfe) into ign-utils1 (e9c439b) will increase coverage by 1.72%.
The diff coverage is 95.23%.

@@              Coverage Diff               @@
##           ign-utils1      #30      +/-   ##
==============================================
+ Coverage       89.58%   91.30%   +1.72%
==============================================
  Files               3        4       +1
  Lines              48       69      +21
==============================================
+ Hits               43       63      +20
- Misses              5        6       +1

Impacted Files
Coverage Δ

src/Environment.cc
95.23% <95.23%> (ø)

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 e9c439b...db81dfe. Read the comment docs.

Perhaps we can just document this in Environment.hh.

Added documentation.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1


Add Phase for alpha or beta

This PR adds a phase attribute for a Content Item. It can only be present with
alpha or beta as the attributes. This can then be used by the Rendering
Application to switch functionality or visual features based on the phase.

@tommyp after removing the 'phase' from frontend_links_definition.json you'll also need to re-run make to regenerate the frontend schema.json files.

@heathd Good point. Done!

IMO, content-store should be updated to support this field before this is merged.

https://github.com/alphagov/content-store/pull/138 has been merged and deployed, so I guess this is good to merge too, right?

@tijmenb Yeah. This can be merged now.


Makefile: move list of K8S system tests to var

This PR moves the list of Kubernetes system tests to be run to a variable. This will allow us to split the tests to run into multiple separate runs which take less time to run.
This addresses CNTV-36.

build PR

build PR

I see Jenkins raised Java exception when sending email to<EMAIL_ADDRESS>http://<IP_ADDRESS>/job/Netplugin_CI/9051/console
Sending e-mails to<EMAIL_ADDRESS>ERROR: Could not connect to SMTP host: localhost, port: 25
javax.mail.MessagingException: Could not connect to SMTP host: localhost, port: 25;
  nested exception is:
	java.net.ConnectException: Connection refused (Connection refused)
	at com.sun.mail.smtp.SMTPTransport.openServer(SMTPTransport.java:1934)
	at com.sun.mail.smtp.SMTPTransport.protocolConnect(SMTPTransport.java:638)

This PR can be tested after https://metacloud.jira.com/browse/CNTV-68 is fixed to make the Contiv Jenkins gate use the new options 00SSH|TestBasic|TestNetwork|TestPolicy. Will give my +1 after https://metacloud.jira.com/browse/CNTV-68 is fixed.

build PR

@vhosakot: No, this can be tested right away.

LGTM

@srampal @rchirakk: Can you merge this when it's done running on the CI, please?

LGTM!

This is green.
@rchirakk @srampal: Can you merge this, please?


Refactored restart logic with replan for workload containers

Issue
Clean up the restart logic in the charms' _configure method.
Solution
The node_exporter workload container was successfully switched to pebble.replan() logic (although the pebble layer will not change throughout the charm's lifecycle). Since loki has "startup": "disabled" set in the pebble layer, there is still a requirement for some restart logic.
Context
pebble.replan() will only restart services that have "startup": "enabled" set in the pebble layer.
Testing Instructions
Haven't tested integration tests locally due to charm timeout.
Upgrade Notes
None

the failing itest is likely due to a missing can-connect guard in the _update_certs path

@PietroPasotti There is already a con_connect guard in the _update_cert method. Were you referring to somewhere else in the code?

huh. Interesting. Then I misread the message, or even can_connect errors for some reason


get elements from board as JSON

It would be nice to have a function to export the elements to JSON.
Using this it should be easier to send the elements to ajax for example.

This is nice. I had to keep jKanban data in a var to save on database. Having a function to export kanban to json will make that simpler.

Can I look into it?

Can I look into it?

For sure. Pull requests are always welcome.

This is very tricky. We have two approaches according to my knowledge.

Create an array attribute and change it on addElement, addBoards, removeElement and removeBoard calls to keep it even with DOM.
or
Read jKanban DOM and output a JSON.

I would like to vote which you guys think it's better.

i think approach 1. could have some potential inconsistency when the user could be frantically draging the boxes here and there.
therefore i consider approach 2 more trustworthy.
this could help

 methods: {
        renderJSON(){
            const obj = this.$refs.demo2
            let boards = []
            obj.querySelectorAll('.kanban-board').forEach(el => {
                let items = []
                el.querySelectorAll('.kanban-item').forEach(i => {
                    items.push({
                        title: i.innerHTML,
                    })
                })
                boards.push({
                    id: el.getAttribute('data-id'),
                    title: el.querySelector('.kanban-title-board').innerHTML,
                    items,
                })
            })
            return boards
        }
    },

I think this needs to be updated to return the name of the item not the HTML
renderJSON = ()=>{       const obj = document.getElementById("myKanban");       let boards = []       obj.querySelectorAll('.kanban-board').forEach(el => {           let items = []           el.querySelectorAll('.kanban-item').forEach(i => {               items.push({                   id: i.childNodes[1].innerHTML,               })           })           boards.push({               id: el.getAttribute('data-id'),               title: el.querySelector('.kanban-title-board').innerHTML,               items,           })       })       return boards     }


Rename ProductSpace as BlockVectorSpace

Fixes #92. Class ProductSpace from module psydac.linalg.block caused a name clashes with class ProductSpace from module sympde.topology.space. Therefore we have renamed it as BlockVectorSpace, which is consistent with the other names in the same module.

New naming seems more consistent indeed: Vector -> VectorSpace / BlockVector -> BlockVectorSpace ...

In fact this was your suggestion in the first place! :smile:


Fix ambiguous call with untyped int literal in {JSON,YAML}::Any.new

Adds a workaround for #11645

Perhaps. But the main intention is to get it working with untyped literals.
Ideal solution would be fixing the autocasting.

I don't think we need to restrict ourselves to the number types supported by the Anys, instead we could just support all of them:
def initialize(@raw : Type)
end

# this overload goes below the `Type` one to avoid an infinite recursion,
# as the two overloads are unordered
def self.new(raw : Int)
  new(raw.to_i64)
end

def self.new(raw : Float)
  new(raw.to_f64)
end


Unable to install

Could not find nokogiri-1.8.0 in any of the sources

Please fill out the issue template, you have not provided enough information for me to help you.


Closes #1979 Add a flag_all function argument to enhance derive_var_extreme_flag()

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Could we get a test or two with some dirtier data?  Like a subject missing an AVISITN or missing AVAL?

Should we mention this flag_all in the Details section as well?

Should we mention this flag_all in the Details section as well?

I can add some flavoring


Rely more official Jekyll plugins for common behavior

This pull request is a series of (hopefully) atomic commits, with some targeted suggestions for how the site could rely on some of the official Jekyll Plugins a bit more, but I'm equally as glad to break this up into several PRs, or help cherry-pick as desired.
Looking at the site to debug a GitHub Pages issue, I noticed there were several opportunities to rely on several of the official Jekyll plugins, many of which are simply shared, battle-tested templates, that better account for edge cases and enforce many Jekyll style conventions, with the goal of reducing overall complexity and the maintainability burden (if there's less custom code in the repo, there's less to update and less that can go wrong).
There's more details in each commit, but here's what I'm proposing in this pull request:
Use the GitHub Pages Gem (ad99f86c357b45ceeb2e9eb6e82ab3a0d8c72dc3, e971f9cb)
The GitHub Pages Gem (https://github.com/github/pages-gem) is a meta-gem that does two things:

It locks dependencies to the same version used by GitHub Pages ensuring that when you build the site locally, you're using the same version of plugins and other dependencies used in production.

When loaded as part of the :jekyll_plugins group, it allows the Gem to set certain configuration defaults and overrides (such as activating default plugins) to ensure, once again that your local preview replicates the production version as closely as possible.

Rather than lock each dependency explicitly (which will cause the local and staging preview to differ from production as it does today), let the Pages Gem manage the dependency versions for us.
Use Jekyll Feed to generate the Atom feed (	f08e807)
Jekyll Feed (https://github.com/jekyll/jekyll-feed) is an official Jekyll plugin which should replicate the existing feed nearly identically, but with a shared, battle-tested template that accounts for all sorts of edge cases like relative links in feed entries.
It should be a drop in replacement and "just work" without any additional configuration. That said, it doesn't appear that there are any posts, as the existing feed is blank. It may be best to remove the feed all together?
Use Jekyll Sitemap to generate the sitemap (8f1f8d4, 5de1103)
Jekyll Sitemap (https://github.com/jekyll/jekyll-sitemap) is an official Jekyll plugin, which should serve as a drop-in replacement for the existing sitemap.xml. The resulting sitemap should be largely
similar to the existing sitemap, but with a shared, battle-tested template that accounts for all sorts of edge cases, handles collections and static files, etc.
Here's what the sitemap looks like today:

And here's what it looks like with the plugin:

The lastmod fields are removed, because they're the time the site is last modified, and thus are largely ignored/inaccurate.
Add title and description to _config.yml (cb9c9dc)
Although the title already has a "name" entry, this commit replicates the name as "title", and adds a "description" entry, both of which are shared across common, official Jekyll plugins like jekyll-feed
and jekyll-sitemap.
This allows the feed and sitemap to properly output the title and description in various places (along with Jekyll SEO tag). I grabbed the description from the existing human-readable subhead.
Remove "baseurl" from _config.yml (69a6958)
Remove baseurl from the config for three reasons:

It's not used anyplace within the site

The site’s base URL is only to be used when the resulting site lives at a subpath of the domain. Otherwise, the subpath should be nil, not / (see http://ben.balter.com/jekyll-style-guide/config/#baseurl)

When the Pages Gem is loaded as part of the :jekyll_plugins group, baseurl and url are set automatically.

Use the `absolute_url filter to generate the canonical URL (91634d1)
Starting with Jekyll v3.3, Jekyll ships with relative_url and absolute_url filters. Rather than manual concatenating strings, we can rely on Jekyll to handle the logic in Ruby land, which takes into account the site's URL (locally and in production), and handles things like double "/"s.
While less important for the canonical URL, for other URLs, when previewed locally, the URL will be http://localhost:4000, rather than the live URL, allowing you to use the filter for things like assets, without frustration or local overrides.
Use Jekyll SEO Tag to generate the search engine HEAD meta (731f5ba, e6675060)
This change is the most "aggressive", and I'd be glad to talk through it more. There may be some incompatibilities with the site's custom logic, but at first glance, it appears to work as expected. The change moves the site to use the official Jekyll SEO Tag plugin (https://github.com/jekyll/jekyll-seo-tag) to generate the search-engine meta in the HEAD of each page. The title and canonical URL output should be largely the same as before, but with additional metadata, such as JSON-LD (for richer indexing), Twitter card metadata, etc.
Like the other plugins, this should largely work out-of-the-box, with no additional day-to-day configuration, but is customizable where desired.
Here's the title SEO metadata now for the homepage:
  <title>Kubernetes - Production-Grade Container Orchestration</title>

And here's what it'd look like with the plugin:
	<!-- Begin Jekyll SEO tag v2.1.0 -->
<title>Kubernetes - Production-Grade Container Orchestration</title>
<meta property="og:title" content="Kubernetes" />
<meta name="description" content="Production-Grade Container Orchestration" />
<meta property="og:description" content="Production-Grade Container Orchestration" />
<link rel="canonical" href="http://localhost:4000/" />
<meta property="og:url" content="http://localhost:4000/" />
<meta property="og:site_name" content="Kubernetes" />
<meta name="twitter:card" content="summary" />
<meta name="twitter:site" content="@kubernetesio" />
<script type="application/ld+json">
{"@context": "http://schema.org",
"@type": "WebSite",
"name": "Kubernetes",
"headline": "Kubernetes",
"description": "Production-Grade Container Orchestration",
"publisher": {"@type": "Organization",
"logo": {"@type": "ImageObject",
"url": "http://localhost:4000/images/favicon.png"}},
"url": "http://localhost:4000/"}</script>
<!-- End Jekyll SEO tag -->

There's a lot there, and again, I'm glad to talk through any of the above, or break the commits into individual pull requests, but I believe that relying more heavily on the Jekyll community may help simplify the day-to-day workflow of building and maintaining the site.

This change is

Thanks for these changes. Can you sign the CLA for the linux foundation?

wow thanks so much

wow this looks a lot better, thanks so much, as far as I can tell all the changes look good!
LGTM (after the cla of course)

Can you sign the CLA for the linux foundation?

I believe my colleague @bluemazzoo initiated the process of establishing a corporate CLA on behalf of GitHub this afternoon. GitHub ❤ Kubernetes.

I believe I should now be a member of the CNCF - GitHub organization, which means that there should be a valid CLA on file, under which my contribution is covered.

@jessfraz, https://github.com/kubernetes/kubernetes.github.io/issues/1959


Update OpenEXR.cpp to compile on Windows (ssize_t missing)

ssize_t is only defined in windows as SSIZE_T as it is only in posix.

Hey! Ty!
However, this repo is about to be closed as long as the python wrapper is
getting merged on the upstream
On Fri, 27 Oct 2023, 17:57 Andrew Selle, @.***> wrote:

ssize_t is only defined in windows as SSIZE_T as it is only in posix.
You can view, comment on, or merge this pull request online at:
https://github.com/sanguinariojoe/pip-openexr/pull/37
Commit Summary

8ebef7e
https://github.com/sanguinariojoe/pip-openexr/pull/37/commits/8ebef7eb5049dd458701f964dc2571bcfed8ccf6
Update OpenEXR.cpp to compile on Windows (ssize_t missing)

File Changes
(1 file https://github.com/sanguinariojoe/pip-openexr/pull/37/files)

M OpenEXR.cpp
https://github.com/sanguinariojoe/pip-openexr/pull/37/files#diff-91ce01dbae84d5f9ccc6faea8a3c03577d9f565a8c72cfa83bd15b06117adfb3
(5)

Patch Links:

https://github.com/sanguinariojoe/pip-openexr/pull/37.patch
https://github.com/sanguinariojoe/pip-openexr/pull/37.diff

—
Reply to this email directly, view it on GitHub
https://github.com/sanguinariojoe/pip-openexr/pull/37, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMXKKCE2UUV2NBS4DSYQILYBPKVBAVCNFSM6AAAAAA6TBCVCGVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3DKNZZHA2TONA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Fix the '--no-colors' option

This patch fixes the --no-colors option that currently is a no-op by reordering the --colors and --no-colors options so that program.colors is bound to --no-colors. --colors is already handled separately by inspecting the arguments directly.

shit, thanks!


Image Optimizer Missing

I'm new to Rust and leptos. I was trying to configure leptos-image to my best knowledge. However I get following error.
      Notify watching paths src, public, style
     Serving at http://<IP_ADDRESS>:3000
listening on http://<IP_ADDRESS>:3000
thread '<unnamed>' panicked at /home/s1n7ax/.cargo/registry/src/index.crates.io-6f17d22bba15001f/leptos_image-0.2.0/src/provider.rs:29:18:
Failed to retrieve image cache: ServerError("Image Optimizer Missing.")
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I made following changes to my existing leptos website.
https://github.com/s1n7ax/my-website-ssr/compare/main...1-feat-image-optimizations
It would be nice if you can tell me what I'm doing wrong.

I'm missing .leptos_routes_with_context(&state, routes, state.optimizer.provide_context(), App) when building app in main.


28.9999999999999996%

Description
The zombie round end screen is stupidly accurate. Could round it down to the 3rd place or something
Screenshots

free one line PR by just using Math.Round wherever that gets passed

Should probably use https://docs.microsoft.com/en-us/dotnet/standard/base-types/standard-numeric-format-strings#use-standard-numeric-format-strings i.e. $"{number:0}"

use Math.round or math.floor method to round down or round do nearest number.

Would close but no triage, close pls


palmTree-Daniel

Hello!
I haven't touched programming much since December-ish (I was preoccupied with CAD and other obligations such as AP Exams and Finals), and this project was a great warm-up! It was fun to learn Javascript (a new programming language for me) and pull out my Geometry knowledge that I forgot about (last time I used it was in 8th Grade). It was painful to deal with javascript syntax, but it was much easier the second time (learning Java syntax was torture). Because of that, I felt that I had the motivation to design features that were more aesthetically pleasing. I had a fun time getting re-acquainted with programming with this project!

Daniel

Nice! I think the palm tree is the best part. I renamed your snapshots folder to be lowercase.
Congrats you've earned a Blot, you can order it here.


fix(deps): returned generating spring metadata file #967

 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.


ImportError : No module name mla

When I run nnet_convnet_mnist.py in the IDE , it throw the error.

Thank you @rushter

I could run it after  pip install . .
But how to run it without install it into site-packages

http://stackoverflow.com/questions/11536764/how-to-fix-attempted-relative-import-in-non-package-even-with-init-py

You can copy mla directory into examples.
I don't know what kind of IDE you are using, but PyCharm can automatically detect examples directory.

Hi all,
I am also getting same error as no module named mla. I tried to install it via pip unable to install it.
Traceback (most recent call last):
File "knn.py", line 6, in
from mla.base import BaseEstimator
ModuleNotFoundError: No module named 'mla'
when i tried to install it via pip then i am getting the following error
Could not find a version that satisfies the requirement mla (from versions: )
No matching distribution found for mla

Try copying mla  into the directory of python/ lib folder

There exists a problem about the function ''ax.scatter()" in kmeans.py.
TypeError: scatter() got  multiple values for argument 'c'


Balder/counters 2 sequencenumber

@toregge PR

probably easier to review commits in sequence as the last one is a pure refactoring.


Fix deserialisation of additional_kwargs and tool_call_id

Currently, when deserialising, additional_kwargs are passed to constructor as additionalKwargs

Thank yoU!


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for phenomenal-custard-76e941 processing.

Name
Link

🔨 Latest commit
1fef4eabb3e1d52b4be822b084a805d590b8bc04

🔍 Latest deploy log
https://app.netlify.com/sites/phenomenal-custard-76e941/deploys/62f4746a1ac2f50008324506


Tidy up this here issue tracker

All User Stories from "ideas" repo in "scratch" section of User Stories Trello
All completed issues closed

Duplicate of #262


MAINT apply sorting of imports

As discussed in #2265, this PR removes the temporarily introduced # noqa: I001, fixing the sorting of these imports.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Usage of 'text' annot type

Can you tell me what is expected to put in the Appearance()? I tried to add content = 'test' but kept giving me the 'NoneType' object is not subscriptable error.
from pdf_annotate import PdfAnnotator, Location, Appearance
a = PdfAnnotator('a.pdf')
a.add_annotation(
'text',
Location(x1=50, y1=50, x2=100, y2=100, page=0),
Appearance(),
)
a.write('b.pdf')

I managed to add text amending the ReadMe example like this:
from pdf_annotate import PdfAnnotator, Location, Appearance
a = PdfAnnotator('a.pdf')
a.add_annotation(
    'text',
    Location(x1=50, y1=50, x2=100, y2=100, page=0),
    Appearance(stroke_color=(1, 0, 0), stroke_width=5, content="hello world", fill=(0.705, 0.094, 0.125, 1))
)
a.write('b.pdf')  # or use overwrite=True if you feel

'content' is the text you want to write and 'fill' is the colour. 'Fill' seems to accept RGB(A) normalised decimal colours. I used this colour picker to get the value:
http://doc.instantreality.org/tools/color_calculator/
Thanks to the developers for the nice project - a few more usage examples would be great!

I found the usage inside the test cases. This is the snippet from my script,
            a.add_annotation(
                'text',
                Location(x1=x1+5, y1=y1, x2=x2, y2=y2, page=page),
                Appearance(
                    fill=[0.4, 0, 0],
                    stroke_width=1,
                    font_size=10,
                    content=filename,
                ),
            )


observable:version's definition does not cover class definitions

Six classes in observable.ttl include observable:version in their restriction listings:

:Application
:ArchiveFile
:OperatingSystem
:PDFFile
:Software
:X509Certificate

However, the definition of :version only has :Application and :PDFFile in its domain.
Maybe it would be beneficial not declare a rdfs:domain of version?

This issue should end up resolved once we can decide on a solution for issue #104

This is a topic that I have been talking about for a while. Not specifically these six classes, but a broader discussion about how to specify the rdfs:domain.
Setting a domain does not restrict another class from using the predicate. It just means a Description Logic Reasoner will not type the Subject node as an Application and PDFFile (in this case).
And as Sean notes: we are working and discussing this issue.


Copy directory doesn't work with 'ImageFromDockerfile.withFileFromPath(...)'

In my Dockerfile, I have something similar to:
COPY ./folder /path/to, where folder is a directory.
When I try to start a generic container (using ImageFromDockerfile), I get an exception in ImageFromDockerfile:
Caused by: java.lang.RuntimeException: Can't transfer file ...
  at org.testcontainers.images.builder.traits.FilesTrait$1.transferTo(FilesTrait.java:43)
  ...
Caused by: java.io.IOException: Is a directory

Ouch - sorry. Let's try and solve this with #279, which gives us a good chance to clean up file/resource handling for good.

Hi @tourea
Sorry it's taken a while to give you an update on this - it's turned out quite a bit harder than I initially thought. However, I have some draft code on a branch which allows recursive copying of files in.
You can see an example here.
I'll keep working on this and tidying up - if you really want to try it in the meantime you could use jitpack to obtain a branch build (https://jitpack.io/#testcontainers/testcontainers-java/285-recursive-tar-copy-SNAPSHOT should work)
@bsideup FYI - this branch is too messy to deserve review yet. However, you might want to know that (so far) I had to make some fairly big changes to Transferable to allow enough flexibility for recursively populating the context TAR. I will make some further changes to the interface hierarchy, but thought I should give you a heads-up.
Richard

This change is getting closer to being ready; I'd hope to get it released soon. Sorry for the long wait on this.

The 'This change' link in your comment (https://github.com/testcontainers/testcontainers-java/issues/!313) doesn't work for me.  This one does though.

Support for this was added in #313 so I shall close.


CORE-2232

https://liquibase.jira.com/browse/CORE-2232

Thanks!

@nvoxland
It seems that current_schema without () doesn't work for PostgreSQL < 8.4
http://www.postgresql.org/docs/8.3/static/functions-info.html
http://www.postgresql.org/docs/8.4/static/functions-info.html
So Current Liquibase Core Release: 3.4.1 (July 28, 2015) doesn't work for PostgreSQL < 8.4.


Bearer token authentication

Is there a way to authenticate in cmis through Bearer token using this lib?
On the first look I thought that it doesn't implemented yet but maybe I can do it using Guzzle and "Authorization" header? Can somebody comment on this?

That should be possible if supported by guzzle. Just authenticate via
guzzle and than pass this guzzle connection into the lib. See the examples
for details.
2016-10-06 21:44 GMT+02:00 Xunto<EMAIL_ADDRESS>
Is there a way to authenticate in cmis through Bearer token using this lib?
On the first look I thought that it doesn't implemented yet but maybe I
can do it using Guzzle and "Authorization" header? Can somebody comment on
this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dkd/php-cmis-client/issues/29, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABky1jZk7O6f1y3JtnMZocdtx9GZmykdks5qxU-MgaJpZM4KQVqO
.


Manage permissions

Defaults, projects, workspaces, and relevant data sources to list them Docs

@GtheSheep have you started working on it? We need it and we are thinking to implement it. It would be great if you already have some results and we can continue it.

@GtheSheep for now, we need project, workbook, and data source permissions.
I see you did a lot of work so that we can use the existing code as an example for new resources.

We already have a Python script for it, but the issue is revoking when permissions are deleted. I don't want to handle it manually, that's why I want to extend the provider.

Oh nice! Sounds like you must be pretty familiar with the api responses already then, halfway there! 😅

we did that over tableauserverclient not bare api calls
but anyway...

Ahh fair, yeah that project is super useful 🙌


fix tr with delete flag if more than 1 operand given

fixes the issue #5944 where tr -d flag should not take more than one operand
so its like this now
❯ echo "acsd5c"| ./target/debug/tr -d "c"
asd5

~/sad/coreutils main
❯ echo "acsd5c"| ./target/debug/tr -d "c" "p"
./target/debug/tr: extra operand 'p'
Only one string may be given when deleting without squeezing repeats.
Try './target/debug/tr --help' for more information.

problem tho is that adding more arguments errors out tr when it should just display the same massage
where gnu tr
❯ echo "acsd5c"| tr -d "c" "p" "m"
tr: extra operand ‘p’
Try 'tr --help' for more information.

coreutils one
❯ echo "acsd5c"| ./target/debug/tr -d "c" "p" "m"
error: unexpected value 'm' for '[sets]...' found; no more were expected

Usage: ./target/debug/tr [OPTION]... SET1 [SET2]

For more information, try '--help'.

not sure this is related to  #5944 since i could not grep for the error expect in whom test so don't know where to look for that

looks good but could you please add a test to make sure we don't regress in the future ?
thanks

added a test for 2 operands with -d flag

Thanks for your PR :)


Cannot use mismatched architectures in containers

Docker supports running several CPU containers whose architecture does not precisely match the host OS. For example, on an x64 laptop:
$ docker buildx ls
NAME/NODE DRIVER/ENDPOINT STATUS  PLATFORMS
default * docker
  default default         running linux/amd64, linux/386

And an ARM64 server:
$ docker buildx ls
NAME/NODE DRIVER/ENDPOINT STATUS  PLATFORMS
default * docker
  default default         running linux/arm64, linux/arm/v7, linux/arm/v6

The values here can be passed with --platform
e.g.
$ docker run -it ubuntu /bin/bash
root@013fa9713271:/# apt update && apt -y install file
[..]
root@013fa9713271:/# file $0
/bin/bash: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 3.2.0, BuildID[sha1]=12f73d7a8e226c663034529c8dd20efec22dde54, stripped

vs
$ docker run --platform linux/i386 -it i386/ubuntu /bin/bash
root@4a6921d4cb75:/# apt update && apt -y install file
[..]
root@4a6921d4cb75:/# file $0
/bin/bash: ELF 32-bit LSB shared object, Intel 80386, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux.so.2, for GNU/Linux 3.2.0, BuildID[sha1]=d9e0b016588211a89ee9640eb1ed9ad38bb9c2a3, stripped

This is convenient, because it allows for a single pool of multi-arch-capable systems to execute docker images from more than one architecture (i.e. 64-bit processors can run both 64-bit and 32-bit containers).
However, this doesn't work with the Azure DevOps agent, which always assumes an architecture match between the agent itself, and any containers it executes - specifically because it bind-mounts the externals directory and tries to execute its contents. There's no obvious way to get e.g. the 32-bit externals into a 32-bit Docker container running on ARM64, and as such, the container initialization fails (because the 64-bit Node binary lacks any 64-bit system libraries inside the 32-bit container)
This can be worked around by running the 32-bit DevOps agent on the same hardware as the 64-bit agent, and running multiple pools, but this is far less convenient and runs the risk of oversubscribing jobs on the same hardware (since that hardware ends up in multiple pools at once which might all have jobs running on them)

I worked around this using the documentation in https://github.com/Microsoft/azure-pipelines-agent/blob/master/docs/design/non-glibc-containers.md but it would be great if this could be overridden in yaml, rather than needing a whole-ass new Dockerfile

This is still an issue.

Hi @directhex we are working on more prioritized issues at the moment, but will get back to this one once be able to.

This is still an issue.

This is still an issue.

This is still an issue.


Thread 'main' panicking resulting in history file not being properly extracted

After running through the program the history file is not being properly extracted. Attached is the error that I am getting.
Edit to provide more information:
Tried autosave as well as save made during debug mode. Running 1.13.2 unmodded with all dlc except legends of the dead, courture of the capets, and north african atire.

Thank you for letting me know. Is there a way you could provide the save file you used? Without it there's not much I can gather, but based on the crash information I can tell that your save file contains something extraordinary indeed

https://file.io/iDEJsYe70cuU
here are 3 saves. Crashed save is the one I first shared. I decided to try again with 2 other saves.
Crashed save was orginally multiplayer but launched again in single player
Sultan tahir is single player game
and autosave 2 is the same as crashed save but without launching it in debug singleplayer afterwards
Note each save seemed to have similar errors but none of them were at the same point

If you need any additional information just ask. I am figuring this out as I go along. Fine to download software that may help with troubleshooting

I have managed to replicate your issue, though the progress bar didn't match your screenshot. Weird, probably down to thread shenanigans.
I also have managed to fix your issue, your two issues actually. One save file you have provided had a declared player have their id equal -1, didn't know that could happen. Probably down to them dropping out right? So that's Crashed Save.ck3 no longer crashing, at least on my machine. Sultan_Tahir_of_Persia_958_07_12.ck3 was crashing because I had forgotten to adjust how I compile the Windows executable. Worrisome because that implies that all the people who have downloaded the Windows executable since like a version ago have had their instances crashing and did nothing about it. Lovely. The autosave is ironman encoded and there's not much I can do about it. I refer you to the readme for instructions on how to deal with it. Anywho thank you for letting me know about the issue, I will try to release a hotfix by the week's end and once you download that the issue should be resolved. If you don't want to wait you can always try compiling the source code yourself.

Thank you so much. I didn't realise it was ironman encoded to be honest didn't check I was just checking a bunch of saves which I had to make sure it wasn't just that one

The issue is now fixed in release v.2.3.2


AzureFunctionApp@1: Clarify the azureSubscription property

Hi!
The AzureFunctionApp@1  has an azureSubscription property and the comment in the docs says "Required. Azure subscription.".
To me, it looked like I was supposed to add the Azure subscription GUID here and I even tried the name.
What was needed though, was a service connection, which neither the name or comment indicates. 🤔
Can you add something to the comment that indicates this? 🙏🏻

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

ID: 84a23158-e322-4577-5dcc-de0c380dc186
Version Independent ID: 84a23158-e322-4577-5dcc-de0c380dc186
Content: AzureFunctionApp@1 - Azure Functions v1 task
Content Source: task-reference/azure-function-app-v1.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

Thanks for your feedback! I'll update the doc now.


refactor: move ica connection identifiers from port to version metadata bytestring

Description
The following PR refactors the interchain accounts handshake flow to move the connection identifier pair from controller port ID to a JSON encoded metadata type encapsulated within the ICS004 version bytestring.
See: https://github.com/cosmos/ibc/tree/master/spec/core/ics-004-channel-and-packet-semantics#Versioning
Apologies for the large diff as these changes affected a decent chunk of the test suite.
NOTE: Changes will be required to /docs/app_modules/interchain_accounts/ica_auth.md when this is merged. I can make the changes in a follow up PR to not bloat this any further.
closes: #615

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[x] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[x] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #700 (873d8cd) into main (087bc5d) will increase coverage by 0.03%.
The diff coverage is 89.15%.

@@            Coverage Diff             @@
##             main     #700      +/-   ##
==========================================
+ Coverage   79.62%   79.65%   +0.03%
==========================================
  Files         145      144       -1
  Lines       10656    10613      -43
==========================================
- Hits         8485     8454      -31
+ Misses       1755     1747       -8
+ Partials      416      412       -4

Impacted Files
Coverage Δ

modules/apps/27-interchain-accounts/types/keys.go
100.00% <ø> (ø)

...7-interchain-accounts/controller/keeper/account.go
75.00% <64.70%> (-11.37%)
:arrow_down:

...ps/27-interchain-accounts/host/keeper/handshake.go
95.23% <88.46%> (+9.75%)
:arrow_up:

...interchain-accounts/controller/keeper/handshake.go
100.00% <100.00%> (+17.64%)
:arrow_up:

...dules/apps/27-interchain-accounts/types/account.go
78.87% <100.00%> (+3.06%)
:arrow_up:

modules/apps/27-interchain-accounts/types/port.go
100.00% <100.00%> (ø)

I think we can merge (this is blocking future work). Aditya gave a general ACK.
With regards to his main requested change. Since relayers can only initialize ChanOpenInit after an active channel has been closed, this is an unnecessary improvement at this point in time. Furthermore, the handshake improvements linked have not been implemented for ChanOpenInit in ibc-go. We can address the improvements for allowing empty version on ChanOpenInit for all applications when this issue is addressed in #722


generate quark files from protobuf - protoc plugin request

As far as I can tell, quark files are at a minimum, a superset of protobuf proto file functionality. If, for example, I have a gRPC microservice, the endpoints and messages are defined in .proto files. From these, thus I can generate my language stubs, plus other things like a REST proxy, swagger json, or if I wanted to leverage datawire-connect, I could generate quark definitions as well.
https://developers.google.com/protocol-buffers/docs/reference/other
https://github.com/google/protobuf/blob/master/docs/third_party.md

Thanks for the request. Yes, it's definitely possible, and we've discussed providing importers for Swagger, Protobuf, et al. If you have a specific test case or example, we'd love to get some details!
(We view Quark as a way to decouple the details of your network protocol & data contract from your code. Quark today supports HTTP and Websockets as transport, but it could just as easily use gRPC.)


too many notifications when pylance crashes

also, pylance is constantly crashing

Can you upload the log from the Python Language Server tab? It should show the callstack for the crash.

2024-06-25 10:13:15.764 [info] (Client) Pylance async client (2024.6.1) started with python extension (2024.9.2024062506)
2024-06-25 10:13:17.622 [info] c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\pyright.bundle.js:1

{the entire contents of that bundle? (2.6m characters)}

TypeError: Cannot read properties of undefined (reading 'split')
    at x (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\pyright.bundle.js:1:1381535)
    at _0x987c9 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:569922)
    at _0x3bce49 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:569261)
    at e.autoImportDetail (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\pyright.bundle.js:1:1438456)
    at 94576 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\pyright.bundle.js:1:1288606)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 87472 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\pyright.bundle.js:1:1265878)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 68968 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:915625)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 71880 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:956284)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 3740 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:418670)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 78268 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:3246)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 80332 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:582858)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at 47100 (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1096126)
    at _0x48e40e (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1121829)
    at c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1122070
    at _0x48e40e.O (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1123085)
    at _0x48e40e.x (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1122120)
    at _0x48e40e.x (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1125636)
    at c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1125684
    at Object.<anonymous> (c:\Users\aamunger\.vscode-insiders\extensions\ms-python.vscode-pylance-2024.6.1\dist\server.bundle.js:1:1125724)
    at Module._compile (node:internal/modules/cjs/loader:1256:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1311:10)
    at Module.load (node:internal/modules/cjs/loader:1098:32)
    at Module._load (node:internal/modules/cjs/loader:945:12)
    at c._load (node:electron/js2c/node_init:2:13672)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:96:12)

Node.js v20.9.0

2024-06-25 10:13:18.376 [info] [Error - 10:13:18 AM] Server process exited with code 1.
2024-06-25 10:13:18.380 [info] [Error - 10:13:18 AM] Server initialization failed.
2024-06-25 10:13:18.380 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-06-25 10:13:18.381 [info] [Error - 10:13:18 AM] Pylance client: couldn't create connection to server.
2024-06-25 10:13:18.381 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097

Thanks. That's this line of code here:
https://github.com/microsoft/pyright/blob/1168be715e64961106b6cd18c01db87ed285d715/packages/pyright-internal/src/localization/localize.ts#L178
What's your locale set to? I don't know how the locale can be undefined but it looks like that's the problem.

I'm not sure how to check that - I don't have any settings for locale set

Looking up the callstack, I'm guessing it's coming from a VSCODE env var?
https://github.com/microsoft/pyright/blob/1168be715e64961106b6cd18c01db87ed285d715/packages/pyright-internal/src/localization/localize.ts#L121
VSCODE_NLS_CONFIG might be set? And then it just assumes that's a json file and it blindly loads the 'locale' setting in the json file.

This issue has been fixed in prerelease version 2024.7.100, which we've just released. You can find the changelog here: CHANGELOG.md


Deprecate std::error

Summary
Deprecate the category from the error code, and remove std::error module.

Closes #1184

After removing the category, can we consider whether move stdlib, moveos stdlib and rooch framework allocate different error code ranges in a conventional way, so that the use and error location will be more straightforward.

After removing the category, can we consider whether move stdlib, moveos stdlib and rooch framework allocate different error code ranges in a conventional way, so that the use and error location will be more straightforward.

The error code is located via module + error code. It is hard to maintain an error code range for every module. So, keep it simple.


[FEATURE] Revising the Arm32 Targets for CI Build

Is your feature request related to a problem? Please describe.
Over the weekend we had CI Build Issues that took a while to resolve:

https://github.com/apache/nuttx/pull/14229
https://github.com/apache/nuttx/pull/14252
https://github.com/apache/nuttx-apps/pull/2716
https://github.com/apache/nuttx-apps/pull/2720

Our CI Build for Arm32 Targets requires 2.5 hours to run (arm-01 to arm-14). It takes some time to catch all the errors, patch them, re-run the build. And we might hit errors again. Let's improve this.
Describe the solution you'd like
What if we could validate the defconfig files earlier, before any builds? (Suggested by @GUIDINGLI)

Might be possible, I'll check through the CI Build Script

Do we really need to build so many Arm32 Targets every time?

If we build only the latest targets (e.g. SAMV7, Goldfish), then the problems will show up earlier. (And reduce our cost of GitHub Actions)
Or we should move SAMV7, Goldfish etc to the first job arm-01, keep the job small, so that it will fail earlier.

These are the Arm32 Targets that we should keep for the CI Build, suggested by @GUIDINGLI:

armv8-m, armv8-a, armv8-r
armv7-m, armv7-a, armv7-r
armv6-m
For each of them: Keeping two/three boards is enough
For each of them: Build for Flat mode, Kernel mode (if supported), Qemu (if supported)

Here's the complete list of 932 Arm32 Targets (search for arm-01 to arm-13)
Hi @xiaoxiang781216 @acassis @cederom what do you think? Thanks!
Describe alternatives you've considered
No response
Verification

[X] I have verified before submitting the report.

Yes in the light of overloading CI we need optimizations. Also current complexity makes things broken quite often and hard to analyze.
Optimizations should not jeopardize quality of testing - so we still should test all possible configurations (as mentioned above) but maybe on lower count of boards or with smaller tasks that are easier to manage I agree.
We still need to build on Linux, macOS, Windows. There are mostly used, others are not available on GitHub (i.e. BSD).
I would suggest to add details to worker names on what is built inside (i.e. linux-arm-01-armv8m-sim-stm32, linux-xtensa-01-esp32, linux-rv-01-esp32c3, etc). Maybe its more convenient to use single worker per OS like in msys2 right now there is only one worker that makes all of the tasks (decreased setup time per build)? Some top-bottom architecture would be nice to have here :-)
We should really think more about distributed testing platforms that would involve various boards that people have at hand. This would take burden from CI and would reveal other problems not caught by CI. But its a separate complex project I know. And with underlying tools that does not provide self-compatibility (i.e. recent problems with Python and PIP changes) its going to be maintenance nightmare. Welcome to the brave new world of constant change ;-)

Hi @GUIDINGLI: This script will validate all defconfig files for all targets: arm-01 to arm-14, risc-v-01 to risc-v-06, etc. Lemme figure out how to add this to our CI Workflow, meanwhile we can run it manually. Thanks!
https://github.com/lupyuen/nuttx-riscv64/blob/main/check-config.sh


Htmx

Besides handling issue #85, I've added a bootstrap class to to the "copy link" button and replaced  in monitor.html with bootstrap margin classes.


Default import not working?

What I'm trying to do:
I'm taking an exisiting create-react-app project that I've added bsb into and has been working so far. I just run bsb along side the JS server. However I found it hard to get Jest to work properly so I was trying to switch to this. It seemed like it would be a better experience overall.
What is happening
When I try to do a react-scripts start on the project I get this:

Any idea what might be causing this? It seems to complain on default imports everywhere which we use a lot of.

I think I found the source of the problem. It seems in this configuration if a file doesn't have a jsx extension then it defaults to the reason file with the same name. I have a lot of this because I'm converting things to Reason.

This was configured so we can support in-source without the .bs extension 😕 I don’t really see a way to reconcile your problem and supporting that, sorry 😔


French localization ready but building issues

Hi everybody
So i'm not a dev but i wanted to add french localization so that's it, files are ready.
But i can't build Clipy to test the translation (getting errors with AEXML), probably because i run Xcode 9.2 and right now, i can't upgrade to OS 10.13
About the error, i'm getting this one 3 times :
/Clipy/Pods/AEXML/Sources/Element.swift:100:24: Value of type '[AEXMLElement]' has no member 'compactMap'
Questions are : Any chance to build Clipy with Xcode 9.2 ? If not, is it a good idea to trigger a pull request and let someone else build a test version ?
Thank you

@Go-rom Thank you for trying to contribute to localization :+1:
Clipy can only be built with Xcode 9.3 and swift 4.1 or later 😢
If you have prepared the localization files you can attach it to this issue and I can add localization.
Localization target file is the contents of this zip file.
Localizable.zip

Hi @Econa77
here's the files for the french localization.
Don't hesitate to send me a test build in order to check the translation.
Regards.
Localizable-fr.zip

Hello @Econa77
any chance to get the french localization in the next release ?
Thank you !


How to receive and Handle Messages from APN in ios nativescript.

How to retrieve and handle the payload data from the app in background & foreground (APN Message)in ios.In Payload key reference i receive multiple items.How to i check and get the data.Please give some reference.
Please give some reference.
{
"aps" : {
"alert" : {
"body" : "great match!",
"title" : "Portugal vs. Denmark",
},
"badge" : 1,
},
"customKey" : "customValue"
}

@priyangamani use the notification callback for iOS as shown here:
notificationCallbackIOS: (message: any) => {
    console.log("Message : " + JSON.stringify(message));
}

I tried this callback but it's not firing.
On Fri, Apr 27, 2018, 6:57 PM Nick Iliev<EMAIL_ADDRESS>wrote:

@priyangamani https://github.com/priyangamani use the notification
callback for iOS as shown here
https://github.com/NativeScript/push-plugin#using-the-plugin-in-ios:
notificationCallbackIOS: (message: any) => {
console.log("Message : " + JSON.stringify(message));
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NativeScript/push-plugin/issues/216#issuecomment-384937488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFwiNR0fLYtfya7fG_8HBy4mFdNra4Lgks5tsvmygaJpZM4Tp5US
.

@priyangamani are you receiving the notifications? DO you have your APNS certificate set?

yes able to receive the notification .I have the APN cert.
How to handles some keys in the data object like message, title, and uses them to construct a notification entry in the tray on ios?
In android i able to retrive the payload actions but ios cannot.
application.on(application.resumeEvent, function(args) {
if (args.android) {
var act = args.android;
var intent = act.getIntent();
var extras = intent.getExtras();
if (extras) {
// for (var key in extras) {
//     console.log(key + ' -> ' + extras[key]);
// }
var msg = extras.get('someKey');
}
}
if(app.ios){         //Here what i need to do?
}
});

I got the push notifications are working, but I don't know how to extract the response.
So my question is how do I access the data inside of alert for the 'body' and 'title'?

I am interested in the solution for iOS, too.
Here's our current setup in Android:
A device receives a push notification while the app is closed. The push notification contains data that tells the app which page should be opened. After the push notification is tapped, the app is opened, and the resume event of the app is called. The push notification's data is read (using the code that priyangamani listed above) and the correct page is displayed. This is all working as intended.
In iOS, a device receives a push notification while the app is closed. The push notification contains the same data as an Android one (we're using Firebase to send the push notifications) but I don't know how to access that data. If the push notification is tapped, the app is opened and the homepage is displayed, but we'd prefer a different page to be displayed. How can I access the push notification's data so that we can control which page is displayed?
Thanks in advance!

@JDillon08 could you share the code snippet how you implemented Android setup so you can open specific page. Thanks

@bachras Here's the code that we use to navigate to a specific page after resuming the app via tapping n a push notification:
`// App resume event
application.on(application.resumeEvent, function(args) {
if (args.android) {
var act = args.android;
var intent = act.getIntent();
var extras = intent.getExtras();
    if (extras) {
        var data = extras.get('DATA_NAME');

        // Navigate to a specific page based off of data's value
    }
}

});`


Booting time too long on Android (GLES2)

Godot version:
3.1-alpha2
OS/device including version:
Android 7.0, GPU: Mali-T860MP2
Issue description:
The booting time using GLES2 is taking ~15s on Android, while using GLES3 is ~1s.
Steps to reproduce:
Just export and run any GLES2 project on Android.
Minimal reproduction project:
Any GLES2 project.

Try to remove the default environment. Project/Project Settings/Rendering/Environment/Default Environment

Thanks @NewNodeGames, that worked! But why?

Because Godot always load the environment if there is one, even if is a 2D project.


U-38

Fix: Modal bubble edge for display of events
Fix: Modal displays events immediately when modal is shown

Codecov Report
Patch and project coverage have no change.

Comparison is base (7e184d1) 8.90% compared to head (238c010) 8.90%.

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #24   +/-   ##
=====================================
  Coverage   8.90%   8.90%
=====================================
  Files         11      11
  Lines        483     483
=====================================
  Hits          43      43
  Misses       440     440

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

This is part of sprint 2 still. Its just a patch to fix the bugs created by their merges before starting sprint 3


fix(oauth2) Issue with application/json GET requests

Hello Kong maintainers,
Side note -- I've tried to be a good open source citizen, but getting a Kong development environment turned out to be too time consuming for this project, therefore I can't provide tests for this PR, besides my word that it's working for us.
Summary
We've noticed an issue in the OAuth2 plugin that would return a server error on HTTP GET requests of content-type application/json. This commit fixes it.
Full changelog

Fix OAuth2 plugin for GET requests with a content-type of application/json

Issues resolved
GET requests can come through with a content-type of 'application/json',
but will lack a body. The existing code will break on this case, by
trying to parse a body that doesn't exist -- this commit adds the
condition that the body must exist before we try parsing it.

(PS: please let me know if I can help further with getting this PR merged)

We are also getting reports from clients who get 500 response on GET requests. Turns out they were passing the "Content-Type: application/json" header on GET requests.
Although they should not pass this header on GET requests, returning a 500 is nevertheless a bit rude :)

This has been fixed by https://github.com/Mashape/kong/pull/1915
Thanks everybody!


Fix debugShaders output when shaders fail to compile

In ShaderProgram, the createAndLinkProgram function was deleting the shaders before calling WEBGL_debug_shaders.getTranslatedShaderSource(). As a result, the problematic shader code was never output to the console as intended.
This PR reorders the createAndLinkProgram function to complete all logging and debug messages before calling gl.deleteShader.
This addresses the original problem in https://github.com/CesiumGS/cesium/issues/10256, although that issue mentions some ideas for further improvement, which are not implemented here.

Thanks for the pull request @jjhembd!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

@jjhembd If I introduce a syntax error in the shader, am I expecting to see the full shader source in the console log? I just tried introducing a syntax error on main and on this branch, and the output is identical.

Thanks for checking @sanjeetsuhag!
I just confirmed your test. A syntax error will make an individual shader fail to compile, in which case debugShaders.getTranslatedShaderSource returns nothing.
The difference will be when the vertex and fragment shaders both compile, but gl.linkProgram fails. Then debugShaders.getTranslatedShaderSource will return the translated source from the previous compile stages.
I'm not sure yet how to make a simple example of a linker error.

I just pushed another edit:
For the case where the vertex or fragment shader fails to compile, it now logs the raw GLSL instead of attempting to get the (non-existent) translated source.
This should be more useful for finding simple syntax errors in Custom Shaders.

Courtesy of @lilleyse, I was able to trigger a link error by adding a varyings of different types in a shader (I used the SkyAtmosphereVS.glsl and SkyAtmosphereFS.glsl) and reading the value in the fragment shader. And the code output was in HLSL.

Thanks! I tried that and the output makes sense. On my machine (Chrome on Windows) it prints both GLSL and HLSL. Both are output from getTranslatedShaderSource:
[Cesium WebGL] Shader program link log: Types of varying 'v_test' differ between VERTEX and FRAGMENT shaders.
FRAGMENT varying v_test does not match any VERTEX varying

[Cesium WebGL] Translated vertex shaderSource:
// VERTEX SHADER BEGIN

// GLSL BEGIN

#define LOG_DEPTH
#define OES_texture_float_linear

/* ... */

void main()
{
    czm_log_depth_main();
    czm_vertexLogDepth();
}

// GLSL END

// INITIAL HLSL BEGIN

#pragma warning( disable: 3556 3571 )
struct _czm_ray
{
    float3 _origin;
    float3 _direction;
};


issue=1042:map NERDTreeTabsToggle to F1, so default scroll down screen(CTRL-E) can work

issue=1042:map NERDTreeTabsToggle to F1, so default scroll down screen(CTRL-E) can work

 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.chenpeng08 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Revert back to use glide for vendor package update

This PR reverts back to use glide for below consideration:
Device Plugin API change to support NUMA is blocked due to issue, change back to glide allows us to have NUMA support be merged.
go mod support will be added later once we figure out how to resolve kubernetes package dependency with go module.

I downloaded this branch and re-ran "glide update -v".  After that, "make test" worked.

@zshi-redhat Perhaps it's better to do a proper revert? Because even with this new changes(specially it changes the vendor again) the other PRs with vendor modification will have new conflicts. And it won't be easy to resolve those conflicts. Let's wait before we introduce/update or make any more changes in vendor until those PR's are merged?

@ahalim-intel My PR https://github.com/intel/sriov-network-device-plugin/pull/160 is already rebased on this, but with another commit that re-runs "glide update -v" and updates the vendor code.
As we can see here, the checks are failing.  If I check out the code from this PR and run "glide update -v" then "make test" passes.
If you want to do a formal revert, it'd be no problem for me to rebase on top of that.

@ahalim-intel @cbf123 I can do a quick formal revert if you prefer to it.

Updated PR with git revert.


Fix to null/zero byte db handling

Description
Please include a brief summary of the change that this pull request proposes. Include any relevant motivation and context. List any dependencies required for this change.

There was a bit of confusion about how all-zero byte values work in the db: they tell the db to delete the associated key. I've fixed how these zeroes are handled so that if a key does not exist then retrieving such a key returns null always.
This was how the behaviour as originally defined, the DoubleDataWord broke this contract in a few cases, so it's back to as expected.

Fixes Issue # N/A.
Type of change
Insert x into the following checkboxes to confirm (eg. [x]):

[ ] Bug fix.
[ ] New feature.
[x] Enhancement.
[x] Unit test.
[ ] Breaking change (a fix or feature that causes existing functionality to not work as expected).
[ ] Requires documentation update.

Testing
Please describe the tests you used to validate this pull request. Provide any relevant details for test configurations as well as any instructions to reproduce these results.

There are existing unit tests for DataWord and DoubleDataWord, I've added more thorough testing to the db itself. Also tested against our existing test suite.

Verification
Insert x into the following checkboxes to confirm (eg. [x]):

[x] I have self-reviewed my own code and conformed to the style guidelines of this project.
[x] New and existing tests pass locally with my changes.
[x] I have added tests for my fix or feature.
[x] I have made appropriate changes to the corresponding documentation.
[x] My code generates no new warnings.
[x] Any dependent changes have been made.

@AlexandraRoatis any idea about why the ContractDetailed does not allow to put zero value into storage?

Made the requested changes and added some documentation to IContractDetails and its two concrete subclasses. Discovered the two handle missing key-values differently so it is probably worth doing a PR in the future to make them consistent


[LeMondeInformatique] Handle special articles

Fix content extraction for special article compiling previous articles (sample)

Whoops. Fixed!

Merged. Thanks!


Puppeteer assembly doesn't have a strong name

Description
An attempt to use PuppeteerSharp.dll from a DLL which has a strong name causes the following error:
System.IO.FileLoadException: Could not load file or assembly 'PuppeteerSharp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. A strongly-named assembly is required. (Exception from HRESULT: 0x80131044)
File name: 'PuppeteerSharp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
Expected behavior:
I would expect Puppeteer Nuget to work out of the box with any .Net project, signed or not.
Actual behavior:
Currently I have to download the source code and make my custom build of the Puppeteer. I cannot just use the Nuget package maintained by the Puppeteer developers.
Versions
Puppeteer Nuget 2.0.3

I can fix that if it's OK to keep the SNK file in the repository. It seems that falls into Microsoft recommendations: https://docs.microsoft.com/en-us/dotnet/standard/assembly/strong-named

If you are an open-source developer and you want the identity benefits of a strong-named assembly, consider checking in the private key associated with an assembly to your source control system.

If it's not OK, only a person with appropriate rights in this GitHub project will be able to accomplish that task.

@johnnyjob I could take a look at that.
My concern is not breaking existing users and, as you said, being able to build the assembly in our CI.

@johnnyjob I would accept a PR for that.

My understanding is that if we do add a strong name it will be considered a binary breaking change by .NET. So any library that built against 2.0 won't be compatible with the strong-named assembly.
It would be nice not to break with SemVer, so I'd suggest either going to 3.0 when we do this OR releasing a side-by-side PuppeteerSharp.StrongName package built from the same code that people who care about strong names can use until PuppeteerSharp decides to go to 3.0 for other reasons.

I wouldn't bump the version. Matching Puppupeteer version is rule number 1 (or 2).
We could ship a StrongName assembly for these scenarios.

What are your thoughts here @Meir017 ?

I'm ok with making puppeteer-sharp use strong naming. as long as we check-in the strong-name key file

This is the reason why assembly binding to Microsoft.Extensions.Logging <IP_ADDRESS> doesn't work?


[FEAT] Decrease time of recognised state - MQTT

Is your feature request related to a problem? Please describe.
Now, the state of person recognised is assigned to a camera for 10 minutes (600 seconds). This is too long, considering automations, like door opening. It means that automation won't work again unless 10 minutes have passed. So for instance, going just to take bins out, or a corner shop, lock won't open, if I return within 10 minutes (camera recognises me going out and sends mqtt)
The MQTT is:
{
  "name": "michal",
  "object_id": "double_take_michal",
  "icon": "mdi:account",
  "value_template": "{{ value_json.camera }}",
  "state_topic": "homeassistant/sensor/double-take/michal/state",
  "json_attributes_topic": "homeassistant/sensor/double-take/michal/state",
  "availability_topic": "double-take/available",
  "unique_id": "double_take_michal",
  "expire_after": 600
}

Describe the solution you'd like
I would like to have option to decrease "expire_after" to just 120 seconds
Additional context
Add any other context or screenshots about the feature request here.

Implemented in the last beta (skrashevich/double-take:beta)
Please check

How to pull the beta as a HA Addon? I can see only stable release.

Unfortunately, there is no easy way to do it :(
I'll publish new release with this feature soon (after some tests...)

Thanks!
On Fri, 1 Dec 2023, 00:04 Sergey Krashevich, @.***>
wrote:

Unfortunately, there is no easy way to do it :(
I'll publish new release with this feature soon (after some tests...)
—
Reply to this email directly, view it on GitHub
https://github.com/skrashevich/double-take/issues/181#issuecomment-1834960106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMNLFK25WCZVKLJPBMFTPNDYHENKDAVCNFSM6AAAAABABYMMRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZUHE3DAMJQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Any news on the HA Addon with new version please?

Any news on the HA Addon with new version please?

you can use beta add-on from same repository

Beta shows as 1.13.1 from 2022
On Thu, 18 Jan 2024, 19:12 Sergey Krashevich, @.***>
wrote:

Any news on the HA Addon with new version please?
you can use beta add-on from same repository
—
Reply to this email directly, view it on GitHub
https://github.com/skrashevich/double-take/issues/181#issuecomment-1899060680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMNLFK372PUXAHWU2DD2WOLYPFX25AVCNFSM6AAAAABABYMMRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJZGA3DANRYGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

How to set up the mqtt expire in config? Thanks


Map selection behavior should match list selection behavior (ArcGIS Pro)

Current behavior
When you select points in the map, it adds to the highlighted list, instead of highlighting that single coordinate.
Repro steps

Create three points in the map
Use the Select Point  tool to select a point in the map
the point highlights, this is correct
Use the `Select Point' tool to select a different point
there are now two points highlighted in the list. This is not correct.

Both points should only be selected if the user used Ctrl+click when selecting the second point. If the user did just a regular single click on the map, only that point should be highlighted in the list. This is the way the list functions when you click points on the list.

Expected behavior
List to map

[ ] Click a coordinate in the list

[ ] Corresponding point is highlighted on the map

[ ] Click a different point in the list

[ ] New corresponding point is highlighted on the map

[ ] Ctrl+click a point in the list

[ ] Two points are now highlighted in the list and on the map

[ ] Click a highlighted point in the list

[ ] Corresponding point is no longer selected in the map

[ ] Right-click any point in the list

[ ] Clear all selections is added to the context menu and all points in the list and map are unselected

Map to list

[ ] Use the Select Point tool to click a point in the map

[ ] Corresponding point is highlighted in the list

[ ] Click a different point in the map

[ ] New corresponding point is highlighted on the list

[ ] Ctrl+click a point in the map

[ ] Two points are now highlighted in the map and in the list

[ ] Click a highlighted point in the map

[ ] Corresponding point is no longer selected in the list

@dfoll Do we need deselection of points from map(click highlighted point from map) too ?

@saip yes. when a point is highlighted on the map, clicking that point should remove the map selection and the list selection for the same point


Minor UI related issue in Safari on iOS

--
I'm using Safari on iOS 8.1.3

Ya, I saw. There are some minor other problems too, once I've got the css cleaned up, I'll take care of those.

Two other small ones:

iPhone 6, iOS 8.1.3


Update contract address to use the constant we made

Update contract address to use the constant we made in the const folder for easier user experience

New pr #24 takes care of this


make docker command more robust

I was told --ipc=host would unlock shared memory size (I didn't double check). I think it would be fine to leave the top one as original to not complicate it.

LGTM!


[BUG] path.split is not a function when hideExpression specified in repeat field

I'm submitting a ...  (check one with "x")
[X] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
When using hideExpression in FormArray, I got this error:
TypeError: path.split is not a function
    at _find (forms.js:1885)
    at<EMAIL_ADDRESS>(forms.js:2285)
    at<EMAIL_ADDRESS>(ngx-formly-core.js:701)
    at ngx-formly-core.js:625
    at Array.forEach (<anonymous>)
    at<EMAIL_ADDRESS>(ngx-formly-core.js:623)
    at ngx-formly-core.js:627
    at Array.forEach (<anonymous>)
    at<EMAIL_ADDRESS>(ngx-formly-core.js:623)
    at ngx-formly-core.js:627

After debugging, it happen in formly.form.expression.ts line 93 when trying to get a control using the property field._formlyKeyPath.path which contain an array of number but [number].split doesn't exist
Expected behavior
To not have an error ;)
Minimal reproduction of the problem with instructions
Check https://stackblitz.com/edit/angular-uy7bxp?file=src/app/app.component.ts
Angular 6.0.4
Formly 4.4.3

Thx !!!


Update info about ActivityStarter support

Update https://github.com/johncarl81/parceler-site as well?


Connector IP in docker container.

Hi,
I am testing to see if I could wrap up pomelo servers as docker container running in my digital ocean server, and access it via public website. The test  works fine via chat server running out side of container, but once I put them in the container, web page could not find the servers.
Here is what I did with the config/servers.json when 'pomelo start' is running outside of the container...
{
"development": {
"connector": [
{ "id": "connector-server-1", "host": "(VPS public IP)", "port": 4050, "clientPort": 3050, "frontend": true },
{ "id": "connector-server-2", "host": "(VPS public IP)", "port": 4051, "clientPort": 3051, "frontend": true },
{ "id": "connector-server-3", "host": "(VPS public IP)", "port": 4052, "clientPort": 3052, "frontend": true }
],
"chat": [
{ "id": "chat-server-1", "host": "<IP_ADDRESS>", "port": 6050 },
{ "id": "chat-server-2", "host": "<IP_ADDRESS>", "port": 6051 },
{ "id": "chat-server-3", "host": "<IP_ADDRESS>", "port": 6052 }
],
"gate": [
{ "id": "gate-server-1", "host": "(VPS public IP)", "clientPort": 3014, "frontend": true }
]
},
}
This config won't work with container since the public IP is not available inside the container. so what I did is publishing 3050-3052 and 3014 with -p option, and set the connector and gatway IP to the virtual IPs assigned by docker like ( <IP_ADDRESS> ) instead of my digital ocean server public IP. Web page client seems to reach the gateway, but it trys to make persistent websocket connection with the unreachable docker virtual IP. What needs to be done here to make the servers running in the docker container accessible via gate?

Perhaps this will work
docker run --add-host=host:<IP_ADDRESS>(your host ip address) ....
then use "host" in your config file

Thanks. The example IP of <IP_ADDRESS> is the IP of 'eth0' of my container which hosts the game server and the pomelo chat server, so I presume "host" is the public routable IP address of my linux server, the host IP address that runs the docker application.
I  run a heroku-like continue delivery system (dokku) that generates docker image on demand, and I might not be able to determine IP in advance directly and pass it with "add-host" command. I end up just manually test your suggestion by adding a /etc/hosts entry like "<IP_ADDRESS>   X.Y.Z.W(my public IP)", and use X.Y.Z.W in game server config. Unfortunately, it doesn't work. When I use the container IP of <IP_ADDRESS> in game config, at least the console prompt would print something when I initiate contact to the gate server ( in my case when I hit 'join' button of pomelo chat server ).
I've also considered the possibility of creating an additional custom network interface in docker (docker create network ... ) , and map that network interface with the public IP via add-host. What do you think?  The manual modification of etc/host won't work, would creating extra IP help?
Any idea? Could Pomelo frontend work with docker?  In my opinion, connector plays an important role not only for performance but also regarding user experience. It is much easier to manage deployment for admin/op if it could work with docker.
Thank you.


How to set default mode to auto for air purifier?

Hi, first of all, thanks for your plug-in.
I have Mi Air Purifier Pro H model.
and install miot plug-in and set name, ip, token, model, pollingInterval.
buzzer and led control is set false, fan level and mode control is set also false.
after restart my homebridge, everything worked only except one thing.
the one thins is when I turn on air purifier by apple homekit, the air purifier is always set manual mode(favorite mode).
and manual mode(favorite mode)is always full fan speed.
so what I want to do when turning on the air purifier is set auto mode as a default fan mode.
how can I set default air purifier fan mode to auto mo when I turn on it ?

There should be a possibility to set the air purifier mode to auto in the home app, i think this is what you are looking for.


Publish a package to NPM and build a tagged release

Ideally, the following would happen automatically when we push a new tag:

the code is compiled/built (to the library version) using the appropriate webpack config
this makes a versioned release available from github
this release is pushed to NPM.

However, we can do this manually for a version 0.0.1, which we should do pretty much immediately to learn the ropes and figure out what we need to do to automate it. In particular, there's an open question from the MVP (#17) about whether we need to commit folders like demo-build into the repo or not.

Done in #44 - it's not automated yet, but is otherwise complete.


Does the Clash.meta kernel support xtls-rprx-vision sir?

I look forward to xtls-rprx-vision being supported in the next update.

Duplicate #254.


Missing the first letter of folder name in path while running .py file in terminal

VSCode Version: 1.44.0
OS Version: Windows 10 Enterprise

Steps to Reproduce:

Open my virtual env for python programming (
Run the python file in terminal.
And this is it. Sometimes vscode miss a first letter of folder name in path while running
It does not happen everytime. Rarely, it's working fine. The first letter do not  miss and all is going good.

Does this issue occur when all extensions are disabled?: Yes

Please reopen this on the python extension: https://github.com/Microsoft/vscode-python
Thank you!