Update README.md

Thanks for the fixes @fernandosanchezmunoz and one suggestion, going forward: if you phrase your commit messages a bit more specific it's later easier to understand what's going on. For example, instead of Update README.md (which there are many from in the repo), you might wanna use Fixes wording in Registry (since this gives the reader both the scope as well as how important the commit was).


interfaces: 6RD tracking with /64 prefix

I think I can correctly set up a 6rd tunnel (a /64):
root@net6501:~ # ifconfig wan_stf
wan_stf: flags=4041<UP,RUNNING,LINK2> metric 0 mtu 1280
	inet6 2001:b07:xx:yy:: prefixlen 64
	nd6 options=1<PERFORMNUD>
	v4net <IP_ADDRESS>/32 -> tv4br <IP_ADDRESS>
	groups: stf

root@net6501:~ # route -6 get default
   route to: default
destination: default
       mask: default
    gateway: 2001:b07:xx:yy::51d0:32d6
        fib: 0
  interface: wan_stf
      flags: <UP,GATEWAY,DONE,STATIC>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1280         1         0

root@net6501:~ # ping6 -c1 ipv6.google.com
PING6(56=40+8+8 bytes) 2001:b07:xx:yy:: --> 2a00:1450:4002:80a::200e
16 bytes from 2a00:1450:4002:80a::200e, icmp_seq=0 hlim=58 time=7.225 ms

--- ipv6.l.google.com ping6 statistics ---
1 packets transmitted, 1 packets received, 0.0% packet loss
round-trip min/avg/max/std-dev = 7.225/7.225/7.225/0.000 ms

