could not determine kind of name for C.rados_set_pool_full_try

[root@node1 ceph]# go build  pool.go
github.com/ceph/go-ceph/rados
/root/odev/src/github.com/ceph/go-ceph/rados/ioctx_octopus.go:23:2: could not determine kind of name for C.rados_set_pool_full_try
/root/odev/src/github.com/ceph/go-ceph/rados/ioctx_octopus.go:35:2: could not determine kind of name for C.rados_unset_pool_full_try

Your local version of the ceph may not be "octopus". From the README:

go-ceph tries to support different Ceph versions. However some functions might only be available in recent versions, and others may be deprecated. In order to work with non-current versions of Ceph, it is required to pass build-tags to the go command line. A tag with the named Ceph release will enable/disable certain features of the go-ceph packages, and prevent warnings or compile problems. ...

If you are using ceph from packages, you can use the package manager to get the ceph version and then backtrack to the release series code name. So if you are on ceph version 14, that's "nautilus". See this ceph versions index for more details.
Once you do, use build tags to select the ceph version as noted on the readme.
Lastly, do note that the go-ceph's recent releases only support nautilus and newer.

thanks!


fix: ensure storages are force flushed during uplink shutdown

Closes #

Changes

Why?

Storage might contain publishes that aren't enough to overflow from in-memory buffer onto disk, in this situation, during shutdown, uplink must force them to be written into file to save them from being lost.
Trials Performed

Consult linked PRs for more detail

Let's consider this situation with regards to full-graceful shutdown:

