[5.2] Allow Validator Interaction on ValidatesRequests

Allow interaction with the Validator instance on ValidatesRequests by calling passing a callback as last parameter in validate or validateWithBag.
Example:
/**
 * Store a new game collector.
 *
 * @param  Request  $request
 * @return Response
 */
public function store(Request $request)
{
    $this->validate($request, [
        'email' => 'required|email',
        'games' => 'required|numeric',
    ], [], [], function($validator) {
        $validator->sometimes('reason', 'required|max:500', function($input) {
            return $input->games >= 100;
        });
    });

    // The game collector is valid, store in database...
}

:+1:

To be honest I think this could get a bit messy. The ValidatesRequest shouldn't be responsible for that kind of thing, what about if you wanted to add ->each and stuff like that to the validator? I would much prefer to allow passing of a closure to $validate I don't like the whole inversion of control here.. I would MUCH rather have a closure supplied to the validate method that will be called before validation occurs. That will allow you to interact with the validator object and add sometimes each etc
/**
 * Store a new game collector.
 *
 * @param  Request  $request
 * @return Response
 */
public function store(Request $request)
{
    $this->validate($request, [
        'email' => 'required|email',
        'games' => 'required|numeric',
    ], [], [], function($validator) {
    	$validator->sometimes('reason', 'required|max:500', function($input) {
        	return $input->games >= 100;
        });
    });

    // The game collector is valid, store in database...
}

@garygreen That's an excellent idea!

Might want to squash those commits down as well.

Well, I tried.


Make Scrolling Possible

Hi,
Will be possible to make scrolling also possible?
Thanks!

The idea is to make possible to have both: scrolling and arrows. For the moment I add some css to make .scrtabs-tabs-fixed-container scrollable:
.scrtabs-tabs-fixed-container {
   overflow-x: auto;
   overflow-y: hidden;
   display: -webkit-box;
   display: -moz-box;

   &::-webkit-scrollbar {
      display: none;
   }
}

Maybe add also -webkit-overflow-scrolling: touch;, but is not needed maybe.
The problem are now the arrows, I tried to use the scroll event on the container and run a .scrollingTabs('refresh');. But is not working als expected.
Do you have any idea how can do it better?
Thanks!

v1.1.0 adds the option to allow swiping of the tabs on a touch screen. Details in the README here:
https://github.com/mikejacobson/jquery-bootstrap-scrolling-tabs#allowScrollbar
It's only really practical on WebKit browsers, though, because, to allow swiping, the horizontal scrollbar needs to be enabled, and only WebKit-based browsers allow the scrollbar to be hidden.
Not sure if that helps you or not, but it's available. :)

Thanks! I will try with it.

@mikejacobson you should add this to the next version, would be great.

v2.0.0 implements better swiping functionality. I hope it works for you all. :smile:


Only use wl-copy if WAYLAND_DISPLAY is set

A user may have wl-copy installed, but not currently using Wayland.
This is a tweak of #120, which is now out-of-date.

This should be merged immediately.

I have wl-copy installed but use X in most times. This issue made the plugin totally unusable for me.

This should be merged asap. Who can do that?

I had to debug today why tmux-yank did not work anymore. Could this PR please be merged?


[WIP] Smokescreen Prototype: React, Chakra, Plugin Setup Commands

Closes: #2894
What's Changed

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Completed as prototyped.


what is the valor of this variable CONFIG_SERVER_URL

please a need some help for this .

Hi @Spiritbe4r. The answer depends the location you deploy the configuration server.
If you run the app on your local desktop, you can reach it at the URL http://localhost:8888/

Please reopen if needed

For question, we could now use the Discussions tab: https://github.com/spring-petclinic/spring-petclinic-microservices/discussions


api_call("users.list") doesn't return custom fields in profile

Github username in response for api_call("users.list")
when doing the request for slack_client.api_call("users.list") it's not returning the github username,
{
    "ok": true,
    "user": {
        "id": "W012A3CDE",
        "team_id": "T012AB3C4",
        "name": "spengler",
        "deleted": false,
        "color": "9f69e7",
        "real_name": "Egon Spengler",
        "tz": "America/New_York",
        "tz_label": "Eastern Daylight Time",
        "tz_offset": -14400,
        "profile": {
            "title": "",
            "phone": "",
            "skype": "",
            "real_name": "Egon Spengler",
            "real_name_normalized": "Egon Spengler",
            "display_name": "spengler",
            "display_name_normalized": "spengler",
            "status_text": "Print is dead",
            "status_emoji": ":books:",
            "status_expiration":<PHONE_NUMBER>,
            "avatar_hash": "ge3b51ca72de",
            "first_name": "Matthew",
            "last_name": "Johnston",
            "email"<EMAIL_ADDRESS>            "image_original": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "image_24": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "image_32": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "image_48": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "image_72": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "image_192": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "image_512": "https://.../avatar/e3b51ca72dee4ef87916ae2b9240df50.jpg",
            "team": "T012AB3C4"
        },
        "is_admin": true,
        "is_owner": false,
        "is_primary_owner": false,
        "is_restricted": false,
        "is_ultra_restricted": false,
        "is_bot": false,
        "is_stranger": false,
        "updated":<PHONE_NUMBER>,
        "is_app_user": false,
        "is_invited_user": false,
        "has_2fa": false,
        "locale": "en-US"
    }
}

should also return something like
"github_uesrname"= "username"

@ymahawarcldcvr
I do understand this is not so convenient but custom fields are not included in users.info/users.list responses.
Please use users.profile.get API method for fetching custom fields. Probably, having include_labels=1 in its arguments makes handling the response easier. Refer to https://api.slack.com/methods/users.profile.get for details.

Let us know if you need more help here. Please allow me to close this issue after waiting for your response for a few more days.

Please allow me to close this issue now. Feel free to write in if you have further comments.


Add more details in Log4j default settings

What changes are proposed in this pull request?
Add filename:linenumber:threadid details into default log4j settings so logs could be more informative when doing debugging.
Why are the changes needed?
During debugging, it is hard to trace events for one particular file/event/thread with current default log4j settings
Does this PR introduce any user facing changes?
N/A

@lucyge2022 Do we know if this adds overhead to the logging process? i am assuming not much, because line number and file names are static and thread id is a threadlocal var call.

@lucyge2022 Do we know if this adds overhead to the logging process? i am assuming not much, because line number and file names are static and thread id is a threadlocal var call.

I think this is nothing compared to with problems caused by RollingFileAppender which internally have lock contention when dealing with large concurrency:
https://stackoverflow.com/questions/50747813/log4j-rollingfileappender-performance
AsyncAppender with lock-free settings is more recommended nowadays : https://logging.apache.org/log4j/2.x/manual/async.html
https://logging.apache.org/log4j/2.x/performance.html#logging-library-performance-comparison
If we haven't seen any log panicking similar to this, where all threads are stuck at log4j trying to compete for locks, this pattern change should be fine, and we gain more debugging benefits than overhead
https://issues.apache.org/jira/browse/LOG4J2-953

alluxio-bot, merge this please


[Question] Local authentication mixed with user registration?

Describe the issue
These questions have to do with local user registration from caddy auth portal.

can I use the existing "user_db.json" as the starting file for the "registration_db.json" so that we keep the superuser and registered users for authentication in the same file?  UPDATE: YES

I'm assuming we would edit the javascript and template to allow for a "role" to be defined in the user registration screen as well.

has anyone worked out a "change password on first login" flow for the local auth...

