Higher ed threshold modification

Right now, we see aberrations due to low shares of college students and high low income percentiles. If we want to remove students from contributing to income, we should explicitly do this.
The cutoff for college students should be better aligned to the share of students attending elite schools.

Our goal is to modify income in tracts that have under some high threshold of students (80% is my starting number!) so they can continue to be included.
May require refinement going forward.

This will involve two parts:

Right now, we see aberrations due to low shares of college students and high low income percentiles. If we want to remove students from contributing to income, we should explicitly do this.
The cutoff for college students should be better aligned to the share of students attending elite schools.

Our goal is to modify income in tracts that have under some high threshold of students (80% is my starting number!) so they can continue to be included.
May require refinement going forward.

I agree with revisiting whether educational enrollment should be such a significant piece of the threshold criteria.
Current enrollment is highly correlated with the age distribution -- Areas with older populations tend to have lower enrollment.
Because of this Age relationship, there is an inverse relationship between current enrollment and attainment. On average, areas with lower current enrollment tend to have higher high school and college attainment
Below is analysis of all census tracts in New York state.
The first table is by Age quintile (using % over Age 65).
The second is by Higher Ed Enrollment quintile (from the CEJST data)
Therefore I don't think that low educational enrollment is a great measure of disadvantage - It seems to reflect the age distribution.
This has big implications for areas with lots of students who may not be permanent residents.
The map shows the Old North End of Burlington Vermont. This area has low income, many renters, poor housing quality, high linguistic isolation (thriving immigrant community), and poor health outcomes. However it fails to pass any of the thresholds because of high educational enrollment, due to high numbers of students attending local colleges. I would encourage reviewers to look at other lower-income, high-renter areas in student towns/cities.

Closing because it appears the income imputation takes care of this

also -- thank you so much for the super thoughtful analysis @adwel81!


Detect percent-encoded list params that use the compact (comma-delimited) schema

When the client percent-encodes the commas for a value in the query string, the framework always treats that value as a blob, rather than a list of strings. What we would like is for this:
things=id1%2Cid2%2Cid3
...to be interpreted as a list by req.get_param_as_list(), i.e.:
`['id1', 'id2', 'id3']
For consistency, req.get_param() should also realize that things is a list and only return a single element as is done when form-style lists are encountered.
We'll have to see if this can be done efficiently. If it can not be done without significantly reducing the performance of parse_query_string(), then we may have to just let the bug stand and make sure it is documented.

@kgriffs How about adding an extra keyworded argument delimeter to the get_param_as_list(), ie.
req.get_param_as_list('things', delimeter=',')
# ['1', '2', '3']

@kgriffs Any updates on this ?

Given that we have now cemented the new default of not splitting on commas (see also the above link to #1547 ), it is unlikely that we will want to invest in this special-case  detection (which would also make the framework's logic more complex).


"actSig" nil error

Repro'ing for me with tsserver

It seems to be an old version.
Would you check the latest one? If the issue still existed, please send me the function signature as well. Thanks!

Huh! I was sure I did a PlugUpdate and tried again before posting but now I can't reproduce that error.
I did run into this while impatiently waiting for tsserver to spin up, but may not be related to this plugin:

The signatures don't show super consistently with tsserver, but that could just be tsserver's fault; rust_analyzer seems pretty quick and responsive.
The signature in question was from zustand, for what it's worth:

When the tsserver is startup, it may not send back the correct response and may crash the plugin...

Also, I saw this wired error the other day from another lsp. It is more a fault from tsserver. When it specifies the active parameter to be 1 but send no signatures and parameters back.
You may want to raise it to tsserver team.


Missing items: [Brunnhildar Axe]

[Brunnhildar Axe] (https://www.wowhead.com/wotlk/item=41752/brunnhildar-axe "https://www.wowhead.com/wotlk/item=41752/brunnhildar-axe")
Also note that the in-game tooltip has an additional + 4 - 6 damage (no damage type listed)
This is not on the item in wowhead for some reason

We intentionally don't include common or junk items because there isn't any purpose to adding them. Is there some need for this item?

it's really slow so I was hoping to test it with fan of knives for different numbers of targets


Device roles

Be able to assign a role to a device that serves the purpose of denoting what the device is.
Use case
assign a role to a device that is able to have config added to it for the role. this would then link the device life cycle from SOE Provisioning (the config group) to deployed service/role.
Features

[ ] Can have config assigned
[ ] API endpoint available to pull device role config
[ ] role config is rendered as part of device config

marked this issue as blocking #33

marked this issue as a duplicate of #70

closing in favour of #70

marked this issue as related to #70

added 5m of time spent


Round Robin Troupe Messages

I have an actor set up in a troupe configuration and each member holds a network connection to a server.  I'm running into an issue where during light loads the same member is used for all messages (every 4s) and this causes the other troupe member's network connection to timeout.
I wonder if it would be possible to do something like round robin the messages to members of the troupe so they don't become stale.  Thoughts?

Hi Kevin,
You could definitely experiment with that: change line 96 of
thespian/troupe.py from "self._idle_troupers.append(troupe_member)" to
"self._idle_troupers.insert(0, troupe_member)".
I don't really see any downsides to this approach, so I think this is a
reasonable change to make (if you get a chance to experiment with the
above recommendation, let me know how it goes).
The one caution I would offer is that I would suspect that there are
many other ways you could encounter stale connections to your server, so
you may want to handle this situation anyhow, and also possibly use a
self.wakeupAfter to schedule sends of noop messages as a keepalive.
Regards,
Kevin
Kevin Colyar writes:

I have an actor set up in a troupe configuration and each member holds a network connection to a server.  I'm running into an issue where during light loads the same member is used for all messages (every 4s) and this causes the other troupe member's network connection to timeout.
I wonder if it would be possible to do something like round robin the messages to members of the troupe so they don't become stale.  Thoughts?

My ideal setup would be to set the idle_count=1, that way most of the time my troupe would just contain one actor with one connection, and just spin up additional actors during peaks.
However, I also use my actor system with a simulator for testing and it will send many messages a second to my troupe.  When the idle_count is 1, it's continually spawning and killing troupe members, which is opening/closing connections to my server.  It really slows it down.  I assuming it's killing troupe members when they are done processing their message, when it sees that self.troupe_work_in_progress is False.  Is there some way I could not have them be considered idled so quickly?

Hi @kevincolyar,
Sorry to have not responded to this earlier.  The troupe workers should not be killed if there is active work still available (https://github.com/kquick/Thespian/blob/master/thespian/troupe.py#L90-L95).
I would like to implement the round-robin and delay before killing ideas, so I've created https://github.com/kquick/Thespian/issues/12 which is where the currently active Thespian work is occurring (and thus, I'm closing the issue here).  Please feel free to watch that issue if you are still interested in this functionality.


Publish Codecov report in PRs

Show the Codecov code coverage change report in pull requests

We will use SonarCloud instead


[FLINK-18756][table-api] Support IF NOT EXISTS for CREATE TABLE statement

What is the purpose of the change

This pull request support IF NOT EXISTS for create table statement.

Brief change log

Support parse IF NOT EXISTS in parserImpls.ftl
Simplify SqlCreateTable.java 's constructor

Verifying this change

Add parse tests in FlinkSqlParserImplTest.java
Add catalog related test in TableEnvironmentTest.scala

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@danny0405 could you have a more look?

Could you add a test in SQL CLI to make sure it also works in SQL CLI? Maybe we can add it in LocalExecutorITCase.


RHIDP-2596: update OIDC auth provider configurations

Description
Update the OIDC Auth Provider documentation to reflect the changes made in https://github.com/janus-idp/backstage-showcase/pull/1309

/cherrypick 1.2.x


Evaluation Metrics

Could you please provide the code of evaluating the generated content for computing PPL, coherence socre and MAUVE score ?

Hi,
Thanks for your interest in our work. Please check out the latest commit: 5a966ff1cb3ab6955cf528b28e2a8088a217ab05.

Thanks，but it seems you forgot to upload “simcse” package


Update ubuntu version in def file from 21.10 to 22.04; 21.10 eol was July, 2022

##Error
The error occurs when one tries to build an image from def file. Since Ubuntu 21.10 reached its end of life on July 2022, apt-get update fails.
This is an error I got
+ apt-get update
Ign:1 http://security.ubuntu.com/ubuntu impish-security InRelease
Ign:2 http://archive.ubuntu.com/ubuntu impish InRelease
Err:3 http://security.ubuntu.com/ubuntu impish-security Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Ign:4 http://archive.ubuntu.com/ubuntu impish-updates InRelease
Ign:5 http://archive.ubuntu.com/ubuntu impish-backports InRelease
Err:6 http://archive.ubuntu.com/ubuntu impish Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:7 http://archive.ubuntu.com/ubuntu impish-updates Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:8 http://archive.ubuntu.com/ubuntu impish-backports Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Reading package lists... Done
E: The repository 'http://security.ubuntu.com/ubuntu impish-security Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: The repository 'http://archive.ubuntu.com/ubuntu impish Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: The repository 'http://archive.ubuntu.com/ubuntu impish-updates Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: The repository 'http://archive.ubuntu.com/ubuntu impish-backports Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
FATAL:   While performing build: while running engine: exit status 100

##Solution
To solve the error substitute from: ubuntu:21.10 with from: ubuntu:22.04 in the `environment/mpra_oligo_barcode.def. Then the image is built nicely.

Thanks! Sorry I didn't see this sooner. I will fix it now.


Include update to handle latest Nidium server

Please include an example and a target in the Makefile to compile it.
I have never heard nor used libapenetwork (what is nidium?) and so have no way to ensure that the adapter keeps working.


Alignment on underscores in Blazor examples

It looks like we won't have complete alignment between engineering and docs on explicit use of the private access modifier. Templates do it now, but engineering samples often (perhaps usually) don't. It's consistently in use now in the docs.
Next question is on underscores for private fields:
https://github.com/aspnet/AspNetCore/wiki/Engineering-guidelines#coding-style-guidelines--general

Engineering:

No underscore for private fields in the @code block root. Slight inconsistencies tho: Templates have _stopPropagation but currentCount, forecasts, and collapseNavMenu.
The underscore is used for a private field inside of a class in the @code block in engineering samples.

Docs: Mostly using underscores for private fields ... not in every in-text example tho. There are a handful that still need a touch of work if we're going to use them everywhere in docs.

How would you like the underscore used with private fields across the doc examples?

Always
Inside classes only, not for fields at the root of @code blocks
Nowhere

btw - I plan to put an engineering PR in to standardize the approach in templates, if needed ... plz 🛑 me if you prefer that I not do that.

@danroth27 Need your ruling on this one in order to work it. :point_up:


Close this ticket, but will need a larger discussion about what to do with all 'avoid' terms in the futute


feat: add minio and local-path-rwx to dev stack. refactor dev stack into chart

disable default local-path provisioner
add minio and local-path-rwx to dev stack
refactor dev stack manifests into helm chart

Description
...
Related Issue
Fixes #

Relates to #
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[ ] Test, docs, adr added or updated as needed
[ ] Contributor Guide Steps(https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md#submitting-a-pull-request) followed

Do we want to start adding tests that the dev stack components work as expected here? If so, do we want to update this project to use uds run?

Yes we can do that in another or 🦄


Fix typo

Description

Linked issue(s)/Pull request(s)

issue_number

Type of Change

[ ] Non-breaking bug fix
[ ] Breaking bug fix
[ ] New feature
[ ] Test
[ ] Doc update
[ ] Docker update

Related Component

[ ] Simulation toolkit
[ ] RL toolkit
[ ] Distributed toolkit

Has Been Tested

OS:

[ ] Windows
[ ] Mac OS
[ ] Linux

Python version:

[ ] 3.6
[ ] 3.7

Key information snapshot(s):

Needs Follow Up Actions

[ ] New release package
[ ] New docker image

Checklist

[ ] Add/update the related comments
[ ] Add/update the related tests
[ ] Add/update the related documentations
[ ] Update the dependent downstream modules usage

Codecov Report

Merging #242 (c49ee4a) into v0.2 (951eaaf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             v0.2     #242   +/-   ##
=======================================
  Coverage   64.97%   64.97%
=======================================
  Files         111      111
  Lines        5333     5333
=======================================
  Hits         3465     3465
  Misses       1868     1868

Flag
Coverage Δ

unittests
64.97% <ø> (ø)

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

Continue to review full report at Codecov.

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

Codecov Report

Merging #242 (c49ee4a) into v0.2 (951eaaf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             v0.2     #242   +/-   ##
=======================================
  Coverage   64.97%   64.97%
=======================================
  Files         111      111
  Lines        5333     5333
=======================================
  Hits         3465     3465
  Misses       1868     1868

Flag
Coverage Δ

unittests
64.97% <ø> (ø)

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

Continue to review full report at Codecov.

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


Netty 4.1.0.Final Release Date

We're considering moving to Netty 4.1.0 for the newly added HTTP/2 codec classes. We noticed that it is still at version .CR4, do you anticipate backward incompatible interfaces with .Final version. When is the planned release date? Thanks!

@ssheng http2 and dns codec will remain experimental even in 4.1.0.Final. This is not because we thing the code not works but because we may want to break the API later on without a need for a major release.
We not have a fixed release date but aim for the end of this month.
Hope this helps

@normanmaurer thanks.


Update token_handler.py

Type
enhancement, bug_fix

Description

Introduced error handling in the token calculation process to improve the robustness of the _get_system_user_tokens method. Now, if an error occurs, the method returns -1 instead of raising an exception.
Made a minor formatting fix in the count_tokens method to ensure consistency in the codebase.

Changes walkthrough
Relevant filesEnhancement

token_handler.pyEnhance Token Calculation Robustness and Minor Formatting Fix

pr_agent/algo/token_handler.py
Wrapped the token calculation logic in _get_system_user_tokens within a try-except block.  Returns -1 if an exception occurs during token calculation.  No change in the logic of count_tokens method, just formatting adjustment.

+10/-7

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/help


[Storage] Authentication failure ending parallel sync upload test

When ending a parallel sync upload test, an AuthenticationFailed error is raised:
~/python/sdk/storage/azure-storage-blob$ perfstress UploadTest --size 1048756 --sync --warmup 0 --duration 3 --parallel 2

=== Test ===
Current         Total           Average
30              30              30.57
31              61              30.89
30              91              30.62
Exception: Server failed to authenticate the request.
Make sure the value of Authorization header is formed correctly including the signature.
ErrorCode:AuthenticationFailed
authenticationerrordetail:The MAC signature found in the HTTP request 'REDACTED' is not the same as any computed signature.
Server used following string to sign:

I have been able to repro under the following conditions:

Client: Azure VM, size D4ds_v5
OS: Ubuntu 20.04
Python Versions: 3.7 and 3.11
Blob Size: >= 1048756
Parallel: >= 2
Sync/Async: Sync

It does not repro if any of the following are true:

OS: Windows 10 or Windows Server 2019
Blob Size: 10240
Parallel: 1
Sync/Async: Async

It repros in our nightly perf runs:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2051826&view=logs&j=42ded549-05ee-5cdb-7ba7-7a948a0cc056&t=49e32a0b-4245-531b-0723-9d45113ffbd3&l=5530
I was not able to see the failed request in the service diagnostic logs.  It's possible the service diagnostic logs do not include requests that failed authentication.
Capturing HTTP traffic in a tool like Fidder or WireShark might be helpful, though this is a little more difficult since it doesn't repro on Windows.
We suspect the issue might be related to the threading code in the perf framework, where the first thread to complete causes other threads to abort, which leads to this error.
https://github.com/Azure/azure-sdk-for-python/blob/1a39de24a23423df4766e3e87302c480865744b5/tools/azure-devtools/src/azure_devtools/perfstress_tests/_perf_stress_runner.py#L191-L194

It seems this may not be caused by exiting the ThreadPoolExecutor context prematurely - I restructured this code using the following structure across sync and async:
if args.sync:
    with ThreadPoolExecutor(max_workers=args.parallel) as ex:
        tasks = [ex.submit(test.run_all_sync, duration) for test in tests]
        wait(tasks, return_when=ALL_COMPLETED)
else:
    tasks = [asyncio.create_task(test.run_all_async(duration)) for test in tests]
    await asyncio.wait(tasks, return_when=asyncio.ALL_COMPLETED)
for task in tasks:
    task.result()

And it seems like the authentication errors are still present.

An observation regarding that auth error: From the error message we see:
Server used following string to sign: 'PUT
[08:40:30.755] [STDERR]
[08:40:30.755] [STDERR]
[08:40:30.755] [STDERR] 1048576
[08:40:30.755] [STDERR]
[08:40:30.755] [STDERR] application/octet-stream
[08:40:30.755] [STDERR]
[08:40:30.755] [STDERR]

...

The 1048576 is suspect since my content length when testing is actually 10485760.  I'm thinking something might be up with the RandomStream since the server seems to be pulling that number based on what we have here.
In UploadTest, specifying: self.upload_stream = RandomStream(self.args.size, initial_buffer_length=self.args.size)
seems to also get rid of the error.
Seems the error arises when the size is greater than the initial_buffer_length.

Confirmed that this is a race condition in the implementation of the RandomStream in the perf framework. I will add a quick lock as a short-term fix and we can do a more significant refactor of this class longer term.

The race condition is in method get_random_bytes():
https://github.com/Azure/azure-sdk-for-python/blob/a585e63970a7c83b1709785e713d37e02699a411/tools/azure-devtools/src/azure_devtools/perfstress_tests/_random_stream.py#L12-L16
If two threads call this concurrently, with the same buffer_length, and buffer_length > _BYTE_BUFFER[0]:

Thread A reaches the end of line 14, updating _BYTE_BUFFER[0], then is interrupted.
Thread B sees that buffer_length == _BYTE_BUFFER[0], so skips the if, and returns _BYTE_BUFFER[1][:buffer_length].

The problem is that _BYTE_BUFFER[1] has not been updated yet, so the returned buffer is smaller than requested, which violates the method contract and leads to the downstream bug reported above.
A lock should work as a short-term fix, but in the longer term we should be able to implement RandomStream without locks and without changing the size of the underlying buffer.


Mismatch of the head movement (raw video vs. generated video)

Hey there,
first of all great project! Had a lot fun playing around with it. My next step is to put the generated video back on top of the raw video.

To remain the real eye and forehead movement i want to mask out just the nose mouth and jaw:

Unfortunateley the movement of the raw head and the "new" head doesnt match 100%. There is still a little mismatch. It looks like the head is wobbling.
https://github.com/yerfor/GeneFacePlusPlus/assets/58291636/c773ca8e-cbb3-4883-944c-0a504c70150f
I already set the hparams['infer_smooth_camera_path_kernel_size'] = 7 to 0 and isabled the smooth featues
 smo_euler = smooth_features_xd(batch['euler'])
 smo_trans = smooth_features_xd(batch['trans'])

setting drv_pose to static didnt help. Are there any other parameters I can try to edit to achieve a perfect match of the landmarks? Especially for the nose and eyes?
Thanks in advance!

i have the same problem, hoping someone can answer this!

I have in not looked closer at the repo, but was thinking that some of the comments in thread below may be related to this issue
https://github.com/yerfor/GeneFacePlusPlus/issues/104


Core.calculateChecksum encodes the result as Big Endian instead of Little Endian

The spec for ID states that the checksum bytes should be encoded as Little Endian: https://docs.iden3.io/protocol/spec/#identifier-format

checksum: Addition (with overflow) of all the ID bytes Little Endian 16 bits ( [ type | genesis_state ] )

However, the code in Core.calculateChecksum returns the bytes as Big Endian:
https://github.com/iden3/js-iden3-auth/blob/v0.1.3-intermediate/src/core/core.ts#L39
Compare with the corresponding code in js-iden3-core, which reverses the bytes to encode as Little Endian:
https://github.com/iden3/js-iden3-core/blame/fb25d9b0bb1cab2e32ba3a2774feec84fb38d88e/src/elemBytes.ts#L41
This means an ID created by js-iden3-core (or go-iden3-core) cannot be decoded by js-iden3-auth, and vice-versa. e.g. I hit this in the AtomicQuerySigPubSignals constructor when it parses the public signals with a user ID generated from go-iden3-core.
There have been issues with this in the Go libraries as well, but go-iden3-core has recently been changed to use Little Endian: https://github.com/iden3/go-iden3-core/blob/master/id.go#L189 (though not in a release yet). The go-iden3-auth library uses the ID implementation from go-iden3-core. Perhaps the JS libraries could do similarly.
FYI @OBrezhniev

FYI @jimthematrix, Chengxuan

Hi @nedgar, @jimthematrix, @Chengxuan. As I mentioned in https://github.com/kaleido-io/kaleido-iden3-samples/issues/14 we've changed that to match the specs. This is a breaking change that impacts the whole protocol and all the parts need to be updated (issuer, verifier, circuits, smart contracts, wallet, and sdk on different languages). We plan to release everything in the end of February. And until we do a release I suggested to stick to the already released versions (tagged in the repositories, not master branches) and preferably using go libs, because only the verifier part of js libs (js-iden3-auth) was released during the previous release. You should not use js-iden3-core for now (even though we just tagged it and released on the npmjs.org), because it was not part of the release, and we were not implementing compatibility with the old release and started directly with implementation for the new one.

Thanks for clarifying @OBrezhniev. One pain point with the current version of js-iden3-auth is that it doesn't re-export the core classes and helper functions, so e.g. Id isn't available to consumers. Perhaps that could be patched until the new library versions are available?

Also, should we be using js-iden3-auth 0.1.3-intermediate (5 months ago, still marked as current) or 0.1.4 (7 months ago)?
https://www.npmjs.com/package/@iden3/js-iden3-auth?activeTab=versions

Looks like the support for basic auth in the state resolver is only in 0.1.4, according to:
npm diff --diff='<EMAIL_ADDRESS>--diff='<EMAIL_ADDRESS>So we'll use that for now unless you tell us otherwise.

@nedgar We noticed that npm was pulling updated version of js-jwz for tagged versions of js-iden3-auth, that broke verification of jwz tokens. You can update to js-iden3-auth version 0.1.5 to fix this if you had such a problem too.

And exported core lib in v0.1.6.

Thanks, @OBrezhniev. Will switch to 0.1.6 until the new release is available.

the new version is released. Core versions are synchronized.


Improving github workflow

It would be great to improve the github workflow:

A better usage of labels to differentiate the issues, at least the basic ones if we don't create custom labels
Using milestones to determinate which features should be implemented in the next version
Something more?

Make this will definitely helps us to priorities and ease the development process.
Any thought?

I like the workflow, we can start with labels.

Closing this issue since the first step is done. We can reopen it later if we think it's not enough.


Improve exception handing logic of deploy-agent cleanup

Currently, the operations to delete the actual build directory and the tarball are considered within a single exception handler.
We have seen cases on very large directories where shutil will throw an error and thus exit the exception statement. As a result, the archive is left behind as well, which is not optimal. This has also caused issue as the error thrown is not always an OSError exception, so the entire application halts.
This PR ensures that each operation is performed in an isolated exception handler to prevent the shutil operation from skipping the the gzip removal.
Apologies for the git mess - we were up all night working on another project and happened to detect this issue.
Thank you @ellezv for the help as well!

LGTM


MUMPS seems to be super slow on Ubuntu 18

Not sure if this is the right place to ask. I modified the fenics hyperelastic demo to make it with 768k tets (80x80x20 intervals).
On Ubuntu 18, I use Anaconda3 fenics package with MUMPS. Per Newton step, it took about 10 min per Newton step (one direct solve) whether or not MPI is used (when using MPI I used -np 8, the machine is with 8 cores 16 threads).
On my macbook with Anaconda fenics (same version: 2018.1.0), one Newton step took a minute or so, though parallel didn't help much either.
Not sure what's causing the difference.
Thanks

Can you share conda list?

Sometimes unexpected slow MPI performance happens due to threads (effectively overloading the CPUs). For MPI jobs you might want to constrain to 1 thread per MPI process by setting
export OMP_NUM_THREADS=1

This should be fixed now, otherwise feel free to open a new issue.


docs(firebase_crashlytics): Crashlytics setup guide

Previous guidelines resulted in the Crashlytics dashboard failing to initialise.
This PR updates documentation to produce an updated guide on how to Install Crashlytics.
Based on issue https://github.com/FirebaseExtended/flutterfire/issues/3669

Looks like one of the images isn't coming through, last one: https://flutter-firebase-docs--pr3860-dackers86-crashlyti-8zf0n9lb.web.app/docs/crashlytics/usage

@dackers86 see my previous comment about the broken image


[INTERNAL] improve test coverage: AbstractFormatter

Adds ava test for AbstractFormatter
Add JSDoc for dirExists function

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

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

Pull Request Test Coverage Report for Build 495

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

Totals

Change from base Build 491:
0.2%

Covered Lines:
1479

Relevant Lines:
2506

💛  - Coveralls

Apparently, you merged the master into this branch. This should have rather been a rebase. We should remove the merge commit and do a rebase 👍


Fix registry

With the current implementation in master, there is no way to have a custom registry. This is because we initialize a registry and then every primitive element closures around it. When this happens, any time a new custom registry is used, the original elements will not know about that new registry.
To fix this, we have to generate new elements for each registry. It also requires changing the interface to expose the new registry.
One other thing we could do would be to have minim.common or minim.global as a kind of global registry that simply calls init() on require (and is subsequently cached).
@danielgtaylor Let me know what you think. Sorry about this interface change, but I think it's necessary.

The interface I'd like to see:
var minim = require('minim');
var ElementRegistry = minim.ElementRegistry;
var data;

// Use the default registry to convert to/from refract elements
data = minim.toElement(['abc', 123]);
data = minim.fromRefract({element: 'string', content: 'foo'});
data = minim.fromCompactRefract(['string', {}, {}, 'foo']);

// Load namespaces into the default registry
minim.use(require('minim-data-structures'))
     .use(require('minim-api-description'))
     .use(require('minim-parse-result'));

// Now we can load a parse result with API elements!
data = minim.fromRefract(require('./parse-result.json'));
console.log(data.api.title); // => 'My API'

// Using a custom registry
registry = new ElementRegistry();
// The registry is now loaded with the default element set, but we can add more if we like:
registry.use(require('minim-data-structures'))
        .use(require('my-custom-namespace'));

data = minim.fromRefract(require('./custom-result.json');
console.log(data.custom.title); // => 'Custom test'

// Instantiate a single element by its class
var StringElement = registry.getElementClass('string');
console.log((new StringElement('foo')).toValue()); // => 'foo'

// This also works for subclassing, e.g. your namespace module might have:
exports.init = function (registry) {
  var StringElement = registry.getElementClass('string');

  var CustomElement = createClass(StringElement, {
    // define you custom class here...
  });

  registry.register('custom', CustomElement);
}

// Then the `.use(...)` mechanism above is really simple to implement:
ElementRegistry.prototype.use = function (mod) {
  mod.init(this);
}

Hope this makes sense. Sorry for the brain dump!

@danielgtaylor While I like this, I unfortunately think, without doing something like I've done or other fancy tricks, that your proposal above will not work when you get to a custom registry. I'll use your example.
First, how things currently work:

A registry is initialized in base.coffee
Elements are loaded and passed this registry
Each element method that uses this registry closures around the registry from base.coffee

So now, even if I create a new registry as you've done, when I do:
registry.register('custom', CustomElement);

The original Minim elements will have no visibility of this new element, even though they are included in the new registry! Of course, if you register this custom element in the global registry it will work, but here with a custom registry, it won't.
I have a thought for a proposal, but I'd like to make sure I'm right here (I think I am but that usually doesn't mean much).

@smizell any more progress made on this? Here is my proposal, hopefully written out to be a bit easier to understand:
https://gist.github.com/danielgtaylor/fe17d92c3e0679fb8ac9
It's very similar to what you've proposed but changes the public interface slightly to match up with Node.js norms and be a bit simpler (in my biased opinion). How it works:

The minim module's default export is just an instance of an ElementRegistry that has been loaded with the default elements. It also exposes the ElementRegistry class so you can use custom registries.
Namespaces are loaded via the registry's use method, which in turn just calls an init function in the namespace module. This is just like tons of other Node.js modules.
By default, no elements are loaded in a new registry. A useDefault method is added to load the default elements. This means one more step when using custom registries but allows you to build the registry entirely from scratch. Alternative idea: auto-load defaults and allow a method to clear the registry? Maybe make this a constructor option instead, but what about loading it later?
When creating a new namespace of elements, you must use the registry.getElementClass method to fetch previously defined elements. This ensures that you get the right element for the given registry, particularly if previously loaded namespaces may have replaced the default elements (is this safe and reasonable?) You must also define your element classes within the registry closure, just like you have done for Minim already.

Let me know if this isn't clear or you still think it wouldn't work!

@smizell I've done a sample implementation in #77 to show how it would all work together. Let me know what you think. This is blocking me from splitting code out of Fury, which I'd like to get to within the next couple of weeks if possible.

@danielgtaylor I like what you've done with use. I think when you create a new registry, it should have the defaults there with a method to remove them. Without them, it's just a registry of some sort (which might actually be a pattern we could abstract, but not now).
I've added some comments on #77. Would be happy to carry on the conversation there so we aren't bouncing back and forth :)


Added box for gh pr's and issues

Thanks for your contribution! And i made some minor adjustments


Patch

Better modal styles
Sequential fetching in SSG
Fix missing (tm)s in titles
Fix config files (lint staged, now -> vercel)

📝Changed routes:

/blog
/

(view full size)
(view full size)

/store/[name]/source

(view full size)

Commit 885c87b66822eced6a8c9e2b946ccbace1f7dabc (https://hyper-site-git-fork-vassbence-master.vercel.now.sh).


Leica scan file reader

I'd like to have a PTS reader, which I could use to read Leica scans in the browser and render them in 3D.
Acceptance criteria:

Browsers with memory limitations on string value should accept files up to 2GB, possibly bigger.
The reader is framework agnostic.
If the point cloud is too big, it should be possible to read evenly distributed points or randomly taken.

At the moment only the PTS format is supported.


Robert Longson: Watch orthoindehospital.in

Robert Longson requests the watch of the watch_keyword orthoindehospital\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

orthoindehospital\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Cody Gray in Charcoal HQ


[Feature Request]: Integrates Infinity with RAGFlow

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Describe the feature you'd like
Besides vector search and fulltext search, Infinity also support sparse vector search, tensor search and RRF reranker in v0.2.0. Now, we can integrate Infinity as the database of RAGFlow.
Also refer to: https://github.com/infiniflow/ragflow/issues/697, https://github.com/infiniflow/ragflow/issues/1308

Because the storage format will be changed in Infinity 0.3.0, integrating Infinity into RAGFlow is postponed in 0.4.0.

Following PRs are related to this issue.
#1861
#1862
#1803
#1944


Fix useless parameters

Some functions have parameters that don't do what they are supposed to. These need to be fixed before next release.

Can you provide an example? I am looking to contribute.

@mushahidq In multiplicationFunc the parameter maxRes should restrict the value of c so that it is not higher than maxRes. However, this is not the case as the max value of variable b is too high. Look at additionFunc and subtractionFunc to see what the parameters should do. DivisionFunc and moduloFunc don't need maxRes parameters because c will always be smaller than maxDivid or maxModulo.

I understand what the problem. I hope its alright if I use a for loop. I think a better solution would be to have all the factors stored in an array and then generate a random number with highest being the last index of the array. The problem elements would be the element stored at that position and the  quotient of its division with the solution which was generated first. Is it alright if I implement this?

I have made a PR with a fix.


Databrowser url

The data browser now updates the browser history and refreshes the url every time something has been clicked it so that now it is possible to share data-browser urls with other people.
The input-fields for the time-range now support pressing the Enter-button in order to apply time range
deleted a few now longer needed Node-Libraries
deleted a few unnecessary lines in the templates
satisfied the current version of black

LGTM


idea: support global CSS

Would it be possible to support injecting global styles via set content? Our components are basically headless without our design tokens which get loaded externally.
Instead of adding a new prop to the stencil-playwright.json, we could probably just import the stencil.config and check for globalStyle.

Hello @markacianfrani thanks for the contribution!
Question about this change, is the goal to inject the same globalScript and globalStyle from the stencil.config.ts into the test template? If so, we could potentially solve this in a more automated way by either having an option to pass through the Stencil config or somehow detecting it automatically.
The original opinionation of the stencil-playwright.json was to workaround me not being able to easily solve the problem of passing the Stencil config context to Playwright. However, if that is necessary to improve the Dx of this integration, I think it is a problem that could be solved.

Yep that was exactly the intent. I agree too that auto detecting the stencil config would be better DX. For global styles, I think it's safe to assume the developer will want those.
For global scripts, maybe not. If Im using global scripts to import another library like icons, it might be unnecessary overhead if the component Im testing doesn't use icons. Or would that not matter with stencils lazy loader?

I have a webcomponent lib. That have global style css to load fonts and styling for the components. This feature would be helpful.

This feature would be very useful.
@sean-perkins any update on the state of this PR?

thx


Pressing cancel in auth dialog should show badge on auth icon

Testing #117290

have the snippet below
select 'cancel' in the auth-allow-dialog (see screen shot)
you are now lost. I didn't allow or deny auth and I would expect that the auth icon shows the badge to guide me to retrying this

	let disposable = vscode.commands.registerCommand('extension.helloWorld', async () => {

		const session = await vscode.authentication.getSession(
			'microsoft',
			["https://management.core.windows.net/.default", "offline_access"],
			{ createIfNone: true }
		)

		console.log(session);
	});

I still don't see the badge when pressing cancel

@jrieken I'm having trouble repro'ing, any additional steps?

This is what I am doing/having

logged in to GH and allowed GHNB extension
request auth from my test extension and press cancel
now I don't see the "grant auth for test"

https://user-images.githubusercontent.com/1794099/109195839-4cf9af00-779b-11eb-862b-9a4273464e4d.mov

 vscode.commands.registerCommand('extension.helloWorld', async (args) => {
        const session = await vscode.authentication.getSession('github', [],
            { createIfNone: true }
        )

        console.log(session);
    });

This is an interesting issue because I'm not sure I'd call this a bug completely. The model flow is suppose to be for "I can't do anything else unless you give me auth" scenarios and hitting cancel there says "no sorry you can't have it I don't care what functionality I'm missing out on".
I could see people saying that the opposite should be true "I don't want to see the badge for the modal auth flow because I said I didn't wanna auth".
So I'm more leaning on this being "by-design". The badge is really only for silent notification that auth'ing gives you extra features... not for if you explicitly say you don't want auth.
What do you think @jrieken?

Fair enough.


Update shelljs to 0.8.4 to prevent circular dependency in Node v14

Current Behavior
Run tsdx test in Node v14 will output this warning
Expected behavior
Output no warnings.
Suggested solution
Simply upgrade shelljs to 0.8.4 will solve this problem
Additional context
Your environment

Software
Version(s)

TSDX
0.13.2

TypeScript
3.8.3

Browser
Chrome 81

npm/Yarn
yarn

Node
14

Operating System
Mac 10.15.4

So TSDX has ^0.8.3 in the package.json. A patch bump fits into the range, so you don't need TSDX to update that, you can update your yarn.lock or reinstall TSDX and you'll have the newer version of shelljs


Fix #1460: Update to Fable 3

fixes #1460
Requires ionide/ionide-vscode-helpers#35
About Experimental Build:
It seems like experimental builds aren't done via the build script, but by hand with probably a command like webpack --mode=production --env.ionideExperimental (which passes IONIDE_EXPERIMENTAL to Fable).
With Fable 3, compilation to JS and bundling are now split in two between Fable and webpack. Which makes the command more complicated.
I added a Build Target to for that: BuildExp -- existed already before, but was empty. Now it compiles with Fable with IONIDE_EXPERIMENTAL defined, and bundles with webpack and --env.ionideExperimental into release-exp. As far as I can tell that's how an experimental build was created before.
If experimental build should be kept out of the build script (or requires more options): The full command is now:
dotnet fable ./src/Ionide.FSharp.fsproj --outDir ./out --define IONIDE_EXPERIMENTAL --run webpack --mode=production --env.ionideExperimental

I think Experimental build is not needed - it was used back before 4.0 release, and we're not using it any more.
I've merged the helpers PR so you can bump dependencies here.

Awesome, thanks a lot for this ❤

Wow. @alfonsogarciacaro did you see this?

Sorry, didn't I tell you that all CLI options are changing in Fable 3.1? ...Just kidding 😜 Great work @Booksbaum!

Sorry, didn't I tell you that all CLI options are changing in Fable 3.1? ...Just kidding 😜 Great work @Booksbaum!


How to get values from isteven multiselect

In the getting started section there is the following code that retrieves the selected values:
angular.forEach( $scope.outputBrowsers, function( value, key ) {
/* do your stuff here */
});
I am an angular newbie. Where is this code to be placed?

Well that's a very broad question, but one example is when - let's say you need to ajax-save a list of selected IDs from the multi-select.
// First we get the list of IDs
var selectedIds = [];
angular.forEach( $scope.outputBrowsers, function( value, key ) {
    selectedIds.push( value.itemId );
});

// Then do your AJAX post, sending the list of IDs
$http({
  method: 'POST',
  url: '/someUrl',
  data: JSON.stringify( selectedIds );
}) ....

I really like your directive. Great stuff. Thanks for sharing.
Do I need to handle a close event to retrieve all selected values?
I have a child view that includes your dropdowns, I am having difficulty
executing the controller from within this form. Do you happen to know how
this is accomplished?
TIA,
Ron Tornambe
On Mon, May 30, 2016 at 6:09 AM, Steven<EMAIL_ADDRESS>wrote:

Well that's a very broad question, but one example is when - let's say you
need to ajax-save a list of selected IDs from the multi-select.
// First we get the list of IDsvar selectedIds = []; angular.forEach( $scope.outputBrowsers, function( value, key ) {
selectedIds.push( value.itemId );
});
// Then do your AJAX post, sending the list of IDs$http({
method: 'POST',
url: '/someUrl',
data: JSON.stringify( selectedIds );
}) ....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/isteven/angular-multi-select/issues/443#issuecomment-222459843,
or mute the thread
https://github.com/notifications/unsubscribe/AAraSfoLTtBa-fnskyzyXWDx76iOT3n9ks5qGrdNgaJpZM4Imo5Q
.

