Using rem as unit in transforms

Draggable works just as I need, with one exception. The CSS transforms are done with pixels, but I need them to be rems, as my application uses them for scaling stuff down 1:1. Pretty much everything in my app with a pixel unit is converted with this little helper: (x) => ${x / 16}rem
Is it possible to change those without ditching the batteries and using DraggableCore? I took a quick peek at the source and looks like no, this requires some changes.

export function getTranslation({x, y}: ControlPosition, positionOffset: PositionOffsetControlPosition, unitSuffix: string): string {
  let translation = `translate(${x}${unitSuffix},${y}${unitSuffix})`;
  if (positionOffset) {
    const defaultX = `${(typeof positionOffset.x === 'string') ? positionOffset.x : positionOffset.x + unitSuffix}`;
    const defaultY = `${(typeof positionOffset.y === 'string') ? positionOffset.y : positionOffset.y + unitSuffix}`;
    translation = `translate(${defaultX}, ${defaultY})` + translation;
  }
  return translation;
}

Maybe with a customizer that takes the translation and params used to build it, and does whatever it wants with it? Not everyone is using 16 as the baseline value so it might be hard to support rems directly, so this might be an ok compromise.

export function getTranslation({x, y}: ControlPosition, positionOffset: PositionOffsetControlPosition, unitSuffix: string, customizer?: (translation: string, xy: ControlPosition, offset: PositionOffsetControlPosition, unitSuffix: string) => string): string {
  let translation = `translate(${x}${unitSuffix},${y}${unitSuffix})`;

  if (customizer) {
    return customizer(translation, { x, y }, positionOffset, unitSuffix)
  }

  if (positionOffset) {
    const defaultX = `${(typeof positionOffset.x === 'string') ? positionOffset.x : positionOffset.x + unitSuffix}`;
    const defaultY = `${(typeof positionOffset.y === 'string') ? positionOffset.y : positionOffset.y + unitSuffix}`;
    translation = `translate(${defaultX}, ${defaultY})` + translation;
  }
  return translation;
}

I haven't tested this, just wrote it inline. Is there an easier / better / already supported way?
I tried using the scale property, but either I'm using it wrong or it doesn't work in my case. ATM I have my application scaled to ~13px so everything on the "canvas" fits the screen. That means that items on the canvas (draggable) are 0.8102232667450059% of their full size, and I used that as the scale value. The item is supposed to be at the bottom of the canvas, but instead of that, it's outside, not visible. If I scale my app to 100%, it's where it's supposed to be.
If I manually convert the pixels in the transform to rem, everything works as expected. So I'd just like to use rem, if that's possible.
I don't know if my use case is somewhat special or not. It's a "design your own product" kind of app.  Instead of working top-left, my app works bottom-center, meaning [0,0] is dead center on the canvas bottom axis. That way I don't have to deal with repositioning everything when screen size changes, or calculate where the center is. If I want an element 200px on the left, I set x to -200. If I want it 50px from the bottom, I set y to 50. Don't know if that messes with the internal calculations of this component, but FYI.

Yeah - your use case makes sense, and I'd love to be able to support it with a stable API and without requiring forking. Doing so would require refactoring such that the major calculations (such as getTranslation itself) had a default implementation but could be overridden via props.
I don't really have a good story for this yet, because doing so would require some very deliberate API design; if we just let users override anything they like via props, we're sure to cause breakage, all the time, as we release future minor versions. I'm open to suggestion here.

Now, rather insanely enough, you could get around this by defining a child component that always translates its style attribute, whenever it is passed, to rem from px, like so:
class RemWrapper {
  render() {
    const child = React.Children.only(this.props.children);
    const style = {
      ...this.props.style,
      transform: transformCustomizer(this.props.style.transform),
    };
    return React.cloneElement(child, {style});
  }
}

Thanks for the reply.
If you think your fix was insane, just look at this.
For now, I can get by with this monster. I'd rather kill it, but I also gotta ship a demo this week.
document.querySelectorAll('.react-draggable').forEach(el => {
  if (el.style.transform) {
     const containsPx = el.style.transform.indexOf('px') > 0

     if (containsPx) {
       const [x, y] = el.style.transform.replace('translate(', '').replace(')', '').split(', ').map(pxd => pxd.replace('px', '')).map(px => parseInt(px, 10) / 16).map(x => `${x}rem`)
       const newTransform = `translate(${x}, ${y})`

       el.style.transform = newTransform
       console.log(el)
     }
  }
})

Fair enough. I'm not against the customizer in principle, but I believe there are more things that users will want to customize than just this - and if we do it, I would like a unified API that I'm prepared to not break between releases.

Yes, I agree. I just have my tunnel vision goggles on as a dumb user :wink:
I'm OK with any reasonable solution, if RemWrapper works, that's perfect for me. I'll close the issue if it does, and maybe send a little PR documenting it's use?

Sounds good, thanks.
I like your personal site btw - nice work.

Looks like it works. I haven't done extensive testing yet but at least things render in the right place, and dragging seems to work! It even scales down beautifully when I resize the window.
I had to ditch TS because it wouldn't stop bugging me about this.

TS2605: JSX element type 'RemWrapper' is not a constructor function for JSX elements.
Types of property 'render' are incompatible.
Type '({ children, style: styles }: { children: any; style: any; }) => DetailedReactHTMLElement<{ style: any; }, HTMLElement>' is not assignable to type '() => ReactNode'.

I also had to make some changes to RemWrapper to get it to work. I know you probably wrote it as pseudo. I have no idea why this works, but it does.
import React from 'react'
import PropTypes from 'prop-types'

class RemWrapper extends React.Component {
  translateTransformToRem(transform, remBaseline = 16) {
    const convertedValues = transform.replace('translate(', '').replace(')', '')
      .split(',')
      .map(px => px.replace('px', ''))
      .map(px => parseInt(px, 10) / remBaseline)
      .map(x => `${x}rem`)
    const [x, y] = convertedValues

    return `translate(${x}, ${y})`
  }

  render() {
    const { children, remBaseline = 16, style } = this.props
    const child = React.Children.only(children)

    const editedStyle = {
      ...child.props.style,
      ...style,
      transform: this.translateTransformToRem(style.transform, remBaseline),
    }

    return React.cloneElement(child, {
       ...child.props,
       ...this.props,
       style: editedStyle
    })
  }
}

RemWrapper.propTypes = {
  style: PropTypes.shape({
    transform: PropTypes.string.isRequired
  }),
  children: PropTypes.node.isRequired,
  remBaseline: PropTypes.number,
}

export default RemWrapper

If that looks ok to you, I can add a section to the README with an example usage. It assumes that the transform is always translate(x, y), and will break if it encounters anything else. It's fine for me, but it may have some unintended consequences for someone else.
Also thanks, you're like the 20th person to visit my site! I created that about a year ago just before I became a full-time freelancer, and didn't have anything to do for about 3 weeks. I haven't been able to touch it after that...
It's an interesting combination. Nginx routes known bot useragents to a "static" Puppeteer-rendered version of the site so it's indexable, and if I wanted to, I could switch that on for every user for a SSRd first load. I'm going to do something like that in one of my bigger projects.

Thanks, I think that's a helpful idea.


Fixes #3074 by describing the model in AppsResource_create, change th…

…e default example and define the schema correctly

LGTM – though that MarathonSchedulerActorTest is failing a lot, recently.


Upgrade graphql-parser to v0.2.2

There is no breaking change.
v0.2.0...v0.2.2

Just curious, was the default of ^ used by cargo not sufficient to get a newer one in a local Cargo.lock?

Indeed, I guess in this case we could have kept a lower required version.


Fix mapset() not working for

Fix #12697

Can you please update and rebase this PR to the latest?


feat: upgrade to middy v2.x

What did you implement:
Middy recently upgraded to v2, with some breaking changes. This PR brings us up to date with Middy.
Todos:

[x] Write tests
[x] Fix linting errors
[x] Make sure code coverage hasn't dropped
[x] Enable "Allow edits from maintainers" for this PR
[x] Update the messages below

Is this ready for review?: YES
Is it a breaking change?: YES

Nice, I was also looking into this, since I was thinking about changing our typing when I saw the middy update.
The problem I saw is this update is actually a breaking change:

It breaks our typescript types- though this can be easily fixed and isn't a major blocker
Our pattern- packages we return the middy object and since middy has breaking changes it most likely will pass down.
Middy no longer supports node.js v10 but lambda and our CI still do

@lkhari good shout, we should mark this as a breaking change instead.
Lambda is starting the deprecation process for node 10.x from July so it shouldn't be a big issue, but middlewares that worked with it previously would need to upgrade to the middy 2.0 interface (e.g. no more next)

@lkhari @gsingh1 I've updated the PR to mark the PR as breaking, so hopefully lerna would see as breaking change and publish a new major version. I've removed test-node10 from the CI config as well since middy has dropped support for Node.10 and Lambda is deprecating it soon as well.

Seems good, to me, I've made a PR for this feature branch that contains typings to reflect more closely with what middy has written. I've exposed Contexts as one of the generics since a few of our patterns put the data in the context object.
But I'm not a maintainer so I'll leave that up to you @gsingh1 ! 😄

This is awesome, thanks @theburningmonk. We should probably add a node14 in the CI. It would be good to add a note in the readme about the node runtimes that we support too.

ci/circleci: test-node10 hasn't completed but it looks like middy 2.0 dropped support for node 10 https://middy.js.org/docs/UPGRADE.html

@demsey2 I disabled node10 in this PR, which is why the node10 is not running, but looks like the checks is not respecting that, maybe it needs to be configured elsewhere?
@gsingh1 @simontabor do you guys know?

There is another problem, I am testing that and it looks like MiddlewareObj interface changed the signature, it was
 MiddlewareObject<T, R, C extends Context = Context>

and now we have
MiddlewareObj<TEvent = any, TResult = any, TErr = Error>

not sure why they did remove the Context but that is probably a question we should ask them

@demsey2 if you refer to middy's documentation there's some examples of what the middleware accepts now - essentially just a request object, no more next.
As @lkhari noted, this affects anyone who writes custom middlewares and use them with the built-in patterns, which is why I made a dummy commit to make this PR a breaking change.

@theburningmonk the problem is that definition,
export default function <T, R, C extends Context = Context>(params: {
  sampleDebugLogRate: number;
}): middy.MiddlewareObj<T, R, C>;

it should be
export default function <T, R, E extends Error>(params: {
  sampleDebugLogRate: number;
}): middy.MiddlewareObj<T, R, E>;

