CB-10242 API changes to support semi-private networks

Adds new optional fields 'usePublicEndpointAccessGateway' and
'endpointGatewaySubnetIds' to the EnvironmentNetworkV1Request API object. These
fields are persisted to the EnvironmentNetwork and NetworkDto BaseNetwork
objects. Adds new columns for these fields to the environment_network table
and saves them to the database.
Tested with unit tests, and by manually setting the new fields in the IDE and
verifying they were converted/persisted in the database correctly.
See detailed description in the commit message.

Thanks @hreeve-cloudera. I gave the PR a quick look. This is in an area that I am less familiar with, so I can only say that I don't see any obvious problems and someone else should give the PR a full review.
Thanks for adding several tests.

lgtm fwiw.
I have noticed that this PR does not contain:
* Using the provided flag / subnets in the LB creation

* logic to auto-fill the subnet to use if none has been provided.

Is the plan to add these as a follow up? I think it makes sense to merge this in since it is contained to the API related changes and persisting of those.

@enis Yes, I was going to add that logic as part of https://jira.cloudera.com/browse/CB-10095, which is partially done and just had placeholders for these API changes. I wanted to get this in first, so that neither of us will be blocked on it.

I noticed that the API response changed:

please modify the e2e test and EnvironmentNetworkTestAssertion to check the response
probably we should include this response into the public API
probably we should add this somehow to the UI as well

lgtm fwiw.
I have noticed that this PR does not contain:
* Using the provided flag / subnets in the LB creation

* logic to auto-fill the subnet to use if none has been provided.

Is the plan to add these as a follow up? I think it makes sense to merge this in since it is contained to the API related changes and persisting of those.

@enis Yes, I was going to add that logic as part of https://jira.cloudera.com/browse/CB-10095, which is partially done and just had placeholders for these API changes. I wanted to get this in first, so that neither of us will be blocked on it.

I noticed that the API response changed:
* please modify the e2e test and EnvironmentNetworkTestAssertion to check the response

* probably we should include this response into the public API

* probably we should add this somehow to the UI as well

Thanks @doktoric. I have some follow up question:

I'll update EnvironmentNetworkTestAssertion. For the e2e test, where exactly do I make those changes? I haven't modified the e2e tests before.
Is the public API the thunderhead API? @enis is making the Thunderhead API changes in https://jira.cloudera.com/browse/CDPCP-3435. If there's another place I need to make corresponding changes please let me know.
These options will be exposed in the UI during environment creation in https://jira.cloudera.com/browse/CB-10093. Is that what you mean, or do you mean we should add the updated response to the UI?

@doktoric I've added a new e2e test, EnvironmentWithPublicEndpointAccessGatewayTests, and have added validation logic for the new fields to EnvironmentNetworkTestAssertion.


Introduce ability to disable Panel Init Strategy wait for specified panels

For now PanelFactory implements following init logic:

init panel;
apply global Panel Init Strategy;
if present apply local init strategy (@WhenPanelInit annotation method)
But for some situations it is not acceptable. I assume we should implement ability to disable global panel init strategy for specified panels.

Use case
Application under test consists of two parts: frontend and backend. Let's assume frontend UI is powered by jQuery lib and backend is pure HTML. To deal with frontend WaitForJquery strategy was configured. In one of test cases work with both frontend and backend is required. But during init of panels on backend test fails, because of jQuery lib absence. Disabling solution will help to avoid its.
Mechanics
During panels init its class should be tested by presence of special annotation. If annotation present, global panel init strategy invoking should be skipped. This logic should be provided in PanelFactory.
Clases hierarchy
DisableGlobalInitStrategy - annotation (applicable only on class level)
Package com.github.invictum.panels

Implemented. Will be available on version 0.1.14+


[QUESTION] Am I allowed to embed the debloater into my application?

I will leave credit

I'll find a way to use this within my application I am using vb.net

.. why does this need to be asked?
It's under the MIT license so technically you can literally do whatever you want with it as long as you follow the conditions

.. why does this need to be asked?
It's under the MIT license so technically you can literally do whatever you want with it as long as you follow the conditions

you right, but it is also polite from him to ask...

I appreciate you asking! But yes, what @jebjordan said.


BLD: DOC: make building docs easier by parsing python version.

Taken over from NumPy.

Looks good to me. I've had trouble building locally on master since I use 3.7, and it looks like this should fix it. I'll test on Monday to make sure it does, but others feel free to merge in the meantime if you're happy.

This fixes doc builds for me, looks pretty straightforward, and is taken from NumPy, so we should be good here. Thanks @rgommers


Prepend the fork to the structs under Types

https://github.com/lambdaclass/lambda_ethereum_consensus/pull/779#pullrequestreview-1911698099

What do you think about, instead of having "Types.ExecutionPayload" and "Types.ExecutionPayloadDeneb", we can have "Types.Capella.ExecutionPayload" and "Types.Deneb.ExecutionPayload". This would make it simpler to change the code (only change aliases instead of changing the names everywhere, plus the changes in the use cases).

No longer relevant


Added js baldrick example

Thanks Richard.

although I'm hoping to replace this with Conduit - when its ready - https://github.com/CalderaWP/Conduit

Epic mate, let me know if I can help out at all :)


[Inference] Add debug mode

Debug mode with logs and some improvements & questions.

@KepingYan  could you help to review this and discuss with @Deegue


Switch between windows of the same program