But when I configure my LAN IPv6 to track the WAN interface (prefix id 0, I don't have other choice), the IPv6 configuration stops working: I can't ping ipv6.google.com anymore. I think this is happening because the default route gets overwritten (em1 is LAN):
root@net6501:~ # route -6 get default
   route to: default
destination: default
       mask: default
    gateway: 2001:b07:xx:yy::51d0:32d6
        fib: 0
  interface: em1
      flags: <UP,GATEWAY,DONE,STATIC>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1500         1         0

while I think I still should be getting interface: wan_stf here, as it was before. On the webui the Interface for the WAN_6RD gateway is still WAN (see attached screenshot). Is this an OPNsense issue, or am I misunderstanding something? Suggestions and feedback will be appreciated. Thank you!

It’s a reported 6rd limitation. We don’t know yet if this can be worked around.

On 22. Aug 2018, at 11:57, Paride Legovini<EMAIL_ADDRESS>wrote:
I think I can correctly set up a 6rd tunnel (a /64):
root@net6501:~ # ifconfig wan_stf
wan_stf: flags=4041<UP,RUNNING,LINK2> metric 0 mtu 1280
inet6 2001:b07:xx:yy:: prefixlen 64
nd6 options=1
v4net <IP_ADDRESS>/32 -> tv4br <IP_ADDRESS>
groups: stf
root@net6501:~ # route -6 get default
route to: default
destination: default
mask: default
gateway: 2001:b07:xx:yy::51d0:32d6
fib: 0
interface: wan_stf
flags: <UP,GATEWAY,DONE,STATIC>
recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
0         0         0         0      1280         1         0
root@net6501:~ # ping6 -c1 ipv6.google.com
PING6(56=40+8+8 bytes) 2001:b07:xx:yy:: --> 2a00:1450:4002:80a::200e
16 bytes from 2a00:1450:4002:80a::200e, icmp_seq=0 hlim=58 time=7.225 ms
--- ipv6.l.google.com ping6 statistics ---
1 packets transmitted, 1 packets received, 0.0% packet loss
round-trip min/avg/max/std-dev = 7.225/7.225/7.225/0.000 ms
But when I configure my LAN IPv6 to track the WAN interface (prefix id 0, I don't have other choice), the IPv6 configuration stops working: I can't ping ipv6.google.com anymore. I think this is happening because the default route gets overwritten (em0 is LAN):
root@net6501:~ # route -6 get default
route to: default
destination: default
mask: default
gateway: 2001:b07:xx:yy::51d0:32d6
fib: 0
interface: em1
flags: <UP,GATEWAY,DONE,STATIC>
recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
0         0         0         0      1500         1         0
while I think I still should be getting interface: wan_stf here, as it was before. On the webui the Interface for the WAN_6RD gateway is still WAN (see attached screenshot). Is this an OPNsense issue, or am I misunderstanding something? Suggestions and feedback will be appreciated. Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @fichtner. So does 6rd work properly on OPNsense only when more than a /64 is assigned?
The point is: it does work with the ISP provided router, so there has to be a way. I have tried to set up 6rd directly on the LAN interface (which is how I found out https://github.com/opnsense/core/issues/2661), but routes are not advertised in this way. Suggestions are welcome.

6rd was just recently brought back with 18.7 and probably suffers from a longer code slumber. It probably shouldn’t pull a WAN address as is configurable in DHCPv6 mode to be able to yield the prefix to LAN. I’ll look into it in September when I’m back at the office.

I don’t see code for stripping of the default route so this seems to be a side effect although I’m not sure where. Tracking on 6rd works differently than on „traditional“ dhcp6 and the code suggest tracking more than one address is not supported because the LAN address always ends up as xxxx::1. Still a bit of work ahead of us. 😊

Thanks for looking into it, @fichtner. I am very willing to test stuff and provide feedback, just ping me.

A code audit later it looks like 6rd has issues with setting up system routes in general. This will take a while, but working on it.

@paride hi! :) is this still the case on 18.7.3 ? do you have means to try the development version and see if that makes a change if 18.7.3 hasn't ? I'm flying mostly blind for 6RD unfortunately. Appreciate all feedback.

My 6RD stopped working with 18.7.2, but started working again with 18.7.3. Looked like route issue with 18.7.2.

@thomasjsn yup, that was https://github.com/opnsense/core/issues/2698

@fichtner I will try 18.7.3 tomorrow and let you know. I am fine with trying the development version if there's the need to. Thanks a lot for your work.

@paride super, thanks ❤

Bummer, ok... here we go...
Do you have aliases? If so, please make a copy (config.xml download) for your configuration via System: Configuration: Backups. You need this for restoring your aliases on 18.7.3 as the development version uses the newer alias system. If you don't have aliases, download a backup just to be safe.
Changing is easy: System: Firewall: Settings: Set "release type" to development. Save. Check for updates. Install updates. Since this affects 6RD please reboot as the interfaces for 6RD have been renamed to e.g. em0_stf.
The same transition for going back. Import your config.xml to retain your old aliases. Reboot due to 6RD interface name changing back to the old format.

@fichtner same problem with 19.1.a_227-i386 :(
(There is also another issue: the 6rd configuration only works if I configure it with the hack I described in https://github.com/opnsense/core/issues/2662#issue-352897137 and reboot. If I use the "6RD IPv4 Prefix address" option it doesn't work, despite the stf interface being configured in an almost identical way. But I think this is a different issue, and it is probably better to tackle one thing at a time...)

ok, thank you for confirming. let's go back to 18.7.3 and solve this ticket first.
What I know is that "tracking" injects an IPv6 into the tracking interface with ::1. My suspicion is that this also causes the routing table to behave differently, but it's very hard to see why. Maybe it thinks the adapter is in the same subnet and thus traffic is not forwarded to the correct interface anymore?

Is radvd running for you?
pgrep radvd

Maybe it thinks the adapter is in the same subnet and thus traffic is not forwarded to the correct interface anymore?

I think this is the case. If I manually remove the default route (route -6 del default), then add it again (route -6 add default <ip6>), and then I check how the route has been configured, I see that the chosen interface is the LAN interface, as if the route address belonged to its subnet.
If I set a default route with: route -6 add default -interface em0_stf (as in "far gateway"?) then it seems to work as expected. I still wasn't able to ping the ipv6 Internet from my LAN, but this may be a different issue.
Once I set up a tracking interface then radvd is started.

Ok, I have a suspicion... please use "Manual configuration" for your tracking LAN, go to Services: Router Advertisements: [LAN] setting "Router Advertisements" to "unmanaged" and "Advertise Default Gateway" to off. I have the feeling that WAN snatches a route from radvd pointing to the LAN interface but that would only work for clients in LAN, not the WAN interface on your box.

@fichtner done, but the gateway interface still changes to the LAN interface once the ipv6 tracking is set up.

Can you provide a full ifconfig + netstat -nr  ? ... can send over to<EMAIL_ADDRESS>for privacy if you want.

(in the misconfigured state and working state)

@fichtner done

@paride thanks, it moves the default route over to your LAN. how odd. Does issuing this command fix it?
# /usr/local/etc/rc.routing_configure

@fichtner no, it does not.
I think the point is the following: em1 has inet6 2001:b07:xx:yy::1 prefixlen 64, while em0_stf has inet6 2001:b07:xx:yy:: prefixlen 64. These are kind of the same subnet (2001:b07:xx:yy::/64), and the ipv6 gateway falls in both of these ranges. FreeBSD happens to set the gateway interface to one of these two interfaces that both have a matching subnet, and apparently it chooses em1 (because it is the last one created)? But I may be wrong of course.

can you give all the "ROUTING" entries from the system log for when that script was issued. If it's what you say we set it correctly but the system decides otherwise oO

Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: entering configure using defaults
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: IPv6 default gateway set to wan
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: no IPv4 default gateway set, assuming wan
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: setting IPv4 default route to <IP_ADDRESS>
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: keeping current default gateway '<IP_ADDRESS>'
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: setting IPv6 default route to 2001:b07:xx:yy::<IP_ADDRESS>
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: removing /tmp/em0_stf_defaultgwv6
Sep 24 16:09:47 net6501 opnsense-devel: /usr/local/etc/rc.routing_configure: ROUTING: creating /tmp/em0_stf_defaultgwv6 using '2001:b07:xx:yy::<IP_ADDRESS>'

is there any error  in the raw log after "setting IPv6 default route to" ?
This will dump all entries if the GUI doesn't show it:
# clog /var/log/system.log

@fichtner No, the rc.routing_configure scripts generated only those ROUTING lines in the system log, nothing else.

That would also explain why you're the only one reporting... others have larger prefixes so that's not an issue... makes sense? :)

Makes sense and setting prefixlen to /128 does work! The gateway interface stays set to em0_stf now. Still this is not truly a solution, right?
I did try to add the default ipv6 gateway using route the -ipf modified (route -6 add default -ifp em0_stf ), but the gateway interface is set to em0` in this case...

Actually it is. I'd prefer 128. It means it's a host-only address. Transit interface for LAN only. How are your clients in this scenario? http://test-ipv6.com/ and http://ipv6-test.com/ are good tests.
Besides, there's precedent in GIF and GRE tunnels which are very similar to 6RD: https://github.com/opnsense/core/blob/772ea4ace1c76543d75eb93dfc5fc7ac68697cc2/src/etc/inc/interfaces.inc#L849
I know that IPv6 implementations refuse to accept forced gateway interfaces and produce strange errors in the result.

@fichtner If I set prefixlen to something different from 64 (I tried: /128, /63, /96) my clients do not get autoconfigured anymore...

# clog /var/log/routing.log | grep radvd

Is radvd starting when we have the wrong prefix? There is a Base6Interface parameter in radvd that we could use to follow the real interface maybe.

@fichtner it does start but complains: Sep 24 16:57:55 net6501 radvd[64234]: prefix length should be 64 for em1.
@fichtner I'm sorry this turned out more convoluted than expected, but hopefully I won't be the only OPNsense user to benefit of this work.

I appreciate the help in getting to the bottom of this. Already learned a few things along the way. Give me a day to propose a test patch. :)

@paride alright, 49a498195 is a radically different approach to try. Run
# opnsense-patch 49a498195

to try (should also reboot) and patch again to remove.

@fichtner will do; does it apply on 18.7.4 or to the development version? (I still have to revert to the stable version...).

I used dev, but both should work

@fichtner seems to behave as before: the interface of the default gateway is set to LAN once I configure LAN to track WAN. Don't we have the same prefixlen = 64 problem?

The issue is the same of course, but I was hoping to side-step it through this... but that would further point to "it's impossible" with only a /64 from the ISP. I have no other ideas. :(

@fichtner well, the ISP router somehow manages to do it, there must be a way. Can't be force the inet6 address assigned to LAN to have a 128bit prefix? I don't see it specified in the patch.

I was hoping to get rid of the other code, but let's try the hybrid solution then.

November has been extremely busy, sorry. Bumping this to feature.

19.1 will not allow tracking from LAN so we hope to get more people involved to find a suitable solution.

I think I'm having the same issue.
ifconfig: ```
wan_stf: flags=4041<UP,RUNNING,LINK2> metric 0 mtu 1480
inet6 xxxx:xxxx:xxxx:xxxx:: prefixlen 64
If I set track interface on LAN, I can't ping ipv6.google.com from the opnsense.

OPNsense 19.1.5_1-amd64
FreeBSD 11.2-RELEASE-p9-HBSD
OpenSSL 1.0.2r 26 Feb 2019

I'm dropping this issue. With a "dead end" prefix of /64 we won't be able to anything clean and since 19.1 the system refuses to allow tracking.


Feature Request: Drawer

The drawer is a complex component commonly used in mobile terminals. If 'park-ui' implements it, we would no longer need to make an effort to integrate third-party libraries.
links:

https://www.solid-ui.com/docs/components/drawer
https://corvu.dev/docs/primitives/drawer/

https://park-ui.com/docs/panda/components/drawer

https://park-ui.com/docs/panda/components/drawer

just a dialog

Not sure why you gave @dannylin108 a thumbs down, he is correct. You can simply add a variant to the drawer that changes its position to the bottom of the screen, such as what my team implemented here: https://github.com/animareflection/ui/blob/master/src/components/core/Drawer/Drawer.recipe.ts#L107-L120
Demo (Storybook)

Not sure why you gave @dannylin108 a thumbs down, he is correct. You can simply add a variant to the drawer that changes its position to the bottom of the screen, such as what my team implemented here: https://github.com/animareflection/ui/blob/master/src/components/core/Drawer/Drawer.recipe.ts#L107-L120
Demo (Storybook)

These are two entirely different things, merely sharing the same name. You can refer to https://corvu.dev/docs/primitives/drawer/ for more details; it represents a mobile-native design. It is not a simple dialog.

Thank you for removing the thumbs down, I did see that but it appears to be the same thing except adding draggability (which is easy to implement), even rendering as a div just the same. I understand you seem to be asking about not using other libraries for dragging though, it was just unclear from the OP

For the primary UI, the crucial aspects are anatomy and state machines part, while templates are not of significant importance. Otherwise, @corvu/drawer would not have so much code, even though it is built upon @corvu/dialog.


Missing huggingface from setup.py

The topline model needs huggingface and it seems like that wasn't kept in the setup.py

Hi @siddalmia , I guess these changes have been included in #7, so I close this for now. Please feel free to reopen it.


Add "Show Password" Feature to Login Page

Here’s an example of an issue for adding a Show Password Feature on the login page:
Title: Add "Show Password" Feature to Login Page
Description
The login page currently requires users to enter their password without the option to view it. To improve user experience and reduce typing errors, we need to implement a "Show Password" feature that allows users to toggle the visibility of their password.
Acceptance Criteria
UI Design:
Add a clickable "eye" icon (or similar) inside the password input field that allows users to toggle between showing and hiding the password.
The icon should change state depending on whether the password is visible or hidden.
Functionality:
When the "eye" icon is clicked, the input type should change between "password" and "text" to either hide or show the password.
Ensure that the password visibility toggle works smoothly and does not affect form validation.

can you assign it to me?


Build fail with official caddy builder dockerfile

Hi !
Since the commit of yesterday the build based on the official docker builder fails. The error is quite explicit :
<EMAIL_ADDRESS>requires go >= 1.22.0 (running go 1.21.10; GOTOOLCHAIN=local)
It would be nice to make version tags in the git repository. Currently it is not easy to fallback to the previous version.
Here is a dockerfile for complete reproduction :
FROM caddy:2-builder AS builder

RUN xcaddy build \
    --with github.com/greenpau/caddy-security \
    --with github.com/mholt/caddy-ratelimit

FROM caddy:2-alpine

COPY --from=builder /usr/bin/caddy /usr/bin/caddy

PS : It's a nice occasion to say that using Caddy and all this plugins is really great and thank all the maintainers and developers.

As a temporary solution, one can use the image instead.
FROM caddy:2.8.0-beta.2-builder AS builder

Hello, same issue here as of today. Upvoting. (The temporary solution is also working for me).

@sebMathieu @Matasx Don't forget to set the environment variable CADDY_VERSION=latest or replace xcaddy build with xcaddy build latest if you don't want to run the beta version v2.8.0-beta.2.

Should be fixed on master now


Constant propagation computes wrong values

Given the following test file, computing the constant values for x using scalpel results into a dictionary with three key-value pairs.
x = "a"
x = "b"
x = 1

with open("test_file.py", "r", encoding="utf-8") as source:
    code_str = source.read()

cfg = CFGBuilder().build_from_src(name="", src=code_str)

_, const_dict = SSA().compute_SSA(cfg)

print(const_dict)

{('x', 0): <ast.Constant object at 0x0000021BC9EFA430>, ('x', 1): <ast.Constant object at 0x0000021BC9EFA430>, ('x', 2): <ast.Constant object at 0x0000021BC9EFA430>}

That is what I expect. However, parsing the ast.Constant objects to get the actual values does not result into the correct values. Instead, I get the same value three times.
 for key, value in const_dict.items():
        print(key, ast.unparse(value))

('x', 0) 1
('x', 1) 1
('x', 2) 1

This is not what I expect. Do you agree or am I missing something here?

It looks like a edge case. thanks for providing this test case

Thanks for raising this. The problem has now been fixed. It's indeed an edge case.


Fix type conversion for int value

Fix #158
Signed-off-by: Frank Jogeleit<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.15%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2736

Relevant Lines:
2906

💛  - Coveralls


Tags out of order with includeControllerXmlComments

Hi
I am facing same issue related to #918
I use bellow packages
<PackageReference Include="Swashbuckle.AspNetCore" Version="5.6.3" />
   <PackageReference Include="Swashbuckle.AspNetCore.Annotations" Version="5.6.3" />
   <PackageReference Include="Swashbuckle.AspNetCore.Filters" Version="6.0.1" />
   <PackageReference Include="Swashbuckle.AspNetCore.Newtonsoft" Version="5.6.3" />
   <PackageReference Include="Swashbuckle.AspNetCore.ReDoc" Version="5.6.3" />

and facing same ordering issue ,not fix with bellow code
// TagReOrderDocumentFilter.cs
public class TagReOrderDocumentFilter : IDocumentFilter
{
    public void Apply(SwaggerDocument swaggerDoc, DocumentFilterContext context)
    {
        swaggerDoc.Tags = swaggerDoc.Tags
            .OrderBy(tag => tag.Name)
            .ToList();
    }
}

// Startup.cs
options.IncludeXmlComments(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "Test.Web.Host.XML"), true);
options.IncludeXmlComments(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "Test.Application.XML"), true);
options.DocumentFilter<TagReOrderDocumentFilter>();

@domaindrivendev  as you comment here

I believe the following commit should resolve this - 38dc191 But, you'll need to wait for and upgrade to v5.0.0 to avail of it.

I use version above 5 ,but still the issue is there.
Please let me know if I am doing any thing wrong.

Please create a minimal app (i.e. starting from an empty project) that repro's this issue and post it to github so I can pull down and troubleshoot.

Please create a minimal app (i.e. starting from an empty project) that repro's this issue and post it to github so I can pull down and troubleshoot.

Closing due to inactivity

I see the same issue.  The json file itself is in the expected order, but the swagger UI is changing the sort order when xml comments are included, such as putting paths without comments as the bottom.

I can also confirm that this is still an issue. I have a special sort key for ordering controllers out of alphabetical order.  Without includeControllerXmlComments: true I get the correct controller ordered correctly... Tokens, and then Authorization.
{     "openapi": "3.0.1",     "info": {         "title": "Test Web API",         "version": "0.1.0"     },     "paths": {         "/Tokens/GenerateAdministrativeUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates an administrative user token.",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Tokens/GenerateManagementAdministratorUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates a management user with administrative privileges token.",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Tokens/GenerateManagementUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates a management user token.",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Tokens/GenerateUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates a user token (with no groups).",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Authorization/Test": {             "get": {                 "tags": [                     "Authorization"                 ],                 "summary": "",                 "description": "\r\n\r\nAuthorization requires  the following policy: Manager",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         }     },     "components": {         "securitySchemes": {             "Bearer": {                 "type": "apiKey",                 "description": "JWT Authorization header using the Bearer scheme. Example: \"Authorization: Bearer {token}\"  A token can be acquired using /authentication/authenticate.",                 "name": "Authorization",                 "in": "header"             }         }     },     "security": [         {             "Bearer": []         }     ] }
With includeControllerXmlComments: true I get the incorrect controller ordered based on the tag order... Authorization, and then Tokens.
{     "openapi": "3.0.1",     "info": {         "title": "Test Web API",         "version": "0.1.0"     },     "paths": {         "/Tokens/GenerateAdministrativeUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates an administrative user token.",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Tokens/GenerateManagementAdministratorUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates a management user with administrative privileges token.",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Tokens/GenerateManagementUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates a management user token.",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Tokens/GenerateUserToken": {             "get": {                 "tags": [                     "Tokens"                 ],                 "summary": "Generates a user token (with no groups).",                 "description": "\r\n\r\nAuthentication/authorization is not required.",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         },         "/Authorization/Test": {             "get": {                 "tags": [                     "Authorization"                 ],                 "summary": "",                 "description": "\r\n\r\nAuthorization requires  the following policy: Manager",                 "responses": {                     "200": {                         "description": "Success"                     }                 }             }         }     },     "components": {         "securitySchemes": {             "Bearer": {                 "type": "apiKey",                 "description": "JWT Authorization header using the Bearer scheme. Example: \"Authorization: Bearer {token}\"  A token can be acquired using /authentication/authenticate.",                 "name": "Authorization",                 "in": "header"             }         }     },     "security": [         {             "Bearer": []         }     ],     "tags": [         {             "name": "Authorization",             "description": "The authorization test API calls."         },         {             "name": "Tokens",             "description": "Generate tokens API calls."         }     ] }

