Set permissions: write-all

Setting permissions: write-all is required in new repositories for the GITHUB_TOKEN to work. See also https://github.com/JuliaDocs/Documenter.jl/pull/1819.
Without it, people might get an error like
fatal: unable to access 'https://github.com/rikhuijzer/MyProject.jl.git/': The requested URL returned error: 403

Please double-check whether I haven't missed a place in PkgTemplates.jl where this setting should be set too

Thanks for the contribution, @rikhuijzer
Looks like a few more files in test/fixtures/ need to the same update for reference tests to pass.
More generally, are there more restricted permissions we could set then write-all that would enable what we want?
PkgTemplates is fairly widely used to whatever permissions we set here can end up being set across many packages, so i'd be keen to set the minimal required :)

More generally, are there more restricted permissions we could set then write-all that would enable what we want?

Yeah, I wojld recommend

More generally, are there more restricted permissions we could set then write-all that would enable what we want?

Yes. Well spotted. Would you agree that
    permissions:
      contents: write

is good? I've tested it at https://github.com/rikhuijzer/DiscountedCashFlows.jl/commits/main commits 4f85500df75d6efc78e999ed78e201a17f74216d and d19106abd4de732fdecc8d64aa1e176058dc855c.

Yeah, permissions: contents: write sounds good to me!
I think you'll also need to rebase this on master, as we just updated the reference tests to run on v1.7.2 (#349).
If you then run tests locally with v1.7.2 it should offer the option to update any reference test files that need updating to match this change.
Finally, if you bump the Project.toml version to the next non-breaking version, i'll make a release with this change as soon as it's merged
Thanks again!

I think you'll also need to rebase this on master, as we just updated the reference tests to run on v1.7.2 (https://github.com/invenia/PkgTemplates.jl/pull/349).
If you then run tests locally with v1.7.2 it should offer the option to update any reference test files that need updating to match this change.

Finally, if you bump the Project.toml version to the next non-breaking version, i'll make a release with this change as soon as it's merged

Great. Thanks for the instructions.
I won't have time to continue working on this. If you want to get it merged quickly, then feel free to make some further adjustements

thanks for your work, @rikhuijzer
hopefully i can get this over the line from here. i must admit Github Actions is not something i'm too familiar with. I wonder if @DilumAluthge might be able to give this PR a review?


documentation

Port relevant portions of documentation to README

Okay,  my first hw assignment was a failure.  I did per below to start out with modifying the text files you created with what I have in the developer console:

Edit the files.
Save and check changes ("git status" shows changed files while "git diff" shows actual changes)

This worked okay, but this did not:

Commit the changes ("git commit -a")
Push the changes ("git push" provide your username/pw when prompted)

I did change the editor:
git config --global core.editor "brackets"
The error message I got follows:
MacBook-Air:alexa-vera-bridge mprice$ git commit -a
error: cannot run brackets: No such file or directory
error: unable to start editor 'brackets'
Please supply the message using either -m or -F option.
MacBook-Air:alexa-vera-bridge mprice$ brackets
-bash: brackets: command not found
MacBook-Air:alexa-vera-bridge mprice$
I assume this is because brackets isn't properly pathed, but rather than struggle with it, I thought I'd query you first.
Also, since I cloned the directory, I assume I lost the diff history, so I'm not sure of the changes you did to the python files.  Are those ready to replace and try?
Dad

Oops, not sure how to use this.

need to figure out
[ ] uploading images
[ ] links (document/external)
[ ] various other small formatting changes
[ ] write quick start section

Will get brackets or some other editor working on Mac for git and python
Will continue to chip away at python tutorial

[x] testing list
[ ] testing list incomplete
:cactus:


Refactoring: don't interpolate in log function (deprecated technique)

Description
Refactoring: don't interpolate in log function (deprecated technique)
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

/lgtm

/approve

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

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

Project coverage is 94.66%. Comparing base (617d32f) to head (3f13f04).
Report is 12 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1473   +/-   ##
=======================================
  Coverage   94.66%   94.66%
=======================================
  Files          58       58
  Lines        2531     2531
=======================================
  Hits         2396     2396
  Misses        135      135


Initial commit

Without reviewing what you actually committed, initial commits to master without discussion are generally a bad idea.Sent from my iPhoneOn Mar 31, 2024, at 6:33 PM, CzK87PL @.***> wrote:
You can view, comment on, or merge this pull request online at:
  https://github.com/dogecoin/dogecoin/pull/3504
Commit Summary
01fca12  Initial commit
File Changes  (0 files)
Patch Links:
https://github.com/dogecoin/dogecoin/pull/3504.patch
https://github.com/dogecoin/dogecoin/pull/3504.diff
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Provide efficient way to reassemble split Bytes

BytesMut::unsplit[1] can be used to reassemble a large BytesMut object that has been split multiple times.  However, it takes O(n) time for what in many cases could be an O(1) operation.  Even with unsafe, there is currently no faster way to reassemble split BytesMut objects.  I see a few ways to solve this:

Remove the at == 0 optimization in Bytes::split_to, and remove self.cap = end from set_end.  But this would mean that calling extend_from_slice on the new Bytes could corrupt another.  So that's a nonstarter
Create a new struct BytesShadow which keeps a reference to a Shared, but allows neither read nor write operations.  BytesShadow would be upgradable to either Bytes as long as there are no BytesMut objects referencing the same Shared, or to BytesMut if there are neither Bytes nor BytesMut referencing the same Shared.  This would require adding a field or two to Shared, but should have negligible performance impact.
Provide an unsafe way to create a BytesMut and Bytes that share storage.
Provide Bytes::into_vec which would consume the Bytes and return its inner Vec.  The user could then use Vec::set_len to recover the entire buffer.

Does any of these appeal to you?  Do you have any better ideas?  I'm willing to try implementing whatever sounds good to you.
[1] And after PR #182 Bytes::unsplit too

I didn't say safely.  Most of the options I'm considering are unsafe.  Intuitively, it's obvious how you would do this in C or C++: just save the pointer to the beginning of the buffer.  The BytesShadow option, however, might be safe.  I'm working on a PoC right now.  However, it might turn out to be incompatible with the truncate and extend_from_slice methods.

Ok, I have something working.  It's similar to the BytesShadow option I mentioned above.  The idea is that there's a single object that owns the real storage (DivBufShared), which can create multiple child objects that each reference a portion of the total storage.  Crucially, the parent DivBufShared may coexist with one or more mutable DivBufMuts that use its storage.  That's how you can quickly recover the original buffer after splitting it up.
There were a lot of problems caused by methods like BytesMut::extend_from_slice and BytesMut::clone that COW data.  So in my version data is never implicitly copied.  Remove those methods, of course, makes it incompatible with bytes.
I've temporarily posted the documentation here.  Do you see a way to add this kind of functionality to bytes?  If so, offer your suggestions.  If not, I'll just publish this as a new crate.
https://people.freebsd.org/~asomers/rust/doc/divbuf/

I looked at divbuf and I'm still not sure I follow exactly. It seems like you are saying that if you split a BytesMut, modify one half in such a way that it grows the buffer, then try to unsplit, it will not work because the two handles do not share memory anymore.
This seems like correct behavior.
If you always want to be able to unsplit, don't call fns that could grow the buffer.

Ah, I think I see now.
Yeah, I don't know if there really is a way for Bytes to get this functionality. I'm going to close this for now, but if there are no insights on how to add this functionality w/o fundamental API changes, then we can continue the discussion.


Unit gets stuck in maintenace status when pebble_ready events occur

Event handlers need to be re-designed to reach Active status after handling pebble_ready events.

unit reaches active status on install at first, but gets stuck in maintenance in case one of the pebble_ready events take place because their handlers don't call main.

unit reaches active status on install at first, but gets stuck in maintenance in case one of the pebble_ready events take place because their handlers don't call main.

At the end of Pebble Ready event we should call main event handler (main()).

Closing as it's solved by #83


VSCODE gutter does not display

VS Code Version: 1.78.0 (user setup)252e5463d60e63238250799aef7375787f68b4ee 2023-05-03T20:09:00.748Z
OS version: Windows_NT x64 10.0.22000
Sandboxed: Yes

Steps to Reproduce:

My gutters have been gone for months, which makes my development inconvenient
I have tried several solutions online but none of them worked (including reinstalling vscode).

Did you check the state of your editor.glyphMargin setting?

Did you check the state of your editor.glyphMargin setting?

This state is on

I've tried rolling back the version to version 1.76.2 where I have previously encountered this situation.
The problem is not solved, now I am using version 1.73.1, which shows the gutter normally

Can you try with default settings?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


📝[Docs]: Enhancing README.md

What Docs changes are you proposing?
I am aiming to enhance readme.md under gssoc.
Why do the Docs need this improvement? What is the motivation for this change? How will this change benefit the community?
The readme is good but not well mentained, adding feature in section which showcase that the project is a part of gssoc in table form will make readme more attractable also add table of content of easy navigation, link all .md files in main readme for contributor convenience
Describe the solution you'd like
As mentioned above I want to enhance overall readme, readme serves as the first source of interaction between project admin and viewers so it should be well maintained

any reference? @ananyag309

something similar

Hi @ananyag309 ,
I'm Atharva, and I'm new to contributing to open source. I'd love to contribute by enhancing the README file for StartConnect-Hub. May I take on this task? Also, are there any specific guidelines I should follow?
Thank you!

@ananyag309 assigned to you, you can enhance the readme file