Uplink is running on device, shutdown triggered, inflight file created with 10 unacked(MAX_INFLIGHT_SIZE) and 10 in_channel(Capacity of channel between AsyncClient and EventLoop) publishes persisted into it.
Uplink restarts, loads 10 publishes into channel(because of cap and considering EventLoop ain't handling requests), rest 10 publishes still left unhandled as network is down.
Now if uplink were to shutdown again before getting network, would we want to lose the 10 packets that were not pushed onto channel or duplicate the 10 packets that were pushed into channel?
This is something that has been bugging me for a while.

Illustrating the scenario, consider 10 unacked publishes are (A, B) and 10 in_channel publishes are (C, D), first shutdown of uplink writes them to inflight file as (A, B, C, D), on restart only first 10(i.e. A, B) publishes are loaded while rest 10(C, D) are not. The two possible ways to handle next shutdown would be:

lose last 10 and be left with only first 10(i.e. on disk: A, B)
Duplicate first 10(i.e. on disk: A, B, C, D, A, B)

NOTE: those 10 publishes in_channel after restart will now avalanche if we had uplink restarts continuously(A,B,C,D,A,B,A,B...), while only 10 packets will remain lost in the other scenario(A,B). The second path is what I have taken in the current implementation, but wanted to document this decision and ask for feedback.

Code to ease readability of control handles as requested by @tekjar is included in the PR https://github.com/bytebeamio/uplink/pull/316 and this commit


Update retrieving-query-suggestions-using-the-search-rest-service.md

Category

[ x] Content fix
[ ] New article
Related issues: fixes #2703

What's in this Pull Request?
Fix invalid code sample link.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/retrieving-query-suggestions-using-the-search-rest-service.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Support panning inertia or continued movement?

Part of #974.
Microsoft calls a move or pan gesture that is removed with sufficient speed to continue movement an "intertia" event so reusing the term here. Is this something we want to support? Previously it was extra work to include this. But the solution we landed on in #974 implements most of this anyway.
We may still decide not to support this, there were concerns that this could be distracting (among other things). But it should no longer be a technical challenge. @terracoda @emily-phet what are your thoughts on this now?

I'm still on team 'minimize possible distractions' for this one. I can just imagine young kids figuring out they can 'throw' the sim screen. I would prefer we hold off on this unless and until we find that it would fill a specific need.

OK sounds good, lets keep out for now. CLosing and we can revisit if we receive other feedback or evidence that this is important.

Apologies for missing the comment period. I agree we can wait to see what specifically students with low-vision need in this area.


WhereIn with integer returns partial match on string. (int) 1 returns (string) '1ab1fdeb-7e83-4be9-a1b4-52da267d2733'

Laravel Version: 6.12.0
PHP Version: 7.4.0
Database Driver & Version: MySQL 8.0

Description:
Passing an integer 1 into a whereIn against a string column (i.e. UUID), returns a match against 1ab1fdeb-7e83-4be9-a1b4-52da267d2733
Model::whereIn('uuid', [1])
    ->each(function(Model $model) {
        dd($model->uuid); // '1ab1fdeb-7e83-4be9-a1b4-52da267d2733'
    });

To prevent this from happening you have to force the whereIn array to a string, which is not very intuitive, even if technically correct.
Model::whereIn('uuid', array_map('strval', $this->ids)); // returns no results as expected

This is the way MySQL compares values and not a Laravel issue.
It's the application's responsibility to prevent comparisons like this, e.g. by validating user input.

I understand its not a Laravel issue... but wonder if there is something that could be built into eloquent in a similar way that you can cast model values on the way out. What if we can cast them on the way into the query too?

Please see the answer by @staudenmeir


test: add a test for binary column

What problem does this PR solve?
Add a test for the binary columns to make sure that all binary number inside 0x00 to 0xFF is good to encode to canal-json format.
Check List
Tests

Integration test

Release note
None

/run-all-tests

/run-kafka-tests

/run-kafka-integration-test

/run-kafka-integration-test

/run-kafka-integration-test

Unit test TestRegionWorkerExitWhenIsIdle failed.

/run-all-tests

/run-integration-test
/run-leak-test


Update codeql.yml to manually start code ql jobs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.724%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
30728

Relevant Lines:
68919

💛  - Coveralls


[Doc] old version of document links  are pointed in Configuring IWA on Linux page

Please update the link in the "For more information about IWA" in [1]. This link is referring to IS 5.2.0 Document.
[1] https://docs.wso2.com/display/IS540/Configuring+IWA+on+Linux

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Added CanFulfillIntentRequest feature

For more information about CanFulfillIntentRequest, visit these 2 links:
https://developer.amazon.com/docs/custom-skills/understand-name-free-interaction-for-custom-skills.html
https://developer.amazon.com/docs/custom-skills/quick-start-canfulfill-intent-request.html
The way it's meant to work is that in every intent should validate the scenario when this value is coming:
skill.onIntent('MyIntent', (alexaEvent) => {
    if (alexaEvent.request.isCanFulfillIntentRequest) {
        const canFulfillIntent = { canFulfill: 'YES' };
        return { canFulfillIntent };
    }

    // DO YOUR NORMAL CODE
});

Coverage decreased (-0.1%) to 98.839% when pulling b4683347f7a0a2c97e5abde87daff8664ef5b4f8 on omenocal:dev into de20cc6af99da4c0d92b23955287b673d0a1bddb on mediarain:dev.

After discussing with @omenocal we decided to go this way:
skill.onCanFulfillIntentRequest((alexaEvent) => {});

Agreed about avoiding imitating this as an intent. I know we've done that elsewhere, but it was a mistake. We should be more faithful to the how Alexa actually works.


Data set directory structure

Hi, I am very interested in your research, but I ran into some problems when replicating the code: the program could not read the data.
What is the data set directory structure?
What should be the argument to train.py --data_dir?
Thank you.

Same as the following question:
https://github.com/arthurchen0518/DirectionNet/issues/4#issuecomment-1475642915

What is the version of your tensorflow?


feat(#1061): unify records results title

closes #1061
This PR includes show records results title on top of records list in Explore and Annotation Mode and unify styles

To me, this message should be more explicit and visually more pronounced, saying something like:

Filters are not part of the Rule, but are applied to the following record list.

And maybe only show this when filters are set.

The "Records with..." message will be applied only for "Define Rules" mode. The "Annotate" and "Explore" modes will keep intact.


small edit: improve GetFirstWords (don't use fixed sized string buffer when not checking overflow)

update STWeil's "Fix function GetFirstWords and modernize function GetPrefixes": GetFirstWords : don't bother with the extra string copy and fixed-sized string buffer which could be attacked by using a 1K+ length word prefix (insane; I know. Anyway)
This also accounts for these commit duplicates: 5943d081e20a87832d575c67b31a3bbd632bcdeb, 47fe923bbb624349eed6155322cf8e828979b81c, 0231ff750e4f146ee8eae9b890b7638401120a8e, c10667d6a83674ec0a8a0d834031211e95a7267e, b04fbb44ebaef0bd811d95bc30ef0e023e796e67, 054dba3e2a61b50d76b693a7d5fa1b45dace72d9

Thanks!


Processing backlog, reduced functionality

Hi all,
I encounter the following message in the overview page: "Your system is currently processing a backlog of audio. This can take several hours before normal functionality of your BirdNET-Pi resumes."
Generally the BirdNet-Pi still works and detects birds and uploads the findings to BirdWeather, for example, but I cannot listen to recordings anymore.
The system runs for over a year now and this has not occurred up to now.
The Birdnet-Pi has all the updates, I restarted it several times, and I let it run with a switched-off microphone for over three days to enable it to work through the backlog.
Any suggestions what to do?
Best regards,
                     -- Wolfram

Additionally I now deleted all data. The error message was gone, but immediately after detecting the first bird re-appeared.
Please help


Feature Suggest / Datapack Check

Hey, i had a simple Question
is there a way, to look for datapack updates on modrinth?
I had allways the issue, i keep all my mods up to date, but sometimes, i don't see updates for datapacks,
Could you implement this feature or is this way not possible?
Thanks

I'm not sure how possible it is cause some data packs on modrinth are package as mods (Modrinth feature) some not. That is good request but I will left it to next year🫡

Thanks, if you had an solution, i will thank you for that :)

Ah yeah, could it be pissible with file hashes? I think modrinth give something about file hashes and it would display only if a new hash in online.
Had only looked in automodpack, there it compare the files and version with hash etc

Ah yeah, could it be pissible with file hashes? I think modrinth give something about file hashes and it would display only if a new hash in online.
Had only looked in automodpack, there it compare the files and version with hash etc

That's how all mods works. My, Mod Menu, AutoModPack and else


A way to assess normalization quality

Dear all (@jcohenadad  @kennethaweberii ),
Currently, we are using the Dice Coefficient (between manual vs automatic mask) as a metric to assess the performance of newly trained model.
I had a meeting with @rohanbanerjee and we were discussing possible metrics to assess the quality of EPI to template normalization/registration (as this is one of the most important outcomes of automated segmentation from the user perspective, right?).
Therefore, the question is: how to assess and quantify the quality of template normalization?
I encountered this recent preprint.
Here they use a multi-step normalization method and compare it with single-step normalization (details are not important for my question), using the following approach:
`

To evaluate the registration performance, the binary cord segmentation mask in the EPI space was registered to PAM50 space using each method, and the result was compared to the binary template cord mask. Two indices were defined by calculating: i) the number of voxels in the template mask that are missed in the registered mask, normalized by the total number of template cord voxels, and ii) the overlap between the registered and the template cord masks, normalized by the total number of voxels in both masks.

`
What do you think? Is this a valid approach to evaluate the performance of registration?

I'm not sure I fully understand the context of this issue. The current repository is about cord segmentation on EPI scans. Why are we talking about registration to the template here?

I'm not sure I fully understand the context of this issue. The current repository is about cord segmentation on EPI scans. Why are we talking about registration to the template here?

Hi Julien,
I posted this here as it is relevant to this project and the publication that will come out of this project (instead of an email). Sorry that it was not clear.
I thought that one of the main reasons that we need to manually segment the cord is to be able to properly register EPI data to the PAM50 template (from the perspective of an SCT user who is analyzing fMRI data ).
Then, for someone interested in spinal fMRI analysis, would not it help to see that the normalization using automated segmentation performs comparable to manual segmetation?
If yes, would not it be important to demonstrate it in the paper?

I thought that one of the main reasons that we need to manually segment the cord is to be able to properly register EPI data to the PAM50 template (from the perspective of an SCT user who is analyzing fMRI data ).

indeed, it is

Then, for someone interested in spinal fMRI analysis, would not it help to see that the normalization using automated segmentation performs comparable to manual segmetation? If yes, would not it be important to demonstrate it in the paper?

Demonstrating improvements in registration is quite tricky. One thing we could do, though, is run the registration pipeline and, for each site, show the group average EPI in the PAM50 space (one fig: one sub-panel per axial view)

Demonstrating improvements in registration is quite tricky. One thing we could do, though, is run the registration pipeline and, for each site, show the group average EPI in the PAM50 space (one fig: one sub-panel per axial view)

Thank you, Julien!
Yes, definitely!! I assume that quality would be tricky to distinguish from the average (hopefully not!) . We can maybe accompany it with some sort of heat map (like a standard deviation map) to show consistency across subjects.
What did you think about the metric that was employed in the recent preprint?

Yes, definitely!! I assume that quality would be tricky to distinguish from the average (hopefully not!) . We can maybe accompany it with some sort of heat map (like a standard deviation map) to show consistency across subjects.

I wouldn't go that far-- I think that a group average per site is enough. If reviewers want more meat, we will produce more meat.

What did you think about the metric that was employed in the recent preprint?

What preprint are you referring to?

I wouldn't go that far-- I think that a group average per site is enough. If reviewers want more meat, we will produce more meat.

Good to know, thank you!

What preprint are you referring to?

This one that I posted above!
https://www.researchsquare.com/article/rs-3889284/v1


Add new api for Search addresses

Search Districts

List Of Districts By Search Params
http://ehealth.edenlabllc.com/api/uaddresses/districts
Districts by Region
http://ehealth.edenlabllc.com/api/uaddresses/regions/290f1ee-6c54-4b01-90e6-d701748f0851/districts
Settlements by district
http://ehealth.edenlabllc.com/api/uaddresses/districts/290f1ee-6c54-4b01-90e6-d701748f0851/settlements

till April both api should work

@svetavedmed deployed


Update widget with message when no execution are founds

What feature do you want to see added?
Update widget with message when not execution are founds
To be consistent with other widget (like build queue) that display No builds in the queue.

Upstream changes
No response
Are you interested in contributing this feature?
No response

Hi @jonesbusy, Sorry but I'm not able to understand what kind of message update are you looking for in the widgets and what should trigger this update. Could you please clarify this a bit more?

See screenshot there is no message now.
Something similar of the build queue widget as you can see.
It's confusing now to see an empty widget that is not indicating that there is no execution

Okay, So, Correct me if I am wrong, you require a new widget which displays the jobs which are not in execution right now or a message which displays that that no job execution is taking place right now.

Like the build queue. Just a message like "No next executions found."
If you look carefully at the screen, the UI is not consistent. All widgets are empty, but only the next execution one doesn't display a proper message.

Okay... I get it now. Sorry, I wasn't able to spot it at first. Fine, I believe I can try to fix this. Could you please assign this issue to me?

We don't assign issues so that everyone can pick it up.
Just submit a PR and I will review it.

Oh. Okay. I'll try to submit a PR by next week. I need some time to understand the codebase first and Again thanks for the clarification.

Needed to revert because of regression

Implemented


Ack/nack hangs when channel is closed after cancellation

When a channel gets cancelled with basic_cancel() while messages are being processed, then:

When in the Closing state, calling nack() on a delivery returns an error as expected: invalid channel state: Closing.
After the channel gets closed and removed, the call to nack() never resolves.

I suspect this is somewhat similar to amqp-rs/lapin#409, maybe?
I can try to put together a reproducing example if that's helpful.

This is very interesting thanks.
A reproducer would be appreciated to speed up the debugging

Will do!
I've been unable to reproduce with a smaller example, so I'll need to dig deeper.

I can only reproduce inside a #[tokio::test]. Inside #[tokio::main] it seems to work just fine! 😕
At this point I'm suspecting that this is some tokio/executor/runtime-related "I'm holding it wrong" issue, and probably not a lapin issue! So feel free to ignore/close if you suspect the same.
I've made a repro here with both a main function and a test function if you're interested.


add kaggle starter link

Changes:

Fixed logic for mounting Google drive outside Colab.
Added Kaggle starter notebook edit link with Open in Kaggle  badge.

@abhishekkrthakur If u have some time to spare, please take a look at this.


compaction-filter: consider  mvcc-delete as redundant key to trigger rocksdb compaction

What is changed and how it works?

Issue Number: Close #17269

What's Changed:
compaction-filter: consider mvcc.delete as redundant key to trigger Rocksdb compaction

Release note

compaction-filter: consider mvcc.delete as redundant key to trigger Rocksdb compaction

Could this enhancement be cherry-picked to both v7.5 and v8.1 ?

Could this enhancement be cherry-picked to both v7.5 and v8.1 ?

You can add type/bug to the issue and then cherry-pick to other LTS branches. 😅

@AndreMouche
This change has a similar PR before https://github.com/tikv/tikv/pull/13352, it's not merged because there could be performance regression in some cases.
We need a complete and comprehensive test and decide whether this should be enabled by default.

@cfzjywxk  I think the PR you mentioned #13352 is similar with https://github.com/tikv/tikv/pull/15342  which is already merged long day ago. So the current impact of this PR on performance is controllable, because the main feature that may impact performance  (#13352 or #15342)has already been merged in #15342


detailed report, fix report creation

@Arieg419 - do you know if this passes the test suite (or if it runs via docker compose)?

hey @onlydole , i've moved this back to draft for now and am working through it --
The current state is that the 'detailed report' end report looks good but streams redundant intermediate steps to the UI.
I tried to bisect and reverted to about two weeks ago, and I am also getting a janky, concise 'detailed report' (shorter than a basic report), so I am wondering when it last worked. If there was a known hash where it functioned well, I'm happy to work off that, @assafelovic; otherwise, I think I'll have a fix for this tomorrow.

cc @assafelovic detailed report looks good now, however, it's still missing references. will add tomorrow, and publish the PR for review.


[storefront] review counts display "undefined" incase no ratings has been made

the feature works fine


add toml 1.1 support

available now, https://github.com/notnmeyer/tsk/pull/37

https://github.com/notnmeyer/tsk/pull/39


Update README.md

Closes #3 .

we merged a similar change during summer, sorry for letting this one linger


Remove Konduit and gieffektivt.no titles

[ ] Auth0

[x] Create new tenant
[x] Migrate applications, rules and actions
[ ] Migrate users

[x] Send support ticket for password hashes and salts
[ ] Import users to new tenant

[ ] Change backend check from konduit-user-id to gieffektivt-user-id
[ ] Test new applications locally
[ ] Switch prod application client ids and secrets etc...

[ ] Update google cloud secrets

[ ] Test

[ ] Profile page
[ ] Admin panel
[ ] Scheduled routines
[ ] Swagger

[x] Sanity

[x] Update default studio URL
[x] Update CORS policies as needed

[ ] Website

[x] Update title on profile page
[ ] Update all client side calls to use gieffektivt-user-id
[x] Change readme and package.json

[ ] AvtaleGiro i nettbank
[x] Vipps recurring

Sanity Studio domain, for reference

Tab title when logged into Min Side, for reference

Continued in issue #363


Fix "Timers cannot be stopped from another thread" issue during shutdown

Fix "Timers cannot be stopped from another thread" issue during hyperiond shutdown
@m-seker Thanks for providing the code changes fixing the issue.
What kind of change does this PR introduce? (check at least one)

[X] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[X] No

If yes, please describe the impact and migration path for existing setups:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated (docs/docs/en)
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

Ready to merge?

Works nicely at my end!


fix: load seeders using typeorm (#324)

I prefer the solution from #339 since it relies even more on TypeORM than this one.

Closing this one since #339 is a nicer solution


Unsupported Radiator Valve TS0601 ( Moes Zigbee Radiator Valve)

Hello, I have chosen zigbee2mqtt mainly to put the thermostatic valves of the Moes model TS0601 directly in my Home Assistant (of which I am attaching the image) and I find that while doing the scan the device is detected but it does not connect me as Manifacturer errors come out Unsupported. Now I have 5 thermostatic valves that I can't get to work. Can you put them in the code so that they are displayed to me and I can use them through zigbee2mqtt?
My Dongle is the Sonoff Zigbee USB Dongle.
Thanks
P.S : Small note. When I get the series of errors in red at the bottom right the device is called TZE200 while then in the model it is called TS0601
EXAMPLE OF VALVES

ERROR MODEL UNSUPPORTED

ERROR RED IN HOME ASSISTANT ZIGBEE2MQTT PAGE

I have the last version of Zigbee2Mqtt ( Current version: 1.18.1-1 )

Versione
core-2022.2.8

Tipo di installazione
Home Assistant OS

Sviluppo
false

Supervisore
true

Docker
true

Utente
root

Ambiente virtuale
false

Versione Python
3.9.7

Famiglia del sistema operativo
Linux

Versione del sistema operativo
5.10.92-v8

Architettura della CPU
aarch64

Fuso orario
Europe/Rome

Thanks

Hi!
I am just a user so feel free to judge my comments based on that:

latest version is 1.25.0, not 1.18.1-1, so try to update first
I think I have the same radiator valve, and it is working, but I am using the edge version of Zigbee2MQTT:

Hope this helps.


Add auto merge

@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/kivy-garden-feedstock#2.


[fix][test] Fix TransactionTest failure due to  clean up after class

Motivation
Fix testGetTxnID: transaction Id is not begin at 0;
testSubscriptionRecreateTopic : topic is already exsit.
and testPendingAckMarkDeletePosition: failed to get TP manager stats due to testEndTPRecoveringWhenManagerLedgerDisReadable set a mock 'TransactionPendingAckStoreProvider'  into pulsar.
Modification
fix them.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Matching PR in forked repository
PR in forked repository: https://github.com/liangyepianzhou/pulsar/pull/3
After opening this PR, the build in apache/pulsar will fail and instructions will
be provided for opening a PR in the PR author's forked repository.
apache/pulsar pull requests should be first tested in your own fork since the
apache/pulsar CI based on GitHub Actions has constrained resources and quota.
GitHub Actions provides separate quota for pull requests that are executed in
a forked repository.
The tests will be run in the forked repository until all PR review comments have
been handled, the tests pass and the PR is approved by a reviewer.
-->

@liangyepianzhou We should reduce the beforeMethod afterMethod, it will slow down the test, and waste the CI resources. The fix looks like moving some test from the beforeClass and afterClass to beforeMethod afterMethod. I think the right way is to make the test can work with beforeClass and afterClass, you can use beforeMethod and afterMethod to clean some metadata and data by using Pulsar Admin API.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c952f3c). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #17746   +/-   ##
=========================================
  Coverage          ?   70.92%
  Complexity        ?      437
=========================================
  Files             ?       26
  Lines             ?     2246
  Branches          ?      245
=========================================
  Hits              ?     1593
  Misses            ?      480
  Partials          ?      173

Flag
Coverage Δ

unittests
70.92% <0.00%> (?)

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


updating to allow custom settings

This allows per-field custom CSS. It keeps the basic defaults but writes anything the user enters after all the default CSS, so anything can be overridden. It uses a CSS parser to prepend the field ID/class information, so no one has to worry about adding all the specific crap - just very simple selectors.

Looks good to me, thanks!


tfa.image.rotate and tfa.image.gaussian_filter2d

Hi 👋🏼 ,
Since TFA has been deprecated, I wanted to ask if there is an equivalent implementation (functional) in Tensorflow or Keras ?
Thanks in advance

You have only the random rotation but not the rotation itself:
https://keras.io/api/layers/preprocessing_layers/image_augmentation/random_rotation/
Also in TF you have 90 degree rotation:
https://www.tensorflow.org/api_docs/python/tf/image/rot90
For gaussian filter your have:
https://keras.io/api/keras_cv/layers/augmentation/random_sharpness/
Or:
https://www.tensorflow.org/io/api_docs/python/tfio/experimental/filter/gaussian

Thannks @bhack 👍🏼


add to do list code

I  made the following changes

updated readme file
add todolist application

STATUS APPROVED
Hello @yuvenalmash ,
Your project is complete! There is nothing else to say other than... it's time to merge it 👍 👍 😄 👏
Congratulations! 🎉
Highlights

Professional Readme File ✔✔
Linters are setup correctly ✔✔
JavaScript best practices ✔✔

Cheers and happy coding 👏🏿👏🏿👏🏿

Thank you


Added sigin with google option and chnged style of signin and signup page

Describe the feature

Add ScreenShots
No response
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@trishanu-init assign me this under Vsoc24

@trishanu-init created pull request for the same and added screenshots there as well

can you merge this i will create new pull request for that buttons in navbar

can you give me points for this

yes i have already done that


core: allow custom roles

Role previously mandated that all role strings starts with "role/".
If the given string doesn't start with the prefix, Role prepends it.
This is incorrect, since custom roles have format "projects/X/roles/Y".
This PR makes Role a little smarter at prepending:
we only prepend if the given string contains no slash; this keeps the
previously-valid use valid.
Fixes #2687.

Cool. Could you link to an official reference for this role behavior?

@vchudnov-g I'm not sure what you mean by "behavior" here. Do you mean how it automatically prepends "roles/"?
If so, this is not a "standard behavior". The problem is Roles is trying too hard to be useful. If you run Roles.of("owner"), "owner" is not a valid role but "roles/owner" is, so Roles helpfully prepends.
However this makes custom roles impossible. Since custom role is in format "projects/X/roles/Y", calling Roles.of("projects/X/roles/Y") creates role "roles/projects/X/roles/Y". This PR makes it possible to refer to custom roles.
Am I understanding you correctly?

That explanation is helpful. My comment was about including a link to an official reference to what you explain about "roles/owner" vs "projects/X/roles/Y", just for documentation purposes.


Optimize gossip

Currently we do flood all blocks to all of our neighbors when a block is scheduled or skipped, on the main engine and the candidate engine. This could be optimized to eg only do that on the main engine and not while warp syncing.

Should be done in the same step as #35 and #597


Sistent site draft

Notes for Reviewers
"This PR represents the draft of the Sistent website. Please note that all the website's code is present inside the site folder.
Signed commits

[x] Yes, I signed my commits.

// @leecalcote

There's some great work in here, @coder12git. 🙂

While I understand that Yarn has its perks, and we are all learning, is it intentional to have site outside of the packages or apps?

Not to my knowledge, no. One Makefile to rule all components of the system sounds nice to me.

Oh, my word. What is going on with my today?

While I understand that Yarn has its perks, and we are all learning, is it intentional to have site outside of the packages or apps?

I only ask because we have a way to yarn ignore site in its building process.

@nebula-aac i agree, i will move this site folder inside apps with sistent-website as a folder name

@coder12git i added  draft label, you can remove when done


[sil-optimizer] Make compile time concatenation of constant strings work again

This guaranteed optimization got lost when a + operator for Strings was made a static method on type String.
Resolves SR-4348.

@swift-ci please smoke test


The point clouds are vertical after records as pcd files.

HI,
I am working with Autoware and I want to use this algorithm to prepare the pcd map for using. I met a problem. I tried to use pointcloud_to_pcd to record and export the pointclouds as pcd files. After that, I import the pcd file into autoware. However I found that the pointclouds map is vertical.

When I runing the algorithm, I saw the pointclouds are horizental in Rviz. Do you rotate the point clouds in the algorithm? I recorded the pointclouds from topic: laser_cloud_surround. What is the reason of this situation?

Yes, the points are rotated here [https://github.com/RobustFieldAutonomyLab/LeGO-LOAM/blob/744254789c91ebe4851b4678be2c9f97e2f56c27/LeGO-LOAM/src/featureAssociation.cpp#L487].

Thank you for your reply, can you give me some hint of how to modify it? It does not work if I only modify the x,y,z.

@xingtongli
You need to rer-write all the code that is related to point cloud computation. It's probably not that easy.


workaround: error when defectDojo integration is not set.

The following error occurs where config.defectDojo is not set:
Traceback (most recent call last):
File "/home/rapidast/rapidast.py", line 153, in
if defect_d and hasattr(scanner, "data_for_defect_dojo"):
NameError: name 'defect_d' is not defined
Workaround
Add the following config, regardless of whether or not defectDojo export is in use:
config:
  defectDojo:
    url: "http://localhost"

https://github.com/RedHatProductSecurity/rapidast/pull/88 will fix it.


[BUG] Rescue spell can't be cast by warlock past lvl 2 spell slots. v4.3.1

Describe the bug
When attempting to cast the Rescue spell as warlock with lvl 2 spell slots the spell gives you multiple options for the rescue locus. However, if you have a lvl 3 and above spell slot for warlock you are unable to cast the spell as it no longer offers you a choice of locus.
To Reproduce

Acquire the Rescue spell for Warlock.
Reach level 5.
Attempt to cast the Rescue spell with upgraded spell slots.
Observe spell no longer offers options for casting.

Expected behavior
I assume that the spell should still allow you to choose what locus you want to use and then let you cast it despite having a higher level spell slot.
Screenshots
lvl 2 spell slots

lvl 3 spell slots

Requirements Versions
Script Extender:
(v20)
Baldur's Gate 3 Mod Manager:
(<IP_ADDRESS>)
Mod Configuration Menu:
(<IP_ADDRESS>)
Load Order
LoadOrder.json

I know why this happens, I'll fix it in the future updates

Upcasting now works however, you when trying to target anyone it considers them an invalid target, regardless of if you upcast or not. I'm not sure if I'm missing something though.

I couldn't reproduce the issue. What spell doesn't work?

I will attempt to reproduce the issue and I will give a more detailed set of steps to reproduce in a new bug report.


fixes #406

What kind of change does this PR introduce?
Bug fix
What is the current behavior?
Graphql query fails if postgres is in recovery mode.
What is the new behavior?
Read-only queries will work even when postgres is in recovery mode. Note that mutations will still not work.
Additional context
See this comment for details about this fix.
Fixes #406

@imor could you pls add an entry to the changelog for read-only replica support so we don't forget about this one?


wrappers: remove wrappers dir, decouple from ci/codeql

-we want to decouple the wrappers directory from libdogecoin and let external maintainers maintain the bindings independently. the new de facto repos live in https://github.com/dogeorg
-consolidate test architectures switch statement in ci
-remove go from language analysis in codeql

added some additional removals in various files across the project and fixed some formatting issues.


[Bug]: Multiple checkbox widgets do not get selected using the Shift key

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I try to select multiple checkbox widgets using the shift key, they do not get selected.
Screen recording
Steps To Reproduce

Create a new app
Add multiple checkboxes
Using the shift key try selecting multiple checkboxes

Public Sample App
No response
Version
Cloud - 1.7.9

Closing this issue since we can't do shift click on the widget, It has to be done on the settings.


Stream execution develop

This resolved issues in code review.

I'd like to keep this branch as a reference.  I created another branch from up-to-dated develop branch which addresses the last a few issues (pull request #175).  Close this pull request.


Add IsDevelopment guard to Swagger UI in JSON transcoding docs

@captainsafia  We're doing all the Swagger UI and we should have it done next week.


[move-lang] use copy instead of rename for tempfiles

Motivation
rename(from, to) will fail when from and to are on separate filesystems. This is usually not an issue for macOS but might be for certain Linux distributions where the /tmp is in tmpfs and others in ext4 or sth.
The tempfile will be deleted when it goes out of scope (which is next line after the copy).
Test Plan
cargo xtest

(Discussion had offline but bringing this in for posterity)
We use rename because it is threadsafe. But it requires the /tmp dir to be in the same filesystem mount as the target file. We could do a copy+rename, where we first copy from the tmp to some random garbage in the target dir, and then rename from the random garbage to the target file. But that is probably overkill for now


(PUP-4744) Fixes reposdir split by whitespace

reposdir can be split by whitespace - http://yum.baseurl.org/gitweb?p=yum.git;a=blob;f=yum/config.py;h=97e5e3d3b063294a96227a67cce43a9655cd3aa6;hb=30c993c19b8b85f44e7cec436484862ab55a6961#l171

Puppet previously only split by commas
Now changed to a regex of [,|\s] which will catch whitespace and commas
Added new spec for this new regex

Ticket https://tickets.puppetlabs.com/browse/PUP-4744

:+1:

This could go against stable.

In reading the ticket, it sounds like several people are affected by this. :+1: to retargeting for stable (sorry @petems I should have noticed that sooner)

@joshcooper No problem, do you want me to change the target?

@petems yeah. You'll need to close this PR and open another. (I wish wish wish that github allowed changing target branches on open PRs. Sigh.)


Graph Runtime for Heterogeneous Execution

Problem
The current graph runtime assumes that all the op are on a single context (OpenCL, or ARM), in reality we usually need switch between devices, especially for last few layers in detector models, where it might be hard to get a GPU version of the detector operators (@Laurawly is working on one, but that is a different story). A better solution would be enable the graph runtime and builder to split the graph into multiple devices, and insert copy operator when necessary.
Steps of Changes

As a first step, let us support mixed host(CPU) and device code, which will solve most of the problems

We will need API discussion for multiple devices, the current graph runtime relies on a single device context, which do not handle this need well. This is a longer term problem and can be resolved in second phase.

Proposed API Changes

Allow operators to register FComputeFallback, which indicates the operator need to fallback to host cpu implementation
Have a rewrite pass that insert copy node after seeing the nodes that fallsback to host.
Enable an additional column in the deploy graph indicate the device placement plan.

Related Issues
https://github.com/dmlc/tvm/issues/1242

@zheng-da @reminisce Since they are working on subgraphs.

For the first step, is there a way we can detect that whether op is registered for current target in tvm? In this case, we don't need to add an extra attribute. If this is unrealistic, then in FComputeFallback, we may want to specify for what targets this operator needs to be fall back to cpu.

@tqchen, Someone already working on this? or any conflict with relay related works going on?
How about fusing graph nodes with different device placement, this has to avoid?
I am interested to take up this.

@tqchen It seems to me that the subgraph approach currently developed in MXNet can fit well for the heterogeneous execution.
https://cwiki.apache.org/confluence/display/MXNET/Unified+integration+with+external+acceleration+libraries
We might need to allow users to specify what device an operator should run.
What kind of target hardware are you thinking about? A machine with one type of accelerators? or with multiple types of accelerators (e.g., GPU, ASIC, FPGA)?

@tqchen @joyalbin @zheng-da I have actually started working on the step one. The current graph partitioning approach is based on the one developed in MXNet.

@zhiics this sounds awesome, do you want to provide more details, feel free to post discussions here

@tqchen The basic idea is to partition a computational graph into a bunch of subgraphs according to some rules. Each subgraph, therefore, contains a number of nodes. These subgraphs will be compiled into modules individually, potentially each one could be for a specific device. In the end, each module will be executed using tvm runtime. Any suggestions or comments will be appreciated.

I see, I was thinking a more basic way, to extend the runtime a bit with a context field, and insert cross device copy, so the final evaluation still runs on a single graph, but each entry can use different context

If the purpose is for executing different nodes on different devices, I think @tqchen 's method is more suitable. It would be similar to using context group in simple_bind in MXNet.
The subgraph method developed in MXNet is mostly for executing the subgraphs of a nnvm graph using different accelerators, such as TVM, MKLDNN, and TRT. Each accelerator can take an nnvm subgraph and applies operator fusion, layout conversion accordingly.

@zhiics building individual modules lead to many outputs which complicates the deployment. inserting a context field make it more transparent to user.

Really interesting topic! I have once developed similar mechanism in caffe for an accelerator (could be one of the fast ones in this plant currently :) ).
Personally, I think it would be cool if there were generic graph for every backend device - especially when you are talking about Heterogeneous Execution. Basically, accelerator drivers only handle generic neural networks interfaces - one unified optimized graph of TVM could bring in difficulty when enabling enormous accelerators.
I don't have much knowledge of TVM, neither not sure about whether TVM aims to enable different backends on one devices, but if that is the case, a generic graph solution sound cool maybe - you can perform different optimization for different backend device inside the graph. Anyway, basic solution requires less effort at this time... (ignore me, i am just balabala...)

I believe @tqchen's solution should be the most suitable and (also the most general) one. Do program slicing and synchronize only necessary data is proven promising.
Suppose we have a higher-level IR and its interpreter (probably with JIT), and this IR also supports program slicing by reusing some simple type inference system. Is it worth discussing the advantages and disadvantages between synchronizing in high-level and low-level?
Thanks :)

Thank you all for sharing. We (@yidawang @yzhliu ) also thought about @tqchen 's solution. It's a very nice and generic approach. But we think it is also helpful to keep another hierarchy for optimization and deployment on different devices. I agree that dividing into modules may lead to multiple outputs sometimes. But we could provide some wrapper to make the process of module build and deployment transparent to user.  We may currently still stick to our method. However, it would be very interesting to see the other implementation.

I think this task has two parts. The first task is to compile part of a graph for a particular device. I think the simplest way is to partition the graph into subgraphs so that we can reuse everything TVM has right now to compile individual subgraphs. Without partitioning, graph-level scheduling search can be difficult.
The second task is to execute the graph. I think both solutions work. @tqchen's suggestion might be simpler for the TVM runtime. All we need to do is to stitch subgraphs back to the whole graph while inserting cross device copy at the boundary of subgraphs.

I agree with @zheng-da thought on having two mindshares on this:

Runtime: Think of what do we need for runtime(this case like bytecode), we want a minimum design that can be deployed
Compiler: Think of what needs to be done for optimizations, this case might benefit a bit handling of hierarchy etc.

I am advocating for the simple context-based method on the runtime side, regardless of how compiler can be implemented. The reason is that it keeps the current interface minimum and clean an the users.

@junrushao1994 still in confidential. Also, caffe is so lightweighted that we simply feeds abstracted networks data into underlying device driver without any optimization that TVM does. The driver itself handles optimization, compiling, running.
My point is, when we are talking about something like interface or heterogeneous, the abstracted description passed from high level to low level should be self-contained, which means that when you passing a so-called subgraph (or context keyword you like), the subgraph information should be no more no less - it just like another network (regardless the purpose of it). This is just like you are running a single graph context on one device. The interface is so important that it will impact all backend drivers. It will be pretty weird for a device driver knows that it has a big graph, but it should only execute part of the graph. You guys can look into Android Neural Networks API (check partitionTheWork) which aims only to abstract backend devices.
So, i vote for @zhiics (though i have contributed nothing to tvm...)

@jackwish My understanding for this problem is that we might want to keep abstract graph for compiler so that different optimizations can be done easily for heterogeneous devices, but not necessarily for runtime. Since we might not want to execute these subgraphs individually, a wrapper to pack all subgraphs and data flow information is what we want for runtime. I think both solutions will work, but change the runtime interface might be a cleaner way to implement.

@jackwish I am not sure if my understanding is correct. Are you referring to program slicing, which is the solution in TensorFlow Swift for synchronizing data between different devices?
For example, the compiler knows the entire graph, which is heterogeneous computation on many devices. It simply does something like type inference or heuristics to figure out which part of the program should reside on which device, and dispatch this part to the corresponding device.

@kevinthesun Thanks for your explanation. I know exactly that TVM aims to provide generic optimization regardless the backend device, and I was very excited when I got to know this. However, when it come to practice, especially in context of this heterogeneous execution issue, the primitives of the backend is mostly inconsistent across different devices. What I am trying to say is that, yes we can optimize IRs that finally feed to LLVM-like backend which happily takes some IR; but when it comes to accelerators, a driver is usually provided by the accelerator vendor - the primitives of the driver could be very high level such as Conv2D. I am new to TVM, but intuitionally, the IR (output of the optimization) could be of lower level than Conv2D like primitives. If that's going to be the case, the graph which is composed by operators and tensors should feed to accelerator driver directly.
@junrushao1994 If we take neural networks as program, yes I am referring to something like program slicing.

I will use the TVM stack to make my point clear. Basically we have different categories of backend devices (for intuition only):

high level primitives: cuDNN, mklDNN like drivers, and accelerators drivers
mid-level primitives: CUDA, OpenCL like libraries
low-level primitives: LLVM with x86, ARM as devices
I guess the generic optimization is able to handle mid-level and low-level primitives. But for high level, the drivers knows much better how to optimize or schedule.

However, this issue is really depends on what TVM is intended to be. If TVM is going to support only one (or very limited) non-CPU backend devices, I think a context based approach should be cool. Otherwise, if TVM aims to schedule ONE model across different devices, I guess the subgraph approach? The answer is really deeply tighten with TVM's future. There is no better or worse, there will only be the more suitable. If the decision is that TVM is going to supported only limited devices for a specific network running routine, @tqchen 's approach may come in charge. And, to be honest, though I am more likely to be interested in subgraph approach, I am wondering schedule networks across devices is really a must in practice - first we don't have so many devices on one machine, second the memcpy could be heavy.
Anyway, thanks you guys for be so patient as I really don't know much about TVM while still talking too much :)

To be clear, the simple context-based approach works for multiple devices mentioned by @jackwish , including cases that do not fall back to CPU.  What we can do is to annotate the context for each node.
The only "drawback" of not introducing subgraph abstraction is that subfunction might be helpful to do things with recursion(which we do not consider). Note that even if we introduce subgraph abstraction, from high level point of view, the master graph that executes different subgraph still have to handle different device context(which falls back to the context-based approach)

@tqchen Yes, the execution can be done on different devices for both solution. Device drivers don't need to know the context semantic as it receives a graph and execute it in the subgraph approach.
A subgraph approach is not necessarily to have a recursion behavior. Say, we get a graph, we partition it into several subgraphs - the module, which holds subgraphs, is simply an executor. The master graph and subgraphs are described in the same way, and master graph doesn't knows there are many subgraphs. The executor has the executing context - there could be one subgraph or multiple subgraphs. For each subgraph, executor feeds to device driver directly.
I am talking about how people stack software in a system. For both solutions, a layer that translates TVM's representation to driver representation is required (assume that the driver has generic NN interface that targets many frameworks and runtime). Context approach needs to translate context to something like subgraph and feeds the subgraph; subgraph approach feeds subgraph. See, the question is where you put the representation translation - delay it before drivers (context), or at early stage (subgraph). Both are cool solutions :)

Good discussion and contributions are always welcomed @jackwish

Thanks for all inputs. After more discussions, we think it would be more convenient for runtime to take a single graph because 1) there are usually not that many devices on a single machine in practice, 2) compiling each subgraph and executing them individually would require the construction of the subgraphs in a way to keep their correct execution order. It does complicate the design and runtime.
We've started a discussion thread with some concerns at https://discuss.tvm.ai/t/graph-partitioning/504
Thank you very much for any comment and suggestion.