@rontornambe ,
Usually you'd want to retrieve the directive values during AJAX form submission (well, in fact, all the form values).
For the child view thingy, can you replicate in Plunker or JsFiddler so I can have a look?

It is a little complicated to do a fiddle. Perhaps my test-site
http://writers-tryst.ron-tornambe.com will do. We are only interested in
the Enablers view. The angular main controller is in pages.js, which is
accessible from View Source. I have attached the enablers.html which
includes the JS for the child controller. You will need to create an
account, which should only take a minute.
best,
Ron
On Tue, May 31, 2016 at 11:52 PM, Steven<EMAIL_ADDRESS>wrote:

@rontornambe https://github.com/rontornambe ,
Usually you'd want to retrieve the directive values during AJAX form
submission (well, in fact, all the form values).
For the child view thingy, can you replicate in Plunker or JsFiddler so I
can have a look?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/isteven/angular-multi-select/issues/443#issuecomment-222886000,
or mute the thread
https://github.com/notifications/unsubscribe/AAraSe6WQ4I49PimUIFLiYsSXPDjLzQfks5qHQIYgaJpZM4Imo5Q
.

@rontornambe ,
I think I'll need the fiddler / plunker.. it's kinda hard following piece of codes like the one you provided :)

No worries. I decided on another plugin,
On Mon, Jun 6, 2016 at 1:04 PM, Steven<EMAIL_ADDRESS>wrote:

@rontornambe https://github.com/rontornambe ,
I think I'll need the fiddler / plunker.. it's kinda hard following piece
of codes like the one you provided :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/isteven/angular-multi-select/issues/443#issuecomment-224021628,
or mute the thread
https://github.com/notifications/unsubscribe/AAraSdbALphDaE2AeF7KSQfXCwF_hib_ks5qJFMZgaJpZM4Imo5Q
.


feat: welcome screen

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


test: Benchmark

TODO

[x] Koa
[x] Deno
[x] uWebSocket
[ ] Summary and performance ranking

:tada: This PR is included in version 1.19.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


<EMAIL_ADDRESS>decompress error

Failed to extract files from D:\scoop\apps\postgresql13\13.8\postgresql-13.8-1-windows-x64-binaries.zip.
Log file:
D:\scoop\apps\postgresql13\13.8\7zip.log

Scoop Configuration

//# Your configuration here

7zip.log

Cannot reproduce with current version.
> scoop install postgresql13
Installing 'postgresql13' (13.13) [64bit] from versions bucket
postgresql-13.13-1-windows-x64-binaries.zip (313.3 MB) [======================================================] 100%
Checking hash of postgresql-13.13-1-windows-x64-binaries.zip ... ok.
Extracting postgresql-13.13-1-windows-x64-binaries.zip ... done.
Linking ~\scoop\apps\postgresql13\current => ~\scoop\apps\postgresql13\13.13
Creating shortcut for pgAdmin 4 (pgAdmin4.exe)
Persisting data
Running post_install script...
'postgresql13' (13.13) was installed successfully!
Notes
-----
Run run 'pg_ctl start' or 'pg_ctl stop' to start and stop the database or
register it as a service by running 'pg_ctl register -N PostgreSQL' from an elevated shell.
Default superuser login: postgres, password: <blank>


Microsoft Outlook accounts had bad outbound defaults for manual setup

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.400
Where did you get the app from?
F-Droid
Android version
Pixel Experience 12 (SP2A.220405.003) (Stock but old ROM)
Device model
Pixel 4
Steps to reproduce
With setting up a Microsoft account, optionally has 2FA enabled as I did:

