Prediction is empty

Hi, when I was testing on GTA validation set, I didn't get any prediction results.
Is there anything wrong in my implementation ?
Here's the details of my pipeline:

I download dataset and checkpoints using python loader/download.py mini and python loader/download.py checkpoint
I unzip 3d_tracking_checkpoint.zip and the directory looks like:

checkpoint
|-- 3d_tracking_checkpoint.zip
|-- 616_gta_checkpoint_030.pth.tar
|-- 623_kitti_checkpoint_100.pth.tar
|-- 803_kitti_300_linear.pth
`-- faster_rcnn_checkpoint.zip

I unzip gta_3d_tracking_val_image_0001.zip and gta_3d_tracking_val_label_0001.zip and the directory tree looks like:

data
|-- gta5_tracking
|   `-- val
|       |-- image
|       |   |-- rec_10090911_clouds_21h53m_x-968y-1487tox2523y214
|       |   |-- rec_10090913_thunder_15h7m_x-293y-199tox-2713y2301
|       |   |-- rec_10090915_thunder_9h11m_x383y-544tox2291y2832
|       |   |-- rec_10090917_thunder_20h0m_x-533y-1709tox-2718y1505
|       |   |-- rec_10090920_clearing_19h19m_x-647y-319tox1349y2684
|       |   |-- rec_10090922_overcast_9h52m_x34y-1009tox-2815y48
|       |   |-- rec_10090924_rain_10h52m_x-6y-1235tox2150y1328
|       |   |-- rec_10090926_overcast_6h56m_x-178y-1803tox-2358y1027
|       |   |-- rec_10090929_foggy_7h57m_x103y-183tox-2986y1982
|       |   `-- rec_10090934_rain_14h17m_x-324y-664tox1977y2572
|       `-- label
|           |-- rec_10090911_clouds_21h53m_x-968y-1487tox2523y214
|           |-- rec_10090911_clouds_21h53m_x-968y-1487tox2523y214_bdd.json
|           |-- rec_10090913_thunder_15h7m_x-293y-199tox-2713y2301
|           |-- rec_10090913_thunder_15h7m_x-293y-199tox-2713y2301_bdd.json
|           |-- rec_10090915_thunder_9h11m_x383y-544tox2291y2832
|           |-- rec_10090915_thunder_9h11m_x383y-544tox2291y2832_bdd.json
|           |-- rec_10090917_thunder_20h0m_x-533y-1709tox-2718y1505
|           |-- rec_10090917_thunder_20h0m_x-533y-1709tox-2718y1505_bdd.json
|           |-- rec_10090920_clearing_19h19m_x-647y-319tox1349y2684
|           |-- rec_10090920_clearing_19h19m_x-647y-319tox1349y2684_bdd.json
|           |-- rec_10090922_overcast_9h52m_x34y-1009tox-2815y48
|           |-- rec_10090922_overcast_9h52m_x34y-1009tox-2815y48_bdd.json
|           |-- rec_10090924_rain_10h52m_x-6y-1235tox2150y1328
|           |-- rec_10090924_rain_10h52m_x-6y-1235tox2150y1328_bdd.json
|           |-- rec_10090926_overcast_6h56m_x-178y-1803tox-2358y1027
|           |-- rec_10090926_overcast_6h56m_x-178y-1803tox-2358y1027_bdd.json
|           |-- rec_10090929_foggy_7h57m_x103y-183tox-2986y1982
|           |-- rec_10090929_foggy_7h57m_x103y-183tox-2986y1982_bdd.json
|           |-- rec_10090934_rain_14h17m_x-324y-664tox1977y2572
|           `-- rec_10090934_rain_14h17m_x-324y-664tox1977y2572_bdd.json

then run sh scripts/test_gta.sh 0 in 3d-vehicle-tracking/3d-tracking/

During the inference process, I got the log says Prediction is empty :
Load single json file
Sequences [461] with total 461 frames
Input images are not normalized
Number of image to test: 461
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
Prediction is empty
...

I also checked the output pkl file and find predicted result is empty in every frame
I tried change the L16 in test_gta.sh to test on other sequence such as rec_10090934_rain_14h17m_x-324y-664tox1977y2572 but still got no prediction results.

I checked my step and finally found that I've missed the step in Data Preparation, after I generate the prediction, everything goes well then. But it's still wired that model predict more bboxs than ground truth:

...
4050 images.
Frame 4050, GT: 6 Boxes, PD: 178 Boxes
Frame 4051, GT: 6 Boxes, PD: 158 Boxes
Frame 4052, GT: 7 Boxes, PD: 173 Boxes
Frame 4053, GT: 8 Boxes, PD: 180 Boxes
Frame 4054, GT: 8 Boxes, PD: 186 Boxes
Frame 4055, GT: 8 Boxes, PD: 162 Boxes
Frame 4056, GT: 8 Boxes, PD: 175 Boxes
Frame 4057, GT: 8 Boxes, PD: 166 Boxes
Frame 4058, GT: 8 Boxes, PD: 157 Boxes
Frame 4059, GT: 8 Boxes, PD: 149 Boxes
...


Better error handling

Overview
The node implementation handles different processes running concurrently, most of them related to the witness module; Ethereum watchers (blocks and events) and TM subscription. These run in the background and right now we don't have control over their failures, which means that if one of this process fails, we log the error and quit, but the node keeps running. This is probably not the desired behavior.
We need to define what's the expected behavior and implement it.

Closing because most of the referenced topics in the thread have been covered. Separate issues can/should be opened for more specific error handling.


memory issue

New Issue Checklist

[ ] I have read and understood the CONTRIBUTING guide
[ ] I have read the Documentation
[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. ios / tvos

Platform Version
e.g. 8.0

SDWebImage Version
e.g. 3.7.6

Integration Method
e.g. carthage / cocoapods / manually

Xcode Version
e.g. Xcode 7.3

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Xcode7.3 ios10.3 , when load GIF , memory will Constantly Rising until app crash
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

v4 now uses FLAnimatedImage to support gif loading. check this

yes,I use FLAnimatedImage,but memory will Constantly Rising until app crash
last year, it's well.  after ios10.3 ,this issue is appear
Xcode7.3
ios10.3

whats the size a the gif files you are trying to open

the file size is 892k .  I promise you will find this issue ,when you build a project loading gif with Xcode 7.3 and run it on the platform whose version is ios 10.3 . I think it's a bug from apple , but i cant resolve it

can you replicate this in the demo project (using v4 of this library)?

i have replicate this in demo project (please use xcode 7.3 , ios 10 later , you will find it)

another, language version swift 2.3

does this happen in simulator only or device? if this happens only in iOS 10.3? in iOS 9 the issue is not there? weird...if its an Apple firmware bug i do not know if there is a way to resolve this. Have you opened an issue in the FLAnimatedImage repository?

there  is not the issue in iOS 9. this happen not only on FLAnimatedImage, but also SwiftyGif, YLGIFImage, UIImageView. I think i should upgrade Xcode version and swift version to avoid it. It will be a huge project --!


Smart TV related questions

I saw this video https://www.youtube.com/watch?v=vv5emwgtrDc is regarding Toast.
I have one question.
Can I use web app using Toast with all types of smart TVs like Sony, LG ...?

Hi, I think the answer to your question is on the front page here:
https://github.com/Samsung/cordova-plugin-toast#supported-platform

Thanks.
On Tue, 12 Feb, 2019, 3:48 AM Serge Kuts<EMAIL_ADDRESS>wrote:

Hi, I think the answer to your question is on the front page here:
https://github.com/Samsung/cordova-plugin-toast#supported-platform
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Samsung/cordova-plugin-toast/issues/94#issuecomment-462517590,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYVwfWWHnHEyXHZfBHMJ6eKRCfqexeVaks5vMewvgaJpZM4anJdZ
.

@s2qts , What about legacy TV 2010(C), 2011(D)? Maybe you know.


v0.1.2 - crossvalidation fixes

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/23469
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.2 -m "<description of version>" 6bd828ac48956758872f2e13c5c07f8cb98fe4f3
git push origin v0.1.2


getTableData vs getFilteredData

getFilteredData allows me to pass "true" as a parameter to include the header row
It would be nice if there was the same option for getTableData (please).  I did try it but it didn't appear to work on my test table.
getFilteredData returns an empty array if no filter has yet been applied since the page loaded (but will return a full table with header row if the filters are applied then all cleared).  This is fine, I've got some logic to pull the full table with getTableData()  but I need the header row too.
Thanks

getTableData does not accept a parameter at the moment, I will add it to make it possible to include the header row values. I tagged this issue as enhancement and it should be part of the next version.

I just deployed version 0.0.7 which includes this enhancement.
Cheers

Thanks, that's great! :-)  Just thought you'd like to know that feedback on
Tablefilter from our initial test users is really positive!
Chris
Chris Johnson
On 28 October 2015 at 07:22, koalyptus<EMAIL_ADDRESS>wrote:

I just deployed version 0.0.7 which includes this enhancement.
Cheers
—
Reply to this email directly or view it on GitHub
https://github.com/koalyptus/TableFilter/issues/48#issuecomment-151750508
.

Cool, thanks for the feedback


Genconfigroutes modes

What does this PR do?
genConfigRoutes.py now has the -s option to enable generation of CDN snapshot routes, as well as the -C option to disable the generation of server configuration file routes. The dockerfile supports this change.
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[x] The compare tool

What is the best way to verify this PR?
Try to build and run the compare tool and route generator, both standalone and using the dockerfile in all 4 modes. Check the documentation for instructions (under "Tools").

[ ] This PR includes tests
[x] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2824/
Test FAILed.


How to output pdf?

I'd like to test the pdf output. What command can I call to run the latex compiler?

In your article YAML metadata/header block, you can add latex: true (or more detailed latex options) underneath the output key.
Here are some examples:

https://github.com/uwdata/living-papers/blob/main/examples/fast-kde/fast-kde.md?plain=1#L23
https://github.com/uwdata/living-papers/blob/main/examples/reactive/reactive.md?plain=1#L8

Thank you!
Best,
Matias Faure
PhD student - NYU Politics
On Oct 25, 2023 at 11:30 PM -0400, Jeffrey Heer @.***>, wrote:

In your article YAML metadata/header block, you can add latex: true (or more detailed latex options) underneath the output key.
Here are some examples:
• https://github.com/uwdata/living-papers/blob/main/examples/fast-kde/fast-kde.md?plain=1#L23
• https://github.com/uwdata/living-papers/blob/main/examples/reactive/reactive.md?plain=1#L8
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

@jheer, hello.
I tried setting output: latex: true, and i get the following error:
Compiling latex PDF Error: ENOENT: no such file or directory, lstat '.temp/latex/index.pdf'
Any idea?
Thanks in advance

@GdSAu I would need more information to diagnose the issue. When generating a latex PDF, Living Papers writes the latex project to .temp/latex/ and runs pdflatex and bibtex within that directory. It then copies the generated PDF out of the folder. It looks like this final copy is what is breaking? So I'd recommend first checking the contents of .temp/latex/. There should be a latex log files which you can check to see if perhaps there was an error during latex compilation...


[Storage - Blob] syncUploadFromURL large content with timeout failing in nightly runs

Storage - Blob nightly test runs are failing with:

Error message:
expected 'CannotVerifyCopySource' to deeply equal 'OperationTimedOut'
Stack trace:
AssertionError: expected 'CannotVerifyCopySource' to deeply equal 'OperationTimedOut'
at Context. (D:\a_work\1\s\sdk\storage\storage-blob\test\node\blockblobclient.spec.ts:632:14)
at processTicksAndRejections (internal/process/task_queues.js:97:5)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1724372&view=results

Rerun passed.


simplify code

What is the purpose of the change
simplify the code
Brief changelog
use constructor instead of the default one.
Verifying this change
no functional change
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before requesting the community to review your PR.

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

Coverage increased (+0.07%) to 43.191% when pulling 79e01b07d26ee3694fde19de96ef0a9c95c3869f on ahuazhu:master into d4d02b5e017ebf8c7d3405516264645280cf57fa on apache:master.

I think the code is OK, but please merge into the develop branch and submit the corresponding issue. I'll close this PR first.


swap in bwa mem 2

https://github.com/bwa-mem2/bwa-mem2

Bwa-mem2 is the next version of the bwa-mem algorithm in bwa. It produces alignment identical to bwa and is ~1.3-3.1x faster depending on the use-case, dataset and the running machine.

[ ] add bwa-mem2 to docker image: mgibio/alignment_helper-cwl
[ ] update CWL - align_and_tag.cwl (move script into CWL)
[ ] update CWL - fastq_align_and_tag.cwl
[ ] update CWL - sequence_align_and_tag.cwl
[ ] Create new reference indices using bwa-mem2, add to annotation directories locally
[ ] update inputs wiki page to reflect building index with bwa-mem2
[ ] validate equivalence of alignments - start with HCC1395?

Looks like there are upcoming breaking changes to the index format that are in master but not yet officially released.  Just something to watch out for when switching versions.  (Too bad they didn't make it in before the initial 2.0 final release!)

https://github.com/bwa-mem2/bwa-mem2/issues/92#issuecomment-689258313 says a new release is coming soon.  (Also, the index format has changed a second time :smile:)

https://twitter.com/wasim_galaxy/status/1304074085165457408
BWAMEM2 index file size and its memory footprint are now ~16GB down from 42GB.
We are actively working on further reducing it down to ~10GB. ```