Description of the new feature / enhancement
To have a keyboard shortcut like Alt-Tab (cycle through all programs windows) but restricted to all windows of the current focused application, like in MacOS, i.e. via the shortcut Alt-` (backtick) or, better, the key above Tab, for layouts other than ANSI-US.
There's already a GitHub repository by @akiver (see link in down below section) that, could be maybe further improved and embedded in, say,  PowerToys Keyboard Manager.
Scenario when this would be used?
Everyday, to improve productivity and replace the far less comfortable and limited shortcuts Win+(App#InTaskbar).
Supporting information
Link to existing GitHub repository: https://github.com/akiver/AltBacktick

This looks a duplicate of £10061

This looks a duplicate of #10061

Yes. it almost is. Sorry for not having found it before posting. Anyway, the quoted GitHub project here  does it a little better than Windows Switcher in #10061, so I'll post the same link there too.

Duplicate of #10061

/dup #278


Add MarketFactory.

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[X] Company name
[X] Company description
[X] Company location
[X] GitHub link
[X] How Elixir is used
[X] Sorted alphabetically

Sweeet!  Thank you @bdupharm 👍


ROADMAP

@nesterone
do you have any idea of roadmap for this project?

what is missing to release vue-vega as "final" release?

Hi, @xmnlab
I think it's about to fix:
Must Have:

[ ] Export the vega API #39
[ ] Exclude vega and vega lite libs from our js bundle #6
[ ] console.log #43

Nice to Have

[ ] Provide tooltip support #36

And 1.0.0 version would be ready to release

@xmnlab
Not sure about

dataset
and
line with point

Could you bring more info ?
I'm going to take a look at #39, #43  an #6
I you don't I can propose you to take a look at #36

Hey @nesterone !
datasets: https://vega.github.io/vega-lite/docs/data.html#datasets
line with points mark: https://vega.github.io/vega-lite/examples/line_overlay.html
but it is already working (using spec) with the last PR merged.
here is an example using dataset and line with points:
https://vega.github.io/editor/#/url/vega-lite/N4KABBYEQCQM4GMAWBTAtgQygLmkgLvgA5zYD0ZAbigOYYB0NAlvkgK4BG9TA9mYqkxVaGALQAbFiioAmegDZ6ABnoArODwB2UADThIUBFoBmTGjjChI16JSYoA7has3XUVGYIWAzEqV7XNwcmABNWCwAWP31XAF8YsFiAgxCMfAw4FHw4ZwTrKFT00QwZFA4OAE4K+Q4MYw4Adm95GTqEMqUIhoBGEPkUJQrjFAsAbTzAl0DAqAAPC27k6bc2TIAnTQw0EdwoNm7dCeXbDHE2HbBu5SObJJvrKeODedwZJaeIPfXN7Ys9g-eHyglFO5wsciU90gdw+jyBLzAbyhK2+Wwuexkhw++RBZwuV0hHxhTzhTzmPkBZNWKA2aL+bExlOOwNBF281yJNwAugliZ9KEZNAg0mMbqS3IUsLhxcsoD90ZLiqVylUanVGs1WsZ2hxOj0+gMhiNkYkmTYoCghTwQkxNOZpSb8kZxDw1rlsTNTChxCF6ajfmagfgAJ5EdGaHhoW2nKCO6GBmb4Hg8cT4JhEUUe1wyrPQL0+v00+VY3OJ0PhyPR8Sx0vxuPZ+uy-O+3a4sEJj1QENhv4ARzYGE0afSaeoNdriUbXI75oRObJzb+8xn0y75b7A6HLDSTDHjb52KgwfdE6gi9brJLtbXPd2-cHw53e9z8SJM-cKE8+AWAFZ-CaoEwNYAGsT07IgeFtb9cHwNZzhXAxu3RSRNGNN8AMycQUAQNMtDAw9MOwpM1iUboKnw3MoA4W0W2gRBThQHIELcS0jBtO0clwcYJ1nK8eOgY9G2sachMQ9ddigmlcXHD1XyeA9V2HLC-mQDA1m-d9gjCJBImiaY5IgHlYiAA

console.log #43

it seems #44 has already fixed that!

Hi, great project but I wonder what is going on with it. The last PR was merged 8 months ago... Any updates?

Any news @nesterone ..?


kubectl --prune does not delete last managed item(s) in namespace

Is this a BUG REPORT or FEATURE REQUEST?:
BUG
Kubernetes version (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.0", GitCommit:"91e7b4fd31fcd3d5f436da26c980becec37ceefe", GitTreeState:"clean", BuildDate:"2018-06-27T20:08:34Z", GoVersion:"go1.10.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.3", GitCommit:"a4529464e4629c21224b3d52edfe0ea91b072862", GitTreeState:"clean", BuildDate:"2018-09-09T17:53:03Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Environment:

Cloud provider or hardware configuration: baremetal/vmware cluster
OS (e.g. from /etc/os-release): CoreOS 1855.4.0
Kernel (e.g. uname -a): 4.14.67-coreos
Install tools: vanilla kubernetes
Others:

What happened:
kubctl apply --prune won't delete a resource (or multiple resources) if there is no other resource with the selected label left in this namespace.
What you expected to happen:
prune should delete all resources with the specified label if they are not in the set of applied resources.
How to reproduce it (as minimally and precisely as possible):
Apply a pod to a namespace and two pods to another namespace.
$ cat all.yaml
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: default-http-backend
    testlable: managed
  name: default-http-1
  namespace: first
spec:
  containers:
  - image: gcr.io/google_containers/defaultbackend:1.4
    imagePullPolicy: IfNotPresent
    name: default-http-backend
    ports:
    - containerPort: 8080
      protocol: TCP
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: default-http-backend
    testlable: managed
  name: default-http-1
  namespace: second
spec:
  containers:
  - image: gcr.io/google_containers/defaultbackend:1.4
    imagePullPolicy: IfNotPresent
    name: default-http-backend
    ports:
    - containerPort: 8080
      protocol: TCP
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: default-http-backend
    testlable: managed
  name: default-http-2
  namespace: second
spec:
  containers:
  - image: gcr.io/google_containers/defaultbackend:1.4
    imagePullPolicy: IfNotPresent
    name: default-http-backend
    ports:
    - containerPort: 8080
      protocol: TCP

$ kubectl apply --prune -l 'testlable=managed' --cascade=true -f all.yaml
pod/default-http-1 created
pod/default-http-2 created
pod/default-http-1 created

If you delete one pod definition in the second namespace prune apply works as expected:
$ cat /only-two.yaml
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: default-http-backend
    testlable: managed
  name: default-http-1
  namespace: first
spec:
  containers:
  - image: gcr.io/google_containers/defaultbackend:1.4
    imagePullPolicy: IfNotPresent
    name: default-http-backend
    ports:
    - containerPort: 8080
      protocol: TCP
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: default-http-backend
    testlable: managed
  name: default-http-1
  namespace: second
spec:
  containers:
  - image: gcr.io/google_containers/defaultbackend:1.4
    imagePullPolicy: IfNotPresent
    name: default-http-backend
    ports:
    - containerPort: 8080
      protocol: TCP

$ kubectl apply --prune -l 'testlable=managed' --cascade=true -f all.yaml
pod/default-http-1 unchanged
pod/default-http-1 unchanged
pod/default-http-2 pruned

But if I want to delete both resources from the second namespace, nothing happens:
$ cat only-one.yaml
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: default-http-backend
    testlable: managed
  name: default-http-1
  namespace: first
spec:
  containers:
  - image: gcr.io/google_containers/defaultbackend:1.4
    imagePullPolicy: IfNotPresent
    name: default-http-backend
    ports:
    - containerPort: 8080
      protocol: TCP

$ kubectl apply --prune -l 'testlable=managed' --cascade=true -f all.yaml
pod/default-http-1 unchanged

Anything else we need to know:
This happens regardless if I delete one or multiple resources from the second namespace. Prune won't touch the second namespace if there are no manged resources left in there.

@TheKangaroo can you reopen this?
I seem to have run into the same issue


Default Notification support

I'm doing some testing and it seems that containers created by Autokuma are not given default 'on' notifications even if my provider (in this case ntfy) is set to 'default enabled' as in the below screenshot.

Hi, yes this is handled using special client side logic by the uptime kuma webui (the same is btw true for proxies) which I don't plan to replicate, however you can achieve basically the same using autokumas default_settings config:
    environment:
      AUTOKUMA__DEFAULT_SETTINGS: |-
         *.notification_id_list: { "1": true }

Thanks for that!
For anyone else who stumbles upon this, https://github.com/louislam/uptime-kuma/pull/4411 has been merged but Uptime Kuma 2.0 (as of this writing) isn't released yet, so you'll likely note errors and no notification fired for docker containers and groups as a URL is currently expected but not included when monitoring those types of monitors.
Example log:
2024-04-13T05:09:49.060409423Z 2024-04-12T22:09:49-07:00 [MONITOR] WARN: Monitor #39 'Dozzle Web': Failing: Request failed with status code 404 | Interval: 60 seconds | Type: http | Down Count: 0 | Resend Interval: 0
2024-04-13T05:09:49.773562660Z 2024-04-12T22:09:49-07:00 [MONITOR] ERROR: Cannot send notification to Ntfy
2024-04-13T05:09:49.775668020Z Error: Error: AxiosError: Request failed with status code 400 {"code":40018,"http":400,"error":"invalid request: actions invalid; parameter 'url' is required for action 'view'","link":"https://ntfy.sh/docs/publish/#action-buttons"}
2024-04-13T05:09:49.775687100Z     at Ntfy.throwGeneralAxiosError (/app/server/notification-providers/notification-provider.js:38:15)
2024-04-13T05:09:49.775689558Z     at Ntfy.send (/app/server/notification-providers/ntfy.js:72:18)
2024-04-13T05:09:49.775691305Z     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2024-04-13T05:09:49.775692975Z     at async Monitor.sendNotification (/app/server/model/monitor.js:1421:21)
2024-04-13T05:09:49.775694606Z     at async beat (/app/server/model/monitor.js:958:21)
2024-04-13T05:09:49.775696280Z     at async Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:1026:17)
2024-04-13T05:09:49.776387315Z 2024-04-12T22:09:49-07:00 [MONITOR] WARN: Monitor #41 'Dozzle': Failing: Child inaccessible | Interval: 60 seconds | Type: group | Down Count: 0 | Resend Interval: 0

@BigBoot  this doesnt seem to work for me, is there any logging that can give me some hints as to why my default notifier is not checked?

@johntdyer You will likely need to run a jq query to check that your default notifier is in fact ID of 1, similar to https://github.com/BigBoot/AutoKuma/issues/2#issuecomment-1934498704. My default notifier (before blowing away my old config folder) was actually 2.

@undaunt  mine is actually number 1
[
  {
    "id": 1,
    "name": "Uptime",
    "active": true,
    "userId": 1,
    "isDefault": true,
    "config": {
      "applyExisting": true,
      "discordWebhookUrl": "https://discord.com/api/webhooks/xxxxxx/xxxxxx",
      "isDefault": true,
      "name": "Uptime",
      "type": "discord"
    }
  },```

any thoughts ?

The AUTOKUMA__DEFAULT_SETTINGS works, I've looked at your logs in #11 and autokuma seems to have problems connecting to your uptime kuma instance, which is probably also the reason for the excessive connection count.

Actually I just noticed you seem to be using static monitors, these are completely static and don't go through any processing right now (i.e. no default settings), if that is what you're trying to do.

yes, I am using a mix of static and label derived ... trying to codify my entire config....


[Bug]: MelonLoader does not work with game "Dream Quest"

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Full Latest.log file included. If no file exists then leave this unchecked and state so.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
I tried to create a Chinese mod for the game dream quest, but an error in the log occurred when I added the melonloader and ran it in dnspy for the first time.
Dreamquest can be downloaded on steam, and the unity version used to make the game seems to be 4.6.5
I want to know whether it is because melonloader does not support unity of too low version or my operation error. Thank you for your help.
Latest.log
Did you attach your log file?

[X] Yes, I attached my log file to the text box above.
[ ] No, I could not find a log file at {Game_Directory}\MelonLoader\Latest.log

For who's gonna look into this... It's a very old game made in 2014, Unity version is 4.6.5f1, AssetsTools.NET is failing, it only has the "mainData" file to read the game information
Here's a copy of it if the AssetsTools.NET dev wants to take a look: https://anonfiles.com/e9j3nd70yd/mainData

This is now fixed in the latest alpha-development nightly build.


[ISSUE #7658] Fix bug of  timer message metrics with setDelayTimeMs

Which Issue(s) This PR Fixes

Fixes #7658
Brief Description

How Did You Test This Change?

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

Comparison is base (8a36471) 43.17% compared to head (7cf20eb) 43.26%.

Files
Patch %
Lines

.../rocketmq/broker/metrics/BrokerMetricsManager.java
0.00%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7743      +/-   ##
=============================================
+ Coverage      43.17%   43.26%   +0.08%
- Complexity      9823     9839      +16
=============================================
  Files           1166     1166
  Lines          84573    84574       +1
  Branches       10972    10973       +1
=============================================
+ Hits           36512    36588      +76
+ Misses         43519    43442      -77
- Partials        4542     4544       +2

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


3.x + YouTube + iOS (iPad) not working any longer

Just tried upgrading from 2.x to 3.x and encountered this error which is also reproducable on the YouTube demo page: http://mediaelementjs.com/examples/?name=youtube
Desktop (Chrome) works fine, but opening the page on iPad results in non working YouTube video, meaning black screen and when trying to play throwing JS errors. This is also reproducable in Chrome browser emulation set to iPad.
DevTools console shows this:

mediaelement-and-player.min.js:25 Uncaught TypeError: Cannot assign to read only property 'duration' of object '#<HTMLVideoElement>'

(2 times)

Uncaught (in promise) DOMException: Failed to load because no supported source was found.

Thanks for pointing this. Can you download the version from github and test it again please? I just did a fix that might have solved this

When using the current master both TypeError messages are gone :)

mediaelement-and-player.js:726 Uncaught (in promise) DOMException: The element has no supported sources.

Is still thrown though and no video showing up. There is also no YouTube iframe generated inside the <mediaelementwrapper>

Ok I'll take a look and keep you posted

@MorisatoK OK Seems to be fixed now. Download the latest package from GitHub and let me know

Thanks a lot @ron666, YouTube is now working in Mobile Safari and Chrome emulation :+1:

Perfect. Closing this issue. This change will be posted in the oncoming release (3.0.1)

I do have a similar problem. On Chrome (Win10) I still get this:
mediaelement-and-player.min.js:25 Uncaught TypeError: Cannot assign to read only property 'duration' of object '#<HTMLVideoElement>'