Enter in your login email
Tap manual setup (required if you're using 2FA, OAuth seems to not give K-9 the ability to authenticate correctly when the account has it enabled.)
Enter in details for inbound then continue
Check details of outbound. Note that it uses SSL/TTS by default, which seems to be unsupported by Microsoft.

Expected behavior
Ideally to be defaulting to STARTTLS for @outlook.com, @hotmail.*, @live.com and @msn.com.
Actual behavior
K-9 is instead defaulting to SSL/TTS, which the Microsoft servers seem to not support for outbound. The client hangs indefinitely as it tries to connect to the server and seems to not get a response.
I know this is rather minor, but it could be a slight improvement to UX if there can be known good defaults for such things. Ideally, the OAuth setup would be sufficient for allowing quick setup even with 2FA enabled on the accounts.
Logs
No response

Improving account setup is on the roadmap. But even now, automatic setup for these domains should work fine. The fact that authentication for the outgoing server currently doesn't work, is a bug on Microsoft's side. See https://forum.k9mail.app/t/outlook-com-mail-settings/6154/6
I don't see us adding support for auto-detecting server settings when the user has explicitly selected manual setup.

Fair. It was more necessarily pointing out that the SSL/TLS route is generally broken for Microsoft's stuff, not about generally being unable to send.
Tho, it does seem that the OAuth2 route likely does need a bugfix for accounts with 2FA enabled.
Should I create a new issue for that, or would that also fall under the same forum post you linked?

Possibly related to the account setup issue?
https://blog.thunderbird.net/2023/01/important-message-for-microsoft-office-365-enterprise-users/


Tags/Lists

Description of the new feature / enhancement
When I install new software I would like to tag it / add it to a list so that when I export json file I can choose if I want full setup or just items that are tagged / on a list
Proposed technical implementation details
winget install Microsoft.WindowsTerminal --tag my-tag
or
winget install Microsoft.WindowsTerminal --add-to-list my-list

We've recently been discussing mechanisms for excluding packages from winget upgrade --all.

https://github.com/microsoft/winget-cli/issues/2339

One of the thoughts has been to also exclude those from winget export packages.json.
Would something like that express what you are asking for?

I at least would love it to be optional whether it is omitted from export or not. Usually, I just want to keep an app from being upgraded, but I use export to back up apps so I can install them on another computer or on a new installation say after a fresh install of Windows.  Just my two cents

@Massamune3210 I could see a winget export packages.json --include-excluded

that would be perfect, at least for me, though I don't see any way someone else could complain about it

One of the thoughts has been to also exclude those from winget export packages.json.
Would something like that express what you are asking for?

Yes, it will do what I have in mind, but it will also make it difficult.
I will have to exclude every default program to achieve what I want.
That looks like a lot of work.
Maybe that is something that would make it easier:
winget exclude --all


Mirror compatibility

Hello,
Is it possible to use this relay with the Mirror framework? If not how to add support for it?
Thanks.

Yes this can be used with Mirror. As long as you use the RelayTransport instead of the NetworkTransport for all UNET operations.

Great :) What about the dll in the library folder? Do I've to put them into my Unity project too? We've maybe a better place to talk about that?


Google play store link added, about screen changed

@ty221 sorry I'll correct this.

@mananwason Thank you. Can you rebase your work to one commit please, if that is not any problem?

Yeah I guess this would be cool !!

@ty221 where should the base be for the rebase?

Please rebase all your commits to one comparing them to current master branch status.

ok thanks :D

@ty221 I did rebase it, but could you please check. Thanks :D

@mananwason Hi, it seems there are still 5 commits here. Please follow cherry-picking and rebasing instrunctions as here: http://stackoverflow.com/questions/14635672/git-rebase-a-single-commit best luck :D

thanks @ty221 :D

@mananwason Just curiosity - why did you close this PR?

@ty221 I was trying rebasing, I was not sure what to do. It says rebasing done, but doesn't do anything here.


chore(DTFS2-6999): replace moment in tfm-ui

Introduction :pencil2:
Moment js is deprecated, it should be replaced. This PR will not cover removing moment from tests - see #2761 .
Resolution :heavy_check_mark:

Replace moment() with vanilla Dates where possible and date-fns where necessary in trade-finance-manager-ui

Misc

add dev test scripts to all packages

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.48%. Comparing base (fbe7ec4) to head (b4748d5).
Report is 70 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #2791       +/-   ##
==========================================
- Coverage   22.57%   0.48%   -22.10%
==========================================
  Files        1136     104     -1032
  Lines       27255    1452    -25803
  Branches     4486     361     -4125
==========================================
- Hits         6153       7     -6146
+ Misses      18388    1230    -17158
+ Partials     2714     215     -2499

Flag
Coverage Δ

api-test
0.48% <ø> (ø)

ui-test
?

unit-test
?

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

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


fix origin x of TOPasscodeButtonLabel's label.

numberLabel and letteringLabel  was just a little out of position.
before

after

diff

Hey @RyoAbe! Thanks for the PR.
Hmm. Centering the text and making the label view widths the same as the button view is certainly one way to get more granular control, but I feel like that might also be incurring a performance hit.
I chose to use [UILabel sizeToFit] to make sure the labels were only as big as they needed to be. When they're bigger than that, Core Animation needs to do more blending passes with the extra pixels in the bigger label, which means slower performance (And this view controller is already pretty heavy).
Here's an example of what I mean:

The red should only encompass where the text is being drawn, but it accounts for the whole label bounds.
The reason the labels are probably off-center is because CGRectIntegral rounds decimal numbers to whole numbers, which means it's not taking advantage of the Retina display.
It might be more appropriate to replace CGRectIntegral with something that can handle 2x on Retina and 3x on Retina HD. :)
What do you think?


Unable to import module located in a folder with "." in its name.

When importing a module that exists in a folder with "." in its name, the compiler is not able to resolve that module, e.g.:
{-- File location: /Project.One/View.elm --}
module Project.One.View where

..

{-- File location: /Project.One/Main.elm --}
module Project.One.Main where

import Project.One.View exposing (..)
..

and when I compile the Main.elm, I got a compile error:
I cannot find find module 'Project.One.View'.

Module 'Project.One.Main' is trying to import it.

One workaround for this is to use symbolic link to the folder with "." with a link without the ".", but it will be great if we can use folder with "." in it especially when working on legacy project that already has a lot of folders that can't be easily renamed.

If we were to allow this, how would the compiler know if you were looking for Project/One/View or Project.One.View, particularly if both were present?

I'd recommend making the source-directories field in your elm-package.json point to the deeper thing. Like, put the Elm all in one place, and have it point to Project.One/src/ and have it all be in there.
Basically, I think there are work arounds here, and unless you can prove that there is literally nothing that can be done, I wouldn't want to support this.


ntfs.yaml updates

Added $LogFile as an artifact specific to NTFS.

@macuisdein thx for the proposed changes, sry for the slower response on my side.
I've made some minor tweaks but will merge the proposed changes.


Clarification on goals and PRs for firmware

CC @ImUrX @Noxime @carl-anders @kitlith
Some links may be broken. I'll fix that eventually
Please note the scope of feedback that I'm primarily interested in:

Any missing clarifications that are needed?
Typos, prose
Broken links

Please note that while I am happy to hear opinions about the goals as listed, I am not necessarily going to change them.

Gave it a second pass to make goals simpler.

will merge tomorrow end of day (GMT-5) if no further clarification is needed


Cortex secret management

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[ ] All new and existing tests passed.

@infa-bapa Thanks for the Pull Request. Due to changes in policy i will have to push it from my branch though.

@ranberant  can you also release this helm chart please?
$ helm template cortex-xdr --repo https://paloaltonetworks.github.io/cortex-helm --version 1.1.0
Error: chart "cortex-xdr" version "1.1.0" not found in https://paloaltonetworks.github.io/cortex-helm repository


Fix {target_harness} expansion in template

This seems to have been inadvertently broken by 41f973184ec7653c7048872362e47e2ca9999f89.
The missing variable expansion was exposed by #2789, so tasks using {target_harness} (which is not a valid expansion) fail.

Codecov Report

Merging #2866 (0bc3178) into main (1b07b7d) will decrease coverage by 1.50%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2866      +/-   ##
==========================================
- Coverage   28.09%   26.60%   -1.50%
==========================================
  Files         302      125     -177
  Lines       35629    13408   -22221
==========================================
- Hits        10011     3567    -6444
+ Misses      25618     9841   -15777

Impacted Files
Coverage Δ

src/agent/input-tester/src/test_result/mod.rs

src/agent/onefuzz-task/src/tasks/fuzz/generator.rs

src/agent/input-tester/src/logging.rs

src/agent/stacktrace-parser/src/lib.rs

...c/agent/dynamic-library/src/bin/dynamic-library.rs

src/agent/coverage/src/timer.rs

src/agent/srcview/src/modoff.rs

src/agent/coverage/src/cobertura.rs

src/agent/onefuzz/src/memory/tests_linux.rs

src/agent/onefuzz/src/monitor/tests.rs

... and 167 more

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

This is a real bug, but it only lingered because that template has been superseded by what we currently call libfuzzer dotnet_dll.
Since a major release is coming up, now would be a great time to just remove the old POC implementation entirely. PR for that here: #2875

Closing in favour of #2875


Update version: egoist.devdocs-desktop version 0.7.2

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

(Automated response - build 602.)

This package appears to be of portable type.

@wingetbot waivers Add Validation-Unattended-Failed

/AzurePipelines run


testRemovePartiallyUploaded test in gsc gateway fails: "The bucket you tried to delete is not empty"

During minio gcs gateway testing, testRemovePartiallyUploaded test fails with "The bucket you tried to delete is not empty" error message. That is because the partially uploaded file, which is supposed to be not created at gcs, is created and causes the problem.
$ docker run -e MINT_MODE=full -e SERVER_ENDPOINT=<IP_ADDRESS>:9000 -e ACCESS_KEY=<access_key> -e SECRET_KEY=<secret_key> -e ENABLE_HTTPS=0 minio/mint:edge minio-go
Running with
SERVER_ENDPOINT: <IP_ADDRESS>:9000
ACCESS_KEY:      GOOG2LZARBHVS5VB63L
SECRET_KEY:      ***REDACTED***
ENABLE_HTTPS:    0
SERVER_REGION:   us-east-1
MINT_DATA_DIR:   /mint/data
MINT_MODE:       full

To get logs, run 'sudo docker cp 40ea7a24e3b6:/mint/log /tmp/mint-logs'
(1/1) Running minio-go tests ...

