Pamela's feedback

This is fantastic! Here's my running feedback as I use it:

I wasn't in the allowed judges list. :-( [Awesome that the feature exists, though!]

I'd like to default to seeing the code.

Maybe add an "Unknown" for Level?

Can there be different criteria for each contest? Like for Emoji, I need to be able to rate their use of functions. For Organ-ic code, we needed to judge also based on scientific accuracy.

Can you show # of lines of code next to the tiel? I don't judge entirely based on that, of course, but it's a useful metric to figure out how much work someone put into something.

How about a slider instead of 1-5? I prefer using a slider for things like this, I get less decision paralysis. E.g. http://imagine-it.org/svg/sliderplugin.html

What do you think about hiding other people's scores? I think there's a tendency to score based on what other people scored, it biases people. I was kind of thinking that only I (the contest administrator) would see the final scores. I don't want people feeling bad about having super low scores. :-(

Hello Pamela!
• I'll give you a script to run that'll add you to the allowed judges list.
• As you wish! https://github.com/Team-Delta-KA/KA-Contest-Judging-System/commit/26f9c2e19d57d757569881e3b5a42e96a13c3876
• Sure thing! I'll add that shortly.
• In our Trello board, I posted a card for an idea of allowing administrators (IE. you), to create different rubrics. Trello link
• I'll definitely look into doing this (shouldn't be hard, just look at the student's code, and implement linesOfCode every time we see \n).
• Sure; I'll implement sliders shortly.
• We can definitely do that. Before that however, I'll need to finish implementing role-based-authentication.
We appreciate your feedback!! If you have any other ideas, don't hesitate to post them here on GitHub, or email me!
~ Gigabyte Giant

Also, it's worth noting, the allowed judges list goes based on Google UIDs, so we have to add your Google UID to the allowed judges list before you can actually judge entries.
The reason we do this, is so we can prevent random people from logging in and judging, and also so you can't judge more than once!

Exciting to see so much progress! :)

Update on fixing these issues:

I wasn't in the allowed judges list. :-( [Awesome that the feature exists, though!]
You're still not in the allowed judges list. You might have been before when @Gigabyte-Giant gave you that script, but we've changed how judge authentication works since then. However, if you go http://gigabytegiant.com/kacjs/Client/, I think it should log you into our database and we can easily identify you by your Google display name "Pamela Fox".
I'd like to default to seeing the code.
Done!
Maybe add an "Unknown" for Level?
Done!
Can there be different criteria for each contest? Like for Emoji, I need to be able to rate their use of functions. For Organ-ic code, we needed to judge also based on scientific accuracy.
We're still working on this, but we'll be done soon.
Can you show # of lines of code next to the tiel? I don't judge entirely based on that, of course, but it's a useful metric to figure out how much work someone put into something.
Done!
How about a slider instead of 1-5? I prefer using a slider for things like this, I get less decision paralysis. E.g. http://imagine-it.org/svg/sliderplugin.html
Done!
What do you think about hiding other people's scores? I think there's a tendency to score based on what other people scored, it biases people. I was kind of thinking that only I (the contest administrator) would see the final scores. I don't want people feeling bad about having super low scores. :-(
Done! Only admins like me, @Lokio27, and @Gigabyte-Giant can see scores. However, you're not an admin yet, so you have to follow the instructions at the top so we can make you an admin.

We've finished everything on this list. @pamelafox, if you have more feedback or ideas, please make another list for us so we can complete it. Thanks!


