[iOS] Wrong preview size on iPad devices using CameraPreview.cover

Steps to Reproduce
Setup cameraawesome and run on iPhone and iPad for comparison. On iPhone devices it will work as expected filling the screen but the same configuration on iPad leaves some blank spaces on the sides. If we switch to fitWidth it will work as expected on iPad but will leave blank spaces for top and bottom on iPhone

Expected results
No blank spaces on the sides for the preview
Actual results
Blank spaces on the sides for the preview.
About your device

Brand
Model
OS

Apple
iPhone X
15.5

Apple
iPad Pro (9.7-inch)
14.6

Your flutter version
Run this in your command line
flutter --version
Flutter 3.7.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision b06b8b2710 (5 weeks ago) • 2023-01-23 16:55:55 -0800
Engine • revision b24591ed32
Tools • Dart 2.19.0 • DevTools 2.20.1

Hi,
I fixed a bug on the PreviewFit.cover mode yesterday.
#256
It's not merged yet as we wanted to test on more devices.
But I think this would fix your problem.
If you want the preview to take all your screen you have to you the cover mode.
For the fitWidth mode. I think it's totally normal.
This mode will try to make the view exactly as large as it can be. But the height will stay proportionnal.
That's why height can be not taking all the screen.
For more explanation you can get a look at this great doc on web fit..

It did fix on the iPad now but on the iPhone now appears some black boxes on top and bottom:

Fixed


Connection timed out after not receiving any message for 10000ms. Disconnecting.

I set Deplay server at multipaler tab and test at UnityClient but it time out every time.
How to fix this issue.
KCP: Connection timed out after not receiving any message for 10000ms. Disconnecting.
UnityEngine.Debug:LogWarning (object)
kcp2k.KcpConnection:HandleTimeout (uint) (at Assets/Mirror/Runtime/Transport/KCP/kcp2k/highlevel/KcpConnection.cs:154)
kcp2k.KcpConnection:TickIncoming_Connected (uint) (at Assets/Mirror/Runtime/Transport/KCP/kcp2k/highlevel/KcpConnection.cs:250)
kcp2k.KcpConnection:TickIncoming () (at Assets/Mirror/Runtime/Transport/KCP/kcp2k/highlevel/KcpConnection.cs:362)
kcp2k.KcpClient:TickIncoming () (at Assets/Mirror/Runtime/Transport/KCP/kcp2k/highlevel/KcpClient.cs:89)
kcp2k.KcpTransport:ClientEarlyUpdate () (at Assets/Mirror/Runtime/Transport/KCP/MirrorTransport/KcpTransport.cs:117)
Mirror.NetworkClient:NetworkEarlyUpdate () (at Assets/Mirror/Runtime/NetworkClient.cs:1198)
Mirror.NetworkLoop:NetworkEarlyUpdate () (at Assets/Mirror/Runtime/NetworkLoop.cs:182)

have you configured your ports correctly?

have you configured your ports correctly?

Yes, I configured my port correctly.
I use port 30000 at kcpTransport.
I get port from Playfab->MultiPlayer->Servers
Can I config more?

Have a look at this  #47
Is it possible you changed the server or client scene?

I change client scene.

Read the NetworkManager docs. Specifically Scene Management. Scenes have to be the same.

Closing, please feel free to re-open if the issue is not resolved. Thanks!


Fix inferring module for unsupported data files

This PR raises a FileNotFoundError instead:
FileNotFoundError: No (supported) data files or dataset script found in <dataset_name>

Fix #5785.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

I think you can revert the last commit - it should fail if data_files={} IMO


terminal: helpful message in SIGINT handler