because this is the definition of middy.middlewareObj
MiddlewareObj<TEvent = any, TResult = any, TErr = Error>

they have removed Context and now they always use LambdaContext
this is the previous version
MiddlewareObject<T, R, C extends Context = Context>

Whoops. Yea @demsey2  is correct. I must have just updated the middy object name, but missed out the correct generics there. I was probably more focused on exposing the context for our other stuff.
If you change packages/lambda-powertools-middleware-correlation-ids/index.d.ts lines 15 -17 to look like below, then it should fix the issue:
export default function <TEvent = any, TResult = any, TErr = Error>(params: {
  sampleDebugLogRate: number;
}): middy.MiddlewareObj<TEvent, TResult, TErr>;

@theburningmonk I've disabled the required check for node:10, so you're all green now. I'll add node:14 as required as soon as it's added too

thanks @simontabor and @lkhari!


智能合约中能否使用  regex

#include< regex >
regex_match
编译通过，但是部署时提示如下错误
Exception Details: 3070000 wasm_exception: WASM Exception
env.__ctype_get_mb_cur_max unresolveable
{"module":"env","export":"__ctype_get_mb_cur_max"}
thread-0  wasm_interface.hpp:42 resolve
{"mod_name":"env","export_name":"__ctype_get_mb_cur_max"}
thread-0  wasm_interface.hpp:44 resolve

pending console output:
{"console":""}
thread-0  apply_context.cpp:61 exec_one

Try prefixing your eosio-cpp command line with export LC_ALL="C";.  If you are still using eosiocpp, you should switch to eosio-cpp, from https://github.com/EOSIO/eosio.cdt.

Per EOSIO/eosio.cdt#125, regex in smart contracts is not supported and not recommended because of its dependency on std::locale.


Error: The type "PromiseConstructor" is not callable.

https://hegel.js.org/try#MYewdgzgLgBADjAvDMBTA7jACgJxAWwEsJUAKASiA

there is also Property "then" does not exist in "async () => Promise<true>"
https://hegel.js.org/try#IYZwngdgxgBAZgV2gFwJYHsIxACgJQwDeAvjAFBkgB0yAFgKYQ6owC8AfEWTDMADb0ATsmZ4yxPEA
but I believe this is related


Overriding Notification Message Text in Plugin