Blocked CNAME of allowed domain not able to easily be allowed

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[ x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
I've tested this with 23.1.11 but  I don't believe any changes in 23.7 would have affected this.  If the CNAME of an allowed domain is on a blocklist, there is not a simple way to allow that query to successfully complete.
To Reproduce
Steps to reproduce the behavior:

Enable DNSBL in Unbound using the Steven Black list.
Enable Unbound Reporting
Query click.redditmail.com
Visit Unbound Reporting and whitelist click.redditmail.com
Configure Unbound to flush cache on restart and restart Unbound
Query click.redditmail.com
Visit Unbound Reporting and see that A and AAAA records are allowed but CNAME is blocked
CNAME shows as already whitelisted but query for click.reddit.com still fails

Expected behavior
Whitelisted click.redditmail.com should resolve
Describe alternatives you considered
Current workaround is to manually query the allowed domain, get the resulting CNAME and then whitelist that in the DNSBL screen.  Alternative would be for Unbound Reporting to show the both the original and resulting CNAME and/or recognize that the CNAME is what is still blocked and therefore show the whitelist option instead of block.
Probably the least feasible option would be for Unbound to realize that the original query is allowed and therefore not check the resulting CNAME against the blocklist.  This would likely cause more problems than it solves as well as reducing security provided by the blocklist.
Screenshots
https://imgur.com/N3dFp03
https://imgur.com/79oddsl
Additional context
See thread for more context.  https://forum.opnsense.org/index.php?topic=35218.0

Probably the least feasible option would be for Unbound to realize that the original query is allowed and therefore not check the resulting CNAME against the blocklist. This would likely cause more problems than it solves as well as reducing security provided by the blocklist.

This is entirely correct.

Alternative would be for Unbound Reporting to show the both the original and resulting CNAME and/or recognize that the CNAME is what is still blocked and therefore show the whitelist option instead of block.

The statistics are from the perspective of the incoming queries, therefore showing two entries here will likely pollute the counters. The current code is somewhat aware of this situation, but pretends to be the original query so domains don't appear out of nowhere for the person reviewing the logs. The easiest solution would likely be to drop logging the incoming query (e.g. click.redditmail.com) and instead log <resulting CNAME> (CNAME of <original queried domain>).

Alternative would be for Unbound Reporting to show the both the original and resulting CNAME and/or recognize that the CNAME is what is still blocked and therefore show the whitelist option instead of block.

The statistics are from the perspective of the incoming queries, therefore showing two entries here will likely pollute the counters. The current code is somewhat aware of this situation, but pretends to be the original query so domains don't appear out of nowhere for the person reviewing the logs. The easiest solution would likely be to drop logging the incoming query (e.g. click.redditmail.com) and instead log <resulting CNAME> (CNAME of <original queried domain>).

Isn't that exactly what I listed as the first alternative?

I am simply eliminating ambiguity.. "exactly" does not quite fit.

I've discovered another issue related to Unbound DNSBL and CNAME records.  I'm not sure if it needs to be opened as a separate issue or rolled into this one.
https://forum.opnsense.org/index.php?topic=36068.0
The TLDR is that CNAME records that resolve to a CNAME record appear to be broken.

I am also experiencing the same issue. Some domains in my "Whitelist" work fine but the most recent one added does not, regardless of restarts and waiting out TTL.
Interesting observations that I hope can shed light on this:

REPORTING: UNBOUND DNS logs it as a "block", but the "Command" button only offers to "Block Domain", when "Whitelist Domain" is expected
Resolution issues coming from the browser with the following error "DNS_PROBE_POSSIBLE"
Using nslookup (windows) does resolve the domain in question (I see it as a block in REPORTING: UNBOUND DNS
Domain in question: "learn.cisecurity.org", being blocked from at least AdAway and EasyPrivacy lists

Hello, I'm currently using the versions listed below and I can confirm this bug
OPNsense 23.7.11-amd64
FreeBSD 13.2-RELEASE-p7
OpenSSL 1.1.1w

The domain I tried to whitelist is mparticle.weather.com but unbound still blocks the requests with the same behavior reported by @CJ-Ross and @AJSNetworks even after whitelisting and reloading the service

Hello,
Running same version reported by OP here and same issue.
OPNsense 23.1.11_1-amd64 FreeBSD 13.1-RELEASE-p8 OpenSSL 1.1.1u 30 May 2023
In this case the domain that is not being whitelisted is go.missionbio.com.	2250	IN	CNAME	go.pardot.com.
Disabling DNSBL "solves" the issue, but is not really what we want.

I have plans to work on this issue along with some other improvements to the whole DNSBL handling and reporting but with everything else I have going on I have no idea when I'll be able to dedicate time to it.
In the meantime, anyone coming across this issue can use a DNS lookup tool(local or online) to find out what the next CNAME in the chain is until they get to the actual domain and unblock each one of those.  While cumbersome, it does work to allow you to access something that is on a downloaded DNSBL.

I must be doing something wrong. In order to watch live sports on Paramount+ I have to allow pubads.g.doubleclick.net. I added it and its other cname pubads46.g.doubleclick.net to whitelist domains and its still blocked by DNSBL. I don't watch often but when I do I have to disable blocklist altogether and clear my local dns cache to get it to work.

The problem is still here...
Any news?


Update project visuals: new favicon and refined

colors

Cam you make site responsive ....

Ok. I will get it done

Raise the issues


Error when compiling

8   const trackNames = []
          ~~~~~~~~~~

node_modules/soundcloud-downloader/src/download-playlist.ts:10:24 - error TS2345: Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
  Type 'undefined' is not assignable to type 'string'.

10     const p = download(track.permalink_url, clientID, axiosInstance)
                          ~~~~~~~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/download-playlist.ts:15:19 - error TS7005: Variable 'trackNames' implicitly has an 'any[]' type.

15   return [result, trackNames]
                     ~~~~~~~~~~

node_modules/soundcloud-downloader/src/download.ts:91:29 - error TS2532: Object is possibly 'undefined'.

91   return await fromMediaObj(info.media.transcodings[0], clientID, axiosInstance)
                               ~~~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:1:19 - error TS7016: Could not find a declaration file for module 'soundcloud-key-fetch'. 'D:/Bernd/BerndPC/Documents/Github/Javascript/Discord-MusicBot/node_modules/soundcloud-key-fetch/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/soundcloud-key-fetch` if it exists or add a new declaration (.d.ts) file containing `declare module 'soundcloud-key-fetch';`

1 import sckey from 'soundcloud-key-fetch'
                    ~~~~~~~~~~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:24:32 - error TS2532: Object is possibly 'undefined'.

24   const filtered = filterMedia(info.media.transcodings, { format: format })
                                  ~~~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:51:3 - error TS2564: Property 'STREAMING_PROTOCOLS' has no initializer and is not definitely assigned in the constructor.

51   STREAMING_PROTOCOLS: { [key: string]: STREAMING_PROTOCOLS }
     ~~~~~~~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:52:3 - error TS2564: Property 'FORMATS' has no initializer and is not definitely assigned in the constructor.

52   FORMATS: { [key: string]: FORMATS }
     ~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:57:3 - error TS2564: Property 'axios' has no initializer and is not definitely assigned in the constructor.

57   axios: AxiosInstance
     ~~~~~

node_modules/soundcloud-downloader/src/index.ts:247:5 - error TS2322: Type 'string | undefined' is not assignable to type 'string'.
  Type 'undefined' is not assignable to type 'string'.

247     return this._clientID
        ~~~~~~~~~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:274:7 - error TS2322: Type 'string | undefined' is not assignable to type 'string'.
  Type 'undefined' is not assignable to type 'string'.

274       return this._clientID
          ~~~~~~~~~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:287:7 - error TS2769: No overload matches this call.
  Overload 1 of 4, '(path: number | PathLike, options: { encoding?: null | undefined; flag?: string | undefined; } | null | undefined, callback: (err: ErrnoException | null, data: Buffer) => void): void', gave the following error.
    Argument of type '"utf8"' is not assignable to parameter of type '{ encoding?: null | undefined; flag?: string | undefined; } | null | undefined'.
  Overload 2 of 4, '(path: number | PathLike, options: string | { encoding: string; flag?: string | undefined; }, callback: (err: ErrnoException | null, data: string) => void): void', gave the following error.
    Argument of type '(err: NodeJS.ErrnoException, data: string) => void' is not assignable to parameter of type '(err: ErrnoException | null, data: string) => void'.
      Types of parameters 'err' and 'err' are incompatible.
        Type 'ErrnoException | null' is not assignable to type 'ErrnoException'.
          Type 'null' is not assignable to type 'ErrnoException'.
  Overload 3 of 4, '(path: number | PathLike, options: string | { encoding?: string | null | undefined; flag?: string | undefined; } | null | undefined, callback: (err: ErrnoException | null, data: string | Buffer) => void): void', gave the following error.
    Argument of type '(err: NodeJS.ErrnoException, data: string) => void' is not assignable to parameter of type '(err: ErrnoException | null, data: string | Buffer) => void'.
      Types of parameters 'err' and 'err' are incompatible.
        Type 'ErrnoException | null' is not assignable to type 'ErrnoException'.
          Type 'null' is not assignable to type 'ErrnoException'.

287       fs.readFile(filename, 'utf8', (err: NodeJS.ErrnoException, data: string) => {
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
288         if (err) return reject(err)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
...
309         }
    ~~~~~~~~~
310       })
    ~~~~~~~~

node_modules/soundcloud-downloader/src/index.ts:322:31 - error TS2322: Type 'string | undefined' is not assignable to type 'string'.
  Type 'undefined' is not assignable to type 'string'.

322       if (isFirebaseURL(url)) url = await convertFirebaseURL(url, this.axios)
                                  ~~~

node_modules/soundcloud-downloader/src/info.ts:168:24 - error TS2345: Argument of type 'number' is not assignable to parameter of type 'never'.

168       splitIds[i].push(ids[x])
                           ~~~~~~

node_modules/soundcloud-downloader/src/likes.ts:43:23 - error TS2532: Object is possibly 'undefined'.

43   while (nextHref && (options.limit > 0 || options.limit === -1)) {
                         ~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/likes.ts:53:89 - error TS2769: No overload matches this call.
  Overload 1 of 2, '(...items: ConcatArray<never>[]): never[]', gave the following error.
    Argument of type 'Like' is not assignable to parameter of type 'ConcatArray<never>'.
      Type 'Like' is missing the following properties from type 'ConcatArray<never>': length, join, slice
  Overload 2 of 2, '(...items: ConcatArray<never>[]): never[]', gave the following error.
    Argument of type 'Like' is not assignable to parameter of type 'ConcatArray<never>'.

53     query.collection = query.collection.reduce((prev, curr) => curr.track ? prev.concat(curr) : prev, [])
                                                                                ~~~~

node_modules/soundcloud-downloader/src/likes.ts:54:10 - error TS2454: Variable 'response' is used before being assigned.

54     if (!response) {
            ~~~~~~~~

node_modules/soundcloud-downloader/src/likes.ts:63:7 - error TS2532: Object is possibly 'undefined'.

63       options.limit -= query.collection.length
         ~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/likes.ts:65:11 - error TS2532: Object is possibly 'undefined'.

             ~~~~~~~~~~~~~

node_modules/soundcloud-downloader/src/likes.ts:79:10 - error TS2454: Variable 'response' is used before being assigned.

79   return response
            ~~~~~~~~

Found 20 errors.```

This is the error log from tsc

@zackradisic It is set to true and still throws errors. i have "skipLibCheck": true, in my tsconfig

@kaaax0815 Ok try importing the lib like this:
import scdl from 'soundcloud-downloader/dist/index'

This will force import of the transpiled JavaScript, instead of the Typescript you will have to compile. This is not an ideal solution and I'm working on v2.0.0 which fixes these problems. Please let me know if you still have problems

Could not find a declaration file for module 'soundcloud-downloader/dist/index'. 'd:/Bernd/BerndPC/Documents/Github/Javascript/Discord-MusicBot/node_modules/soundcloud-downloader/dist/index.js' implicitly has an 'any' type.
Try npm i --save-dev @types/soundcloud-downloader if it exists or add a new declaration (.d.ts) file containing declare module 'soundcloud-downloader/dist/index';

@zackradisic Do you have any support patch until 2.0.0

@kaaax0815 Yeah forgot to mention that importing the transpiled JavaScript means you won't get any typings, and you'll have to declare the module in a .d.ts file. This is probably the best option for right now.
If you still want typings for intellisense, I'm working on v2.0.0 on the v2.0.0 branch. You can install the module directly from there npm install zackradisic/node-soundcloud-downloader#v2.0.0. I can't quite remember, but I think I fixed the type errors there. Just keep in mind that I'm developing on this branch and so I'm likely going to be introducing breaking changes.

@zackradisic okay ill go with the custom decleration. but i would recommend you only ship the js with the decleration. that is what most do. but thanks for your help

Do you have a full declaration? because if i import it from the typescript path I get errors again

@zackradisic Do you have a full declaration? because if i import it from the typescript path I get errors again. And If i declare an empty module everything is 'any' which defeats the whole purpose of typescript

Yeah so the declarations file was lazily made and imports the .ts files, which does not help your problem 😅
Have never heard of dts-bundle, but it seems to have done the job, nice find!


Cookie testing

Change description
Testing google analytics with the cookie banner from 1290 & 1277.
This PR should not be merged.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Other PRs cover this change, closing this one.


Explicitly request RWO storage for tests

What this PR does
Test "should reject a migration of a vmi with a non-shared data volume" requries a RWO Datavolume in order to actually test non-shared volume.

Fixes #
Why we need it and why it was done in this way
Links to places where the discussion took place:
Special notes for your reviewer

Release note

NONE

/cc @akalenyu

/cherry-pick release-1.4
Other releases should not be affected.

/approve

/approve


role for creating easuser type ldap_user accounts

Hi
Can someone please guide me on which role to use to created the easuser type accounts?
Thanx
DJW

There is no role for this at present - please add one. You can clone set_ldap_user_pwd and call is add_ldap_user perhaps. The role needs to call the python function ibmsecurity.isam.web.embedded_ldap.user.add()

Hi
I'll try but am no prorammer and am not familiar with python so don't
laugh.
Challenge accepted !
Regards
Djw
On Fri, 08 Mar 2019, 19:33 Ram Sreerangam<EMAIL_ADDRESS>wrote:

There is no role for this at present - please add one. You can clone
set_ldap_user_pwd and call is add_ldap_user perhaps. The role needs to call
the python function ibmsecurity.isam.web.embedded_ldap.user.add()
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IBM-Security/isam-ansible-roles/issues/109#issuecomment-471011144,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ak07siFqHi1y_h2sB5xJyyxkJSw0aKSqks5vUp7mgaJpZM4bkylm
.

I have some code i can share later this evening. Now enjoying skiing.


include: net: remove __packed from network structs

GCC 9 produces a warning about taking the address of a member
of a packed structure. Since the networking headers should all
use unaligned accesses and are manually packed then we can
remove the attribute.
Fixes #16587

@nashif - you may want to take a look at this as well.

All the structs that are describing network packet and are typically cast directly to memory need to have __packed in them so that we do not get any holes in them. I do not think this is a correct thing to do here.

That's what I would have thought as well. However, @ozhuraki thought otherwise, according to this.

Without __packed the compiler is free to insert padding between struct member fields to avoid unaligned access. With __packed, the alignment is set to 1 and therefore no padding is added (at the cost of potentially expensive unaligned access).

Right. I know what __packed does but I thought I would investigate @ozhuraki 's claims.
https://github.com/zephyrproject-rtos/zephyr/issues/16587#issuecomment-912338336
The implication is that fields are manually aligned and use byte-aligned accessors.

This change will break direct serializing of the network structs.

It does not break any of our existing unit tests that I can tell.


As a User I want to be able to join tournaments outside of my organisation so that I can compete in inter-company competitions

Great idea!


make app-tools-lib-for-java the default runtime

@nbashirbello

To clarify, this change implies that when you create an App Engine project (non-maven and maven) via the New Project Wizard, a new app-tools-lib runtime will be created if it didn't exist?

yes

LGTM


DDH Manager Ping Worker, Worker State Checked

DDH Manager Ping Worker, Worker State Checked
DDH Manager 周期向 Worker 发送 ping 命令，对 Worker 状态进行检测，并把检测状态结果维护到 Host 表中

Can you upgrade your high-risk dependencies?

upgrade depends to highest vesion


Multiple fixes to the data set and code to get things working

Fixes to the data csv sheets as well as named column headers to get code working

I resolved the conflicts


Adding integration not working

Region and Brand of car
Europe/Kia e-Niro
Describe the bug
Add the integration will not work (See screenshots)
Debug logs if an error occurred

Nothing interesting :
2022-06-04 23:32:40 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-06-04 23:32:40 WARNING (SyncWorker_1) [homeassistant.loader] We found a custom integration kia_uvo which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
Outlined here: https://github.com/fuatakgun/kia_uvo/blob/master/README.m
To Reproduce
Steps to reproduce the behavior:

Go to Integration
Click on Add
Search for "UVO"
Click on it
-> Error after 30sec

Expected behavior
Add the integration is successful
Screenshots

I installed it with HACS

After reinstallation it is working, I checked HACS experimental features, maybe it helped.

Please enable debug logs and post those.

On Jun 4, 2022, at 3:47 PM, tho071088 @.***> wrote:

I installed it with HACS
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.


Create global text styles

Bootstrap allows:
text-muted
text-danger
text-warning
text-success
text-primary
We should add similar global text color classes.

Not strictly needed.  Options:

Use bundled material-ui palette colors: http://www.material-ui.com/#/customization/colors
Define colors in palette.css.  Can the defined color names (e.g. "lightAccentColor") be directly referenced within components or do they have to be re-defined in the .css file for the component?  @mike-robertson ?


Add variable resolvement to "image" template tag

It would be super useful to allow us to pack image formats into variables.
For example:
## models.py
class Banner(Model):
    SIZES = [
        ('fill-200x200', 'small'),
        ('fill-400x400', 'big'),
    ]
    image = ForeignKey(Image)
    size = CharField(choices=SIZES)

{# template.html #}
{% image banner.image banner.size %}

Hacked solution:
## wagtailimages_tags.py
class ImageNode(template.Node):
    ...
    def render(self, context):
        try:
            self.filter_spec = template.Variable(self.filter_spec).resolve(context)
        except template.VariableDoesNotExist:
            pass
        ...

Duplicate of #2090

@diveru4i I was thinking about this idea I call "filter-spec macros" which I think might help with this.
You can add your own filter specs to Wagtail using the register_image_operations hook: https://github.com/torchbox/wagtail/blob/master/wagtail/wagtailimages/wagtail_hooks.py#L69-L77. "macros" would allow you to build filter specs from others and use them in your template:
@hooks.register('register_image_operations')
def register_image_operations():
    return [
        ('banner_small': MacroOperation(r'^$', 'fill-200x200'),
        ('banner_big': MacroOperation(r'^$', 'fill-400x400'),
    ]

Then in the template: {% image myimage banner_big %}
Just to describe the idea, here's an example which has parameters (example from #1694):
@hooks.register('register_image_operations')
def register_image_operations():
    return [
        ('pad_with_bg': MacroOperation(r'^(\d+)-(\d+)-([0-9a-f){6}$', 'pad-{0}-{1} bg-{3}'),
    ]


[Feature] RPC rate limiter

🚀 Feature

Add controls that allow snarkOS to adjust 2 rates:
a. The rate that RPC requests from all IP addresses are processed
b. The rate that RPC requests from a single IP address is processed

Add a "restricted_peers" list for RPC IP addresses that exceed the rate.

Motivation
Some IP addresses are spamming the sync node RPC endpoints.

Note: jsonrpsee just declared that it's more or less stable a few days ago; I'll check it out, it might be the time to close https://github.com/AleoHQ/snarkOS/issues/754 as well.

Oh, and just so that no one else tackles this in parallel, I'm already working on this.

What's the status on this?

Let's consider the the things we've used the RPC server for so far:

obtaining various information on our own node
actively interacting with our own node
providing data for the Aleo explorer from the a dedicated Aleo node
taking some measurements of the entire network

And some of the related issues we've encountered:

sync nodes (and possibly any other node with their RPC server open to the network) being overwhelmed with the volume of RPC requests, slowing them down

[x] resolved by setting a global jsonrpsee resource limit

sync nodes were sometimes getting a lot of requests from specific hosts (indicating malicious intent), slowing down the rate at which legitimate requests could be processed

[ ] open issue (the remaining part of this very issue)

many nodes have had their RPC server closed/unreachable, causing network-wide measurements to be only partial

[x] mitigated by the freshly merged network-based crawler, which will be extended with further features

some users didn't want to expose the RPC server at all (e.g. #1355) due to security concerns

[ ] partially mitigated by using a dedicated RPC server implementation (as opposed to a "hand-written" one), but still open

Having all the above in mind, I'd like to propose to start shifting towards dedicating the RPC server to local/restricted use; this means:

setting the default RPC server IP address to <IP_ADDRESS>
using jsonrpsee's AccessControl to restrict the hosts that can send RPC requests to the sync nodes (this would be employed by the nodes we want to use for Aleo explorer's purposes)
only using the network-based crawler to obtain much more complete information on the whole network (while also distributing lists of peers)

This proposal would completely resolve any concerns related to malicious RPC use; even if we used per-IP bans, a well-organized malicious actor could still set up a lot of RPC clients that would individually send requests at a rate right above the one we permit.
Cc https://github.com/AleoHQ/snarkOS/issues/1381


Fighting Gloves - Unit Test Cases, OR IF THAT'S NOT POSSIBLE (it wasn't), manual testing cases

Which are the cases needed?
OK (#112) - I have to MAKE a basic structure for test cases.
OK (#113) - Values added to the charge variable are really added.
OK (#117) - If the variable receives a value and ends as lower than zero, it becomes zero and triggers a logger error message.
OK (#118) - The save system saves the charge variable.
OK (#119) - The load system loads the charge variable.
OK (#120) - The clean system cleans the charge variable.
OK (#121) - The charge variable increases by one every 5 rooms entered.
OK (#123) - Add visible number of charges.
OK (#124) - The save, load and clean systems are updated for the new variables.
OK (#125) - Clicking with the right button on it in a Rest Room (and nowhere else) gives a message and zeroes the positive charges.
OK (#126) - Clicking with the right button on it in a Rest Room (and nowhere else) gives a message, opens the card list if there are more cards than upgrades and let's the player choose the cards, zeroing the positive charges at the end.
OK (#130) - Clicking with the right button as above if there are more charges then card to upgrade does the same as above but the charges used are only the number of cards left to upgrade.
OK (#131) - If there aren't cards to upgrade, the relic does not work.
OK (#132) - Make relic reacts when it gains a charge.
OK (#133) - Make relic reacts when has its charges spent.
OK (#135) - Update description
OK (#138) - Add proper description to the card grid screen.

The tests will wait until I have an answer from the devs. Let's keep programming the relic.

Ok, all done.


Need to explain big increase in recent estimated recruitments compared to last year's assessment

e.g. we now think there were 1.76 billion more fish spawned in 2010 than we did in last year's assessment. That's based on medians - confidence intervals also increase.
I added some calculations (and commented example plots) in custom-knitr-variables.R, and preliminary text in exec summary. I was going to carry on doing a quick figure (showing medians and credibility intervals), but realise it will get done automatically when bridging model plots are updated (see commit that will get referenced below; it's on my andy-dev branch for now). So, we do need to

[x] make sure the bridging plot of recruitments clearly shows the increased 2010 estimate; may even need to make it just one bridging step in the plot
[x] add something to main text to explain this
[x] edit exec summary as necessary

Seems to me a big increase, so should be highlighted.

Aaron wrote some text in exec-summary that I already copied into main assessment section. Agree that the 2010 bump may not show up in bridging plots.
Aha - Aaron mentioned squid plot - there is a slight bump in 2010 one. I'll make sure to add something when reading through. So leaving open for now....

No tweaks to bridging plots, added text regarding squid plot in the above commit. Haven't crunched the numbers, but squid plot indeed suggests that it's the final year of data rather than model tweaks that bump up the 2010 recruitment.


ci: support ansible-lint and ansible-test 2.16

Fix yamllint issue with markdownlint config
Add cleanup for tests_include_vars_from_parent.yml
Signed-off-by: Rich Megginson<EMAIL_ADDRESS>

@richm I added a commit to return mssql-related vars and rescue block.

[citest]


Variable for string 942 causes mixed language sentence

String: 942 Official YouTube Creator - Last updated %time_ago%
Problem::  The replaced text is in English, so we end of up mixed languages.
Example::  "Kreator YouTube Rasmi - kemaskini terakhir pada a week ago"
I think this is because the string from relativeDate() is not localized.

Thanks for filing this issue @infinite-persistence ! Yea, that must be it. This also happens in other places we use relative dates. Want to help us look for a solution?

Actually, we do have it working in some places, but not on the homepage where I first checked.

Ok, I'll give this a shot. (I switched accounts due to key and various problems, abonding that account).
There are currently 2 ways of getting relative time:  the one mentioned above, and DateTime. They are not in sync:

(1) I'll try to sync the implementation of these 2 functions, unless told otherwise.
(2) Are there any rules on editing or removing existing string entries in app-string.json?

Looking further, I think I'll:

Leave relativeDate untouched.
Switch all usages of relativeDate in lbry-desktop to DateTime.  I think this one is more correct, and was recently adjusted to match YouTube.


Fix RLOO checkpointing

This PR fixes RLOO checkpointing (in the same way as a recent fix for PPOv2 PR #2080).
This is needed after changes to _save_checkpoint method introduced in transformers v4.45.0.dev. Specifically we will get KeyError: 'TrainerControl' during saving of the trainer state (here is the exact line causing the issue). By passing stateful_callbacks to OnlineTrainerState explicitly the TrainerControl object is stored and can be properly accessed in _save_checkpoint.

Nice, thanks @bartoszzuk. Without your fix, does it cause any error when running RLOO?

make sure to run make precommit by the way to make the CI happy

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@qgallouedec Yes when using transformers v4.45.0.dev I'm getting
...
File "/usr/local/lib/python3.10/dist-packages/trl/trainer/rloo_trainer.py", line 449, in train
  self._save_checkpoint(model, trial=None, metrics=metrics)
File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 3016, in _save_checkpoint
  if isinstance(self.state.stateful_callbacks[cb_name], list):
KeyError: 'TrainerControl'

Because the self.state.stateful_callbacks is an empty dict
Sorry, totally forgot about make precommit will fix it ASAP

Thanks. I'm not sure to understand why this failing mode doesn't break our CI

I am not sure if this is related, but I have observed strange behaviour in RLOO checkpointing. For example, I have set it to checkpoint every 500 steps and it follows that for some time, but after a while it starts generating checkpoints every 2 steps. Is this an intended functionality?

Hey @lewtun, sorry for late response. I added a simple regression test for RLOO checkpointing. Hopefully it somewhat follows the conventions found in other tests (let me know If any improvements are required). The test should:

Succeed without the fix for transformers<4.45.0
Fail without the fix for transformers>=4.45.0
Succeed with the fix for transformers>=4.45.0

I also changed the test function to ensure that tokenizer matches SFT and Reward model.

Succeed without the fix for transformers<4.45.0
✅
Fail without the fix for transformers>=4.45.0
✅
Succeed with the fix for transformers>=4.45.0
✅

Yep, not related


Impl onLoggedOut, update onLoggedIn to only add data.extra non-empty obj

Implemented the onLoggedOut hook for logout
Updated the onLoggedIn hook to only append data.extra when a non-empty object was returned
added tests for onLoggedOut hook in new file
moved onLoggedIn hook tests to same file as onLoggedOut hook tests
Updated README to no longer say '(coming soon)' for onLoggedOut hook data.extra

Looking good! I'm a bit busy today but I'll review this later tonight. If you don't hear back from me by tomorrow come back here and yell at me :) Thanks so much for taking care of all of this.
The one thing that jumps out to me that you'll probably think I'm crazy for is the commit messages. They are all slightly out of spec. The first needs to be shortened up, the second should use the imperative tone, and the 2nd and 3rd both need sentence capitalization. Just reword/rebase those and force push to this PR branch (otherwise you have to create a new PR – your choice). I do accept subject lines up to 80 chars, since those display properly everywhere in GitHub. I need to update the guidelines to include that asap.
Just know that I'm this crazy even to members of the Meteor Developer Group :) Nobody is exempt from my git OCD :P
Thanks again for what appears to be an awesome PR. I'll get this reviewed and merged in ASAP.

@kahmali must have missed this as a notification! I've created a new PR with a single commit (with better/shorter wording :) ).
I'm going to close this PR.


Update validations.py python script.

Closes: #1
Updated validations.py python script
Fixed the behavior of validate_user function in validation.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I fixed it again.


Review On Mouse Inputs Functionality Guide

Review documentation for Tutorial on Mouse Inputs Functionality Guide
The Introduction to Mouse Inputs Functionality Guide in SplashKit tutorial is checked the python and C++ are working without any errors.
Making sure that all C# Code follows top-level statements and made the changes fixed the mouse location functionality error in C# and minor changes to the text to make the content and code more coherent. This document covers all the checks and links to the PR for this audit.

Good tutorial review, and great work on this doc. Just one small change requested


cmake support to pass in a ssl library

This allows ATS to be built with BoringSSL

Looks like it is conflicting with OPENSSL_ROOT_DIR and OPENSSL_ROOT_DIR doesn't play nice with BoringSSL.  I will take a look at this later.

OPENSSL_ROOT_DIR doesn't play nice with BoringSSL

What was the issue? OPENSSL_ROOT_DIR worked for me several weeks ago.

creating a new pr to use OPENSSL_ROOT_DIR


fix infer predicates generate wrong result

We use two facilities to do predicate infer: PredicatePropagation and PullUpPredicates. In the prvious implementation, we use a set to save the intermediate result of PredicatePropagation. The purpose is infer new predicate though two equal relation. However, it is the wrong way. Because it could infer wrong predicate through outer join. For example
select a.c1
   from a
   left join b on a.c2 = b.c2 and a.c1 = '1'
   left join c on a.c2 = c.c2 and a.c1 = '2'
   inner join d on a.c3=d.c3

the predicates a.c1 = '1' and a.c1 = '2' should not be inferred as filter to relation a.
This PR:

revert the change from PR #22145, commit 3c58e9ba
Remove the unreasonable restrict in PullupPredicate.
Use new Filter node rather than new otherCondition on join node to save infer predicates

run buildall


Packing problem

I am using webpack configuration that described in this path:
https://angular.io/docs/ts/latest/guide/webpack.html
now I adde ng2-bootstrap to my project but when I tried to create a build version all angular2 files also added to my app.js file, what's wrong with this configuration or it is a bug in ng2 bootstrap?

error stack trace?

There is no error, I am trying to extract vendor and polyfills scripts to separate files and any java script codes that I am writing to another script file(app.js), any thing is good until I used ng2-bootstrap in my project,
now when I wanna run build and get dest files I see that vendor scripts(such as angular) added to my app.js.

I am afraid this is about webpack configuration issues
I would recommend using angular-cli
Regards


[Tracking issue] Emacs updates and QOL cleanups

This issue serves as a placeholder for all the issues and pull requests related to Emacs ecosystem in Nixpkgs.

https://github.com/NixOS/nixpkgs/issues/330595
https://github.com/NixOS/nixpkgs/issues/331053


Fix comment

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @CarsonF - to get the merge through, the CLA needs to be "signed": https://cla-assistant.io/sakuraapi/cli?pullRequest=14


Implement Sieve of Eratosthenes in prime.h

On a second thought, I started to feel like class is not necessary for this. What about having a function with the following signature?
vector<bool> sieve(const T &limit);

On a second thought, I started to feel like class is not necessary for this. What about having a function with the following signature?
vector<bool> sieve(const T &limit);

We can have such a function for users' sake. But I suggest we keep the Sieve class as well, so the code is consistent with other complex sieve classes.

I am also unsure about defining a free function version of class APIs. Let's ignore that for now.
I have added the unit tests and made some slight adjustments. PTAL.


All examples must include complete JSON

We must avoid abbreviating JSON outside of strings... for example:
this is ok
{
"foo": "bar...baz"
}

this is not.
{
"foo": { ...}
}

Discussed on July 8, 2021 WG call:

Example (only one?) of this: https://identity.foundation/edv-spec/#example-29-successful-data-vault-stream-read-response.

We could use some of this magic to make sure that "..."s aren't selectable if people try to copy-paste examples:
https://github.com/w3c/vc-data-model/blob/main/index.html#L108-L115

Addressed in PR https://github.com/decentralized-identity/edv-spec/pull/75 (using the appropriate div classes).


Tell if connected to D415 or D435

Required Info

Camera Model
D400

Firmware Version
5.10.3

Operating System & Version
Ubuntu 16

Kernel Version (Linux Only)
4.14.13

Platform
NVIDIA Jetson

SDK Version
2.16.1

Language
C++ / Python

Issue Description
I am using the Python and/or C++ APIs to talk to Realsense cameras. How can I tell if I am connected to a D415 vs. a D435? They take slightly different parameters, and I want to start a slightly different set of streams for both types of cameras. What is an easy (programmatic, through the API) way to tell which type of camera I am connected to?
I can upgrade firmware, update my library version, etc. if this is necessary to make this happen.

[Realsense Customer Engineering Team Comment]
Hi @mprat,
You can refer to the sample code "python-rs400-advanced-mode-disabled.py" to check the PID which is different between D415 (0AD3) and D435 (0B07).

Awesome, thanks!


Unable to set Smtp Type via UI

When setting up the EmailServerCredentials block via the UI, the field Smtp Type accepts only JSON type. Therefore the value must be supplied as quoted string, e.g. "INSECURE".
This results however in the following error message when trying to send a message:
ValueError: Must be one of ['SSL', 'STARTTLS', 'INSECURE']; got '"INSECURE"'

When the input field is left empty, the error message looks as follows:
ValueError: Must be one of ['SSL', 'STARTTLS', 'INSECURE']; got ''

Thanks for reporting this. There will be an incoming UI upgrade; in the meantime, a work around is saving the value in quotes and stripping the quotes afterwards:
from prefect_email import EmailServerCredentials
email_credentials_block = EmailServerCredentials.load("testing")
email_credentials_block = email_credentials_block.smtp_server.strip('"')

Once the UI is updated, you can edit the existing block to drop the quotes and code to drop strip:

Thank you for the temporary workaround. Which UI update will include the fix?
For anyone else stumbling upon this, the following workaround works for me. Note that both smtp_server and smtp_type have to be stripped and reassigned.
email_server_credentials = EmailServerCredentials.load("my-mail-block")
email_server_credentials.smtp_server = email_server_credentials.smtp_server.strip('"')
email_server_credentials.smtp_type = email_server_credentials.smtp_type.strip('"')

I couldn't find the issue; but I believe @desertaxle talked to the UI team about this.


Add Favicons for all platforms

Support for Apple, Android, Windows, and high res favicon options.
Added blade template sub-view to ease implementation.
Also, tangentially includes some NPM fixes

Fixes #42


add --version option with full credits

With short licensing info for all dependency libraries.

I can take this one as well!

Please do! However please try to do it smartly, not dumping all details of all the licenses on the output. Please also provide here links to all the license files in all the dependency libraries you tracked down.
Please then also try to verify that with some external tool, tailored specifically to checking licensing of source code, as it's not enough to check "LICENSE" files in the repositories to be reasonably sure. Please comment here with the link to the tool you used, and a copy of its report (here verbatim if reasonably short, or as a link to a gist/pastebin if it's very long).
Thanks!

The reporting tool that I decided to use is located here:
https://github.com/src-d/go-license-detector
I chose this tool because it has the most accurate results compared to other similar tools.
The output is as follows:
~/projects$ ./versions.sh
https://github.com/gdamore/tcell 100% Apache-2.0, 90% ECL-2.0
https://github.com/mattn/go-isatty 99% MIT, 93% JSON, 85% MIT-feh, 82% Xnet
https://github.com/spf13/pflag 93% BSD-3-Clause, 88% BSD-Source-Code, 79% BSD-3-Clause-No-Nuclear-License-2014, 78% BSD-4-Clause


Add tomcat8 at 8.5.16

This could use https://github.com/lukesampson/scoop/pull/1630

Why not use the existing tomcat.json from scoop-extras as tomcat85.json? Using tomcat8.json is a bit confusing since there is a tomcat80.json for the 8.0-releases.

Closing per https://github.com/scoopinstaller/versions/issues/44#issuecomment-338705862


Issue 841 fix - some arguments to the compiler might be processed as a path

Fixes the Issue 841
Detecting a file path within the arguments list by comparing a postfix with the file name does not work correctly.
Checking that the file-name argument should not start with the dash should fix it for now.

Merge is blocked by the pre-existing test failure


Fix issue #62. Set isCli true when lounch so on Windows

Thanks @darmo82! I will make a new release.

Looks good


Is it possible to create CDC in babelfish db

Hi @ALL,
Is it possible to enable change data capture in babelfish db.

Babelfish is adding support for PostgreSQL native logical replication in upcoming release. This might help to replicate changes from source tables to destination table. You can give it a try on BABEL_4_X_DEV or BABEL_3_X_DEV branches.


[BUG] Using Get.to with Tear-offs will generate a route with unnecessary characters

When navigating using Get.to<T>(() => Page()) with unnecessary_lambdas lint enabled, On IDE or Code Editor, we get suggestion to convert the function literal to a tear off.
i.e., Get.to<T>(() => Page()) becomes Get.to<T>(Page.new)
Tear off will add unnecessary characters to the navigation route name which is generated by GetX.
example:
void _onMenuPressed() {
    Get.to<void>(
      DrawerPage.new,
      binding: DrawerBinding(),
      transition: Transition.leftToRight,
    );
  }

On navigation, when we check the logs, we see
[GETX] GOING TO ROUTE /(%7BKey?%20key,%20_Location?%20$creationLocationd_0dea112b090073317d4%7D)%20=%3E%20DrawerPage
When we URL Decode it, we get
/({Key? key, _Location? $creationLocationd_0dea112b090073317d4}) => DrawerPage
Also, when i switch back Lambda for page field, everything is working as expected.
Result:
void _onMenuPressed() {
    Get.to<void>(
      () => DrawerPage(),
      binding: DrawerBinding(),
      transition: Transition.leftToRight,
    );
  }

[GETX] GOING TO ROUTE /DrawerPage
[GETX] CLOSE TO ROUTE /DrawerPage
Expected behavior
This should not be happening. However, when we navigate, there are no errors and it successfully navigates to the specified page. The problem occurs when we use
Get.until((route) => Get.currentRoute == 'DrawerPage');
This code fails as there is no route with this name in the stack due to the bug.
Flutter Version:
2.10.3
Dart Version:
2.16.1
Getx Version:
4.6.1
Describe on which device you found the bug:
I have tested this on:

Pixel 2XL - Physical Device
iPhone 12 - Physical Device
Pixel 2 (Android 11) - Emulator

This is really an interesting question, I hadn't tested this before the inclusion of new in dart, marking it here with a point of attention, because I'll see that later.

If my memory serves me right, constructor tear offs were introduced in Dart 2.15
Also, back to the point, i haven't had a chance to debug it completely from my side, but i have noticed on Get.to method, the runtimeType of the page parameter is used.
i belive constructor tear offs may not generate correct string as we require.


fixed router navigation

This addresses to issue #11

Main Component is displayed by default to "/"

Other route component will be added in the similar fashion as and when taken up, now it shows up empty.

@manjeetaR Thanks for your help..!


prevent line-wrapping in code blocks

Currently, when a line of code is longer than than the screen allows, the line wraps. This makes certain samples harder to read.
Adjust the css to prevent line-wrapping; offer a horizontal scrollbar instead.

Duplicate of #33


Replace the slow loop in Teddy with Aho-Corasick.

This can have a significant impact on performance when the match rate is
low (common) and the number of patterns is high (e.g., for a case
insensitive search).

As a meta comment: it's kind of silly I didn't just do this in the first place. There's almost no downside (sans a very small additional amount of memory required) and the code is far simpler than it was before.


Destructure payload from decode function properly

The decode function returns an object containing a header and payload properties. Assigning the whole object to payload fails nbf and exp checks on verify JWT as those properties not found in decode return object directly. Instead, destructure payload property from decode return data that contains those values and check them correctly.

Thank you so much for your PR, this must have slipped through :)


Step Functions definition is unreadable when generating Cloud Formation Template

Hello,
Is there a way to get the generated Step Functions definition nicely formatted inside the Cloud Formation Template?
Below is the current behavior. Is it a known issue ?

We don't have any current plans to optimize the format of the CloudFormation output. We see the synthesized CloudFormation templates as "compiled artifacts" which are meant to be read by machines and not humans. Can you please provide some more details about your use case and describe why the format of the CloudFormation output is of importance?

Our use case restricts us from directly using cdk.deploy() and therefore we would like to generate CF templates using CDK and deploy the stacks manually. And because of that having a "human" readable output would make the code review process for us much easier.

Thanks for the details about your use case. I doubt that this is something we will prioritize within the coming months, but we are open to contributions!

Thanks for the quick reply :)


Retry workflow for files that cannot be deposited

This issue is a placeholder for the whole branch of the workflow that will occur after we've identified the versions and permissions for all of a publication's files and determined that we don't have any files that can be deposited (see https://github.com/psu-libraries/researcher-metadata/issues/690). It may end up needing to be decomposed into a handful of separate issues depending on how we want this workflow to behave.
Some things that we may want to do automatically when we encounter this condition:

delete the files that can't be deposited from Activity Insight (if possible)?
update the publication's status in Activity Insight?
delete the local copies of files that cannot be deposited from RMD and update RMD's associated database records to indicate that the files were not able to be deposited
keep any metadata that we've gathered earlier in the workflow (confirmed DOI, open access information, etc.) that will be useful in the next iteration of the workflow if/when we receive a new file for the publication
either notify the faculty member directly that we couldn't deposit the file that they provided, or notify @nmg110 so that she can contact the faculty member and help them to provide the correct version

RE: either notify the faculty member directly that we couldn't deposit the file that they provided, or notify @nmg110 so that she can contact the faculty member and help them to provide the correct version
If it is a file that we cannot deposit what if they then get thrown in the OA Email group that automatically goes out from RMD? And then a list is of accounts, AI ID's, Titles, and file names are emailed to<EMAIL_ADDRESS>for those attachments to be removed from AI? Just a thought.

Sometimes faculty is not sure what the versions are and sending an email directly to them confuses them when the version is wrong, so sending an email to the OA group might be a better option than send to the faculty.

We'll need to determine what exactly kicks this off.

Thanks for uncovering this additional complexity right away, @anaelizabethenriquez.
I supposed that it's possible that if one author has a batch of publications that require action, then each of those publications may require action for a different one of the reasons that you listed?
I think that by the time that a publication/file reaches this point in the workflow, we'd have enough information about the version to automatically specify the reason why the file was rejected, so that may not be an issue. If that turns out to not be the case, then we could allow the RMD admin to choose a reason (or even a whole different email template) for rejecting the file.
I also don't think that it would be too difficult to group files in this list by the Activity Insight users that uploaded them and then send one email per group that lists each file. If the files were grouped in this way, would you still want the option to send an email per file in addition to the batch email?

If the application ends up automatically deciding the reason for rejection, then I imagine that we'd want the RMD admin to be able to see that reason before they send the email in order to verify that it's correct.
I'm imagining that we may need to indicate some sort of additional state in this list also. As we've discussed, a file will eventually drop off of this list automatically once the author uploads a new (correct) file for the same publication and that file passes the version checking step. But in the meantime, the file will remain on this list. So we'll want to keep track of whether or not action has been taken for a given file (that is, whether or not an email has been sent) and probably group those files at the bottom of the list so that the files that still require action float to the top.

Thanks, @EricDurante. This all sounds right to me. In particular:

I supposed that it's possible that if one author has a batch of publications that require action, then each of those publications may require action for a different one of the reasons that you listed?

Yes

If the files were grouped in this way, would you still want the option to send an email per file in addition to the batch email option?

I think so, because of the possibility that one or more files has something very weird going on and we want to just send an email about an "easy" file to get it out of the way.

So we'll want to keep track of whether or not action has been taken for a given file (that is, whether or not an email has been sent) and probably group those files at the bottom of the list so that the files that still require action float to the top.

Ah, right. Yes, we will definitely want to keep track of whether an email has been sent (and ideally, exactly what text was sent and when).

This all sounds good and possible to me.  To summarize we want to:

Include the reason the file is "wrong" in the list.  This can be automatically determined by comparing the preferred version and the determined version stored in RMD.
Group the publications by author.
For authors with groups of publications, add a button to send a single email to that author describing all the "wrong" files.
Add a button for every individual publication (even the ones in groups) to send an email out to that author for just that one publication.
Store and display info telling us if an email has already been sent.  We could track this with something like a last_email_sent_at timestamp.
Order the list so that publications who've already had email notifications sent are pushed to the bottom.

This is minor, but just to clarify: "Group the publications by author" means the author who uploaded the file in Activity Insight, not the author(s) listed in RMD.

Per discussion with @Smullz622 , we have just a single template email for all possible cases (single vs. batch and need accepted manuscript vs. need published version). Here's the proposed text:
From<EMAIL_ADDRESS>Reply-to<EMAIL_ADDRESS>Subject: Open Access Post-Print Publication Files in Activity Insight
Hello,
We are writing to follow up on the publication file(s) uploaded to your account in Activity Insight in the section titled “Open Access Post-Print Publication files."
When researchers upload files to this section, a team in the Libraries works to make them available to the public via ScholarSphere according to the default terms of the journal's publishing agreement. This process is designed for the “post-print” or “accepted manuscript” version of the article, because most journals allow that version to be shared widely. Some journals prefer that you share the final published version.
One or more of the files you uploaded appear to be a version of the article that the journal's default policy does not allow to be deposited in ScholarSphere. If you have the version of an article that can be deposited, you can send it by reply email or upload it in Activity Insight. Here is a summary of what we have and what we need:

Title
Version we have
Version that can be deposited

[Title1]
[ArticleVersion1]
[PreferredVersion1]

You can view a list of all your publications and their open access statuses on your Manage Profile Publications page. If you have uploaded other files in Activity Insight, we will process those files and update you in due course.
If you have any questions, please do not hesitate to contact us at<EMAIL_ADDRESS>All the best,
Office of Scholarly Communications and Copyright
Penn State University Libraries
<EMAIL_ADDRESS>

Thanks, @nmg110 ! I just updated the text to "One or more of the files you (or an Activity Insight Proxy) uploaded..."

Just a quick note to say that if "Preferred Version" is "none" for a publication, we don't want to include it in this email/bucket. Those items should be in a different bucket.


Logging functionality

There needs to be a way in which logs from remote machines are sent to the server machines as well and shown. These can then be passed to the reduce function. Idea is to implement this logging as socket.io streams, which are sent to server and server console.logs them.

Fixed with #3 as of release 0.1.0


[$250] IOU - Workspace name is duplicate on report header for submitter

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.64-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5238580&group_by=cases:section_id&group_order=asc&group_id=309128
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Precondition:

Login as the owner of the workspace
Create a workspace
Invite the approver and employee
Navigate to more features
Enable "workflows"
On the "Workflow" editor - enable "Add Approvals"
Set the Approver account as the Approver

Steps:

As employee submit 2 expenses in workspace chat
As approver open the report with 2 expenses
check the header to make sure it shows the Report ID, workspace and submitter.

As the submitter, notice the report ID and workspace name are shown twice.

Expected Result:
For the submitter, the report header should not include in %workspaceName% as it's repetitive. The workspace chat is the workspace name for them.
Actual Result:
For the submitter, we're repeating the workspace name.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/ba6230fb-1f75-4c13-926c-9019a52a34b6

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021859415514696293689
Upwork Job ID:<PHONE_NUMBER>696293689
Last Price Increase: 2024-11-21

Issue OwnerCurrent Issue Owner: @
Issue OwnerCurrent Issue Owner: @Christinadobrzyn

Asking the team for some feedback. https://expensify.slack.com/archives/C05LX9D6E07/p1732081764396159

Hello, I am Tomek from callstack, I would like to work on that issue

I was able to reproduce it, I am working on the fix not

Awesome! Let me know if you want me to move this to weekly while you work on the PR @rinej!

In some cases the workspace chat and workspace itself was the same, I added the condition check to diplay the workspace only once in such case.
Here is the PR with the fix: https://github.com/Expensify/App/issues/52763
And the video with the change:
https://github.com/user-attachments/assets/aea7db08-cc9c-4d4a-8fb7-e8e6645c5f23

Nice, that video looks pretty good to me

looks like the PR is under review - https://github.com/Expensify/App/pull/52914

monitoring PR https://github.com/Expensify/App/pull/52914

PR is already created with the fix, we are waiting for the C+ review

that's right, thanks for the update!

@thesahindia @Christinadobrzyn do you know who else can we ping for the review of PR? @jasperhuangg doesn't appear to respond

Sorry for all the labels, I'm trying add a new engineer to the issue. I'll reach out also to see if we can get someone to help.

Thanks @mallenexpensify! We've moved this to daily to get this on @jasperhuangg K2 for reivew

Thanks @thesahindia and @Christinadobrzyn! The PR is merged :)

Awesome! monitoring PR https://github.com/Expensify/App/pull/52914

Preparing for payment - @thesahindia do we need a regression test for this?

Payment day

Contributor+: @thesahindia owed $250 via NewDot

@thesahindia do we need a regression test for this?


apply headers with copyright/license text to source code files

@jeff-cohere @bnmurphy - what do each of our organizations want in terms of copyright/licensing? NCAR typically uses Apache 2.0, but I don't think there is any problem with using another license if EPA or DOE has stricter rules. I can also include each organization in the copyright statement if you send me the text you usually use.

Most DOE labs prefer the BSD 2- or 3-clause licenses. I haven't asked lately how they feel about the Apache one.
Here's some license boilerplate for an open-source tool I developed with funding from Sandia:
//-------------------------------------------------------------------------
// Copyright (c) 2021,
// National Technology & Engineering Solutions of Sandia, LLC (NTESS).
//
// Under the terms of Contract DE-NA0003525 with NTESS, the U.S. Government
// retains certain rights in this software.
//
// Redistribution and use in source and binary forms, with or without
// modification, are permitted provided that the following conditions are
// met:
//
// 1. Redistributions of source code must retain the above copyright
// notice, this list of conditions and the following disclaimer.
//
// 2. Redistributions in binary form must reproduce the above copyright
// notice, this list of conditions and the following disclaimer in the
// documentation and/or other materials provided with the distribution.
//
// 3. Neither the name of the Sandia Corporation nor the names of the
// contributors may be used to endorse or promote products derived from
// this software without specific prior written permission.

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

We use the Creative Commons license. There has been a lot of chatter about licensing software over the last 4-5 years. I think this might be mandatory for us. However, if you want to go with something different, I can make the case that I am merely contributing to this code which is distributed by other parties; it is not an EPA project.
EPA Disclaimer
This software/application was developed by the U.S. Environmental Protection Agency (USEPA). No warranty expressed or implied is made regarding the accuracy or utility of the system, nor shall the act of distribution constitute any such warranty. The USEPA has relinquished control of the information and no longer has responsibility to protect the integrity, confidentiality or availability of the information. Any reference to specific commercial products, processes, or services by service mark, trademark, manufacturer, or otherwise, does not constitute or imply their endorsement, recommendation or favoring by the USEPA. The USEPA seal and logo shall not be used in any manner to imply endorsement of any commercial product or activity by the USEPA or the United States Government.

I'm checking with our system admin, but I don't think there will be any problem on my end with using any of these licenses. It might be more difficult to resolve the DOE/EPA disclaimers. I guess we could include both? But, it seems like they overlap in terms of warranties, etc. How important do you think it is to include these? At least in the Apache license there is already language about there being no warranties, and I assume the same is true for these other licenses.
I was looking up Creative Commons (because I know the least about this one), and it seems like there are multiple Creative Commons licenses. @bnmurphy - does EPA have a particular one they use, or could we use any of them?


WRP-5951: Fixed to pass a ref object to theme libraries instead of a literal object

Enact-DCO-1.0-Signed-off-by: Seungcheon Baek<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Theme libraries can assume that a passing object reference is fixed, but useScrollBase is passing a literal object that is newly created for each call.
Resolution
Instead of a literal object, pass a ref object from useRef.
Additional Considerations
Links
WRP-5951
Comments

Would you please update your branch with the recent develop?


[In progress - questions] initial bar chart for test counts on hour chart

Initial bar graph for test counts

Questions:

Right now the barchart is rendering inside of the HourChart, because presumably we always want the test graphs. This makes things a little weird in the render method. Is this the correct way to do this? Should this be inside of the individual pages instead?

The export controls take a single id, but it is set on the SVG for the HourChart. How can the two share an SVG element so that both can be on a single element?

I'd love your thoughts @pbeshai and @vlandham.

My typical approach would be to have hour chart and count chart be separate and then create a new component HourWithCountChart that composed the two, but the way you have done it works as well.

This raises the problem of doing the data transformation inside the chart instead in a selector - if you now want to share the data between two charts, you either need to transform it twice or you need to nest them and pass the data from one to another. Third option is to create the composite component and do the data transformation there and pass it to both.

This will require some modification of how we compose the charts, but should be doable. I'll take a look today.

+1 to keeping them as separate components.
I think the composite component that holds the two the strategy i would suggest starting with. If we see lots of repeat patterns in data processing, we can move them up to a utility or into the api somehow.

@vlandham I composed the charts as HourChartWithCounts. You must pass inSvg to HourChart to have it be rendered inside another SVG element.
Renamed BarChart to CountChart and made it slightly less generic. We may want to refactor a bar chart out at some point... but until that point, let's keep it less generic. I expect it will make things simpler.


Fix/wgpu/matmul web

Checklist

[ ] Confirmed that run-checks all script has been executed.

Related Issues/PRs
Fixes #2178
Changes
Work around the cubecl/wgpu issue by forcing the simple matmul strategy until read write strategy is fixed.
Testing
The MWE works and the mnist-inference-web results are coherent.

After some offline discussion we decided to go another route to solve the issue closer to the source.


Interface to formalize how tooling obtains the application's service provider

This a more general purpose alternative to the feature described in https://github.com/aspnet/EntityFramework/issues/8331.
Currently tools that require access to the application's runtime services and configuration (e.g. EF Core tools, ASP.NET Core Scaffolding, possibly Razor tooling and others) need to hardcode heuristics which

often require for tools to take additional unwanted dependencies, e.g. EF tools brings ASP.NET Core Hosting in 1.0 even if it can be used in non-ASP.NET Core applications
only work if the application follows specific (often version-specific) conventions to build the service provider (e.g. Startup.ConfigureServices() in ASP.NET Core 1.0, Program.BuildWebHost() in ASP.NET Core 2.0, etc.)
and require restoring to bespoke fallback mechanisms (like IDbContextFactory in EF) if the convention is not matched

This issue is about defining a common accessor interface that tooling can scan for at design time which provides access to the application's service provider. We believe this approach:

Could be used in ASP.NET Core applications as well as any other application that uses our DI
Could be leveraged by any tool that requires access to the application's services, rather than being component-specific
Removes the need of hardcoding heuristics in tools by moving the necessary code to the application
Enables project templates can incorporate an implementation of the interface that is tailored to the patterns they used to construct the application's service provider, e.g. for the ASP.NET Core 2.0 templates the implementation can return Program.BuildWebHost(args).Services. If the mechanism changes in the future, the template can be updated
Addresses all the concerns described in https://github.com/aspnet/EntityFramework/issues/8331, e.g. it allows EF Core tooling to discover any DbContext registered with AddDbContext regardless of their location, making IDbContextFactory unnecessary for most common cases

Example:
public class DesignTimeServiceProviderAccessor : IDesignTimeServiceProviderAccessor
{
    public IServiceProvider GetServiceProvider(string[] args)
        => Program.BuildWebHost(args).Services;
}

Open issues:

Still need to run the idea by the architect :smile:
General shape and naming of the interface (or abstract type?) is up in the air
Need to decide where we want to put it in templates

cc @bricelam @ajcvickers @DamianEdwards @davidfowl

This doesn't have anything to do with DI

Sure, where should we put it?

Honestly, I don't understand what this solves... Are we going to have shared code that calls into this interface or something?

Not necessarily, but yes we could.
As the title says and hopefully the description explains this interface formalizes the convention that tools need to implement in order to obtain the application's service provider at design time. E.g. instead of having to fake a host and instantiate Startup to call ConfigureServices() based on yesterday's templates or calling Program.BuildWebHost() based on tomorrow‘s template or whatever happens later or however customers decide to factor their code, we establish that tools will look for a type implementing this interface in the startup project and we start including one by default.
The main affinity with DI is that it is about obtaining the IServiceProvider. I guess it could go in a tooling abstractions package if we have such thing.

It is an interesting idea for Universal apps as well. There is always the problem of being unable to instantiate ViewModels at design time which have constructor arguments that are meant to come from DI at runtime. Everybody goes about crafting a solution, and they aren't always pretty. Sometimes they're awful. This isn't about 'tooling' per se, not in the .NET Core sense, but how could would it be if XDesProc could use this? Well, phase one would be me using this with if (DesignModeEnabled)...

@divega @davidfowl @ajcvickers @DamianEdwards @muratg @bricelam  - what is the status of this? We met last week, and I thought maybe we arrived at a conclusion, but I don't recall the concrete next steps. Whatever we decide, we should try to get it in this week.

@Eilon Diego, Brice, and I talked about it a little while ago. Brice is going to send some pull requests and we will take it from there.

Can you comment a bit on bringing some UI folks onboard? Xamarin, UWP? Is this something you see benefit in? Or should I raise this issue in a few other places?

@ericwj It's worth raising in the respective places.
I think it's a little less interesting with MVVM because most people probably aren't using Microsoft.Extensions.DependencyInjection.Abstraction there like we do in ASP.NET Core.

No, not today, but behold, .NET Standard is coming. I don't think there is mention of a date for UWP to arrive at 2.0, but once it's there, it'll make quite an impact I'm sure. Sharing basics like this just removes friction and making switching project types less of a culture shock. And it'd be opt in anyway I'm sure...
I've tried to find a suitable repo for XAML, but I couldn't really find one. Can you direct me somewhere, or should I pile it at the bottom of the LILO queue at https://visualstudio.uservoice.com?

VS Feedback would be a good place to start. Maybe dotnet/project-system would also get the right people looking at it.

And Xamarin? They don't have issues. Only pull requests 🥇

Sorry, no idea on that one

TY anyway, I'll get there

Clearing up milestone so that this can be re-triaged. We are not planning to do it for 2.0.0. Also some relevant comments in the conversation on the PR: https://github.com/aspnet/DependencyInjection/pull/527#issuecomment-302557915.

@divega What milestone should this go in?  2.2 or backlog?

@muratg I think it is fine for this to be in the backlog. It still seems nice and from time to time we hear from people that want something that works with DI without ASP.NET Core and without going all the way down to IDesignTimeDbContextFactory, but it does not seem to be a high priority.
Cc @bricelam and @ajcvickers in case they think we should plan to have it sooner.

This issue was moved to aspnet/Home#2342


Ensure FunctionEmitter synthetic vars don't name clash

This is a follow-up from #2800.

This will be easier to deal with after some "cosmetic" changes in the IR data structures, that I plan for M2, so I'll delay this issue until then.


A11y_VisualStudioCodeClient_Problems Panel_Validate that Eslint populates the problems view_Status Messages: Voice Over is not announcing any State like expand or collapsed when user is pressing enter on Python button in problems panel.

[Check out Accessibility Insights!]- Identify accessibility bugs before check-in and make bug fixing faster and easier.
GitHubTags:
#A11yWCAG2.1; #A11yTCS; #A11ySev3; #Visual Studio Code Client; #BM-VisualStudioCodeClient-MAC-Aug2023; #Voiceover; #MAC;  #WCAG1.3.1;
Environment and OS details:
Visual Studio Code insider for MAC version 1.82.0
Mac Version: 13.5
Screen reader: Voice Over
Repro Steps:

Open Visual Studio Code Insiders editor
Press ctrl+shift+M
With a JavaScript file open, activate the ESLint extension and use keyboard navigation to arrive at the "error & warning" output button on bottom left pane.
Navigate to the error output window ; hit ENTER when you arrive at each line item to go the destination of the error/warning.
Use keyboard navigation and other ATs to fix errors and remove warnings.
Validate that Eslint populates the problem view.
Turn on Voice over and observe that Voice Over is not announcing any State like expand or collapsed when user is pressing enter on Python button in problems panel.

Actual Result:
Voice Over is not announcing any State like expand or collapsed when user is pressing enter on Python button in problems panel.
Expected Result:
Voice Over should be announcing the State like expand or collapsed when user is pressing enter on Python button in problems panel.
User Impact:
Screen readers users will face difficulty if Voice Over is not announcing any State like expand or collapsed when user is pressing enter on Python button in problems panel.
Attachment:
https://github.com/microsoft/vscode/assets/98728920/748273f1-743a-4bcd-8c3f-d696319db840

With a JavaScript file open, activate the ESLint extension and use keyboard navigation to arrive at the "error & warning" output button on bottom left pane.

Can you please clarify where is the error & warning button in the UI?

Use keyboard navigation and other ATs to fix errors and remove warnings

What do you mean by ATs here?

Voice Over is not announcing any State like expand or collapsed when user is pressing enter on Python button in problems panel.

Can you please clarify where is Python button?

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


Aggregate documentations files into docs folder

thanks for this - I have this change in https://github.com/securesign/sigstore-ocp/pull/26 also - but I've been sitting on that for too long now so let's merge this and I'll rebase mine!

Moving PR to #124


Set sane JVM memory options based on cgroups limits

If Xmx is not set the JVM will use by default 1/4th (in most cases) of the host memory
This can cause the Kernel to kill the container if the JVM memory grows over the cgroups limit
because the JVM is not aware of that limit and doesn't invoke the GC
Setting it by default to 0.5 times the memory limited by cgroups, customizable with JVM_HEAP_RATIO

Maybe the heap could be over 0.5 of the memory, @kohsuke?
@reviewbybees

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

imho this should better be set in java parent image as this issue is not specific to jenkins.

the script could go into java but I don't know how hard could that be
JAVA_OPTS is a Jenkins specific

yes, but JAVA_OPTS is a reasonable common pattern, all derived images could then adopt

:bee:

Since you're using Bash, you should need bc. Instead, you can use the $((...)) arithmetic context.

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

Looks good to me. :bee:

I still think this should be implemented in Java docker image, as this
issue is not Jenkins specific
Le 29 déc. 2015 07:48, "Oleg Nenashev"<EMAIL_ADDRESS>a écrit :

Looks good to me. [image: :bee:]
—
Reply to this email directly or view it on GitHub
https://github.com/jenkinsci/docker/pull/182#issuecomment-167733757.

:bee:

actually it is better to set _JAVA_OPTIONS as it will be used by any child jvm process
http://progexc.blogspot.com.es/2013/12/what-i-discovered-while-trying-to-pass.html

Reworked to not require bc and use _JAVA_OPTIONS

Can we assume that the proper approach is now java -XX:+UnlockExperimentalVMOptions -XX:+UseCGroupMemoryLimitForHeap?

This is solved in new versions of JDK8 with UseCGroupMemoryLimitForHeap
https://blog.csanchez.org/2017/05/31/running-a-jvm-in-a-container-without-getting-killed/


SetBasePath method in ApiClient should be marked as obsolete

The SetBasePath method is marked as obsolete in every class except the ApiClient class.
It's current implementation also results in unintended behaviour:
var apiClient = new ApiClient();
apiClient.SetBasePath(Demo_REST_BasePath); // RestClient path is not updated
apiClient.SetOAuthBasePath(); // OAuth path is now updated based on new base path

This results in an ApiClient which is pointing at production for the Rest API and demo for the OAuth API.
Or another, more realistic use case:
apiClient.SetBasePath(account.BaseUri); // Has no effect on RestClient

The above will seemingly work for US accounts (as the account base path is likely the same) but will then fail if an EU users authenticates.
I suggest removing the implementation leaving only a stub and marking as obsolete for consistency across the repo.

Agree on that, the SetBasePath method is misleading for developers

Good news: Engineering is working on this issue. Will be fixed in the next release.

This has been resolved with v5.1.0 for v2.1 and v3.7.0 for v2


docs: Metadata changes

Added SEO changes to Kargo docs
In short:

meta title is inherited from the first H1 of the document
sidebar_label exists only in the context of navigating through the docs
description serves as metadata description and should be a bit longer (55+ chars)

@morey-tech please help  me out - I tried adding the Signed-off-by: Wojtek Cichoń<EMAIL_ADDRESS>line but it still somehow fails the tests. Would appreciate your help.

I tried adding the Signed-off-by: Wojtek Cichoń<EMAIL_ADDRESS>line but it still somehow fails the tests.

@wojtekidd don't worry about this. The real purpose of this is to offer your personal assurance that no one -- your government or employer for instance -- would object to you making this contribution. In your case, we already know that to be true.

@wojtekidd if you're happy with the current state, I am as well. Is this g2g from your perspective?

@krancour g2g please proceed


Is the apikey linked the kite user id or anyone can use it from the hosted environment by authorizing it.

It is linked to a user ID which you entered in the app's settings. Ask further queries like this in our forum. This issue section is only for the client library for .Net.


The target has dasappeared, but the model still has predicted results. How to solve it ?

Hi, there are some criteria you can use to judge the credibility of the tracking results (e.g. OSTrack estimated scores). When the scores are below a threshold, you can consider that the target has disappeared, and there are some other methods that you can find in long-term tracking papers as well.


AttributeError: 'AuthError' object has no attribute 'message'

Python3 exceptions have no "message" attribute. We should simply print the exception directly, since that produces the same text.

Resolved by this commit


Login screen Enhancements

After the user enters Username and password, the page should go to next page/Language page instead of clicking on the next button. The red Logout button is confusing to the user.
Maintain same blue color of button.

https://github.com/unfoldingWord/gitea-react-toolkit/pull/25/commits/6485f42741b3bcbaa2756cead9799c58f3bf01d1
Included in PR:

Should be available on beta.9 and above.

Verified in Beta-10. Works as expected. The red Logout button is now in Blue. Looks good.
When username and password are entered the page logs the user in and moves to the next page.


Feature request: fan device for PWM-controlled fan

See Discord.

Thank you! Will provide feedback as soon as there is something to test!

Could you try beta v1.8.25-0?

@ebaauw Bedankt. Thanks. It's working perfectly.
Amazing how all these tools work so nicely together. Super easy to install a beta version from HomeBridge UI, change the config, and see it all working in a glitch!
Thanks a million!

@ebaauw I was a little bit to fast. From Homekit it seems to be ok, but controlling the Fan from the Home App does not seem to trigger the actual commands. Switching it back to a Lightbulb does work, so it seems not all parts are fully connected yet.
Let me know if there is anything I can do to narrow it down more.

I suspect turning the fan on won’t work, but changing its speed does?

I suspect turning the fan on won’t work, but changing its speed does?

That's correct!

Could you try beta v1.8.25-1?

Could you try beta v1.8.25-1?

@ebaauw Just tested it, and works great, both on/off as well as setting a specific rotation speed.

Published v1.8.25.


Utils for alloy

In alloy, there is going to be an included Utils class that contains common rather simple self-contained algorithms to aid in writing code. For example, a constrain method. What other methods should be added?

A few math functions have been implemented on the latest version, including a sign function that returns the sign of a value (-1 if negative, +1 if positive). Aswell as a second power function that preserves the sign. Eg. pow2(-5, 2) = -25.


[Snyk] Fix for 2 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements/docs.txt

⚠ Warning
sphinx 5.3.0 has requirement docutils<0.20,>=0.14, but you have docutils 0.20.1.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

484/1000  Why? Has a fix available, CVSS 5.4
Cross-site Scripting (XSS) SNYK-PYTHON-JINJA2-6150717
jinja2:  3.1.2 -> 3.1.3
No
No Known Exploit

424/1000  Why? Has a fix available, CVSS 4.2
Information Exposure Through Sent Data SNYK-PYTHON-URLLIB3-6002459
urllib3:  2.0.6 -> 2.0.7
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Checkmarx One – Scan Summary & Details – b00e4047-78b2-4703-8eba-f070af268c12
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

Command_Injection
/src/werkzeug/_reloader.py: 177
Attack Vector

Hardcoded_Password_in_Connection_String
/examples/httpbasicauth.py: 39
Attack Vector

Hardcoded_Password_in_Connection_String
/examples/httpbasicauth.py: 39
Attack Vector

Missing_HSTS_Header
/src/werkzeug/serving.py: 266
Attack Vector

Path_Traversal
/src/werkzeug/_reloader.py: 177
Attack Vector

Path_Traversal
/src/werkzeug/_reloader.py: 177
Attack Vector

Path_Traversal
/src/werkzeug/_reloader.py: 177
Attack Vector

SSRF
/tests/live_apps/run.py: 10
Attack Vector

Log_Forging
/src/werkzeug/sansio/response.py: 260
Attack Vector

Fixed Issues

Severity
Issue
Source File / Package

Object_Access_Violation
/tests/live_apps/run.py: 10

Use_Of_Hardcoded_Password
/tests/test_wrappers.py: 181

Use_Of_Hardcoded_Password
/tests/test_http.py: 231

Use_Of_Hardcoded_Password
/tests/test_http.py: 154


release: v0.4.0

Codecov Report

Merging #76 (930a525) into main (df8cb5a) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   83.71%   83.71%
=======================================
  Files          30       30
  Lines        1578     1578
=======================================
  Hits         1321     1321
  Misses        219      219
  Partials       38       38

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Upstreaming of functionalities

Part of the developments in oemoflex could be upstreamed to the oemof dependencies. E.g. the Bev facade and some generalizations in postprocessing.

For the Bev facade, it would be best to wait for the adaption of oemof.tabular to oemof v0.4.0, as this will enable us to simplify the class using GenericStorage's constant loss terms.


cookie (and header) support

it would be good to have cookie and header support for some apis

your not active for 3 years nvm

your not active for 3 years nvm

How do you envision the API looking like?

RESTful api like niceeli/netech-backbone


feat(helm): allow the use of a existing priorityclass

Description
Allow to precise existing priorityclass instead of creating a dedicated one.
We use higher priority than the maximum user created one, which makes this DS have less priority than our "cluster" priority class. This makes the DS not able to provision on some nodes.
How can this be tested?

Set .Values.csidriver.existingPriorityClassName to an existing priority class name.

Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly
[x] I have read and understood the contribution guidelines

@aslafy-z are you planning on getting this merged? If not, we can finish it up and add this functionality at a later point.

@chrismuellner I did the requested changes. Please have a new look!


⚠ Bofrost.at has degraded performance

In c6200cd, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1091 ms

Resolved: Bofrost.at performance has improved in 80a894c.


More keyword args

[x] addresses some of the places in #8331
[x] Tests added / passed
[x] Passes pre-commit run --all-files

This was a fun one to do, as it was perfect for me to help me dig deeper into Dask.
Along the way I made some changes that add documentation that was not there previously:

Added out to elemwise
Added chunks to to_hdf5
Added split_every to einsum
Added like to arange

Also found some unrelated things:

Fix for 2 doctest fails in dask/array/slicing.py
Opened issue for #8353

Last, there are 2 changes I am not sure about

In to_tiledb there is a wired overwrite of the key from both the storage_options and key inputs.
Not sure what I did there makes sense, I am not a tiledb user...

I removed the add keyword and the associated test, this was removed >3y ago, hope it is ok)

Thanks for an amazing project!

Can one of the admins verify this patch?

I think test failures are unrelated. Ready for review.
Thanks!

This is ready for another pass (@jsignell if you have time...)
The failure is unrelated (TypeError: argument of type 'WindowsPath' is not iterable) for python 3.7 (I think this is related)
I also commented on the original issue with an updated list of kwargs.pop( locations.
Thanks again!

Not sure what happened with the docs, they are building fine locally...

Thanks @boazmohar for sticking with this! It's a nice improvement :)


Part 4 of completing S110

Continuing #1128, #1129, and #1130, this is the fourth and final PR in a series to complete the traits of S110, the strong ultrafilter topology. This one added the three remaining traits: P32 (Countably paracompact), P33 (Countably metacompact), and P194 (Submetacompact). In addition, I've included a proof for P22 (Pseudocompact), which was originally just "asserted in the general reference chart" of Counterexamples.
This PR got delayed a bit, mostly because I was waiting the earlier three parts to be approved so that, in the final part, I can check to make sure there is no redundant trait. (And then because of the holidays.) There is indeed none.

I reviewed Countably metacompact and Submetacompact. No problem for those.
I'll leave the other two to someone more knowledgeable.