Hi @ananyag309 ,
I'm Atharva, and I'm new to contributing to open source. I'd love to contribute by enhancing the README file for StartConnect-Hub. May I take on this task? Also, are there any specific guidelines I should follow?
Thank you!

She is also a contributor pls look for some other issues, welcome to the open source!

Hi @ananyag309 ,
I'm Atharva, and I'm new to contributing to open source. I'd love to contribute by enhancing the README file for StartConnect-Hub. May I take on this task? Also, are there any specific guidelines I should follow?
Thank you!

She is also a contributor pls look for some other issues, welcome to the open source! @atharvamulam

No issues. Thank you!


[WIP] Create generic task notifications

These notifications types are intended to be use with provider-initiated tasks when there is a need to notify the user about the progress. The expires_in and audience values are just my assumptions and they might be wrong, this is one of the reasons why this is WIP.
I also added helper methods for initiating these notifications from models, you just have to include the NotificationMixin concern and you can use the notify_task(start|finish|fail|update) methods.
Related demo: https://github.com/ManageIQ/manageiq-providers-amazon/pull/468
@miq-bot add_reviewer @martinpovolny

@miq-bot add_reviewer @Ladas

@miq-bot assign @agrare
@miq-bot add_label enhancement

@skateman can you unwip this and rebase? I think this can get in


Bump-up pytest-asyncio to 0.24.0

Description
Bump-up pytest-asyncio to 0.24.0
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[x] Bump-up dependent library
[x] Bump-up library or tool used for development (does not change the final image)

/retest

/retest

/lgtm

/approve

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

Project coverage is 94.82%. Comparing base (d3076b3) to head (275b952).
Report is 38 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1534      +/-   ##
==========================================
+ Coverage   94.75%   94.82%   +0.07%
==========================================
  Files          59       60       +1
  Lines        2631     2669      +38
==========================================
+ Hits         2493     2531      +38
  Misses        138      138

/override "Red Hat Konflux / lightspeed-service-on-pull-request"


more headers

Partially addresses #4535
the following headers were touched:
Last-Modified
Location
Max-Forwards
Proxy-Authorization
Range
Range header contains some additional code and I left it as it be, since I wasn't sure what to do with it.
Edit: I've added the Origin header as well, but I'm not sure if that Header instance is ok.

Oops.  I broke this handling the merge conflict.  My fault.


DEPTH_TEXTURE incorrect data

Godot version:
Godot 3.1-beta3
Godot git 463123a661151f119132f2ee9af78925a58a068a
OS/device including version:
OS: Arch Linux x86_64
Kernel: 4.19.4-arch1-1-ARCH
GPU: NVIDIA GeForce 940MX
GPU: Intel HD Graphics 620

Issue description:
In a ShaderMaterial, DEPTH_TEXTURE seems to have incorrect data (usually 0.0 or 1.0, but sometimes glitches out with a couple random pixels with arbitrary values) with the GLES3 backend. With a similar project running Godot 3.0.6-stable (not the same project, since 3.1 didn't seem backwards compatible), the same shader code works properly (you can see geometry depth behind the plane if you put the camera close enough.) SCREEN_TEXTURE still works properly in 3.1-beta3 as well as 3.0.6-stable.
Minimal reproduction project:
depth_texture_test_case.zip

I can confirm this bug. I tested on a few versions (3.06, a1, b1, b2) and the bug was only present in beta3.

677d4400de9ae73713be9f33600f59a10a079c19 is the faulty commit. Adding an OmniLight with contact shadows enabled does indeed fix the issue.
This can likely be fixed by using a check similar to SCREEN_TEXTURE for DEPTH_TEXTURE, ie. writing to it when a shader uses it.

This issue should be fixed with #25557

Reopening as #25557 was reverted in 0eb61c3106 as it triggered #26258 (and @reduz says it's not the correct fix for this issue).

Out of curiosity, what would a more correct fix for #25557 be?

@akien-mga I think this should be part of the 3.1 milestone. Or at least part of an early round of bug fixes. Reading from depth is important for a lot of SpatialMaterial effects (depth fade, proximity fade, etc.).

I don't know if this is related to this problem, but I wanted to play around a bit with the Depth-buffer in an Shader and found this in the Godot-Asset-Library: https://godotengine.org/asset-library/asset/76
After downloading it, I stumbled upon the issue, that the intersection parts are not highlighted in Godot 3.1-Stable (this actually use the Depth-buffer to achieve the white borders) but is correctly working with Godot 3.0.6. So I guess this a bug and hopefully related to this issue here. I've uploaded two screenshots to make it more clear what I mean.
Godot 3.0.6-Stable

Godot 3.1.0-Stable

@Nophlock yes, this is the result of the depth buffer not working.

Sorry to bother you. I don't quite understand the problem. The values I get in the fragment are all 1.0. Can you tell me how to get the correct value, or under what circumstances it is invalid.❤


Simplify CI into one file

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Made a few cleanups and committing. Thanks!


Keepalive

net:support TCP_KEEPINTVL,TCP_KEEPCNT,TCP_USER_TIMEOUT for tcp keep-alive
except TCP_KEEPIDLE, linux provide TCP_KEEPINTVL,TCP_KEEPCNT,TCP_USER_TIMEOUT for tcp keep-alive. support this then users have more control over tcp keep-alive.
add two new api uv_tcp_keepalive_ex and uv_tcp_timeout for user to control tcp keepalive

Any movement on this? I would like to be able to set TCP_KEEPCNT and TCP_KEEPINTVL


Threshold Switch does not accurately calculate the level of storage views with different capacity

Describe the Bug
Hello!
In ThresholdSwitchBlockEntity.java, the method updateCurrentLevel inaccurately calculates the fill level when storage views have different capacities. It treats all storage views as if they have the same weight, leading to incorrect results.
Here's a simple example to illustrate the problem:
Suppose there are two storage views, one with a capacity of 100 and one with a capacity of 200.
Suppose the first storage is completely full (100/100) and the second storage is half full (100/200).
Using the current code, occupied would be calculated as (100/100) + (100/200) = 1 + 0.5 = 1.5, and totalSpace would be 2.
Therefore, currentLevel would be occupied / totalSpace = 1.5 / 2 = 0.75.
However, the actual fill level of the storage system would be 200 out of a total capacity of 300, or approximately 66.67%.
Proposed Fix:
Calculate Total Space by Summing Actual Capacities: Add the actual space of each storage view to totalSpace instead of incrementing by 1.
Calculate Occupied Space by Summing Actual Counts: Add the actual count of each storage view to occupied instead of calculating count * (1f / space).
As I'm not sure about potential edge cases, I preferred to post about it here instead of submitting a PR (I'm new to the Minecraft mod scene)
Thank you!
Reproduction Steps

Use a Threshold Switch on a storage system with at least two views with different capacity
Fill one and leave the other one empty
Look at the level reported by the Threshold Switch

Expected Result
Accurately calculate the fill level
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
I've discovered this issue while using the Extended Drawers mod for Fabric but confirmed with the author that it was an issue with the Create block.
Additional Context
No response

Same bug was posted today here: https://github.com/Creators-of-Create/Create/issues/5299
I'm closing this report


Malformed Attribute: Missing quotation mark

Hi! When I try to parse a html file, I get this error:
Malformed attribute: Top: Blending in linear color space produces expected blending results<br - Missing quotation mark at around index 9873

The minimal reproducible example is
#[cfg(test)]
mod tests {
    use html2md_rs::parser::safe_parse_html;
    use html2md_rs::to_md::safe_from_html_to_md;

    #[test]
    fn test_safe_from_html_to_md() {
        let input = r#""#.to_string();
        let parsed = safe_parse_html(input).unwrap();
    }
}

I guess this is because you didn't take care the case when a string in html has a HTML tag.

@ifsheldon Thanks for the issue! Let me know if there's anything else I can do :)


[Bug] TC-CNET-4.9 Test Failure

Describe the bug
ERROR      | 2024-08-02 10:55:46.946765 | Test Step Error: Failed parsing with error: Step using "verification" key should either set "disabled: true" or "PICS: PICS_USER_PROMPT"
INFO       | 2024-08-02 10:55:46.948951 | Test Step Completed [ERROR]: Start chip-tool test
Steps to reproduce the behavior
No response
Expected behavior
No response
Log files
4-9_2024_08_02_04_16_33.log
PICS file
Network Commissioning Test Plan.zip
Screenshots
No response
Environment
No response
Additional Information
No response

DUP of https://github.com/project-chip/matter-test-scripts/issues/345


Pr stats

My review statistics are skewed by the fact that I take care of the weekly upstream sync…


Mwpw 132061 library

Adds the library feature and a quick link to our new docs to the sidekick
Resolves: MWPW-132061
If you want to test the new library feature, go to your sidekick settings and configure like so

Test URLs:

Before: https://stage--express--adobecom.hlx.page/express/?lighthouse=on
After: https://MWPW-132061-library--express--adobecom.hlx.page/express/?lighthouse=on

Code change looks good and makes perfect sense but I can't get the Library button and Docs button to show up after configuring the branch url in sidekick. Did I miss something?

Code change looks good and makes perfect sense but I can't get the Library button and Docs button to show up after configuring the branch url in sidekick. Did I miss something?

@qiyundai hmm, that's interesting. Did you configure it for the right repo ? If not we can take a look :D


Create group within splits