In the context of TVMContext passed to graph runtime and with heterogeneous we are now passing a vector of TVMContext.
As I understand TVMContext is tightly coupled with compiled function.
Ref #1695

The vector of TVMContext is used for checking storage device type in the given TVMContext list or not.
There would be only one version of tvm_op in the module (host or device) for each operation.

Hence I don't see why we need to pass TVMContext to graph runtime if storage already mapped to device_id.
Am I missing any thing here ?
Or
Is this vector of TVMContext is a backup plan to support a future version of Heterogeneous execution where

Compiled module holds multiple version of tvm_op for each target device.
Dynamically user can pass TVMContext list to prioritize the device.

The context is passed to map the virtual device index(which is already in the plan) to the specific context. This can still be useful say we compiled a GPU module and we can map it to gpu(0) or gpu(1) during runtime

closed by https://github.com/dmlc/tvm/pull/1695


Nimble Markets #31

@phuclh After talking to @joshtorres about this, can you go through the Market model, factory, nova resource, and anywhere else to remove the references to these fields?

@phuclh After talking to @joshtorres about this, can you go through the Market model, factory, nova resource, and anywhere else to remove the references to these fields?

Removed them. I also removed some references in Market Nova resource at: https://github.com/tipoff/locations/commit/aaac37ef4b378aa856862ba435cc64fd2955f8b3


