refactor imports of logger dependencies

What does this PR do?
Refactoring imports around optional loggers, @awaelchli
resolves partially #2266
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?
[x] Did you verify new and existing tests pass locally with your changes?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified; Bugfixes should be including in bug-fix release milestones (m.f.X) and features should be included in (m.X.b) releases.

Did you have fun?
Make sure you had fun coding 🙃

@awaelchli do you have suggestions why the mocking tests for loggers are failing?

@awaelchli it seems to be still falling, so far I see it seems that with mocking we cannot use the actual _module_available so not sure what would be better if the func also accepts mock packages, but then it degrades the functions or redoes logger mocking...
Well, an alternative would be to add extra arg to the _module_available(..., strict: bool = True) and with not strict it would also, work with mocked packages... What do you think?

so far I see it seems that with mocking we cannot use the actual _module_available so not sure what would be better if the func also accepts mock packages, but then it degrades the functions or redoes logger mocking.

of course it can :)
you deleted some lines, that's why tests couldn't mock the experiment objects. See my commits, I reverted it and now it should be working.
You can test this also locally, all you have to do is uninstall the loggers (pip uninstall wandb, for example) and then run the tests and they should still pass.

@awaelchli mind check the final version? :]

@awaelchli this seems to be the last failing test test_mlflow_logger_dirs_creation mind have a look?

it seems something is wrong with the mlflow installation in the docker. locally everything is fine but when running inside docker importing mlflow fails even though it is installed. so the only idea i have is to either re-install mlflow or have it completely uninstalled (in this case the tests will run with the mocked version).

Let's see if an update helps :]

@awaelchli mind recheck? 🐰

@awaelchli mind recheck? 🐰

looks like some tests are failing on master.

yes, but the last commit here touching code b2cbf7c is fine...

looks like some tests are failing on master.

yes, but the last commit here touching code b2cbf7c is fine...


Refactor ACA original KG for containerization API (Community Edition)

We need to refactor the ACA original to add data for community edition. We need to generate the enterprise edition as well for GBS consumption.

DB has been updated
KG utils has been updated but need to be verified.


train_lmdb and val_lmdb  location

Hi
Where can I find train_lmdb and val_lmdb output from digits?
The one generated by new DataSet as classification problem
Thanks

You can find the job directory on the job page. That directory on your server will contain the LMDBs.

Thanks for your reply.
I actually searched this location and I can't find any files with LMDB extension
or do you mean data.mdb?
is data.mdb the LMDB file?

The directory containing data.mdb is the lmdb "file".

I can't find the documentation for this, but it's discussed here: https://github.com/BVLC/caffe/pull/2384.

Thank you so much for your help

@lukeyeager sorry for reviving this post, but you meant that the whole directory containg data.mdb is the lmdb file or just the data.mdb file? Because usually there is a lock.mdb there also.
Should I point data.mdb as my lmdb file or the whole directory as source for lmdb?
Thank you.


Update Students' details in batches

When a student is COVID-19 positive, there should be a straightforward process (method) to batch update students related to the COVID-19 positive student.
Group Conversation:

Change JSON format to categorize Students
Array of Class Codes which stores an Array of Persons (Another implementation of JSON Address Book)

To discuss: What if the student who was COVID-19 positive changed their status back to Negative, what should be the follow-up action?
During our team discussion, we could not derive a standard solution. However, some suggestions include:

Utilize the same method call to return close contact status to Negative. This approach seems viable but might have confused edge cases.
Let the change of status be independent. In addition, add a timer element to the status (epoch library?).

In conclusion, the team will further discuss future iterations to implement this feature.


SSE: Just use Stream instead of SseSender

Looking at the SSE examples, an odd part for me is to use an SSE specific channel call to get a SseSender and use that in a producer. This PR shows that it can be independent of the channel type by sticking to the concept that any Stream trait object can be a producer of server-sent events. So all I need in a producer is something which is Stream eventually.
If an unbounded channel or a different kind of channel is preferred by the user it should be obvious that SSE is not imposing any special requirement on the channel type. It just needs to be Stream eventually.
I view the SSE Responder in this case as a consumer of SSE events and the producer is the user defined thing producing eventually SSE events or things which can be turned into SSE events.
Funnily tokio mpsc bounded Receiver is not implementing Stream itself but tokio-stream provides a ReceiverStream wrapper type which takes the Receiver and implements Stream. Here in the PR I introduced Sse::from_receiver to be a bit more convenient in this regard. The user does not need to use the tokio-stream dependency by himself.
There are multiple Sse::from_* functions now and it is maybe not the best pattern here. I need feedback if maybe another trait and From implementations for this trait and Receiver (and all the other tokio receiver types) are better such that Sse::from can be used in all cases.
After wrapping a Receiver in  a ReceiverStream a user could now also call map on it. So it is more composable than receiver types by themself. For example this allows patterns where the data producer is independent of SSE, for example produces a user defined type or String and than the stream is mapped into Sse::Data.
Other feedback to this SSE interface
When the producer is not channels based but a stream by itself, for example a streaming answer from some database query, I can imagine that this is mapped quite well to SSE events. So this is good.
I got confused by the use of Infallible in the examples because I did not knew Infallible. It also feels a bit clunky to produce Results of Events in a stream. It is discouraging when the example needs to use turbofish to say that there are no errors.
Ok::<_, Infallible>(sse::Event::Data(data)) Can this be more convenient? Would have expected just Ok(data).
Sse::Event::Comment confused me and I needed to look SSE comments up in MDN to find out that they are actually not showing up on the client side javascript events. I think they don't need to be so prominent in the examples.
Thanks @cjrh for the great discussions on this during #EuroRust23 🦀

Thanks so much for both your work during the impl workshop!


cross is not building my project in containers.

Because it says "the x86_64-unknown-linux-gnu target may not be installed".
How to debug cross itself? I'm running cross 0.2.1 on the new MacBook Pro with M1 Max. There is no this kind of problem on my old iMac.

Can you post the entire invocation and command output?

$ cross build --target x86_64-unknown-linux-gnu
   Compiling cfg-if v1.0.0
   Compiling pin-project-lite v0.2.7
   Compiling bytes v1.1.0
   Compiling lazy_static v1.4.0
   Compiling futures-sink v0.3.17
   Compiling slab v0.4.4
   Compiling glob v0.3.0
   Compiling futures v0.1.31
   Compiling futures-io v0.3.17
   Compiling pin-utils v0.1.0
error[E0463]: can't find crate for `core`
  |
  = note: the `x86_64-unknown-linux-gnu` target may not be installed
  = help: consider downloading the target with `rustup target add x86_64-unknown-linux-gnu`

For more information about this error, try `rustc --explain E0463`.
error: could not compile `cfg-if` due to previous error
warning: build failed, waiting for other jobs to finish...
error[E0463]: can't find crate for `std`
  |
  = note: the `x86_64-unknown-linux-gnu` target may not be installed
  = help: consider downloading the target with `rustup target add x86_64-unknown-linux-gnu`

error: build failed

This output is after I manually uninstalled the target on the host machine, previously, it just fails at somewhere requires the gcc toolchain.

I've been using my own docker image for build. But even if I removed the Cross.toml, let it use its default configuration, it's not using the containers to build. I don't know if there is any other config files or environment variables that affects the behaviour of cross.

I'm running into this same issue too after building my own rustembedded/cross:aarch64-unknown-linux-musl-0.2.1 image for aarch64 instead of x86_64. Looks like cross is requiring the x86_64-unknown-linux-gnu toolchain even on aarch64 systems, see: https://github.com/rust-embedded/cross/blob/master/src/rustc.rs#L62
If you comment out those three lines and rebuild cross it'll use the native aarch64-unknown-linux-gnu toolchain instead.

Finally I come to write a lightweight wrapper by myself, since running cargo in docker is as simple as just typing docker run --rm -v `pwd`:/project -w /project my_docker_image cargo build --release --target x86_64-unknown-linux-gnu

I'm also running into this issue on an M1 Mac Pro. Works fine on x86 linux / mac but even with a custom Cross.toml it doesn't work.
To reproduce, clone https://github.com/wasmCloud/capability-providers, cd to httpserver-rs, run make par-full.

I'm also having issues with this as well, I had noticed that the version 0.2.1 was tagged on the 20th of June 2020.
So I thought I might have a dig through the code and see if I could find a solution, but it seems that if you clone the master branch and cargo build --release then it seems to be working.
So I had a look at https://github.com/rust-embedded/cross/blob/master/CHANGELOG.md and I think these issues have been resolved in https://github.com/rust-embedded/cross/pull/489
Just need to build cross manually or I guess hopefully a new version with these updates/fixes gets tagged?

Thanks @michaelr0 installing latest unreleased fixed the issue for me. You can do cargo install cross --git https://github.com/rust-embedded/cross --force also

Thanks @michaelr0 installing latest unreleased fixed the issue for me. You can do cargo install cross --git https://github.com/rust-embedded/cross --force also

Awesome, thanks for that, I didn't know you could do that. I just manually built it and moved it to a bin path haha.

Does this reproduce for anyone as of the latest main or as of v0.2.4? I don't have an aarch64 machine to test this.

this was an issue with toolchain lookup, should have been solved with https://github.com/cross-rs/cross/pull/489 I believe


plotmafsummary percentage order

Hello,
I just noticed that the percentages do not match the height of the barplot, I thought that the percentages would be ordered highest to lowest?

Thanks.

Hello,
Yes that is the case. Bars are sorted based on total mutations. However labels represent % of samples mutated. In this case, DNMT3A has more mutations in less samples (54 variants from 48 samples) i.e, some of the samples have more than one mutation in DNMT3A (multihit).
> getGeneSummary(x = laml)[,.(Hugo_Symbol, total, MutatedSamples, AlteredSamples)]
       Hugo_Symbol total MutatedSamples AlteredSamples
   1:        FLT3    52             52             52
   2:      DNMT3A    54             48             48
   3:        NPM1    34             33             33
   4:        IDH2    20             20             20
   5:        IDH1    18             18             18
  ---