This creates the group within the splits contract instead of passing it in. This makes the frontend flow a lot easier as you don't have to 1) create a group, 2) search for the group, 3) create splits with the group address.
To discuss: who should be the admin of the group contract, if any? In this PR it's set to the sender.

Replaced by #492


RemoteCommand

Does the extension have the ability to process RemoteCommand? Like this:
RemoteCommand echo '#!/bin/bash' >> /home/ops/askpass.sh && echo "echo 'user'" >> /home/ops/askpass.sh && chmod +x /home/ops/askpass.sh && export SUDO_ASKPASS="/home/ops/askpass.sh" && sudo -Ai bash -c 'rm /home/ops/askpass.sh && /bin/bash'

@pavel-hushcha you want to set an environment variable on the destination host using an interactive query on the shell and then spawn a bash using this 'extended' environment, right?

@qknight , yes, that's correct.


Terminate build on unknown variant type

Fixes: #2508
Signed-off-by: Morgan Davies<EMAIL_ADDRESS>

That logic implies more hardcoding of variants in places that are not so obvious. What is the problem with using regexes on arrays in shell?

That logic implies more hardcoding of variants in places that are not so obvious. What is the problem with using regexes on arrays in shell?

Two reasons:

I find them quite a bit more difficult to read and parse than alternatives (The array is being defined in your PR only for the purpose of this at the moment - it could just as easily being a space-separated string to be used in my grep example)
(Personal one) I've always preferred to keep as close to standard POSIX shell syntax as far as possible as opposed to relying on bash-specifics as possible for portability

But as I said, not something I'd insist on being changed - we've filled the scripts with lots of bash-isms already and they're not going away ;-)


Disabling DLSS crashes GPU Driver

Describe the bug
When disabling DLSS in Remix, it consistently crashes my GPU driver. This only appears to happen at 1080p or above for me. This occurs on Portal RTX, Flatout 2, and various other games. It does not appear to be directly related to VRAM either, as it doesn't max out before the crash.

Remix builds: Various, including latest Portal RTX and 77900b5
3070 ti
535.98
Flatout 2: Steam
Portal RTX: Steam

Attach logs!
Portal RTX Logs:
hl2_d3d9.log
d3d9.log
NvRemixBridge.log
Flatout 2 Logs:
NvRemixBridge.log
d3d9.log
FlatOut2_d3d9.log
To Reproduce
Steps to reproduce the behavior:

Load a game with Remix at your monitor's full screen resolution
Open the Remix overlay
Disable DLSS (the crash should occur here)

Expected behavior
The upscaling method should switch successfully, without a driver crash
Screenshots

REMIX-2019 for tracking.

@Kim2091 Can you reproduce this after the ImGUI fix? I believe all these issues with toggling reflex, dlss, enhanced assets had to do with ImGUI and got fixed with that patch.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.4 to before v5.4.128",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9d71dd0c70099914fcd063135da3c580865e924c",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=22cba878abf646cd3a02ee7c8c2cef7afe66a256"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "9d71dd0c70099914fcd063135da3c580865e924c",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "22cba878abf646cd3a02ee7c8c2cef7afe66a256",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "can: j1939: fix Use-after-Free, hold skb ref while in use\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.128 by commit 22cba878abf646cd3a02ee7c8c2cef7afe66a256, it was introduced in version v5.4 by commit 9d71dd0c70099914fcd063135da3c580865e924c. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000928


Add .all static properties to Expansions and Fields

close #129

Codecov Report

Merging #130 (26a98ef) into main (89308a3) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   80.32%   80.33%   +0.01%
==========================================
  Files         255      255
  Lines        6805     6809       +4
==========================================
+ Hits         5466     5470       +4
  Misses       1339     1339

Impacted Files
Coverage Δ

Sources/TwitterAPIKit/APIv2/ExpansionsV2.swift
100.00% <100.00%> (ø)

Sources/TwitterAPIKit/APIv2/FieldsV2.swift
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89308a3...26a98ef. Read the comment docs.


DBZ-7695 Backport to 2.6 add _id field to tombstone rewrite events, for delete events

@ismailsimsek Hi, thanks for the PR (ant the other one). As this is a new feature not bugfix we prefer not to do the backports.


Support to validate connection host and port.

Feature Request

Is your feature request related to a problem? Please describe
Currently, it only validate whether connection is open when verify the validity of redis connection. But sometimes, redis master will be switched, the vip address will pointed to the new redis master. But in redis connection pool, the connection still points to the discarded redis instance which has be slave, so it cannot write data by this connection.
// io.lettuce.core.support.ConnectionPoolSupport.RedisPooledObjectFactory#validateObject
public boolean validateObject(PooledObject<T> p) {
    return p.getObject().isOpen();
}

Describe the solution you'd like
There is a way to point the latest redis master. When validate redis connection, it can validate whether the connection host and port is correct.

Connection pool support isn't aware of clustering arrangements. If you want to use clustering and pooling, then you need to provide that kind of support for eviction yourself.


Added HTML5 Canvas Notes for Professionals

What does this PR do?
Add resource
For resources
Description
The HTML5 Canvas Notes for Professionals book is compiled from Stack Overflow Documentation, the content is written by the beautiful people at Stack Overflow. Text content is released under Creative Commons BY-SA. See credits at the end of this book whom contributed to the various chapters. Images may be copyright of their respective owners unless otherwise specified
Book created for educational purposes and is not affiliated with HTML5 Canvas group(s), company(s) nor Stack Overflow. All trademarks belong to their respective company owners
180 pages, published on May 2019
Why is this valuable (or not)?
This book contains best answers from Stackoverflow with respect to HTML5 Canvas
How do we know it's really free?
Author of this book has kept this book free of cost.
For book lists, is it a book? For course lists, is it a course? etc.
Yes, It is a book.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

:heart: Thanks for taking your time filling the pull request

Thanks for this extra secure https but think in conficts that could produce in other merges. Next time, try to scope fitting to your contribution 😉

Thank you for reviewing this pull request. I'll keep that in mind. 😄


Adding workqueue disabling if user is on SSD

This is a pretty easy fix, but currently only if you have one SSD partition, so the command will be more complex in the end:
lsblk -d -o name,rota | grep -E '^nvme|^sd' | awk '{print $1}' | head -n 1 && printf 'discard,no-read-workqueue,no-write-workqueue' | sudo tee -a /etc/crypttab && notify-send -a "optimizer" "Encrypted SSD performance optimized"

This command detects if you have any SSD, and just adds the parameters to the last line of the crypttab file.
Problems:

only working if one SSD works
encrypted SSD + encrypted HDD not working

Something building on top of that will be helpful for performance.

I feel like this is out of scope for ublue, if this really gives a significant performance boost you should go to upstream Fedora and try to push it there.


Update for 2.0?

First of all thanks for all the wonderful mods you make available for us.
Many have already been updated to 2.0 but not this.
Hope it is planned as well?

I only have so much time. I have to prioritize my Wube work and then the next priority is Krastorio 2. So it might be a while.

Do you accept PRs?

I do, but for something like this it will be more efficient if I just do it myself. I can update the mod myself far quicker than I can comb through someone else's code doing the same.

Really waiting for 2.0 version, building early game now via hand is really pain

@raiguard
I almost ported stuff - except quality 😁
My two main issues so far

It seems that if entity is being upgraded to the same prototype with different quality, to_be_upgraded returns false and get_upgrade_target returns nil, nil
There's ghost_prototype but no ghost_quality. Not sure how to deal with entity-ghost cases

Or am I doing something utterly wrong?

Sorry for missing your PR, I just pushed my own update. Oh well. :laughing:


No values for Roll/Pitch!

First of all thank you so much for doing this.
i was just wondering if you can add Roll/Pitch values so we can know if the bot is flipped.
and it will help so much in making bots fly!
Thank you!

There are roll and pitch values.
What version of the framework are you using?
Might be easier to talk on the discord

Im asking for a Roll/Pitch value reader block on the scratch.rlbot.org editor
i followed the instructions and downloaded python like an hour ago
im pretty sure im using the latest version
send discord link thx

The discord link is listed on the github on the readme
Here is another one:
https://discord.gg/W37mPd

I'll be adding these soon!

Thank you!

A boost value also would be great!

Added pitch, roll, and boost value. Sorry it took forever!


can't use with gradle?

Thank you for good question. We are working on this. Check for updates.

Thank you for sharing this beautiful side menu, but I received a msg at Cradle
Error:A problem occurred configuring project ':app'.

Cannot evaluate module circualreveal : Configuration with name 'default' not found.

I think it's can help you
http://stackoverflow.com/questions/24453971/building-gradle-for-wear-app-failed
and be sure that you use gradle version 2.2.1

I see.thx


rewrite carl.envs.init

rewrite loading of environments. loading all takes too much time.

This should not be an issue anymore because base envs are instantiated during init of CARL env.


