Improve README

As a new user, I'd like to understand what I am installing and how it is configured to do what I want.
There are two things that could be improved in the README:

correct the description of koverage.coverageFilePaths (it's currently the same as koverage.coverageFileNames)
better describe the Features of the extension

For the features, I'd appreciate screenshots of how it would look like to better understand what is being provided by the extension.

@FieteO Sorry for the late reply, life and work are keeping very busy !
I'm open to PR, if you have an idea/time to get things more clear.

@FieteO Sorry for the late reply, life and work are keeping very busy.
I'm open to PR if you have an idea/time to get it more clear.

Had a look, the repo was in a bad state (2 years of commits were gone 😄  ). Got it fixed. (BTW the ReadMe was fine in vscode, in the extension page)


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

This work has been completed with PR #8.


Test: tab opening blocked dialog

Refs: https://github.com/microsoft/vscode/issues/118731

[x] macOS @bpasero
[x] macOS @Tyriar

Complexity: 3
Create Issue

Safari automatically blocks opening of new tabs, when we detect that a new tab or window opening has been blocked we now show a dialog on which the user can react and allow for the tab or window to get opened.
Do the following:

Use VSCode and verify opening new windows or clicking on links in editor does not show any unexpected dialog and everything works as before
Open our web bits in Safari or use codespaces and verify:

Open a starter VS Code extension and verify you can debug it. Once you start debugging it you see the newly introduced dialog and clicking on "Continue" properly opens a new tab and you can debug
Clicking on a link in the editor also shows this new dialog, clicking on "continue" properly opens a new tab
Do an action that results in a folder, workspace or file to open in a new window
Try to be creative with commands that open new tabs or new windows and verify that you see this dialog any time Safari blocks opening. Verify you do not see this dialog when it is not expected

Brilliant 👍


Kong File Log Rolling Appender

Summary
So I wonder whether is that possible to create log file but with a rolling appender in which the log will be written into a new file whenever the specified parameter has been reached ?
Thank you

Hi,
No, the logging plugin would currently not support such a feature. Contributions weelcomed!


Avoid redundant onDidAddPaneItem notifications when items are move between panes

/cc @steelbrain
Refs #9012

:heart: Thanks you for this!

Failure is a license check issue, which has been resolved on master.

:zap: fast fix.


User creation fails when just ~10 users created

This pull request just includes the test that reproduces the bug, for reporting. Right fix of code and tests will follow.
Fail log: https://gist.github.com/kuenishi/cf174e2bf3d796fcc378

Is this a bug of test code or of source code?

I still don't know. Needs investigation.

According the fail log, the test failed at line 41. That means admin user test failed before running 10 users test.

Could not reproduce.


[new] 兼容pytorch的DataLoader，替换Batch为DataSetIter

Description：兼容pytorch的DataLoader，替换Batch为DataSetIter

[new] DataSetIter, TorchLoaderIter, BatchIter
[remove] Batch
[change] Trainer 的API

Main reason: 使Trainer更快也更易使用
@修改过这个文件的人
@核心开发人员

Codecov Report

Merging #166 into dev0.5.0 will increase coverage by 0.27%.
The diff coverage is 87.97%.

@@             Coverage Diff              @@
##           dev0.5.0     #166      +/-   ##
============================================
+ Coverage     64.68%   64.95%   +0.27%
============================================
  Files            88       88
  Lines          9083     9103      +20
============================================
+ Hits           5875     5913      +38
+ Misses         3208     3190      -18

Impacted Files
Coverage Δ

fastNLP/__init__.py
100% <ø> (ø)
:arrow_up:

test/models/model_runner.py
98.75% <100%> (ø)
:arrow_up:

fastNLP/core/__init__.py
100% <100%> (ø)
:arrow_up:

test/core/test_trainer.py
95.71% <100%> (ø)
:arrow_up:

test/core/test_callbacks.py
100% <100%> (ø)
:arrow_up:

test/core/test_batch.py
99.1% <100%> (-0.9%)
:arrow_down:

fastNLP/core/predictor.py
73.17% <100%> (ø)
:arrow_up:

test/test_tutorials.py
100% <100%> (ø)
:arrow_up:

fastNLP/modules/encoder/embedding.py
21.33% <50%> (ø)
:arrow_up:

fastNLP/core/tester.py
85.88% <62.5%> (-2.87%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 87.11656% with 21 lines in your changes missing coverage. Please review.

Project coverage is 64.96%. Comparing base (1a4c3c2) to head (4b5113c).

Files with missing lines
Patch %
Lines

fastNLP/core/batch.py
88.54%
11 Missing :warning:

fastNLP/core/trainer.py
73.68%
5 Missing :warning:

fastNLP/core/tester.py
62.50%
3 Missing :warning:

fastNLP/core/field.py
80.00%
1 Missing :warning:

fastNLP/modules/encoder/embedding.py
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           dev0.5.0     #166      +/-   ##
============================================
+ Coverage     64.68%   64.96%   +0.27%
============================================
  Files            88       88
  Lines          9083     9107      +24
============================================
+ Hits           5875     5916      +41
+ Misses         3208     3191      -17

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


[ISSUE #6445] Fix the attribute of the compaction topic doc

Make sure set the target branch to develop
What is the purpose of the change

fix #6445
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

I will close this pr because It has be fixed


Adding "addvl" instruction for efficient stack pointer calculation

In order to manipulate RVV stack (prologue, epilogue, stack reference, register alignment, function argument in stack) efficiently, we need a scalar "addvl" instruction to add an integer multiple of vlenb value to stack pointer. Without this instruction, we need several instructions to calculate the stack pointer value and need to allocate several scalar tmp registers for this calculation which may lead to extra register save/restore code.
Example 1: allocate RVV stack in prologue:
csrr  t0, vlenb
addi  t1, zero, 7
mul t0, t0, t1
sub sp, sp, t0
Example 2: reference vector stack object, offset #7
addi a0,sp,#unscaled_offset
csrr  a1, vlenb
addi  a2, zero, 7
mul a2, a1, a2
add a0, a0, a2
vl1r.v  v24, (a0)
Is it possible to add such an instruction for efficient RVV stack support?

+1, that will significantly simplify the code gen for spilling vector register.

The basic form of "addvl" instruction should use the vlenb CSR as an implied source operand. And I think whether LMUL needs to be considered as another implied source operand can be discussed.

The basic form of "addvl" instruction should use the vlenb CSR as an implied source operand.

I disagree.
Since the operation is defined as sp+=( VLEN / 8) * (imm or rs1), and vlenb is defined as VLEN /8 there is a correlation, but the Uarch can determine if it wants to read vlenb or pull the bits from an internal VLEN representation.

And I think whether LMUL needs to be considered as another implied source operand can be discussed.

I don't see any need for LMUL to be factored into the operation . The code needs to know LMUL to operate. It can then dynamically scale rs1 or at compile time the imm.

we need a scalar "addvl" instruction to add an integer multiple of vlenb value to stack pointer.

Adding to sp does not help with Example 2.
The problem you are stating is the difficulty deriving a multiple of vlenb to add/subtract from a register, especially sp.
Since vlenb is a Read Only CSR, the csrrw and csrrwi are unused.
These variants could be used to request a multiplier for vlenb.
Thus examples 1 and 2 become.
Example 1: allocate RVV stack in prologue:
csrrwi t0, vlenb, 7
sub sp, sp, t0
Example 2: reference vector stack object, offset #7
csrrwi a0, vlenb, 7
addi a0,a0,#unscaled_offset
add a0,a0,sp,
vl1r.v v24, (a0)
Note: csrrwi on vlenb will likely be assigned an alias (maybe vlenxi and vlenx?)
Some may object to this unorthodox use of csrrw, however, such use is planned for fast interrupt TG and been floated for other uses as well.

I am sorry that I did not describe the "addvl" instruction envisioned clearly. What I am actually thinking about the addvl instruction has the following syntax and functionality.
"addvl  rd, rs1, signed_imm_6bit"
"rd = rs1 + (signed_imm_6bit * vlenb)"
So it should be useful for example 2 as well. But it seems that I cannot find an existing instruction format with 6-bits of immediate values.
Your proposal of using "csrrwi rd, vlenb, unsigned_imm_5bit" with the function of "rd = unsigned_imm_5bit * vlenb" is a good idea.

I'm not much of a fan of the "magic CSR" approach.  It creates an awkward special case, since many styles of implementation perform CSR accesses at commit, whereas here we want to execute the instruction earlier in the pipeline.  It also adds a special case to the illegal-instruction detection logic, since currently all CSRs with address bits 11:10=3 trap on writes.

Then we can define 2 instructions in the OP opcode space as follows:
"addvl rd, rs1, uimm5" --> "rd = rs1 + (uimm5 * vlenb_const)" // when rs1=x0, rd = uimm5 * vlenb_const
"subvl rd, rs1, uimm5" --> "rd = rs1 - (uimm5 * vlenb_const)" // when rs1=x0, rd = - (uimm5 * vlenb_const)

TG should discuss whether to add such instructions, but if they are approved, I think your signed-immediate version is fine.

On Thu, Aug 6, 2020, 04:36 Andrew Waterman<EMAIL_ADDRESS>wrote:

I'm not much of a fan of the "magic CSR" approach.

See below

It creates an awkward special case, since many styles of implementation
perform CSR accesses at commit, whereas here we want to execute the
instruction earlier in the pipeline.

For vlenb we also want early CSR pipeline access. No stall if we want ooo
and vector :vl, vtype, etc. to interleave.
It also adds a special case to the illegal-instruction detection logic,

since currently all CSRs with address bits 11:10=3 trap on writes.

I had thought that this issue could be deferred until after v1.0. But your
concern is valid and we may best carve out a section on 11:10=3 for csrrw
csrc/s allowed variants.
The original scheme is valid but as with debug in S and H it must evolve.
Magic case.
There is a camp that is adverse to  creating awkward special case
instructions with multiple ways of accessing CSRs, registers that exist
because of special handling raising such issues as already mentioned,
including synchronization. Better to keep the complexity out of the
"normal" instructions.
vsetlv is an intentional exception. It is so prevalant that it needs to be
understood and corner cases addressed.
I fear addvl would not get that scruitiny.
But we are otherwise not in a rush to resolve it. V1.1 s ok.
But the vector CSR mapping we want to get right for v1.0
—

You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-v-spec/issues/544#issuecomment-669793582,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAWIKJFWXGJ2D3BLJ2WTZDR7JTQFANCNFSM4PPV3DPA
.

Have you worked out what this would look like with a larger example?
This code snippet is to save or load a single vector register. Normally,
the code would be saving/restoring multiple registers. This takes a long
time to execute. For each vector register saved/loaded, a new base address
needs to be computed. I don't think you would use addvl to compute the
address from scratch each time, when you can simply do a subi/addi to
modify sp each time.
Instead, perhaps we need vload/vstore instructions that compute the
required effective address using the register number? For example:
vl1r.v vX,(a0)
where EffectiveAddress = a0 + (vlenb * X)
The "*X" is a variable shift.
Guy
On Fri, Jul 31, 2020 at 12:41 AM chuanhua<EMAIL_ADDRESS>wrote:

In order to manipulate RVV stack (prologue, epilogue, stack reference,
register alignment, function argument in stack) efficiently, we need a
scalar "addvl" instruction to add an integer multiple of vlenb value to
stack pointer. Without this instruction, we need several instructions to
calculate the stack pointer value and need to allocate several scalar tmp
registers for this calculation which may lead to extra register
save/restore code.
Example 1: allocate RVV stack in prologue:
csrr t0, vlenb
addi t1, zero, 7
mul t0, t0, t1
sub sp, sp, t0
Example 2: reference vector stack object, offset #7
https://github.com/riscv/riscv-v-spec/issues/7
addi a0,sp,#unscaled_offset
csrr a1, vlenb
addi a2, zero, 7
mul a2, a1, a2
add a0, a0, a2
vl1r.v v24, (a0)
Is it possible to add such an instruction for efficient RVV stack support?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-v-spec/issues/544, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMAPWPD4E726MHKOEEOI7TR6JYUBANCNFSM4PPV3DPA
.

Moved to post v1.0.


Blob Trigger should let me change my connection

Right now, Blob trigger isn't showing the connection selector UX. It should allow me to choose a different storage account than the default.

Note that this storage connection isn't actually currently plumbed into the runtime - the runtime ignores the value. See https://github.com/Azure/azure-webjobs-sdk-script/issues/55


Initialized variables and other minor issues presented by Cppcheck in 5 files

Initialized class variables inside core/variant/variant.h

Created initialization lists for variables assigned in the constructor's body inside editor/plugins/visual_shader_editory_plugin.h , modules/lightmapper_rd/lightmapper_rd.h , and modules/mono/editor/bindings_generator.h

Added breaks and default case to switch statement inside of modules/mono/editor/GodotTools/GodotTools.ProjectEditor/IdentifierUtils.cs

Likely Hacktoberfest spam, closing.

Likely Hacktoberfest spam, closing.

Hi, this was not Hacktoberfest spam. Is there something I have to modify to get this approved?

Is there something I have to modify to get this approved?

No, there's nothing salvageable in this PR. Fixing the static analyzer reports might look like an easy task, but in reality actual issues are likely already fixed and most of cppcheck reports are false-positives.

Initialized class variables inside core/variant/variant.h

It's union, only one member of union can be initialized, and is already done in the _mem ({0}), so this change wrong.

Added breaks and default case to switch statement inside

This change is absolutely wrong, these are fall-through cases and original code is correct and do not need any changes.

Created initialization lists for variables assigned

Arbitrary style change, we allow both styles and changing few random initializers does not make any sense.


Consider making oio::Write take a pointer to the data

I have a layer that implements a local fs cache for any service, the main data structure being a TeeReader. Porting it to be async will be a nice ergonomics lift, however the new trait consumes the bytes when writing which feels awkward. Prior art:

https://docs.rs/futures/latest/futures/io/trait.AsyncWrite.html
https://doc.rust-lang.org/std/io/trait.Write.html

Is it too late to refactor these to take a &Bytes instead (or even just a &[u8])? Same applies for Read and &mut [u8]
/// A reader impl that reads from a source while writing to a sink
///
/// We cannot assume that multiple calls to poll_read will be given
/// the same buffer, meaning we cannot use that to store data
/// between reads and writes.
pub struct TeeReader<Src, Sink> {
    src: Src,
    sink: Sink,
    outstanding_write: Option<(usize, usize)>,
    buffer: Vec<u8>,
}

impl<Src, Sink> TeeReader<Src, Sink> {
    pub fn new(src: Src, sink: Sink) -> Self {
        Self {
            src,
            sink,
            outstanding_write: None,
            buffer: vec![0; 4096],
        }
    }
}

impl<Src: Read, Sink: Write> Read for TeeReader<Src, Sink> {
    fn poll_read(
        &mut self,
        cx: &mut std::task::Context<'_>,
        buf: &mut [u8],
    ) -> std::task::Poll<Result<usize>> {
        if let Some((idx, total)) = self.outstanding_write {
            match self.sink.poll_write(cx, &&self.buffer[idx..total]) {
                Poll::Ready(Ok(written)) => {
                    let idx_next = idx + written;
                    if idx_next == total {
                        self.outstanding_write = None;
                        buf[0..total].copy_from_slice(&self.buffer[0..total]);
                        Poll::Ready(Ok(total))
                    } else {
                        self.outstanding_write = Some((idx_next, total));
                        Poll::Pending
                    }
                }
                Poll::Ready(Err(e)) => Poll::Ready(Err(e)),
                Poll::Pending => Poll::Pending,
            }
        } else {
            match self.src.poll_read(cx, &mut self.buffer) {
                Poll::Ready(Ok(0)) => Poll::Ready(Ok(0)),
                // we have read some bytes, initiate a write
                Poll::Ready(Ok(n)) => {
                    self.outstanding_write = Some((0, n));
                    self.poll_read(cx, buf)
                }
                Poll::Ready(Err(e)) => Poll::Ready(Err(e)),
                Poll::Pending => Poll::Pending,
            }
        }
    }

    fn poll_seek(
        &mut self,
        cx: &mut std::task::Context<'_>,
        pos: std::io::SeekFrom,
    ) -> std::task::Poll<Result<u64>> {
        self.src.poll_seek(cx, pos)
    }

    fn poll_next(
        &mut self,
        cx: &mut std::task::Context<'_>,
    ) -> std::task::Poll<Option<Result<bytes::Bytes>>> {
        self.src.poll_next(cx)
    }
}

New impl for comparison. This has at least two allocations per call to read compared to none in the previous impl.
/// A reader impl that reads from a source while writing to a sink
pub struct TeeReader<Src, Sink> {
    src: Src,
    sink: Sink,
}

impl<Src, Sink> TeeReader<Src, Sink> {
    pub fn new(src: Src, sink: Sink) -> Self {
        Self { src, sink }
    }
}

impl<Src: Read, Sink: Write> Read for TeeReader<Src, Sink> {
    async fn read(&mut self, limit: usize) -> Result<bytes::Bytes> {
        let data = self.src.read(limit).await?;
        self.sink.write(data.clone()).await?;
        Ok(data)
    }

    fn seek(
        &mut self,
        pos: std::io::SeekFrom,
    ) -> impl futures::prelude::Future<Output = Result<u64>> + Send {
        self.src.seek(pos)
    }
}

Thank you for initiating this discussion. I'm currently working on the new API design and will return to this thread once completed.

Ok, I got a brilliant idea. Please stay tuned.

Tracked at https://github.com/apache/opendal/discussions/4402


Anything else important to do?

Aside from creating a module, I believe that we have everything important done (without starting anything major, e.g. code for proper testing).

If the answer is no, then the main thing to do would be a final pass for bugs, readability, comments, user friendliness etc.
I'm also gonna add a small section which allows the option of outputting as a csv for use in a different plotting program.

I cc'd you in the email so you can have a read of what i sent

If someone has a bit of time before the deadline, perhaps add a small section that displays the output automatically.
Or maybe change the location it saves to, to somewhere more useful e.g. pictures

i think he changed the deadline for tuesday so I can do some of these small things now

while you do, be sure to run checks every so often, just to make sure you haven't accidentally broken anything (it happens to everyone).

I believe that there is no longer anymore work to be done, I think its safe to assume that this project has finished. Thanks to everyone who contributed!


Any elegant way to get SQLAlchemy result objects JSONencoded?

Description
I would like to get Custom Objects JSON encoded by the connexion framework automatically. ATM i am trying to fiddle around with CustomJSONEncoders, but i don't find the right spot to hook them into connexion.
Expected bahaviour
A call to e.g.:
INFO:werkzeug:<IP_ADDRESS> - - [20/May/2016 19:33:46] "GET /v0/floor/1/rooms HTTP/1.1" 500 -
pointing to:
def get_floor_rooms(floor_id):
rooms = FloorRoomListObject.query.filter_by(floor_id=floor_id)
return [room for room in rooms]
should directly work out of the box without raising the error below
Actual behaviour
Traceback (most recent call last):
File "/Library/Python/2.7/site-packages/flask/app.py", line 1836, in call
return self.wsgi_app(environ, start_response)
File "/Library/Python/2.7/site-packages/flask/app.py", line 1820, in wsgi_app
response = self.make_response(self.handle_exception(e))
File "/Library/Python/2.7/site-packages/flask/app.py", line 1403, in handle_exception
reraise(exc_type, exc_value, tb)
File "/Library/Python/2.7/site-packages/flask/app.py", line 1817, in wsgi_app
response = self.full_dispatch_request()
File "/Library/Python/2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/Library/Python/2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/Library/Python/2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
rv = self.dispatch_request()
File "/Library/Python/2.7/site-packages/flask/app.py", line 1461, in dispatch_request
return self.view_functionsrule.endpoint
File "/Library/Python/2.7/site-packages/connexion/decorators/validation.py", line 251, in wrapper
response = function(_args, *_kwargs)
File "/Library/Python/2.7/site-packages/connexion/decorators/produces.py", line 128, in wrapper
data = [json.dumps(data, indent=2), '\n']
File "/Library/Python/2.7/site-packages/flask/json.py", line 126, in dumps
rv = _json.dumps(obj, **kwargs)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/init.py", line 250, in dumps
sort_keys=sort_keys, **kw).encode(obj)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/encoder.py", line 209, in encode
chunks = list(chunks)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/encoder.py", line 431, in _iterencode
for chunk in _iterencode_list(o, _current_indent_level):
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/encoder.py", line 332, in _iterencode_list
for chunk in chunks:
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/encoder.py", line 442, in _iterencode
o = _default(o)
File "/Library/Python/2.7/site-packages/connexion/decorators/produces.py", line 46, in default
return json.JSONEncoder.default(self, o)
File "/Library/Python/2.7/site-packages/flask/json.py", line 83, in default
return _json.JSONEncoder.default(self, o)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/encoder.py", line 184, in default
raise TypeError(repr(o) + " is not JSON serializable")
Steps to reproduce
Get a result set from postgres using SQLAlchemy and try to return it in the view.
Additional info:
Output of the commands:

python --version
Python 2.7.10
pip show connexion | grep "^Version\:"
Version: 1.0.97

Ok found that in the official doc...will try :)
Connexion allows you to customize the JSONEncoder class in the Flask app instance json_encoder (connexion.App:app). If you wanna reuse the Connexion’s date-time sezialization, inherit your custom encoder from connexion.decorators.produces.JSONEncoder.

How are your room objects defined?

Pretty Straight forward i would say ;)
from sqlalchemy import Column, Integer, String
from database import Base
class FloorRoomListObject(Base):
tablename = 'rooms'
id = Column(Integer, primary_key=True)
floor_id = Column(Integer)
name = Column(String())
frontend_css_position_x = Column(Integer)
frontend_css_position_y = Column(Integer)
frontend_css_position_h = Column(Integer)
frontend_css_position_w = Column(Integer)
def __init__(self, floor_id=None, name=None, frontend_css_position_x=None, frontend_css_position_y=None, frontend_css_position_h=None, frontend_css_position_w=None ):
    self.floor_id = floor_id
    self.name = name
    self.frontend_css_position_x = frontend_css_position_x
    self.frontend_css_position_y = frontend_css_position_y
    self.frontend_css_position_h = frontend_css_position_h
    self.frontend_css_position_w = frontend_css_position_w