This more of a "how-to" question than an issue, but I couldn't find any other forum where I could ask this question.
I am writing a plugin for Rainloop to change the password via Postfixadmin. I have got everything working except the Notifications. When the password doesn't fit Postfixadmin's requirements, I want to display something like "Password must have at least 2 numbers" but the only option is "Password is too weak". I am trying to find a way to display additional information but nothing works.
RainLoop version, browser, OS:
v 1.17.0
Expected behavior and actual behavior:
I put a string (e.g. "Password needs at least 2 numbers" in the $sAdditionalMessage parameter of the "ClientException" but it doesn't show in the browser.
Steps to reproduce the problem:
Logs or screenshots:

You need to modify https://github.com/RainLoop/rainloop-webmail/blob/master/dev/Settings/User/ChangePassword.js


fix(jest-haste-map): don't throw on missing mapper in Node crawler

Summary
The mapper option passed to HasteMap is not supported in Node crawler, so we shouldn't pass it there. It's likely a cause of a regression in React Native 0.60 RC: https://github.com/facebook/react-native/pull/25241#issuecomment-501442208.
The regression was a part of this refactoring: https://github.com/facebook/jest/pull/8056/files#diff-5c5055968e3661a18f280d5e3f10c969L738
Test plan
Added extra assertion to validate that mapper is not passed to Node crawler.
cc @scotthovestadt @arcanis

Codecov Report

Merging #8558 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8558      +/-   ##
==========================================
+ Coverage   60.57%   60.58%   +<.01%
==========================================
  Files         269      269
  Lines       11054    11056       +2
  Branches     2696     2697       +1
==========================================
+ Hits         6696     6698       +2
  Misses       3772     3772
  Partials      586      586

Impacted Files
Coverage Δ

packages/jest-haste-map/src/index.ts
81.37% <100%> (+0.1%)
: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 3748557...ac5a94e. Read the comment docs.

what's the interplay between this and the referenced metro bug 418 linked above? It just bit me and I'm trying to figure out if this is an unreleased change that needs to be incorporated in a release of jest here, then referenced in metro, or if it's already released here and metro just needs to update? @thymikee you merged this, is it released? @motiz88 this seems needed in metro but I'm not even sure how to propose it...

Update jest-haste-map in your lock file

I don't like messing around in lock files :-), but I'm okay with patch-package
jest-haste-map+24.8.0.patch.txt
It will tell me when it's released, and I can build for now (if anyone else needs this, just install patch-package, make a patches folder and drop this in there without the .txt extension (that was needed so github would accept the attachment) and add patch-package as a postinstall hook
The question of how to get it integrated is open though ? with a patch I can move alone for now though

Just remove it from lock file and yarn install again :p

Ah! I see what you mean. Would it better the same to say that you could just delete the whole lock file and it works, or do you have to install, then delete just jest haste map from the lock file, and install again?

Deleting just the jesthaste-map` entry should be enough, yarn/npm will see it's missing and download the latest version (which contains this fix). You can of course throw away the entire lockfile if you want, but it's not necessary

Metro should probably update, though?

Opened up https://github.com/facebook/metro/pull/432


As a room creator, I want to click home to and edit in the room creation process in order to go back to home or edit my room.

Is your feature request related to a problem? Please describe.
Room creator uses the back button to fix an error, which is a bad user experience.

Describe the solution you'd like
When the room creator clicks "create party" to go to the room creation screen, inside of the /create view they can click a "home" button rather than a "back" button that currently exist. It would be helpful if there was a tooltip at this point, pointing out that all information will be deleted when hovering over "home" - but i'll leave that up to the devs to decide.
Subsequently, in the /create screen, when the room creator is reviewing and wants to go back to the "create party" screen - they should click an "edit" button where the current "back" button is.

Describe alternatives you've considered
none really here

Additional context
other stickies related to this one:

"user is forced to restart their election creation everytime they click 'Back'"
"user expects to go back edit from review when they click 'back'"
"user did not see the edit button"
"Create Party: having to restart a party (after being made) was annoying (not sure if she was using version with review page before). She liked what we did with tip.party (people responsible for signing themselves in)"
"Review Party: Found edit button (but only after she hit back and started up the party again)"
"Review Party: When we asked her to add a missing address to the list, she chose to hit Back instead of Edit. Clicking back exited her out of making the room to the "All Room" page and frustrated her that it didn't take her back to the Create Party page."

I'm working on this.

@hmrtn I've made a pull request to add the above feature request. Please review it. Thanks.

This ticket is completed, yes?

Yes, the PR with the requested changes has been merged.


Develop a Spring Boot Web Application Which Support Login by Microsoft Work or School Account

Introduction of Spring Security and Microsoft Work or School Account
Develop a Spring Boot Web Application Which Support Login by Microsoft Work or School Account
2.1. Create Application by Spring Initializer
2.2. Modify the Application
2.3. Run Application and Validate Result
Reference

what's this?

and the priority feild is not supposed to be set

https://github.com/microsoft/spring-cloud-azure/discussions/categories/blog-proposal


useCustomer and other hooks return undefined even though api returns data

I'm having a problem using storefront hooks. After successful SSO login and having all the api endpoints on NextJS application for BigCommerce in place, the hooks return undefined result on the first page load, even though if I look into the Network tab of the browser, I can see SWR does successful requests to /api/bigcommerce/customers and this endpoint returns logged in user data.
Whats interesting, if I update a code and NextJS running in development mode will refresh the page in browser automatically, the hook returns data as expected. But then if I refresh the page manually again, it returns undefined.
const { data: customer } = useCustomer(); // It gets invoked in the component at least 2 times.
console.log(customer); // <--- is undefined on first page load, and returns data only if NextJS
                       // refreshes the page automatically after code change.

On the first call of useCustomer I can see api request in the Network tab to the /api/bigcommerce/customers endpoint, which returns expected data.
I have also tried to revalidate the cache, but it also doesn't help.
const { data: customer, revalidate } = useCustomer();

revalidate().then(() => {
  console.log(customer);
});

The same problem with useCart hook.
It looks like hooks don't revalidate SWR cache properly. Does anyone experience the same issue? I have already read all issues/topics and debugged everything as much as possible.

This has nothing to do with storefront-data-hooks, that appears to be the same behavior for all useSWR hooks I have. Closing this issue.

Nope, wrong again :)
My other useSWR hooks work in all components. Custom implemented useCustomer also works. But whenever I try to use useCustomer from storefront-data-hooks - it always returns undefined.....

I noticed that swr in storefront-data-hooks is very outdated - 0.3.6, which has been released more than 2 years ago.
The same applies to the https://github.com/storyblok/nextjs-bigcommerce-starter template, which is using the older swr and nextjs 10. Could it be because of version incompatibility? I'm using the latest next and swr in my project.


I hope to move cnfcluster to China region, what suggestions?

How to modify this file “cfncluster/cloudformation/cfncluster.cfn.json”
Any  suggestions?

Hi,
Sorry for the late reply. We added support for China regions cn-north-1 and cn-northwest-1 as part of the last release of AWS ParallelCluster (2.1.1).
Francesco


Fix tautulli so it can see plex logs

Currently tautulli can't see plex logs, reducing its usefulness. This PR maps plex logs to a static directory and lets tautulli see them.

@davestephens Tested in VM and working - ready to merge


Question: missing line-break when using XSLT identity transformation

It seems that I set a wrong configuration ...
I have a simple "XSLT identity transformation", and using it with xsl.exe it works fine and keeps all line-feeds.
When I use it with XmlNotepad, it transform the file too, but removes all line-feeds. I have to reformat it with xmlint.exe.
I changed all my settings for XmNotepad to "Default" - no solution either.
How can I keep the line-feeds?

Hmm, this one is a bit tricky, if I turn on "Preserve Whitespace" in the XML Document then you will see this:

I assume you do not want to see whitespace like this, or do you?  In order to have full control over whitespace in the document (and see it in the XSLT transform) you kind of need this, but it's pretty tedious...?
The alternative (1) is I hide all these nodes in the XML notepad UI, but I load the whitespace nodes into the document so XSLT transforms can see the whitespace.
Alternative (2) which saves memory, would be to not load any whitespace into the document, but before I run the XSLT transform I save the document with pretty printing turned on (which xml notepad save is doing already) and then I feed that saved document with whitespace to the XSLT transform.  The downside of this is that for very large documents this could add a lot of time before you see the XSLT output due to having to save and reload that doc into the transform.
What is your preference?

I ended up adding a new Option under formatting called "Preserve Whitepsace" so you can play with that and see what you think, after turning it on you will need to reload your XML doc and you will see the above #whitespace nodes.

Hi Chris
I'm not sure we are talking about the same topic.
I have this XML in UTF-8:
<?xml version="1.0" encoding="utf-8"?>
<DSS_2015_LV95.Siedlungsentwaesserung.Einzugsgebiet TID="12684">
  <OBJ_ID>12684</OBJ_ID>
  <Metaattribute>
    <SIA405_Base_LV95.Metaattribute>
      <Letzte_Aenderung>20211124</Letzte_Aenderung>
    </SIA405_Base_LV95.Metaattribute>
  </Metaattribute>

</DSS_2015_LV95.Siedlungsentwaesserung.Einzugsgebiet>

This XSLT (ident copy):
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
  <xsl:template match="@*|node()">
    <xsl:copy >
      <xsl:apply-templates select="@*|node()"/>
    </xsl:copy>
  </xsl:template>
</xsl:stylesheet>

With your xsl.exe 1.0 from your XMLTools 2023-01-27 I use this command:
xsl -s ident_kopie.xslt mini.xml
and get this result:
<?xml version="1.0" encoding="utf-8"?><DSS_2015_LV95.Siedlungsentwaesserung.Einzugsgebiet TID="12684">
  <OBJ_ID>12684</OBJ_ID>
  <Metaattribute>
    <SIA405_Base_LV95.Metaattribute>
      <Letzte_Aenderung>20211124</Letzte_Aenderung>
    </SIA405_Base_LV95.Metaattribute>
  </Metaattribute>

</DSS_2015_LV95.Siedlungsentwaesserung.Einzugsgebiet>

Using the same in XMLNotepad I get this result - with missing line-breaks:
<?xml version="1.0" encoding="utf-8"?><DSS_2015_LV95.Siedlungsentwaesserung.Einzugsgebiet TID="12684"><OBJ_ID>12684</OBJ_ID><Metaattribute><SIA405_Base_LV95.Metaattribute><Letzte_Aenderung>20211124</Letzte_Aenderung></SIA405_Base_LV95.Metaattribute></Metaattribute></DSS_2015_LV95.Siedlungsentwaesserung.Einzugsgebiet>

Please re-read my answers, this is about "preserving whitespace" in the XML Document, which XML notepad does not do by default.  Try my new Preserve Whitespace setting and you will get the XSL output you are looking for.

New Preserve Whitespace formatting option is available in https://github.com/microsoft/XmlNotepad/releases/tag/<IP_ADDRESS>

Thanks, works as expected.


[WIP] Ci job smoke test #420

Created a Github Action workflow - local-up-kpng.yaml - for local-up-kpng.sh.
Solves #420

Thank you sir !!!!
/lgtm
/approve


New network type (L2) - Internal

Internal PR for New network type (L2)
JIRA Ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-10102
Feature Specification: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=74680920
Introduction
This feature allows CloudStack administrators to create layer 2 networks on CloudStack. As these networks are purely layer 2, they don't require IP addresses or Virtual Router, only VLAN is necessary (provided by administrator or assigned by CloudStack). Also, network services should be handled externally, e.g. DNS, DHCP, as they are not provided by L2 networks.
As a consequence, a new Guest Network type is created within CloudStack: L2
Description

Network offerings and networks support new guest type: L2.
L2 Network offering creation allows administrator to select Specify VLAN or let CloudStack assign it dynamically.
L2 Network creation allows administrator to specify VLAN tag (if network offerings allows it) or simply create network.
VM deployments on L2 networks:

VMs should not IP addresses or any network service
No Virtual Router deployed on network
If Specify VLAN = true for network offering, network gets implemented using a dynamically assigned VLAN

UI changes
A new button is added on Networks tab, available for admins, to allow L2 networks creation

Added functional fixes for test cases reported by @borisstoyanov
Ping for review @DaanHoogland @rhtyd

Thanks @nvazquez looks good, however, I'll continue with my review on the upstream PR.

Thanks @rhtyd @borisstoyanov. No problem @borisstoyanov, I'll close this one


How to Use Cart ?

im getting this error on Laravel 7

Hi, @maulayyacyber
Enhance docs in this pr https://github.com/melihovv/laravel-shopping-cart/pull/12
Check it out


OpenBuddy models

https://scisharp.github.io/LLamaSharp/0.4/GetStarted/
It is mentioned here that the OpenBuddy models are fine with the software
but...
In practice it doesn't work, it should work when the base of the software is updated, but until this update is done, it still doesn't work with these models
I mean the llama.cpp has been updated to work with these models
But it is not updated here anymore.

Could you please have a try with the current master branch? I've updated the native libraries of windows and linux with corresponding changes in C#.

I will check, thank you very much

LLama.Exceptions.RuntimeError: 'Failed to load model W:\Models\llam\Uncensored\openbuddy-llama-ggml-7b-v4-q5_K.bin.enc.'
this is what i get

Karen-The-Editor.ggmlv3.q5_K_M.bin
I have now tried it, and it does work nicely, thanks for the update
[Although what are the differences? I haven't found out yet]

The difference is the native library file and some low-level APIs in C#. :)


docs: fix doc as spot instances are implemented

Description
Fix the documentation as spot instances are no longer planned but implemented.
Verification
None
Checklist

[ ] My code follows the style guidelines of the project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] I have used pre-commit hook to update the Terraform documentation

:tada: This PR is included in version 1.9.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[v6 rc5] Changing Field props does not re-render

Example:
@reduxForm({ name: 'myForm' })
class Foo {
    state = { highlighted: 0 }

    render () {
         return

            <button onClick={() => this.setState({highlighted: Math.random()})}>Click Me!</button>
            <Field name='bar' highlighted={this.state.highlighted} component={renderField} />

    }
}

const renderField = ({input, highlighted}) =>

{highlighted}

Clicking on the button changes the component's (Foo) state, re-renders's Foo, passes the updated custom props (highlighted) to the Field, but Field does not re-render
In the current implementation it makes it very hard to work with custom props as they don't force a re-render and thus aren't as useful as they can be.
A workaround is to perform these operations on the store itself, but that doesn't make sense for state that might be shared by multiple componenets

There is a unit test that shows that this is working. I've updated it (see commit above) to be more like your example. I wonder what is different in your scenario?

ok that is odd, re-trying the example and it works for me now as well :|
Thanks for the reply! I'll close this and re-open if I can reproduce it again.


Slimbot support commands ?

Sorry my english not well.
Does slimbot support commands feature ?
I can't find that in slimbot documents.

Hi @phuongnguyenkim, Slimbot can respond to commands as well. You need to first set up commands for your bot with @BotFather using /setcommand. Once that is done, just listen for it with your bot. Do you need an example for this?

@edisonchee please, can you clarify: I set command via /setcommand in @Botfather
How I can check command calls in bot? For now I parse command
slimbot.on('message', message => {
  const currentCommand = commandManager.parseCommand(message.text);
switch (currentCommand) {
        case 'help':
            console.log('help')
            bot.slimbot.sendMessage(currentChatId, 'Help');
            break;
}
});

commandManager.parseCommand = (text) => {
    if (text.charAt(0) === '/') {
        if(text.indexOf(' ') === -1){
            return text.substr(1, text.length);
        }else{
            return text.substr(1, text.indexOf(' '));
        }
    } else {
        return false;
    }
};

Have a look at https://core.telegram.org/bots/api#message and see what kind of information you can find about bot commands in the field entities. That makes parsing a lot easier.
Remember that a command can be of the form /help or /help@botname

@KnorpelSenf Thanks for helping to answer this!
@phuongnguyenkim @lgg If your questions are answered perhaps you can help to close the thread, or maybe open another issue. :)


Slow Action Server Response

I recently ordered a turtlebot4 standard for my research, which has an create 3 base. I am trying to drive my turtlebot with the action server using the instructions in the documentation. Specifically, I call this command in my CLI:
ros2 action send_goal /drive_distance irobot_create_msgs/action/DriveDistance \
"distance: 0.5
max_translation_speed: 0.3"

The prompt shows: Waiting for an action server to become available...
And then it takes 20~30 seconds before I actually get a response from the robot and it starts to move.
This delay is way too long for my application to bear and I wonder how to deal with this issue?
Here is some additional information about my setup and debugging process:

I am using a WLAN network where only one turtlebot is present.
I am using ROS2 Humble and the latest firmware from the irobot create3 website: H.2.3
My operating system is Ubuntu 22.04 and all software packages have been updated to the latest version.
I have sync the clock on my PC and Rasp Pi and icreate 3 using NTP. (But I still see user.notice ntpd: ntpd: reply from [<IP_ADDRESS>](http://<IP_ADDRESS>/): delay 0.004474 is too high, ignoring in the create 3 log. Not sure how to fix it...)
I tried both CLI and rclpy action client. The delays are similar, both 20~30 seconds.
I log the time at which the goal request is sent and the create3 receives the goal message. The delay between these two is around 20~30 seconds. After the goal message is received, the robot starts actuating quickly.
I tried the other action servers available on create 3 and all of them suffer from this delay issue.
I wrote an action server myself and ran it on a Rasp Pi in the same network and interacted with it using the same way. The delay is negligible.
I monitor the WiFi traffic and notice that create 3 has a lot of uplink data (0.6Mbps) and fewer downlink data (3 kbps) from the create3 robot. This throughput should not saturate the channel and cause the delay issue.

I have searched on the internet but could not find a similar issue. Any help would be appreciated
Thank you so much for your help!

@LuHaofan, Disclaimer:  I am just a Create3 owner.   What follows is my personal opinion.
Haofan, you have "discovered" a secret:  There are many things the Create3 performs at 62 to 100 times per second, and a few things on the order of seconds per request, efficiently interleaved by a processor slightly more powerful than what might  be considered a micro-controller operating at 70 to 80% and sometimes even 100% CPU at 70% of 59MB available memory.
The driving action servers are useful for initial learning in ROS 2, but have some disadvantages for complex robot environments, such as:

The IR sensors will fail to "see black obstacles" - (affects /wall_follow action server most)
The action servers will ignore /cmd_vel messages coming from an external processor that is more aware of the environment

The "slow initiation" of the Create3 native action servers is likely not something that will (or can) be addressed by the iRobot team.  If the action servers are useful, great, if not the distributed nature of ROS allows fast, external action servers tailored to users' needs.


replace macos-10.15 with macos-12

This PR:

replaces macos-10.15 with macos-12 in the GH workflows, due to deprecation. See https://github.com/actions/runner-images/issues/5583

Signed-off-by: Thomas Sjögren<EMAIL_ADDRESS>

hm. I should have done this earlier, as I did  for molecule-vagrant months ago. Thanks for doing it :)


Enable logging bugs via Github instead of via "App Center in-product support experience".

I am trying to log a bug but apparently you have to do this via App Center now? The "Open" button opens a new browser window with app center, there is a button for "Submit feature request" which brings me back to this page, but where is the button to log a bug?

+1
Diverting these conversations away hides information from users of AppCenter.
I would also suggest posting questions to Stackoverflow and only posting reproducable bugs here, but its hard to get the general public to adhere to that.

Thanks for the feedback! I definitely see the value in having support threads on GitHub or Stack Overflow, especially for common issues.
Our current support model enables our team to work closely with users and provides a private space for users to share more sensitive information with us like stack traces, app info, user emails etc. However, we are continuously exploring ways to improve our customer support experience and processes so please continue to share feedback with us.
Thanks!

I just shared my feedback and I still don't know how to submit a bug for app center, so I logged a feature request instead... The current setup has a dead end. Why not simply use github the way it is supposed to? You can always invite users for chat in app center when necessary or use other means of communication.

Hey Bot, we still want this fixed.

Is app center being abandoned? With so many issues going to auto-close I afraid to bring
My clients to App Center.

Still needed...


How to save direct in s3 Bucket?

How to save direct in S3 Bucket, using Storage?

FFMpeg::fromDisk('input_disk')
->open('input')
->addFilter(function ($filters) {
$filters->clip();
})
->export()
->toDisk('output_disk')
->inFormat(new \FFMpeg\Format\Video\X264())
->save('output');

This would require saving it locally first and then uploading it to the cloud. .)


Install split meryl tools

These will need TPV rules and the container won't be built until the next planemo-monitor run.
Installation sequence for tool-installers

[ ] Test using @galaxybot test this
[ ] Inspect CI output for expected changes
[ ] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

Just FYI you can trigger the monitor runs at any point at https://github.com/galaxyproject/planemo-monitor/actions


[Feature Request] Forward/Backwards Daily Notes buttons

I'm currently using the buttons plugin to create these forward/backwards buttons in my daily notes but it looks very clumsy and I think it would be much better if it was integrated with your feature suite. Here's a screenshot from Clover, which is the best UI pattern I can think of for this:

Although the forward backward arrows in clover are for weeks and you can select the day from the persistent week view bar. I would be happy to see either my suggestion or clovers implementation! Thanks for the awesome plugin :)

The Timeline complication already has a weekly inline calendar when you click it.
I haven't used Clover, but I think the one in the weekly calendar might behave in a similar way?
Though there are currently no dedicated "forward/backwards buttons" in there. So I'm sticking with custom keybinds for the Periodic Notes: Open (previous|next) daily note commands for now...


CLI Ergonomics

See #7

coverage tests fails because of CLI boilerplate that will be covered later in integration tests.


Improve js builds

Status
READY
Description

Separate webpack configs for dev and prod
Remove react hot module reloading (will add when React Hot Loader 3 is released)

Todos

[ ] Tests
[ ] Documentation
[ ] License

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
git fetch --all
git checkout <feature_branch>
npm test

@boundlessgeo/spatial-connect

Maybe we should change name of web/webpack.dev.js to  web/webpack.local.js?  Otherwise I think it could be confusing b/c the dev environment refers to the remote development environment....so building with npm run build:dev would be not be using the web/webpack.dev.js and if we rename, it's explicit

yes that makes sense

👍


feature: Generate Code for resource adoption by annotation

Issue #, if available:
Description of changes:
These changes introduce a new generated function in all controllers
that attempts to populate the resource spec/status with fields defined
by the user.
This change will require for developers to add hooks for this function to
all controllers that already have a SetResourceIdentifiers hook.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest

/retest

/retest

/retest

/retest


[ci:component:github.com/gardener/apiserver-proxy:v0.6.0->v0.8.0]

Release Notes:
Updated base image of apiserver-proxy to alpine 3.16.2

The golang version is updated to `1.19.2`.

The following dependencies are updated:
- k8s.io/api: v0.19.2 -> v0.23.5
- k8s.io/apimachinery: v0.19.2 -> v0.23.5
- k8s.io/apiserver: v0.19.2 -> v0.23.5
- k8s.io/client-go: v0.19.2 -> v0.23.5
- sigs.k8s.io/controller-runtime: v0.7.0-alpha.4 -> v0.11.2

An issue causing the `apiserver-proxy-pod-webhook` to wrongly remove the `grpc` field from `livenessProbe`s, `readinessProbe`s and `startupProbe`s when defaulting a Pod is now fixed.

Native arm64 builds (builds on arm hosts) are now supported. Previously only arm64 builds with qemu were supported.

`bazel` is no longer used for builds ands tests. As alternative a Makefile with equivalent targets is now provided.

/kind enhancement

managed resource garden/nginx-ingress is not does not seem to be related to this change => Retry test.
/test pull-gardener-e2e-kind


Error buildind the app

Hi all,
today I tried to build the app in my ubuntu 14.04 64bit but with errors:
steps:

git clone https://github.com/shprink/wordpress-hybrid-client.git

cd wordpress-hybrid-client

npm install

npm run installWPHC

npm run devserver

up this point, all works fine, and I can see the default julien's blog
now when I try to build for android... first error

ionic state restore

You can see the log here
http://pastebin.com/XUCvhmgh
...so I tried to uninstall the cordova-plugin-google-analytics an reinstall by hand all plugins presents in the previous version. Here the list
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ cordova plugin list
com.ionic.keyboard 1.0.4 "Keyboard"
com.telerik.plugins.nativepagetransitions 0.5.7 "Native Page Transitions"
cordova-plugin-console 1.0.2 "Console"
cordova-plugin-crosswalk-webview 1.4.0 "Crosswalk WebView Engine"
cordova-plugin-device 1.1.0 "Device"
cordova-plugin-dialogs 1.2.0 "Notification"
cordova-plugin-file 3.0.0 "File"
cordova-plugin-file-transfer 1.4.0 "File Transfer"
cordova-plugin-globalization 1.0.2 "Globalization"
cordova-plugin-google-analytics 0.8.1 "Google Universal Analytics Plugin"
cordova-plugin-googleplayservices 19.0.3 "Google Play Services for Android"
cordova-plugin-inappbrowser 1.1.1 "InAppBrowser"
cordova-plugin-network-information 1.1.0 "Network Information"
cordova-plugin-splashscreen 3.0.0 "Splashscreen"
cordova-plugin-statusbar 2.0.0 "StatusBar"
cordova-plugin-velda-devicefeedback 0.0.2 "DeviceFeedback"
cordova-plugin-whitelist 1.2.0 "Whitelist"
nl.x-services.plugins.socialsharing 4.3.19 "SocialSharing"
nl.x-services.plugins.toast 2.1.1 "Toast"
org.pushandplay.cordova.apprate 1.1.7 "AppRate"
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$
then remove the platforms an reinstall android:
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ cordova platform remove android ios
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ cordova platform add android
No errors... but....building
http://pastebin.com/fA9XJUx6
for information... here my system
my ionic is the last
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ ionic -v
1.7.12
my android sdk is updated and (I think) complete for this system

someone has found same errors?
MaX

Thanks for this details issue!
Can you try changing<EMAIL_ADDRESS>to<EMAIL_ADDRESS>and ionic state restore again?

there is a problem with this cordova-plugin-google-analytics plugin and googleplayservices

max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ cordova plugin remove cordova-plugin-google-analytics
Uninstalling cordova-plugin-google-analytics from android
Removing "cordova-plugin-google-analytics"
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ cordova plugin add<EMAIL_ADDRESS>Fetching plugin<EMAIL_ADDRESS>via npm
Installing "cordova-plugin-google-analytics" for android
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$ ionic state restore
Attempting to restore your Ionic application from package.json
Restoring Platforms
cordova platform add android
cordova platform add ios
Restore platforms is complete
Restoring Plugins
cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add https://github.com/Telerik-Verified-Plugins/NativePageTransitions#0.5.5
cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>Restore plugins is complete
Ionic state restore completed
max@U9200:~/XDK/WPHC/wordpress-hybrid-client$
:-D
ok, no errors!
now I try to build

sorry, but there are more errors:
http://pastebin.com/XDm7qKR8
for the moment I won't use google analitic... can I unistall  cordova-plugin-googleplayservices and
cordova-plugin-google-analytics?
the app can works without there twho plugins?

Just remove it for now. Will check this issue, thanks

where is the list of plugins to install used from "ionic state restore" ?

at the end of package.json

I have updated to the latest google play service and I can reproduce!

good!! :)
meanwhile I'm compiling without these two plugins. I tried to "cordova build" but although the building reaches the end without errors, then android5, not let me install the apk on my mobile ... strange.

I can build usign "npm run build", sign the apk and install in my phone.
but now... "Error: class cordova not found" and then white screen. :(
I will investigate how to downgrade in android studio, the google play service

how is npm run android doing ?

sorry... too late :)
I have just donwgrade the google_play_service to the version 25 and I'm restarting the WPHC environment, in order to see if I can see working WPHC in my phone.

houston, we have a problem :(
downgrading the google_play_service to #25 is not enougth
So I have remove the cordova-plugin-google-analytics from the list.
So I can compile and build without errors... but with
"npm run android" I can read in my phone:
Error Class cordova not found
and then
Application error the connection to the server was unsucessfull.( file://android_asset/www/index.html)
I'm so sad... :(

There is a conflict with push notif plugin. WOrking on it

we are all with you! :D

Getting there :P

Why do not you join the chat? https://gitter.im/shprink/wordpress-hybrid-client

@maxlinux2000 can you please:
Update android dependencies like this:

then :
git checkout develop
git pull origin develop
rm -rf plugins/ platforms/ node_modules/
npm install && npm run restore
npm run android (with device plugged in)

If it works (works on my side) I will release beta02 right away.

 YOU ARE THE CHAMPIOOOOON!!!!!
https://45.media.tumblr.com/db40d681639e0e53535e5e57577e7f07/tumblr_n82tqtEc7g1tbx44fo1_500.gif
now the build is working in my phone! :dancer:

Haha thanks for your help
releasing beta02 :)


Azure Cognitive Search: Custom index and scoring profile support

Description: Adding support for custom index and scoring profile support in Azure Cognitive Search
@hwchase17

@pablocastro @farzad528

@ruoccofabrizio is there any way to filter by those new extra fields and delete only the filtered vectors?

Think i'm doing something wrong because i'm getting an error while trying to filter..
(then I realized this is not merged yet) hahahaha
Do you have an estimated time for the merge? @ruoccofabrizio @farzad528 ??

Any news on this one?

Thanks @pablocastro .
@hwchase17 Really appreciated whenever you can merge!

Any issue on this one @hwchase17 ??

thanks all!


Move squbs-admin-exp to squbs-admin

Remove the old squbs-admin, too. Do it in a way that is most sensitive to git.

Fixed by #314


Update: W3Devices-1.2.0 - Installer hash value corrected

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I will mak a new pr


Tokenizer maintains Tokens in a linked list.

This is the first part of addressing #3 (implementing the linked list). It also includes a tweak that (probably) improves the performance of parsing declarations. Previously, we were slicing the string and then using indexOf to find the colon and then splitting the string on its index. Now we can just traverse the Token linked list and compare Token instances to do the same thing.

/cc @azakus

LGTM++


Failed to resolve: com.pawegio.kandroid:kandroid:0.8.8

Issue is appeared when upgrading:
Android Studio 3.6 -> 4.0
Gradle 3.6.3 -> 4.0.0
and
-distributionUrl=https://services.gradle.org/distributions/gradle-5.6.4-all.zip
+distributionUrl=https://services.gradle.org/distributions/gradle-6.1.1-all.zip

Temporarily downgraded to version 0.8.7 to resolve.

@janzzend
Thanks, really working but no need to close the issue because this is only hotfix

I guess the reason is because the maven-metadata.xml at bintray is wrong.
As you can see in this copy I've just uploaded to GitHub, the latest version is named 0.8.7.
So Gradle is unable to find the 0.8.8 because, well, there is according to the XML no 0.8.8 🙃

any update sir? @pawegio


winpty automatic slash-to-backslash conversion messes commands that need slashes

Environment

Windows 10 64x
mintty 2.0.3 from Git Bash
winpty 0.2.2-dev from Git Bash
Bash 4.3.42

Problem
Running docker inside mintty fails and there's a correct recommendation to use winpty:
$ docker run -it mrts/python-selenium-xvfb
the input device is not a TTY.  If you are using mintty, try prefixing the command with 'winpty'

However, when using winpty, slashes are converted to backslashes that messes up docker that expects slashes:
$ winpty docker run -it mrts/python-selenium-xvfb
docker: Error parsing reference: "mrts\\python-selenium-xvfb" is not a valid repository/tag.

Expected behavior
Slashes are not converted to backslashes or there is a way to turn this off.

FWIW, upstream winpty passes arguments through unmodified, which I think matches Cygwin's behavior.
If you can get upstream winpty working with your Git Bash environment, then I think you'd avoid this problem.  You could try downloading winpty-0.4.0 from https://github.com/rprichard/winpty/releases/tag/0.4.0.  I think you'd need the 32-bit or 64-bit MSYS2 packages according to uname -m.
MSYS2 (and MSYS, IIRC) uses a heuristic to automatically detect Unix-style paths in arguments and convert them to Windows-style paths.  IIRC, that conversion behavior isn't exposed from the MSYS2 DLL.
When winpty was packaged for MSYS2, the packager added the argument slash-to-backslash conversion.  Eventually, the argument conversion code from MSYS2 was brought into the MSYS2-packaged version of winpty: https://github.com/Alexpux/MSYS2-packages/commit/f6440fbde4b9df9a2ce89e45b65abda1c1f9e7f3.  That may be something this project should also do.  I'm guessing you don't have that version yet, because the MSYS2 conversion heuristic seems to leave foo/bar alone.  (OTOH, it will convert something like /foo/bar.)
Slightly related to https://github.com/rprichard/winpty/issues/81, but not really.

Thanks for the thorough explanation!

@mrts Did this solve your problem, i'm having the same issue with git bash.

MSYS2's automatic path conversion can be controlled by an environment variable MSYS2_ARG_CONV_EXCL.
See: https://sourceforge.net/p/msys2/wiki/Porting/#filesystem-namespaces
(I'm not sure this is the case.)

MSYS2 has a MSYS2_ARG_CONV_EXCL variable, which is a semicolon-delimited list of prefixes that are matched against arguments, with a special '*' value that matches anything.
It looks the process spawn's argument heuristic has a bunch of additional special cases in Git-for-Windows that aren't in MSYS2.  e.g.:

Git-for-Windows adds a MSYS_NO_PATHCONV variable that (I think) disables the heuristic if get() returns non-NULL.
Mangling is disabled for paths starting with @@ or paths that look like IPv6 addresses.
Compare Git-for-windows (https://github.com/git-for-windows/msys2-runtime/commits/msys2-master%2Bgit-for-windows/winsup/cygwin/msys2_path_conv.cc) versus MSYS2 (https://github.com/Alexpux/Cygwin/commits/msys2-master/winsup/cygwin/msys2_path_conv.cc).

The MSYS2-patched winpty uses the argument heuristic on the first argument, which is a mistake.  MSYS2 uses path::get_win32() for the first argument instead.  (See linebuf::fromargv in the MSYS2 runtime.)
I'm wondering again whether winpty should try to access the MSYS argv -> cmdline translation code.  If nothing else, it could use POSIX spawn to start winpty-agent.exe and have it dump the Windows command line (and environment?) to a pipe, then run winpty_open like it does now.  It would slow winpty startup.


Normalizing scores for different versions

From d4rl/infos.py it can be seen that the same reference score is used for computing the normalized values for versions v0-v2. Is this correct? Shouldn't each version have its own reference score?

I think this is correct to use the normalized values for different versions of datasets. See https://github.com/rail-berkeley/d4rl/wiki/Tasks for the differences between different versions of datasets. The *-v2/v1 datasets only have improved metadata thus I think the score of this dataset should be the same as their v0 counterparts.


Feat/mining reward multiplier

Description
Added mining reward pool. This (number) indicates an amount of tokens reserved from the treasure specifically for mining reward payouts.
Added "reverse scaling" for mining reward payouts. This means the less tokens are in the mining reward pool (reserved from the treasury) the smaller the reward become.
Added constant factor called "hashDevaluationFactor" to scale the payouts further. This is a simple get/set.
Special notes:
Please carefully review the code written for the actual scaling formula in the SearchSECORewardingFacet.sol file. This is crucial code and the way it was written makes it only the more prone to errors/bugs. Do suggest ways to improve the readability of the code.
I've tested this bit of code in Test_SearchSECORewardingFacet.ts, but the code is just as unreadable over there so please review that too.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
See Test_SearchSECORewardingFacet.ts.

Should probably add a separate test for the monetization part of the process

don't merge after approve please

should add comments to helper functions

check jira for other todos regarding this pull request


chore: add service now destination support

Signed-off-by: Nitish Kulkarni<EMAIL_ADDRESS>PR summary
PR includes changes of support for new destination service now.
Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Go Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
no support of service now destination
What is the new behavior?
added support for service now destination
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.1.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Calculate regressors and save them in _confounds.tsv table

[ ] confounds in native space

[ ] Frame Displacement regressor (input is HMC movpar)
[ ] after HMC & unwarping:

[ ] White Matter, Grey Matter, CSF signal regressors, Global Signal regressor
[ ] dvars
[ ] TCompCor
[ ] ACompCor

[ ] Put outputs in bids-derivatives according to standard

I have the same concern as in #44

For CompCor, please see https://github.com/nipy/nipype/issues/1594
For the Global Signal and the tissue-wise signal regressors we could submit another new interface to nipype.

OR

Have only one interface for all of this (ComCor, Global Signal, tissue-wise signal regressors).

In both cases, the FD regressor is a bit more complicated and we already have all the building blocks in place to generate it. That's why I would not mix it with the rest of them.
I prefer the two separated interfaces, and let it up to the user to merge all of them in a single file (using AddCSVColumn, for instance)
WDYT @chrisfilo ?

@oesteban At your suggestion, I took a look at AddCSVColumn in nipype.algorithms.misc, and it doesn't match the requirements for this project as written--would you recommend I modify/refactor AddCSVColumn to make it work (may take longer, but perhaps more useful in the future), or write a simple T/CSV manipulation function just for this (less work now, but not as useful to others)?

@shoshber @chrisfilo let's go for a quick and dirt implementation, probably just with something like numpy.savetxt(delimiter='\t')

Agreed - or a Function Interface that uses pandas.
On Fri, Sep 30, 2016 at 2:03 PM, Oscar Esteban<EMAIL_ADDRESS>wrote:

@shoshber https://github.com/shoshber @chrisfilo
https://github.com/chrisfilo let's go for a quick and dirt
implementation, probably just with something like
numpy.savetxt(delimiter='\t')
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/issues/109#issuecomment-250851704,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp9KlMMUHRP39KgSxqZZgnOpE5gdWks5qvXktgaJpZM4JkGFS
.

Cool, time to learn me some pandas!

pandas adds another dependency and it is probably overkill the problem, but as you want. Sooner or later pandas will end up as a dependency.

You are right - it's probably an overkill.
On Fri, Sep 30, 2016 at 6:01 PM, Oscar Esteban<EMAIL_ADDRESS>wrote:

pandas adds another dependency and it is probably overkill the problem,
but as you want. Sooner or later pandas will end up as a dependency.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/issues/109#issuecomment-250882866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp0o-hfCyDiWBJiNjI3dNfPcF6AT5ks5qvbDPgaJpZM4JkGFS
.

Pandas fatalism wins again.

That's why they are endangered.
On Fri, Sep 30, 2016 at 6:28 PM, Ross Blair<EMAIL_ADDRESS>wrote:

Pandas fatalism wins again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/issues/109#issuecomment-250884416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkpzx1wCIBF8_iDCJLJkZftD4D1KSEks5qvbdMgaJpZM4JkGFS
.

Pandas make a comeback! http://www.cbsnews.com/videos/adorable-baby-pandas-make-their-debut-in-china/
Also I finished writing it before I saw these comments. So ...


QnAMakerOptions using StrictFilters how to check "contains" for metadata filters

Hi,
I using QnA Maker API to get response for StrictFiletrs as
{
"question": "qna maker and luis",
"top": 5,
"strictFilters": [
{
    "name": "category",
    "value": "api"
}]

}
getting response if metadata key and value matches with given input. (if metadata key is "category" and value is equal to "api" getting response)
Please let me know is there any way to check given input contains in metadata (if metadata key is "category" and value contains "api")
Thanks in advance

@dmvtech

Hi @saikiran521,
My apologies, but could you clarify a little bit further? I'm not quite clear on what you are asking. Are you wanting to check that the strictFilters you supply in the request match the strictFilters you get in the response?

Hi @dmvtech
Using strict filters data checks equals with metadata key value pair. For example if metadata is "category:api" we can retrieve by strict filters as
"strictFilters": [
{
"name": "category",
"value": "api"
}]
I need an example if metadata is "category: api response" I need to get response using same strict filter as
"strictFilters": [
{
"name": "category",
"value": "api"
}]
need a solution if value of the strict filter is a sub-string of the metadata value, how to give the input and get the response

I believe the strictFilters are meant to be pretty strict. That's why it doesn't work that way.
If you have specific scenario/situation where you have a a set list of values, you could (conceivably) add logic in your code before applying strictFilters to QnAMakerOptions to handle that. For example, a function that would strip the  response from the value before setting the metadata.
Otherwise; (since this isn't an bot feature, but QnAMaker), you could open a request to have this added changed. Do this by creating a UserVoice ticket for QnAMaker.

thank you for your response @dmvtech, will raise a ticket in QnAMaker


Stacking Text and Icons

It would be nice to have the ability to stack icon and text, as described here.

Thanks for issue. Welcome to pull requests. We will be grateful for the help in the development of the extension.

Hi. Pr merged.


Refactor to just be jupyter-desktop

Fork from https://github.com/manics/jupyter-omeroanalysis-desktop
to remove all components other than XFCE desktop.

Move everything to base directory, since this repo is only
for jupyter-desktop
Remove packages not needed for base functionality from apt.txt
and environment.yml
Remove extra desktop shortcuts

@manics and @ryanlovett did all the actual work!

I didn't mean to make this a PR here, but for https://github.com/yuvipanda/jupyter-desktop. Eventually I think this should move into the JupyterHub org.
<3


%load_ext magic

Would it make sense to support the python kernel's %load_ext magic?
Even if the internals of sparqlkernel are too different from the ipython kernel, it would be useful for loading custom (or customized) extensions.

By way of background: I came across the ipython autotime extension, and tried to load it like this to see if it would work with sparqlkernel:
%load_ext autotime
But sparqlkernel does not recognize this magic, so I got no further. (So, independently of this feature request: Any tips for how to time cell execution in sparqlkernel? ;-))


build: enable Ubuntu 22.04

Signed-off-by: Tim Whisonant<EMAIL_ADDRESS>

Coverage remained the same at 65.481% when pulling f776770108a9b1bb4fd550eed0bbb19aaed59d83 on tswhison/ubuntu22 into 6c149ded26cae3c61460d8fc2216e2522f48bdfe on master.


Get all 3 opts in one call?

What if I want all the names, urls, and IDs in one call?  Is there a way to do that?

Currently, there is no way you could do that, but I think something like this should be part of this API -
ytlist(url, 'all').then(res => {
  console.log(res);
  // => {...}
});

I'll work on this feature when I get somet time. Thanks for pointing it out.

Sent PR


Add spell check script to hack

Add a script to check commonly misspelled English words.

We already run mispell through golangci-lint which covers all Go files. golangci-lint has the advantage that we don't have to worry about installing individual tools, among other things. Did you check if https://github.com/vmware-tanzu/antrea/blob/master/.golangci.yml could be tweaked so that mispell also runs on documentation files?

I tired adding the following which did not quite catch those errors locally either.. seems like it only checks for go code?
linters-settings:
  misspell:
    source: text

@abhiraut Too bad but that makes sense, golangci-lint invokes each linter on a file-by-file basis.
Could you make sure this is part of CI? You can add a job to .github/workflows/go.yml.

/test-all

/test-e2e
/test-conformance

@abhiraut there is some issue with the jenkins testbed right now. There is also no need to run the jenkins tests for this PR.

/skip-all


The "New Supervision" list item isn't activated correctly

Previously I had set:
`<mwc-list-item
        hasMeta
        class="sup-ln mitem"
        graphic="icon"
        data-ln="NEW"
        value="New ${supervisionType} Supervision"
        ?noninteractive=${this.availableSupervisionLNs === 0}
      >
        New ${supervisionType} Supervision
          ${this.instantiatedSupervisionLNs > 0
            ? html`— ${this.availableSupervisionLNs} available

`
            : nothing}

        <mwc-icon slot="graphic">heart_plus</mwc-icon>
      </mwc-list-item>`;

However this means there are situations when deleting a used control block and then connecting a new one works once but not twice (this test: can carry out a sequence of create, disconnect, delete and connect).
To avoid this we now have:
// TODO: Could not get the following noninteractive expression to work
    // for the "NEW" mwc-list-item
    //  ?noninteractive=${(this.selectedIedSupervisedCBRefs.length ===
    //  this.selectedIedSubscribedCBRefs.length) ||
    //  this.availableSupervisionLNs === 0}

    return html`${this.getSelectedIedSupLNs(used, unused)
        .filter(supLn => this.searchUnusedSupervisionLN(supLn))
        .map(lN => html`${this.renderUnusedSupervisionNode(lN)}`)}
      <mwc-list-item
        hasMeta
        class="sup-ln mitem"
        graphic="icon"
        data-ln="NEW"
        value="New ${supervisionType} Supervision"
        ?noninteractive=${this.availableSupervisionLNs === 0}
      >
        New ${supervisionType} Supervision
          ${this.instantiatedSupervisionLNs > 0
            ? html`— ${this.availableSupervisionLNs} available

`
            : nothing}

        <mwc-icon slot="graphic">heart_plus</mwc-icon>
      </mwc-list-item>`;

Which is not correct and means the "New LGOS Supervision" can be clicked on when it shouldn't be possible (i.e. when there are no control block references in the right hand side), see for the picture below:

There is something I don't understand about the reactive update or the internal state which is causing this and I will need some help to resolve it.
I thought that I should be able to rely on editCount causing a full re-render of all templates, but the this ?noninteractive attribute appears to not be correctly calculated even though within the browser debugger I don't see that it is noninteractive when I can no longer click on it (after the first delete and reconnect round). However removing / reducing the noninteractive conditions seems to improve this and I don't know why.

To trigger this, use the "GOOSE-2007B4-LGOS.scd" fixture and the issue-7-showing-issue branch with only the small change to noninteractive

Select "GOOSE_Subscriber1"
Delete with the delete button LGOS 2
Click on "New LGOS Supervision" at the bottom left list
Click on the control block on the right.
Repeat steps 2,3,4. The bug is that the same supervision cannot be reallocated (the list item is either disabled or not activated or something).

The cause appears to be dynamic changing of interactivity without calling layout()


osquery agent host name changes

Osquery agent host name changes isn't reflected to Kolide fleet server. Despite the host names changed they still keep appearing as they were first added

Test
I didn't waited for an hour but I ran the (distributed) query "select hostname from system_info" a short time after I changed the hostname and I saw the host name changed on UI.
lastly, after I changed the hostname one more time, I ran the query again and I noticed the hostname and computer_name is different
(query result is that: hostname: win2008-2 but computer_name: win2008-2-23)
When I check on the computer via login I see that computer name, full computer name and netbios name is same win2008-2-23

what could be the situation?

If Fleet is showing the correct data from osquery (after up to an hour delay), then things are working fine.
The Fleet UI is using the value returned for the hostname column of system info. I think this is the most generally useful value we can show (note that computer_name is marked as "optional" in the osquery schema).
It sounds like there things are working as expected with Fleet, so I'm going to close out this issue. Happy to continue discussing if there are further clarifications needed.

I think you are right. the only thning I wondered is the difference between hostname and computer_name columns for a while. Thanks.

Possibly this is worth a new issue, but I'd really like the ability to specify the query used to populate the host name in the UI, since I'm monitoring end user workstations that like to change hostname when the change networks. ie. Have the hosts identified by system_info.local_hostname, or even system_info.uuid


Certificate Issue for Particular Websites

Recently I was navigating this particular website https://www.iditect.com/ using titanium Proxy.
For some reason this website was not loaded, Then I browsed without Titanium this website was opening. I found this issue through crome.

I think the problem is with the certificate what we creating.
I there anyway to skip issuing certificate and  allow with load their own certificates, if any problem occurs.

This problem shuold be fixed in the latest package with enabled ssl decryption.

I there anyway to skip issuing certificate and allow with load their own certificates, if any problem occurs.

Yes, you can always disable the SSL decryption for any particular website with the following code:
            explicitEndPoint.BeforeTunnelConnectRequest += onBeforeTunnelConnectRequest;

        private async Task onBeforeTunnelConnectRequest(object sender, TunnelConnectSessionEventArgs e)
        {
            string hostname = e.HttpClient.Request.RequestUri.Host;

            if (hostname.Contains("dropbox.com"))
            {
                // Exclude Https addresses you don't want to proxy
                // Useful for clients that use certificate pinning
                // for example dropbox.com
                e.DecryptSsl = false;
            }
        }

Of course if you set this, you won't be able to see/modify the decrypted traffic, loaded urls.

I close this issue, feel free to reopen if needed.


Incorporate additional features from sample handout

The tufte-latex folks have a sample document with some neat features that we could copy, including:

full-width text blocks
indented lists (for, e.g., references)
indented text blocks (e.g. the introduction)
increased letter spacing for all-caps

Blockquotes and indented lists have been fixed. Full-width text blocks and all-caps letter spacing are still on the to-do list.

These are no longer project goals.


Array.from doesn't support iterables

Before the recent removal of corejs polyfills I had this in my code:
let s = new Set();
s.add(1);
s.add(2);
let a = Array.from(s.values());
```
And it worked (as it should).

Now it doesn't work anymore.

A brief look at `aurelia-polyfill` makes me think that `Array.from` only supports array-like parameters (with `length` and an indexer). Per the specs it must also support iterables (objects with a `next()` function that returns `{ value, done }`).

This is a breaking change.

We're happy to accept a pull request to fix that and would release it asap once provided.

In the end I added back corejs to my project.
We have a large, complex LOB application and it was written from the scratch with the assumption that we could assume ES6 level of features (although not extensively).
After the last update, many things started to break: for of loops because Symbol polyfill was missing (we use Babel to transpile). Missing Promise polyfill. Array.from(new Set().values()) was not properly supported in Aurelia. string.repeat() was missing.
So I made the call that the risk of regression was just too big and not worth it. We put back a custom build of core-js that only contains ES6 polyfills. It weights around 60Kb minified (not gzipped). For an intranet app it's acceptable, especially given the full app size and the fact that after first loading it will stay in browser cache.
Now the question for me is rather: is there an easy way to remove aurelia-polyfills? It seems like an unrequired dependency now.
I created a PR for you anyway: #5. I think that it makes no sense to expose Set and Map polyfills and not properly support Array.from on the results of .values() and .keys().
The PR seems pretty straightforward to me, but please note that because of above rollback to corejs I did not test this exhaustively.
I did not find a test suite, otherwise I would have added a few tests along the lines of:
var s = new Set();
s.add(1); s.add(2); s.add(3);
expect(Array.from(s.values()).join(",")).toBe("1,2,3");

We've added a Symbol polyfill as well as Symbol.iterator (for/of) support for all collections. We've fixed up Object.assign to work with Symbols as well.

My PR should be modified then.
In my PR, because there was no Symbol.iterator I looked at the next() method directly.
It would be cleaner (and more specs compliant) to modify it so that:

Array.from first checks for @@iterator. If there is one it should do the next() loop on [Symbol.iterator](), which actually just returns this but serves as a marker interface (more or less).
I don't close the iterator correctly. On arbitrary operators there might be a return() method.
Otherwise Array.from should fall back on the array-like code.

At this point, I think with Symbol.iterator built-in aurelia-polyfills it would be best to grab a correct Array.from polyfill from existing sources such as CoreJS. My PR was merely a work-around to make Array.from(set.keys()) & co. work (as they were also in the included polyfills) but it is not completely spec-compliant.

Would you be interested to locate such a polyfill or "port" one. We would like something without any external dependencies beyond what we already have in our polyfills library.

I updated my PR by taking the CoreJS implementation (mostly). Because CoreJS is made of tons of tiny reusable parts I had to merge stuff and simplify a little bit the paranoid checks that litter this code.
Beware that I did this directly in Github during a commute... so I did not run this code even once! Maybe it does not work.
It seems to me it should be OK though and hopefully it help you get a better Array.from more quickly.
Sorry about that but as I'm using CoreJS again at work it doesn't leave me a lot of time to test alternative polyfills...

Understood. Thanks for the assistance!

The main thing we wanted was to remove the hard dependency that we had on core-js which left developers without options. We wanted to also have our own solution and open up choices for developers. Thanks for helping!

I'm afraid you've opened Pandora's box, though... now people (like me?) will find edge cases that don't work (I saw something about Object.keys(null)) and you'll need to maintain your own polyfill stuff :(
Why didn't you use the custom build option of CoreJS to get just the bits you needed?
Also, a good thing would be to put aurelia-polyfills as a totally optional dependency that needs to be required first. So that people really have the choice to use another (e.g. corejs), use none at all (if target platform has enough support -- ES6 browsers are practically here now), or use aurelia-polyfill.

The core-js project is in danger, so we don't want any dependency on it in our core.
It's possible to avoid our own polyfills by writing your own bootstrapper, which isn't hard, or manually bootstrapping the framework in code. But, I'm not sure it's worth it for most people, especially since our polyfills are pretty small.

It's true that they are small. Any code example? We already bootstrap Aurelia in code (if I understand you correctly).
Just out of curiosity could you quickly clue me in as to why core-js is in danger? Thanks!

@jods4 I think Rob is referencing to this issue: https://github.com/zloirock/core-js/issues/139.

The bootstrapper module is just a small bit of "startup" code that gets things working. It could be replaced with your own code. It's the only thing that references our polyfills, so that could be removed. Once we all move to jspm 0,.17, the bootstrapper will get even simpler...making it easier to write your own if needed.
@Gheoan is referencing to the correct issue. It's best for us to have as few hard dependencies as possible. So, even if core-js is ok, we all feel better not "forcing" developers to use it and not tying ourselves to things we can't control.

@Gheoan thanks for the pointer.
I feel like polyfills should never be a hard-dependency anyway.  They are an implementation of a standard spec, you should be able to swap them at any point. As long as you know (and document) what features you depend on, you should be good.
We use AMD/requirejs here. When this project started JSPM seemed too "fresh" for us to place a confident bet on it. We had a lot of experience and past success with AMD, so that felt like a better choice.

Awesome! I'd like to hear about how require.js has worked out. Please send me some email rob at durandal dot io  I've used require.js a lot in the past. That's what Durandal was based on. I'd like to get some feedback from you on using it with Aurelia. Maybe we can make it easier. We could even do a custom bootstrapper if needed, like we just added for Webpack.

I'd just like to register my support for this being part of aurelia-polyfills. I too have had to revert to core-js but if this were included in aurelia-polyfills I could lose my dependency on core-js.
Rob, I'm happy to see you've been receptive to its inclusion despite not being needed by aurelia itself. I think these polyfills cover the majority of commonly required cases. This is the last piece of the puzzle that needs filling for me :+1:

Yes, we can add it. We just need to test the new version submitted based on iterables.

@EisenbergEffect you can close this issue now :-)
When do you plan to do a release containing the PR? I was about to update and realised it's not included in a release yet.