1237:      ZNF689     1              1              1
1238:      ZNF75D     1              1              1
1239:      ZNF827     1              1              1
1240:       ZNF99     1              1              1
1241:        ZPBP     1              1              1

But I understand how it can be confusing. I will change fix the % to show MutatedSamples.
Thanks for noticing.

Okay it seems the current option is the best way. Other way would be to order based on the % of samples mutated which would shift the bars. So I will let it be the way it is.

Hi, Is there a way to only show plot of top mutated genes?

Hi,
Yes. Try maftools::mafbarplot().


[FEATURE] Directories

Hi, client-zip is a great lib, thank you very much for your work.
It would be awesome to be able to organize files in sub-directories into the output zip (maybe it is already and I didn't found how?).
Thanks

Hello. While there is no explicit suport for it, sub-directories are in fact possible because they're not an actual entity in ZIP files, instead they are deduced from pathnames: if you create a file whose name begins with "folder/" then it will, de facto, be in that folder.
That said, we could add a more obvious sub-directories feature. But how would it work ? I've seen other libraries let you create a directory object with methods to add files to it. I don't want to do that because it would completely change the current design with a single streaming function. I am open to suggestions.

For example, we could have elements in the input iterable that have this form: {name: "dir", items: [await fetch(…), {name: "subdir", …}]}. It would make typing much harder because TypeScript doesn't like recursive types, though. And in the end, all it would do is prefix file names with the directory name and a slash.

That's awesome, exactly what I was looking for.
Thanks a lot :)


[DOC] Add documentation related to allocator time out

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request.
Relevant PR - https://github.com/opensearch-project/OpenSearch/pull/15239
*Version: List the OpenSearch version to which this issue applies, e.g. 2.14, 2.12--2.14, or all.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

@imRishN: Is this related to 2.17?

@Naarcha-AWS yes

[Catch All Triage - 1, 2, 3, 4, 5]


Add .proxify(), .path() and .type

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

Comparison is base (af1c12d) 93.49% compared to head (07ec67c) 92.15%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   93.49%   92.15%   -1.35%
==========================================
  Files           6        6
  Lines         323      395      +72
  Branches        2       11       +9
==========================================
+ Hits          302      364      +62
- Misses         21       31      +10

Files
Coverage Δ

src/main.cc
97.05% <100.00%> (+0.18%)
:arrow_up:

lib/index.cjs
88.88% <86.84%> (-11.12%)
:arrow_down:

src/JSON.cc
90.58% <81.81%> (-1.45%)
:arrow_down:

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


rez-build with platform variants

Hey there.
I'm trying to do a rez-build --install with platform-windows and platform-linux as variants.
On a windows machine. The 2 first variant are platform-windows variant. The 3rd and 4th are Linux one.
So rez-build --install is doing the 2 first variant and error (and stop) on the 3rd.
Error: "The following package conflicts occurred: (platform-linux <--!--> ~platform==windows)"
The issue if I'm going to a Linux machine to build the Linux platform variant.
rez-build --install will error on the 1st variant.
I did change to the code to fix that:
rez-build is now continuing  on error and printing how many variants were build successfully.

I would like to do a pull request of it. Can you grant me the access?
Regards,
Clement

You don't need additional access to do a Pull Request.
You just need to fork the repo and do your pull request from there.

Yes, feel free to PR from a fork.
In terms of the problem, yes this needs to be fixed. I think the best way
to do this is to add a --skip-conflicting flag to rez-build. Or, more
specifically, add this option to the 'local' build process type, and
expose as a CLI arg. I think in this case it would also make sense to add a
rezconfig entry to turn this behaviour on by default.
Thx,
A
On Wed, Feb 21, 2018 at 1:45 PM, Brendan Abel<EMAIL_ADDRESS>wrote:

You don't need additional access to do a Pull Request.
You just need to fork the repo
https://guides.github.com/activities/forking/ and do your pull request
from there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/484#issuecomment-367195931, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABjqSrsficDD2BT78HJ1qKfO8994A0NQks5tW4NIgaJpZM4SL19u
.


Fixes waveform thumbnail issues + textBox focus improvements for windows

Tested on Windows.
To test it in Mac, make sure the issues in #93 , as well as in #91 are fixed

fixes #93


Qtmain

Checklist

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

I think this might be on the right track, but not working yet.

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

@conda-forge-admin please rerender

@conda-forge/qscintilla2 any chance someone can give me a hand with the windows build? I think it should be modified to use sip-build as in recipe/build.sh but I don't have a local windows machine to test it out on.

I usually just use the Windows CI builds to check. Or are the iteration times too long for you to wait on those?

Basically I suck at windows, I would be happy if someone else can take a shot at it.

@conda-forge-admin please rerender

It would be good to merge this, several packages need it for the qt515 migration.


Intersil's DF Driver Wrapper

TODO: add commit to bump to DF master with https://github.com/PX4/DriverFramework/pull/61 pulled in.

Please run:
./Tools/fix_code_style.sh src/platforms/posix/drivers/df_isl_wrapper/df_isl_wrapper.cpp

2 comments, other than that it looks fine.

@julianoes I ran fix condestyle on df_isl_wrapper, not much changes there, also fixed the unused option issue.

Thanks @bugobliterator for the fixes, unfortunately after the renaming, the build was not completely happy:
CMake Warning (dev) at cmake/common/px4_base.cmake:313 (add_dependencies):
  Policy CMP0046 is not set: Error on non-existent dependency in
  add_dependencies.  Run "cmake --help-policy CMP0046" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  The dependency target "df_isl" of target
  "platforms__posix__drivers__df_isl_wrapper" does not exist.
Call Stack (most recent call first):
  src/platforms/posix/drivers/df_isl_wrapper/CMakeLists.txt:36 (px4_add_module)
This warning is for project developers.  Use -Wno-dev to suppress it.

I renamed everything all the way, and pushed it to: #4392.


Generating large areas causes player timeout

When generating large areas for new locations the player experiences timeout before the generation is completed.

See the network plugin code for how a wait system can be achieved.
Wait system = make the next stages of location creation wait on the terrain generation to finish.
Network does this on tpll.


add validator webhook for harvester addon

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

With the wide usage of harvester addon, webook is required for following scenarios.
(1) cannot create more than one instance for one addon
(2) cannot change addon field chart
We did not find feasible solution for below 2 checks, leave them blank now.

(3) check the existing of given chart (chart + repo + version)  (TBD)
(4) check the valuesContent is an valid yaml/json format (TBD)

Solution:

Add webhook to validate addon.
Related Issue:
https://github.com/harvester/harvester/issues/3660
Test plan:

(1) cannot create more than one instance for one addon
harv31:/home/rancher # kk create -f new_addon.yam

Error from server (Conflict): error when creating "new_addon.yam": admission webhook "validator.harvesterhci.io" denied the request: addon with Chart "harvester-vm-import-controller" has been created, cannot create a new one

(2) cannot change addon field chart
kk edit addons.harvesterhci.io -n harvester-system pcidevices-controller

error: addons.harvesterhci.io "pcidevices-controller" could not be patched: admission webhook "validator.harvesterhci.io" denied the request: chart field cannot be changed.
You can run `kubectl replace -f /tmp/kubectl-edit-2576031672.yaml` to try this update again.

cc @ibrokethecloud @WuJun2016
when more checks are needed, please add here, thanks.

@w13915984028 LGTM. can we please add some basic tests for this and we can keep building on top of these as we add more validations.

@w13915984028 LGTM. can we please add some basic tests for this and we can keep building on top of these as we add more validations.

@ibrokethecloud please take a look of the tests, thanks.
BTW: for deleting an addon, should user disable them at first ?

deleting the addon, should trigger gc for the helm chart, which should be same as disabling the addon.

@bk201 This commit is more an enhancement, it would be optional for v1.1.3, depends on the planned effort to this release. thanks.


Bug in AMA

https://brianlovin.com/ama
Anyone could just spam likes using an auto clicker on any question and it doesn't stop that. Maybe it should have an code to limit one Ip address or 1 visitor can like only once.

It's mostly intentional that it can be spammed – I do have some rate limiting on the API side. I'm planning some future changes to the site that will gate likes behind a sign in 😊
Thanks for the heads up and taking the time to open this!


Multi-origin support for CDN

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

I had the same change reviewed and signed off in the private repo of azure-rest-api-specs- here is the link to that PR- https://github.com/Azure/azure-rest-api-specs-pr/pull/871

Can one of the admins verify this patch?

WaitForARMFeedback

Was reviewed in the private repo here:  https://github.com/Azure/azure-rest-api-specs-pr/pull/871

@njuCZ  - can you take a look at this PR?Or assign it to someone who can?


add mips64 reg

Checklist

[ ] Language changes are updated in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Hi, Can you fix the clang format warnings and make this mergable? Thanks!

I can't  resolve CI check failed! Close  this commit.


From private repo

@v-alje : Thanks for your contribution! The author, @MarkMcGeeAtAquent, has been notified to review your proposed change.


using movie or file instead of camera

Can you please advice on how to use Lucas optical flow to detect motion for movie file instead of the webcam? so I don't want to click (or use the mouse_down function)
thanks

Have a look into this: http://docs.opencv.org/master/d7/d8b/tutorial_py_lucas_kanade.html#gsc.tab=0

thanks


7.9.2 release

This PR prepare 7.9 branch for 7.9.2 release:

bump chart and Docker images version from 7.9.2-SNAPSHOT to 7.9.2
remove notices on README that this is a development branch and development branch specific install instructions