when includeControllerXmlComments: true   we can see Tags being generated in the json file.
When this Tags Property is present, the controllers are ordered by this tags.
BUT in the code "src\Swashbuckle.AspNetCore.SwaggerGen\XmlComments\XmlCommentsDocumentFilter.cs"
Tags are created only for controllers that have description added to them.

i am experiencing the same thing. having the docs is great for codegen but now the swagger UI order is out of whack. any way to have the docs and keep alphabetical sort by path?


Operational plugin: application info

Coverage increased (+0.03%) to 99.52% when pulling 31ba0ae50e47ee218122dc7a81fcebf3330a393a on 43-Operational-Plugin-Application-info into cb7ebd13d176527c0a39901b84596971022bb53f on release-candidate.


Make design system for Store Buttons

Make and awesome design system for Store Buttons Component.
See the Store Buttons and make one.
Refer to : Button Design System for any reference
Tip :
You can use Tailwind Playground

Hey for now I am closing this issue. What you can do is raise and issue to fix the Store buttons and work it. @itz-deepak
Then according to that we can make a design system. Make sure to show me once what you have done and after checking you can make the design system and make all the other buttons as well. @itz-deepak
For better communication you can use the discord that would be better I guess.


Inkscape opening over tiled windows

In the blog, when Castel uses ctr + f an inkscape window opens up on top of both the tilted terminal and zathura. This is seen from here https://castel.dev/a48c6ef9b7407f13e4e0e86b2649333f/including_figure.gif.
On a closer look, it looks as if desktop environment number is changed.
When I try execute ctr + f however, I get third window in the sense that my screen is split into 3 things - the terminal, zathura, and inkscape.
Does anyone know how I can get the behavior in the gif?
PS: I am using bspwm as I think Castel uses it too?

Add this to your bspwmrc file:
bspc rule -a Inkscape desktop='7'


Release v3.0.0

Just the CHANGELOG.md file?

Just the CHANGELOG.md file?

Yep, it's just indicating that all of the currently unreleased changes are now going into a release. Nice simple diff!


Fix new gosec G112 Potential Slowloris Attack issues

While updating golangci-lint (https://github.com/submariner-io/shipyard/pull/899), we're seeing new gosec failures in various projects.

Consuming Projects / Lint (coastguard)

pkg/healthz/healthz.go:36:16: G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
		httpServer: &http.Server{Addr: address},
		             ^

Consuming Projects / Lint (lighthouse)

pkg/agent/main.go:151:10: G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
	srv := &http.Server{Addr: ":8082"}

Consuming Projects / Lint (submariner)

pkg/globalnet/main.go:145:10: G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
	srv := &http.Server{Addr: ":8081"}
	        ^
main.go:321:10: G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)
	srv := &http.Server{Addr: ":8080"}
	        ^

Consuming Projects / Lint (subctl)

internal/restconfig/restconfig.go:375:4: error is not nil (line 371) but it returns nil (nilerr)
			return nil
			^
pkg/broker/ensure.go:177:4: error is not nil (line 174) but it returns nil (nilerr)
			return false, nil
			^

Interesting context: https://medium.com/a-journey-with-go/go-understand-and-mitigate-slowloris-attack-711c1b1403f6

There's very little information out there about this. The golangci-lint PR/golangci-lint Issue linked to the same blog post as above for justification/explanation. There's also recent discussion about ReadTimeout being used if ReadHeaderTimeout isn't set.

This blog is the the most recent/relevant docs about proper Go net/http timeouts I've been able to find: https://ieftimov.com/posts/make-resilient-golang-net-http-servers-using-timeouts-deadlines-context-cancellation/
Seems like there's a lot to consider, I'm not sure it's as simple as adding a ReadHeaderTimeout. Any guidance @tpantelis @skitt @sridhargaddam?

Seems like there's a lot to consider, I'm not sure it's as simple as adding a ReadHeaderTimeout. Any guidance @tpantelis

Discussing on slack, I think we should start by addressing this specific security issue and let the tooling inform us going forward. Perhaps we can optimize performance by following the guidance in that article you posted but it's unclear what usage patterns these http servers will experience to warrant possible premature optimization. That's my take.


Fix UnicodeDammit support (bytes -> unicode)

parse() in common.py shows that UnicodeDammit can be used to send bytes to boilerpipy. Of course lxml cannot do tostring() on unicode because it does not know what encoding to choose, so I fixed that too.

Thank you for the quick merges @harshavardhana.

Thank you for the quick merges @harshavardhana.

Thank you for your changes.


add CCNet

This PR adds the initial files for the corpus callosum segmentation module.

added CCNet directory, containing the complete module (with duplicate files from FastSurferCNN)
edited run_fastsurfer.sh to be able to run the CCNet

TODO:

Upload checkpoints and add to download checkpoint script
The module is based on another fork that has not been merged into dev yet. It needs to be merged, before this module can be comitted. Afterwards, the CCNet can be cut down, to the necessary  files to avoid redundancy.
The CCNet currently uses the norm.mgz registered to upright space. Changes in the run_fastsurfer.sh could adress this constrain

Please remove duplicate files first to avoid inflating our repository. Most of them are probably not even needed by CCNet?
Also merge current dev into your branch via a re-base, resolve conflicts and force push to your repo.
Thanks.

TODO:

[ ] Merge FSTumor branch into dev and rebase
[ ] Strip legacy FSTumor code from CCNet structures
[ ] Remove repeated methods, that were modified by the FSTumor module
[ ] Add checkpoints  download from websites functionality
[ ] Modify FastSurfer pipeline to create aligned images using asegs and use those as input for the CCNet
[ ] Add (Soft-) label mapping of CCNet segmentation into unaligned space
[ ] Post-processing cleanup of CC-oversegmentation
[ ] Retrain localization to use continuous distributions

Please remove data files (hdf5 files) from the commit and then squash your commits, we do not need the history to be "polluted" by really large data files.


Error: The argument type 'String' can't be assigned to the parameter type 'Uri'.

`/C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/vin_decoder-0.1.3/lib/src/nhtsa_model.dart:89:37: Error: The argument type 'String' can't be assigned to the parameter type 'Uri'.

'Uri' is from 'dart:core'.
final response = await http.get(path);`
pls update

Thanks, this was already fixed in the 0.2.0-nullsafety version. Could you please try with this?

Ok, I have now cut new stable and pre-releases. Please use 0.1.4+1 if you require a stable release, or 0.2.1-nullsafety if you plan to have null safety enabled in your application.


Plugin Not Working.

Hello,
This plugin is not working on all android phones. It is working fine in almost all samsung phones, but in one plus one it is behaving abnormal i.e. it is working at times and freezes at times.
Please let me know if you have any solution for this

It is not working on my Android phone...

Hey there,
You can do one thing, you can use timeout functions and within timeouts you can write orientation change code. It hepled me, let me know if you have any issues.

@Naseem16, Hey can you help me out. It's showing video can not play. Please let me know.

Can you be more specific please.
It will be better if you can share the code snippet

It opens youtubeplayer but a alert message comes 'Can't play this video'. That's it.


MacOS Cosmetic issues thread - latest release

I saw some issues with UI, I think they're not difficult to fix and make our product as perfect as possible.
Screenshots

My Environment

Verge Release: VERGE-Qt-4.0.1.dmg
Mac pro MF840.
OS:

[x] I'm using Mac High Serria 10.13.2

posting @Positivism 's screenshots here

Can anyone confirm these issues still exist on the latest release?  Both of these issues were posted prior to latest.

can anyone confirm if these are issues in the latest release?  both of your posts were made early January, before the latest release.
@Positivism
@quangnd

Still there, we're also getting 'out of sync' even if it's fully sync'd.

Are we there yet?  Are we there yet?  4.0.2 ?  :smile:

Out of sync gone, rest still there.


Remove and replace all "opencv" usage - remove build dependency

Overview
The use of opencv has become a liability for DSL, for a number of reasons, and needs to be replaced. After a great deal of investigation, it appears that FFmpeg will provide a much lighter/better solution for what DSL needs.
This will require the installation of four new libraries that are core to FFmpeg
sudo apt update && sudo apt-get install \
    libavformat-dev \
    libavcodec-dev \
    libavutil-dev \
    libswscale-dev \

Current opencv usage
... and how it will be replaced. Note: the current implementation is CPU based. The FFmpeg implementation will be as well, however, there may be an opportunity to move to a hardware accelerated solution with FFmpeg moving forward.
Usage 1. To obtain Input source file attributes.

Used by the URI, File, and Image Sources to determine the dimensions and frame-rate of an input source file.
This will now be done using libavformat.

Usage 2. To capture/encode frames and objects to JPEG
... this includes
Packing an RGBA NvBufferSurface

Each memory-aligned image is copied/packed into a contiguous buffer.
The packing (removing of the padding) will now be done directly in DSL.

Converting the RGBA buffer to BGR