Coming tomorrow most likely.

Sorry to resurrect this one from the dead. We're running into issues using the polyfills and Array.from with Maps, and specifically when we use a map function. The error we get running this code is:
var map = new Map();
map.set('foo', 1);
Array.from(map, function(x){
  return x[1] = x[1] + 1
});

TypeError: fn is not a function. (In 'fn(a1, a2)', 'fn' is an instance of Array)

This is coming from this line https://github.com/aurelia/polyfills/blob/master/src/array.js#L11
Can anyone else confirm? Is this a known issue?

As I noted in the PR, the problems seems to be that L32:
https://github.com/aurelia/polyfills/blob/master/src/array.js#L32
should be
result[index] = mapping ? iterCall(iterator, mapfn, step.value, index) : step.value;
instead of
result[index] = mapping ? iterCall(mapfn, step.value, index) : step.value;


Add a loading screen when opening the app:

fetch every useful data from firebase (eg. trophies, stars, shop items, statistics...) and then store them locally

Resolved by PR #105


Unable to change redirect_uri after provider has mounted

Effectively I have a component with logic similar to:
const MyComponent = () => {
     // Hook that changes with every single page move returning the current URL
     const path = useBrowserPath();

     const oidcConfig = {
         // ...
         redirect_uri: path,
     }

      return <AuthProvider {...oidcConfig}>
              <App />
       </AuthProvider>;
}

