Monitor external solar panel/battery voltage

Need
Want to be able to monitor voltage of external solar panel/battery which is plugged into Boron's V_USB via the micro-USB socket.
Challenge
Particle has methods for monitoring VBATT where a LiPo battery would be connected (via the FuelGauge methods: https://docs.particle.io/reference/device-os/firmware/#fuelgauge) but unclear how to monitor V_USB. Assumption may be that V_USB comes from a stable power supply and does not need to be monitored.
Steps to Solve

[ ] Read Particle docs
[ ] Check Particle forum/post
[ ] Could feed into a Particle GPIO via voltage divider and use analog input if needed, but this wastes power

@efarq, can you check this out?

Here's a reference design for a voltage divider that Particle uses to measure the battery voltage on an Argon: https://docs.particle.io/reference/device-os/api/battery-voltage/battery-voltage-photon-2/#battery-voltage-argon. The voltage on VUSB on the Boron will be higher but the nominal resistances used here are helpful nonetheless. At ≈ 1 MΩ, current wasted will be ≈ 5 µa which isn't too bad.

fuel gauge does not seem to work with usb input: https://community.particle.io/t/fuel-gauge-reading-on-the-usb/62568/3

Yes, I didn't think the fuelgauge would work but it was worth double-checking. See the voltage divider reference above for a proposed solution. I think it will be critical to have solar panel/battery monitoring ability and a voltage divider is very easy to implement and for @ebgoldstein to include in the PCB.

@efarq to design/test voltage divider to scale 5 V nominal solar panel output down to 3.3 V max acceptable Boron ADC analog input. Using equation Vout = Vin*(R2/(R1+R2)) in standard voltage divider circuit, R2 to be ≈ 2 MΩ and R1 to be rounded up to nearest available resistor that keeps Vout ≤ 3.3 V.

Just following up - I think I got it worked out! With Phil's help, I found that R1 = 1.125 megaohms, so I rounded it up to 1.3 megaohms, and R2 = 2 megaohms. Then I set up the Boron with some resistors (photo and circuit diagram attached) and ran the code on it. The Boron V(out) was consistently reading about ~2.7V while it was receiving usb power from my laptop. I checked with the multimeter, and that matched what the Boron was saying too.
I'm uploading the script folder to the Firmware folder, and the Boron+resistors+breadboard are on the counter in Phil's lab. Let me know if anything looks like it needs to be changed.

Great, thanks, @efarq. Nice work. I see the test script in your commit here https://github.com/TinyCamML/Boron-and-OpenMV/commit/e2b713a82b8a3a851c1bc34a9584dc5fcb082bdf and in the new subdirectory here https://github.com/TinyCamML/Boron-and-OpenMV/tree/main/Firmware/TestingVoltageSplitter.
To close this issue, I recommend the following:

Create a branch of this repo
Integrate solar panel/battery voltage monitor firmware into main Boron firmware in that branch
Test new firmware
Iterate between 2<—>3 as needed
Once firmware works as desired, merge branch back into main.
Close issue with link to merge

(@ebgoldstein, workflow seem appropriate to you? As we add teammates here, I'm more than open to thinking through desired github workflow together/following your lead.)

Great! That looks good at a first glance. Confirming that you tested this on the hardware? If so, please say so and click "close with comment."
I recommend removing the percent charge variable and report. Battery charge is non-linearly related to voltage (google "battery discharge curve" for visuals) and can easily be inaccurate. Voltage will suffice. No branch/pull request needed for that change; just remove relevant parts of .ino file and recommit/push, please.

Yes, I tested it on the Boron and OpenMV that bentley was using, and it seems to work.

I'm reopening this so I remember to look at it this week. I just need to double check the code/voltage with a multimeter for this because it's still reporting about 2.8-2.9 V when plugged into usb power. I thought I scaled it up properly by multiplying by an extra ((R1 + R2) / R1)
https://github.com/TinyCamML/Boron-and-OpenMV/blob/3a03ea4b75fd1438a5236e86fadcb9e209097110/src/boron/src/TinyCamPublish.ino#L72

Solved with https://github.com/TinyCamML/Boron-and-OpenMV/commit/8a047e3b0cbdd27e7818fff782918f5fbd4ddb85

I'm actually not 100% convinced this is truly resolved. So currently, I have the TinyCamML running off of my computer usb, so it should be reading pretty close to 5V. However, it is reporting values around 4.36V, suggesting maybe my scaling is off? Any ideas of things for me to review?

Reopening based on @efarq's comment above and preliminary data showing no apparent trend in voltage (screenshot below from analysis in https://github.com/TinyCamML/Boron-and-OpenMV/blob/main/analysis/tinyCamML_analysis_20240812.ipynb).

Notes:

Unclear if data show anything unexpected after relatively short deployment
Hypothesis: a voltage regulator on Voltaic panel/power supply may "try" to supply 5 V no matter what the battery's actual charge is and we may not be able to monitor this after all. Unsure, but more investigation needed.

Closing this now after talking to Voltaic Systems and having them tell us there is no way to monitor battery voltage from the solar panel, likely due to a voltage regulator

Seems appropriate to close; I just referenced this in a new issue at https://github.com/TinyCamML/Boron-and-OpenMV/issues/43 so that this idea, even if not the solution pursued in this issue, is retained.


for which katex functions are these fonts used?

We use Katex in Joplin, but run into rendering issues now and again. Most likely because we haven't added all fonts. Laurent is reluctant to add all fonts without knowing what they are actually used for.
Is there a list or matrix of why these fonts are used and what for?
I checked the contributing doc, but there's no reference to a help forum or something else.
So I hope it's ok to ask a question here.

Ah, I see. I can't handle all the replies and comments.

@tessus did you find the info you were looking for?

Nope, we've added all fonts instead.
However, a matrix or info in the doc would still be nice as a reference. Should you ever add that info, please let m know in this issue so that I get a notification.

It's tough since there isn't a one-to-one mapping between commands and fonts.  Some things like delimiters will load different fonts based on the size of the group they wrap.

Does joplin limit which commands can be used when entering math?

No, it doesn't.

In that case including all of the fonts was the right call. 🙂


Zero pad the HESA codes

As per https://trello.com/c/IwGtWrkv/796-zero-pad-hesa-codes-in-reference-data - does this look right, Ed? I did a cunning super-nerdy regexp search and replace thing, hopefully I didn't miss any cases!

I think we'd have to consider this a breaking change, as "001" != "1" if we're treating these as strings...?

I think we'd have to consider this a breaking change, as "001" != "1" if we're treating these as strings...?

Well, we might consider it a FIXING change if the codes were meant to be padded and aren't :-)
Now, code consuming this data might be already manually padding the strings if they're too short, in which case I hope this change would make that code redundant and removable, but not harmful to service operation.
But, I am not particularly familiar with where these HESA codes go to in the services and how they're used and what the implications might be... Please advise :-)

I don't think we entirely know how these codes are consumed by downstream services - we'd have to check with the vendors who integrate with Apply. It's possible they convert them all to integers anyway, in which case it'd make no difference (and would possibly indicate we should convert them to integers too). Alternatively, possibly HESA accept both the zero padded and non zero padded ones when collecting data?
I don't know enough to be confident either way. Our API docs suggest we return them as strings in our API: https://www.apply-for-teacher-training.service.gov.uk/api-docs/v1.1/reference#qualification-object

Hopefully @edwardhorsford knows the rationale (he's the one who asked me to do this)!

I reckon we should probably consider this a breaking change, since we might mess up an integration doing strict equality checking.

Are HESA codes zero padded? https://www.hesa.ac.uk/collection/c21051/a/qualgrade doesn't appear to zero pad purely numerical grades. Could you explain why this PR/Change is necessary?

@steventux It depends on the specific data item. Many HESA codes are zero padded. The data items referenced in this repo do.
DEGCLSS
DEGTYPE
DEGEST

@edwardhorsford So are these changes https://github.com/DFE-Digital/dfe-reference-data/pull/52/files#diff-0ccfd37a88d146b53710ba30a9b8f5316fd381b4b1b4a427020cb7c8e1800e92R4-R57 effectively going to zero pad QUALGRADE codes?

@steventux and I chatted on Slack. This repo does not contain QUALGADE. The items being zero padded are ones where HESA themselves zero pad. The intention is that where we use others' codes, we'll match the form they use. In this case it means the three items above being zero padded appropriately. As and when other items get added, we'd only zero pad where HESA themselves do so.

@alaric-dfe @edwardhorsford not for this PR, but would having the HESA short name closer to these mappings be useful?
If the intention of Dfe::ReferencesData::Degrees::GRADES is to provide HESA mappings for DEGCLSS would it be useful to somehow signify this in the codebase? Perhaps a comment is enough.
One of the challenges working with HESA codes in our codebase is often which HESA data collections map to our application data. I've already made the mistake of seeing the constant name GRADES and assuming it to mean QUALGRADE. I wonder if it will help to either document or specify this as this library grows?

One of the challenges working with HESA codes in our codebase is often which HESA data collections map to our application data. I've already made the mistake of seeing the constant name GRADES and assuming it to mean QUALGRADE. I wonder if it will help to either document or specify this as this library grows?

Would something like this:
| `hesa_code` | string | The HESA [DEGCLSS](https://www.hesa.ac.uk/collection/c22053/e/degclss) code for this degree grade. |

...in the table row for hesa_code here:
https://github.com/DFE-Digital/dfe-reference-data/blob/main/docs/lists_degrees.md#dfereferencedatadegreesgrades
...be a good way to do that? If so I'll knock up a separate PR for it.

@steventux Agreed where we reference hesa codes we should be explicit about which data item we're talking about!
There won't always be a 1:1 mapping between a HESA field and one here. So a data set in the reference data might include HESA items, but it's not necessarily the same set of items.


Link to Splunk for logs when updating ECS service

This PR links to Splunk for logs when updating an ECS service. This change is necessary as we're no longer logging to AWS CloudWatch Logs, so the call to ecs-cli logs is failing. As we have the runtimeId of ECS containers logged to Splunk (via the container_id property) we can instead track ECS updates there.

LGTM thanks for this!
Could we do the same for concourse/tasks/run-task.sh (and other places where we are using ecs-cli logs)?

Absolutely, apologies for missing those. Will add them to this PR.


[QUESTION] - Use custom server list filter for load balancer with Eureka

How can i implement/use a custom server list filter based on Eureka service metadata (api version for example). Or how to overrride the default LoadBalancerClientFilter to return the correct server instance base in metadata that will be passed in request header.
For example:
I will register two services (serviceA) on Eureka. In the first instance i will put a metadata named eureka.instance.metadata-map.api-version=V1, and in the second service the metadata will be eureka.instance.metadata-map.api-version=V2.
In the service call (http:///serviceA) i will put a request header like 'api-version', 'V1' and read this header on the loadbalancerclientfilter or on the serverlistfilter.
Thanks for the help
PS: sorry for the confusing english, :(

You would want to provide your own LoadBalancerClientFilter bean.  You would then probably want to extend LoadBalancerClientFilter and override the choose method.

@ryanjbaxter Thanks for the answer,
I try to do that (in fact i did), but i can't  replace the bean "loadBalancerClientFilter" with this name. Only worked when i change the bean name. The problem is that the two filters are callers.
What a did kind of worked out, because i changed the order of my filter to be called before the "loadBalancerClientFilter" and change the "GATEWAY_SCHEME_PREFIX_ATTR",
this way when the filter is called  it sees that it has already been modified and ignores.

Yes the bean name might need to be different, but the bean in GatewayLoadBalancerClientAutoConfiguration should not be created as long as your bean is created first since it is @ConditionalOnMissingBean(LoadBalancerClientFilter.class)

In the version 2.0.1.RELEASE of spring-cloud-gateway-core don't have this annotation @ConditionalOnMissingBean(LoadBalancerClientFilter.class)
Here the implementation
@Configuration
@ConditionalOnClass({LoadBalancerClient.class, RibbonAutoConfiguration.class, DispatcherHandler.class})
@AutoConfigureAfter(RibbonAutoConfiguration.class)
public class GatewayLoadBalancerClientAutoConfiguration {
// GlobalFilter beans

@Bean
@ConditionalOnBean(LoadBalancerClient.class)
public LoadBalancerClientFilter loadBalancerClientFilter(LoadBalancerClient client) {
	return new LoadBalancerClientFilter(client);
}

}
i'm using this versions:
Spring boot: 2.0.4.RELEASE
Spring cloud: Finchley.SR1

In that case using the same name should work, you said it didn't, but did say how it failed.

Closing due to age of the question. If you would like us to look at this issue, please comment and we will look at re-opening the issue.


import 报错

torch 2.0.1+cu117
ubuntu22
安装bmtrain成功，import报错：
Traceback (most recent call last):
File "", line 1, in
File "/home/kidd/anaconda3/envs/cpmbee/lib/python3.8/site-packages/bmtrain/init.py", line 2, in
from .init import init_distributed
File "/home/kidd/anaconda3/envs/cpmbee/lib/python3.8/site-packages/bmtrain/init.py", line 8, in
from . import nccl
File "/home/kidd/anaconda3/envs/cpmbee/lib/python3.8/site-packages/bmtrain/nccl/init.py", line 4, in
from . import _C as C
ImportError: /home/kidd/anaconda3/envs/cpmbee/lib/python3.8/site-packages/bmtrain/nccl/_C.cpython-38-x86_64-linux-gnu.so: undefined symbol: ncclBroadcast

torch>=2.0 is not yet supported.

torch>=2.0 is not yet supported.

I switched to torch1.9.1 still report this error

try pip install bmtrain with extra flag --no-cache-dir. torch 1.10~1.13 is preferred.

pip uninstall bmtrain and then pip install bmtrain --no-cache-dir.

解决了吗

Torch >= 2.0 is now supported. (since PR #115 )


Add launch_template usage docs

Reference Issues or PRs
closes #524

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

@joneszc can you have a look if this PR is okay?

@joneszc can you have a look if this PR is okay?

Hello @viniciusdc,
The docs looks great to me, clear and concise.  I like the tip on creating a new node group with the desired launch template and migrating workloads to avoid issues with modifying/destroying a node group.


[feat]: Logs color blue unreadable

Prerequisites

[X] I have searched for existing issues that already suggest this feature, without success.

Describe the Feature Request
Blue color for DEBUG is very hard to read.
(I actually had to highlight it to read it)

Describe the Use Case
Make it easy on eyes
Describe Preferred Solution
Make it brighter
Describe Alternatives
No response
Anything else?
Color for WARN could be a bit more yellow

Note to self: alter colors according to lib configuration


Bump OCP versions: 4.10, 4.8, 4.11

OpenShift updates: 4.10.35 -> 4.10.36, 4.8.50 -> 4.8.51, 4.11.7 -> 4.11.8
RHCOS updates:
/test edge-e2e-metal-assisted-4-10 edge-e2e-metal-assisted-4-8 edge-e2e-metal-assisted-4-11
/cc @osherdp
/hold

Closing as duplicate of https://github.com/osherdp/assisted-service/pull/159

Closing as duplicate of https://github.com/osherdp/assisted-service/pull/160


Check the CPU and memory of a new host before joining it to the cluster

Describe what this PR does / why we need it
Does this pull request fix one issue?

Fixes #362
Describe how you did it
Describe how to verify it
master: cpu >= 2 memory >= 4GB
node:   cpu >= 2 memory >= 4GB
otherwise throw exception (show a warning if execute shell to get infomation failed)
Special notes for reviews

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

@wisheen could you add my dingtalk or wechat?  fangnux

I think we should add those checks into the checker module.

The runtime interface is very important, if you want to change any module interface, we need to discuss it in detail.

type CPUChecker struct {
    cluster *v1.Cluster
}

func NewCPUChecker(cluster *v1.Cluster) Checker{
....
}

func (c *CPUChecker)Run() error {
 .....
}


time out

Closing this for now - If you still have issues, please provide more details so that we can find a solution.


Get appdag.html working with data

Story
Parent Issue
Complete port of AppDagCtrl.scala

merge into parent issue #952


Publishing the project

Should we continue to use bower ?
Should we publish on npm ?

IMO we can drop bower, and just support npm.
maybe even add it to a public cdn like cdnjs or something similar

https://docs.npmjs.com/misc/scope
Should we publish on npm as bare , or as scoped package ?
Scope would allow us to later publish other packages under the same umbrella.
It also allows us to take a name that is already taken as bare publish.
I propose to publish as @core/core dropping .js as it is technically not limited to js.

for public cdn https://unpkg.com/ is also a possiblity

https://github.com/cdnjs/cdnjs/blob/master/PULL_REQUEST_TEMPLATE.md#essential-checklist
cdnjs is not inclusive to all and does not support latest version

well, to make things simpler, I'd only publish on npm then... at least for the initial release. We can use cdn's in case other users request it
About the naming, it's really up to your personal preference... if @core is available, that would be interesting, I also don't like the .js in the package name

@core is not available, what about @eroc ?
That would make a @eroc/core package

sounds good :)
Small and easy to remember 💃

OK, I already created eroc org, ask me if you want to be added as well. I will proceed with https://bower.io/blog/2017/how-to-drop-bower-support/ and publishing to npm

Published https://www.npmjs.com/package/@eroc/core


missing logsoftmax operation in example Rnnlm

in example/Rnnlm, we define a class Rnnlm with forward function:
std::tuple<Variable, Variable, Variable>
forward(const Variable& input, const Variable& h, const Variable& c) {
auto output = embed(input);
Variable ho, co;
std::tie(output, ho, co) = rnn(output, h, c);
// Truncate BPTT
ho.setCalcGrad(false);
co.setCalcGrad(false);

output = linear(output);
return std::make_tuple(output, ho, co);

}
and use CategoricalCrossEntropy to calc loss, but CategoricalCrossEntropy need logsoftmax as input, so: Maybe this is a bug!
RnnLm::forward  function ignores operation: logsoftmax, which should use after linear and before return.

cc @awni

This does indeed seem to be a bug! Could you submit a PR?

The example still seems to miss the logsoftmax fwd in the RnnLm model forward:
https://github.com/facebookresearch/flashlight/blob/1a47844ea1afbcbee563b5e8a784a6d32cdb4da8/flashlight/fl/examples/RnnLm.cpp#L109
and indeed, neg/crazy loss as reported in:
https://github.com/facebookresearch/flashlight/issues/137
Retouching that way:
    output = linear(output);
    LogSoftmax lsm(0); // max on the main dimension
    output = lsm(output);
    return std::make_tuple(output, ho, co);

seems to do the job
$ ./flashlight/fl/examples/RnnLm simple-examples/data/
Vocab size: 10000
Epoch 1 - Train Loss: 5.69 Validation Loss: 5.25 Validation Perplexity: 191 Time per iteration (ms): 10.9
Epoch 2 - Train Loss: 4.96 Validation Loss: 5.01 Validation Perplexity: 150 Time per iteration (ms): 10.8
Epoch 3 - Train Loss: 4.7 Validation Loss: 4.93 Validation Perplexity: 138 Time per iteration (ms): 10.8
Epoch 4 - Train Loss: 4.52 Validation Loss: 4.88 Validation Perplexity: 132 Time per iteration (ms): 10.8
Epoch 5 - Train Loss: 4.26 Validation Loss: 4.8 Validation Perplexity: 121 Time per iteration (ms): 10.7

@tlikhomanenko  @awni  Still worth a PR ?

closable

Thanks a lot for the PR and catching this!


[AIRAVATA-2619] Refactoring App Catalog implementation - GatewayResourceProfile

The pull request contains the registry refactoring changes for the AppCatalog->GatewayResourceProfile. Basically a start to the registry refactoring task. GatewayResourceProfileRepository has been implemented instead of the existing Impl implementation and the changes have been done to the RegistryServiceHandler to call this repository instead of the Impl.
Changes:

Added a directory for app_catalog in the existing registry-refactoring module.
Added GatewayResourceProfileRepository class .(app_catalog/GatewayResourceProfileRepository.java).
Added StoragePrefRepository and ComputePrefRepository classes. Note that these classes still need to be implemented and currently have been added just to support the GatewayResourceProfile methods.
Added AppCatalogJpaUtils  for the app catalog entity manager  (JpaUtil/AppCatalogJpaUtils).
Added tests for the repository methods GatewayResourceProfileRepositoryTest.

Next Steps:
Implement ComputePrefRepository and StoragePrefRepository methods.

@sachinkariyattin are there any DB changes compared to gateway resource profile in develop branch?

@smarru No. I have used the same DB script (appcatalog-derby.sql) that currently exists in develop branch.


Update Understand-determine-basal.md

added description of short_avgdelta and long_avgdelta
put json keys in code blocks (only for Summary of Inputs -- haven't gotten to Summary of Outputs section (yet))

@scottleibrand is that a blocker for merging this, or just "nice to do" in future?


[backport] Use THRUST_OPTIONAL_CPP11_CONSTEXPR

Backport of #5002

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit ee8b5b2d7410ddd6874a844a2540ebff84e57325, target branch v9) failed with status FAILURE.


[Prototype] Keccak benchmark p3

I assume we're not merging the PR as is?

Yeah, don't think so. But I'll keep this branch in sync with main so we can can have a benchmark for monolithic Plonky3.

Yeah, I think with the integrated prover, this is obsolete!


github rate limit: use new method of getting headers

After bumping the ocktokit version we didn't realize the method of
extracting the headers (for ratelimit) had changed.
CC @addison-grant
Closes iFixit/pulldasher#184

QA 👍
We are now parsing and utilizing the rate limit headers again:

CR :shipit:

CR :coffee:

dev_block 👍  on adding more logs.

un_dev_block 👍

CR :coffee:

CR :old_key:

QA 👍


Question for lines 308 - 317

I am receiving the following error when trying to use models. Not sure what the issue is.
return this.attribute(_.deepCopy(attribute)).code(code).control();
                                                                   ^
TypeError: Object [object Object] has no method 'control'

 // reading
 ModelMaker.prototype.i = function (code, attribute) {
 console.log('hello');
 return this.attribute(_.deepCopy(attribute).code(code).reading());
} ;

// control
ModelMaker.prototype.o = function (code, attribute) {
return this.attribute(_.deepCopy(attribute).code(code).control());
};

Or this issue, basically for the 'code' call.
return this.attribute(_.deepCopy(attribute).code(code).control());
                                                ^
TypeError: Cannot call method 'code' of undefined

Hi Richard,
How is this being called?
On Tue, Feb 17, 2015 at 6:25 PM, Richard<EMAIL_ADDRESS>wrote:

Or this issue, basically for the 'code' call.
return this.attribute(_.deepCopy(attribute).code(code).control());
^
TypeError: Cannot call method 'code' of undefined
—
Reply to this email directly or view it on GitHub
https://github.com/dpjanes/node-iotdb/issues/2#issuecomment-74777924.

I am calling it via 'node model.js' for homestar-lifx.

Can I get the complete stack trace?
D.
On Tue, Feb 17, 2015 at 8:29 PM, Richard<EMAIL_ADDRESS>wrote:

I am calling it via 'node model.js' for homestar-lifx.
—
Reply to this email directly or view it on GitHub
https://github.com/dpjanes/node-iotdb/issues/2#issuecomment-74793837.

I think I've got the problem fixed.
D.
On Wed, Feb 18, 2015 at 3:14 AM, David Janes<EMAIL_ADDRESS>wrote:

Can I get the complete stack trace?
D.
On Tue, Feb 17, 2015 at 8:29 PM, Richard<EMAIL_ADDRESS>wrote:

I am calling it via 'node model.js' for homestar-lifx.
—
Reply to this email directly or view it on GitHub
https://github.com/dpjanes/node-iotdb/issues/2#issuecomment-74793837.

I haven't had a chance to take a look, will come back to this tomorrow.


Implement Email Sending Functionality in React App

Description:
As a part of our user authentication flow, we need the capability to send emails from our React application. This issue encompasses the necessary frontend functionality for the email-sending feature.
Acceptance Criteria:

Users should be able to input their email.
There should be validation in place to ensure only emails with our accepted domains are submitted.
On email submission, a POST request should be made to our Flask backend to trigger the email sending process.
Users should receive feedback on the UI if the email is sent successfully.
Users should also receive feedback if there are any errors, especially validation errors.

I'm confused as to why this was closed as completed @CFIALeronB ? Isn't this what @k-allagbe is working on?

I'm confused as to why this was closed as completed @CFIALeronB ? Isn't this what @k-allagbe is working on?

This issue is located in the membrane-frontend repository. Shouldn't this issue be in the membrane-backend repository instead?

@CFIALeronB most user-visible feature request will have both a frontend issue and a backend issue. the frontend issue should probably come with mockups in the description while the backend issue will detail sequence diagrams and api endpoints.


Fix Path Traversal Vulnerability

Fixes #16

webserver.py is going to be removed soon and it currently has not real functionality. I will reject this PR since the CLA is not signed.


Mds 340

individuals-if-api-stub
Usage

This stub should be switched out with the IF Stub in the external test environment.

This stub has a post endpoint for each IF endpoint that will allow the consumer to post the
payload they wish to get back from IF

On hitting the GET endpoints for a scoped request the stub should serve back the correct payload

When the consumer posts the JSON payload they will need to provide the use case

Use cases can vary per API; for example:

Individuals Details API

LAA-C3-residences
LAA-C4-residences
HMCTS-C3-residences
HMCTS-C4-residences
LSANI-C1-residences
LSANI-C4-residences
NICTSEJO-C4-residences
LAA-C4-contact-details
HMCTS-C4-contact-details

Individuals Benefits and Credits API

LAA-C1-working-tax-credit
LAA-C2-working-tax-credit
LAA-C3-working-tax-credit
LAA-C1-child-tax-credit
LAA-C2-child-tax-credit
LAA-C3-child-tax-credit
HMCTS-C2-working-tax-credit
HMCTS-C3-working-tax-credit
LSANI-C1-working-tax-credit
LSANI-C3-working-tax-credit

Individuals Employments API

LAA-C1
LAA-C2
LAA-C3
LAA-C4
LSANI-C1
LSANI-C3
HMCTS-C2
HMCTS-C3
HMCTS-C4
NICTSEJO-C4

Individuals Income API (PAYE)

LAA-C1
LAA-C2
LAA-C3
LAA-C4
HMCTS-C2
HMCTS-C3
HMCTS-C4
LSANI-C1
LSANI-C3
NICTSEJO-C4

Individuals Income API (SA)

LAA-C1
LAA-C2
LAA-C3
LAA-C4
HMCTS-C2
HMCTS-C3
HMCTS-C4
LSANI-C1
LSANI-C3
NICTSEJO-C4

Example API POSTs

Individual Details API

individuals/contact/details/nino/CS700100A?startDate=2019-01-01&endDate=2020-03-01&useCase=LAA-C4-residences
individuals/contact/details/nino/CS700100A?startDate=2019-01-01&endDate=2020-03-01&useCase=LAA-C4-contact-details

Individual Benefits and Credits API

individuals/tax-credits/nino/CS700100A?startDate=2019-01-01&endDate=2020-03-01&useCase=LAA-C1-working-tax-credit
individuals/tax-credits/nino/CS700100A?startDate=2019-01-01&endDate=2020-03-01&useCase=LAA-C1-child-tax-credit

Individual Employments API

individuals/employments/nino/CS700100A?startDate=2019-01-01&endDate=2020-03-01&useCase=LAA-C1

Individual Income API (PAYE)

individuals/income/paye/nino/CS700100A?startDate=2019-01-01&endDate=2020-03-01&useCase=LAA-C1

Individual Income API (SA)

individuals/income/sa/nino/CS700100A?startYear=2019&endYear=2020&useCase=LAA-C1

License
This code is open source software licensed under the Apache 2.0 License.

TODO

Test
Fix
Update spec tests
PR and merge

TODO

Test
Fix
Update spec tests
PR and merge


dnvm crashes when run with no arguments on osx-arm64

I'd imagine this should show usage instead of a stack trace.
zsh ❯ dnvm
dnvm 0.6.10 c310a3458c8551fb511ef9c07e49dc3853b9670c

Unhandled Exception: System.InvalidOperationException: Unknown command
   at Dnvm.CommandLineArguments.ParseRaw(IAnsiConsole, String[]) + 0x6c4
   at Dnvm.CommandLineArguments.TryParse(IAnsiConsole, String[]) + 0x10
   at Dnvm.Program.<Main>d__2.MoveNext() + 0x104
--- End of stack trace from previous location ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x24
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x100
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task, ConfigureAwaitOptions) + 0x68
   at Dnvm.Program.<Main>(String[] args) + 0x3c
   at dnvm!<BaseAddress>+0x57a528

I am getting the same:

dnvm 0.6.10 c310a3458c8551fb511ef9c07e49dc3853b9670c
Unhandled Exception: System.InvalidOperationException: Unknown command
at Dnvm.CommandLineArguments.ParseRaw(IAnsiConsole, String[]) + 0x6c4
at Dnvm.CommandLineArguments.TryParse(IAnsiConsole, String[]) + 0x10
at Dnvm.Program.d__2.MoveNext() + 0x104
--- End of stack trace from previous location ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x24
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x100
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task, ConfigureAwaitOptions) + 0x68
at Dnvm.Program.(String[] args) + 0x3c
at dnvm!+0x57a528
[1]    3819 abort      dnvm

Yeah this is an oversight on my part -- it's viewing "plain" dnvm as an invalid command line.
What would people expect to be show? The help page?


Consider migrating from dep to go modules.

I'm submitting a ...  (check one with "x")
[] bug report => search github for a similar issue or PR before submitting
[x] enhancement request
[] feature request
[] support request
[] general question

Current behavior (how does the issue manifest):
Flogo uses dep.
Expected behavior:
Flogo could use go modules instead.
Minimal steps to reproduce the problem (not required if feature enhancement):
What is the motivation / use case for changing the behavior?
Modules are the future of package management in Go, well for now at least.  :)
Please tell us about your environment (Operating system, docker version, browser & verison if webui, etc):
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
Additional information you deem important (e.g. issue happens only occasionally):

Hi Matthew! Good point and we’re in the process of doing just that and moving it to another organization on GitHub in the process. You can find the work in progress code (which is pretty stable) at https://github.com/project-flogo. The new CLI, which uses modules, can be found at https://github.com/project-flogo/cli. Feel free to check that out and let us know if you have any thoughts or suggestions.

Excellent, thank you Leon.
Should I switch over to using project-flogo now?
On Wed, Mar 27, 2019 at 11:35 PM Leon Stigter<EMAIL_ADDRESS>wrote:

Hi Matthew! Good point and we’re in the process of doing just that and
moving it to another organization on GitHub in the process. You can find
the work in progress code (which is pretty stable) at
https://github.com/project-flogo. The new CLI, which uses modules, can be
found at https://github.com/project-flogo/cli. Feel free to check that
out and let us know if you have any thoughts or suggestions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TIBCOSoftware/flogo-cli/issues/129#issuecomment-477444489,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABO5b8Rhms6Br84NgNEulb-sz2jstgvGks5vbEaEgaJpZM4cPRk9
.

We’re finishing up the last things before the 0.9 release so most things should work. I’d recommend trying it out and letting us know if there are any issues with your specific use case so we can tackle them


Python Functions ModuleNotFoundError after Oryx installs Virtual Environment to Unexpected Path

When building and deploying my python functions using Github workflow, the functions build using oryx. The virtual environment in oryx gets set up in the pythonenv3.8 folder (see logs below). But at runtime, I'm getting an error "Failure Exception: ModuleNotFoundError: No module named 'requests'".  I believe the Static Web Apps functions expect the virtual environment to be in the .venv folder, not the pythonenv3.8 folder.
There are two solutions to this problem that I can think of: (1) change oryx to install packages to an expected folder, or (2) search for packages in pythonenv* when running functions.
Downloading and extracting 'python' version '3.8.6' to '/tmp/oryx/platforms/python/3.8.6'...
Downloaded in 2 sec(s).
Verifying checksum...
Extracting contents...
Done in 4 sec(s).

Python Version: /tmp/oryx/platforms/python/3.8.6/bin/python3.8
Python Virtual Environment: pythonenv3.8
Creating virtual environment...
Activating virtual environment...
Running pip install...
[04:44:20+0000] Collecting azure-functions
[04:44:20+0000]   Downloading azure_functions-1.5.0-py3-none-any.whl (124 kB)
[04:44:21+0000] Collecting requests==2.25.0
[04:44:21+0000]   Downloading requests-2.25.0-py2.py3-none-any.whl (61 kB)
[04:44:21+0000] Collecting idna<3,>=2.5
[04:44:21+0000]   Downloading idna-2.10-py2.py3-none-any.whl (58 kB)
[04:44:21+0000] Collecting chardet<4,>=3.0.2
[04:44:21+0000]   Downloading chardet-3.0.4-py2.py3-none-any.whl (133 kB)
[04:44:21+0000] Collecting urllib3<1.27,>=1.21.1
[04:44:21+0000]   Downloading urllib3-1.26.2-py2.py3-none-any.whl (136 kB)
[04:44:21+0000] Collecting certifi>=2017.4.17
[04:44:21+0000]   Downloading certifi-2020.12.5-py2.py3-none-any.whl (147 kB)
[04:44:21+0000] Installing collected packages: azure-functions, idna, chardet, urllib3, certifi, requests
[04:44:22+0000] Successfully installed azure-functions-1.5.0 certifi-2020.12.5 chardet-3.0.4 idna-2.10 requests-2.25.0 urllib3-1.26.2
WARNING: You are using pip version 20.2.1; however, version 20.3.1 is available.
You should consider upgrading via the '/github/workspace/api/pythonenv3.8/bin/python -m pip install --upgrade pip' command.
Preparing output...

Copying files to destination directory '/bin/staticsites/ss-oryx/api'...
Done in 0 sec(s).

Looks like we're currently not installing Python packages to a location that the Functions Python worker is expecting (.python_packages/lib/site-packages). We'll need to change the Static Web Apps action or Oryx to use the correct path.
In the meantime, as a workaround until this is fixed, you can add a step before the Static Web Apps action to install packages into the right place. See this commit for an example.

I will take a look at it ... @molson194  by any chance do you have any hardcoded dependency in your code/config where you are referring .venv as path or something?

@arroyc Nothing hardcoded. Take a look at this repro that simply pulls in requests and uses it in the httptrigger function. https://github.com/anthonychu/20201210-test-python-swa


Mistake in Cross Validation for Classification of credit-g data

Hi,
In https://github.com/Prasham8897/Applied_Machine_Learning/blob/master/Classification %26 Regression/pds2136_Task1.ipynb In [21]:, preprocess = make_column_transformer((OneHotEncoder(handle_unknown="ignore"), categorical_columns_index)) is being used to preprocess the data without scaling (as compared to scaling in the next cell). However, make_column_transformer by default drops columns not specified explicitly. Thus, remainder='passthrough' should be specified.

Thanks for pointing that out. Will update it.


[cudnn] add port and enable it in darknet

This will be very tricky to test in CI. I will open it for review once I will be able to test it more locally

@cenit, is there any rush on this PR? @rastaban is out of office right now and he's the one most familiar with updating the software on all the CI machines (since we need to get CUDA 10).

There is no Cuda installed in any CI machine, unfortunately. That's why it's so difficult to test this PR, since it starts from a broken port for the CI point of view. It works locally if you try (don't need a CUDA card to install CUDA software, if you know how to proceed ;) )
No rush for it anyway, but it might be nice if we ever get Cuda installed for future CI tests :)
(I have some scripts for headless installation on linux/windows if you need them)

I think we'd happily take those scripts. We definitely want our CI systems to be capable of building all the ports.

@cbezault here it is. Taken directly from the darknet CI infrastructure
install_cuda.cmd.txt
install_cuda.sh.txt

@vicroms while integrating cudnn into some work, I found some bugs in OpenCV (3 and 4) dealt directly for my simplicity inside this PR.

@cbezault any news here?

I am going to start bringing up a new set of VMs to update some other things, I will add this to the script while I am at it.  It may not be until next week sometime that they are available though.

Ok! I hope this pr can be merged next next week!
Thanks for your work!

@rastaban sorry for pinging again. News?

@vicroms while integrating cudnn into some work, I found some bugs in OpenCV (3 and 4) dealt directly for my simplicity inside this PR.

since this is still true, I also added the fix to issue #7981

cuda is now installed on the windows VMs.

wonderful. I merged with master to trigger a rebuild. I hope it will be a green check :)

It looks like cuda is passing on all windows machines.  Based on the warning messages in cudnn it should probably build for x64-windows-static, let me know if there are additional changes that are needed on the CI machine to get that passing.
I just installed cuda on the Linux CI machines and have started a rebuild on the vcpkg-Linux-PR pipeline.  It may fail due to #8247, I will watch for the results.

if you prepared the environment as requested by the port file, it should work on linux x64 (I use it locally).
I will analyse the x64-windows-static now

@Rastaban x64-windows-static is an expected failure unfortunately.
Since cudnn is a dynamic-linking-only library, Vcpkg just exits saying Refusing to build unexpected dynamic library against the static CRT. (you have to request explicit tolerance for this situation in a custom triplet, I learned that with @vicroms when we were working on removing all references to CMAKE_EXPORT_ALL_SYMBOLS https://github.com/microsoft/vcpkg/pull/5937#issuecomment-488960198 )

@Rastaban is there anything I can do to help deciding if merge this PR?

@cenit thanks for adding cudnn port. However, I think the way cuda port was implemented is better, as long as  cudnn lib is put in the same directory as cuda (should be easy: they have the same directory structure). I've made one here. It's less cumbersome and works fine on Windows 10 so far.
By the way, darknet CONTROL file, feature cudnn should also depends on package cudnn.

@ChaoJia I agree that a cudnn dependency is missing from darknet (I don’t know how it happened, it was the main reason of that pr... :D )
I don’t understand what’s the problem with CUDNN installed inside vcpkg folder. It works, at least in my cases. If possible, I’d like to be able to install CUDA also with vcpkg, not just checking if it is installed. Did you experience any problem? Why do you think this way is not good?

@cenit the main reason is that it is cumbersome. cudnn64_7.dll is 401 MB. If you have multiple projects using cudnn, vcpkg will make multiple copies of it. That could add up quickly.
cudnn is not built by vcpkg, so it does not come with a debug version and release version, while other non-empty packages do. The same goes for cuda.
Some projects might rely on cuda but not necessarily use vcpkg. If cuda is physically installed within vcpkg instead of the place where it used to be, CMake (and maybe some other software which rely on something provided by cuda) would probably not be able to find cuda naturally without any tweaks. In that case, cuda has to be installed twice. That's about 2 GB.
If cuda port remains the way it is right now, then it is advantageous to put cudnn in the cuda install path and keep an empty vcpkg port for cudnn too (at least path has already been set). No multiple copies are needed. Since cuda installation is taken care of by user, it is kind of straightforward to leave cudnn to user too.
Anyway, it's just a matter of personal taste, but please leave cuda port the way it is if it does not  irritate you too much.

of course CUDA will remain like it is now, most of all because the user setup inside vcpkg is impossible.
Understood your point about cudnn


debugger current line vertical spacing

I'm testing the plugin with some test php code and it appears that the current line vertical spacing is huge. Anyway to change this?

In Visual Studio code

Sublime Text 3.2.2 on Windows 10

Unfortunately there isn't a great way to draw the current line without phantoms which mess with the vertical spacing

thank you for the reply, better late than never :)


kubernetes: add pod, service and RC overview

Can you take a look at this @kelseyhightower?

FYI: for some reason Github won't show the SVGs inline :/


Testing a 'yield put'

It doesn't seem right to just assert that a PUT exists... I want to test exactly what that PUT was.
How would you test this? I'm finding testing sagas to be a difficult task... all the examples I find are to test the trivial things, but there are no examples for testing 'real-life', complex sagas...
Thank you for any help you might provide to this poor soul 👍

you just create an effect in your test and check its deep equality against yielded one
next = gen.next()
expect(next.value).toEqual(put(itemDeleteSuccess()))

Haha looks so simple once you know it!
Thanks @Andarist !!

@Andarist what if the yield put is called with a TON of arguments? How do you test without passing all those args?
next = gen.next()
expect(next.value).toEqual(put(itemDeleteSuccess({
  superComplexObject: {
    thatNeverEnds: {...}
  }
})))

Well, it probably depends on what do you want to test rly. You could play with this https://github.com/yelouafi/redux-saga/blob/b29b4315e729e45f71feb06f69c5afd7eee0e73e/src/utils.js#L2 to grab effect's payload and test what you want about it. Cause effects are just plain object, similarly to actions.


Downgraded pubnub version

Description:
Downgrade the pubnub version prior to when they started using pycryptodome.
Related issue (if applicable): fixes #2411
Checklist:
If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

The problem with this change is that now it works on Linux but it no longer works on Windows 10.
What would be a good way of resolving this ?

I never had an issue with the newer version of pubnub in Linux. Is there
another component that requires pycrypto? They only people I saw that
reported an issue with the original version were people using docker, can
something be changed in docker?
On Jul 4, 2016 12:42 PM, "Paulus Schoutsen"<EMAIL_ADDRESS>wrote:

The problem with this change is that now it works on Linux but it no
longer works on Windows 10.
What would be a good way of resolving this ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2420#issuecomment-230325714,
or mute the thread
https://github.com/notifications/unsubscribe/AGIoAjMPbBQraGazVGp2p1xH-GzwA2k9ks5qSTf-gaJpZM4JDsIs
.

The only thing that the Docker image does is that it installs all the requirements at once. I wouldn't be surprised if somewhere something depends on pycrypto.

BTW I used that installer on Windows 10 64-bit.
trying to install the usual way resulted in an error when it was unable to find C++ 10.0 even though it was installed.

I received a replay from pubnub. Based on their response I think it would
be best to switch to their reduced crypto branch. This doesn't have any
crypto dependencies so it should work on all platforms. They also mention
switching to cryptography.io witch appears to work in Windows, but not sure
when they will be done with that. Pubnub response is below.
" cryptography.io is the library we are going to use for PubNub Python SDK
v4, we have scoped this as one of the dependencies. The exact timeline for
v4 is still in the works but we hope to start seeding the first beta in the
next month or so.
For the reduce_crypto branch (a version of the SDK with no encryption
library dependency), the ssl_on and cryptography are two separate features.
ssl_on ensures that the client communicates to PubNub via https://
protocol, while the cryptography dependency is required to encrypt the
actual payload of the published messages and is activated only if you are
passing a cipher_key."
On Jul 4, 2016 12:50 PM, "william scanlon"<EMAIL_ADDRESS>wrote:

I never had an issue with the newer version of pubnub in Linux. Is there
another component that requires pycrypto? They only people I saw that
reported an issue with the original version were people using docker, can
something be changed in docker?
On Jul 4, 2016 12:42 PM, "Paulus Schoutsen"<EMAIL_ADDRESS>wrote:

The problem with this change is that now it works on Linux but it no
longer works on Windows 10.
What would be a good way of resolving this ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2420#issuecomment-230325714,
or mute the thread
https://github.com/notifications/unsubscribe/AGIoAjMPbBQraGazVGp2p1xH-GzwA2k9ks5qSTf-gaJpZM4JDsIs
.


Problems extracting imu-pose data fron BAG files using rs-convert

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>85

Operating System & Version
Win (7/10)

Kernel Version (Linux Only)

Platform
PC

SDK Version
2.5.1 for rs-convert

Language
C

Segment
Infrastructure images/depth

Issue Description
rs-convert problems in extracting imu-pose data.
I performed a number of Viewer collections using two D455 cameras. Bag file sizes varied between 1-68 Gigabytes.  Of 10 BAG files, only 3 fully converted the image and imu-pose data.
I ran the latest version of rs-convert -v and then rs-convert -p separately. I noticed that the imu-pose output csv file was often truncated. All output files should have the same computed elapsed time given number of samples / sample rate. For the smaller BAG files, this was not a problem, but for bigger BAG files, discrepancies in file counts (png, txt, csv) or an expected number of imu-pose Gyro and Accel records were too few or absent.
Attached find a spreadsheet that summarizes the results for ten bag files. Same Viewer configuration used in each collection.
Is there a documented BAG filesize limit that rs-convert can process?  It appears that rs-convert is better at handling color and depth but has problems with imu-pose.
When watching the imu-pose file in a folder as rs-convert is processing the bag file, imu-pose filesize alternates between 0 and a growing number as if the file is being opened and closed repeatedly with the final filesize being much smaller. As the imu-pose file contains first the Gyro data followed by Accel data, this would indicate that rs-convert has to keep track of the Gyro and Accel data for the entire BAG file before it can output the final imu-pose file. How this is done in rs-convert may be the problem whether in memory or temporary files.
I am using Windows 10 with 32GB of memory and SSD disks.
Question 1.  What is the BAG filesize limit for rs-convert?
Question 2.  Can the imu-pose portion of rs-convert be modified to handle these large BAG filesizes?
Question 3.  What else may be causing a problem?
As always, thank you for your support.
14Sept2022_summary.xlsx

Hi @jimmek  I am not aware of a bag file limit in rs-convert or in librealsense in general.  Loading a large file volume into the computer's memory will have a cost to the available memory space though, even if the amount of memory used is less than the total size of the file(s).
You could monitor available memory in Windows whilst rs-convert is running by bringing up the Task Manager window with Ctrl-Alt-Del and then selecting the Task Manager option and left-clicking on the Task Manager's Performance tab.

In terms of bag size, 68 gb is very large and such file sizes are not usually encountered.  Around 20 gb would be a more common maximum size reported by RealSense users.  The more streams that are active simultaneously and the higher the resolution / FPS settings used, the greater the bag size will be.  Some RealSense users have reported that the RGB stream seemed to them to use a much greater proportion of file size than depth data.
You could explore how much a difference it would make to enable the bag compression option in the RealSense Viewer.  This can be done using the instructions at https://github.com/IntelRealSense/librealsense/issues/7494#issuecomment-704411013
Another way to reduce bag file size would be to record the bag in the RealSense ROS wrapper with the rosbag record command instead of using the Viewer and use rosbag record's --split command to automatically close a bag recording and start a new bag file when a specified maximum limit was reached, such as file size or duration.  More information about this can be found at https://github.com/IntelRealSense/librealsense/issues/2424#issuecomment-423284173
In regard to rs-convert code changes: as your computer has a very large memory capacity, I wonder whether adding the Keep() instruction to store frames in the computer's memory instead of writing them to disk may improve performance.  Memory capacity is an important factor because Keep() can consume a lot of the computer's memory space when storing the frames.  Once all frames are stored in memory, the pipeline can be closed and batch processing performed on all the stored frames in a single action (such as applying post-processing and / or alignment and then saving the frames to file.  An example of a C++ discussion about Keep() is at https://github.com/IntelRealSense/librealsense/issues/1942

Hello Marty,
I reran rs-convert (-v option) with the Windows Performance Monitor tracking the rs-convert memory requirements directly. It used a baseline of 56.6 MB of memory initially and slowly worked its way up to 63.4 MB. The program uses very little memory; that is not the problem. Non-imu data is extracted and written sequentially. Behavior of the program suggests that it is reading imu-pose data sequentially and both storing in data memory for sorting and continually rewriting the segregated Gyro and Accel data into a newly opened imu-pose file each time. I believe this is the cause of the problem in that rs-convert memory management and writing of the imu-pose data is flawed. The easiest solution would be to write the Gyro and Accel data into separate files (sequential action) which would eliminate the apparent memory management - file management coding problem.

Do you have the ability to create a support ticket to Intel on the Zendesk support channel, please?  This would enable you to submit a request to the RealSense team for the adjustments to rs-convert that you suggest in order to improve the tool's handling of very large bag files.

Hi @jimmek  Do you have an update about this case that you can provide, please?  Thanks!

Still working on the issue. Will contact Zendesk this week.
-----Original Message-----
From: MartyG-RealSense @.>
To: IntelRealSense/librealsense @.>
Cc: jimmek @.>; Mention @.>
Sent: Sun, Oct 2, 2022 3:14 am
Subject: Re: [IntelRealSense/librealsense] Problems extracting imu-pose data fron BAG files using rs-convert (Issue #10934)
Hi @jimmek Do you have an update about this case that you can provide, please? Thanks!—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks very much for the update, @jimmek

Hello Marty G,
Reference Issue #10934
Looking at the INTEL’s latest SDK, we have found pieces ofcode that would help us in generation trajectory from the imu-pose data.
We need to:
Read Pose-Frame from BAG file, then for each frame,
a. Compute POSE_DATA using INTEL’s algorithm,s to get(translation/velocity/rotation/Angular velocity/Angular acceleration/trackerconfidence/mapper confidence) with time tag.
b. With POSE_DATA, Compute MAT(16) quaternion.
c. Save POSE_DATA and MAT(16) in output files.
As a minimum we need translation and rotation with time tagusing the Gyro and Accel Data in the BAG files from our D455 cameras.
Please Help.
Jim Mekemson
Starodub, Inc.
Message ID: @.***>

Hello Marty G,
Reference Issue #10934
We have looked at INTEL’s latest SDK. Rs_convert providesthe imu-pose file with two tables in a single file, one for the GYRO with sevencolumns with 3 angular velocities in radians/sec and one for ACCEL with sevencolumns with three accelerations in m/sec2.
We studied the most recent downloads of the SDK. Looking at RS_POSEfor the T265, it looks like there are functions that compute pose-data with thefollowing data elements in a real-time stream:
·      Translation
·      Velocity
·      Rotation
·      Angular Velocity
·      Angular Acceleration
·      Tracker Confidence
·      Mapper Confidence
QUESTION 1:  We areusing D455 cameras. Do you have  thean example where one can read a BAG file andcompute the same information, at least the Translation data element?
QUESTION 2:  Are?readditional resources that INTEL can share about pose computations inpost-process beyond the information in the SDK?
QUESTION 3: The examples in the SDK are related to T265cameras. Is there another SDK for D455 cameras, or is the code for T265 applicableto D455 camera?
Jim Mekemson
Starodub, Inc.
Message ID: @.***>

Hi @jimmek  Pose stream data is only supported on the RealSense T265 Tracking Camera model.  A T265 can be paired with a 400 Series camera such as your D455 (for example, mounting both on the same bracket), as described in Intel's T265 and 400 Series guide Better Together at the link below.
https://www.intelrealsense.com/depth-and-tracking-combined-get-started/
The only way of plotting a trajectory with a D455 may be to use the ROS guide at the link below.
https://shinkansan.github.io/2019-UGRP-DPoom/SLAM.html
The T265 has an rs-trajectory SDK example program.
https://github.com/IntelRealSense/librealsense/tree/master/examples/trajectory

In regard to your other set of questions:

If you recorded gyro and accel data to a bag file with a D455, I believe that you could use SDK script code to obtain most of the items in the list.  Tracker and Mapper confidences would likely not be obtainable from that D455 gyro and accel bag data though.

If you seek information about the IMU that is not available publicly then I would again recommend contacting Intel through a private Zendesk support ticket.

The RealSense SDK 2.0 (librealsense2) supports both D455 and T265.  There is not a separate SDK.  T265-only examples such as rs-pose and rs-trajectory cannot be used with 400 Series cameras.

I have been able to confirm that rs-convert for imu-pose data extraction does not always extract imu-pose data satisfactorily. For different BAG files, no data, truncated, full data extracted has been observed. The current number of ConnID and op-code within the BAG files are present. I feel the BAG files themselves are good. It may also be interfering with csv depth data as I sometimes get more png depth than csv depth files.
I need rs-convert.exe before SDK 2.47 when imu-pose extraction was added to rs-convert to insure that al the png files are being extracted from the BAG files I am currently working with. Unable to find rs-convert.exe prior to SDK 2.47 in the SDK librealsense download. I will continue to search across our other company computers for the older version.
Thank you for your assistance in getting an earlier version of rs-convert.exe.

SDK 2.45.0 has an earlier version of rs-convert.  I have uploaded the exe to a Google Drive download link.
https://drive.google.com/file/d/1kn5gSLXsB2HnV6G2klkBfZRZ0jOUCbGs/view?usp=sharing

Hi @jimmek  Did using the 2.45.0 version of rs-convert provided in the download link above resolve your problem, please?  Thanks!

Hello MartyG,
I did some extensive testing of rs-convert on different sized BAG files.
For Viewer BAG files less than 4GB, rs-convert (all SDK versions) reported consistent png color and depth images, plus depth matrices (.csv) and imu-pose.
For Viewer BAG files greater than 4GB, (6, 18, 68, 64, 64, 62, 65) rs-convert (with imu-pose, two versions) reported inconsistent depth matrices (.csv) relative to png depth images, and truncated imu-pose files sometimes with no records/frames.  Rs-convert 2.45.0 version (prior to imu-pose inclusion) had no problems with png image or depth matrix (csv) files.
BAG files are good, but processing of the BAG file by rs-convert with imu-pose code included results in little or no valid imu-pose data and interferes with depth matrix extraction on BAG files bigger that 4 GB.

I will discuss with my Intel RealSense colleagues this issue with truncated / missing data with large bag files after 2.45.0.

I created my own test bags in sizes of 5 gb and 31 gb and ran extraction tests with rs-convert but was not able to replicate the effects that you described.
I note in https://github.com/IntelRealSense/librealsense/issues/10934#issuecomment-1281453831 that you are apparently recording depth, color and IMU data into the bag simultaneously using the RealSense Viewer.  Is that correct please?
There is a known issue with the Viewer tool for some RealSense users (not all) where enabling three stream types simultaneously (such as depth, RGB and Motion Module) can cause a No Frames Received error in one of those three streams where frames stop arriving for that particular stream.
It does not matter which order the three streams are enabled in, as enabling Motion Module is always the trigger for the problem.  There is no known fix for this problem in the Viewer for RealSense users who experience it other than to only enable two streams simultaneously instead of three.
If you are recording depth, color and Motion Module simultaneously in the Viewer, is there any point during recording when the message No Frames Received appears in red text in the center of the panel?  If so then this could explain why you have missing frames.  Would it be possible to record a large bag with only Depth and Motion Module enabled if you have not done so already in order to test whether the same problems occur with that two-stream bag?

Hello MartyG,
We do collect all three streams for two D455 cameras with Viewer.
We check DAQ time consistency for imu-pose, RGB, and Depth output by dividing each of these output record/file counts by their input FPS to compute data acquisition time for each. This provides a method to confirm that the number of records/files found by rs-convert for checking if the number of imu-pose gyro and accel records are consistent with images. For BAG files less than 4 GB, data acquisition time for all streams is consistent. For BAG files greater than 4GB, this is where problems occur. This would indicate that rs-convert is the problem.
When running rs-convert on a large BAG file, I was monitoring (via Windows TaskManager Performance monitor on the rs-convert program) how much memory the program was using and also watch the imu-pose output file size in Windows Explorer. Memory used changed rapidly up and down and the imu-pose output file appeared to be opening, writing, and closing repeatedly as file size repeatedly changed from 0 to a value and back again.
Imu-pose output has all the gyro data followed by the Accel data. If the gyro and accel data is written to separate files in sequence as they are encountered, this would eliminate the repeated open, write and close and the program needing to reorganize the data for the combined data imu-pose file. It would also simplify memory management.
In regard to your question: "If you are recording depth, color and Motion Module simultaneously in the Viewer, is there any point during recording when the message No Frames Received appears in red text in the center of the panel?"    Answer:  I do not know as I am normally not monitoring that screen (out of view) as I am monitoring another screen with two other acquisition programs running. I will perform a test where I can monitor the screen for the error message and will let you know the results.
In regard to your question: "Would it be possible to record a large bag with only Depth and Motion Module enabled if you have not done so already in order to test whether the same problems occur with that two-stream bag?"  I will test this with the same computer and let you know the results.
Regards,
James Mekemson

I ran tests with test bags of 5 gb and 31 gb size and was not able to replicate the described effects myself on my own computers.
I look forward to your next set of test results.   Good luck!

Collected 2 cases of on Depth and Motion data.  Ran a 3B and a 6.8GB BAG files.  Imu-pose file had no useful output for the 3GB file. For the 6.8GB file, had good depth data and gyro data, but Accel data was abut 2700 records short.

Thank you for the new test results.  I have responded to your email.

I suggest a possible easy fix to rs-convert problem where imu-pose (motion) data has been collected along with color and depth.
It has been observed that rs-convert processes imu-pose data as follows:

Read some gyro and accel records from BAG file
Gather/Organize step 1 data for imu-pose file. First gyro data then accel data.
Open imu-pose file.
Write imu-pose csv file
Close imu-pose file

Repeat 1-5 until all of BAG file read.
Opening and closing the same named file requires OS overhead and HD/SSD response time.  If program open is attempted before the OS has closed the file in the previous loop, the two actions may conflict. I have seen this conflict in command line scripts with slow external hard disks. This type of conflict may be occurring in rs-convert as cpu's and memory has gotten faster.
How to fix. Two Methods.
Method 1. Rewrite rs-convert to write gyro and accel to separate files. "One" open and close for "each" file. Steps 3-5 above is simplified. Do not know what in steps 1-2 would need to be modified.
Method 2. Use REWIND.  A faster code change that would require very little existing code modifications.

Read some gyro and accel records from BAG file
Gather/Organize step 1 data for imu-pose file. First gyro data then accel data.
Open imu-pose file ONLY FOR THE FIRST WRITE ATTEMPT OF GYRO AND ACCEL DATA.
Write imu-pose csv file
REWIND csv file

Repeat 1-5 until all of BAG file read. Here, step 3 is executed only once. REWIND is just a simple change of file pointer. Eliminating repeated open and close will greatly reduce OS overhead.
Maybe Method 2 is an easy quick fix to solve rs-convert imu-pose issue.

Thanks so much @jimmek for your detailed suggestions for improvements.  If yourself or another RealSense user would like to make these changes to the code and submit them for inclusion in rs-convert as a Pull Request (PR) then the contribution would be welcomed.
https://github.com/IntelRealSense/librealsense/pulls

Hi @jimmek  Bearing in mind my comment above, do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Update database info in readme

In the current readme mariadb is listed as the database.
Should be changed to postgres.

fixed in #107


Integrating CUDA properly in OpenQuantumBase

To get CUDA to work in DiffEq solvers of OpenQuantumTools with minimal changes, I had to add support for CuArray type in OpenQuantumBase. In particular, the initial state u0 of A::AbstractHamiltonian(H, u0) is now allowed to be CuArray.
https://github.com/USCqserver/OpenQuantumBase.jl/blob/e6778bc881f1f93fc5e22c87504fddbc7f10ede6/src/OpenQuantumBase.jl#L11
https://github.com/USCqserver/OpenQuantumBase.jl/blob/e6778bc881f1f93fc5e22c87504fddbc7f10ede6/src/OpenQuantumBase.jl#L48
This brings up several issues. The two most important are
(I) How should we integrate CUDA with OpenQuantumBase.jl?
(II) Is there a way to make CUDA an optional dependence?
My proposed solution to (I) is to make CuHamiltonian and CuAnnealing constructors which inherit from the Abstract versions. When passed to a solver in OpenQuantumTools, we just use multiple-dispatch on "Cu" types to do GPU accelerated solvers.
Pros:
(1) CuHamiltonian/ CuAnnealing data can be optimized for GPU (i.e. Float32 and whatever else is necessary)
(2) Solvers will have GPU support via multiple-dispatch (no additional arguments/ "seperate gpu solvers")
(3) If partially solved in one GPU run, final state uf is CuArray, so supports future runs as u0 "natively"
Cons:
(1) Users have to define a separate CuH/CuA types if they want to run on GPU.
(2) CUDA is now native dependence.

I agree with your proposal. Here are some of my thoughts:
(1) I think we should still keep CuDenseHamiltonian and CuSparseHamiltonian to distinguish CuArray and CuSparseArray.
(2) CuA and Annealing could share the same constructor, so the user only needs to define a CuH type to use GPU. Because Hamiltonian is necessary for every simulation, using it as a flag for CPG/GPU seems a reasonable choice.
(3) In the future, if the GPU library grows, we could separate it into another component package. Right now, loading CUDA seems to be a reasonable choice. The drawbacks of making CUDA a dependent package are: 1. We need to load CUDA each time OpenQuantumTools is loaded. It will consume more memory, but I don't think it will have a huge impact because of the JIT compiler. 2. There is an additional complication of the conditional usage, meaning we may need to manually check if CUDA is functional or not each time OpenQuantumTools is launched. I am not sure how this will impact OpenQuantumTools. And I will open a separate issue for this topic.

CUDA conditional usage: #45.


fix(interfaces): update companyId param

Update companyId param to be string
Fix tests to reflect changes
Update README.md

:tada: This PR is included in version 1.0.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


moq-relay: init at v0.6.0

Description of changes
Add the moq-relay rust project to run a Media over QUIC relay server.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This should be closed, considering you already added the package in #342442 (the first commit)


Add Notify MFA module

Description:
Add a multi-factor authentication module that using notify service to delivery one-time password to user during the login process
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#6131
Dev document: home-assistant/developers.home-assistant#80
Example entry for configuration.yaml (if applicable):
homeassistant:
  auth_mfa_modules:
     - type: notify

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Almost done. Need add translation, tweak some wording, and do more tests on real notify service

Change to increase counter before generate code, allow user try 3 times before login flow about.

I resolved several easy-fix code review comment. @balloob are you okay we merge in current state, and leave the mfa re-structure in a separate PR, e.g. create a separated MFA flow


Please use subdomains for wildcard dns entries instead of path substitution

Is there an existing issue for this?

[X] I have searched the existing issues

What did you expect to happen?
Customer could request a simple firewall change to allow all possible hosts behind registry.k8s.io
Debugging Information
Customer in a restricted environment needed to pull csi-provisioner from registry.k8s.io.  Was unable to get docker or podman installed in a non-secure location to download and pass the image bundles in.
Worked with firewall team to open up registry.k8s.io, only to find we also needed to add something-central-docker.pkg.dev and firewall team said they cannot do a partial hostname match in their rules preventing the request of *-docker.pkg.dev.  It was days later when we found out that the request was again forwarded to an AWS bucket.
Anything else?
I know we as a community like to stay modern and help guide customers to modern approaches, but having multiple redirects without a way to resolve (or use a simple match) to allow access to the artifacts seems a bit arrogant.
Either publish a complete list of redirect sites or change to subdomains.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please read our README

I know we as a community like to stay modern and help guide customers to modern approaches, but having multiple redirects without a way to resolve (or use a simple match) to allow access to the artifacts seems a bit arrogant.

.. this is a free service operated by volunteers, running on donated vendor resources.
We don't currently maintain a list of backends and they are subject to change at anytime to prioritize keeping them online at anytime. There's a detailed description of this at the top of the readme, which is the landing page for https://registry.k8s.io
There are many alternatives for your paying client.
We have found in the past that users will not switch when we ask them politely to migrate to a new more cost effective host, so we're not backing ourselves into any corners this time. It's a free service.

The reason we do redirects to multiple backends is also outlined in detail in the docs.

Either publish a complete list of redirect sites or change to subdomains.

Sorry no.
/close


Sub-String

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #4360

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[x] DSA
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@nimisha-yadav kindly review and merge my pull request.✨

@nimisha-yadav done


Cannot find module '/app/.output/server/index.mjs (start)

Describe the bug
When deploying with Coolify i get the following error:
[2024-Aug-28 05:23:48.265808] Attempt 1 of 10 | Healthcheck status: "unhealthy"
[2024-Aug-28 05:23:48.268764] ----------------------------------------
[2024-Aug-28 05:23:48.271215] Container logs:
[2024-Aug-28 05:23:48.365974] ><EMAIL_ADDRESS>start

vinxi start

<EMAIL_ADDRESS>start
vinxi start

[2024-Aug-28 05:23:48.369933] at defaultResolve (node:internal/modules/esm/resolve:1193:11)
at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:404:12)
at ModuleLoader.resolve (node:internal/modules/esm/loader:373:25)
at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:250:38)
at ModuleLoader.import (node:internal/modules/esm/loader:336:34)
at importModuleDynamically (node:internal/modules/esm/translators:143:35)
at importModuleDynamicallyCallback (node:internal/modules/esm/utils:176:14)
ERROR Cannot find module '/app/.output/server/index.mjs' imported from /app/node_modules/vinxi/bin/cli.mjs
Your Example Website or App
https://stackblitz.com/github/tanstack/router/tree/main/examples/react/start-basic?embed=1&theme=light&preset=node&file=src/main.tsx
Steps to Reproduce the Bug or Issue
npm run start
Expected behavior
should deploy successfully
Screenshots or Videos
No response
Platform

OS: [Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Concerns about failed migrations being marked as applied

First, thank you for this package and specifically the migration logic (from Bun). That said, I have some concerns about failed migrations being marked as applied. I understand that this allows the rollback logic to be run, but I'm not sure if the desirability of that outweighs the risks.
The major risk I'm worried about is another "up" being performed before the "rollback" and the failed migration being skipped. CI/CD pipelines and developers not aware of this behavior may try to apply the migrations again, like in the case of a timeout or transient error, and be surprised by the behavior. I would argue it violates the principle of least surprise and that the defacto standard is for a following "up" to try applying the failed migration again, not skipping it.

It is pretty easy to create an option that will mark migration as applied after the migration completed successfully - see https://github.com/uptrace/go-clickhouse/blob/master/chmigrate/migrator.go#L152-L155 . Please send a PR if you are interested.

On it.

How do you want tests to be written? I don't see a mocking tool in the project, which I would need to make assertions that Migrator isn't trying to write to the ch_migrations table. Any objection to adding gomock?

No objections, but for a such simple change I won't ask for a test.


Added new command line arguments

Closes #41
I think adding a new flag as mentioned here could lead to some issues with pvi... We could end up with bobfiles the screen-dir that unintuitively  aren't actually matched to the PVs of the IOC currently running:
Running
PandABlocks-ioc softioc <IP_ADDRESS> TEST-PREFIX --screens-dir=screens --skip-bobfile-en-if-dir-not-empty
in the case where screens isn't empty and the panda has changed since the .bob have been created will lead to a gui that isn't necessarily up to date with the pvi of the ioc.
I think it would be cleaner to have --overwrite-files, but it would depend on if there's really a use case for this.

It's a good question whether --overwrite-files or --skip-bobfile-gen-if-dir-not-empty is a better option. The original behavior of the IOC before the latest changes was to always overwrite files. This may still leave screens with invalid PVs if the new uploaded app has less blocks of a certain type.

@AlexanderWells-diamond  Thoughts on this?

I think overwrite files is nice, I'll bring it back

I added the flag --clear-bobfiles, this will remove all bobfiles from the specified screens-dir. An error will  be thrown if it's passed in without a --screens-dir too.
I also did all the work for https://github.com/PandABlocks/PandABlocks-ioc/issues/43 since I was having problems with softioc during the tests.


Option to show only what's in appnames.txt is missing in the latest release (works in the prior one, 0.4)

As per subject, that option is missing now.

Thanks for looking into this

I removed this option when I added support for custom tabs: You can turn off auto-tabs and define custom tabs in appnames.txt, effectively achieving the same thing. Have you tried that?

Hello, yup I figured. I'm doing that, but the issue is there's still an
"All" tab that shows everything. Just being picky to be honest. Thanks for
the hard work on this!
On Thu, 29 Aug 2019, 12:48 tverona1<EMAIL_ADDRESS>wrote:

I removed this option when I added support for custom tabs: You can turn
off auto-tabs and define custom tabs in appnames.txt, effectively achieving
the same thing. Have you tried that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tverona1/QuestAppLauncher/issues/13?email_source=notifications&email_token=AABFWY2WDLEY73HDXJWIZELQG5IJDA5CNFSM4IKB6MR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5NHABA#issuecomment-526020612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABFWYY37C6RHW6SPZ2AAGTQG5IJDANCNFSM4IKB6MRQ
.

Ok :). I"m going to close this issue then given the above works for you.


python3Packages.markdown: 3.3.6 -> 3.3.7

Description of changes
https://github.com/Python-Markdown/markdown/blob/3.3.7/docs/change_log/index.md

Things done

Built on platform(s)

[x] 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.05 Release Notes (or backporting 21.11 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

[x] Fits CONTRIBUTING.md.

@ofborg build python3Packages.markdown


[CSBindings] Open collection binding associated with @autoclosure arg…

…ument
To preserve subtype relationship between element types of a collection
passed as an argument to a parameter represented by another collection
type let's extend opening of the collection types to be done for arguments
to @autoclosure parameters as well because implicit closure is transparent
to the caller.
Resolves: SR-13135
Resolves: rdar://problem/65088975

@swift-ci please smoke test


NAN appears when doing time series problems.（做时间序列问题时候出现NAN）

Dataset Create

I test the effect of KAN through artificial data, and the process guarantees that NAN will not appear.
Training

when I first train,this is look fine
Problem
Then I use the "prune" function,and train again,the loss start begin NAN

I adjusted the learning rate of all the fit functions above to 0.001, but the training after pruning is still nan.
How can I solve this problem

I saw that other people's methods include adjusting "updata_grid=False". My training after pruning was successful, but when I used the "refine" function to continue training, it was still NAN.

PS.3
I adjusted my network structure

That is, to train on a smaller-scale network. At this time, all the training is normal, and I don't need to use "updata_grid=False" anymore.
This question makes me a little puzzled.

same question when I using deeper KAN

make your grid range bigger can solve it. dense grid lead to more oscillation, and your gradient can be explored....large grid range can reduce this oscillation


Figure out HTTP AUTH stuff for connecting with other groups

from the forum:

As a server admin, node to node connections can be authenticated with HTTP Basic Auth Q: When doing node to node connections I was wondering if when you request access from a server do you sign up with a username and password and that will be used for HTTP basic auth?

Answer:
It's manual, you ask the people. So you give your collaborating group an account. If you want to make it automatic, go for it, but I didn't ask for it.

Document your service address, port, hostname, prefix(if used), and the username/password for HTTP Basic Auth(if used) in your README so that HTTP clients can connect to your API.

Answer:
Provide a default just to enable collaboration but you're going to have make accounts for nodes individually as well. Having a test account allows people to try stuff.
The suggestions below are suggestions, but you should allow for PREFIXES if possible.
When building your API, try to adhere to these rules for easy compatibility with other groups:
REST API calls may be prefixed. ie. http://service_address/api/author/{AUTHOR_ID}/posts/
Document your service address, port, hostname, prefix(if used), and the username/password for HTTP Basic Auth(if used) in your README so that HTTP clients can connect to your API.

As a server admin, I can limit nodes connecting to me via authentication.

Answer:
This means that you can stop hosts from connecting to you simply by not granting them authentication credentials or revoking them.
It just means you should have authentication between hosts.

I think other people are having trouble posting comments on our api... kyle was. even when logged in it wouldn't let him at api/posts/post_id/comments/

I switched to my heroku to fix this part


Implement filter plugin

requires https://github.com/Uninett/argus-htmx-frontend/pull/42/ to be merged
closes #5

merged through #19


FTP Upload….

I have the time lapse, keograms and startrails working on the rpi however no matter what I do I can’t get ANYTHING to upload to my new website….
I have changed all the ftp settings to the required details but something still must be wrong….
Any help is appreciated please…..

I also just noticed the Aurora part along the top doesn’t show either

@Shand359, try the attached command.  Move it to the "allsky/scripts" folder and make it executable, then run it with no arguments and it will display a usage message.  Then run with the arguments.  Try it with a dummy file.  It will produce a log file you can look at to see what's happening.
A couple common causes for ftp problems:

the FTP server doesn't have the necessary directories.  If you want the keogram to go into "/keogram" on the FTP server, that directory has to already exist.
SSL Certificates are missing or have the wrong server name or something else.  To allow lftp to work with unknown certificates, add:   set ssl:check-hostname false  to /etc/lftp.conf

Let me know how it goes.
runLftp.zip

@Shand359 you can ignore the runLftp.zip file.  Instead, download the latest files.  It includes the "upload.sh" command that you can try executing.  If it fails it will display some lines that should help debug the problem.


changes regex to allow additional customized saved object ids

Signed-off-by: Zhongnan Su<EMAIL_ADDRESS>Description
This code logic is contributed by @inf17101, in this #289. Since we need this fix to go out the upcoming release, and we decided to take the logic and commit,  instead of waiting for the author to reply
Better would be /(?:[0-9a-z]+-[0-9a-z]+)+?/ as regex expression because of non-capturing group.
Or to allow only a certain range of characters: /(?:/)([0-9a-z-]{1,100})?/
Issues Resolved
#281
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks for the fix- I'll open a branch to investigate the link checker errors further and see what alternate links we can include if no other solution is found

Thanks for the fix- I'll open a branch to investigate the link checker errors further and see what alternate links we can include if no other solution is found

Thanks David

Codecov Report

Merging #295 (8902e89) into main (d444010) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #295   +/-   ##
=========================================
  Coverage     72.75%   72.75%
  Complexity      369      369
=========================================
  Files            94       94
  Lines          3928     3928
  Branches        609      609
=========================================
  Hits           2858     2858
  Misses          894      894
  Partials        176      176

Flag
Coverage Δ

dashboards-reports
77.21% <ø> (ø)

reports-scheduler
68.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Enlarge the config

We found a CSAF file that is ~100MB: https://access.redhat.com/security/data/csaf/v2/advisories/2018/rhsa-2018_3140.json
We will enlarge our restriction.

Postponed - investigation needed as the file is incomplete.


Update LICENSE

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[ ] HISTORY.rst has been updated (with summary of main changes)

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

Adds names and dates to our license

Does this PR introduce a breaking change?
Other information:

Coverage: 92.483%. Remained the same when pulling 181a72fa76ff8e985f79b122f79fb48388721e04 on update-license into c16cb06f4d42b9e2aaf03864a705811715936d46 on master.


:bug: logicalcluster-deletion: skip bound resources

Skip deleting bound resources in the logicalcluster deletor as this can
race with the apibinding deletion controller. If the apibinding deletion
controller deletes an APIBinding and all its instances and then the
logicalcluster deletor tries to delete the collection of the same
resources (because it saw the resource in discovery), it will get a 404
and go into rate limited backoff, which we don't want.
Taken from @ncdc's rebase PR https://github.com/kcp-dev/kcp/pull/2945.

/retest

Closing & re-opening this PR to migrate it to the new Prow. Please do not be alarmed.

/retest all

@sttts is this still valid to merge? Looks green'ish

/test all

Yes. I just extracted it at the time because we didn't know where some flakes came from.

/lgtm
/approve