Conversion to a format without an alpha channel before encoding to JPEG
The RGBA buffer will now be converted to YUVJ420P (required mjpeg codec format) using libavutil and libswscale.

Optionally annotate each image, Frame Capture only

Each Image Frame is (optionally) annotated with bounding boxes and labels.
This is the one item that can not be easily replaced and will be omitted - at least for the next release

Encoding the BGR buffer to JPEG

Opencv was used to encode the BGR buffer (as well as the packing and converting defined above)
The new YUVJ420P will be encoded using the installed MJPEG codec from libavcodec.

Note: the above steps were previously done in the context of the Pad-Probe-Handler... which was not ideal. The new solution will execute these tasks in a g_idle_thread callback.
New Artifacts
A new AvFile utility module will be created to implement the above requirements while abstracting the use of FFmpeg. This should simplify any future consideration/implementation if need be.

Complete and merged into v0.26.alpha


option to have array of extra data template on the slider? (instead of badges)

HI,
really loving these custom sliders!
Ive made some extra badges on the light to show temperature and brightness. But ive only been abel to show 1 value in grey on the slider itself, just below the name:

Would it be possible to show more than 1 value there? The badges might be a bit too much, but i would  love to be able to see those values, and believe the small grey ones on the slider would be perfect.
Ive tried something like this, but it wont work obviously, showing only the brightness =B attribute:
light.bureau:
  extra_data_template: >-
    - if (attributes.brightness) return attributes.brightness + 'B'; return null;
    - if (attributes.color_temp) return attributes.color_temp + 'C'; return null;
  state_card_mode: break-slider
  stretch_slider: true
  hide_control: false
  show_last_changed: false
  theme: happy
  #confirm_controls_show_lock: true
  slider_theme:
    min: 0
    max: 255
    pin: true
    off_when_min: false
    report_when_not_changed: false
  extra_badge:
    - attribute: brightness
    - attribute: color_temp

----- edit --------
wishlist part 2:
if we could have more than one slider, things would get even better. Im imagining a setup where I wouldn't have to click the light to go to the sliders and change them. Have the custom-ui allow for a multiple slider setup would not force us to do so, and have all controls needed and wanted in 1 majestic view.....
I know, I am asking a lot, but really really hope this would be an option for the development of Custom-ui.
----- end edit -----
Cheers,
Marius

Multiple extra_data_template support was added in 20171227.
Multiple sliders don't scale - the line breaking code is too complex as-is.


Update pod.js

Dont cast to string when a tag is not defined.

@keithballdotnet thanks for your contribution!


Moved Installation.md and fixed Travis badge in README.md

Pull Request Test Coverage Report for Build 9

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.257%

Totals

Change from base Build 7:
0.0%

Covered Lines:
119

Relevant Lines:
167

💛  - Coveralls


Allow passing flags to underlying kubectl where applicable

I have multiple k8s clusters, (dev and prod) which I operate from the same jump host.  To avoid accidental applications to one cluster,  I don't put kubeconfigs in the standard location, but rather apply kubectl --kubeconfig=path/to/kubeconfig.   For verbs like pachctl port-forward, it uses kubectl without flags.
If one could do pachctl --kubeconfig=/path/to/kubeconfig  or pachctl --kubectlflags='--kubeconfig=...'  this would be helpful.

pachctl port-forward is actually only a thin wrapper around kubectl port-forward and it can be used directly for use cases such as this.

Yeah, all pachctl port-forward does is find a pachd pod and do kubectl port-forward <pod>

Fixed in #1441


Very large shields (badges) on the docs page

In the docs/berty site the shields are very big, taking up almost the entire screen.

The link is wrong, it links to this issue, but you are right!

The link is wrong, it links to this issue, but you are right!

Thanks, I've changed the link now :), I also send a pull request where i fixed it, but it has not yet been reviewed :)

Thanks for your contribution @LFGaming ;)


feat: +Reverse engineering related logic

Features

Based on the source code, reverse engineer class diagrams, sequence diagrams, and use cases.
There is still room for improvement in the logic for generating sequence diagrams of classes.
The logic for symbol extraction only covers common Python usages, and there is still room for improvement.

Codecov Report
Attention: 503 lines in your changes are missing coverage. Please review.

Comparison is base (e8c333b) 85.28% compared to head (34225d0) 36.44%.

Files
Patch %
Lines

metagpt/actions/rebuild_sequence_view.py
15.91%
206 Missing :warning:

metagpt/repo_parser.py
32.14%
133 Missing :warning:

metagpt/utils/visual_graph_repo.py
36.26%
58 Missing :warning:

metagpt/actions/rebuild_class_view.py
22.72%
34 Missing :warning:

metagpt/schema.py
31.57%
26 Missing :warning:

metagpt/utils/graph_repository.py
43.58%
22 Missing :warning:

metagpt/utils/common.py
28.57%
15 Missing :warning:

metagpt/utils/di_graph_repository.py
27.27%
8 Missing :warning:

metagpt/utils/project_repo.py
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              dev     #800       +/-   ##
===========================================
- Coverage   85.28%   36.44%   -48.84%
===========================================
  Files         186      186
  Lines        9871    10368      +497
===========================================
- Hits         8418     3779     -4639
- Misses       1453     6589     +5136

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

should tests/data/graph_db/networkx.json be ignored?
Yes. networkx.class_view.json and networkx.sequence_view.json are currently used in unit tests and will be used later for the further development of reverse engineering.

could you provide some readme or comments for the files?
Yes. Google style doc strings have been added already.

how to use the results as context, any design here?
All symbols and their extracted information are stored in a graph database. You can use the existing information to search within this database and see what information is available and usable.

could we use vector search to search the symbols?
Yes. Various data storage engines can be implemented by inheriting the GraphRepository class. The reason for using a graph database is that it can effectively manage refined results of the same data across different dimensions. If you prefer not to use a graph database, you can also read out the graph repository data and write it into the corresponding data management engine.


Need admin/'root' permissions/privileges to install pommerman

It appears that we need admin privilege on Windows and equivalents on Linux systems to install pommerman with:
pip install .
even though we are installing in a virtual environment or a conda environment.
It is not a problem on my own machine, but on clusters where I do not have such privileges, I cannot install using the command above. A workaround is
python setup.py install
but I need to uninstall pommerman before reinstalling to update the env version with my changes, because otherwise, it won't update.

@maym2104 I just did a fresh install in a new environment and did not come across this permissions issue. Can you give a full trace of the error? Including steps that led you to this?

Ok turns out I could not reproduce it on Windows but could on the cluster (CentOS).

Steps:
virtualenv testenv
source testenv/bin/activate
git clone https://github.com/MultiAgentLearning/playground
cd playground
pip install -U .
If I can reproduce it on Windows or on local Ubuntu I will let you know but it might be an issue of the cluster server only... I already contacted a support guy of the cluster but he could not reproduce it.

Ok I could install it when I clone it directly in my home space... Must be the permissions of the specific space (scratch) I was using. Thank you for your help and sorry to have bothering you guys about that.


Toplevel or Window should allow set IsMdiContainer property.

Case IsMdiContainer is true the user is responsible to create another Window class and call Application.Run<myChildClass> in the code.

Generated with the TerminalGuiDesigner but testing with the PR https://github.com/gui-cs/Terminal.Gui/pull/2778.
using Terminal.Gui;
using YourNamespace;

public class Program {
	public void Main(string[] args)
	{
		Application.Run<MyView>();
		Application.Shutdown();
	}
}

//------------------------------------------------------------------------------

//  <auto-generated>
//      This code was generated by:
//        TerminalGuiDesigner v<IP_ADDRESS>
//      You can make changes to this file and they will not be overwritten when saving.
//  </auto-generated>
// -----------------------------------------------------------------------------
namespace YourNamespace {
	using Terminal.Gui;

	public partial class MyView {
		MyChildView _childView1;
		MyChildView _childView2;

		public MyView ()
		{
			InitializeComponent ();
			Title = "Parent";
			_childView1 = new () { Width = 20, Height = 10, Title = "Child1" };
			_childView2 = new () { X = Pos.Right (_childView1), Y = Pos.Bottom (_childView1), Width = 20, Height = 10, Title = "Child2" };
			Ready += () => Application.Run (_childView1);
			_childView1.Ready += () => Application.Run (_childView2);
		}
	}
}

//------------------------------------------------------------------------------

//  <auto-generated>
//      This code was generated by:
//        TerminalGuiDesigner v<IP_ADDRESS>
//      Changes to this file may cause incorrect behavior and will be lost if
//      the code is regenerated.
//  </auto-generated>
// -----------------------------------------------------------------------------
namespace YourNamespace {
	using System;
	using Terminal.Gui;

	public partial class MyView : Terminal.Gui.Window {

		private void InitializeComponent ()
		{
			this.Width = Dim.Fill (0);
			this.Height = Dim.Fill (0);
			this.X = 0;
			this.Y = 0;
			this.Modal = false;
			this.IsMdiContainer = true;
			this.Border.BorderStyle = Terminal.Gui.BorderStyle.Single;
			this.Border.Effect3D = false;
			this.Border.Effect3DBrush = null;
			this.Border.DrawMarginFrame = true;
			this.TextAlignment = Terminal.Gui.TextAlignment.Left;
			this.Title = "";
		}
	}
}

//------------------------------------------------------------------------------

//  <auto-generated>
//      This code was generated by:
//        TerminalGuiDesigner v<IP_ADDRESS>
//      You can make changes to this file and they will not be overwritten when saving.
//  </auto-generated>
// -----------------------------------------------------------------------------
namespace YourNamespace {
	using Terminal.Gui;

	public partial class MyChildView {

		public MyChildView ()
		{
			InitializeComponent ();
		}
	}
}

//------------------------------------------------------------------------------

//  <auto-generated>
//      This code was generated by:
//        TerminalGuiDesigner v<IP_ADDRESS>
//      Changes to this file may cause incorrect behavior and will be lost if
//      the code is regenerated.
//  </auto-generated>
// -----------------------------------------------------------------------------
namespace YourNamespace {
	using System;
	using Terminal.Gui;

	public partial class MyChildView : Terminal.Gui.Window {

		private Terminal.Gui.Label label;

		private Terminal.Gui.TextField textField;