I'm using ME 3.0.1 (NPM)

@NemoStein The error only appears on Chrome and it shouldn't prevent you to use the plugin. I'm still figuring out the best way to get rid of that error. I'll keep you posted about this

@NemoStein can you download the latest version via GitHub and test this please? I just added a conditional to avoid these errors. Let me know if it worked for you. Thanks


Update Issue - Automate_undefined_4012

URL tested: https://accounts.google.com/v3/signin/identifier?continue=https%3A%2F%2Fmail.google.com%2Fmail%2Fu%2F0%2F&emr=1&followup=https%3A%2F%2Fmail.google.com%2Fmail%2Fu%2F0%2F&ifkv=AcMMx-eyEzPpidfNhvzRncy_XRvK-BtD3uwd-83PQ-kqEerVQc8-RAI26rxHjX5X-F7d3SLBkYs6&osid=1&passive=1209600&service=mail&flowName=GlifWebSignIn&flowEntry=ServiceLogin&dsh=S1593937898%3A1732681222188228&ddm=1
Open URL on Browserstack
default description

Property
Value

Browser
Chrome 131.0

Operating System
OS X Big Sur

Resolution
1688x784

Screenshot Attached
Screenshot URL

Updated Screenshot Attached


CraftyJS & NodeJS

I have made a node module, which runs a slim version of the current crafty master version (0.5.3). It's still in development: [npm_crafty] (https://npmjs.org/package/npm_crafty)
According to a Google Group Discussion about NodeJS & CraftyJS there is already some motivation to allow easy integration of crafty into a node server.
I was able to filter out the bulk of "unneeded" stuff for a server. As you can see [here] (https://github.com/mucaho/npm_crafty/tree/master/diff#manual-filtering), some manual fine-tuning is required. If you allow a bit of refactoring, the whole process could be more automatized and clean:

the bold outlined components and Crafty features could be moved into seperate files

Hey, are there any updates on this? What's the current state of crafty for use on nodeJS side?

Crafty development has progressed with logic separation of components in mind (see #578). In its current state it is tremendously easier to build a server version than it was before. Coincidentally I am currently working on a new version of npm_crafty, so I will test out how to build Crafty for node and will report my findings here.

Update: Majority of server-side stuff (except isometric) should run on nodejs now, you can also create multiple instances. (I just tested with an example game).
You can test it for yourself if you copy the develop branch to the root folder of your node project and add "craftyjs": "./Crafty" to your package.json dependencies. You can read more about local npm modules here. After that you do npm install and you should be able to create seperate instances by require('craftyjs')() (note the additional function call).
@starwed @kevinsimper
Would it make sense to transfer npm_crafty over to the craftyjs organization (and possibly rename it to Crafty Net or something like that)? It has low-level client-server networking and automatic matchmaking is almost done.

By the way, @dekdevy and @Sythe2o0:
I saw from your comments from another issues that you have used Crafty for networked games. Have you tried and/or had any issues require-ing and using Crafty on a node server?

@mucaho
While we used a node server, we didn't have any crafty code server-side, only client-side, sorry.

@mucaho Same here, the client is crafty based, the server is not really using any game libraries since its only logic.


Confusing documentation

Operating System: Ubuntu 16.04.3 LTS
Node Version: v8.9.4
NPM Version: 5.6.0
webpack Version: 2.2.1
webpack-serve Version: ^0.3.0

This issue is for a:

[ ] bug
[x] modification request
[ ] feature request

New Feature Use Case
First,
There is a mismatch in docs on what is returned by serve function.

Returns an {Object} containing... https://github.com/webpack-contrib/webpack-serve#serveoptions
But here you show that a promise is returned, not an object https://github.com/webpack-contrib/webpack-serve#serveoptions

serve({ config }).then((server) => {
  server.on('listening', () => {
    console.log('happy fun time');
  });
});

Second,
Mismatch on structure of options passed to serve function

serve({ config }) means serve({ config: {...wpconfig...} })
serve([options]) means serve({...})

So, above means that the stucture should be as follows. And this is wrong - all options in one object. It throws:
serve({
    config: { webpack config },
    ...all other webpack-serve options
})

And for some reason this doesn't throw. But it doesn't match plugin's docs
serve({
    config: { webpack config },
    options: { ...all other webpack-serve options }
})

(as an aside, please stop putting spaces in the checkboxes. the issue template does note not to. it won't render the checkboxes properly when you do)

Returns an {Object} containing...

Correct, that should read Returns a {Promise} which returns an {Object} containing

Mismatch on structure of options passed to serve function

That is not correct. See: https://github.com/webpack-contrib/webpack-serve/blob/master/lib/options.js#L9. It looks like you may have a misunderstanding of how restructuring works in newer Node versions.
Also of note: Only webpack v4+ is supported by this module.


decrypt9 cant be startet ?!

hey,
decrypt9 (latest version) via hbl (QR) does not start. only a black and a red square appears .
then automatically the hbl start again.
for example cant open emunand9 too..
any idea ?
fw 9.2

what is your entrypoins? can you load other homebrew apps? have you tried copying back the Decrypt9 files? Do you have the aeskeydb.bin file on the root of your SD card? (or in the files9 folder?)

If sysDowngrader crashes at any point during the downgrade (which can happen occasionally), you can power off the system by holding the power button. You will be left with what's called a "partial downgrade" or "partial", which is recoverable.

Although a partially downgraded system may show 9.2.0 as the system version, you are not done. You must restart from the beginning of this page and downgrade a second time to install any files that were not installed. Browserhax will work for partial downgrades as long as you are using the recommended version of sysDowngrader.


stack level too deep (SystemStackError)

When dumping a large redis database (1gb+ in size), redis-dump raises an exception after dumping about 262mb of data.
This is the last few lines of output (running with -D):
...
1320554699.0341: dump[-2199019514904509058, {"db"=>0, "key"=>"customer+43573945", "ttl"=>-1, "type"=>"set", "value"=>["1"], "size"=>1}]
1320554699.0341:  customer+43573945 (set): 1B
/Users/paul/.rvm/gems/ruby-1.9.2-p180/bin/redis-dump:19: stack level too deep (SystemStackError)

I wasn't able to reproduce the issue. However, I just pushed a new release (0.3) which includes some memory optimizations, speed improvements, and related changes. There's a new chunk_size option which specifies how many records to process at one time (default is 10,000).


Issue with Dialog Detection Prevents Falling Back to Curses

A user reported this error on Alpha.9, running Fedora:
./LogosLinuxInstaller
2024-07-08 21:40:57 ERROR: An error occurred in run_control_panel(): 'tuple' object has no attribute 'expandtabs'

This is most likely related to the package installer code and the code for displaying installed packages, as it switches between a dict and a list of tuples in tui_dialog

I thought I had fixed this but it appears not.


Soft-deletes onlyTrashed() with where() and orWhere() returns non-deleted results

I have a search form to find deleted users who has email or username that matches the search query:
$user->onlyTrashed()->where('username', 'like', "%$query%")
    ->orWhere('email', 'like', "%$query%")->get()

But the results includes non-deleted users. Is it a bug?

I think this ends up coming out as
select * from user where trashed == false and username like %query% or email like %query%.

You need to put the where in a closure.

$user->withTrashed()->where('username', 'like', "%$query%")
->orWhere('email', 'like', "%$query%")->get();
try this


Is this package is compatible with Laravel 5?

Didn't see anything in the documentation, so just checking if this package is compatible with Laravel 5?

Hi, I haven't tested it yet with Laravel 5 but it shouldn't be a major problem. I'll test it next week.

Hey, sorry for the delay. I can assure that the package is Laravel 5 compatible. The current master and v1.1.* will support Laravel 5.
For Laravel 4 you can use the 1.0.* versions.


Disable forced updated for device trackers

Forced updates are only needed if we are not polling. Since the coordinator
is effectively doing the polling instead of Home Assistant internals doing it
via should_poll set to True, we need to set the property manually to avoid
writing a state update every time the coordinator callback happens to avoid
a state changed event when nothing has really changed.


Feature/atlantins init

Ran Plan for 2 projects:

dir: . workspace: default
dir: module workspace: default

1. dir: . workspace: default
Plan Failed: Pull request must be approved according to the project's approval rules before running plan.

2. dir: module workspace: default
Plan Failed: This project is currently locked by an unapplied plan from pull #13. To continue, delete the lock from #13 or apply that plan and merge the pull request.
Once the lock is released, comment atlantis plan here to re-plan.

Plan Summary
2 projects, 0 with changes, 0 with no changes, 2 failed

:fast_forward: To apply all unapplied plans from this Pull Request, comment:atlantis apply

:put_litter_in_its_place: To delete all plans and locks from this Pull Request, comment:atlantis unlock

atlantis plan

Ran Plan for 2 projects:

dir: . workspace: default
dir: module workspace: default

1. dir: . workspace: default
Plan Failed: Pull request must be approved according to the project's approval rules before running plan.

2. dir: module workspace: default
Plan Failed: This project is currently locked by an unapplied plan from pull #13. To continue, delete the lock from #13 or apply that plan and merge the pull request.
Once the lock is released, comment atlantis plan here to re-plan.

Plan Summary
2 projects, 0 with changes, 0 with no changes, 2 failed

:fast_forward: To apply all unapplied plans from this Pull Request, comment:atlantis apply

:put_litter_in_its_place: To delete all plans and locks from this Pull Request, comment:atlantis unlock

Warning: The plan for dir: . workspace: default was discarded via the Atlantis UI.
To apply this plan you must run plan again.


Ardana qe tests nova

Please review the changes for nova and ceilometer

Add subunit protocal

Running those tests fails when trying to send the subunit results to OS-Health, see: http://<IP_ADDRESS>:8080/job/ci-ardana-qe-tests/83/console
Also, there are some tests failing, are they expected to fail? Is it possible to have a filter containing only tests that are expected to pass?

Add condition on the checkpoint in source code, once the change is merged into ardana-qe repo, the subunit test result should pass.

looks good to me for subunit and rocketchat output


Cloudwatch exporter - Support custom service labels and portName

What this PR does / why we need it:
Added support for add additional k8s labels on the service object and for overriding the portName.
This allows, for example, easier integration with prometheus-operator configurations, which scrape exporters based on custom labels and portNames.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/CC @gianrubio

/ok-to-test
/lgtm


Bump version for 0.22 release

@sholderbach
do you want this to be ready before Nushell 0.83?

As I landed https://github.com/nushell/nushell/pull/9787 we have to release a new reedline (also for the good fixes)

As I landed nushell/nushell#9787 we have to release a new reedline (also for the good fixes)

mm i do not see the change, see nushell/nushell#9787 :confused:


Replace deprecated actions/setup-ruby with ruby/setup-ruby

It seems ruby/setup-ruby has taken over from the deprecated actions/setup-ruby, see: https://github.com/actions/setup-ruby#setup-ruby

You're absolutely right, thanks for the PR 😄


Remove leading slash

From baseUrl leading slash needs to be removed in case user will time <IP_ADDRESS>/ isntead of <IP_ADDRESS>

wrong repo


Fixed the body response stream so it is reusable.

Desired Outcome
Fixed the body response stream so it is reusable.
Implemented Changes
Describe how the desired outcome above has been achieved with this PR. In
particular, consider:
I reset the steam so it is reusable.
Connected Issue/Story
Resolves #18
Definition of Done
Changelog

[ ] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[ ] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: insert issue ID
[ ] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[ ] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[ ] There are no security aspects to these changes

Great, thanks :)


Log mesh u vs

for #4
I don't think this covers every edge case regarding multiple UV sets and I'm making some assumptions on how mesh UVs are handled here, but I figured I'd clean up some low-hanging fruit if I could and put it somewhere it could be easily maintainable.

Hi @domportera ,
this was bad timing indeed! You code looks fine and is valid, but I was simultaneously working on [LoadObj] and ObjMesh to overcome this issue: Now when loading Obj-files without UV coordinates we generate a fallback UV-setup via box projection:
https://github.com/still-scene/t3/commit/446e8ab41b930c7b6af73f3c83666b168927a10e
https://github.com/still-scene/t3/commit/cb63b4ad5dace9ce34114d89f35209ab81fc1fe1
So I believe these warnings are no longer required.
Maybe we should align to avoid such conflicts. Depending on what you want to work on, it might be good to go through the RoadMap
I tagged some tasks here

no worries! that sounds like a much better solution. fortunately this was a super quick thing that just helped me learn the codebase a little more. I'll take a look!


Update readme for release 0.5

Related to #253
Update code for SciPy 1.11.0.

scipy.stats.mode for non numeric data is deprecated from SciPy 1.9.0 and is removed in SciPy 1.11.0.
Tests are broken, because transfer learning module uses non numeric extended labels.
@plcrodrigues , any idea?

I solve the problem, replacing scipy.stats.mode by np.unique, which supports non numeric data.

@plcrodrigues , any idea?

This is indeed annoying but doesn't sound too hard to bypass with your np.unique approach.
In any case, we should soon take some time to make use of the new sample-props feature from sklearn -- PR #24027 -- in the transfer learning module of pyRiemann. This would avoid recurring to string in the labels.

Last test for Windows Python 3.7 which is OK installed joblib 1.2.0
https://github.com/pyRiemann/pyRiemann/actions/runs/5321719371/jobs/9637189077#step:5:24
5:10
Now, it is KO installing joblib 1.3.1
https://github.com/pyRiemann/pyRiemann/actions/runs/5415226192/jobs/9846294273?pr=254#step:5:24
I freeze joblib to 1.2.0, but now, all others tests for Python 3.8 and 3.9 are KO.
Any idea?

Waiting scikit-learn bugfix, I decide to detect when _init_centroids must be used with a non default sample_weight.
I also clean code, removing support for version before scikit-learn 0.24.

FYI I had a look and the joblib 1.3.1 issue is actually due to loky (default joblib parallel backend). The loky issue is tracked in https://github.com/joblib/loky/issues/411.
Not entirely sure this is going to get fixed in loky. FYI scikit-learn requires Python 3.8+ since scikit-learn 1.1 (released May 2022), so a possible work-around for pyRiemann would be to drop Python 3.7 support in the 0.5 release.

Close-reopen for tests.

Documentation fails, it is probably related to #255. @agramfort, what do you think ?

For documentation build, I can't solve the dependency problem between sphinx and jinja2.

Doc is now OK!
I let you merge @agramfort , @plcrodrigues , @sylvchev


[3274] Remove trainees

Context
https://trello.com/c/pIPfv1SW/3274-remove-duplicate-trainee-records-on-register
These trainees were duplicated in CRM (subsequently remove). The Register versions should never have been created as they already have an older version on the DTTP CRM.
Changes proposed in this pull request

Data migration

 Remove duplicate Trainee records on Register


support ?raw imports like vite does

Describe the feature
context:  https://github.com/nuxt/nuxt/issues/21487
currently if a have universal-logic.ts that looks like this:
import file1 from './hello.txt?raw`

... other logic

Nitro (and thus Nuxt server) is not able to find the file. It would be great to have the ability to have truly universal imports across the server and the client and to allow even more code reuse
Additional information

[ ] Would you be willing to help implement this feature?

We already have a raw loader. For sake for Vite consistency, ?raw seems a good alternative (also there were legacy reasons we were not resolving relative paths, now we don't have those)


Add "bundle_errors" options to send multiple errors for one parse

Hi, I just started using webargs with the purpose to fully replace reqparse from Flask-RESTful in my application.
So far I am very happy with webargs nested arguments capabilities (which I could not properly work out with reqparse).
However, with reqparse you have a bundle_errors option, which when set to true, tries to validate all arguments and does not stop after the first error, but rather aggregates all errors and throws an exception which message field is a dictionary with all errors found per field.
This option is very useful to avoid many roundtrips between client and server when you have a complex form or web service.
I could not find an equivalent feature in webargs, does such a feature exist? If not, it would be very nice to have and that would put webargs to par (and even better) with reqparse.

"Error bundling" has not yet been implemented in webargs, though it's definitely a feature I would like to see.
One solution is to use marshmallow in tandem with webargs (I see Flask-RESTful is working towards replacing it's marshalling module in favor of marshmallow) because it does have the error-bundling you're looking for. You would use webargs solely to pull arguments off of the request, then leave validation up to marshmallow. There is even some work being done to provide tighter integration between the two libraries: https://github.com/hartror/webargs-marshmallow .
I would accept a PR implementing this feature in webargs, but for the time being, marshmallow's validation is the way to go.

Thanks for the comment.
Currently I don't use marshmallow in my project as the marshalling support of Flask-RestFul fits my needs for output; I am not sure yet I would want to add it as a new dependency just for this.
I will take a look at webargs source and see if I can easily add this feature and propose a PR for it. However, I think that won't be until a few weeks more.
I will put updates here if needed.

A related change that I had considered making was to catch ValidationError exceptions during parsing and ensure the arg name was set -- as validation functions probably won't know the correct arg name when raising the exception.

This will be solved by using marshmallow's Nested field.


Update to latest NIO with updated protocol negotiation SPI

Motivation
A recent PR to https://github.com/apple/swift-nio/pull/2497 adjusted the SPI for protocol negotiation.
Modifications

Bump minimum NIO version to 2.58.0
Update use of NIOTypedApplicationProtocolNegotiationHandler.

Result
Compiles again with latest NIO.

@simonjbeaumont There are a bunch of warnings now since we deprecated something and we have to migrate those API usages as well.

My daily Android CI, which builds trunk NIO HTTP/2 against trunk NIO, just broke because of that NIO pull. I can confirm that this pull fixes it on Android.

@simonjbeaumont any idea what's happening with the api breakage build?

@simonjbeaumont any idea what's happening with the api breakage build?

Yes, the API checker relies on being able to compile the previous commit, which is failing because of the SPI change, which is contained in this PR.
13:55:24 [305/311] Compiling NIOHTTP2 HTTP2Stream.swift
13:55:24 /tmp/.check-api_DOZ6tg/repo/.build/x86_64-unknown-linux-gnu/apidiff/493897c4744de24fe7c48940c3654e41fb6673fc-checkout/Sources/NIOHTTP2/HTTP2PipelineHelpers.swift:503:27: error: no exact matches in call to initializer
13:55:24         let alpnHandler = NIOTypedApplicationProtocolNegotiationHandler<NIONegotiatedHTTPVersion<HTTP1Output, HTTP2Output>>(eventLoop: self.eventLoop) { result in

Oh good catch. Thanks!


Uppercase day codes so that they are case-insensitive

Addresses #330. This bugged me the other day so I thought I'd fix it.

Model citizen :clap: @kochman

Thanks for the fix!


Glorfindel: Blacklist newlifeherbalclinic

Glorfindel requests the blacklist of the following keyword(s):

newlifeherbalclinic - MS search

newlifeherbalclinic has been seen in 3 true positives, 0 false positives, and 0 NAAs.


Add AsyncTaskCodeActivity type

Currently for those wishing to build a custom asynchronous activity, one must implement AsyncCodeActivity and work with the NetFx 3 async library, implementing BeginExecute and EndExecute. For many newer developers, this model is foreign and may be difficult to find adequate documentation of how to implement an activity using this model.
I would like to propose a new type, AsyncTaskCodeActivity, which instead exposes an abstract ExecuteAsync() method which returns a Task or Task<T> as appropriate. This can reuse much of the existing code available in the Windows Workflow Foundation library, such as AsyncCodeActivityContext.
Proposed API Changes
public abstract class AsyncTaskCodeActivity : AsyncCodeActivity
{
    protected abstract Task ExecuteAsync(AsyncCodeActivityContext context, CancellationToken cancellationToken);
}
public abstract class AsyncTaskCodeActivity<TValue> : AsyncCodeActivity<TValue>
{
    protected abstract Task<TValue> ExecuteAsync(AsyncCodeActivityContext context, CancellationToken cancellationToken);
}

Considerations/changes:

Should System.ValueTask be used instead of System.Task? If a user wraps a MemoryStream or some other existing ValueTask that can return synchronously, ValueTask allows us to wrap the return value and avoid allocating. There are however a couple of potential downsides:

ValueTask/ValueTask<T> is less usable than its Task/Task<T> counterpart. However, since ValueTask/ValueTask<T> is best used in environments where it is simply going to be awaited, this makes it a particularly good candidate for UiPath. If it is being called outside of the UiPath code runner, then it is likely being done in a unit test by the developer who is building the activity. It will be on them to handle it appropriately.
ValueTask/ValueTask<T> is larger than Task/Task<T> when wrapping Task/Task<T>. This is a measurable difference in microbenchmarks, however for utilization in an environment like UiPath, I doubt it will make a big difference.

Is AsyncCodeActivity the correct target for extension? Should it instead extend Activity like AsyncCodeActivity does? The way my current implementation of AsyncTaskCodeActivity works, uses BeginExecute to start the task and uses a TaskCompletionSource to track the lifetime of the task. Then it uses EndExecute to return the value of the task. In the case of AsyncCodeActivity (without a return type), it uses an empty VoidResult struct as its result (since I can't use void as a return type). This type works similarly to MediatR's Unit  type.

Should there be an interest in this change, I would be happy to submit a PR.

Sounds good! A PR is welcome.

We're not a high performance library, so Task seems best.
It seems there's useful stuff to reuse in AsyncCodeActivity. But make sure to make BeginExecute/EndExecute sealed .


An online judge

An online judge is needed by the community to practice their coding skills and also to host competitions during fests such as Advaita.
This issue description is a WIP and I will add more details after dinner.

Work has already been started on this here https://github.com/p-society/Online-Judge by @IamRaviTejaG and @knrt10


Update pihole_dns_search.xml

Input selector to choice from two different ways to set source

Thanks for Contributing!
I like the idea. I'll be testing throughout the day. I switched the base to the dev branch so I can track the changes for the next version release on Splunkbase.


RuntimeException: No supported encrypter found. The cipher and / or key length are invalid.

No matter what I do, I can't seem to get the guy to boot up with an APP_KEY that it'll take with consistency.
docker-compose.yaml reads this, under the environment:
- APP_KEY=odsoIqIptj9ADwVS9onivtmarnFKQhG3
This is what spits out in the run time logs:
Backup skipped!
Backup completed!
Application key [base64:vGSwhwLZjWiucsWF6jEyF6Mk/jgfNU0uAdK2Q1sTR4w=] set successfully.
Configuration cache cleared!
Configuration cached successfully!
Route cache cleared!

When I go into the deployment, and look at the .env file this is what I see set:
APP_KEY="base64:oRx0GVFwZRvofbeo1Et/EoSCh+Fel+GQF3yUyGGbGYg="
No wonder when the app tries to start up it gives me this error:

[2020-06-26 02:14:47] production.ERROR: RuntimeException: No supported encrypter found. The cipher and / or key length are invalid. in /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php:45

It's probably unable to find consistency in the values. cachet v2.3.18

Hi @DevMan01 I had the same problem for a bit.  The issue for me was that I used an incorrect APP_KEY, which looks like it may be your issue as well.  From your logs, it looks like you need to set your APP_KEY to the string base64:vGSwhwLZjWiucsWF6jEyF6Mk/jgfNU0uAdK2Q1sTR4w=

API_KEY needs to be 32 chars: https://github.com/CachetHQ/Docker/issues/330#issuecomment-490574998

Just noticed that the README of this repo tells you to use artisan to generate the key (which has the correct format and length), but the actual guide doesn't explain it yet:
docker exec -i ID_OF_THE_CONTAINER php artisan key:generate

For anyone finding this issue, just use that command to generate the key.


Set pydantic v2 as a requirement

If trying to use the Opper package with Pydantic v1, it fails:
ImportError: cannot import name 'computed_field' from 'pydantic'

here: https://github.com/opper-ai/opper-python/blob/main/src/opperai/types/init.py#L10
This is only available starting on v2

Hi! Thanks for the PR. Sounds good. 🙂


Fixes #7044 Handle databse filter in ingestion pipeline when connection specifies a database from UI side  #7044

Describe your changes:
#7044

Handle databse filter in ingestion pipeline when connection specifies a database

When data base name is not given

When data base name is given

Type of change:

[ ] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

[x] I have added a test that covers the exact scenario we are fixing. For complex issues, comment the issue number in the test for future reference.

@pmbrull @ShaileshParmar11 we should show the database is configured in the database filter place. Instead of completely not showing the filter itself. Its going to be confusing for the users why the database filter suddenly disappeared

in redshift we ask users to enter database name as mandatory right @ulixius9 . What will happen in those cases?

@ShaileshParmar11 @ulixius9

cc @pmbrull

@ShaileshParmar11 in case of redshift & postgres we will have to enable the database filter pattern when ingestAllDatabases flag is enabled


Include ppc64le hash

Signed-off-by: Yussuf Shaikh<EMAIL_ADDRESS>

/cc @Prajyot-Parab @cs-zhang @aishwaryabk


mcumgr-smp: Create AsyncSMPTransport trait and implement for serial

Implements the async route of #2

@SZenglein could you already briefly look at 3ef656b4ec9a0bf3ff5a0c61bec3818e00655389 ? had to do it like this for object safety

Without going into detail, I don't like that you removed the sync transports without replacement. Please at least move the implementations into the library.
On that note, I am also not convinced  smp-tool should be converted to tokio.  There is only ever one future (main) with no benefit at all.

All the transport are in the async in some way, as its always transporting to a different device. how would you feel about a single threaded tokio runtime? I feel like tokio makes sense here because its async. mutli threated rt not so much, thats true

Also the thing with converting to tokio is that I dont see a way to implement ble without async. Thats part of the reason why I would like to use tokio for the CLI as well

Okay, at a cursory glance you're right about async bluetooth.  Considering that, leaving the CLI on tokio is fine.

Build a newer transport, but thinking about removing mcumgr from my project, it feels like to much work for to less advantage for just a personal project. Feel free to take the new code from https://github.com/SysBadge/smp-rs/tree/kloenk/lib/transport-frame (ota fw update seems to not work. firmware informations did work)

Thanks for the update. Is it too much work in terms of changes I requested, or in terms of implementing the Bluetooth transport?

Implementing the bluetooth things, and having something to test. I don't have a full zephyr here, so I can only trivial build zephyr including ble transport, others fail to build. And overall for my project mcumgr seems to be much more work then I need. Currently trying without mcumgr at all, therefore not needing this debug tool

I understand. I have not looked at the SMP/Mcumgr BLE transport yet, so I cannot comment on the complexity, especially compared to rolling your own BLE.
We can leave this PR open. If you decide to come back to it and give mcumgr another try, just comment.
If you don't maybe someone else wants to pick it up, or we might reuse your code.
Good luck with your project.

Hi @SZenglein. I'm already implemented BLE. My way looks a bit simpler than @kloenk, with all respect.  I was removed all the sync transports at first, but after that found this PR. So will try to consider all the recommendations. Think that it can be done in a week-two.

Hey @aectaan, I'm on vacation right now, but looking forward to your contribution! 🎊

Thanks to both of you. There is now an implementation for async transports (maybe simpler due to async_trait usage) and a BLE transport, so I'm closing this PR.


archivebox: use hash instead of sha256

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

error: builder for '/nix/store/jrp96x8mpjphn4wrhidv2d98zy2ln4jq-python3.10-django-extensions-3.2.0.drv' failed with exit code 1;
       last 10 log lines:
       > removing build/bdist.linux-x86_64/wheel
       > Finished executing setuptoolsBuildPhase
       > installing
       > Executing pipInstallPhase
       > /build/source/dist /build/source
       > Processing ./django_extensions-3.2.0-py3-none-any.whl
       > ERROR: Could not find a version that satisfies the requirement Django>=3.2 (from django-extensions) (from versions: none)
       > ERROR: No matching distribution found for Django>=3.2
       >
       > builder for '/nix/store/jrp96x8mpjphn4wrhidv2d98zy2ln4jq-python3.10-django-extensions-3.2.0.drv' failed with exit code 1
       For full logs, run 'nix log /nix/store/jrp96x8mpjphn4wrhidv2d98zy2ln4jq-python3.10-django-extensions-3.2.0.drv'.
error: 1 dependencies of derivation '/nix/store/k5f62hs8fgy2c4sb44a60pxa43d285ar-archivebox-0.6.2.drv' failed to build

Heh, yeah that only works if the original builder also uses hash

Django uses hash since the 3.2.15 upgrade, so this change is warranted.

error: builder for '/nix/store/jrp96x8mpjphn4wrhidv2d98zy2ln4jq-python3.10-django-extensions-3.2.0.drv' failed with exit code 1;
       last 10 log lines:
       > removing build/bdist.linux-x86_64/wheel
       > Finished executing setuptoolsBuildPhase
       > installing
       > Executing pipInstallPhase
       > /build/source/dist /build/source
       > Processing ./django_extensions-3.2.0-py3-none-any.whl
       > ERROR: Could not find a version that satisfies the requirement Django>=3.2 (from django-extensions) (from versions: none)
       > ERROR: No matching distribution found for Django>=3.2
       >
       > builder for '/nix/store/jrp96x8mpjphn4wrhidv2d98zy2ln4jq-python3.10-django-extensions-3.2.0.drv' failed with exit code 1
       For full logs, run 'nix log /nix/store/jrp96x8mpjphn4wrhidv2d98zy2ln4jq-python3.10-django-extensions-3.2.0.drv'.
error: 1 dependencies of derivation '/nix/store/k5f62hs8fgy2c4sb44a60pxa43d285ar-archivebox-0.6.2.drv' failed to build

This seems like another issue, but we don't have it on 22.05. So I would merge this first and  backport and maybe investigate later into this new issue


Fix OS inventory JSON generation

The code is not generating in correct format. Cleanup code for cleaner json format file.

[x]  Execute command to avoid sudo issue
[x]  Format to JSON pretty print to a file

https://gerrit.openbmc-project.xyz/#/c/4599/


Custom JS and css

Could you document how to add a few custom javascript and css snippets to an existing theme (like you did with reveal-ck.css?
Do I need to create my own template?
Assuming I can find night in the reveal-ck/files/reveal.js directory, where would I put this?
I can the instructions into a doc if you could start me in the right direction

If you're talking about CSS tweaks, you can create a css/ directory and create a css file in it. That CSS file will be automatically included and linked up in the presentation.
It's described here:
https://relishapp.com/jedcn/reveal-ck/docs/how-to-store-and-reference-css
The docs at https://relishapp.com/jedcn/reveal-ck are generated from the feature/*.feature files in this repository.
I try to write up .feature files for every thing-- so-- if it's possible it'll be on https://relishapp.com.

I've never used a reveal.js plugin or used custom JavaScript-- so-- I think I put something minimal in-- described here:
https://relishapp.com/jedcn/reveal-ck/docs/how-to-store-and-reference-js
But I've never used it.
If you can give me a use case (for something you're doing), we can work backwards to figure out how to make it happen.

Worked like a charm
.reveal cite {
  font-style: italic;
}
@media print {
  /* overriding .reveal section img */
  .reveal section h1 img, .reveal section h3 img {
    display: inline;
  }
}


rife not working

Is there an extra command line option or something to kick rife off? I've installed as per the instructions but it doesn't seem to be doing anything. Also how to you specify movie output instead of gif?

One more question - what denoising strength is it using to generate the images? I'm trying to reproduce the initial image using A1111 with the same seed/sampler/steps but no luck so far. Is this even a thing that can be done or does the diff process not work like that?

Is there an extra command line option or something to kick rife off? I've installed as per the instructions but it doesn't seem to be doing anything. Also how to you specify movie output instead of gif?

Apologies for the slow reply 😅 RIFE is a separate subcommand - animatediff rife interpolate --help will print details. Generally it's something like;
animatediff rife interpolate -z -x -F 50 -M 3 ./00-13163998037304110560

One more question - what denoising strength is it using to generate the images? I'm trying to reproduce the initial image using A1111 with the same seed/sampler/steps but no luck so far. Is this even a thing that can be done or does the diff process not work like that?

It's text-to-image, so technically a denoising strength of 1.0, but it's not really a meaningful/configurable thing. As for duplicating outputs, this uses the HuggingFace implementation of the SD pipelines rather than the AUTOMATIC1111 webui implementation and (as far as I know) there's no way to make HF pipelines output exactly the same images as the auto webui 😢
On top of that, the temporal unet/motion module alters the generation outputs (since that's how it works), so even if you did hack up the Unet/sampler implementations to make them identical, the outputs would never be exactly the same. Just the nature of the thing, really.


How to generateOpenApiDocument?

This is a noob question but how do I use this particular code snippet for creating an openapi document? Or what route do I need to access to access documentation? I'm using this trpc-openapi with Next by the way.
import { generateOpenApiDocument } from 'trpc-openapi';

import { appRouter } from '../appRouter';

/* 👇 */
export const openApiDocument = generateOpenApiDocument(appRouter, {
  title: 'tRPC OpenAPI',
  version: '1.0.0',
  baseUrl: 'http://localhost:3000',
});

That code should be placed in an openapi.json.ts file in your api directory. Once there and once your endpoints are set up you can navigate to that page same as any other endpoint to see the json.
example: http:\\localhost;3000\api\openapi.json

Thanks @toddjudd!
I also noticed that you can paste the the contents of openapi.json.ts into editor.swagger.io and see well-organized UI for the documentation. I'm thinking of display that same Swagger UI within my app via a NextJS Route. Is this possible? Or is this not recommended to do?

I've never done it myself, but it sounds like a solid choice. Might be some helpful information here. https://next-swagger-doc.productsway.com/

Thanks for this, I was able to successfully do it.
I opted for a local dev environment approach for viewing the api documentation, so that I won't need to create an API specifically for viewing documentation.
I got the idea from next-swagger-doc, but I used swagger-ui-cli instead.
Basically, I write the openApiDocument as a JSON and view it with the swagger-ui-cli.


bundleignore doesn't ignore while copying files to temp dir

Describe the bug
When there are files that have permissions incompatible with whatever yarn bundle does with them (or yarn/fslib does with them), adding such files to .bundleignore does not help because apparently ignoring happens later on in the pipeline.
In my case, I'm using nix + direnv, which creates a .direnv directory that contains symlinks to readonly files somewhere else on the system. Even if I add .direnv to .bundleignore, yarn bundle copies them to a temp dir and then runs chmod on them, which fails.
To Reproduce
Steps to reproduce the behavior:

cd into a project that uses yarn v2/v3 workspaces
touch ~/bar need a file somewhere else for this reproduction
sudo chown root ~/bar root or any other user, any user other than yours.
sudo chmod 0444 ~/bar make it readonly for everybody.
ln -s ~/bar foo gotta create a symlink for reproduction
Add foo to .bundleignore
cd into a workspace and run yarn bundle.

I think this can also be reproduced by creating a symlink to a non-existing path.
Expected behavior
Everything works fine.
Actual behavior
EPERM: operation not permitted, chmod '<omitted tmp dir path>/foo'
Desktop (please complete the following information):

MacOS

Additional info
This could be solved by not even copying ignored dirs/files to a tmp dir for further processing.

Yep I think you're correct it's happening too late in the process.
The current process they're deleted after being copied to a tmp directory.
https://github.com/ojkelly/yarn.build/blob/2f000664c0255a0bd63e1180e679c5ede2e63fac/packages/plugins/plugin-bundle/src/commands/bundle/index.ts#L330
This can probably be solved by using something like fs-extra which has the ability to take in a function to filter files to copy
https://github.com/jprichardson/node-fs-extra/blob/master/docs/copy.md#copysrc-dest-options-callback
it would be replacing this line https://github.com/ojkelly/yarn.build/blob/2f000664c0255a0bd63e1180e679c5ede2e63fac/packages/plugins/plugin-bundle/src/commands/bundle/index.ts#L286
If this feels like something you might be able to work on let me know.


flowforge-device-agent doesn't start without internet connection

Current Behavior
When I'm connected to the internet, the device agent work has expected when the device boot with the flowforge-device.service is provided in that git. The device agent connect to the internet check if it needed update and install his module and start node-red when all these step are finish.
But when I disconnect from the internet, and reboot the device agent, nothing works anymore. the device agent start on boot but its unable to start node-red, its send me that message in the systemctl :
[AGENT] 28 Mar 2023, 19:34:46 warn: Error whilst starting Node-RED: RequestError: connect ENETUNREACH <IP_ADDRESS>:3009 - Local (0.0.0.>

Expected Behavior
I'm expected the device agent to start node-red instead of waiting for an internet connexion.
Steps To Reproduce
Disconnect the device agent from the internet
Environment

FlowForge version:1.3.0
Node.js version:16.16.0
npm version: 8.19.2
Platform/OS: Raspberry Pi OS
Browser: chrome

This was an intentional choice to prevent devices continuing to run deployed snapshots even after they had been either updated or removed in the FlowForge applications. Having the device check in before starting allows that check to be made.
How ever we can see that this may be something that a user may wish to control and allow existing snapshots to start and run until the device agent can check in with the FlowForge application.
We will look at making this configurable.

We will look at making this configurable.

Sorry, on reflection it probably won't be configurable, but we can look at flipping the behaviour to starting with what is currently deployed until it can check in

In our case this is critical, I base a 1 year project on your platform, and before choosing your platform I've did the tests in early 2022 and that was not an issue. Your tech support and your CEO have tell me that is suppose to work like I ask.
The only way I found to bypass that issue is to change the service file to ping google.com and if its not reachable its starting node-red instead with the flowforge directory, But thats not keeping the env variable of the flowforge-device-agent, so its not viable for us.
flowforge-device.service file:
[Unit]
Description=FlowForge Device Agent

[Service]
Type=simple
# Run as normal pi user - change to the user name you wish to run FlowForge Device as
User=pi
Group=pi
WorkingDirectory=/opt/flowforge-device

Environment="NODE_OPTIONS=--max_old_space_size=512"

ExecStart=/bin/bash -c "if ping -q -c 1 -W 1 google.com >/dev/null; then /usr/bin/env flowforge-device-agent -p 1881; else /usr/bin/env node-red -p 1880 -s /opt/flowforge-device/project/settings.js -u /opt/flowforge-device/project; fi"

# Use SIGINT to stop
KillSignal=SIGINT
# Auto restart on crash
Restart=on-failure
RestartSec=20
# Tag things in the log
SyslogIdentifier=FlowForgeDevice

[Install]
WantedBy=multi-user.target


call_phase argument not documented

From the documentation it is unclear how to get phased genotyped from the input.

The call_phase argument is not documented - I see it in the example code, but not in the manual. It is also not completely clear to me whether I need both, -calling_threshold and -call_phase

It would be good to mention here in the outputs that the phase file is only produced, if the user specifies the -calling_threshold and -call_phase (I think it would help with the understanding)

@janaobsteter we (@XingerTang and @AprilYUZhang) are well aware of how these docs are unclear and we are working on a rewamp of the docs as well as more functionality in this space. See all the options opened and also work going on at https://github.com/AlphaGenes/AlphaPeel/tree/devel/docs

@janaobsteter could you work with the devel version and report what is not clear in that version?


Joined inline & non-inline operations pt. 1.

@diegoalejogm can you address the merge conflicts? looks good to me otherwise
unfortunately a bunch of conflicting changes were merged before this one. going forward we should try to merge any refactoring as quickly as possible. maybe we need a refactor label in github ?

Working great!


Replace deprecated calls to Atom API

Usage of WorkspaceView is deprecated in most cases (in fact atom throws warnings in as of version 0.187.0 when using this plugin). Therefore we're now using the new simplified API to gain access to the editor content and the list of commands.

Sorry for the slow reply... good work! :ship:'ing


Search Applicant Profile

As an IAO analyst
I want to search for an applicant profile
so that I can match an applicant to an existing applicant profile

Assumptions & Scope
What are the assumptions for this story?
Applicant profiles from AXIS will need to be migrated into FOI Mod
Every applicant will require a profile.
Updating an applicant profile will not update the applicant contract details for Closed requests.
When an analyst updates an applicant profile, they will have the option of updating the contact details for other Open requests.
Only IAO analysts can view, search, match, create and update Applicant Profiles.
When an applicant makes a request they will provide their name and contact information. Upon review of the request, the IAO Intake analyst will need to search and check if the applicant previously has made FOI request(s) and if they already have an applicant profile in the system. The IAO Intake analyst will attempt to match the applicant name and contact information manually with existing applicant profiles and contact information already in the system. If there is no match, the Intake analyst will create a new applicant profile for the applicant.
Searches will match the information provided by the applicant when making the request ( First Name; Middle Name; Last Name; Date of Birth; Email; Primary Phone) with the same data fields for applicant profiles already on file.
Closest matches will be displayed at the top of the search results.
Assumption - searches in the Search Applicants profile will be client side and only search existing results to narrow the existing search, rather than start a new search. Confirm with team that is correct?
What is IN scope?
Searching for an applicant profile
What is NOT in scope?
View and select existing Applicant Profile #4761
Creating a new applicant profile
Searching and selecting profiles for On Behalf of requests
Adding an additional applicant category profile (i.e. different applicant type) #4763
Updating an existing applicant profile
Request History for an applicant
Applicant Contact History
Acceptance Criteria
Scenario 1: Search Applicant Profile from Request View

GIVEN an IAO analyst is on a Request View
AND an Applicant Profile has not been selected for the request
WHEN the IAO analyst opens Address and Contact Details
THEN the 'Search Applicant Profile' link will be there
https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/663873000

Scenario 2: Click on 'Search Applicant Profile - No match

GIVEN no Applicant Profile has not been selected for the request
AND there are no matches
WHEN the IAO analyst clicks on the 'Search Applicant Profile' link will be there
THEN the Search Applicants modal will open
AND there will be no results returned in the search
AND there will be a search bar and buttons to 'Create New Profile' and 'Cancel'
And columns for: First Name; Middle Name; Last Name; Date of Birth; Email; Primary Phone
https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/987389728

Scenario 3: Click on 'Search Applicant Profile - 1 or more matches

GIVEN no Applicant Profile has not been selected for the request
AND there are matches
WHEN the IAO analyst clicks on the 'Search Applicant Profile' link will be there
THEN the Search Applicants modal will open
AND the results will be returned in the search
AND there will be a search bar and buttons to 'Create New Profile' and 'Cancel'
And columns for: First Name; Middle Name; Last Name; Date of Birth; Email; Primary Phone
AND these columns will be filled if that data has been captured in the profile
AND the existing Applicant Profile with the most matches (First Name; Middle Name; Last Name; Date of Birth; Email; Primary Phone) will be displayed at the top of the results
AND the existing Applicant Profile with the least matches (First Name; Middle Name; Last Name; Date of Birth; Email; Primary Phone) will be displayed at the bottom of the results
https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/987389727

**Scenario 4: Cancel 'Search Applicant Profile' modal **

GIVEN the IAO analyst is on the the Search Applicants modal will open
WHEN the IAO analyst clicks on the 'Cancel' button
THEN the modal will close
AND the IAO analyst will remain on the Request View
AND no Applicant Profile will be matched with the Request

**Scenario 5: Search from 'Search Applicant Profile' modal **

GIVEN the IAO analyst is on the the Search Applicants modal
https://jacklyn808742.invisionapp.com/console/FOI---Intake-ckpq19frj032r013q6gpm8t1m/clp8r0umw0g3m016n9wh9gobi/play
WHEN the IAO analyst enters information in the Search Bar (e.g. DOB, Primary Phone, Email, etc.)
THEN search will update with the refined results

...
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

As discussed, On Behalf of will be a different story that will include this AC:
Scenario 2: Search Applicant Profile from Request View

GIVEN an IAO analyst is on a Request View for a personal request
AND the request is On Behalf Of
AND an Applicant Profile has not been selected for On Behalf of person
WHEN the IAO analyst opens on Behalf Of
THEN the 'Search Applicant Profile' link will be there
https://jacklyn808742.invisionapp.com/console/FOI---Intake-ckpq19frj032r013q6gpm8t1m/ckpx6mtxp0128013ecxv9f3nw/play

Search Applicant Profile#4758 - 13/21
#############################################
front end

develop react components for "Search Applicants" - default to max 100(config) - same listing of Queue and client side search - 8

back end

Analyze unique of applicant on Back end - FOIRequestApplicants,FOIRequestApplicantMappings, RequestorTypes - from General, Personal Request - if there is problem, need to fix - 3

Develop Search API - takes the input from request details!- FirstName, LastName, email etc? - 5

Based on the analysis on implementing Applicant profiles as per the current design, information. Below are issues development team is trying to highlight to business, @JHarrietha-AOT , and this section is followed by the suggested business rules, which development team is waiting a approval and/or other flows in UX as per @JHarrietha-AOT 's opinion.
Main points around Applicant Profiles, how it is handled in AXIS vs FOI MOD application.

In AXIS, Applicant Profiles(Person Name, Contact Info etc.) are Mandatory, and to be created/selected to Open/ run a life cycle of a FOI request. Also, in AXIS Contact Information is tied up with Person  On the other hand , on FOI MOD Contact Information is tied with the FOI Request and a Person linked to his/her contact info via a FOI request. This is based on the fact that, a person can create FOI request for his personal and /or official needs and can be fully anonymous while he/she requests a FOI records
Because of the above reason, AXIS have a process which involves communication with the applicant that can fair, good amount of information which can almost have realistic data. So till this date, this is not an issue even with AXIS Sync to FOI MOD.
Now to coming to the main point,  when we start implementing this/current UX design based on our current back end implementation, issue is like we will be listing will have many entries on search results of applicant profiles if the search is NOT following a near to unique identifying business rules. - Find our business rules on the section below.
Also, as per the current UX design based on our current back end implementation, Updates to a contact information can mistakenly applied to another request, without FOI analyst knowing where(or what all requests!) this contact information will get updated. This is because on FOI MOD, contact information is tied up with FOI request , not to person whose identity cant be  validated - known to all business team, devs etc.

Suggested business Rules, User flow.

Search applicant profiles will only list applicant details as per
[FirstName and/or LastName] **AND** (Email or Phone) provided EMAIL and/or Phone is NOT null on search filter coming from the request or criteria deciding . Address we can defer when we have confidence on Address data
This will reduce the risk of listing users with similar names and mistakenly update contact info.
We actually need a Link Contact Information between FOI requests so that the original business problem of updating contact information is for single user to multiple requests will get resolved.
Also, this way business know how many requests has a particular user has created.

During the meeting on 04/Dec, these are points we discussed and decided - as per my current understanding

The business rule to show up applicant search results will be based on Email by default. Means the requests from applicants who has not provided Email will NOT show up in search even though their FirstName and LastName matches in the system, this is the default behavior.
There will be an alternative option on the system, @JHarrietha-AOT brings up to the current wireframes, which will allow FOI Analyst to override the default action of searching profiles/requests by email ,like toggle or something, to allow analyst to search applicant profile by FirstName, LastName of her/his choice which has no emails, phone provided.
Dev Team indicated that, this alternative option will bring unrelated applicant profiles on the search results , since there is no unique identification in the system, cant rely just on names. Also, going with the alternative option results and updating it can bring issues, but business is ready to take risk and those results will not be much, as per stats @m-prodan shared(<1%). This situation will be more specific after decommissioning of AXIS though.
@nkan-aot has brought up the concern POST linking situation of profiles and indicated that Names will sync'd without any audits left back and inquired what should we do for closed requests, Yes that was a valid concern. @richard-aot , other members suggested a versioning logic for applicant profiles and sys, generated comments for open requests. and closed requests should be left as is. Business indicated this will be a new user story as per priority. @abin-aot , suggested its better to do along with this, but @m-prodan , @lmullane , and other business will decide the priority soon
The auto-sync of contact information, based on the email on FOI MOD can be done as an exercise on test-marshal, and demo to the business once we reach a logical end on these tasks  if they are OK, then we will   make it PROD release step. cc: @abin-aot
So from UX/UX design perspective, Team will require UX for toggling default search by Email based business rule and by open ended results of user specific search criteria. Also, additional UX updates to make analyst know while updating Address and/or any contact information on FOI MOD on applicant profile that will be impacting other requests, or any wording of that lines

cc: @JHarrietha-AOT @richard-aot @nkan-aot @lmullane @m-prodan

12/5 standup

search applicant process should be mandatory before moving request form intake to open
remove email or address validation from unopened to intake state change
intake still needs to be able to manually search even if no existing results return
if an applicant profile has different contact info (email/phone/address) from the request form, highlight the fields in the modal (Jackie to provide color)

@JHarrietha-AOT @abin-aot

Applicant Profile Sample.xlsx
This is the sample data file team used to explain various Person to multiple contact info scenarios(Work vs Personal), then multiple Personal addresses based on life situation, but same email and/or phone - which derived the above mentioned business rule

12/12/2023 standup

clarified, update all button on applicant profile must be clicked before saving if there are differences between request and profile
request form fields related to applicant will not be editable after opening. any changes needed to contact info will be made in the applicant profile modal

From stand-up today - One thing to be mindful of,  with our fees process, sometimes users need to place a request 'On Hold', or beyond, without sending the automated fee email. Right now this is done by removing their email, saving, placing request 'On Hold', then adding the email back. This will bypass the automated email being sent. If we remove their ability to edit this field after opening, there may be side effects / issues (they would need to do this in the applicant profile then).

Latest code committed to test-marshal-RQ-4758
Endpoints:

[ ] Search (auto/manual)
[ ] Create or save changes to applicant (with applicant version)
[ ] Save changes to all open requests (with applicant version)
[ ] Fetch request history by applicant ID
[ ] Fetch applicant history
[ ] Create profile hook for intake to open (if create new profile, save info to db
if existing profile, pass applicant id along with save request object
after open state, disable any edits to applicant information)

my branch for front end is test-marshal-NK-4761

https://miro.com/app/board/uXjVMtjQyiU=/

Made a few changes on backend:

Added "applicantprofileid" to "FOIRequestApplicants" as the unique identifier for applicant profile
Removed version from "FOIRequestApplicants" and "FoiRequestApplicantMappings"
New process for updating applicant profile:
3.1 Create a new row in "FOIRequestApplicants" if any of the "FOIRequestApplicants" column values got updated, and link the new row to old ones with "applicantprofileid"; then add new mapping in "FoiRequestApplicantMappings"
3.2 Create a new version in "FOIRequests", "FOIMinistryRequests" if any of the column values in those two tables are updated; then add new mapping in "FoiRequestApplicantMappings"


support uploading to s3 from lambda

Turns out the region change wasn't needed. I was explicitly adding it to the s3 bucket name but it is included when the region --all flag is added to the execution options so we're good to go as-is.


F6 should move focus to another UI element

Similar to how users can press Tab to navigate the UI tree, F6 is expected to work the same way.
We should use this as a way to explicitly move focus into the tab bar. I think generally F6 is used to move "up" the UI tree so that would make sense.

Similar to Tab, using the Shift modifier reverses the direction of navigation. So Shift+F6 goes "down"

What about the fact that commandline applications might want to use F6 themselves?
Even COOKED_READ uses it:
    case VK_F6:
    {
        _insertCtrlZ(cookedReadData);
        break;
    }

Closing. It's really important that we don't interfere with the underlying client app, and this behavior seems to be inconsistent across other tabbed applications.


Complete the serial workflow

Make the changes needed to run a simple workflow on a single worker

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 40.242%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
3856

Relevant Lines:
9582

💛  - Coveralls


GUI needs to be defined

It must be consistent with the mega_analysis package API.

Please explain in more detail ?

The Slicer graphical user interface (GUI) depends on what we want the user to be able to input into the system. Whatever can be specified in the GUI must be compatible with the mega_analysis API. The output of mega_analysis should be a data frame with shape (num_semiology_terms, num_structures), and the way they are combined will be handled by Slicer. Is that correct?
Once there's a single branch and there are no conflicting issues, we can maybe have some docs to know what every single function does, and how they need to be called. We should also have some unit tests.

yes unit tests using the pickled dataframes on my to do list after I can open slicer and get it to work.

Closing as there are issues about this already.


scale_event is never released,  ComputeLibrary\examples\cl_events.cpp

Output of 'strings libarm_compute.so | grep arm_compute_version':
Platform:
Operating System:

Problem description:
scale_event in the code below is never released
void do_run() override
{
    // Enqueue and flush the scale OpenCL kernel:
    scale.run();
    // Create a synchronisation event between scale and median:
    cl::Event scale_event = CLScheduler::get().enqueue_sync_event();
    // Enqueue and flush the median OpenCL kernel:
    median.run();
    // Enqueue and flush the Gaussian OpenCL kernel:
    gauss.run();

    //Make sure all the OpenCL jobs are done executing:
    scale_event.wait();        // Block until Scale is done executing (Median3x3 and Gaussian5x5 might still be running)
    CLScheduler::get().sync(); // Block until Gaussian5x5 is done executing
}

Hi @hxxfff
Have you made any changes to the test https://github.com/ARM-software/ComputeLibrary/blob/master/examples/cl_events.cpp ?
I could not reproduce the problem on my device:
root@acl_hikey_dev:~/tmp/pabtel01/graph# LD_LIBRARY_PATH=$LD_LIBRARY_PATH:. ./cl_events

./cl_events

Usage: ./build/cl_events [input_image.ppm]

No input_image provided, creating a dummy 640x480 image

Test passed

What device do you use to run the test?

No, I have not test it. scale_event should be released, in my opinion.
On 10/24/2019 20:42, Pablo Marquez wrote: Hi @hxxfff
Have you made any changes to the test https://github.com/ARM-software/ComputeLibrary/blob/master/examples/cl_events.cpp ?
I could not reproduce the problem on my device:
root@acl_hikey_dev:~/tmp/pabtel01/graph# LD_LIBRARY_PATH=$LD_LIBRARY_PATH:. ./cl_events
./cl_events
Usage: ./build/cl_events [input_image.ppm]
No input_image provided, creating a dummy 640x480 image
Test passed
What device do you use to run the test?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/ARM-software/ComputeLibrary/issues/760?email_source=notifications\u0026email_token=ABYU4QENRWST2ZIEIVL6NJDQQGJZNA5CNFSM4I7YHBO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECE4FWQ#issuecomment-545899226",
"url": "https://github.com/ARM-software/ComputeLibrary/issues/760?email_source=notifications\u0026email_token=ABYU4QENRWST2ZIEIVL6NJDQQGJZNA5CNFSM4I7YHBO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECE4FWQ#issuecomment-545899226",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Hi @hxxfff
What device are you running the test? Have you tried running other tests?

Can you find the call of clReleaseEvent(scale_event）in the code?
On 11/06/2019 23:29, Pablo Marquez wrote: Hi @hxxfff
What device are you running the test? Have you tried running other tests?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/ARM-software/ComputeLibrary/issues/760?email_source=notifications\u0026email_token=ABYU4QAIKQNACILYHQR5E3DQSLPGXA5CNFSM4I7YHBO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDG5XAQ#issuecomment-550361986",
"url": "https://github.com/ARM-software/ComputeLibrary/issues/760?email_source=notifications\u0026email_token=ABYU4QAIKQNACILYHQR5E3DQSLPGXA5CNFSM4I7YHBO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDG5XAQ#issuecomment-550361986",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Hi @hxxfff
Where are you running the example? On a phone? what device? What OS?

I haven’t run it.
On 11/07/2019 18:23, Pablo Marquez wrote: Hi @hxxfff
Where are you running the example? On a phone? what device? What OS?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/ARM-software/ComputeLibrary/issues/760?email_source=notifications\u0026email_token=ABYU4QEJTM2YHGPCWUDTHW3QSPUB5A5CNFSM4I7YHBO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDL5WLI#issuecomment-551017261",
"url": "https://github.com/ARM-software/ComputeLibrary/issues/760?email_source=notifications\u0026email_token=ABYU4QEJTM2YHGPCWUDTHW3QSPUB5A5CNFSM4I7YHBO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDL5WLI#issuecomment-551017261",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Hi @hxxfff
No, there is no need to call clReleaseEvent()
The CLScheduler is just adding a marker in the command queue: https://github.com/ARM-software/ComputeLibrary/blob/master/arm_compute/runtime/CL/CLScheduler.h#L163
See https://www.khronos.org/registry/OpenCL/sdk/1.0/docs/man/xhtml/clEnqueueMarker.html


Allow ECS metadata service to use hostnames

Describe the feature
The ECS credentials provider added in #475 only allows <IP_ADDRESS>/[::1] IP addresses when AWS_CONTAINER_CREDENTIALS_FULL_URI is used with http. Metadata services should be allowed to provide any hostname they want as long as all of the IP addresses for said host resolve to the loopback device.
Is your Feature Request related to a problem?
N/A
Proposed Solution
We need a KMP compatible DNS host resolver, something like:
fun interface HostResolver {
    suspend fun resolve(hostname: String): List<IpAddr>
}

class IpAddr(...) {
    fun isLoopbackDevice(): Boolean
    ...
}

This needs to be added to the validate full URI check (or at construction time) and verify that all IP addresses are the loopback device:
class EcsCredentialsProvider(...) {
    suspend fun validateFullUri(uri: String) {
        ...

        val ips = hostResolver.resolve(hostname)
        return ips.all { it.isLoopbackDevice() }
    }
}

Describe alternative solutions or features you've considered
N/A
Acknowledge

[ ] I may be able to implement this feature request

AWS Kotlin SDK version used
N/A
Platform (JVM/JS/Native)
N/A
Operating System and version
N/A

Depends on implementation of smithy-kotlin#745.


Incorrect Output in a specific scenario

Scenario -
Use Dictionary for multiple keywords and one keyword is present in more than one class/category, then, output is incorrect and shows only one class/category.
Below mentioned is example code for detailed explanation.
#Import library
from flashtext import KeywordProcessor

#Define Dictionary
dict_keywords={}

#'Russia' is present in both classes/categories - Europe and Asia
dict_keywords["Europe"]=['France','Germany','Russia']
dict_keywords["Asia"]=['India','Russia','China']

keyword_processor = KeywordProcessor()
keyword_processor.add_keywords_from_dict(dict_keywords)

#Content has 'Russia' keyword
content="Russia is a federal semi-presidential republic"
flashtext_out=keyword_processor.extract_keywords(content)

print(flashtext_out)
#Output - ['Asia']
#Output should consist of both classes/categories - Asia and Europe

@biswajitpatra - If the commit which you have done in your forked instance working fine, could you please create a pull request to this main repo so that this could be the part of release package which we can use
@vi3k6i5 - Could you please confirm if this can be merged into this primary repo?


regarding customised message componet

I saw some comments reagarding the customised message components, is there any news about this components?

@Rushsoul Yes'm working on it.
It will be available partially by the end of this week.

@Rushsoul Please track original issue for news about this feature: https://github.com/chatscope/chat-ui-kit-react/issues/2


23 implement logic for calculating a1 a3 impacts for template model

Logic implemented and used the updated BOM in #37

Hey @kiunana, two things about this pull request:

Please pull in the changes I made today regarding the Impact Calculator from #40
Based on this information from the pandas documentation, I would prefer we don't use iterrows unless we need really need to. The tl;dr from the link is that it's slower and comes with some other data type issues that could potentially be problematic. I think we're not breaking the bank here computationally, but if we have the opportunity to be more in line with the libraries we're using, I'd like to do so. Therefore, I think this calculation can be done with either the map or merge functions. Let's try to refactor with these methods in mind, and let's discuss if it creates any new issues. Regarding your try and except, I think the missing values will be filled as NaNs, so we could output the rows with NaN values using loc.