Update - cleaned up the configuration file and included what we are attempting to use for registration, below
Configuration
{
        storage file_system {
                root /opt/caddy/storage
        }
        local_certs
        http_port 80
        https_port 443
}
https://example.com, https://<IP_ADDRESS> {
        log {
                level INFO
                format console
                output file /var/log/caddy/caddy.log
        }
        route /auth* {
                auth_portal {
                        path /auth
                        backends {
                                local_backend {
                                        method local
                                        path /opt/caddy/assets/conf/local/auth/user_db.json
                                        realm local
                                }
                        }
                        jwt {
                                token_name access_token
                                token_secret AnExampleSecretString123
                        }
                        ui {
                                theme basic
                                links {
                                        "Elite Manager" /
                                        "Auth Portal Settings" /auth/settings
                                        "who am i check" /auth/whoami
                                        "Add MFA Authentication App" /auth/settings/mfa/add/app
                                }
                        }
                        registration {
                                dropbox /opt/caddy/assets/conf/local/auth/user_db.json
                                title "User Registration"
                        }
                }
        }
        route /gr/* {
                jwt {
                        enable claim headers
                }
                reverse_proxy http://localhost:3000
        }
        route /version* {
                respond * "2.0.0-a" 200
        }
        route /ui/* {
                jwt {
                        enable claim headers
                }
                reverse_proxy http://localhost:1880
        }
        route /* {
                jwt {
                        primary yes
                        trusted_tokens {
                                static_secret {
                                        token_name access_token
                                        token_secret AnExampleSecretString123
                                }
                        }
                        enable claim headers
                }
                reverse_proxy http://localhost:1081
        }
}

Version Information
Provide output of caddy list-modules -versions | grep auth below:
caddy list-modules -versions | grep auth
http.authentication.hashes.bcrypt v2.2.2
http.authentication.hashes.scrypt v2.2.2
http.authentication.providers.http_basic v2.2.2
http.authentication.providers.jwt v1.2.1
http.handlers.auth_portal v1.3.3
http.handlers.authentication v2.2.2

Expected behavior
I would expect that the user_db.json and registration_db.json are interchangeable?
Additional context
I'm just checking if anyone has extended the local user authentication in a "register users by a admin" and "users change password on first login" kind of use case.

Also - updated to latest caddy & auth portal last night too:
./linux_arm7_custom version
v2.3.0 h1:fnrqJLa3G5vfxcxmOH/+kJOcunPLhSBnjgIvjXV/QTA=

./caddy_linux_arm7_custom list-modules -versions | grep auth
http.authentication.hashes.bcrypt v2.3.0
http.authentication.hashes.scrypt v2.3.0
http.authentication.providers.http_basic v2.3.0
http.authentication.providers.jwt v1.2.3
http.handlers.auth_portal v1.3.10
http.handlers.authentication v2.3.0

Also - updated to latest caddy & auth portal last night too:
./linux_arm7_custom version
v2.3.0 h1:fnrqJLa3G5vfxcxmOH/+kJOcunPLhSBnjgIvjXV/QTA=

./caddy_linux_arm7_custom list-modules -versions | grep auth
http.authentication.hashes.bcrypt v2.3.0
http.authentication.hashes.scrypt v2.3.0
http.authentication.providers.http_basic v2.3.0
http.authentication.providers.jwt v1.2.3
http.handlers.auth_portal v1.3.10
http.handlers.authentication v2.3.0

One thing I'm not clear on is what URI to use to hit the registration page in the auth portal?  /auth/reg or /auth/register does not work...

One thing I'm not clear on is what URI to use to hit the registration page in the auth portal?  /auth/reg or /auth/register does not work...

Ok so the base URI is /auth/register.  It appeared to properly register my user over the common user_db.json.  It sets the user's role to "registration pending" in that file.
I'm wondering if I can make the /auth/register path a protected path by JWT so that only our "admin" and "superuser" roles can access the registration page.
I'll continue to research the path to customizing this.

Ok so the base URI is /auth/register.  It appeared to properly register my user over the common user_db.json.  It sets the user's role to "registration pending" in that file.
I'm wondering if I can make the /auth/register path a protected path by JWT so that only our "admin" and "superuser" roles can access the registration page.
I'll continue to research the path to customizing this.

@tgelite , my original idea was to have a way to promote registered user to user db. However, I did not get to it yet.

@tgelite , my original idea was to have a way to promote registered user to user db. However, I did not get to it yet.

The two files are not interchangeable. They just use the same format.

The two files are not interchangeable. They just use the same format.

Ah ok, that makes sense, I was wondering if there was some kind of "planned" portion of this to pull from registration and update user db.
I'll take a look at that with our devs as a process we could build for user registration "approval".  We're working in node.js not go unfortunately but if we work something out we'll post to public github.
I think we'll modify from admin register and use the user register with code so password management is user self maintained, and then set the role as we update user_db.json from the registration_db.json.
Thanks for the confirmation and additional detail.

Ah ok, that makes sense, I was wondering if there was some kind of "planned" portion of this to pull from registration and update user db.
I'll take a look at that with our devs as a process we could build for user registration "approval".  We're working in node.js not go unfortunately but if we work something out we'll post to public github.
I think we'll modify from admin register and use the user register with code so password management is user self maintained, and then set the role as we update user_db.json from the registration_db.json.
Thanks for the confirmation and additional detail.

The two files are not interchangeable. They just use the same format.

Hi, just started using your plugins today, they are working very well so far.
What happens if an external program loads these files and does the transfer?
Would caddy-auth-portal pick up the updated file on the next authentication call, or would I need to reload caddy?
Would caddy-auth-portal overwrite user_db.json when restarting caddy?
I ask (instead of just trying it out) because maybe caddy-auth-portal assumes that nobody else is touching those files, and in that case I will go in a different direction.


feat: moved and updated prevOpenings.md

Moved previousOpportunities.md to the outside of Onboarding Docs Folder.
Updated previousOpportunities.md to include recent positions.

I think this link should be updated too alongside with it @LonelyTree
https://github.com/nodejs/mentorship/blob/main/README.md#previous-opportunities


check the config

Summary

优化配置文件里面的配置项检查
优化代码中的异常处理部分，将异常堆栈记录在log中方便定位问题

Codecov Report

Merging #122 into develop will increase coverage by 0.02%.
The diff coverage is 37.77%.

@@              Coverage Diff              @@
##             develop     #122      +/-   ##
=============================================
+ Coverage      45.03%   45.06%   +0.02%
- Complexity      1375     1376       +1
=============================================
  Files            182      182
  Lines          10035    10062      +27
  Branches        1286     1286
=============================================
+ Hits            4519     4534      +15
- Misses          5005     5017      +12
  Partials         511      511

Impacted Files
Coverage Δ
Complexity Δ

...webank/weid/suite/persistence/sql/SqlExecutor.java
1.44% <ø> (+0.01%)
1 <0> (ø)
:arrow_down:

...k/weid/service/impl/CredentialPojoServiceImpl.java
45.14% <0%> (-0.05%)
117 <0> (ø)

...ervice/impl/engine/fiscov2/CptServiceEngineV2.java
55.41% <0%> (-0.25%)
13 <0> (ø)

...m/webank/weid/suite/persistence/sql/SqlDomain.java
53.06% <0%> (ø)
7 <0> (ø)
:arrow_down:

...rvice/impl/engine/fiscov2/WeIdServiceEngineV2.java
74.89% <0%> (-0.96%)
41 <0> (ø)

...m/webank/weid/suite/endpoint/EndpointDataUtil.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...rvice/impl/engine/fiscov1/WeIdServiceEngineV1.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ansportation/json/impl/JsonTransportationImpl.java
75.72% <100%> (+0.48%)
13 <0> (ø)
:arrow_down:

...ortation/qr/impl/QrCodeJsonTransportationImpl.java
92.4% <100%> (+0.19%)
7 <0> (ø)
:arrow_down:

.../com/webank/weid/service/impl/WeIdServiceImpl.java
76.17% <100%> (+0.07%)
62 <0> (ø)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


live data

Hi,
I noticed there are two functions that can possibly provide real-time data.

stock_zh_a_spot
stock_zh_a_tick

I've questions/suggestions regarding both, respectively.

Is it possible to provide a symbol argument to stock_zh_a_spot? eg. stock_zh_a_spot(symbol = '000001').
Does stock_zh_a_tick provide live data during regular trade hours?
I'm interested in symbol = 'sh204007' (GC007, repo rate). Can stock_zh_a_spot fetch this data?

Thanks.

btw, stock_zh_a_tick does not seem to be able to fetch 'sh131802'.
Instead 'sh204007' is available. Thoughts? Thanks.
http://stockhtm.finance.qq.com/sstock/ggcx/131802.shtml

Hello,
I have checked the function 'stock_zh_a_spot', sina website should run forloop to get all page stocks, then you can run select, so whatever, you have to wait for a forloop time.

sh204007 is bond, 'stock_zh_a_spot' is just for stock, akshare will design 'bond_zh_a_spot' in future

sh204007 is bond, 'stock_zh_a_spot' is just for stock, akshare will design 'bond_zh_a_spot' in future

Hi, fortunately sh204007 actually works. I'm wondering if there is a way to get sh131802 to work as well.

Hello,
I have checked the function 'stock_zh_a_spot', sina website should run forloop to get all page stocks, then you can run select, so whatever, you have to wait for a forloop time.

Thanks for the info. It may be a good idea to put this message somewhere.

‘stock_zh_a_tick ’ can only provide data after trading day at16:00

In order to achieve the data, you can use
stock_zh_a_spot_df[stock_zh_a_spot_df["symbol"].str.contains("000001")]
to replace
stock_zh_a_spot(symbol = '000001')

sh204007 is bond, 'stock_zh_a_spot' is just for stock, akshare will design 'bond_zh_a_spot' in future

Hi, fortunately sh204007 actually works for historical data. I'm wondering if it also works for live data.

I will consider a new interface to handle this type historical data

Hello,
I have checked the function 'stock_zh_a_spot', sina website should run forloop to get all page stocks, then you can run select, so whatever, you have to wait for a forloop time.

Thanks for the info. It may be a good idea to put this message somewhere.

I will put the detail in docs and code docstrings

btw, stock_zh_a_tick does not seem to be able to fetch 'sh131802'.
Instead 'sh204007' is available. Thoughts? Thanks.
http://stockhtm.finance.qq.com/sstock/ggcx/131802.shtml

yeah, thanks for your good idea, i will add func named bond_repo_zh_tick to handle it

btw, stock_zh_a_tick does not seem to be able to fetch 'sh131802'.
Instead 'sh204007' is available. Thoughts? Thanks.
http://stockhtm.finance.qq.com/sstock/ggcx/131802.shtml

yeah, thanks for your good idea, i will add func named bond_repo_zh_tick to handle it

Great! Look forward to bond live data and historical tick data.

btw, stock_zh_a_tick does not seem to be able to fetch 'sh131802'.
Instead 'sh204007' is available. Thoughts? Thanks.
http://stockhtm.finance.qq.com/sstock/ggcx/131802.shtml

yeah, thanks for your good idea, i will add func named bond_repo_zh_tick to handle it

Great! Look forward to bond live data and historical tick data.

ok

btw, stock_zh_a_tick does not seem to be able to fetch 'sh131802'.
Instead 'sh204007' is available. Thoughts? Thanks.
http://stockhtm.finance.qq.com/sstock/ggcx/131802.shtml

yeah, thanks for your good idea, i will add func named bond_repo_zh_tick to handle it

Great! Look forward to bond live data and historical tick data.

ok

In fact, since bonds (and futures contracts) can expire, it might be a better idea to only post daily repo rates, reverse-repo rates, deposit rates, etc.

btw, stock_zh_a_tick does not seem to be able to fetch 'sh131802'.
Instead 'sh204007' is available. Thoughts? Thanks.
http://stockhtm.finance.qq.com/sstock/ggcx/131802.shtml

yeah, thanks for your good idea, i will add func named bond_repo_zh_tick to handle it

Great! Look forward to bond live data and historical tick data.

ok

In fact, since bonds (and futures contracts) can expire, it might be a better idea to only post daily repo rates, reverse-repo rates, deposit rates, etc.

what's the meaning of 'only post daily repo rates, reverse-repo rates, deposit rates'

what's the meaning of 'only post daily repo rates, reverse-repo rates, deposit rates'

Sorry please discard my comment. :)

what's the meaning of 'only post daily repo rates, reverse-repo rates, deposit rates'

Sorry please discard my comment. :)

ok

Just want to follow up on interest rates live data fetch. Any recent plan? Thanks.

interest rates live data

do you have any good data-source website for interest rates live data?

do you have any good data-source website for interest rates live data?

I think the sina website that you are currently using is a good data source.
For example,

I believe SHIBOR data are already present in akshare so needless to replicate.
There are additional rates data, like sz131802 (R-014), but it's unclear how to structure the codes.

do you have any good data-source website for interest rates live data?

I think the sina website that you are currently using is a good data source.
http://finance.sina.com.cn/bond/
For example,

I believe SHIBOR data are already present in akshare so needless to replicate.
There are additional rates data, like sz131802 (R-014), but it's unclear how to structure the codes.

In akshare version 0.3.95:
add 沪深债券 and 沪深可转债
https://akshare.readthedocs.io/zh_CN/latest/data/bond/bond.html#id7

In akshare version 0.3.95:
add 沪深债券 and 沪深可转债
https://akshare.readthedocs.io/zh_CN/latest/data/bond/bond.html#id7

Great! Thanks~

I'm afraid there is a need to reopen the ticket.
bond spot data does not contain any interest rate data...
It would be ideal to access the following website to simply provide symbol and extract data.

Otherwise live repo rates are tabulated at http://finance.sina.com.cn/bond/

I'm afraid there is a need to reopen the ticket.
bond spot data does not contain any interest rate data...
It would be ideal to access the following website to simply provide symbol and extract live data.

Otherwise live repo rates are tabulated at http://finance.sina.com.cn/bond/

real tick data should maintains the requests to interface, just like the interface used in futures functions? or what kind of data?

real tick data should maintains the requests to interface, just like the interface used in futures functions? or what kind of data?

The first provided link, http://money.finance.sina.com.cn/bond/quotes/sz131802.html, or the generic http://money.finance.sina.com.cn/bond/quotes/{symbolname}.html, can give users the flexibility to fetch live data in which they are interested. It might be difficult to implement, I guess.
In terms of another live data source for interest rates (repo rates), I've found this one.
https://www.chinabond.com.cn/jsp/include/EJB/realtimeV2_jm_hg.jsp
Again, thanks a lot for providing such a great tool.

closing the ticket for now. maybe useful when there are more use cases from other users.


app template

Let's get the primitive app online. As you have already started working on it following the tutorial I'll just leave this to you.

meteor provides 4 template apps: leaderboard, todo, clock, local market. After we finish the tutorial, we should take a look at them and identify their components our app also needs.

Can you check out the 4 app templates meteor provides and pick one as our starting point? Then we can have some well constructed code base to build upon.

I've been looking at template candidates and I just found a good match: wekan. Its features can be mapped to ours perfectly:
create tasks -> create animals
tasks' labels -> animals' species
tasks change stages -> animals grow
assignees get tasks -> adopters get animals
@ylhou what do you think?

If we use telescope as our template instead of one of our package, how do we pull in telescope's upgrades?

by adding or replacing template content and style, we can separate own code with in-package code
refer to https://telescope.readme.io/docs

I see. That'll be great!


Adds persistent volume claims encryption with LUKS

Adds encryption in StorageClass as a parameter. Encryption passphrase is stored in kubernetes secrets per StorageClass. Implements rbd volume encryption relying on dm-crypt and cryptsetup
using LUKS extension
The change is related to proposal made earlier. This is a first part of the full feature that adds encryption with passphrase stored in secrets.
Signed-off-by: Vasyl Purchel<EMAIL_ADDRESS>Signed-off-by: Andrea Baglioni<EMAIL_ADDRESS>Signed-off-by: Ioannis Papaioannou<EMAIL_ADDRESS>Signed-off-by: Paul Mc Auley<EMAIL_ADDRESS>Signed-off-by: Sergio de Carvalho<EMAIL_ADDRESS>

I would like to propose the following change to the design/implementation here:
Assumptions:

Initial encryption can be performed by the nodeserver rather than the controller server, IOW it is acceptable the the node that is going to mount the encrypted device also performs the initial encryption setup.

Revision:

Perform initial encryption post attachRBDImage called from NodeStageVolume

This needs to be conditional to the rbd image not having been formatted prior to this
Leverage suggestion by @dillaman to stash a rbd image-meta set <key> <value> that denotes initial encryption is done or not

Perform OpenEncryptedVolume subsequently in NodeStageVolume

This is as done in the current version of the code

Why:

Avoid the controller plugin from having to map and encrypt the image
Keep all map related requirements within the scope of the nodeplugin, and leverage initial map that needs to be performed anyway, to further encrypt the image

@ShyamsundarR, we have thought about such approach, but it is not safe enough to encrypt volumes when attaching.
If for any reason volume was provisioned as not encrypted and it got some data on it, the last thing you want is to loose that data. If we encrypt with LUKS a volume on a first attach, it is possible that we will format the disk that was not encrypted previously and loose all the data.
Encrypting when creating an image is much safer approach as you know that you just created this volume (hence in a change I'm about to push to address previous comments we fail if volume was created but not encrypted) and it is safe to assume that there is no data on it yet.

If for any reason volume was provisioned as not encrypted and it got some data on it, the last thing you want is to loose that data. If we encrypt with LUKS a volume on a first attach, it is possible that we will format the disk that was not encrypted previously and loose all the data.

How do you get a volume w/ data on it when it's dynamically provisioned? I don't think the CSI should ever fallback to unencrypted mounting if something fails (when encryption is enabled).

attach process currently configured in such a way that if you requested encrypted volume to be attached, but it is not encrypted - the process fails
as a possible scenario I can think of when volume have data and not encrypted is:

someone upgrades to a new version and sets encrypted: true for existing volume - on next attach process will see volume as not encrypted and try to encrypt it - hence delete all data

even though in documentation I've added that it is not supported, we still do not want to delete customer data for a mistake in configuration

even though in documentation I've added that it is not supported, we still do not want to delete customer data for a mistake in configuration

That's where that metadata property on the image can be used to prevent such an occurrence. If encryption is enabled, the controller could record the state that encryption should be used. Then on attach, if encryption is enabled, you can test the block via blkid. If it shows nothing exists, you can create the LUKS layer. If it shows something exists and it's not LUKS, error out.

ok, I'll check such approach with my team and see if we can implement it

even though in documentation I've added that it is not supported, we still do not want to delete customer data for a mistake in configuration

That's where that metadata property on the image can be used to prevent such an occurrence. If encryption is enabled, the controller could record the state that encryption should be used. Then on attach, if encryption is enabled, you can test the block via blkid. If it shows nothing exists, you can create the LUKS layer. If it shows something exists and it's not LUKS, error out.

Let me combine what I am saying with what @dillaman is here,

Let the provisioning stage add an image meta key csi.encrypted with an initial value of false

This helps distinguish encrypted volumes from non-encrypted volumes, only encrypted volumes will have this key
The VolumeContext will also carry the provisioning parameters, so that can also be used to distinguish between encrypted volumes and others (although the CO may request back the volume context, at which time the CSI driver should be able to regenerate it, the same as before, where the image meta key again can help)

The Staging call can then check both the VolumeContext and the presence of csi.encrypted, to decide to first encrypt, and then flip the csi.encrypted to true and then map and proceed.

This safeguards older volumes from being encrypted (even when the CO admin forces an recreate of the StorageClass (in kube) to add the encryption parameter)
Also provides a means to encrypt it only once before it is used the first time

When a volume is dynamically provisioned, we do have the leverage of taking action during the first time it is Staged (mapped and mounted). We even create a filesystem on rbd images, the first time we map and  mount them, I see this as no different at present.

@ShyamsundarR and @dillaman the proposition to change original workflow that you have asked is implemented, could you please check if it is what you were thinking about?

@humblec @nixpanic @ShyamsundarR @dillaman PTAL

@vasyl-purchel.et al, Thanks for the PR ! overall PR looks good to me. Have few review comments. Also I would like to get this in for release-2.0.0, so flagging this PR accordingly.

@ShyamsundarR regarding:

How are volumes created from Snapshots and other volumes retaining the encryption status? Or is this a future concern? Can we just add a safeguard in the code, if it is a future concern, to return an error if source volume is encrypted?
I don't really know how snapshots work in ceph-csi and how to check this. For now I think it was discussed that this will be a future concern for snapshots. We could add a safeguard in the code though I don't know how I can check that and verify it works (from what I can see snapshot e2e tests are commented out)

@ShyamsundarR regarding:

How are volumes created from Snapshots and other volumes retaining the encryption status? Or is this a future concern? Can we just add a safeguard in the code, if it is a future concern, to return an error if source volume is encrypted?

I don't really know how snapshots work in ceph-csi and how to check this. For now I think it was discussed that this will be a future concern for snapshots. We could add a safeguard in the code though I don't know how I can check that and verify it works (from what I can see snapshot e2e tests are commented out)

Cloned images should carry the encrypted keys, so that these are staged by LUKS. As image-meta maintain copying the image keys across clones, as long as we store the encryption keys on the image (as concluded here) clones will not be a concern.

Cloned images should carry the encrypted keys, so that these are staged by LUKS. As image-meta maintain copying the image keys across clones, as long as we store the encryption keys on the image (as concluded here) clones will not be a concern.

I think for short-term we need to prevent k8s snapshots and clones -- at least if they are not using the same encryption settings + key. We should open a follow-up issue ticket since I don't think that snapshot/clone support has merged yet(?) so that it doesn't get lost.
Longer term, we will also want a design that can support re-encrypting upon snapshot/clone operations with new keys since w/o re-encryption you can start to attack the encryption [1][2].
[1] https://cloud.google.com/compute/docs/disks/customer-supplied-encryption#create_snapshot
[2] https://mbroz.fedorapeople.org/talks/DevConf2016/devconf2016-luks2.pdf

@vasyl-purchel CI is failing can you please fix it.
@ShyamsundarR can you please open a new issue to track the things which need to be taken care of later?

Cloned images should carry the encrypted keys, so that these are staged by LUKS. As image-meta maintain copying the image keys across clones, as long as we store the encryption keys on the image (as concluded here) clones will not be a concern.

I think for short-term we need to prevent k8s snapshots and clones -- at least if they are not using the same encryption settings + passphrase key. We should open a follow-up issue ticket since I don't think that snapshot/clone support has merged yet(?) so that it doesn't get lost.

My understanding is that the encryption parameter added as image-meta on the image will be retained by clones (as a part of rbd copying the keys across when a clone operation is performed). Hence CSI will also treat such images as encrypted (as the image-meta parameter is present), and Stage them appropriately.
The gap would be, if a different storage class is used to create a volume from a snapshot or an existing volume, as the volume content source, at which point the volume_options that gets carried over to the Stage request may not have the encryption parameter.
I opened issue #745 for this purpose, to check and ensure we do not end up in any unsupported or unidentifiable situation.

Thanks all !!


Fix link to connect API

Link to connect.md works in preview but is gives 404 for website Update internal link from ../api/connect.md to ../api/connect to fix.

Hm, maybe this is not the right fix. Will have another look.

Intra-page links should always point to the Markdown file name, not the final URL.
If the link is 404-ing, then there's something else that needs to be done to make the link correct.

Actually, this may be due to a bug in Docusaurus. We had to do the same sort of thing in dd17c64791baaebb1b50910f2febf3271416ac22. Can we see about upgrading to 2.0?

FWIW I'm noticing that the Netlify deploy preview isn't updating in another pull request: #1543

Yes, I found that as well. Didn't have much luck with the Docusaurus previews unfortunately.

Closing as this is not the right fix for the problem.


Fixes: Allow saving SP page with an existing collapsible section. Closes #6462

Closes #6462
Changes in page clientsidewebpart add and page text add commands to handle existing collapsible sections when saving an SP page.
In this PR, I have also updated the page text add test file to align with the latest recommendations.
Just one comment, I believe we also need to add some handlers for the controlType":0,"pageSettingsSlice... element, which is also visible in the page context.

Thanks @mkm17! We'll try to review it ASAP.

Just one comment, I believe we also need to add some handlers for the controlType":0,"pageSettingsSlice... element, which is also visible in the page context.

Could you clarify this a bit more, please?

hi @milanholemans, I noticed that on a page, there is an additional element with controlType: 0 and some page settings. In the page text add command, during the changes to the command, this setting is omitted (during updates in clientsidepage.ts and Page.ts). In the page clientsidewebpart add command, as far as I remember, it is handled better, but the method used to add new parts to the page is implemented differently.
To be honest, I am not sure what we can do with it. I noticed that the clientsidepage.ts and Page.ts are also used in different commands. We could use a similar approach to page clientsidewebpart add and implement it in the page text add and other commands, or simply update page text add (and possibly other commands) to handle controlType:0.

To be honest, I have no idea why we have Page.ts and clientsidepage.ts. It feels like it should be one util file.
Do you have more info about this @pnp/cli-for-microsoft-365-maintainers?


NLIC-2342 log4j upgrade

Updated version of the Labs64/NetLicensingClient-java yet to be released.


spring-cloud-starter-parent Angel.SR5 incorrect dependency

The recently released spring-cloud-starter-parent Angel.SR5, tries to include org.springframework.cloud:spring-cloud-netflix:pom:1.0.5.RELEASE which was never released to maven central.
I guess this is suppose to be 1.0.6.RELEASE

Fixed in Angel.SR6. Thanks.


Google Drive refusing to connect

When converting, the link to google drive may be messed up when using multiple google accounts. It defaults to my Chrome synced account when I am using my school email.

This is a known issue that Google has not addressed: see https://github.com/evbacher/gd2md-html/wiki#drivegooglecom-refused-to-connect for more detail.


docs: add link to invitation issue

think it would be a good thing to add a link to the "Invite me to the Github Organization" issue

Is it called Organization or Organisation
Our Action uses the option with s
see here

GitHub with z
see here

I've found an article about this

The only difference is that organization is the sole spelling used in American English, while both terms are common in British English.

see here

Both are correct:
's' is the correct letter in the UK and then 'z' in the US.
Let's use organisation for Eddie :).

Let's use organisation for Eddie :).

👍🏼


use current_app to get app?

handle(data, app):  # current
   pass
handle(data):          # after
   app = current_app

我下午试试


Setup changelog

This PR adds the same changelog flow added to shx.  Also generates the first changelog.

This PR brings up a larger point, v1.0.0.  Technically, when used in production you are at 1.0.  I think we're there and should release a 1.0 as the next release.
The biggest advantage here is that breaking changes are properly noted by the major version number.  I'm updating some projects with shelljs right now and I'm not sure what 0.6 to 0.7 may have broke or if it did.
Lastly, we could take a note from facebook/reactjs.  They recently went from v0.14.0 to v15.0.0 on the next release.  This was because reactjs is in production use all over the place, including facebook.

LGTM.
With regards to v1.0.0, semver defines v0 as a beta, so that unstable packages can be developed easily.
I personally think that we need to start progressing to 1.0.0, but I have a few things I want to work out before doing so, I'll create a milestone when I'm off mobile.

Big picture ShellJS discussion in #444.

@nfischer we good on this PR then?  Assuming you'll make script updates on another PR.

LGTM, but why has this not passed CI?

It did pass on my last commit.  But the auto changelog commit [ci skip] seems to put the test in permanent suspense?

If that's the case, we should probably wait to merge until we can figure that out (don't want to accidentally freeze our CI)

I just mean only that commit is not reporting for some reason.  Tests are still running, and passing as seen here: https://travis-ci.org/shelljs/shelljs/builds/130655225
I'll try to remove that commit and re-add another one to see if that works.

If this doesn't work, we can actually remove the [ ci skip ] for our setup.

OK, removed the ci skip so the statuses show through on the PR.  Tests were still running, but they never reported back.  Suppose the extra test for changelog-only commits doesn't hurt.
Just need another sign off now.


⚠ Fortnite has degraded performance

In 11cf25f, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1534 ms

Resolved: Fortnite performance has improved in f72ceef.


Core: Add InputModels, adapt CLI & GUI

Now both CLI and GUI work off the same gamefile "parsers". This probably needs lots of fixes. Check TODOs.

Note: The compile errors when compiling for net472 weren't caused by me.

The compiler errors must have been caused by you, because development compiles fine for net472 right now. It looks like you removed the using LibCpp2IL at the top of the Program file, this is what caused it.

The compiler errors must have been caused by you, because development compiles fine for net472 right now. It looks like you removed the using LibCpp2IL at the top of the Program file, this is what caused it.

 OmniSharp probably marked it as unused and it got removed. I'll probably have to disable some relevant functions in VSCodium later

I'll write a better rewrite later

added fix, if it doesn't work I will cry

it works


:sparkles: feat: Update landing page based on feedback

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #185

PR #186

PR #187

PR #188

PR #189   👈

This comment was auto-generated by Graphite.


Hand models need animations

For OpenXR runtimes other than SteamVR, if player uses controller, hand skeleton data is not provided and current implementation of physical hands breaks.
Without skeleton data, hand model should follow controllers (XRController3D node) and play animations based on received button clicks (ex. grip).
I have uploaded hand animation files extracted from SteamVR here. These need to be properly transformed for hand models used by physical-movement module, but I don't have 3d/animation skills to do it.

Using the valve hands and its animations doesnt make sense, adding on this,
the Hand models i made for the xr tools kit, are made with OpenXR standard, bone naming etc. in mind and they come with a highpoly and lowpoly variant along with 3 different types.
It was a lengthy process that involved many Updates.
They come with a dozen of premade animations and in theory they should work out of the box with your xr kit.
when i get time, will test my theory and report Back, anyways it should be almost a drag n drop

Thanks @DigitalN8m4r3. Adding your hands is next on my roadmap. I believe multiple working models are beneficial, that's why I want to keep valve hands and make them work fully.

hmm, i do understand your motivation... anyways glad that you are considering the xr tools hand models.
Thanks

Unfortunately, @DigitalN8m4r3 's hand models were unsuitable for hand tracking (bone rotations incompatible with OpenXR), so I had to adapt valve hands animations for that. Anyways, hand models are now animated and plugin works with controllers, too.


React Native  component quick fix weird import suggestion

*Templated added by @mjbvz
TypeScript Version: 3.7.0-dev.20190919
Search terms

React
quick fix
add missing import

original issue below

VSCode Version: 1.38.1 and 1.39.0
OS Version: macOS

Very weird issue... In React Native if you write <Text></Text> and you use Quick Fix... on the first <Text> you will get Import default 'React' from "react", but if you use Quick Fix... on the second </Text> (closing) you will get Import 'Text' from module "react-native" (which is correct).
It's hard to describe the issue so these screenshots will explain everything.

I expect to get the same behavior like with </Text> (closing) if I use Quick Fix on the <Text> (opening).
Steps to Reproduce:

yarn add react react-native @types/react @types/react-native
Create tsconfig.json with the content below.
Create new file App.tsx with the content below.
Move cursor to the first <Text> and press CMD+. or use Quick Fix... from Command Palette.

tsconfig.json
{
  "compilerOptions": {
    "jsx": "react-native",
    "lib": ["es6"],
    "esModuleInterop": true,
    "moduleResolution": "node"
  }
}

package.json
{
  "dependencies": {
    "@types/react": "^16.9.2",
    "@types/react-native": "^0.60.15",
    "react": "^16.9.0",
    "react-native": "^0.60.5"
  }
}

App.tsx
import React from 'react';

function App() {
  return <Text>Hello, World!</Text>;
}

I have this bug too!

This bug still exists and it’s kinda weird. Maybe somehow we can help to fix it?

@a-tarasyuk @sandersn TS: 5.6.3 problem still exists unfortunately :(
As you can see if I use Quick Fix it doesn't have Text as an import and it should suggest me it from react-native. I do think it overlaps with Text from lib/lib.dom.d.ts, because if I Go To Definition it opens it.

How I can avoid this, because it is really frustrating since in React Native this is most used component.

I can't remember--does react-native require your tsconfig's "lib" to include "dom"? If not, you could try getting rid of it from your tsconfig.
(A quick look at, say, @types/react-native-md5 on Definitely Typed just has "lib": ["es6"], which suggests that you don't need it either.)

@sandersn Even if I use "lib": ["es6"] it still does that, tried restarting VS Code, same. I do see expo ("extends": "expo/tsconfig.base") contains "lib": ["DOM", "ESNext"], but even if I remove that extends and add "lib": ["ESNext"] issue is still the same Text from lib/lib.dom.d.ts appears as first and using quick fix it doesn't suggest to add Text import from react-native.

@a-tarasyuk @sandersn Seems like you were right, that the problem was with tsconfig...
Seems like problem was not only with lib containing dom since expo decided to include it, but also missing this part: "types": ["react-native"] which solved an issue for me, also this now removes dom suggestions like Text from imports, finally!
Sorry for the chaos 😄


Gas allocation at 0

Processing 3 tier ICO, at least 6 contracts needed to be signed where gas limit was not filled in, was unsure of amount and might be a reason why contracts failed not knowing the amount of gas to send to each one.

We can't reproduce the issue with 3-tier contract.
Could you reproduce it to share the video with us? Or to point us, what parameters of crowdsale you've set?

https://pastebin.com/d0J7dsBe
while running icowizard locally there are some errors, check out the paste bin for some of the information.. I can get the whole thing if needed, but this one was run in a multiplexer so I couldn't scroll.

I did notice however, these contracts are still using "throw" which means they aren't updated.

it's ok. we use proven TokenMarket contracts as a base. We will incrementally update them and include Safemath from OpenZeppelin in the future.

is there a chat channel you reside in? gitter, irc, slack? I am attemmpting another single tier and it came up with a transaction with 36 bytes of data gas limit set at 0 and gas price at 21... i have a screen shot of it but its on my other computer.. lmk

@freshbtcer we have a telegram chat here https://t.me/oraclesnetwork

lol figures i cant get into that right now as my phone broke and ive been locked in this room for a week losing my mind and cant sign up for telegram without sms and i dont feel like searching for a pc version.. im guessing a transaction sending only 36 bytes should only be a little over $1 usd in gas?

while running icowizard locally there are some errors, check out the paste bin for some of the >information.. I can get the whole thing if needed, but this one was run in a multiplexer so I couldn't >scroll.

These warnings don't affect ICO wizard logic. We should ignore them.
@freshbtcer if you have any screenshot or video describing your problem, it'll be very helpful to solve it.

@freshbtcer could you check https://wizard-stage.oracles.org/ ? Are you faced with this bug there?
If it is fixed there, we'll merge it to the main branch.

Anyway, we merged this fix to https://wizard.oracles.org/. We'll appreciate your feedback.

Hey, sorry didn't get back to you sooner. I tried it out on the testnet and all the gas worked out and it deployed, I just didn't get to test it out because got called away to help with the kids and forgot about it. It happened on the main net though, I will try again soon let you know.

We have repaired the life cycle of ICO Wizard in mainnet. Thank you for the report!


Recursion vectorisation and fix 253

Closes #253
Implements the ability to vectorise recursion
Fix some other things pointed out by LGTM

This PR seems to have quite a few changes which are unrelated to recursion vectorisation, and 253.

This PR seems to have quite a few changes which are unrelated to recursion vectorisation, and 253.

because there was an emergency issue with the textbox display on the site that needed an urgent patch. Also, there were some little things pointed out by auto-code-review that needed fixing.


sync back changes in child cluster

What type of PR is this?
add informers for resources on child clusters, requeue Descriptions when resources change.
What this PR does / why we need it:
roll back all changes on resource changes on child cluster.
Which issue(s) this PR fixes:

Fixes #135
Supercede #189
Special notes for your reviewer:

Codecov Report

Merging #194 (8e49b5f) into main (40fe10b) will increase coverage by 0.59%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##            main    #194      +/-   ##
========================================
+ Coverage   7.77%   8.37%   +0.59%
========================================
  Files         29      31       +2
  Lines       3780    3928     +148
========================================
+ Hits         294     329      +35
- Misses      3464    3574     +110
- Partials      22      25       +3

Impacted Files
Coverage Δ

...ernet/clusternet/pkg/hub/apiserver/shadow/utils.go
89.28% <0.00%> (-10.72%)
:arrow_down:

...lusternet/clusternet/pkg/utils/resourceinformer.go
0.00% <0.00%> (ø)

.../clusternet/clusternet/pkg/utils/resourceclient.go
0.00% <0.00%> (ø)

...et/clusternet/pkg/registry/shadow/template/rest.go
2.74% <0.00%> (+2.74%)
:arrow_up:

...ub.com/clusternet/clusternet/pkg/utils/deployer.go
5.15% <0.00%> (+5.15%)
:arrow_up:

Continue to review full report at Codecov.

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


Issue getting capture done (camera write FIFO done flag)

Hello,
For a class project, we're trying to interface with the 2MP Arducam Mini through a Raspberry Pi using bare metal programming. We have working I2C and SPI drivers and are attempting to get any kind of image data from the camera. To guide us, we've heavily examined your code for the Arduino, as we assumed it'd be closer to the bare metal programming we're doing than using your code for the Raspberry Pi with Linux. (ArduCAM_SPI_OV2640_FIFO_UART.ino) Using an Arduino, we modified it to print BMP. It confirms that the capture is done and successfully dumps the values to the serial monitor.
Our problem is that we can't seem to get a capture done from the camera. Here's what we can get:

We can successfully check the SPI communications to the camera by querying the test register 0x00 with any value, and we can read the same value back
We can read the ArduChip version through the 0x40 register and get the version (0x40) back.
We set the mode to MCU to LCD mode by writing 0x00 to 0x02. It's listed as reserved in the manual, but we noticed this was done in your code so we tried it anyway.
We can read the sensor address through i2c successfully
We can successfully write the long list of commands in ov2640_regs.h, and we're writing the correct commands for Bitmap.
We then flush the fifo, clear the fifo, and start a capture. We've confirmed that we're writing the same values as your code (0x84,0x01    0x84,0x01    0x84,0x02)
We then write 0x41, 0x00, and read back the value in this register. In your code, it takes approximately 20-30 reads before capture is complete. Our code never reaches that point. Your code returns numbers like 0x01 until it hits 0x08 or 0x09 which has the done bit set (bit 3), our code returns 0x40, sometimes 0x00 or 0x044

Do you have any insight as to why we can't get a capture done? What do the other bits correspond to in 0x41? We've spent 20+ hours debugging this with a logic analyzer to confirm that our SPI and I2C work, and we're writing the same addresses and values as your code does. We've stepped through your code and made sure to send identical commands. The project is due in a few days so a speedy reply would be appreciated.
As an aside, we notices that flush_fifo() and clear_fifio_flag() are identical functions, is there a reason to send this command twice?

We figured out the issue!
The datasheet here says that the polarity is 0 and the phase is 1. We found experimentally that the phase should be 0, and after this we see that the timing diagram shows that the phase should be 0. Please update this!

@Eric, yes you are right, it is typing mistake. we will update the document soon.

Wrong eric.

Sorry , it should be Eric Fritz.


galaxyRoot checker

Hi,
Thank you so much for this. When I watched your talk I immediately installed it.
I needed to fight a while because my galaxyRoot was said invalid.
Reading the code I found:
https://github.com/galaxyproject/galaxy-language-server/blob/a22b1e79f747fd476587680ba0c74ee92b7d667a/client/src/configuration/galaxyToolWorkspaceConfiguration.ts#L69
Mine was ending by galaxy-dev.
I think it would be great to also consider it as valid.
Thanks

Hey @lldelisle! Thank you so much for trying it and for reporting issues! they are much appreciated.
I will fix it so it is less restrictive and a bit "smarter" :)

Thank you very much you are so reactive...


Battery charge switch doesn't trigger AC charging

Version 1.5.0
Describe the bug
Battery charge switch isn't taking any affect on the givenergy battery.
Battery details: 102 Ah | 5.22 kWh | (1 Pack)
Battery Firmware Version: 3009
Software version: D0.535-A0.535
Inverter model: GIV-AC3.0
Inverter Firmware Version: D0.535-A0.535
Debug log

2023-01-24 20:30:57.169 ERROR (SyncWorker_4) [givenergy_modbus] Modbus Error: [Input/Output] No Response received from the remote unit/Unable to decode response
NoneType: None
2023-01-24 20:30:57.175 ERROR (MainThread) [custom_components.givenergy_local] Function failed Unexpected response from remote end: Modbus Error: [Input/Output] No Response received from the remote unit/Unable to decode response

Possibly two things going on here:

The A/C charging switch alone will not enable charging. The current time must also fall within one of the charge slots. This is just how the inverter behaves, rather than any logic baked in to the integration.
The inverter connections are notoriously unreliable. For this reason, the integration will attempt up to 5 times to apply a new configuration value before giving up with that error message. Unfortunately there's not much we can do about that, but certainly for my usage it's extremely rare for it not to work witin the 5 attempts.

Unfortunately there's not really anything that can be changed within the integration to fix the error you're seeing.

Thanks for the charge slot tip.
Seems if I use the giv mobile app, I can set it to “solar and grid” and set charge times from 00:00 till 23:59 and the I can use the battery charge switch in home assistant!
Will keep monitoring incase it messes anything else up, but seems to work well at the moment.
Many thanks
Colin
Sent from my iPhone

On 25 Jan 2023, at 21:29, cdpuk @.***> wrote:

Possibly two things going on here:
The A/C charging switch alone will not enable charging. The current time must also fall within one of the charge slots. This is just how the inverter behaves, rather than any logic baked in to the integration.
The inverter connections are notoriously unreliable. For this reason, the integration will attempt up to 5 times to apply a new configuration value before giving up with that error message. Unfortunately there's not much we can do about that, but certainly for my usage it's extremely rare for it not to work witin the 5 attempts.
Unfortunately there's not really anything that can be changed within the integration to fix the error you're seeing.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


refactored predict module so it can be made generic easier

Not much, just refactored some of the messages in the prediction module so it's easier to make it generic at a later point. The default is still the 100in10 arena challenge

Don't mind the repeated requests, I'm using the autocompiler to test the build because I can't be bothered getting it all up and running on my end today, I'm just having a bit of fun figuring out how your project is put together.

Alright, that should do it I hope

thx :sun_with_face:


Black image after compression on iOS

Overview
I've seen mentions of this issue earlier and apparently, it has been fixed on v2.0.2.
But even after the update, I am still facing this issue on iOS (16.4.1, iPhone 12 and 13, Safari and Chrome) only (Android, or other desktop browsers are fine).
The issue happens when trying to perform a bulk image upload (Currently using AntDesign - Upload for that).
Single images or a smaller number (up to 10) works fine, but any higher than that will result in black images.
These are the compression options I am using at the moment:
{
  maxSizeMB: 1,
  maxWidthOrHeight: 1500,
  useWebWorker: false, // Tried with both true/false, but the issue persists
}

Can it be related to running the compression simultaneously? Any idea why it works on Android (even on older devices like Pixel 5), but fails on an iPhone 12?

@rafavalerio There is a limit on memory usage in browsers which may be the reason for the black image outputs you are experiencing. I suggest trying to run the compression process sequentially instead of in parallel to determine if the issue persists.
If it works fine, we may need to find a way to restrict concurrent compression in order to prevent this problem from occurring.

Found a similar issue on Ipad pro (images turning all black) see: https://github.com/Donaldcwl/browser-image-compression/issues/222


Add PR auto assigner

This will trigger on pull_request:ready_for_review and will assign to ssec-jhu/teamwhose settings will auto assign 1 member from that team in a round-robin fashion.
To use the team settings otherwise, just manually request a review from ssec-jhu/team. The idea for the committed workflow is for when opening a draft PR and then bumping it to ready for review.
Note: if the PR gets converted back to a draft and then marked again for review the auto assigner will assign a new member so be mindful of this and manually re-assign the original.
However, I've checked the following team setting which should fix this:

Only notify requested team members
If both a team and one or more of its members are requested for review, don't notify the entire team.

See https://github.com/pozil/auto-assign-issue

Whoops, nope - this fills the assignee field and not the review field.


Add installation command for the missing packages in debian

https://github.com/Vitexus/FirefoxNightlyDeb#installation
checking here:
sudo wget -O /etc/apt/trusted.gpg.d/vitexsoftware.gpg http://repo.vitexsoftware.cz/keyring.gpg
wget is not installed by default in debian, and i saw that later in the installation process it need bzip2 package which is as well not installed by default.
with apt-transport-https package mentioned here #12 the commands will be:
sudo apt install wget apt-transport-https bzip2
echo "deb https://repo.vitexsoftware.cz default main" | sudo tee /etc/apt/sources.list.d/vitexsoftware.list
sudo wget -O /etc/apt/trusted.gpg.d/vitexsoftware.gpg https://repo.vitexsoftware.cz/keyring.gpg

Instructions was updarted


simple multicast registry

I don't think this is what we need. We want to optimise mdns. We still want to use a standard to discovery.

Why you want mdns?what benefits do you have with it?With plain multicast we
can avoid double encoding, and have very simple registry that uses all
other core micro stuff
сб, 18 апр. 2020 г., 11:43 Asim Aslam<EMAIL_ADDRESS>
I don't think this is what we need. We want to optimise mdns. We still
want to use a standard to discovery.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/micro/go-micro/pull/1548#issuecomment-615774100, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADVQG3J6ZBBWRPEEAQIY5LRNFRZ5ANCNFSM4MLEFVCQ
.

Because I want to operate on standards as opposed to ad hoc custom things. What I learned from my client/server experience is that inevitably standards win out. And in the case of local discovery mdns is a staple. So I only want to optimise. I also see further long term benefits to it.

ok,


k3d support

Currently k3d is not listed in
https://docs.px.dev/installing-pixie/requirements/
It would be great to support it or just add in the list, thanks!
If I continue installing, it seems installing successfully.
➜ px deploy
Pixie CLI

Running Cluster Checks:
 ✔    Kernel version > 4.14.0
 ✔    Cluster type is supported
 ✔    K8s version > 1.12.0
 ✔    Kubectl > 1.10.0 is present
 ✔    User can create namespace
 ✕    Cluster type is in list of known supported types  ERR: Cluster type is not in list of known supported cluster types. Ple
 ✕    Cluster type is in list of known supported types  ERR: Cluster type is not in list of known supported cluster types. Please see: https://docs.px.dev/installing-pixie/requirements/
Some cluster checks failed. Pixie may not work properly on your cluster. Continue with deploy? (y/n) [y] : y
Installing Vizier version: 0.9.7
Generating YAMLs for Pixie
Deploying Pixie to the following cluster: k3d-west
Is the cluster correct? (y/n) [y] : y
Found 1 nodes
 ✔    Installing OLM CRDs
 ✔    Deploying OLM
 ✔    Deploying Pixie OLM Namespace
 ✔    Installing Vizier CRD
 ✔    Deploying OLM Catalog
 ✔    Deploying OLM Subscription
 ✔    Creating namespace
 ✔    Deploying Vizier
 ✔    Waiting for Cloud Connector to come online
Waiting for Pixie to pass healthcheck
 ✔    Wait for PEMs/Kelvin
 ✔    Wait for healthcheck

==> Next Steps:

Run some scripts using the px cli. For example:
- px script list : to show pre-installed scripts.
- px run px/service_stats : to run service info for sock-shop demo application (service selection coming soon!).

Check out our docs: https://docs.withpixie.ai:443.

Visit : https://work.withpixie.ai:443 to use Pixie's UI.

In Pixel UI, some scripts like px/namespaces work:

Some scripts like px/cluster do not work:

Found more info at https://github.com/pixie-io/pixie/issues/273
and
https://pixie-community.slack.com/archives/CQ63KEVFY/p1630003944005600?thread_ts=1630001589.004300&cid=CQ63KEVFY

Pixie currently requires access to the host kernel for eBPF. Environments where k8s is running run on bare-metal or on a VM (e.g. kvm2) meet this requirement, and so that's why Pixie works.
In environments where there's an extra containerization layer, Pixie can't currently properly punch through to the host. That's why things like minikube --driver=docker or kind are not yet supported.


Add conflicts: true option to mango selectors

This allows for using Mango queries for finding docs with conflicts.
Closes #1101

script to repro:
#!/bin/sh -ex
S=http://a:a@<IP_ADDRESS>:15984
DB=x1101

# delete test db
curl -X DELETE $S/$DB

# create test d
curl -X PUT $S/$DB

# create doc

curl -X PUT $S/$DB/doc -d '{"a":2}'

# create conflict
curl -X PUT $S/$DB/doc?new_edits=false -d '{"a":1,"_rev":"1-23202479633c2b380f79507a776743d5"}'

# validate we have created a conflict:
curl $S/$DB/doc?conflicts=true

# create mango index def
curl -X POST $S/$DB/_index -d '{"index":{"fields": ["_conflicts"]}}' -Hcontent-type:application/json

# query mango index
curl -X POST $S/$DB/_find -d '{"selector": {"_conflicts": { "$exists": true}},"conflicts":true}' -Hcontent-type:application/json

# query mango index without conflicts option
curl -X POST $S/$DB/_find -d '{"selector": {"_conflicts": { "$exists": true}}}' -Hcontent-type:application/json

Successful run:
 ~/t.sh
+ S=http://a:a@<IP_ADDRESS>:15984
+ DB=x1101
+ curl -X DELETE http://a:a@<IP_ADDRESS>:15984/x1101
{"ok":true}
+ curl -X PUT http://a:a@<IP_ADDRESS>:15984/x1101
{"ok":true}
+ curl -X PUT http://a:a@<IP_ADDRESS>:15984/x1101/doc -d '{"a":2}'
{"ok":true,"id":"doc","rev":"1-3975759ccff3842adf690a5c10caee42"}
+ curl -X PUT 'http://a:a@<IP_ADDRESS>:15984/x1101/doc?new_edits=false' -d '{"a":1,"_rev":"1-23202479633c2b380f79507a776743d5"}'
{"ok":true,"id":"doc","rev":"1-23202479633c2b380f79507a776743d5"}
+ curl 'http://a:a@<IP_ADDRESS>:15984/x1101/doc?conflicts=true'
{"_id":"doc","_rev":"1-3975759ccff3842adf690a5c10caee42","a":2,"_conflicts":["1-23202479633c2b380f79507a776743d5"]}
+ curl -X POST http://a:a@<IP_ADDRESS>:15984/x1101/_index -d '{"index":{"fields": ["_conflicts"]}}' -Hcontent-type:application/json
{"result":"created","id":"_design/42ac6b715ea8fb5be44a74e6cf515eaa334238d6","name":"42ac6b715ea8fb5be44a74e6cf515eaa334238d6"}
+ curl -X POST http://a:a@<IP_ADDRESS>:15984/x1101/_find -d '{"selector": {"_conflicts": { "$exists": true}},"conflicts":true}' -Hcontent-type:application/json
{"docs":[
{"_id":"doc","_rev":"1-3975759ccff3842adf690a5c10caee42","a":2,"_conflicts":["1-23202479633c2b380f79507a776743d5"]}
],
"bookmark": "g1AAAABheJzLYWBgYMpgSmHgKy5JLCrJTq2MT8lPzkzJBYozA1kgKQ6YVA5QkBFMgKSVDHWNjI0MjEzMLc2MjZONkowtDNLMLU0NzBPNzc3MTYxTTLOysgCY2ReV"}

+ curl -X POST http://a:a@<IP_ADDRESS>:15984/x1101/_find -d '{"selector": {"_conflicts": { "$exists": true}}}' -Hcontent-type:application/json
{"docs":[
],
"bookmark": "nil"}

test case in the mango test suite please

Would love some help here. I have a hard time grocking that particular flavour of python.

LGTM but I'd want to see a +1 from garren or tonysun or similar :)

Also docs update please :cat:

No, not all all. Exists: false will affect index selection since no field with exists false can be in an index.

From: Joan Touzet<EMAIL_ADDRESS>Sent: Thursday, December 20, 2018 10:53 PM
To: apache/couchdb
Cc: garren smith; Mention
Subject: Re: [apache/couchdb] Add conflicts: true option to mango selectors (#1425)
@wohali commented on this pull request.

In src/mango/test/19-find-conflicts.pyhttps://github.com/apache/couchdb/pull/1425#discussion_r243414057:

+CONFLICT = [

{

   "_id": "doc",

   "_rev": "1-23202479633c2b380f79507a776743d5",

   "a": 1

}
+]

+class ChooseCorrectIndexForDocs(mango.DbPerClass):

def setUp(self):

   self.db.recreate()

   self.db.save_docs(copy.deepcopy(DOC))

   self.db.save_docs_with_conflicts(copy.deepcopy(CONFLICT))

def test_retrieve_conflicts(self):

   self.db.create_index(["_conflicts"])

@garrensmithhttps://github.com/garrensmith sorry to ask this so late, but are you saying that all $exists queries always perform a full DB scan and ignore secondary indexes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/couchdb/pull/1425#discussion_r243414057, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAK9As9Sbonh0IkmlQ1zkwzkP72Oqhj2ks5u6_i_gaJpZM4VGu6Q.


⚠ Bofrost.fr has degraded performance

In ef5a3de, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1834 ms

Resolved: Bofrost.fr performance has improved in d2a7baa after 6 minutes.


Fix addon-prometheus by updating some version numbers and file permissions

Proposed Changes

Bump prometheus to 2.37.0
Downgrade wheels to alpine-3.14
Eliminate /data/prometheus permission issue by adding chmod -R to prometheus.sh script
Correct location of libraries

Related Issues

(Github link to related issues or pull requests)

this fix can be commited to fix the "open /data/prometheus/queries.active: permission denied" error ?

Thanks for the PRs!
However, I've decided to no longer pursue the further development of this add-on. Up until now it has only been in edge, and was never released stable.
Feel free to pick up the sources to publish your own.
../Frenck


Non-Rectangular Rooms

Hello again @gvlfm78 ! I apologize for not simply putting this in the same message as my other ticket, but it's really its own problem entirely. Though, I suppose it's more of a general question.
Due to the shape of my hotel, the rooms are not just simply rectangles. Many of them have a curve or offset piece to them that, if marked as a rectangle, would overlap some of the floor's hallway.
Is there a way to properly make rooms that are non-rectangular, or is that out of the question of this plugin?

Not how the plugin is currently coded, but I will loook into allowing this for the next version(s).

I was looking into this, but I would need to know what kind of non-cuboid regions you had in mind.

Think of the shape of the L piece in Tetris, essentially. The room is shaped like that, with the hallway outside of it.

And what way would you select it using woldedit? (Through the //Sel commands?)

I normally just use //pos1 and //po2

But would you end up with a non cuboid selection? I mean, if you then defined a region using the /rg command with the L selection, would the region become non-cuboid? You can check by right-clicking the ground with a piece of leather)

I hadn't thought of that, actually... To be totally honest, I'm not proficient in WorldEdit, so I didn't actually know you could make non-cuboid region selections.

Well I have added support for hotel & room polygonal regions and it will be coming in the next version of the plugin. For how to select polygonically, I found this tutorial on youtube: (https://www.youtube.com/watch?v=1Ew5lQUi33w) (you don't have to expand vert, you can //expand 5 up and //expand 56 down etc.)


Update networking-partners-msp.md

Updated incorrect URL for Liquid Telecom.

@Prasheen24 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@skishen525
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi,
This is done.
Thank you.
From: Courtney Wales @.>
Sent: Friday, July 1, 2022 4:35 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Prasheen Dehda (Liquid C2) @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update networking-partners-msp.md (PR #95171)
@skishen525https://github.com/skishen525
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/95171#issuecomment-1172414760, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARSXOOI46UGBUFJRWROIEN3VR36YNANCNFSM52L74DMQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>This email disclaimer applies to the original email, all attachments and any subsequent emails sent by Liquid Telecom. This email contains valuable business information that is privileged, confidential and/or otherwise protected from disclosure, intended only for the named person or entity to which it is addressed. If you are not the intended recipient of this email and you received this e-mail in error, any review, use, dissemination, distribution, printing or copying of this e-mail is strictly prohibited and may be unlawful and/or an infringement of copyright. Please notify us immediately of the error and permanently delete the email from your system, retaining no copies in any media. No employee or agent is authorized to conclude any binding agreement on behalf of Liquid Telecom with another party or give any warranty by email without the express written confirmation by an authorized representative or a director of Liquid Telecom. Nothing in this email shall be construed as a legally binding agreement or warranty or an offer to contract. Liquid Telecom will not be responsible for any damages suffered by the recipient as a result of the recipient not taking cognizance of this principle. Liquid Telecom accepts no liability of whatever nature for any loss, liability, damage or expense resulting directly or indirectly from the access of any files which are attached to this message. Any email addressed to Liquid Telecom shall only be deemed to have been received once receipt is confirmed by Liquid Telecom orally or in writing. An automated acknowledgment of receipt will not suffice as proof of receipt by the Liquid Telecom. This email disclaimer shall be governed by the laws of South Africa.

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @skishen525

Invalid command: '#label'. Only Microsoft employees can use this command.

Good day,
Kindly inform when this will be updated?
@.***
From: Prasheen Dehda (Liquid C2)
Sent: Saturday, July 2, 2022 4:06 PM
To: MicrosoftDocs/azure-docs @.>; MicrosoftDocs/azure-docs @.>
Cc: Mention @.***>
Subject: RE: [MicrosoftDocs/azure-docs] Update networking-partners-msp.md (PR #95171)
Hi,
This is done.
Thank you.
From: Courtney Wales<EMAIL_ADDRESS>Sent: Friday, July 1, 2022 4:35 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Prasheen Dehda (Liquid C2<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update networking-partners-msp.md (PR #95171)
@skishen525https://github.com/skishen525
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/95171#issuecomment-1172414760, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARSXOOI46UGBUFJRWROIEN3VR36YNANCNFSM52L74DMQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>This email disclaimer applies to the original email, all attachments and any subsequent emails sent by Liquid Telecom. This email contains valuable business information that is privileged, confidential and/or otherwise protected from disclosure, intended only for the named person or entity to which it is addressed. If you are not the intended recipient of this email and you received this e-mail in error, any review, use, dissemination, distribution, printing or copying of this e-mail is strictly prohibited and may be unlawful and/or an infringement of copyright. Please notify us immediately of the error and permanently delete the email from your system, retaining no copies in any media. No employee or agent is authorized to conclude any binding agreement on behalf of Liquid Telecom with another party or give any warranty by email without the express written confirmation by an authorized representative or a director of Liquid Telecom. Nothing in this email shall be construed as a legally binding agreement or warranty or an offer to contract. Liquid Telecom will not be responsible for any damages suffered by the recipient as a result of the recipient not taking cognizance of this principle. Liquid Telecom accepts no liability of whatever nature for any loss, liability, damage or expense resulting directly or indirectly from the access of any files which are attached to this message. Any email addressed to Liquid Telecom shall only be deemed to have been received once receipt is confirmed by Liquid Telecom orally or in writing. An automated acknowledgment of receipt will not suffice as proof of receipt by the Liquid Telecom. This email disclaimer shall be governed by the laws of South Africa.

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @skishen525

Invalid command: '#label'. Only Microsoft employees can use this command.

Hi,
Thank you for your help thus far.
any idea on when this will be updated?

Hi,
Thank you for your help thus far.
any idea on when this will be updated?

#sign-off

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi,
Kindly update on the below. We have customers still going to this page and clicking on our offer which takes them to the incorrect page.
Your assistance will be greatly appreciated
From: Prasheen Dehda (Liquid C2)
Sent: Saturday, July 2, 2022 4:06 PM
To: MicrosoftDocs/azure-docs @.>; MicrosoftDocs/azure-docs @.>
Cc: Mention @.***>
Subject: RE: [MicrosoftDocs/azure-docs] Update networking-partners-msp.md (PR #95171)
Hi,
This is done.
Thank you.
From: Courtney Wales<EMAIL_ADDRESS>Sent: Friday, July 1, 2022 4:35 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Prasheen Dehda (Liquid C2<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update networking-partners-msp.md (PR #95171)
@skishen525https://github.com/skishen525
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/95171#issuecomment-1172414760, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARSXOOI46UGBUFJRWROIEN3VR36YNANCNFSM52L74DMQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>This email disclaimer applies to the original email, all attachments and any subsequent emails sent by Liquid Telecom. This email contains valuable business information that is privileged, confidential and/or otherwise protected from disclosure, intended only for the named person or entity to which it is addressed. If you are not the intended recipient of this email and you received this e-mail in error, any review, use, dissemination, distribution, printing or copying of this e-mail is strictly prohibited and may be unlawful and/or an infringement of copyright. Please notify us immediately of the error and permanently delete the email from your system, retaining no copies in any media. No employee or agent is authorized to conclude any binding agreement on behalf of Liquid Telecom with another party or give any warranty by email without the express written confirmation by an authorized representative or a director of Liquid Telecom. Nothing in this email shall be construed as a legally binding agreement or warranty or an offer to contract. Liquid Telecom will not be responsible for any damages suffered by the recipient as a result of the recipient not taking cognizance of this principle. Liquid Telecom accepts no liability of whatever nature for any loss, liability, damage or expense resulting directly or indirectly from the access of any files which are attached to this message. Any email addressed to Liquid Telecom shall only be deemed to have been received once receipt is confirmed by Liquid Telecom orally or in writing. An automated acknowledgment of receipt will not suffice as proof of receipt by the Liquid Telecom. This email disclaimer shall be governed by the laws of South Africa.


Added node 20 support

https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

My apoligies in advance if this is an ignorant question but is there any way I can help to get this PR approved? It would help me a lot in my personal projects to get rid of the node16 error that I keep getting when using this action.

My apoligies in advance if this is an ignorant question but is there any way I can help to get this PR approved? It would help me a lot in my personal projects to get rid of the node16 error that I keep getting when using this action.

Looks like maintainer doesn't care about this anymore :(

Oh no, is there anything we can do?

This will be prioritized soon. We need to keep it in parity with azure devops extension.
If this is blocking, you can for now build your private branch and use it in your workflow. It should continue to work as expected.

Thank you so much and sorry for the disturbance, best of luck and thank you so much for all your hard work!

Thank you so much and sorry for the disturbance, best of luck and thank you so much for all your hard work!

No worries at all. We'll get this released on priority.  Thank you for your contribution

If there's anything I can do to help please let me know, I'm relatively new to the whole thing but I'd be happy to help in any capacity I can.

Will this change resolve the issues in Azure Devops pipelines too? That complains about usage of Node 10 :-|
Issue


Texture Changes for The Smoke Particle

The smoke particle's json file is a complete mess(for reference, there are only 12 images, but 52 lines of json code! There should only be 16!). There is a small feature for the smoke where it is slightly orange at the beginning of the explosion, then returns back to it's normal color. This is done via the textures, which makes the json file a mess in the first place. Instead, it should be done via the particle factory(the code). The particle factory can actually adjust the particle's colors without needing to touch the particle's texture at all. This is a benefit because

json file becomes less messy
Textures become less messy
The amount of time it takes for the smoke to return to it's normal color can easily be adjusted

I also think I can pull some inspiration from Terraria's particles with how they disappear. Based off of memory, they either fade away and or shrink in size. This can also be done in Minecraft's particle factories, producing a cleaner particle overall.

I have made an oversight here. Whilst the particle factory can be used to change the particle colors, it actually can't really make them brighter. I would have to do some odd things to actually recreate the effect that is achieved with the current, but janky, textures. Basically, the amount of work that would have to go in isn't really worth the amount of effect that comes out.


Bump Capsule Chart Version to v0.1.3

Description
This pr bumps the capsule char version to v0.1.3 and the capsule go package to v0.1.1
How Has This Been Tested?

[x] locally
[x] existing unit and e2e tests

/test

/test

/merge


Account Screen stuck on refresh

Account screen is stuck on loading.

The account screen is stuck on refresh and not loading.

A very heavy app with lots of F&F and a donation account. When you pull down, the account refresh use to take a long time. Now it does not progress.

To replicate kill app while account refresh is in progress and this will result in the account refresh getting stuck in most cases.

-Verified this on Dev app version 1.3.1(161),it is taking almost 8/9 sec to refresh on account.
-There are 2 F&F and donation account in my wallet also having 4/5 transactions.

-Verified this on Dev app version 1.3.1(161),it is taking almost 8/9 sec to refresh on account.
-There are 2 F&F and donation account in my wallet also having 4/5 transactions.

@cakesoft-swati This issue was to fix an app which gets stuck in refresh mode.
Refresh taking too long is covered by this ticket https://github.com/bithyve/hexa/issues/1944

@cakesoft-swati - please test on Android

Working as expected


Crash with Aether II

The game crashed whilst there was a severe problem during mod loading that has caused the game to fail
Error: net.minecraftforge.fml.common.LoaderException: java.lang.NoClassDefFoundError: net/minecraft/world/chunk/Chunk
Minecraft: 1.12.2
Forge: <IP_ADDRESS>60
RoughlyEnoughIDs: 2.0.8
Mixinbooter: 9.1
Aether II: 0.3.0+build411-universal
logs:
crash-2024-03-06_01.04.48-client.txt
latest.log
debug.log
launcher_log.txt

Looks like this is a compatibility issue related to Phosphor
Its source code is no longer available to the public, and I recommend you turn off support for it in The Aether II's configuration. This problem can be solved

@Robolightning I have submitted a patch, and when this patch is applied, please replace "Phosphor" with "Hesperus". This will fix compatibility with this one

Better yet, use this Aether II fork instead and use the lighting engine of your choosing between Hesperus or Alfheim. Although I think the OP already got the message in CleanroomMC/MixinBooter#73 for this particular issue.

It seems that this issue can be closed directly with the unplanned option

@jchung01 Oh, I have a mod that I want to port to 1.16.5, but I can't port it completely. Every time I try it crashes. Can you help me take a look?


🐞[bug]:  Incorrect Images and Descriptions in Explore Section for Vintage Collection

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In the Explore section of the Vintage Collection, images of unrelated food items and irrelevant text are displayed instead of vintage-themed content. This misalignment is causing confusion and detracts from the website’s purpose, which is to showcase vintage items. The presence of unrelated images and text disrupts the visual consistency and thematic focus of the collection, potentially impacting user engagement and reducing the effectiveness of the Explore section.
Expected behavior
The Explore section should display images and descriptions that are contextually aligned with the vintage collection, maintaining the website's theme and enhancing the user experience.
Add ScreenShots
screen-capture (1).webm
What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@Anjaliavv51 Please review this issue once and would be great if you can assign it to me I wlll fix it

@Anjaliavv51 have been 2 days and igot no response would be great if you can tell me about the issue and even on #756


 Can you check to see if the omics_credentials.sh script is marked as executable?
In the replace_shell branch, this file has the executable bit set in GitHub. The executable permission should be preserved when the file is copied into the container image.

You are right, it's not.  I actually am embedding the dockerfile and shell script from this repo into my own hence the scripts were not set as executable.  That also explains why I had to make them executable


Add ability to export server data from Traffic Portal

I'm submitting a ...

[ ] bug report
[x] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[x] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:
Currently the only way to export server data is to copy/paste it out of the portal or to use the API.  Copy/pasting does not give you the ability to control which fields you are exporting, and using the API requires some basic knowledge on scripting/writing code.  Neither of these options are great, we should provide the ability to let a user export server data.
Expected / new behavior:

User can export server data to csv.  Traffic Portal would allow them to choose servers from a specific CDN, Cache group, etc as well as letting the user choose which fields they want to export.  This would be useful for external groups that use the Traffic Ops data for other purposes (e.g. network monitoring)
Minimal reproduction of the problem with instructions:

Anything else:

Does "server data" include associated Profile/Parameters and/or Delivery Service assignments?

Not as part of this issue, this is more focused on the actual server object

it's easier to just do this globally for all tables fyi.


KAFKA-7051: Improve the efficiency of ReplicaManager

Improve the efficiency of the ReplicaManager when there are many
partitions.
The 'fetchInfo' argument to ReplicaManager#fetchMessages can be an
iterator rather than a sequence.  This avoids the need to materialize
a new collection in some cases.
Reduce the number of times we loop over the read results obtained from
'readFromLog'.
Fix a case where we were performing an O(N^2) operation to join the set
of partitions we wanted info about, and the set of partitions we
obtained information about.

Thanks for the PR. Nice catch regarding the quadratic issue! Not sure about the iterator optimization though. It makes the code more brittle and the benefit seems less clear given the general style of code.

Nice catch!
Regarding the iterator optimization, the added code complexity is worthwhile if it does give us observable performance improvement, e.g. it reduces time for handling FetchRequest by more than 5%. The major overhead for handling FetchRequest should be disk IO or CPU (e.g. for SSL). Not sure if can it can give us observable performance improvement by reducing the amount of in-memory operation by constant factor.

Thanks for looking at the patch.  I agree that we need to be careful not to make the code too complex.
  In a lot of cases, I think it is making our latencies worse by causing unnecessary GC activity.  There has to be a better way than making so many copies.  It's especially bad when the copies can persist for a few seconds -- they may become tenured and trigger a full GC eventually.
I'm going to be doing more benchmarks which may shed more light on this.

@cmcabbe my suggestion is to do the obvious and simple change in this PR. We can include that in 2.0. The rest can considered later.

I think the iterator optimization is justified here. I've just spent the past week profiling a Kafka cluster dropping throughput significantly due to number of partitions + replication and the collectFirst / traversal block was at the top of the cpu usage time by a significant amount.


Make prettier target run in privileged container

What type of PR is this?
/kind ci
What this PR does / why we need it:
This avoids issues when running on systems using SELinux.
Which issue(s) this PR fixes:

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

None

@cri-o/cri-o-maintainers PTAL

/approve
/lgtm


WIP: Turn on d3d binaries - Windows

Changes in appveyor configuration file to be able build d3d10 and d3d11 tests.

I am confused with this change because these tests uses ARCH variable. Now I am setting for both platforms -DARCH=x86_64. However for _win32 platform there is a need to set i686  Do you know how to pass in appveyor.yml different variable for different platform? Maybe it can stay as is now? Maybe these settings in CMakeLists.txt are obsoleted and can be removed?

Finally I've found solution how to set these variables


Add new features to Dan

Some features that would be nice: 8ball, wolfram alpha integration, weather, recipes, github, dice roll, countdown, dictionary definiton, TLD information, hug, url shortener.

Add website resolution templates: (improve youtube), wikipedia, twitter, facebook, minecraftwiki.net

help system that lists plugins available to the user requesting help.

do liek

Another idea: Add command prefix from config to the help lines when requesting help from a command
-Dan- **\**hash <algo> <text> - Hashes <text> with <algo>

8ball - planned
dice roll - planned
url shortener - planned
TLD information - planned
hug - in consideration
wolfram alpha - in consideration
weather - in consideration
recipes - in consideration
countdown - Unfortunately can't do, unless I add pThreads to the requirements

(of course, you can always make your own and open a PR to add it)
I do plan on adding more website-specific searches and title resolutions, specifically for twitter and github. I'll look into adding the others, but they may come at a later date.
@matthijs186 I have two possible ways to go about that. I can assume and automatically prepend the command prefix to every help line that gets sent back, or I can do a search replace for something like {p}.

IMO, the search replace identifier could be nice if it could/would be used in more places than only the help lines.
But yeah, it's an idea. Up to you.

Commands 8ball and dice roll has been added in 4.0.14-dev, as well as help prefixing using {cp}.


About Gridview  question,

Hi, I am making a simple Girdapp using Windows-universal-samplecode(Listview part).
How can I  change  Gridview shape When I changed source data  that relative to Gridview  during Runtime?
Regarding to Sample code, When page create It seems to setup  data to make gridview.
like this
public SimpleGridViewSample()
{
this.InitializeComponent();
ContactsCVS.Source = Contact.GetContactsGrouped(250);
}
When I change to  data , it didn't change Gridvew figure on the page
It seem to need method to update gridview or source for make gridview.
Like this ,
forexample.
private void AppBarPlayButton_Click(object sender, RoutedEventArgs e)
{
ContactsCVS.Source = Contact.GetContactsGrouped(12);
       Do I have to call added methods?
       Example.,
        ContactsCVS.. view( )????
        ContactsCVS...()?????
    }

Thanks You
Derrick

The issue you are running into is not about a broken sample and requires additional debugging to further isolate the problem you are experiencing. You can open a support case with our Developer Support team who can further help with the specific problem you are experiencing. Please use these steps below for further help:

Visit http://aka.ms/storesupport and sign-in with your Microsoft account
Under the “App Development” section choose "Windows 10 universal app development", then choose the appropriate problem type and category to get engaged through a formal support channel.