The problem is that changing any of the configuration does not cause the auth provider to update the settings nor does it reconstruct the user manager.
As my goal here is to just update the redirect uri periodically I did also try to set the useAuth().settings.redirect_uri etc but since the OidcClient has its own copy of the settings it does not change the redirect_uri it uses.
I believe this is a bug as it should be possible to change this configuration, however please do let me know if theres something fundamentally incorrect with attempting to change the settings/redirect_uri.

Typically the redirect_url must be registered in the auth server configuration, as it is security depended. There is no need to change the redirect_url dynamically, as this library intercepts the query parameters dealing with the auth process. You must just ensure the configured redirect_url points some where into your SPA application. You do *not" need to setup that route!
e.g. http://localhost:5000/signin-callback is a good candidate for development.

will close the issue soon, unless you comment...

if you need to deal with different environments (development vs production), best way is to compile the application code separately per  environment by using something like this in package.json:
...
  "scripts": {
        "start": "env-cmd -f .env.development react-scripts start",
        "build": "env-cmd -f .env.development react-scripts build",
        "build:production": "env-cmd -f .env.production react-scripts build",
        ...
  }
...

.env.development:
REACT_APP_OIDC_AUTHORITY=https://my-auth.com
REACT_APP_OIDC_CLIENT_ID=12345
REACT_APP_OIDC_REDIRECT_URI=http://localhost:5000/signin-callback


