text:
Change spacing policy of PLFA

Currently, PLFA is written using auto-fill-mode enforcing a line-length limit of 80 characters. This is far from ideal for prose, and not at all what we want for Agda code. We should move to a policy one line for every paragraph, which works well with line diffs, and avoids introducing issues such as #498 and #500.

But there is a major problem with the suggested approach, which is that it is much less readable in Emacs :(

For me, the readability isn't really a problem, since Emacs automatically wraps lines after a certain length, and my buffers don't tend to be too wide? However, it does lead to problems for me where the lines get harder to navigate. I think it's worth the trade-off, though: Markdown is generally layout-sensitive, and so, as we've noticed, auto-fill-mode can easily introduce rendering errors.

Nod

This was rejected by @wadler.

authors: jsiek wenkokke


text:
Last Update Lists.lagda

This is my last adjustment at the moment.
Before I proceed with the revision of Part 2, I have to read it carefully several times.

Ok, I agree with you. I will continue the update of Part 2.

source: gharchive/pull-request

license_type: permissive

license_source: github-api


text:
Revert "Add helper BodyClass for appending classes to the body tag from View …"

Reverts plone/plone-react#246
Reverting because noticed a problem that breaks the deal... :(

Coverage decreased (-0.2%) to 50.809% when pulling 8154b44ceca9a0a621fdee5276dba909aac44974 on revert-246-improved-helmet-body-classname-appendable into 60a40693c56557ad942549baa9e6f14b909761c3 on master.

authors: coveralls sneridagh

repo: plone/plone-react

license: MIT


text:
document the env vars feature

we need to add documentation on the new environment variables feature

fixed in https://github.com/ploomber/doc/commit/5b2cade17ba077582bb9f790e65077b271924536

source: gharchive/issue

repo: ploomber/doc


text:
Developer Experience Audit

Hey there! This is a general documentation audit. I've changed a few things:

Added a Code of Conduct. This is the same one used on ropensci/plotly.
Added a link to Github in the bugs and homepage field in the package.json.
Edited the headers on contributing and readme files (mostly depth).
Added 2017 to the license year.

Looks great. Thanks!

authors: RichardLitt chriddyp

repo: plotly/plotly-database-connector


text:
Error: Missing required POST parameters: un key origin when passing array of text

This is my setup for trace:
(It's TypeScript - ES6 syntax)
    x: this.users.map((u) => u.followerCount),
    y: this.users.map((u) => u.postCount),
    text: this.users.map((u) => u.username),
    // ...

The rest is just copied off the Colored and Styled Scatter Plot example.
When I run this, error from title is reported: Error: Missing required POST parameters: un key origin.
Although if I remove text: ..., it runs successfully.
I have checked the lengths and they match, all usernames are strings and there is also no falsy values.
Although if I change value mapped value to some fixed string, it runs. I'm unsure why it won't work for dynamic strings.
There are some special characters (emojis, etc.), could this be a problem, same as in #15?

Update 1: removing all non-ascii characters didn't help either. It repots a different error:
Error: Hm... Plotly had some trouble decoding your 'args'. ...
Which is as unhelpful as the first one.
Update 2: using regex [^a-zA-Z] as .replace(/[^a-zA-Z]/g, "") on each name finally worked. This is very dirty hack and I'm disappointed on that I had to use it.

@markogresak : Should be resolved in this PR, please let me know if it works for you

authors: alexander-daniel markogresak

repo: plotly/plotly-nodejs

license: mit

license_source: bigquery


text:
Support querying multiple measurements/series

Currently, when querying multiple series with chunked=True, there is no way to know which series a data point belongs to; the interface only provides the point and does not pass along the name of the measurement.
I fixed this for myself here: https://github.com/plugaai/aioinflux/compare/master...miracle2k:multiseries?expand=1 but it's not a clean pull request because it depends on my changes to make the library run on Python 3.5 (for PyPy).

Thanks for raising the issue!
I thought quite a lot about this and decided to totally drop built-in parsing for chunked responses in v0.2.0, so this issue doesn't apply anymore.
With the new iteration mechanism (iter_resp), users can customize point parsing as they prefer (for both chunked and non-chunked responses). It should allow for querying with multiple statements/series/measurements as long as you write an appropriate parser.
Things should now be more consistent (across chunked/non-chunked responses), more flexible (with the use of custom parsers) and easier to use (simple iteration via iter_resp).

repo: plugaai/aioinflux


text:
🛑 Dick Pluim.com is down

In 70134c8, Dick Pluim.com (https://dickpluim.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Dick Pluim.com is back up in fd764ff after 10 minutes.


text:
🛑 Dick Pluim.com is down

In 2c50870, Dick Pluim.com (https://dickpluim.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Dick Pluim.com is back up in 7024463 after 10 minutes.


text:
🛑 Pierced Arrows is down

In 69cb79c, Pierced Arrows (https://www.piercedarrows.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Pierced Arrows is back up in 2aeafb5 after 9 minutes.


text:
🛑 KCG is down

In 634c9af, KCG (https://kunstrijclubgroningen.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: KCG is back up in 7a1214d after 3 minutes.


text:
Add Skiff to notes / office

Skiff

Skiff
License: GNU AGPLv3 (partially open source)
Why do you think this helps users privacy?: end-to-end encrypted collaboration (Google Docs / Notion)
Under what section should this service be listed?: Notes and Office
Additional comments / info: The beta is currently free with no storage ceiling (limited to docs / pdfs)

Looks neat, I will keep a close eye on the project and I will list once I know more about it and I have tested!

Blogpost  Nov 16: The wait is over — Skiff opens to the public https://www.skiff.org/updates/waitlist-open
(I have not opened an account and have no opinion on the service.)

Just made a pr adding skiff drive. https://github.com/pluja/awesome-privacy/pull/344

authors: amarksnightwatch dhrm1k hkienle pluja

repo: pluja/awesome-privacy


text:
Not all axes and camera track included in output

I have a glb file with animated joints and a camera moving along a path, when load the model in gltf-viewer.donmccurdy.com the animation plays completely and I can switch to the Camera "Main_Camera" to see the camera movement.
Here a quick recording of how it should look like, excuse the choppiness my macbook has a hard time recording and rendering the webgl at the same time. :D
https://user-images.githubusercontent.com/3080903/121669396-8fc26700-caac-11eb-8549-c72d8f963da5.mp4
But when I convert the model to JSX using gltfjsx, load it in react and play the animation it only plays one of the animation tracks which is one joint, not the whole animation which is composed of multiple tracks, the camera movement being one of them.
Here also a quick screen recording of how it looks, also confusing is the weird lighting change.
https://user-images.githubusercontent.com/3080903/121669878-1bd48e80-caad-11eb-942a-40008b6f4fc6.mp4
This is the content of actions["animation_0"]._clip. As you can see there are multiple tracks with axes and the Main_Camera animated.

The gltfjsx output also only contains one group with the name "Axis-5", the actual model has "Axis-1" to "Axis-7",  with 1,2,3 and 5 animated.
Given that the animation plays perfectly in the gltf web viewer using three.js the .gtlb file must be correct and the problem happening during the conversion to jsx.
You can reproduce this by loading the attached .glb file in gltf-viewer.donmccurdy.com and under cameras selecting "Main_Camera".
I've also attached my output of gltfjsx using the command
yarn gltfjsx public/model.glb -p=6 .  I've set makeDefault to true in PerspectiveCamera and added a useEffect to play animation_0. I'm not sure if the exported PerspectiveCamera is the same as Main_Camera.
Archiv.zip

fixed it by doing a verbose output using the -v flag in gltfjsx.

oh thats interesting, could still be a bug if it took away something that was needed for skinned meshes

oh thats interesting, could still be a bug if it took away something that was needed for skinned meshes

Yeah thats true, I just played around with it and it fixed it by chance haha. Shall we reopen the issue again?

authors: AlexanderProd drcmda

repo: pmndrs/gltfjsx


text:
touch-action on iOS Safari

Describe the bug
On iOS 15.4.1, Safari browser, the touch-action behaviour suggested in https://use-gesture.netlify.app/docs/extras/#touch-action does not work well. If I try to drag the boxes horizontally, I need to slide perfectly horizontally -- if not, the drag stop and the screen scrolls instead.  NOTE: it works very well with the Chrome browser on iOS!
A workaround/solution to this is to set preventScroll: true (https://use-gesture.netlify.app/docs/options/#preventscroll) -- I'm reporting this more as a documentation issue.  (Also, preventScroll is advertised as an experimental feature.)
Sandbox or Video
https://user-images.githubusercontent.com/1161208/164729457-b6acd4a4-eb89-4064-9c2c-e7b5f43dbd2d.MOV
Information:

Use Gesture version: Whatever's currently on your web site
Device: iPhone 12 -- this has also been seen
OS: iOS 15.4.1
Browser: Stock/Safari.  (Chrome does not have this problem)

Checklist:

[x] I've read the documentation.
[x] If this is an issue with drag, I've tried setting touch-action: none to the draggable element.

I've noticed that for a while and this is just infuriating. Chrome works ok but for some reason pan-y stopped working properly in Safari. This is absurd. In those situations I'm now looking at methods similar to https://kor-ui.com/components/swipe-actions involving overflow.

You can also use preventScroll: true as a config option in useDrag.

authors: dbismut ukrutt

repo: pmndrs/use-gesture


text:
.member deprecated, replace with .contains

Hi Louis,
While trying to compile the test.chpl file there were warnings for the ".member" function being deprecated. I modified the files to replace ".member" with ".contains" on my end and tried to commit them but it seems as though I cannot push them to the master branch.

Hi Mandy, sorry for the late reply (totally slipped my mind).
If you submit a PR, I should be able to merge this in if you want.

authors: LouisJenkinsCS mandysack

repo: pnnl/chgl


text:
Script samples could also show CLI version

Currently, in our docs we do not show which version of CLI was used when some script was written.

This may lead to problems when someone tries one of our scripts with latest version of CLI. Some things could already changed and some options could already be deprecated. Of course it would be best if we would keep our scripts inline with latest major version but it is a lot of work and we may still just miss something.
in the Microsoft Adoption sample gallery this info is shown

It's kept in the sample.json in which we store all sample metadata

It would be best if we could also show this on our docs also coming from the sample.json asset so we won't need to maintain the version in two places.

@pnp/cli-for-microsoft-365-maintainers any feed on this? What is your take?

Seems like a good addition. On the other hand, our script samples should always works with the latest version of CLI. Unfortunately I know this is sometimes overlooked by us.

Sounds like a good idea! That would make it a bit easier to use for folks.

Seems like a good addition. On the other hand, our script samples should always work with the latest version of CLI. Unfortunately I know this is sometimes overlooked by us.

I think the problem is when you're on an older CLI version than the one used to build the script.

authors: Adam-it Jwaegebaert milanholemans waldekmastykarz


text:
Add 'planner plan details get' command closes #3051

Closes # 3051
New Command = Five Files

src/m365/planner/commands.ts
src/m365/planner/commands/plan/plan-details-get.ts
src/m365/planner/commands/plan/plan-details-get.spec.ts
docs/mkdocs.yml
docs/docs/cmd/planner/plan/plan-details-get.md

Thanks very much for the review @waldekmastykarz , my lack of knowledge of typescript is failing me. Please amend the line  132 from plan-details-get.ts  to remove reference to any.
const planDetailsItem: any | undefined = response;
I tried const planDetailsItem: PlannerPlanDetails | undefined = response.value;  but somehow the return data is not mapped to PlannerPlanDetails and nothing gets returned.
Thanks in advance for your help.

@waldekmastykarz : Thanks very much for all the guidance so far.  I need help , the Check PR / test (windows-latest ,16, 16) failed with error
Process completed with exit code 1.

Please advise on next steps.

Merged manually. Thank you! 👏


text:
Added commands to list messagingSettings, memberSettings and guestSettings.

Addressing #810, #811 and #812

Coverage decreased (-0.09%) to 99.908% when pulling 98c2d5a9348b0d26bb6884042ce0549cfeea8979 on vman:getteamssettings into 8762b1df83812151ff527ee1df0070b4e1ef5d09 on pnp:dev.

Thanks! I'll have a look shortly!

Merged manually. Thank you! 👏

authors: coveralls vman waldekmastykarz


text:
Added new command to list planner tasks #990

Hey @waldekmastykarz ,
I have completed my first command with reference to the issue #990 to list planner tasks of the user.
Please review my PR and if everything goes well merge it with master branch. I'm looking for your feedback.
Thanks,
Jayakumar B

Coverage decreased (-0.005%) to 99.995% when pulling ae0e148bd0f6725c69cdff03e2ed7b8068f1f28c on JayakumarB:list-planner-tasks into 8a81ecdcf004ff502a3083b4efc27d46387d0cf6 on pnp:dev.

Thank you @JayakumarB. We will review your PR shortly. We follow a different process with regards to PR where we merge them in to the dev branch first, release them as a beta version for a few weeks and if nothing out of the ordinary has been reported the dev branch is merged with master once per month to release a new version. If all is well, your PR will be included in the next beta release next week and then next release end of June/beginning of July.

Okay thank you!  Let me focus on next command #991 to get planner task by task id.

Excellent! Thank you @JayakumarB

The implementation looks ok, but when trying to retrieve tasks for a specific user, I'm getting the following error:
You do not have the required permissions to access this item.

The access token that I'm using has the Group.ReadWrite.All permission which should be enough to retrieve the tasks according to the docs. Am I missing something?

Reply
I have a raised an issue with Microsoft Graph team, it seems for now only current user tasks are only readable via Microsoft Graph planner API https://github.com/microsoftgraph/microsoft-graph-docs/issues/4766. Please suggest if I want to make any change in the command.

If it's not possible to retrieve other users' tasks, then we should not offer the userId/userName parameters to avoid confusion. When they will get introduced later, we can extend the command with additional functionality.

Thanks for the feedback. Let me remove the parameters and update readme/spec/command accordingly.

Hi @waldekmastykarz ,
I'm done with the changes to remove options from the respective files.
Thanks,
Jayakumar B.

Merged manually. Thank you! 👏

Thank you for your patience on reviewing this PR. I will make sure that the guidelines learned in this PR will be followed perfectly in next command.

I appreciate your effort. Looking forward to your next PR 👍

authors: JayakumarB coveralls waldekmastykarz


text:
[BUG] Invoke-PnPBatch : The maximum number of bytes allowed to be read from the stream has been exceeded. After the last read operation, a total of 1064000 bytes has been read from the stream; however a maximum of 1048576 bytes is allowed

Reporting an Issue
I have a SharePoint script inventory that runs some cmdlet like "Set-PnPListItem" and "Add-PnPListItem" with bach parameter.
Then every 50 cmdlets, I run "Invoke-PnPBatch -Batch $batch -Connection $ConnectionSiteInventory"
When executing "Invoke-PnPBatch", randomly I receive the error below :
Le nombre maximum d'octets pouvant être lus à partir du flux a été dépassé. Après la dernière opération de lecture, un total de 1064000 octets a été lu à partir du flux alors qu'un maximum de 1048576 octets est autorisé.
In English I think it's: The maximum number of bytes allowed to be read from the stream has been exceeded. After the last read operation, a total of 1064000 bytes has been read from the stream; however a maximum of 1048576 bytes is allowed
Here is some statistic for 4 complete execution of my script:

77 Invoke-PnPBatch, I get 3 errors
30 Invoke-PnPBatch, I get 6 errors
35 Invoke-PnPBatch, I get 5 errors
73 Invoke-PnPBatch, I get 6 errors

Here is the SharePoint item property that I set:

Actual behavior

What is the version of the Cmdlet module you are running?

Which operating system/environment are you running PnP PowerShell on?

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

Am moving this to discussion since this is specific to your code/script.
Would suggest maybe reducing the number of items to be sent in the batch and then try.
If anyone in the community has a suggestion, would be great to hear from them as well.

authors: gautamdsheth jolscr

repo: pnp/powershell


text:
[BUG] Invoke-PnPSiteTemplate messing up / and \ in current nightly

Category

[x] Bug

Describe the bug
Invoke-PnPSiteTemplate -Path "./tmp/pages/de/Page.xml" fails with "Object reference not set to an instance of an object." in Version  2.2.104-nightly when PageName in Page.xml is "folder1/folder2/Page.aspx" contains more than one subfolder. I guess this is the problem, slash and backslah messed up:

Even if correct in XML

Expected behavior
No Error, as using just one subfolder, like ./tmp/pages/Page.xml with PageName in Page.xml is "folder1/Page.aspx"
Environment details (development & target environment)

PNP version: 2.2.104-nightly
OS: Windows 11
PSVersion: 7.3.8
PSEdition: Core

hi @reply-jens , thanks for opening the issue.
Could you please open it here instead: https://github.com/pnp/pnpframework/issues ?
PnP PowerShell consumes the provisioning engine, the actual logic resides in the above mentioned repo. Will be closing it here.

authors: gautamdsheth reply-jens


text:
Teams Documentation - Fixes

Type

[X] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
What is in this Pull Request ?
Several more Teams documentation fixes:

Fixes to denote id, mailnickname or display name can be used for team
Channel id or channel display name
Correcting and adding examples

Thanks

Great fixes!

Great fixes!

authors: LeonArmston erwinvanhunen


text:
po-page-dynamic-table: O modal está com problemas responsivos

Reprodução

https://stackblitz.com/edit/po-ui-1jnqyd?file=src/app/app.component.html

Passos para reproduzir:

Defina alguns campos para o filtro avançado;
Defina as opções opções para estes campos;
Durante a execução altere o tamanho da interface

Qual o comportamento atual?
O modal para seleção do filtro avançado esta com problema responsivo e quanto maior a tela maior é o problema e isso dificulta a visibilidade das opções.
Que comportamento você esperava ver?
O modal para seleção do filtro avançado deve ser responsivo e dificulte a visibilidade das opções.
Ambiente

PO UI: 6.11.0
Angular: 13,0,2
Browser(s):

[x] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari
[ ] IE 11
[ ] Outros

Operating System:

[x] Windows
[ ] macOS
[ ] Ubuntu
[ ] Outros

Seguem abaixo alguns prints com a simulação do problema:

Olá @alinelariguet, existe alguma previsão de correção a exibição dos "radio buttons"?

Ainda não temos @wsteixeira, houve reclamação dos usuários sobre esse comportamento?

olá, @alinelariguet,
Sim houve, não é critico, mas quanto maior for a tela pior fica a visualização e parte do rotulo vai desaparecendo.

Comportamento já existe hoje nas versões mais atuais do PO-UI. Conforme doc abaixo:
https://po-ui.io/documentation/po-page-dynamic-table

authors: alinelariguet renanarosario wsteixeira

repo: po-ui/po-angular


text:
Como fazer a requisição do tipo GET em uma page dynamic table passando os parâmetros necessários de token e header?

Estou utilizando uma PAGE DYNAMIC TABLE para carregar os dados que vem da minha api, no entanto para realizar o get dos dados, preciso passar não somente a URL mas também o token para validação. Como e onde fazer isso?

@whalyf In Angular, you can make something called an "interceptor." Think of it like a helpful tool that automatically adds a specific header (like a label) to every message your app sends out. Instead of adding that label to each message individually, you create this tool once, and it takes care of it for you every time. This makes things easier and keeps your code cleaner!

Thanks, i've implemented this way. Yeah it will be better to deal with api calls.

Thanks, i've implemented this way. Yeah it will be better to deal with api calls.

authors: abananyabera whalyf


text:
Color picker field

This was discussed in https://github.com/pocketbase/pocketbase/discussions/400
We coud add new field type color for easier validation and management of Hex/RGBA color codes.

yes goood idea §

Possible using a text field and a regex:
Hexcode: ^#(?:[0-9a-fA-F]{3}){1,2}$
Source: https://stackoverflow.com/questions/1636350/how-to-identify-a-given-string-is-hex-color-format#answer-1636354
Not sure about RGBA.

I love to see support for other color formats, particularly oklch colors. Support just came to chrome with 111 and safari has supported it for a while. I understand if you don't want to/can't implement it, but I personally would get a lot of use from this feature.
Here are some resources about oklch if you're interested

https://chriscoyier.net/2023/01/22/ok-oklch-👑/
https://blog.jim-nielsen.com/2023/ok-lch-im-convinced/
https://evilmartians.com/chronicles/oklch-a-color-picker-made-to-help-think-perceptively

I'm working on implementing a Color field in  Go codebase. Could you point me to the relevant folders or files where similar field types are defined? Any examples or best practices would be really helpful. Thanks!

authors: Jothsa Pratham16112002 brewhousedigital ganigeorgiev zarqizoubir

repo: pocketbase/pocketbase


text:
Feature. Core. Unlimited Limit (Till manual cancellation)

Background
User want's to execute scenario until manual cancellation.
There is no limit for it.

Create an ILimit implementation and creator for this scenario

The task is In Progress by

repo: podkolzzzin/QueryPressure


text:
Kubelet: DRA code cleanup

Fixed some of the numerous golangci-lint findings to make the code more readable.

@klueska @pohly Thank you for the review. I've updated the PR according to your comments. PTAL again.

Tests failed after including these changes and worked again after I removed them. I have not checked which particular changed caused this.
@bart0sh : please have a look and submit this anew.

repo: pohly/kubernetes


text:
Precision checking algorithm is very slow

Describe the bug
When snapshot testing images with precision lower than 1, if the snapshots don't exactly match the reference images, the tests take much longer to run.
We currently have 66 snapshot tests in a framework

Precision
Passing Tests
Failing Tests
Pixel-Imperfect Images
Total Time

1
57
9
36
38s

0.97
66
0
36
483s

Those 9 failing unit-tests are recording 4 images each to account for different device sizes, so those 36 images that are not pixel-perfect are responsible for a 12.7x increase in the total duration for running 66 unit tests. The recorded images are not very large, ranging from 81KB to 255KB.
To Reproduce
Set the precision to any value lower than 1 and modify the tested view so they're not pixel-perfect with the reference images anymore.
Expected behavior
Checking whether or not the snapshots are within the accepted precision should be faster. A 12.7x increase for just 36 images is a whole different order of magnitude.
Environment

swift-snapshot-testing version 1.9.0
Xcode 13.2.1
Swift 5.5
iOS 15.2

I believe the issue is just this for-loop, but I don't have a concrete suggestion for how to fix it. Maybe parallelize it with a multi-threaded queue?
https://github.com/pointfreeco/swift-snapshot-testing/blob/88f6e2c0afe04221fcfb1601a2ecaad83115a05f/Sources/SnapshotTesting/Snapshotting/UIImage.swift#L105-L108

FYI: There's a promising open PR (#571) that aims to address three issues involving that loop, including its surprisingly long execution time (when compiled without optimizations, as is default for "Debug" builds).

Another option would be to move the image diff analysis from the CPU to the GPU.
The package already includes functions to create diff images (which are part of the failure output), so throwing something like the CIAreaAverage Core Image filter on it would allow for very quick evaluation.
The problem solved by https://github.com/pointfreeco/swift-snapshot-testing/pull/571 could also be tackled by additionally running CIAreaMaximum to see whether any of the pixels is above a certain threshold.

The use of CoreImage filters in https://github.com/pointfreeco/swift-snapshot-testing/pull/628 improves the speed by 90-97%: going from ~1s to ~2ms per snapshot. This is accomplished using the CILabDeltaE filter, but could also be done with a CIColorAbsoluteDifference to calculate the Euclidean color distance instead of a perceptual color difference.

@ejensen I'm working on a few tests and the precision makes the tests super slow e.g a test takes 0.5s to complete but with 0.98 precision it takes 6s on the latest version.

@ejensen I'm working on a few tests and the precision makes the tests super slow e.g a test takes 0.5s to complete but with 0.98 precision it takes 6s on the latest version.
Edit: It's faster if I also add a preceptualPrecision of 0.98 with precision

The speedup added in #628 was only enabled when preceptualPrecision < 1. I have added another https://github.com/pointfreeco/swift-snapshot-testing/pull/664 that introduces a similar speedup when preceptualPrecision == 1

authors: Stengo ejensen fjcaetano gohanlon tahirmt

repo: pointfreeco/swift-snapshot-testing


text:
fix: enforce fallback responses

The fallback responses are the ones coming from our backup system. They come into play when the node selected from the network can't do it's job, and doesn't return any response. In our mission to be as fast as we can, we use our internal system to serve that request (doesn't generate revenue).
These responses were only enforced to a certain format, but the truth is that there are other several server errors from blockchain's node clients that indicate that the call wasn't effective. This PR adds some additional text enforcement to abstract these errors from users and let it fail without any specific blockchain's node message that could add confusion to end user.
Example error:
{"error":{"code":-32000,"message":"rpc error: code = Unavailable desc = connection error: desc = \"transport: Error while dialing dial tcp 127.0.0.1:9090: connect: connection refused\""},"id":732830328053361800,"jsonrpc":"2.0"}

Another fix was also introduced at this PR, since it's error-related. It's the fix to the problem outlined in #862.
The current approach is matching server errors, and user error is anything with -32k error and NOT a server error. Since server errors are fewer, it is easier to handle and node runners won't be penalized by false positives on EVM unknown user error messages.

Another fix was also introduced at this PR, since it's error-related. It's the fix to the problem outlined in #862.
The current approach is matching server errors, and user error is anything with -32k error and NOT a server error. Since server errors are fewer, it is easier to handle and node runners won't be penalized by false positives on EVM unknown user error messages.

repo: pokt-foundation/portal-api


text:
Add golangci-lint config files and GitHub Actions

Let's just add the configs for now and then improve one step at a time.
Supersedes #9

We agreed to merge this and have golangci-lint as an optional CI job in our heads until we fix it.

repo: polarsignals/arcticdb


text:
feat: enhance API Reference documentation and create / root endpoint

closes #4497
[server]

created root endpoint / with a helpful response and a direct link to the documentation for the user

[clients]

enhanced the documentation at the endpoint https://docs.polar.sh/api to improve clarity and provide the user with next steps
enhanced the feedback section to include links for GitHub issues and the discord server for easy success

hi @birkjernstrom, could you please take a look at this?

Very nice @naineel1209 – thank you for this 🙏🏼 Looks great at first glance; I'll test it locally before approving & merging.

repo: polarsource/polar


text:
fmriprep anat placement is unintuitive with multiple sessions

fmriprep places T1w files in different places depending on the number of T1w images across sessions. If there are multiple T1w images, fmriprep averages them and creates a top level "anat" folder to store the average. When only one session has a T1w image, no top-level folder is created.
(folder structure if ses-1 and ses-2 had a T1w image):

anat
ses-1

anat

affine_transform.txt

ses-2

anat

affine_transform.txt

(folder structure if only ses-1 had a T1w image):

ses-1

anat

ses-2

Considering that both sessions are referencing the same T1w image regardless, I think it'd make sense to move the T1w image to a top level anat folder in the second case.

Ah, this is going to be a function of the source filename. When there's only one, we don't bother manipulating it to find a "common" name. We should probably just always pop the session out.

authors: IanEisenberg effigies

repo: poldracklab/fmriprep


text:
Add Geocentric Solar Ecliptic (GSE) frame

🎯 Goal
Produce plots like this:

And this:
https://twitter.com/planet4589/status/910720207701897221
💡 Possible solutions
Perhaps it's similar to this one, or even exactly this one! http://docs.astropy.org/en/stable/api/astropy.coordinates.GeocentricTrueEcliptic.html

📋  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 🎉

So, where do I contribute the frame?

I have to contribute methods to convert one frame to this one, as done here:
https://github.com/astropy/astropy/blob/master/astropy/coordinates/builtin_frames/ecliptic_transforms.py

Check out https://github.com/poliastro/poliastro/blob/master/src/poliastro/frames.py

Closed in #562

repo: poliastro/poliastro


text:
Home work 7 (Red-black-tree)

Реализация красно-черного дерева (с левой красной связью).

Прошу прощения за прицепившиеся изменения  readme.

Принято полностью.

authors: KateMoreva dormidon


text:
fix typo from brick to break

I do not think you intended to use the word brick, and think that break was intended instead

Actually here the use is indeed intentional.
In electronic devices to „brick something“ means to make it non-functional, typically via an update.

I did not know that.

repo: polkadot-js/docs


text:
解绑中过多时的显示优化

Restake on STAKING stats panel #135
Kusama/Polkadot

https://miro.com/app/board/uXjVOIzN9iI=/?share_link_id=609857322191
新增unstaking页面，改善stats页面中unstaking的显示

authors: MarchAlice StrawberryFlavor

repo: polkawallet-io/app


text:
Clarify/update disconnect/connect procedure

This fails:
reachy = ReachySDK(host="localhost")
reachy.disconnect()
reachy = ReachySDK(host="localhost")

This works:
reachy = ReachySDK(host="localhost")
reachy.disconnect()
ReachySDK.clear()
reachy = ReachySDK(host="localhost")

This (I think) works:
reachy = ReachySDK(host="localhost")
reachy.disconnect()
reachy.connect()

I think this is OK, it just has to be clear for the user. Suggestions :

Give an example where several sdk clients are used
In the error message, suggest the use of "ReachySDK.clear()"

Should be fixed by the previous merged

authors: RemiFabre glannuzel


text:
Narrowing conversion inside { } is ill-formed

The way we initialize the color structs in the demo code is actually "ill-formed" in C++11.  For example:
(rgb_color){ 0, 255 - x, 0 };

The problem is that the middle field is a uint8_t, but the value we are trying to stick into it is actually an int because of integer promotion.  I found out because I tried to compile code like that in Atmel Studio, which is using a pretty modern version of the GCC C++ compiler and got warnings like this:
narrowing conversion of '(255 - ((int)x))' from 'int' to 'unsigned char' inside { } is ill-formed in C++11 [-Wnarrowing]

When the Arduino IDE starts using a modern compiler and people get this warning, we should introduce a macro or inline function like this:
#define color(r, g, b) ((rgb_color){ (uint8_t)(r), (uint8_t)(g), (uint8_t)(b) })

It is OK to have a narrowing happen in a normal assignment, like c = a + b, but if narrowing happens during an initializer they think that is special and deserves a warning.

Hello EveryOne
I want to On/Off and change the color if this Rgb Dtrip using Android tablet with Arduino uno and these all thin are connected via Usb Cable .i will wait your reply
Thanks & Regard
Munesh parmar

Hello Every One
I want to On/Off and change  color of  this Rgb strip using Android tablet with Arduino uno and these all think are connected via USB Cable.so can you give me a any suggestion i will wait your reply
Thanks & Regards
Munesh parmar

Hi, @MuneshThakur.
As @DavidEGrayson mentioned in his reply to you in #9, this is not the appropriate place to ask for this kind of help. Please stop posting asking for help here and try on our forum, where people other than the library developers are more likely to read it.
Ryan

This is fixed in version 4.2.0 with the new constructors for rgb_color.

authors: DavidEGrayson MuneshThakur ryantm

repo: pololu/pololu-led-strip-arduino


text:
specify separator by blocks

I would like to be able to set a a different separator by block. As far as I now sperator is global to a bar, or used in spcefici modules bspwm or i3.
I'm not sure it would be used by most users, but it would allow more customization of polybar.
Here is an idea on how the configuration could look like:
[bar/foo]
separator = |
modules-left-separator = ""
modules-center-separator = /

Maybe we could even add the ability to completely customize all separators:
modules-left = mpd <sep1> date
sep1 = --
sep1-margin = 3

Is this still being considered as a possible feature? It has been a few years since the author posted this issue. I am interested in having different separators for left and right modules.

authors: bricewge kociap patrick96

repo: polybar/polybar


text:
Unable to retrieve elevation data

Getting this error:
Unable to retrieve elevation data

Which leads to "unable to convert building"
Unsure if this is Bing rate limits – their docs suggest it shouldn't be happening, although it may be throttled due to high demand.
Is there any way to catch this kind of error more specifically?

Interestingly, this problem went away after I threw away the buildings I'd already converted and started over from scratch. Could be a coincidence.

so once I've re-started a conversion after an out-of-memory error I always see this error eventually – but one interesting thing I've found is that I'll start seeing things like this:
Building has already been converted: output/DEB_LOD2_UUID_8bf7884d-b01c-48da-a9a2-ebca7648c347.obj
Unable to convert building: BLDG_0003000f00040f85
Unable to retrieve elevation data
Building has already been converted: output/BLDG_0003000e009a3895.obj

So in this case, BLDG_0003000f00040f85 is a previously-existing building – and then these apparent false hits become more common, until every building is reported this way, until (as far as I can tell from my tests) the last existing building is reached – but then the errors continue, naming non-existing buildings, until an out-of-memory error.

Here's a typical log:
https://gist.github.com/meetar/629d8caa286b9d1ef3b0

So it turns out the whole log didn't paste in there, and I checked the original log file and found this:

One thing that's certain is that I need to implement some more robust checks and behaviour when the elevation API isn't accessible (whether because of rate-limiting or whatever). That should solve the original issue here, though it doesn't solve the problem of where to get semi-accurate elevation data from…
I'm not sure about the other errors yet – they might be as a result of the elevation stuff, or something else entirely. I'll make the elevation behaviour more robust first and we can see how that changes things.

1a406f1d3c102073103df4d0b160bae3b35cd1ca moves to a key-less Google elevation API so that should avoid some issues. Let me know if there are more elevation-related errors and I can dig further.

After some more tests it looks like it's still easy to start getting rate-limit messages. It seems that you can only do 10 requests per second with the Google API.
I'm going to need to have a think about how best to approach this, as if we only do 10 requests a second then it'll take over 1m30s for every 1,000 buildings. It's possible to batch the elevation requests for multiple buildings, though that'll require quite significant changes to the architecture to be able to match up the elevation results with the worker processes responsible for each elevation request.

I'm less sure about the rate-limiting now, especially since the errors seem to go away when I restart the conversion from scratch – so Bing might be fine once the "out of memory" problem is resolved.

Perhaps. Moving to the Google API for now makes sense anyway as it doesn't have a hard usage cap over a month like the Bing one (unless we shell out for an extortionate plan).
I'm going to have a think about where the memory issues are and see what I can do.

So I'm told our elevation API is being tested in dev – it's based on SRTM with some other sources mixed in, but I don't know where or how. I presume there's one to cover the +60 lat range, but I don't know anything about the data quality – it probably hasn't been filtered for building heights, so there's likely some noise in the system in urban areas. I'll want to try it anyway, it will be a very common use case.

This was also just brought to my attention, I have to assume it won't work for our purposes but here it is for reference: http://www.liedman.net/elevation-service/

Hmm, I forgot that SRTM data is affected by buildings. Still, it may be good enough for first version that's not reliant on Google or Bing. I imagine the original heights within the CityGML data is based on accurate LIDAR data so we may still get floating buildings.
All this effort does make me wonder why the ground height isn't included in the CityGML data, or why the relative height to ground level isn't included. I'll see if my CityGML friends have any suggestions.

Seems to be running more smoothly now – but I certainly ran into elevation rate limits, it converted 2500 and is skipping the rest, with "Elevation values not present in API response" errors.
But no queue or memory errors (so far), so: progress! Assuming it skips the rest I'll retry the conversion later today.

That's good news then! The elevation errors will persist until we either use the new internal API or work on a way to batch and rate-limit requests to Google / Bing (a lot harder to implement).

Okay, let's leave this open for now and I'll update when we're ready to test the new API.

So this is interesting, and looks relatively simple:

Im running an elevation lookup service using open terrain data(SRTM) and https://github.om/perliedman/elevation-service Script used for fetching DEMs is at https://gist.github.com/JesseCrocker/1275882b1a48756e19db i filled in missing areas with 3 arc second SRTM, which is not covered in that script.

The service currently runs on an AWS t2.micro instance with a 400gb ebs volume to store the data, and is capable of doing ~5k points/seconds, for around $30/month.

Feasibly we could do the same thing until the Routing team has their service up and running. cc @bcamper

Looks nice! I couldn't see any setup instructions so I assume there's some manual tweaking to get something like this up and running. Certainly an alternative option though!
I've pinged my CityGML contacts to see if I can get a better understanding on how height is used in CityGML and whether I've mis-understood something – it's entirely possible the data we need is hidden in the (hugely verbose) CityGML files somewhere and I just don't know what I'm looking for.

So it looks like elevation values in CityGML aren't expected to be relative to the ground and there doesn't seem to be any elements or properties to surface that data within the CityGML either. Saying that, there are a couple of leads that may prove useful for us.

The boundedBy element for each building should include the lowest point for the building, however this may actually be below ground level (eg. the bottom of a basement)
The measuredHeight attribute (if it exists) could be used to calculate the building offset from ground-level (eg. maxZ - minZ - measuredHeight == 0)

All in all, the guidance and approach recommended is to poll an external elevation API for each building. This is what we're already doing and it's what they CityGML guys are doing too – not practical but we don't have much choice it seems, at least for now.

I've run some initial tests with the dev elevation API and it's certainly better from a rate-limiting standpoint. However, from the buildings I've checked there can be a difference in height between the Google Elevation API of up to 10 metres (eg. run this Google result through the dev API) – possibly more (though some are ok – within a few metres). I imagine this is due to the low pixel density, or noise created by buildings / cities in SRTM data.
Overall, this means that buildings could be offset up to +/- 10 metres more than they should, resulting in buildings embedded into or floating above the ground – which is bad considering some buildings may be shorter than the margin of error (ie. they make themselves new lives as underground bunkers). It really comes down to what kind of accuracy we want, which I would argue needs to be high considering building elevation is incredibly important in defining how real a city looks.
One potential approach would be to forgo rough elevation APIs entirely and work out the lowest point that would leave no gaps between the building and ground plane – a more advanced approach to simply snapping the lowest building point to the ground plane. It's entirely possible that the margin for error here is significantly lower than that of the various elevation APIs, potentially a couple metres or so with the average being less than a metre – due to the fact that the building vertical-axis values are derived from LIDAR and as such only accounts for what it can see above-ground anyway.
Either way, I'd be interested to run some tests here for comparison.

I've managed to find Berln's 2 and 5-metre accuracy terrain data (which is open licensed!) so I'm going to use that as part of the comparison process to compare the APIs / approaches.

Ok so I've done a quick comparison using the dev API (SRTM), Google API and the lidar data from Berlin.
Assuming the following point (sprawling but relative short building) @ 52.53444783854353, 13.37425159625227:

Lidar: 33.5m
Google: 34.3m
Dev: 39m
Lowest point recorded by CityGML (not necessarily the ground): 31m

And another point (large road intersection) @ 52.52363, 13.35689:

Lidar: 34.4m
Google: 35m
Dev: 35m

And another:

Lidar: 36.5m
Google: 37.7m
Dev: 39m

All in all, the results all differ which is expected. I'm still none-the-wiser about which one would be most accurate, considering that for one building the lowest point is 31m and yet none of the elevation APIs return values below 33.5m. The highest point of the building should be correct though (as it's lidar), so my gut says that highest point minus lidar terrain value equals origin point for ground level.

I was afraid of that – in that case I agree, working from the data we already have is more likely to result in "good-enough" data. Maybe we can work in some kind of weirdness-detection to look for cases where there are numerous z-values near the calculated ground plane.

On Aug 3, 2015, at 5:15 AM, Robin Hawkes notifications@github.com wrote:
I've run some initial tests with the dev elevation API and it's certainly better from a rate-limiting standpoint. However, from the buildings I've checked there can be a difference in height between the Google Elevation API of up to 10 metres (eg. run this Google result through the dev API) – possibly more (though some are ok – within a few metres). I imagine this is due to the low pixel density, or noise created by buildings / cities in SRTM data.
Overall, this means that buildings could be offset up to +/- 10 metres more than they should, resulting in buildings embedded into or floating above the ground – which is bad considering some buildings may be shorter than the margin of error (ie. they make themselves new lives as underground bunkers). It really comes down to what kind of accuracy we want, which I would argue needs to be high considering building elevation is incredibly important in defining how real a city looks.
One potential approach would be to forgo rough elevation APIs entirely and work out the lowest point that would leave no gaps between the building and ground plane – a more advanced approach to simply snapping the lowest building point to the ground plane. It's entirely possible that the margin for error here is significantly lower than that of the various elevation APIs, potentially a couple metres or so with the average being less than a metre – due to the fact that the building vertical-axis values are derived from LIDAR and as such only accounts for what it can see above-ground anyway.
Either way, I'd be interested to run some tests here for comparison.
—
Reply to this email directly or view it on GitHub.

I'll have a think about what an approach for that might look like and compare the results. I think that edge-cases might make it difficult (eg. a building with legs like the Eiffel Tower) but we'll see how it goes.
On the plus side, the dev API is great for fairly accurate (sparse) terrain and this general problem is only applicable when rendering onto a flat plane like in ViziCities or Tangram. If we rendered onto a 3D terrain then everything should look ok with the original elevation values in the CityGML.

c327dfab87458f848b7b48383c853cbbc4355864 (0.1.24) experiments with using the highest groundSurface element (if exists) as a crude way of finding the best guess of ground elevation. This isn't perfect as groundSurface can be used to define any part of the building that touches the ground (eg. the bottom of a basement), though we may not notice anything too abnormal if the CityGML data is derived from above-ground lidar (ie. no subterranean groundSurface elements can be seen).
Seeing as this is a general library, it will probably be a good idea to enable these various elevation approaches as a user-definable option – some people will be happy with using the lowest point, some with using the highest groundSurface, others will want to use an external API like Google.

Still seeing a discrepancy between the citygml building count and the converted building count – checking the conversion log I see that the converter is still occasionally returning things like:
Unable to convert building: UUID_060cce13-df69-4ecc-9bc9-836a9efbfa82
[Error: Unable to retrieve elevation data: getaddrinfo ENOTFOUND valhalla-elevation-dev-1428989121.us-east-1.elb.amazonaws.com valhalla-elevation-dev-1428989121.us-east-1.elb.amazonaws.com:80]

It could be a rate limit, but these errors are quite rare (23 out of 19k)... regardless, the converter never queues those buildings back up, and just fails on them.

Do we still need Valhalla (or any elevation service) if we're using groundSurface?
Can we requeue the failed buildings?

The conversion process should only be requesting elevation data from Valhalla if it can't work out a sane height from the CityGML data. The fact that this isn't happening is definitely odd as it implies that the building doesn't have a bldg:GroundSurface CityGML element, which would make it invalid.
However, the fallback is for exactly this situation and should work. I'll do the following things:

Update the Valhalla endpoint to whatever it is now
Look into why building UUID_060cce13-df69-4ecc-9bc9-836a9efbfa82 doesn't report a ground surface in CityGML

I've pushed 0.1.27 which updates the Valhalla endpoint and will now work as a fallback for buildings without a bldg:GroundSurface. You'll need to add the valhallaKey option when starting up citygml-to-obj (it's in the readme).

After digging through the Mitte set I found building UUID_060cce13-df69-4ecc-9bc9-836a9efbfa82 and it definitely doesn't have a bldg:GroundSurface, nor any bldg:WallSurface elements – in short, this means we can't easily work out what's the ground and what's a roof or a wall so there isn't much we can do, though it should still output a valid 3D object.

authors: meetar robhawkes

repo: polygon-city/citygml-to-obj


text:
上传覆盖

使用 upx put file 不会覆盖之前的文件吗，通过 upx put --help 也未看到详细的参数详解

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

put 会覆盖之前的文件的。CDN 刷新可能需要一定的时间。您可以 ls 看下文件的修改时间或者大小这些信息有没有改变

不至于回复我这么多遍吧
我就是通过ls看了，时间没有改变，然后就删除了之前了，再重新put就可以了，所以我想知道是否不需要我删除，直接覆盖

不好意思，刚才家里网太烂了，回复了多次。put 是直接覆盖的。如果还出现这个问题，可以提供一些具体信息，我这边排查下

既然是覆盖的，重新put后，ls 查看到的文件列表，时间没有变化呢

对，这个问题需要具体分析下，理论上不可能出现这样的问题

暂时先关闭了

authors: cismous polym

repo: polym/upx


text:
add mouseovers for our 'contacts' rellated icons

I think it would be useful if these icons  each had a mouse-over popup describing what each thing is:
Here are some suggested text for editing:

pombelist. The fission yeast community (and friends) mailing list for any discussions, announcements or questions related to fission yeast.
pombeSlack  Many discussion channels including conferences, talks, technical ????
PomBase twitter feed -announcements, anything we spot related to fission yeast and some fun stuff.
PomBase facebook (we don't use this much)
PomBase GitHub, all our project tasks organised by curation, FYPO, website, Canto (curation tool), Chado(database) feel free to browse, join, suggest....
PomBase Linked in (we don't use this much)

They all do have mouseovers with very brief text descriptions (often just the name spelled out).
I'm honestly not convinced most of them need any more than that. Anyone who doesn't already know what Twitter is won't click the icon; those who do will want to see for themselves what's there, and will likely know to expect a Twitter-ish grab bag. I suspect the same applies to Facebook and maybe also LinkedIn.
Specific suggestions:

icon
mouseover text
my comment

pombelist
Fission yeast community mailing list (pombelist)
links to the info page so I don't think we need more in a mouseover

Slack
pombeSlack - Community discussion channels
not sure how much more to say; don't want any of these to be wordy

Twitter
@PomBase on Twitter
No change

Facebook
@PomBase on Facebook
No change

GitHub
PomBase code on GitHub - Canto, Chado, website, etc.
could use different examples, but don't want to laundry-list all the repos

LinkedIn
PomBase on LinkedIn
No change

Some of the rest would probably be better handled by adding some FAQ entries, and maybe a new FAQ category for "community" or something to that effect. (I'm thinking of things like how to join pombeSlack or switching digest mode off for pombelist ...)

OK. I never saw the mouse overs. I see them now.

repo: pombase/website


text:
Improve display of histone modifications

Histone alles and modifications are universally described using the processed (- initMet) for the histones.
For alleles we do this:
hht3-K56R(K57R aa)
The name follows the community usage, and the description matches the underlying protein sequence. We use the underlying protein sequence to check that the alleles and modifications conform with the sequence in our QC pipeline without adding a special case for histones.
At the moment we Display modifications using only the preprocessed form which will be very confusing for end users. We would like to change the display so that the universal nomenclature is presented with the processed form in parentheses
K14(K15) processed/preprocessed.
I will add a list of the histones that this applies to.

Applies to

Gene systematic ID
SPAC1834.03c
SPBC1105.12
SPBC8D2.03c
SPAC1834.04
SPBC8D2.04
SPBC1105.11c.1
SPAC513.01c
SPCP31B10.07
SPCC622.08c
SPAC19G12.06c
SPCC622.09

Applies to

Do these have a something like a GO term in common so we don't need to maintain a gene list in the configuration file?

extended the "applies to" list with Jo's additions

Do these have a something like a GO term in common so we don't need to maintain a gene list in the configuration file?

https://www.pombase.org/results/from/id/593d5a9e-c876-467e-a531-336791ef7e8b
Unfortunately not -these are described variously as adapters or structural molecules. There is not even a protein family that is specific for the group because this is only a subset of all histone faults.

Could we add some sort of /controlled_curation to annotate these genes?  That way the annotation will end up in Chado and the web code will be identify the genes.

Yes, I can add
/controlled_curation="histone"
or does it also need to have a "type"?

or does it also need to have a "type"?

It's been so long, I can't remember.  I'll refresh my memory and let you know.
But now I think about it, I wonder if /SO=SO:0000418 would be better?

this is for signa peptide, not modified histone?
but PRO might have a grouping, will check

nope PRO does not have a grouping (it would be a bit odd anyway)

this is for signa peptide, not modified histone?

Sorry!  Please ignore me.  :-)  I was think about this is at the same time: pombase/website#2115

I thought you were. They are both removal of N-terminal regions, but a bit different ;)

I could find, or request the PRO_ID for each one and add
controlled_curation=display(PRO:xxx)

Should I do it like this?

/controlled_curation="histone

I think something like this:
/controlled_curation="term=warning, histone"

K14(K15) processed/preprocessed.

Where does that need to be shown on the web site?  Sorry, I don't understand histones.

I could find, or request the PRO_ID for each one and add
controlled_curation=display(PRO:xxx)

I'm not sure about that but I don't really understand what's needed.

Don't worry, we can discuss in person, it's easier.

Action for kmr: make a Chado check that we always have 11 histones - 11 annotations to GO:0000786 nucleosome

How's this?:
https://desktop.kmr.nz/gene/SPAC1834.04

Good, but I think it needs to be visible in the full view as well as  show details.

Good, but I think it needs to be visible in the full view as well as show details.

It will be.  That's what the two images show.  The code isn't finished yet so my desktop version is a bit dodgy.  I took the screenshot while it was briefly doing the right thing for hht1.  :-)

Got it!

Hi Val.
On the documentation page for the modifications section (https://www.pombase.org/documentation/gene-page-modifications) it says:
Note: for histones, residue numbering assumes that the initiator methionine is removed.

Is that correct?  If so, I think I'm misunderstanding things.

I think that is because the documentation predates the manus script,and that previously  the instructions were to represent histone modifications using the histone code standard.
I'm not sure whether  everybody adhered to that guidline, because presumably they would need to manually edit their mass spec output to make the histones match (unless that mass spec processing does this automatically, this seems unlikely because even uniprot reports the modifications on the unprocessed version https://www.uniprot.org/uniprotkb/P09988/entry#ptm_processing).
It might be useful to know how many manus script  'fixed' to the non-modified form @manu do you know this?
But I think we  probably  just need to update the modification now to say that histones should be reported in the unprocessed form.
Basically, we changed the way we do this to make it the same for every protein, but weren't aware of the existing instructions for histones.

We will also need to correct
https://www.pombase.org/
The Residue column indicates the position modified. For protein modifications, use one-letter amino acid code. Multiple entries are allowed, but only for cases where two or more of the same modification are known to be present at the same time. Separate entries with commas (e.g. S72,T85). Position numbering should reflect the current sequence data in PomBase. Please refer to the Gene Coordinate Changes page to ensure that your residue position entries are up to date. Also note that histones are conventionally numbered assuming the initiator methionine is removed (i.e. every position in the mature protein is numbered, and is 1 less than the apparent numbering predicted by translating the ORF).

For protein modifications, use one-letter amino acid code. Multiple entries are allowed, but only for cases where two or more of the same modification are known to be present at the same time. Separate entries with commas (e.g. S72,T85)

this section isn't correct, we don't  conjoin because this wi make modifications impossible to collapse and it isn't sure informative since  it's biased for close residues which are like to be on the same peptide fragment.
Also we report the phase when known so we can get modifications that co-occur this way.
2.- we can tell people this isn't necessary, and why
Please refer to the Gene Coordinate Changes page to ensure that your residue position entries are up to date.
I can rewrite a shorter version tomorrow.

The residues are displayed correctly for histones now.  I had some bugs to fix when displaying on pages that have a mixture of histones and non-histones like:
https://www.pombase.org/term/MOD:00408

That's now fixed but let me know if you see any problems.
We talked about being more explicit in the detailed display (eg."K56(K57)` processed(preprocessed)").  Should we do that?  Here's how it looks in the test version on my desktop:

We talked about being more explicit in the detailed display (eg."K56(K57)` processed(preprocessed)"). Should we do that? Here's how it looks in the test version on my desktop:

It doesn't hurt, but I  think most biologists will understand without.  Put it in for now as  there is space...

OK, I'll added that.  The main site will have the change in a little while.

Add text at the top in the "Notes" section.
We have a QC pipeline in place to check that the modified residues match the current protein sequence coordinates. For most proteins where the protein sequence coordinates have changed, we will be abo to automatically "lift over" to the current sequence residue numbering.
Change text
Also note that histones are conventionally numbered assuming the initiator methionine is removed (i.e. every position in the mature protein is numbered, and is 1 less than the apparent numbering predicted by translating the ORF).
to
Histones should be represented using the unprocessed protein sequence coordinates, not the processed coordinates conventionally used to describe histones. Histone modifications will be  represented on the gene pages as
K4(K5) processed(preprocessed), but our checking pipeline will expect unmodified forms.

Edited text above.
@kimrutherford can you make this change and ten this ticket can close

can you make this change and ten this ticket can close

Excellent, thanks.  I've made that text change.

authors: ValWood kimrutherford


text:
Change the GO section filter from  Throughput to Evidence type and expand options

Change the GO section filter from Throughput to Evidence type and expand options
This would allow people to filter inferences.

We already have the filter for non-experimental under  the full-view:

but  for some reason we didn't include "experimental" as an option

I think adding "Experimental" could be confusing.  The three current menu items are mutually exclusive.  With "Experimental" added there will be overlap between the options.

add the following evidence groups

Those are the three groups we have already.  Do you mean rename rather than add?

Here is the current configuration for evidence/throughput types.  This config is how GO annotation gets assigned a throughput type on loading.
IRD, NR and RCA don't have a configured type but we don't have any of those annotations.

type
codes

high throughput
HDA HEP HGI HMP HTP

low throughput
EXP IDA IGI IMP IPI

non-experimental
IBA IBD IC IEA IEP IGC IKR ISA ISM ISO ISS NAS TAS

unset
IRD ND NR RCA

I've implemented this on my desktop: https://desktop.kmr.nz/gene/SPAC1565.08

That's great.
I think the labels should be
Low  => Experimental (LTP)
High => Experimental (HTP)
All experimental => Experimental (All)
and Inferred
If we think everyone won't know what HTP/LTP is then we need to put this in full. The labels would be quite long but I think that is OK?
@PCarme thoughts?

Hi.
I've rudely ignored your suggestions and changed the labels to:
- High throughput experiment
- Low throughput experiment
- Inferred
- All experimental annotation
(Sorry I got the order wrong - I'll fix that)
I've done this to see if they are too long.  What do you think?:

Looks good!

I think so


text:
browser track for Lantermann UTR/ncRNA dataset

Data in spreadsheet attached to Jira PB-2623. Kim may have converted it to GFF at some point already; if not, that's to do.
Datatype: "Curated transcripts"
Method: Manually curated from PMID: 18641648 transcriptome data-
Strain: h90 WT
PMID:20118936

Do we still have access to this file ?

moved to
https://github.com/pombase/website/issues/1441

It's GSE16040 in GEO

I'm looking for the curated transcripts file that was converted to GFF?

I'm looking for the curated transcripts file that was converted to GFF?

We have some files that used to be in Dropbox and are now archived on the server.  Do these look right?:
Lantermann_dutrow_utrs_chr1.embl
Lanterman_utrs_chr2.embl
Lanterman_utrs_chr3.embl
original_1.embl
original_2.embl
original_3.embl
utrs-Chr1.embl
utrs-Chr2.embl
utrs-Chr3.embl
I don't remember converting these to GFF and I can't find any corresponding GFF files.
/data/pombase/archive_from_dropbox/Data/Broad/utrs/

I have a feeling these might be the files we used to do
https://www.pombase.org/faq/how-do-you-determine-gene-s-full-length-transcript-utr-coordinates-transcription-start-and-end-sites
I will try to figure out the difference between the 3 sets in Artemis....
I will also check the format in the paper.
This isn't the broad dataset though (as the directory label states) (maybe some of the embl files are the broad and  we are only interested here in the ones labelled "Lanterman"
I now remember that we had this data somewhere in a local directory , but I can't remember where that was...

It might have been under svn,  or somewhere on the old Oliver 0. This also might have had the scripts for the file priorities so it might be useful to locate...

I have been looking for this. @kimrutherford  could you have a look at some point. I think this dataset would be useful to align with our curation.

All I can find the files mentioned in this comment: https://github.com/pombase/website/issues/62#issuecomment-671910129=
Do you have any more clues you can give me?

Isn’t there a dismiss of everything that was in jira? Maybe search it for pb-2623?

I  haven't had access to Jira since Po-xit. I'll go back to the paper. There are a few papers which mention revised gen annotation sets that I need to dig out and read.
@kim I will get onto this after the 9th. Assigned to me for now.

Isn’t there a download of everything that was in jira? Maybe search it for pb-2623?

Thanks Antonia!  You're right.  I had forgotten that we downloaded a copy of the Jira tickets.  I've put a copy in Dropbox/pombase/temp/ebi_jira_tickets_backup
They are HTML files.  They render OK if you open them in Firefox.  These files mention "Lanterman":
jira-2623.html
jira-2624.html
jira-2003.html
jira-1353.html
jira-1088.html
These files mention "Dutrow":
jira-2624.html
jira-1353.html
jira-1088.html
jira-1018.html
I've had a look but I can't see any filenames that would help.

2623 mentions a file called
nsmb.1741-S2.xls
"
This is the other set of manually curated features.(will upload)Its a
apreadsheet so it will need converting to gff. I'll assign to Kim first
because we used this dataset in the current UTRS (with lower precedence
than the Rhind data). I wonder if Kim already did the conversion somewhere.
I looked in the UTR files in the drop box, and I could only find EMBL
files.
It will appear under "transciptome"
Datatype- "Curated transcripts"Method-Manually curated from PMID: 18641648
transcriptome data-Strain h90 WTcitation PMID: 20118936"
On Thu, Mar 31, 2022 at 10:22 AM Kim Rutherford @.***>
wrote:

Isn’t there a download of everything that was in jira? Maybe search it for
pb-2623?
Thanks Antonia! You're right. I had forgotten that we downloaded a copy of
the Jira tickets. I've put a copy in
Dropbox/pombase/temp/ebi_jira_tickets_backup
They are HTML files. They render OK if you open them in Firefox. These
files mention "Lanterman":
jira-2623.html
jira-2624.html
jira-2003.html
jira-1353.html
jira-1088.html
These files mention "Dutrow":
jira-2624.html
jira-1353.html
jira-1088.html
jira-1018.html
I've had a look but I can't see any filenames that would help.
—
Reply to this email directly, view it on GitHub
https://github.com/pombase/website/issues/62#issuecomment-1084315273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBDJUVPEBZ45WOPNY6TEM3VCVVEXANCNFSM4CNCMWAA
.
You are receiving this because you were assigned.Message ID:
@.***>

you might be able to request your username, and then password, using your
email if required..
https://www.ebi.ac.uk/panda/jira/browse/PB-2623
I can log in but not see this issue - I think because my pombase account
got merged with my new ebi account when I started here & I lost
permissions.
On Thu, Mar 31, 2022 at 10:31 AM Antonia Lock @.***>
wrote:

2623 mentions a file called
nsmb.1741-S2.xls
"
This is the other set of manually curated features.(will upload)Its a
apreadsheet so it will need converting to gff. I'll assign to Kim first
because we used this dataset in the current UTRS (with lower precedence
than the Rhind data). I wonder if Kim already did the conversion somewhere.
I looked in the UTR files in the drop box, and I could only find EMBL
files.
It will appear under "transciptome"
Datatype- "Curated transcripts"Method-Manually curated from PMID:
18641648 transcriptome data-Strain h90 WTcitation PMID: 20118936"
On Thu, Mar 31, 2022 at 10:22 AM Kim Rutherford @.***>
wrote:

Isn’t there a download of everything that was in jira? Maybe search it
for pb-2623?
Thanks Antonia! You're right. I had forgotten that we downloaded a copy
of the Jira tickets. I've put a copy in
Dropbox/pombase/temp/ebi_jira_tickets_backup
They are HTML files. They render OK if you open them in Firefox. These
files mention "Lanterman":
jira-2623.html
jira-2624.html
jira-2003.html
jira-1353.html
jira-1088.html
These files mention "Dutrow":
jira-2624.html
jira-1353.html
jira-1088.html
jira-1018.html
I've had a look but I can't see any filenames that would help.
—
Reply to this email directly, view it on GitHub
https://github.com/pombase/website/issues/62#issuecomment-1084315273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBDJUVPEBZ45WOPNY6TEM3VCVVEXANCNFSM4CNCMWAA
.
You are receiving this because you were assigned.Message ID:
@.***>

Thanks again Antonia.  You're good at this.  :-)

Sorry, hit return too quick

2623 mentions a file called
nsmb.1741-S2.xls

I think that must be S2 of: https://www.nature.com/articles/nsmb.1741

I woud quite like to be abe to view this dataset in the New Year for the genome update paper. I'd like to compare to the curated transcripts

OK, let's chat about this soon.  I have questions about the dataset.

Hi @PCarme
Let's have a look at this dataset in January.  This has been planned for a long time.  :-)

yep, I have suggested opening a new ticket, one or each dataset, using a template.

authors: Antonialock ValWood kimrutherford mah11


text:
Fix tooltip position

Problem
When use this plugin on sub-window, tooltip will shown with wrong contents(too big image) on wrong position.

screenshot

Solution
import poi/views/components/etc/overlay instead direct @blueprintjs/core
(same as poi-plugin-ezexped/ui/fleet-picker/fleet-button)

screenshot

Note
TP value tooltip is not well tested (changes on a863467). Because transport operation is closed currently.

LGTM

authors: KochiyaOcean yshkflyer

repo: poooi/plugin-prophet


text:
Maproute

A route represents a line connecting 2 spots
These data are based on map images
format: mapspot[:MapArea:][:MapCell:][:RouteNo:] = [:Spot1:, :Spot2:]
for duplicated spots, only one is chosen
Special cases:

map 2-3, point E to point I is not possible, vice vesa, though it is drawn on the image
map 6-3. the reconnaissance point is not included

Example:
Map 6-4:

mapspots including duplicated ones:

maproute:

@KagamiChan excellent

authors: KagamiChan yukixz


text:
Cache the monospace fallbacks buffer in FontSystem

See #308

Needs a rebase after merge of #307

repo: pop-os/cosmic-text


text:
There's a hide method but not a open method

Feature description

A open method to complete with the current existent hide();
Why should this feature be part of the Popper's core?
If someone wants to keep the element reference and open by calling a method than a button.

It should be more easier and would be nice to have.

Popper.js doesn’t provide any hide() method

authors: FezVrasta ianguuima

repo: popperjs/popper-core


text:
OpenAPI Optional types

Hack the openapi generator for Pydantic to allow support for nullable fields in the openapi spec. Ignore one case (nullable schema) to avoid hitting an openapigenerator 5.x.x bug (we're pinned to 5.x.x).

Codecov Report
Base: 68.42% // Head: 68.11% // Decreases project coverage by -0.30% :warning:

Coverage data is based on head (060e712) compared to base (edf159c).
Patch coverage: 10.34% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #294      +/-   ##
==========================================
- Coverage   68.42%   68.11%   -0.31%
==========================================
  Files          71       71
  Lines        5437     5466      +29
==========================================
+ Hits         3720     3723       +3
- Misses       1717     1743      +26

Impacted Files
Coverage Δ

api/utils/openapi.py
24.00% <10.34%> (-18.86%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I've validated this better, and it does work only with openapi-generator 5.3.0 - it does NOT work in 6.2.0 as python-prior, I keep seeing external_id: str | None type turn into external_id: (ExternalId,) with no definition for ExternalId. I can't track this down in their generator code, and not worth trying to upgrade at this stage.
It does have one limitation that function parameters don't work well to be optional, just set it to a sensible non nullable default. I'm following up in another PR to add more types to endpoints.

authors: codecov-commenter illusional

repo: populationgenomics/sample-metadata


text:
Removed support for k8s 1.13 and below and added support for 1.22 and above

I tried to install portainer on k8s version 1.22.5 and noticed it fail as the ingress apiVersion is now incorrect and it's been deprecated. This PR fixes this by using networking.k8s.io/v1 for v1.19 and above (where it was introduced), and networking.k8s.io/v1beta for anything below v1.19.

@rupertbenbrook
Thank you for the PR. This is being added as part of a larger change. I'll close this in favour of https://github.com/portainer/k8s/pull/91

Thanks @samdulam, that makes sense.

authors: rupertbenbrook samdulam


text:
Support persisting portainer data into external database

In case we can recover from machine crash safely without data losing.

I believe that this is related to https://github.com/portainer/portainer/issues/523
Closing as might be a duplicate and also, you can already persist the data outside of the Docker host where Portainer is running, by using a network shared filesystem for example.

repo: portainer/portainer

license: Zlib


text:
The endpoint-init view is cached when redeploying an instance

Description
When redeploying Portainer (fresh database) over the same IP:PORT or domain without logging out from the previous instance, the user will go through the admin-init view and then be redirected directly to the home view without going through the endpoint init view.
Steps to reproduce the issue:

Deploy a fresh instance of Portainer and configure it (admin-init, endpoint-init)
Without logging out of the instance, re-deploy a new instance (with fresh database) on the same IP:PORT
Access the instance, you are being redirected to the admin-init view
After configuring the admin credentials, you are redirected to the home view

Technical details:

Portainer version: 2.0-CE

Closing this one as we're not able to 100% reproduce it.

I don't get this 100% but I have seen it probably 90% of the time. Same steps as above, except I don't use a volume in my command and i use windows with chrome

authors: deviantony itsconquest


text:
Business Edition - Custom RBAC roles

Is your feature request related to a problem? Please describe.
Currently the RBAC roles are limited to the build in roles: https://docs.portainer.io/admin/users/roles#built-in-roles
This works as follows:

Describe the solution you'd like
Allow administrators to define custom roles.
Adding or editing a custom role:

To add a new role for Docker:

The “All resources in an endpoint” vs “Resources assigned to an individual/team“ determines if the role is for all resources in an environment, or only the ones that are explicitly assigned to a Portainer user.
There is a different tab for Docker, Kubernetes, ACI etc.
The different levels portrayed by the radio buttons are accumulative. So Operate implies both the Read and Operate permissions. And Create includes Read, Operate and Update permissions.
The default selection for the radio buttons is Read.
The quick select “Set all below“ sets all radio buttons below it to what the user selected in the button group.
If a certain group of radio buttons doesn’t have the same level, we pick the one with less access. E.g. if the user sets all to Operator, we’ll select Read for the docker secret.
Kubernetes example:

ACI example:

How about https://github.com/portainer/portainer/issues/5402 for Kubernetes Custom role?

I upvote this.
Yesterday I had my first contact with the roles in portainer and it took not many minutes, that I was looking for adding own custom roles, because the factory did not satisfy. Because I did not find a way to do that, I was starting googling and found this smart request.
I would add, in the main-roles-page it shoult be possible to add, delete, edit and clone a role. I don't know, if following would make sense, but instead of radio buttons to use checkbox to set up the permissions.
BTW. In my yesterdays test drive with roles I was able to force delete other users image of a stopped container as a standard user. I think that should not have been possible.

I was able to force delete other users image of a stopped container as a standard user

That will actually be fixed in the 2.12.0 release.
The release notes for that are:

Standard users will no longer be able to remove or export images. Also, Operators, Help Desk, and ready only users will no longer be able to export images.

+1 from me here. I need to give users the ability to star/stop/restart their own container and nothing more.

Another upvote from me. I just noticed that a standard user can remove a container but not recreate it, at least the button is missing in users dashboard. From my point of view, that makes no sense at all, it should be the other way around (if there should be a meaningful restriction), that the user may recreate a new container and pull the latest image but not remove the container.

count with plus 1 vote, please

Are there any updates on this?

+1 from me here. I need to give users the ability to star[t]/stop/restart their own container and nothing more.

+1. Even a new built-in role that covers exactly this case would be sufficient.

+1

authors: BenHippynet CM2Walki bduff-walleye geigervibe huib-portainer mcc-mcannizzaro samdulam wil-m


text:
fix(networks): review how networks are loaded for usage in multiple v…

…iews
Reviews how the networks are loaded in different views & using different providers:

Provider Docker Swarm mode:

templates: local networks + swarm attachable networks (API version > 1.25)
container-creation: local networks + swarm attachable networks (API version > 1.25)
container-details (join list): local networks + swarm attachable networks (API version > 1.25)
service-creation: all swarm networks

Provider Docker Swarm standalone:

templates: global networks
container-creation: global networks
container-details (join list): global networks

Provider Docker standalone:

templates: local networks
container-creation: local networks
container-details (join list): local networks
Fix #1103

An image is available for this implementation: portainer/portainer:pr1104 (linux amd64 build)


text:
2.5.1

@rultor release tag is 2.5.1

@rultor release tag is 2.5.1

@portlek OK, I will release it now. Please check the progress here

@rultor release tag is 2.5.1

@portlek Done! FYI, the full log is here (took me 8min)

authors: portlek rultor

repo: portlek/SmartInventory


text:
If I want to hide the "comments count",  what should I change?

You could do it just with CSS. Inspect the code, extract the right CSS selector and set a display:none in your CSS

@blatinier the  embed.js?  I cant find which line control the "comments count"

authors: blatinier goxofy

repo: posativ/isso


text:
Issue for non-admin users to change their email address

Describe the bug
When a non-admin user changes their email, the settings page gets "stuck". The email address modification works as it should, but the screen doesn't refresh.
Following error is being generated in the logs:
[10028] [2024-01-12T10:42:13.410] INFO -- : Completed 500 Internal Server Error in 248ms (ActiveRecord: 7.0ms | Allocations: 1888)
[10028] [2024-01-12T10:42:13.414] FATAL -- :
NoMethodError (undefined method `verify_path' for #<UserController:0x000000000162d8>):

app/controllers/user_controller.rb:72:in `update'
lib/postal/tracking_middleware.rb:12:in `call'

Indeed, verify_path doesn't seem to be defined anywhere in the code.
To Reproduce

Login as non-admin
Go to My Settings
Enter your password, and replace the email address.

Expected behaviour
An email verifying the new email address should be sent I guess? Or at least a refreshed screen.
Environment details
version 2.1.4

This will be resolved in Postal 3.0.0.

authors: adamcooke olegbliaher

repo: postalserver/postal


text:
Fix/requeuer time

Please see #1789 for information on why.

whups.


text:
Add CommandKit::Options::VerboseLevel

Add a module which defines a -v,--verbose option that can be specified multiple times to increase the verbosity level.

Implemented by 27589fb.


text:
Python 3 Compatible?

It looks like cocotb is NOT compatible with Python 3 yet?

RefactoringTool: Warnings/messages while refactoring:
RefactoringTool: ### In file ./cocotb/generators/bit.py ###
RefactoringTool: Line 67: You should use 'operator.mul(1)' here.
RefactoringTool: There was 1 error:
RefactoringTool: Can't parse ./tests/test_cases/test_exit_error/test_exit.py: ParseError: bad input: type=1, value='a', context=('', (14, 20))

Cheers
Pei

Hi Pei,
I'm not exactly sure what the issue is, but I run it with Python 3.5. And the Travis CI build tests for python3 support.
Perhaps there is a bug in your RefactoringTool software?

Cocotb is compatible at least with Python 3.4 and 3.5 (on Linux)

repo: potentialventures/cocotb


text:
Ci test 2

Another test of #928.
This is re-triggering CI of #917 to try and pinpoint why the branch CI passed but the merge CI failed.

@eric-wieser you can also enable Travis on your repository.
Just login with gihub account end enable ;-)

@themperek: Sorry for the noise, I've gone ahead and done that now :)

authors: eric-wieser themperek


text:
🛑 PowBot SDN is down

In c364cea, PowBot SDN (https://api.powbot.org/products) was down:

HTTP code: 502
Response time: 349 ms

Resolved: PowBot SDN is back up in b8f1630 after 8 minutes.

repo: powbot/status


text:
Left-censoring

I wrote before in the README of the deleted test branch:

Unsure to be implemented

Left-censoring (I have difficulty understanding and implementing it)

I think this is a good feature to have, but I also think lots of refactoring of the hsmm_core is needed.

See the left-censoring branch for progress. Aside from implementation of left-censoring itself, I am making the repo more "venv-friendly" for testing.

Initial implementation completed. Further testing needed. See commits a33f43dcad81081fa90de1d3d559eacedde379d5 and c6dc4532b66e55de1f41933fc2af414fe721ef56.

repo: poypoyan/edhsmm


text:
Update 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:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
Some tools you may like, much more efficient

I wrote a set of tools, RemoveByName and RemoveByGUID that lets you specify a list of applications to remove, and it will remove them for you. It uses WMI, but it does it intelligently - only running the commands it needs to - So your not running removals on software that isn't installed. You interested?

Definitely. I want to use WMI as much as possible. It's insanely useful. Someone mentioned that if WMI is broken it would stop the script from running, but if that's the case, there's more of a reason to start over with a new install of Windows.
So in the interest of not being too long winded, please feel free to rewrite Stage 2. Thank you.

If WMI is broken, that actually indicates an issue with the PATH variable. In any case, I am not sure if my application be affected, because I am using direct calls to the system to do the work, not the WMI binary.

authors: PFCKrutonium pplude

repo: pplude/TronScriptEvo


text:
关于FcaNet: Frequency Channel Attention Networks

Hi♪(･ω･)ﾉ，建议FcaNet: Frequency Channel Attention Networks的publish可以更新为ICCV2021~[Link]~

已修改，感谢提醒

authors: dedekinds pprp

repo: pprp/awesome-attention-mechanism-in-cv


text:
Add personal info, change layout and styles

Create new layout for personal variant of portfolio.
Add info which were written in the associated tasks.
Add custom styles  so webpage will look at least readable.
Create PR and merge it into feature/body.

Done ✅

repo: ppvlo/homepage


text:
Wrappers around aleju/imgaug?

https://github.com/aleju/imgaug seems to have very nice augmentors. And the library even have the same name as our augmentors!
Maybe we could provide a wrapper on the interface to make it easy to use those augmentors in tensorpack. One thing is that they only support uint8 images.
@vfdev-5 any comments? since you seem to have experience with both libraries.

Yes, there are nice features as iaa.Sometimes, iaa.SomeOf etc (similar to RandomApplyAug and RandomChooseAug), keypoints transformations, iaa.ElasticTransformation and iaa.PiecewiseAffine, and other color augmentations, dropouts etc. Image augmentation randomness is implemented in a developed way, various sampling provided.
However, it mostly uses skimage vs opencv (in tensorpack) and bounded by uint8 and range 0-255.  In my experience, transformations like iaa.PiecewiseAffine can slow down dataflow.
@ppwwyyxx I do not know what kind of wrapping you would like to provide. Probably, the simplest way to use aleju/imgaug within tensorpack is with MapData. And it would be up to user to handle type of transformations, sampling, determinism/repeatability of aleju/imgaug transformations.
HTH

What I had in mind is a MapData as well.
Just found that the keypoints transformations in iaa seems to assume integer coordinates -- this is going to hurt training for coordinate sensitive tasks.

repo: ppwwyyxx/tensorpack


text:
Add internal interface to 4-way Keccak and implement C-fallback

Partly implements: #33
Acceptance criteria:

[ ] The reference implementation is modified to leverage an internal C interface to a 4-way parallel Keccak permutation.
[ ] The interface is implemented by fallback to the standard 1-way Keccak permutation in C.

Adding AArch64 assembly for 4-way parallel Keccak is not part of this issue.

@cothan Do you have time to give this some thought?

I made a version of gen_matrix work at compile time for Keccak x3 and x4 based on the defined KECCAK_WAY.
The idea is naive, I consider the matrix in gen_matrix as 1d array, apply KECCAK_WAY, and feed to rej_uniform. For the number of left over vectors less than KECCAK_WAY will fall back to 1-way Keccak version (can use switch/case to select the best left over x2, x3, x4...etc).
Is it something worthwhile to implement?
I think the interface you want would be some abstract API on top of x2,....x8-way Keccak at runtime.

authors: cothan hanno-becker


text:
Create documentation for pq cp general info, projects summary and processes,

As we build up the pqca project I think we should start documenting our processes
For example #11 discusses how repos may be named - others relate to codes of conduct, security.
Whilst we have the external website at https://pqca.org/ , I'm thinking of more operational/developer type information.
Each project will also have detailed docs, and in some cases established documentation sites.
I think we have another set of content in between where we document how the overall PQCA project is being run, what it offers, conventions & standards/ what our processes are etc.
A few approaches I can think of include

Github pages (mkdocs can add nicer formatting to plain github markdown, and there are various other options too) - As a developer I like this approach as it feels natural
Wiki (github or other) - this can work better for some non-developers (to which I include anyone creating content including docs, tests, ci etc) ,though I feel they are less likely to write content here
Website - where? LF? I'm far less comfortable with this

I think it would be useful to get something up asap - even if only temporary

I'm thinking this would be useful to have in place for the first virtual hack day.
We could create a 'docs' repository (or any other name suggested!) and use it to capture cross-subproject docs. I suggested mkdocs as it allows some structure, but simple github pages is an alternative.
Specifically we could post a few starter links/pointers/notes for the hack day
It would start very basic - and we may later decide to change to something different, good to start somewhere

We have setup a new repository 'documentation' using mkdocs material.
This is published at https://docs.pqcodepackage.org
Currently empty but a candidate for the TSC to consider, and a mechanism to invite doc contributions during the hackathon.

Further updates have been made to the docs site in an effort to try and build an initial structure

Closing this as we have a docs website, and a CI process to publish. -- will open specific issues, prs for content that needs adding

repo: pq-code-package/tsc


text:
filePond renders differently

Is there an existing issue for this?

[X] I have searched the existing issues

Have you updated FilePond and its plugins?

[X] I have updated FilePond and its plugins

Describe the bug
I tried to create a very simple image uploader. Here is the html:
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Title</title>
    <link href="https://unpkg.com/filepond@^4/dist/filepond.css" rel="stylesheet"/>
    <link rel="stylesheet" href="{{ url_for('static', filename='admin/test.css') }}">
</head>
<body>

<div id="pickupImageContainer">
    <input id="pickupImageInput" type="file" class="filepond" name="filepond"
           multiple data-allow-reorder="true" data-max-file-size="3MB" data-max-files="4">
</div>

<script src="https://unpkg.com/filepond-plugin-image-preview/dist/filepond-plugin-image-preview.js"></script>
<script src="https://unpkg.com/filepond-plugin-image-exif-orientation/dist/filepond-plugin-image-exif-orientation.js"></script>
<script src="https://unpkg.com/filepond-plugin-file-validate-size/dist/filepond-plugin-file-validate-size.js"></script>
<script src="https://unpkg.com/filepond-plugin-file-encode/dist/filepond-plugin-file-encode.js"></script>
<script src="https://unpkg.com/filepond@^4/dist/filepond.js"></script>
<script src="{{ url_for('static', filename='admin/reporteditor.js') }}"></script>
</body>
</html>

and the css: (copy, pasted from your own doc page)
.filepond--item {
    width: calc(25% - 0.5em);
}

/* use a hand cursor intead of arrow for the action buttons */
.filepond--file-action-button {
    cursor: pointer;
}

/* the text color of the drop label*/
.filepond--drop-label {
    color: #555;
}

/* underline color for "Browse" button */
.filepond--label-action {
    text-decoration-color: #aaa;
}

/* the background color of the filepond drop area */
.filepond--panel-root {
    background-color: #eee;
}

/* the border radius of the drop area */
.filepond--panel-root {
    border-radius: 0.5em;
}

/* the border radius of the file item */
.filepond--item-panel {
    border-radius: 0.5em;
}

/* the background color of the file and file panel (used when dropping an image) */
.filepond--item-panel {
    background-color: #555;
}

/* the background color of the drop circle */
.filepond--drip-blob {
    background-color: #999;
}

/* the background color of the black action buttons */
.filepond--file-action-button {
    background-color: rgba(0, 0, 0, 0.5);
}

/* the icon color of the black action buttons */
.filepond--file-action-button {
    color: white;
}

/* the color of the focus ring */
.filepond--file-action-button:hover,
.filepond--file-action-button:focus {
    box-shadow: 0 0 0 0.125em rgba(255, 255, 255, 0.9);
}

/* the text color of the file status and info labels */
.filepond--file {
    color: white;
}

/* error state color */
[data-filepond-item-state*='error'] .filepond--item-panel,
[data-filepond-item-state*='invalid'] .filepond--item-panel {
    background-color: red;
}

[data-filepond-item-state='processing-complete'] .filepond--item-panel {
    background-color: green;
}

JS:
FilePond.registerPlugin(
    FilePondPluginFileEncode,
	FilePondPluginFileValidateSize,
	FilePondPluginImageExifOrientation,
    FilePondPluginImagePreview
);

pickupImageInput = document.getElementById("pickupImageInput");
const pond = FilePond.create(pickupImageInput);

and the result is this image. What am I doing wrong?

e
Reproduction
as explained in bug description
Environment
- Device:
- OS:
- Browser:

It looks like you've not loaded the styles for the image preview plugin.

authors: mDadashi rikschennink

repo: pqina/filepond


text:
how to get file in +page.server.ts

my code:
in +page.svelte
<FilePond
					bind:this={inputFile}
					server='?/updateAvatar'
					{name}
					file={afile}
					allowMultiple={false}
					oninit={handleInit}
					onaddfile={handleAddFile}
				/>

in +page.server.ts
updateAvatar: async ({locals, request}) => {
        const dataImg = await request.formData()
        console.log("🚀 ~ file: +page.server.ts:36 ~ updateAvatar: ~ dataImg", dataImg)
        const file = dataImg.getAll('avataImageUserUpload')

        return {success: true}
    }

The file i get is blob, not file, so how can i get file (File format ) like normal input file form?

https://pqina.nl/blog/convert-a-blob-to-a-file-with-javascript/

repo: pqina/svelte-filepond


text:
Add Facebook Analytics

Ticket: https://praekeltorg.atlassian.net/browse/GEMMOLO-448
I have also moved other tags that result in <script> tags, to the bottom of our web page, in order to reduce page render time.

🍰 @nathanbegbie
I'm not too sure about the bbm analytics though.

authors: Mitso nathanbegbie

repo: praekelt/molo-gem


text:
Start of mainmenu+contentrepo integration

Not complete yet but want to keep PRs small

+1

repo: praekeltfoundation/vaccine-eligibility


text:
fix content feedback trigger

This won't only occur on new sessions, this might occur in exiting sessions, so we need to move it to the process_message function

@KaitCrawford Yes, 1, 2, and yes are fairly common responses. We could possible remove yes, because that isn't explicitly mentioned, but 1 and 2 are in the content of the message sent to the user, so the only way we can avoid that is to change the content.

@KaitCrawford Yes, 1, 2, and yes are fairly common responses. We could possible remove yes, because that isn't explicitly mentioned, but 1 and 2 are in the content of the message sent to the user, so the only way we can avoid that is to change the content.

bah humbug! ☹

authors: KaitCrawford rudigiesler


text:
Added Drum Sound Web App

Hello Project Admin,
I have added a Drum Sound Web App which I mention in the issue no. #2660 .
Screenshot -

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

It is super ready to Merge

@MohitGupta121  finally all set to go with

repo: pranjay-poddar/Dev-Geeks


text:
4483 Added the download button

#4483

Close #4483 >

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 , @TusharKesarwani  please review it

@MohitGupta121 made the required changes now there is only single project in this pr


text:
breaking change

Signed-off-by: Phil Prasek prasek@gmail.com

/rebase

repo: prasek/supergraph-demo


text:
Downloading speed is very slow

Bro increase downloading speed
You create a very good app keep improving

Patched

repo: prateek-chaubey/YTPro


text:
🛑 ChatGPT Beyond 2021 is down

In 84ff3c2, ChatGPT Beyond 2021 (https://chatgpt-beyond-2021.onrender.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: ChatGPT Beyond 2021 is back up in 79e314a after 3 minutes.

repo: prateekralhan/status


text:
Height of a BST

Title: Height of a BST
About: The aim is to form a Binary Search Tree of the integers entered and display its height.
Name: Manasi Chhibber
Label: Feature Request
Assignee: 'Manasi2001'

Define You:

[x] DevIncept Participant
[x] Contributor

Is your feature request related to a problem? Please describe.
The purpose is to explore trees.

@prathimacode-hub ma'am please assign me this issue :)

Issue assigned. @Manasi2001

Its been 6 days. 9th is the last date to accept PR's. If you wouldn't do it by then, obviously you can submit later but points won't be counted towards DCP program. @Manasi2001

Its been 6 days. 9th is the last date to accept PR's. If you wouldn't do it by then, obviously you can submit later but points won't be counted towards DCP program. @Manasi2001

Sorry ma'am for the delay. I got stuck in some college assignments. Have made the PR now. 👍🏻

repo: prathimacode-hub/PyAlgo-Tree


text:
re2.exe not responding

I have tried both nightly and regular build.  When I launch the game with the mod installed i just get the black game window. And it says not responding. Have to force it to shutdown.
SteamVR 1.22.11
Valve Index
Nvidia 3090 driver 512.15
Any ideas?

Ah ok i gotta start it from steam. I get a warnign about

I can press ok and it works. But it says it will affect perfoamnce. And I do get alot of frame drops in game

Disable Hardware Accelerated GPU Scheduling as pointed out here: https://github.com/praydog/REFramework/issues/147
Lower your SteamVR resolution
Switch to DX12

Read the Troubleshoting FAQ

authors: AndersMalmgren praydog

repo: praydog/REFramework