Our handling of SIGINT is confusing to people that expect SIGINT to be
delivered to inferior when they press ^C (see #483 for example).
Let's make an effort to point out what's happening and how one can send
SIGINT to inferior.

Let's consolidate those messages, and omit specific instructions. Let's just say something like: "recieved SIGINT, stopping process (will not forward signal)".

Done.


Bulk render a list of lines

Currently, you can call NuiLine:render to update a single line.
Is there a way to batch together many lines and call the api and update the buffer only once with all the line.
Alternatively, a NuiBlock or similar construct contains lines similar to how NuiLine "contains" NuiText objects.
E.g
local block = NuiBlock({
    NuiLine({NuiText("Foo")}),
    NuiLine({NuiText("On Line 2"), NuiText("Bar)})
})

block:render(bufnr, 1)

which will do a single nvim_buf_set_lines and apply the highlights.

Is there a way to batch together many lines and call the api and update the buffer only once with all the line.

which will do a single nvim_buf_set_lines and apply the highlights.

Are you concerned about the performance? If you want to render multiple NuiLine with a single nvim_buf_set_lines call, you'd still need to apply highlights using multiple calls and also keep additional states to keep track of the boundaries to apply highlights. I don't think it's worth it.

If you want the convenience of rending multiple lines together, you can write a small wrapper:
local function render_lines(lines, bufnr, ns_id, linenr_start)
  for i, line in ipairs(lines) do
    line:render(bufnr, ns_id, linenr_start + i - 1)
  end
end

and use it as:
render_lines({
  NuiLine({ NuiText("Foo") }),
  NuiLine({ NuiText("On Line 2"), NuiText("Bar") }),
}, bufnr, ns_id, 1)


[zh] Typo in docs/setup/production-environment/container-runtimes

启动命令缺少一个w
sudo systemctl enable crio --now

This is a Bug Report

Problem:
Typo in zh/docs/setup/production-environment/container-runtimes
Proposed Solution:
sudo systemctl enable crio --no -> sudo systemctl enable crio --now
Page to Update:

zh: https://kubernetes.io/zh/docs/setup/production-environment/container-runtimes/#cri-o
(en: https://kubernetes.io/docs/setup/production-environment/container-runtimes/#cri-o )

/retitle [zh] Typo in docs/setup/production-environment/container-runtimes
/kind bug
/language zh

Page is a little bit lagging behind. Will fix.
/assign


Proper Error handling

Most of the system components are built in a fail-safe way to begin, but nonetheless we should go over the complete system once and check where we'd need to introduce proper error handlig. A good example might be for ActorWrapper<A>.send() to return an errror if the mailbox is already stopped, so that users know to that they can drop the object.
This includes error handling for constructors and Actor trait functions

resolved with 0.6.0


UI visibility can be added

can you please assign me this issue?

I would like to work on this project, please assign me


Typecript + Zod error with Delegate and relation

https://github.com/eqqe/reprod-zenstack

Fixed in 2.9.0


Show rejection reason in exception

Is your feature request related to a problem? Please describe.
I am unable to make useful assertions on rejection reasons because the exception that gets raised in a generic exception that does not reveal any of the information.
Describe the solution you'd like
In client.wait_for_tx(), raise a custom error TransactionRejected(result["message"]) that includes that failure reason from the result.
Describe alternatives you've considered
Implementing a custom wait_for_tx().
Additional context
I was failing to sign a transaction correctly but the error message was not telling me that. I had to set a breakpoint in the lib to find out the details of the failure.

I'm actually seeing some strange properties on the TransactionInfo object...

Hey, thanks for the suggestion about custom exceptions.
TransactionInfo object are actually created internally in cairo-lang library, they're not part of starknet.py. If you think it's a problem with starknet.py I'd need more details on what's failing.

TransactionInfo object are actually created internally in cairo-lang library, they're not part of starknet.py. If you think it's a problem with starknet.py I'd need more details on what's failing.

Ah, I don't actually think it is a problem with starknet.py but the error is so strange, I would be remiss if I did not at least hint at it.
Basically, I can see some random OS paths in my error message that are definitely not me. I am running starknet-devnet as my provider.

Of course. We're actually using starknet-devnet for our end-to-end tests and for this limited use it seems to be working most of the time. But I don't dismiss you findings, there may be some hidden problems - I'd recommend investigating more and opening an issue with devnet developers.
If you manage to find it is actually related to starknet.py, please let us now!

Yay, the PR looks great. I had one comment through (I know it is already merged, but just in case! https://github.com/software-mansion/starknet.py/pull/115/files#r846394692)


Combine styles at stylex.create() level

Is your feature request related to a problem? Please describe.
I would like to be able to combine styles at stylex.create() level.
Also, i would like to apply existing styles, with a different pseudo class (ie. define a style, and apply it to another element with :hover pseudo class).
To illustrate my point, let's say I have a shadow style:
export const shadowStyles = stylex.create({
  base: {
    '::before': { /* ... */ },
    '::after': { /* ... */ },
  },
  level0: {
    '::before': { boxShadow: `0 0 0 0 ${colorScheme.shadow}` },
    '::after': { boxShadow: `0 0 0 0 ${colorScheme.shadow}` },
  },
  level5: {
    '::before': { boxShadow: `0 4px 4px 0 ${colorScheme.shadow}` },
    '::after': { boxShadow: `0 8px 12px 6px ${colorScheme.shadow}` },
  },
});

I want it to be applied like this:

Describe a solution you'd like
Ideally, I would like to use it like this:
export const buttonStyles = stylex.create({
  container: {
    className: {
      default: stylex.apply(shadowStyles.base, shadowStyles.level0),
      ':hover': stylex.apply(shadowStyles.level5),
    },
  },
});

export const Button = ({ children }) =>
{children}
;

Does it make sense to you?
Describe alternatives you've considered
I don't have any alternative.

I would like to be able to combine styles at stylex.create() level.

This is possible by using and spreading the result of stylex.include

Also, i would like to apply existing styles, with a different pseudo class

This is not possible. You can only spread styles on the top level.

This is possible by using and spreading the result of stylex.include.

Thanks, this works well and definitely should be documented ^_^

@olivierpascal We haven't documented it for a couple of reasons:

We have seen near-zero usage of stylex.include internally, so we're not convinced developers will find much use for it.
You can only spread it at the top-level of styles. Trying to use it within ::before will break. This exception makes the API a bit messy in my opinion.

We're going to leave it undocumented for a while and see how often people ask for or need something like it.

I've also been wondering if stylex.include actually works because I haven't been able to get it to work so far. I can see it being used to encapsulate small groups of css into reusable components essentially that can be used to build more layered style collections inside stylex rather then in components. An example would be as follows:

const commonFrame = stylex.create({
  base: {
    height: `100%`,
    width: `100%`,
    paddingTop: `0`,
    paddingBottom: `0`,
    paddingLeft: `30px`,
    paddingRight: `30px`,
    position: 'relative',
    margin: 'auto',
    minHeight: `100px`,
    display: 'block',
  },
});
const frame = stylex.create({
  content: {
    ...stylex.include(commonFrame.base),
    paddingLeft: {
      // eslint-disable-next-line @stylexjs/valid-styles
      [greaterThanIncSmallDesktop]: `210px`,
    },
    paddingRight: {
      // eslint-disable-next-line @stylexjs/valid-styles
      [greaterThanIncSmallDesktop]: `210px`,
    },
  },
});
const flex = stylex.create({
  container: {
    display: 'flex',
    minHeight: 0,
    minWidth: 0,
    flexDirection: 'row',
    flexWrap: 'wrap',
    justifyContent: 'space-between',
    alignItems: 'flex-start',
  },
});
const s = stylex.create({
  wrapper: {
    ...stylex.include(frame.content),
    ...stylex.include(flex.container),
    height: 'auto',
    width: '100%',
    flexFlow: 'column',
    flexWrap: 'wrap',
    justifyContent: 'flex-start',
    backgroundColor: '#060908',
    gap: `50px`,
    paddingTop: `100px`,
    paddingBottom: `100px`,
  },
});

export const El = () => (

item

)

At the moment the above does not seem to work, it only works if applying the separate stylex classnames directly to the component itself:

export const El = () => (

item

)

I'm investigating why stylex.include is not working for you.
However, I do see a couple of issues:
content: {
  ...stylex.include(commonFrame.base),
  paddingInline: {
    default: 30
    [greaterThanIncSmallDesktop]: 210,
  },
},

You can't override just for a media query. If you override paddingLeft you have to override all of it. (I combined paddingLeft and paddingRight into paddingInline for you)

I'm investigating why stylex.include is not working for you.
However, I do see a couple of issues:
content: {
  ...stylex.include(commonFrame.base),
  paddingInline: {
    default: 30
    [greaterThanIncSmallDesktop]: 210,
  },
},

You can't override just for a media query. If you override paddingLeft you have to override all of it. (I combined paddingLeft and paddingRight into paddingInline for you)

Was there a reason for not allowing overrides just for a media query? I'm thinking of situations where you might want a base space for mobile devices, and then larger ones for different sections on desktop where you wouldn't want to necessarily duplicate default values where they could just be built up?

Was there a reason for not allowing overrides just for a media query?

Yes. A lot of thought has gone into this. I'll try to summarise:

It's a design philosophy: You shouldn't think about "mobile" styles vs "desktop" styles. You have styles and some of them adapt to screen size. This incentives fluid design and reduces over-reliance on media queries and reduces the total amount of CSS needed.
Consistent merging of styles. If you pass a style with margin:0 to a component, you want the margin to be 0. You don't want to go check the source for all the media queries it might be using etc. The current model makes this reliable.
Future RN compatibility. Without getting too in the weeds, RN doesn't natively support Media queries, and doesn't have the same specificity design. The current design is compatible, separate media queries would break.

Was there a reason for not allowing overrides just for a media query?

Yes. A lot of thought has gone into this. I'll try to summarise:

It's a design philosophy: You shouldn't think about "mobile" styles vs "desktop" styles. You have styles and some of them adapt to screen size. This incentives fluid design and reduces over-reliance on media queries and reduces the total amount of CSS needed.
Consistent merging of styles. If you pass a style with margin:0 to a component, you want the margin to be 0. You don't want to go check the source for all the media queries it might be using etc. The current model makes this reliable.
Future RN compatibility. Without getting too in the weeds, RN doesn't natively support Media queries, and doesn't have the same specificity design. The current design is compatible, separate media queries would break.

@nmn any chance includes becomes official and documented? Being able to use it becomes essential to crating a wide-scale design system; creating styles that inherit others' is a basic OOP feature to keep everything sane and make Stylex really usable.

Use the array syntax to combine styles. There is no functional difference from includes. We've explained this many times over
@necolas Thanks for your response, can you point me on the right direction where I can find these explanations? I've tried and could not find anything

https://stylexjs.com/docs/learn/styling-ui/using-styles/#merging-styles


Stack channel logging issue

Hey! I'm having an issue when using the package because of line 46 of BrefServiceProvider:
// We change Laravel's default log destination to stderr
$logDriver = Config::get('logging.default');
if ($logDriver === 'stack') {
    Config::set('logging.default', 'stderr');
}

We're using multi-channel logging, and our stack channel already include stderr. Config bellow:
'stack' => [
    'driver' => 'stack',
    'channels' => ['stderr', 'bugsnag'],
    'ignore_exceptions' => false,
],

When using the package the bugsnag just stop working.
Couldn't we just leave this to user or at least change approach to just ensure stack contains stderr instead of changing logging.default (like below example)?
// We change Laravel's default log destination to stderr
$logDriver = Config::get('logging.default');
if ($logDriver === 'stack') {
    $channels = Config::get('logging.channels.stack.channels', []);
    array_push($channels, 'stderr');
    Config::set('logging.channels.stack.channels', array_unique($channels));
}

Hi, that's an interesting approach and I like it.
However, we need to make sure that the logger does not try to write to disk: would you solution make sure of that?

Well we could filter channels to remove all that use filesystem as storage. User still could use some custom channel that writes to disk, but I think the risk is very low.
// We remove single and daily channels and ensure that stderr is used
$logDriver = Config::get('logging.default');
if ($logDriver === 'stack') {
    $channels = array_filter(Config::get('logging.channels.stack.channels', []), function ($value) {
        return !in_array($value, ['single', 'daily']);
    });
    array_push($channels, 'stderr');
    Config::set('logging.channels.stack.channels', array_unique($channels));
}

It's not a perfect solution, but I think can solve most use cases. Syslog and errorlog aren't a problem, right?
What do you think?

Right, that makes sense. However that sounds fragile to me: new versions of Laravel could very well rename single to something else for example.
I would love to find a solution that's a bit more reliable for the future, but I'm not sure what/how 🤔

Right, that makes sense. However that sounds fragile to me: new versions of Laravel could very well rename single to something else for example.
I would love to find a solution that's a bit more reliable for the future, but I'm not sure what/how 🤔

As one who also uses Bugsnag and is looking to use this project in production, I'd rather accept the risk of the filesystem being implemented in a Laravel upgrade where my radar is up for potential issues than sacrifice Bugsnag support.  So I'd be a fan of just eliminating the logging file aliases.

The variable name $logDriver is a bit misleading. The code reads from the configuration 'logging.default', which is actually the channel name.
I think the easiest solution just set the LOG_CHANNEL environment variable to be a custom one, e.g. lambda-stack, and put only stderr and bugsnag into the channels array.
        'lambda-stack' => [
            'driver' => 'stack',
            'channels' => [
                'bugsnag',
                'stderr',
            ],
            'ignore_exceptions' => false,
        ],

Just got stung by this issue while trying to add Flare to the stack log channel.
One solution might be to add a new log channel named serverless to the logging config, then change the code snippet in the OP to always set the default logging channel to serverless. By default that could only include the stderr driver, but users could add whatever other drivers they want as well. The docs would need to point this out and mention that channels that expect access to a filesystem shouldn't be used.
In any case, this issue definitely needs mentioning in the docs.


Fix overflow-x issue for coding posts

Bug
Due to the browser vertical scrollbar, having 100vw for the coding posts can cause a horizontal overflow and horizontal scrollbar for posts that have a coding section like this demo article
Fix
overflow-x: hidden on <main>
Because div.highlighter.rouge does not have a parent with width: 100% a conventional solution would involve using hacky code as presented in this article
Unless, you would prefer a structural change in the code of this project I think my solution is sufficient and to my knowledge does not present any additional issues.

Could you send a screenshot maybe? I can't reproduce this.

Yes, I have a suspicion this might be OS dependent but I can't verify that as of now.
I'm Chrome 81 and Windows 10

You can see in the above screenshot that there is an horizontal scrollbar due to the content overflow. This only happens in posts that contain coding sections.

@thelehhman To reproduce in Chrome devtools, you can use the device tool and set
width: 1491
height: 800

Okay. I was able to reproduce it on windows. Works fine on macOS though.

Thanks!


Cherry index sample

cherry-pick index sample op cn doc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.17 out of 18 committers have signed the CLA.:white_check_mark: zhouwei25:white_check_mark: silingtong123:white_check_mark: cryoco:white_check_mark: JepsonWong:white_check_mark: ceci3:white_check_mark: wzzju:white_check_mark: suytingwan:white_check_mark: lfchener:white_check_mark: GaoWei8:white_check_mark: littletomatodonkey:white_check_mark: tianshuo78520a:white_check_mark: MrChengmo:white_check_mark: lijianshe02:white_check_mark: zhiqiu:white_check_mark: willthefrog:white_check_mark: zhupengyang:white_check_mark: qingqing01:x: Zhang TingZhang Ting seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


EndpointSlice 1.22 Docs Update (KEP 752)

This PR includes the doc updates required since EndpointSlice is going GA in 1.22.

KEP: EndpointSlice Enhancement: EndpointSlice KEP
Enhancement Issue: https://github.com/kubernetes/enhancements/issues/752
Relevant Kubernetes PRs:

kubernetes/kubernetes#103306
kubernetes/kubernetes#103451
kubernetes/kubernetes#103520

thank you for opening this, please remember to hold until the enhancement is no longer labelled 'At Risk'.
/sig network
/assign @ritpanjw

https://github.com/kubernetes/kubernetes/pull/103596 suggests this PR might also need to cover EndpointSliceTerminatingCondition.

The enhancement is no longer at risk. All relevant PRs were merged yesterday.
This PR is to cover the changes related to https://github.com/kubernetes/enhancements/issues/752
kubernetes/kubernetes#103596 is for a different KEP. I believe EndpointSliceTerminatingCondition is for https://github.com/kubernetes/enhancements/issues/1672 and will be covered by a different docs PR

Hello @swetharepakula :wave: please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 20th of July, 11:59 PM PDT.  In comments, you mentioned it is WIP, hence sending reminder. Thank you!

Hello @swetharepakula 👋 please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 20th of July, 11:59 PM PDT. In comments, you mentioned it is WIP, hence sending reminder. Thank you!

Hi @swetharepakula friendly reminder that today is the deadline for PRs ready for review, please make sure this PR is ready for review by the end of the day, thank you!

Thanks for the reminder @ritpanjw & @PI-Victor! The PR is now ready for review

/cc @robscott @aojea

This PR LGTM from technical side, one question for docs reviewers like @sftim and @kbhawkey: Do we ever allow docs to reference previous versions? In this case it might be helpful to note that the warning annotation value would be set in 1.21 if users want to know if any of their Endpoints will be truncated when they upgrade to 1.22.

Do we ever allow docs to reference previous versions?

Usually, the way we'd help readers is by advising them to check they're viewing the docs for their version of Kubernetes.
We can backport advice to older docs revisions if that makes sense to do so.

LGTM

Usually, the way we'd help readers is by advising them to check they're viewing the docs for their version of Kubernetes.
We can backport advice to older docs revisions if that makes sense to do so.

Thanks for the quick response @sftim! That approach makes sense to me. I think the docs for 1.21 are already sufficient here.

Next step is a tech review from SIG Network
@kubernetes/sig-network-pr-reviews please check this one out - it should be straightforward to verify.

/approve
/hold
Release docs folks, feel free to unhold

@swetharepakula can you please rebase, a fix that fixes the feature gates merged and cause this to conflict.

PR has been rebased.
Is there anything left for this PR? Or do we still need the hold?

a tech LGTM from @kubernetes/sig-network-pr-reviews is all that is needed, when this is done we can remove the hold.

We have the tech review I think: https://github.com/kubernetes/website/pull/28745#issuecomment-883752047
/hold cancel

/lgtm


Define BLST_NO_ASM for other ISAs

I have no way of testing this, but I believe this is what we need to do in order to support other architectures.
Does anyone have a system they could verify this works?
Might fix #398.

See https://github.com/ethereum/c-kzg-4844/issues/398#issuecomment-1960929643
It may be a toolchain/cross-compilation setup issue, continuing to investigate.


trusted proxies gets overwritten

Hi
I have a case where the trusted proxies should allow all ($proxies = '*';).
It seems that the package overwrites the config that's been set in the TrustProxies middleware.
My solution now is to overwrite the config again in the AppServiceProvider. Is this the preferred way?
Shouldn't it be better to provide a config/serverless.php configuration file, and make that behaviour configurable?

The package automatically allows <IP_ADDRESS>/0 which essentially would be the same as *, no?
Can you help me understand a use case that works with * only?

I'm relatively new to serverless so I'm not sure if I can provide much details.
In my case I have to overwrite the setting again or else my redirects, generated urls, asset urls, ... are all generated with the wrong domain.

Someone faced this issue again today (see https://github.com/brefphp/bref/discussions/1159)
I backtraced this to this PR in laravel https://github.com/laravel/framework/pull/38295
Since laravel 8, they backported the Trustproxies middleware inside Laravel itself but dropped the possibility to configure it through config

In contrary to the original package, all configuration is done through overwriting properties

Not sure what is the proper Laravel way to handle this. Maybe we should just update the Bref documentation as it's done for Symfony ATM https://bref.sh/docs/frameworks/symfony.html#trust-api-gateway

@t-richard thanks a lot for tracking this!
Maybe our immediate option would be to document this then indeed. PRs are welcome!

@mnapoli actually someone already went ahead and opened a PR https://github.com/brefphp/bref/pull/1160

👍  it's merged and live: https://bref.sh/docs/frameworks/laravel.html#trusted-proxies


remove timestamp

Hello bertranddemiddelaer,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.
Status report is not available.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

the author

one peer

tests will fail as long as the artifacts server is not at least 4.0.9

LGTM, Merge it when tests are green :)

/approve

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/1.0

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.
The following options are set: approve

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/1.0

Please check the status of the associated issue RELENG-5233.
Goodbye bertranddemiddelaer.


Issues with checkboxes

I've been having issues getting checkboxes to work.
When using the exact code in
http://davidkpiano.github.io/react-redux-form/#/recipe/complex-values
The events are fired, the model updates but the checkbox does not get checked.

I ideally want to get it working with ReactBootstrap, I have tried both using controls.checkbox and controls .text.
export const CheckboxFormField = createFieldClass({
    'Input': controls.checkbox
});

<CheckboxFormField model="options.type[]"  validators={{required: (vals) => vals.length}}>
                        <Input type="checkbox" label="SMS" value="sms" />
                        <Input type="checkbox" label="Email" value="email" />
                    </CheckboxFormField>

Sorry about that! This change is now fixed and published in<EMAIL_ADDRESS>The issue had to do with how <Field> knew which "updating" action to choose for each control. It didn't think that Input was a checkbox, but now it does, as long as it has type="checkbox".

So this is working with CheckboxFormField now? We have react-bootstrap as well and the complex-values examples doesn't work properly: instead of appending hobbies to the list, it just overrides it. Even we use plain input elements.
I'll report if the CheckboxFormField works for use.

@sattaman were does the controls.checkbox property come from?

Sorry. import {controls} from 'react-redux-form';

Yes, it should be working with checkboxes now. Let me know if you have further issues.

@muuki88 yes that's exactly what I found, I got it working with a singe checkbox in the end like this:
export const CheckboxFormField = createFieldClass({
    'Input': (props) =>
        ({onChange: (e, val) => { props.onChange(!props.checked)} })
});

   <CheckboxFormField model="myModel.field">
                <Input type="checkbox" label="Check" checked={myModel.field} />
            </CheckboxFormField>

@davidkpiano Another issue I had was, since all the bootstrap elements use the same control 'Input', something like
export const CheckboxFormField = createFieldClass({
    'Input':controls.checkbox
});

export const TextFormField = createFieldClass({
    'Input': controls.text
});

Didnt work

We figured it out. Our error was that we didn't use the correct model paths. Because nesting reducers with combineReducers is the same as model in nested state.
This jsfiddle demonstrates how things are working now. Why it took us so long: all other form inputs seem to work (text fields, radio buttons). The checkbox however has an internal checked state.
Anyway, thanks for your great library and the extensive documentation :)

Awesome, glad you got it working @muuki88 ! Let me know if there's anything you'd like to see in the docs and I'll be sure to add it.


split a video file by given time period

Problem Statement
split a video file by given time period

Hey,
I would like to do this project
If nobody has taken it, assign this to me .
cheers!

Ok @kana800


MAINT: sparse: Simplify sputils.isintlike

This works around gh-7860 by not calling np.isscalar, as suggested by @eric-wieser here.
Some objects that fail sputils.isscalarlike will pass the np.ndim check (i.e. object), but these are handled by the try-except check, which should be cheap for anything where np.ndim(x) == 0.

Should this actually use operator.index?

Yes, but that would be a compatibility break. Perhaps
try:
    operator.index(x)
except TypeError:
    if _old_intlike(x):
        print_a_scary_warning()
        return True
    else:
        return False
else:
    return True

Of course, really the error should propagate too

yeah, for 1.0.0 we probably want the backward compatible solution, we can iterate more in 1.1.0

I added the suggested deprecation warning, for the case where opertor.index(x) fails but int(x) == x.

@eric-wieser I thought I was forgetting something with that warning...
I also added a filter to the unit test so pytest doesn't fail when the warning is thrown.


Update README.md

Corrected a few typos and a broken link. Made a few editorial changes to have sentences flow better and use more accurate terminology where appropriate.

Thanks a lot!


DSA container failed due to operation not permitted

Hi, I am trying to run DSA plugin on my Ubuntu using kubectl command below.
kubectl apply -k https://github.com/intel/intel-device-plugins-for-kubernetes/deployments/dsa_plugin?ref=main
The command deploys the daemonset but the pod is in CreateContainerError status.
NAME                           READY   STATUS                 RESTARTS        AGE
intel-dsa-plugin-d4qgj   0/1     CreateContainerError   0               4s
By checking on the pod message, there is an error mentioning about operation not permitted.

Any ideas?

Any ideas?

Is the idxd module loaded?

Thank you for pointing that out, I didn't load idxd module.
Can I know how to get it?

@yengliong it's part of mainline kernel, we have been using 5.15+ kernels.

After loading idxd driver, the container is running.
I want to verify the plugin with command below.
kubectl get nodes -o go-template='{{range .items}}{{.metadata.name}}{{"\n"}}{{range $k,$v:=.status.allocatable}}{{"  "}}{{$k}}{{": "}}{{$v}}{{"\n"}}{{end}}{{end}}' | grep '^\([^ ]\)\|\(  dsa\)'
The output only shows me node1.
I didn't see the information like dsa.intel.com/wq-user-dedicated or dsa.intel.com/wq-user-shared.
Btw, my kernel is 5.11.0-16. Maybe the kernel version causes the issue?

I didn't see the information like dsa.intel.com/wq-user-dedicated or dsa.intel.com/wq-user-shared.

idxd does not create any of these automatically but they must be configured using accel-config. We provide an initcontainer that provisions a default configuration. DSA README documents it.

@yengliong OK to close?

Hi @mythi Yes, please go ahead to close it. I will go through the DSA README, thank you :)

@yengliong Thanks! Please re-open or submit another issue if you run into problems.


Use the native scheduler for tcloud demos

Just going to merge this given it's a simple change. I'm intentionally leaving the daily cron job in github actions to show how you can have the best of both worlds with the native scheduler and a 3rd party cron job.


Fix Test_gui_run_cmd_in_terminal

When the terminal feature is not available (e.g. winpty.dll is missing),
the Test_gui_run_cmd_in_terminal test fails.
Check the terminal feature.

Codecov Report

Merging #6084 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6084   +/-   ##
=======================================
  Coverage   87.24%   87.24%
=======================================
  Files         142      142
  Lines      156832   156832
=======================================
+ Hits       136832   136834    +2
+ Misses      20000    19998    -2

Impacted Files
Coverage Δ

src/gui.c
63.30% <0.00%> (-0.76%)
:arrow_down:

src/misc1.c
89.18% <0.00%> (-0.22%)
:arrow_down:

src/gui_gtk_x11.c
57.69% <0.00%> (-0.20%)
:arrow_down:

src/sign.c
94.77% <0.00%> (-0.18%)
:arrow_down:

src/message.c
88.59% <0.00%> (-0.05%)
:arrow_down:

src/terminal.c
88.11% <0.00%> (-0.04%)
:arrow_down:

src/os_unix.c
70.85% <0.00%> (+0.26%)
:arrow_up:

src/gui_gtk.c
31.33% <0.00%> (+0.27%)
:arrow_up:

src/netbeans.c
75.25% <0.00%> (+0.37%)
:arrow_up:

src/ui.c
83.51% <0.00%> (+3.84%)
: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 a5d0077...cd446aa. Read the comment docs.


Create a chart type "template" or call it "Basic chart type".

Expected Behavior

Create a basic chart type to illustrate common chart component part creation progress.
Declare common variable which suitable for most of chart type,make common component variable definition have same name,and the data key value mappings could have same key name,like xAxisLabel,or just call it xLabel somelike.
Some variable could have same default value.
Implements common methods make other chart type could be use .
Code it and document it ,so anyone want to contribute a new chart type to britecharts would have acknowledgments to implement the chart type.
Current Behavior

have not create yet.
variable used to same purpose have different name.different default value.

Related to https://github.com/eventbrite/britecharts/issues/264

I've just write some piece of typescript code to illustrate the parts of chart creation.may this help?
// / <reference types = 'd3-selection' />

import {Selection} from 'd3-selection';

declare function define(...args: any[]): any;

interface ChartCreation{
       (this:void):ChartInterface;
}

interface ChartInterface {
    (_selection:Selection<any,any,any,any>):void;
    exportChart (filename) :void;
    width () :this|number;
    height():this|number;
    margin():this|number;
    on():this;
    colorSchema();
}

define(function(require){
    const {exportChart} = require('./helpers/exportChart');
    let svg;
    let exports:ChartInterface;
    let chartCreation:ChartCreation;
    chartCreation = function():ChartInterface{
        return exports;
    }
    exports =  <ChartInterface> function(_selection:Selection<any,any,any,any>){
          _selection.each(function(_data){

            });
    }
    exports.exportChart = function(filename):void{
         exportChart.call(exports, svg, filename);
    }

    return chartCreation

});

Hi @bung87!
Thanks a lot, however I think typescript is not super evident for everybody (it doesn't feel like that to me). I am working already in a small markdown file with some guidelines as you commented on the issue.
Also, I was thinking on grabbing one of the simplest charts and comment a lot inside, so that we can have a demo chart that is also a real chart.
Thanks anyways!

@Golodhros Good to know,we can move forward first,when this done back to exist code make them unified.still have lot work to do.

We would point to one of our charts from the code styleguide document.


Dart sockets don't respect iOS's VPN

Originally filed for Flutter: https://github.com/flutter/flutter/issues/41500
The issue reports that using Dart to access resources over VPN doesn't work.
I looked through Dart's source code and it appears to be using posix sockets.  Apple's documentation recommends against that: In iOS, POSIX networking is discouraged because it does not activate the cellular radio or on-demand VPN. Thus, as a general rule, you should separate the networking code from any common data processing functionality and rewrite the networking code using higher-level APIs.
source: https://developer.apple.com/library/archive/documentation/NetworkingInternet/Conceptual/NetworkingTopics/Articles/UsingSocketsandSocketStreams.html
We should implement a socket implementation based on CFSocket.

cc @ZichangG

I think this should be added.
But I'm not sure whether VM is able to distinguish ios from mac. We only have a mac implementation so far. @a-siva @rmacnak-google

We do have HOST_OS_IOS defined when TARGET_OS_IPHONE is defined, see runtime/platform/globals.h

We do distinguish between iOS and Mac, it's just that the implementations are so similar we put them in the same files (unlike how we have separated files for Android and Linux) that have a small number of ifdefs.

@ZichangG The CoreNetworking framework exists for iOS and macosx.  If you change iOS and macosx to use it, it will be easier on you since you'll be able to test it locally instead of having to test on the simulator / device all the time.

CFSocket should be available on both iOS and macOS.
An alternative to this is https://github.com/dart-lang/sdk/issues/39104, but this is likely easier to implement and should not require any breakages in the API surface.

Right. That should be easier for testing.
If we move to CFSocket, it will be another set of implementation which won't use our kqueue-based eventhandler. It is basically rewriting whole socket with higher level CFSocket and CFSocketCreateRunLoopSource .

Looked at some docs and code examples. CFSocket can be embedded into our eventhandler system.
Unfortunately, this link  explicitly says

In iOS, using sockets directly using POSIX functions or CFSocket does not automatically activate the device’s cellular modem or on-demand VPN.

What @dnfield proposed might be a good choice.
Since the only problem is activation of modem and VPN. Is it possible to turn them on programmatically for this case?

@ZichangG @dnfield  That's a bummer.  I think we are stuck with the higher level API's if CFSocket doesn't work.
#39104 sounds good and necessary but the one problem is that it doesn't solve the issue for every client of Dart.  This is going to be a problem for everyone that uses Dart on iOS, it would be nice to solve this problem as the default implementation on iOS.

The idea with 39104 is to keep the API compatible with dart:io, so that you can just switch your import from dart:io to dart:net.  I've been planning to write a doc for this, but have been delayed.

We do have some discussions for splitting dart:io into multiple smaller packages. I haven't started but this is the plan for this quarter.

what is the current state of this issue? i face app vpn from mobile iron at every enterprise customer. it is a show stopper in many cases if the app vpn is used to secure a connection.

Hi @tobiaszuercher,
As a work around you can create a custom http client and use http overriders to return your custom client when a HttpClient is requested.
Inside the custom client you can get the device proxy so that it is dynamic.
Please see the examples below of our Proxy aware http client that utilises the device_proxy package from pub dev
`import 'dart:io';
import 'package:device_proxy/device_proxy.dart';
class ProxyHttpClient implements HttpClient {
HttpClient _client;
String _proxy;
ProxyHttpClient({SecurityContext context, HttpClient client}) {
_client = client != null ? client : new HttpClient(context: context);
_client.findProxy = (uri) {
return _proxy;
};
_client.badCertificateCallback = ((
X509Certificate cert,
String host,
int port,
) =>
// TODO Disable in release mode
true);
}
Future updateProxy() async {
ProxyConfig proxyConfig = await DeviceProxy.proxyConfig;
_proxy = proxyConfig.isEnable ? 'PROXY ${proxyConfig.proxyUrl};' : 'DIRECT';
}
@override
bool get autoUncompress => _client.autoUncompress;
@override
set autoUncompress(bool value) => _client.autoUncompress = value;
@override
Duration get connectionTimeout => _client.connectionTimeout;
@override
set connectionTimeout(Duration value) => _client.connectionTimeout = value;
@override
Duration get idleTimeout => _client.idleTimeout;
@override
set idleTimeout(Duration value) => _client.idleTimeout = value;
@override
int get maxConnectionsPerHost => _client.maxConnectionsPerHost;
@override
set maxConnectionsPerHost(int value) => _client.maxConnectionsPerHost = value;
@override
String get userAgent => _client.userAgent;
@override
set userAgent(String value) => _client.userAgent = value;
@override
void addCredentials(
Uri url, String realm, HttpClientCredentials credentials) {
return _client.addCredentials(url, realm, credentials);
}
@override
void addProxyCredentials(
String host, int port, String realm, HttpClientCredentials credentials) {
return _client.addProxyCredentials(host, port, realm, credentials);
}
@override
set authenticate(
Future Function(Uri url, String scheme, String realm) f) =>
_client.authenticate = f;
@override
set authenticateProxy(
Future Function(
String host, int port, String scheme, String realm)
f) =>
_client.authenticateProxy = f;
@override
set badCertificateCallback(
bool Function(X509Certificate cert, String host, int port) callback) {
// _client.badCertificateCallback = callback;
}
@override
void close({bool force = false}) => _client.close(force: force);
@override
Future delete(String host, int port, String path) async {
await updateProxy();
return _client.delete(host, port, path);
}
@override
Future deleteUrl(Uri url) async {
await updateProxy();
return _client.deleteUrl(url);
}
@override
set findProxy(String Function(Uri url) f) {
_client.findProxy = f;
}
@override
Future get(String host, int port, String path) async {
await updateProxy();
return _client.get(host, port, path);
}
@override
Future getUrl(Uri url) async {
await updateProxy();
return _client.getUrl(url.replace(path: url.path));
}
@override
Future head(String host, int port, String path) async {
await updateProxy();
return _client.head(host, port, path);
}
@override
Future headUrl(Uri url) async {
await updateProxy();
return _client.headUrl(url);
}
@override
Future open(
String method,
String host,
int port,
String path,
) async {
await updateProxy();
return _client.open(method, host, port, path);
}
@override
Future openUrl(String method, Uri url) async {
await updateProxy();
return _client.openUrl(method, url);
}
@override
Future patch(String host, int port, String path) async {
await updateProxy();
return _client.patch(host, port, path);
}
@override
Future patchUrl(Uri url) async {
await updateProxy();
return _client.patchUrl(url);
}
@override
Future post(String host, int port, String path) async {
await updateProxy();
return _client.post(host, port, path);
}
@override
Future postUrl(Uri url) async {
await updateProxy();
return _client.postUrl(url);
}
@override
Future put(String host, int port, String path) async {
await updateProxy();
return _client.put(host, port, path);
}
@override
Future putUrl(Uri url) async {
await updateProxy();
return _client.putUrl(url);
}
}
`
A custom HttpOverrides that returns your new proxy aware client
`import 'dart:io';
import 'package:modules/core/shelf.dart';
class ProxyHttpOverrides extends HttpOverrides {
@override
HttpClient createHttpClient(SecurityContext context) {
return ProxyHttpClient(
context: context,
client: super.createHttpClient(context),
);
}
}
`
Regards,
Tarek

@thaoula thank you a lot for your code! i still don't have access to an app-vpn, but i'll try as soon as the environment is there!

With per-apn vpn the suggested solution doesn't work, since it might not be "just" a proxy.
I've tested with VMware Airwatch, using flutter and the host cannot be reached.
Bypassing the dart HTTP ( by the usage of a custom plugin that routes requests to native ios code ), the per-app VPN captures and forwards those requests/responses successfully.

Any news on this issue? Will there be a built-in solution in the near future?

Hello @gaaclarke
Is there any timing for this issue?
Thanks!

@JordiGiros no

Did anyone try this with F5 BIG IP VPN? I dont have access to a working environment just yet.

cupertino_http is a new experimental Flutter plugin that provides access to Apple's Foundation URL Loading System - which honors iOS VPN settings.
cupertino_http has the same interface as package:http Client so it is easy to use in a cross-platform way. For example:
late Client client;
if (Platform.isIOS) {
  final config = URLSessionConfiguration.ephemeralSessionConfiguration()
    # Do whatever configuration you want.
    ..allowsCellularAccess = false
    ..allowsConstrainedNetworkAccess = false
    ..allowsExpensiveNetworkAccess = false;
  client = CupertinoClient.fromSessionConfiguration(config);
} else {
  client = IOClient(); // Uses an HTTP client based on dart:io
}

final response = await client.get(Uri.https(
    'www.googleapis.com',
    '/books/v1/volumes',
    {'q': 'HTTP', 'maxResults': '40', 'printType': 'books'}));

I would really appreciate it if you can try cupertino_http out and see it if solves the VPN issues for you.
Comments or bugs in the cupertino_http issue tracker would be appreciated!

Are there any news?
There's quite some impact for the company I'm working with as most of their customers have some sort of VPN setup going..

@komaxx have you tried https://pub.dev/packages/cupertino_http ?

@a-siva Good point, I'll give it a try!
However, it's still marked "experimental", right? I'm a bit reluctant to include not-yet-stable code in this app since there are business app stores and certifications in play - updating the app is not a quick process :/

@komaxx it is currently marked as "experimental" as this package is fairly new and we are soliciting feedback from users, our plan is to address all the initial feedback we receive and move it out of the "experimental" state.

@GoncaloPT have you tried  https://pub.dev/packages/cupertino_http for your VPN problem.

Hello @a-siva. Would it support websocket connections?
Last time i checked the package was very rudimentary and lacked support for websocket connections.
Also, as probably all of us that post in this thread, I use flutter in a enterprise context; so using experimental packages is not something i'm eager to jump into :)

An interesting note about using VPN with BSD sockets: https://developer.apple.com/forums/thread/76448

I filed a bug to track adding websocket support in package:cupertino_http.


[SPARK-6972][SQL] Add Coalesce to DataFrame

Test build #30442 has started for   PR 5545 at commit 9fdf3f6.

Test build #30442 has finished for   PR 5545 at commit 9fdf3f6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

/cc @rxin

LGTM. Merged in master.

How do I can implement this? I've built spark using the newest version of master, that contains this code, but IntelliJ still doesn't recognize the coalesce(1)

Maybe reimport the project?


Rename .cs file does not change  in csproj. files; changes to .csproj not applied until vscode is reloaded.

VS Code version: 1.19.2
C# Extension version: 1.13.0
These issues make renaming files .cs files in vscode a pain in the butt. Not sure if these are bugs/feature requests, how hard it would be to fix them or if they were reported in the past (couldn't find dup issue), but here is the situation.
I tested the same thing in Rider and Rider smoothly handles both of them.

Have foo.cs in bar.csproj "Compile Include=foo.cs"
Rename foo.cs to foo2.cs
foo2.cs is outside of the project now and c# support breaks down, OR foo2.cs is kept as if in project but without actually changing csproj file, so issue surfaces on the next reload.

Correct thing to do is to change matching Compile Include-s when renaming .cs files.

open bar.csproj
manually change "Compile Include=foo.cs" to "Compile Include=foo2.cs"
change is not applied until vscode is fully reloaded

Vscode should listen to changes to csproj files and respond accordingly. Not that a lot of people would be manually changing csproj files, but 3rd party like Unity, which uses VSCode as IDE, do.
edit: after testing some more I think the second part actually works, or fails very rarely. I will add more info if I find a repro. The first part fails consistently

I've had the same issues in the past. I think what I've done is used unity for the creation/deletion of files and used VSCode for the actual code-editting. It's not an optimal workflow but I learned to deal with it. Alternatively if you want full project a .csproj support, the easiest workflow would probably be to use VSCommunity as this is both supported by Unity, and has extensions to provide additional Unity support (IRC)..
I know MonoDevelop is provided out-of-the-box but I've seen a lot of really cruddy and strange behaviour when using MonoDevelop with Unity, so I would still recommend VSCode + Unity for file management or full on VSCommunity, at least until Omnisharp/VSCode decide that they want to undertake the management of .csproj files.

As per OmniSharp/omnisharp-roslyn#1159 this problem should be fixed.
@SirIntruder Can you confirm if we can close this now ?
Can someone here try setting "omnisharp.path": "latest" and checking if the error still persists?

Thanks! You can close this, sure thing
Note that there will still be errors. Difference is that they errors will go away as soon as Unity3d is focused (which regenerates .csproj files), and people shouldn't need to restart omnisharp anymore

Ya I've noticed that OmniSharp doesn't run regular design time builds so it ends up missing added Compile items. My 2cents

@SirIntruder Thanks 💃


Fix/tabs discard dead tab

Fix #487
Fix #496
Fix #505

Added that this fixes #487.

@bsclifton I approved but new commits were pushed marking it 'stale'. FWIW all 3 issues are resolved as far as I'm concerned, and I've found no new issues.


Fix bug where malformed module headers would cause null pointer exceptions

This PR makes PSFile#getModule return PSModule? instead of PSModule. The reason is that purescript files with invalid module headers may not get a module element at all, and that causes null pointer exceptions down the line.
This change trickled down in a lot of different places, and I took the opportunity to go through and clean up some of the boilerplate in the tests.

@simonolander the null pointers have been helpful in finding that there are files that did not parse correctly. I have been thinking that there maybe should be a error thrown with more details, what file did not parse etc.
Now when we don't throw any exception but instead return empty collections etc, we don't get this warnings. do you have any idea on how to re introduce warnings without crashing?


Add new presentations to README.md

Signed-off-by: Ed Warnicke<EMAIL_ADDRESS>

This still has the formatting problem ...


Update publish-an-extension.md

Description

Added a paragraph about sparse checkouts to improve the speed of git operations. The solution is based on this discussion in the community.
Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Just tried it and it worked, but there is a missing step that I think some people will miss. After cloning, the folder will be empty, so you need to create an /extensions folder at the root, and add your extension folder in there (ex: emoji-search).
What do you think?

The extensions repository is big and to keep your git operations fast locally

Maybe be more specific with: "... and to avoid pulling all its content"?

This is especially useful when iterating on an existing extension.

I agree but it's useful also when you create an extension, so maybe rephrase it to "This is also useful"?


cluster setup refactor

Signed-off-by: qingwave<EMAIL_ADDRESS>
What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Reduce cyclomatic complexity of function cluster.Up()
Which issue(s) this PR fixes:

Part of #107
Special notes for your reviewer:
Does this PR introduce a user-facing change?

cluster setup method refactoring.

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/assign @wzshiming


Desktop viewport is not rendered

Issue description
When playing on flat (desktop) mode, the desktop viewport is not rendered, with the screen being completely pink instead.

This renders the game unplayable.
Steps to reproduce

Open the game.
Go to a map. Start or join a server.

Minimal reproduction project (For SDK or engine issues)
No response
Using a released version?
No
What OS? Any other relevant information? (V-Sekai version, Godot revision, System information)
Windows 10 (but it also happens under Linux Mint), godot-groups-4.2 (v4.2.beta.custom_build [dd18872ba]). NVIDIA GeForce GTX 1060 with Forward+ (Vulkan) renderer.

Previously, I unmerged a pr but I decided against it because it's an unnecessary divergence - https://github.com/V-Sekai/godot/commit/d917975a67b9bcb887a1141b38a4c64f9f7a31b4

Fixed by https://github.com/V-Sekai/v-sekai-game/pull/348.

See https://github.com/V-Sekai/v-sekai-game/issues/351 also


drivers/spi: Add SPI Slave character device driver

Summary
This PR intends to add a new feature to the NuttX RTOS: a SPI Slave character device driver, which implements the SPI Slave device interface.
Impact
New feature, no impact if not selected.
Testing
Test setup consisted of 2 ESP32-C3 DevKit boards, the first assuming the Master role and the other Slave.
Slave enabled SPI_SLAVE_DRIVER_COLORIZE_TX_BUFFER config, with default values for configs SPI_SLAVE_DRIVER_COLORIZE_PATTERN and SPI_SLAVE_DRIVER_COLORIZE_NUM_BYTES.
Master initiates a SPI Exchange operation via spi tool application and appropriately receives the answer:
NuttShell (NSH) NuttX-10.1.0-RC1
nsh> spi exch -x 30 DEADA55ABEEFA55ADEADA55ABEEFA55ADEADA55ABEEFA55ADEADA55ABEEF
Sending:        DE AD A5 5A BE EF A5 5A DE AD A5 5A BE EF A5 5A DE AD A5 5A BE EF A5 5A DE AD A5 5A BE EF
Received:       A5 A5 A5 A5 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
nsh>

The SPI Slave driver file system operations were validated using a custom app that performed each one of the supported ops.
This is the output from Master after Slave's answer to the first transfer:
nsh> spi exch -x 30 DEADA55ABEEFA55ADEADA55ABEEFA55ADEADA55ABEEFA55ADEADA55ABEEF
Sending:        DE AD A5 5A BE EF A5 5A DE AD A5 5A BE EF A5 5A DE AD A5 5A BE EF A5 5A DE AD A5 5A BE EF
Received:       00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F 10 11 12 13 14 15 16 17 18 19 1A 1B 1C 1D
nsh>

@jlaitine @gregory-nutt I believe it is important your feedback on this. Could you please take a look? Thanks.

LGTM. So basically this implements a similar character driver for testing purposes, which exists for SPI master already. I can imagine that this may be an useful test feature, and it also demonstrates the use of QPOLL/ENQUEUE.

LGTM. So basically this implements a similar character driver for testing purposes, which exists for SPI master already. I can imagine that this may be an useful test feature, and it also demonstrates the use of QPOLL/ENQUEUE.

Yes, it is a fairly minimal driver that enables basic testing of the SPI Slave controller implementation. It has with some opportunities for improvements due to its current simplified design decision, e.g. buffer management. I'll improve the description with this limitations.

Thank you @jlaitine ! If you have the chance, please create glue for your stm32h7 SPI SLAVE to use this upper half.

Hi!
That slave works only as a receiver, and unfortunately I don't currently have any stm32h7 hw to actually test it on. So I don't think that I will continue on that for now. As a receiver only, it should work pretty much out-of-the-box also with this upper half.
The hw setup which I used was such that there were two stm chips, both having master and slave spi busses in simplex mode. So both were sending to the other one as master and receiving as slave (two chip selects, two clock lines and mosi/miso lines crossed). The slave controller was not limited to simplex mode though, it is just that I didn't implement slave writing (full duplex).
It is actually pretty cool setup, since both devices can initiate sending whenever they need, and the other one can just poll on the IF on regular basis to check if anything has arrived to the DMA buffer.
Alan Carvalho de Assis kirjoitti torstai 27. toukokuuta 2021:

Thank you @jlaitine ! If you have the chance, please create glue for your stm32h7 SPI SLAVE to use this upper half.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-nuttx/pull/3787#issuecomment-84971791


F6 Shorcut to hide Results Pane

recently updated to version 5.0.5 and the F6 keyboard shortcut (To hide the Results Pane ) is no longer working. According to the Wiki it should be working.
there's small arrow to hide it, but it does not serve the same purpose, because after executing a new query, stays hidden. (with the F6 shortcut did not)
Thanks

well.. after a few time looking through the menus, I found that now is binded to Ctrl+6 hotkey... changed back to F6 in Preferences -> General -> Keys: Togle Results Panel .
Way more comfortable with just one key
Please Update the Wiki

Wiki updated, thanks for noticing.
We have changed default shortcut because functional keys aren't always applicable (e.g. they have special meaning on MacOS and Linux, on some laptops you need to use fn+FX combination which is also not very convenient)

The "Toggle results panel" shortcut is now CTRL+T, at least on Windows.


Could not load file or assembly System.Runtime.Serialization, Version=<IP_ADDRESS>,...

Hi Daniel,
NuDoq service is great, but sometimes it fails to index assemblies with innocently-looking native .NET Framework dependencies. Here is a project that wasn't indexed because of missing System.Runtime.Serialization assembly (which belongs to .NET Framework):
http://www.nudoq.org/#!/Projects/Zyan
Error generating Zyan
Version: 2.5
Generated Date: 10/31/2013 3:09:13 AM
http://www.nuget.org/packages/Zyan
Error
System.IO.FileNotFoundException: Could not load file or assembly 'System.Runtime.Serialization, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e' or one of its dependencies. The system cannot find the file specified.
at System.ModuleHandle.ResolveType(RuntimeModule module, Int32 typeToken, IntPtr typeInstArgs, Int32 typeInstCount, IntPtr methodInstArgs, Int32 methodInstCount, ObjectHandleOnStack type) at System.ModuleHandle.ResolveTypeHandleInternal(RuntimeModule module, Int32 typeToken, RuntimeTypeHandle[] typeInstantiationContext, RuntimeTypeHandle[] methodInstantiationContext) at System.ModuleHandle.ResolveTypeHandle(Int32 typeToken, RuntimeTypeHandle[] typeInstantiationContext, RuntimeTypeHandle[] methodInstantiationContext)...

The service has been discontinued a long time ago. Sorry for not replying earlier. This project is now only about the XML doc API.

The service has been discontinued a long time ago. Sorry for not replying earlier. This project is now only about the XML doc API.


Add zipDefaultWith

Implementation taken from https://stackoverflow.com/a/43312255/503377
When I found myself copying this from StackOverflow I knew it was time to make a PR to extra. :D But maybe it's already somewhere else and I haven't found it.

It seems there should be a package for all the zip combinations people want - error on mismatch, extend, Nothing, a default value. I'm a bit hesitant about this patch for a few reasons:

The name doesn't quite imply what it does. Default could mean a few things.
Why not zipWithFoo :: (Maybe a -> Maybe b -> c) ?
Should Foo be Extend? It's not quite right. I have no idea what it should be.
Is this the useful version of the combinator? It's not one I've ever gone for or wanted, but that may be because it wasn't available - it's hard to tell in advance.

(And sorry for the delay in replying - caught COVID and have been recovering and getting back to normal.)

I'd welcome a more general version. Other languages call this zip_longest.

I'm very sorry to hear about COVID. Glad you're recovering!

Looking at Python, they take a single default value, and use it for both lists - the advantage of an untyped language - there's something somewhat unpleasant about passing two defaults, with the knowledge that at most one will ever be used.
Looking at Rust, they have essentially zipWithLongest :: (Maybe a -> Maybe b -> c) -> [a] -> [b] -> [c]. I prefer that type, although note that zipWithLongest :: (These a b -> c) -> [a] -> [b] -> [c] is a more accurate type, using These.

How do you feel about a new dependency on these?

I could also define a type just for this function that is analogous to These. Or...use anonymous sums... heh

For the extra library, definitely not, as its number of dependencies is deliberately very low. That said, it doesn't seem unreasonable for these to be in base, so if that happened, then it would be fine.

I suspect that a pair of Maybe's and a stated invariant is probably the pragmatic way - new data types need functor instances etc, and then it's competing with these library. Seems like a lot of surface area to the library for insufficient benefit.

Yes agreed.

So we've landed on zipWithLongest :: (Maybe a -> Maybe b -> c) -> [a] -> [b] -> [c] as a signature and prototype? I'm fine with that.

I added zipWithLongest so am closing this PR.

Thanks. I'm OOO for a bit and wasn't able to get to this.


Ubuntu - Nothing really happens

Hi,
I've successfully installed the pocketnet.core deb.
I've downloaded manually the checkpoints db and put in in /home/.pocketcoin/checkpoints
Open terminal
pocketcoin-qt
0000ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff
0000ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff
Segmentation fault (core dumped)

Open Log
2022-01-19T15:25:38Z Pocketnet Core version v0.20.17 (release build)
2022-01-19T15:25:38Z InitParameterInteraction: parameter interaction: -whitelistforcerelay=1 -> setting -whitelistrelay=1
2022-01-19T15:25:38Z Assuming ancestors of block 61276732ac5226282d46937b3a4609ed06e98f6621a68941c649ac507dd143bc have valid signatures.
2022-01-19T15:25:38Z Setting nMinimumChainWork=00000000000000000000000000000000000000000000002e9b7b6eb4a23574f0
2022-01-19T15:25:38Z Using the 'shani(1way,2way)' SHA256 implementation
2022-01-19T15:25:38Z Using RdRand as an additional entropy source
2022-01-19T15:25:38Z Default data directory /home/ddy/.pocketcoin
2022-01-19T15:25:38Z Using data directory /home/ddy/.pocketcoin
2022-01-19T15:25:38Z Config file: /home/ddy/.pocketcoin/pocketcoin.conf (not found, skipping)
2022-01-19T15:25:38Z Using at most 125 automatic connections (1024 file descriptors available)
2022-01-19T15:25:38Z Using 16 MiB out of 32/2 requested for signature cache, able to store 524288 elements
2022-01-19T15:25:38Z Using 16 MiB out of 32/2 requested for script execution cache, able to store 524288 elements
2022-01-19T15:25:38Z Using 8 threads for script verification
2022-01-19T15:25:38Z init message: Loading Pocket DB...
2022-01-19T15:25:38Z scheduler thread start
2022-01-19T15:25:38Z SQLite usage version: 3037000
2022-01-19T15:25:38Z ErrorLogCallback: 283; Message: recovered 302 frames from WAL file /home/ddy/.pocketcoin/pocketdb/main.sqlite3-wal
2022-01-19T15:25:38Z Creating Sqlite database main.sqlite3 structure..
2022-01-19T15:25:38Z Creating Sqlite database web.sqlite3 structure..
2022-01-19T15:25:38Z Using wallet directory /home/ddy/.pocketcoin/wallets
2022-01-19T15:25:38Z init message: Verifying wallet(s)...
2022-01-19T15:25:38Z Using BerkeleyDB version Berkeley DB 4.8.30: (April  9, 2010)
2022-01-19T15:25:38Z Using wallet wallet.dat
2022-01-19T15:25:38Z BerkeleyEnvironment::Open: LogDir=/home/ddy/.pocketcoin/wallets/database ErrorFile=/home/ddy/.pocketcoin/wallets/db.log
2022-01-19T15:25:38Z Cache configuration:
2022-01-19T15:25:38Z * Using 2.0MiB for block index database
2022-01-19T15:25:38Z * Using 8.0MiB for chain state database
2022-01-19T15:25:38Z * Using 160.0MiB for in-memory UTXO set (plus up to 286.1MiB of unused mempool space)
2022-01-19T15:25:38Z init message: Loading block index...
2022-01-19T15:25:38Z Opening LevelDB in /home/ddy/.pocketcoin/blocks/index
2022-01-19T15:25:38Z Opened LevelDB successfully
2022-01-19T15:25:38Z Using obfuscation key for /home/ddy/.pocketcoin/blocks/index:<PHONE_NUMBER>000000
2022-01-19T15:25:38Z LoadBlockIndexDB: last block file = 0
2022-01-19T15:25:38Z LoadBlockIndexDB: last block file info: CBlockFileInfo(blocks=0, size=0, heights=0...0, time=1970-01-01...1970-01-01)
2022-01-19T15:25:38Z Checking all blk files are present...
2022-01-19T15:25:38Z Initializing databases...
2022-01-19T15:25:38Z Opening LevelDB in /home/ddy/.pocketcoin/chainstate
2022-01-19T15:25:38Z Opened LevelDB successfully
2022-01-19T15:25:38Z Using obfuscation key for /home/ddy/.pocketcoin/chainstate: 8076e700f0e92c4a
2022-01-19T15:25:38Z init message: Rewinding blocks...
2022-01-19T15:25:38Z  block index              14ms
2022-01-19T15:25:38Z Opening LevelDB in /home/ddy/.pocketcoin/indexes/txindex
2022-01-19T15:25:38Z Opened LevelDB successfully
2022-01-19T15:25:38Z Using obfuscation key for /home/ddy/.pocketcoin/indexes/txindex:<PHONE_NUMBER>000000
2022-01-19T15:25:38Z init message: Loading wallet...
2022-01-19T15:25:38Z txindex thread start
2022-01-19T15:25:38Z txindex is enabled
2022-01-19T15:25:38Z txindex thread exit
2022-01-19T15:25:38Z Failed to open mempool file from disk. Continuing anyway.
2022-01-19T15:25:38Z mapBlockIndex.size() = 1
2022-01-19T15:25:38Z Best block: 00000fd0f6633d395541056e8adc32961e15f8133674b2e3937c4d210ced6f3f (0)
2022-01-19T15:25:38Z Staker thread started for
2022-01-19T15:25:38Z Bound to [::]:37070
2022-01-19T15:25:38Z Bound to <IP_ADDRESS>:37070
2022-01-19T15:25:38Z init message: Loading P2P addresses...
2022-01-19T15:25:38Z torcontrol thread start
2022-01-19T15:25:38Z Loaded 0 addresses from peers.dat  1ms
2022-01-19T15:25:38Z init message: Loading banlist...
2022-01-19T15:25:38Z init message: Starting network threads...
2022-01-19T15:25:38Z net thread start
2022-01-19T15:25:38Z dnsseed thread start
2022-01-19T15:25:38Z Loading addresses from DNS seeds (could take a while)
2022-01-19T15:25:38Z addcon thread start
2022-01-19T15:25:38Z 0 addresses found from DNS seeds
2022-01-19T15:25:38Z dnsseed thread exit
2022-01-19T15:25:38Z msghand thread start
2022-01-19T15:25:38Z opencon thread start
2022-01-19T15:25:38Z init message: Done loading
2022-01-19T15:25:38Z GUI: Platform customization: "other"
2022-01-19T15:25:38Z GUI: PaymentServer::LoadRootCAs: Loaded  125  root certificates
Any idea?
Then, how do I configure everything, including my wallet addres?!
The instructions in the readme.MD are not clear at all...

Maybe try to run in debugging using GDB ? This may give more information about the problem
$ sudo apt install gdb
$ gdb pocketcoind

Not that useful:
ddy@ddy--50909:~$ gdb pocketcoind
GNU gdb (Ubuntu 11.1-0ubuntu2) 11.1
Copyright (C) 2021 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
https://www.gnu.org/software/gdb/bugs/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from pocketcoind...
(No debugging symbols found in pocketcoind)
(gdb)
Am I missing something?

Everything is fine, you need to enter "r" to run

From the terminal output it is crashing with a segmentation fault.  What Ubuntu version are you running?  What is your system hardware configuration?

From the terminal output it is crashing with a segmentation fault. What Ubuntu version are you running? What is your system hardware configuration?

Ubuntu 21.10
CPU i5-1135g7
16 gb ram
250 GB SSD

I have a repro on this segmentation fault when performing a clean install and attempting to run pocketcoin-qt on Ubuntu 21.10.  If possible it would be good for us to get a fix into 20.18.

I have a repro on this segmentation fault when performing a clean install and attempting to run pocketcoin-qt on Ubuntu 21.10. If possible it would be good for us to get a fix into 20.18.

@tawmaz Do you have a solution?

I will investigate some more today.  The segmentation fault does not show a stack, and appears to be crashing inside the QT library.  As a workaround it is possible to run pocketcoind, but the GUI does not appear to work on Ubuntu 21.10.
There are two messages printed to the console before crashing:
"Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome.  Use QT_QPA_PLATFORM=wayland to run on Wayland anyway."
"Attribute Qt::AA_EnableHighDpiScaling must be set before QCoreApplication is created."

I think it can no longer be released in 0.20.18 and is not urgent enough. In any case, we plan to abandon the bitcoin GUI in the future, replacing it with a web interface.

I am running into a some issues with Ubuntu 21.10 requiring me to re-install.  I would suggest we recommend Ubuntu 20.04 in the setup guide.

Fix: https://github.com/pocketnetteam/pocketnet.core/pull/176

@Dylan-86 please try with 0.20.18 as it has a segmentation fault fix with pocketcoin-qt.  Feel free to re-open this issue if the problem is still present.


add imple shim gRPC api

current containerd define gRPC api for shim, youki can imple this gRPC api.

@lengrongfu Thanks for creating the issue. Youki is a low-level container runtime, isn't shim. Why do you want to shim?
You can use runc's shim as youki's shim because youki meets the OCI Runtime spec.

you are right, I remembered it wrong.


python37Packages.PyChromecast: 4.1.0 -> 4.1.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-pychromecast/versions.
meta.description for python37Packages.PyChromecast is: '"Library for Python 3.4+ to communicate with the Google Chromecast"'.
meta.homepage for python37Packages.PyChromecast is: '"https://github.com/balloob/pychromecast"

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.1.1 with grep in /nix/store/wa5n7mvvjp7bs18cg163mz65v9blgswp-python3.7-PyChromecast-4.1.1

directory tree listing: https://gist.github.com/5b082057be07a08cbfdad7a0b976ccce

du listing: https://gist.github.com/0a9c2a45b837832e188bcf2e6f6ed7a7

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)
4 package rebuild(s)
4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
catt
gnomecast
python37Packages.PyChromecast
python38Packages.PyChromecast

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wa5n7mvvjp7bs18cg163mz65v9blgswp-python3.7-PyChromecast-4.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.PyChromecast https://github.com/r-ryantm/nixpkgs/archive/c6494f73846d8124e4552fdea44df0a7407c2f4b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wa5n7mvvjp7bs18cg163mz65v9blgswp-python3.7-PyChromecast-4.1.1
ls -la /nix/store/wa5n7mvvjp7bs18cg163mz65v9blgswp-python3.7-PyChromecast-4.1.1/bin

cc @abbradar for testing.

@dtzWill in the future, could you use the review functionality? It makes it easier to select PR's that are likely ready to be merged.

@dtzWill in the future, could you use the review functionality? It makes it easier to select PR's that are likely ready to be merged.

Eep, will do! o7


Using LaravelCollective HTML as a real dependency

Related to missing dependency from https://github.com/bllim/laravalid/pull/21.
Original bug report: https://github.com/bllim/laravalid/issues/19

wrong PR lol


Support instantiation based on url params

Description
We need the ability to start the form itnstansiation by providing a standard link including query parameters. The case for this is correspondance messages and regular emails containging a link to start the form. In Altinn 2 one of the most used features is to send a notification to citizens/businesses about the information they need to know (roughly 60 millions per year). Typically this is information about correspondence received or a new form they need to fill out.
Additional information
Example url from GitHub: https://github.com/Altinn/app-lib-dotnet/issues/new?assignees=&labels=kind%2Fchore%2Cstatus%2Fdraft&projects=&template=chore.yml
Documentation from GitHub: https://docs.github.com/en/issues/tracking-your-work-with-issues/creating-an-issue#creating-an-issue-from-a-url-query
Acceptance criteria

[ ] Instantiation can happen with a call containing url parameters which can be used for prefill for the new instance
[ ] A hook which allows the application developers to read the parameteres in an easy way and to add custom logic before the instance is created. For example checking if a provided id is allready registered/used. The case for this is typically survey links sent to users which should not be shared between users.
[ ] If the instance shouldn't be created we should present information to the user that this link is no longer valid.
[ ] The information page should be configurable.
[ ] A way to map query parameters to the data model for Studio configurable prefil setup

Digital gravferdsmelding

Application developers should be in control of which query parameters should map to which data model fields
Application developers should be able to define valid values to go into the data model fields
How should we handle encoding of values to prevent security issues?
Individual query params vs base64 encoded param => individual
We need to fix redirect in Altinn Innboks otherwise you will end up in the inbox when you are not logged in when following the link
Should we route this via an stateless form first instead of creating an instance directly? Yes
Example url: https://app.serviceowner.altinn.no/org/app/layoutName/new?key1=value1&key2=value2
Prefil config for data model connected to stateless layout should get url params as a new source.

Discussion about this on slack:

https://altinn.slack.com/archives/C02EJ9HKQA3/p1724749995459069

Denne er viktig for oss i NVE også, så bumper denne.


Tests should run from the root directory

Today mvn test fails if run at root level. Running it at module level works fine. This needs to be fixed

Fixed via acd76083ebae5895d0d70bfc6e828c69ebffd8de


Only include pinned versions in environment name.

re: #169.
For a dependency matrix such as...
"matrix": {
    "a": [],
    "b": ["5"],
    "c": ["2.24", "2.26"]
}

... this PR changes the resulting environment names from:

py2.7-a-b5-c2.24
py2.7-a-b5-c2.26

to:

py2.7-b5-c2.24
py2.7-b5-c2.26

NB.

As mentioned on #169, this affects both the name of the testing environment and the name of the result files.
This PR does not change the circumstances under which result file names will change (e.g. going from an unpinned version to a pinned version). The obvious exception to this is the initial transition from the current naming scheme to the scheme proposed in this PR.

Thanks for submitting this.
A couple of thoughts: This doesn't really solve the problem when there are a large number of explicitly-versioned dependencies.  Also, even though it's not supported at present since the existing dependency matrix stuff is very simplistic, it may be the case that the user wants to create environments both with and without a particular dependency (to benchmark a Python vs. Cython implementation of an algorithm, for example).  What if the environment directory were the MD5 hash of the list of dependencies?  Then we can limit the number of characters to something sane in all cases, even in terrible ones.  Sorry to share this brain dump only after you filed the PR -- I didn't have a chance to get back to this yesterday.
There's also the transition problem you mention.  For the environment directories it's not a big deal -- you just throw out the old environments and there's a rebuild time hit for the first time with the new method.  For the results though, the files would all have to be renamed.  Maybe we could do this name simplification --- whether as you've proposed here or with my hash proposal -- only on the environment name (where conda cares) and not on the result files.

This doesn't really solve the problem when there are a large number of explicitly-versioned dependencies.

True. But I don't have that problem. :wink:

it may be the case that the user wants to create environments both with and without a particular dependency

That certainly sounds plausible.

What if the environment directory were the MD5 hash of the list of dependencies?

Maybe we could do this name simplification --- whether as you've proposed here or with my hash proposal -- only on the environment name (where conda cares) and not on the result files.

:+1: If necessary there's always the option of writing out one or more files that describe the contents of the environment directory.

Sorry to share this brain dump only after you filed the PR -- I didn't have a chance to get back to this yesterday.

Not a problem at all. I needed to make these changes anyway so I can keep making progress. :grinning:

Closed in favor of #181.

:+1:


lv_indev_get_point strange behaviour

i'm trying to implement calibration screen, but i'm having strange issue.
my touch driver seems to work fine, if i hardcode some offset and scale values inside it.
but when i try to read lv_indev_get_point() function, it just returns [605,0] everytime i execute it.
i'm using the same method, that is used in calibration app - big button and state machine.
i've tried going with my own idea first, without button and using lv_indev_wait_release(), but somehow it was not working at all (may be it is also a bug, i'm not sure, but it was returning without a delay at all).

i've even done this in SysTick handler that fires every ms:
	char buf[16];
	lv_indev_t *indev = lv_indev_get_act();
	lv_indev_get_point(indev, &p1);
	sprintf(buf, "%u : %u" , (uint16_t)p1.x, (uint16_t)p1.y );
	lv_label_set_text(label, buf);

and it is really returns the same value.
but at the same time i have enabled LV_INDEV_POINT_MARKER and it plots points right.
am i using this method in the wrong way, or its data updates only once?

ok. i know the reason. lv_indev_get_act() returns null pointer. what is the right way to get indev then?

i've got it working in SysTick using lv_indev_next(NULL) function call.
and now i think that somehow before button callback gets executed, the actual coords are already overwritten by another indev read in the backgound and it just returns coords for unpressed screen.
but why?

ok. i fixed this by editing line 260 of lv_indev.c.
it was
if(i->driver.type == LV_INDEV_TYPE_POINTER)
and i did
if(i->driver.type == LV_INDEV_TYPE_POINTER && i->proc.state == LV_INDEV_STATE_PR)
i'm not sure if it is a poper fix, but i think for touchpad you need to check whether it is pressed, before updating coords. but i'm not very good in LVGL architecture and i'm also a newbie in C too, so this fix can be a wrong way.

ah. not fixed. button now just crashes the MCU. and here i don't know how to debug further.

fixed it in lv_indev.c
            if(i->driver.type == LV_INDEV_TYPE_POINTER)
            {// for touchpanel it is better update coords only while it is pressed
                if (i->proc.state == LV_INDEV_STATE_PR) {
					i->proc.act_point.x = data.point.x;
					i->proc.act_point.y = data.point.y;
                }

                /*Process the current point*/
                indev_proc_point(&i->proc);
            }

first i've just added a second condition with &&, but it prevented  indev_proc_point(&i->proc); to be executed, thus preventing button from calling the callback.
i don't know whether it is a right fix, because i'm newbie in C and a don't know LVGL architecture well.

Hi,
Thank you for the detailed tracking.
Here is how indev processing works:

Get the first input device and set  it as active
Read its point (or key)
Process the new point
Go the next input device (point 1) until have others
Set the active indev to NULL if there is no more indev.

In other words you can only see the active indev in action functions (e.g. button click) called during "Process the new point" phase to determine which indev triggered the action. Outside of action functions, (in general somewhere in the code) there is no active (being processed) input device.
Your technique with lv_indev_get_next(NULL) is a good idea, it gives you the lastly added indev. However I suggest rather save the return value of lv_indev_drv_register(). It's an lv_indev_t * variable.
It's dangerous to set a label's text in the sys tick handler because the systick can interrupt lv_task_handler in a wrong time. LittlevGL is designed to do all the things in init functions before lv_task_handler or action callbacks, lv_tasks etc. It is better to read a value (In your case the touchpad coordinates) in the systick interrupt and create and lv_task (which is a periodically function) to refresh a label's text there.  Or read the last coordinates in the lv_task too.
If I'm correct then your issue was that you read the touchpad when it was released hence you read dummy values. To solve this you should set  the last x and y coordinates in you indev_read function regardless to the state (pressed/released) of touchpad.
Your fix seems good for touchpad, but it's not good if the POINTER is a mouse. The mouse uses the released coordinates too to position the cursor.

thank you for your great explanation!


Update 2021-04-10

There's a couple of package version updates needed, but most importantly this PR updates the information text with details about our re-opening

Let me know what you think @James-BuildBrighton , I think we should publish this soon (and make further updates to the content in general after some more discussion)


Repo description

Since I can't make a PR for that 😱
"📦🔭🌠 Create your own universe of packages a la tidyverse." should be "📦🔭🌠 Create your own universe of packages à la tidyverse."
😉 🇫🇷

Done 🎊🕺🏻🎊


setPingConnectionInterval for single server configuration is not propagated to underlying MasterSlaveServersConfig

### Expected behavior
Setting the pingConnectionInterval on a single server config should enable periodic ping on connections.
Actual behavior
No pings take place.  Similar to #1415, this appears to be due to SingleConnectionManager.create not copying the configured value to the MasterSlaveServersConfig.
Steps to reproduce or test case
Create single server redisson client with config:
        config.useSingleServer()
                .setAddress(nodeAddress)
                .setPingConnectionInterval(5000);

Redis version
Any
Redisson version
3.7.0
Redisson configuration

Fixed


Chart title links pointing to Twitter search

I'm using the PeakDetectorAgent to monitor a stream of Facebook post activities. I'm still testing it, but the agent seems to be doing its job. However, I've noticed a small bug in the agent summary page. The link in the charts title always point to a twitter search. It would be great to let the user customize that url.
Best, Fabio

You are right, that would be a nice addition, I assume you would like the base URL to be configurable in the agent with the term being dynamically replaced like it is done at the moment for the twitter search term?

Yes :)
Il dom 19 giu 2016, 12:08 AM Dominik Sander<EMAIL_ADDRESS>ha
scritto:

You are right, that would be a nice addition, I assume you would like the
base URL to be configurable in the agent with the term being dynamically
replaced like it is done at the moment for the twitter search term?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cantino/huginn/issues/1552#issuecomment-226967743,
or mute the thread
https://github.com/notifications/unsubscribe/AI-7MxsEyAudQkIr2GNRhZz-cpvwaV7oks5qNGxagaJpZM4I4FGO
.


Cross platform frameworks could be clarified

Currently, the "Preparing test for Upload" page group contains pages for cross platform and platform specific all thrown together in the same group. It would be easier for readers to reason with their decision making on which framework to use if they were grouped by "cross platform" and "platform specific".

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

ID: ed7c0d11-e60a-f088-e983-7465aca41515
Version Independent ID: b2c6a308-1fcb-a385-239d-862b2c14114c
Content: Preparing Appium Tests for Upload - Visual Studio App Center
Content Source: docs/test-cloud/preparing-for-upload/appium.md
Service: vs-appcenter
GitHub Login: @glennwester
Microsoft Alias: glwest

@luisnaranjo733 thanks for the suggestion. We're in the planning process to do a refresh on the docs, and doc structure is one of the prime motivators. I doubt we'll split things into cross-platform and platform, but we will try to organize better.


Text is not animated

I am trying to plot some iterative algorithm. I have multiple subplots and want to annotate the entire figure with a text Iteration <i> where i is an integer.
The text of all frames is displayed over each other instead of just one text per plot.

The issue only appears if fig.text() is called, not with ax.text()


[NodeJS] How to validate input on Prompts.text

Hi,
First, I must congratulate you guys for the excellent work :)
I'm trying to validate an input from a Prompts.text, if it's invalid I'll keep asking for a valid text.
How can I acomplish that ?

So thanks :) and first let me ask... I've been contemplating adding a Prompts.pattern() that would take a RegEx and only return when the pattern is matched. Would that cover your needs in this case?
Now how to do custom input validation. I was planning to add an example of this (which I will) but here's the basics of how I think you could do it. I would create a new dialog like below:
bot.add('/myCustomPrompt', function (session, results) {
    var valid = false;
    if (results && results.response) {
        // validate response
        valid = (results.response == '42');
    }
    if (!valid) {
        builder.Prompts.text(session, 'what's the meaning of life?');
    } else {
        session.endDialog(results);
    }
});

To call your custom prompt just say session.beginDialog('/myCustomPrompt'); in a waterfall and your validated results will be passed to the next step of the waterfall.
You can further customize this by passing in additional arguments in the beginDialog() call but you'll have to keep in mind that your initial args and the results from the call to Prompts.text() are going to come back in via the same param of your custom prompts dialog handler. That's why I added the second clause of the if test to look for results.response.  There's also a results.resumed you can look for which is probably a more accurate test.  If you look at the src for BotBuilder look at how DialogAction.waterfall() is implemented. Its a good example of a single closure that knows if it's being called initially or with the result of a prompt.
Hope that helps.

Sorry... I can never help myself with these kind of questions :) Here's a slight variant that uses session.dialogData to know how many times its prompted the user and will eventually give up.
bot.add('/myCustomPrompt', function (session, results) {
    var valid = false;
    if (results && results.response) {
        // validate response
        valid = (results.response == '42');
    }
    if (valid || (results && results.resumed == builder.ResumeReason.canceled)) {
        session.endDialog(results);
    } else {
        if (!session.dialogData.promptCount) {
            session.dialogData.promptCount = 1;
            builder.Prompts.text(session, 'What's the meaning of life?');
        } else if (session.dialogData.promptCount < 5) {
            session.dialogData.promptCount++;
            builder.Prompts.text(session, 'Sorry that's not it. Guess again. What's the meaning of life?);
        } else {
            session.endDialog({ resumed: builder.ResumeReason.notCompleted });
        }
    }
});

This version also accounts for the user saying "nevermind" to cancel the prompt.

I just checked in a new example for how to create a validated prompt. We were talking about it and I think we're going to add new DialogAction.validatedPrompt() that streamlines most of the boilerplate code from the example and lets you define a custom prompt by simply providing the validation function as a closure.

Hi,
Thanks for the answer. The second option you gave worked well.
But the possibility to provide a validation function will be great.
Thanks again


Update Forms dependency to Forms 5 stable

Description of Change
Hooray! It's here! This means we can launch too... 🚀
Please check if I got all the NuGet references and a quick check if everything still works as intended :D

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

What about essentials dependency? the latest stable version is 1.6.0.

What about essentials dependency? the latest stable version is 1.6.0.

What about Essentials dependency? the latest stable version is 1.6.0.

XamarinCommunityToolkit has no dependency on Essentials :) You saw at Sample projects which are no matter for this PR 👍
But anyways, nice catch)

What about Essentials dependency? the latest stable version is 1.6.0.

XamarinCommunityToolkit has no dependency on Essentials :) You saw at Sample projects which are no matter for this PR 👍
But anyways, nice catch)