		private void InitializeComponent ()
		{
			this.textField = new Terminal.Gui.TextField ();
			this.label = new Terminal.Gui.Label ();
			this.Width = Dim.Fill (0);
			this.Height = Dim.Fill (0);
			this.X = 0;
			this.Y = 0;
			this.Modal = false;
			this.IsMdiContainer = false;
			this.Border.BorderStyle = Terminal.Gui.BorderStyle.Single;
			this.Border.Effect3D = false;
			this.Border.Effect3DBrush = null;
			this.Border.DrawMarginFrame = true;
			this.TextAlignment = Terminal.Gui.TextAlignment.Left;
			this.Title = "";
			this.label.Width = 4;
			this.label.Height = 1;
			this.label.X = 0;
			this.label.Y = 0;
			this.label.Data = "label";
			this.label.Text = "Name:";
			this.label.TextAlignment = Terminal.Gui.TextAlignment.Left;
			this.Add (this.label);
			this.textField.Width = 10;
			this.textField.Height = 1;
			this.textField.X = 6;
			this.textField.Y = 0;
			this.textField.Secret = false;
			this.textField.Data = "textField";
			this.textField.Text = "";
			this.textField.TextAlignment = Terminal.Gui.TextAlignment.Left;
			this.Add (this.textField);
		}
	}
}

https://github.com/gui-cs/TerminalGuiDesigner/assets/13117724/7f11df75-1947-41f2-af1f-3f833ab702a8

Can you please create a PR that just contains 91a1a22
I have cherry-picked it but get null references when I set it to true and continue using editor.
For example:

Open a window
Add a FrameView
Delete the FrameView
Repeat after setting IsMdiContainer=true