add ContainerImage field

change the order of fields in protobuf
add missing fields (i.e., ContainerImage) in gRPC

Signed-off-by: Jaehyun Nam<EMAIL_ADDRESS>

Codecov Report

Merging #603 (4cfffb4) into main (b4ff028) will decrease coverage by 0.88%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #603      +/-   ##
==========================================
- Coverage   41.46%   40.58%   -0.89%
==========================================
  Files          24       24
  Lines        8920     8922       +2
==========================================
- Hits         3699     3621      -78
- Misses       4787     4873      +86
+ Partials      434      428       -6

Impacted Files
Coverage Δ

KubeArmor/feeder/feeder.go
53.55% <100.00%> (+0.23%)
:arrow_up:

KubeArmor/monitor/hostProcessTree.go
0.00% <0.00%> (-69.24%)
:arrow_down:

KubeArmor/monitor/systemMonitor.go
39.39% <0.00%> (-9.57%)
:arrow_down:

KubeArmor/feeder/policyMatcher.go
40.41% <0.00%> (-0.24%)
:arrow_down:

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 3e731f3...4cfffb4. Read the comment docs.


Expand all is happening even if i click in the empty are/ when i dont display any indicator

Describe the bug
Hi I am using the custom expand indicator to implement the collapse only functionality( i dont need to have expand all )  in the expandHeaderColumnRenderer
as per the storybook example, i have implemented the custom indictors to show the collapse all (...) when i click on that and i gave the other condition as null  , so as(not to perform expand)
Here is my code :  as suggested by @AllenFang  in #1420 , i have added expandByColumnOnly: true, but this doesn't seem working for me.
const expandRow = {     renderer: row => (

{This Expand row is belong to rowKey ${row.id}}

           You can render anything here, also you can add additional data on           every row object

           expandRow.renderer callback will pass the origin row object to you

     ),     showExpandColumn: true,     expandByColumnOnly: true,     expandHeaderColumnRenderer: ({ isAnyExpands }) => {       if (isAnyExpands) {         return -;       }       return null; //it means i should not perform any expand     },     expandColumnRenderer: ({ expanded }) => {       if (expanded) {         return -;       }       return ...;     }   };
expandHeaderColumnRenderer: ({ isAnyExpands }) => { if (isAnyExpands) { return -; } return null; },
As per the above condition, I will not see any indicator for expand all., which is correct But the issue is when I click on that area which is null/ empty I am seeing the expand is still happening, which should not be the case in this scenario. Is there any way I can restrict only expand functionality in such a case
Please find the code sandbox link in below:
@AllenFang  request you to please review this, Thank you
codesandbox
https://codesandbox.io/s/react-bootstrap-table-next-basic-example-zsu58?file=/src/index.js

HI @sonusandeep  boss, so far we can't do this((for your requirement, but I got an idea.. ugly idea.. lol
Make the empty element to full cover the th element:
First, make it cover the th element:
expandHeaderColumnRenderer: ({ isAnyExpands }) => {
  if (isAnyExpands) {
    return -;
  }
  const style = {
    position: 'absolute',
    top: 0,
    right: 0,
    left: 0,
    bottom: 0
    width: '100%'

  }
  return
 e.stopPropagation()} />;
},

Second: change it's parent(th) as position relative:
.react-bootstrap-table > table > thead > tr:first-child > th:first-child {
  position: relative;
}

I tested it and work perfect. Let me know if above isn't work.

it's my link:
https://codesandbox.io/s/react-bootstrap-table-next-basic-example-z3qde?file=/src/index.js

Awesome boss @AllenFang , working perfect :)