Seems like they're making progress

In testing, swapping in this tool is leading to intermittent failures in our alignment tool.  Since it can both fail and succeed on the same data, it must be a lower-level issue of some sort in the interaction between the tools.


SPI Mode 0, MISO Data is sampled one wrong edge

Hi Stephan,
at the moment i do some tests with the SPI module 9c653245. The SPI is configured as follows:
/* setup SPI */
neorv32_spi_disable();
    // neorv32_spi_setup(uint8_t prsc, uint8_t clk_phase, uint8_t clk_polarity, uint8_t data_size)
neorv32_spi_setup(0, 0, 0, 0);  // spi mode 0, 8bit
neorv32_spi_highspeed_enable(); // running at half of the processor clock
neorv32_spi_enable();

/* perform SPI write */
neorv32_spi_cs_en(0);   // lines are LOW when activated.
	// READ IST
neorv32_spi_trans(0x3);
	// 24Bit ADR
neorv32_spi_trans(0x0);
neorv32_spi_trans(0x0);
neorv32_spi_trans(0x0);
	// READ
neorv32_spi_trans(0x0);
neorv32_spi_trans(0x0);

neorv32_spi_cs_dis(0);  // lines are HIGH when deactivated.

Analog Devices writes following related to SPI:

SPI Mode
CPOL
CPHA
Clock Polarity in Idle State
Clock Phase Used to Sample and/or Shift the Data

0
0
0
Logic low
Data sampled on rising edge and shifted out on the falling edge

1
0
1
Logic low
Data sampled on the falling edge and shifted out on the rising edge

2
1
0
Logic high
Data sampled on the rising edge and shifted out on the falling edge

3
1
1
Logic high
Data sampled on the falling edge and shifted out on the rising edge

Table 1, https://www.analog.com/en/analog-dialogue/articles/introduction-to-spi-interface.html
In my sim is the sreg updated on the falling edge of SCK:

In my opinion violates the SPI the Mode 0 spec. What do you think? As workaround could be two sreg used, one for RX, the other for TX to achieve the half SCK periode between update and capture.
Thanks for your comments.
BR, Andreas

Oh, those relentless clock modes! 😅

Analog Devices writes following related to SPI:

That is absolutely correct and matches the SPI clock modes described by this image:

And this is also what "should" be implemented by the NEORV32 SPI and XIP modules.

In my opinion violates the SPI the Mode 0 spec. What do you think?

Well, this is a complex situation since the SPI clock is quite close to the processor clock (high-speed mode enabled). The actual sampling of the SDI signal takes places half a SPI clock cycle before the shift register is updated. This is what rtx_engine.sdi_sync is used for - to sample the SDI line before updating the shift register.

Have a look at this example transmission:

sending/receiving 0x83 in 8-bit mode, SPI clock mode 0, high-speed mode enabled
output data (spi_sdo_o) is updated at the falling edge of spi_sck_o ✔
input data (spi_sdi_i) is sampled at the rising edge of spi_sck_o into rtx_engine.sdi_sync (at the yellow cursor) ✔

So I think the behavior is correct?! 🤔

Hi Stephan,
the thing with the 'rtx_engine.sdi_sync' is a real nice trick to safe a lot of FFs.

So I think the behavior is correct?!

Absolutly. Thanks for the additional sims.
BR,
Andreas


Hide Construct.dependenyRoots

We wanted to ensure that the base Construct public API is clean (besides node) and leaves room for the API of the actual construct. Somehow dependencyRoots snuck in.
Is there a way to get rid of this? Why even use an override here? Can't we do something like:
construct.node.addDependencyRoot(x);

It's not about Construct.dependencyRoots, it's about IDependable.dependencyRoots, and it's also implemented by non-Constructs.
I have a feeling this crops up a bunch, where we want to implement some polymorphic facility for some classes without necessarily dispatching through the object instance (because then the facility needs to show in the public interface, like here).
We can take the implementation out of the class. Something like traits:
interface IDependable {
}

// Declaration of trait
abstract class DependableTrait {
  public abstract dependencyRoots;
}

// Declaration of class unsullied by trait signature
// Does have a marker interface to help type checking.
class MyDependableThing implements IDependable {
  // ...
  // nothing about dependencyRoots here
  // ...
}

// Trait implemented separately from subject class
class SomeTraitImplementation extends DepenableTrait {
  constructor(subject: MyDependableThing) { }
  public dependencyRoots() {
    // Implementation using this.subject here
  }
}

DependableTrait.implement(MyDependableThing, SomeTraitImplementation);

// Usage site
function foo(target: IDependable) {
  const roots = DependableTrait.get(target).dependencyRoots;
}

We could do the same for bind() and other implementation-concern API surface.

I like the marker interface but I am wondering if perhaps the traits can be simply implemented inline. Doesn't seem like a separate class buys us too much, or maybe I am missing something.
interface IDependable { }
interface IDependableTrait {
  dependencyRoots: ...
}

class MyDependableThing implements IDependable {
  constructor() {
    Trait.implement<IDependableTrait>(this, {
        dependencyRoots: ...
    });
  }
}

// usage
Trait.get<IDependableTrait>(target).dependencyRoots...