Here is the captured info from log file:
{
  "args": {
    "bucketName": "minio-go-test-ltg23df1dt3tqfah",
    "objectName": "minio-go-test-ltg23df1dt3tqfah-resumable"
  },
  "duration": 1403,
  "error": "The bucket you tried to delete is not empty",
  "function": "RemoveIncompleteUpload(bucketName, objectName)",
  "message": "Cleanup failed",
  "name": "minio-go: testRemovePartiallyUploaded",
  "status": "FAIL"
}

opened issue in server repo: https://github.com/minio/minio/issues/5216


Support Button Row

What's in this PR?
This PR aims to support button row/header/footer.
This DSL
 val f2 = buttonFooter {
  onButtonSetupListener = {
    it.setBackgroundResource(R.drawable.widget_button)
    it.text = getString(R.string.app_name)
  }
  onClickListener = { _, _ ->
    toast("Button is clicked")
  }
}

produces this

Merging this in as no one reviews this 😢


Adds settings.pages to enable one bot instance to listen to more than

(Related to https://github.com/botmasterai/botmaster-messenger/issues/5)
This is needed because eg. in FB, one app can subscribe to changes in
more than one page, so we need to enable one bot to listen to more than
one pages.

Coverage remained the same at 100.0% when pulling e5a0631819b87cea9d234f3cc0b4c0a82147a3e9 on andremw:feature/multi-pages into c1da4b267f5c911a0ef24eb19a8ec1dd923db593 on botmasterai:master.

Thanks for the pull request @andremw. I think you have the general right idea for what is needed to add what you would like to botmastger-messenger. However, The changes would have to be made only in the botmaster-messenger package. That is because pages are really only used in FB Messenger and this should not be referenced in the core package.
I realise the issue you were trying to solve was that of requiredCredentials. It looks like the easiest way to solve that by only working on botmaster-messenger would be to add something like
this.optionalCredentials = ['pages', 'pageToken']

In the botmaster-messenger contructor. Then adding a _checkOptionalCredentials method where .you perform a logic similar to what you have here.
There is another issue on the PR you have made to botmaster-messenger. But I'll expand on it on the other PR.
Thanks again for working on this. Hopefully the next PR to botmaster-messenger will be the one


Make ArmResetISR generic

Moved ResetISR from the individual startup.cpp files and put them in the
exceptions.cpp file.
Changed name of HardFaultHandler and ResetIsr, to ArmHardFaultHandler
and ArmResetHandler.
Resolves #642

Coverage remained the same at 75.692% when pulling 220eba853673fb62f3ced1921bcee42065812c77 on generic-arm-resetisr into 8839d0d79fd9408769651d38b894b437a1570805 on master.


Traefik gateway time out with certain apps

Welcome!

[x] Yes, I've searched similar issues on GitHub and didn't find any.
[x] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Migration from Docker in a Debian Jail to Native Docker on Truenas Scale....
Applications:
Prometheus
Grafana
Nextcloud
each were showing gateway time out..... At first I did not think it was traefik...
I striped nextcloud down to bare bones and even reinstalled it (clean install), it will work through traefik for a few moments then get a gateway time out..
Next I moved on to troubleshoot graphana and Prometheus and I realized as soon as I published a port and access them with Ip and port  they worked.....
Now that I knew it was a traefik issue I began looking into the app traefik config which is simple for graphana and Prom...
Prom compose labels
labels:
      - traefik.enable=true
      - traefik.http.routers.prometheus.entrypoints=websecure
      - traefik.http.routers.prometheus.rule=Host(`${PROMETHEUS_DN}`)
      - traefik.http.routers.prometheus.tls=true
      - traefik.http.services.prometheus.loadbalancer.server.port=9090
      - traefik.http.routers.prometheus.middlewares=authelia@docker

after not being able to fix it I went back to the other docker install and everything was working....
All using the same version v3.2.0
then I realized the version were not the same as the image had been updated, index digest was different.... also the pinned versions all recent ones were updated and the issue is present even on v3.1.7....
I then went back to a version that the image had not been updated v3.1.6 and alleviated the issue
What did you see instead?
Gateway time out on
Applications:
Prometheus
Grafana
Nextcloud
***The other 43 applications are working fine
What version of Traefik are you using?
v3.2.0
sha256:d12741b971f5c833919db2b680191dc23d5d96ef17e41a914e3c593a8c663ac4
Current on docker hub
I moved back to v3.1.6 to relieve the issue
What is your environment & configuration?
My ingress Stack with Traefik
networks:
  authelia:
    name: authelia
    ipam:
      driver: default
      config:
        - subnet: <IP_ADDRESS>/24
          ip_range: <IP_ADDRESS>/24
          gateway: <IP_ADDRESS>
  proxy:
    name: proxy
    external: true
services:
  traefik:
    container_name: traefik
    image: traefik:${TRAEFIK_VERSION}
    hostname: traefik
    user: 0:568
    ports:
      - 80:80
      - 443:443
    expose:
      - 8080
    restart: unless-stopped
    environment:
      - TZ=${TZ}
      - CF_DNS_API_TOKEN=${CFT}
    networks:
      proxy:
        ipv4_address: <IP_ADDRESS>
      authelia: null
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    labels:
      - traefik.enable=true
      - traefik.http.routers.traefik.entrypoints=websecure
      - traefik.http.routers.traefik.rule=Host(`${TRAEFIK_DN}`)
      - traefik.http.routers.traefik.tls=true
      - traefik.http.routers.traefik.tls.certresolver=cloudflare
      - traefik.http.routers.traefik.tls.domains[0].main=${FQDN}
      - traefik.http.routers.traefik.tls.domains[0].sans=${SDN}
      - traefik.http.routers.traefik.service=api@internal
      - traefik.http.services.traefik.loadbalancer.server.port=8080
      - traefik.http.routers.traefik.middlewares=authelia@docker
    volumes:
      - /var/run/docker.sock:/run/docker.sock:ro
      - ${TRAEFIK_YAML}:/etc/traefik/traefik.yaml:ro
      - ${TRAEFIK_CONFIG}:/etc/traefik/config.yaml:ro
      - ${TRAEFIK_CERTS}:/etc/traefik/certs/
  lldap:
    container_name: lldap
    image: lldap/lldap:${LLDAP_VERSION}
    hostname: lldap
    user: 0:568
    expose:
      - 17170 #WEB
      - 3890 #Database
    restart: unless-stopped
    environment:
      - PUID=568
      - TZ=${TZ}
      - LLDAP_JWT_SECRET=${SECRET}
      - LLDAP_KEY_SEED=${SECRET}
      - LLDAP_LDAP_BASE_DN=${DN}
    networks:
      - proxy
      - authelia
    labels:
      - traefik.enable=true
      - traefik.http.routers.lldap.entrypoints=websecure
      - traefik.http.routers.lldap.rule=Host(`${LLDAP_DN}`)
      - traefik.http.routers.lldap.tls=true
      - traefik.http.services.lldap.loadbalancer.server.port=17170
      - traefik.http.routers.lldap.middlewares=authelia@docker
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    volumes:
      - ${LLDAP_DATA}:/data
  authelia:
    container_name: authelia
    image: authelia/authelia:${AUTHELIA_VERSION}
    hostname: authelia
    user: 0:568
    expose:
      - 9091
    restart: unless-stopped
    environment:
      - TZ=${TZ}
      - PUID=568
    networks:
      - proxy
      - authelia
    labels:
      - traefik.enable=true
      - traefik.http.routers.authelia.rule=Host(`${AUTH_DN}`)
      - traefik.http.routers.authelia.entrypoints=websecure
      - traefik.http.routers.authelia.tls=true
      - traefik.http.middlewares.authelia.forwardauth.address=http://authelia:9091/api/verify?rd=https://${AUTH_DN}
      - traefik.http.middlewares.authelia.forwardauth.trustForwardHeader=true
      - traefik.http.middlewares.authelia.forwardauth.authResponseHeaders=Remote-User,Remote-Groups,Remote-Name,Remote-Email
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    depends_on:
      - redis
      - mariadb-authelia
    volumes:
      - ${AUTHELIA_CONFIG}:/config
  redis:
    container_name: redis
    image: bitnami/redis:${REDIS_VERSION}
    user: 5000:0
    expose:
      - 6379
    restart: unless-stopped
    environment:
      REDIS_PASSWORD: ${SECRET}
      TZ: ${TZ}
    networks:
      - authelia
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    volumes:
      - ${REDIS_DATA}:/bitnami/
  mariadb-authelia:
    container_name: mariadb-authelia
    image: linuxserver/mariadb:${MARIADB_AUTHELIA_VERSION}
    user: 0:5000
    expose:
      - 3306
    restart: unless-stopped
    environment:
      MYSQL_ROOT_PASSWORD: ${SECRET}
      MYSQL_ROOT_USER: root
      MYSQL_DATABASE: ${AUTHELIA_USER}
      MYSQL_USER: ${AUTHELIA_USER}
      MYSQL_PASSWORD: ${SECRET}
      PUID: 5000
      TZ: ${TZ}
    networks:
      - authelia
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    volumes:
      - ${MARIADB_AUTHELIA_CONFIG}:/config
  homarr:
    container_name: homarr
    image: ghcr.io/ajnart/homarr:${HOMARR_VERSION}
    hostname: homarr
    user: 568:568
    expose:
      - 7575
    restart: unless-stopped
    environment:
      - DEFAULT_COLOR_SCHEME=dark
    networks:
      - proxy
    labels:
      - traefik.enable=true
      - traefik.http.routers.homarr.entrypoints=websecure
      - traefik.http.routers.homarr.rule=Host(`${HOMARR_DN}`)
      - traefik.http.routers.homarr.tls=true
      - traefik.http.services.homarr.loadbalancer.server.port=7575
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    volumes:
      - ${HOMARR_CONFIG}:/app/data/configs
      - ${HOMARR_ICONS}:/app/public/icons
      - ${HOMARR_DATABASE}:/data
  uptime-kuma:
    container_name: uptime-kuma
    image: louislam/uptime-kuma:${UPTIMEKUMA_VERSION}
    expose:
      - 3001
    restart: unless-stopped
    environment:
      - UPTIME_KUMA_DISABLE_FRAME_SAMEORIGIN=true
    networks:
      - proxy
    labels:
      - traefik.enable=true
      - traefik.http.routers.uptime-kuma.entrypoints=websecure
      - traefik.http.routers.uptime-kuma.rule=Host(`${UPTIMEKUMA_DN}`)
      - traefik.http.routers.uptime-kuma.tls=true
      - traefik.http.services.uptime-kuma.loadbalancer.server.port=3001
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G
    volumes:
      - ${UPTIMEKUMA_DATA}:/app/data
      - /var/run/docker.sock:/var/run/docker.sock:ro
  nginx:
    container_name: nginx
    image: tws101/homelab:${WEBSERVER_VERSION}
    restart: unless-stopped
    expose:
      - 80
    networks:
      - proxy
    labels:
      - traefik.enable=true
      - traefik.http.routers.nginx.entrypoints=websecure
      - traefik.http.routers.nginx.rule=Host(`${FQDN}`)
      - traefik.http.routers.nginx.tls=true
      - traefik.http.services.nginx.loadbalancer.server.port=80
    deploy:
      resources:
        limits:
          cpus: "1"
          memory: 2G

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
Nothing was logged, the log was empty

Hello @tws101,
Thanks for reaching out!
To keep the repository focused, we ask that all questions be asked in the community forum. It is pretty active, so you might find that your question has already been answered there.  If not, you can ask and get help from other community members pretty quickly.


Using MMMarkdown, only system font will show bold or italicized text

Hi,
It works very nice unless I want to use my custom font.
NSString *htmlString    = [MMMarkdown HTMLStringWithMarkdown:result extensions:MMMarkdownExtensionsGitHubFlavored error:nil];
TRIED TO CHANGE EXTENSIONS EACH OF THEM ASKING FOR.
NSError *error = nil;

NSAttributedString *attString = [[NSAttributedString alloc] initWithData:[htmlString dataUsingEncoding:NSUTF8StringEncoding] options:@{NSDocumentTypeDocumentAttribute:NSHTMLTextDocumentType, NSCharacterEncodingDocumentAttribute:@(NSUTF8StringEncoding)} documentAttributes:nil error:&error];

if (error) {
    NSLog(@"Error: %@ %s %i", error.localizedDescription, __func__, __LINE__);
} else {
    // Clear text view
   myTextView.text = @"";
    // Append the attributed string

  [myTextView.textStorage appendAttributedString:attString]; TRIED THIS

  myTextView.attributedText = attString;  TRIED THIS

}

Any solution any approach to solve this would be great
Thanks.

It is a uitextview in interface builder. font is set there, also I tried to change font in code.
Not worked.

I'm not sure. This isn't really MMMarkdown-specific.


make two widgets with the same type working ?

could  someone give me a hint on  how to make two widgets  with the same type working in a same time?

This feature is work in progress.

@samyzemmouri  that feature is not available yet.


Request: add recipe to emscripten-forge

Currently cramjam is being built as a package for Pyodide, but there is no recipe for emscripten-forge. It would be really nice to add this recipe so that one could have more options to use it on WASM. All that needs to be done is place a couple of files here, which should be pretty much the same as what is already present in the cramjam feedstock.

Sounds cool! I've got some other TODOs here in this project as well but I've been extra busy with family and work lately; many apologies.

No worries, thanks so much!

xref: https://github.com/emscripten-forge/recipes/pull/1321


Explore Onyx Data Caching for heavy components

Problem
Components wrapped with withOnyx often spend a lot of time in render phase.
One of the issues is that it appears like on subsequent re-renders these components take equally long to retrieve the data. It seems like this is something that onyx should handle better.
Ideally, re-rendering onyx component w/o changes to the data it fetches shouldn’t result in long query times but rather should be instant.
Solution
Explore better data caching in app components

Tracking this here so I can find it later:
https://github.com/Expensify/react-native-onyx/pull/401


fix(permissions): order permissions alphabetically before setting them

Also exempted some transactions from requiring permissions (most of balances, all of staking, sudo
and session)

:tada: This PR is included in version 4.0.0-alpha.30 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-beta.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: do8uiwgngj0w6
Region: us-east-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

@useorder,  we're closing this ticket because we haven't heard back from you. Please re-open if this is still an issue.


pgsql: Reduce cost of GetNotification by 2.5x

This PR optimizes specifically searchNotificationLayerIntroducingVulnerability, called twice by GetNotification.
By delaying the Layer join to the very end, we can cut the query costs from 540,836 to 219,477. Below are results of EXPLAIN ANALYZE on one of the worth case found today. We can observe a reduction of execution time from 5470.466 ms down to 2521.073 ms. This query fully uses the index added in #278. When executed with the parameters used in #278 (redacted), a >10x gain is seen.
Original query
WITH subquery AS (
  SELECT l.ID, l.name
  FROM Vulnerability_Affects_FeatureVersion vafv, FeatureVersion fv, Layer_diff_FeatureVersion ldfv, Layer l
  WHERE l.id >= $2
    AND vafv.vulnerability_id = $1
    AND vafv.featureversion_id = fv.id
    AND ldfv.featureversion_id = fv.id
    AND ldfv.modification = 'add'
    AND ldfv.layer_id = l.id
  ORDER BY l.ID
)
SELECT *
FROM subquery
LIMIT $3;

Limit  (cost=540862.93..540864.93 rows=100 width=278) (actual time=5466.087..5466.611 rows=100 loops=1)
  CTE subquery
    ->  Sort  (cost=540836.30..540862.93 rows=10650 width=92) (actual time=5466.078..5466.186 rows=100 loops=1)
          Sort Key: l.id
          Sort Method: external sort  Disk: 11456kB
          ->  Nested Loop  (cost=217.06..540123.90 rows=10650 width=92) (actual time=282.663..5170.733 rows=132227 loops=1)
                ->  Nested Loop  (cost=216.62..478346.31 rows=13379 width=4) (actual time=39.503..3052.648 rows=144392 loops=1)
                      Join Filter: (vafv.featureversion_id = ldfv.featureversion_id)
                      ->  Nested Loop  (cost=0.72..165.36 rows=15 width=8) (actual time=0.022..0.056 rows=3 loops=1)
                            ->  Index Only Scan using vulnerability_affects_feature_vulnerability_id_featureversi_key on vulnerability_affects_featureversion vafv  (cost=0.43..40.56 rows=15 width=4) (actual time=0.011..0.018 rows=3 loops=1)
                                  Index Cond: (vulnerability_id = 1367197)
                                  Heap Fetches: 3
                            ->  Index Only Scan using featureversion_pkey on featureversion fv  (cost=0.29..8.31 rows=1 width=4) (actual time=0.005..0.006 rows=1 loops=3)
                                  Index Cond: (id = vafv.featureversion_id)
                                  Heap Fetches: 3
                      ->  Bitmap Heap Scan on layer_diff_featureversion ldfv  (cost=215.90..31692.20 rows=14922 width=8) (actual time=13.179..914.892 rows=48131 loops=3)
                            Recheck Cond: (featureversion_id = fv.id)
                            Rows Removed by Index Recheck: <redacted>
                            Filter: (modification = 'add'::modification)
                            Rows Removed by Filter: 190
                            Heap Blocks: exact=35226 lossy=80070
                            ->  Bitmap Index Scan on layer_diff_featureversion_featureversion_id_layer_id_idx  (cost=0.00..212.17 rows=15060 width=0) (actual time=10.916..10.916 rows=48320 loops=3)
                                  Index Cond: (featureversion_id = fv.id)
                ->  Index Scan using layer_pkey on layer l  (cost=0.44..4.61 rows=1 width=92) (actual time=0.009..0.010 rows=1 loops=144392)
                      Index Cond: ((id = ldfv.layer_id) AND (id >= $2))
  ->  CTE Scan on subquery  (cost=0.00..213.00 rows=10650 width=278) (actual time=5466.083..5466.409 rows=100 loops=1)
Planning time: 1.421 ms
Execution time: 5470.466 ms

New query
EXPLAIN ANALYZE WITH LDFV AS (
  SELECT ldfv.layer_id
  FROM Vulnerability_Affects_FeatureVersion vafv, FeatureVersion fv, Layer_diff_FeatureVersion ldfv
  WHERE ldfv.layer_id >= $2
    AND vafv.vulnerability_id = $1
    AND vafv.featureversion_id = fv.id
    AND ldfv.featureversion_id = fv.id
    AND ldfv.modification = 'add'
  ORDER BY ldfv.ID
)
SELECT l.id, l.name
FROM LDFV, Layer l
WHERE LDFV.layer_id = l.id
LIMIT $3

 Limit  (cost=218662.94..219477.65 rows=100 width=92) (actual time=2517.235..2518.658 rows=100 loops=1)
   CTE ldfv
     ->  Sort  (cost=218633.61..218662.50 rows=11556 width=8) (actual time=2517.210..2517.323 rows=100 loops=1)
           Sort Key: ldfv_1.id
           Sort Method: external sort  Disk: 2328kB
           ->  Nested Loop  (cost=179.94..217853.79 rows=11556 width=8) (actual time=35.591..2299.412 rows=132227 loops=1)
                 Join Filter: (vafv.featureversion_id = ldfv_1.featureversion_id)
                 ->  Nested Loop  (cost=0.72..165.36 rows=15 width=8) (actual time=0.022..0.065 rows=3 loops=1)
                       ->  Index Only Scan using vulnerability_affects_feature_vulnerability_id_featureversi_key on vulnerability_affects_featureversion vafv  (cost=0.43..40.56 rows=15 width=4) (actual time=0.011..0.017 rows=3 loops=1)
                             Index Cond: (vulnerability_id = 1367197)
                             Heap Fetches: 3
                       ->  Index Only Scan using featureversion_pkey on featureversion fv  (cost=0.29..8.31 rows=1 width=4) (actual time=0.007..0.009 rows=1 loops=3)
                             Index Cond: (id = vafv.featureversion_id)
                             Heap Fetches: 3
                 ->  Bitmap Heap Scan on layer_diff_featureversion ldfv_1  (cost=179.22..14351.46 rows=12888 width=12) (actual time=11.860..667.419 rows=44076 loops=3)
                       Recheck Cond: ((featureversion_id = fv.id) AND (layer_id >= $2))
                       Rows Removed by Index Recheck: <redacted>
                       Filter: (modification = 'add'::modification)
                       Rows Removed by Filter: 190
                       Heap Blocks: exact=24973 lossy=80092
                       ->  Bitmap Index Scan on layer_diff_featureversion_featureversion_id_layer_id_idx  (cost=0.00..176.00 rows=13008 width=0) (actual time=10.231..10.231 rows=44265 loops=3)
                             Index Cond: ((featureversion_id = fv.id) AND (layer_id >= $2))
   ->  Nested Loop  (cost=0.44..94148.66 rows=11556 width=92) (actual time=2517.233..2518.458 rows=100 loops=1)
         ->  CTE Scan on ldfv  (cost=0.00..231.12 rows=11556 width=4) (actual time=2517.215..2517.548 rows=100 loops=1)
         ->  Index Scan using layer_pkey on layer l  (cost=0.44..8.12 rows=1 width=92) (actual time=0.004..0.005 rows=1 loops=100)
               Index Cond: (id = ldfv.layer_id)
 Planning time: 0.762 ms
 Execution time: 2521.073 ms

FWIW, straight-forward request to ensure correctness.
WITH
  S1 AS (
      SELECT l.ID, l.name
      FROM Vulnerability_Affects_FeatureVersion vafv, FeatureVersion fv, Layer_diff_FeatureVersion ldfv, Layer l
      WHERE l.id >= $1
        AND vafv.vulnerability_id = $2
        AND vafv.featureversion_id = fv.id
        AND ldfv.featureversion_id = fv.id
        AND ldfv.modification = 'add'
        AND ldfv.layer_id = l.id
      ORDER BY l.ID
  ),
  S2 AS (
      SELECT ldfv.layer_id
      FROM Vulnerability_Affects_FeatureVersion vafv, FeatureVersion fv, Layer_diff_FeatureVersion ldfv
      WHERE ldfv.layer_id >= $1
        AND vafv.vulnerability_id = $2
        AND vafv.featureversion_id = fv.id
        AND ldfv.featureversion_id = fv.id
        AND ldfv.modification = 'add'
      ORDER BY ldfv.ID
    )
SELECT COUNT(*) - $3 FROM
(
  (
    SELECT *
    FROM S1
    LIMIT $3
  )
  UNION
  (
    SELECT l.id, l.name
    FROM S2, Layer l
    WHERE S2.layer_id = l.id
    LIMIT $3
  )
) x


Are typings for Client correct?

Perhaps I'm misunderstanding something...
Describe the bug
import pyEX as iex

client = iex.Client(
            api_token="Tpk_99999, version='sandbox')

return client.exchanges()

is valid use of the API (from my understanding), and seems to functionally work fine (i.e. it returns an array of exchanges).
However this gives me a type error (using Pyright)
error: Cannot access member "exchanges" for type "Client"

whereas if I access exchanges directly from the API the types work fine:
return iex.exchanges()

To Reproduce
Steps above
Expected behavior
Would expect the same types to be returned for client.exchanges as for iex.exchanges
Is there a way of workaround/solution?

I don't see any bugs in this library, perhaps the typing tool does not fully support what python as a language supports

Not sure I fully understand - shouldn't the typings given by the library match what the library offers? Does your type tooling give you the correct results in the above case @timkpaine ?


IE8 & IE9 not loading images

I'm getting the following error when trying to view in IE8 and IE9 through modern.ie Win7.
HTML1201: github.io is a website you've added to Compatibility View.
picturefill
SCRIPT438: Object doesn't support property or method 'querySelectorAll'
picturefill.js, line 1004 character 3

@julzmon Looks like you're running IE8 and IE9 in compatibility mode. Is there a reason for that or was it an accident? If it's on purpose, what version of IE are you running compatibility mode as?

@mike-engel Should we maybe add:
<meta http-equiv="X-UA-Compatible" content="IE=edge">

to the <head> to ward this off going forward?

@mike-engel Yeah I was testing it both ways. Without compatibility mode I still get this error.
'picImg' is undefined  picturefill.js, line 1221 character 4

It was a fresh download from modern.ie pictured IE8 7601.17514

It seems this picImg thing is a merge error. There is no picImg and no setIntrnisicSize in 3.0. Somehow it is a mixture between 2.0 and 3.0 version. If you use the real 3.0 (https://cdn.rawgit.com/scottjehl/picturefill/3.0/dist/picturefill.js) you should not get these errors.


Neuralangelo vs Bakedangelo: differences

Hi, I'm working on some surface reconstruction tasks and I've wondered what is the difference between neuralangelo and bakedangelo, if any of you could sum it up in some points. I also haven't found any paper about bakedangelo so far, only neuralangelo and bakedsdf individually.
Thanks in advance.

Hi, bakedangelo has a better background modeling where we use proposal networks for both SDF and the background NeRF model. And since we are using proposal networks, the sampling is more efficient and can be trained with only a singel GPU instead of 8 GPUs in neuralangelo.

Hi, bakedangelo has a better background modeling where we use proposal networks for both SDF and the background NeRF model. And since we are using proposal networks, the sampling is more efficient and can be trained with only a singel GPU instead of 8 GPUs in neuralangelo.

Thanks for answering.
Is there any paper or resource for bakedangelo i can read more closely and which I can quote? I'm planning on writing a thesis work and I would like to cite all the resources I can. I would like to justify the usage of bakedangelo over typical neuralangelo watching also various metrics that I'm evaluating.

Hi, we don't have a paper for sdfstudio. You can check the code for details. You can simply cite sdfstudio :)

Thanks for the help then, I most definitely will!


Feature/av user port

This is initial implementation of user-authorizations service based on avUser from availity web.

Can pull out caching logic.


v1.7.0 lacks APK

Could you please attach the APK to the release of v1.7.0 – for those outside of Playstore? Thanks!

@chinesedfan @housseindjirdeh any news? The APK is still missing.

@chinesedfan @housseindjirdeh still nothing?

Ugh, any chance – or must we give up on the app?

@IzzySoft Be patient or forget this app temporarily. I have been ready to build a new release for several months. But without @housseindjirdeh's help/account/permissions, nothing can be done.

Sad, but what can we do? @housseindjirdeh please don't delay this for too long!

@IzzySoft Finally. I built a new release, v1.7.1.

Thanks! But you forgot to increase versionCode (which is still set to 7 as was v1.5.0 already). Could you please fix and replace, @chinesedfan? Let me know when it's done, as this time I need to fetch the update manually (as the APK with the wrong versionCode is "in the way").

@IzzySoft Because git only contains 6(1.4.1), I increased it to be 7(1.7.1). Do you know how to determine the right next versionCode?

@chinesedfan it must simply be bigger than any one you've used before. You can use it as "release counter", or simply use the current timestamp (I'd not really recommend the latter – but it's just to give you an idea). And it must be integer.
To find out which you've used before, you can run aapt d badging on the other APKs, and grep for versionCode. this is the result for 1.5.0 and 1.7.1:
package: name='com.gitpoint' versionCode='7' versionName='1.5.0' platformBuildVersionName='8.0.0'
package: name='com.gitpoint' versionCode='7' versionName='1.7.1' platformBuildVersionName='8.0.0'

@IzzySoft Updated versionCode to 10. I guessed by the following way. :P

7, 1.5.0
8, 1.6.0
9, 1.7.0
10, 1.7.1

Apologies @IzzySoft, I've been hands off this project for the past few months so I'm sorry about that. @chinesedfan has been leading the project thus far so now I'm making sure he has complete permissions/control over releases and not just me.
@chinesedfan thank you,I wish I kept note of the last versionCode but as long as it's large that's fine. Okay to start and move from 10 onwards :D

Thanks @chinesedfan and @housseindjirdeh! I've just grabbed the APK manually and checked, it looks fine! Triggered a manual update in my repo, so it should show up in a few minutes.
Guess this issue can be closed then. Please keep up attaching the APKs to the tags, and of course increasing versionCode with each new release :smile:


rem function doesn't work

Using the rem function results in this error:
error: 'remainder' was not declared in this scope
                          __result = obj<number>(remainder(number::to<real_t>(num), number::to<real_t>(div)));

This also means that the mod function does not work.

Apparently avr-gcc does not implement remainder function. Replaced the call with % operator.


chore(avm): migrate to template engine

Only lookups and permutations missing.

#7316  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

proof_construction_time_poseidon_hash_100_ms (16): 719 (-24%)
proof_construction_time_poseidon_hash_30_ms (16): 203 (-29%)
proof_construction_time_poseidon_hash_30_ms (32): 221 (-16%)
proof_construction_time_sha256_ms (16): 708 (-23%)
avm_simulation_time_ms (Token:mint_public): 68.8 (-87%)
avm_simulation_time_ms (Token:assert_minter_and_mint): 180 (+195%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,762 (+1%)
1,558 (-1%)
:warning: 708 (-23%)
774 (-7%)
777 (-5%)

proof_construction_time_sha256_30_ms
12,039
3,183 (+1%)
1,412
1,423 (-1%)
1,473

proof_construction_time_sha256_100_ms
44,036
11,889 (+1%)
5,487
5,491 (+1%)
5,396 (+1%)

proof_construction_time_poseidon_hash_ms
78.0
34.0 (-3%)
34.0 (-23%)
57.0 (-12%)
89.0 (-1%)

proof_construction_time_poseidon_hash_30_ms
1,527 (+1%)
418 (+1%)
:warning: 203 (-29%)
:warning: 221 (-16%)
265 (-8%)

proof_construction_time_poseidon_hash_100_ms
5,787 (+1%)
1,576 (+1%)
:warning: 719 (-24%)
783 (-11%)
792 (-6%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,464
9,462
9,464

l1_rollup_execution_gas
611,203
611,352
611,505

l2_block_processing_time_in_ms
762
1,422 (+1%)
2,745 (+1%)

l2_block_building_time_in_ms
20,953 (-1%)
42,798 (+2%)
82,292

l2_block_rollup_simulation_time_in_ms
20,953 (-1%)
42,798 (+2%)
82,291

l2_block_public_tx_process_time_in_ms
17,942 (-1%)
39,502 (+2%)
79,097

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,117 (+2%)
10,000 (+2%)

node_database_size_in_bytes
12,259,408
16,142,416

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
68.4 (-3%)
13,790

GasToken:set_portal
17.7 (+18%)
3,339

Token:constructor
92.5
23,692

FPC:constructor
65.4 (+1%)
13,592

GasToken:mint_public
52.7 (-1%)
10,158

Token:mint_public
:warning: 68.8 (-87%)
19,034

Token:assert_minter_and_mint
:warning: 180 (+195%)
12,925

AuthRegistry:set_authorized
33.2 (+2%)
7,812

FPC:prepare_fee
118 (-9%)
15,062

Token:transfer_public
39.7 (+23%)
31,218

FPC:pay_refund
137 (-1%)
25,260

Benchmarking:increment_balance
2,243 (+2%)
15,267

Token:_increase_public_balance
56.1 (+3%)
15,006

FPC:pay_refund_with_shielded_rebate
118 (-1%)
26,347

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.159 (+5%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.5
16.8
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.605
0.517
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.7
76.3 (+1%)
131 (-1%)
246 (+1%)
477 (+1%)

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.497
0.468
0.451
0.445 (+1%)
0.444 (+1%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
60.4 (+1%)
113 (+2%)
183 (-1%)
355 (+1%)
704 (+1%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.509 (+1%)
0.504 (+1%)
0.484
0.479 (+1%)
0.482 (+1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
73.4 (+1%)
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.521 (+1%)
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
74,057
667,850

Transaction size based on fee payment method
| Metric |  |
| - |  |