Gas cost changes for IO-heavy operations

https://github.com/ethereum/EIPs/blob/master/EIPS/eip-150.md

This issue still exists
https://etherscan.io/tx/0x19d5907ebfcd3fc8640b2e7c042c7f2c34be5fec12a2fa9283bd5dccbb8604e9

@rmeissner add some steps to reproduce

@koeppelmann @rmeissner A fix for this has just been deployed to production, let's see if it works


Add macOS pytest tests in CI

Make sure pytest tests also work under macOS, see #814.

It's a timeout error.

Yes, seems to be a temporary problem - the next build succeeded, so I'm going to ignore this.

Looks good to me


Logo for the Higher Education Engagement project

Context
We need a logo for the project.
References

https://github.com/Green-Software-Foundation/projects/issues/83
https://github.com/Green-Software-Foundation/community-wg/issues/89
Thoughts on University engagement (supporting Student events, providing curriculum templates)

Actions

[ ] @Jenya-design Design logo
[ ] Add to website project listing
[ ] Update repository with logo

@russelltrow could you add this task to Design project? Do you know what might be the issue, why I can't add some tasks there?
Please let me know what's my deadline here. Thank you

@russelltrow please check two logo options and let me know your thoughts

These look great. Both are great options. I like the v2 just slightly more, but either would be great!

