Feature/attachment upload athene

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[ ] Server: I added multiple integration tests (Spring) related to the features
[ ] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[ ] Server: I implemented the changes with a good performance and prevented too many database calls
[ ] Server: I documented the Java code using JavaDoc style.
[ ] Client: I added multiple integration tests (Jest) related to the features
[ ] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[ ] Client: I documented the TypeScript code using JSDoc style.
[ ] Client: I added multiple screenshots/screencasts of my UI changes
[ ] Client: I translated all the newly inserted strings into German and English

Motivation and Context

Description

Steps for Testing

Log in to Artemis
Navigate to Course Administration
...

Screenshots

Lecture attachments are now uploaded to Athene to be used as training material for the ELMo model of the respective course.

I think we should first discuss if uploading lecture slides to other systems is ok for instructors. As we now have more courses from other departments, they might not be happy if we upload and process their files without permission. Also, not all the attachments in a lecture include slides.
So we should have at least a checkbox in the attachment upload that the instructors needs to tick to approve the upload. Only then, the PDF should be uploaded to other systems

I think we should first discuss if uploading lecture slides to other systems is ok for instructors. As we now have more courses from other departments, they might not be happy if we upload and process their files without permission. Also, not all the attachments in a lecture include slides.
So we should have at least a checkbox in the attachment upload that the instructors needs to tick to approve the upload. Only then, the PDF should be uploaded to other systems

@krusche
"I think we should first discuss if uploading lecture slides to other systems is ok for instructors"
Yes, I discussed this with @jpbernius and we thought that Athene is somehow also a part of the Artemis system.
"So we should have at least a checkbox in the attachment upload that the instructors needs to tick to approve the upload".
yes that's definitely something I should add.
"Also, not all the attachments in a lecture include slides"
For now we can extract training data from all pdfs and text files, But I will for now add a condition to only upload pdf files.

I guess if we have a checkbox and you only upload files if

The instructor has ticked the checkbox
The file is a PDF file

this would be fine!

The instructor has ticked the checkbox

Would we define this (a) once for the course or (b) for every attachment?

The instructor has ticked the checkbox

Would we define this (a) once for the course or (b) for every attachment?

I would say for each attachment

The instructor has ticked the checkbox

Would we define this (a) once for the course or (b) for every attachment?

Yes, I would also say for each attachment, as this enables the instructor to choose not to add attachments that are somehow out of the general context of the lecture (I'm thinking about attachments like event advertisements, Ferienakademie flyers, .... 😛   )

I'm not sure if it happens in reality but I managed to get:

Exception in de.tum.in.www1.artemis.service.connectors.EmbeddingTrainingMaterialService.uploadAttachment() with cause = 'NULL' and exception = '404 Not Found: [no body]'
by not building the right branch in Athene.
Exception in de.tum.in.www1.artemis.service.connectors.EmbeddingTrainingMaterialService.uploadAttachment() with cause = 'NULL' and exception = '500 Internal Server Error:
by giving wrong credentials to cloud.

The problem is I didn't get any message that my attachment is not saved to Athene.
And deleting an attachment does not delete it from the cloud I suppose. (idk if it is intended)
Except from these, it worked as it is described :)
Edit: Maybe you can also say in your checkbox text that it only works for pdf files because I thought I could send a png too.

The first exception is expected if you don't run the Athene component.
Second exception should is dealt with in Athene now.
" I didn't get any message that my attachment is not saved to Athene."
you get  message by successful upload and the trace of the error otherwise.
deleting an attachment does not delete it from the cloud I suppose
@jpbernius : what do you think ? should we implement delete requests as well ?

should we implement delete requests as well ?

I would consider this for a flow up but keep it as is for now

I'm not sure if it happens in reality but I managed to get:

Exception in de.tum.in.www1.artemis.service.connectors.EmbeddingTrainingMaterialService.uploadAttachment() with cause = 'NULL' and exception = '404 Not Found: [no body]'
by not building the right branch in Athene.
Exception in de.tum.in.www1.artemis.service.connectors.EmbeddingTrainingMaterialService.uploadAttachment() with cause = 'NULL' and exception = '500 Internal Server Error:
by giving wrong credentials to cloud.

The problem is I didn't get any message that my attachment is not saved to Athene.
And deleting an attachment does not delete it from the cloud I suppose. (idk if it is intended)
Except from these, it worked as it is described :)
Edit: Maybe you can also say in your checkbox text that it only works for pdf files because I thought I could send a png too.

The first exception is expected if you don't run the Athene component.
Second exception should is dealt with in Athene now.
" I didn't get any message that my attachment is not saved to Athene."
you get message by successful upload and the trace of the error otherwise.
deleting an attachment does not delete it from the cloud I suppose
@jpbernius : what do you think ? should we implement delete requests as well ?
Maybe you can also say in your checkbox text that it only works for pdf files because I thought I could send a png too.
I will do that !
Thanks for the feedback !

"Second exception should is dealt with in Athene now."
Yes, I saw that 👌
"I will do that !"
It looks good, I really liked it 👍
"you get message by successful upload and the trace of the error otherwise."
I meant in the user interface for the user but maybe it is not in the scope of this PR (could be good to have in the future in terms of UX)

Works as expected for me. Just experienced an issue when deleting an attachment, that the dialog stays open even if you already hit the Delete-Button (see Screenshot):

I wasn't able to deploy this in test server 1 or 3 using the following release: https://bamboo.ase.in.tum.de/deploy/viewDeploymentVersion.action?versionId=277152368
After deployment on both servers, I'm not able to access the URLs.
Deploying the "develop" branch on top fixes the issue after some time.
Are we supposed to deploy on local only?
Example message on test server 3 after deploying. Neither server is accessible even after 10+ minutes.

I wasn't able to deploy this in test server 1 or 3 using the following release: https://bamboo.ase.in.tum.de/deploy/viewDeploymentVersion.action?versionId=277152368
After deployment on both servers, I'm not able to access the URLs.
Deploying the "develop" branch on top fixes the issue after some time.
Are we supposed to deploy on local only?
Example message on test server 3 after deploying. Neither server is accessible even after 10+ minutes.

Thanks for testing,
It's because we need to deploy an up-to-date version of Athene + update config files in test servers

Closing this for now. The PR is totally outdated. Please first finish the PR properly and then reopen it or even create a new one.


Add git commit sha to output of --version

TODO:

[ ] output is using crate_name!() now yielding alacritty_terminal not alacritty

