text:
feat: add --fast mode

This PR changes the progress calculation to optionally skip frames that are between ones that have similar progress toward the target. When enabled, it speeds up processing by about 40% and has minimal impact when not enabled via the shortcut. This also aligns perceptual progress and visual progress by sharing the same computation code and fixes #48 by virtue of the fact that we can't wait for the global min when we only parse some of the JPEGs.
Timings
( for i in $(seq 10); do; speedline cnn.json --pretty; done; )  55.15s user 4.63s system 98% cpu 1:00.44 total

( for i in $(seq 10); do; speedline cnn.json --pretty --fast; done; )  34.28s user 3.32s system 99% cpu 37.737 total

1 - (34/55) = ~38%
Open Questions

Expose the direct threshold value as the option or keep as binary? If so, name? similarProgressThreshold?

Skipping frames can artificially make the visual jitter (jankiness) go away from the calculations. Please make sure that layout instability effects that perceptual speed index captures aren't sacrificed due to this frame-skipping (technically, this is temporal downsampling).

@pahammad by its nature, the impact on the indexes cannot be avoided as you point out. However, in my experience there are a number of issues with using these metrics to as a signal of layout stability in the first place and frequently I've seen jitter artificially be rewarded by speed index rather than punished. Would a warning and tunable threshold address your concerns or are you against including the option at all with these drawbacks?

I am OK with including the frame-skipping as an option if it comes with appropriate warning when the option is exercised. I am very curious about your sentence: "frequently I've seen jitter artificially be rewarded by speed index rather than punished". Are you referring to the classical (histogram-based) speed index, or are you referring to SSIM based perceptual speed index (PSI) ? If you are referring to PSI in that sentence (where it rewards jitter instead of punishing it), I would love to see an example or two - this is opposite to what I've seen.

@pahammad disclaimer text added and I've filed #50 to discuss layout stability tracking 👍

changes lgtm but 2 test failures we need to work out.

changes lgtm but 2 test failures we need to work out.

done

speedline@1.2.0 published with this.

authors: pahammad patrickhulce paulirish

repo: pmdartus/speedline


text:
🛑 RPB Law is down