Locale config option with callable as array syntax

Looking at the newest release 5.3.0.
https://github.com/optimistdigital/nova-menu-builder/blob/master/src/MenuBuilder.php#L42
is_array is checked before is_callable, which was not the case on earlier version as you can see.
So if i have configured my locales as callable in static method notation:
'locales' => [\App\Facades\Locale::class, 'getSupportedLocaleArray'],
this doesnt work anymore and kinda breaks. I have to change it to a method call as string:
'locales' => \App\Facades\Locale::class . '::getSupportedLocaleArray',

@raraworks if locales is an array, then it should contain locales. I cannot understand how it could work previously.

:)
In short, if configuration allows passing a callable as configuration value, then is_callable check should be called before is_array check...
Should be an easy fix?

@raraworks if locales is an array, then it should contain locales. I cannot understand how it could work previously.
As you can see here, nothing changed when locales is an array.
P.S.: verified and I can confirm the bug, since is_callable should be called before is_array.

:)
In short, if configuration allows passing a callable as configuration value, then is_callable check should be called before is_array check...
Should be an easy fix?

Heya! I reordered the if statements so that the is_callable is first. It should now work as expected. :) The fix was release in version 5.3.1. Thanks again and good luck!

Heya! I reordered the if statements so that the is_callable is first. It should now work as expected. :) The fix was release in version 5.3.1. Thanks again and good luck!