def __repr__(self):
    return self

but i replaced the return value in repr already with dicts and stringified dicts. But sicne i am totally new to python and flask/connection and i didn't have time to look into the inner workings of serialization etc. But i will try the documented approach...

I'm really a beginner myself with Flask, Connexion and API's in general. Working with SQLite I found that, as long as you're using numbers and strings only, it's sufficient to copy dict to repr without any attribute whose name begins with underscore.

Hmm, did that...but did't work for me...Maybe the underscores in the names 'frontend_css_position_x' are the problem here...

will camelcase everything and lets see ;)

@basisklasse if you have a minimal example with  SQLAlchemy working we could add it to our Connexion examples folder, I think it would be useful for many people :smile:

Ok, i think i have a nice solution.
I added the flask-jsontools extension:
from flask.ext.jsontools import DynamicJSONEncoder
set up SQLAlchemy to use declarative extension with Serializable Base
from sqlalchemy import create_engine
from sqlalchemy.orm import scoped_session, sessionmaker
from sqlalchemy.ext.declarative import declarative_base
from flask.ext.jsontools import JsonSerializableBase

engine = create_engine('postgresql://xxx:yyy@localhost:5432/mydatabase, convert_unicode=True)
db_session = scoped_session(sessionmaker(bind=engine))

Base = declarative_base(cls=(JsonSerializableBase,))
Base.query = db_session.query_property()

And the models just return themselves as repr:
def __repr__(self):         return self
Works like a charm for normal fields, but you have to add own encoding for specific datetime etc stuff...

@hjacobs I created a variant of your own connexion-example project that uses SQLAlchemy. Note that I really am a beginner with both SQLAlchemy and Connexion and it is still very much work in progress, but you may check it out here.

@nmusatti cool, thanks! I just checked it out :smile: --- can you do a PR to put your example in examples/sqlalchemy ?

@hjacobs Certainly. I'll just clean it up and streamline it a bit.

We now have a simple SQLAlchemy example: https://github.com/zalando/connexion/tree/master/examples/sqlalchemy
Thanks to @nmusatti :smile:

Ohh no, and the link is dead... @hjacobs

The current link is https://github.com/zalando/connexion/tree/master/examples/swagger2/sqlalchemy


test/040924_2217_review_without_body

🎉 Snyk hasn't found any issues so far.
✅ security/snyk check is completed. No issues were found. (View Details)


limit / offset should auto try cast to int

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

select 1 offset '0'
Parsing this SQL using JSqlParser with this statements
Exception

org.junit.ComparisonFailure: Expected query to parse but it failed to parse (error: Encountered unexpected token: "\'0\'" <S_CHAR_LITERAL>
    at line 1, column 19.

Expected behavior
query to parse fine
System

postgres
Java 11
JSqlParser 4.2

Character literals are not allowed as parameter there. Using an integer should do fine.

Duplicate of #933


Update Guide-Headers.md

Adding some information to the section on 'Header interaction with screen component'. In my app which has ~30kLOC I have only one setParams call because I use mobx for state management, meaning I instantiate an object with all the observable data and functions a component needs, and then navigate to that screen, passing the object through navigation.state.
Then, both static navigationOptions block and the component render() method have access to the object, and I do not need to use setParams. This is also a benefit for testing.
I feel like this should be mentioned in the docs, and I want to ask some redux user (since I mentioned redux in the text) if what I wrote makes sense to them - if it is possible to work with redux like this (I use mobx so I don't know). Thanks!

LGTM, maybe add a link to the libraries?
https://github.com/mobxjs/mobx
https://github.com/reactjs/redux

I have added links to HOC docs, mobx and redux

Awesome, looks good. Let's ship it 🚀

Thanks @vonovak & @matthamil 💪


Loop function / Active Class

Loop option added to allow the text to cycle around for a set amount amount of loops.
Active class added on plugin initialised. The class can be used to hide the words prior to plugin initialization and avoid the flash of all the words.

More ideas forth coming.

Apologies for not getting back to you sooner. I'll have a look this week.


CI: v1.8: Suite-k8s-1.11.K8sVerifier Runs the kernel verifier against Cilium's BPF datapath

/home/jenkins/workspace/Cilium-PR-K8s-1.11-net-next/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:461
test-verifier pod not ready after timeout
Expected
    <*errors.errorString | 0xc0003dd6a0>: {
        s: "timed out waiting for pods with filter test-verifier to be ready: 4m0s timeout expired",
    }
to be nil
/home/jenkins/workspace/Cilium-PR-K8s-1.11-net-next/src/github.com/cilium/cilium/test/k8sT/Verifier.go:58

Seen on #14015: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.11-net-next/1473/ - test_results_Cilium-PR-K8s-1.11-net-next_1473_BDD-Test-PR.zip
Also occured on v1.8 upstream branch at least two times:
https://jenkins.cilium.io/view/Cilium-v1.8/job/cilium-v1.8-k8s-1.11-kernel-net-next/726/ - test_results_cilium-v1.8-k8s-1.11-kernel-net-next_726_BDD-Test-PR.zip
https://jenkins.cilium.io/view/Cilium-v1.8/job/cilium-v1.8-k8s-1.11-kernel-net-next/722/ - test_results_cilium-v1.8-k8s-1.11-kernel-net-next_722_BDD-Test-PR.zip
/cc @pchaigno

I think I know the fix. We need to extend the tolerations for the test-verifier pod to include node.kubernetes.io/unreachable.

@tklauser Not 100% sure on the fix in #14074, so worth keeping an eye for this flake in case we need to reopen.

@pchaigno Thanks, will help to keep an eye on it.


Remove azureml-contrib-interpret

Updating model interpretability doc to remove instances and code referencing a deprecated contrib package azureml-contrib-interpret as all functionality will now move to the main azureml-interpret package. PR here: https://msdata.visualstudio.com/Vienna/_git/AzureMlCli/pullrequest/432499

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


Feature: ngTrueValue should accept non-string values

For ng-model checkboxes, ngTrueValue only supports string. It should really just accept an angular expression and eval to get the value so it supports say a js object.

@eekboom
When I was migrating from 1.2, I switched to following:
ng-true-value="{{proxyOnValue}}"

to
ng-true-value="'{{proxyOnValue}}'"

I just had an interesting realization. What I was trying to do - and from the looks of it, what a lot of you have been trying to do - can actually be accomplished with:
ng-true-value="{{ myVariable | json }}"

This supports string values, booleans and numbers. Not sure yet whether I actually like this, but I certainly found it entertaining. And maybe this will help somebody else.

getu-lar i've tried what you said, with a complex object, it works.  So, many thanks!

getu-lar 's solution worked for me. Thanks.

I was trying to use an expression which resolved to a string with underscore in it, e.g. "101_3". I used getu-lar 's solution and it simply works. Feels like a hack though. Thanks for sharing it !

+1 for expressions

+1

It doesn't seem like there is much hope for Angular team to provide a sane checkbox directive for 1.x series... It can't be too difficult. Expression support would be fully backwards compatible due to the mandatory quoting.
Until then the suggested workaround is working for me:
ng-true-value="{{ myVariable | json }}"

Ensure that myVariable is defined in the controller. I tried to define it in post-link and it did not work.


Paket.Core: honor DOTNET_ROOT env var [note: UNTESTED, DO NOT MERGE yet]

Fixes https://github.com/fsprojects/Paket/issues/4141

It doesn't build because ProcessHelper.fs and Utils.fs would have a cyclic dependency after this change :'-( any suggestion is appreciated.

Why do you need to move it above?


The terminal can't input full width Chinese characters(in win10, Windows Termial with powershell or cmd)

The terminal can input Chinese words, but can't input  full width Chinese characters while using powershell.exe or command prompt.exe

@LingYiLing you still have to give us more info than that.

What version of Windows are you using?
Are you using the Windows Terminal, or the vintage console?
If you're using the Terminal, what version of the Terminal are you using?
What version of PowerShell? What version of PSReadline (if enabled)?
Which IME are you using to input characters?
What happens when you try inputing chinese characters? A screenshot would be helpful here. Protip: you can just paste images in this textbox on GitHub and it will upload and link the image for you ☺

I'm using Windows Terminal, version: 1.5.10271.0

The versions and problem is below

IME I using is 微软拼音

Ah, gotcha. So the issue is that the full-width  characters aren't working wile you're using ssh. That sounds like an issue with ssh.exe, you should probably report this on their repo. Their issue tracker can be found here: https://github.com/PowerShell/Win32-OpenSSH/issues


Command line interface

A cabinetry command line interface should allow to run core tasks from the command line. It should also be possible to overwrite configuration arguments from the command line (possibly only a subset).
The interface could be built with libraries such as click or typer (which uses click).

#81 implements a basic CLI. It does not include functionality to override configuration file parameters from the command line yet.

The CLI has been further expanded and is documented on readthedocs. The functionality implemented in #143, which allows reading configuration files from stdin, partially addresses overriding configuration arguments via the command line (for example by piping through sed). Additional expansion of the CLI capabilities in this regard is out of scope for now.


Does the LCOW VM need to be licenced on Windows Server Standard?

A tricky licencing question:

Since even process-isolated Linux containers consume a VM together, does that VM count towards the 2 VM's per licence-pack for Windows Server Standard?  Or is that VM 'thrown in for free' since it is incidental infrastructure to achieve an outcome equivalent to process-isolated Windows containers?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a92c8c80-59f8-8cc4-3da4-82e4c6df9b0d
Version Independent ID: 55b07147-4154-0daf-72a7-7b412d572912
Content: Windows Containers FAQ
Content Source: virtualization/windowscontainers/about/faq.md
Service: windows-containers
Product: windows-containers
GitHub Login: @PatrickLang

I guess this question is assuming that every VM - whether occupied by a VM running Windows Server or not - might consume a licence.  I'm not entirely sure this is the case.  But anyhow, if the licencing of LCOW in both the process-isolated and hyper-v-isolated scenarios could be clarified somewhere official ... that would be helpful..

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


refactor(logger): test logger with '--loglevel' flag support

append '-- --loglevel=debug' in CLI to get log outputs in the console (in debug mode)
Signed-off-by: Xavier Coulon<EMAIL_ADDRESS>

Codecov Report

Merging #60 (8bcc766) into main (95ceed8) will decrease coverage by 1.73%.
The diff coverage is 48.57%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   79.58%   77.84%   -1.74%
==========================================
  Files          14       15       +1
  Lines         813      835      +22
==========================================
+ Hits          647      650       +3
- Misses        117      135      +18
- Partials       49       50       +1

Impacted Files
Coverage Δ

pkg/logr/logger.go
48.14% <31.57%> (-51.86%)
:arrow_down:

testsupport/logger.go
68.75% <68.75%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


stratified code update 7/27

Shane - please see the revised code that addresses your comments. Responses are included to the comments. We also addressed the comments relating to pulling the runoff functions into the .py files.
The nested hyetograph notebook can be deleted now. I renamed it _void_nested_hyetograph for now. Also the Runoff Notebook can be deleted per Mark's update.

Hydro4_samples.ipynb:
- The inputs description is still the same description copied from EventsTable.ipynb, should be updated.

PrecipTable.ipynb:
- Want to add an if statement when getting the list of zip files, where if you are calculating nested hydrographs you want to add the following to the list comprehension: ('l.zip' not in d) and ('u.zip' not in d)] else you want to use the original list comprehension. I would just add a variable under the Papermill cell that says 'nested_hyetograph =True', and then use that variable for your condition in your if statement. Same thing for saving the PrecipTable.xlsx and the final nrcs_PreciptTable.xlsx. Finally, no need to create the outputs directory twice.

EventsTable_Stratified.ipynb:

Looks much better!
Can remove: durations = [6, 12, 24, 96]
Could add more markdown to describe individual cells/for readability but looks good overall

durations

EventsTable_Stratified.ipynb:

Looks much better!
Can remove: durations = [6, 12, 24, 96]
Could add more markdown to describe individual cells/for readability but looks good overall

Thanks for adding a few more headers to the notebook for readability.

This PR adds the ability to calculate runoff hydrographs using NOAA Atlas 14 or NRCS nested hyetographs. Results are saved to .xlxs, dss, and json files, closing #45. Resolves #50 by adding legends to plots in EventsTable_Stratified.ipynb. There are still open issues, including the need to update the readme and Docs for this repository but those can be addressed later.


Not logical behavoir in joinWith and relation queries

I've got one model with relation one-to-many to another:
class Commodity extends ActiveRecord
{
     public function getDiscount()
     {
         return $this->hasOne(Discount::className(), [Discount::commodityId() => self::id()]);
     }
     public static function findForCurrentCompany()
     {
         return static::find()->...(conditions)...->joinWith('discount');
     }
     public function afterFind()
     {
          // Discount calculations for multiple prices
     }
}

What will happen:
Yii2 will execute join query, will populate Commodity models with data and call afterFind method.
In afterFind i will call to $this->discount and execute SELECT ... id IN() query and receive an error due to database limitations. Only after that Commodity models will be filled with Discound models, which is not logical - i expect afterFind in 'base' model called after filling with relations, not before.
Or is there anything i can use instead that i just don't know of?

If you want people over here to understand what your problem is, you should formulate your issue better. You are talking about multiple issues and a example of code which doesnt really say anything.
About afterFind and its chronology, take a look at the code over here:
https://github.com/yiisoft/yii2/blob/master/framework/db/ActiveQuery.php#L229
If you still think something is wrong, please repost all the code necessary to understand your issue, as slim as possible (without redundant/unnecessary code).


Make Slice Syntax Pythonic

Also, for naming specific tests, respect C4M_TEST_DIR when there isn't an absolute path provided.
Removed the terminal width assert.
Fixes #79

LGTM. Though I suppose to claim full "pythonic slice syntax", we'd also have to support e.g. x[::-1] and x[1::2].

No, I have no interest in the second colon.

Non-blocking question: I guess it's intended that we can no longer pass a relative path to c4test? I know I can still write either:

./dev run foo.c4m
cd debug && c4test foo.c4m

It's because the cwd when running the 'dev' script is where the binary is, to prevent any kind of confusion with testing.  As such, the test runner assumes "../test/". You can set CON4M_TEST_DIR.  Mine is just set to the test directory.
When I get to a real compiler command line, this will be moot, it's just for the test runner.

but I was previously writing e.g. debug/c4test tests/foo.c4m sometimes. I guess partly because that allows tab completion, and because ./dev run tests/foo.c4m doesn't work.


Use osm cli's bug-report command to collect logs in e2e framework

Please describe the Improvement and/or Feature Request
Use the osm support bug-report command to collect mesh related logs for CI failures (primarily e2e failures).
Currently, some log collection logic is baked into the e2e framework. If there are any gaps in the bug-report tool to triage e2e failures, address these gaps in the tool.
Scope (please mark with X where applicable)

New Functionality          [X]
Tests                      [X]

Possible use cases
Leverage osm bug-report tool for log collection in CI.

https://github.com/openservicemesh/osm/pull/4504


feat(ui): added Hero Button

I Have added the Hero Button
https://github.com/user-attachments/assets/9ccf6c13-bcd0-464c-a7f8-b99b9445221a
This is the Preview of the Button

Hi @NishantSinghhhhh Please link the associated issue in this PR. Thanks

@sudhashrestha ma'am,  the PR Number is #373

@hari  sir I have opened a new PR for the hero button , please have a look at it and review it , it is having all the changes that you requested


[RFE] Create Operator using Helm Chart

[ ] Create Helm Charts to simplify the deployment
[ ] Create an Operator based on the Helm Charts to publish on OperatorHub.io

This task has its own repo, temporarly available here: https://github.com/Trousseau-io/trousseau-operator
Current status is:

Helm Chart drafted with a successful deployment but a failed to connect to KMS due to worfklow issue
Helm Chart based Operator built with a successful deployment, CRD creating the trousseau.kms.trousseau.io resources but fails on existing resources

Helm charts generated for v2.
Potential backport could be done for v1.


WIP --enable= lll

Description of the change:

--enable= lll

Motivation for the change:

--enable= lll

Closes #2224

@I-am-Dishon Are you still planning to work on this? If so, can you please rebase this PR on the master branch. Seems like you've currently merged the master onto your branch.

please reopen when you have it rebased and ready to proceed.  I'll close this till then.


The cache file is too large

If a function is repeatedly called with different parameters, I find that the file in the cache directory still has only one file named after the function name. This leads to a huge file size when the function is called many times, so it reads
And the speed of rewriting is not as good as when the cache is not used.

Hmm. Understandable.
I never really got into dealing with cleaning old cache values, or with dealing with such cases.
Do you want to perhaps contribute code to solve this problem?

I think a straightforward solution is to create a different file for each parameter set used to call the function (can make this optional maybe). Would be happy to contribute if you think this is a good idea.

First, we need to make sure there's no performance issue here. At the moment, if I remember correctly, the single-file-per-function-scheme might make things faster by not opening and acquiring locks on different files. I'm not sure if that's the case, since the pickle cache has some intermediate layer in-memory, but there also a different thread opened for checking and reading from cache.
So, anyway, first thing is reading through the code and understanding how this affects the amount of disk reads.
Secondly, we need to benchmark the change (or lack-thereof, if believe this doesn't impede performance).
Finally, if this does work properly, we also need  a naming scheme that doesn't break for lots of parameters, or long name parameters. Maybe a constant
I would LOVE a contribution by you, @erap129 , so please go ahead with examining these issues.

actually I am trying to get started with the project, and I can't seem to get the tests running:
(cachier_env) C:\Users\eladr\Documents\GitHub\cachier>python -m pytest ImportError while loading conftest 'C:\Users\eladr\Documents\GitHub\cachier\tests\conftest.py'. tests\conftest.py:6: in <module>     from .test_mongo_core import _test_mongetter tests\test_mongo_core.py:71: in <module>     ??? cachier\core.py:128: in cachier     core = _MongoCore(mongetter, stale_after, next_time) cachier\mongo_core.py:47: in __init__     self.mongo_collection = self.mongetter() tests\test_mongo_core.py:56: in _test_mongetter     _test_mongetter.client = _get_cachier_db_mongo_client() tests\test_mongo_core.py:40: in _get_cachier_db_mongo_client     host = quote_plus(CFG[CfgKey.HOST]) ..\..\..\Anaconda3\envs\cachier_env\lib\site-packages\birch\core.py:276: in __getitem__     val = self._getitem_helper(key, self._val_dict) ..\..\..\Anaconda3\envs\cachier_env\lib\site-packages\birch\core.py:268: in _getitem_helper     raise KeyError("{}: No configuration value for {}.".format( E   KeyError: 'cachier: No configuration value for TEST_HOST.'
This happens also when running any non-mongo related tests. Do I need to add certain system variables?

No, that's ok. Mongo tests are supposed to work locally. They will be run by Travis when you submit a PR.
But it does seem like we'll have to set up some kind of memory-local mock so you can run all other tests.
Set the following environment variables with dummy values and let me know if that helps:

@erap129
It seems like you tried to fix test_delete_cache_file, however it still fails:
https://github.com/shaypal5/cachier/runs/3612547559?check_suite_focus=true
It also fails on master. Let's and figure this out.


Android: Trigger notification stopped working after restarting android phone. xiaomi device

Versions
"react-native": "0.71.4",
"react": "18.2.0",
"@notifee/react-native": "^7.7.0-0",

I have scheduled a notification every day at 5PM and I used to get everyday but after few days I had to restart my phone bcz of some reason and then after I never received any push notification, Waited for one more day for the same time but didn't receive notification.
Again created a similar notification every day bases but same problem again
I am testing this on Redmi note 10 device.
Please help me on this issue like if there is any configuration require to do or it's a bug?

What did adb logcat say when you sent the FCM?

This looks very critical bug.
I am using local scheduling notification completely.
const items = await notifee.getTriggerNotifications();
I have tested in real device by viewing JSON.stringify(items)
First case ->

Scheduled a notification at 7:55PM  and got the notification on time (as expected) and checked in the getTriggerNotifications list and the item removed. working fine

Scheduled a notification at 8:00PM now I restarted my android phone at 7:56PM and I checked the getTriggerNotifications list at 7:57PM by opening my app and the log was there as expected but At 8:00PM I didn't receive notification of 8:00PM.  I checked getTriggerNotifications list at 8:01PM and still the notification log was there (but it should have been removed).

What I can say that.  After restarting android device, notification not working and logs still there in

This looks very critical bug.
I am using local scheduling notification completely.
const items = await notifee.getTriggerNotifications();
I have tested in real device by viewing JSON.stringify(items)
First case ->

Scheduled a notification at 7:55PM  and got the notification on time (as expected) and checked in the getTriggerNotifications list and the item removed. working fine

Scheduled a notification at 8:00PM now I restarted my android phone at 7:56PM and I checked the getTriggerNotifications list at 7:57PM by opening my app and the log was there as expected but At 8:00PM I didn't receive notification of 8:00PM.  I checked getTriggerNotifications list at 8:01PM and still the notification log was there (but it should have been removed).

What I can say that.  After restarting android device, notification not working and logs still there in

What did adb logcat say at the time the notification should have triggered?

How can I test this 'adb logcat' in production build?

What did adb logcat say at the time the notification should have triggered?

Nothing showed on logcat at the time of trigger.

@helenaford
Notification scheduled for 12.00 13.00 14.00
Notification fired at 12.00 and 13.00 on time.
Reboot phone at 13.50 and waiting for the last notification.
Notification didn't trigger at 14:00

@helenaford
Notification scheduled for 12.00 13.00 14.00
Notification fired at 12.00 and 13.00 on time.
Reboot phone at 13.50 and waiting for the last notification.
Notification didn't trigger at 14:00

@ashishmangukiya there appears to be plenty of advice on the same on the wider internet:
https://search.brave.com/search?q=how+can+I+test+adb+logcat+on+production+build

This shows when my android phone restarted and logs for my android app without opening
2023-04-02 11:53:10.312  1824-2390  SystemServerTimingAsync system_server                        I  InitThreadPoolExec:Update app-ops uidState in case package com.routineplanner changed
2023-04-02 11:53:11.932  1824-1824  StorageManagerService   system_server                        V  Package com.routineplanner does not have legacy storage
2023-04-02 11:53:15.323  2959-3105  Launcher.Model          com.miui.home                        W  loadShortcut, id=254, intent=Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.routineplanner/.MainActivity }, userHandle=UserHandle{0}, type=0
2023-04-02 11:53:15.339  2959-3105  Launcher.Model          com.miui.home                        W  loadShortcut, after getShortcutInfoFromApplication, info=ShortcutInfo, id=254, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=-1, container=-1, cellX=-1, cellY=-1, spanX=1, spanY=1
2023-04-02 11:53:15.340  2959-3105  Launcher.Model          com.miui.home                        D  loadShortcut, bind item to launcher, item=ShortcutInfo, id=254, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=5, container=-100, cellX=3, cellY=1, spanX=1, spanY=1
2023-04-02 11:53:15.344  2959-2959  Launcher.CellLayout     com.miui.home                        D  updateCellOccupiedMarks, update, ShortcutInfo, id=254, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=5, container=-100, cellX=3, cellY=1, spanX=1, spanY=1
2023-04-02 11:53:16.131  2959-3105  SQLiteLog               com.miui.home                        W  (28) double-quoted string literal: "com.routineplanner/com.routineplanner.MainActivity#0"
2023-04-02 11:53:16.846  2959-3105  Launcher.AllAppsList    com.miui.home                        D  updatePackage, packageName=com.routineplanner, loadMissing=false, user=UserHandle{0}, matchedSize=1
2023-04-02 11:53:16.846  2959-3105  Launcher.AllAppsList    com.miui.home                        D  updatePackage, find appInfo=AppInfo, id=-1, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=-1, container=-1, cellX=-1, cellY=-1, spanX=1, spanY=1 from ComponentInfo{com.routineplanner/com.routineplanner.MainActivity}
2023-04-02 11:53:24.117  2959-3105  Launcher.Model          com.miui.home                        W  loadShortcut, id=254, intent=Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.routineplanner/.MainActivity }, userHandle=UserHandle{0}, type=0
2023-04-02 11:53:24.123  2959-3105  Launcher.Model          com.miui.home                        W  loadShortcut, after getShortcutInfoFromApplication, info=ShortcutInfo, id=254, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=-1, container=-1, cellX=-1, cellY=-1, spanX=1, spanY=1
2023-04-02 11:53:24.124  2959-3105  Launcher.Model          com.miui.home                        D  loadShortcut, bind item to launcher, item=ShortcutInfo, id=254, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=5, container=-100, cellX=3, cellY=1, spanX=1, spanY=1
2023-04-02 11:53:24.167  2959-2959  ActivityManagerWrapper  com.miui.home                        E  getRecentTasks: taskId=9   userId=0   baseIntent=Intent { act=android.intent.action.MAIN flag=268435456 cmp=ComponentInfo{com.routineplanner/com.routineplanner.MainActivity} }
2023-04-02 11:53:25.870  2959-2959  Launcher.CellLayout     com.miui.home                        D  updateCellOccupiedMarks, update, ShortcutInfo, id=254, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=5, container=-100, cellX=3, cellY=1, spanX=1, spanY=1
2023-04-02 11:53:28.708  2959-3105  Launcher.AllAppsList    com.miui.home                        D  updatePackage, packageName=com.routineplanner, loadMissing=false, user=UserHandle{0}, matchedSize=1
2023-04-02 11:53:28.709  2959-3105  Launcher.AllAppsList    com.miui.home                        D  updatePackage, find appInfo=AppInfo, id=-1, itemType=0, user=UserHandle{0}, mIconType=0, pkgName=com.routineplanner, className=com.routineplanner.MainActivity, screenId=-1, container=-1, cellX=-1, cellY=-1, spanX=1, spanY=1 from ComponentInfo{com.routineplanner/com.routineplanner.MainActivity}

this was tested not so long ago so would be surprised if this is a bug in the library... but doesn't mean to say that it's not a bug.
Would you be able to test with a google pixel emulator to rule out any specific device issues? This would help a lot to narrow down the issue. Also if you're in dev mode, if you set a trigger notification, as soon as you restart the device/emulator, you can check the database to see if it's been set again

@helenaford  Thanks for your reply on my issue 🙌
I checked the same issue in different devices as well like - one plus nord ce 2 5g but still same issue, not receiving after restarting my android app.
As you said to check work_data database so I just checked now, The database still shows those triggers which didn't happen after restarting the app and not getting removed even the timestamp expired.

@ashishmangukiya can you paste your trigger object? are you using the alarm manager?

@helenaford  Here is a scheduled notification object
Data taken through notifee.getTriggerNotifications()
[
{
"notification": {
"title": "Task at 6PM",
"data": {
"type": "task",
"id": "96b05e3c-4e15-41ed-a99a-4c8a790b5591"
},
"id": "15a8b602-defc-4865-b364-793c218269d3",
"body": "06:00 PM",
"android": {
"importance": 4,
"groupSummary": false,
"pressAction": {
"launchActivity": "default",
"id": "press-outside"
},
"lightUpScreen": false,
"loopSound": false,
"vibrationPattern": [
300,
500
],
"localOnly": false,
"defaults": [
-1
],
"chronometerDirection": "up",
"asForegroundService": false,
"ongoing": false,
"smallIcon": "ic_small_icon",
"showTimestamp": true,
"badgeIconType": 2,
"onlyAlertOnce": true,
"actions": [
{
"pressAction": {
"id": "done"
},
"title": "DONE"
},
{
"pressAction": {
"id": "dismiss"
},
"title": "DISMISS"
}
],
"circularLargeIcon": false,
"groupAlertBehavior": 0,
"showChronometer": false,
"colorized": false,
"color": "#5200ff",
"channelId": "NOTIFICATIONS_SOUND_VIBRATE",
"timeoutAfter": 64800000,
"autoCancel": true,
"visibility": 1,
"category": "reminder"
}
},
"trigger": {
"alarmManager": {
"allowWhileIdle": true
},
"repeatFrequency": -1,
"timestamp":<PHONE_NUMBER>000,
"type": 0
}
}
]
Let me know if you need any other info.

@helenaford I know you may be busy with your daily work but This bug is very important for me to get fixed as it is blocking my new release and bcz of the bug users complaining us on not receiving notification after restarting their devices.
I have been checking this thread every 30 mins 😬 just to get your replay.

Hey there, reading this now...
Oh wow, can't believe that's the issue... it's so hard with Android when there are tonnes of different manufactures and devices! Sorry, this has been horrible to get working!
As we're an open-source library we rely on the community to help us make the dev exp easier for everyone. So very much appreciate how much you are debugging and sharing your outcomes here.
What Android version are you on? The auto start setting could be a device specific setting.
For the delay, does it happen for a notification if say you set it 5 minutes after you restart? I'd imagine not as the app should be running at the point.
I'm not really sure what to suggest for that. 🤔 have you checked out our docs on battery optimization? https://notifee.app/react-native/docs/android/background-restrictions Maybe there's another setting somewhere such as adding your app to an allowlist.
This is also a good website https://dontkillmyapp.com/xiaomi

Hi @helenaford
Using android 12 version, Yes not all types of devices having the autostart option.
No, like the app hasn't opened after restart, waited for the notification and received late those which scheduled before restarting.
Xiaomi is the third-largest smartphone brand, with 12.51% of the total market share so the issue should be fixed somehow 😊

Hi @helenaford
One plus devices also have option to enable Autostart https://ibb.co/THVJ7P5
Without enabling the option I couldn't receive triggered notification after restarting one plus device.
After enabling I could receive all notifications on time.
Looks like many other type of devices can have this option.

@helenaford  For now I am going to use this https://www.npmjs.com/package/react-native-autostart library to take autostart permission from user side.
private static final Intent[] AUTOSTART_INTENTS =
{
        new Intent().setComponent(new ComponentName("com.miui.securitycenter", "com.miui.permcenter.autostart.AutoStartManagementActivity")), //MIUI
        new Intent().setComponent(new ComponentName("com.letv.android.letvsafe", "com.letv.android.letvsafe.AutobootManageActivity")),
        new Intent().setComponent(new ComponentName("com.huawei.systemmanager", "com.huawei.systemmanager.optimize.process.ProtectActivity")),  //EMUI
        new Intent().setComponent(new ComponentName("com.oppo.safe", "com.oppo.safe.permission.startup.StartupAppListActivity")),
        new Intent().setComponent(new ComponentName("com.iqoo.secure", "com.iqoo.secure.ui.phoneoptimize.AddWhiteListActivity")),
        new Intent().setComponent(new ComponentName("com.iqoo.secure", "com.iqoo.secure.ui.phoneoptimize.BgStartUpManager")),
        new Intent().setComponent(new ComponentName("com.vivo.permissionmanager", "com.vivo.permissionmanager.activity.BgStartUpManagerActivity")),
        new Intent().setComponent(new ComponentName("com.asus.mobilemanager", "com.asus.mobilemanager.entry.FunctionActivity")).setData(Uri.parse("mobilemanager://function/entry/AutoStart")),
        new Intent().setComponent(new ComponentName("com.samsung.android.lool", "com.samsung.android.sm.ui.battery.BatteryActivity"))
};

The library having above device list which have the autostart permission.

Hi
I was roaming around a little that I saw this and I thought I give my two cents
I've got a Xiaomi myself, and this is a manufacture specific issue
the best thing devs can do for now is to instruct their users about auto start permission and help them by using an intent to open app/related settings

we have some apis so you can navigate the user to the correct app settings screen, @ashishmangukiya do they work?

@ashishmangukiya thanks for sharing. Did you try the api inside notifee first? Asking as if it doesn't work, it's a bug and should be fixed.

ya we're facing the same issue, we are using alarm manager and its not firing in many devices after restart.
Here's my payload :
{
  "notification": {
    "title": "Wake-up Chant",
    "data": {},
    "id": "Wake Up To Chant Alarm_once",
    "android": {
      "importance": 4,
      "groupSummary": false,
      "pressAction": {
        "launchActivity": "com.ishafoundation.app.AlarmActivity",
        "id": "sg_alarm_press"
      },
      "fullScreenAction": {
        "launchActivity": "com.ishafoundation.app.AlarmActivity",
        "id": "sg_alarm_press"
      },
      "lightUpScreen": true,
      "loopSound": false,
      "localOnly": false,
      "defaults": [
        -1
      ],
      "chronometerDirection": "up",
      "asForegroundService": false,
      "ongoing": true,
      "smallIcon": "ic_small_notification",
      "showTimestamp": false,
      "badgeIconType": 2,
      "onlyAlertOnce": false,
      "actions": [
        {
          "pressAction": {
            "id": "sg_alarm_dismiss"
          },
          "title": "Dismiss"
        },
        {
          "pressAction": {
            "id": "sg_alarm_snooze"
          },
          "title": "Snooze (5 min)"
        }
      ],
      "circularLargeIcon": false,
      "groupAlertBehavior": 0,
      "showChronometer": false,
      "colorized": false,
      "color": "white",
      "channelId": "Wake Up To Chant Alarm",
      "autoCancel": true,
      "visibility": 1,
      "largeIcon": "ic_launcher",
      "category": "alarm"
    }
  }
}

@helenaford can you please mention which exact method takes the user to autostart switch screen ? 🙏🏻


can't find devenv.exe

Hi,
I need to find devenv.exe in order to upgrade an sln, in an environment configured by build tools. So the VCINSTALLPATH does not contains the path to devenv.exe.

Please find above the PR to solve my issue :)


Fix log_subscriber monkey patch for Rails 5.0.3.

We just updated to Rails 5.0.3 and noticed we were getting exceptions when running rails db:reset. It turns out that the API changed in 5.0.3 to match 5.1.
This patch changes the version sniffing to assume >= 5.0.3 will use the new API. It's possible that it may change back to 5.0.2 behaviour in a 5.0.4 release, but we should be able to handle this with additional predicates.
Before

After

Couple of questions:

Are you okay with changing the version sniffing to do string comparisons? It lets you easily test against the patch number.
I wasn't sure how far to go with testing. Do you want to test against 5.0.3 as well as 5.x? It looks like we assume the latest stable version is good enough.

Feedback welcome.

It would be great to get this merged - we are seeing the same issue that this fixes

Thank you for the PR.
v4.1.2 has been released. Let me know if you run into issues with it.


Wonder if you would reproduce the i2i function? Thanks.

aka "My Subject in My Style"

please stay tuned


scrollbar updated

Thank you.
On Tue, 22 Aug 2023, 9:59 pm Aaditya Chowdhury, @.***>
wrote:

@.**** approved this pull request.
Looks Great!!
—
Reply to this email directly, view it on GitHub
https://github.com/AadiXC0DE/QR-NINJA/pull/3#pullrequestreview-1589877683,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCGU4VSPIY3G3NMH7ZJW53XWTM6HANCNFSM6AAAAAA3ZUGZNA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Export-CsOnlineAudioFile is not usable for uploading?

I have successfully uploaded an audio file using the Import-CsOnlineAudioFile.
But exporting the file back seems to get a different format unusable to Import again:
$content=Export-CsOnlineAudioFile -Identity <> -ApplicationId huntgroup
[System.IO.File]::WriteAllBytes("test.wav", $content)
Either options give me the same error "Audio file content is not supported"
(directly from the export)
Import-CsOnlineAudioFile -ApplicationId HuntGroup -FileName $audiofile.FileName -Content $content
(or from a saved file)
$contentFile = Get-Content "test.wav" -Encoding byte -ReadCount 0
Import-CsOnlineAudioFile -ApplicationId HuntGroup -FileName "test.wav" -Content $contentFile

@TheITguy666 Thanks for the feedback. Please open a Microsoft support case and they will work with you to get to the bottom of this.

@yogkumgit
MS support confirmed today: The file that you get from the command export-csonlineaudiofile is not a supported format to be able to upload using the import-csonlineaudiofile.

Closing this as #9183 is linked. thanks


Create a duckDB CMS using DuckDB and store the website content of duckdb.org using that CMS

A website is a graph, make use of graph database technology for creating the website at https://duckdb.org.
The ISO GPM is stable:
https://www.gqlstandards.org/ July, 2022 – GQL Status Update
"SQL/PGQ is interesting to people focused on GQL because the Graph Pattern Matching (GPM) language in SQL/PGQ is essentially identical to GPM in GQL. Since the technical work on SQL/PGQ is now stable, GPM in GQL is now stable."

We are quite happy with our statically generated website.

"hannes closed this as completed" - how that? shouldn't it be "not done"?

"hannes closed this as completed" - how that? shouldn't it be "not done"?

That's just the default message.

We are quite happy with our statically generated website.

Who is we?
Why is the only message before closing as "completed" referring to the happiness of "we" and not to the unhappiness and damage the flaws of the website create elsewhere?
Why do you mention one feature "statically generated" and not other features, e. g. stored in dozens of text files of different formats (md, yml, txt, json, html) and the generated website containing inconsistencies because of that (e.g. different URLs presenting the same content, menu hierarchy different when going different routes) and a large amount of white space, probably also because of that?

With the proposal the data would be stored in the database, getting rid of "md, yml, txt, json, html" and can still be "statically generated" while containing less inconsistencies.
Additionally other people (not included in "we") using DuckDB would have the possibility to run their websites and their data analyses based on the same technology.

Howdy!
Hannes is the CEO of DuckDBLabs and a co-creator of DuckDB. That is the "we"! 😀
The changes you are proposing sound like a significant amount of work. If you wish to reimplement our website, I suggest that you fork our repo and show us what the completed reimplementation looks like. Please keep in mind that this is open source, so this website is provided as a free service to the community!
Also, would you be able to clarify the benefit a bit more? Portions of our site are automatically generated with Python scripts, but then stored in github as static files for loading speed. I don't see how your approach is superior.


Community mode

This creates a new mode "community" for community projects.
It removes all internal URLs from config to enable trying the project outside of the RH firewall.
The openstack-rdo product streams doesn't have any builds because they exist in the Centos Koji instance.

@RedHatProductSecurity/corgi-devs I updated the tests so they can be run without any internal URLs and fix the the few suggestions Martin had.
This should be ready for review. It works well for Fedora. We'll have to figure out a solution for openstack-rdo.

I removed VCR from the last 2 tests which were using them, so removed pytest-recording dependency from the project as well. This also allowed to discontinue the use of git-lfs, so all files are now directly part of the repository. Git LFS can be removed as a dev dependency.

Now with SCA enabled

LGTM

I see a bunch of LFS files that changed. The diff in GitHub only says "Git LFS file not shown" - did you mean to change these, or should they have been completely removed instead?

The old files are not removed, however we shouldn't be using them anymore.

I think we might be able to run "community mode" without needing a separate settings module or docker-compose.yml file. I only mention because I think you said you would have liked to keep this even simpler (or maybe I'm misremembering?)
If so, we can add public defaults to internal env vars in docker-compose. The Pypi value will only be used if the env var is unset: ${PIP_INDEX_URL:-"https://pypi.org/simple"}
And document which env vars should be set / unset for community mode: UMB_BREW_MONITOR_ENABLED turned off COMMUNITY_MODE turned on
I think that should remove the need for a separate docker-compose and settings file. The Dockerfile might still need some tweaking.

The main reason why we have 2 docker-compose files is so that we can use a separate Dockerfile for each. We need a separate Dockerfile to skip the step where we download and installed the internal CA certificate. Having 2 docker-compose files also allows a separate docker volume for the postgres data, incase you wanted to save the community data but delete the enterprise data.

I think we might be able to run "community mode" without needing a separate settings module or docker-compose.yml file. I only mention because I think you said you would have liked to keep this even simpler (or maybe I'm misremembering?)
If so, we can add public defaults to internal env vars in docker-compose. The Pypi value will only be used if the env var is unset: ${PIP_INDEX_URL:-"https://pypi.org/simple"}
And document which env vars should be set / unset for community mode: UMB_BREW_MONITOR_ENABLED turned off COMMUNITY_MODE turned on
I think that should remove the need for a separate docker-compose and settings file. The Dockerfile might still need some tweaking.

The main reason why we have 2 docker-compose files is so that we can use a separate Dockerfile for each. We need a separate Dockerfile to skip the step where we download and installed the internal CA certificate. Having 2 docker-compose files also allows a separate docker volume for the postgres data, incase you wanted to save the community data but delete the enterprise data.

I resolved this by adding a '| true'  after the download ROOT_CA in the Dockerfile. This allows a community user without that access to skip adding that certificate. However we now have both community and enterprise using the same docker volume.


Non operator-sdk operators

To enable a new operator, the admin needs to

Install CRD and RBAC for that operator
Enable subscription.
To simplify this, can we have the
1 catalog/OLM operator install CRD once an install plan is executed?

Many operators out there install CRDs on startup and don't have a separate CRD yaml. Is there a guide on how to use them with OLM?

Hi @smanpathak
To clarify - if you are creating a Subscription, then CRDs and RBAC will be configured automatically

Many operators out there install CRDs on startup and don't have a separate CRD yaml. Is there a guide on how to use them with OLM?

There's not a way to do that directly - however, for any operator that creates CRDs on startup, it would not be hard to modify them to check for the existence of the CRD on startup as well. The operator should handle the existence gracefully anyone (for example, if the pod is rescheduled and must restart). Operators written like this can be easily packaged by OLM.
Thanks!


ord env consistently panics at shutdown on Windows

going back to the first inclusion of the ord env command, I have never been able to get it to shutdown gracefully.  It always panics after a single CTRL-C.  In 0.17.1, I receive this message:
`ord` server URL:
http://<IP_ADDRESS>:9001
Example `bitcoin-cli` command:
bitcoin-cli -datadir=env getblockchaininfo
Example `ord` command:
C:\Users\nine\ord\target\release\ord.exe --datadir env wallet balance

Shutting down gracefully. Press <CTRL-C> again to shutdown immediately.
Shutting down gracefully. Press <CTRL-C> again to shutdown immediately.
thread 'main' panicked at src\subcommand\env.rs:10:8:
called `Result::unwrap()` on an `Err` value: Error { kind: NotFound, message: "program not found" }
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
thread 'main' panicked at src\subcommand\env.rs:10:8:
called `Result::unwrap()` on an `Err` value: Error { kind: NotFound, message: "program not found" }
stack backtrace:
   0:     0x7ff622a0cef2 - <unknown>
   1:     0x7ff622a3276d - <unknown>
   2:     0x7ff622a07611 - <unknown>
   3:     0x7ff622a0cd1a - <unknown>
   4:     0x7ff622a0f129 - <unknown>
   5:     0x7ff622a0ede5 - <unknown>
   6:     0x7ff622a0f644 - <unknown>
   7:     0x7ff622a0f519 - <unknown>
   8:     0x7ff622a0d7f9 - <unknown>
   9:     0x7ff622a0f1e6 - <unknown>
  10:     0x7ff622a868b7 - <unknown>
  11:     0x7ff622a86df3 - <unknown>
  12:     0x7ff6221f2f1a - <unknown>
  13:     0x7ff6221c17eb - <unknown>
  14:     0x7ff622207684 - <unknown>
  15:     0x7ffef5431030 - <unknown>
  16:     0x7ffef5434f5e - is_exception_typeof
  17:     0x7ffef543f500 - _C_specific_handler
  18:     0x7ffef543416c - is_exception_typeof
  19:     0x7ffef543fe41 - _CxxFrameHandler3
  20:     0x7fff175b449f - _chkstk
  21:     0x7fff1752fd54 - RtlUnwindEx
  22:     0x7ffef543f9ce - _C_specific_handler
  23:     0x7ffef5432e25 - is_exception_typeof
  24:     0x7ffef5433244 - is_exception_typeof
  25:     0x7ffef5434268 - is_exception_typeof
  26:     0x7ffef543fe41 - _CxxFrameHandler3
  27:     0x7fff175b441f - _chkstk
  28:     0x7fff1752e466 - RtlFindCharInUnicodeString
  29:     0x7fff17564475 - RtlRaiseException
  30:     0x7fff14dd5b0c - RaiseException
  31:     0x7ffef5436720 - CxxThrowException
  32:     0x7ff622a1ff1e - <unknown>
  33:     0x7ff622a0f999 - <unknown>
  34:     0x7ff622a0f6ad - <unknown>
  35:     0x7ff622a0f519 - <unknown>
  36:     0x7ff622a0d7f9 - <unknown>
  37:     0x7ff622a0f1e6 - <unknown>
  38:     0x7ff622a868b7 - <unknown>
  39:     0x7ff622a86df3 - <unknown>
  40:     0x7ff6221f2f1a - <unknown>
  41:     0x7ff6221c17eb - <unknown>
  42:     0x7ff622206175 - <unknown>
  43:     0x7ff6221e7af3 - <unknown>
  44:     0x7ff621d314a3 - <unknown>
  45:     0x7ff621ca1006 - <unknown>
  46:     0x7ff621ca101c - <unknown>
  47:     0x7ff6229fdf92 - <unknown>
  48:     0x7ff621ca107c - <unknown>
  49:     0x7ff622a5638c - <unknown>
  50:     0x7fff1674257d - BaseThreadInitThunk
  51:     0x7fff1756aa58 - RtlUserThreadStart

Please let me know if there is any further information I can provide to help with the fix.

Thanks for the report!
I think this might be related how we shutdown and send signals to the processes. Specifically how Windows handles SIGTERM differently


Add tests for module init returning/panicking errors

Description:
$title, once jBallerina changes are introduced for the same.
This should include tests for check at module level.

Need to enable ServiceTest.testServiceInitNegativeTest or add a similar test too.

PR https://github.com/ballerina-platform/ballerina-lang/pull/18215 adds tests related to this

@MaryamZi I think test scenarios are covered now, please check on ServiceTest.testServiceInitNegativeTest you mentioned above and close the issue

@MaryamZi I think test scenarios are covered now, please check on ServiceTest.testServiceInitNegativeTest you mentioned above and close the issue

Yeah, sent https://github.com/ballerina-platform/ballerina-lang/pull/18448 to enable this.


Self-hosted CI-mpi

Update CI-mpi workflow to use our self-hosted runners.

MPI runs took ages on this machine - will switch back to another machine and add a few more runners. Should be ready later today.

Temporarily closing for some testing.


Ironsworn: Show images for progress boxes

Changes the progress track boxes to use gradient based backgrounds to display 'tick' marks instead of just the characters - + * #

Changes progress bars in: Failure track, Legacy Vows, Repeating Vows, Progress Tracks, Bonds Track, Sites Track
Does not add any new attributes or remove any attributes.

Character Sheet Info Roll20 Internal Use only.


Usage questions

Thank you @hurshi for the awesome package! I got a few questions, maybe you can add the answers in the future to README?

how to invalidate manually the cache? should I set max-age to 0? (when user makes pull to refresh for example)
how to delete ALL items from cache (when the user logs in with another account)

Thanks for your report, I have updated README, and add some example.
And this is my suggestion for your question:

_manager.delete(url);
 _manager.clearAll();


Update heartbeats.md

See https://github.com/rabbitmq/rabbitmq-website/issues/1386

@cressie176 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Thanks! I just copied the existing text in #1387. I'll let @michaelklishin decide which to use.

@cressie176 Thank you for signing the Contributor License Agreement!

Thank you. This suggestion makes practical sense: both sides should insist on disabling heartbeats for best predictability.
FTR, here is the server function that performs negotiations of values such as the heartbeat timeout and
channel_max: https://github.com/rabbitmq/rabbitmq-server/blob/v3.10.x/deps/rabbit/src/rabbit_reader.erl#L1349-L1358.

Perfect. I figured that you would have suggestions.

Thank you, @cressie176


Problem with quickstart on Mac OSX

I am having problems with getting the quick start to work on Mac OSX. I have the prerequisites met. When I run:

make launch-dev

It fails to launch the hyperkube container. I get the following error:

Error response from daemon: Cannot start container 36d5c85c1f894a52945a98b3f047795554f9292cd06a840a00b3968c121ab949: [8] System error: stat /mnt/sda1/var/lib/docker/aufs/mnt/36d5c85c1f894a52945a98b3f047795554f9292cd06a840a00b3968c121ab949/sys/fs/cgroup/cpuset/docker/36d5c85c1f894a52945a98b3f047795554f9292cd06a840a00b3968c121ab949: no such file or directory
Error: failed to start containers: [36d5c85c1f894a52945a98b3f047795554f9292cd06a840a00b3968c121ab949]

Hmm it looks like one of the kubernetes containers isn't being created properly. What's the output of docker ps following that error?
We've seen problems like this before with different Docker storage drivers. We've had the best results with devicemapper. Would it be a pain to redeploy with devicemapper and try again? Here's a tutorial on how that I think should still work: http://muehe.org/posts/switching-docker-from-aufs-to-devicemapper/.

This also happens to me on linux to. I got it to work by mounting the etc/kube directory to the kubelet container. I created a pull-request for this here. I tested it on both Mac OSX and Ubuntu Linux.

So changing the storage driver to devicemapper works. I will delete my pull request and close this issue, as you have already added the fix to the documentation.


qemu: add curl to buildInputs

Motivation for this change
Enables support for accessing files over HTTP:
qemu-system-x86_64 -drive media=cdrom,file=http://host/path.iso,readonly

Increases the closures size from 445 to 447 MiB.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

A 0.4% increase!!! Impossible! faints

@copumpkin: Yeah, I know, big stuff! ;-) I'll merge it...


Extend Analysis-Ready Data for SAR

[x] The EODC section should contain more details and also code snippets
[x] The Terrascope code snippet should ideally be translated into other languages, too.
[ ] EODC: Review content

All done, but EODC still needs to verify my code snippets work for the given extents. I couldn't check it as the EODC back-end is not working right now. @ValentinaHutter @pstranner You could also extend the description a bit more, it's very short at the moment.


[BUG] Can't ping anymore after changing settings

Describe the bug
Can't ping after changing some settings in the menu
To Reproduce
Steps to reproduce the behavior:

Go to game, ping -> Works
Got to settings in game
change setting (was graphics but I don't think this is linked)
try to ping (T or G) -> no ping

same.


[wip][no ticket] Set up Percy

Purpose
Visual diffs in Percy!

Summary of Changes

Install ember-percy
Seed faker in tests, so it'll have the same "random" data every time
Switch mirage guid factory to use seedrandom instead of seeding faker, to avoid interference with the global faker seed above
Define the guid alphabet in only one place

Side Effects

Less work
Better app

Feature Flags
n/a

QA Notes
You'll like this.

Ticket

n/a
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Pull Request Test Coverage Report for Build 1886

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 66.261%

Totals

Change from base Build 1882:
0.006%

Covered Lines:
4164

Relevant Lines:
5846

💛  - Coveralls


Component mutates array provided in tags prop

When an array is passed in as a prop for tags, it is mutated. This has a the side effect of changing the parent component's state without triggering an update.
See below for example code and console output.
Potential Fix
A simple fix would be to use Array.prototype.concat to make a copy of the tags array from this.props when setting the initial state.
getInitialState: function () {
  return {
    tags: [].concat(this.props.tags) || [],
    currentInput: null
  };
},

https://github.com/tutorialhorizon/react-tagged-input/blob/e155c0f34738a5f4a733cc48704a6fc80a0ddf50/src/TaggedInput.jsx#L70

Example:
var MyComponent = React.createClass({
  getInitialState() {
    return { tags: ['foo'] };
  },

  onAddTag(value) {
    console.log('onAddTag', value, this.state.tags);
  },

  onRemoveTag(value) {
    console.log('onRemoveTag', value, this.state.tags);
  },

  render() {
    return (
      <div>
        <TaggedInput tags={this.state.tags}
                     onAddTag={this.onAddTag}
                     onRemoveTag={this.onRemoveTag} />
      </div>
    );
  }
});

Console output
> onAddTag bar ["foo", "bar"]
> onRemoveTag foo ["bar"]

Given that MyComponent never updates its state, the values of this.state.tags should never change. However, the console output above shows that TaggedInput has modified the tags array from the state of MyComponent.

@alistairjcbrown Thanks for reporting the issue!
Would you like to send a PR for the same?

@thmain No worries - thanks for the great component. I'm using it in a side project at the moment and so I didn't forget, I fired up a few issues.
I'll see about getting some PRs up tonight for them :+1:

Fixed!


Determine running environment

This PR ensures that the MNC is less tied to GitHub Actions. I'm not yet sure what will be required to run in GitLab but this at least provides proper hooks and prevents GitHub-specific things from happening there.

Code looks fine to me. Have you tested this in GHA and GitLab to ensure it does what you expect?

@cole-h Things seem to be working as normal:
GHA: https://github.com/lucperkins/mnc-action-test/actions/runs/9117773497/job/25069402172
GitLab: https://gitlab.com/DeterminateSystems/flakehub-cache-test/-/jobs/6872721027


importModule always formats with brittany

We now have formatProviders, but the importModule function always uses brittany if formatOnImport is enabled.
Rather use the configured formatter.

How are plugins supposed to access other plugins? Currently, it doesnt seem possible.

I think we are doing this incorrectly, we should maybe follow lead of what other languages' tooling are doing. I for instance like what Golang does: whenever you automatically insert an import, the whole of the import section gets automatically formatted.
We should probably do that: find the range of text were all the imports are and format that range with the chose formatter. We could expose a custom action from hie to format imports and use the custom action from any plugin inserting stuff into that section of code.

This is what I am currently trying to do.
The problem I am currently facing is, that the type
type FormattingProvider = Uri -> FormattingType -> FormattingOptions -> IdeDeferM (IdeResult [TextEdit])

Can not be easily called within a IdeGhcM

I meant doing something like this: https://github.com/haskell/haskell-ide-engine/pull/889/files#diff-df7cc8b4c1792382d5165b401f9ba6d0R536
The ghcmod plugin instructs the client to request an action from the hare plugin. We could do the same here. Just build and action for formatting the range, and the right formatter will be used

@fendor maybe don't listen to me too much, I thought that lien was instructing the client to execute something, but it seems like the user still needs to select the action to be executed.


Error when running the dockercompose file

Before reporting an issue

[X] I have searched existing issues
[X] I have reproduced the issue with the latest release

Environment information
version: '3'

Minimal setup for a running EHRbase. Contains the server component as well as the required postgres instance.

services:

EHRBase container. see .env.ehrbase for configuration details.

ehrbase:
image: ${EHRBASE_IMAGE:-ehrbase/ehrbase:2.0.0}
env_file:
- .env.ehrbase
environment:
DB_URL: jdbc:postgresql://ehrdb:5432/ehrbase
DB_USER_ADMIN: ehrbase
DB_PASS_ADMIN: ehrbase
DB_USER: ehrbase_restricted
DB_PASS: ehrbase_restricted
links:
- ehrdb
depends_on:
ehrdb:
condition: service_started
ports:
- "8080:8080"
networks:
- ehrbase-net

Pre-configured ehrbase postgres.

ehrdb:
image: ${EHRBASE_POSTGRES_IMAGE:-ehrbase/ehrbase-v2-postgres:16.2}
environment:
POSTGRES_USER: postgres
POSTGRES_PASSWORD: postgres
EHRBASE_USER_ADMIN: ehrbase
EHRBASE_PASSWORD_ADMIN: ehrbase
EHRBASE_USER: ehrbase_restricted
EHRBASE_PASSWORD: ehrbase_restricted
healthcheck:
test: [ "CMD-SHELL", "pg_isready -U postgres" ]
interval: 5s
timeout: 5s
retries: 12
ports:
- "5432:5432"
networks:
- ehrbase-net
networks:
ehrbase-net: {}
Steps to reproduce
I've run this docker compose file as all in one and also run the docker files separately.
The postgres image works fine but when I run the ehrbase image it crashes.
I get the following error when booting up the ehrbase image:
| Caused by: org.flywaydb.core.internal.sqlscript.FlywaySqlScriptException: Migration V2__ehr.sql failed
localenvironment-ehrbase-1  | ----------------------------
localenvironment-ehrbase-1  | SQL State  : 42883
localenvironment-ehrbase-1  | Error Code : 0
localenvironment-ehrbase-1  | Message    : ERROR: function ext.versioning() does not exist
localenvironment-ehrbase-1  | Location   : db/migration/V2__ehr.sql (/file:/ehrbase.jar!/BOOT-INF/lib/base-0.33.0-SNAPSHOT.jar!/db/migration/V2__ehr.sql)
localenvironment-ehrbase-1  | Line       : 146
localenvironment-ehrbase-1  | Statement  : CREATE TRIGGER versioning_trigger BEFORE INSERT OR UPDATE OR DELETE ON ehr.status
localenvironment-ehrbase-1  | FOR EACH ROW EXECUTE PROCEDURE ext.versioning('sys_period', 'ehr.status_history', true)
localenvironment-ehrbase-1  |
localenvironment-ehrbase-1  |   at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.handleException(DefaultSqlScriptExecutor.java:275)
localenvironment-ehrbase-1  |   at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.executeStatement(DefaultSqlScriptExecutor.java:222)
localenvironment-ehrbase-1  |   at org.flywaydb.core.internal.sqlscript.DefaultSqlScriptExecutor.execute(DefaultSqlScriptExecutor.java:126)
Expected behavior
Docker image up and running
Actual result
Caused by: org.flywaydb.core.internal.sqlscript.FlywaySqlScriptException: Migration V2__ehr.sql failed
Further information
No response

Nevermind, after I deleted all my docker containers and images I pulled them again from the server. Now it is working.


Support for Camera Roll

A comment in the image component references support for local assets that are part of the bundle, but also other places such as the camera roll.
https://github.com/facebook/react-native/blob/6e04a3c964848c3beb0966dcfb54fff370ab68d7/Libraries/Components/Image/Image.ios.js#L30
Does this mean I can feed in URL's of the scheme assets-library://, or is this not supported? As the component currently reports that the image is not found. If it isn't, would I need to implement it in RCTConvert similarly to Gif support?
eg.
  <Image style={styles.icon} source={ix('assets-library://asset/asset.JPG?id=x&ext=JPG')}  />

@vjeux I have a catch here. If I load a GIF from asset library with a path like: assets-library://asset....&ext=GIF, then I just end up getting the first frame but not the whole GIF. How do I get it as a GIF image so I can display it as a playing GIF with the Image tag.


Self-host run_isolated.zip/.exe on isolate server

Goal:
- Stop archiving run_isolated.zip for swarming task request. It's silly.

AI:
The same way swarming_bot.zip is generated by the server itself, the isolate
server should have the code to run an isolated task available for download.

- ln -s ../../client appengine/isolate/client
- Add code to generate run_isolated.zip
- Add urlendpoint to download the code, similar to /get_slave_code on swarming.

It'd be good to add /<cpu_arch>/<os_type>/ in the url so we can future proof
ourselves for pre-compiled binaries (e.g. golang). It would just default to the
python implementation when no precompiled binary is available. I think it's the
best design.

Future:
To enable compile-client-on-upload, tools/update.py will have to be updated
accordingly.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 14 Nov 2014 at 12:36

Blocking: #197

We embedded it into swarming bot instead.