Note

remove notices on README that this is a development branch and development branch specific install instructions

This is required because released Helm charts will be packaged from this PR commit including the README files that will be available inside the chart packages and in Helm Hub pages (example: Elasticsearch chart page).
This change will be reverted in the next PR after release when we will bump 7.9 branch to 7.9.3-SNAPSHOT version.

7.9.2 - 2020/09/24

7.9.2 as the default stack version
Bump Helm version to 2.16.12 (@jmlrt)

Elasticsearch

PR
Author
Title

#729
@floretan
Include pre-releases in the semver range.

#810
@luanguimaraesla
Add loadBalancer externalTrafficPolicy option

#778
@jmlrt
Fix secrets names in examples

Metricbeat

PR
Author
Title

#778
erihanse
Support secrets


Simplify Materialization Scheduling

The current configure-a-materialization API is quite complex and can be simplified, since most of the metadata provided by users can be inferred from existing node metadata.
Pre-Materialization Step
For nodes where we expect to materialize partition by partition, the user can set columns on the node to be temporal partition columns via their column attributes:
curl -X POST 'http://localhost:8000/nodes/default.repairs_cube/columns/default_DOT_total_repair_cost/attributes'
-d '{
    "namespace": "system",
    "name": "temporal_partition_day"
}'

We'll provide two new attribute tags for this purpose: temporal partition day and temporal partition hour. Setting these column attributes implies that any materialized version of the node should be partitioned by that column.
Configure Materialization
This means that the user wants DJ to schedule the appropriate materialization workflows in their scheduler of choice, based on the provided config. This will reuse the existing POST /nodes/{node_name}/materializations endpoint, but with some changes to the input schema.
User input that we'll keep at this stage:

node to materialize
schedule (cron)
engine
optional engine-specific config

We'll remove partitions from the expected input, since this can now be derived from column attributes. Additionally, if it's a Druid materialization, we'll remove the requirement for any Druid-specific config, as all of it can be derived at runtime based on column attributes.
DJ will store this materialization config and create a scheduled workflow and an adhoc workflow (which is used for kicking off backfills and adhoc runs). Note: If there are no temporal partitions, it will assume the node is meant to be materialized fully at the specified schedule.
Example updated API call:
curl -X POST '/nodes/default.repairs_cube/materialization/'
-d '{
    "engine": {"name": "druid", "version": ""},
    "config": {
        "spark": {}
    },
    "schedule": "@daily"
}'

Set Backfills
For a given node with a materialization configured, users can ask for the node to be backfilled (across some time range). This will trigger the adhoc workflow that was scheduled previously. This will be a new API endpoint:
curl -X POST 'http://localhost:8000/nodes/{node_name}/materializations/{materialization_name}/backfill'
-d '{
    "range": [
        "20230901",
        "20230930"
    ]
}'

If it's a Druid materialization, based on the configured partitions, it will generate the Druid config at runtime:

we can assign granularity based on column attributes
we can assign the timestamp_column based on column attributes
intervals are generated based on the backfill partition (i.e., we should only load one interval at a time, whereas previously we were loading all intervals for each backfill job)

@agorajek what do you think of the above? I know we don't have to change materialization atm, since it works, but I think it would be nice to have a UI for users to configure materialization in (especially if they want to build cubes), and it's better to create the UI after we clean up the API layer.

@shangyian this sounds great! One question I have is how do we translate the $dj_logical_timestamp into the values of temporal partition day and temporal partition hour for the scheduled job, if the user did not make that translation in their SQL code?

@agorajek good question... I think right now we do that in the query service (as in, we add an extra wrapper that adds in this where clause for the partition), but we should change that so that the core DJ service generates that query with the parameterized $dj_logical_timestamp and passes it to the query service for scheduling during the "configure materialization" phase.

Right... so maybe instead of asking for  temporal partition day|hour lets ask the user to specify temporal partition and then format using some standard code (say Python), https://docs.python.org/3/library/datetime.html#format-codes ?
Remember that they already tell us the schedule, so we don't need to know the ... day|hour part for anything other than backfills. But for backfills we can actually ask for that specifically as in:

Pick one partition column to run the backfill for.
a. You picked a categorical partition, ok so please tell us the list of values to run the backfill for.
b. You picked a temporal partition, ok so please tell us the time interval and the min, max values to run it with.

That makes sense, except that then it's harder to use the existing column attributes for. I guess we could add some column-attribute-level metadata that stores the timestamp format.

+1 on providing input for all partitions. And for the categorical it should be ok to skip, as-in no values set.

That makes sense, except that then it's harder to use the existing column attributes for. I guess we could add some column-attribute-level metadata that stores the timestamp format.

Sorry for jumping around, but I just recalled this https://github.com/DataJunction/dj/discussions/421 and realized we could just ask for an expression when they define the temp partition, say: CAST(FORMAT(${dj_logical_timestamp}, "YYYMMDD") AS INT) and we can evaluate this expression to make sure it matches the type of the column. With this in hand asking for backfill range could be based on pure timestamp, cause we would know how to translate it into the partition values.


add computed decorator

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         4    +1
  Lines          120       127    +7
  Branches        18        18
=========================================
+ Hits           120       127    +7

Impacted Files
Coverage Δ

cabina/_computed.py
100.00% <100.00%> (ø)

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 a8b8b6b...c2d052b. Read the comment docs.


Add notebooks to fill the basic CLV grid

The idea is to write a notebook with pure PyMC model(s) for each of these CLV scenarios. We can start with the Lifetime part (not-value yet), but ideally we will include value as well by the end. This might be a constant with time-decay penalty in the simplest cases.
Hopefully this will give us a good picture of the building blocks that are necessary for a minimum viable package, and can also serve as the base documentation. Overtime we would replace the custom PyMC code with imports from the CLV sub-package.

[x] Continuous Non-contractual #16
[x] Continuous Contractual #36
[ ] Discrete Non-contractual (up for grabs)
[x] Discrete Contractual #32

Is the goal to add more than one model to each quadrant? The CNC quadrant now has @larryshamalama 's BG/NBD distribution (#16), but there are several other models available in that quadrant. I wasn't sure if the idea was to get one per quadrant max or if the goal was broader than that. Also, the distribution added in #16 is named ContNonContract, which seems to imply that there aren't going to be other continuous non-contractual distributions.

Sorry for not replying to your question initially. Effectively, there are other continuous non-contractual distributions, as I realized later on (e.g. ParetoNBD). We briefly touched upon this point in #128

Discrete contractual is covered by https://github.com/pymc-labs/pymc-marketing/pull/133.  Looks like a PR to address https://github.com/pymc-labs/pymc-marketing/issues/176 would cover the final quadrant.

We have a distribution block for Continuous Contractual, but no modeling options per https://github.com/pymc-labs/pymc-marketing/issues/279. I've gotten questions on LinkedIn about modeling purchases made through active memberships and/or phone app sessions, so there's demand for it.

We have a distribution block for Continuous Contractual, but no modeling options per https://github.com/pymc-labs/pymc-marketing/issues/279. I've gotten questions on LinkedIn about modeling purchases made through active memberships and/or phone app sessions, so there's demand for it.

Related: https://github.com/pymc-labs/pymc-marketing/issues/64


About testing data

Hi, thanks for sharing the code and the impressive paper.
I am running experiments on the fastMRI singlecoil knee data with 4x acceleration using you pre-trained model.
Could you provide which slices of each scan (referring to the num_slices & num_start in the mri_data.py) that you use in the test experiments?
Since I have run the experiments using all the slices and the PSNR result is very low, only 28dB (3dB higher than zero-filled) instead of 32.73dB in your paper. Please see the details below.

Many thanks and I look forward to your reply!