Global Exception
Object reference not set to an instance of an object.
   at Terminal.Gui.Toplevel.PositionCursor()
   at Terminal.Gui.View.PositionCursor()
   at Terminal.Gui.Toplevel.PositionCursor()
   at Terminal.Gui.Application.RunMainLoopIteration(RunState& state, Boolean
 wait, Boolean& firstIteration)
   at Terminal.Gui.Application.RunLoop(RunState state, Boolean wait)
   at Terminal.Gui.Application.Run(Toplevel view, Func`2 errorHandler)

This crash isn't nessesarily a show stopper.  It may be that the property is not designed to be changed during runtime of app.  In this case I think we could add the concept of a 'MemoryOnlyProperty` (needs a better name).
In this case the Property would store the state (true/false) but not actually change the value on the View.  This would be useful for the case of View.Visible too which it would be nice to edit but we really don't want to actually result in vanishing from the editor!

I have added the concept of a SuppressedProperty which is useful for Visible but I think also could be handy for this use case too.  See: https://github.com/gui-cs/TerminalGuiDesigner/pull/228
I've added you as a reviewer to get your impressions of the feature.
If accepted, I will need to port the PR to v2 branch also.

Can you please create a PR that just contains 91a1a22
I have cherry-picked it but get null references when I set it to true and continue using editor.
For example:
* Open a window

* Add a FrameView

* Delete the FrameView

* Repeat after setting IsMdiContainer=true

I tried before doing a separate PR to this but it only works with the #227. Can you test on that PR and check if the error persist, please?

I already fixed in the https://github.com/gui-cs/Terminal.Gui/pull/2778/commits/c8f2d067bd0c6ca244d614b7f10f05a2b01cb619, but it need wait until is merged and published.

Forget, throws the same error. I'll investigate the cause. Thanks for related this. I'll create a PR to this issue.

Done.


[regression] Workaround MSVC preprocessor issue triggered by REQUIRE_THROWS

In 2.3.0 REQUIRE_THROWS goes through one extra level
when passing its expr argument, and this seems to
cause MSVC 19.16 preprocessor to refuse to do
stringification with C2026 (string too big, trailing
characters truncated).  The string is indeed big (6.8KB),
but not that big (16KB).

Can you paste an example of what the code looks like (with the string shortened from 6.8kb to a few chars)? This might be a problem for other macros as well.
I'll go ahead and merge this since it is a simplification anyway but you'll have to use the dev branch for now

Can you paste an example of what the code looks like (with the string shortened from 6.8kb to a few chars)? This might be a problem for other macros as well.

I did more research and found the cause.  The code used to work because the traditionally MSVC preprocessor only expand one layer of macros in stringification each time you pass it down.
My code looks like this
REQUIRE_THROWS(somefunc(
         KWARG(.labels, "x1", "y1"),
         KWARG(.ppg, 0.3),
         ...);

Only 5 lines, but each KWARG expands to a complicated PEGTL_STRING (strongly typed compile-time string) inside tuple along with some macros -- at this stage, it's 6.8K.  But if all macros are expanded, it's 799K.  The standard seems to require this to be fully expanded.  However, it works in gcc because gcc has no string literal size limit.  It worked in msvc because passing it down once doesn't cause the whole thing to be fully expanded.  It does not work with msvc next-gen preprocessor because it has both the "correct" behavior and the limit (https://devblogs.microsoft.com/cppblog/msvc-preprocessor-progress-towards-conformance/).  So what I should probably is to change the call to a named lambda instead.

I'll go ahead and merge this since it is a simplification anyway but you'll have to use the dev branch for now

I'm wandering whether there is a way to not to expand the nested macros at all.  Because after all users only want to see the code they wrote, not the code compiler wrote, when encounters a test failure.

(Anyhow the change you just merged has its value [although I sent the PR without digging into the issue too deep]: without the change REQUIRE_THROWS's test failure message is different from that of REQUIRE_THROWS_AS under msvc preprocessor's "friendly" behavior.)


Add user pronouns to OIDC claims for use in meeting registration

Description
There is a request to update meeting registration to provide the option to show pronouns on badges. If the user has already defined pronouns in Datatracker, it would be useful to pre-populate this field with data from DT
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Already provided as extra scope.
https://github.com/ietf-tools/datatracker/blob/main/ietf/ietfauth/utils.py#L223


Feedback canceling out

Hi,
I don't know if this is a module related issue or an overall issue, I will try to explain the best I can below.
The best way to explain is that I have 2 buttons which use feedback from itself as well as from each other, however, if button one has changed the feedback of button 2 from its default, when button 2 is used, its state changes to the feedback of what the operation is doing which is fine but when you deselect it, it doesn't return to its existing feedback state (if the feedback is from button 1), but rather the default state.
Is there a way around this of knowing which feedback state it needs to display?
Any Help would be much appreciated.
Dan.

Hi @Brennan-Dan
Could you perhaps tell me what actions and feedbacks you are using on each of the two buttons?

@Brennan-Dan
Any chance you can give me information?


Analysis- IDIM Response (BC Services Card)

Acceptance Criteria
Facilitate IDIM-AEST meetings to:

[ ] Compile initial report of Business related issues
[ ] Compile initial report of Technical related issues
[ ] Reports should relate to each other (Business-Technical)
[ ] Confirm list valid and accurate with Alys and Schwenker

Additional Context
Thanks all for your time today great discussion! Very helpful. I’ve captured some notes here with the intention that they are companion to the great table we reviewed as part of the meeting. Please share as appropriate (I feel like there’s a name missing from the CITZ team) and please let me know if I’ve misrepresented or missed anything critical. Also to inform those that weren’t able to join us.
We talked about our stated intention to halt the attestation reliance as best we can – having a handle on what situations cannot be rectified any other way via these conversations and not having attestation as the default option so we can uncover any additional potential gotcha’s is the goal. I asked for us to have a solid understanding of those situations, who (BCSC or SABC) will be leading any ‘fix’, and what the alternate supports if any are no later than the end of January.
Note: Current wording that ServiceBC staff uses ‘please contact SABC to see if there is an alternative’ if BCSC is unable to rectify situation.
The areas SABC frequently assists with are:
•	Students with Single Names (mononym or double mononym)
o	Initial discussion indicates that a mononym is often replicated in both last name and first name fields on BCSC
o	ICBC can support mononym but Health Insurance BC cannot (HIBC needs it in both fields and issues a non-photo BCSC with that – therefore they end up with a BCDL and a non-photo BCSC that do not match)
o	Marlowe’s thought – I believe we may be able to tackle this one via business rule in SIMS
•	Students with multiple first names (seems to be biggest volume)
o	May have potential to tackle SOME of these through communication/training of ICBC counter staff
o	WE find out either:
	Our account creation does not match BCSC, we coach them to change the account login
	Sometimes still caught at SIN check
	Sometimes school won’t confirm enrolment
	Last catch can be federal account mismatch
o	Marlowe’s thought – in our workflow would there be some way to flag for manual intervention? Need to understand in more detail how to identify.
•	Double last name (ie. hyphenated)
o	Per Jeremy P usually problematic only when also have multiple first name
•	Students whose name on BCSC does not match SIN or school record
o	Ie. married but did not change SIN record (takes 6-9 months)
o	Reissuance of BCSC with spelling mistakes could be 4-6 weeks but more often a couple of weeks
o	Marlowe’s thought – ties into communications to schools (post-sec AND K-12) and students in that we may be able to direct earlier and inform re SIN name mismatch potential
•	Two legal middle names
o	Health Insurance only allows one middle name (non-photo ID)
•	Student with two BC Services Card
o	Ie. Health Insurance with divorced parents
o	Workaround available via BCSC with agreement of student/parents to cancel/deactivate second card
	Better training on this particular scenario will help
•	Expired/Relinquished BC Services Card for out of province students
o	6-10 occurrences last week per Jeremy
o	Students have given up BCSC and/or BCDL after moving for education
o	Currently ‘guarantor’ attestation process
o	OOP users that don’t have MSP (non-MSP credential) currently out of luck but it is on the roadmap of the future
Q for SIMS:
•	Do we have system limitations for name length?
Jeremy’s situation of BCSC reissue (GUID mismatch) – currently a minor in-house workaround. Let’s talk this one through.
Communication for our applicants:
•	We need to ensure that our communications include strong recommendation for students and soon-to-be students to get their BCSC, review for correctness…

Passed Task to Anna - No longer required


Storage refactor in prep for per node manifests

tldr: use composition over inheritance.
Context
Currently Storage uses inheritance and mixins heavily.  Methods for accessing backup manifests ("jsons") as well as actual file contents ("hexdigests") are all together in a single large class.
A change we have talked about is per node manifests, ie a list of all the files on each node for each manifest.  Further adding a new mixin for the node manifests to Storage would be unwieldy and be too much to have in one class.  So as preparation, I am splitting the Storage class.
Changes
Split Storage into smaller, composable classes.

Storage and it's implementations are the "storage backend".  They handle actually reading and writing from object storage / the file system / memory / anywhere else.  Caching is also transparently handled by this backend class, see CachedStorage.
MultiStorage is similar to before.  It routes access to multiple Storage classes keyed by name.
JsonStore and HexDigestStore now wrap a Storage class rather than mixing in.  Since they are not a Storage class themselves, I changed the suffix (if someone has a better name pls let me know).
StorageManager combines all these things to provide access to JsonStore and HexDigestStore for both the node and the coordinator APIs.

Some other small changes:

The current inheritance model for ThreadLocalStorage won't work well if we need to cache multiple things instead of just one class, which will be necessary for Uploader with per node manifests.  Use composition here, renaming to CopiedThreadLocal.
Use tmp_path: Path over tmpdir: py.path.local

aiven-core tests ran successfully here https://github.com/aiven/aiven-core/pull/44634


NFT Drop page mockup added

Created a mockup the Join drop page

Hey, this is good! Your design is better than the one of mine.
Do you think, we could make a carousel or sidescroll component out of NFT drop list?

It's also very fine to use the counter. I think, could be useful to have a date + a counter but this is up to your decision ;-) .
If you use the counter value, I rather would name it "Remaining Time" or "Time Left" instead of "Date" otherwise this is nice.


Set up RISC-V CI test

Setting up RISC-V cross-compile CI test so we can catch the potential breakage sooner.

[x] Setup docker in GCR
[ ] Setup Kokoro script
Setup buildkite yml script

If you're setting up a Kokoro script (as in #5762) there's no need to also set up buildkite

Oh, okay. I put the item out to see If the long-term plan is to move to buildkite. I will remove it.

If it is, then we can do it as part of that :slightly_smiling_face:

Turned the presubmit on check today.


hello world

Please share you email id then we can mail you the participation certificate

Please share you email id then we can mail you the participation certificate

<EMAIL_ADDRESS>


Method Chaining

Hello again! I was wondering if you would be open to using the method chaining design pattern for the Setters so that changing values can look much better.
Usage example:
cp := astiav.AllocCodecParameters().
                  SetHeight(1).
                  SetHeight(2).
                  SetSampleRate(3)

Hey, I'm not looking for this change right now, but I'll keep the issue open in case that changes


Device is not secure

I am getting this error on android when trying to use the plugin, "Device is not secure",
Here is my implementation:
`
import { SecureStorage } from 'ionic-native';
import {Constants} from '../../model/Constants';
export class SecStorage {
static instance: SecStorage;
static isCreating: Boolean = false;
private secureStorage: SecureStorage;

public static LOGGED = "LOGGED";
public static TokenId = "TOKEN_ID";

constructor() {
    if (!SecStorage.isCreating) {
        throw new Error("You can't call new in Singleton instances!");
    }
}

static getInstance() {
    if (SecStorage.instance == null) {
        SecStorage.isCreating = true;
        SecStorage.instance = new SecStorage();
        SecStorage.instance.init();
        SecStorage.isCreating = false;
    }

    console.log("INSTANCE -> " + SecStorage.instance);
    return SecStorage.instance;
}

private init() {
  ;debugger
    this.secureStorage = new SecureStorage();
      ;debugger
    this.secureStorage.create(Constants.SECURE_STORAGE_KEY)
        .then(
        () => console.log('*SECURE STORAGE SUCCESS*'),
        error => console.log("*SECURE STORAGE ERROR*" + error)
        );
}

getValue(key) {
    return this.secureStorage.get(key);
}

setValue(key, value) {
    this.secureStorage.set(key, value)
        .then(
        data => console.log(data),
        error => console.log(error)
        );
}

removeValue(key) {
    this.secureStorage.remove(key);
    // .then(
    // data => console.log(data),
    // error => console.log(error)
    // );
}

}
`
Can i Know what is wrong ?

As of the last version, if the device has no screen lock or an screen lock with insufficient security that error is thrown. There is an example in the README, you should the user and call secureDevice to bring up the screen lock settings.

Can't we skip this security check?
On Aug 8, 2016 2:05 PM, "Yiorgis Gozadinos"<EMAIL_ADDRESS>wrote:

As of the last version, if the device has no screen lock or an screen lock
with insufficient security that error is thrown. There is an example in the
README, you should the user and call secureDevice to bring up the screen
lock settings.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Crypho/cordova-plugin-secure-storage/issues/44#issuecomment-238205685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK49WwS3QsgaAXI1YJ33ItU-OeBZ9fotks5qdw1igaJpZM4Je6P-
.

Please read the docs. No you can't, we cannot store anything on the credential storage unless appropriate screen lock settings are there.

Could you maybe provide a function to check if device is secure?

@ezamelczyk Isn't that what init() does?

@ggozad Yeah, you're right! Thanks.


build error with gflags

I got a fatal build error as the following:
~/DynSLAM/build$ make
Scanning dependencies of target InstRecLib
[  0%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/SegmentationDataset.cpp.o
[  1%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/SparseSFProvider.cpp.o
[  2%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/InstanceTracker.cpp.o
In file included from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/Track.h:7:0,
from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/InstanceTracker.h:13,
from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/InstanceTracker.cpp:3:
/home/tutu/DynSLAM/src/DynSLAM/InstRecLib/../InfiniTamDriver.h:11:27: fatal error: gflags/gflags.h: No such file or directory
compilation terminated.
src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/build.make:110: recipe for target 'src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/InstanceTracker.cpp.o' failed
make[2]: *** [src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/InstanceTracker.cpp.o] Error 1
CMakeFiles/Makefile2:1247: recipe for target 'src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/all' failed
make[1]: *** [src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

Try going to src/gflagsand building that using CMake. It should get placed in the CMake registry and found by CMake.

Thanks! But I encounter another build error after building the gflags
cd ~/DynSLAM/src/gflags
mkdir build && cd build
export CXXFLAGS="-fPIC" && cmake .. && make VERBOSE=1
sudo make install
cd ~/DynSLAM/build
build
[  1%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/InstanceTracker.cpp.o
[  1%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/PrecomputedSegmentationProvider.cpp.o
[  2%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/InstanceReconstructor.cpp.o
[  3%] Building CXX object src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/VisoSparseSFProvider.cpp.o
/home/tutu/DynSLAM/src/DynSLAM/InstRecLib/VisoSparseSFProvider.cpp: In function ‘Eigen::Matrix4d instreclib::VisoToEigen(const Matrix&)’:
/home/tutu/DynSLAM/src/DynSLAM/InstRecLib/VisoSparseSFProvider.cpp:9:28: error: passing ‘const Matrix’ as ‘this’ argument discards qualifiers [-fpermissive]
return Eigen::Matrix4d((~viso_matrix).val[0]);
^
In file included from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/../../libviso2/src/matcher.h:34:0,
from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/SparseSFProvider.h:7,
from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/VisoSparseSFProvider.h:7,
from /home/tutu/DynSLAM/src/DynSLAM/InstRecLib/VisoSparseSFProvider.cpp:3:
/home/tutu/DynSLAM/src/DynSLAM/InstRecLib/../../libviso2/src/matrix.h:106:11: note:   in call to ‘Matrix Matrix::operator~()’
Matrix  operator~ ();                // transpose
^
src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/build.make:182: recipe for target 'src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/VisoSparseSFProvider.cpp.o' failed
make[2]: *** [src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/VisoSparseSFProvider.cpp.o] Error 1
CMakeFiles/Makefile2:1247: recipe for target 'src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/all' failed
make[1]: *** [src/DynSLAM/InstRecLib/CMakeFiles/InstRecLib.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

Hmm, try https://github.com/AndreiBarsan/DynSLAM/issues/33 as it seems like a similar problem. (I don't have access to a CUDA-enabled workstation to test these things at the moment. I'm sorry about that... :( )

The solution on #33 works. I modify VisoSparseSFProvider.cpp as the following:
Eigen::Matrix4d VisoToEigen(const Matrix &viso_matrix) {
//  The '~' transposes the matrix...
Matrix a=viso_matrix;
return Eigen::Matrix4d((~a).val[0]);
}
Then make can go on until meet the the following link error:
[ 65%] Linking CXX executable DynSLAMGUI
CMakeFiles/DynSLAMGUI.dir/src/DynSLAM/DynSLAMGUI.cpp.o: In function backward::TraceResolverLinuxImpl<backward::trace_resolver_tag::libdw>::find_fundie_by_pc(Dwarf_Die*, unsigned long, Dwarf_Die*)': /home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1394: undefined reference to dwarf_child'
/home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1409: undefined reference to dwarf_attr' /home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1410: undefined reference to dwarf_formflag'
/home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1422: undefined reference to dwarf_siblingof' ..... ..... /home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1373: undefined reference to dwarf_formudata'
CMakeFiles/DynSLAMGUI.dir/src/DynSLAM/DynSLAMGUI.cpp.o: In function void backward::details::deleter<void, Dwfl*, &dwfl_end>::operator()<Dwfl*>(Dwfl*&) const': /home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:303: undefined reference to dwfl_end'
CMakeFiles/DynSLAMGUI.dir/src/DynSLAM/DynSLAMGUI.cpp.o: In function backward::TraceResolverLinuxImpl<backward::trace_resolver_tag::libdw>::die_has_pc(Dwarf_Die*, unsigned long)': /home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1371: undefined reference to dwarf_attr'
/home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:1373: undefined reference to dwarf_formudata' CMakeFiles/DynSLAMGUI.dir/src/DynSLAM/DynSLAMGUI.cpp.o: In function void backward::details::deleter<void, Dwfl*, &dwfl_end>::operator()<Dwfl*>(Dwfl*&) const':
/home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp/backward.hpp:303: undefined reference to `dwfl_end'
collect2: error: ld returned 1 exit status
CMakeFiles/DynSLAMGUI.dir/build.make:821: recipe for target 'DynSLAMGUI' failed
make[2]: *** [DynSLAMGUI] Error 1
CMakeFiles/Makefile2:74: recipe for target 'CMakeFiles/DynSLAMGUI.dir/all' failed
make[1]: *** [CMakeFiles/DynSLAMGUI.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2
the cmake result is the following:
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- backwards-cpp-enhanced stack traces? ON
-- Found Eigen3: /home/tutu/DynSLAM/src/eigen (Required is at least version "2.91.0")
-- Found GLUT: /usr/lib/x86_64-linux-gnu/libglut.so
-- Found OpenGL: /usr/lib/x86_64-linux-gnu/libGL.so
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- CUDA found: TRUE
-- Found libdw: /usr/lib/x86_64-linux-gnu/libdw.so
-- Found libbfd: /usr/lib/x86_64-linux-gnu/libbfd.so
-- BACKWARD_HAS_UNWIND=1
-- BACKWARD_HAS_BACKTRACE=0
-- BACKWARD_HAS_BACKTRACE_SYMBOL=0
-- BACKWARD_HAS_DW=1
-- BACKWARD_HAS_BFD=0
-- Found Backward: /home/tutu/DynSLAM/src/InfiniTAM/InfiniTAM/ORUtils/third_party/backward-cpp
-- Looking for C++ include unistd.h
-- Looking for C++ include unistd.h - found
-- Looking for C++ include stdint.h
-- Looking for C++ include stdint.h - found
-- Looking for C++ include inttypes.h
-- Looking for C++ include inttypes.h - found
-- Looking for C++ include sys/types.h
-- Looking for C++ include sys/types.h - found
-- Looking for C++ include sys/stat.h
-- Looking for C++ include sys/stat.h - found
-- Looking for C++ include fnmatch.h
-- Looking for C++ include fnmatch.h - found
-- Looking for C++ include stddef.h
-- Looking for C++ include stddef.h - found
-- Check size of uint32_t
-- Check size of uint32_t - done
-- Looking for strtoll
-- Looking for strtoll - found
-- CUDA found: TRUE
-- OpenNI found: FALSE
-- LibRoyale found: FALSE
-- libuvc found: FALSE
-- librealsense found: FALSE
-- libpng found: TRUE
-- Compile: 52
-- compiled -> 0
-- worked
-- Using CUDA compute capability: 52
-- Using CUDA arch: 52
-- Configuring done
-- Generating done
-- Build files have been written to: /home/tutu/DynSLAM/build
Before doing cmake, I apt-get install  libdw-dev binutils-dev in order to let cmake find libdw and libbfd.
So how to do next to fix the current link issue?
Thanks!

Hmm, interesting. Could you try to disable libdw support from CMake? Alternatively, you could try to add #define BACKWARD_HAS_DW 0 to DynSlamGUI.cpp, but I'm not 100% sure it would work.
I only used libbfd during development and didn't see any error like that, so it may be due to some libdw weirdness.

After executing sudo apt-get remove  libdw-dev, I make successfully at last. My whole script for building on Ubuntu 16.04 is:
sudo apt-get install git libxmu-dev libxi-dev freeglut3 freeglut3-dev glew-utils libglew-dev libglew-dbg
sudo apt-get install cmake doxygen libboost-python-dev libopencv-dev libboost-all-dev python-opencv
sudo apt-get install binutils-dev libpng++-dev
git clone --recursive https://github.com/AndreiBarsan/DynSLAM
cd DynSLAM/src/Pangolin/
mkdir build
cd build
cmake ..
make
cd ../../gflags
mkdir build && cd build
export CXXFLAGS="-fPIC" && cmake .. && make VERBOSE=1
sudo make install
nano ~/DynSLAM/src/DynSLAM/InstRecLib/VisoSparseSFProvider.cpp
Eigen::Matrix4d VisoToEigen(const Matrix &viso_matrix) {
//  The '~' transposes the matrix...
Matrix a=viso_matrix;
return Eigen::Matrix4d((~a).val[0]);
}
cd ~/DynSLAM
mkdir build
cd build
cmake ..
make

That's awesome news.
I really appreciate you sharing the setup script!!
Have you managed to get the system to run on an example sequence, such as this short one? http://www.cs.toronto.edu/~iab/dynslam/mini-kitti-odometry-seq-06-for-dynslam.7z

Yeah.  I run "build/DynSLAM --use_dispnet --dataset_root=path/to/extracted/archive" with mini-kitti-odometry-seq-06-for-dynslam.7z successfully.
But is still on the way to test with KITTI Odometry dataset.


Unnecessary use of Random embeddings in EMG example

The example implements the temporal encoder shown in the paper "Hyperdimensional biosignal processing: A case study for EMG-based hand gesture recognition", which is depicted in the paper's Figure 4.
The encoding of temporal information is done solely by the permutation of spatial encoded samples, and there is no binding to random vectors. Thus, I believe that the binding to "timestamps" Random embedding is unnecessary.
https://github.com/hyperdimensional-computing/torchhd/blob/83fdf340d0ca74938e496815339466815cf7d909/examples/emg_hand_gestures.py#L42
I've ran some tests using different random seeds and the accuracy values remain the same (sometimes with small changes) when removing the mentioned line.

Thank you for reporting this issue, great catch. And thank you for opening a pull request already.


Project attribute doesnt recognize a todo if its above a todo with a threshold tag

Bug Report
**App Version:**2.0.11
Platform: macOS
Installation Method: homebrew
Expected Behavior:
I expect the project attributes to have a +1 for a project
Actual Behavior:
There is no +1 for a project
Steps to Reproduce:

Open the app.
Paste the following todos seen in this code block into your todo.txt:

2024-02-21 entry 1 +bug
2024-02-21 entry2 due:2024-03-20 +bug rec:+1m pri:A t:2024-03-18

Have threshold date in future toggle off
Screenshots:

@ndmvisuals can I ask you to check if this bug still occurs in this pre-release? https://github.com/ransome1/sleek/releases/tag/v2.0.12-rc.1

For sure! I'll check tonight and get back to you

@ransome1 Yep that fixed it! Thank you so much :)


make racket's psyntax benchmark work

Benchmark is here, see all the files that start with psyntax: https://github.com/plt/racket/tree/master/pkgs/racket-pkgs/racket-benchmarks/tests/racket/benchmarks/common
Needs various racket-y things to be able to work unmodified.

This turns out to need eval.

This might work now.


[Feature] Global installation

Description
Make an installer able to install Ungoogled Chromium for all users
Who's implementing?

[ ] I'm willing to implement this feature myself

The problem
Currently, the installer installs itself only for the user installing the software
I think there should be a way to install it for all users
Even by using the following winget command winget install --scope machine --id eloston.ungoogled-chromium, it is installed only for the current user
Possible solutions
The installer could be able to use the Program Files directory instead of the local one
Alternatives
No response
Additional context
No response

Cannot reproduce.

Does winget install -e --id eloston.ungoogled-chromium install the latest version? I'm noob at using winget and would like ot make it easier to update Chromium. It seems that the version available using winget is 111.0.5563.65 but I don't know if it's an older version or not related to Ungoogle Chromium.

winget should have all packages up to M127, see https://github.com/microsoft/winget-pkgs/commits/master/manifests/e/eloston/ungoogled-chromium


mobile scroll bug

Too large or something,  press scroll up and disappear the guess thing.  Scroll guesses to top and can't see title fully.  Probably also why it is not scrolling to word perfectly

pretty solved


New release

I don't know the criteria timing for releases, but can I ask for a new release?
I'm interested in https://github.com/edmondchuc/flask-htmx/commit/93855a7e14f5422f0472b47975cae77a583d2e3e.
Thanks!

Thanks for the nudge! It's available now in 0.4.0.
https://github.com/edmondchuc/flask-htmx/releases/tag/0.4.0


Pass build parameters to Slack

We use Jenkins for deployment and have a single job that deploys any of our projects based on a build parameter.  Rather than just knowing the job has run in Slack it would be great if the build parameter for the project could be passed too.

Apologies, I've found this is already possible by setting a custom message after hitting the Advanced button when configuring the job.

Glad you found a work-around.


Reduce scope of Amazon SageMaker Prompt Driver to only Jumpstart

By reducing the scope of our integration with Amazon SageMaker to just Jumpstart, we can reduce the request/response variance, getting rid of the need for Prompt Model Drivers.

@dylanholmes also updated PR to include embedding drivers.

Ignoring integration test, will be fixed in separate PR.


Using class decorators causes class to be skipped

See https://github.com/wycats/javascript-decorators#class-declaration
@F("color")
@G
class Foo {
}

ESDoc fails to generate any docs for class Foo.

To be clear, I'm not looking for support to generate esdoc in this use case, I'm simply hoping for either (A) an error message or (B) silently ignoring the new syntax and continue generating esdoc normally.

@jasonsanjose Hi! Thanks for this issue :smile:
Sorry, I don't have plan to support ES7. Please see https://github.com/h13i32maru/esdoc/issues/29 and FAQ for more information.

(A) an error message or (B) silently ignoring the new syntax and continue generating esdoc normally.

Now, the both plan are hard. because Espree(used in ESDoc) doesn't support ES7. So ESDoc can not handle ES7 syntax.
Thanks for your understanding.

@jasonsanjose Hi!
I published ESDoc ES7 Plugin :smile:
This plugin enables to generate documentation from source code including ES7.
You can use this plugin with<EMAIL_ADDRESS>

Thanks! I'll try it out this week.

:+1:


Show links on app bar for full screen or  drawer for mobile

I think hiding the links to different pages in the hamburger menu makes the website harder to navigate for some users. Would be nice to have a responsive drawer instead, where the links are on the header for full screen and in the responsive drawer with the hamburger menu for smaller screens.

What if instead of pushing the items into the Title Bar the drawer was just responsive, meaning always stays open on wide enough screens.  Also, for those smaller screens it should probably auto-close after selection.  I think standard material design would keep navigation within the drawer or or bottom nav (https://material.io/components/bottom-navigation).

Go for it, Ron. Make it look good 👍
On Sun, Oct 18, 2020, 9:28 AM ronaldewatts<EMAIL_ADDRESS>wrote:

What if instead of pushing the items into the Title Bar the drawer was
just responsive, meaning always stays open on wide enough screens. Also,
for those smaller screens it should probably auto-close after selection. I
think standard material design would keep navigation within the drawer or
or bottom nav (https://material.io/components/bottom-navigation).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opennebraska/landlord/issues/39#issuecomment-711176558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2HQ5EESG5OFTLZZ4Q7K5LSLL3P5ANCNFSM4R7CPL6A
.

i agree that it is a good idea. Might also provide a good example for the nav on the next project

Here's what it looks like


For those who have 404 error when running script, CHECK THIS

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

Title

[X] The title is no longer "[BUG] Title" and I edited it with the right error name.

Describe the bug
Use Win+R to run
%LOCALAPPDATA%\Programs\Python*PythonVERSION*\Lib\site-packages
change PythonVERSION to your version like Python38, Python311,etc.
Then find
patcher.py
Open it with notpad o any coding program like Visual Studio Code
Using Ctrl+F find
download_url = "https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/%s/%s/%s"
should be in line 283
then replace it with
download_url = "https://storage.googleapis.com/chrome-for-testing-public/%s/%s/%s"
save and quit
run script as usual
Copy and paste your error
Credit to @klept0 and @dudeua
Screenshots

Value of dashboard variable
none

already reported #439

most people don't check closed issues, I keep it open and on top so people can see the solution at first sight.

This has been fixed in undetected-chromedriver version 3.5.5

PR raised to resolve issue.

The original repo is not being maintained as far as we can tell hence why I started mine. Feel free to look at it and make any PR that is necessary

set up everything, but when i run "main" it automatically closes. when i tried this method, i couldn't find patcher.py...

https://github.com/klept0/MS-Rewards-Farmer

I'm gonna close it. the problem should be already fixed anyway.

set up everything, but when i run "main" it automatically closes. when i tried this method, i couldn't find patcher.py...

https://github.com/klept0/MS-Rewards-Farmer

Will try it tomorrow…


Add size filter

This filter aims to resize an image when responsive images isn't available.
Mergin this PR will:

Define a new size filter that take a width parameter
Explain this size filter in the README.md usage section
Close #1

Looks good @d9beuD, thanks for putting this together! I'll get an update out to RubyGems shortly.

Now pushed to RubyGems as version 1.2.0. Thanks again for the contribution @d9beuD !


DX: drop non-maintained Symfony <3.4 and <4.4

I was struggling with some issue which looked related to --prefer-lowest (ref https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5242#issuecomment-723671891 ) and having 1000 possible combinations to check is pointless.
(not fully dropping v3, as it's still in serious usage in prod: https://packagist.org/packages/symfony/console/stats#major/all )

Might as well bump to ^5.1.8 too, since 5.1.6 for example, is older than 4.4.16.

5.x is current major line of Sf, i would keep it as-is, only bumping the deps for lines without active development

Seems odd to bump 3.x and 4.x to versions newer than what is allowed in the 5.x series?

i see this as first step to drop v3/v4 support. i don't plan to drop v4 support

:-1:  for less users being able to install because locking out these versions. Tool worked on these versions before. If this is an issue with tests than I would fix or drop the tests, not the packages.


CI pass

Before submitting
minor fixes to gets the CI passing @neggert
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Could you also increase num_epochs to 4 in test_restore_models.py::test_running_test_pretrained_model_dp? Same issue, but only shows up when running on GPUs.

@neggert done ^^

LGTM

@williamFalcon ^^


"Test if mount_requests is set correctly" on empty list

Proposed feature
I think it would be better if mount_requests is allowed to be an empty list (or something the like).
That way, I could just configure an empty variable for hosts which should not have additional mounts.
For now, I have to introduce a new group or do a when on my own.

Additionally as the assert.yaml has run_once: yes (and maybe delegate_to: localhost?) this seems to have an odd behavior if defined per host or group. But maybe I'm doing some bad practice here...

Thanks, mount_requests should be optional indeed. Changed with the last commit.
The assert.yml (which checks variables) runs on localhost (to speed up the checks) and indeed run_once. Please let me know what issues you are running into. Likely in a new issue.
Thanks @debuglevel


How to make background's alpha?

now,I can't set background's alpha by setAlpha();
eg.
m_view = new View;
m_view->setWidthFilling();
m_view->setHeight(150);//It's px? I advise to use dp.
m_view->setTop(0);
m_view->setBorderColor(slib::Color::Red); // test Color
m_view->setBorderWidth(2);
//m_view->setAlwaysOnDrawBackground(true);
m_view->setBackgroundColor(slib::Color::Blue);
m_view->setAlpha(0.8);

m_view->setAlpha(0.8)

This will set the alpha of the view object.
To set the background window's alpha, use following snippet
getMainWindow()->setAlpha(0.8);

Yes,I want to set alpha of the view object.
But it seems to be ineffective.

I want to set area of title bar's alpha only,not all pager.
thanks.

Thanks.


how to zoom in or out images on double tap in mobile device?

Hi, I use ImageViewer in javascript. Web browser is ok.  But ImageView is not zoom in and out of images on double tap in mobile device. How can i resolve this problem? Thank you.

It looks like the library uses the touch API instead of the Pointer API.  I may look at forking and authoring a PR.  Might be able to use https://github.com/CamHenlin/TouchPolyfill as a bridge, too.


BC Farsi: Legal system and Immigration-Hate crimes and help for victims...

There are four issues in this topic.

Red arrow highlights a missing letter at the beginning of the word "گر " it should be "اگر " The missing letter is " ا "
See screenshot below from the guide.

Yellow arrow refers to mixed up order of words. The words in english should be at the beginning of the sentence.
See screenshot below from the guide.

Blue arrow refers to the word " mailto" that should be removed.

Issue # 4: There is a missing second link here. In the guide and english version there are two links.  Attached a screenshot from English content.

The second link is not clickable in the ncg. It appears to be a dead link when I type it in manually. I am going to leave it out of the content


Add crowdloan summary into api proxy

Add crowdloan summary into api proxy.

I do think adding prettier and agreeing on the standard should be brought in the group for discussion.

@geastwood @ishrak-imam this came up with @jdomingos when I was showing him the gateway earlier. We have a prettier config in there, but some commits have snuck through unformatted somehow.
We can add a pre commit hook or something like that to enforce it, this will prevent issues in future. Also, we should get rid of all prettier configs that are not at the root so there is just the 1 for the whole repo.
I don't have strong opinions about what rules we have in the prettier config (except 2 spaces not 4!). I think we only have 3 or 4 rules specified, the rest are defaults.

Great. As long as everyone on the same page I'm onboard. 👌

Also, we should probably do this throughout all our JS repos. As the team is growing relying on all developers having prettier in their editor (with format on save set to true) is likely to become more of a problem.

Currently we have same but multiple configs for prettier in the root and substrate api proxy.
Don't have any strong opinion for the rules as long as same across all projects.

Also, we should get rid of all prettier configs that are not at the root so there is just the 1 for the whole repo.

This sounds good.

Throughout you mean each repo has a prettierrc file to specify the formatting rather than unify the format?

If we want to unify it we just need to publish the config as an npm package (or use an existing one, e.g. like some projects do with airbnb's eslint rules).
I don't mind though if different sub teams/projects want to have different prettier rules, as long as it's automatic and enforced in each repo.

If we were like a company and lot of private repo, having company wide non styling package make sense. However, if we do OS, having a config seems more transparent option.


where is testnet faucet not shelley-testnet

I think I don't understand the question. Could you clarify, please?

Here it is https://testnets.cardano.org/en/byron/tools/faucet/

This helped me solve the problem, thank you very much!


Add CropImageActivity into your AndroidManifest.xml

Android 12 we need to add android:exported attribute for activity, service , broadcast and provider.It is not mentioned in library.

please read #858
And update to the new repository


Open SEG and RTSTRUCT side panel automatically

Request
When loading or dragging in a seg or rtstruct the change in the image may be subtle and people may not realize anything has happened (real users have reported confusion on this).  Currently one needs to know that opening the right side panel will show the list of segments/structures.
The request would be to either open the panel whenever it gets populated, or at least make it visually change, perhaps with a little badge telling the number of segments/structures that are available.  The badge option may be preferable is people don't want the side panel re-opening every time and they need to repeatedly close it to see the images.  Also if it's possible that there are both structs and segments in the same study then it would be more clear that there is data in one or both and avoid ambiguity over which one to open.
...
Why should we prioritize this feature?
Current behavior has been shown to be confusing.
...

@igoroctaviano is this something you could have a look at?

@igoroctaviano is this something you could have a look at?

Sure, I'll do that asap. The optional badge in the panel tab makes sense.


Revised codes for no toxin chains, Manual epitope design

Excellent work, I tried to generate antibody for COVID NP protein.
So, I revised the code as below;
if name == 'main':
ckpt = './checkpoints/cdrh3_design.ckpt'
root_dir = './demos/data'
pdbs = [os.path.join(root_dir, '6wzo.pdb') for _ in range(4)]
toxin_chains = []
remove_chains = None
receptor_chains = ["A", "B", "C", "D"]
epitope_defs = [os.path.join(root_dir, c + '_epitope.json') for c in receptor_chains]
identifiers = [f'{c}_antibody' for c in receptor_chains]
I manually design epitope.json file as;
A_epitope.json
[
["A", [299, ""]],
["A", [300, ""]],
["A", [302, ""]],
["A", [303, ""]],
["A", [305, ""]],
["A", [306, ""]],
["A", [345, ""]],
["A", [347, ""]],
["A", [348, ""]],
["A", [349, ""]]
]
However, I encounter the messeges as;
/home/jhs9301/anaconda3/envs/dyMEAN/lib/python3.8/site-packages/numpy/core/fromnumeric.py:3464: RuntimeWarning: Mean of empty slice.
return _methods._mean(a, axis=axis, dtype=dtype,
/home/jhs9301/anaconda3/envs/dyMEAN/lib/python3.8/site-packages/numpy/core/_methods.py:184: RuntimeWarning: invalid value encountered in divide
ret = um.true_divide(
/mnt/c/Users/jhs93/Downloads/dyMEAN-main/dyMEAN-main/data/dataset.py:63: RuntimeWarning: invalid value encountered in cast
X[0] = center  # set center
I guess the invalid values were obtained in Epitope data.
Epitope data: {'X': array([[[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808],
[-9223372036854775808, -9223372036854775808,
-9223372036854775808]]]), 'S': [22], 'residue_pos': [0], 'xloss_mask': [[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]]}
Then, How can i fix the problem? please let me know.

Thanks! It was fixed. there is one more question. Is there any way to obtain the complex structure of an antibody sequence that I already know and an antigen? I wanna optimize the already developed antibody but there is no complex structure.

Hi, I've updated the API for structure prediction here. Corresponding explanations are also updated in README. You can have a try!

Excellent work! I have confusion about the structure prediction tool. Predicte the complex need to provide antigen structure, antibody sequence and epitope definition, but how can we get epitope when there is no complex structure? Looking forward to hearing from you!

Hi, the structure prediction tool is mostly a local docking & structure prediction tool, which assumes that the user already has some knowledge on the binding site of the antibody, just like popular docking softwares for small molecules (e.g. glide, vina). For example, if you know that the antibody is blocking some protein-protein interactions, than it is highly possible that the antibody is approximately binding to the interface of the PPI. Under such circumstances, you can use the protein-protein complex to extract the epitope definition for local docking & structure prediction.
Global docking/cofolding of antibodies to proteins is still a challenging problem, on which most existing models cannot produce satisfactory performance, and the most appealing one might still be Alphafold 3.