Adding java file

please merge my request

Good job @sakshidond! Congratulations on making your first open-source contribution. Keep learning!


Write a test for fasta_manager that tests whether the first and last sequences have been ingested correctly

Write the test
Run the test
Modify the code until the test works

@akim135 are you interested in writing this test?


Loading thumbnail image in gallery (configurable product)

Current behavior
As a placeholder for images, we use the main product image. It's problematic especially when we show all gallery images as thumbnails
https://github.com/DivanteLtd/vue-storefront/blob/dfbe73f2ffeab38bce4ac3e219db6a09a257b261/core/modules/catalog/helpers/index.ts#L536
Expected behavior
We should see proper image or placeholder with logo
Can you handle fixing this bug by yourself?
No - decision needed

i believe @pkarw was working on this

I don't exactly get the problem we must discuss this @mdanilowicz by the next chance we're on Hangout

It has been fixed with #2481


Failed to become healthy before their configured timeout or in 120 seconds.

Failed to restart billingapp-backend: container(s) failed to become healthy before their configured timeout or in 120 seconds. This might be a problem with the healthcheck and not a functional problem. (ddev-billingapp-backend-sqlsrv container exited, more info with [
        ddev logs -s sqlsrv
        docker logs ddev-billingapp-backend-sqlsrv
        docker inspect --format "{{ json .State.Health }}" ddev-billingapp-backend-sqlsrv | docker run -i --rm ddev/ddev-utilities jq -r
])