Hi,
We don't exclude slices during inference. It's weird to see such low PSNR. Are you using the provided DataLoader (https://github.com/guopengf/ReconFormer/blob/451761991aca18fb7adf3fd9d5fb95314ccef109/data/mri_data.py#L16) and processed dataset (https://livejohnshopkins-my.sharepoint.com/:f:/g/personal/pguo4_jh_edu/EtXsMeyrJB1Pn-JOjM_UqhUBdY1KPrvs-PwF2fW7gERKIA?e=uuBINy). If not, please make sure that you normalize data in a similar way shown in those lines (https://github.com/guopengf/ReconFormer/blob/451761991aca18fb7adf3fd9d5fb95314ccef109/data/mri_data.py#L136-L138)

Hi @guopengf ,
Thanks for your reply. Yes, I used exactly the same data and code that you provided.
I have figured out the problem.
The reason is that when I calculate the PSNR, I calculate it on every single slice. While in your original code, you calculate the PSNR of the whole set of slices at the same time.
That leads to the difference in data_range in the PSNR function, thus causing the different PSNR results that I have.
So I just want to make sure is that the common way in the MRI community to calculate the whole set of slices at once instead of calculating on each slice?
Many thanks!

I'm glad that you figured it out. Yeah, it's common practice that we calculate metrics based on the 3D volume instead of individual slices. The evaluation code is adopted from fastMRI repo (https://github.com/facebookresearch/fastMRI).

@guopengf Got it, thanks so much!


Integrate upstream changes

[ ] https://github.com/succinctlabs/sp1/pull/912
[x] https://github.com/succinctlabs/sp1/pull/922
[ ] https://github.com/succinctlabs/sp1/pull/940
[x] https://github.com/succinctlabs/sp1/pull/995
[x] https://github.com/succinctlabs/sp1/pull/1013
[ ] https://github.com/succinctlabs/sp1/pull/1046
[x] https://github.com/succinctlabs/sp1/pull/1119
[x] https://github.com/succinctlabs/sp1/pull/1123

I added most in #59. We probably don't want https://github.com/succinctlabs/sp1/pull/1046. Not sure about the others. They're related and we can revisit this again later.


Update NAG interface for cvxpy 1.1

The NAG interface (created through PR #776) relies on one or more functions from the ConicSolver class which were removed from cvxpy, in the switch from version 1.0 to 1.1. In particular, NAG.block_format is broken, because ConicSolver.format_constr does not exist anymore.
From what I can tell, the NAG apply() function was modeled off the MOSEK apply() function. With that in mind, many of the necessary changes to the current NAG interface can be identified by following https://github.com/cvxgrp/cvxpy/commit/badc0a79701da0785acaaf994b9d072bafb64efb#diff-209669af1b5a91178295a651bbea34f3. (That commit wasn't perfect though; exponential cone changes were needed in https://github.com/cvxgrp/cvxpy/pull/862.)

Thanks for opening this, Riley!
The NAG developers are welcome to submit a PR to update the interface. NAG is a commercial solver; we don't test it in CI.

Making a note before I close this issue.
Steven attempted to get NAG working on 1.1, but wasn't successful (https://github.com/cvxgrp/cvxpy/issues/913#issuecomment-582197969). The lack of readily available academic or trial licenses makes it unrealistic for us to support NAG on our own. Since no NAG users or developers stepped up to tackle this issue, we've opted to remove NAG support in the CVXPY 1.1 release (https://github.com/cvxgrp/cvxpy/issues/913#issuecomment-582222934).
We'll update https://github.com/cvxgrp/cvxpy/blob/master/CONTRIBUTING.md to provide guidance on contributing solver interfaces (commercial or otherwise) in the future.

Hi, now the NAG interface has been updated #967. Thanks a lot!


Initial mqtt calls for endpoints

Summary
Initial implementation of mqtt requests for engines and overview of engines for admins.

When i go to /admin/engines i get an error.

@jjoderis did you have a mqtt broker running and the env vars set?

Ah i see. No i didn't. Would it be a valid option to hide the entry from the Dashboard if the mqtt environment values are not set? From what i understand the route cannot work without them.
And are there upsides and downsides to using environment variables instead of configuration entries for this? I guess it would not really be possible to change the mqtt server address or password while the MS is running using environment variables

I think it would be better to show a message that mqtt isn't set up, what do you think?
I remember Kai saying that he wanted the mqtt credentials in the environment variables, I think for now it is ok.


Neo.ClientError.Database.DatabaseNotFound

Hello! I use Neo4j commynity edition 4.0.4.
Is there any option to set default database name in your package? Because I change default neo4j db to my own, and I have an error:
Neo4j errors detected. First one with code "Neo.ClientError.Database.DatabaseNotFound"
and message "The database requested does not exists. Requested database name: 'neo4j'."

Thanks!

Hello,
Yes, there is an option! You can use the HttpInjections or BoltInjections respectively while using the client builder. Here is an example:
$client = Laudis\Neo4j\ClientBuilder::create()
    ->addHttpConnection('http', 'http://neo4j:password@localhost', HttpInjections::create()->withDatabase('myDatabase'))
    ->addBoltConnection('bolt', 'neo4j:password@localhost', BoltInjections::create()->withDatabase('tags'))
    ->build();

Works greate, thanks!
But how annoying to switch between db's in community edition of Neo :)

Yeah, it's not so simple but I am glad I could be of help!
Best of luck


with different value but same timestamp" num_dropped=1

exporter data

  # HELP cpu_load_1m cpu load 1m
  # TYPE cpu_load_1m gauge
  cpu_load_1m{agentId="xxxx"} 11.0<PHONE_NUMBER>.101
  # HELP cpu_load_1m cpu load 1m
  # TYPE cpu_load_1m gauge
  cpu_load_1m{agentId="xxxx"} 12.0<PHONE_NUMBER>.101

What did you expect to see?
expect wirte all data！
log
ts=2024-07-23T11:47:34.423Z caller=scrape.go:1783 level=debug component="scrape manager" scrape_pool=my_machine target=http://<IP_ADDRESS>:9400/metrics msg="Duplicate sample for timestamp" series="cpu_load_1m{agentId="xxxx"}"
ts=2024-07-23T11:47:34.423Z caller=scrape.go:1744 level=warn component="scrape manager" scrape_pool=my_machine target=http://<IP_ADDRESS>:9400/metrics msg="Error on ingesting samples with different value but same timestamp" num_dropped=1
System information
linux x86
Prometheus version
2.53.1
Prometheus configuration file

Alertmanager version
No response
Alertmanager configuration file
No response
Logs
ts=2024-07-23T11:47:34.423Z caller=scrape.go:1783 level=debug component="scrape manager" scrape_pool=my_machine target=http://<IP_ADDRESS>:9400/metrics msg="Duplicate sample for timestamp" series="cpu_load_1m{agentId="xxxx"}"
ts=2024-07-23T11:47:34.423Z caller=scrape.go:1744 level=warn component="scrape manager" scrape_pool=my_machine target=http://<IP_ADDRESS>:9400/metrics msg="Error on ingesting samples with different value but same timestamp" num_dropped=1

This issue seems to be a duplicated of this one https://github.com/prometheus/prometheus/issues/14391 and somewhat similar to this one https://github.com/prometheus/prometheus/issues/14503.
Besides the current implementation, do we have a source of truth regarding what should be the behaviour of prometheus when scraping data and what the contract that application that publish metrics through a metrics endpoint should follow?

This issue seems to be a duplicated of this one #14391 and somewhat similar to this one #14503.
Besides the current implementation, do we have a source of truth regarding what should be the behaviour of prometheus when scraping data and what the contract that application that publish metrics through a metrics endpoint should follow?

"The actual scenario I'm dealing with has a high requirement for the timeliness of data reporting. The agent sends an index data to my backend every second, and my backend uses the Java version of prometheus-exporter. Therefore, Prometheus will have multiple timestamped data points during a single scrape."

FYI, this PR seems to fix this issue: https://github.com/prometheus/prometheus/pull/14685

@zhangfanv can you clarify why you don't think this is a duplicate, please?


Add note on the relationship between OUTPUT_PATH and terraform_root in the README

docs/tfengine/README.md:250:1 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 4]


parachain ismp relayer messaging integration tests

tesseract messaging relayer integration tests between 2 parachains, Testing ismp get and post requests.

After merging this,
I need to improve ismp Demo get request to emit event so that the relayer can fetch the request

So merge this first, the get request I need to make a seperate PR on tesseract messaging as it filters out GETREQUESTs @Wizdave97

Context:
Once u submit any request. the dest chain will check in the proof.state_id ( originaty request proof chain id)  to fetch the height of the statemachine in dest chain
But then in the RequestResponse::Request Get ( fn handle). it checks if the request detination chain is not equal to proof.state_id.  but this will always fail
for example chain A (2000) chain B (2001)
Get request dispatched from chain A ( so it will contain proof object with state id 2000), dest chain: 2001.
in chain B ( 2001)
Verifications: it will check the proof state id ( 2000) and fetch the height of that state machine. ✅
But now in the handle function of the Get request it checks if dest chain(2001) != proof.state_id(2000) and it returns error
So these 2 verifications contradict

Overall looks good, please update your PR description, and add these new tests to the integration tests step otherwise, CI isn't going to work

Let's fix the tests and wrap this off ASAP, it's been open for about two weeks.

yes

yes, so Ive tested alot on this, running them altogether they are flaking but its not because of the tests but the chain handling the transactions and all,
But if i just run them seperately like cargo test 1 && cargo test 2, They do not fail at all
I don't think that's the issue, the tests are independent of each other, you should be able to run them successfully with the previous command.

@MrishoLukamba what's the latest with this PR?

Am updating it. Pushing commit today. I landed yesterday night.

Hyperclient should pass if you rebase on main


Rancher-monitoring: query issue on daemonset

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Enable rancher monitoring, deploy a daemonset, view daemonset
Result:
cpu, memory, memory usage, cpu usage - all appear to be inaccurate, some are double the actual amount
Other details that may be helpful:
Looks like it should be resolved by including container_name!="POD", container_name!="",  in the prometheus query - this happens on other dashboards but not here
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.2.2
Installation option (single install/HA): HA

CPU:
sum(rate(container_cpu_usage_seconds_total{container_name!="POD", container_name!="", namespace="$daemonset_namespace",pod_name=~"$daemonset_name.*"}[5m]))
Memory:
sum(container_memory_working_set_bytes{namespace="$daemonset_namespace",pod_name=~"$daemonset_name.*", container_name!="POD", container_name!=""})
CPU Usage:
sum by (pod_name) (rate(container_cpu_usage_seconds_total{namespace="$daemonset_namespace",pod_name=~"$daemonset_name.*", container_name!="POD", container_name!=""}[5m]))
Memory Usage
sum by (pod_name) (container_memory_working_set_bytes{namespace="$daemonset_namespace",pod_name=~"$daemonset_name.*", container_name!="POD", container_name!=""})
This is the updated dashboard json from grafana after I made these updates - happy to PR the system-charts repo if this looks correct
updated-json.txt

Seeing something similar on statefulset dashboard with cpu and memory

I too am observing this. It's actually fairly noticeable if you compare rancher charts to the grafana ones grafana is showing 2x in most cases.

As I think about it, I wonder if the graphs embedded in the rancher ui are also affected - I'd try to verify but I'm not sure where that code is.

@spencergilbert , for rancher ui, the expressions are in https://github.com/rancher/rancher/blob/master/pkg/controllers/user/monitoring/metricExpression.go. This is indeed a bug for ignoring conatiner_name!="" in some expressions, but I prefer to keep the counting for POD container, as the pause container is also a part of one workload.

Sounds good to me, I'm still wrapping my head around the kubernetes monitoring here so I'll keep that in mind with the POD container.
Thanks!

The fix is in system-charts dev branch, and can be tested with master build

Fixed in v2.2.4-rc1 in backport issue #20242
When I have time I will check in latest master (v2.3) in this issue as well.

Thanks everyone!

Latest master has been tested; I have the same findings as my tests done recently for the backport issue #20242 (v2.2.4-rc1); this is working now.
Version: Master (v2.3) (5/29/19)
The PR https://github.com/rancher/system-charts/pull/43 contains a fix and here is the overview:

Problem: There are two kubelet scraping targets on Prometheus, one is scraping /metrics, another one is scraping /metrics/cadvisor. The metrics from /metrics endpoint are not including container_name. So the container_* expression will double the actual mount without
conatiner_name!=""

Fix: Add container_name!="" into the expression

This should fix the inaccurate Grafana dashboards that were in some cases (when container_name!="" was not there) showing 2x the values.
I tested again and now I observe data that matches what we see in Rancher charts. Grafana is no longer showing 2x the values.
I tested by creating a CPU load on an ubuntu workload. I utilized the stress package to put 50% of cpu cores under load then 100% cpu cores under load. This way we have consistent cpu utilization more or less. I ensured that for each test the Rancher wheels and charts matched what we see in Grafana -- down to the Deployment level this appears accurate now (check Cluster, Node, Pod, Deployment [memory], etc and compare each to Rancher UI). I did spot checks as well for memory usage, etc which also appears accurate now.


Metadata and screenshots for multiple "Branded" Targets of the same underlying App

I am using the setup described in the  FAQs using dotenv for multiple targets (see also issue #88 ).
However, I am still having trouble to figure out how to handle screenshots and metadata. I need to store them in target-specific folders like TARGET_NAME/screenshots or TARGET_NAME/metadata
I am setting the custom paths in the Fastfile like this:
    deliver(
      screenshots_path: ENV['SCREENSHOTS_PATH'],
      metadata_path: ENV['METADATA_PATH']
    )

Now that works. I'd prefer to declare them in the Deliverfile itself, but it does't seem to work with dotenv (or am I missing something?).
But the real issue is:
How can I download the initial metadata and screenshots from ITC to my custom folders?
For screenshots, there is an API deliver download_screenshots but I don't know how to make it use my custom paths.
For metadata, the only way to get them seems to be a deliver setup but then again how do I get it to use my custom paths!

deliver init will always just init the files in the current directory. There is no way to change this. You can easily write a script to clone everything somewhere into /tmp and then move the resulting files.

I was trying just that, but deliver setup requires user interaction. I didn't manage to set bundle id etc. via script.
I also tried pre generating the Deliverfiles but then deliver setup wouldn't run.
deliver download_screenshots should work however.
So from an API perspective I think it would make sense to expose an Interface download_metadata.
I think that should also resolve my issue.
What do you think?

Ok, I have a setup script for my app targets like this:
# iterate apps
apps.each do |app|
  app_name = app[:ProjectName]
  bundle_id = app[:id]

  # create Deliverfile
  out_file = File.new("#{app_name}/Deliverfile", "w")
  out_file.puts("app_identifier \"#{bundle_id}\"\nusername \"#{username}\"")
  out_file.close

  # load screenshots
  %x( cd "#{app_name}" && deliver download_screenshots && cd - )

  # TODO: load metadata
  # %x( cd "#{app_name}" && deliver download_metadata && cd - )
end

Works fine except of deliver download_metadata of course.
Any chance you would add that API?

@baarde sorry, no download_metadata planned

That's OK. I don't need it. 😉

Ok, too bad. Thanks anyway for your incredible work!
Why am I the only one with the "multiple branded targets" setup who is running into this issue...?
Help from someone who's been through this would be very much appreciated! What am I missing?
I'm looking at setting up Deliver 50+ times....

Don't laugh as I'm sure there is a better approach, but I used something like this to run through all my branded targets which get pushed to the App Store.
`echo "Setting up Deliver for all targets"
mkdir "x1";
cd x1;
deliver init -u<EMAIL_ADDRESS>-a "com.xxx.apps.x1";
cd ..
mkdir "x2";
cd x2;
deliver init -u<EMAIL_ADDRESS>-a "com.xxx.apps.x2.";
`
Hope that helps

Oh, I'm not laughing at all! That was exactly what I was looking for. I was messing around with deliver init but the options -u, -a were not obvious to me.
So I would still consider a download_metadata API to be the better option, but as for me this issue can be closed.

@deberle Glad it helped!

Hey everyone! I am going to close this issue for now as there has not been any activity for a while! Please feel free to reopen if necessary! :+1:


Login

Epic to track login for the deposit app.
We've decided to use web3auth as the primary login method.

[ ] #5

This task is essentially the task #5 , no?


Converter by Power2Apps Connector

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Please merge my renamed connector. Thank you

Hello @stsiek,
Thank you for an update!
Your pull request is approved and merged. Please update your submission in ISV Studio.
Let me know if you have any questions or need help!
Thank you very much for working with us.


[BUG] Swagger documentation is set to null for objects that are @ManyToOne for TypeORM

Information

Version: 6.14.3
Packages: @tsed/swagger, @tsed/typeorm

Swagger documentation disappears of objects that are set to @OneToMany relationships.
A Simple Example
import { Entity, ManyToOne, OneToMany } from 'typeorm'
import { CollectionOf } from '@tsed/schema'

@Entity()
class User {
  @CollectionOf(Photo)
  @OneToMany(() => Photo, photo => photo.user)
  photos: Photo[];
}

@Entity()
class Photo {
  @ManyToOne(()=> User, user => user.photos)
  user: User
}

This results in the following swagger being generated:
User:
      ...
      photos:
        type: ''
      ...

Acceptance criteria
the swagger definition should be
User:
      ...
      photos:
        type: array
        $ref: `#/definitions/Photo
      ...

@gkazarian add @Property on user prop.


Add runtime_assets property to filters resource

Pull Request Checklist
closes #64 @webframp I took a quick look and I did not see any tests for filters, am I missing something or did they not get done. I can take a closer look over the weekend possibly. I also need to actually look more closely at the docs since it was not clear to me what
General

[ ] Update Changelog following the conventions laid out here

[ ] Update README with any necessary configuration snippets

[ ] Cookstyle (rubocop) passes

[ ] Foodcritic passes

[ ] Rspec (unit tests) passes

[ ] Inspec (integration tests) passes

New Features

[ ] Added a Testing Artifact as either an automated test or a manual artifact on the PR.

[ ] Adedd documentation for it to the README.md

Purpose
Add support for runttime_assets on filters.
API Docs: https://docs.sensu.io/sensu-go/5.13/api/filters/
Known Compatibility Issues
none

@majormoses Looks good, seems like they were just missed. They only existed on the check resource so far from what I can tell.


Issue with juicer output hic file

Hi everyone,
Thank you for developing this tool. I run juicer and it completed by reporting the following messages:
(-: Postprocessing successfully completed, maps too sparse to annotate or GPUs unavailable (-: (-: Pipeline successfully completed (-: Run cleanup.sh to remove the splits directory Check ./opt/juicer/HiC-HUMAN-CD14-1-1/aligned for results
So, I went to have a look at the hic output. At first, I noticed that the splits folders were still there. Further by looking at the hic files, they looked like that:

To be honest, it seems there is not any intrachromosomal interaction reported and I am not sure why. Any idea what could have gone wrong?
Thanks,
IG

Hello - can you please post the command you ran, what system you're running
under, and the JAR version?
On Wed, Mar 24, 2021 at 11:22 AM iacopogentile @.***>
wrote:

Hi everyone,
Thank you for developing this tool. I run juicer and it completed by
reporting the following messages:
(-: Postprocessing successfully completed, maps too sparse to annotate or
GPUs unavailable (-: (-: Pipeline successfully completed (-: Run cleanup.sh
to remove the splits directory Check
./opt/juicer/HiC-HUMAN-CD14-1-1/aligned for results
So, I went to have a look at the hic output. At first, I noticed that the
splits folders were still there. Further by looking at the hic files, they
looked like that:
[image: Screen Shot 2021-03-24 at 11 11 14 AM]
https://user-images.githubusercontent.com/79707258/112334266-ab7a5600-8c91-11eb-8c1a-3a3a862aba91.png
To be honest, it seems there is not any intrachromosomal interaction
reported and I am not sure why. Any idea what could have gone wrong?
Thanks,
IG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/215, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW7H2R5U7SWPHQTVLY3TFH7SVANCNFSM4ZXPNJ7A
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Hi. Thanks for the reply! I used juicer_tools.1.9.9_jcuda.0.8.jar. and I run the following bash script on the cluster:
HiC_seq () {

	local run=$1

./opt/juicer/scripts/juicer.sh -g GRCh38 -d ./opt/juicer/$run -s MboI -p ./opt/juicer/mygenome.chrom.sizes -y ./opt/juicer/restriction_sites -z ./opt/juicer/references/GRCh38_latest_genomic.fna -D ./opt/juicer/ -t 1
}

for folder in $folderlist; do HiC_seq '$folder' & done

thx,
IG

There is something wrong with your restriction site file most likely, since
it seems to be thinking all intra chromosomal reads are intra fragment.
Run at the “final” stage (-S final) and use -s none (or fix your
restriction site file, but the above is easier)
On Wed, Mar 24, 2021 at 1:44 PM iacopogentile @.***>
wrote:

Hi. Thanks for the reply! I used juicer_tools.1.9.9_jcuda.0.8.jar. and I
run the following bash script on the cluster:
HiC_seq () {
local run=$1
./opt/juicer/scripts/juicer.sh -g GRCh38 -d ./opt/juicer/$run -s MboI -p ./opt/juicer/mygenome.chrom.sizes -y ./opt/juicer/restriction_sites -z ./opt/juicer/references/GRCh38_latest_genomic.fna -D ./opt/juicer/ -t 1
}
for folder in $folderlist; do HiC_seq '$folder' & done
thx,
IG
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/215#issuecomment-806029954,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW3BZRE5MSQCUXCGNQDTFIQI7ANCNFSM4ZXPNJ7A
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Hi, Thank you very much for the reply and I really apologize for the late response. I tried what suggested and it did not affect the output. The fact that the process is not actually terminated (even though it was printed so) since the split folders and other files in the aligned folder are still there and have not been removed yet.

The splits files don't get removed until you explicitly remove them - in
case the pipeline fails and you need to restart at some midway point.
You ran at the final stage with site=none and got the same result?
On Thu, Apr 1, 2021 at 12:34 PM iacopogentile @.***>
wrote:

Hi, Thank you very much for the reply and I really apologize for the late
response. I tried what suggested and it did not affect the output. The fact
that the process is not actually terminated (even though it was printed so)
since the split folders and other files in the aligned folder are still
there and have not been removed yet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/215#issuecomment-812027636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW2ZDTC44IUTCVKSYBDTGSOB5ANCNFSM4ZXPNJ7A
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Sorry, you should also update to a newer jar.
Basically it's clear that your fragments are assigned incorrectly so that
all intra-chromosomal reads are defined as intra-fragment. And old versions
of the jar will remove this. If you don't want to have to rerun from the
chimeric handling stage (another possibility), you should just run from
final with a new jar and without specifying a fragment file or RE.
On Thu, Apr 1, 2021 at 12:39 PM Neva Durand @.***> wrote:

The splits files don't get removed until you explicitly remove them - in
case the pipeline fails and you need to restart at some midway point.
You ran at the final stage with site=none and got the same result?
On Thu, Apr 1, 2021 at 12:34 PM iacopogentile @.***>
wrote:

Hi, Thank you very much for the reply and I really apologize for the late
response. I tried what suggested and it did not affect the output. The fact
that the process is not actually terminated (even though it was printed so)
since the split folders and other files in the aligned folder are still
there and have not been removed yet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/215#issuecomment-812027636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW2ZDTC44IUTCVKSYBDTGSOB5ANCNFSM4ZXPNJ7A
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Ok, Thx! I just replaced juicer_tools.jar from the previous version with the 1.22 one and I will repeat as you suggested. Do, I need to remove only the odd-looking hic files before running it again?

No, they will get overwritten
On Thu, Apr 1, 2021 at 12:56 PM iacopogentile @.***>
wrote:

Ok, Thx! I just replaced juicer_tools.jar from the previous version with
the 1.22 one and I will repeat as you suggested. Do, I need to remove only
the odd-looking hic files before running it again?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/215#issuecomment-812040644,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW5IVCUC77Y4RWN6VVTTGSQT7ANCNFSM4ZXPNJ7A
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Hi, Thx! the hic output looks normal. I have checked the sam output flagstat and it reported that 0% of reads were properly paired. I guess this is not really expected and it could be link to the original problem

They are aligned separately, I wouldn’t worry about it
On Fri, Apr 2, 2021 at 12:38 PM iacopogentile @.***>
wrote:

Hi, Thx! the hic output looks normal. I have checked the sam output
flagstat and it reported that 0% of reads were properly paired. I guess
this is not really expected and it could be link to the original problem
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/215#issuecomment-812607009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EWZ6MTYHYIMQXYCPVJDTGXXHVANCNFSM4ZXPNJ7A
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Great! Thx!


Tons of "declared but never defined" errors

(I'm not a C expert, so excuse me if this is stupid)
dsgmLib fails to build, and I couldn't find a reason for this (under Cygwin, as cmd won't work and msys crashes):
In file included from C:/devkitPRO/dsgmLib/include/DSGM.h:40:0,
                 from C:/devkitPRO/dsgmLib/source/DGM_wireless.c:1:
C:/devkitPRO/dsgmLib/include/DSGM_quaternion.h:52:19: warning: inline function 'Quaternion_fromEuler' declared but never defined
 inline Quaternion Quaternion_fromEuler(int32 roll, int32 pitch, int32 yaw);
                   ^
In file included from C:/devkitPRO/dsgmLib/include/DSGM.h:11:0,
                 from C:/devkitPRO/dsgmLib/source/DGM_wireless.c:1:
C:/devkitPRO/libnds/include/dsgmDSWiFi.h:12:13: warning: inline function 'setWirelessMode' declared but never defined
 inline void setWirelessMode(enum WIRELESS_MODE mode);

This error seems to be there for every single function. I looked up the error but the only solution was to remove "static" which isn't there to begin with.

Note: I know it's actually a warning, but it becomes an error when building an example.

Update: the errors go away with the use of --std=gnu89.

Should probably be using --std=c11 instead, but I'm glad that it works for you.

Do you think this has something to do with devkitPro's updates? A few months ago I didn't have this problem at all.

Could well be. I'm not currently able to this further, but I'm glad you solved the problem. Have fun with the library!

Never got around to mentioning that -fgnu89-inline works perfectly.
https://gcc.gnu.org/gcc-5/porting_to.html
GCC 5 defaults to gnu11 rather than gnu89, causing a change in how inline functions are handled.

I don't understand how can I use the "--std=c11" thing :/
Someone can help me?

alright, I ported the thing to not use inline function which i think i ported it to gnu11.

I don't understand how can I use the "--std=c11" thing :/ Someone can help me?

You should modify the makefile to add the --std=c11 However c11 have the declared but never defined errors.
Modify the Lib Makefile and add the CFlag -fgnu89-inline.
However even if you do this it wouldnt even work, multiple definition errors.


implement simplified version of vcl::Vector

Maybe we can now try to implement a simplified version of vcl::Vector since we've abandonned the methods and friend functions signatures with buffers.
Maybe.
This will be some attempt done on a separate branch.

Just tried -> compiler errors and no optimization.  Let's abandon this idea.


Send email even if copy request is refused

References

Fixes #8451

Description
Allow the email to be sent even when the copy request is rejected.
Instructions for Reviewers
List of changes in this PR:

Placed the email.send(); method outside of the ri.isAccept_request() condition.

@pilasou already described how to reproduce the problem, see #8451
Checklist

[X] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[X] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[X] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[X] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

Hi @tdonohue! OK, it makes sense to add a configuration for this use case.


Proper way to set up metadata event?

Hi. I can't seem to get a metadata event fired up. I tried as follows:
player = flvjs.createPlayer(mediaDataSource, {
                enableWorker: false,
                lazyLoadMaxDuration: 2 * 60,
                seekType: 'range',
            });

			player.on("metadata_arrived",(e)=>console.log("First attempt to get metadata"));
			player.on('error', (e)=> console.log(e));
			player.on("onMetaData",(e)=>console.log("Seecond attempt to get metadata"));
            player.attachMediaElement(element);
            player.load();

What am I doing wrong?
Thanks in advance!

player.on(flvjs.PlayerEvents.METADATA_ARRIVED, (metadata) => console.log(metadata));

Please pay attention that I do not guarantee whenever onMetaData received, the event will be triggered.

Thanks!!!!

@xqq Good morning. Unfortunately, the metadata event doesn't fire, and I can't understand why.
What is confusing is that other player.on events do fire, but not metadata_arrived. For example, the STATISTICS_INFO, placed immediately after the METADATA_ARRIVED, does fire:
player.on(flvjs.Events.METADATA_ARRIVED, (metadata) => console.log(metadata));
player.on(flvjs.Events.STATISTICS_INFO, e => console.log(e));

Also, metadata does arrive, as I the logger produces the following records:

[FLVDemuxer] > Found another onMetaData tag! f6a40bc6-2ddc-4234-bb7b-35c2c03118aa:1076:4902
[FLVDemuxer] > Parsed onMetaData

Any idea what might be different about metadata_arrived?
thanks again
Christo

MARK


Change value

Any way of changing the value after it has been initialized via code?
For example, I initialize the color picker, and on my "edit" view , I get info via ajax or something, and I have Key value of the color i want to select. Instead of rebuilding it from scratch an option like
$("#randomInput").paletteColorPicker().select(dummy.key,dummy.val);

Hi!
I'm not sure but maybe you can do:
$('#unique-id-7').val("#808080");
$('#unique-id-7').data('paletteColorPickerPlugin').reload();

Please take a look to sample 7 I'm not sure is exacly what you need.


Edges not rendered after upgrade from v6 to v8

After i upgraded from version 6.2.1 to 8.1.2 my edges are not rendered at all.
I didn't have custom handles for any of them and everything was working fine and now i am constantly getting this error:
"couldn't create edge for source handle id: null; edge id: E42-F2".
After few hours of debugging i couldn't find more than that the bounds are missing when 'getHandleBounds' is calling getHandleBoundsByHandleType.
I didn't see anything mentioned that the Handle's are now mandatory with the new version.
Can you please check if there is some bug or breaking changes that were not documented and we need to consider?

Hey!
that sounds strange.. could you try to set connectionMode="loose"?

I also faced same issue when I upgraded version from 6.1.3 to 8.1.1,
Then I figure out problem and changed my code.
Issue is with the way we are using edge id is changed in newer version
In 6.1.3 I was using ==>
[
{ id: '1', className: 'dark-node', position: { x: 0, y: 0 }, ...getCustomeNodeData('1', 'Node 1')},
{ id: '2', position: { x: 300, y: -150 }, ...getCustomeNodeData('2', 'Node 2')}
{ id: 'e1-2', source: '1__src_right_center', target: '2__trg_left_center', ...getCustomeEdgeData(EdgeType.custom, true, '')},
]
in Newer version I had to change code to ==>
[
{ id: '1', className: 'dark-node', position: { x: 0, y: 0 }, ...getCustomeNodeData('1', 'Node 1')},
{ id: '2', position: { x: 300, y: -150 }, ...getCustomeNodeData('2', 'Node 2')}
{ id: 'e1-2', source: '1', target: '2', sourceHandle: 'src_right_center', targetHandle: 'trg_left_center',, ...getCustomeEdgeData(EdgeType.custom, true, '')},
]

Hey!
that sounds strange.. could you try to set connectionMode="loose"? We are not allowing source to source or target to target connections anymore. That can be changed with the connectionMode prop. When do you get this error?

I get this when the graph tries to render each edge. I don't have custom edges defined neither custom handles so i was relying on the default behavior so far. Example of 2 nodes and an edge between them:

``

Hey!
that sounds strange.. could you try to set connectionMode="loose"? We are not allowing source to source or target to target connections anymore. That can be changed with the connectionMode prop. When do you get this error?

The 'connectionMode' didn't helped. I have custom nodes without any  in them and with v6 that worked perfectly with some defaults (sourcePosition - bottom / targetPosition - top). Now after the upgrade to v8 i need to put  in each Custom node i have in order to work, the defaults probably don't work anymore.
Therefore, i wouldn't consider this as an issue maybe but a breaking change which should be mentioned in the documentation so other don't encounter the same problem.

I am glad that you found a solution but could you explain the finding once again? I didn't get it. How did you solve the problem?

With the old version there were default handles even for the custom nodes. After the upgrade i needed to define them for each custom node like:

Thanks for clarification!


Style skeleton button on import screen

Fixes #98723
There is no white skeleton svg available currently, so I kept the red one, which is consistent with the Skeleton3D node, so it kinda make sense.
But now, when it's enabled, you get the square backdrop, similar to other toggle buttons in the editor, like the Snap to grid button.

There is no white skeleton svg available currently

I made one: AnimationPreview.svg.zip

There is no white skeleton svg available currently

I made one: AnimationPreview.svg.zip

Thank you, I brought it in.


ignore ATTR_TRANSITION when comparing to last_service_data

Ignore ATTR_TRANSITION when we compare last_service_data to current.
Will now skip the check if force is True
Moved new context creation below the check - for the debug logger.

LGTM 😄

closed as it looks like this PR was implemented in another commit


Closes: #5

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

no


Update casing in docs headers (phase 1)

Overview
Our documentation guidelines recommend using sentence case for headlines, something our docs currently do inconsistently. This is phase 1 of a few (I broke them up so it isn't 100+ files all at once) to fix up our headers.
NOTE: These PRs are only intended to update headers. Some non-header casing changes may have made their way in, but I'm trying to split these changes into more reviewable chunks.

/azp run mrtk_docs


Identical workflows for copying files between host and container?

The workflows to copy a file from a docker container to the current host and to copy a file from a host to a container seem identical? I haven't used docker in a bit, but I think the
My guess is this one is the wrong way around and needs to reference the host file first and the container destination second.

Hi @wzzrd, thank you so much for filing! I believe this was fixed by https://github.com/warpdotdev/workflows/pull/37 so going to mark as closed, but feel free to reopen if that's incorrect.

@wzzrd if you're in our discord server what's your username so I can add the bug catcher role?

I am now :) It's wzzrd, just like here


[$500] Chat - No gradient between the solid color and the image in the background

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Open any new chat.

Expected Result:
There is color gradient between the solid color and the image in the background.
Actual Result:
There is no gradient between the solid color and the image in the background.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d44d0270ef23b10b
Upwork Job ID:<PHONE_NUMBER>124833280
Last Price Increase: 2023-12-04

Proposal
Please re-state the problem that we are trying to solve in this issue.
There is no gradient between the solid color and the image in the background
What is the root cause of that problem?
imgbot made an optimization to the background images which caused the colour gradient on them to break. This was already fixed in a previous PR by ignoring the file: https://github.com/Expensify/App/pull/30512
However, due to changing the file names (by introducing a dark and light version of the background) in https://github.com/Expensify/App/commit/5a5be5ef74a1a12209fc3c65ed0f6729cbfad25e the image was optimized by imgbot again before the files were updated in .imgbotconfig: https://github.com/Expensify/App/commit/0feb15519163fe60b0da7bf7517efab3a3cc6d1d
What changes do you think we should make in order to solve the problem?

Reverting the automated optimization of ImgBot (https://github.com/Expensify/App/commit/67c42cbd1dfa4c6e05b8d9a488fcf086e1ef7477) should do the trick. Afterwards the background images won't be re-optimized due to the updated .imgbotconfig

I am not convinced this is a bug. it looks like either the expected behaviour or extreme UI polish:


inline block

Do you want to request a feature or report a bug?
not sure... possible a feature
What is the current behavior?
when inserting a custom block, it always uses a new line, I want to use the inserted block in the middle of another blocks
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/gmertk/e61z7nfa/.
there are many people asking for this, but there is no specific answer, it seems is possible using custom blocks but not sure how
What is the expected behavior?
the inserted block will be between for ex the selected anchor, so there will be text before and after. it can be inserting text or an image for example
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
last version from npm, OSX... not sure if is a bug. a feature or if it ever worked
Thanks

It would be a nice feature. I'm also faced with it and would be great to make atomic blocks inline.

Yeah..I am also on same boat..can't insert image(atomic block) in between block without splitting that block...
Alteast if some workaround is provided then that will also be very helpful.

here is the inline block solution issue852

here is the inline block solution issue852


File Location

File Location
User Story
As a bootcamp student,
I want the files for my project to be in the correct directories,
so that the webpage displays correctly.

Acceptance Criteria
GIVEN a Prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can see the fully formatted version of the webpage, with relevant Javascript and CSS.

Resolved. Moved all files up a directory using Git Bash

I had to add this extra Issue in the Repository and resolve it.
I had created a folder within a folder, somehow.
The path of the directory was showing as:

"User 1/bootcamp/prework-study-guide/prework-study-guide"

and there were some files housed inside that path, and some just in:

"User1/bootcamp/prework-study-guide"

I resolved the issue using Git Bash and by consulting the Github Docs on how to move files and push the changes to the main Repository.


Does not recognize daily note folder

OS: Windows 10
I have a daily note folder to keep my daily notes away from the main pane. However the plugin isn't recognizing it. It will only recognize daily notes that are in the main folder. When I make a daily note thru the app and then move it, it also breaks the recognition and says it doesn't exist.

Hi @lillianhorin, what do you have as your "New Note Location" in Settings > Daily Note?

The calendar will only recognize notes placed in that directory.

This happens to me also on Windows 10
These are my Daily notes settings & other info:

I am also having this issue. I have the Daily Notes setup to put new daily notes in "01 Notes", with title format "YYYY-MM-DD". When I use this plugin it simply creates new notes in the main folder.

This should be fixed now in version 1.1.0. Please let me know if you're still seeing this issue after updating!

Hi, I have the same issue.  I try to create a new day note in the future and the create button is not responsive.  All folders and template locations are good.  I'm on obsidian v9.10

Unfortunately now the plugin fails to install via the settings menu

Version 1.3.1 fixes the problem for me.

This should finally be resolved in v1.1.3. Sorry for the earlier confusion!

Excellent, thanks so much!
This is an awesome plug-in, glad you made it 😄

The issue is resolved for me too - thanks!

Apologies for not getting to your question earlier! And it's all fixed for
me now too. Thank you!
Best wishes,
Lillian Horin
Ph.D Student, Harvard University
Pronouns: she, her, hers
Website http://lillianhorin.com
On Sat, Oct 31, 2020 at 12:48 PM akicebear<EMAIL_ADDRESS>wrote:

The issue is resolved for me too - thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/liamcain/obsidian-calendar-plugin/issues/14#issuecomment-719958810,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF224MD266O5QPB4B2GHHRTSNQ5WLANCNFSM4TFKTVIA
.

Apologies for not getting to your comment sooner! It is all resolve for me. Thanks for the great plugin!


Add screen for configuring authentication

The new configuration screen is available from a button on the home screen.
A basic global authentication service AuthenticationService has been added. The service implements login and password update/reset. Besides this service, the authentication status is available to the app in the cubit Authentication which is updated in response to (async) calls to AuthenticationService in accordance with the usual bloc patterns.
The password is persisted using the library flutter_secure_storage. There's not yet special handling of web.

When I run
flutter pub get

it changes the lock file compared to what is currently there.

When I run
flutter pub get

it changes the lock file compared to what is currently there.

Yeah I've seen that quite a bit. It looks to me like we need to agree on the version of the CLI/SDK we're running. The easiest is probably that we always use the latest version.
I just ran flutter upgrade followed by flutter pub get and didn't get any changes.


Delete key vault with the 'Soft-delete' option 'Enabled'

Hi,
As far as i know, once the Az Key Vault is created, it's Soft Delete feature is enabled by default. I recently created an Az Key Vault with the Soft Delete feature enabled. Is it possible to permanently delete it? as i can't find any option to 'Disable' the Soft Delete feature.
When i deleted it, it told me that 'it will remain in my subscription as a hidden vault.' Does that mean, i'll be charged during it's 90 days retention period?
Let me know please.
Thanks.

[Enter feedback here]

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

ID: 5fb5d787-c259-1083-c021-97fb0790122e
Version Independent ID: 352d7d3f-da52-bcd0-3efb-587e99066967
Content: Azure Key Vault soft-delete
Content Source: articles/key-vault/general/soft-delete-overview.md
Service: key-vault
Sub-service: general
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@SAIMMINHAS
Thanks for your feedback! We will investigate and update as appropriate.

@SAIMMINHAS
Thank you for your time and patience throughout this issue!
I recently created an Az Key Vault with the Soft Delete feature enabled. Is it possible to permanently delete it?

Yes, you can delete or permanently delete (purge) the Azure Key Vault even with the Soft Delete feature enabled.
To delete the AKV, you can do this within the Azure Portal, PS, or CLI.
Remove-AzKeyVault -VaultName 'ContosoVault'
To Purge a soft-deleted key-vault (WARNING! THIS OPERATION WILL PERMANENTLY DELETE YOUR KEY VAULT)
Remove-AzKeyVault -VaultName ContosoVault -InRemovedState -Location westus

When I deleted it, it told me that it will remain in my subscription as a hidden vault. Does that mean, I'll be charged during its 90 day retention period?
Billing implications:

In general, when an object (a key vault or a key or a secret) is in deleted state, there are only two operations possible: 'purge' and 'recover'. All the other operations will fail. Therefore, even though the object exists, no operations can be performed and hence no usage will occur, so no bill. However there are the following exceptions, for more info.

If you have any other questions, please let me know.
Thank you again for your time and patience throughout this issue.


Provide support for different locations of release notes

Background
Before refactoring, RLS notes were located in PR comments and migrated to Issue comments.
After refactoring, RLS notes expected location is in the PR description only.
Feature
Add logic to control where RLS notes could be located.
Possible options (one location from following):

Issue

description
comments

PR

description
comments

Extra issue field or property - defined by project (not part of repository !!!!)

I would suggest to break this up to 3 (probably) separately deliverable tickets.

From PR comments (already existed)
From Issue body and comments (similar to PR)
From a project field (completely new logic I suppose)


Allow python 3.12 for cli

Don't want to have to recreate another set of envs on the GAE next time we upgrade. So allowing greater than 3.10

I wonder what the reason was we restricted it 🤔

I wonder what the reason was we restricted it 🤔

It was because of the pyyaml version not being able to be installed on 3.12, but we've fixed that now


px-dropdown-click is not firing the event once an item is selected.

HTML:
<px-dropdown>                 <px-dropdown-text class="px-dropdown-text">Text</px-dropdown-text>                 <px-dropdown-chevron class="px-dropdown-chevron"></px-dropdown-chevron>                 <px-dropdown-content class="px-dropdown-content" max-cont-character-width="10"                 extend-dropdown="true" extend-dropdown-by="15" items={{data}} px-dropdown-click={{downdownHandler()}}>                 </px-dropdown-content>               </px-dropdown>
I see the function being called during the page load. The function is called for each item in the drop down but after the page is loaded the function is not called once an item is selected.

@runn-vermel

Hi galva-ge,
px-dropdown-click is an event that is fired whenever an item in the dropdown is clicked -  it passes the click event when it's fired.  no need to specify it as an attribute in your code.
You have to listen to this event being fired, not specify it in the tag itself.
Please let us know if you need more clarification.
Thanks!

Hi runn-vermel,
Could you please provide code snippet for this.

HI @YSambaiah
You can find all the info you need about event listeners right here: https://www.polymer-project.org/1.0/docs/devguide/events


Widget not showing up?

Hello,
first thank you for this plugin! Seems to work, matomo is showing data.
But when I want to add the widget - it is not showing up? Maybe I forgot a step? I configured the auth token.
Thanks in advance!

Nevermind I didn't notice that I can scroll down... :D
I will close this.
congrats on this plugin 👏  I will test it in the next days and leave a review on ocms


Added Insertion sort

PR accepted @madhu-ram2012
Successfully merged to HacktoberFest_22 repo.
Need to Star this repo incase not done.
🚀🚀👍


[Bug] Remove tooltip from 3d Bar when we remove focus from the chart

Version
14.0.0
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=bar3d-dataset&gl=1
Steps to Reproduce

Focus on an element of the 3d Bar chart
Verify that we show the tooltip
Remove focus from the chart
The tooltip doesn't disappear

Current Behavior
The tooltip stays in the chart
Expected Behavior
The tooltip should disappear like in the other charts (ex: https://echarts.apache.org/examples/en/editor.html?c=bar-simple)
Environment
- OS: Windows 10
- Browser: Chrome 104.0.5112.82, Edge 104.0.1293.54, Firefox 103.0.2
- Framework: Angular 14

Any additional comments?
No response

+1
Also, the example tooltip code needs to be fixed:
          encode: {
            x: 'Year',
            y: 'Country',
            z: 'Life Expectancy',
            //tooltip: [0, 1, 2, 3, 4] = incorrect display
          },
          tooltip: [0, 1, 2, 3, 4]

Still not fixed as of Oct 2024.
There is a JS error TypeError: Cannot read properties of undefined (reading 'toFixed') coming from Grid3DView #L184 calling core/echarts.ts
There is also a related issue with legend coming from this same error.


wait_recording seems busted

I'm trying to get picamera to stream indefinitely and I think the wait_record is close to --timeout in raspivid. When I try to use it, its throwing exceptions.
pi@raspberrypi:~ $ pip3 list | grep picamera
picamera          1.13
pi@raspberrypi:~ $ cat thing.py
import picamera
import subprocess
import time
if name == "main":
gstreamer = subprocess.Popen([
'gst-launch-1.0', '-v',
'fdsrc',
'!', 'h264parse',
'!', 'rtph264pay', 'config-interval=1', 'pt=96',
'!', 'gdppay',
'!', 'tcpserversink', 'host=<IP_ADDRESS>', 'port=5006'
], stdin=subprocess.PIPE)
camera = picamera.PiCamera(resolution=(1280, 720), framerate=30)
camera.hflip = True
camera.wait_recording(10)
camera.start_recording(gstreamer.stdin, format='h264', bitrate=1000000)
print("Hello World!")
#time.sleep(30)
#camera.stop_recording()
#gstreamer.stdin.close()
#gstreamer.wait()
pi@raspberrypi:~ $ python3 thing.py
Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
/GstPipeline:pipeline0/GstTCPServerSink:tcpserversink0: current-port = 5006
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/picamera/camera.py", line 1161, in wait_recording
encoder = self._encoders[splitter_port]
KeyError: 1
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "thing.py", line 17, in
camera.wait_recording(timeout=10, splitter_port=1)
File "/usr/lib/python3/dist-packages/picamera/camera.py", line 1165, in wait_recording
'port %d' % splitter_port)
picamera.exc.PiCameraNotRecording: There is no recording in progress on port 1
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstSystemClock
Got EOS from element "pipeline0".
Execution ended after 0:00:00.000381678
Setting pipeline to PAUSED ...
Setting pipeline to READY ...
Setting pipeline to NULL ...
Freeing pipeline ...

Your usage of wait_recording is incorrect. It is "wait whilst recording", ie call it after start_recording. It is preferable compared to time.sleep in that it handles any exceptions thrown by the library.
See docs https://picamera.readthedocs.io/en/release-1.13/api_camera.html#picamera.PiCamera.wait_recording
https://picamera.readthedocs.io/en/release-1.13/recipes1.html#recording-video-to-a-file


mermaid.js support for diagrams

Will there be mermaid support https://mermaid-js.github.io to render diagrams

Yes, I think it is necessary to add a diagram support.

发件人: Dharmjit Singh<EMAIL_ADDRESS>发送时间: Thursday, December 31, 2020 12:14:12 AM
收件人: wangchucheng/hugo-eureka<EMAIL_ADDRESS>抄送: Subscribed<EMAIL_ADDRESS>主题: [wangchucheng/hugo-eureka] mermaid.js support for diagrams (#26)
Will there be mermaid support https://mermaid-js.github.io to render diagrams
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/wangchucheng/hugo-eureka/issues/26, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJEXGNABFJ2QNGL672CKZODSXNGVJANCNFSM4VOPFRGA.


InvalidOperationException during compilation of some erroneous list comprehensions.

Description
Compiling an erroneous list comprehension that has a misplaced literal where the receiving variable should be results in an InvalidOperationException instead of a syntax error.
Example reproducing code:
lst = ["a", "b", "c"]
total = sum(1 for 1 in lst)

This happens on IronPython 2.7.10, embedded inside Unity in my case - but it reproduces in ipy on the command line as well.

Thanks for the report, seems to be an issue with all types of for statements:
for 1 in []: pass


Test notify_team_new_comment workflow

Description
Goal is to test notify_team_new_comment before adding the same changes to kolibri, studio, ricecooker, kolibri-design-system
Issue addressed
https://github.com/learningequality/kolibri/issues/12567
Addresses #PR# HERE
Before/after screenshots

Changelog

add notify_team_new_comment workflow

Description: Summary of change(s)
Products impact: Choose from - none (for internal updates) / bugfix / new API / updated API / removed API. If it's 'none', use "-" for all items below to indicate they are not relevant.
Addresses: Link(s) to GH issue(s) addressed. Include KDS links as well as links to related issues in a consumer product repository too.
Components: Affected public KDS component. Do not include internal sub-components or documentation components.
Breaking: Will this change break something in a consumer? Choose from: yes / no
Impacts a11y: Does this change improve a11y or adds new features that can be used to improve it? Choose from: yes / no
Guidance: Why and how to introduce this update to a consumer? Required for breaking changes, appreciated for changes with a11y impact, and welcomed for non-breaking changes when relevant.

Steps to test

Comment on issue
Check if notification is visible
...

(optional) Implementation notes
At a high level, how did you implement this?

Does this introduce any tech-debt items?

Testing checklist

[ ] Contributor has fully tested the PR manually
[ ] If there are any front-end changes, before/after screenshots are included
[ ] Critical and brittle code paths are covered by unit tests
[ ] The change is described in the changelog section above

Reviewer guidance

[ ] Is .github repository reference changed to https://github.com/learningequality/.github/tree/main ?

After review

[ ] The changelog item has been pasted to the CHANGELOG.md

Comments
MAKE SURE CHANGE DOES NOT REFER TO FORK!

Resutl of testing with https://github.com/rparadowski/.github/tree/12567_issue :