chore!: fix semantic-release conventionalcommits ! syntax

What does this change?
conventionalcommits syntax should allow a ! postfix to commit types to signal a major release to semantic-release.
At some point the conventionalcommits preset configuration for semantic-release stopped working for ! with types (build, chore, ci, docs, style, refactor, perf, test) and now only work for fix or feat.
A related discussion around ! is here:
https://github.com/semantic-release/semantic-release/issues/2339
The current preset configuration looks correct according to the docs but this post suggested a change to the way the preset is applied to the semantic-release plugins.
Using this handy guide to run semantic-release dry-run locally:
https://blog.elantha.com/semantic-release-local-dry-run/
I can see that the preset change does now pick up a major release:
before:
[semantic-release] [@semantic-release/commit-analyzer] › ℹ  Analyzing commit: docs!: Update node instructions
[semantic-release] [@semantic-release/commit-analyzer] › ℹ  The commit should not trigger a release

after:
[semantic-release] [@semantic-release/commit-analyzer] › ℹ  Analyzing commit: docs!: Update node instructions
[semantic-release] [@semantic-release/commit-analyzer] › ℹ  The release type for the commit is major

I am deliberately using chore! for this PR to test if a major release is performed (we have backed up major releases which have not happened so imo this is okay).

This fix did work so a ! on the commit type will do a major release.
It also looks like its fixed the release notes so all changes are listed:
https://github.com/guardian/commercial/releases


Using in Nuxt 3 project. page_view works, but trackEvent doesn't

I'm using @gtm-support/vue-gtm in Nuxt 3 App
Info

Tool
Version

Plugin
v2.0.0

Vue
v3.2.45

vue-gtm.client.js:
import { createGtm } from '@gtm-support/vue-gtm'

export default defineNuxtPlugin((nuxtApp) => {
  nuxtApp.vueApp.use(createGtm({
    id: 'XXX-XXXXXXX',
    defer: false,
    compatibility: false,
    enabled: true,
    debug: true,
    loadScript: true,
    vueRouter: useRouter(),
    trackOnNextTick: false
  }))
})

Then page_view works. Then I try use trackEvent in the setup:
composables/useGTM.js:
import { useGtm as useVueGtm } from '@gtm-support/vue-gtm'

export default () => {
  const gtm = useVueGtm()
  return gtm;
}

myComponent.vue:
const $gtm = useGTM();
$gtm.trackEvent({
  event: 'add_to_cart',
  category: 'ecommerce',
  action: 'click',
  label: 'Adding item to cart',
  value: 5000,
  noninteraction: false,
})

When I'm adding item to cart nothing's happened in the Googlу Analytics. There is also no new request in the "Network". But in the console I see
[GTM-Support]: Dispatching event {event: 'add_to_cart', category: 'ecommerce', action: 'click', label: 'Adding item to cart', value: 5000}

What am I doing wrong?

Please make sure that you are in browser context and you have correctly enabled it
https://github.com/gtm-support/core/blob/9391b2d691dcff0dd75446c4ddbce3b916e60652/src/gtm-support.ts#L222-L238
You can test/debug/log $gtm.enabled() and $gtm.isInBrowserContext() and see if both returns true
The debugging is performed (by design) even if the trigger gets not executed

@Shinigami92,
console.log:
$gtm.enabled(): true
$gtm.isInBrowserContext(): true
[GTM-Support]: Dispatching event {event: 'add_to_cart', category: 'ecommerce', action: 'click', label: 'Adding item to cart', value: 5000}

Looks like everything is ok but still not working

Okay, sadly in that case I don't know what's going on
Feel free to also log what's happening with window.dataLayer, but I'm done with my knowledge

The datalayer gets new records every time the event is called, as expected

Then the issue is not anymore on the plugin's side, but on something with GTM itself
Check your GTM-ID or anything like that, look into the account
Search or ask on Stack Overflow
...


Add more charts for better understanding for Amazon Reviews Sentiment Analysis

Deep Learning Simplified Repository (Proposing new issue)
issue in Amazon Reviews Sentiment Analysis

Feature Description
Data Analysis/Daily News and Stock Data Analysis/DailyNews.ipynb
Hi, I am Simran Shaikh. I’ve been reviewing the project as part of my contribution to GSSoC Extended 24 and noticed that the code performs by adding more charts for better understanding we can used different library of python like seaborn or matplotlib . By using this library its easy to understand Job Satisfaction Analysis . Also we can include some data cleaning concept for more accurate and efficient data. As i had find an exixting issue there is no more charts .
This would help increase in more visualization . I would like to work on this.
If you like this idea, assign the task to me and add the corresponding charts to it.
Thank you for your time and consideration!
liked my idea assign me a work @abhisheks008

@abhishak3 can you assign me work !

@abhishak3 can you assign me work !

Have liked my idea ?

@abhishak3  can you assign me this work ?

@abhishak3  can you please review mu issue!

@abhishak3 can you please review mu issue!

I think you are tagging a different person instead of me!
Anyways, this work will be an enhancement work of the existing project. Do not remove any files/folders from the existing project folder, add your file(s) only in the designated folder then create a pull request for review.
I hope you have understood my point. Assigning this issue to you @SimranShaikh20
Updated the issue title as per the required naming convention.

@abhishak3 can you please review mu issue!

I think you are tagging a different person instead of me!
Anyways, this work will be an enhancement work of the existing project. Do not remove any files/folders from the existing project folder, add your file(s) only in the designated folder then create a pull request for review.
I hope you have understood my point. Assigning this issue to you @SimranShaikh20
Updated the issue title as per the required naming convention.

Oh sorry my bad typo error

@abhisheks008  can you pls change assigned level !

hey @abhisheks008  can you please upgrade the assigned level !


Contributing & Forking Issue

So... let's say that I wanted to contribute this project and began forking it. Even after following the contributing guidelines, I'm left with an non-working and broken local project. It would be impossible to make the changes you want to make without having to make more unrelated changes that should have already been there.
Things like, the .env file and a guide to getting that to work with the code, so that any and all secrets required by the project is provided without having to change the main codebase just to get it working for you locally.
Although they are free, you'll have to make accounts in order to get those API keys and secrets needed.
I made the necessary fixes for these issues and also updated the CONTRIBUTING.md file for documentation.

Hi @stacknomad ,
Thank you so much for your valuable feedback and for taking the time to work on the necessary fixes for the project. I appreciate your initiative in resolving the issues related to the .env file and improving the contributing guidelines.
As a suggestion, it would be great if you could break down your changes into smaller, focused commits. This will make it easier to review and merge the updates one step at a time, ensuring each set of changes is tested thoroughly. For example:

One commit could address the fixes for .env and related secrets.
Another commit can focus on updating the CONTRIBUTING.md file for documentation.

Once you've made these commits, feel free to open separate pull requests, and I’ll be happy to review them promptly.
Thank you again for your efforts, and I look forward to seeing your contributions!

Hey @stacknomad , are you still working on this issue? If not, please let me know so I can assign it to someone else. Thanks!


Add instance ID to handler callback

Warning: I have not a damn clue what I'm talking about when it comes to node, javascript, or perhaps literally anything.
Is it possible to either attach an opaque callback object or otherwise get some sort of instance ID with the callbacks so that if multiple people use the portal simultaneously, a host application can differentiate between the two users?
Unless that's already available, and I'm an idiot...

@jnsnow technically that can be done based on the remoteAddress, but if a user is behind NAT that wont help. For multi-user setup you'd likely have to spin a separate instance of a process with a port for each given user

Yeah, I'm sure it could be hacked up, but I really assume that the backing libraries here MUST keep some sort of identifying instance ID per-each connection, I just don't know how to get at it because I am the furthest possible thing from a webdev.
My initial thought was to keep track of the user's playername, but that isn't sent with every command, just during GetPlayer.
If I can correlate GetPlayer to some identifying feature (e.g. a Thread ID for the particular socket instance communicating with PoGo) I can correlate all future events with a playername.
thinks (owwww)
TCP packets maintain continuity via their IP and port source/destination combo. These can be muxed to form a unique session ID. If pogo-mitm can fetch these properties, it can compute a unique hash that can be provided back to the callbacks so that the callbacks can correlate playername and session.
Or, perhaps, upon intercepting a packet with a new unique hash, pogo-mitm can call an application-provided callback to construct an opaque object that will stay for the lifetime of the connection. That way the user application can store information related to the session in the opaque object, like playername and a few other settings.

The user (or at least a session) could be identified with the contents of auth_ticket field in request envelope. The first envelope of session however has auth_info instead.
Solution could be passing some generated unique id or a read-only copy of the the envelope as another parameter to handlers and let the programmer identify the users.

I think we should extract the auth_ticket from the envelope and pass a session object (stored within some global object keyed by that auth_ticket) to handlers. For the initial GetPlayer maybe still create the same object, and discard later if there is no corresponding auth_ticket.
Could limit it to callbacks that accept three parameters (identified by Function.length), since I think it's cleaner to have it as an optional first parameter.

#202 has my current implementation with the semantics I previously described. It should correctly handle reauth after token or ticket expiration; let me know if there are any problems (I haven't extensively tested the new handlers yet).


Added MarshalAny() support

This is not breaking current API but provides a huge effort if you need a fallback for a marshaller.
In cases you don't need a speed, but want to be sure that your object will be marshalled, if it's possible by default golang json implementation.
Thank you!

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #31   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          30     30
  Lines        3388   3394    +6
=====================================
+ Hits         3388   3394    +6

Impacted Files
Coverage Δ

encode.go
100% <100%> (ø)
: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 ca0442d...b1c88ac. Read the comment docs.

We could also check if it's not an implementation of MarshalerJSONArray, a string, or a number and use gojay, else we use standard json encoder. What do you think?

@francoispqt i will update PR, but it will BC changes IMO. Let me check this one.

@francoispqt added all failover stuff & speed up

@francoispqt any news on this one?


Introduce reduce metaops