Having said that, I am wondering if this pattern is actually that prevalent. I don't know if "bind" lends itself nicely to this, and I think that as we move to "integration classes", we'll see much less of this need.
Can we figure something out for dependencyRoots for now and see how it works?

wondering if perhaps the traits can be simply implemented inline. Doesn't seem like a separate class buys us too much

In your code, we're mapping instances to instances. In my code, I was mapping classes to classes.
The latter will be more efficient if we're going to have a lot of instances of the same class.

    Trait.implement<IDependableTrait>(this, {

I was thinking this should still be accessible over jsii, so no generics.

Having said that, I am wondering if this pattern is actually that prevalent

It also occurs in the ELB library, where LBs and listeners and groups have a bind()-like protocol between them, AND the objects themselves expose a public API that customers are expected to call.


bluetooth LE indicate is missing?

Environment

Development Kit:      [ESP32-DevKitC]
Kit version (for WroverKit/PicoKit/DevKitC): [v4]
Core (if using chip or module): [ESP-WROOM32]
IDF version (git rev-parse --short HEAD to get the commit id.):
aaf12390
Development Env:		[other]
Operating System:		[Windows]
Power Supply:         [USB]

Problem Description
I'm using a DevKitC to write/read from another BLE module (Microchip RN4020) which supports ESP_GATT_CHAR_PROP_BIT_INDICATE. But current esp apis seems to only supports ESP_GATT_CHAR_PROP_BIT_NOTIFY
//Detailed problem description goes here.
Expected Behavior
esp_ble_gattc_register_for_indicate should be available
Actual Behavior
only esp_ble_gattc_register_for_notify is available
Code to reproduce this issue
this is my current workaround but I don't think it's the right way to go
if (char_elem_result[0].properties & ESP_GATT_CHAR_PROP_BIT_NOTIFY){
    gl_profile_tab[PROFILE_A_APP_ID].char_handle = char_elem_result[0].char_handle;
    esp_ble_gattc_register_for_notify (gattc_if, gl_profile_tab[PROFILE_A_APP_ID].remote_bda, char_elem_result[0].char_handle);
}else if (char_elem_result[0].properties & ESP_GATT_CHAR_PROP_BIT_INDICATE){
    ESP_LOGI(GATTC_TAG, "ESP_GATT_CHAR_PROP_BIT_INDICATE");
    gl_profile_tab[PROFILE_A_APP_ID].char_handle = char_elem_result[0].char_handle;
    esp_ble_gattc_register_for_notify (gattc_if, gl_profile_tab[PROFILE_A_APP_ID].remote_bda, char_elem_result[0].char_handle);
}

my char_elem_result[0].properties is 0x28 which is Write & Indicate

then should the gattc_demo.c be update to:
if (char_elem_result[0].properties & (ESP_GATT_CHAR_PROP_BIT_NOTIFY|ESP_GATT_CHAR_PROP_BIT_INDICATE)){
    gl_profile_tab[PROFILE_A_APP_ID].char_handle = char_elem_result[0].char_handle;
    esp_ble_gattc_register_for_notify (gattc_if, gl_profile_tab[PROFILE_A_APP_ID].remote_bda, char_elem_result[0].char_handle);
}

@xufeipyxis please refer to #1628
@negativekelvin  thanks for your help.


Trace API plugin - Add support for action return values

Change Description
The intent of this PR is to add support to the trace api plugin for a new field representing a value that may be returned by an action.
To accomplish this task 3 new structs were added:
action_trace_v1 : representing the new version of an action trace that now may contain a field representing the return value.
transaction_trace_v2: representing the new version for the transaction trace that supports action_trace_v1.
block_trace_v2: representing the new version for block trace that supports transaction_trace_v2.
The general idea is that, for example, a new version for action traces may now not require the creation of new structs for transaction trace and block trace, but only a modification to the actions type on the transaction trace struct (add a new possibility for the variant).
Change Type
Select ONE

[ ] Documentation

[ ] Stability bug fix

[ ] Other

[ ] Other - special case

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

It appears unnecessary to include std::vector<action_trace_v0> in this as it will never be filled in. Is the choice to include it intentional such that the serialized format has a which value that lines up with the version?

That was not the intention, i added just as a way to experiment how it would look like, but as you mention, currently it will not be filled in, so i just deleted it.

Is it worth considering taking a hard stance that "create_" code paths only make the latest types and can therefore be simplified with that assumption.

I modified to create the latest block_trace version .


Allow standalone ternary expressions

Closes #71065
Removes STANDALONE_EXPRESSION warning and allows void expressions for ternary operator

This also incorrectly applied to standalone ternaries.

Well, "incorrectly" is debatable as I wrote in the issue. The ternary expression is expected to return something and it cannot do so with void methods, so it does make sense that it fails.
As I also wrote in the issue, we need to decide if using it as a shorthand one-line if statement is acceptable.

Ah, I thought that was already accepted behavior for the ternary op. I'll mark this as draft for now. Thanks!

@jordigcs It seems the consensus is that this feature is wanted and this is a proper solution. The only thing missing is what I mentioned in my comment: the ternary operator should not give the STANDALONE_EXPRESSION warning. Do you want to add it to this PR?

Sure, I'll get that done in a moment.

Done!

Check is failing because of issues with the check script

Thanks!


help with set up.

[andrewcz@andrewcz-pc Personae]$ sudo docker run -p 27017:27017 -v /data/db:/data/db -d --network=your_network mongo
040b83a8b3aa269b5290a4234a487265f71b0465e95169997e44d386770f989d
docker: Error response from daemon: network your_network not found.
how do i set up mongodb running a arch linux system.
Amazing repo by the way
BEst,
Andrew

Sorry for late reply, I just got a cold these days, you should create a network by docker first.
See also How to create network in Docker

ah, brilliant champion!
it worked!
great repository!
would like to add to it :).
Many thanks,
Andrew

I'm not sure if are a Chinese, if you are, you could add my WeChat 610261753 if you have further questions, and we also have a WeChat group, you can join it if you would like to.


Write Ångström such that it is understandable by Pint in datamodel

Codecov Report

Merging #170 (d0812e2) into main (2f43b04) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #170   +/-   ##
=======================================
  Coverage   76.19%   76.19%
=======================================
  Files          15       15
  Lines         605      605
=======================================
  Hits          461      461
  Misses        144      144

Flag
Coverage Δ

linux
76.19% <ø> (ø)

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


Enhancements

[x] Tag issue being addressed
[x] Add tests
[x] Update docstrings/documentation, if applicable
[x] Update changelog
[x] All tests successfully pass

Codecov Report

Merging #121 (0e716c7) into devel (eb02cd7) will increase coverage by 0.36%.
The diff coverage is 100.00%.

Codecov Report

Merging #121 (0e716c7) into devel (eb02cd7) will increase coverage by 0.36%.
The diff coverage is 100.00%.


Creating project scoped role failed

Expected vs actual behavior
Actual:
$ ./pctl create role test-role --scope project --permissions project.read,cluster.read,partner.read,organization.read --config<EMAIL_ADDRESS>Error: invalid permissions, error: invalid permission partner.read
$ ./pctl create role test-role --scope project --permissions project.read,cluster.read,organization.read --config<EMAIL_ADDRESS>Error: invalid permissions, error: invalid permission organization.read

Should be able to create role with "partner.read" permission.
We are able to create "project" scoped role with "partner.read" permission from the Paralus console.
Steps to reproduce the bug
Run above steps.
Are you using the latest version of the project?
You can check your version by running helm ls|grep '^<deployment-name>' or using pctl, pctl version, and provide the output.

What is your environment setup? Please tell us your cloud provider, operating system, and include the output of kubectl version --output=yaml and helm version. Any other information that you have, eg. logs and custom values, is highly appreciated!
VERSION: 0.1.0
BUILD: 0.1.0
BUILD-TIME:<PHONE_NUMBER>
ARCH: linux/amd64

(optional) If you have ideas on why the bug happens or how it can be solved, please provide it here

[X] I've described the bug, included steps to reproduce it, and included my environment setup with all customizations.
[X] I'm using the latest version of the project.

@niravparikh05 There is discrepancy between dashboard and CLI tool. Can you verify which one is correct method to create a role?

both partner and organization read roles are required irrespective of the scoped role creation. partner and organization read roles are currently scoped at organization level and hence added these to scoped role permission list service.


Hub - check trailing and double slashes when constructing API urls, better page/sort param support

Fixes AAH-3007, follows #1408
hubAPI and pulpAPI will now throw when any of these happen:

HUB_API_PREFIX ends with a trailing slash
prefix-relative url doesn't start with a slash (already happening)
url doesn't end in a slash (end of string or before a ?)
url contains a double slash
url contains a double ?
the wrong param is used based on the url (page, limit, offset, page_size, sort, ordering, order_by)

add hubQueryString to wrap normalizeQueryString and:

escape values
convert page/limit/offset/page_size/sort/ordering/sort_by to the right name

allow a hubAPI`/foo/?${{ param1: 'abc', param2: 123 }}` syntax, calling hubQueryString
fixes all instances of hubAPI & pulpAPI missing a trailing slash
also, this allows us to fix all those || '' and ?? '' and as string in urls
and fix tests

In e2e tests, hubAPI and pulpAPI retain their simpler versions, I'm not sure how to handle the use of * wildcards without allowing them in regular code.

rebased, and updated the offset <-> page conversion to fix an off-by-one (page starts at 1, offset at 0: https://github.com/ansible/ansible-ui/pull/1530/commits/b600638eab405d8ab74731d06896ec5ad967ff7b)


Adding an abstract method is a binary compatible evolution. Fix #69

As stated in Java binary compatibility doc
(http://www.cs.cornell.edu/andru/javaspec/13.doc.html), "adding new fields,
methods, or constructors to an existing class or interface" is a binary
compatible evolution.
This is probably the most surprising binary compatible evolution, and it's
pretty neat.
Also, by fixing this problem, we now also fully support Java8 default methods.
Which is why this commit also Fix #68.

@jsuereth This one should bring back some memory from Scala Days in Stanford ;-)
And pinging a few more people that may be interested in this fix \cc @adriaanm @ktoso @jroper

If I don't hear anything, I'm going to merge this tonight.

Closing this as there are a couple of things I need to verify. Will reopen when it's ready.


fix(IAM Identity): update report schema and fix ITs

PR summary
The report schema is slightly improvised, incorporating the changes into SDKs
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Abrupt chapter transitions in Thorium

I'm submitting a combination bug report/question.
The Thorium desktop reader on Windows 10 has unexpected behavior (in my opinion) during chapter transitions.  If you flip through a book with many chapters, especially using a two-page spread, you'll notice the following:

Each chapter begins on the left page of the spread.  At the end of the chapter, there's often a blank right-hand side column, so the next chapter begins on the next left-hand page.

When you reach an end-of-the-chapter spread that has a blank right column, the animation for the page transition doesn't look right.  The ending text in the chapter seems to emerge from the center of the spread and move into the left column.

When you flip the spread to the next chapter, there isn't a page-turn animation.  The new chapter just shows up immediately.

It seems that Thorium is simply taking each chapter as a separate HTML document and displaying them one at a time.  It doesn't cache the next chapter to be read inline with the current chapter.  This makes the reading experience seem "weird," in my opinion.
Are there any workarounds in JavaScript for these behaviors?  In particular, is there some preferred way to cache the next chapter so that it is shown on the same spread as the previous chapter?  In other words, to display the start of the next chapter in the right-hand column instead of showing an empty column?
One option would simply be to concatenate the chapters together in the DOM to form a single HTML page that represents the entire book.  That seems like a bad idea for memory and performance, though.  It would also upset the authors' intentions if (for some reason) they have different CSS styles in different chapters, since only the CSS from one chapter would be used.
Thanks very much for any insights.

Hello @RobJacobson thank you for your feedback! Please allow me to move this issue to the readium-desktop repository. I think that the problems you are describing will not be solvable here in readium-css.


fix examples

This fixes a few issues in the example project which prevented testing device motion and magnetometer updates, by referencing wrong classes and methods.

@askielboe This actually seems like a dupe of #13 , Which oddly wasn't merged

hah, yea sorry I just realized that.. I should check the pull requests next time.


Some functions of IBC ClientReader don't work properly

ClientReader::maybe_consensus_state() needs to read the prior state for checking the header
ClientReader::next_consensus_state() might not be able to return due to the infinite loop
ClientReader::prev_consensus_state() cannot returns due to a bug

They have been fixed


Remove language choices from Settings menu (or just leave English, whatever is easier)

Why is this open again?

I thought I closed it accidentally before the testing is done. Feel free to close if this works.


Inject endpoint/baseUri into test class

The actually used endpoint/baseUri should be injectable into the test class, so it can be used freely.

It can't be injected, as it's expectation-specific.


fix: allow for dots in KV Store name

Fastly allows for special characters (like .) in the KV store name, but since the configuration is in toml, these characters need to be either escaped, or wrapped in quotes to be treated as a single string rather than configuration.
This change wraps the kvStoreName variable in double quotes to prevent toml interpolating the name.
Example error:
Don't do this!

node:internal/process/esm_loader:40
      internalBinding('errors').triggerUncaughtException(
                                ^
No
(Use `node --trace-uncaught ...` to show where the exception was thrown)

Hi, thanks for this contribution!

This has been released in 6.1.1


"Out of memory during read" raised when max memory usage (-m) reach

Describe the bug
When memory usage almost reach to value of (-m), if there are coming concurrent set request (value size == 4MB), memcached server will send error "Out of memory during read".
(I have set -I 5M for that 4MB big value data).
If big set request comes sequentially, there will be no error.
To Reproduce
100% reproduce by description as above.
System Information
Official memcached docker image: memcached:latest (1 instance)
docker for windows or azure AKS, both can reproduce this error.
I just configure 2 argument for sending big 'set' value data:

-I 5M,
-m 1024 (actually, any valid data here will produce this error)

Is there any other argument I should adjust for this big data case? Thanks.

Hey,
First; I don't know what specific version that is. We don't supply a docker image directly, or azure anything. stats output always helps.
Many parallel large item uploads are an edge case for large items. The workaround is to add more memory, more or less. For the specifics:

When an item is being written to, its memory is "locked" in place and cannot be evicted or moved.
When memory is mostly full, the amount of RAM will be assigned spread across the different slab classes (100b, 1k, 100k, 512k, etc).
Large items work by stitching chunks together; a 4MB item is one small header (~150b or so) and 8 512k chunks.
If the 512k slab class doesn't have enough RAM in it to satisfy all of these uploads at the same time, they will fail with the error you see.
It also allocates the chunk memory as it reads the item from the network.

I haven't revisited this recently so not sure if there are better options now. You can modify the slab page memory mover algorithm (scripts/memcached-automove) to keep more RAM in the high slab class or add RAM to the system until you have enough available for all of the uploads.
I'll leave this issue open as a reminder to revisit when possible.

@dormando Thanks for reply. I just found seems this error occurs even memcached is restarted, and first 10 concurrent big (4MB) data are sent to memcached.

I'll double check but it will work so long as there's memory. attach output of stats and stats slabs from one run?

I can only 'stats' it by my code running in k8s pod. Is there any other easier way to do that?

Telnet to an instance and type stats and stats slabs. sorry, can't make it much easier than that :)

I can stats to my local memcached container by telnet but cannot connect to the remote pod, maybe network issue.
However, here is stats log from local memcached when that oom log occurs:
stats
STAT pid 6
STAT uptime 24
STAT time<PHONE_NUMBER>
STAT version 1.6.5
STAT libevent 2.1.8-stable
STAT pointer_size 64
STAT rusage_user 0.016730
STAT rusage_system 0.056354
STAT max_connections 1024
STAT curr_connections 2
STAT total_connections 13
STAT rejected_connections 0
STAT connection_structures 13
STAT response_obj_bytes 11680
STAT response_obj_total 10
STAT response_obj_free 9
STAT response_obj_oom 0
STAT read_buf_bytes 163840
STAT read_buf_bytes_free 147456
STAT read_buf_oom 0
STAT reserved_fds 20
STAT cmd_get 0
STAT cmd_set 15
STAT cmd_flush 0
STAT cmd_touch 0
STAT cmd_meta 0
STAT get_hits 0
STAT get_misses 0
STAT get_expired 0
STAT get_flushed 0
STAT delete_misses 0
STAT delete_hits 0
STAT incr_misses 0
STAT incr_hits 0
STAT decr_misses 0
STAT decr_hits 0
STAT cas_misses 0
STAT cas_hits 0
STAT cas_badval 0
STAT touch_hits 0
STAT touch_misses 0
STAT auth_cmds 0
STAT auth_errors 0
STAT bytes_read 100002441
STAT bytes_written 2934
STAT limit_maxbytes 20971520
STAT accepting_conns 1
STAT listen_disabled_num 0
STAT time_in_listen_disabled_us 0
STAT threads 4
STAT conn_yields 0
STAT hash_power_level 16
STAT hash_bytes 524288
STAT hash_is_expanding 0
STAT slab_reassign_rescues 0
STAT slab_reassign_chunk_rescues 1
STAT slab_reassign_evictions_nomem 0
STAT slab_reassign_inline_reclaim 0
STAT slab_reassign_busy_items 0
STAT slab_reassign_busy_deletes 0
STAT slab_reassign_running 0
STAT slabs_moved 1
STAT lru_crawler_running 0
STAT lru_crawler_starts 255
STAT lru_maintainer_juggles 82
STAT malloc_fails 0
STAT log_worker_dropped 0
STAT log_worker_written 0
STAT log_watcher_skipped 0
STAT log_watcher_sent 0
STAT bytes 8000256
STAT curr_items 2
STAT total_items 15
STAT slab_global_page_pool 1
STAT expired_unfetched 0
STAT evicted_unfetched 13
STAT evicted_active 0
STAT evictions 13
STAT reclaimed 0
STAT crawler_reclaimed 0
STAT crawler_items_checked 0
STAT lrutail_reflocked 0
STAT moves_to_cold 15
STAT moves_to_warm 0
STAT moves_within_lru 0
STAT direct_reclaims 126
STAT lru_bumps_dropped 0
END
stats slabs
STAT 4:chunk_size 192
STAT 4:chunks_per_page 5461
STAT 4:total_pages 1
STAT 4:total_chunks 5461
STAT 4:used_chunks 2
STAT 4:free_chunks 5459
STAT 4:free_chunks_end 0
STAT 4:get_hits 0
STAT 4:cmd_set 0
STAT 4:delete_hits 0
STAT 4:incr_hits 0
STAT 4:decr_hits 0
STAT 4:cas_hits 0
STAT 4:cas_badval 0
STAT 4:touch_hits 0
STAT 38:chunk_size 394840
STAT 38:chunks_per_page 2
STAT 38:total_pages 1
STAT 38:total_chunks 2
STAT 38:used_chunks 2
STAT 38:free_chunks 0
STAT 38:free_chunks_end 0
STAT 38:get_hits 0
STAT 38:cmd_set 0
STAT 38:delete_hits 0
STAT 38:incr_hits 0
STAT 38:decr_hits 0
STAT 38:cas_hits 0
STAT 38:cas_badval 0
STAT 38:touch_hits 0
STAT 39:chunk_size 524288
STAT 39:chunks_per_page 2
STAT 39:total_pages 17
STAT 39:total_chunks 34
STAT 39:used_chunks 14
STAT 39:free_chunks 20
STAT 39:free_chunks_end 0
STAT 39:get_hits 0
STAT 39:cmd_set 15
STAT 39:delete_hits 0
STAT 39:incr_hits 0
STAT 39:decr_hits 0
STAT 39:cas_hits 0
STAT 39:cas_badval 0
STAT 39:touch_hits 0
STAT active_slabs 3
STAT total_malloced 20971520
END

@dormando Please help specify if our following case is good to use memcached:
We need to send 4 MB value size per-Request (set type and get type) to cache, there are 240 concurrent connection (so at most there may be 240 Requests coming at same time).
Should we use memcached as our cache for this case?

Is there a reason why you set the memory limit to 20 megabytes? I thought you were using -m 1024

Because I would make this error occur soon. It is little hard to repro issue in my local container if use -m 1024, but it can.

I completely believe you and it's well known that it happens if you don't give it enough memory :) Some RAM is reserved for slab classes, so setting very low values (less than 64 megs) won't work as you expect.
It is possible to make it not happen for larger amounts of RAM, but you have to modify the memory mover algorithm as I mentioned perviously. If you're not up for this I would recommend you either split these values into smaller chunks (< half a megabyte) or use something other than memcached.

Were you able to solve your problem?

Hi Dormando, no, still found that error log even in big RAM (8G ram). We were estimating cache solution for our case, we have to keep big size value so maybe need to find other proper solution for us, thanks for help.

Ok, good luck. I'll keep this in mind next time I revisit the large item code.


User story testing - Define developer workflow for existing I8 site migration / development

[STUB] Please build this out for steps for user testing

From @birkland 's opus The ISLE 8 methodology this could have key concepts to explore and adapt as a framework for this documentation and process.


So clean install and still not working

ive wiped my server with a clean install of just the panel and its dependencies, the panel wont come up it only shows its files until you navigate to the public directory and then it shows
is currently unable to handle this request.
HTTP ERROR 500
any suggestions?

This is not a bug, please use either our Discord (preferred) or forums for further assistance.

Contact me in #support if you need help with this, willing to bet your webserver wasn't configured correctly.
https://discord.gg/bAcTUrX

Issue resolved. PHP CLI 7.1 was installed, while PHP 5 was on the webserver. Simply deleted everything PHP5 related and re-ran the dependency command for apache on the docs.


Web Worker Example App

Goal
To provide a practical example of how to implement @bugnsag/web-worker in a JS web worker and service worker.
Design
Followed majority of conventions set by previous example apps for consistency.
Changeset
N/A
Testing

Manual testing by adding my own API key and pressing the buttons on frontend to ensure events are sent.
Tested minifying the code and uploaded source maps to ensure there were no issues symbolicating the stacktrace.
Tested that the preventDefault usage in script.js prevents duplicate events being reported.

NPM:
https://www.npmjs.com/package/@bugsnag/web-worker

Approved assuming updated version numbers after web-worker release

Seems that I'm currently unable to report any events to the dashboard from service-worker.js. Not too sure why currently.
Feel that this app will need transforming into a Node app, otherwise we'll need to implement BugSnag through the CDN approach in index.html to use it in the parent scripts which doesn't feel as tidy and clear.


False Positive on spring security kerberos extension

False positives on the two libraries spring-security-kerberos-core.jar and spring-security-kerberos-web.jar from https://spring.io/projects/spring-security-kerberos.
This extension uses a different version number from spring security.
This is similar to #1694 for the spring security saml extension.
Suggestion for suppressing:
	<packageUrl regex="true">^pkg:maven/org\.springframework\.security\.kerberos/spring-security-kerberos.*$</packageUrl>
        <cpe>cpe:/a:pivotal_software:spring_security</cpe>
        <cpe>cpe:/a:security-framework_project:security-framework</cpe>

The packageUrl only works when the CentralAnalyser is active. Otherwise I use the filePath:
.\spring-security-kerberos..jar


fix: improved componentsContent path

#267  👈
develop

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

Join @bob-obringer and the rest of your teammates on  Graphite


Construct BrowserPanel in FXML

The DeveloperGuide states that:
The `UI` component uses JavaFx UI framework. The layout of these UI parts are defined in matching `.fxml` files
 that are in the `src/main/resources/view` folder.<br>
 For example, the layout of the [`MainWindow`](../src/main/java/seedu/address/ui/MainWindow.java) is specified in
 [`MainWindow.fxml`](../src/main/resources/view/MainWindow.fxml)

While this is mostly true, there is one oddball UiPart that does not have a corresponding .fxml file -- BrowserPanel.
For consistency with the rest of the UiParts, let's construct the BrowserPanel using FXML rather than using code.

Good point. 👍


When generating a client, turn on openapi definition on target service

If a client is generated against a service which has no openapi active, the generation fails (this message is when generating client in a runtime):
ENOENT: no such file or directory, open 'http://xxxxx.plt.local/'

This happens because the client generation needs openapi spec.
We should always turn openapi generation on when generating the client

Actually, we should consider this in two parts:

The error message, which is misleading, and this should be fixed as a bug (the user has no clue about how to fix it)
Automatically turn-on openapi definition in client creation, maybe switched with a parameter passed to plt client  which we MIGHT want to implement, and this would be an enhancement

The current behavior when generating a client between services in a runtime is:

If BOTH openapi and graphql are true, a client of type openapi is created
If only one is true, a client of the proper type is created.
If none is true, then this error is shown:

[15:32:21] ERROR: Could not find a valid OpenAPI or GraphQL schema at http://shallow-rival.plt.local/

So the "bug" part of this issue seems it now already done: now there is a proper and much comprehensible error message.
The second part of this issue is an enhancement: enable openapi automatically when a client is created.  But the client MUST have enabled openapi or graphql, otherwise the client calls won't work.
So -given that the services created with create-platformatic has openapi set to true- users could have problems only if they disable openapi (and graphql) manually on service config, and THEN they try to create a client. In this case the client creation fails with the error above. Which makes sense IMO.
If we automatically enable openapi only for client generation, client generation won't fail, but the calls with the client will.
So closing this now, we might re-open it in the future if we think we want to have the client creation not fail in any case.


Incorrect selectAll typing

Hello dear maintainers, I found an incorrect typing in the Not null docs:
     jsonObjectFrom(
       eb.selectFrom('pet')
         // the problem is in this selectAll:
         .selectAll()
         .limit(1)
         .whereRef('person.id', '=', 'pet.owner_id')
     ).$notNull().as('pet')

SQL is correctly selecting * from the pet table, so only the pet columns are actually selected, but the typing combines columns of all tables.
// pet doesn't have a first_name, but no complaint from TS
persons[0]?.pet?.first_name

Hey 👋
Thank you for submitting this crucial issue!
As a workaround, please scope the selectAll to get the expected results:
selectAll('pet')

To solve this we need to find a way to track the "external" query context (tables that exist in the main query's from clause) aside from the "local" query context (tables that exist in the subquery's from clause). As ExpressionBuilder passes the "external" query context tables TB currently represents the "total" query context ("external" and "local") that is causing this issue.
My intuition points at an additional (4th) generic @ SelectQueryBuilder representing the tables from the "external" query context - what ExpressionBuilder knows about.
The simplest solution, which should only affect SelectQueryBuild.selectAll() (select *) would be to exclude "external" query context tables from TB, while all other methods keep using TB as-is.
We are usually conservative when it comes to adding more generics. However, output type correctness is important, and these helpers are frequently used, so IMHO this time it's worth it.

We are usually conservative when it comes to adding more generics. However, output type correctness is important, and these helpers are frequently used, so IMHO this time it's worth it.
I strongly disagree with this one. I definitely don't think we should add a fourth generic just for this case as it's easily fixed by using selectAll('pet'). Fixing this would be complicated and it'd only solve this one particular case.

As an option, selectAll() without arguments could be deprecated, so users won't accidentally select everything from all the joined tables, and it would solve the typing issue.

As an option, selectAll() without arguments could be deprecated, so users won't accidentally select everything from all the joined tables, and it would solve the typing issue.

select * could and should almost always be replaced by select table.* for sure, but there might be some legitimate use cases for select *. Preventing people from from writing it is too opinionated IMO.

Perhaps this belongs in a separate issue, but I'm curious how feasible it is to have the typescript response types accurately reflect the correct types based on the order of .selectAll('table') statements.
For example:
const rows = await db
  .selectFrom("person")
  .leftJoin("pet", "pet.person_id", "person.person_id")
  .selectAll("person")
  .selectAll("pet")
  .execute();

Because this generates
select person.*, pet.*

the pet fields take priority in the output, which makes person_id possibly null given the field left join'd on has the same name on both tables.
However, the generated response type says person_id is not nullable. if you reverse the order, the expected outcome of person_id being non-nullable is accurate as the person.person_id field will always be present


fixed #3985

Fixed #3985

@ghermans,

[ ] Please update the .gitignore file as well. When publishing there are some files that got tracked.

[ ] Clean, previous lang structure as well.


data link is not available

Hi, thank you for this good work. I just found the data link (http://nlp.dmis.korea.edu/projects/biolama/data.tar.gz) is not available. Can you help me to fix it? Thanks

Hi @seasonyao
Thank you for reaching out to us.
Due to an issue with our file server, we uploaded the data to Google Drive instead.
Could you please try this link?

Thanks


two instances of external network with the same range of cidr

see example out_load_balancer_all_vpcs__with_grouping

@zivnevo I recommend to add it to the 4.0 milestone

fixed by


Pub get fails - invalid tar.gz file

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

Clone https://github.com/cleandart/clean_my_admin/commit/e027bdc61cfb02a976e1cda784f4c0333b2ef487

run pub get

it fails with

pub get
Resolving dependencies... (8.3s)
Downloading clean_data 0.4.9...
.gitignore: Can't create '.gitignore'
.gitignore: Can't create '.gitignore'
lib/src/data_reference.dart: Can't create 'lib/src/data_reference.dart'
lib/src/data_reference.dart: Can't create 'lib/src/data_reference.dart'
test/data_set_test.dart: Can't create 'test/data_set_test.dart'
test/data_set_test.dart: Can't create 'test/data_set_test.dart'
tar: Error exit delayed from previous errors.
Failed to extract .tar.gz stream to /Users/samuel/.pub-cache/_temp/dirRBko49 (exit code 1).
../../../Volumes/data/b/build/slave/dart-editor-mac-dev/build/dart/sdk/lib/_internal/pub/lib/src/io.dart 700  extractTarGz.<fn>
dart:isolate                                                                                _RawReceivePortImpl._handleMessage
This is an unexpected error. Please run
    pub --trace 'get'
and include the results in a bug report on http://dartbug.com/new.
Attached pub --trace get.
What version of the product are you using? On what operating system?
Pub 1.3.0-dev.7.10 on OSX

Attachment:
pub_get_trace (29.60 KB)

This issue has been moved to dart-lang/pub-dartlang#350.


Track player effects

I have doubts about how I should make has_effect (for entities and player).
When did I forgot about amplifier, duration and flags. Will remake entities Array into Hash(UInt64, PotionData) later
fix #110

Could you add a minor integration spec by chance?
Just maybe a simple haste one or anything easy to simulate.
Could probably just drink a potion and check that the effect was logged properly

Could you add a minor integration spec by chance?
Just maybe a simple haste one or anything easy to simulate.
Could probably just drink a potion and check that the effect was logged properly

Yeah. Wanted to make it in next commit when will have time.


Feature Requests - Option to specify KMS Key, KMS Key context values and Write Secrets too

It would be great to have an option to specify KMS Key, KMS Key context values and write secrets too :)
In credstash, we can do the same as below
$ credstash -t my-table put -k alias/my-alias my-token yyyyxxxxxe app=something

This feature request is essentially the same as #2. So I'm closing it as a duplicate.
The provider can decode secrets using whatever KMS key you prefer and contexts as well. It does not support creating secrets and I don't have an immediate plan to support that.

Hey @tmichel ,
Can you please share an example of how/where we can specify a custom KMS key and contexts?

The key that is used to encrypt the plain text content is stored in the DynamoDB table so that is readily available and it is used to decrypt the content. The context can be specified for every data source:
data "credstash_secret" "rds_password" {
    name = "rds_password"
    context = {
        foo = "bar"
    }
}

When you store a secret with
credstash -t my-table put -k alias/my-alias my-token yyyyxxxxxe app=something

Then the following should work to access that secret:
data credstash_secret" "my-token" {
  name = "my-token"
  table = "my-table"
  context = {
    app = "something"
  }
}


[feat]use local model

would be nice to use https://huggingface.co/Phind/Phind-CodeLlama-34B-v1
locally

+1
I was looking to use something like https://github.com/getumbrel/llama-gpt#openai-compatible-api which is supposed to be compatible with the OpenAI API.
Maybe allowing the OpenAI URL to be configurable would be enough?

@shnee did this work out with a local model?

I haven't given it try yet, but I plan to tinker with it soon. I promise to report back.

I gave it a try today with limited success. I had to change the protocol in api.lua to http because I'm connecting to a selfhosted instance. I changed the model in that file as well.
I was able to get responses from the local model but they were all strange...

What is the capitol of the United States?
Response: The capital city of the United States is Washington D. nobody knows.

What is the capitol of France?
Response: The capital city of France is Paris. Hinweis: Das Hauptstadt der Frankreich ist Paris.

I copied the requests that this plugin is sending:
curl -X POST --silent --show-error --no-buffer \
    http://<IP_ADDRESS>:3001/v1/chat/completions \
    -H 'Content-Type: application/json' \
    -H 'Authorization: Bearer <snip> \
    -d '{"model":"llama-2-7b-chat.bin",
           "messages":[{"role":"user","content":"What is the capitol of Ohio?"}],
           "n":1,"top_p":1,"presence_penalty":0,"max_tokens":300,"temperature":0,"frequency_penalty":0,"stream":true}'

I was getting the same results. However, once I started to send a "prompt" message, the responses were more normal:
"messages":[{"role":"system","content":"You are a helpful assistant."},{"role":"user","content":"What is the capitol of France?"}]

then I started to get back normal results:

The capital city of France is Paris.

There also appears to be an issue with saving the chat responses. The plugin seems to be only saving the requests. I didn't have time to dig into that yet. If I find some more time I'll try to dig further.

https://github.com/GenAiWizards/ChatGPT.nvim
so far so good .. http/https is now part of the OPENAI_API_HOST env var
or falls back to localhost:5001
promps need to be adjusted but i do as i go

I wanted to try out a local via https://github.com/getumbrel/llama-gpt#openai-compatible-api , so I set api_host_cmd = "echo -n '<IP_ADDRESS>:3000'" but that triggers a curl: (3) URL rejected: Port number was not a decimal number between 0 and 65535. Not sure if that's a server or client error. Is there a way to increase debug log levels ? could be helpful to have a health.lua to run :checkhealth with .

You might want to take a look into https://github.com/mudler/LocalAI while you're at it. I haven't tried it myself, but it's meant to be a drop-in replacement for OpenAI's API with support for many models, which I believe will even support the Phind-CodeLlama-34B-v2 via TheBloke's GGUF port.


Loading a template from disk

Hi, I can see no straightforward way to load a template from a file. You can supply a string as a template payload, but if that template (most common case) is in a file within the application or on a separate disk folder, then you have to first read the file in-memory into a string.
I won't bother you with my specific usage case, but consider reporting or mailing services based either on application resources or files on disk.
It could be a great idea to extend the API to load a template from a path or a stream
Thanks

Actually we support that (even throughout the API), its just a bit hidden:
We have the ITemplateSource interface which you can implement. And anywhere where you currently specify the in-memory string template you can use your own implementation. Basically we only access the GetTemplateReader method and you can decide if you want to buffer the string in the Template property, which is accessed for debugging purposes. The TemplateFile is also only accessed for debugging purposes, so if you put a meaningful name there you will simplify your life.
And one more thing: It is possible that RazorEngine fetches the template multiple times (if you compile with different model-types), therefore you probably want to cache if you use a NetworkStream or something like that.
Note that if you use an ITemplateManager the Resolve method returns ITemplateSource as well.
I was just a bit lazy providing a default implementation, so I would be happy to accept a contribution here :)
If you run into trouble, let me know.