Thank you @tholewis @russelltrow please check the files for logo and icon:
svg format
png format
all files

@tholewis apologies, can't share with you the whole folder at the moment, I'll try to figure it out on Monday
Please find all png files attached. Let me know if it won't work for you or you urgently need other formats

These are just awesome! Love it!


[5.5] PSR2 Can not be followed

Laravel Version: 5.5.26

Description:
It is not possible to correctly follow PSR2 standards when developing a laravel application fully.
The most obvious example is the database migrations, as they can not be namespaced because the Migrator will not be able to find it, but PSR2 requires all class to be namespaced.
Steps To Reproduce:
Create a migration in laravel, using artisan
Install PHP Code Sniffer and run it on your project, using the --standard=PSR2

Is there anything that can be done, or any plans to upgrade the framework to make following PSR2 100% possible?

Hi @sfgn,
This is best to be placed in https://github.com/laravel/internals for discussion there.

Thanks - I did try looking for a existing one but must have missed it.


Revert "event cleanup (code+golden files) (#2226)"

This reverts commit 152a20c72af8b855f4caec1355abbce863192fb5 and restores the old event behavior.
We have several issues on the UI and presumably they come from the reverted commit.

This reverts commit 152a20c and restores the old event behavior.
We have several issues on the UI and presumably they come from the reverted commit.


Fail to build thirdparty from source

I tried to compile the third-party libraries on Linux, but I encountered the following compilation errors:
In file included from /home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.cc:4:
/home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.h:17:2: error: #error This file was generated by an older version of protoc which is
   17 | #error This file was generated by an older version of protoc which is
      |  ^~~~~
/home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.h:18:2: error: #error incompatible with your Protocol Buffer headers. Please
   18 | #error incompatible with your Protocol Buffer headers. Please
      |  ^~~~~
/home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.h:19:2: error: #error regenerate this file with a newer version of protoc.
   19 | #error regenerate this file with a newer version of protoc.
      |  ^~~~~
In file included from /home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.grpc.pb.cc:5:
/home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.h:17:2: error: #error This file was generated by an older version of protoc which is
   17 | #error This file was generated by an older version of protoc which is
      |  ^~~~~
/home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.h:18:2: error: #error incompatible with your Protocol Buffer headers. Please
   18 | #error incompatible with your Protocol Buffer headers. Please
      |  ^~~~~
/home/alex/code/starrocks/thirdparty/src/grpc-1.43.0/starrocks_build/gens/src/proto/grpc/reflection/v1alpha/reflection.pb.h:19:2: error: #error regenerate this file with a newer version of protoc.
   19 | #error regenerate this file with a newer version of protoc.
      |  ^~~~~

After troubleshooting, I found that the problem was that my system installed a lower version of libprotobuf than the self compiled version in thirdparty. When compiling, the system installed libprotobuf was used instead of the one in thirdparty.
➜  ~ which protoc
/usr/bin/protoc
➜  ~ protoc --version
libprotoc 3.12.4

The same problem occurs when compiling the Arrow library:
[ 83%] Building CXX object src/arrow/flight/CMakeFiles/arrow_flight_objlib.dir/middleware.cc.o
[ 83%] Building CXX object src/parquet/CMakeFiles/parquet_objlib.dir/arrow/reader.cc.o
[ 83%] Building CXX object src/parquet/CMakeFiles/parquet_objlib.dir/arrow/reader_internal.cc.o
In file included from ^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.cc:4^[[m^[[K:
^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.h:17:2:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K#error This file was generated by an older version of protoc which is
   17 | #^[[01;31m^[[Kerror^[[m^[[K This file was generated by an older version of protoc which is
      |  ^[[01;31m^[[K^~~~~^[[m^[[K
^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.h:18:2:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K#error incompatible with your Protocol Buffer headers. Please
   18 | #^[[01;31m^[[Kerror^[[m^[[K incompatible with your Protocol Buffer headers. Please
      |  ^[[01;31m^[[K^~~~~^[[m^[[K
^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.h:19:2:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K#error regenerate this file with a newer version of protoc.
   19 | #^[[01;31m^[[Kerror^[[m^[[K regenerate this file with a newer version of protoc.
      |  ^[[01;31m^[[K^~~~~^[[m^[[K
In file included from ^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/src/arrow/flight/protocol_internal.h:23^[[m^[[K,
                 from ^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/src/arrow/flight/serialization_internal.h:22^[[m^[[K,
                 from ^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/src/arrow/flight/client.cc:39^[[m^[[K:
^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.h:17:2:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K#error This file was generated by an older version of protoc which is
   17 | #^[[01;31m^[[Kerror^[[m^[[K This file was generated by an older version of protoc which is
      |  ^[[01;31m^[[K^~~~~^[[m^[[K
^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.h:18:2:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K#error incompatible with your Protocol Buffer headers. Please
   18 | #^[[01;31m^[[Kerror^[[m^[[K incompatible with your Protocol Buffer headers. Please
      |  ^[[01;31m^[[K^~~~~^[[m^[[K
^[[01m^[[K/home/alex/code/starrocks/thirdparty/src/arrow-apache-arrow-16.1.0/cpp/release/src/arrow/flight/Flight.pb.h:19:2:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K#error regenerate this file with a newer version of protoc.

StarRocks version (Required)

main branch ae49f13b850accdab0e08206696c383fbe08e59a

I have built thirdparty on centos7. There are many errors, which I handled manually one by one.


add new script

add new script

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

mistake.


Fix issue #40

Not sure why all the old commits got here but most of it is stuff that got merged before.
Accidentally wrote in one commit fix for # 3. It was suppose to be Fix for # 40.

I have added fox for #3 as well.

Not sure why all the old commits got here

Because we haven't merged dev into main, maybe that's why it's showing all the previous commits.

This pr is to dev branch so probably it is other reason.

Seems like this by design, but have some workaround. Although no harm, let it be like this. Source - GitHub pull request showing commits that are already in target branch

I've added another feature here so I should also update the version and the change log.
You can check everything else in the meantime.


Document or fix linting on shallow clones

Shallow cloned repository yield errors via git-raw-commits. conventional-changelog-lint should either document how to circumvent these limitations or support commit listing on shallow clones, too.

see #6

Here's a workaround for TravisCI (which does shallow clones): https://github.com/marionebl/conventional-changelog-lint/issues/6#issuecomment-231186598

My workaround had an issue with tag commits since those result in a detached head. My current workaround is to do this in my before_install TravisCI yaml section:
git remote set-branches origin master && git fetch
git checkout master
# Go back to the original commit that TravisCI checked out:
git checkout -

I'm not sure exactly why but this makes the master branch visible to the linter.

After chewing around on this for quite a while I'll go with the following approach:

There is no way to get the git log messages from a shallow clone
Document the need to pull in the full history for shallow clones. Provide TravisCI snippet.
Detect shallow clones when running getMessages and print a warning with a link to said documentation


pip

Hey
Could you release this on pip so we can easily install it

Hey, I think it should work, I've uploaded a release to pypi recently, pip intall pynYNAB :-)

I ended up just using pip install {git_repo} :)


Species is checked for in PP configuration, this is not necessary

As the PP data is not species related, the system should not expect species information for that configuration.

Update in progress

Corrected in the code


Changing from css style hinting to HTML tag

Hello team,
Please I need your help, I don't know how to get my editor to work right.
I am watching a tutorial where the instructor is literarily writing css in a html file and it's converted to html tags, but I cannot get it to work.
Like div.someClass#SomeID -->
Thanks for your help. I have tried to research on my own, but it's all in vain.

Sounds like Emmet to me. The extension you are looking for is called brackets-emmet and it can be found via the extension manager!

Hi Petetnt,
That's right. I just figured that out and then came here to see that you commented the same. Thanks a lot for your response.
I can close this now.


Fixed directory path and language abbreviation in i18n readme

The path to the i18n directory was wrong
The abbreviation in your example did not match the one in your link.

Thank you for the catch 🙇
I resolved the merge conflicts as I merged another PR right before this one 👌


Generic type 'Promise' requires 1 type argument(s)

In the tab content type, for typescript support, the beforeChange method returns a boolean or a promise, but the promise needs a type argument like "Promise".

I fixed it, using the boolean type for the promise, but is nice to have it fixed for a future release.

Thanks @Capellan29 for pointing out.
My experience with TS basically started with this component 😄
If you have time feel free to open a PR and I will merge it since the fix is minor

@cristijora I did it, an update to the npm package with this change will make me so happy.
And thanks for this componen, is really great.

Released 0.8.2

Thanks.