If want, you could also use this crate (it's already used in rls and clippy):
https://crates.io/crates/rustc_tools_util

@matthiaskrgr cool, I've updated this to use that crate. Thanks.
I'm pretty happy with this overall atm. One crazy idea I had though, would be to check for local changes to the repo and print a alacritty 0.3.2 (531976a*)  (notice the *) when detected. Worth added complexity? I'm happy to keep this simple, but just thought it might help detect weird patches and other discrepancies.


Implement Methode AuthenticateClient as in Documentation

AuthenticateClient(string user, string password, Uri baseAddress, IEnumerable scopes)

Here you go - at least the one Methode that actually IS Documeted should be available no?

Hello @AisoPlee ,
I have merged your pull requests in the solution and made an update to the integration tests, they should now pass with your changes.
Regards,
Alexandru-Constantin Pascariu


PDS OWASP ZAP AjaxSpider No Implementor found

Problem
Starting the AjaxSpider inside the current PDS OWASP ZAP results in an ClientApiException, with the reason No Implementor. The reason for this kind of error message means, that the ajax spider extension could not be accessed via the API. Some modules were not installed correctly inside the Image.
Solution
After some research I noticed, that some dependencies for the OWASP ZAP AjaxSpider Extension are prompted to be installed using the desktop version. The installation used inside the PDS OWASP ZAP missed installing these dependencies. So we need to install these dependencies manually.

After analyzing the error it occurs, that firefox needs to be installed using snap install firefox. We don't want to use snap inside the docker container. Therefore we change the base image as mentioned here: #1421


[Suggestion] Add ability to enter location names

We should be able to enter the name of a location (e.g. Portland, Oregon, USA or a specific address such as  2 Bryant St #220, San Francisco, California, USA) instead of hassling around with coordinates. There are a ton of APIs (including a Google Maps one IIRC) to convert these location strings into coordinates if necessary.

Repeat of issue #41

@fenixasin Nope, this is entering location strings. #41 is remembering saved longitude/latitude coordinates to use again in the future.


Add myself as maintainer on git-instafix

Description of changes
I write git-instafix and I want to be added to the reviewers list when it gets updated in nixpkgs.
Things done
No changes or testing except adding myself as a maintainer.

Add yourself to this file:
https://github.com/NixOS/nixpkgs/blob/master/maintainers/maintainer-list.nix
It should be a different commit. And it should come first. This order:

maintainers: add quodlibetor

git-instafix: add quodlibetor as maintainer

The order is necessary because second commit depends on first. Or else, eval fails!

thanks, lmk if it's not quite right.


Month Scrape Limit

It seems tweet scraping is limited to a month back from the specified date, even when using the following commands:
python twint.py -u BBC --since 2006-03-21 -o BBC.csv --csv
python twint.py -u BBC --since 2006-03-21 --until 2017-06-01 -o BBC.csv --csv
This might be an issue, or am I overlooking an option?
I am running Twint on Windows 10, with Python 3.6.4 (Anaconda 64 bit).

I have the same problem with Twint while the previous Tweep release seems to work with older posts

Hi all and thanks for reporting,
in the while _since < _until loop, after else remove break . This should fix the problem.
I firstly added the feature as this, but maybe this breaks the code somehow with the newest features (since a break was added). I'm going to fix this.

I have a quick fix, will update in a few mins

Fixed in this commit
Opted for a seperate loop

Thanks for the fix! This is a nifty piece of software

Thank you :)


Guarding against null values before unserializing

Question
Answer

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
yes

Tests pass?
yes

Fixed tickets

License
MIT

Doc PR

Description
Guarding against null values before calling unserialize. This addresses E_DEPRECATED notices in PHP 8.2 when unserializing as the function signature now only accepts string values.
TODO

[ ] Add tests
[ ] Add documentation
[ ] Updated Changelog.md

🙏


Improper generation of ctypes bindings for Lib.IntVector.Intrinsics.vec256

When generating ctypes bindings with krml on the below file:
module KremlinVector

open FStar.HyperStack.ST
open LowStar.Buffer

#set-options "--z3rlimit 25 --ifuel 0 --fuel 0"

let t2 = UInt32.t & Lib.IntVector.Intrinsics.vec256
let f2 (x : t2) : t2 = x

We get the following .ml:
open Ctypes
module Bindings(F:Cstubs.FOREIGN) =
  struct
    open F
    let kremlinVector_f2 =
      foreign "KremlinVector_f2"
        (kremlinVector_t2 @-> (returning kremlinVector_t2))
  end

Note that kremlinVector_f2 references the undefined kremlinVector_t2, preventing the file from compiling

Hey @victor-dumitrescu, would you be able to take a look at this? I think it's relatively simple to fix, when you encounter a struct type that contains forbidden types, just remember the name of the struct, then:
https://github.com/FStarLang/kremlin/blob/master/src/GenCtypes.ml#L85
when you see one of those struct types later on (the Qualified case), then you can also return false.

No problem, I'll submit a fix soon.

I think this should fix it but let me know if you think I'm missing any potential cases.
For the code above it should now still work but throw these warnings:
Warning 22: : Dropping declaration from KremlinVector bindings because it uses unsupported type Lib.IntVector.Intrinsics.vec256
Warning 22: : Dropping declaration from KremlinVector bindings because it uses unsupported type KremlinVector
.t2

and neither t2 not f2 should be bound.


For 2D and 3D transforms it is not possible to supply all temporary buffers

It would be great to allow the user to supply all needed temporary buffers for 2D and 3D transforms. Currently internal 1D transforms allocate their only temporary buffers. This is a problem when baking using large number of plans, because these temporary buffers are not deallocated until the plan is destroyed.
For now my workaround is to decompose 2D and 3D FFTs into 1D FFTs myself, but this way I do not profit from optimized transpose kernels...

Yes, this is a known issue. It is on the list of items to be improved. Unfortunately, I am not in a position to say when this would get worked on.

I am just running into this issue again. There could be different solutions to this problem:

Allow the user to force deallocation of a plans internal buffers (Does not change current API, but allocation of temporary buffers still can not be done by the user)

Extend the API to allow passing multiple temporary buffers.

Allow the user to pass a pointer to a callback function delivering temporary buffers.

Ok, let me reopen. But not sure when we can get to this one. Also, does releasing plans after its use not an option for you?

Depending on how long recreating plans takes, releasing plans is an option. Will recreating plans be faster than the first time or is the code compiled again?


Updated for QnA Maker

Updated for QnA Maker Fairfax GA (Scheduled for 10/01)

@prabansa : Thanks for your contribution! The author(s) have been notified to review your proposed change.


serde support for prost-types

Hi!
I'm trying to support serde-based serialization on some Protobuf types I have generated using prost. I do this conditionally by putting the following in the build script for my crate.
fn main() {
    let protos =
    prost_build::Config::new()
        .type_attribute(".", "#[cfg_attr(feature = \"serde\", derive(serde::Serialize, serde::Deserialize))]");
        .compile_protos(&[/* list of files */], &[/* list of includes */])
        .unwrap();
}

This works great until I use any of the well-known types. The problem, of course, is that the well-known types have already been generated and exist in prost-types. The workaround is to use compile_well_known_types(), like so.
syntax = "proto3";

import "google/protobuf/timestamp.proto";

message LogWithTime {
    string log = 1;
    google.protobuf.Timestamp timestamp = 2;
}

fn main() {
    let protos =
    prost_build::Config::new()
        .type_attribute(".", "#[cfg_attr(feature = \"serde\", derive(serde::Serialize, serde::Deserialize))]");
        .compile_well_known_types() // <---- added this line!
        .compile_protos(&[/* list of files */], &[/* list of includes */])
        .unwrap();
}

The problem now, however, is that the well-known types I generated are no longer compatible with any other crate's well-known types. So, it becomes difficult to do transformations between the types generated across crates.
My workaround is to generate the well-known types with the appropriate serde attributes in a new shared crate that can be used by all the other crates in my workspace. But, I was wondering: would it be possible to add a feature flag to prost-types that would allow one to turn on support for serde? Or perhaps the feature flag is better situated in prost-build? I'm not sure exactly where the best place to put it is, but generally having support for serde would be nice.

By the way, I did see there was some desire for this at some point, but figured it might be worth bringing up in a separate issue in case things have changed.

I'm very keen to add support for the protobuf JSON encoding to prost by having an option to output the right serde annotations, however I haven't found time to actually work on this. Once that's done, they could be enabled for prost-types through a cargo feature. I can't give a timeline on when this might land, though.

Any update on this? I cannot compile it to use Struct.


ExpatError: unbound prefix

This happens on the first PDF file I tried to extract. The file can be downloaded from here
The error is not printed on the console, but in

000.000.001/000.000.001.header.tei

<?xml version='1.0' encoding='UTF-8'?>
<error>ExpatError: unbound prefix: line 2, column 0</error>

It's likely the academic paper identifier failed. Which extractor were you using and does it work without the framework?

I am using PDFBox 2.0.1, the newest version. The text extraction result was
good. If the classifier fails, it should explicitly give an error about it.
It is hard to determine the classifier fails from the error I saw. Let me
try more documents today and see how it works.
Jian
On Tue, May 17, 2016 at 9:50 PM, huaiyudavid<EMAIL_ADDRESS>wrote:

It's likely the academic paper identifier failed. Which extractor were you
using and does it work without the framework?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/SeerLabs/pdfmef/issues/3#issuecomment-219904065


Cannot read property 'chart' of undefined when upgrading to 2.0.0

I'd upgraded from 1.4.0 to 2.0.0 and have the error:
Cannot read property 'chart' of undefined

I've returned to 1.4.0 and it works.
Happy to poke around some more, but I'm having difficulty recreating it as a standalone test. If someone can orient me, happy to dig around. The stack trace didn't help me. Here it is:
react-jsx-highcharts.min.js:1 Uncaught TypeError: Cannot read property 'chart' of undefined
    at n.value (react-jsx-highcharts.min.js:1)
    at finishClassComponent (react-dom.development.js:10249)
    at updateClassComponent (react-dom.development.js:10226)
    at beginWork (react-dom.development.js:10605)
    at performUnitOfWork (react-dom.development.js:12573)
    at workLoop (react-dom.development.js:12682)
    at HTMLUnknownElement.callCallback (react-dom.development.js:1299)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:1338)
    at invokeGuardedCallback (react-dom.development.js:1195)
    at performWork (react-dom.development.js:12800)
    at batchedUpdates (react-dom.development.js:13244)
    at performFiberBatchedUpdates (react-dom.development.js:1646)
    at stackBatchedUpdates (react-dom.development.js:1637)
    at batchedUpdates (react-dom.development.js:1651)
    at Object.batchedUpdatesWithControlledComponents [as batchedUpdates] (react-dom.development.js:1664)
    at dispatchEvent (react-dom.development.js:1874)
value @ react-jsx-highcharts.min.js:1
finishClassComponent @ react-dom.development.js:10249
updateClassComponent @ react-dom.development.js:10226
beginWork @ react-dom.development.js:10605
performUnitOfWork @ react-dom.development.js:12573
workLoop @ react-dom.development.js:12682
callCallback @ react-dom.development.js:1299
invokeGuardedCallbackDev @ react-dom.development.js:1338
invokeGuardedCallback @ react-dom.development.js:1195
performWork @ react-dom.development.js:12800
batchedUpdates @ react-dom.development.js:13244
performFiberBatchedUpdates @ react-dom.development.js:1646
stackBatchedUpdates @ react-dom.development.js:1637
batchedUpdates @ react-dom.development.js:1651
batchedUpdatesWithControlledComponents @ react-dom.development.js:1664
dispatchEvent @ react-dom.development.js:1874
proxyConsole.js:54 The above error occurred in the <n> component:
    in n (created by HighchartsProvider(n))
    in HighchartsProvider(n) (at ResultsChart.js:15)

Any chance you could share the chart component you're attempting to render? Would help me debug.
Are you using the new HOC withHighcharts in your component? It is required for version 2.
You can see an example of it's usage here

That's what it was---I wasn't aware of that change.
When I built a simple test case to try and isolate the issue, the chart worked, though---which is why I was struggling to locate it. Possibly caching then. But, regardless, the HoC seems to have been the issue.

No worries, I'll make it way more obvious in the readme. Was worried this might happen!

I looked at the list of notes and the top of the README and saw 1.2.0 and 1.3.0 listed, so I presumed there was no 2.0.0 info. So, even just adding 2.0.0 to that list and mentioning the HoC probably worthwhile.


[T4A5][W13-B1] Lang Yanbin

Ready for review

@langyanbin0314
Some comments added. Please ack & close the PR after reading comments.


JWT::JWK::EC needs require 'forwardable'