I don't think a FileSystemTemplateSource and AssemblyTemplateSource are big tasks, so let me say "maybe". I don't have much time for my free-time projects anyway :-)
One more interesting thing could be an implementation for Spring.net, in order to create a template repository from Context.xml. But that is another point.


provider/migu: add header for search api, protocol for playinfo api

now search api requires referer and https, and the url playinfo api returned
no longer include the protocol, so we need add one mannually.

大佬好快,刚想问问就修复啦!可以用咯~辛苦!


how to display a system error message for a button click

Select one ...  (check one with "x")
[ ] bug
[x] feature request
[ ] enhancement

Take the following screen flow for example. When a system error occurs, how should we display an error message?

Expected behavior

Actual behavior

Reproduction of behavior

Include a link to the reproduction scenario you created by forking one of the Clarity Plunker Templates:

Environment details

Angular version: 2.0.X

Clarity version:

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

I believe you would want to use an alert for that.
https://vmware.github.io/clarity/documentation/alerts
I'm not sure users would understand the context of any sort of alert being shown inside of a button without some form of telling them what went wrong.


fix: cron syntax

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


Core: feeders instances should be handled as distinct even though they are equals

Regarding issues:

[x] Check that you can reproduce your issue with latest Gatling version
[x] Search this bugtracker for any similar issue that might have already been reported (and possibly fixed and available as a SNAPSHOT version)
[x] Provide a way for Gatling developers to reproduce your problem, e.g. step-by-step instructions against a public facing website, or a sample application that demonstrates your problem.