I think I just stumbled over our first slang.
I started with thinking up a way to is parsed reduction ops:
macro term:reduce(op: Q::Infix, terms: Q::Expr[])
    is parsed(/
        "[" <infix> "]"
        <.ws>
        <EXPR>* %% ["," <.ws>]
    /) {

    # Ignoring the zero-op case for now -- exercise for the reader
    my ast: Q::Expr = terms[0];
    for terms[1..*] -> term: Q::Expr {  # assuming [1..*] syntax
        ast = quasi { {{{ast}}} {{{op @ Q::Infix}}} {{{term}}} };
    }
    return ast;
}

Wow, I love this "build-your-own-AST" (last seen in #134).
But then I thought about it a bit more. The above would handle hard-coded elements just fine:
say([+] 1, 2, 3);           # 6

But there would be no mechanism (short of quasi hackery at compile time) to inject an array in there. Which might even be a fairly common case. Perl 6 has prefix:<|> for that. I think 007 would go the JavaScript route and use ...:
my values = [1, 2, 3];
say([+] ...values);         # 6

So now we end up with something like this:
is parsed(/
    "[" <infix> "]"
    <.ws>
    ["..."? <EXPR>]* %% ["," <.ws>]
/)

But hold on. Unlike in #134, we just took a first step down a dangerous garden path: we did with is parsed what could very well be done with the expression parser itself!
Hence, slangs. For the purpose of this issue at least, a slang is a textually scoped, extended parsing environment. In this particular case, the slang is defining two operators:
infix:<,> (which might want list associativity, by the way)
prefix:<...>

I don't have any great idea about the actual underlying mechanisms, but it seems that the requirements involved are clear enough:

The slang is entered whenever we parse the /"[" <infix> "]"/ prelude in term position.
"Inside" of the slang, when we parse the list of elements, the two new operators have been magically added to the grammar. (That's what I meant by "extended".) As we leave the slang, the operators go out of scope, so to speak. (That's what I meant by "textually scoped".)
The slang emits a Q type that we can call Q::Term::Reduce. This type has the same properties as the parameter list above: a Q::Infix operator and an array of Q::Expr.
The .eval method of Q::Term::Reduce is basically just the macro body.

So... a slang is... a Q class? With an associated is parsed? I don't know if that will work, but I kind of like the sound of that.
Another thing about infix:<,>: note how we used %%, which means this should work:
my sum = [+] 1, 2, 3,;      # note extra comma
say(sum);                   # 6

If we introduce the comma as an infix, there's currently no way to do that. Maybe together with list associativity we should also introduce is lastable, or something like that.

do we not have an issue for is parsed? I was wondering if is parsed should always be installed at EXPR level.

It gets installed in the category of your choice, for example term or infix.
On Aug 17, 2016 14:19, "ven"<EMAIL_ADDRESS>wrote:

do we not have an issue for is parsed? I was wondering if is parsed
should always be installed at EXPR level.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/masak/007/issues/176#issuecomment-240394342, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABQkw0QTDWMynz8mVvDvnwAlC-Z1Z2Oks5qgvw2gaJpZM4JmWS1
.

Oh hey, wait. <EXPR>* %% ["," <.ws>] — where have I seen that before?
By some remarkable coincidence, that's the argumentlist rule in the 007 grammar. (It's written a bit differently, but it comes down to the same.)
So, yes Virginia: we can get our commas for free!
is parsed(/
    "[" <infix> "]"
    <.ws>
    <argumentlist>
/)

Two immediate objections:

This feels like abusing <argumentlist>.
Great, now we lost prefix:<...>.

But no, it's re-using <argumentlist>. As seen in Perl 6, the reduce metaop is a callable thing, taking arguments:
$ perl6 -e 'say [~](1, 2, 3)'
123

And if we implement #178 separately, then we're back to the proposed syntax of this issue:
$ perl6 -e 'say [**] 2, 3, 4'
2417851639229258349412352

And prefix:<...> is by right something that should be defined so that ordinary calls/arguments benefit from it too. Let's say the <argumentlist> rule was instead this:
<argument>* %% ["," <.ws>]

And the new argument category was defined by default as
rule argument:expr { <EXPR> }

Then we could get our prefix:<...> analogue like this:
rule argument:spread { "..." <EXPR> }

Which feels better anyway, because now we don't have the problem of someone using the prefix twice in a row, or somewhere non-toplevel.

Of course, a real implementation of this macro would also take into account the associativity (or lack thereof) of the infix operator.