[R-package] [docs] Simplified examles to cut example run time (fixes #2988)

See #2988 . The R package is very close to hitting this R CMD CHECK NOTE:
* checking examples ... NOTE
Examples with CPU or elapsed time > 5s
     user system elapsed
dim 5.426  0.264    5.79

In this PR, I cut the runtime for our examples by:

removing all library(lightgbm) (these are not necessary and it is conventional to omit them)
cutting nrounds from  10L to smaller numbers in all examples with training
wrapping examples for code that relies on the filesystem in \donttest
wrapping plotting code in \donttest

These tests run in about 2.9 seconds on my laptop.  That means CI won't catch those examples being broken
I think it's worth it to get rid of the R CMD CHECK note. We test all those functions in unit tests, so if we break the functions themselves  we'll know about it.
Won't using nrounds = 5L and other parameter changes result in terrible models?
Yes probably. But we don't have strong guarantees about how well the models from our example code work anyway.  I think the role of examples here is just to show minimal valid sets of arguments to each function. See  https://github.com/microsoft/LightGBM/issues/2988#issuecomment-611726510 for my full opinion on this.

@Laurae2 or @guolinke could you please take a look whenever you have time? This NOTE is blocking #2985 and even if we work around it there it's going to block other PRs (since the  runtime right now is so close to 5  seconds)

Should we really be using \donttest? That means CI won't catch those examples being broken

Sorry for maybe stupid question, but I'm still far away from R and learning it only by our conversations here 😃
As we do not wrap them into \dontrun (but in \donttest), it means that these examples are will be running at our RTD site and failing docs will indirectly indicate that our examples may be broken, right?

As we do not wrap them into \dontrun (but in \donttest), it means that these examples are will be running at our RTD site and failing docs will indirectly indicate that our examples may be broken, right?

@StrikerRUS The examples would fail silently (but still show the error in doc).

As we do not wrap them into \dontrun (but in \donttest), it means that these examples are will be running at our RTD site and failing docs will indirectly indicate that our examples may be broken, right?

@StrikerRUS The examples would fail silently (but still show the error in doc).

I thought that if you added \dontrun, it will literally add the comment "## don't run" in the examples which I think would be confusing. I swear it used to work like that. But I just tested and I guess not?

@Laurae2 is right (https://github.com/microsoft/LightGBM/pull/2989#issuecomment-613026665). I tried adding stop("I am an error") to the examples in lgb.cv() and then re-running pkgdown::build_site().
Results when that example is wrapped in \donttest

Results with \dontrun

In which case I think the \donttest is still an ok choice. I'd rather that we have an error message that someone sees and alerts us to than just having the code be silently broken until someone runs it. I'd hate to have people run that code, get an error, and get discouraged and thing they're doing something wrong.

Change the learning_rate to adapt to the hyperparameter changes

Thanks! Just made these changes

Not sure that the root cause is this PR, but we have the following right now at our docs:

https://lightgbm.readthedocs.io/en/latest/R/reference/lgb.cv.html

Also, it seems that two other examples are broken:
https://lightgbm.readthedocs.io/en/latest/R/reference/lgb.Dataset.set.categorical.html
https://lightgbm.readthedocs.io/en/latest/R/reference/lgb.plot.interpretation.html

Thanks @StrikerRUS ! I don't think they are related to this PR. I just created #2999 to address them though, so appreciate you pointing them out!


Multiple DNS Zone Support

Description
This PR is to add support for multiple DNS zones.
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
E2E and unit tests have been updated to test this functionality.
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a breaking change which will impact consuming tool(s)?

[x] E2E suite and terraform remade to use multiple zones
[x] Fixture tests updated
[x] Unit tests to test new CLI args

Checklist:

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

/ok-to-test a376759

/ok-to-test sha=a376759

/ok-to-test sha=a376759

/ok-to-test sha=42ac613

/ok-to-test sha=b4dc991

/ok-to-test sha=680455d


Trim string values in requiredValidator empty check

https://github.com/vivid-planet/comet/pull/2499#discussion_r1749910436

I would not expect that, and class-validator @IsEmpty also doesn't do this

Imho I would not. You are creating more problems (you need a custom @IsNotEmpty server side) than you solve with this.

Haven't thought about this when making the suggestion. I'm fine with not making this change. At least we then don't need to backport it 😁 Sorry @thomasdax98!


Extract html

Feature request: I'd love the ability to extract html from a CSS selector. Currently it seems there's no good way to do so. Perhaps like so:
const extractedHtml = new HTMLRewriter().on('#my-id', {
  element(e) {
    e.extract();
  }
}).transform(response);

Not sure that'd be the most appropriate syntax given the nature of extracting vs. modifying/removing.
More context: https://community.cloudflare.com/t/htmlrewriter-extract-and-serve-single-dom-node/136769

@dsbudiac We have internal discussions about such API some time ago. It's relatively easy to implement, but we didn't prioritise it as we wanted customer feedback first.
The API will probably be similar to text handlers:
new HTMLRewriter().on('#my-id', {
innerHTML(htmlChunk) {
//...
}
}).transform(response);

That would be awesome.
In case you're looking for feedback, here's my particular use case. We have a fairly large site with a lot of dynamic pages. I'm trying to both:

Minimize initial payload and number of DOM elements by loading more content on DOM ready (or lazy load/on-demand)
Minimize the number of cache items I need to manage (which in turn should increase cache hit ratio).

P.S. I'm already maximizing s-maxage for HTML and busting using the REST api as needed.
My idea is this: Class and id DOM nodes on the origin that I'd like to load after DOM ready. Cloudflare would store the entire origin response in cache (single cache item), however the worker would strip out/serve the appropriate content based on the request.
For example let's say I have the following HTML served by the origin:
GET /my-page/

Content to include on initial payload

Content to load async via js after DOM ready

Cloudflare would store that as a single cache item. However when the user requests the resource, the Cloudflare worker would modify initial payload to:

Content to include on initial payload

With CSS .async.placeholder { display: none; }. On DOM ready, the client javascript would look for any .async.placeholder items and request like:
GET /my-page/?async=my-content

The Cloudflare worker would handle again, but see the async query string flag. It would pull from the single (ideally cached) item, strip out just the necessary content and serve:

Content to load async via js after DOM ready

The client would then rip and replace the response.

All this said, this begs the question: am I actually improving overall performance with this idea?
Cloudflare workers would effectively be modifying the DOM of every single response w/ HTMLRewriter. While HTMLRewriter is supposed be very efficient, it has to require some amount of overhead.
Would I be much better off just maintaining separate cache items?

This would be incredibly useful for docs.rs as we're looking into switching to lol_html because of its performance and resource usage bonuses, has there been any progress?

@Kixiron As far as I'm aware @ObsidianMinor is currently working on this.

Not at the moment sadly. I'll probably get a chance to work on it again next quarter. If you want it sooner we're open to PRs.

This would be incredibly useful for docs.rs as we're looking into switching to lol_html because of its performance and resource usage bonuses, has there been any progress?

We ended up not needing this after all :) https://github.com/rust-lang/docs.rs/pull/930

@ObsidianMinor did this get put together for this quarter by chance?

+1 for this too
I use Workers as a proxy (along with WebPageTest) for experimenting with and demonstrating potential performance improvements to clients
I often wrap script blocks such as

With an event handler for window onload e.g.
class deferInlineScript {
  element(element) {

    const wrapperStart = "window.addEventListener('load', function() {";
    const wrapperEnd ="});";

    element.prepend(wrapperStart, {html: true});
    element.append(wrapperEnd,  {html: true});
  }
}

Sometimes the selectors to extract correct DOM node are fragile e.g. head > script:nth-of-type(1) so being able to get  the contents of the node so I can check I'm operating on the correct one would be helpful.

Adding some additional use-cases incase it helps. At @phishdeck we are looking to use lol_html a part of a specialised MitM proxy. Most selector works can be done on tags & attributes, however some tags require us to peak into the inner content. Right now we only have set_inner_content which doesn't fit the use-case (flushes previous value) so something like this in the library would be fantastic.
If there is a PR open I'm willing to help contribute to it. :-)

Some questions about this:

Do we want to pass the entire HTML contents to the handler, or do we want it to be chunked, just like text ones?
Do we want innerHTML, outerHTML, or both?
When there is a HTML handler active, do we need to resynthetize the HTML representation of each token we created, or is there a way to just pass along the original HTML input we received and parsed?

AFAICT, we are talking about 2 separate features in this ticket:

The first one is the ability to filter out everything except some tags that match a selector (in the issue description, the forum post, and this comment);
the second one is the ability to read the inner HTML contents from a handler and transform it (in this comment and this one).

The first feature is pretty easy to implement, it corresponds more or less to the sed -n s/foo/bar/p trick where you neuter the output by default with -n and then explicitly include it back for some actions with the p command.
This could be done with:
new HTMLRewriter().print(false).on('#my-content', {print: true}).transform(response)

@inikulin Do you think this would be a viable API for this specific use case?

The second feature is more convoluted. I understand that @inikulin wants them to be like text handlers, but I'm curious about the implications here. Let's say the input to the rewriter is:

    abc

      def

    ghi

If we set up an inner HTML handler on div and an element handler on p, should the inner HTML handler be fed chunks before the element handler on p is ever invoked, thus giving the user the opportunity to completely remove the p element before it's even parsed by the rewriter, or should it always be invoked last after any other handler had the opportunity to do its job?
And once the inner HTML handler was invoked on a chunk, should the rewriter reparse the output that was just produced for further processing, maybe invoking other handlers on it?

With all that said, I'll start working on the first use case, which is pretty straightforward.

I think for the first use case we can do something like:
new HTMLRewriter().removeExcept('sel1).removeExcept('sel2').transform();

For the second use case:

We can keep the text handlers-like behaviour
innerHTML handler takes precedence over rewritable unit handlers
innerHTML handler produces chunks in rewritable unit boundaries, e.g. it will never produce
<spa chunk. It will wait till we get the whole start tag:
.
Rewritable unit handlers for the chunks captured by innerHTML handler are still executed, but the rewritable units are never rendered to the output if innerHTML handler modified underlying input chunk.

I think for the first use case we can do something like:
new HTMLRewriter().removeExcept('sel1').removeExcept('sel2').transform();

I think this would be less flexible than an e.print(true) rewrite action in an element handler: the action lets us, for example, filter all the
 tags in

foo

bar

qux

 with:
new HTMLRewriter({print: false})
  .on('div', { element(e) { e.print(true); } })
  .on('p', { element(e) { e.print(false); } })
  .transform(response)

I don't think such a thing is possible with the more declarative .removeExcept("div") API.

* if `innerHTML` handler modified underlying input chunk.

I'm confused about this part, wouldn't that make the rewriting non-deterministic? For example, let's say the innerHTML handler only modifies the first ever chunk it's invoked with. This will neuter the rewritable units for that chunk, which means that different mutations will be discarded depending on whether the handler received
 or
.

I don't think such a thing is possible with the more declarative .removeExcept("div") API.

removeExcept(':not(p)')
What I don't like about print thing is the combination of 2 flags which makes the API quite convoluted IMHO.

I'm confused about this part, wouldn't that make the rewriting non-deterministic? For example, let's say the innerHTML handler only modifies the first ever chunk it's invoked with. This will neuter the rewritable units for that chunk, which means that different mutations will be discarded depending on whether the handler received  or .

It will make the rewriting non-deterministic if your innerHTML handler is non-deterministic. And it's fine by us. With your example that will only happen if you perform mutations only on a first chunk, disregard its content.
Let's say you want to remove all the innerHTML. More likely your handler will remove on each chunk, so all underlying rewritable units will be affected.

removeExcept(':not(p)')
What I don't like about print thing is the combination of 2 flags which makes the API quite convoluted IMHO.

This would also remove all div tags in a p (I should have made that clear in my example sorry).

Let's say you want to remove all the innerHTML. More likely your handler will remove on each chunk, so all underlying rewritable units will be affected.

I see, thanks for the details.

Ok, so let's focus on the second case for now and create a separate issue for the first one and continue the API discussion there.

I created #78 for the second case instead, as most examples and comments here are about the first one.

I would really like to be able to extract specific elements from a response.
For example, for the following response

<element1/>
<element2/>

return the following:
<element1/>

Does anyone know if this is actually possible with workers? I'm relatively new to CF Workers & still not sure...


split workshop menu, fixes 386

oh have to fix a link from front page before merging this

ok should be merge ready now

Thanks, @wikfeldt ! I have limited skills to review the file changes in the browser (i.e., as it is to be shown in HTML), but one thing I am a bit afraid of is having separate pages for "upcoming" and "past" workshops make the "upcoming" page a bit empty. But let's try to look at it.

My change suggestions are not needed for correct rendering but I wanted to avoid that the dots proliferate when copy pasted in future. They are not needed.

But I guess my changes are lost now. I will send them as pull request.


Improve getNodeProperty by using conditional types

Looks like this:
export type TsNodePropertyNames<NodeType extends ts.Node> = {
    [KeyName in keyof NodeType]:
        KeyName extends "_tsSimpleAstBrand" | CompilerApiNodeBrandPropertyNamesType ? never : KeyName;
}[keyof NodeType];
export type TsNodeToWrappedType<NodeType extends ts.Node, KeyName extends TsNodePropertyNames<NodeType>, NonNullableNodeType = NonNullable<NodeType[KeyName]>> =
    NodeType[KeyName] extends ts.NodeArray<infer ArrayNodeType> ? Node<ArrayNodeType>[] :
    NodeType[KeyName] extends ts.Node ? Node<NodeType[KeyName]> :
    NonNullableNodeType extends ts.Node ? Node<NonNullableNodeType> | undefined :
    NodeType[KeyName];

But unfortunately only compiles in typescript@next due to bugs so I'll have to wait. Maybe it will work in the next 2.8.x version. In the meantime, I've checked this into the getNodeProperty branch.

Hello, Just curious, I've noticed that sometimes is hard to access the AST using getters that return Node|undefined. I wonder if this issue try to attack that ? For example, because ts-simple-ast uses methods and not properties sometimes I end up using local variables and && like the following snippet if tsc is configure with strict=true:
function importDeclaration2Structure(importDecl: ImportDeclaration): ImportDeclarationStructure {
  let defImport:Identifier | undefined, namedImport:  ImportSpecifier[], namespaceImport: Identifier | undefined,
  return {
    namedImports: (namedImport = importDecl.getNamedImports()) && namedImport.map(ni => ni.getText()),
    defaultImport: (defImport = importDecl.getDefaultImport()) && defImport.getText(),
    namespaceImport: (namespaceImport = importDecl.getNamespaceImport()) && namespaceImport.getText(),
    moduleSpecifier: importDecl.getModuleSpecifier().getText()
  }
}

@cancerberoSgx that's a different issue than this one.
I've often wondered about switching this library to using properties instead of methods, but it would probably be too large of a breaking change at this point.
So with properties you would be able to do:
defImport: importDecl.defaultImport && importDecl.defaultImport.getText()

With methods you can do:
defImport: importDecl.getDefaultImport() && importDecl.getDefaultImportOrThrow().getText()

Basically, once typescript supports the null propagation operator then our troubles will go away:
defImport: importDecl.getDefaultImport()?.getText()

Probably best to just wait for that.

Didn't knew that one, nice! Also , a helper (at least temporary for debuging) is using ! for example, without "!" the following fails in strict=true:
node.getExpression().getSymbol()!.getDeclarations()
my two cents

Yes, or by using getSymbolOrThrow().

I'm personally using orThrow variants so much I'd almost prefer them being the regular ones and the others orNull. :sweat_smile:

@lazarljubenovic it might be nice in some scenarios, but it's good for people to prompted about something possibly being possibly null and then from there decide whether they want to handle it being null or do something else like throwing an exception.


Wrong dashboard load after signinup up with SAML into an org

Steps to Reproduce

Signup into an organization using SAML

Current Result
After signing up via SAML, and account creation finishes, you are shown a corrupted dashboard:

All requests authenticated via cookie return 401.
After refreshing the page (F5), you get redirected through the login process like this:

And you finally get to the dashboard, correctly.
Expected result
You are correctly authenticated and the dashboard loads fine.
Browser and version
Tested Firefox and Chrome
Additional info
It seems like cookies are not set correctly upon account creation. I suspect a mixup of user and organization scopes.

Moved to TODO cc @saleiva @xavijam @jorgesancha

Confirmed that this is also happening for Google signups. Note that in staging/production the dashboard loads correctly, but you can still see the requests failing in the network console, until you refresh the dashboard.


refactor: workflow trigger

change pull_request to pull_request_target to enable comments to be
written

Contributes to: #60
Signed-off-by: Matthew Chirgwin<EMAIL_ADDRESS>Note: this will not trigger on this PR. This will take effect on all future PRs

Change looks good to me. Have you verified it works in your fork?

Indeed! https://github.com/matthew-chirgwin/strimzi-ui/pull/5/checks?check_run_id=1332944932 - with the pull_request_target in the repo, this PR, which changes the workflow, does not have the additional step run - meaning the workflow in master is being run, not the one in the PR


problems about sys.stdin.read() in ipython console

Problem Description
These days I am trying to write a remote shell with python. I used winpwn module, which based on threading and standard io. Now I want to improve these things and found a quesion:
sys.stdin.read() is unavaliable in IPython Console.
this problem might relate to another quesion:
Unsupported operation sys.stdout.fileno() in IPython Console
because of these questions, I cannot use standard io by os.read or os.write in IPython Console, which is meaningful to my continued work
Now I am using Spyder 5.1.5 | Python 3.9.7 64-bit | Qt 5.9.7 | PyQt5 5.9.2 | Windows 10 | IPython 7.29.0
I tried the same operations above in pycharm's IPython Console (7.29.0), and it all work in pycharm.
What steps reproduce the problem?

import sys and input "sys.stdin.read()"
input "sys.stdout.fileno()"

What is the expected output? What do you see instead?

it should be like input function but I got flash and no input.
it should return an integer about file descriptor (1 if stdout) according to pycharm but I got Exception.

Paste Traceback/Error Below (if applicable)
import sys

sys.stdin.read()
Out[2]: ''

In [3]: sys.stdout.fileno()
Traceback (most recent call last):

  File "C:\Users\10310\AppData\Local\Temp/ipykernel_24744/2810973561.py", line 1, in <module>
    sys.stdout.fileno()

  File "C:\Users\10310\anaconda3\lib\site-packages\ipykernel\iostream.py", line 308, in fileno
    raise io.UnsupportedOperation("fileno")

UnsupportedOperation: fileno

Versions

Spyder version: 5.1.5
Python version: 3.9.7
Qt version: 5.9.7
PyQt version: 5.9.2
Operating System name/version: Windows 10

Hi @DuanYuFi thank you for the feedback! I'm not totally sure what could be happening but I was able to reproduce this :+1:
Pinging @ccordoba12 and @impact27 (maybe they could have ideas about what is happening here)

I think this has to do with the ipykernel redefining stdin and stdout so they can be sent over the network. I therefore wouldn’t expect to have a fileno. I would recommend to run your code in python directly if you want to develop a console application, or to use the input function.

I would recommend to run your code in python directly if you want to develop a console application, or to use the input function.

I agree with @impact27. It's not possible to develop a console app using our IPython console. The only way would be going to the menu
Run > Configuration per file

and selecting there the option called
Execute in an external system terminal

I'm going to close this as "Won't fix" because there's nothing else we can do about this, sorry.

Thankyou guys for generousing helps! I will take deep considerations about them.

I would recommend to run your code in python directly if you want to develop a console application, or to use the input function.

I agree with @impact27. It's not possible to develop a console app using our IPython console. The only way would be going to the menu
Run > Configuration per file

and selecting there the option called
Execute in an external system terminal

Hi ccordoba12, it is a nice solution to my demand. Thanks!


bug: if one line is very wide then you can't scroll to the left side in desktop view

Problem/Need:

you need to be able to scroll to the left side, even when one line inside the <textarea> text is very long and imperfectly triggers overflow scrollbars (not a problem in mobile as it always wraps nicely)

Tried:

position: absolute; left: 0; seems to work but then prevents having the caret start in the middle in desktop view

Suggestions:

JS: detect when <textarea> text should wrap, and then turn on position: absolute; left: 0;
CSS: somehow get <textarea> text to wrap automatically

this doesn't seem to be a problem on Chrome, but it is on Firefox


Search Organization option in Fork Window

It's time consuming and hard to scroll and look for the organization in the Fork Window, it would be awesome if there was a search box where one could enter the starting characters of the organization name and matching organization or orgnaizations automatically get filtered and shown in the window.
Right now fork window is like this:

And one way of required fork window could be:

Wow 😲, how many organizations do you have?
There was design suggestion that could have included an searchbar.

I don't think most people have nearly that many organizations. I think you'll just have to ctrl+f


Added faster conversion method

This method also doesn't hang Emacs, and can take advantage of multiple cores.
It does add a dependency on fd (I am conspiring to get it installed everywhere, but we can drop it for find if you don't want another dependency.)
This also makes the setup a bit easier, as the user doesn't have to bother with typing in their $RUSTUP_HOME and long default target name.

Wow, that is super fast. Very impressive, and fixes the main issue I see with rustdoc at the moment.

It does add a dependency on fd (I am conspiring to get it installed everywhere, but we can drop it for find if you don't want another dependency.)

I'm fine with it. I also prefer fd and I expect most people who want to use rustdoc already have it installed.
The default Pandoc org writer inserts a lot of empty newlines. After the old conversion rustdoc-remove-whitespace was run to clean this up. We should keep that functionality or add similar to convert.sh.
I also would like to retain rustdoc-convert-current-package. That way users can easily convert documentation specific to their current project.
I am not sure about the save location for the org files. There is a slight advantage to keeping it categorized under toolchain, but it also makes rg slower while searching, and it is not obvious where one would put the package-specific documentation as per the previous point. It should continue to respect rustdoc-search-directory, or some other user-specified variable.
Many of the files in std are redirects, and there is no need to convert them. They were filtered out previously with
(< 10 (count-lines (point-min) ; redirect files are 10 lines long
                               (point-max)))

It's not a huge deal, but it would be nice to have similar functionality in the script.
In summary, the speed has increased massively but some functionality has been lost. Could we dump the script into rustdoc.el, and run it with start-process? This, along with some glue code, would restore most/all flexibility lost from switching from elisp to bash, make future packaging easier (End goal is that users should be able to install rustdoc through melpa), while still being insanely fast.
What do you think?

@samhedin
Adapting those functions to call the ./convert.sh script sounds like a very good idea, I'll do that.
Removing newlines should be easy to do with an awk/sed filter. With this excerpt from the converted Vec documentation, what newlines did the old solution remove?:

*** pub const fn new() -> Vec<T>

Constructs a new, empty =Vec<T>=.

The vector will not allocate until elements are pushed onto it.

#+BEGIN_SRC rust
let mut vec: Vec<i32> = Vec::new();
#+END_SRC

*** pub fn with_capacity(capacity: usize) -> Vec<T>

Constructs a new, empty =Vec<T>= with the specified capacity.

The vector will be able to hold exactly =capacity= elements without
reallocating. If =capacity= is 0, the vector will not allocate.

It is important to note that although the returned vector has the
/capacity/ specified, the vector will have a zero /length/. For an
explanation of the difference between length and capacity, see /Capacity
and reallocation/.

#+BEGIN_SRC rust
let mut vec = Vec::with_capacity(10);

// The vector contains no items, even though it has capacity for more
assert_eq!(vec.len(), 0);

// These are all done without reallocating...
for i in 0..10 {
    vec.push(i);
}

// ...but this may make the vector reallocate
vec.push(11);
#+END_SRC

*** pub fn into_raw_parts(self) -> (*mut T, usize, usize)

This is a nightly-only experimental API. (=vec_into_raw_parts= #65816)
new API

Decomposes a =Vec<T>= into its raw components.

Returns the raw pointer to the underlying data, the length of the vector
(in elements), and the allocated capacity of the data (in elements).
These are the same arguments in the same order as the arguments to
=from_raw_parts=.

After calling this function, the caller is responsible for the memory
previously managed by the =Vec=. The only way to do this is to convert
the raw pointer, length, and capacity back into a =Vec= with the
=from_raw_parts= function, allowing the destructor to perform the
cleanup.

#+BEGIN_SRC rust
#![feature(vec_into_raw_parts)]
let v: Vec<i32> = vec![-1, 0, 1];

let (ptr, len, cap) = v.into_raw_parts();

let rebuilt = unsafe {
    // We can now make changes to the components, such as
    // transmuting the raw pointer to a compatible type.
    let ptr = ptr as *mut u32;

    Vec::from_raw_parts(ptr, len, cap)
};
assert_eq!(rebuilt, [4294967295, 0, 1]);
#+END_SRC

I think we might want to keep those redirects, but instead turn them into symlinks, because we might eventually want to turn the  links into org links between the converted documents. That is, if those links are used internally, if they are solely there because of external references then they should be removed, but if that's the case then we can probably do the following:

Copy the doc-sets into something like ~/.local/share/emacs/rust-docsets/<crate>

IMO It's probably better to not pollute the main docset directory anyway

Remove all the redirects using something like rg -l "
Redirecting to <a href=\"[^\"]*\"" $path | xargs rm
Run the ./convert.sh script

Then we could probably bikeshed where to place the docsets, if we both agree they should be moved from $RUSTUP_HOME.

@samhedin This script runs in 6.1 seconds on my computer, so filtering the old way is definitely doable:
#!/usr/bin/env bash

DOC_PATH="$(dirname "$(rustup doc --std --path)")"

for file in $(fd . -ehtml "$DOC_PATH"); do
    # Use head first to avoid reading the whole file
    if [ $(head -n 11 "$file" | wc -l) != 10 ]; then
        echo "$file"
    fi
done

@samhedin rg -l "
Redirecting to <a href=\"[^\"]*\"" really seems to be the way to go, it only takes 108ms and should be way more reliable (but obviously not perfect.)

Adapting those functions to call the ./convert.sh script sounds like a very good idea, I'll do that.

Cool! This will really up the quality of rustdoc.

Removing newlines should be easy to do with an awk/sed filter. With this excerpt from the converted Vec documentation, what newlines did the old solution remove?:

The old filter removed all newlines except those followed by a header (defined as a line starting with *).

*** pub const fn new() -> Vec<T>
Constructs a new, empty =Vec<T>=.
The vector will not allocate until elements are pushed onto it.
#+BEGIN_SRC rust
let mut vec: Vec<i32> = Vec::new();
#+END_SRC

*** pub fn with_capacity(capacity: usize) -> Vec<T>
Constructs a new, empty =Vec<T>= with the specified capacity.
The vector will be able to hold exactly =capacity= elements without...

and so on. The end result is not set in stone. It was a quick cleanup because I found all the whitespace distracting and this was an easy way to make it more compact.

Copy the doc-sets into something like ~/.local/share/emacs/rust-docsets/

Do you mean the .HTML docs? No need to copy them or mix them with .org files.
The user should be able to specify a directory and then have output.sh place the converted org files in rustdoc-search-directory. The subdirectory structure can be kept by taking the relative file name.
I might be misunderstanding something here, so please clarify if I just said something strange.

rg -vl [...] really seems to be the way to go, it only takes 108ms and should be way more reliable (but obviously not perfect.)

Great! As far as changing the links we want to keep, it might be easier to do the conversion inside the pandoc filter, though I am not sure. Either way, that is probably a later issue.

@samhedin convert.sh is now a bit faster, as it will skip the redirects, thankfully I found the --ignore-file option of fd which made this much easier (and without adding anything to the run-time.)
It also means that it doesn't have to do a silly "copy, then remove" process.

@samhedin As for the newline issue, is it possible to handle this in the Lua filter?

@samhedin I think it would have to be handled by the Lua script, if at all, as the simple fix would crunch these longer paragraphs in such a way that they just become harder to read (this is subjective of course):
Original

Creates a =Vec<T>= directly from the raw components of another vector.

This is highly unsafe, due to the number of invariants that aren't
checked:

- =ptr= needs to have been previously allocated via =String=/=Vec<T>=
  (at least, it's highly likely to be incorrect if it wasn't).
- =T= needs to have the same size and alignment as what =ptr= was
  allocated with. (=T= having a less strict alignment is not sufficient,
  the alignment really needs to be equal to satsify the =dealloc=
  requirement that memory must be allocated and deallocated with the
  same layout.)
- =length= needs to be less than or equal to =capacity=.
- =capacity= needs to be the capacity that the pointer was allocated
  with.

Violating these may cause problems like corrupting the allocator's
internal data structures. For example it is *not* safe to build a
=Vec<u8>= from a pointer to a C =char= array with length =size_t=. It's
also not safe to build one from a =Vec<u16>= and its length, because the
allocator cares about the alignment, and these two types have different
alignments. The buffer was allocated with alignment 2 (for =u16=), but
after turning it into a =Vec<u8>= it'll be deallocated with alignment 1.

The ownership of =ptr= is effectively transferred to the =Vec<T>= which
may then deallocate, reallocate or change the contents of memory pointed
to by the pointer at will. Ensure that nothing else uses the pointer
after calling this function.

Crunched

Creates a =Vec<T>= directly from the raw components of another vector.
This is highly unsafe, due to the number of invariants that aren't
checked:
- =ptr= needs to have been previously allocated via =String=/=Vec<T>=
  (at least, it's highly likely to be incorrect if it wasn't).
- =T= needs to have the same size and alignment as what =ptr= was
  allocated with. (=T= having a less strict alignment is not sufficient,
  the alignment really needs to be equal to satsify the =dealloc=
  requirement that memory must be allocated and deallocated with the
  same layout.)
- =length= needs to be less than or equal to =capacity=.
- =capacity= needs to be the capacity that the pointer was allocated
  with.
Violating these may cause problems like corrupting the allocator's
internal data structures. For example it is *not* safe to build a
=Vec<u8>= from a pointer to a C =char= array with length =size_t=. It's
also not safe to build one from a =Vec<u16>= and its length, because the
allocator cares about the alignment, and these two types have different
alignments. The buffer was allocated with alignment 2 (for =u16=), but
after turning it into a =Vec<u8>= it'll be deallocated with alignment 1.
The ownership of =ptr= is effectively transferred to the =Vec<T>= which
may then deallocate, reallocate or change the contents of memory pointed
to by the pointer at will. Ensure that nothing else uses the pointer
after calling this function.

@samhedin Should rustdoc-to-org be based on manually searching, or should it also support getting information with lsp and then searching for that?
I'm thinking that most people writing mid-large size Rust projects in Emacs are probably using LSP, I'm thinking of hacking an org-mode display into lsp-describe-thing-at-point, but I'll attempt that in a different pull request. This work could then be merged into rustic-mode.

convert.sh is now a bit faster, as it will skip the redirects, thankfully I found the --ignore-file option of fd which made this much easier (and without adding anything to the run-time.)

Sick!

@samhedin As for the newline issue, is it possible to handle this in the Lua filter?

The filter function does not include the newlines, the issue arises when Pandoc takes the filtered data and converts it to org. Luckily, it's about to get much better: https://github.com/jgm/pandoc/issues/6554
Initially, I thought that this change in Pandoc would be nice but not sufficient, but I am starting to think that it might be just right.
We can leave this for now - going by the release frequency of Pandoc this change should be released soon enough.

With std being default for library, the default for target is whatever rustup show says it is.

This solution is convenient for converting std, but doesn't this make it impossible to convert package-specific documentation?
DOC_PATH="$RUSTUP_HOME/toolchains/$TARGET/share/doc/rust/html/$LIBRARY"

If it's just changed to take a full directory path (we can query the user for it in emacs to make it easy to enter), it can be used for anything.
DOC_PATH="$LIBRARY" # Something like this

An alternative solution is fine too, but it should be straightforward to generate any folder with documentation. When that's done, I think this is pretty much ready to be merged. There is some elisp to write to call and interact the script, but it should not be too bad. If you want to you are very welcome to do it, otherwise, I can take care of it.

Should rustdoc-to-org be based on manually searching, or should it also support getting information with lsp and then searching for that?
I started this project because I was using LSP, expected there to be thorough documentation when using lsp-describe-thing-at-point, and finding out that there wasn't (and this seems quite far down/nonexistent on the priority list of rust analyzer/rustdoc). So I'm all for integrating with LSP. I even looked at it a bit previously but decided to leave it for later.

EDIT: Or maybe merged into lsp-mode if made sufficiently generic, every language mode could be supplied with a custom pandoc filter to generate .org files.

Yeah, this would be super cool. This project does a similar thing (and was an inspiration for rustdoc), but is further along in development: https://github.com/danielmartin/swift-helpful

I believe it is possible to make a pure elisp solution almost as fast as the current one. I worked on an async one previously but abandoned it because it kept throwing a "too many filedescriptors open" error at me. Maybe this can be solved, in which case such a solution would be preferable.

@samhedin Yeah, the script is currently only for converting documentation in the system directories, as a test.
I'm going to write the elisp-code tomorrow, I'm thinking it should enumerate the libraries in the current cargo project (rustic-mode mostly has that figured out I think) and just start converting all of it. Now that the process doesn't take too long I don't know if people need fine-grained control over which specific libraries they want to convert. I'll cache everything in $HOME/.local/share/... so that users won't have to redo the conversion if they use the same library in many crates.
Later on there should probably also be an mtime check on the documentation and the org output to see if they need to be re-generated.

@samhedin I think it's pretty much done now.
The last thing to be done is to just change rustdoc-source-user to samhedin.

Super great job! Rustdoc is much better now.


Yaw/Roll toggle

Roll and Yaw being mapped to the same axis is a bit difficult to handle in certain flight situations, because rolling requires letting go of the left stick to hold the alt mode button.
It would be somewhat easier if the flight mode switch could optionally be set to toggle instead of having to hold it.
A more complicated solution would be if the roll mode button turned the grips into the two sides of the roll axis, allowing access to all flight controls at the same time (but preventing the use of tools)

Have you checked the Steamvr input bindings? You should be able to change the roll mode binding to a toggle there, or assign it to a more comfortable button.

SteamVR does not seem to do toggle controls. It was possible to set a "chord" so Roll Mode is activated, for example while both grips are pressed, but it did not seem possible to set a pressure threshold so it was somewhat unreliable
I tried to bind the roll axes directly, but since the game does not expose them the way it does with ThrustUp and ThrustDown, they can't be bound without going deeper than I was able to

It does do toggle controls for at least some of them. The Vive bindings use toggle controls for the grips, or at least they used to.


Formik Bindings

We mean to create formik bindings to all our form components so that they can be used seamlessly with formik.

 Formik Bindings

https://github.com/palmetto/palmetto-components/pull/52


Creation a Tag for the current master

Please create a Tag for the current master release to fix the composer issue. Currently works composer only with the dev-master branch
Thanks
Bjoern

Done.


ColorMatrix PreFX not working in Firefox on Linux (image goes black)

Version

Phaser Version: 3.87.0 (or lower)

Operating system: Ubuntu, Arch Linux, Steam OS

Browser: Firefox 132.0.2 (or lower)

Description

When using any ColorMatrix PreFX, the image goes completely black (or white, in some cases). I tried Ubuntu, Arch Linux and Steam OS (running on my Steam Deck). Doesn't happen on Android version of Firefox.
Example Test Code

Everything is apparent from this Labs example.
Additional Information
Screenshot from Ubuntu:

This is a known issue that we've resolved in the v4.0.0 renderer. In the meantime, please use PostFX and then upgrade when the time is right.


critical vulnerabilities

I'm sharing a list of critical vulnerabilities that are present in Label Studio right now. These come from a scan using OWASP Dependency Check (https://github.com/jeremylong/DependencyCheck). These vulnerabilities are publicly disclosed. Most of the high and critical vulnerabilities impacting Label Studio come from SQLite. These vulnerabilities are currently preventing me from using Label Studio inside my organization right now. I figured I'd report this information to help the developments.
/tmp/tmp-1299CwfySr4yfG4a/label_studio/core/utils/windows_sqlite_fix/sqlite-dll-win32-x86-3350100.zip/sqlite3.dll
CRITICAL
CVE-2017-10989
https://bugs.launchpad.net/ubuntu/+source/sqlite3/+bug/1700937
The getNodeSize function in ext/rtree/rtree.c in SQLite through 3.19.3, as used in GDAL and other products, mishandles undersized RTree blobs in a crafted database, leading to a heap-based buffer over-read or possibly unspecified other impact.
/tmp/tmp-1299CwfySr4yfG4a/label_studio/core/utils/windows_sqlite_fix/sqlite-dll-win32-x86-3350100.zip/sqlite3.dll
CRITICAL
CVE-2020-11656
https://www.oracle.com/security-alerts/cpuoct2020.html
In SQLite through 3.31.1, the ALTER TABLE implementation has a use-after-free, as demonstrated by an ORDER BY clause that belongs to a compound SELECT statement.
/tmp/tmp-1299CwfySr4yfG4a/label_studio/core/utils/windows_sqlite_fix/sqlite-dll-win32-x86-3350100.zip/sqlite3.dll
CRITICAL
CVE-2019-19646
https://security.netapp.com/advisory/ntap-20191223-0001/
pragma.c in SQLite through 3.30.1 mishandles NOT NULL in an integrity_check PRAGMA command in certain cases of generated columns.
/tmp/tmp-1299CwfySr4yfG4a/label_studio/core/utils/windows_sqlite_fix/sqlite-dll-win64-x64-3350000.zip/sqlite3.dll
CRITICAL
CVE-2019-19646
https://security.netapp.com/advisory/ntap-20191223-0001/
pragma.c in SQLite through 3.30.1 mishandles NOT NULL in an integrity_check PRAGMA command in certain cases of generated columns.
/tmp/tmp-1299CwfySr4yfG4a/label_studio/core/utils/windows_sqlite_fix/sqlite-dll-win64-x64-3350000.zip/sqlite3.dll
CRITICAL
CVE-2020-11656
https://www.oracle.com/security-alerts/cpuoct2020.html
In SQLite through 3.31.1, the ALTER TABLE implementation has a use-after-free, as demonstrated by an ORDER BY clause that belongs to a compound SELECT statement.
Environment:

OS: RHEL 7
Label Studio Version 1.0.1

These might be false positives considering that Label Studio 1.0.1 is using SQLite 3.35.2 on my server. This version is later than the concerned versions as described in the vulnerabilities. I'm looking more into it.

@dalekube This SQLite version will be used only if your environment doesn't have SQLite dll. And it will be prompt you with the confirmation about this usage.

Please forgive me for my lack of knowledge for SQLite. Does this mean that Label Studio will use the 'dll' if SQLite is not already installed via a package manager (e.g. apt in Ubuntu)?

Does this mean that Label Studio will use the 'dll' if SQLite is not already installed via a package manager (e.g. apt in Ubuntu)?

Yes, you are absolutely right! For ubuntu this dll is irrelevant, because there are .so files only.

What version of SQLite is the 'dll'? I assume it's earlier than 3.35 given the vulnerability scan is returning the above critical items.

Ok. I've updated it to 3.35.5:
https://github.com/heartexlabs/label-studio/pull/854/files

https://github.com/heartexlabs/label-studio/pull/854/files is in master, I hope this problem is solved.


chore: disable cra eslint

Description
Disable CRA built-in ESLint checks since we have our own config and a separate step for this
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] Jest
[ ] Cypress

Checklist:
Dev activity

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

Tested the bug scenario on my local setup with this fix and it is working now. The lint was previously not working for this commit.
LGTM


Use bootstrap-select for searchable user list organized by platform

Closes https://github.com/amyxzhang/policykit/issues/445
This is the same style select we use for the trigger selection on the policy editor.

Related issue https://github.com/amyxzhang/policykit/issues/515 is for separating users by platform on the dashboard user list

Beautiful, thank you!


[WIP] Created component for editing account settings

Working:

Displaying current user display_name
Users routes created with one route for updating display_name
The buttons for connecting/disconnecting from Github and ORCHID exist but are not currently working either

Issue to fix:

When closing the form without submitting but modifying the display_name, it'll keep the modification in the form until page get refreshed
When submitting, the form doesn't close and the new display_name need the page to be refreshed manually to be displayed correctly

Everything should be working now, to sum up:

Updated LoginDetails component: now using the store to update dynamically the user's display_name

Account settings is now enabled with the EditAccountSettingsModal component displaying:

Name = editable user's display_name (required)
Contact email = editable user's contact_email (can be None)
Link to user's github page || Login via Github
Link to user's orcid page || Login via ORCID (disabled)

New users routes in pydatalad to save_user (currently supporting updating display_name and contact_email only)

And the saveUser fetch in server_fetch_utils.js

For some reason all the e2e tests are failing here @BenjaminCharmes, would you be able to take a look when you get a chance before we merge?


CAS admin is able to sort/filter the list of registration submissions

Description:
As a CAS admin, I want to be able to filter/sort the list of pending operation (facility) registration submissions so that I can make sense of the information
Acceptance Criteria:
Given
When
Then
Development Checklist:

[ ] Checklist item
[ ] Checklist item
[ ] Checklist item
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

Probably overlap with with #97


glob pattern string required - version 2.0.0-beta.17

✗ Caught exception:
TypeError: glob pattern string required
at new Minimatch (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/glob/node_modules/minimatch/minimatch.js:108:11)
at setopts (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/glob/common.js:112:20)
at new Glob (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/glob/glob.js:117:3)
at FSWatcher.processEvent (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/index.js:92:35)
at emitTwo (events.js:87:13)
at FSWatcher.emit (events.js:172:7)
at FSWatcher. (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/chokidar/index.js:172:38)
at FSWatcher._emit (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/chokidar/index.js:212:5)
at FSWatcher._remove (/home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/chokidar/index.js:525:50)
at /home/mark/Projects/meds/node_modules/ionic/node_modules/ionic-app-lib/node_modules/gulp-watch/node_modules/chokidar/index.js:451:20
at FSReqWrap.oncomplete (fs.js:82:15)
Mind letting us know? https://github.com/driftyco/ionic-cli/issues

What does your ionic.config.js look like?

module.exports = {
proxies: null,
paths: {
html : {
src: ['app//*.html'],
dest: "www/build"
},
sass: {
src: ['app/theme/app.+(ios|md).scss'],
dest: 'www/build/css',
include: [
'node_modules/ionic-framework',
'node_modules/ionicons/dist/scss'
]
},
fonts: {
src: ['node_modules/ionic-framework/fonts//.+(ttf|woff|woff2)'],
dest: "www/build/fonts"
},
watch: {
sass: ['app/**/.scss'],
html: ['app//*.html'],
livereload: [
'www/build//.html',
'www/build/**/.js',
'www/build/**/*.css'
]
}
},
autoPrefixerOptions: {
browsers: [
'last 2 versions',
'iOS >= 7',
'Android >= 4',
'Explorer >= 10',
'ExplorerMobile >= 11'
],
cascade: false
},
// hooks execute before or after all project-related Ionic commands
// (so not for start, docs, but serve, run, etc.) and take in the arguments
// passed to the command as a parameter
//
// The format is 'before' or 'after' + commandName (uppercased)
// ex: beforeServe, afterRun, beforePrepare, etc.
hooks: {
beforeServe: function (argv) {
//console.log('beforeServe');
}
}
};
On 9 February 2016 at 21:18, Tim Lancina<EMAIL_ADDRESS>wrote:

What does your ionic.config.js look like?
—
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ionic-cli/issues/739#issuecomment-182071025.

Thanks, everything seems to be in order there.
It looks like: https://github.com/floatdrop/gulp-watch/issues/187, but I'm still unsure where or when it's happening.  Does that error throw when you have edited or deleted a specific file? It's hard to track it down just from that error alone.

It was when I deleted a file, but not sure I can reproduce.  Sorry.
On 10 February 2016 at 01:21, Tim Lancina<EMAIL_ADDRESS>wrote:

Thanks, everything seems to be in order there.
It looks like: floatdrop/gulp-watch#187
https://github.com/floatdrop/gulp-watch/issues/187, but I'm still
unsure where or when it's happening. Does that error throw when you have
edited or deleted a specific file? It's hard to track it down just from
that error alone.
—
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ionic-cli/issues/739#issuecomment-182158423.

I had this happen when I deleted a page from the starter whose Sass file was referenced by the App Shared Imports in app/theme/app.core.scss.  It looks it was likely triggered by the Sass builder encountering a missing path, but I can't recall exactly what steps I took to get the error.

I have this error in beta.10. Probably because I moved around a component between a few folders/directories. How do I fix it?

Take a look at app/theme/app.core.scss.  For me it was trying to import a  may be trying to import an SCSS file I had removed.


Prune the React Surface Area

Move internal-only components to an internal/ folder
Remove project-management components not used in ShoWhy, move to webapp
Remove vestigial components unused anywhere

In this case the per-hooks imports can look pretty ugly. By having a single hook export we can unify all of our lib-hook imports into a single import statement per component.
I'd be game for an internal.js file as an alternative that we can use for internal imports.

RenameCallout is used in the TableName component. TableCard and TableList have been removed

RenameCallout is used in the TableName component. TableCard and TableList have been removed

Yeah, but we won't be allowing that so all of the associated logic can disappear


include axiom solver in Houdini

Issue submitted via PipeBot
Reporting user: Isaac Criddle

Install and roll Axiom by Theory Accelerated into Dungini. should be simple enough.

Closed by #48


[Feature] Protobuf Encoding / Decoding per Subscription

Motivation
As mentioned in #555 protobuf encoding is now supported in MQTTX, which is great, but sadly one can only configure one message type for all subscriptions. Often it is necessary to subscribe to multiple topics, which all use a different protobuf message to en-/decode.
Detailed design
I would like to be able to set up a mapping from either subscription or topic to protobuf (or other schema) message both for sending as well as receiving. This means that the "Run Script" should either be available by subscription, or that a certain mapping from (wildcarded) topics to encodings is possible. The best case would be if it were possible to take different messages from different encoding files.
Alternatives
I have considered writing a script that would do this en-/decoding dynamically based on the topic, however, since the functions are not yet properly documented I was not able to easily do this. I furthermore think, that this would be an important feature to have, as writing a custom script is a lot more effort / higher barrier to entry for something that could be quite common to need (imo).

Hi @Neverbolt 👋
Thank you very much for your suggestions and feedback. We are currently planning improvements in the scripting use, and your suggestions are of great help to us. We plan to implement and optimize the script functionality for topics in the future. At the same time, we apologize for not being able to fully meet your requirements.
If possible, you could consider using MQTTX CLI as an alternative. It also supports protobuf, and you can use multiple commands to use different proto files for different topics. For example:
mqttx sub -t 'testtopic/1' -h broker.emqx.io -Pp ./TestPerson.proto -Pmn Person

mqttx sub -t 'testtopic/2' -h broker.emqx.io -Pp ./Message.proto -Pmn Message

We will continue to strive to enhance the functionality and user experience of MQTTX. Thanks again for your valuable suggestions.