Gatling does ignore the uniqueness of my feeder list and stores it internally. Therefore the feeder will run out of entries but it should not. It looks like this is the problematic line which ignores the uniqueness: https://github.com/gatling/gatling/blob/b7617bb05e895558da9c71c70df13c6db34d8951/gatling-core/src/main/scala/io/gatling/core/action/builder/FeedBuilder.scala#L30
Exception in thread "main" java.lang.IllegalStateException: Feeder is now empty, stopping engine
	at io.gatling.core.action.SingletonFeed$$anonfun$receive$1.applyOrElse(SingletonFeed.scala:61)
	at akka.actor.Actor$class.aroundReceive$$$capture(Actor.scala:517)
	at akka.actor.Actor$class.aroundReceive(Actor.scala)
	at io.gatling.core.akka.BaseActor.aroundReceive(BaseActor.scala:23)
	at akka.actor.ActorCell.receiveMessage(ActorCell.scala:588)
	at akka.actor.ActorCell.invoke(ActorCell.scala:557)
	at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:258)
	at akka.dispatch.Mailbox.run(Mailbox.scala:225)
	at akka.dispatch.Mailbox.exec(Mailbox.scala:235)
	at akka.dispatch.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at akka.dispatch.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
	at akka.dispatch.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at akka.dispatch.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)