And, as it turns out, we can take the <argumentlist> idea one step further — by not parsing that bit at all.
All we really want is a term that resolves to something that you can call, either with parentheses or as a listop, and it Does The Right Thing:
use rest_parameters;
macro term:reduce(op: Q::Infix) is parsed(/"[" <infix> "]"/) {
    my name = ...;   # something nice and introspectable
    return quasi {
        (sub {{{name @ Q::Identifier}}}(*values) {
            return values.reduce(sub (accum, val) {
                return accum {{{op @ Q::Infix}}} val;
            });
        );
    };
}

...I believe that's our first case of two-layers-of-sub-between-quasi-and-unquote.

prefix:<...>

Actually, I don't remember my own decisions/speculations, it seems: we've already decided in #112 to go with * for this (from Python and Perl 6, instead of from JavaScript). That's for parameters, but it makes sense to use it for argument spread too (like Python, unlike Perl 6).
Instead of going back and fixing this whole issue, I kindly ask the reader to please just hallucinate that any ... really says * above.

Here's a shorter version:
use rest_parameters;
macro term:reduce(op: Q::Infix) is parsed(/"[" <infix> "]"/) {
    my name = ...;   # something nice and introspectable
    return quasi {
        (sub {{{name @ Q::Identifier}}}(*values) {
            return values.reduce({{{op.identifier}}});
        })
    };
}

The term:reduce rule has as its declarative prefix [... as does the built-in term:array. The place where they differ is in the next atom, where the former has <infix> and the latter has <expr>.
It's lucky that these two are unambiguous. In the core language. But then someone comes along and declares term:<*> (for their every Whatever and WhateverCode needs)... and then there's an ambiguity. It's a "latent ambiguity".
When and how is this ambiguity resolved? I don't know. I feel [*] should be interpreted as a term:reduce, even in the face of a term:<*>.
This section of S02 seems to indicate that this is a place where look-ahead more than one "longest token" is employed. I... that makes sense, I think. So maybe we need to do it too.

We haven't even talked about [\+] yet. I wouldn't mind it. Dunno if it should be in a separate module? Probably not.


Pass **kwargs through to the paged function in Pager

This should fix #444 and any other time we need to page over a 'get' that has some extra parameters

Thanks for doing this! 🚀


Support flag to remove the right carrot in the trigger

Is your feature request related to a problem? Please describe.
The design of auro-combobox is a version of dropdown without the right carrot. Dropdown needs to support this.
Describe the solution you'd like
Add a new attribute. When present, the carrot will not be rendered in the UI.
Describe alternatives you've considered
Use the CSS Parts to hide the carrot.

This is already supported via the chevron attribute.


[Task]: create test instance for UQI

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
No response

Hosted at http://<IP_ADDRESS>/


GSE156989 - Multimodal analysis for human ex vivo studies shows extensive molecular changes from delays in blood processing

Project short name:
-Human-10x3ptranscriptionprofiling (Is someone working on it already?)
Primary Wrangler:
Secondary Wrangler:

Link to Ingest

https://contribute.data.humancellatlas.org/projects/detail?uuid=0dcdb978-d420-483d-8cc9-234fd0ed164f

Associated files

Google Drive:

Published study links

Paper: https://doi.org/10.1016/j.isci.2021.102404

Accessioned data: GSE156989

Key Events

[ ] Convert published metadata to HCA spreadsheet
[ ] Manually curate dataset to meet HCA metadata standard
[ ] Collect any matrix and cell-type annotation files
[ ] Upload sheet to validate metadata
[ ] Check linking using ingest graph validator
[ ] Transfer raw files to ingest to validate data files
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Agreed with wranglers on Friday that if we are focusing on something else, immune atlas datasets should drop from the Queue. Therefore closing the ticket as nothing was done


When is 2.3 planned for a release

Sorry not an issue, but it is overdue and just want to have an upgrade plan in place.
Does an upgrade of etcd cluster from 2.2.0 to 2.3 requires an upgrade to proxies as well, or is it optional?

It is optional. We will release it in the middle of March. Sorry for the delay.

@xiang90 Thanks, np just checking to plan :smile:


backend: delete unused bridge query

since this query is already move to other repo and bridge query that not related to explorer will not put in explorer backend, delete bridge query part

@icerove Thanks for cleaning it from the Explorer side!


Extension started showing "und..."

Recently the extension started showing und... without new notifications.
Chrome: 55.0.2883.75
Notifier for GitHub: 2.5.1

can you take a look into network panel to see what the API response was?

I received a 403 (Forbidden), which makes sense as I was just rate-limited by GitHub 🤕


Add scripts for ci build

Signed-off-by: Breezewish<EMAIL_ADDRESS>
What have you changed? (mandatory)
This PR allows actions to be run on CI controllable by the content of this repo.
Resolves https://github.com/tikv/tikv/issues/4623

Since this requires a change to all current release branches, I will close this PR temporarily and restart working on it after GA.


Improve login/sign up page

Improve design, add new features or totally redesign the login page.

hi @QAZIMAAZARSHAD .
i will like to work on this issue

I will assign you this issue when the contribution starts then you can start working on this issue.

ok on it..

Do you want to remove the background image under the form or you need it to be there?

Your wish, it's up to you. You can totally redesign the login page to make it better.

@QAZIMAAZARSHAD  Can I work on this issue? I'm a participant of LGM Soc21.

I'll assign you this issue when the contribution period starts from 10th June, then you can start working on this issue.
Other participants are also willing to resolve this issue so If you want your pull request to be merged then your design should be better than the others.

are we supposed to display the result on the same page under form or we have to navigate to another page to display?

It's up to you if you want to display the result as a pop-up on the same page or navigate to another page, it's your wish.

Great work @Dhanushreddy09, I'll merge your pull request when the contribution period starts.

@QAZIMAAZARSHAD Needless of me working on this issue right.. 🙂

Since many participants are willing to contribute or resolve a particular issue, I'll accept the best solution.
You can still work on the same issue, you can also create new issues if you find any mistake or chance of improvement in any solution.
I will start assigning issues and merging pull requests from 10th June.

@QAZIMAAZARSHAD thats cool..

There is no way users can get back to the home page once they complete the registration, I am trying to relocate the URL to the home page but since javascript doesn't support relocation of URLs on submit type inputs, it's really difficult to cope up. Is there a suggestion I could rely on?

@Dhanushreddy09 this solution worked ->

I came across this solution earlier but it isn't working for me. Anyways, I will commit it, however, I think we need to validate the user on the client-side. May I add javascript validation?

Yes, go-ahead
but don't make any pull requests before 10th June

Hey mentors and project admin as the coding period for LGMSOC starts today could you assign the issue to me

@nisha331 this issue has already been resolved by @Dhanushreddy09 and will be closed soon.
Once that PR is merged you are welcome to open a new issue to improve the design or fix bugs if any.

@coding-geek21 this issue has already been resolved and will be closed soon.
You are always welcome to open a new issue to improve login page design again if you have some better idea.
Have patience till the other PR is merged.

@coding-geek21 this issue has already been resolved and will be closed soon.
You are always welcome to open a new issue to improve login page design again if you have some better idea.
Have patience till the other PR is merged.

Sure thank you


non-sandbox build builds unnecessary things

As reported by Hanno Böck:
I'm confused because it seems to me it refuses to do a
non-sandbox-install. According to the readme the sandbox-install should
be non-default.
However if I do ./configure; make the first thing it's trying to do is
to download and compile httpd.
I already have http with alpn patched installed system wide.
Is this a bug?

Fixed in release 0.3.0


date hover on mouseover

#108

Error when window is resized:

Need to fix:
date is shown as an overlay on the entry box instead.


Kruskal-Wallis Test

The Kruskal-Wallis Test is a non-parametric statistical test used to determine whether there are significant differences between three or more independent groups. It extends the Mann-Whitney U test for more than two groups and is appropriate for ordinal or continuous data that do not meet parametric assumptions. The test ranks all data points from all groups together, compares the ranks between groups, and calculates a H statistic based on these ranks. A significant H value indicates that at least one group differs significantly from the others in terms of central tendency, but it does not identify which specific groups differ.
A non-parametric version of ANOVA used to compare three or more groups.

Can you assign it to me? @Niketkumardheeryan
Full name: Manish Rana
GitHub profile link: https://github.com/ranamanish674zu
Email ID<EMAIL_ADDRESS>Approach for this : python
What is your participant role?: GSSoC-2024 contributor
Can you add the label for GSSoC, i want to work on it
Thanks.


Scorer is not very good for matching on filenames

VSCode Version: 1.5.2
OS Version: OS X 10.11.6

The repo is Microsoft/vscode.

This is actually a corner case where our scoring is bad: Parts of the filename (explorerview.ts) end up matching on the folder instead of the file name. It is just bad luck that in this case explorerView.ts has parent folders that match more on the query compared to explorerViewModel.ts.
Overall we need to revisit our scoring functions and improve it to work better with files.

Fixed via https://github.com/Microsoft/vscode/commit/9d40f0c647bf1de0aaa813e6519b85a24e98da14


RemovePrivateMethodNeverUsedAnalyzer (CC0068) throwing cast exception

RemovePrivateMethodNeverUsedAnalyzer is throwing when analyzing Mono Cecil:
The Compiler Analyzer 'CodeCracker.Usage.RemovePrivateMethodNeverUsedAnalyzer' threw an exception with message 'Unable to cast object of type 'Microsoft.CodeAnalysis.CSharp.Syntax.MemberAccessExpressionSyntax' to type 'Microsoft.CodeAnalysis.CSharp.Syntax.IdentifierNameSyntax'.'.

See details on 395: build https://ci.appveyor.com/project/code-cracker/code-cracker/build/<IP_ADDRESS>5
The analyzer is being disabled so the build does not fail anymore.

@julianosaless can you work on this one?

@giggio OK I will work :)


How to notify List component that its row item height has changed?

I have a react-virtualized List, each item in the rows have an expand button, when the button is clicked the height of the row changes and it expands and show more info.
the issue here is, after the button got clicked the inner Grid (ReactVirtualized__Grid__innerScrollContainer) CSS height property is not updating or the Gid is not re-rendering/notified of the changes in row height
What should I do to make it rerender the row everytime the row item height gets updated?
I tried using forceUpdateGrid() with useRef() but it does not seems to work? or maybe I did it wrong?
Sandbox code:
https://codesandbox.io/s/agitated-pond-xq1csq?file=/pages/index.js

you need use CellMeasurer and CellMeasurerCache, when someone item has update height, you should clear one of CellMeasurerCache, then rerender

@yelnyafacee You should warp your list item with CellMeasurer. CellMeasurer. It passes the measure render prop to your component so you can call it in a function that changes it's layout.


Implement undo/redo function for networkbook data

As an NUS computing student, I want to be able to undo and redo commands that alter the networkbook data state so that I can maintain data integrity and recover from accidental modifications to the networkbook data.


V10 sdk 12/15/2015

Update to latest V10 of the PHP SDK, this fixes #22

Any ETA on when this is going to be merged? (:

Sorry for the late reply.
I should be working on this with in the next few days, just want to make sure I put the latest SDK v9, before merging this one.


Change default instance disk type to SSD

Changes the default EC2 disk type from magnetic to GP2 (non-provisioned SSD).
They cost the same amount and have 1/3 to 1/10 the IOPS performance.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   18.12%   18.12%
=======================================
  Files          54       54
  Lines        2714     2714
  Branches      305      305
=======================================
  Hits          492      492
  Misses       2186     2186
  Partials       36       36

Impacted Files
Coverage Δ

lib/python/treadmill_aws/ec2client.py
35.05% <ø> (ø)
: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 c1dd214...f64e6dd. Read the comment docs.


Cannot run dagit

The docs for scheduler says to run dagit -h ghost.lan -w workspace.yaml. However, even after installing all requirements files it says the following:
It seems to be because of this:
https://github.com/dagster-io/dagster/discussions/18783
(.venv) cloftus@Coltons-MacBook-Pro ~/g/s/d/implnets (master)> dagit -h ghost.lan -w works
pace.yaml
Traceback (most recent call last):
  File "/Users/cloftus/github/scheduler/.venv/bin/dagit", line 5, in <module>
    from dagster_webserver.cli import main
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster_webserver/__init__.py", line 1, in <module>
    from dagster._core.libraries import DagsterLibraryRegistry
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/__init__.py", line 84, in <module>
    from dagster._config.config_schema import ConfigSchema as ConfigSchema
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/_config/__init__.py", line 5, in <module>
    from .config_type import (
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/_config/config_type.py", line 6, in <module>
    from dagster._annotations import public
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/_annotations.py", line 9, in <module>
    from dagster._utils.backcompat import (
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/_utils/__init__.py", line 50, in <module>
    import dagster._seven as seven
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/_seven/__init__.py", line 16, in <module>
    from .compat.pendulum import PendulumDateTime as PendulumDateTime  # re-exported
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/cloftus/github/scheduler/.venv/lib/python3.12/site-packages/dagster/_seven/compat/pendulum.py", line 31, in <module>
    pendulum.DateTime if _IS_PENDULUM_2 else pendulum.Pendulum  # type: ignore[attr-defined]
                                             ^^^^^^^^^^^^^^^^^
AttributeError: module 'pendulum' has no attribute 'Pendulum'. Did you mean: '_pendulum'?
(.venv) cloftus@Coltons-MacBook-Pro ~/g/s/d/implnets (master) [0|1]>

seems to have dependency issues when trying to fix this
        File "/private/var/folders/ff/zdlbl2jn4jx2h5qfdzjrdqvc0000gn/T/pip-build-env-rd7l2wa7/overlay/lib/python3.12/site-packages/poetry/core/masonry/builders/wheel.py", line 262, in _run_build_script
          subprocess.check_call([self.executable.as_posix(), build_script])
        File<EMAIL_ADDRESS>line 413, in check_call
          raise CalledProcessError(retcode, cmd)
      subprocess.CalledProcessError: Command '['/Users/cloftus/github/scheduler/.venv/bin/python', 'build.py']' returned non-zero exit status 1.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pendulum
Failed to build pendulum
ERROR: Could not build wheels for pendulum, which is required to install pyproject.toml-based projects

[notice] A new release of pip is available: 24.0 -> 24.2
[notice] To update, run: pip install --upgrade pip

we no longer run dagit but instead use dagster. dagit appears deprecated


Add download icon for files and attachments

Currently the name of each file is a hyperlink. This leads to unintended downloads. I have watched numerous users download the file by accident, when they actually want to just view the file.
I think we should stop hyperlinking the file name and use a 'download' button that will initiate the download.

+1
And while we're at it, we should hide the permalinks behind a button.
I could imagine a single column, maybe called "Actions", holding the three icons: View, Download, Permalink.


Missing format specifier for arguments of image status

Fixes https://github.com/Kilian/Trimage/issues/73
To reproduce this issue, select a bunch of files, ensure some are queued and ensure that status displays.
Tested with Python 3.7.7.

I didn't add the file names there on purpose to keep things looking clean, but I suppose it can't hurt having them. Thanks!


Dockerfile available?

Hello,
I would like to change the docker container you provide for the project, but I struggle to find the dockerfile so I can edit it. Can you help me there @yangxue0827 ?
I looked here: https://hub.docker.com/r/yangxue2docker/py3-tf1.15.2-nv-torch1.8.0-cuda11/tags and when I click here: https://hub.docker.com/layers/yangxue2docker/py3-tf1.15.2-nv-torch1.8.0-cuda11/v1.0/images/sha256-89819321b7277f6def51a9c349caca62b4c9ef881f2ed0af4402d298b465f95e?context=explore it also shows what I believe are the commands of the dockerfile, but I do not find the file, so I can alter it. I am using Ubuntu on WSL2 if that matters.
Thank you and best regards

You can use my docker image as the base image, and then write the dockerfile by yourself.

I tried this but had no luck, as there are some errors then when running the code.
Basically my main issue is: When I use the Dockerfile I made without you base image it does not find the *.cpp files that are imported. So I was thinking maybe you set some ENV where is has to look for the imports.
When I use your Docker as the Base image there is conflicts with the Python and Ubuntu Version, as I would like to use Python 3.7 or 3.9, but as in your base Docker it is Ubuntu 18.04 I believe it can only run Python 3.6
Do you have an idea how to resolve this? I am not an expert on Docker and maybe having your Dockerfile could help me to understand it better.
Thank you very much and best regards


feat: Auto layout appsmith editor

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma or any other documents that might be relevant to the PR

Add a TL;DR when description is extra long (helps content team)

Fixes # (issue)

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=006e486

/ok-to-test sha=d00e2af

/ok-to-test sha=7c4dd7a

/ok-to-test sha=42e00a5

/ok-to-test sha=2ee5f77

/ok-to-test sha=d817986

/ok-to-test sha=e881ba5

/ok-to-test sha=c27e754

/ok-to-test sha=d633e7e

/ok-to-test sha=142d6fe

/ok-to-test sha=bc71f0a

/ok-to-test sha=33cae06

/perf-test

/build-deploy-preview

/ok-to-test sha=4c2ed8a

/ok-to-test sha=caaec16

/ok-to-test sha=6435c5d

/perf-test

/perf-test

/perf-test

/perf-test

/perf-test

/perf-test

/deploy-build-preview

/build-deploy-preview

/perf-test

/build-deploy-preview

/perf-test

/build-deploy-preview

/build-deploy-preview

/ok-to-test sha=58c0db0

/ok-to-test sha=7c31f61

/build-deploy-preview

/ci-merge-check

/ok-to-test sha=233f6dc

/ok-to-test sha=3a1d327

/ok-to-test sha=f9df3c7

/build-deploy-preview

/perf-test

/build-deploy-preview

/perf-test

/build-deploy-preview

/build-deploy-preview

/perf-test

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/build-deploy-preview env=release

/perf-test


collections do not display in live preview

The documentation mentions you can access collections out of the box, but the live preview shows nothing. Also, if I apply a filter like "reverse" it errors and does not render any Bookshop components in the preview.  I know that filters are limited, but am I missing a step to see collections in live previews? The documentation mentions you should be able to access as normal 'collections.*' which works fine in a normal 11ty render, just not live preview.
Running all the latest packages:
"@11ty/eleventy": "^2.0.0",
"@bookshop/browser": "^3.4.0",
"@bookshop/eleventy-bookshop": "^3.4.0",
"@bookshop/eleventy-engine": "^3.4.0",
"@bookshop/generate": "^3.4.0",
"@bookshop/sass": "^3.4.0",
On Node: 18.13.0

This appears to be a documentation issue. It reads "out of the box" yet you still need to apply it to the collections to the data_config. This is confusing as I would assume out of the box does not require me to make any additional configurations.

Hmm @edmeehan can you share what you had to add to data_config to get this to work? Collections should indeed work out of the box without touching data_config, but maybe something else is going on here.
For some insight — this is the integration test for live editing an Eleventy collection. That test has no CloudCannon configuration file at all.

Yeah, something is happening with my setup, I can access a collection, but no data objects are being returned. So I can see the count of items but can't access the data. I am waist-deep on a project I need to deliver in 24 hours. So when I am done, I will revisit this and post my config. This is my first CloudCannon project and have had a lot of trail and error getting the config correct.

I am setting the tags using in a directory using an eleventy.js file - https://www.11ty.dev/docs/data-template-dir/

I tested adding the tags property directly in a few files in a collection and it still didn't work. When I output the collections items into a template I see [FILTERED] instead of [OBJECT] as expected.

Hmm.
Overall, this is definitely a case where I need to improve the documentation (which I am currently doing!)
Firstly just to check — if by live preview you're referring to the Bookshop component browser, collections will not work there. In the CMS Visual Editor though, these should be accessible.
Regarding the Visual Editor:
When Bookshop considers "collections" it isn't looking directly at any 11ty concept, but is instead using CloudCannon's understanding of collections. So the collections that are visible in your sidebar (or configured with collections_config) are what will be available in the live preview.
Are the collections you're trying to access shown by CloudCannon in the CMS?
(Also, if you have any code base on hand that you can share I'd be happy to spin it up and dig in, which might be the quickest resolution)

Yeah, this is in CC visual editor live preview. The only thing I am finding that could be the issue, is when my site builds I notice some added messages regarding collections.
The collection I am working with is "events" - but in the screen shot you can see it is empty and it processes all the subfolders with collections names like "events/2020" with X files.
I wonder if this has something to do with CC collection vs 11ty collection.

Yes, I can confirm now that my issue is related to how CC interprets collections (based on the log) differs from what 11ty defines as collections.
As a test, I tried the "pages" collection and was able to access the data.
I was not able to access the "events" collection (but 11ty can access it), but I could access the "events/2017" collection.

To summarize the issue, this appears that collections are not the same between CC and 11ty.
It appears CC collections are based on folders in the app, while 11ty is defined by the tags property in the front matter.
This can be seen in the build log above.

I read thru the CC documentation related to collections. I only see mention of folders defining collections. In my case, I have a collections folder but also subfolders, and I want all files on those subfolders to be a part of the parent collection and not a new subfolder collection.
I do define collections in my config as such:
collections_config: {     events: {       path: 'events'     } }

Going to close this again since you can work around it with configs... but should allow collections to reflect the SSG like eleventy.
Just incase someone hits this same wall. I solved this by using the dynamic permalink option in front matter for 11ty.
screen shot of my CC collections now that all files are in a single folder:

Yes, CloudCannon collections correspond strictly to folders rather than SSG concepts like tags — since the core CMS is generic over SSGs it's the best path there, so that is unlikely to change.
I'll keep this in mind though for whether Bookshop could support remapping content to match an 11ty collection — it depends on what data we have ans when. I'll jump back into this thread if I make any progress on that in the future. In the meantime I'm writing new Bookshop 11ty documentation, so I'll make the semantics of this super clear.


SolidWorks path fix try 2

It worked to include a  PropertyGroup in my csproj.user file that overrides the on in the .csproj file.  However, to make it work I had to modify the hintpath in the .csproj file.  (see the Solidwork path fix pull request for more details on the issue I am trying to fix)

__  (see the Solidwork path fix pull request for more details on the issue I am trying to fix) __ is not generally too helpful. Try https://github.com/Weingartner/SolidworksAddinFramework/pull/9 when referencing an issue.

Ok, that's not a problem. Thanks for that.
Can you provide an example of the csproj.user file?
We would like to include it in the docs.

I attached 2 csproj.user files
DemoMacroFeature.csproj.zip

Thanks bradphelan.  Next time I'll reference the issue number


slightly cleaner WHERE construction

Instead of catenation and then cleaning up the extra delimiter at the end, how about creating an array of predicates and then joining them - which will create the exact number of '&&'s needed?

Sure thanks for the suggestion


Add support for overriding the redirectUrl in loginWithRedirect

Description
When using a provider that has a separate loginurl and log out url, the redirect back to the login url isn't occurring, instead they are being redirected to the location.href.
Expected outcome
The user should be redirected to the login redirect url.
Actual outcome
The user is being redirected to the location.href instead of the login redirect url.
Browsers Affected

[X] Chrome
[X] Firefox
[X] Safari 10
[X] Safari 9
[X] Safari 8
[X] Safari 7
[X] Edge
[X] IE 11
[X] IE 10

This is intended behavior.
Basically this ensures the user is brought back to where they started when using loginWithRedirect.
https://github.com/salte-io/salte-auth/blob/master/src/salte-auth.js#L124-L132
Example

User hits https://salte.io/my-custom-view
User is redirected to the IDP and logs in
User is redirected back to https://salte.io
salte-auth then sends the user back to their initial request,
which in this case was https://salte.io/my-custom-view

This doesn't work well in our case.
The user https://myhomepage
The user is redirected to the IDP and logs in
The user is redirected to https://myhompage/dashboard
Then another redirect occurs send the user back to https://myhomepage   The user needs to land on https://myhomepage/dashboard after logging in.

One option would be to allow the developer to provide an override redirectUrl to loginWithRedirect.
const auth = new SalteAuth({
  // ...
});

auth.loginWithRedirect('https://myhomepage/dashboard'); // This will take the user to /dashboard
auth.loginWithRedirect(); // This will take the user back where they started

@nick-woodward
I am good with this approach if you are?  I can open a PR for your review.

This change will be deployed in 2.10.0 in a few minutes,
NPM invalidated everyones tokens due to a security incident.

This change is now available in v2.10.0.


Add "link" property to StoredEvent when using resolveLinkTos flag

When unpacking an event message that has a defined "link" property (which would be a sibling of the "event" property), create a "link" property in the unpacked StoredEvent to hold the deserialized link information. This "link" would become another StoredEvent instance.
Such a structure should only exist if the stream was read using the resolveLinkTos flag.
NOTE: I also corrected a small error in the README and the TypeScript defs, which indicated that the "number" property in StoredEvent represents the event number.  Based on what I saw in connection.js, the property is "eventNumber", not "number".
Original discussions:

Issue #5
Google Groups thread

Thanks for contributing!

My pleasure. If we go forward with this client, I may be back.


how to use pool like guzzle?

a guzzle httpclient can use pool to do requests like official example.

You can use the pool from the official example and create the client as shown in the readme:
$loop = \React\EventLoop\Factory::create();
$handler = new \WyriHaximus\React\GuzzlePsr7\HttpClientAdapter($loop);

$client = new \GuzzleHttp\Client([
    'handler' => \GuzzleHttp\HandlerStack::create($handler),
]);

$requests = function ($total) {
    $uri = 'http://<IP_ADDRESS>:8126/guzzle-server/perf';
    for ($i = 0; $i < $total; $i++) {
        yield new Request('GET', $uri);
    }
};

$pool = new Pool($client, $requests(100), [
    'concurrency' => 5,
    'fulfilled' => function ($response, $index) {
        // this is delivered each successful response
    },
    'rejected' => function ($reason, $index) {
        // this is delivered each failed request
    },
]);

// Initiate the transfers and create a promise
$promise = $pool->promise();

$loop->run();

(Quick ontested hackup.)


Add feature for removing constraints and indexes from schema

This feature is supposed to add functionality similar to neomodel_install_labels, just instead of creating new indexes and constraints it removes all existing ones.
Code is modeled after install_labels example.
Test is also added that remove all indexes and constraints and then reinstall all removed again not to mess up with other tests.
This feature was missing because manually removing installed labels especially when there is a lot of them is tedious work.

Thanks for this, any chance you could add some documentation to the docs/ so people know about the feature.

I added short doc into getting_started.rst.
Feel free to move it to some other place and modify something.

Super thank you :-)


Implement new stylings based on the new approved mockups

Implement in the stylings https://github.com/montevideo-tech/hybrid-meetup-platform/issues/132

@fyoungqualabs @Unnil can you share the mockups here?
It would be good if this kind of discussions could be held in GitHub so more people have a chance to be part of the decision making, since this is a community project 🙂

Ciao Marco! totally agree, once we have the mockups we'll send them here so the community can have a voice in this decision.

Thanks Felipe, I appreciate!


Missing a Javadoc comment

Hi everyone, I received this error when I submit my Log.java. Does anyone know how to fix this?

Can you share your code and show where the error occurs?

you must write a javadoc to solve the issue. a javadoc looks like this for example:
/**
* Applies the given function to this Log's value, returning a new Log of type {@code }. This
* method transforms the value to another Log instance. If this log message is empty, it
* directly returns the function's result, otherwise combines the log messages.
*
* @param  The type parameter for the returned Log, determined by the function.
* @param mapper The function to apply, taking a value of type {@code } or its supertype and
*               returns a Log of type {@code }.
* @return A new Log instance of type {@code }. If this instance's log is not empty,
*         the returned Log's message combines this instance's and the result's logs. Otherwise,
*         it returns the result directly.
*/

Thank you! @brianxlim


Issues with configs

While it works fine as I first launch it, if i set it to launch an application (in this case, battle.net) it switches to a default config. Setting GloSC to open nothing and then opening the application doesn't fix the issue.

You mean the controller is switching back to desktop-config?
I just checked with latest Steam-Beta, it still works perfectly fine for me...
You have to configure your individual shortcuts controller config in Steam

No, I mean it switches to a keyboard+mouse config when I have set it up as a kb+m and controller config. It no longer falls back to a default config, but the controller part of the config doesn't work, forcing me to use a solely kb&m config

Can you try this https://github.com/Alia5/GloSC/releases/tag/<IP_ADDRESS> ?

Sorry for the very late response, I had just given up on the problem by using another program, but now that program doesn't work...
I tried that version of GloSC, didn't help
Tried setting steam to run as admin, didn't help either


student has transferred msc twice in oodi, shows correctly in sis

011048721 is all good in sis. transfer script doesn't catch this, because msc has been transferred twice.
wontfix

also 012083633


Remove allocs monitoring, fix docs, fix parameter tests

This PR:

Removes the allocation monitoring (a step towards #1766)
Fixes some doc strings for file includes
Fixes the parameter tests and includes them in the test suite.

I did these all together because the doc string kind of spans the two issues

bors r+


add clearHistory, clearCache and clearFormData android webview api.

https://github.com/react-native-community/react-native-webview/issues/380

@punksta Thanks for contributing this! Could you update documentation too, while you're at it?

@jamonholmgren done

@jamonholmgren could you also recheck this https://github.com/react-native-community/react-native-webview/pull/451 ? it's sort of stuck

please udpate types, as theses methods should be on android only, you can create AndroidWebViewCommands extending from WebViewCommands

I'm still okay with merging this if it's fixed

great! I still want to get this merged, will try to find time this week.

@Titozzz hello :) I finally found time

@Titozzz thanks for waiting and reviewing.


CVE-2021-32810 and CVE-2020-15254

https://ossindex.sonatype.org/vulnerability/70ecede9-20b0-4454-9ca5-d1b1908b44ac?component-type=cargo&component-name=crossbeam-queue
https://ossindex.sonatype.org/vulnerability/5469eaee-ca6f-44e8-bf0c-583acc9606da?component-type=cargo&component-name=crossbeam-queue
Could you please update crossbeam?
https://github.com/crossbeam-rs/crossbeam/issues/828

Thanks, I'll bump minor version of the crossbeam for the next release.
For now please run cargo update.


Update resourcelist.md

Updated resourcelist.md to include a link to JS Hero.

The description you've included is great, really helpful! Thank you for you contribution!


add item identification for logs and planks

I added item identification for logs and planks. Let me know if I need to change something. thanks

I made a mistake committing the first time, hence the force push. let me know if i should recreate the pr. thanks


Improve keyboard navigation in column view

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #7139
Closes #6736
Closes #6404
Closes #5681
Closes #5375
Closes #2977

Details of Changes
Add details of changes here.

Use left/right arrow keys to navigate column view
Autofocus file list on navigation (all view modes)
Ability to select first item by keyboard when a folder contains only one item (all view modes)
Open selected folder in a new column when pressing Enter
Remove redundant resources from ColumnViewBrowser
Partially fix back/forward buttons for ColumnView
Switch active column or pane when using Tab key
Scroll focused column into view

To be merged after #7390
Validation
How did you test these changes?

[x] Built and ran the app

@gave92 is #5375 supposed to default to opening folders in one click in the column view or should that be a separate option in the settings dialog to implement in a different PR?

@gave92 can you resolve the merge conflicts?


Point Coordiante with Instance Builtin

I’d suggest another Point, next to Pos and Vec, that coveres all things of Vec, but also has an Instance attached to it.
InstancePos / BlockPos would be a helpful friend :D

Please edit the title/message, I do not understand what is the suggestion/issue

Please edit the title/message, I do not understand what is the suggestion/issue

Now more understandable?

This would just add unneeded complexity, a point in space shouldn't refer to an instance. What would even be a valid use case for this, and even if you have a valid use case, what prevents you from making a InstancePos record yourself?

I tried making it myself. But I have problems extending it from Vec.

Will be an Minestom way of Spigots Location

Whats your use case?

Entity Teleportation or Spawning is simpler :D

record InstancedPos(Instance instance, Pos pos) {}

entity.teleport(instancedPos.instance(), instancedPos.pos());

Something like this would work (even though it's not needed at all). Not a feature for Minestom itself.


Problem with user-changeable properties

In props.js when updating the properties, we compare the new props against the oldProps of that vnode, instead of comparing against the actual properties of that element.
This causes problems when updating user-changeable properties of an element (like value or checked property of an input or textarea). Just typing into an input field already changes the property value of value, the assumptions that only snabbdom changes the property is not true.
For example, we want to clear the value an input field, whenever a button is clicked. We can then patch the node with { "props": { "value": "" }}. But what if the user changes the value of the input field in between. Then comparing only the old properties and new properties of the vnode does not suffice.
An example to illustrate the behavior: https://jsfiddle.net/0q6uueou/4/#.
How about changing the relevant code in props.js to this?
    for (key in props) {
        cur = props[key];
        old = elm[key];

        if (old !== cur) {
            elm[key] = cur;
        }
    }

I've heard that reading from DOM is expensive compared to reading from JavaScript objects. I didn't test this, though.
Seems like snabbdom was designed to not expect changes to DOM by other agents. Perhaps you can write a snabbdom module?

If performance is an issue, we could think about doing this only for certain keys. There is already another exception with value.

Just taking into consideration the value of the property during the diffing algo for value, checked and other special properties is problematic. From the top of my head, using the code that you are proposing has the fallowing problems:

you're touching the DOM many times when you don't have to (think of a change event sending the new value up, to update their vnode, and patch again to reconcile the DOM structure without touching the DOM)
is the actual value of the input is the same as the newValue, which is different from the oldValue, seeing the new value removes the active selection on the element in old browsers (I don't remember all the details though).

The current code is sort-of a compromise because you don't have a way to reset the value of the input to the original value used by the vnode because it will always be triple equal, and therefore skip the operation in favor of a more performant diffing algo. It seems that everyone is fine with that. Maybe we just keep adding more special property names for certain type of elements rather than compromising on the perf impact of that module.

Out of interest:
What is the difference in performance between reading the value form the vnode vs reading from the dom?

Reading from the vnode is just a ECMAScript member expression (it is all in JS), while a DOM property requires calling a getter (each DOM property is a descriptor with a getter and a setter) and then this getter is crossing the bridge to C to access the value of the realm property. That second pass is actually very slower.

Thank you. Are there any benchmarks which show how much slower exactly that is? We have to evaluate, if it makes sense to implement a custom module or use some other workaround.

I just measured and reading from dom seems to be ~10x slower than from virtual dom. But both are still quite fast enough to not matter in most cases (~50M ops/sec).
https://jsperf.com/property-plain/1

@cornerman 10x is best case scenario! Normally is more folds depending on how complex is the getter for the property. Also, this is one of the most common operations during diffing. You touch many nodes and many props on every node, this gets slow really easy. Unfortunately we haven't finish the work to do perf analysis per PR just yet.

This issue would be fixed by #418


add check_csnippets, check_cxxsnippets function support  test code build as a whole unit without automate add main function in its

Is your feature request related to a problem? Please describe.
check_csnippets, check_cxxsnippets now support check code snippet with main function  automatic added. but some time we just need code as a complete unit to build and try to run, xmake should led user to decide whether main function add or not.
code like this can't work properly now:
extern "C" int example_undefined_weak_symbol() __attribute__((weak));
auto f = example_undefined_weak_symbol; // null pointer
return f ? f() : 0; // must compile, link, and run with null pointer

Describe the solution you'd like
add a main  argument  with boolean value to opts
without main:
configvar_check_cxxsnippets("HAVE_WEAK_SYMBOLS",[[extern "C" int example_undefined_weak_symbol() __attribute__((weak));
int main(){
    auto f = example_undefined_weak_symbol; // null pointer
    return f ? f() : 0; // must compile, link, and run with null pointer
  }]], {main=false})

Describe alternatives you've considered
No response
Additional context
No response

try it again. just set tryrun = true, xmake will detect main entry.
xmake update -s dev

    check_csnippets("HAS_INT_4_IN_MAIN", [[
    int test() {
        return (sizeof(int) == 4)? 0 : -1;
    }
    int main(int argc, char** argv) {
        return test();
    }]], {tryrun = true})

thanks


[FLINK-9028][flip6] perform parameters checking before starting cluster

What is the purpose of the change
Perform parameters checking before starting cluster to prevent to setup a problematic cluster.
Brief change log

perform parameters checking before starting cluster

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation
no

CC: @tillrohrmann

@tillrohrmann I made a fixup according to your comments, could you please have a look again?

@tillrohrmann I have addressed your comments, could you please review it again.

Thanks for your contribution @sihuazhou. Changes look good to me. Merging this PR once Travis gave green light.

...found a checkstyle bug in my local built, I made a fixup.

Have seen it as well and fixed it. So no worries.


Error invoking remote method 'GUEST_VIEW_MANAGER_CALL': Error: Script failed to execute, this normally means an error was thrown. Check the renderer console for the error.

On Windows, once built, and you try to enter a prompt I get:
Error invoking remote method 'GUEST_VIEW_MANAGER_CALL': Error: Script failed to execute, this normally means an error was thrown. Check the renderer console for the error.
I will note that I had to force the npm install as nearly every package was complaining.

we've updated the deps #180 so the complaints stop
the error is a very common one and unfortunately an electron issue. it usually means that one of the pages isn't in the right state. can you screenshot the app here when before you enter the prompt? and make sure you're logged in to every one of the open apps

please let me know if this issue persists in the latest release, and record a video of what exactly happens!


Remove unnecesary ember-new-computed addon

If this addons is 1.12+ (and I think that it safe to assume it is), this
addon is not needed anymore.

:umbrella: The latest upstream changes (presumably c623a65) made this pull request unmergeable. Please resolve the merge conflicts.

Rebased

:umbrella: The latest upstream changes (presumably f5002eb) made this pull request unmergeable. Please resolve the merge conflicts.

Rebased again.

This LGTM 👍 . Sorry for the delay in merging!


Clarify the -ASN argument

Added some clarifying text for the -ASN option to make it clearer.

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

#sign-off

@illfated: I'm sorry - only the author of this article, @JasonGerend, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Yes, I know. My sign-off comment is based on a previous agreement with the team.


Show email update form if user is Stripe customer

Only show the form for updating the billable email on an account if the
current user has a Stripe customer ID. Users who aren't in Stripe's
system shouldn't have the ability to update their billable email
address.

LGTM.


More Activ Endgame

Are there Any Plans for a more activ Endgame? All Minigames and the Core Game are only Idle Gaming at some Point. Maybe some more Activ Minigame? Beerbraker (Ballbreaker Clone) or an RPG the Reward can be Some Buffs you can earn. Imagine an Chest in an RPG wich contains 3 Buffs which are aktivated at the same Time.

Hi @Marcnesium87 current plans are to add another sacrifice level after hitting infinity (compare #37). Which features will unlock in that sacrifice level is currently not defined. More minigames for the buildings are possible.
If you have concrete ideas for minigames they are always welcome. I don't like the idea of bringing arcarde games into Beerplop.


Mockable on Private Protocol Cannot Be Initialized Due to Private Initializer

In rare instances, a protocol marked with Mockable may be private and used solely within its file. If this happens, the generated mock cannot be initialized because its initializer is marked as private. While marking the protocol as fileprivate can resolve this issue, it is undesirable. Could the initializer, and generated members of the mock be restricted to an access modifier no less accessible than fileprivate?
example:

Inlining the Mock will give me this error:
Method 'registerUser(_:password:)' must be declared fileprivate because it matches a requirement in private protocol 'RegisterViewInteractor'

I create a PR for this here


micro as a peerDependency

Hi everyone, how are you?
What do you think about define micro as a peerDependency?
For example, in my case I'm using micro 8.0.4 in production but when I run micro-dev I have two differents intances of micro working on (8.0.4 and 8.0.2) because how NPM resolve my dependencies.
This is a problem when micro uses some kind of internal state behaviour like this one: https://github.com/zeit/micro/blob/master/lib/index.js#L114

Seems like the right thing to do, @leo what do you think?


False Negative on Zookeeper

Describe the bug
<dependency>     <groupId>org.apache.zookeeper</groupId>     <artifactId>zookeeper</artifactId>     <version>3.5.3-beta</version>     <type>pom</type> </dependency>
zookeeper-3.5.3-beta.jar is vulnerable as per NVD CVE-2018-8012
cpe : cpe:2.3:a:apache:zookeeper:3.5.3:beta::::::
Version of dependency-check used
The problem occurs using version 5.0.0.M3 and 5.3.0 as well of the OWASP dependency checker
Expected behavior
This should be identified by tool as it is exact match of cpe mentioned in NVD

While there is an underlying bug - I would also contest that you do not actually have the server code in your build that is vulnerable. This is because of the dependency <type>pom</type>. If you remove this then the zookeeper JAR file (as opposed to just the transitive dependencies) would be included in the build and via the OSS Analyzer these findings are flagged.
See the dependency:tree:
[INFO] +- org.apache.zookeeper:zookeeper:pom:3.5.3-beta:compile
[INFO] |  +- org.slf4j:slf4j-api:jar:1.7.5:compile
[INFO] |  +- org.slf4j:slf4j-log4j12:jar:1.7.5:compile
[INFO] |  +- commons-cli:commons-cli:jar:1.2:compile
[INFO] |  +- log4j:log4j:jar:1.2.17:compile
[INFO] |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] +- junit:junit:jar:4.12:test
[INFO] \- org.hamcrest:hamcrest-core:jar:1.3:test
[INFO] ------------------------------------------------------------------

Only the POM is included in the build.


Library Size

The current compiled size of libReactiveSocket.a is 1.6M:
1.6M May 17 10:12 libReactiveSocket.a

The Java version is 185k and is feature complete:
185K May 17 10:14 reactivesocket-0.1.1-SNAPSHOT.jar

Java currently has a dependency on a 200k library, but that can likely be reduced with some effort.
So, reactivesocket-cpp is still not complete, and we are at 1.6M vs the complete Java impl being < 400k.
Folly is > 50M for everything, so I'm assuming the static linking is just pulling from it what is needed (showing my lack of C++ experience):
-rwxr-xr-x  1 root  wheel   5.9M Jan 11 02:15 /usr/local/lib/libfolly.57.dylib
-rw-r--r--  1 root  wheel    68M Jan 11 02:15 /usr/local/lib/libfolly.a

Is this greater size expected for C++? Or is it something we can reduce by being more careful with our dependencies?
I am exploring this due to desire to use this on mobile.

Optimising for code size gives ~800kB. We probably only need IOBuf from folly, which we can strip/reimplement easily. Also, there are plenty of symbols that we don't need to export (mangled strings stored in object files). There is tremendous amount of room for improvements at this stage, even without touching actual code, I think.

did we enable linker to exclude unused symbols like unused global variables?

Which flag exactly?
I'm just going to leave it here for anyone who ends up working on, this https://blog.algolia.com/android-ndk-how-to-reduce-libs-size/.

we are down to 250k
We can remove the rest of template code with some refactoring and get to even smaller size.

Excellent news, thank you Ondrej for the update and work to get to this point.

Can we close this now?


Feature: Multiple Schedulers

This PR is in reference to issue #195
It adds the feature of running multiple scheduler instances (on multiple servers).
Each scheduler registers itself, then tries to acquire a lock to process the scheduling work. If the lock gets acquired it will schedule the jobs to the appropriate queues, remove the lock, and sleep for it's interval.
In the meanwhile another scheduler can try to acquire the lock. If it fails, it will simply sleep for its interval and try again later.
This way we don't have a single scheduler doing all the work. Different schedulers may acquire the lock at different time instances.
I've made sure that even those schedulers who can't acquire a lock have still registered their birth and a heartbeat makes sure the scheduler instance key doesn't expire even if the instance never acquires a lock.
Please let me know how we can proceed :)

@oxalorg this looks like it works well though for some reason when paired with django-rq it seems like the scheduler itself is executing the jobs rather than the rqworker process. My test was to run python manage.py rqscheduler --interval 10 twice and a worker with python manage.py rqworker. The worker process reported nothing however in the 2 ttys for the schedulers they printed out the debug statements from my test jobs.

Hey @mattjegan I've created a dummy repository to test oxalorg's PR and I didn't come across what you experienced. The jobs are run by rqworker, no debug statements from the test jobs in my rqscheduler windows.
You can quickly pull and test this repo: https://github.com/ArionMiles/rqschedtest
Let me know if there's any other issues 😄

@ArionMiles Thanks for this, everything looks good with that repo. Perhaps it was something else in my env.

Can you let me know what version of django you were testing on when you came across your problem? My test repo uses Django 1.11.

Bllllluuuuurrrrrrrrrrrpppppppppppp (placing this useful note here so I get notified when this is merged)

How are we doing here? We'd love to see this merged :)

Another comment wishing for merge and asking for status. @oxalorg: could you rebase your branch? I'll having point my pipenv to your fork for now.

Sure, give me by the end of the day. I'll have it rebased! ^_^
On Mon, Apr 6, 2020 at 4:13 PM Nikolai Prokoschenko <
<EMAIL_ADDRESS>wrote:

Another comment wishing for merge and asking for status. @oxalorg
https://github.com/oxalorg: could you rebase your branch? I'll having
point my pipenv to your fork for now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rq/rq-scheduler/pull/212#issuecomment-609717643, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7774MBH2GWLIMED22JTJDRLGW3VANCNFSM4I7YSR7Q
.

Maybe, just maybe, the lock implementation should switch to redispy's Lock implementation, which uses Lua underneath to atomically extend lock's TTL.

Sorry it took so long for me to review this. Great PR!

One other thing, do you mind writing some docs about running multiple scheduler instances?

:tada: :tada: :champagne: :champagne: :wine_glass: :beers: :beer: :turkey: :cactus: :eggplant:

One other thing, do you mind writing some docs about running multiple scheduler instances?

Thank you @selwin for merging. I'll send in another PR soon with the updated docs. 😸


Merge development into base

Merge development branch into the base branch.

@helrond I tested this locally by creating a new branch based off origin and then merging development into that one. This should remove source and sample data. I've also downloaded a zip copy of base, so we have another copy of the original data we can zip up for Jose somewhere if we want.


System smoke tests on CI

Delivers https://www.pivotaltracker.com/story/show/159034542

I rebased on master and force-pushed. I had not included recent system tests fixes of master, hence CI had failed as expected.

Timing results on d03d52e:

https://circleci.com/gh/aeternity/epoch/28405#build-timing/containers/0
<EMAIL_ADDRESS>

No conflict with PR 1487 - merging.


"Create one" button to create .glsl editor does'nt do anything

I don't really know if this has it's place here but i'll try.
So i'm working with Ubuntu 18.04 and wanted to visualise my webgl canvas in vs code.
I've installed glslCanvas and reloaded my vs code and typed the command "Show glslCanvas" and it opens the window and says there's no active .glsl Editor. The problem is when i click on the button to create one it doesn't do anything. I've tried uninstalling and reinstalling the extension, closing vs code, even rebooting my computer but it doesn't do anything.
As i said i don't really know if this is the right place for that but i haven't found anywhere else.
Thank you for understing.

having the same issue on Windows, can't use it

@ErwanBueche @DJLink hi, running Show glslCanvas command should open a small fragment shader on a secondary panel. focusing that panel you should see this.

If you have problem please submit a more detailed use case.
system, vscode version, installed extensions, screenshot.
please run also this command from the commands palette. "Developer: Toggle Developer Tools"
and copy paste here error messages if found.

@ErwanBueche trying to debug on a fresh install of Ubuntu 18.04 in vmware and seems that vscode cannot find the opengl drivers.
libGL error: unable to load driver: vmwgfx_dri.so libGL error: driver pointer missing libGL error: failed to load driver: vmwgfx libGL error: unable to load driver: swrast_dri.so libGL error: failed to load driver: swrast
To double check try launch this command after Show glslCanvas  ->  Developer: Open Webview Developer Tools
you should see this error.
Uncaught TypeError: Cannot read property 'VERTEX_SHADER' of undefined
but maybe is only a vmware related problem.

@ErwanBueche  ok I did solve the webgl issue in Ubuntu removing vscode from the /snap folder and re installing it manually from the .deb file from here
sudo apt install ./code.deb
after this the extension work as usual.
if you continue to have issue check your OpenGL drivers running this command in terminal
glxinfo | grep OpenGL


The total rows number should plus1 after changing "Does the data contain column headers" from "Yes" to "No".

System Information (please complete the following information):
Windows OS: Windows-11-Enterprise-22H2
ML.Net Model Builder 2022: <IP_ADDRESS>1301 (Main Build)
Microsoft Visual Studio Enterprise: 2022(17.7.6)
.Net: 6.0, 7.0
Describe the bug

On which step of the process did you run into an issue:
The total rows number should plus1 after changing "Does the data contain column headers" from "Yes" to "No".

TestMatrix
https://testpass.blob.core.windows.net/test-pass-data/wikipedia-detox-250-line-data.tsv
To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio start window.
Choose the C# Console App (.NET Core) project template.
Add model builder by right click on the project.
Select any scenario.
On Data page, choose the data source.
Click the "Advanced data options..." link, go to the "Data formatting" tab and then change "Does the data contain column headers" from "Yes" to "No".
You will that the total rows number have no change.

Expected behavior
The total rows number should plus 1.
Screenshot

Additional context

The bug is still repro on the latest main build <IP_ADDRESS>5701.


[Feature Request] in search bar show properties in alphabetically order

When using the search bar, would it be possible to show the list of properties in alphabetical order?
Currently, it is listed in the order as it is stored in the .archimate file.

BTW, I really appreciate the feature of when selecting the property key(s), the Model Tree is showing the relevant elements on the fly. This is very powerful!

Seems a good idea. Will be in the next version.

Fixed in 4.4 beta


[Docs Site] Lint YAML files with Vale

Just a draft to see how good Vale is at linting YAML - it picked up a few issues here and there with the changelogs and glossaries when testing it locally.
cc @kodster28

https://github.com/KianNH/cloudflare-docs/pull/3#discussion_r1476223375


How would I render a swagger spec that is a String or JSON object?

Is there something like:
<div swagger-ui json="{{myObj}}" api-explorer="true"></div>

Hi!
This is not possible for now ...
If several people are interested it could be done in a further release

+1


Sitewide - Implement new Design Spec

Description
Implement new design spec for sitewide ui elements.
Required for Completion

[ ] Rebuild complete
[ ] Create pull request to dev
[ ] Peer review, accepted into dev
[ ] Pushed to live site

Clone framework added in #914


Changes in cachedAccessories killing complete Harmony structure

Hello,
today I removed an old entry (unused power outlet from plugin TplinkSmarthome) so that it will disappear in Apple HomeKit. But the problem is, that everytime I change anything in the file cachedAccessories, that I have to re-create alle automations and set all rooms for each Harmony Hub in HomeKit again.
Why is that?? This is really annoying, cause I have 3 hubs and a lot of automations.
Other plugins does not have that behaviour.

Hi, can you explain a bit more what you did exactly ? Purge cachedAccessories folder ?
If plugins don't rely on cache, but on a static "id", indeed they won't behave like this if you clean cache but plugins that behaves on cache should have the same behavior.
On the opposite, as far as I konw, you could have same problems with plugins that don't use cache if they can't publish the accessory for any reason (hub not available, device not reachable at startup) : device will be lost from home point of view and thus will be lost like automations.
I think the right way to deal with such things is that plugins which rely on cache should adapt themselves if any device is removed / added on the platform itself, which is I agree not always easy to do. In your case,, I think you should have to relaunch homebirdge with a correct config of tplink and the plugin should remove the old devices from cache.
However, I don't think there is a silver bullet for such problems, but f anyone as any idea ...

Hello, I just removed the part below for one device from cachedAccessories file and got the result what I wanted: That this device is gone. I do not complain about any other plugins, I report, that it seems that your Plugins seems to have big problems when one changes something of other plugins in cachedAccessories. All my Harmony icons and automations in HomeKit was mixxed up or stopped working. Why???
Here is the part I removed:
"plugin":"homebridge-tplink-smarthome","platform":"TplinkSmarthome","displayName":"Heizlüfter","UUID":"467435f4-2c2e-4e12-b5c4-daf1bb06d63a","category":7,"context":{"deviceId":"80060532CA0078FA341CCA86B33B408B19E2DA60"},"linkedServices":{"0000003E-0000-1000-8000-0026BB765291":[],"00000047-0000-1000-8000-0026BB765291":[]},"services":[{"UUID":"0000003E-0000-1000-8000-0026BB765291","characteristics":[{"displayName":"Identify","UUID":"00000014-0000-1000-8000-0026BB765291","props":{"format":"bool","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pw"]},"value":false,"eventOnlyCharacteristic":false},{"displayName":"Manufacturer","UUID":"00000020-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"TP-Link","eventOnlyCharacteristic":false},{"displayName":"Model","UUID":"00000021-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"HS110(EU)","eventOnlyCharacteristic":false},{"displayName":"Name","UUID":"00000023-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"Heizlüfter","eventOnlyCharacteristic":false},{"displayName":"Serial Number","UUID":"00000030-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"AC:84:C6:29:02:EA 80060532CA0078FA341CCA86B33B408B19E2DA60","eventOnlyCharacteristic":false},{"displayName":"Firmware Revision","UUID":"00000052-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"1.5.4 Build 180815 Rel.121440","eventOnlyCharacteristic":false},{"displayName":"Hardware Revision","UUID":"00000053-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"2.0","eventOnlyCharacteristic":false}]},{"displayName":"Heizlüfter","UUID":"00000047-0000-1000-8000-0026BB765291","characteristics":[{"displayName":"Name","UUID":"00000023-0000-1000-8000-0026BB765291","props":{"format":"string","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr"]},"value":"Heizlüfter","eventOnlyCharacteristic":false},{"displayName":"On","UUID":"00000025-0000-1000-8000-0026BB765291","props":{"format":"bool","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr","pw","ev"]},"value":false,"eventOnlyCharacteristic":false},{"displayName":"Outlet In Use","UUID":"00000026-0000-1000-8000-0026BB765291","props":{"format":"bool","unit":null,"minValue":null,"maxValue":null,"minStep":null,"perms":["pr","ev"]},"value":false,"eventOnlyCharacteristic":false},{"displayName":"Amperes","UUID":"E863F126-079E-48FF-8F27-9C2605A29F52","props":{"format":"float","unit":"A","minValue":0,"maxValue":65535,"minStep":0.01,"perms":["pr","ev"]},"value":0.02,"eventOnlyCharacteristic":false},{"displayName":"Total Consumption","UUID":"E863F10C-079E-48FF-8F27-9C2605A29F52","props":{"format":"float","unit":"kWh","minValue":0,"maxValue":65535,"minStep":0.001,"perms":["pr","ev"]},"value":66.529,"eventOnlyCharacteristic":false},{"displayName":"Apparent Power","UUID":"E863F110-079E-48FF-8F27-9C2605A29F52","props":{"format":"uint16","unit":"VA","minValue":0,"maxValue":65535,"minStep":1,"perms":["pr","ev"]},"value":6,"eventOnlyCharacteristic":false},{"displayName":"Volts","UUID":"E863F10A-079E-48FF-8F27-9C2605A29F52","props":{"format":"float","unit":"V","minValue":0,"maxValue":65535,"minStep":0.1,"perms":["pr","ev"]},"value":230,"eventOnlyCharacteristic":false},{"displayName":"Consumption","UUID":"E863F10D-079E-48FF-8F27-9C2605A29F52","props":{"format":"float","unit":"W","minValue":0,"maxValue":65535,"minStep":0.1,"perms":["pr","ev"]},"value":0,"eventOnlyCharacteristic":false}]}]}

"why ???" Don't know, especially if you don't share your conf at least first.
Moreover, this plugin is developed as it is, free, and I am doing my best as a hobby developper to support it. You are welcome to code / participate yourself through pull request, learn the not so easy structure of homekit / homebridge plugin, and if you are not pleased with it, choose another plugin .

I understand absolutely, and I don't mean something bad. But I just don't understand why removing something of another plugin effects the harmony plugin.
This is my config, is this OK?
	"platforms": [{ 		"platform": "HarmonyHubWebSocket", 		"name": "Wohnzimmer Fernseher", 		"hubIP": "<IP_ADDRESS>", 		"mainActivity": "Apple TV", 		"switchAccessories": true, 		"otherPlatforms": [{ 			"name": "Schlafzimmer Fernseher", 			"hubIP": "<IP_ADDRESS>", 			"mainActivity": "Apple TV", 			"switchAccessories": true 		}, { 			"name": "Badezimmer Fernseher", 			"hubIP": "<IP_ADDRESS>", 			"mainActivity": "Apple TV", 			"switchAccessories": true 		}] 	}

The config seems ok, don't know what happened, did you reproduce it or not ? I had / remove plugins quite often and don't have such problems, but like I said, I never play with the file in cached accessory, don't even know if it is a homebridge or plugin problem, or an error while editing the file. from my side, accessories are created from cache at startup if they exist as far as I know. If they were removed, you should have a log at startup sayig so in my plugin. what logs did you have after restarting homebridge (after deleting in the cache file, homebridge stoped I think).

I can't reproduce your problem. If you have any backup of your cachedAccessory file before and after the change, might take a look, but can't do more without .

Closing, can't reproduce. Since it was a "manual" and undocumented operation, not sure what happened, sorry.


feat(OakLessonInfoCard): create OakLessonInfoCard add to storybook

crreate OakLessonInfoCard
https://deploy-preview-94--lively-meringue-8ebd43.netlify.app/?path=/docs/components-integrated-oaklessoninfocard--docs
AC

[x]  OakLessonInfoCard - in Integrated components
[x]  test coverage
[x]  storybook
[x]  exported

Another question @benprotheroe:  the Implementation Details for the ticket imply we need to do the 'Static message' version of this component as well:
https://www.figma.com/file/G7z5ufo2T0CHk14CkadeuB/Pupils-high-fidelity-designs?type=design&node-id=4155%3A84044&mode=dev

Nice one @benprotheroe, just needs the 'static message' variant, and the ability to pass a link for worksheet downloads. E.g:


Add Grunt Token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Can't seem to understand the issue here. I am consistently getting a "no newline at end of file" issue

please remove the $ from the name, see https://github.com/solana-labs/token-list/pull/9307/checks?check_run_id=4577415003


onBottomClick

how to listen onBottomClick event
(when click player inventory)

I don't think it's natively possible unfortunately :(

Hey! I have great news for you.
In the latest release 1.5.5 this functionality is now supported 😄
Use it like this,
gui {
  onPlayerInventoryPickupItem = { player, item, slot -> player.sendMessage("Clicked at $slot") }
}

Enjoy!!


[SPARK-12780] Inconsistency returning value of ML python models' properties

https://issues.apache.org/jira/browse/SPARK-12780

Test build #49244 has started for PR 10724 at commit 993dc9c.

Test build #49244 has finished for PR 10724 at commit 993dc9c.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49244/
Test FAILed.

Merged build finished. Test FAILed.

test it please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49312/
Test FAILed.

Test build #49313 has started for PR 10724 at commit becc49c.

Test build #49313 has finished for PR 10724 at commit becc49c.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49313/
Test FAILed.

retest this please

Test build #49314 has started for PR 10724 at commit becc49c.

Test build #49314 has finished for PR 10724 at commit becc49c.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49314/
Test FAILed.

Merged build finished. Test FAILed.

Something wrong with Jenkins?

A PR broke the build, but it should work now.

The feature.py changes look fine to me.
@davies Could you please glance at the serialization change to confirm it's OK?  Thanks!

Test build #2379 has started for PR 10724 at commit becc49c.

Test build #2379 has finished for PR 10724 at commit becc49c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

@jkbradley @holdenk As I mentioned here https://github.com/apache/spark/pull/10186, we can work around the Ser/De issue with type conversion. So same solution here, I remove the Scala side code. But I still think there is a problem when ser/deing Scala Array. How about we merge this first then file another PR to fix it?

File a new JIRA https://issues.apache.org/jira/browse/SPARK-12834

Test build #49454 has started for PR 10724 at commit f5623b6.

Test build #49454 has finished for PR 10724 at commit f5623b6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49454/
Test PASSed.

@yinxusen I spoke with @davies and I think it would be best to use _call_java wherever possible for simplicity.  Otherwise, developers have to remember to put list() around the JVM methods they call.
Since you have created [https://github.com/apache/spark/pull/10772], let's use it to create the fix.  @davies said your original fix (in Scala in [https://github.com/yinxusen/spark/commit/c5a170da04f9941f4515f3263939bab9886c2c85]) seemed reasonable, so perhaps use that if your new one is not working.  After that gets merged, we can return to this and the other PRs.

retest it please

Test build #50111 has started for PR 10724 at commit c0cf68c.

Test build #50111 has finished for PR 10724 at commit c0cf68c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50111/
Test PASSed.

Merged build finished. Test PASSed.

Hi @jkbradley, this is the follow-up PR of https://github.com/apache/spark/pull/10772, shall we merge it?

LGTM
I'll merge this with master
Thanks!


[chain-id] Switch non-prod chain id values back

Summary

Use explicit numbers for all chain ids in the NamedChain enum
https://github.com/libra/libra/pull/5195 changed chain-ids of DEVNET and TESTING. This PR changes them back

We should avoid changing the chain-ids for non-prod environments as they can break the testing environments in various organizations.

@bors-libra r+

:pushpin: Commit 56e6a08 has been approved by davidiw

:hourglass: Testing commit 56e6a080c158db61f597dda0e45518373a02241d with merge e6c8da19bae83f80bb0b860e19f63712ea5df7b2...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing e6c8da19bae83f80bb0b860e19f63712ea5df7b2 to master...


Dec 566 add api end point for metadata configuration.

First pull request for just the api endpoint of the metadata configuration.

:+1:

:+1:


Refactor BranchNameConverter

The current converter/filter for creating branch names from GitHub issues needs an overhaul. Regex would maybe help here.
this could prove helpful: http://stackoverflow.com/questions/3651860/which-characters-are-illegal-within-a-branch-name

Have a look at https://github.com/Praqma/the-phlow/blob/master/perl/git-phlow#L776


Improve the accessibility score of the webapp

Describe the bug
Currently, the accessibility score of https://sql-editor-online.vercel.app/ is 96. It can be improved to 100%.
To Reproduce
Steps to reproduce the behavior:

Go to https://pagespeed.web.dev/ or Chrome dev tools Lighthouse report.
Measure https://sql-editor-online.vercel.app/ page score.
See the accessibility score and how it can be improved.

Expected behavior
Improve the accessibility score.
Screenshots

@Tajinder-Dhoot any updates?

@janvi01 I made some changes but not able to complete. I can work on it over the weekend

@janvi01 - Hello, please reassign this to someone. I haven't been able to work on this. My apologies.

What errors are you getting? @Tajinder-Dhoot

Can you please assign me this issue under JWOC?

Go ahead @Jenis07


A table shared with several groups doesn't appear in group members dashboard

During a stress test of group features one table didn't appear in users dashboard (although "shared with me" table count seemed to count it). After some investigation about it, it looks like it's related to the permission order inside the ACLs and shared entities.
access_control_list:
`[{"type":"group","id":"a18ab1f2-cf9a-4d56-a899-518a68943347","access":"rw"},{"type":"group","id":"8ab5f69e-de2c-405d-94d5-c11da11b4fba","access":"r"}]``
After unsharing and sharing again:
[{"type":"group","id":"8ab5f69e-de2c-405d-94d5-c11da11b4fba","access":"r"},{"type":"group","id":"a18ab1f2-cf9a-4d56-a899-518a68943347","access":"rw"}]
With the new ACL it works as expected.
Before fixing it we also checked shared_entities entries and it only had one entry, referring to the other group. So, it looks like only the first shared_entities entry for ACL list is being created.
cc @xavijam  @viddo

Merged into #4924 branch.


FillHandle very hard to use

You need surgical accuracy to fill down a column using the handle.
The cursor becomes a cross once the handle square is hovered anywhere, however actually dragging down the data only works if one hits the upper left corner of the square:

If you don't hit that specific spot the data will not fill downwards once the dragging is released.
This is the case in the storybook demo and a fresh install of the latest version.
Tested on Chrome Version 103.0.5060.66 (64-Bit) and Firefox 102.0 (54-Bit)
However the datagrid example on the official landing page does not have this bug / does not require the super high precision. So this might came with a certain version, depending on what version is displayed on the official landing page.

Ah woops wonder when that broke! Will fix ASAP.

Fixed in 5.0.0


Update Elemental and Kubewarden to use the most modern publishing practices

Dependent on https://github.com/rancher/dashboard/issues/12030
Currently Elemental and Kubewarden repos have their own dedicated workflow, which is not compliant with our latest practices of using reusable workflows.
We should update Elemental and Kubewarden to use the most modern publishing practices.

[x] Update Elemental https://github.com/rancher/elemental-ui/pull/227
[ ] Update Kubewarden

@jordojordo not a priority, but assigning to you since I've had to update Elemental already, so it's only up to Kubewarden to have this one closed 🙏

Will be accomplished with https://github.com/rancher/kubewarden-ui/pull/917