Hi @ldavidsp - If you do what it says there it will help figure out what the problem is. Do what it says, then attach the results as files:
ddev logs -s sqlsrv

docker inspect --format "{{ json .State.Health }}" ddev-billingapp-backend-sqlsrv | docker run -i --rm ddev/ddev-utilities jq -r

You could explain what the problem was for the next person... :)


vscode  rust-analyzer + remote ssh extension cause internal error: entered unreachable code

when I use vscode extensions and install rust-analyzer and remote-ssh , it shows the error. I guess the extension could not use with remote-ssh extension. thanks
rust-analyzer version: (eg. output of "Rust Analyzer: Show RA Version" command)
all  0.2.x and 0.3.x preleease versions have these debug
rustc version: (eg. output of rustc -V)
rustc 1.58.0-nightly (baba6687d 2021-11-03)

there is something else afoot here - i exclusively use r-a in vscode remote ssh and it is fine.

Can you post the whole error? The one saying "unreachable code".

it shows the macro-error , if the code with  #[async_trait]  , it shows this error

Can you post the whole error? The one saying "unreachable code".

where can i find the error ? I am not sure ,thanks.

If only async_trait has issues and the rest of the features work, it's related to proc macros. Maybe try upgrading to a different nightly: https://github.com/rust-analyzer/rust-analyzer/issues/10837#issuecomment-1007076392.

If only async_trait has issues and the rest of the features work, it's related to proc macros. Maybe try upgrading to a different nightly: #10837 (comment).

Good jobs, it works. thanks


Move from gitlab/flywheel-io to nacdata

The working gear is now here!

I have made the final update (removing DICOM header data from metadata).  So this gear is ready to go


Cleanup emotion/babel deps, package.json

What I did

Move to emotion 10 completel
Cleanup bithound
Cleanup lerna publishConfig

Fixes #4192

@pksunkara why doesn't the global publishConfig work for us?

It's deprecated in lerna@3 because it was not the correct way to do it. Each scoped package.json is supposed to contain publishConfig. https://docs.npmjs.com/files/package.json#publishconfig

Looks like jest-emotion still wants emotion package

Yup, working on it.

Removed emotion@9 only from dependencies for now so that I can unblock the next alpha release. The jest-emotion and emotion@9 are still being used in devDependencies of our tests. Will make a separate PR for it.