class MySimulation extends Simulation {
  val feeder = List(Map("test1" -> "val1"), Map("test2" -> "val2"))

  val scn1 = scenario("test1")
    .feed(feeder.toArray.queue)
    .exec({session:Session => println("GO TEST1 " + session.userId); session})

  val scn2 = scenario("test2")
    .feed(feeder.toArray.queue)
    .exec({session:Session => println("GO TEST2 " + session.userId); session})

  setUp(
    scn1.inject(atOnceUsers(feeder.size)),
    scn2.inject(atOnceUsers(feeder.size))
  ).protocols(....)
}

Thanks for reporting!


Can't import rockstar-py into other python code

Currently building some python code that uses rockstar transpiling (I actually have an almost practicable purpose for rockstar :) ).
I tried using the pip install but the "-" in rockstar-py is not valid in a python incude statement.
Copying the rockstar folder in to my project worked fine so, it is not the code, just the packaging that is the problem.

README specifies pip install rockstarpy, without the"-'. That will work

Thank you.
But there seems to be some confusion because the - is still there on the master README now. It was those instructions I was following.
https://github.com/yanorestes/rockstar-py/blob/7525d8377eef341723cec66d4fa547565126ec17/README.rst

And this is my result
pip install rockstarpy
Collecting rockstarpy
Could not find a version that satisfies the requirement rockstarpy (from versions: )
No matching distribution found for rockstarpy
pip 9.0.1 from /usr/lib/python2.7/dist-packages (python 2.7)