aitäh! :)


Build(deps-dev): Bump @types/lodash from 4.14.185 to 4.14.186 (#4134)

Pull request opened by the merge tool on behalf of #4134

Merged in ba820decf90e2cb47811a92d726a71fc5d1f4332


The whole window is glitching

I know this is hard to reproduce and even hard to report but for the last, I don't know, 2-3 weeks or so, Goofy's window had been randomly glitching. The entire window.
(I don't think it's something with my computer as it's only Goofy doing it)

Could you try explaining what "glitching" means? As it is, it's nigh impossible to understand what you mean.

Right. I'll try.
First of all on closer observation it doesn't happen to the contact list, only the rest of the window.
Second... here's a screenshot of something completely different that illustrates what I mean by glitching: http://i.imgur.com/26qGQwr.jpg
That's all I can say. I know, not a lot to go on.

The same thing happens to me frequently :(

Yikes ... could be a problem with Electron?  I've upgraded Electron to 2.0.x recently (since v3.3.3) - if you could reopen the issue if you still experience it and we can look into it.


ui: improve sidebar panels

Multiple improvements to the side bar:

introduced a panel component to easily add
sidebar blocks
introduced a generic collapse component
removed specific code from object-detail component
removed unused CSS classes

Also upgrade to Vuejs 2.5.16

@eonpatapon I really like what you did with this PR, do you think that you will have time to work on it soon ?

@safchain I was having issues with selenium tests. I think that at some point in the capture test selenium wants to click on a tab but there is a notification that is hidding this tab at the same time

@eonpatapon ok I will have a look for another test with selenium we click on the notification to close it before clicking the tab, I'll try to adapt to this test.

rerun tests

@safchain removed changes done to vendor.json. I don't know why but it happens all the time without changing deps

@eonpatapon maybe you govendor binary is not up-to-date

run skydive-functional-tests-backend-orientdb

@eonpatapon thx ! that's really nicer


feature(text-field): Add helper-text subcomponent for text-field

Codecov Report

Merging #24 into master will increase coverage by 0.28%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   98.01%   98.29%   +0.28%
==========================================
  Files          14       16       +2
  Lines         352      410      +58
  Branches       22       28       +6
==========================================
+ Hits          345      403      +58
  Misses          7        7

Impacted Files
Coverage Δ

packages/text-field/helper-text/index.js
100% <100%> (ø)

packages/text-field/icon/index.js
100% <100%> (ø)

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 a0bc0c0...b5a0e26. Read the comment docs.


Add TapFDSource test

[X] pump frames between taps instead of using veth hack for VM network test
[X] add initial version of TapFDSource test
[x] fix container netns cleanup upon TestTapFDSource failure
[x] in TestTapFDSource, use random pod id to avoid cascading test case failures
[x] check data returned by AddFDs() / GetFDs()
[x] verify network teardown
[x] verify fixing bad CNI info

This change is

Reviewed 6 of 9 files at r1, 4 of 4 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 9 of 9 files at r1, 4 of 4 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Enhancement Request: Multiline Todos

I see you are using an INPUT element for the description of the todo.  I think it would be helpful to allow them to be multiline.  You could use a TEXTAREA or maybe on save of the description, copy the contents to a DIV instead of leaving a TEXTAREA field.

Maybe a markdown description, and then a modal to show the to-do item? If so I can take this =)

Thanks for advice. It's what i am thinking about. I'm finding the best way to control the textarea's height automatically, or another good way to organize multiline description.
I'll do it as soon as I can. 😢

No worries.  I briefly looked at textareas and their height but there doesn't seem to be a great way for it to automatically adjust, sadly.

1.6.0 is comming now and this feature has been implemented.
You can press shift+enter when the cursor is focused on the input to expand the note panel, though it support text only yet.
I know this function is too imperceptible to find out so i'm thinking about how to make it more convenient.

This is fantastic work!  Great job.  You may consider not copying the contents of the input to the textarea when shift+enter is hit.  I see myself using the main part as a header and then adding notes below so it doesn't make sense to copy the contents when shift+enter is hit.  But you may have had other thoughts on its use.


Update SE-0409 to reflect change in the expected Swift 6 behavior

Update the proposal to reflect the language steering group decision to push the previously described Swift 6 mode to a future language mode. Also identify the support for @usableFromInline on imports as not yet implemented.

Thank you @rjmccall for the improvements!


contracts/eoslib/db.hpp does not plumb scope

scope is not plumbed through all the interfaces though the underlying primary_index lookups support this functionality.
Front and Back missing scope
I have included a PR for the fix #1320

I have only fixed it in dawn-2x but I can just as easily open this PR against master too.

Fixed via #1320


Advice for depending on the CML starting tomorrow

I've stared at this repo for a while now, and I think I've just about got the gist of what belongs to the "old" and to the "new" versions of CML.
If I were to start a project tomorrow depending on the CML, would you advise depending on ./rust and migrating in due time,
or going straight to using the new crates?
I appreciate the answer is likely "ummm. Depends on your use case" and it more an invitation to any signposting on:

what the architecture roadmap is
something akin to ADRs that motivated these
how painful a migration would be

This would be super helpful.
I spoke a little to txpipe people, trying to ascertain a comparison between aspects of pallas and CML,
and whether that would be a more stable dependency for a project-starting-tomorrow.
Many thanks

This Q is an abuse of the gh issues, and you may close it as you see fit - no hard feelings.

The new code isn't in any package at the moment (except for CIP25), so it may be a bit tedious to use.
If your goal is to write a project that has its own custom binary format (new tx metadata standard, new Plutus datum format, etc.) and you'd like to code generate it, the CML option might work best so that it's all based on the same primitives (same way the cip25 crate reuses the CML metadatum definitions)
If that's not something you need, then it's up to you which you pick. If CML sounds like a cool project to you in general, you're more than welcome to help contribute as we're near the end of generating the new architecture and will soon have to move into documentation and adding utility functions

Have I understood this correctly

Yes, sounds right

But its a bit strange to now have a process where there are two distinct sets of libs providing serialization: CML and pallas*.

CML has been a project that has spanned multiple years since building this whole codegen pipeline is time-consuming as you can imagine. When Pallas was created, CML didn't support native Rust output (only WASM), which made the Rust interface ugly to work with. Pallas was created to bypass this old limitation of CML which doesn't exist with the new system anymore


Is there a plan for supporting client caching?

The most important feature in redis 6 seems to be client caching. Is there a plan for supporting client caching?

Thanks for bringing this up. It's not on the roadmap for the moment but I think it's nice to have. Any PRs and discussions are welcome.

We are wanting to use server assisted client side caching as well.

Any updates on this topic? Interested to help, but not too familiar with libs internals.

Seems like supporting client-side caching is not likely for the near future https://github.com/luin/ioredis/pull/1504#issuecomment-1030865630

Before the implementation of the resp3 protocol handler will be started, it would be useful to spec out an api and requirements for client side invalidation.
According to https://redis.io/topics/client-side-caching#two-connections-mode RESP2 would also already work with a PUB/SUB approach.
So anybody with a use-case or free time would be welcome to tackle that.


token based licence?

Is the licence token based (same way as user-based licensing) and can the office be fully used on an offline computer for as long as 30 days? Or can the device operate offline with full office as long it is once verified?

Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: e52eb9c1-1755-bec5-4ad8-6df59779e041
Version Independent ID: 10e4cf5d-ea61-806e-f226-1e14265eabe1
Content: Device-based licensing for Microsoft 365 Apps for enterprise
Content Source: DeployOffice/device-based-licensing.md
Service: o365-proplus-itpro
GitHub Login: @jwhit-MSFT
Microsoft Alias: jwhit

Is the licence token based (same way as user-based licensing) and can the office be fully used on an offline computer for as long as 30 days? Or can the device operate offline with full office as long it is once verified?
Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: e52eb9c1-1755-bec5-4ad8-6df59779e041
Version Independent ID: 10e4cf5d-ea61-806e-f226-1e14265eabe1
Content: Device-based licensing for Microsoft 365 Apps for enterprise
Content Source: DeployOffice/device-based-licensing.md
Service: o365-proplus-itpro
GitHub Login: @jwhit-MSFT
Microsoft Alias: jwhit

The license with Device Based Licensing will activate the device itself and does not tie to the user. If a Users chooses to sign in to gain access to their documents they can but it's not required. The device will need connectivity to validate at least once every 30 days similar to User based licensing.   If you want long term offline state I would suggest Office 2019.


This doesn't seem to enable the docker container monitoring

It says the default is newrelic_disable_docker: no ...but I don't think this actually does the step to enable Docker monitoring in the first place:
https://docs.newrelic.com/docs/servers/new-relic-servers-linux/installation-configuration/enabling-new-relic-servers-docker
I'm not getting any Docker monitoring after using this role

Hi, actually haven't used newrelic's docker monitoring myself yet... you probably need to do more than just using this role as it only installs the daemon.
This role has a template for nrsysmond.cfg and I added some variables to be configured as a starting point (can definitely be extended, help welcome) which includes the option:
#
# Option : disable_docker
# Type   : boolean
# Value  : Set to true to disable Docker container statistics gathering.
# Default: false
#
disable_docker={{ newrelic_disable_docker | to_nice_json }}

so till now that's only what it does in account to docker monitoring.
If you figure out what needs to be done we can add it to this role or make a ansible-newrelic-docker...


Look into automatic eligibility checker that uses PHP

https://github.com/mhollander/Expungement-Generator.
Is this possible for us to do?

http://m.asos.com/mt/us.asos.com/Monki-Checked-Shirt/16zfrd/?iid=5571339&cid=11318&sh=0&pge=2&pgesize=50&sort=3&clr=Navy+multi&totalstyles=443&gridsize=3&mporgp=L01vbmtpL01vbmtpLUNoZWNrZWQtU2hpcnQvUHJvZC8.&un_jtt_v_frompage=2

You can  test the PA the site here:   You can log in using<EMAIL_ADDRESS> The password is test1234.

Here is  sample criminal record
CPReport (54).pdf

CPReport (53) (1).pdf

Ok looking at those examples, what text needs to be extracted from them?

So these attached examples are from PA. I will attache the DC ones I showed you last night. I created a repo. Ill paste the link also.

https://github.com/codefordc/Expungement_generatordc

Court Form 2 (3).pdf
Court Form 1 (3).pdf
MPD Form (3).pdf

Above are the DC forms. So the variables to extract are # 1)disposition date 2) crime/charge 3)charge disposition/disposition  And optionally: 1) name 2) PDID

@mchelen

@brianek Are these PDFs (Court Form 2 (3).pdf, Court Form 1 (3).pdf, MPD Form (3).pdf) examples of forms that we need to pull information from, or put information into? One issue with these is they all appear to be scanned images, so it's not possible to directly extract text from them.

Can't we ocr them? I can also try to convert to word. This is the documents
that will have info taken (pull)from them. The info could then go to a html
page or be provided in a doc. Or pdf.
On Oct 15, 2015 3:04 PM, "Mike Chelen"<EMAIL_ADDRESS>wrote:

@brianek https://github.com/brianek Are these PDFs (Court Form 2
(3).pdf, Court Form 1 (3).pdf, MPD Form (3).pdf) examples of forms that we
need to pull information from, or put information into? One issue with
these is they all appear to be scanned images, so it's not possible to
directly extract text from them.
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/issues/165#issuecomment-148490002
.

OCR is possible, but can be much more difficult depending on what info
needs to be extracted. The first step then would be trying some OCR utils
and seeing how the output looks.
It may be worth petitioning DC courts to obtain better formatted PDF files,
but idk how difficult that would be.
On Oct 15, 2015 16:38, "brianek"<EMAIL_ADDRESS>wrote:

Can't we ocr them? I can also try to convert to word. This is the documents
that will have info taken (pull)from them. The info could then go to a html
page or be provided in a doc. Or pdf.
On Oct 15, 2015 3:04 PM, "Mike Chelen"<EMAIL_ADDRESS>wrote:

@brianek https://github.com/brianek Are these PDFs (Court Form 2
(3).pdf, Court Form 1 (3).pdf, MPD Form (3).pdf) examples of forms that
we
need to pull information from, or put information into? One issue with
these is they all appear to be scanned images, so it's not possible to
directly extract text from them.
—
Reply to this email directly or view it on GitHub
<
https://github.com/codefordc/clean-slate/issues/165#issuecomment-148490002
.