In 6ccee15, RPB Law (https://lawbob.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: RPB Law is back up in 49b2411.

source: gharchive/issue

repo: pmjustin/pmuptime

license: MIT

license_type: permissive

license_source: github-api


text:
🛑 JMS Law is down

In 9d7dad3, JMS Law (https://jmslawltd.com) was down:

HTTP code: 503
Response time: 10568 ms

Resolved: JMS Law is back up in 30a5ea2 after 5 minutes.


text:
🛑 JMS Law is down

In 0c90ec9, JMS Law (https://jmslawltd.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: JMS Law is back up in 8aea11d after 1 hour, 23 minutes.


text:
⚠ GitHub has degraded performance

In cdc2258, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 186 ms

Resolved: GitHub performance has improved in 0807d24 after 8 minutes.

repo: pmmmwh/upptime


text:
⚠ Slack has degraded performance

In 4881fc4, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 85 ms

Resolved: Slack performance has improved in 50353d9.


text:
webview setState in each component is not responding

My intention is to customize items in character in webview and sent the items name from react native.
The data is send correctly but when I need to set the state inside the web add event listener.
It have no problem when set state in color, but it crash when set state color and geometry
`function Boy(props) {
const group = useRef()
const { nodes, materials } = useGLTF('/man6.gltf')
const state = proxy ({
  current: null,
  color: {
    skin: "#fff6dd",
    hair: 'Material.010', //RN2_Green.002
    shirt: 'NolmalShirt02.001',
    watch: "Material.007",
    shoe: 'Shoes_Normal.001',  //'Material.009'
  }
})

const stateGeo = proxy ({
  current: null,
  geo: {
    shoe: nodes.Shoes2.geometry,
    shirt: nodes.man_Shirt.geometry,
  }
})

const snap = useSnapshot(state)

const collectionShoesGeometry = {
  "shoe3": nodes.Shoes2.geometry,
  "shoe4": nodes.man_Shoes.geometry,
}

const collectionShirtsGeometry = {
  "shirt3": nodes.Shirt2.geometry,
  "shirt4": nodes.man_Shirt.geometry,
}

window.addEventListener("message", function(event) {
  var requestTrim = event.data + ""
  var value = requestTrim.split("|")
  if (value.length > 0) {
    if (value[0] === "closet") {
      stateGeo.geo['shirt'] = collectionShirtsGeometry['shirt3']
      state.color['shirt'] = "NolmalShirt02.001"
    } else if(value[0] === "watch") {
      state.color["watch"] = "Material.008"
    }
  }
});

useEffect(() => {

}, [])

return (
  <group ref={group} {...props} dispose={null} scale={4.7}>
    <mesh
      geometry={stateGeo.geo['shirt']}
      material-color={snap.color['shirt']}
      material={materials[snap.color['shirt']]}
      position={[0, -0.01, 0.02]}
      rotation={[-1.6, 0, 0]}
    />
    <mesh
      geometry={nodes.watch1.geometry}
      material={materials['Material.001']}
      position={[0, -0.01, 0.02]}
      rotation={[-1.6, 0, 0]}
      scale={1.01}
    />
  </group>
)

}
useGLTF.preload('/man6.gltf')`
I try to put the state outside the function, it's work.
Note: useSnapshot can't use twice in the same function and I'm not sure why.
Did anyone have any suggest about this? Thank you

this doesnt seem correct, color = "NolmalShirt02.001" is not valid

authors: drcmda teenkwn

repo: pmndrs/gltfjsx


text:
Depth buffer issue when using DepthOfFieldEffect

Description of the bug
Follow-up to #420.
By using dofEffect.circleOfConfusionMaterial.adoptCameraSettings and worldFocusDistance, I was able to get a dynamic target to change the focus. I got it to match up / work in @vanruesc's sandbox.
However, I still don't get it to work elsewhere...
For me it looks like the depth buffer is in a wrong format, and that the calculation to go from linearized near/far values to world distances and vice versa doesn't match some setting here. I checked that we're not using logarithmic depth.
Maybe these images of expected and not expected cases help:
In the below images,

the textured plane is cut off by near and far clip planes of 1 and 10
the white cube is the focus target - placed at 1, 10, ~5.5 and ~2.5

✔ Target at near clip plane - as expected: near clip plane is in focus

✔ Target at far clip plane - as expected: far clip plane is in focus

❌ Target at center between near and far - not expected: focus is too close

❌ Target at 1/4 between near and far - not expected: focus is too close

So there seems to be some nonlinearity going on, but I have no idea why.
To Reproduce
I'm unfortunately unsure how to reproduce / what's wrong in this setup so far. Happy to answer any questions to hopefully figure out what I'm doing wrong.
Expected behavior
Ability to set the worldFocusDistance and get that distance in focus.
Screenshots
see above
Library versions used

Three: 0.145.4
Post Processing: 6.29.1

Desktop

OS: Windows 10 and 11
Browser Chrome
Graphics hardware: RTX 2070 Max-Q and RTX 3070

Thanks for the screenshots :+1:
How are you setting the worldFocusDistance? You'd want to set this to the distance from the camera to the cube. Alternatively, try setting dofEffect.target to cube.position;

(my mistake convoluting R3F and three here, of course it's vanilla...)
Yes, this distance is the world distance from the camera to the cube. Also logged these values and made sure they match up with what I'd expect (e.g. in screenshots 2 and 3 the worldFocusDistance values logged are 5.5 and 2.5).
I get the same behaviour when using dofEffect.target = cube.position.
(side note, for that to work the cube must be a at sceen root / must not be transformed by it's parents if I'm not mistaken which I made sure of).

Strange.. I'll take a closer look this weekend.

side note, for that to work the cube must be a at sceen root / must not be transformed by it's parents if I'm not mistaken which I made sure of

Thanks for pointing that out. The implementation currently doesn't use getWorldPosition which is a bug.

For low near clip values (e.g. 0.01) I only get any focus when the target is at far clip distance, and basically all other target distances result in focus planes very close to the near clip plane.

Well, the shader does linearize depth, so this sounds like the camera settings aren't being set correctly for some reason.

Thanks!
Right now I'm literally calling this
        setInterval(() => {

            dof.circleOfConfusionMaterial.adoptCameraSettings(camera);
            dof.target = targetObject.position;

        }, 50);
        ```
to ensure for now not messing up any timing here and that the right camera values are used. (Of course tried without that as well),

What's weird is that it does seem to work for "target distance == near clip" and "target distance == far clip" but nothing inbetween.

Hi again! Were you by any chance able to look into the issue @vanruesc? Thanks!

Sorry, I've been too busy to get back to this. However, I was able to confirm that viewZToOrthographicDepth returns correct values.

I got it to match up / work in @vanruesc's sandbox.
However, I still don't get it to work elsewhere...

Any chance you could provide a failing example with code for me to look at?

@hybridherbst does this sandbox or video demonstrate the same issue you are talking about ?
https://user-images.githubusercontent.com/6885294/204822593-cee0b9d4-25a2-4f03-b472-fb0fc02db443.mp4

@optimus007 Thanks for the sandbox! The unexpected behaviour that can be observed when the target is off-center has to do with perspective projection. The object would probably remain in focus at all times with an orthographic camera. The effect translates the world distance to a linear, orthographic depth value which basically counters the perspective projection. I wonder if we could use perspective depth instead :thinking:
Right now, I'm busy with work and the v7 redesign. I'll eventually get to the DepthOfFieldEffect.
@hybridherbst Does your project use logarithmic depth? That would explain why you're only able to focus objects close to the near and far planes.

The expected behaviour described in this ticket would be achieved by calculating the distance from individual fragments to the camera instead of using the scene depth. This would result in a spherical focus field around the camera instead of a box-like field between the near and far plane.
I don't plan on changing the current CoC implementation in postprocessing v6, but the new implementation in v7 will use the distance-based approach.
Closing this in favor of #569.

authors: hybridherbst optimus007 vanruesc

repo: pmndrs/postprocessing

license: Zlib


text:
New command: Apply a retentionlabel to a file using spo file retentionlabel ensure

Usage
m365 spo file retentionlabel ensure [options]
Description
Apply a retention label to a file
Options

Option
Description

-u, --webUrl <webUrl>
The url of the web

--fileUrl [fileUrl]
The server-relative URL of the file that should be labelled. Specify either fileUrl or fileId but not both.

i, --fileId [fileId]
The UniqueId (GUID) of the file that should be labelled. Specify either fileUrl or fileId but not both.

--name <name>
Name of the retention label to apply to the file.

Examples
Apply the label some label to a file
m365 spo file retentionlabel set --webUrl 'https://contoso.sharepoint.com/sites/sales' --fileUrl '/sites/sales/somelibrary/somefile.pdf' --name 'Some retention label'

More information

This command can be implemented with shared code created for #4158.

Putting this on hold till #4158 is implemented.

looks solid 👍

Hey @martinlingstuyl, why are we waiting with implementation until #4158 is done?

Hey @martinlingstuyl, why are we waiting with implementation until #4158 is done?

Hi @waldekmastykarz, because we might be able to use shared code. In principle files and folders are listItems as well, so we could possibly reuse the listItem command. That was my initial thought.

If you have a better idea though.., do let me know :)

Hey @martinlingstuyl, why are we waiting with implementation until #4158 is done?

Hi @waldekmastykarz, because we might be able to use shared code. In principle files and folders are listItems as well, so we could possibly reuse the listItem command. That was my initial thought.

Make sense. Next time, let's include this reasoning upfront just so that we make it clear to everyone why we're waiting 👍

This command can be implemented with shared code created for #4158.

I'd written something, but I agree it could have been more clear @waldekmastykarz

Can I work on this one please

Awesome, all yours!

authors: Adam-it Jwaegebaert martinlingstuyl nicodecleyre waldekmastykarz


text:
Adds "Create custom views to differentiate SharePoint news page types in Site Pages library" sample script. Closes #1782

Adds "Create custom views to differentiate SharePoint news page types in Site Pages library" sample script. Closes #1782

Thank you @nanddeepn! We'll try to review it ASAP!

Thanks @nicodecleyre
Added a reference to the script from the docs/mkdocs.yml file.

source: gharchive/pull-request

authors: milanholemans nanddeepn


text:
SharePoint Online Search Issue using PnP

Notice
Many bugs reported are actually related to the PnP Framework which is used behind the scenes. Consider carefully where to report an issue:

Are you using Invoke-PnPSiteTemplate or Get-PnPSiteTemplate? The issue is most likely related to the Provisioning Engine. The Provisioning engine is not located in the PowerShell repo. Please report the issue here: https://github.com/pnp/pnpframework/issues.
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? Then please continue reporting the issue in this repo.
If you think that the functionality might be related to the underlying libraries that the cmdlet is calling (We realize that might be difficult to determine), please first double check the code of the cmdlet, which can be found here: https://github.com/pnp/powershell/tree/master/src/Commands. If related to the cmdlet, continue reporting the issue here, otherwise report the issue at https://github.com/pnp/pnpframework/issues

Reporting an Issue or Missing Feature
Please confirm what it is that your reporting
Expected behavior
Please describe what output you expect to see from the PnP PowerShell Cmdlets
Actual behavior
Please describe what you see instead. Please provide samples of output or screenshots.
Steps to reproduce behavior
Please include complete script or code samples in-line or linked from gists
What is the version of the Cmdlet module you are running?
(you can retrieve this by executing Get-Module -Name "PnP.PowerShell" -ListAvailable)
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Closing this as its not clear what the issue is with PnP.
Request you to please answer the issue template questions and reopen the issue.

Hi Elhadj,
My issue has been close due to the following reason.
Regards
Jestine GOH
Senior IT Consultant, Information Technology (Applications)
Tripartite Alliance Limited
80 Jurong East St 21, #05-05/06, Devan Nair Institute for Employment and Employability, Singapore 609607
T: 6956 6407 | @.@.> | TAL Personal Data Policyhttps://apc01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.tal.sg%2Fprivacy-statement%2F&data=02|01|jestine_goh%40tal.sg|401a4a5984c443cc424108d8095629a9|0af12b508f1940928ace4dce8f8253e0|0|0|637269612992643648&sdata=yalG7HcSQL508xcqisIuQPsuOLUbCPKhNwAXPmV25%2FE%3D&reserved=0
@.***D771B7.0167E960]
This message may contain confidential information intended only for the individual named. If you are not the named addressee, do not disseminate, distribute or copy this email. Please notify the sender immediately and delete this email from your system. Any views expressed in the email are those of the individual sender, except where the sender specifically states them to be the view of the Tripartite Alliance Limited (TAL). The information and/or comments provided or expressed in this message are not to be construed as legal advice and are not intended to replace legal advice. You should not rely on any information and/or comments provided or expressed in this message/research memorandum as legal advice. TAL shall not be responsible for any loss or damage arising from your reliance on any information and/or comments provided or expressed in this message.
From: Gautam Sheth @.>
Sent: Monday, July 5, 2021 3:55 PM
To: pnp/powershell @.>
Cc: Jestine Goh @.>; Author @.>
Subject: Re: [pnp/powershell] SharePoint Online Search Issue using PnP (#890)
Closing this as its not clear what the issue is with PnP.
Request you to please answer the issue template questions and reopen the issue.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fpowershell%2Fissues%2F890%23issuecomment-873889554&data=04|01|jestine_goh%40tal.sg|6ce8f0fcd54d4430b58e08d93f8a3267|0af12b508f1940928ace4dce8f8253e0|0|0|637610685698210475|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=AZTkGO0NUO5E5WwsJa2kNDkf2ZgM8BJxgpTMxXEbN3w%3D&reserved=0, or unsubscribehttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUXL4ATYYC4BXS2CNJYVCU3TWFQNJANCNFSM472EKEAA&data=04|01|jestine_goh%40tal.sg|6ce8f0fcd54d4430b58e08d93f8a3267|0af12b508f1940928ace4dce8f8253e0|0|0|637610685698210475|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=NaEwgpsRKk0ivkvlLVm5ZrF2d%2F8FYXBNn882Vd16gPY%3D&reserved=0.

Hello Jestine,
Good day!
Thank you for your email reply, kindly could you provide more description regarding the issue that you are facing?
Best regards,
Elhadj
From: Jestine Goh @.>
Sent: Monday, July 5, 2021 4:01 PM
To: Elhadj Mamoudou Diallo (Shanghai Wicresoft Co,.Ltd.) @.>
Cc: Author @.>; pnp/powershell @.>; pnp/powershell @.***>
Subject: [EXTERNAL] RE: [pnp/powershell] SharePoint Online Search Issue using PnP (#890)
Hi Elhadj,
My issue has been close due to the following reason.
Regards
Jestine GOH
Senior IT Consultant, Information Technology (Applications)
Tripartite Alliance Limited
80 Jurong East St 21, #05-05/06, Devan Nair Institute for Employment and Employability, Singapore 609607
T: 6956 6407 | @.@.> | TAL Personal Data Policyhttps://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.tal.sg%2Fprivacy-statement%2F&data=04|01|v-ediallo%40microsoft.com|d642b24525844526f9b308d93f8b18e0|72f988bf86f141af91ab2d7cd011db47|1|0|637610689586315681|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1hFBkeMJm%2F8Velx7k7eRoL%2Bg3a25B5oSfRgHIY3aDpw%3D&reserved=0
@.***D771BA.389E4070]
This message may contain confidential information intended only for the individual named. If you are not the named addressee, do not disseminate, distribute or copy this email. Please notify the sender immediately and delete this email from your system. Any views expressed in the email are those of the individual sender, except where the sender specifically states them to be the view of the Tripartite Alliance Limited (TAL). The information and/or comments provided or expressed in this message are not to be construed as legal advice and are not intended to replace legal advice. You should not rely on any information and/or comments provided or expressed in this message/research memorandum as legal advice. TAL shall not be responsible for any loss or damage arising from your reliance on any information and/or comments provided or expressed in this message.
From: Gautam Sheth @.@.>>
Sent: Monday, July 5, 2021 3:55 PM
To: pnp/powershell @.@.>>
Cc: Jestine Goh @.@.>>; Author @.@.>>
Subject: Re: [pnp/powershell] SharePoint Online Search Issue using PnP (#890)
Closing this as its not clear what the issue is with PnP.
Request you to please answer the issue template questions and reopen the issue.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fpowershell%2Fissues%2F890%23issuecomment-873889554&data=04|01|v-ediallo%40microsoft.com|d642b24525844526f9b308d93f8b18e0|72f988bf86f141af91ab2d7cd011db47|1|0|637610689586315681|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Jk%2FCld9kxZT1BBw228ChAZWQrt2Ma%2F22O3vJUQbxPmc%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUXL4ATYYC4BXS2CNJYVCU3TWFQNJANCNFSM472EKEAA&data=04|01|v-ediallo%40microsoft.com|d642b24525844526f9b308d93f8b18e0|72f988bf86f141af91ab2d7cd011db47|1|0|637610689586325643|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aYTgAqI0nD%2Bp35ukDAzP17uZyh5pZJpQjFU5qtLpeZ0%3D&reserved=0.

Hi all,
We are using SharePoint web part Search function to search documents. Recently we noticed the search result is not working well. Some time ago, it was working.
If I tried to view source of the page and found the below codes for the Search input. Understand from Microsoft, PnP search is being used and there is a need to log issue with Github.
Please see below codes for your reference. For your advise please.

It looks like your browser does not have JavaScript enabled. Please turn on JavaScript and try again.

Thank you
Regards
Jestine GOH
Senior IT Consultant, Information Technology (Applications)
Tripartite Alliance Limited
80 Jurong East St 21, #05-05/06, Devan Nair Institute for Employment and Employability, Singapore 609607
T: 6956 6407 | @.@.> | TAL Personal Data Policyhttps://apc01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.tal.sg%2Fprivacy-statement%2F&data=02|01|jestine_goh%40tal.sg|401a4a5984c443cc424108d8095629a9|0af12b508f1940928ace4dce8f8253e0|0|0|637269612992643648&sdata=yalG7HcSQL508xcqisIuQPsuOLUbCPKhNwAXPmV25%2FE%3D&reserved=0
@.***D771BA.BFCEB610]
This message may contain confidential information intended only for the individual named. If you are not the named addressee, do not disseminate, distribute or copy this email. Please notify the sender immediately and delete this email from your system. Any views expressed in the email are those of the individual sender, except where the sender specifically states them to be the view of the Tripartite Alliance Limited (TAL). The information and/or comments provided or expressed in this message are not to be construed as legal advice and are not intended to replace legal advice. You should not rely on any information and/or comments provided or expressed in this message/research memorandum as legal advice. TAL shall not be responsible for any loss or damage arising from your reliance on any information and/or comments provided or expressed in this message.
From: Elhadj Mamoudou Diallo (Shanghai Wicresoft Co,.Ltd.) @.>
Sent: Monday, July 5, 2021 4:24 PM
To: Jestine Goh @.>
Cc: Author @.>; pnp/powershell @.>; pnp/powershell @.***>
Subject: RE: [pnp/powershell] SharePoint Online Search Issue using PnP (#890)
Hello Jestine,
Good day!
Thank you for your email reply, kindly could you provide more description regarding the issue that you are facing?
Best regards,
Elhadj
From: Jestine Goh @.@.>>
Sent: Monday, July 5, 2021 4:01 PM
To: Elhadj Mamoudou Diallo (Shanghai Wicresoft Co,.Ltd.) @.@.>>
Cc: Author @.@.>>; pnp/powershell @.@.>>; pnp/powershell @.@.>>
Subject: [EXTERNAL] RE: [pnp/powershell] SharePoint Online Search Issue using PnP (#890)
Hi Elhadj,
My issue has been close due to the following reason.
Regards
Jestine GOH
Senior IT Consultant, Information Technology (Applications)
Tripartite Alliance Limited
80 Jurong East St 21, #05-05/06, Devan Nair Institute for Employment and Employability, Singapore 609607
T: 6956 6407 | @.@.> | TAL Personal Data Policyhttps://apc01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.tal.sg%2Fprivacy-statement%2F&data=04|01|jestine_goh%40tal.sg|d1c8b1283bad4c7cc8eb08d93f8e4f6c|0af12b508f1940928ace4dce8f8253e0|0|0|637610702733310554|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=gFCfq2SS92Z36kbcbB%2Br9l2tfgVAorFcZxMaJitrI7g%3D&reserved=0
@.***D771BA.BFCEB610]
This message may contain confidential information intended only for the individual named. If you are not the named addressee, do not disseminate, distribute or copy this email. Please notify the sender immediately and delete this email from your system. Any views expressed in the email are those of the individual sender, except where the sender specifically states them to be the view of the Tripartite Alliance Limited (TAL). The information and/or comments provided or expressed in this message are not to be construed as legal advice and are not intended to replace legal advice. You should not rely on any information and/or comments provided or expressed in this message/research memorandum as legal advice. TAL shall not be responsible for any loss or damage arising from your reliance on any information and/or comments provided or expressed in this message.
From: Gautam Sheth @.@.>>
Sent: Monday, July 5, 2021 3:55 PM
To: pnp/powershell @.@.>>
Cc: Jestine Goh @.@.>>; Author @.@.>>
Subject: Re: [pnp/powershell] SharePoint Online Search Issue using PnP (#890)
Closing this as its not clear what the issue is with PnP.
Request you to please answer the issue template questions and reopen the issue.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fpowershell%2Fissues%2F890%23issuecomment-873889554&data=04|01|jestine_goh%40tal.sg|d1c8b1283bad4c7cc8eb08d93f8e4f6c|0af12b508f1940928ace4dce8f8253e0|0|0|637610702733310554|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=tC2yxoeDjw04BNdLxkiurn9x1NACazBUoeWtLhrW4o4%3D&reserved=0, or unsubscribehttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUXL4ATYYC4BXS2CNJYVCU3TWFQNJANCNFSM472EKEAA&data=04|01|jestine_goh%40tal.sg|d1c8b1283bad4c7cc8eb08d93f8e4f6c|0af12b508f1940928ace4dce8f8253e0|0|0|637610702733320546|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=qWQfePHEIc1miZstF%2FTfvX5ZBUsVFp6fi45ZygpfGPs%3D&reserved=0.

authors: gautamdsheth jestinegoh

repo: pnp/powershell


text:
Update Add-PnPPageWebPart.md

Correcting Title typo from "Add-PnPWebPart" to "Add-PnPPageWebPart"
Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[x] Typo Fix

Thanks @JimmyHang, well noticed!

authors: JimmyHang KoenZomers


text:
Adding Move-PnPTerm and Move-PnPTermSet commands

Type

[x]  New Feature

Related Issues?
#2978
What is in this Pull Request ?
Adding two new commands

Move-PnPTerm
Move-PnPTermSet

@kunj-sangani - code looks great !!
Some minor changes before I merge it 😊
In both these cmdlets, can you

Replace -DestinationTermSet with -TargetTermSet
Replace -SourceTermSet with -TermSet
Replace -SourceTermGroup with -TermGroup
Replace -DestinationTermGroup with -TargetTermGroup
Replace -DestinationTerm with -TargetTerm

Don't forget to update the docs as well.

Hi @gautamdsheth
Updated the Name of the parameters
Thanks for the help :)

Thanks @kunj-sangani, merged it , much appreciated !

authors: gautamdsheth kunj-sangani


text:
Resize table columns

As per https://github.com/pocketbase/pocketbase/discussions/1542

It uses a fairly generic svelte action, so it doesn't interfere much with the existing code. Took a bit of time to smooth out any bugs, but seems stable now.
Thanks again for your work on the project; the latest release was great 👍.

I'm not sure about this feature.
I haven't tested it locally, nor reviewed the code changes, but just from the screenshot it looks kindof strange since the table is no longer 100% width (maybe because of the fixed layout?).

I'm not sure about this feature.
I haven't tested it locally, nor reviewed the code changes, but just from the screenshot it looks kindof strange since the table is no longer 100% width (maybe because of the fixed layout?).

I did think this after submitting the pull request with the gif.
It makes the implementation slightly less elegant, but I can keep a minimum width for the table. I will revise and see what you think.

I don't think table-layout: fixed is necessary and it could cause some issues with adding for example a new column after the resizing.
Additionally, there should be something that will prevent resizing below some min-width threshold, because from the screenshot the id seems to be cropped and I'm not sure if this is a good idea.

I've updated the code to keep the minimum table width:

This implementation does require a fixed table layout. Trying to keep columns a specific width and the resize-handle tracking the cursor was problematic with a fluid layout.
The columns will clip as you resize. I personally prefer this to min widths.
Let me know what you think, feel free to close if it doesn't make sense for the project.

Sorry, I appreciate the work you've put into this but I don't want to rush it and merge something just because there is a PR for it.
I like the idea of resizable columns but the current implementation feels a little brittle and I'm not confident that we are handling all edge cases (eg. on windows small->big resize I guess we need also to bind to the resize event to recalculate the initial table width? I'm also not sure how it will behave if we do external layout changes, eg. via plain css in a future responsive version or just toggling sibling dom elements, etc.).
I've added it in my local todo to search for other options and eventually something similar could be implemented in the future, but I don't want to invest time into it right now and for now remains out of the scope.

authors: ganigeorgiev mjadobson

repo: pocketbase/pocketbase


text:
Campaign page/ UI-UX elements are missing according to UI mockup (Design)

To Reproduce
Steps to reproduce the behavior:

Go to dev.podkrepi.bg
Navigate to main menu "Дарителство"
Scroll down to 'Кампании.'
Choose random Кампания
5.Select button "Вижте повече"
6.You should be successfully redirected to the new Campaign page

Expected behavior
All UX elements from the Design UI should be according to the Design mock ups.
Actual result:
Campaign page/ UI-UX elements are missing according to UI mockup (Design):

Missing Subheader
Slider for sums (too large according to UI mocк ups from Design)
List of donors and donation sums missing (under the "Сподели" button)
Missing image carousel (under the Description of the Campaign)
Missing section "Последни новини"
Missing section "Коментари"

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows
Browser  chrome
Version - Version 98.0.4758.82 (Official Build) (64-bit)

@ani-kalpachka
@kachar

If any of the @podkrepi-bg/softuni-bootcamp  team wanna try solving this issue it would be great

repo: podkrepi-bg/frontend


text:
[Feature] Use external database for Keycloak

Currently we deploy Keycloak with a Helm chart that also deploys PostgreSQL next to it. Since our own modules will also be using PostgreSQL it's best if we have 1 instance and point all software to it. This will save resources and will make it easier to maintain

I see most of the images and infrastructure are managed in the api repo. Is that infrastructure repo then mandatory?

repo: podkrepi-bg/infrastructure


text:
Crash occuring on Xcode 12 when attempting to do enum reflection on child action.

Describe the bug
Attempting to send an action view the store causes EXC_BAD_INSTRUCTION (code=EXC_I386_INVOP, subcode=0x0) when combining reducers.
Given a child action without any cases
enum ChildAction: Equatable {}
And a parent action
enum ParentAction: Equatable {
	case childAction(ChildAction)
	case parentAction
}

And a main reducer
let parentReducer = Reducer<ParentState, ParentAction, ParentEnvironment>.combine(
	childReducer.pullback(
	 state: \.childState,
		action: /ParentAction.childAction,
		environment: { _ in ChildEnvironment() }
	),
	Reducer<ParentState, ParentAction, ParentEnvironment> { state, action, _ in
		switch action {
		case .parentAction:
			state.childState += 1
			return .none
		case .childAction:
			return .none
		}
	}
)

As soon as you send viewStore.send(.parentAction) a crash occurs.
The crash happens on EnumReflection line 75 from the CasePaths package
This crash does not happen on Xcode 11.4 with the same setup.
A work around is to add a case to the child enum.
enum ChildAction {
	case banana
}

To Reproduce
Send any action viewStore.send(.parentAction) with the above setup is sufficient to cause a crash.
Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots

Environment

Xcode 12 Beta 1
Swift 5.3
OS (if applicable): iOS 14

Additional context
ComposableCasePathCrash.zip

Hey @vibrazy, thanks for the detailed report. We've actually logged this issue on swift-case-paths here: https://github.com/pointfreeco/swift-case-paths/issues/11
We probably won't get around to fixing it for a bit (if you wanna take a pass at it, please do!), but in the meantime, another workaround is to use the .never case path:
 childReducer.pullback(
   state: \.childState,
-  action: /ParentAction.childAction,
+  action: .never,
   environment: { _ in ChildEnvironment() }
 ),

This has been fixed upstream in Case Paths 0.1.2. Be sure to update your package dependencies!

authors: stephencelis vibrazy


text:
Support for Non-Regional Forms and Gender Differences

Was making my Living Dex and noticed the site does not support forms besides regional forms. Unown, Vivillon, Alcremie and gender differences are the big form changes I would be looking for. Perhaps have it as an option to display and/or have the option to display forms/gender differences as seperate boxes.

Hello, I'm already looking for it but it doesn't seem to be about it. please

In contrast to the previous post, I think all "other" forms should be in their own box. This is entirely because what happens when they add another hat for pikachu? You will need to move 1300+ pokemon in HOME to keep the order straight. If they are in a separate box, the worst case is you need to move the whole box one over to make room for another.

This appears to be another duplicate of #256?
(The checkboxes are probably kinda implied, due to being available for Gmax forms)

authors: ALonleyBanana HavocsCall Vrontis fuer-lo

license: mit

license_source: bigquery


text:
[TECHDEBT] [P2P] Raintree scalability improvements

Objective
Tend for the TODOs:
// INVESTIGATE(olshansky/team): Does not scale to 1,000,000,000 nodes

Related to #222 and #246
Origin Document
"make it work ⏩ make it fast ⏩ make it pretty"
#222 improves a lot in terms of time complexity, see what we can do to improve even further.
Some research might be required
Goals

[ ] Ensure that the network can scale
[ ] Verify empirically

Deliverable

[ ] Optimizations
[ ] Tests / Benchmarks

Non-goals / Non-deliverables

_REPLACE_ME: List of things that are out of scope
...

General issue deliverables

[ ] Update the appropriate CHANGELOG
[ ] Update any relevant READMEs (local and/or global)
[ ] Update any relevant global documentation & references
[ ] If applicable, update the source code tree explanation
[ ] If applicable, add or update a state, sequence or flowchart diagram using mermaid

[Optional] Testing Methodology

_REPLACE_ME: Make sure to update the testing methodology appropriately_
Task specific tests: make ...
All tests: make test_all
LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md

Creator: @deblasis
Co-Owners: @Olshansk

Moving this to M4

Closing this out as the scope is too large.

authors: Olshansk deblasis

repo: pokt-network/pocket


text:
Select instrumentation tooling (assumes OpenTelemetry exporter for Azure Monitor)

As a project lead, I want to make a decision on what tooling to use to instrument telemetry data, so that telemetry data can be collected and sent to the centralized monitoring solution

Mentioned in #228

repo: polatengin/indiana


text:
user-home-test branch

As mentioned in neurostars, the last few versions of the fmriprep docker images fail to run when specifying a user (docker run -u myuser ...).
@oesteban created a new docker image (poldracklab/fmriprep:user-home-test).
This new image fixes the permissions problem with $TEMPLATEFLOW_HOME.  However, now I get an error similar to the one in a different Neurostats post when processing the (GRE) fieldmap:
Node: fmriprep_wf.single_subject_Pilot005_wf.func_preproc_ses_day1_task_TASK_acq_normal_run_01_echo_1_wf.sdc_wf.phdiff_wf.meta
Working directory: /tmp/work/fmriprep_wf/single_subject_Pilot005_wf/func_preproc_ses_day1_task_TASK_acq_normal_run_01_echo_1_wf/sdc_wf/phdiff_wf/meta

Node inputs:

bids_dir = None
bids_validate = False
fields = <undefined>
in_file = /data/phelpslab/Linda/BIDSdata/sub-Pilot005/ses-day1/fmap/sub-Pilot005_ses-day1_acq-GRE_run-01_phasediff.nii.gz
undef_fields = False

Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/plugins/multiproc.py", line 69, in run_node
    result['result'] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 473, in run
    result = self._run_interface(execute=True)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 557, in _run_interface
    return self._run_command(execute)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 637, in _run_command
    result = self._interface.run(cwd=outdir)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 371, in run
    outputs = self.aggregate_outputs(runtime)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 472, in aggregate_outputs
    raise error
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 465, in aggregate_outputs
    setattr(outputs, key, val)
  File "/usr/local/miniconda/lib/python3.7/site-packages/traits/trait_handlers.py", line 172, in error
    value )
traits.trait_errors.TraitError: The 'run' trait of a ReadSidecarJSONOutputSpec instance must be a unicode string, but a value of 1 <class 'int'> was specified.

This error was supposed to be fixed after v.1.3.0.post2, so I'm not sure from which version the user-home-test branch was created...
Thanks.

Hi @pvelasco, we've just released 1.3.0.post3 that should take care of both issues.
I'm going to close this one in favor of the neurostars thread (https://neurostars.org/t/singularity-fmriprep-permissionerror-errno-13-permission-denied-cache/3693). Please feel free to reopen if this is still an issue.

Hi @oesteban,
I tested 1.3.0.post3 (specifying a user) and I got a different error (also related to permissions inside the docker image):
Process Process-2:
Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
    self.run()
  File "/usr/local/miniconda/lib/python3.7/multiprocessing/process.py", line 99, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/cli/run.py", line 755, in build_workflow
    err_on_aroma_warn=opts.error_on_aroma_warnings,
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/workflows/base.py", line 218, in init_fmriprep_wf
    err_on_aroma_warn=err_on_aroma_warn,
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/workflows/base.py", line 516, in init_single_subject_wf
    num_bold=len(subject_data['bold']))
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/workflows/bold/base.py", line 399, in init_func_preproc_wf
    bold_reference_wf = init_bold_reference_wf(omp_nthreads=omp_nthreads)
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/workflows/bold/util.py", line 121, in init_bold_reference_wf
    omp_nthreads=omp_nthreads, pre_mask=pre_mask)
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/workflows/bold/util.py", line 302, in init_enhance_and_skullstrip_bold_wf
    'epi_atlasbased_brainmask.json')),
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/ants/registration.py", line 935, in __init__
    super(Registration, self).__init__(**inputs)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/ants/base.py", line 76, in __init__
    super(ANTSCommand, self).__init__(**inputs)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 645, in __init__
    super(CommandLine, self).__init__(**inputs)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 182, in __init__
    self.load_inputs_from_json(from_file, overwrite=True)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 495, in load_inputs_from_json
    with open(json_file) as fhandle:
PermissionError: [Errno 13] Permission denied: '/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/data/epi_atlasbased_brainmask.json'

The problem is that the package_data in fmriprep gets installed with the same permissions as in /src/fmriprep, which are -rw-rw---- (only root and its group have rw access).
I have a fix for it, and will be submitting a PR shortly.
(Note: I can only submit PRs to branches, not tags, so which branch do you want me to submit the PR to?  To master, since the problem is still there?)

This is surprising, why tests would then even work?. Yes, send the PR to master, please.

Sorry, I got it wrong: in the master branch, the permissions are correct.
I think the problem is the --no-cache-dir in the pip install .[all]
I tried building the Docker image for 1.3.0.post3 with
pip install .[all]

(omitting --no-cache-dir) and it runs for a regular user.
Bottom line: tag 1.3.0.post3 is fine except for the --no-cache-dir.
So I'm closing the PR.  Thanks a lot for your help!

Hi, the latest release 1.3.1 is out. Please let us know if that version resolves this problem!

Hi @oesteban,
Yes, it does.  It works fine.
Thanks a lot!

authors: oesteban pvelasco

repo: poldracklab/fmriprep


text:
Cowell method fails to converge with small perturbation accelerations

🐞 Problem

This code converges normally:
from astropy import units as u

from poliastro.twobody import Orbit
from poliastro.bodies import Earth
from poliastro.twobody.propagation import cowell

r0 = [-2384.46, 5729.01, 3050.46] * u.km
v0 = [-7.36138, -2.98997, 1.64354] * u.km / u.s

initial = Orbit.from_vectors(Earth, r0, v0)

def accel(t0, state, k):
    v_vec = state[3:]
    norm_v = (v_vec * v_vec).sum() ** .5
    return 1e-5 * v_vec / norm_v

print(initial.propagate(3 * u.day, method=cowell, ad=accel))

But changing to 1e-6 * v_vec / norm_v fails to converge:
$ python ex0.py
/home/juanlu/.miniconda36/envs/poliastro36/lib/python3.6/site-packages/scipy/integrate/_ode.py:1095: UserWarning: dop853: larger nmax is needed
  self.messages.get(istate, unexpected_istate_msg)))
Traceback (most recent call last):
  File "ex0.py", line 20, in <module>
    print(initial.propagate(3 * u.day, method=cowell, ad=accel))
  File "/home/juanlu/Development/poliastro/poliastro-library/src/poliastro/twobody/orbit.py", line 271, in propagate
    return propagate(self, time_of_flight, method=method, rtol=rtol, **kwargs)
  File "/home/juanlu/Development/poliastro/poliastro-library/src/poliastro/twobody/propagation.py", line 209, in propagate
    r, v = method(orbit, time_of_flight.to(u.s).value, rtol=rtol, **kwargs)
  File "/home/juanlu/Development/poliastro/poliastro-library/src/poliastro/twobody/propagation.py", line 103, in cowell
    raise RuntimeError("Integration failed")
RuntimeError: Integration failed

🖥 Please paste the output of following commands
pip freeze | grep astropy
astropy==3.0.2
pip freeze | grep poliastro
-e git+git@github.com:Juanlu001/poliastro.git@34a9e2c83cd77e918feb0182d2fa162ba06cbd07#egg=poliastro
🎯 Goal

I would expect a zero perturbation acceleration to be equivalent to a keplerian orbit.
💡 Possible solutions

📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #<put issue number here>
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

The current API fails with this case:
def accel(t0, state, k):
    v_vec = state[3:]
    norm_v = (v_vec * v_vec).sum() ** .5
    return 0.0 * v_vec / norm_v

Can you add a corresponding test to #368 to see if the new solvers pass?

repo: poliastro/poliastro


text:
Added option to configure tensorboard docker image

Should be something like that, @mouradmourafiq what do you think ?

Thank you @mouradmourafiq  !

repo: polyaxon/polyaxon-chart


text:
Handle unset experiment in log_artifact(s) call

Not entirely sure why self.experiment is set to None in the Experiment class init, regardless I've added a simple if statement that will allow the use of the log_artifact(s) helper methods and shouldn't impact existing functionality.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ryan Armstrong 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.

I think the implementation was just wrong, there are also 2 other methods, log_output and log_outputs on the base tracker, I am going to clean the client before the v0.5 release, especially these artifacts method, since the platform will be providing annotation for image, dataframes, model among others to specify the type of the artifacts.

Is the v0.5 release happening soon? Should I close this?

@rcarmstrong yes we are testing and pushing hard to have a RC soon.

Just an update, I fixed the docs to reference the log_output(s) methods and fixed the log_artifact(s) or next release since the other method are deprecated.

authors: CLAassistant mouradmourafiq rcarmstrong

repo: polyaxon/polyaxon-client


text:
Polybar crashes when using i3-msg for moving containers quickly

/usr/include/c++/11.1.0/bits/stl_vector.h:1045: std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp, _Alloc>::size_type) [with _Tp = std::__cxx11::basic_string<char>; _Alloc = std::allocator<std::__cxx11::basic_string<char> >; std::vector<_Tp, _Alloc>::reference = std::__cxx11::basic_string<char>&; std::vector<_Tp, _Alloc>::size_type = long unsigned int]: Assertion '__n < this->size()' failed.
this is the log that gives me when executes this piece of code
i3-msg "workspace 5; append_layout ~/.config/scripts/dummy_window.json"&
  termite -e "nvim -u $HOME/.config/nvim/notes.vim -c "startinsert" $HOME/.notes/notes"&
  pid="$!"
  echo ${currentWsName}
  i3-msg "workspace ${currentWsName}"
while : ; do
  winid="`wmctrl -lp | awk -vpid=$pid '$3==pid {print $1; exit}'`"
  [[ -z "${winid}" ]] || break
done
  i3-msg '[id="'$winid'"] floating enable'
  wmctrl -i -r $winid -e 0,$x,50,1000,1000

what it does is that it gives me a container window id and then move it to 5th workspace and take me back to workspace 1 and then bring the container within a window back to where i am, is so quick that it completely freezes polybar and then kill it

I'm unable to reproduce this. Could you share the following:

Your polybar config
The output of polybar -vvv
The entire polybar output (if possible using trace logging -l trace)

Closing due to inactivity

repo: polybar/polybar


text:
AIFF invalid FORM chunk size fix

Hello, this is a fix to a bug with AIFF I just noticed - I forgot to update the FORM chunk header size when the ID3 size changes.
I've also updated the test so it uses temporary file, rather than writing to testdata directory.
Sorry for the issues, thanks for the work.

I could add ffprobe test, since that's how I found this issue, however it is present only in files with actual audio data, the sample AIFF I provided is just minimalistic hand made one without any data other than headers.

FFprobe is fine, there already is another test that uses it anyway ;) Would it be possible to transcode and commit quiet.mp3 so we can use that?

Hello, I've added the quiet.aiff file and updated the test to include ffprobe. Also I've changed the API - read_from_aiff - wants io::Read + io::Seek so it can be used for reading from memory for example. Now read_from_aiff_file does the same as the previous API did.


text:
bringing the terraform state hack inside the cndi binary

Currently the way terraform state is maintained is within GitHub Actions. This works fine, but the code needs to be reimplemented within each supported CI systems (GitLab, etc.). What if the calls to git checkout _state, gpg --symmetric ... git add terraform.tfstate.gpg etc. were made inside the binary?

PR: https://github.com/polyseam/cndi/pull/105

repo: polyseam/cndi


text:
Update googleAnalytics.tsx to properly track goals

I think the messages are not being logged because they had to be set as the label, instead of the value which should be a float.

As per https://developers.google.com/tag-platform/devguides/events


text:
should these 2 links take you to the same page?

both "view  255 genes...."  links go to the same page. Is this intentional?

They don't go to the same page.  The first is a table of genes.  The second link is a table of single allele genotypes and has an Allele column.

Of course. I was not seeing  the "allele". I don't use this link much.
I wonder if it would be better to present the allele column first in this view? (it should not affect anything because  the download options are not available from here).

I wonder if it would be better to present the allele column first in this view?

Sounds sensible.  Shall I go ahead and do it?

Just wait in case @mah11  thinks there is a good reason not to.
I think it would be useful for this page to be a bit more obvious that it's  genotypes (and a bit different from other pages), so I'm for it

no objection; no strong preference

I wonder if it would be better to present the allele column first in this view?

Should we continue order by the product as we do for gene tables?

OK...

All done.

authors: ValWood kimrutherford mah11

repo: pombase/website


text:
Unique URL for every commit

I think to be able to share a git-history URL pointing to a specific commit in the file history would be pretty useful. At least it's definitely useful to me, personally.

That's something I want to add too. Check #42

authors: areebbeigh pomber

repo: pomber/git-history


text:
Ensure config file directory exists before creating config file

If the config file directory doesn't already exist, the application will crash on startup while trying to create the default configuration. Fix this by creating the directory.
Drive-by: Replace os.path functions by pathlib's functions

LGTM. However, please use semantic commit messages.
In this case it would be "fix: ensure config file dir exists".
Correct the commit message and I will merge.

No problem, I've just updated it. Feel free to update the PR title to match if you prefer.

authors: mathieu-lemay pommee

repo: pommee/Pocker


text:
POND-983: Add support for torch_func [upstream]

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[ ] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] signed commit with git commit -s
[ ] Resolves #?
[ ] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date

There's some incidental formatting changes from running black on numpy/arr.py. I tested that a few functions work locally on pushdown (torch.mul, torch.ge, etc.), but I haven't added in test cases since it would require adding an extra dependency.

repo: ponder-org/modin-public


text:
Participación ciudadana

La liga te lleva al mismo landing.

esto no ocupó cambio, sólo ocupabas publicar en plaza pública. cierro el issue

authors: almaosorio ponentesincausa

repo: ponentesincausa/politicadedatos


text:
Update README.zh-CN.md

修正 Pontx 配置指南的地址

Appreciate.

repo: pontjs/pontx


text:
Feature: Explicit partial calls.

This PR implements #1771.

As promised in #1771, I've prepared a script that can be used to automatically migrate your codebase to use explicit partial calls.  You can find it here:
https://gist.github.com/jemc/95969e3e2b58ddb0dede138c737907f5

This is nearly done, but I'm running into issues with SEGVs in the JIT-using compiler tests. @Praetonus, I was hoping you could take a look, since you know the most about those JIT tests.
I don't understand how the SEGVs could be caused by this kind of change. My first thought was that f43d671 might be related since it changes a bit how token objects are freed in the parser, but reverting that commit and running the tests with all the other changes included still saw the same errors.

@jemc I'll take a look.

I'm not seeing any segfaults locally, do you have a minimal case for that?
Also, it looks like you didn't update the grammar file, the CI is failing because of that.

@Praetonus figured out why I was seeing SEGVs, and filed #2047 to fix it :+1:

As soon as this passes, it's ready to merge. However, I've left on the DO NOT MERGE label because I want to clear this wit @SeanTAllen first and make sure we have whatever release notes or other text we need written up to make this a release.

@jemc what platforms has the migration script been tested on?

@SeanTAllen - only my own (Fedora 22 Linux).
I suspect it should work on any Posix-compliant system with bash. I'm not sure what we should do about the Windows crowd - I hear that the latest windows has bash support, so that might work okay.

I can test on OSX sometime in the next few days (maybe this weekend).
I think it would be good to get someone to test on Windows in some fashion or come up with a Windows solution if we can.

@kulibali Would you have a moment to look at this?

Sure, I can take a look this evening.

I have created an equivalent Windows PowerShell script at https://gist.github.com/kulibali/cd5caf3a32d510bb86412f3fd4d52d0f

I am running into an issue where I have a class with an add method that is partial, which I can use the + sugar to call.  Compiling with this update gives the usual error message:
C:\Users\Gordon\Dev\Pony\kiuatan\src\kiuatan\_test.pony:192:24: call is not partial but the method is - a question mark is required after this call
  let next = start + str.size()
                   ^

The script changes the code to
let next = start +? str.size()

But this doesn't compile.  Is there a way to call a partial add method using the + operator?  Or do I have to use add explicitly?

Discussed this in the sync call.
I will update the PR to make the +? syntax work for @kulibali.
I will rebase to fix the merge conflicts.
I will get a thumbs-up from @kulibali and @SeanTAllen on the migration script working before merging.
We'll ignore the pony-stable compilation failures in the CI, then follow up with a PR to fix pony-stable.
After merging we'll want to initiate the 0.16.0 release fairly soon, so users of the "latest release" of pony will be able to have codebases that compile, which also compile for users of the "latest master revision" of pony.

Alright, @kulibali and @SeanTAllen - this is ready for your final testing on Windows and MacOS before we merge.

Works for me 👍

@SeanTAllen gave me permission offline to merge this and fix any possible MacOS issues with the migration script later.
Here we go...

authors: Praetonus SeanTAllen jemc kulibali

repo: ponylang/ponyc


text:
Feature Request: Bind to IP

I was wondering if there is any way to support binding to a particular IP address or if you might be able to address that in a future release?  I have multiple domains on a single dedicated server with 16 individual addresses.  It appears that Ponzu will listen to all addresses on the HTTPS port.  Thanks!

We definitely should support this - and I can try to get it added this weekend.
Would another CLI flag paired with the run command suit your needs?

Are you kidding? YES.  Right now that's my biggest limitation to implementing Ponzu.  Whenever you are able to get to it, please know it will be very much appreciated!

Sure thing! I'll ping you here once I have it complete. Out of curiosity, would you be able to share a bit about:

how you found Ponzu
what you're building
the deployment / hosting environment you're running

Feedback like this is super helpful to make it a better product & dev experience.
Thanks,
Steve

I have been searching for headless cms for sometime.  I have not been satisfied with the shoehorning of established CMS' into this area.  I also believe that website development should be moving towards application-based designs.  So I basically found Ponzu through Google.
My interest is two-fold:  I have need of an API interface for a small chain of movie theatres where imdb/tmdb solutions for information is overkill and limits the owners ability to customize the results.
Second, I would like to build a frontend framework that could take information from Ponzu and generate complete web applications using Angular.
Currently I use NGHTTPX as a reverse proxy to Nginx.  Why?  Because of its HTTP/2 Push ability that Nginx still lacks (among other reasons).  My server is an OVH dedicated server with 64gb a 2TB raid array and modern processor (forgot the nomenclature right now).
I was also curious about the References add-on for Ponzu.  It reminds me of JSON-LD on the surface (I haven't had a chance to dive deeply into it).
Thanks, again, for your prompt response.  I was stunned by the speed with which you responded!

Hey @webeau -
This is now available in the master branch. You can get it by running $ go get -u github.com/ponzu-cms/ponzu/... and then from inside your projects, you can run $ ponzu upgrade to make sure each project has the latest core code.
Let me know if this works for you. The new --bind option for the CLI run command is documented here: https://docs.ponzu-cms.org/CLI/General-Usage/#run
Thank you for the feedback -- that sounds like a great set up and I think Ponzu would be a perfectly suitable option for your CMS / API needs. If you need any other help or have other thoughts about Ponzu, feel free to file another issue or chat with the community on slack on the #ponzu channel at https://gophers.slack.com/messages/C3TBV356D/
I had not seen JSON-LD before, but you are right - the references concept in Ponzu is very similar! I think the added bonus of Ponzu's architecture is that since the JSON responses reference same-origin data URIs, you can easily push them down with HTTP/2 Server Push. You probably already saw Ponzu's Server Push integration, but if not, it's as easy as adding a Push() method to your Content types. Here are the docs for that: https://docs.ponzu-cms.org/Interfaces/Item/#itempushable

Second, I would like to build a frontend framework that could take information from Ponzu and generate complete web applications using Angular.

If you get around to starting this, please let me know -- I'd love to see if I could help or at least follow along :) You might find @natdm's Typewriter project interesting since it helps sync your Go content types (data models) with your front-end code. There is an example using Ponzu.

authors: nilslice webeau

repo: ponzu-cms/ponzu


text:
Lack of removeEventListener in unmount event

As title, the eventListener would never be removed once added.

Added.
df238ef

authors: Chibaheit vuryleo

repo: poooi/plugin-prophet


text:
[BUG]: No checks applicable

Description
There are two bugs I would like to fix.

When the product is added to the cart, there is no requirement for size. Without the size being selected, it is being added to the cart. I want to add a validation check for that.
After checkout, the cart should be empty. Even after checkout, the cart shows all the items which were just purchased. I would like to add a validation check for that as well.

Kindly assign me this issue under GSSoC'24 with an appropriate level.
Screenshots
Bug1

Bug2

Any additional information?
No response
What browser are you seeing the problem on?
Chrome

Congratulations, @abckhush! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Hello @pooranjoyb  I would like to work on this issue under gssoc'24

Assigned to @abckhush based on fcfs basis. @masterboy376 next on the line :)

authors: abckhush masterboy376 pooranjoyb

repo: pooranjoyb/popShop


text:
SingleSample VCF from VDS correction

Another day, another gVCF -> VDS -> MT -> VCF -> Validation hiccup. Here the issue is that MTs that originate from VDSs don't have the FILTERS field, it simply doesn't exist. The 'single sample VCF from MT' script expected a Filters column, so we run into a failure when running the mt.filters.length() == 0 test. When we write the MT to VCF it does generate an empty FILTERS field on all variant rows, so there are no other compatibility issues with downstream tools (VEP, Hap.py, VQSR).
Couple of changes:

Drop gvcf_info completely. We can put it back in later if we need it, but for now we can't export VCFs with this field present, so strip it out early on
Add a --clean flag to the VCF-from-MT script. If used this will remove all non-variant rows, AND filter on mt.filters (but only if filters exists in the MT)
Drops mt.variant_qc before writing the VCF. I think this would have been dropped anyway, but just to make sure
Replace repartition with naive_coalesce - there are genuinely empty partitions (at least in the validation dataset), so reducing the number of partitions can be done cheaply by removing empty ones completely. Computationally this is much cheaper than doing a full repartition. I'm not sure if we'll use this route anyway, as it would be better for us to feed exact partitions into the combiner up front.

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

Project coverage is 26.62%. Comparing base (69e951c) to head (e54b7dc).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1011   +/-   ##
=======================================
  Coverage   26.62%   26.62%
=======================================
  Files           9        9
  Lines        1705     1705
=======================================
  Hits          454      454
  Misses       1251     1251

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

authors: MattWellie codecov-commenter

repo: populationgenomics/production-pipelines


text:
wip: annotated typedocs progress

Quite a few misc complications still with the typedocs. Not entirely sure if the problem is how we do things, how they assume people do things, or something in between.
I did discover that pkg-utils is blocking our ability to wholesale use all the typedocs tags. Should be able to manually add them via package.config.ts until we hit an inline tag. At the point the library will need an update since the type only supports block and modifier right now.
If the groups or categories will actually work, I think we'll be in good shape, but this implementation is still just "okay".

The createMarkdownBehaviors page is a good example of the bizarre way it handles some properties.

repo: portabletext/editor


text:
Using https://my.host.name/v2/ for private repo chokes

I tried to configure a private registry using the host url I docker log'ed in to - which was like
https://my.host.name/v2/
It turns out that portainer didn't like that, but gave no immediate feedback.  Instead it said:
"Image from container: invalid reference format"
when I tried to add a new container.
I recommend that when you add a registry portainer tries to validate it.  If that's not possible then it'd be nice for the failure message to be way more explicit.  Maybe including some technical error information.

You should probably try with https://my.host.name instead of https://my.host.name/v2/
I agree, we will try to test the connectivity before creating the registry.

Yes, that is the fix.  My point was that it wasn't clear what the issue was.  A connectivity test would have nailed it at registry creation time - if that's possible...  Thanks!

i got the same issue
my setting was like below
Registry URL https://njdocker1.nj.thundersoft.com
name harbor
then i want to use pull image
name njdocker1.nj.thundersoft.com/public/rsyslog:1.0
Registry harbor
i was setting the auth info and if i use docker pull njdocker1.nj.thundersoft.com/public/rsyslog:1.0 can pull the img from my own Registry

You need to add the port number to the registry url... port 5000 is Docker default..
Rgds,
Neil Cresswell
On 20/09/2017, at 5:17 PM, zwx168238 <notifications@github.commailto:notifications@github.com> wrote:
i got the same issue
my setting was like below
Registry URL https://njdocker1.nj.thundersoft.comhttps://urldefense.proofpoint.com/v2/url?u=https-3A__njdocker1.nj.thundersoft.com&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=H99wJQGgBWkcoainkjvqEarIuUvyd7LMw--mBBb76rw&s=05gffEv0a9BEG9dhnOJVQ6Idmcy0-e1Qc4BDMsMSMHw&e=
name harbor
then i want to use pull image
name njdocker1.nj.thundersoft.com/public/rsyslog:1.0http://njdocker1.nj.thundersoft.com/public/rsyslog:1.0
Registry harbor
i was setting the auth info and if i use docker pull njdocker1.nj.thundersoft.com/public/rsyslog:1.0http://njdocker1.nj.thundersoft.com/public/rsyslog:1.0 can pull the img from my own Registry
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_1182-23issuecomment-2D330809591&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=H99wJQGgBWkcoainkjvqEarIuUvyd7LMw--mBBb76rw&s=aEJdJjYz1FStqqcwPSyWlyv6KoimWQ9j0aDevWJMVjE&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrla8I91y6j95cuQrEc5TPPl7RFqQFks5skOZRgaJpZM4PQGVM&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=H99wJQGgBWkcoainkjvqEarIuUvyd7LMw--mBBb76rw&s=jzqDW_7Ayi9402eZgpLYK2ZqvY6ILZyKa72_xkpVL1Q&e=.

@ncresswell ths but we use harbor ,the default port was 80.but even add the port it still does't work
throw "Failure:invalid reference format""

@zwx168238 are you using njdocker1.nj.thundersoft.com as the registry URL and public/rsyslog:1.0 as the image you pull ?

@deviantony i use 	njdocker1.nj.thundersoft.com/public/rsyslog:1.0 as the image,it was the full path

@zwx168238 you need to create a registry first using njdocker1.nj.thundersoft.com and then select that registry when creating a container / pulling an image and use public/rsyslog:1.0 as the image name.

@deviantony  actually i was do like this

@zwx168238 and this is not working? Feel free to ping me on Slack to discuss this.

@deviantony  ths  for you hard work , now it was works

Hello! I would like to start my contribution to here. If no one working on this, I would like to take it :)

@asasmoyo Nobody is working on this yet, feel free to open a PR :-)

Great :)
I am thinking to do http request to URL/v2 then check wether it returns 200. Do you think is it enough to just do this?

@asasmoyo yes, that should be enough for a check.

@deviantony please take a look my pr

authors: asasmoyo deviantony kwerle ncresswell zwx168238

repo: portainer/portainer


text:
LDAP Auto create Users

Hi LDAP authentication is great but is it possible to have a switch to enable autocreate users ?

Same opinion! Also add support to get other attributes from LDAP and Attribute Mapping like Name, SureName, Email und Team Membership in relation to LDAP Attributes

We can, but you would still need to define which users can access which endpoints... unless we switch “teams” to be based on an LDAP group..
Rgds,
Neil Cresswell
On 9/12/2017, at 8:29 AM, Yanis LISIMA <notifications@github.commailto:notifications@github.com> wrote:
Hi LDAP authentication is great but is it possible to have a switch to enable autocreate users ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_1483&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=hmnIAffQMcto5tvSPmMMdaWs-QW1o8TV2HHDRZnhMt0&s=ota0FOoR8rNY-fDxPbNmp5-SZC6OIb1qumK7pivMzSE&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrldSZHhVDOIMwagBbaeMroEXfIyTkks5s-2DY4LgaJpZM4Q7ilt&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=hmnIAffQMcto5tvSPmMMdaWs-QW1o8TV2HHDRZnhMt0&s=gBHIZC97XkHu6YovhKkMIWxCksNm4oweW-Oagc6Sd8k&e=.

@zeenlym See my comment, i think this feature is also needed. Don't use only LDAP Groups. I would suggest to make the relation to a team flexible with LDAP Attribute Mapping.

@xoxys i'm agree with you

@xoxys please open another issue for  LDAP attribute mapping. We'll track the user auto-creation feature in this one.

@AlexJakeGreen feel free to open a PR :-)

@deviantony Here is the PR #1839
Thanks!
ps Maybe, it is better no not store ldap users in db at all since Portainer already uses signed cookies and user data (including roles) can be kept there. But I don't know how this approach fits further development plans and looks like it should be a different story.

I just had a look at the PR @AlexJakeGreen
How do you address the fact that even if you automatically creates the user in LDAP they're still unable to access any endpoint? Thus, new users are still blocked at the authentication screen.

New user receives jwt token, but yes, he is not added into any group and thus has to be managed additionally via web ui and this PR solves only a part regarding user autocreation.
So, user's permissions have to be granted in some way, and I can see several strategies,
1 Add new user into some preexisting 'default' team - in this way we skip ldap groups and still need manually give permissions via web ui. Possible, but not flexible for me because all my users live in LDAP
2 Add a third hardcoded ReadOnlyRole (first two are Admin and StandardUser) and assign it to user - user will be able to see resources from start, but admin still needs to assign him to a proper team later. This starts to be ok for me, but community may have different opinion, so confirmation needed. There was an issue for RO, but now it is closed and seems will be implemented in different way
3 Implement mapping ldap group -> team, is the best, but needs much more work in both golang and js code + still need something for readonly access...

IMO the best solution is to implement the LDAP group <> Portainer team mapping.
What's your thoughts @ncresswell ?

Regarding read-only access, this is going to be tackled in https://github.com/portainer/portainer/issues/1259

I agree with ldap - team mapping. When a user logs in, if they are a member of a ldap group that has a corresponding portainer team associated, then autocreate their account in that team

Hy every one,
I think LDAP mapping is the best solution, I can wait for it to be avalaible.
Thanks,

authors: AlexJakeGreen deviantony ncresswell xoxys zeenlym


text:
Swarm visualizer - color by service

Could we implement a color by service model, like in https://github.com/dockersamples/docker-swarm-visualizer

I kwow color are for task status for the moment, but when we show only running tasks, it could be cool to identify that my service run on all nodes !
For information same cluster in portainer swarm visualizer :

Thanx for reading

Hi, I have tried a solution like the image posted below.

The idea is to check service status using task background color and status lable and use the border color to identify the service.
The function used to get the color for service is the same used into docker-swarm-visulizer, who use the service id.
Could this example be a good solution for this feature?

@maocorte looks good, I'll have a look at the PR.

I'm sorry for the necro-posting, but this is the only thing I found about those  frame colors that are litteraly driving me nuts.
Form the PR it seems like they mena pretty much noting (thanks for make me go nuts on the for weeks, btw 🥹)
Checking the current code the function generating the random colour form the container ID is gone, bu and in its place I found this
visualizerTaskBorderColor
https://github.com/portainer/portainer/blob/develop/app/docker/views/swarm/visualizer/swarmvisualizer.html#L110
which I can't honestly find what it should mean.
So I'm pretty much back to square one.
What do the colored frame mean?
Kindly, my brain just goes spinning at 1000% and gets frustrated every time I drop into that view 😭, so many visual clues and not an explanation is just torture :(

authors: WTFKr0 deviantony maocorte unlucio


text:
Container not being listed in Container tab

Bug description
When I search for a container after I restarted it, it disappears in the container tab for a few minutes. docker ps  seems to still find it.
Expected behavior
Portainer should always show every container that docker ps can show, given I have enough privilege for that container.
Steps to reproduce the issue:

Go to a container
Click on restart
Search it in the container tab
Can't find it

Technical details:

Portainer version:  1.23.0
Docker version (managed by Portainer): 18.09.1
Platform (windows/linux): Linux
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
 docker run -d -p 9000:9000 -p 8000:8000 --name portainer --restart always -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainer/portainer
Browser: Firefox 71.0

Additional context
This problem appeared after the update to 1.23.0 and activating the Role Access Extension.
Update: It seems like Portainer has issues displaying containers with healthcheck in a starting state.
It immediately showed up after I stopped the container with the CLI.

Closing as I believe this is a duplicate of #3146
However I think you could mention This problem appeared after the update to 1.23.0 and activating the Role Access Extension. as this is an interesting observation that may help to fix this bug.

authors: itsconquest qmager


text:
Include SVG as available output format for saving table to file

Prework

[X ] Read and abide by the great_tables code of conduct and contributing guidelines.
[ X] Search for duplicates among the existing issues (both open and closed).

Proposal
The ability to save tables to SVG file format would be very useful for including GreatTables into composite scientific figures! Albeit, I am ignorant on how large of an undertaking this may be.

SVG output would be great! I am using Typst and it cannot import pdfs only svg.

I added the following the _save_screenshot in _export.py that does the trick. Kluge fix, so not creating a pull request.
if path.endswith(".svg"):
        # Get the table HTML content
        table_html = driver.find_element(by=By.TAG_NAME, value="table").get_attribute('outerHTML')

        # # Import svg converter
        # _try_import(name="svgwrite", pip_install_line="pip install svgwrite")
        # import svgwrite

        # # Create SVG drawing
        # dwg = svgwrite.Drawing(path, size=(f"{required_width}px", f"{required_height}px"))

        # # Create foreignObject element with proper namespace
        # foreign = dwg.add(dwg.g().add(svgwrite.container.SVG(
        #     insert=(0, 0),
        #     size=(required_width, required_height)
        # )))
        # foreign.set_desc(title='Table HTML', desc=table_html)

        # # Save the SVG
        # dwg.save()
        #
        #
        # Get any styles from the head
        # Get any styles from the head
        styles = driver.execute_script("""
            var styles = '';
            // Get stylesheet rules
            var styleSheets = document.styleSheets;
            for(var i = 0; i < styleSheets.length; i++) {
                try {
                    var rules = styleSheets[i].cssRules;
                    for(var j = 0; j < rules.length; j++) {
                        styles += rules[j].cssText + '\\n';
                    }
                } catch (e) {
                    console.log('Error reading stylesheet:', e);
                }
            }

            // Get computed styles for each element
            var table = document.getElementsByTagName('table')[0];
            var elements = table.getElementsByTagName('*');
            var computedStyles = {};

            // Important style properties to capture
            var styleProps = [
                'font-family', 'font-size', 'font-weight', 'color',
                'background-color', 'border', 'border-color', 'border-width',
                'border-style', 'padding', 'margin', 'text-align', 'vertical-align',
                'width', 'height', 'display', 'position', 'top', 'left',
                'border-collapse', 'border-spacing', 'line-height',
                'border-top', 'border-bottom', 'border-left', 'border-right',
                'padding-top', 'padding-bottom', 'padding-left', 'padding-right',
                'background', 'white-space', 'text-decoration', 'font-style'
            ];

            // Capture styles for each element with a class
            for (var i = 0; i < elements.length; i++) {
                var el = elements[i];
                if (el.className) {
                    var computed = window.getComputedStyle(el);
                    var classStyles = '';
                    styleProps.forEach(function(prop) {
                        var value = computed.getPropertyValue(prop);
                        if (value) {
                            classStyles += prop + ':' + value + ';';
                        }
                    });
                    if (classStyles) {
                        styles += '.' + el.className.replace(/ /g, '.') + '{' + classStyles + '}\\n';
                    }
                }
            }
            return styles;
        """)

        # Function to escape XML attribute content
        def escape_xml_attr(s):
            return (s.replace("&", "&")
                    .replace("<", "<")
                    .replace(">", ">")
                    .replace('"', """)
                    .replace("'", "'"))

        # Process table HTML to escape attributes properly
        import re
        def escape_html_attrs(html):
            def replace_attr(match):
                attr_name = match.group(1)
                attr_value = match.group(2)
                escaped_value = escape_xml_attr(attr_value)
                return f'{attr_name}="{escaped_value}"'

            # Find and escape attribute values
            pattern = r'(\w+)="([^"]*)"'
            return re.sub(pattern, replace_attr, html)

        # Clean up table HTML first
        table_html = (
            table_html
            .replace(" ", " ")  # Replace HTML entities with XML entities
            .replace("", "")  # Convert em to styled span
            .replace("", "")  # Close styled span
        )

        # Then escape attributes
        table_html = escape_html_attrs(table_html)

        # Create SVG wrapper with embedded styles
        svg_content = f'''<?xml version="1.0" encoding="UTF-8" standalone="no"?>
        <svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"
             width="{required_width}" height="{required_height}">
            <foreignObject width="100%" height="100%">

                    {table_html}

            </foreignObject>
        </svg>
        '''

        # Final cleanup of any remaining issues
        svg_content = (
            svg_content
            .replace("&#", "&#")  # Fix double-escaped numeric entities
            .replace("&amp;", "&")  # Fix double-escaped ampersands
        )

        # Save the SVG file
        with open(path, 'w', encoding='utf-8') as f:
            f.write(svg_content)

authors: MarekOzana claysmyth

repo: posit-dev/great-tables


text:
Unable to retrieve the complete list of server APIs: certificates.k8s.io/v1 with default deployment

I'm using the default options in deploy/k8s and only overriding the KUBERNETES_SERVICE_{HOST,PORT} environment variables but getting the following error about a minute or so after the pod starts:
E1211 19:49:34.608041       1 leaderelection.go:332]
{
"level":"ERROR",
"ts":"2023-12-11T19:44:49.789Z",
"logger":"controller-runtime.source.EventHandler",
"caller":"source/kind.go:68",
"msg":"failed to get informer from cache",
"error":"failed to get API group resources:
      unable to retrieve the complete list of server APIs:
          certificates.k8s.io/v1:
              Get \"https://API_SERVER:6443/apis/certificates.k8s.io/v1\":
                 dial tcp: lookup API_SERVER: i/o timeout"
}

I know that pods are able to talk to the api server because I have a running deployment of kube-state-metrics that also overrides the same env vars with the same values.
I'm running kubernetes 1.28.3 if that's helpful.

then I would assume you misconfigured your environment variables 😉
can you show me how you did that ?
also, it's typically not needed to customize the KUBERNETES_SERVICE_{HOST,PORT} envs, because K8s sets those automatically. can you try to run it again without modifying these envs ?

closing without further info

authors: clementnuss onetwopunch

repo: postfinance/kubelet-csr-approver


text:
Add Zesty.io

[x] verified that the CMS I'm adding is still maintained.
[x] read CONTRIBUTING.md.
[x] did not generate README.md.

Thank you, @shrunyan!

authors: mutewinter shrunyan

repo: postlight/awesome-cms


text:
Backup fails if an artifact is missing

Hi,
just tried the backup and it fails, I think as I've currently no lists defined.
Run potatoqualitee/fossilize@v1
Run if ("true" -eq $true) {
VERBOSE: Running script
VERBOSE: Exporting all to ./backups
VERBOSE: Going to https://ruhr.social/api/v1/accounts/verify_credentials
VERBOSE: Processing follows
VERBOSE: Exporting following
VERBOSE: Going to https://ruhr.social/api/v1/accounts/109303980548626757/following?limit=80

    Directory: /home/runner/work/TwitterInflux/TwitterInflux/backups

UnixMode   User             Group                 LastWriteTime           Size
--------   ----             -----                 -------------           ----
-rw-r--r-- runner           docker             [11](https://github.com/Callidus2000/TwitterInflux/actions/runs/3581174700/jobs/6023965634#step:3:12)/30/2022 07:23           1079
VERBOSE: Processing lists
VERBOSE: Exporting lists
VERBOSE: Going to https://ruhr.social/api/v1/lists
Get-ChildItem: /home/runner/work/_actions/potatoqualitee/fossilize/v1/main.ps1:138
Line |
 138 |          Get-ChildItem -Path $filepath
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path
     | '/home/runner/work/TwitterInflux/TwitterInflux/backups/lists.csv'
     | because it does not exist.

Error: Process completed with exit code 1.

I think that if I manually select "all without lists" in my config it would run, but then I'd have to remeber to change this after I've started to use the list feature.

@potatoqualitee - agreed, having no lists causes an issue.

thank you! just fixed and rereleased v1. so you can just rerun the same workflow. sorry for the potato qualitee, I had too much test data appaarnetly 😅

Worked perfectly for me - thank you! ⭐

Works as designed, Thank you.

authors: Callidus2000 jpomfret potatoqualitee

repo: potatoqualitee/fossilize


text:
Introduce metrics.context

Any properties bound to metrics.context will be merged into options for any calls to the service API

Thanks!

authors: opsb poteto

repo: poteto/ember-metrics


text:
Package release dates added to README.md

Updated README.md to reflect the version releases.
Release version is taken from /releases page

What benefit is this ?

I'm not sure that this adds anything useful to the project, but I think we should consider making "releases" a bit more documented using the features on Github to do so - if you look at Bootstrap's Release page, this contains a lot of useful information.  Maybe we can tag 0.14.1 as a fix for the bower version issue and then use that to draft a release?

This idea is from Changelog files, which keep track of major things accomplished in chronological order. Which is followed by many opensource projects.
Another way to achieve the same thing is to tag a version like its described by @glenpike

I'm going to close this as I would favour using the releases page more.

authors: exussum12 glenpike nareshv

repo: powmedia/backbone-forms


text:
Broken Link in README

'developer guides' link in README.md, https://poynt.com/tag/guides/, is broken.
Should it link to https://poynt.github.io/developer-docs/ instead?

@mll11 yup good point, it should

repo: poynt/poynt-python


text:
Update kubernetes/master/golang/master build

This is an automated PR via build-bot

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ltccci
To complete the pull request process, please assign  after the PR has been reviewed.
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
string index out of range

Describe the bug
When I try to build it gives the err
10:56:40 AM: Found section: 2-study/books
10:56:40 AM: Traceback (most recent call last):
10:56:40 AM:   File "__site/convert.py", line 34, in <module>
10:56:40 AM:     nodes[doc_path.abs_url] = doc_path.page_title
10:56:40 AM:   File "/opt/build/repo/__site/utils.py", line 209, in page_title
10:56:40 AM:     [
10:56:40 AM:   File "/opt/build/repo/__site/utils.py", line 210, in <listcomp>
10:56:40 AM:     item if item[0].isupper() else item.title()
10:56:40 AM: IndexError: string index out of range
10:56:41 AM: Building site...

Small Reproducible Example
https://github.com/AzadKshitij/My-Stuff
Steps to Reproduce the Bug or Issue
Check the link and repository
try cloning and hosting on Netlify
Expected behavior
Should build the pages properly.
Screenshots or Videos
What I got

What I have

Operating System Version
Windows.
Visual Studio Code Version
Latest.
Additional context
No response

Hi, I see that your site is up and running. What exactly was the issue? Did the build fail or was it some other error?

My build was failing then I tried deleting the folder and recreating them and it worked.

I would guess there were some symbolic link issues?

authors: AzadKshitij ppeetteerrs

repo: ppeetteerrs/obsidian-zola


text:
请问telegram转发的自定义地址格式是怎么样的

试过https://forward.address/bot<key>，提示无效
谢谢

是不是你的反代，只支持get形式？

是不是你的反代，只支持get形式？

用的是CF worker，试了一下应该是支持post的

（顺带一提吐槽一下，在miui12 20.6.18/小米6 上，tg api输入框是密码控件，没有粘贴选项，只能手打）

试试  https://forward.address/bot/sendMessage

不太清楚你的反代是怎么配置的，目前的机制：填写 http 开头时，不再进行api地址拼接，需要填写的完整的api接口地址

试试 https://forward.address/bot/sendMessage

成功了

多谢您的回答
是需要把~/sendMessage加上

authors: lincww pppscn

repo: pppscn/SmsForwarder


text:
Fix multiple issues related to mania key count stats population

Opening this for self testing / review

Closes #295
[x] Make sure this works as expected.

This should be good for review now. I'm running this in production as a final test run (after manually testing on users in both active and inactive state, confirming that active users see no-change and inactive users see their pp/acc getting updated).
I did take this opportunity to make the logic conform close to the main user stats processor, so the diff might be a bit hard to read. Just viewing the few files directly and ignoring the diff, treating as if it's a new submission, may work better.

repo: ppy/osu-queue-score-statistics


text:
Create main menu

For starters, it could just consist of a "press to start" message.

I will create the start menu.

I have begun creating the start menu. There are some issues with importing font and some features from the Screen class like SetScreen.

Did you get it working in the tutorial?

I am implementing a version of the tutorials so that it fits our project.

authors: carllei pqbyte

repo: pqbyte/coherence


text:
希望能支持 Rime -- 鼠须管/或其他中文输入法的输入法内中英文切换状态显示

安装 鼠须管 和 ShowyEdge 配合有逻辑矛盾，即 shwoedge 并不能显示鼠须管内中英文切换，实际上，rime 本身自动化在不同程序间切换，并替代系统英文输入法是简单又好用的，但为了配合 showyedge，强行出现了这样的用法：
https://pbeta.me/macos-from-zero-3/

关闭鼠须管的英文输入
把中英文输入法之间切换行为独特强调出来
这样才能够使用ShowyEdge的功能
如题，希望能有这样的功能

https://github.com/rime/squirrel/issues/146

我在使用搜狗拼音输入法，期望还能区别出搜狗拼音的中文状态和英文状态，谢谢。
目前只能给搜狗输入法整体设定一个色条，无法给搜狗拼音的中文状态和英文状态分别设定色条。

@Justsoos @yetangye
我用的搜狗输入法，我也想让其区分下搜狗输入法下按 shift 切换中英文的状态。
请这个问题你们有解决办法吗？

authors: GanZhiXiong Justsoos yetangye

repo: pqrs-org/ShowyEdge

license: Unlicense


text:
'iohyve create' doesn't work correctly

iohyve create doesn't work without an explicit pool when no guest is present.
That was my fault ( introduced in #165 ), because I try to get the pool by looking for the first iohyve guest and take the same pool for the creation of the new one.
To fix this problem there are two (probably more) ways to fix it:

Use the old method in create (grep for the first appearance of 'iohyve' in zfs list)
Add a zfs attribute (like iohyve=yes) to all datasets used for iohyve and then search for that

The 'old' method should work in most cases, but can fail in edge cases like someone has a dataset for iohyve backups on another pool named: anotherpool/iohyve-backups which is not intended to be used for iohyve but the old method would use it to create the guest on it.
The 'zfs attribute' method would require the attribute to be present on the iohyve dataset, so there must be some kind of migration to support existing installations.
Like when no iohyve:pool attribute is found, try to find the pool with the 'old' method and add the attribute to it.
So what is the way to go?

I, too, have fallen victim to this trap. No worries.
I say let's add the attribute. I have done similar things things in the past where we included a "fix me" function. I think there's still an archive of that on the wiki.

Input is highly appreciated from the peanut gallery.

What would be the expected behaviour if you would have multiple datasets with the iohyve=yes set?
I can imagine that this would result in an unexpected behaviour.
The example you name your self is already a point of attention, what if you have 3 pools on a machine which one is the backup. Which pool would you decide to use if trying to detect, the attributes will be copied to the backup too in our backup system.
One of the main reasons I prefer to manual define the pool.
"If you pay with peanuts, you get monkeys."

Good point. Haven't thought about that.
So we need a good way to define a default pool or we have to make the pool argument during create required.
If we are using a property to define a dataset, we must make sure that it is unique on a machine.
It would be a possibility to set iohyve=yes (or iohyve:default=yes to be more expressive) in the setup command. When it is set on one pool, it is removed from the other (all other) dataset(s).
Also the clone command must remove it from the clone.
I think it is a lot easier to have a default dataset then to specify it on every create.

The create command need some changes anyway, to support a guest to live in any dataset the user wants (see Roadmap).
We should keep that at the back of our minds.
The changes I made in 'ioh-guest' should already allow that for most of the other commands.

I say let's use a default pool. Most setups only have one pool anyway.

"If you pay with peanuts, you get monkeys."
Lol, I love it.

So, when the user runs iohyve setup pool=poolname the first time, it's assumed that it's going to be their primary pool.
Because of this, I added the following to the __guest_create() function:
if [ -z $pool ]; then
	local pool="$(zfs list -H | grep /iohyve/ISO | cut -f1 -d/)"
fi

I added this because I'm getting ready to submit v0.7.6 to ports.
This fix doesn't need to stay, but it gets the job done.

If there are no more comments from the monkeys in the gallery, I'll go ahead and close this issue out. If you feel this solution is not enough, I have no problem reopening.

For now the solution is fine, the over all create process could be nicer, I'll try to work out a concept first to demonstrate my idea.

:+1:


text:
feat: #536 adicionado referência sobre prazo de solicitação de serviço

ajustei para ficar assim:

authors: andregiachini zorteaadriano

repo: practice-uffs/mural


text:
Include subheadings in navigation tree

Describe the solution you'd like
Pages in my index toctree (with maxdepth > 1) should include subheadings in the navigation menu, similar to alabaster. Having this as optional behavior would be fine, as long as it's available to me. I've written longer pages, and would like the navigation tree to make it easier to jump to the right section without having to first go to that page, then click to expand the content menu, then click on the section.
Describe alternatives you've considered
Tried adding explicit toctree entries, but that does not work for linking to headings/anchors rather than documents, and the only other alternative is making more documents, but that seems like overkill, especially for things like autodoc content.
Additional context
Building docs for aioitertools: https://github.com/omnilib/aioitertools/tree/main/docs
index.rst includes api.rst in a toctree with maxdepth 2. alabaster includes the subheadings from api.rst in the nav tree, but furo doesn't.
With alabaster:

With furo:

Thanks for filing this issue!
I'm not going to be adding this since separating the "page structure" from "site structure" in Furo, is an explicit design choice. The left sidebar is for presenting site structure and the right sidebar is for presenting the page content hierarchy. I find the mixing of the two in other Sphinx themes to be a suboptimal experience when looking for information.

It seems you have since implemented this on your own documentation for Furo, but I see no way of activating it.

Those are shown because they are subpages, ie, TOC within the customisation/index.md page that was included in the main index.md page.

See https://github.com/pradyunsg/furo/discussions/318#discussioncomment-1706902.

authors: jreese pradyunsg rlaphoenix

repo: pradyunsg/furo


text:
Validate against special characters in Message

If a message contains special characters it may be badly encoded when sent over SMS (depending on the carrier and device).
There are some characters which are frequently bad, easily replaceable with ascii characters and are hard to spot by eye.
This commit adds a validator to prevent them from being saved.
We said we'd do this in a separate ticket but it ended up being so straightforward I thought I'd go for it.

So, the reason why we wanted a separate ticket for this is that not all channel types have issues with special characters. So for say SMS, we probably don't want special characters, but for an IP messaging channel, we probably do.
So I don't think that we want an always on thing that we cannot disable.

Ah yeah that's true. I guess in this case I'd argue that the benefit to SMS users of rejecting these 4 characters is significantly greater than the benefit to IP users of including the characters (they all have equivalents that look very similar). Especially given that I think the vast majority of messages go over SMS at the moment (?)

@alexmuller Good point. Yeah I think it's fine because we're only targeting specific characters, not all special characters.

Thanks! I agree - I think when we've got the majority of messages not being sent over SMS we could probably remove this quite easily :)

authors: alexmuller rudigiesler

repo: praekelt/seed-stage-based-messaging


text:
Moon Clip Animation[GSSOC'23]

Describe the project you want to add with tech stack

Hello Sir,
It is my new project of CSS animation where I have made  a animation of MOON CLIP with night views in it. I want to add this project into GSSOC'23.
Expected behavior

I have made it using HTML & CSS. Here we can see a moon clip.
Screenshots (optional)

Additional context (optional )

Please assign me this issue under GSSOC'23.

@MohitGupta121 @TusharKesarwani Please assign me this issue under GSSOC'23.

@MohitGupta121 @TusharKesarwani Please assign me this issue under GSSOC'23.

repo: pranjay-poddar/Dev-Geeks