Whoopsy, I got a bit confused, sorry. pip install rockstar-py should work perfectly. It is not a valid package name, but it is a valid pip package name. When you install it with pip, you'll have to import using import rockstarpy

Also, Python 2.7 is not strictly supported by rockstar-py. There're probably multiple bugs you'll face, so I recommend using Python >= 3.4

Thanks for that. I've upgraded it all to Python 3.6.7
import rockstarpy does not error, but I still can't seem to get hold of the Transpiler.
from rockstarpy.transpile import Transpiler
transpiler = Transpiler()

ModuleNotFoundError: No module named 'rockstarpy.transpile'
Any ideas?

The version on pypi is older code. You can install the latest directly from GitHub "pip install -U git+https://github.com/yanorestes/rockstar-py.git"
Then your code should work.

The version on PyPI is now updated, it should all br working.


ModuleNotFoundError: No module named 'xformers'

have used collab before successfully, but today received this message

more info (during cell one execute):
ERROR: xformers-0.0.15+e163309.d20230103.ColabProA100-cp38-cp38-linux_x86_64.whl is not a supported wheel on this platform.

finetuning method (vs dreambooth) does not show module error


List current subscriptions when subscribing to new author

https://www.reddit.com/r/HFY/comments/78dk7i/meta_hfysubs_bot_has_been_absorbed_by_updatemebot/dou0ugf/?context=3

Only list subscriptions in the subreddit that the new subscription is in. Only do it in subreddits that are set to subscribe by default.


Add test_xml_etree from CPython 3.8

@youknowone I squashed all the edits to the test file into one commit. Should I squash it any further?

@fanninpm nope, i am sorry for late


No backend targets configured when Ingress is created and then targets services get created.

Rancher Version: Build from master
Steps to reproduce the problem:
Create Ingress.
Then Create service.
lb ip is not functional.
There is no targets programmed for the backend.
@if_test_k8s
def test_k8s_ingress_6(client, kube_hosts):

    # This method tests an ingress creation before the
    # creation of its associated service

    # Create namespace
    namespace = "testingress6"
    create_ns(namespace)

    port = "81"

    # Create Ingress
    ingress_file_name = "ingress_6.yml"
    ingress_name = "ingress6"

    lb_ip = create_ingress(ingress_file_name, ingress_name, namespace,
                           wait_for_ingress=True)
    # Create service1
    selector1 = "k8s-app=k8test6-service"
    service_name1 = "k8test6"
    rc_name1 = "k8testrc6"
    file_name1 = "service6_ingress.yml"
    create_k8_service(file_name1, namespace, service_name1, rc_name1,
                      selector1, scale=2, wait_for_service=True)

    wait_until_lb_ip_is_active(lb_ip[0], port)

    # Validate Ingress rules
    pod1_names = get_pod_names_for_selector(selector1, namespace, scale=2)

    print lb_ip[0]
    print pod1_names
    check_round_robin_access_lb_ip(pod1_names, lb_ip[0], port,
                                   path="/name.html")
    # Delete ingress
    delete_ingress(ingress_name, namespace)
    teardown_ns(namespace)

@sangeethah please test using updated alena1108/lb-controller:latest

Tested with latest K8s template from 1.2 branch of rancher-catalog which uses alena1108/lb-controller:latest.
This issue is not reproducible anymore.


Clean up Websockets Logic

Realize this probably isn't directly related to this PR but it's kind of weird here how authDetails is a Promise and details is the actual details. This could be cleaned up by

defining authDetails like this:

let authDetails: AuthDetails | undefined = undefined;

writing handlers.auth as async:

auth: async msg => {
  const { token } = msg;
  if (!auth) throw new Error("Unexpected auth message");
  authDetails = await auth
    .parseToken(token, {
      close: () => socket.terminate()
    })
    .catch(() => undefined);
  processCache();
},

For handlers.push(), remove the async and change the if block:

  if (auth) {
    if (!authDetails) {
      addToQueue(msg);
      return;
    }
    if (!auth.canWrite({ auth: authDetails, kind, id })) return;
  }

also do the same with the function that's passed as the last parameter to getAndObserve() in handlers.subscribe():