—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/issues/165#issuecomment-148513589
.

They give out word documents. Someone scanned it to give to me. I think I
know a good ocr
On Oct 15, 2015 7:49 PM, "Mike Chelen"<EMAIL_ADDRESS>wrote:

OCR is possible, but can be much more difficult depending on what info
needs to be extracted. The first step then would be trying some OCR utils
and seeing how the output looks.
It may be worth petitioning DC courts to obtain better formatted PDF files,
but idk how difficult that would be.
On Oct 15, 2015 16:38, "brianek"<EMAIL_ADDRESS>wrote:

Can't we ocr them? I can also try to convert to word. This is the
documents
that will have info taken (pull)from them. The info could then go to a
html
page or be provided in a doc. Or pdf.
On Oct 15, 2015 3:04 PM, "Mike Chelen"<EMAIL_ADDRESS>wrote:

@brianek https://github.com/brianek Are these PDFs (Court Form 2
(3).pdf, Court Form 1 (3).pdf, MPD Form (3).pdf) examples of forms that
we
need to pull information from, or put information into? One issue with
these is they all appear to be scanned images, so it's not possible to
directly extract text from them.
—
Reply to this email directly or view it on GitHub
<

https://github.com/codefordc/clean-slate/issues/165#issuecomment-148490002

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/codefordc/clean-slate/issues/165#issuecomment-148513589
.

—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/issues/165#issuecomment-148554012
.

Can we get the Word documents?