Using ruby 2.7.1 and trunk version of jwt (dc946fa), I get the following error when trying to verify a token:
lib/jwt/jwk/ec.rb:6:in `<class:EC>': uninitialized constant JWT::JWK::EC::Forwardable (NameError)

Adding require 'forwardable' to the top of ec.rb resolves this.

Thanks for the report.
There are some changes to the new algorithms in the making, this also removes the dependency to the forwardable module.
https://github.com/jwt/ruby-jwt/pull/388

I see. In this case I'll close this ticket. Thanks!


Enhancement/131 refactor metadata reading

This is a major refactoring of the way SQL dialects and metadata reading are handled.

Looks good from my side!


Retry/continue downloads if being uploaded concurrently to reflector

If you are downloading a file while it's being uploaded to reflector, it stops and fails. We used to have code that would re-check for new blobs and download.

I think something similar is happening here with our new satellite reflectors when they need to go all the way back to s3 to grab blobs for the first time. Either something is failing in that process, or the SDK is not handling the connection properly. Downloads will randomly just stop and have to be retried again - each time they go a bit further.
Need to debug with Niko to see if it's the latter part of the flow, but the SDK should handle this better regardless.


Question regarding Debian packaging and external dependencies

Debian usually wants to build a package using upstream copies of all third-party projects instead of using embedded copies.  The primary motivation for doing this is that it simplifies addressing security vulnerabilities, because the upstream project can be updated once and all the programs that incorporate them can be rebuilt automatically.  However, if there are compelling reasons not to do so, exceptions can be made.
From what I can tell, Feather wallet has three levels of external dependencies.

Polyseed and bc-ur in src/third-party.
Monero in monero.
Boost, liblmdb, easylogging++, miniupnp, qrcodegen, randomx, rapidjson, and supercop in monero/external.

I read in your feature comparison documentation that some changes are made to your Monero fork (item 2).  My questions are:

I assume it is OK to replace these files with system packages as the build code looks for a system copies before using the bundled ones.  But I wanted to check to be sure (I noticed you are using an older release of Polyseed).
Because you have already stated in your documentation that you are modifying the Monero code, I would assume that it would not be safe to build against the upstream Monero code.  Are your changes something you expect to eventually be incorporated upstream, or do you expect to maintain them indefinitely?
Have you made any modifications to monero/external?  Bastian, my fellow packager, maintains the Monero Debian package.  He has replaced the majority of the contents of monero/external with system packages without issue, but I wanted to make sure Feather Wallet isn't using them in a different way that could be problematic.

FYI, there is a fair amount of information about the Monero Debian package at:
https://tracker.debian.org/pkg/monero
Thanks for being willing to help us with the packaging.  I really appreciate it.

These are OK.
We apply dozens of non-trivial patches on top of Monero. Building against upstream will simply not work. While most patches get incorporated upstream eventually, we expect to maintain a significant patch-set indefinitely.

miniupnp, randomx, qrcodegen: these dependencies are not used in Feather. I will remove the submodules.
Boost headers and liblmdb, should not be unvendored. Doing so will break wallet file compatibility.
easylogging++ is heavily modified and should not be unvendored.
supercop and rapidjson can be unvendored.

What you need to know about packaging Monero-related software:
Nearly every Monero and Feather release contains security and privacy fixes. The Monero version in Debian stable (v<IP_ADDRESS>) is dangerously out of date and contains numerous vulnerabilities. Specific vulnerabilities are not always disclosed. You can infer if a release includes security fixes by looking at the "recommended version" in the Scheduled software upgrades table in the Monero README. In practice, network participants are expected to run the latest version at all times. Monero may hard-fork on months' notice, rendering all prior versions of the software useless, including wallet applications. If you decide to package Feather, I would wish for more active maintenance going forward. Failing to do so puts user funds and privacy at risk. If Debian release policy does not allow you to keep the software up-to-date, I strongly recommend against packaging Feather and to discontinue packaging Monero.

Looking more closely at Supercop, it doesn't look like it is a good candidate for a Debian package at the present moment because it currently requires part of the greater Monero project to build.  I will keep my eye on it and package it for Debian if that ever changes.

A note on undisclosed security issues: You cannot expect them to be fixed in Debian or any other downstream. The security tracker for the monero package does not show any issue affecting the software in any active release.

Yes, as a matter of good security practice, I would recommend that all known security bugs be disclosed.

A note on undisclosed security issues: You cannot expect them to be fixed in Debian or any other downstream. The security tracker for the monero package does not show any issue affecting the software in any active release.

Security Issues that affect users are usually communicated in a blog post or implied in the changelog. Users are advised to always run the latest version of the software. Issues that affect network stability (such as denial-of-service attacks) are typically only disclosed on a need-to-know basis.
If you can't package the latest version, then don't package the software at all. Monero is not stable software, so perhaps you should reconsider whether it belongs in a stable distribution.
Debian 11 was stable until 10 June 2023. It ships Monero v<IP_ADDRESS>, which is no longer able to connect to the Monero network as of 13 August 2022. For 10 months and until users upgraded to Debian 12, the stable package was rendered useless (i.e. daemon can't synchronize, CLI wallet can't send or receive transactions).
Why was the package never updated? And how will you prevent a repeat of this? We are aiming to hard-fork again next year. This will affect Feather, too. This upgrade will introduce Rust as a build dependency. You will need to package the upgrade in months' notice or the software stops working.

You are right, there should have been a stable upload in Debian 11. At that time I was not involved with packaging Monero. And usually, I run unstable systems, so will rely on users to report their (old)stable software not working. There is no such bug report, so I have not catered to it yet.


Move FR 02-23 beta to public repo

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-formrecognizer


PPTP-1459 - Increasing Obligations integration test coverage

Description of Work carried through
Adding integration test to validate obligationDetails in Obligation
Check list

[x] ./precheck was executed (Integration/Component/Unit tests)
[x] sbt scalafmt test:scalafmt was executed
[ ] Required Environment Config has been amended/added
[ ] Links to dependencies have been included (BE/FE work)
[ ] User Acceptance Tests (UAT) were run locally and have passed.

Merge blocked by earlier unverified commits - worked around with PR #52, closing this one...


Qt4 and Qt5 side-by-side installation breaks pyrpl?

Hi, sorry for the bugspam!  ;)  I don't mean for this to be my personal support forum but I suspect this may be an actual issue with some machines.
I have Pyqt4 and Pyqt5 installed and I believe this is messing up pyrpl.
I tried installing pyrpl from pip, and cloned the git respository, both with the same results: I get import errors for pyrpl:

import pyrpl
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-1-f256f7007682> in <module>()
----> 1 import pyrpl

/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/__init__.py in <module>()
     31
     32 # get QApplication instance
---> 33 from qtpy import QtCore, QtWidgets
     34 APP = QtWidgets.QApplication.instance()
     35 if APP is None:

/usr/lib64/python2.7/site-packages/qtpy/__init__.py in <module>()
    123         import sip
    124         try:
--> 125             sip.setapi('QString', 2)
    126             sip.setapi('QVariant', 2)
    127             sip.setapi('QDate', 2)

ValueError: API 'QString' has already been set to version 1

That's from ipython. From regular python2:

[paul@strontium ~]$ python test.py
Traceback (most recent call last):
  File "test.py", line 1, in <module>
    import pyrpl
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/__init__.py", line 66, in <module>
    from .redpitaya import RedPitaya
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/redpitaya.py", line 19, in <module>
    from . import redpitaya_client
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/redpitaya_client.py", line 28, in <module>
    from .hardware_modules.dsp import dsp_addr_base, DSP_INPUTS
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/hardware_modules/__init__.py", line 23, in <module>
    from .dsp import DspModule, DSP_INPUTS, all_inputs, all_output_directs, \
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/hardware_modules/dsp.py", line 2, in <module>
    from ..attributes import BoolRegister, SelectProperty, SelectProperty, SelectRegister
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/attributes.py", line 20, in <module>
    from .widgets.attribute_widgets import BoolAttributeWidget, \
  File "/usr/lib/python2.7/site-packages/pyrpl-<IP_ADDRESS>-py2.7.egg/pyrpl/widgets/attribute_widgets.py", line 10, in <module>
    import pyqtgraph as pg
  File "/usr/lib/python2.7/site-packages/pyqtgraph/__init__.py", line 13, in <module>
    from .Qt import QtGui
  File "/usr/lib/python2.7/site-packages/pyqtgraph/Qt.py", line 104, in <module>
    from PyQt4 import QtGui, QtCore, uic
RuntimeError: the PyQt4.QtCore and PyQt5.QtCore modules both wrap the QObject class

I need PyQt4 for some older apps so I can't uninstall that... it seems pyrpl is getting confused because of this. Any ideas?

No prob, I am happy if we discuss all issues rather sooner than later.
its been a while i had these issues. if i remember correctly, this bug can be worked around by either loading pyqt4 or pyqt5 before attempting to load pyrpl. so maybe playing with these combinations will get you somewhere:
import PyQt4
import pyrpl

import PyQt5
import pyrpl

from PyQt4 import QtCore
import pyrpl

or something like that. At least i believe the error message should change.
Another thing to try in ipython before loading pyrpl:
%gui qt
import pyrpl

%gui qt4
import pyrpl

%gui qt5
import pyrpl

good luck

Hi
Unfortunately none of those work for me. They all give the same error regarding API version, except for the last one; apparently pyqtgraph requires qt4.
Exception: PyQtGraph requires either PyQt4 or PySide; neither package could be imported.

Is there any easy way I can disable the loading of all the gui elements? I really just need command line control of the RP. Thanks

I figured it out. Since pyqtgraph requires qt4, that has to be used. That's fine.
So pyrpl/init.py line 28 should be (explicitly)
IPYTHON.magic("gui qt4")
In addition, to force the loading of qt4 and NOT qt5 on line 34, first do:
os.environ['QT_API'] = 'pyqt'

This forces qtpy to load qt4, otherwise it will by default load qt5.

Cool thanks! only loading qt4 will not work since many people only have qt5. But maybe we can find another workaround, i will have to play around a little.

It's kind of weird, the above trick only works if I do the following in ipython BEFORE calling import pyrpl:

import os
os.environ['QT_API'] = 'pyqt'
from qtpy import QtCore, QtWidgets

It apparently does no good chaning the init to have the above lines, which makes no sense to me.


Milestone D2

Dockerize application
Change to use cloud hosted MongoDB instance instead of local

LGTM. Note the change required to be made to the env files.


ci(android): update java requirement for cordova-android@11

Platforms affected
CI: Android
Motivation and Context

Paramedic which fetches and uses the latest releases of the platform is now using<EMAIL_ADDRESS>which requires Java 11.
Description

Update the GitHub Action workflow

Use latest version of the actions/setup-java GitHub Action plugin
Use Java 11
Use the Java Distro temurin which is the replacement of adopt

Testing

See Action

Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[x] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[ ] I've updated the documentation if necessary

The failing tests are unrelated to the change and has been existing for a long time.

Error: There was a conflict trying to modify attributes with  in plugin cordova-plugin-paramedic.

The change of this PR's is safe to merge.


It runned but when I look to someones head it looks straight to the ground immediately hauhuahuahus

Its running like this. the strange thing is he looking down everytime someone appears

Any ideas why??

Disable Raw Input

Disable Raw Input

Hello! thank you for your answer. But how do I do that

I used this guide but it didnt work
https://www.tomshardware.com/news/how-to-disable-mouse-acceleration-windows,36886.html

I used this guide but it didnt work
https://www.tomshardware.com/news/how-to-disable-mouse-acceleration-windows,36886.html

It depends on the game. CS:GO allows you to turn off Raw-Input, whilst other games like Overwatch force it on. I myself am trying to fix this issue in Overwatch.

m_rawinput 0 in GAME CONSOLE

m_rawinput 0
it works for me


How to take picture sequentially?

Hi
I want a user can take pictures as many as he/she wants on camera screen.
Without starting a previewActivity how can i achieve this?
Right now i found an unpleasant solution on BaseSandriosActivity.java . which is :
@Override
   public void onPhotoTaken() {
//        startPreviewActivity();
       getCameraController().onPause();
       getCameraController().onResume();
       Toast.makeText(this, "uri   :" +  getCameraController().getOutputFile().toString(), Toast.LENGTH_SHORT).show();
   }

and causes screen freeze 1-2 second.
Thanks in advance

Hello @bahacan19. Currently it is not possible to capture multiple images.
Also by multiple images, what do you mean burst mode or capturing images one by one?

@arpitgandhi9 capturing images one by one, i meant


Create ticket form

Adds a registration form and a create ticket form.

Looks Good But may have conflics down the road


Create 3D Graphs.py

@JituJakhar2000 please star the repo if u think this helped u for hacktoberfest :)


Shift Signature along a timeline during the execution of a test

As discussed in #167, as tests tend to run quickly and produce more than one commit per second, it might be safer to use an explicit signature and make it shift by one second for each commit.

I did this at some point.. forget when but you can see it in https://github.com/ParticularLabs/GitVersion/blob/182782e2274064049ef9c4b4632991077521f542/GitVersionCore.Tests/Helpers/Constants.cs


Add covenants-part-2 review to bitcoin-design/learning-bitcoin-and-design

This PR adds covenants-part-2 transcript review to the bitcoin-design/learning-bitcoin-and-design directory.

Hey @varmur, this is exceptional—truly one of the best I've had the pleasure of evaluating. Your meticulous attention to detail, evident in the careful addition of punctuation marks, removal of filler words, duplicate expressions, false starts, and stammerings, as well as the thoughtful inclusion of chapter divisions, has significantly enhanced the readability and overall quality of the transcript. It's clear you've dedicated a great deal of effort to this task, and the results speak volumes about your commitment to excellence.
Your approach not only improves the reader's experience but also sets a high standard for what we can achieve with thorough and considerate editing. The care you've taken to refine this transcript showcases a deep understanding of the importance of clarity and accessibility in our content.
I would love the opportunity to connect with you on our Discord server and have a discussion about your review process and experience. Your insights could be incredibly valuable for the project.
Thank you again for your outstanding work, @varmur. I'm looking forward to hopefully chatting with you soon on Discord and seeing more of your contributions in the future.


Add License Info for Learning C++ eBook

What does this PR do?
Add info | Improve repo
For resources
Description
This update adds licensing information to the "Learning C++ eBook" entry in the free-programming-books-langs.md file. This eBook is compiled from Stack Overflow Documentation.
Why is this valuable (or not)?
The license information clarifies the terms under which the eBook can be used, ensuring users are aware of their rights and obligations. This transparency is valuable for legal clarity and aligns with the open-source nature of the repository.
How do we know it's really free?
The eBook is available for free download via a direct link, and the license (CC BY-SA) confirms that it is freely distributable.
For book lists, is it a book? For course lists, is it a course? etc.
This entry is for a book, specifically an eBook.
Checklist:

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

Additional Notes
This PR does not remove or add any resources; it merely updates important license information to an existing entry to enhance the repository's usability and legal clarity.

Thanks, and welcome to free-programming-books!


re-added the occuring species

re-added the occuring species that I accidentally removed previously

This line will be not in the climsight.py anymore after PR #87. Or do update in cli branch (directly to #87), or redu PR after #87 is accepted.

You already re-added this line in the cli branch #87 (in climsight_engine.py, line 300), therefore I will close this PR.


Support for Chrome's "Auto-reload CSS upon SASS save" feature

Chrome has a new feature that watches for changes to compiled CSS when you Save a source-mapped file. So I can open a SASS file in Dev Tools, press Save and it will reload the updated CSS without requiring a reload of the page.
Unfortunately this only works when I do "Save As", and therefore I don't get the benefit of this extension's merging of changes and general awesomeness.
To enable this feature, you need to navigate to chrome:flags and enable the "Developer Tools Experiments", restart Chrome, open the inspector, open its Settings, go to the Experiments tab, select "Support for SASS", close the inspector and reopen it, go to the Settings again, open the General tab and select "Enable Source Maps" and "Auto-reload CSS upon SASS save".
Then, you need to have Compass or whatever watching your SASS files for changes and recompiling the CSS on the fly.
If you need any help with setting this up, or getting SASS compiled with the source maps included then please feel free to get in touch with me. It's pretty easy once you know how to do it.
This is very nearly a 'holy grail' workflow, would love to see this working with this extension.
Oh, and thanks for an awesome tool - I love it :)

Same here, can't find the option Auto-reload CSS upon Sass save.


[Vis] Add tensorboard to the project

log the following data during training:

train error metrics
val error metrics
(batch) input image, output image and ground truth

will be done in #13


[calnex] store target name

Add a field to store the target name in the measurement config.

Codecov Report

Merging #167 (b946e9a) into main (60da441) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #167   +/-   ##
=======================================
  Coverage   57.00%   57.00%
=======================================
  Files          74       74
  Lines        6796     6796
=======================================
  Hits         3874     3874
  Misses       2530     2530
  Partials      392      392

Impacted Files
Coverage Δ

calnex/config/config.go
87.24% <ø> (ø)

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 60da441...b946e9a. Read the comment docs.

@pmazzini can you please also update testdata/config.json :)


Update module version to v7

This PR fixes #4380 and fixes #6304 by updating the ATC module path from
https://github.com/apache/trafficcontrol/blob/75c37693f80bc774ca175a8720d418d983bc4ea3/go.mod#L1
to
https://github.com/apache/trafficcontrol/blob/da52e80e24d999a19bc9fcb7674ae8c49e48a240/go.mod#L1
and updates references to the latest ATC Go module around the repo to reference the v7 version. See the git history for the commands used to update files and https://github.com/apache/trafficcontrol/issues/4380#issuecomment-947812818 for the motivation for this PR.
Note that this PR does not move any files.
Once this PR is merged, a message will appear at the top of https://pkg.go.dev/github.com/apache/trafficcontrol lik this:

because the latest module will be at https://pkg.go.dev/github.com/apache/trafficcontrol/v7.

Which Traffic Control components are affected by this PR?

Documentation
Traffic Control Cache Config (T3C, formerly ORT)
Traffic Control Health Client (tc-health-client)
Traffic Control Client
Traffic Monitor
Traffic Ops
Traffic Stats
Grove
CDN in a Box (Enroller)

What is the best way to verify this PR?

Verify that the 8 non-URL, non-local-path instances of github.com/apache/trafficcontrol/ in the repo that did not have v7/ appended to them. They should stay as-is:
repo=github.com/apache/trafficcontrol/
git ls-files | xargs grep -Fn "$repo" 2>/dev/null | grep -vE "(https://|src/)${repo//./\\.}" | grep -vF "${repo}v7"
blueprints/api.guidelines.md:101:[the ATC library's Alert structure](https://pkg.go.dev/github.com/apache/trafficcontrol/lib/go-tc#Alert).
blueprints/refetch-invalidation.md:291:github.com/apache/trafficcontrol/lib/go-atscfg/regexrevalidatedotconfig_test.go
blueprints/refetch-invalidation.md:298:github.com/apache/trafficcontrol/traffic_ops/traffic_ops_golang/invalidationjobs/invalidationjobs.go
blueprints/refetch-invalidation.md:304:github.com/apache/trafficcontrol/traffic_ops/testing/api/v1/jobs_test.go
blueprints/refetch-invalidation.md:305:github.com/apache/trafficcontrol/traffic_ops/testing/api/v2/jobs_test.go
blueprints/refetch-invalidation.md:306:github.com/apache/trafficcontrol/traffic_ops/testing/api/v3/jobs_test.go
blueprints/refetch-invalidation.md:307:github.com/apache/trafficcontrol/traffic_ops/testing/api/v4/jobs_test.go
CHANGELOG.md:694:- Importing Traffic Ops Go clients via the un-versioned `github.com/apache/trafficcontrol/traffic_ops/client` is now deprecated in favor of versioned import paths e.g. `github.com/apache/trafficcontrol/traffic_ops/v3-client`.

PR submission checklist

[x] Existing tests are sufficient
[x] Modified existing documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Closing in favor of #7814


deployment instructions to Cloudflare Pages

Let me know if there's anything to be modified.

Could you please move images to a sub-directory here: https://github.com/flet-dev/website/tree/main/static/img/docs to host them with a website?

Alright.
Will do that.
I saw this: https://github.com/flet-dev/website/blob/main/static/img/docs/callout-demo.gif
You forgot to document it?

I didn't use the yarn. I haven't downloaded it yet.
So, let me know if there is anything I should add or modify.

By the way, remember to update the roadmap. (Check ✅ what has already been done)

Going to go have a look at deployment on vercel. 😉

Cool! And we could describe GitHub too as I tried it and it worked.


[Storage]Add all Missing SAS Permissions

We have some missing sas permissions for container
check sas for blob, queue and file(fileshare and file datalake) to make sure all permissions are provided in sdk.
https://docs.microsoft.com/en-us/rest/api/storageservices/create-service-sas#permissions-for-a-directory-container-or-blob

IS this the same as #8784

Hi @rakshith91
They are not the same. 8784 is (partially)updating the doc for generate_*_sas() operation.
This issue is to add missing permissions for *SasPermissions classes.
For example: ContainerSasPermissions is missing create and add permission currently according to rest doc.
We should check all SasPermissions classs for blob, file, queue, datalake and update all of them.
Also we should update the doc for generate__sas() operation.

Is this a manual error or something we need to bubble up to autorest?  @johanste

hi @jongio
Johan is OOF now.
It’s a manually error, not related to autorest I think.

Re-opening this issue as the PR is not yet merged. Will use this to track adding all missing SAS parameters.


Way to remember position and size of each windowed sessions

-Hello i run 5 accounts at the same time so could be nice if i was not forced to resize and move them between my 2 screens each time i restart them.
-Could we have individuals parameters for each session ? (nomusic for one and not for the other etc) i think it will be implemented with the shortcut functionality but just in case.
Regards.

Hello i run 5 accounts at the same time so could be nice if i was not forced to resize and move them between my 2 screens each time i restart them.

Coming with future additional game instance management.

Could we have individuals parameters for each session ? (nomusic for one and not for the other etc) i think it will be implemented with the shortcut functionality but just in case.

Will come later as the first thing that has to be done is to bind the choosen arguments to a account.


[GH-#181] Adding peers leads to timeout with big blockchain

closes #181, #193

I think that's the right place for it, the task doesn't stop when we reach an invalid block and it shouldn't, if we reach an invalid one we don't add it to our state and continue.

@d-velev My bad, you are correct.

@cytadela8 so are we good to go?
@meivantodorov can you review?

@thepiwo We are good to go, but I would like to get the response from @meivantodorov. I will try reaching him on Slack.


PATH not being set correctly.

For some reason when I try to use this on a SUSE 13 Service Patch 3 node from the tcsh shell, the PATH variable for the t1.script does not contain /bin. This leads to t1.log containing a large number of error messages as you would expect. I am not sure what it is about the particular system configuration that is causing the particular issue.  When I tried to use the test suite on a Linux Mint distribution, the results were much more positive. However, there were still some oddities such as terminal width impact acceptance of a result.
This is what caused the "avail" test to fail.
--- _err.left   2017-01-18 21:52:01.397445390 -0500
+++ _err.right  2017-01-18 21:52:01.405445553 -0500
@@ -2,5 +2,4 @@
ProjectDIR/rt/avail/mf/Core
A/1.0    B/2.0
Use "module spider" to find all possible modules.
-Use "module keyword key1 key2 ..." to search for all possible modules matching
-any of the "keys".
+Use "module keyword key1 key2 ..." to search for all possible modules matching any of the "keys".
Anyways, I am not sure what is causing the SUSE one to fail in the path aspect. Any ideas?

Thanks for the bug report! I have fixed the avail test so that it do the right thing independent of the terminal width.  I have release version 1.1 to fix the avail problem.
The problem with SUSE is likely to be in the way that Hermes tries to keep a minimum PATH.  I have fixed hermes so that it keeps /bin in the PATH.  Please try release 2.3 from hermes to see if it fixes the problem on SUSE for you

Using lua-<IP_ADDRESS> and Hermes-2.3, I am not seeing /bin in the PATH provided by Lmod_test_suite-1.1.

Alright, instead of complaining, here is the issue:
In the local function cleanPath in CleanEnv.lua in Hermes 2.3, the block of code starting at line 56 adds /usr/bin and /bin to pathT as a "keep" only if they are not already in pathT. The directory '/usr/bin' is being kept because of the block of code at starting at line 77.  I think that you want the "==" on line 60 to be a "~=", which fixes this issue for me.

Thanks.  Obviously what I had didn't work.  I had to tweak your fix slightly.  I have pushed a new version of Hermes (2.4).  If you get the chance, please test out Hermes 2.4.  Thanks for the feedback!


enabled source link support

https://github.com/bytecodealliance/wasmtime-dotnet/issues/92

@ben-page thanks for the PR! I'll have time to properly review this tomorrow hopefully.


Allow None as Rounding Precision in clean_weights

the documentation of BaseOptimizer#clean_weights suggests that None is an acceptable rounding value (meaning, "no rounding"), however, there is data validation which requires it be an integer greater than 0.
This commit moves the validation within the rounding is not None condition with a new test to ensure rounding is not being performed.

(please forgive any python style violations... this is my first exposure to the language!)

@pmn4 Thank you for the suggestion! I agree regarding None.
I made a change to the testing though: you are right that the "proper" way to do it is to check whether the function has been called using mock. I removed this for two reasons:

Don't want to have both unittest and pytest
Rounding issues actually are very relevant, so I'd rather the test suite flag them so I can investigate further.

Instead, I've gone for a very simple equality test (with a hacky fix because python 3.5 keeps dicts unsorted).

can't thank you enough for this repo @robertmartin8!
best of luck with the various ventures


Address PR feedback in #4447

#4447 was merged without addressing the (non-breaking) PR feedback, so we could get it in the next release. At some point, we should address the PR feedback; namely:

https://github.com/pachyderm/pachyderm/pull/4447#discussion_r360216167
https://github.com/pachyderm/pachyderm/pull/4447#discussion_r360216206

@ysimonson I provided responses to the comments. It probably makes sense to just discuss in that PR, then make a separate PR with any changes we decide on.

Left some follow-up comments


wlroots 0.4

This is a provisional changelog of breaking changes. It contains changes up to e77e53dae51a3a46e1327c0c53e69d6da695cc9a.

Drag-and-drop has been refactored (https://github.com/swaywm/wlroots/pull/1517)
Compositors need to listen to the wlr_seat.request_start_drag event (just like selection). They need to accept the drag with wlr_seat_start_{pointer,touch}_drag or to reject it. See the related rootston changes.
Additionally, the new_drag_icon event has been removed, use start_drag instead to collect the drag icon.

wlroots 0.4 has been released: https://github.com/swaywm/wlroots/releases/tag/0.4


Provide an adapter for localForage

Please provide a built-in adapter for Mozilla's localForage.

localForage has: getItem(), setItem(), removeItem()
the built-in AsyncStorage adapter should work for you
https://github.com/elgerlambert/redux-localstorage/blob/1.0-breaking-changes/src/adapters/AsyncStorage/adapter.js

Hi @dheerajvs, @vicentedealencar is right; the AsyncStorage adapter should work. You can do the following to get going:
import adapter from 'redux-localstorage/lib/adapters/AsyncStorage';

const storage = adapter(localForage);

I will add it to the list of adapters though to prevent confusion and help others find what they're looking for.

Thanks for the clarification. You may close this issue.

Just realised there is an important difference between AsyncStorage and localForage; AsyncStorage only stores string values, whereas localForage accepts any type of value. The JSON.stringify/parse that's part of the AsyncStorage adapter therefore posses an issue. Which, in a sense, is a similar issue as the one raised in PR #16.

@elgerlambert I'd say just add a new adapter for localForage instead of trying to stuff it into AsyncStorage. Since not all storages limit their data to be strings, JSON.stringify()/.parse() should not be considered as a default part of all async storages.

1.0.0-rc5 includes an adapter for localForage!

Hello !
Somebody can tell me why I do not have any folder named "adapters" in redux-localstorage/lib/ ?
What I did wrong ?
Thanks !

Hi @idwd,
Make sure you npm install redux-localstorage@rc --save. The @latest (0.4.1) doesn't have adapters.


fix: de correction

this PR fixes the issue of not correctly sorting and dropping log2FoldChange and padj columns for the downstream plots
It does NOT fix the statistical issues associated with insufficient data. Here, only optional imputation rules, will help.

forgot to add a revised CI version of the config file!


Vanilla Kubernetes, kube-proxy don't proxy Grafana correctly

Rancher versions:
rancher/server: v1.6.12
rancher/agent: v1.2.7
Infrastructure Stack versions:
healthcheck: v0.3.3
ipsec: v0.13.5
kubernetes (if applicable): k8s:v1.8.3-rancher3
Docker version: 17.09.0-ce
Operating system and kernel: CoreOS 1576.4.0, kernel 4.13.16-coreos-r2
Type/provider of hosts: vSphere self-hosted
Setup details: single-node Rancher + 3 hosts
Environment Template: Kubernetes
Steps to Reproduce:

Add three hosts.
Deploy Kubernetes using the vanilla template.
Access to the web interface and run via cli kubectl cluster-info.
Access to the URL of "monitoring-grafana", such as https://<rancher-server>:8080/r/projects/1a7/kubernetes:6443/api/v1/namespaces/kube-system/services/monitoring-grafana/proxy.

Results:
The kube-proxy component fails to route the requests for some components of the page, which go 404

The resources are available directly (like in http://<rancher-server>:8080/r/projects/1a7/monitoring-grafana:3000/public/css/fonts.min.css) but Grafana can't be opened at the base address of http://<rancher-server>:8080/r/projects/1a7/monitoring-grafana:3000 for "Invalid API Key".

I believe it's somehow related to #8594, but more tied to the proxy component.
Also, I'm pretty new both to Rancher and Kubernetes.

I'm having the same issue the /r/projects/1a244/monitoring-grafana:3000 url is giving me the same Invalid API Key message even though GF_AUTH_BASIC_ENABLED: false is set

More info:
I'm accessing grafana with https://rancher.example.com/r/projects/1a244/kubernetes:6443/api/v1/proxy/namespaces/kube-system/services/monitoring-grafana:80
This page loads but all the urls are not correct and are pointing to https://rancher.example.com/api/v1/proxy/namespaces/kube-system/services/monitoring-grafana:80/public/css/fonts.min.css
It's missing the /r/projects/1a244/kubernetes:6443 part.
I can get to the css and other files files with the following url https://rancher.example.com/r/projects/1a244/kubernetes:6443/api/v1/proxy/namespaces/kube-system/services/monitoring-grafana:80/public/css/grafana.dark.min.af13213c.css

Hi @vlczero did you solve the issue?
I exactly has the same issue. Somehow, when I access the grafana link generated by kubectl cluster-info, the resources will missing the r/projects/1a155/kubernetes:6443 part.
rancher: 1.6.14
rancher/agent: 1.2.9
kubernetes: 1.8.10

@mdgs I "fixed" this by not using the proxy through rancher but make a service that opens a port on the instances. My AWS Application Load balancer has a target group for grafana that points to the open port. Sadly I could not get the proxy to work due to the url prefixes rancher introduces.
My service and deployment files: https://gist.github.com/vlczero/7cf850ea605cae7ae4adeff34a32fb9a
This will overwrite the default grafana deployment and ads persistency for grafana setting/users by storing it in an external mysql database . If you don't want that you can remove the GF_DATABASE_* env values.
Due note that the influxdb is not persistent and you will loose all data when it is rescheduled/crashes. I'm still looking for a way to keep the influxdb data.

im running
   kubectl proxy

this helped in my case
- name: GF_SERVER_ROOT_URL
    value: "http://localhost:8001/api/v1/namespaces/monitoring/services/http:grafana:3000/proxy/"

I had a similar issue, and I finally solved it by adding the following annotation to the L7 Ingress object: nginx.ingress.kubernetes.io/rewrite-target: /
And the following to grafana's configuration:
[server]
root_url = %(protocol)s://%(domain)s/grafana/

Source: https://github.com/kubernetes/ingress-nginx/tree/master/docs/examples/rewrite

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Ensure that we handle indirect objects in all types of Opt entries in ChoiceWidget annotation dictionaries

I haven't got an example where the current code breaks, but given all the previous cases we've seen where PDF generators use indirect objects in Arrays it makes sense to fix this pro-actively.
I've modified the relevant unit-tests slightly, and they would not pass without the code changes in this patch.
Note: Dict_getArray only dereferences Array elements on the "top-level", to avoid recursion issues. Furthermore if you have to loop through the Array at the call-site anyway, then using Dict_get in combination with XRef_fetchIfRef is a tiny bit more efficient.

This change is

/botio unittest

/botio-linux preview

/botio test

Thank you! I agree it's good to be on the safe side here.


fix(ci): Added cgroups for the github runner agent prior to the docker build

steps
Description:
Adds cgroups to push-images.yaml for the runner to hopefully prevent resource utilization errors when building the docker images.
Converts the self-hosted runner for the push-image workflow to use a large runner.
Related issue(s):
Fixes #198

I guess we can safely close this. Thanks @rbarkerSL for the support to solve this problem!

@rbarkerSL let me know if we can close this (and delete the corresponding branch). We can still restore it if needed in the future.


Add material-icons

Description

Add Search Material Icons command that let you search material icons and preview them within Raycast.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed in the metadata folder

Well, this is annoying...

@creasty It seems like we already have a similar extension for this https://github.com/raycast/extensions/pull/1176.  it'd be awesome if you could incorporate your amazing code there! Perhaps by submitting a PR
Just a suggestion though! Feel free to ignore!

Hi @creasty 👋
I'm so sorry for the waiting time, we're working on bringing the queue down, and we're almost back from easter 😄
What do you think about what @Aayush9029 said?

What do you think about what @Aayush9029 said?

Mine is more optimized for Material Icons, intended to mimic functionality of https://fonts.google.com/icons.
Iconify doesn’t come with canonical names, so when you want to view edit icon in the other extension for instance, you have to type that name exactly, whereas in my extension you can get the same icon for other keywords like pen or write.

Cheers! I've just removed the rendundant LICENSE file.


feat: Add Language#normalizeLanguageOptions()

Prerequisites checklist

[x] I have read the contributing guidelines.

What is the purpose of this pull request?
Add normalizeLanguageOptions to the Language type.
What changes did you make? (Give an overview)
Added an optional normalizeLanguaeOptions() method to the Language type.
Related Issues
refs https://github.com/eslint/eslint/issues/19037

Is there anything you'd like reviewers to focus on?

Probably a good idea to avoid confusion. I wish GitHub had a "schedule" feature.


[monitor] update to latest OTEL

Packages impacted by this PR

@azure/monitor-opentelemetry-exporter
@azure/monitor-opentelemetry
@azure/monitor-query
@azure/opentelemetry-instrumentation-azure-sdk

Issues associated with this PR

https://github.com/Azure/azure-sdk-for-js/issues/30324
https://github.com/Azure/azure-sdk-for-js/issues/30323
https://github.com/Azure/azure-sdk-for-js/issues/30322
https://github.com/Azure/azure-sdk-for-js/issues/30321
https://github.com/Azure/azure-sdk-for-js/issues/30320
https://github.com/Azure/azure-sdk-for-js/issues/30319
https://github.com/Azure/azure-sdk-for-js/issues/30318
https://github.com/Azure/azure-sdk-for-js/issues/30317
https://github.com/Azure/azure-sdk-for-js/issues/30316

Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


alert notifications shown as enabled by default when they're not

When a user has not setup nofitications yet, the django view code defaults to creating a new Notifications model instance. This has all notifications types set to be enabled for 'alert'.
So on screen it looks to the user that these notifications are enabled, when in reality they are not.
solution is either to disable them by default in the Notifications model or to somehow create a default Notifications model instance in the database whenever a new user is created.

fixed for users created after https://github.com/DefectDojo/django-DefectDojo/blob/4be8d4c6e4f6e72f3574098e985d067b068bceb1/dojo/utils.py#L1971
existing users may still suffer.

if someone can write a short piece of code to fix this for existing users, we can add that as a migration

@valentijnscholten  are we okay to close this one?


No print output -  Bug Report PyCharm - works in Thonny

Hi Andrew,
Thank you for you reply.  The context is different I believe.
It appears you are using an ESP32, mine is RP2040 - Pico.
I have tried everything including Tools/Reble setting, sadly I have given up, moved to to Pi Zero 2 W
Thanks
Gary


(cquery-xref-find-custom "$cquery/derived") timed out

#include <iostream>

class Base
{
 public:
  Base();
  virtual ~Base();
  virtual void open(){}
};

class Derived: public Base
{
 public:
  Derived();
  virtual ~Derived();
  void open(){}
};

int main(int argc, char *argv[])
{
  Base *b;
  b->open();
  std::cout << "Hello World!" << std::endl;
  return 0;
}

When I ran (cquery-xref-find-custom "$cquery/base")  on Derived::open, the cursor jumped to Base::open, which was expected.
When I ran (cquery-xref-find-custom "$cquery/derived")  on Base::open, I got lsp--send-wait: Timed out while waiting for a response from the language server .
Tested debug and release on 3a21ff0. Debug version didn't trigger any asserts.
I enabled --record=/tmp/cquery, and only saw the request in cquery.in. There was no output in cquery.out.

+1

Function is removed since 7d4b7e2e?

@dimbleby Thanks.
Either lsp-goto-implementation or lsp-ui-peek-find-implementation works. Perhaps only the documentation needs to be updated.


Add run export

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

add run_exports to ensure compatible version

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


This pop-up is appearing not only on dashboard page but also on datasource page as well.
Version
kubernetes: eks v1.20.11
Grafana-operator: 4.3.0
Grafana: 8.5.0
To Reproduce
Steps to reproduce the behavior:

Install grafana operator v4.3.0
Use grafana v8.5.0
Add a dashboard and datasource
Launch grafana UI and login with server admin
Browse for a dashboard and wait for popup for every 2-3 sec
etc.

Expected behavior
This popup shouldnt appear if auto refresh is set off and if time is set then dashboard should refresh according to the auto refresh interval
Suspect component/Location where the bug might be occuring
This happened after migrating to grafana operator v4.3.0 and grafana v8.5.0
Screenshots

Runtime (please complete the following information):

OS: NAME="Red Hat Enterprise Linux Server", VERSION="7.9 (Maipo)".
Grafana Operator Version 4.3.0
Environment: EKS
Deployment type: helm deployment, deployed datasource CR and dashboard CR along with grafana and grfana-operator

Additional context
Running grafana with oauth enabled.
Dashboard used is self built to monitor/display grafana metrics
Grafana-logs:
Error checking server process execution privilege. error: could not get current OS user to detect process privileges
logger=sqlstore t=2022-05-20T09:37:57.52+0000 lvl=warn msg="SQLite database file has broader permissions than it should" path=/var/lib/grafana/grafana.db mode=-rw-rw---- expected=-rw-r-----
logger=context traceID=00000000000000000000000000000000 userId=0 orgId=0 uname= t=2022-05-21T15:40:10.54+0000 lvl=eror msg="bad login data" error="bad content type"  traceID=00000000000000000000000000000000
logger=context traceID=00000000000000000000000000000000 userId=0 orgId=0 uname= t=2022-05-21T15:40:11.15+0000 lvl=eror msg="bad login data" error="bad content type"  traceID=00000000000000000000000000000000
logger=context traceID=00000000000000000000000000000000 userId=0 orgId=0 uname= t=2022-05-21T15:40:11.77+0000 lvl=eror msg="bad login data" error="bad content type" traceID=00000000000000000000000000000000
logger=context traceID=00000000000000000000000000000000 userId=0 orgId=0 uname= t=2022-05-21T15:40:12.38+0000 lvl=eror msg="bad login data" error="bad content type" traceID=00000000000000000000000000000000
logger=context traceID=00000000000000000000000000000000 userId=0 orgId=0 uname= t=2022-05-21T15:40:12.98+0000 lvl=eror msg="bad login data" error="bad content type" traceID=00000000000000000000000000000000
logger=context traceID=00000000000000000000000000000000 userId=0 orgId=0 uname= t=2022-05-22T02:04:27.86+0000 lvl=eror msg="bad login data" error="bad content type"  traceID=00000000000000000000000000000000

Not able to reproduce this issue again.

Try to reproduce the same issue with grafana:8.5.0 and operator:4.1.0. Issue is not there.
Only problem is the operator being upgraded to 4.3.0.

This is solved in 4.4.1. Please upgrade to the latest version.

This is solved in 4.4.1. Please upgrade to the latest version.

Tested it. Working fine. Thanks for the update.

I've still got the same issue with 4.4.1 using the demo dashboard from configmap


Allow SetProperty to work with WhenAny().ToProperty()

MvvmCross convention is to use SetProperty for INPC.
Provide a way to do this that behaves with WhenAny().ToProperty().

awesome.  thanks for the PR


There are some differences between the user's location of react-native-map and the location from navigator.geolocation.

Location is not accurate when you are placing your marker.
See example: https://github.com/react-community/react-native-maps/issues/209#issuecomment-350907665


Visualization with multiple inputs in forward method

Hello, I have a network for which the forward method is defined with two inputs e.g: pred = model(image1, image2).
As for now I haven't been able to use grad-cam. I tried defining input_tensor = (image1, image2) but with no sucess. Is there anyway use grad-cam for such network (from my understanding in theory should be possible).
Thanks in advance! :)

Hi,
I think this could be done by creating a custom reshape_transform,
but I need more details.
How does the model look ?
In the layer you want to target, are there spatial activations from both images?


PadIfNeeded bbox support

Hello, I opened an issue previously, and i decided to push the changes i made to make PadIfNeeded work with bounding boxes.
Looking forward to your feedback.

There is a weird opencv bug preventing me from running tests locally, i will see what i can do and fix the checks soon.

I believe, cases with and without reflection should be treated differently.


WIF encoded private keys?

Looking for a way to WIF encode/decode private keys for testing and then import. Only reference I find is possibly issue 191 re: removing WIF Functions.
Am I missing something? Or is that a DIY feature? Thnx

No, we removed WIF support because we don't use it in


Fix #61

Convert tabs to spaces
Remove circular imports from JSONHelpers.py
Correct view file name in AWGs.py
Correct title in AWGsViews.enaml

Thanks!

My pleasure.  I've got a few more minor things coming that I've come
across while learning the tools.
On 7/28/15 3:48 PM, Blake Johnson wrote:

Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/BBN-Q/PyQLab/pull/62#issuecomment-125733989.


[GAL-138] Deduplicate NFTs by opensea ID in sidebar

[x] Visually ensure sidebar still loads

amazing! remember you bringing up this edge case before, great to see it handled now 🔥


Use local storage so that bucket list persists on refresh

Bonus task from brief.
Which React Hooks could we use in combination with local storage to make the bucket list persist after refreshes?

Import useEffect hook
Run with a console log every time state is changed

Add the bucket to local storage
Make sure the item is stringified for local storage
Log the value retrieved; make sure it is updated immediately

Fetch the initial value for the state using a function, rather than an an array
Moved the function that returns the default values outside of the component, as it is needs to exist before state is initialised for React to find it
Conditional function that checks localStorage for bucket
Make sure to JSON.parse() the value returned by local storage

Manually tested using localStorage.removeItem - this helped me identify a bug
Fixed bug
This now is complete.


Add the "add-reader" behavior to read ACL

Adding a reader may give all prior access, even to data before the point in time when the reader was added. This is needed to generate the document from the change history.
Removing a reader means requests to load will not longer be honored; they may lose all access to the document (i.e. they are not guaranteed to keep any prior states where there was access).


0R0P

Key
Value

View
https://database.shabados.com/sources/3/page/41/line/134

Source
Vaaran Bhai Gurdas Ji

Page
41

ID
0R0P

Line
inj sUry isMG virAwmVy; bhu lwK jgwey [

Related to #1182


Build problem

🐛 Bug
I am trying to build habitat-sim on a previous commit for getting a baseline on another model, but am running into a cmake error at the very end of building.
I am using:

Ubuntu 16.04
gcc 7.5.0

Steps to reproduce:
conda create -n habitat python=3.6 cmake=3.14.0
conda activate habitat
git clone https://github.com/facebookresearch/habitat-sim.git
cd habitat-sim
git checkout 9575dcd45fe6f55d2a44043833af08972a7895a9
pip install -r requirements.txt
python setup.py install --headless

I've tried:

sudo apt install libegl1-mesa-dev
installing without --headless
conda install ninja

Here is the error log:
log.txt

I guess that is before the PR that solved this issue: https://github.com/facebookresearch/habitat-sim/issues/205. Please cherry-pick in https://github.com/facebookresearch/habitat-sim/pull/247


Update Spine version

This PR updates Spine version to 0.9.64-SNAPSHOT, that solves storage-related issues and improves tests for storages. As a result, several issues was detected and fixed.

@armiol PTAL.

@armiol PTAL.

@armiol PTAL.


Make basic routing working again

To move forward faster with an ability to use devworkspace controller as workspace engine in chectl, it's needed to provide some quick solution to make basic routing working again, it's needed:

patch THEIA_HOST from plugin registry meta.yaml to set <IP_ADDRESS>: https://github.com/eclipse/che-plugin-registry/blob/master/v3/plugins/eclipse/che-theia/7.19.2/meta.yaml;
patch che-machine exec to listen to <IP_ADDRESS> https://github.com/eclipse/che-plugin-registry/blob/master/v3/plugins/eclipse/che-machine-exec-plugin/7.19.2/meta.yaml;
switch on TLS for basic routing (if it's still not enabled);
-* unique host may be also needed for webviews to work but that's as best afford

Later these hacks should be replaced with proper routing with authentication enabled.

basic routing works again after https://github.com/devfile/devworkspace-operator/pull/169 except Theia Webview feature.


up_squared: tests/kernel/mem_heap/mheap_api_concept/kernel.memory_heap fail

Describe the bug
tests/kernel/mem_heap/mheap_api_concept/kernel.memory_heap
fail.
To Reproduce
twister --device-testing --hardware-map up2.map -s tests/kernel/mem_heap/mheap_api_concept/kernel.memory_heap -vv
Logs and console output
DEBUG   - DEVICE: *** Booting Zephyr OS build zephyr-v3.5.0-2629-gd70e8d5a8f91 ***
DEBUG   - DEVICE: E: ACPI get table failed: 5
DEBUG   - DEVICE: Running TESTSUITE mheap_api
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_k_aligned_alloc
DEBUG   - DEVICE: PASS - test_k_aligned_alloc in 0.001 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_malloc_in_isr
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/mem_heap/mheap_api_concept/src/test_mheap_api.c:37: tIsr_malloc_and_free: (ptr is NULL)
DEBUG   - DEVICE: bytes allocation failed from system pool
DEBUG   - DEVICE: FAIL - test_malloc_in_isr in 0.019 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_malloc_in_thread
DEBUG   - DEVICE: PASS - test_malloc_in_thread in 0.001 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_mheap_calloc
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/mem_heap/mheap_api_concept/src/test_mheap_api.c:132: mheap_api_test_mheap_calloc: (mem is NULL)
DEBUG   - DEVICE: calloc operation failed
DEBUG   - DEVICE: FAIL - test_mheap_calloc in 0.018 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_mheap_malloc_align4
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/mem_heap/mheap_api_concept/src/test_mheap_concept.c:44: mheap_api_test_mheap_malloc_align4: block[i] is NULL
DEBUG   - DEVICE:
DEBUG   - DEVICE: FAIL - test_mheap_malloc_align4 in 0.017 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_mheap_malloc_free
DEBUG   - DEVICE: PASS - test_mheap_malloc_free in 0.001 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_mheap_threadsafe
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/mem_heap/mheap_api_concept/src/test_mheap_concept.c:60: tmheap_handler: (pool_blocks[thread_id] is NULL)
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/mem_heap/mheap_api_concept/src/test_mheap_concept.c:60: tmheap_handler: (pool_blocks[thread_id] is NULL)
DEBUG   - DEVICE: memory is not allocatedmemory is not allocated
DEBUG   - DEVICE:
DEBUG   - DEVICE: FAIL - test_mheap_threadsafe in 0.037 seconds
DEBUG   - DEVICE: ERROR: cannot fail in test phase 'framework()', bailing
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/mem_heap/mheap_api_concept/src/test_mheap_concept.c:60: tmheap_handler: (pool_blocks[thread_id] is NULL)
DEBUG   - DEVICE: TESTSUITE mheap_api failed.
DEBUG   - DEVICE: memory is not allocated
DEBUG   - DEVICE: ------ TESTSUITE SUMMARY START ------
DEBUG   - DEVICE:
DEBUG   - DEVICE:
DEBUG   - DEVICE: SUITE FAIL -  42.86% [mheap_api]: pass = 3, fail = 4, skip = 2, total = 9 duration = 0.094 seconds
DEBUG   - DEVICE: ERROR: cannot fail in test phase 'teardown()', bailing
DEBUG   - DEVICE: - PASS - [mheap_api.test_k_aligned_alloc] duration = 0.001 seconds
DEBUG   - DEVICE: - FAIL - [mheap_api.test_malloc_in_isr] duration = 0.019 seconds
DEBUG   - DEVICE: - PASS - [mheap_api.test_malloc_in_thread] duration = 0.001 seconds
DEBUG   - DEVICE: - FAIL - [mheap_api.test_mheap_calloc] duration = 0.018 seconds
DEBUG   - DEVICE: - FAIL - [mheap_api.test_mheap_malloc_align4] duration = 0.017 seconds
DEBUG   - DEVICE: - PASS - [mheap_api.test_mheap_malloc_free] duration = 0.001 seconds
DEBUG   - DEVICE: - FAIL - [mheap_api.test_mheap_threadsafe] duration = 0.037 seconds
DEBUG   - DEVICE: - SKIP - [mheap_api.test_multi_heap] duration = 0.000 seconds
DEBUG   - DEVICE: - SKIP - [mheap_api.test_sys_heap_mem_pool_assign] duration = 0.000 seconds
DEBUG   - DEVICE:
DEBUG   - DEVICE: ------ TESTSUITE SUMMARY END ------
DEBUG   - DEVICE:
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: RunID: 497da24b1196b92df88dba66c644470e
DEBUG   - DEVICE: PROJECT EXECUTION FAILED

Environment:

OS: Linux
Toolchain: zephyr-sdk-0.16.3
zephyr-v3.5.0-2629-gd70e8d5a8f91

@KamilxPaszkiet this is just a consequence of the app explicitly setting the following in its prj.conf:
CONFIG_HEAP_MEM_POOL_SIZE=256

There's just no way that's going to work with any platform that uses ACPI, since ACPI requires a heap size of at least a few hundred kilobytes, or in the case of up_squared roughly a megabyte. So this is really a bug with the application overriding the heap size, not something with up_squared. Since there's already an issue tracking this (#65722) and a PR where possible fixes are being discussed (#65908), I don't think it makes sense to open these separate issues. Do you agree?

@KamilxPaszkiet feel free to reopen if this has some other root cause (e.g. try the test by removing that line from prj.conf - if the issue still appears then its about something else than the heap size)


Add power usage report to HA

What would be the way to implement the new wideq power usage report feature to this HA component?

@dacrypt  - how did you get that going?  was pleased to check into this repo today and see @sampsyo has added power consumption...  so did a new git pull.... confirm that the updated wideq appears to be pulling power stats back...

... but it doesnt seem to be showing up in HASS (history page for instance on my W / kWh graph, there is no LG aircon to be found). Should i be expecting to see W values in here? (dev tools -> states UI)

thanks in advance!
Keiran.

Hi! No, we haven't added any support to the HA component. I'm not sure if anyone's working on it… @dacrypt, it looks like you might have a prototype working? Shall we merge any part of it back into this repo?

@dacrypt  - touching this thread again to see if you would consider merging your works so we can suck in power data into HASS? Would love to see it. thanks guys!

@dacrypt - touching this thread again to see if you would consider merging your works so we can suck in power data into HASS? Would love to see it. thanks guys!

Try this: #48
I've been quite busy at work so I haven't had the time to share what I've done. I replaced my raspberry HA  with a vps over vpn and installed a docker version under ubuntu and never managed to upload my version of hass-smartthinq and test it. I need to spend some time to learn how to install the code I have to my own ha.

Any updates on this? Would be great to be able to integrate power usage with the Energy Dashboard of HA.


cluster: add option to specify busybox repository

cluster: add option to specify busybox repository
add 2 options in EtcdCluster.spec:
busyboxRepository. Defaults to "busybox"
busyboxVersion. Defaults to "1.28.0-glibc"
for #1927

Since you changed the API, please also update the generated code: https://github.com/coreos/etcd-operator/blob/master/hack/k8s/codegen/README.md

@hongchaodeng Running the script did not change the generated code. Is it possible, or am I missing anything?

What's the output of running the command?

Generating deepcopy funcs Generating clientset for etcd:v1beta2 at github.com/coreos/etcd-operator/pkg/generated/clientset Generating listers for etcd:v1beta2 at github.com/coreos/etcd-operator/pkg/generated/listers Generating informers for etcd:v1beta2 at github.com/coreos/etcd-operator/pkg/generated/informers

@etcd-bot ok to test

gofmt check failed:
Checking gofmt...
gofmt checking failed:
...

@hongchaodeng Fixed your comments.

I think it is better with two properties, as it is the same as the etcd
image and tag. If you still believe it is better to combine, I'll do it. No
problem
On Mar 5, 2018 21:12, "Hongchao Deng"<EMAIL_ADDRESS>wrote:

@hongchaodeng commented on this pull request.
In pkg/apis/etcd/v1beta2/cluster.go
https://github.com/coreos/etcd-operator/pull/1928#discussion_r172295919:

@@ -143,6 +143,14 @@ type PodPolicy struct {
// etcd cluster.
// The "etcd.version" annotation is reserved for the internal use of the etcd operator.
Annotations map[string]string json:"annotations,omitempty"

// busybox init container repository. default is busybox
BusyboxRepository string json:"busyboxRepository,omitempty"

Sorry to bother gain.
But after a second though, it would be better and easier to combine
BusyboxRepository and BusyboxVersion into one field, e.g. BusyboxImage.
Default to "busybox:1.28.0-glibc".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coreos/etcd-operator/pull/1928#pullrequestreview-101289275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF2uY-s_f7XIelyvMM_c-4kOQMTFBFI7ks5tbY4lgaJpZM4SJQum
.

as it is the same as the etcd image and tag.

The etcd image is a special case. We expect that most of time users would only upgrade versions, aka tags. But that's not the case here.

Ok.
On Mar 5, 2018 21:18, "Hongchao Deng"<EMAIL_ADDRESS>wrote:

as it is the same as the etcd image and tag.
The etcd image is a special case. We expect that most of time users would
only upgrade versions, aka tags. But that's not the case here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coreos/etcd-operator/pull/1928#issuecomment-370531366,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF2uY_oTPeev6Go09vXNYuMGAud_h3kWks5tbY9tgaJpZM4SJQum
.

LGTM

@fanminshi @hasbro17
Can one of you also review and then merge this?

lgtm

@yehiyam @hongchaodeng are there any examples on how to use this to pass in an alternate repo for busybox?

@herikwebb you can look at an example in our helm chart

@yehiyam great, thanks!


[Morpho] change max value of kernel

About https://github.com/Inria-Asclepios/medInria-public/issues/310
Small PR increasing the kernel size in morpho toolbox.
:m:

This PR has been done on integration, like a lot of others. Do we want to merge it on integration, and at the end merge integration of master?

closed for this one https://github.com/Inria-Asclepios/medInria-public/pull/321


PKPDX-11 [Upload data step] Add 'Upload Dataset' button to D&D

Thanks @wniestroj, I'll let @eatyourgreens review this


[acix-255] Remove deprecated notifications for all teams

What does this PR do?
ACIX-255
https://github.com/DataDog/datadog-agent/pull/26326 adds new notifications that will be sent to the teams. This PR removes the old notifications (except for #datadog-agent-pipelines).

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1400 ms

Resolved: Bofrost.at performance has improved in e370b49.


Fix versions used in ansible.

The master branch is still on Girder 1.7.  Use the appropriate versions for ansible.
Fixes issue #239.
ping @cdeepakroy

Codecov Report

Merging #240 into master will not impact coverage.

@@           Coverage Diff           @@
##           master     #240   +/-   ##
=======================================
  Coverage   60.79%   60.79%
=======================================
  Files           8        8
  Lines         352      352
=======================================
  Hits          214      214
  Misses        138      138

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 9a433e5...c062f64. Read the comment docs.

LGTM!
I just tried git clone && vagrant up in this branch and it succeeds. I confirm that this fix works.


As a developer I would like to add tests that checks if a user i properly deleted from the database when using the "forget me" button on the profile page

Acceptance:

User is no longer in the database and can be provably shown using tests for fetching this information
Tests go green

WIP on branch IntegrationTest.DeleteUser.

Would love some help properly creating the playwright test.
@JonasSkjodt, could you lend a hand?

This ticket now has a dependency on #166 :)


pipeline Fix blocking task which is not triggered by 2nd RPC (…

…#38568)
Once a query is cancelled due to any reason, BE may not receive 2nd RPC from FE. If so, we must ensure the execution dependency is ready so tasks will not be blocked.
Proposed changes
Issue Number: close #xxx

run buildall


[#500] Openadas on ITER

Motivations:

the submodule openadas2tofu directly sends a request online to openadas using the built-in library requests, read the answer (html), parse it and extract the info of interest
It works fine at CEA and on my own laptop (and in unit tests on Travis and Github), but not on ITER clusters

Main Changes

A ticket was opened to ask ITER admins their feeling about it
It lead to the creation of a second ticket to the IT team
It seems the requests library used, has its own list (bundle) of certificates for ssl connections, and apparently, for some unknown reason, this bundle of certificates does not include openadas's certificate on ITER's python intsllation
The admin recommend to replace the library's bundle of certificates by the ITER server's bundle of certificates
Instead of doing it from the bash command line, I implemented inside tofu.openadas2tofu a subrotuine that automatically tries to connect, and changes to the server's bundle of certiciates if the first attempt fails.
This way it is transparent for the user who does not have to care about this

Example:
In [3]: tf.openadas2tofu.step01_search_online_by_wavelengthA(lambmin=3.94, lambmax=4, element='ar')
Wavelength  Element  Charge  Data Type  Transition                               Full file name
----------  -------  ------  ---------  ---------------------------------------  ----------------------------------------
3.94817     Ar       16+     ADF15      11513 ^{1}P_{1.0} -> 21 ^{1}S_{0.0}      /adf15/pec40][ar/pec40][ar_ic][ar16.dat
3.94817     Ar       16+     ADF15      11513 ^{1}P_{1.0} -> 21 ^{1}S_{0.0}      /adf15/pec40][ar/pec40][ar_ls][ar16.dat
3.96042     Ar       16+     ADF15      11513 ^{0}S_{5.5} -> 21 ^{0}S_{0.0}      /adf15/pec40][ar/pec40][ar_cl][ar16.dat
3.96042     Ar       16+     ADF15      11513 ^{0}S_{5.5} -> 21 ^{0}S_{0.0}      /adf15/pec40][ar/pec40][ar_ca][ar16.dat
3.96445     Ar       16+     ADF15      11513 ^{3}P_{4.0} -> 21 ^{1}S_{0.0}      /adf15/pec40][ar/pec40][ar_ls][ar16.dat
3.96626     Ar       16+     ADF15      11513 ^{3}P_{1.0} -> 21 ^{1}S_{0.0}      /adf15/pec40][ar/pec40][ar_ic][ar16.dat
3.96731     Ar       15+     ADF15      11512513 ^{2}P_{1.5} -> 12 ^{2}S_{0.5}   /adf15/pec40][ar/pec40][ar_ic][ar15.dat
3.96832     Ar       15+     ADF15      11512513 ^{2}P_{0.5} -> 12 ^{2}S_{0.5}   /adf15/pec40][ar/pec40][ar_ic][ar15.dat
3.97233     Ar       15+     ADF15      11512513 ^{2}P_{2.5} -> 12 ^{2}S_{0.5}   /adf15/pec40][ar/pec40][ar_ls][ar15.dat
3.97612     Ar       15+     ADF15      11512513 ^{2}P_{2.5} -> 12 ^{2}S_{0.5}   /adf15/pec40][ar/pec40][ar_ls][ar15.dat
3.98011     Ar       15+     ADF15      11512513 ^{2}P_{1.5} -> 12 ^{2}S_{0.5}   /adf15/pec40][ar/pec40][ar_ic][ar15.dat
3.98225     Ar       15+     ADF15      11512513 ^{2}P_{0.5} -> 12 ^{2}S_{0.5}   /adf15/pec40][ar/pec40][ar_ic][ar15.dat
3.99122     Ar       15+     ADF15      11512513 ^{0}S_{11.5} -> 12 ^{0}S_{0.5}  /adf15/pec40][ar/pec40][ar_cl][ar15.dat
3.99122     Ar       15+     ADF15      11512513 ^{0}S_{11.5} -> 12 ^{0}S_{0.5}  /adf15/pec40][ar/pec40][ar_ca][ar15.dat
4           Ar       16+     ADF15       ^{3}S_{1.0} ->  ^{1}S_{0.0}             /adf15/transport/transport_llu][ar16.dat
4           Ar       16+     ADF15       ^{3}P_{1.0} ->  ^{1}S_{0.0}             /adf15/transport/transport_llu][ar16.dat

Codecov Report

Merging #508 (fd820c9) into devel (4655c76) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            devel     #508   +/-   ##
=======================================
  Coverage   55.27%   55.27%
=======================================
  Files          48       48
  Lines       22113    22113
=======================================
  Hits        12222    12222
  Misses       9891     9891

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 4655c76...fd820c9. Read the comment docs.


✅ Release 1.0.2

Details

Update index.mjs with named export

Tasks

[ ] ⚙ Update index.mjs with named export

Since we are going to use shared/env.js structure, export style is not important for each application.
Stop to release 1.0.2


[Example] Collaborative Block Text Editor

Adding a new example to demo how Liveblocks can be used to create a collaborative block-based text editor similar to products like Notion and Clover.

Amazing work! 👏
I found a few issues:

Offset is off when splitting a block after formatted text
Merging block erase formatting

See video for the two issues
https://user-images.githubusercontent.com/7273466/180910104-5fff49ff-7335-4785-bc6e-bb7becef9987.mov
Did you consider using a simple loop to go over each character to compute the offset in getHtmlIndexPositionFromInnerTextPosition instead of a regex?

Found a tiny bug:
When focusing the title, the user's focus on the text is not cleared
https://user-images.githubusercontent.com/6947265/181034782-013e8289-e000-4ab6-b2cc-f5ceae6eabe4.mp4


fix: powerbi-docs/desktop-python-in-query-editor.md

Add code fence language

@nschonni : Thanks for your contribution! The author, @otarb, has been notified to review your proposed change.

We're in the process of pulling this commit to the private repository so we can verify the updates on staging. We'll post a new private repo PR # shortly and then close this PR.

@GitHubber17 : Thanks for your contribution! The author, @otarb, has been notified to review your proposed change.

Confirmed code fence identifier. Thanks