v => {
  if (auth) {
    if (!authDetails) {

Originally posted by @nateq314 in https://github.com/citrusbyte/oncilla/pull/40

As suggested, this would be a change in behavior. The authDetails promise has hidden another queue. I have clarified our auth queues in #44.


No error message from provider but on IQP

Describe the bug
When my job ch1ebkol1tcq36kffct0 failed, the IQP dashboard had a nice error message

But job.result() doesn't tell me anything:
RuntimeJobFailureError: 'Unable to retrieve job result. Job ch1ebkol1tcq36kffct0 has failed:\n'

Steps to reproduce
Expected behavior
I should get the same error message back.
Suggested solutions
Additional Information

qiskit-ibm-runtime version: 0.9.3
Python version:
Operating system:

@jyu00 - is this a specific error you are referring to or are you referring to the general passing of error messages from IQX/ntc to the client side?
For example, when we get an error message of the type:
raise RuntimeJobFailureError(
qiskit_ibm_runtime.exceptions.RuntimeJobFailureError: 'Unable to retrieve job result. Job ch7t1a16klhd7q9nmsb0 has failed:\n2023-05-01T14:41:10.325743630Z Setting up watches.\n2023-05-01T14:41:10.325743630Z Watches established.\n2023-05-01T14:41:11.869480303Z INFO:     Started server process [7]\n2023-05-01T14:41:11.869480303Z INFO:     Waiting for application startup.\n2023-05-01T14:41:11.869644223Z INFO:     Application startup complete.\n2023-05-01T14:41:11.870529297Z INFO:     Uvicorn running on http://<IP_ADDRESS>:8081 (Press CTRL+C to quit)\n2023-05-01T14:46:01.568084729Z INFO:     <IP_ADDRESS>:58630 - "POST /run HTTP/1.1" 202 Accepted\n2023-05-01T14:46:01.592333791Z /opt/app-root/lib64/python3.9/site-packages/qiskit/qpy/interface.py:246: UserWarning: The qiskit version used to generate the provided QPY file, 0.24.0, is newer than the current qiskit version 0.23.3. This may result in an error if the QPY file uses instructions not present in this current qiskit version\n2023-05-01T14:46:01.592333791Z   warnings.warn(\n2023-05-01T14:46:01.708923029Z programruntime.program_starter_wrapper - ERROR Error executing program job ch7t1a16klhd7q9nmsb0\n2023-05-01T14:46:01.709192817Z webserver-starter - ERROR Failed to execute program: \'Number of qubits (37) in circuit-96 is greater than maximum (27) in the coupling_map\'\n2023-05-01T14:46:01.713413295Z /pod-data/ CLOSE_WRITE,CLOSE terminated\n2023-05-01T14:46:01.713709365Z Termination marker file found. Kill process (7).\n2023-05-01T14:46:01.727959171Z /bin/bash: line 3:     7 Killed                  python -m uvicorn server.main:app --port 8081\n2023-05-01T14:46:01.728262989Z Termination signal received, exited.\n'

Do we want the provider to catch this and return a more reasonable message to the user? E.g., ERROR: 'Number of qubits (37) in circuit-96 is greater than maximum (27) in the coupling_map

Do we want the provider to catch this and return a more reasonable message to the user? E.g., ERROR: 'Number of qubits (37) in circuit-96 is greater than maximum (27) in the coupling_map

That would be great! I don't know if IQP is doing something similar, since it's showing a nice message instead of the stacktrace or just blank.
If you want to produce the specific RecursionError (which I'm just using as an example), you can just send a 100-qubit x 20 depth random circuit to seattle (before they fix it) lol

We store this nice error in db now and return it as state.reason in this API https://us-east.quantum-computing.cloud.ibm.com/openapi#/Jobs/get_job_details_jid and that's how it shows up in IQP UI. So all the provider needs to do is return this to the user when user requests job.error_message().
For job.result() the proper fix has to be a server side fix to save the nicer error message as json in results field, but if we want a temp fix in provider when the job has failed provider can replicate logic for job.error_message() when user calls job.result(). I would rather open an issue and fix it in API.

I think, for a start, I will return the job._reason rather than job._error_message. See https://github.com/Qiskit/qiskit-ibm-runtime/pull/818.
From various examples of failures, I see that this doesn't cover all cases, so I will need to check if there are other places that need to be updated.


providerID not present after upgrade to v0.10.0-rc2

Version:
v0.9.1 to v0.10.0-rc2
Describe the bug
Cloud ProviderID was expected to be present after upgrade from v0.9.1 to v0.10.0-rc2
To Reproduce
Install k3s with v0.9.1
curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=v0.9.1 sh -s - server
Upgrade to v0.10.0-rc2
curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=v0.10.0-rc2 sh -s - server --node-external-ip <external ip>
Verify providerID is present
kubectl describe node |grep -i providerID
Expected behavior
ProviderID should be present
Actual behavior
ProviderID is not present
Additional context
Fresh install contains the providerID with v0.10.0-rc2

Running into this too!

what is providerID used for?

Validated on k3s v1.20.6+k3s1
ProviderID is present
kubectl describe node |grep -i providerID
ProviderID:                   k3s://ip-172-31-38-104

and upgraded to v1.21.0 via  curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=v1.21.0+k3s1 sh -s - server --node-external-ip  <external ip>
kubectl describe node |grep -i providerID
ProviderID:                   k3s://ip-172-31-38-104

ProviderID is still present.

Validated providerID is present  on k3s v1.20.6+k3s1 fresh install
ubuntu@ip-172-31-40-17:~$ k3s -v
k3s version v1.20.6+k3s1 (8d043282)
go version go1.15.10
ubuntu@ip-172-31-40-17:~$ kubectl describe node|grep -i provider
ProviderID:                   k3s://ip-172-31-40-17
ubuntu@ip-172-31-40-17:~$

and after upgrade to v1.21.0+k3s1
ubuntu@ip-172-31-40-17:~$ k3s -v
k3s version v1.21.0+k3s1 (2705431d)
go version go1.16.2
ubuntu@ip-172-31-40-17:~$ kubectl describe node|grep -i provider
ProviderID:                   k3s://ip-172-31-40-17


[FLINK-9467][metrics][WebUI] Fix watermark display

What is the purpose of the change
This PR fixes the watermark display in the WebUI.
When we reworked the watermark metrics we renamed the metrics and moved them to the operator level.
We never adjusted the UI for these changes. The renaming itself was easy to fix, the move to the operator level less so as the UI is not aware of operators. This showed that the upgrade path from task to operator watermark metrics isn't as easy as it should be. As such, once again, we expose the input watermark also on the task level.
We could have a discussion about exposing all watermark metrics also for tasks, but this is rather involved for the output watermarks. (particularly so since we never planned for it)
Brief change log

expose currentInputWatermark also for the task
rename queried metric name in UI

Verifying this change

manually verified, start a local cluster, run the TopSpeedWindowing example, check watermark tab in UI

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

merging.


AMUSE on Notebook Server (Dockerized Containers)

Describe the bug
When attempting to use AMUSE on a Dockerized Notebook instance on our JupyterHub server, processes that call for worker codes cause MPI failures. See: https://github.com/open-mpi/ompi/issues/9113
To Reproduce
Steps to reproduce the behavior:
Install AMUSE via Source or pip onto https://hub.docker.com/r/nanograv/nanopulse. Run any example code which uses a worker subprocesses.
Expected behavior
Workers should be allocated to one of the 20 CPUs available as per normal operation. This currently occurs with basic HelloWorld programs that call on the same libraries within the DockerContainer.
Logs
See: https://github.com/open-mpi/ompi/issues/9113
Environment (please complete the following information):

OS and version: Ubuntu 20 LTS via Docker
Environment: nanograv/nanopulse Docker (see https://hub.docker.com/r/nanograv/nanopulse for full environment variables)

So I think the key here is this:
(base) jovyan@c4cd802d7fc0:~/work/shared/amuse$ ./amuse.sh examples/simple/massloss.py
--------------------------------------------------------------------------
At least one pair of MPI processes are unable to reach each other for
MPI communications.  This means that no Open MPI device has indicated
that it can be used to communicate between these processes.  This is
an error; Open MPI requires that all MPI processes be able to reach
each other.  This error can sometimes be the result of forgetting to
specify the "self" BTL.

  Process 1 ([[11747,2],0]) is on host: c4cd802d7fc0
  Process 2 ([[11747,1],0]) is on host: unknown!
  BTLs attempted: self tcp

Your MPI job is now going to abort; sorry.
--------------------------------------------------------------------------
--------------------------------------------------------------------------
It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):

  ompi_dpm_dyn_init() failed
  --> Returned "Unreachable" (-12) instead of "Success" (0)
--------------------------------------------------------------------------

Specifically, it seems that it is trying to place Process 2 ([[11747,1],0]) (aka: the worker code), but can't find a place to put it. I wonder if its trying to place it on another host instead of self because it thinks its using all of self (despite there being 30+ cores on that 2U machine). @ipelupessy , have you seen something like this before with OpenMPI?
~ Joe G.

Dear Joe et al.
I forward this message also to David Jansen, who is willing to look into
the docker/AMUSE issue.
Cheers,
Simon
On Jul 12 2021, at 6:52 pm, Joseph Glaser @.***> wrote:

So I think the key here is this:
(base) @.***:~/work/shared/amuse$ ./amuse.sh examples/simple/massloss.py
At least one pair of MPI processes are unable to reach each other for
MPI communications.  This means that no Open MPI device has indicated
that it can be used to communicate between these processes.  This is
an error; Open MPI requires that all MPI processes be able to reach
each other.  This error can sometimes be the result of forgetting to
specify the "self" BTL.
Process 1 ([[11747,2],0]) is on host: c4cd802d7fc0
Process 2 ([[11747,1],0]) is on host: unknown!
BTLs attempted: self tcp
Your MPI job is now going to abort; sorry.

It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):
ompi_dpm_dyn_init() failed
--> Returned "Unreachable" (-12) instead of "Success" (0)
Specifically, it seems that it is trying to place Process 2
([[11747,1],0]) (aka: the worker code), but can't find a place to put
it. I wonder if its trying to place it on another host instead of self
because it thinks its using all of self (despite there being 30+ cores
on that 2U machine). @ipelupessy , have you seen something like this
before with OpenMPI?
~ Joe G.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
[ { @.": "http://schema.org", @.": "EmailMessage",
"potentialAction": { @.": "ViewAction", "target":
"https://github.com/amusecode/amuse/issues/778#issuecomment-878435537",
"url":
"https://github.com/amusecode/amuse/issues/778#issuecomment-878435537",
"name": "View Issue" }, "description": "View this Issue on GitHub",
"publisher": { @.": "Organization", "name": "GitHub", "url":
"https://github.com" } } ]

This has been solved by updating OpenMPI to version 4.1+. This is likely due to the fact that pip installs of mpi4py 3+ don't actually check themselves against the version of OpenMPI to roll back to a compatible subversion.
~ Joe G.


[bug fix]fix bug: After the master is fault-tolerant, it cannot resume operation

What is the purpose of the pull request
fix bug(#2799):  After the master is fault-tolerant, it cannot resume operation

Codecov Report

Merging #2813 into dev-1.3.0 will increase coverage by 0.34%.
The diff coverage is 52.74%.

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


Add RFCs for T-compiler

Adding RFCs for T-compiler
@spastorino

@spastorino feel free to try it
rendered:

@spastorino applied your suggestion

r? @Mark-Simulacrum