Hey Mike: Sorry for the late reply. So here's what I was able to do. I have a program that convert a pdf into html web page or  into word. So I converted them to word and I have added them to this message.
Court Form 1 (4).docx
MPD Form (4).docx

@brianek Hey no worries. You can probably see the formatting of these .docx files are in pretty bad shape though. Ideally we would get the original files before they were scanned. Is that possible?

Yes. We could get the printed copies and scan them in a form that is easiest for a computer to read. Or we can ask for them in a word format.

On Oct 20, 2015, at 12:19 PM, Mike Chelen<EMAIL_ADDRESS>wrote:
@brianek https://github.com/brianek Hey no worries. You can probably see the formatting of these .docx files are in pretty bad shape though. Ideally we would get the original files before they were scanned. Is that possible?
—
Reply to this email directly or view it on GitHub https://github.com/codefordc/clean-slate/issues/165#issuecomment-149620954.

@brianek Yeah if at all possible to get the Word files before they are printed, that would be ideal.

whats the second best option if that is not possible?
On Wed, Oct 21, 2015 at 5:19 PM, Mike Chelen<EMAIL_ADDRESS>wrote:

@brianek https://github.com/brianek Yeah if at all possible to get the
Word files before they are printed, that would be ideal.
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/issues/165#issuecomment-150027313
.

@mchelen here is a message I got from the Expungement generator from Phily. I asked him what line of code reads the criminal records.
To briefly answer your question, the regular expressions from line 100-180 is the code that defines how to read the docket.  The horrible function readArrestRecord is the part that applies the regular expressions and stores everything off into variables.  The function basically takes a line by line look at an arrest record and brute force tries to match that line to one of the aforementioned regular expressions.
readArrestRecord is called in expunge.php, which first calls pdftotext on the pdf docket sheet, which reads the pdf into a text file.  The text file is then fed into readArrestRecord.
Let me know if that helps.
The big challenge that you are going to have is how to OCR the scanned documents.  That is beyond my expertise, but I understand that this is a problem that has been solved by other people.

Yeah the biggest issue is getting the best quality data to start with. What we want is ideally the original document files, or whatever is closest to those files. Each time the files are converted/printed/scanned they will become worse quality.


Bug in TOGGLE CLASSIC GROUP

Toggling Classic Group using the button (TOGGLE CLASSIC GROUP)  only works if the groups above classic are collapsed.
Scenario
Rock group is expanded
Action
Click on  TOGGLE CLASSIC GROUP button.
Expectation
Classic group is expanded.
Result
Classic groups is not expanded, and there is a weird animation (Slide in from bottom)on the group below classic (salsa).

toggle group accepts flat position so if the group above are expanded then you need to compute the flat position before adding it to toggle group.
for flat position read this https://robots.thoughtbot.com/introducing-expandablerecyclerview


array_merge of $options in Database::__construct() loses keys.

According to http://php.net/manual/en/function.array-merge.php, the keys that are being set in https://github.com/FaaPz/Slim-PDO/blob/master/src/PDO/Database.php under the __construct function, are being renumbered, so they are being passed to the parent constructor using the incorrect values.
I'd suggest using "$options += array(default options)" statement instead, which preserves the numeric keys, while allowing what I believe is the intended behaviour here.

 Just released v1.9.2 (f557397c86ab68c379e533a7d1031858f4d88bdd) with the fix included... :grin:

Hi @FaaPz !
First of all thanks for this usefull library ;-)
Anyway, construct still loses given $options, indeed merge on line 31 should be:
$options = $options + $this->getDefaultOptions();
Am I right?


Is Pico still active in terms of future development/enhancement?

Wondering if this project is still in active maintainance including plans for future releases before I would recommend it for a client project. Is there any official information on the current state of the project?

It is. Pico has simplicity built into its core concept, less code causes less issues, requiring fewer updates. The latest v2.1.4 is from August and there's no newer release because we haven't had any new issues since. For plans about Pico 3.0 see #317 and #535. Help is appreciated :heart:


Helm v3.7.0 changed many commands

Current documentation for pushing a Helm chart to ACR is outdated with the release of Helm 3.7.0. For example, helm chart save has been replaced with helm package, and helm chart push with helm push. Please take a look at the release notes for further details.
Using helm chart save results in the following error:
Error: unknown command "chart" for "helm"
Run 'helm --help' for usage.

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

ID: 59c4be11-ec02-7160-e65b-2218bfd70f68
Version Independent ID: 19e08339-390f-ccfa-885c-2643722bd186
Content: Store Helm charts - Azure Container Registry
Content Source: articles/container-registry/container-registry-helm-repos.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@lynshi - Thanks very much for raising this issue and noting the differences in Helm 3.7. We'll update the article soon.

Happy to propose a PR.. if this hasnt been actioned already. @dlepow

@ahmedig Thanks a lot :)


 Thanks.
On 10 August 2017 at 17:24, Maciej Chałapuk<EMAIL_ADDRESS>wrote:

@muroc approved this pull request.
In README.md
https://github.com/quedexnet/java-market-maker/pull/1#discussion_r132501374
:

+* places option orders priced according to Black 76' model,
+* has configurable risk management - stops quoting one side of the order book when delta or vega limit exceeded,
+* enables risk monitoring based on greeks (delta, vega, gamma, theta), per position and in total,
+* cancels all orders when going down or on error.

+## Disclaimer
+
+This document and the code presented in this repository does not constitute any investment advice. By running it, you
+are not guaranteed to earn any bitcoins (rather the opposite).
+
+## Contributing Guide
+
+Default channel for submitting questions regarding the API is [opening new issues][new-issue].
+In cases when information disclosure is not possible, you can contact us at<EMAIL_ADDRESS>+
+In case you need to add a feature to the API, please [submit an issue][new-issue]
I think we may want to remove this sentence.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/quedexnet/java-market-maker/pull/1#pullrequestreview-55589217,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK29JeJfu3kPwCOiCDI6ZqQlEc3dsHB_ks5sWzzOgaJpZM4Ozp0c
.


wings-feathers and FeathersQuery as New Tools

Added wings-feathers (Framework Agnostic) and FeathersQuery (Utilities) as New Tools
Summary
(If you have not already please refer to the contributing guideline as described
here)

[ ] Tell us about the problem your pull request is solving.
[ ] Are there any open issues that are related to this?
[ ] Is this PR dependent on PRs in other repos?

If so, please mention them to keep the conversations linked together.
Other Information
If there's anything else that's important and relevant to your pull
request, mention that information here. This could include
benchmarks, or other information.
Your PR will be reviewed by a core team member and they will work with you to get your changes merged in a timely manner. If merged your PR will automatically be added to the changelog in the next release.
If your changes involve documentation updates please mention that and link the appropriate PR in feathers-docs.
Thanks for contributing to Feathers! :heart:

Thank you!


how to contribute section

add how to contribute to the dzcode.io section.

content
UI design
UI code

how to contribute content:

Chose a project that you like.
Clone the repository.
Start coding!.

UI design already provided in previous issues
UI Code in the latest pull request


add info to use "yarn install" to the readme

Warnings

:warning:

Please include a Pivotal story at the beginning of the PR title (see below).

:warning:

Please include a description of your PR changes.

Example of PR titles that include pivotal stories:

single story: [#123456] my PR title
multiple stories: [#123456,#123457,#123458] my PR title

  Generated by :no_entry_sign: dangerJS


1.19.2 builds are incompatible with Quilt

https://cdn.discordapp.com/attachments/1181233531075170437/1181233645474824193/image.png?ex=6580508f&is=656ddb8f&hm=b3034e8b11fab0934a8d9a83cd7a662e6c059d620bf3ff13c5138380610433b5&
Here's a screenshot

crash-2023-12-04_10.59.02-client.txt
latest(13).log
crash-2023-12-04_<IP_ADDRESS>5-quilt_loader.txt
Crash reports and logs from further investigation into this. Posting for reference.

I am having the same problems with this mod showing up in my crash logs on fabric
crash-2024-06-19_12.54.17-client.txt


Bug: Error on meta fields content app

From https://github.com/patrickdemooij9/uSeoToolkit.Umbraco/discussions/12:

If you press the "Edit" button, make some edits, then switch to the "Content" app, then back to "SEO" app, the fields are all blank - and you can't press the "Finish" button. There is a JS error: Cannot read properties of undefined (reading 'alias').

Fixed by https://github.com/patrickdemooij9/uSeoToolkit.Umbraco/pull/30


JAVA_OPTIONS not merged when using the vertx generator

I'm not sure if it's the desired behavior or a bug...
Description
When the user uses a deployment.yml setting JAVA_OPTIONS, the value is overridden by the generator (
https://github.com/fabric8io/fabric8-maven-plugin/blob/master/generator/vertx/src/main/java/io/fabric8/maven/generator/vertx/VertxGenerator.java#L59)
Anything that needs to be done to "append" both ?
Info

f-m-p version : 3.5.1
Maven version (mvn -v) : 3.5.0
Kubernetes / OpenShift setup and version : v1.5.1+7b451fc, kubernetes v1.5.2+43a9be4 (minishift)

Definitely this is not the desired behavior. Similar issues have been raised in #957 and #937.
There's not a default way to "append" to a env variable in f-m-p. Usually env-vars having multiple "declarations" are subject to a sort of "priority" and only one of them is used. In this case it seems the "priority" for JAVA_OPTIONS is not what a user expects.
I think we can default to an "append" mode for some environment variables like JAVA_OPTIONS, and this can be configurable. Wdyt @rhuss ?

@cescoffier I'm trying to investigate on the current behavior to understand what the plugin should do. I've added a IT test [here])https://github.com/nicolaferraro/fabric8-maven-plugin/blob/19b8f8985e671edcb47f0bfbdecf3e9a95609fde/it/src/it/vertx/expected/kubernetes.yml#L45-L46), but what I see is:

The JAVA_OPTIONS created by the generator are put in the Docker image
The JAVA_OPTIONS in the resource fragment are put in the kubernetes resources

At the end, the first JAVA_OPTIONS (-Dvertx.cacheDirBase=/tmp) are used during the build (S2I or Docker), while the one in deployment.yml is used at runtime in the Pod, and it overrides the one in the docker image (not vice-versa).
Is this correct?

Right now I see the opposite. The options from the generator are used and not the one from the deployment.yml.

Interesting... can you share your project, so I can check what's happening?

Hi @nicolaferraro
Here is a reproducer: https://github.com/openshiftio-vertx-boosters/vertx-http-booster/tree/log4j2
(notice the branch log4j2)
This project uses the Vert.x generator and has a deployment.yml setting JAVA_OPTIONS. When launched, only the options provided by the generator are used:
exec java -Dvertx.cacheDirBase=/tmp -javaagent:/opt/jolokia/jolokia.jar=config=/opt/jolokia/etc/jolokia.properties -cp . -jar /deployments/http-vertx-9-SNAPSHOT.jar

To reproduce the issue:
$ git clone https://github.com/openshiftio-vertx-boosters/vertx-http-booster.git
$ git checkout log4j2
$ mvn fabric8:deploy -Popenshift

Definitely it's the fabric8.mode that changes the behavior. You're using openshift while my example is using kubernetes. There's a OpenShiftS2IEnricher that overrides what's set in the fragment with what's required by the generator.
We need an alternative way to set these env variables, that is consistent with what it's done in kubernetes mode.

I've found a way to uniform the behavior of kubernetes and openshift mode for environment variables.
Basically, we can safely remove the OpenShiftS2IEnricher to avoid polluting Openshift resources with environment variables from the Generator. We can inject those environment variables during binary build.
I've changed the plugin code to inject a .s2i/environment file into the tar binary. This file contains all environment variables that we want to set in the build container and in the final docker image, as explained here.
This way we have a consistent behavior between Kubernetes and Openshift: in both cases, env variables defined at generator level will be encoded in the docker image only, without changing Openshift resources.
Users (through fragments and xml configuration) will be free to set the same env variables at Openshift/Kubernetes level, overriding the values encoded in the docker image.
This does not add a "append" mode for that JAVA_OPTIONS property, but I think it is a lot better than not being able to override it.
@jstrachan do you think the approach is ok?
It requires a small change to the docker-maven-plugin. Also @rhuss ?

I am not sure if this fix is already in the current version of fmp but nevertheless I found a workaround by setting the JAVA_TOOL_OPTIONS env variable which is beeing picked up by the jvm on startup. That way you can extend the JAVA_OPTIONS passed via command line.

@MarWestermann, great. I was looking for a workaround like that but didn't find it. The issue is not fixed yet but we are making progress..


Chunk is already being populated

I'm running WorldBorder to pre-generate my world (partly to reduce load, partly because some structures don't spawn mobs when TC is enabled) and I'm getting this error: http://pastebin.com/9HRCsXGU This will also occasionally happen when WorldBorder isn't running/installed. Latest TC build and latest Spigot 1.11.

It appears as though this issue is related to the frequency at which chunks are populated. It happened every couple of minutes at a frequency of 160 chunks/sec, but it has been running at 40 chunks/sec for around 15 minutes without issue.

Looks like a jungle tree is extending into unloaded chunks - this results in two chunks getting generated at the same time, and therefore the error. I hope I can do something, but for now you can add PopulationBoundsCheck: false to the WorldConfig.

in the config files I saw a warning that setting this to false causes the world to generate differently depending on which direction you explore in, would this affect the terrain heightmap generation or possibly cause visible seams in the world?

@iiegit It doesn't seem to have much effect. First screenshot is with it set to true, second is with it set to false and the terrain explored from a different direction.

I think I fixed the issue, could anyone test this?
Download: http://build.mctcp.com/job/TerrainControl - Master - Gradle/169/default/


FIX: Skip CSRF and login-required checks for webhook endpoints

We don't need to check CSRF tokens for webhook requests because they're sent programmatically by external applications/services where there's no authenticated user and no possibility to forge a request.
Similarly, webhook requests should be allowed to go through even when the site is login-required because they're expected to be sent as unauthenticated requests (no current user) and they can only perform a specific action with no way to exfiltrate arbitrary data from the site.

Will merge and deploy fnord


myzenwallet Error. Error: Request failed with status code 400

When I try to send ZenCash from my wallet I am getting this error.
Is it a problem with the Insight API?
I have tried it on the online version of MyZenWallet too, and with two different browsers. So the problem is not on my side.

Try with fees of 0.000001

It worked!!! Thanks a lot for the help!

This request may close or must a correction be made?


[TASK] Add devcontainer back and add note that linux currently needs vscode insiders

Estimated story points: 2
Description

devcontainers make it really easy to get a reproducible environment.

Acceptance criteria

time: 2.5h


Simplification to menu bar / research mode display

Agreeing with @maryjgoldman, the current menu bar is a mess. I particularly dislike the confusing "DEFAULT MODE/RESEARCH PAGES" toggle, which conflates state (the fact the button label changes) with being a menu.
I propose the following:

(Credit to Mary) Adding a MORE menu button on the far right. Clicking on this button would reveal less common features.

Move "RESEARCH/DEFAULT PAGES" to the MORE button, renaming it "DETAIL MODE"

Move "ABOUT" to more

Later, we can add "VERSION" to this "MORE" menu. (also credit to Mary).
What do others think?

Opening an LShift Trello card: https://trello.com/c/IdE0GJpS/27-add-more-menu-button-to-the-menu-bar

Now that we're down to tweaks on the toggle, it's time to restart with a new issue.


Method 'new' not declared in current class.

Tried installing the project, but I'm getting an error in Argument.gd->build.
I've added Console.tscn as an autoloaded singleton, but this is still happening.
project.godot included in the zip.
project.zip
Ver: Godot 3.1 stable

Hello,
Thanks for providing demo project for reproducing the issue.
I'm aware of this bug as Godot devs are (see: github.com/godotengine/godot/issues/27491 and github.com/godotengine/godot/issues/19345).
Currently I don't have to much time to fix everything that Godot devs broke with 3.1 version.
Sorry for inconvenience. I'll try to fix this repo during this week. Will keep you updated.
Best

Hi, i was able to fix this changing the line:
return new(name, type)
with
return load("res://path/to/script").new(name, type)
both in Argument.gd and accordingly in Command.gd

This issue wasn't active for a long time, I'll close it for now. If you will have any other problem don't hesitate to ask.
Best.

Can this not be closed? Its still broken.


Problem about tracking

Thank you for your great work!
I wonder if you have released the code related to tracking, since the 'tracking for NuScenes3DDetTrackDataset' is False for all configs.
If you have released it, is there an example config?

The dataset supports tracking evaluation, but the code of the multi-object tracking model has not been released yet.
Our previous tracking method was implemented by post-processing of feature similarity matching, but we have given up exploring this method.
We will directly expose the model for joint detection and tracking later.

The dataset supports tracking evaluation, but the code of the multi-object tracking model has not been released yet.
Our previous tracking method was implemented by post-processing of feature similarity matching, but we have given up exploring this method.
We will directly expose the model for joint detection and tracking later.

Thank you for your reply! May I know when you are going to release the joint detection-tracking code?

The time is still uncertain, and it won't be soon, at least 1-2 months later.


Add list of allowed tables for emitting table level metrics

Description
Currently there is one config to enable/disable emitting table level metrics for all tables in a cluster. For big clusters, let's say the ones with a few thousands tables, this is problematic as one might need to set up alerts/monitoring on some the tables that have higher priority than others in the cluster. This PR introduces a config for whitelisted tables to address this problem. Table level metrics are still emitted for whitelisted tables even if emitting table level metrics is disabled for the whole cluster.
Testing Done
For the following scenarios, deployed pinot locally; ran quick-start-batch script; and took a snapshot of metrics using jvisualvm.

enableTableLevelMetrics = false and added baseballStats to the whitelist

enableTableLevelMetrics = false and empty whitelist

enableTableLevelMetrics = true

Aren’t we trying to address the limitations of monitoring systems in Pinot?

I'm not aware that. Could you elaborate a bit more? what's exactly the plan and what's the timeline for it?
The solution presented here is regarding a large cluster at Linkedin for which table level metrics are disabled for all tables. The current situation is risky as for some high priority tables, we don't get alerted. This PR immediately alleviates the existing issue.

Aren’t we trying to address the limitations of monitoring systems in Pinot?

I'm not aware that. Could you elaborate a bit more? what's exactly the plan and what's the timeline for it?
The solution presented here is regarding a large cluster at Linkedin for which table level metrics are disabled for all tables. The current situation is risky as for some high priority tables, we don't get alerted. This PR immediately alleviates the existing issue.

Sorry, I was referring to the changes in this PR. Ideally, we should be logging metrics for all tables/resources. It's up to the operators to set alerts on the right tables that are important for the business.
By adding this new config, we are adding a workaround in Pinot to overcome the limitation with the metrics system which cannot handle thousands of tables.


instructions on how to build the project

is there a step by step guide and requirements on how to build the project?

not that i know of.
AFAIR it is:

install node.js
got clone the repo
run node command prompt
type "npm install"
type "npm run run"


Add support for overriding the web view used for sign in

As per the active directory docs there are a few web views supported in the MSAL iOS version.
The default web view used supports single sign-on (SSO) and shows the " wants to sign in with " permission dialog.
By using either the SafariViewController or WKWebView alternatives, this permission dialog can be avoided. However, when using the WKWebView version, we do sacrifice SSO.

Example usage:
window.cordova.plugins.msalPlugin.signInInteractive(mycbfunction(msg), myerrorfunction(errmsg), {
        webViewType: 'WK_WEB_VIEW'
    }
);

Thanks for your contribution! I apologize for the delay getting to this - I've been putting in some extra hours lately at my day job and haven't been doing much development outside it. It's rewarding, but definitely been crazy with COVID-related things going on.

No problems thanks for getting back to me in the end 😁