MANGO-474. Fix OPENSSL handshake pipeline.

MANGO-474. Fix OPENSSL handshake pipeline.

Codecov Report

Merging #206 (a13df9e) into develop (72c0a30) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #206   +/-   ##
========================================
  Coverage    71.37%   71.37%
========================================
  Files          243      243
  Lines         8691     8691
  Branches       115      115
========================================
  Hits          6203     6203
  Misses        2449     2449
  Partials        39       39

Flag
Coverage Δ

web
71.37% <ø> (ø)

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

Continue to review full report at Codecov.

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


Added highlight matched text

Description
Added highlight matched text
Checklist

[x] All tests passing
[x] Created tests which fail without the change (if possible)
[x] Extended the README / documentation, if necessary
[x] Commits and PR follow conventions

Thanks for updating! Some minor issues, though. 😬

@ro-ka  Done. Also I squashed all commits in a single commit)

Thanks!


Add Firefox support

Hi,
Could you please start supporting Firefox?

It is supported on my setup:

It is supported on my setup:

Ok, weird. Must be related to some privacy settings in uBlock or Firefox that I've added. Thanks for confirming that it works.


Expand in-code documentation to ease potential contributions

[ ] Added some Black formatting to the files
[ ] Added inline comments to the code explaining what it does in each step, to ease potential contributions
[ ] Change name of extract_publications_text in the test files (from ept to full name extract_publications_text to make it easier to understand what's going on = see also #75)

Contributes to #72.
This was a step in the direction for me to wrap my head around what the code does and where. I want to see if I can take a stab at contributing something for #30 next.

Don't know why black fails here since I'm formatting with black...?


Null safety

Can we please migrate to null-safe?

@leocavalcante Is there any timeline for this?

+1

+1

+2

+3

I created the PR https://github.com/leocavalcante/page_view_indicator/pull/22 migrating the code to null safe

I also created the package https://pub.dev/packages/page_view_indicator_ns which is a fork from this package just migrated to null safety. I have no plans to maintain this package. As soon as null safety lands on the original package, i'll abandon this one i made.


Support Kerberos authentication with service discovery

Support Kerberos authentication with service discovery. For a smooth rollout, this requires two authentication strategies in the client until we can deprecate the old version.
The existing clients are latched to DPM01 by default. We can deploy a new incremental update that has the principal hardcoded for DPM01 and supports the new message type that tells the client what principal to use.

So the plan is

Do service discovery to pick a DPM
When Kerberos is needed, start the exchange
If Authenticate_reply is received, continue successfully. Otherwise, if Status_reply is received, switch over to DPM01 and hardcode the service name.

The "old" protocol is no longer used.


Clean up the default wp_nav_menu output?

This frameworks really goes a long way to clean up the default ugly wordpress code. It would be nice to have the option of cleaning up the messy of code that wordpress outputs for nav menus too. I found some guys solution online. Perhaps the code can be cleaned up and put into roots. This code basically strips out the default verbose classes and ids but puts in a current class for the current page.
add_action('wp_loaded','register_nav_menu_class');

function register_nav_menu_class(){
    class agile_walker extends Walker_Nav_Menu  {
        function start_lvl(&$output, $depth) {
            $indent = str_repeat("\t", $depth);
            $output .= "\n$indent
\n";
        }
        function end_lvl(&$output, $depth) {
            $indent = str_repeat("\t", $depth);
            $output .= "$indent
\n";
        }
    }
}

add_filter('nav_menu_css_class','strip_classes');

function strip_classes($a){ return (in_array('current_page_item',$a))? array('active') : array(); }

add_filter('nav_menu_item_id','strip_id');

function strip_id(){ return ''; }

One problem with this code is it leaves the class attributes set but empty.

@benklocek thanks, can you please submit a pull req?

Done. https://github.com/retlehs/roots/pull/229

It's not really a bug. I was just being overly strict on what classes should be there by default but you're right that we should probably some other current* classes.


IE11: Invalid Calling Object

I'm having an issue on IE11 when calling process in timers-browserify I get the error "Invalid Calling Object" in the method process.nextTick().

Would anyone be able to shed some light on this one? It's extremely difficult to debug in IE, but cachedSetTimeout and drainQueue are both defined.

In 2.0.0, we removed process.nextTick from this module. If you use a more recent version, does that solve the issue?

No reply in a while here. If this is still an issue, please let me know.


Traction sandbox reset/redeploy automation

We want the traction sandbox instance to be reset and re-deployed, using the latest published chart, periodically.
The suggested approach is to implement a Github Action (to be stored in https://github.com/bcgov/trust-over-ip-configurations) that will install/uninstall/re-install the sandbox on a schedule.
We have Github actions completing similar work (with different triggers) in the Traction repo (see https://github.com/bcgov/traction/blob/main/.github/workflows/on_pr_closed.yaml and https://github.com/bcgov/traction/blob/main/.github/workflows/on_pr_opened.yaml as examples).
Acceptance Criteria:

[x] A Github action is created. The action will uninstall the helm chart and any leftover deployment artifacts, and re-install it periodically.
[x] The trigger should be set to run every two weeks, on the night between Sunday and Monday at 2am PST

@rajpalc7 checking-in about this. A n updated chart was released so you should be able to proceed and wrap this up as well as #115.

Hi @esune, I was able to make github actions work to uninstall and install the release now. The new release is working fine now. I am going to close this ticket  in favour of #123


Kafka <IP_ADDRESS>

NetflixOSS » suro » suro-pull-requests #4 FAILURE
Looks like there's a problem with this pull request

suro-pull-requests #170 FAILURE
Looks like there's a problem with this pull request

let's not merge it until platform release is ready. today they just released platform with major aws-java-sdk jars. it's better to wait for a week to make sure it stabilizes.

@stevenzwu Can you do same editing in master branch? Please don't merge thrift-0.7.0 to the master branch.

@stevenzwu I guess that this PR can be closed, right?

suro-pull-requests #181 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » suro » suro-pull-requests #12 FAILURE
Looks like there's a problem with this pull request


[IMP] Add Pytorch solutions

Expanding on issue #26 it may be worth adding multiple solutions written using both tf/keras and pytorch. I would be happy to work on solutions to current exercises.

@Syn-Tax Great ! Please go ahead and proceed as you see fit. One organization comment: Since we already organize the current 2 solutions as titanic_classical and titanic_tf_nn it might make sense for you to create a notebook named titanic_pt_nn or something to that effect, and then update the readme.md.
Thanks and look forward to your contribution !


Infinite loop documentation for language server

For the language server installation (https://github.com/reasonml-editor/vim-reason-plus#language-server-installation) there is this comment:

See https://github.com/jaredly/reason-language-server#vim for language-server installation and configuration.

The reason-language-server documentation suggests:

Install the reason-vim plugin, following the readme.

So I can't figure out how you're supposed to install the two together

also cannot figure out where
/absolute/path/to/reason-language-server.exe'
is, reason-language-server#vim sends you back here
and https://github.com/jaredly/reason-language-server doesn't seem to offer any information for regular users rather than contributors.

@SeeThruHead The Vim section of the reason-language-server was missing a step that tells you to go to the releases page. I've submitted a PR to fix this. It's missing this line:

Download the your-platform.zip file from the latest release, unzip it, and put the reason-language-server.exe file somewhere.

The reason-language-server README has been updated now, so I think this issue is no longer relevant.


ADD: More Tableau Platform Properties

Requires shim library changes to work

Coverage increased (+0.03%) to 98.148% when pulling 56bb1427439e5649541e4c1966008f13c7ce2a5b on agreene/platform-props into eaee71e9eaf0836272102f770feb3ea0f47dd807 on dev.

This looks great! Will get the new shim published.

Implemented using the same pattern as the previous options. +1

Can you resolve the merge conflicts @jagreene ?

Coverage remained the same at 98.148% when pulling 4aef3743875b1ce7499e715e39b1c2d5bd8e0a99 on agreene/platform-props into b6952e1950ba6e185d429135a9f1d80893b2e4f9 on dev.


Keywords (reserve words ) in python

added required content in keyword

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hey @username
Closing this as a duplicate of PR #25949.
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Document release process

This is extremely simple, but documenting it aims to avoid having doubt about things that would be necessary but not documented.

Once this is approved/merged, I will cut a new release for 0.2.0.


API: Add Overload Manager  ResponseFlag.

Signed-off-by: Kevin Baichoo<EMAIL_ADDRESS>
Commit Message:  Add Overload Manager ResetReason and ResponseFlag.
Additional Description: For the changes needed in Issue #15791 we will have overload manager reset streams. Given that, we'd like to have a response flag for overload manager resets. This change defines that response flag.
Risk Level: Low
Testing:Unit Tests
Docs Changes: Included
Release Notes: Included
Platform Specific Features: N/A
Related Issue #15791

/assign @antoniovicente

You have test failures which seem related to this PR.
/wait
2021-06-18T18:30:14.7762567Z ==================== Test output for //test/extensions/access_loggers/grpc:grpc_access_log_utils_test:
2021-06-18T18:30:14.7763192Z [==========] Running 1 test from 1 test suite.
2021-06-18T18:30:14.7763881Z [----------] Global test environment set-up.
2021-06-18T18:30:14.7764642Z [----------] 1 test from UtilityResponseFlagsToAccessLogResponseFlagsTest
2021-06-18T18:30:14.7765238Z [ RUN      ] UtilityResponseFlagsToAccessLogResponseFlagsTest.All
2021-06-18T18:30:14.7766559Z TestRandomGenerator running with seed -1346820440
2021-06-18T18:30:14.7767412Z test/extensions/access_loggers/grpc/grpc_access_log_utils_test.cc:51: Failure
2021-06-18T18:30:14.7768045Z Expected equality of these values:
2021-06-18T18:30:14.7768573Z   common_access_log_expected.DebugString()
2021-06-18T18:30:14.7770982Z     Which is: "response_flags {\n  failed_local_healthcheck: true\n  no_healthy_upstream: true\n  upstream_request_timeout: true\n  local_reset: true\n  upstream_remote_reset: true\n  upstream_connection_failure: true\n  upstream_connection_termination: true\n  upstream_overflow: true\n  no_route_found: true\n  delay_injected: true\n  fault_injected: true\n  rate_limited: true\n  unauthorized_details {\n    reason: EXTERNAL_SERVICE\n  }\n  rate_limit_service_error: true\n  downstream_connection_termination: true\n  upstream_retry_limit_exceeded: true\n  stream_idle_timeout: true\n  invalid_envoy_request_headers: true\n  downstream_protocol_error: true\n  upstream_max_stream_duration_reached: true\n  response_from_cache_filter: true\n  no_filter_config_found: true\n  duration_timeout: true\n  upstream_protocol_error: true\n  no_cluster_found: true\n}\n"
2021-06-18T18:30:14.7773421Z   common_access_log.DebugString()
2021-06-18T18:30:14.7775880Z     Which is: "response_flags {\n  failed_local_healthcheck: true\n  no_healthy_upstream: true\n  upstream_request_timeout: true\n  local_reset: true\n  upstream_remote_reset: true\n  upstream_connection_failure: true\n  upstream_connection_termination: true\n  upstream_overflow: true\n  no_route_found: true\n  delay_injected: true\n  fault_injected: true\n  rate_limited: true\n  unauthorized_details {\n    reason: EXTERNAL_SERVICE\n  }\n  rate_limit_service_error: true\n  downstream_connection_termination: true\n  upstream_retry_limit_exceeded: true\n  stream_idle_timeout: true\n  invalid_envoy_request_headers: true\n  downstream_protocol_error: true\n  upstream_max_stream_duration_reached: true\n  response_from_cache_filter: true\n  no_filter_config_found: true\n  duration_timeout: true\n  upstream_protocol_error: true\n  no_cluster_found: true\n  overload_manager: true\n}\n"
2021-06-18T18:30:14.7778378Z With diff:
2021-06-18T18:30:14.7778774Z @@ +27,4 @@
2021-06-18T18:30:14.7779207Z    upstream_protocol_error: true
2021-06-18T18:30:14.7779671Z    no_cluster_found: true
2021-06-18T18:30:14.7780123Z +  overload_manager: true
2021-06-18T18:30:14.7780509Z  }\n

/retest


sell [amount] or quota always fail

Hi. I suppose this issue comes up in combination with other mods I have installed (018d2dad-094a-3f98-957e-29845528b6b4) but "sell [amount]" and "sell quota" always fail reporting there's not enough loot in the ship. Let me know if you need more information.

Can you identify which mod(s) are causing the problem?

I can try to in the next days and get back to you with my findings.

Does Commit 051f9ed fix this?

Ok, I will check and let you know. Just wasn't sure if the fix was directly related to this issue.

So, the good news is: Selling stuff works now (sell amount and sell quota behave the same in my testing), BUT not as expected :D
Here is the scenario: Quota is 130, deadline is reached (0 days left) and I go to the company moon to sell stuff.

Attached PNG (stormytuna.SellFromTerminal) shows my general MOD config.
Second attached PNG (BeforeSellLootScan) shows a scan of all my loot in the ship before selling.
Next PNG (BeforeSellQuotaTerminal) shows the terminal after typing "sell quota" and before confirm.
PNG AfterSellQuotaTerminal shows the terminal after confirmation of selling quota.
Last PNG (AfterSellQuotaScan) shows my ship loot after selling quota.

So, I automatically sold the laser pointer and an axle which are worth 151. There are probably tons of ways to calculate the quota (just think of the laser pointer and pickles) but 151 is out of reach and even beyond my config.
I hope I was more precise this time! :) Let me know if you need something else.

Yep, that was enough - it was an issue with how the allowance amount worked
Closing as fixed in 3ffde0296dc6b5d5282e4a2c932da5eec6d0f7e9


:sparkles: Questionnaire delete confirmation

closes https://github.com/konveyor/tackle2-ui/issues/1276
Adding an explanation of what happens if the questionnaire is deleted and forcing the user to write the questionnaire's name to deletion.
Note: the input is case-sensitive to the name of the questionnaire.
Before:

After:

Hi @sjd78
Thank you for taking the time to review this PR! :smile:

Issues I found:

The Cancel button on the delete confirm modal is never enabled, but the X icon and the Esc key do work.

That's not entirely accurate, the Cancel button is enabled only when submit is enabled as well, but I agree that that is not a wanted behavior, I should expose a new prop the ConfirmModal component to disable the submit only.

In other "type the name of the thing to delete" interactions I've used, the name of the thing is provided on the modal.  Right now, if I don't remember the exact full name of the assessment questionnaire, I need to close the confirm box copy the name, then go back in, then paste it.  Adding the name questionnaire being deleted to the confirm modal would help.

Agreed, will follow the example from Console "Delete project" modal:

@sjd78 I agree, I think there's a cleaner abstraction to be found here. I wonder if we are going to ever want another entity with type-the-name-to-confirm-deletion in this way? If so, maybe we should make a ConfirmDeleteModal or ConfirmDeleteWithNameModal that either simply wraps ConfirmDialog with the state/logic that is currently in the page here, or uses its own Modal instead of ConfirmDialog. Then we can have a ConfirmDeleteQuestionnaireModal that is a trivial usage of it, or just inline that usage in the page (since it would encapsulate all the state).

@mturley @sjd78 @ibolton336 Hi there
Can you please review it again?


Many scenarios fail due to Requested target does not receive transfers

Thanks for filing a bug report :-)
Steps to Reproduce

Expected Result

Actual Result
PFS1,3,6,9
PFS2,4,5,8
MS4
fail due to
replacement fee too low
replacement transaction underpriced
Seems like we are trying to send the transactions to the blockchain too fast in the case the nonce remains the same because the earlier transaction has not propagated and new transactions gas has to be 10% higher than the older ones.
source:
https://github.com/ethers-io/ethers.js/issues/444
https://ethereum.stackexchange.com/questions/67548/ethereum-transaction-fails-in-simultaneous-calls
2021-01-22T01:01:30.482Z [info] retryWhile: retrying {
  count: 0,
  interval: 5000,
  error: Error: replacement fee too low (error={"reason":"processing response error","code":"SERVER_ERROR","body":"{\"jsonrpc\":\"2.0\",\"id\":605,\"error\":{\"code\":-32000,\"message\":\"replacement transaction underpriced\"}}\n","error":{"code":-32000},"requestBody":"{\"method\":\"eth_sendRawTransaction\",\"params\":[\"0xf8ca8085012a05f20083019092943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000946da73853fac1441998fd5c14c7029988ffe9b80000000000000000000000000404ebf40d76a32b0dccdf13f3ff8a9a7e0fad1900000000000000000000000000000000000000000000000000000000000001f42da0178b57fae97031c6ba540b293f3847589d1890ab0b62b56b4ca4a87786800e7ba03d9e1fcf9f21fa759b832c588bee92d5b99fbff5ada6a72c71f5336c9a22c961\"],\"id\":605,\"jsonrpc\":\"2.0\"}","requestMethod":"POST","url":"http://geth.goerli.ethnodes.brainbot.com:8545"}, method="sendTransaction", transaction="0xf8ca8085012a05f20083019092943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000946da73853fac1441998fd5c14c7029988ffe9b80000000000000000000000000404ebf40d76a32b0dccdf13f3ff8a9a7e0fad1900000000000000000000000000000000000000000000000000000000000001f42da0178b57fae97031c6ba540b293f3847589d1890ab0b62b56b4ca4a87786800e7ba03d9e1fcf9f21fa759b832c588bee92d5b99fbff5ada6a72c71f5336c9a22c961", code=REPLACEMENT_UNDERPRICED, version=providers/5.0.19)
      at Logger.makeError (/lc/index.js:7523:21)
      at Logger.throwError (/lc/index.js:7532:20)
      at checkError (/lc/index.js:12155:16)
      at JsonRpcProvider.<anonymous> (/lc/index.js:12576:47)
      at step (/lc/index.js:12102:23)
      at Object.throw (/lc/index.js:12083:53)
      at rejected (/lc/index.js:12075:65)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:93:5) {
    reason: 'replacement fee too low',
    code: 'REPLACEMENT_UNDERPRICED',
    error: Error: processing response error (body="{\"jsonrpc\":\"2.0\",\"id\":605,\"error\":{\"code\":-32000,\"message\":\"replacement transaction underpriced\"}}\n", error={"code":-32000}, requestBody="{\"method\":\"eth_sendRawTransaction\",\"params\":[\"0xf8ca8085012a05f20083019092943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000946da73853fac1441998fd5c14c7029988ffe9b80000000000000000000000000404ebf40d76a32b0dccdf13f3ff8a9a7e0fad1900000000000000000000000000000000000000000000000000000000000001f42da0178b57fae97031c6ba540b293f3847589d1890ab0b62b56b4ca4a87786800e7ba03d9e1fcf9f21fa759b832c588bee92d5b99fbff5ada6a72c71f5336c9a22c961\"],\"id\":605,\"jsonrpc\":\"2.0\"}", requestMethod="POST", url="http://geth.goerli.ethnodes.brainbot.com:8545", code=SERVER_ERROR, version=web/5.0.12)
        at Logger.makeError (/lc/index.js:7523:21)
        at Logger.throwError (/lc/index.js:7532:20)
        at /lc/index.js:19174:32
        at step (/lc/index.js:18942:23)
        at Object.next (/lc/index.js:18923:53)
        at fulfilled (/lc/index.js:18914:58)
        at runMicrotasks (<anonymous>)
        at processTicksAndRejections (internal/process/task_queues.js:93:5) {
      reason: 'processing response error',
      code: 'SERVER_ERROR',
      body: '{"jsonrpc":"2.0","id":605,"error":{"code":-32000,"message":"replacement transaction underpriced"}}\n',
      error: Error: replacement transaction underpriced
          at getResult (/lc/index.js:12174:21)
          at processJsonFunc (/lc/index.js:19217:22)
          at /lc/index.js:19149:46
          at step (/lc/index.js:18942:23)
          at Object.next (/lc/index.js:18923:53)
          at fulfilled (/lc/index.js:18914:58)
          at runMicrotasks (<anonymous>)
          at processTicksAndRejections (internal/process/task_queues.js:93:5) {
        code: -32000,
        data: undefined
      },
      requestBody: '{"method":"eth_sendRawTransaction","params":["0xf8ca8085012a05f20083019092943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000946da73853fac1441998fd5c14c7029988ffe9b80000000000000000000000000404ebf40d76a32b0dccdf13f3ff8a9a7e0fad1900000000000000000000000000000000000000000000000000000000000001f42da0178b57fae97031c6ba540b293f3847589d1890ab0b62b56b4ca4a87786800e7ba03d9e1fcf9f21fa759b832c588bee92d5b99fbff5ada6a72c71f5336c9a22c961"],"id":605,"jsonrpc":"2.0"}',
      requestMethod: 'POST',
      url: 'http://geth.goerli.ethnodes.brainbot.com:8545'
    },
    method: 'sendTransaction',
    transaction: {
      nonce: 0,
      gasPrice: BigNumber { _hex: '0x012a05f200', _isBigNumber: true },
      gasLimit: BigNumber { _hex: '0x019092', _isBigNumber: true },
      to: '0x3EA2a1fED7FdEf300DA19E97092Ce8FdF8bf66A3',
      value: BigNumber { _hex: '0x00', _isBigNumber: true },
      data: '0x0a798f24000000000000000000000000946da73853fac1441998fd5c14c7029988ffe9b80000000000000000000000000404ebf40d76a32b0dccdf13f3ff8a9a7e0fad1900000000000000000000000000000000000000000000000000000000000001f4',
      chainId: 5,
      v: 45,
      r: '0x178b57fae97031c6ba540b293f3847589d1890ab0b62b56b4ca4a87786800e7b',
      s: '0x3d9e1fcf9f21fa759b832c588bee92d5b99fbff5ada6a72c71f5336c9a22c961',
      from: '0x946da73853FAC1441998Fd5c14c7029988FFE9B8',
      hash: '0x6e280ec913da91cecb9377a5bd3f6a80bd3b4a87e09b77856e1a14a45dd5a928'
    },
    transactionHash: '0x6e280ec913da91cecb9377a5bd3f6a80bd3b4a87e09b77856e1a14a45dd5a928'
  }
}

scenario-player-run_pfs1_get_a_simple_path_2021-01-22T00:59:34.log
pfs1_node1_node3_22jan2021.tar.gz

Occured again in PFS9 today

Occured again in PFS9 today

Occurred in BF6 23rd Jan 2021
BF6 gets transfer expired error node reveal this is the similar kind of error that we get here
Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce
2021-01-24T04:35:15.207Z [info] retryWhile: retrying {
  count: 0,
  interval: 5000,
  error: Error: processing response error (body="{\"jsonrpc\":\"2.0\",\"error\":{\"code\":-32010,\"message\":\"Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce.\"},\"id\":590}\n", error={"code":-32010}, requestBody="{\"method\":\"eth_sendRawTransaction\",\"params\":[\"0xf8c98084773594008301909c943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000757eca3c6f2e5f7e0d3274f906a4a6e8d8b85b77000000000000000000000000810e0726f3372f073af461789f5d085f937897d500000000000000000000000000000000000000000000000000000000000001f42ea00fa3f58fa0117d9102a5757fd7bfe1daa6f2d3b1d9d7d9f69a7acc7ee664bbdda0221161eb42bf476d85229d02d3df1655262bf0f68ef8870355b915bdf8c57436\"],\"id\":590,\"jsonrpc\":\"2.0\"}", requestMethod="POST", url="http://parity.goerli.ethnodes.brainbot.com:8545", code=SERVER_ERROR, version=web/5.0.12)
      at Logger.makeError (/lc/index.js:7523:21)
      at Logger.throwError (/lc/index.js:7532:20)
      at /lc/index.js:19174:32
      at step (/lc/index.js:18942:23)
      at Object.next (/lc/index.js:18923:53)
      at fulfilled (/lc/index.js:18914:58)
      at processTicksAndRejections (internal/process/task_queues.js:93:5) {
    reason: 'processing response error',
    code: 'SERVER_ERROR',
    body: '{"jsonrpc":"2.0","error":{"code":-32010,"message":"Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce."},"id":590}\n',
    error: Error: Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce.
        at getResult (/lc/index.js:12174:21)
        at processJsonFunc (/lc/index.js:19217:22)
        at /lc/index.js:19149:46
        at step (/lc/index.js:18942:23)
        at Object.next (/lc/index.js:18923:53)
        at fulfilled (/lc/index.js:18914:58)
        at processTicksAndRejections (internal/process/task_queues.js:93:5) {
      code: -32010,
      data: undefined
    },
    requestBody: '{"method":"eth_sendRawTransaction","params":["0xf8c98084773594008301909c943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000757eca3c6f2e5f7e0d3274f906a4a6e8d8b85b77000000000000000000000000810e0726f3372f073af461789f5d085f937897d500000000000000000000000000000000000000000000000000000000000001f42ea00fa3f58fa0117d9102a5757fd7bfe1daa6f2d3b1d9d7d9f69a7acc7ee664bbdda0221161eb42bf476d85229d02d3df1655262bf0f68ef8870355b915bdf8c57436"],"id":590,"jsonrpc":"2.0"}',
    requestMethod: 'POST',
    url: 'http://parity.goerli.ethnodes.brainbot.com:8545',
    transaction: {
      nonce: 0,
      gasPrice: BigNumber { _hex: '0x77359400', _isBigNumber: true },
      gasLimit: BigNumber { _hex: '0x01909c', _isBigNumber: true },
      to: '0x3EA2a1fED7FdEf300DA19E97092Ce8FdF8bf66A3',
      value: BigNumber { _hex: '0x00', _isBigNumber: true },
      data: '0x0a798f24000000000000000000000000757eca3c6f2e5f7e0d3274f906a4a6e8d8b85b77000000000000000000000000810e0726f3372f073af461789f5d085f937897d500000000000000000000000000000000000000000000000000000000000001f4',
      chainId: 5,
      v: 46,
      r: '0x0fa3f58fa0117d9102a5757fd7bfe1daa6f2d3b1d9d7d9f69a7acc7ee664bbdd',
      s: '0x221161eb42bf476d85229d02d3df1655262bf0f68ef8870355b915bdf8c57436',
      from: '0x757ECa3c6f2E5f7E0D3274F906a4A6e8d8b85b77',
      hash: '0x0cde3bd1a582c1c906d40241c2643a84fa0d87fe91393edbff05496b8616b578'
    },
    transactionHash: '0x0cde3bd1a582c1c906d40241c2643a84fa0d87fe91393edbff05496b8616b578'
  }
}

scenario-player-run_bf6_stress_hub_node_2021-01-24T04:33:09.log
node_4617_003.tar.gz

Occurred in BF6 23rd Jan 2021
BF6 gets transfer expired error node reveal this is the similar kind of error that we get here
Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce
2021-01-24T04:35:15.207Z [info] retryWhile: retrying {
  count: 0,
  interval: 5000,
  error: Error: processing response error (body="{\"jsonrpc\":\"2.0\",\"error\":{\"code\":-32010,\"message\":\"Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce.\"},\"id\":590}\n", error={"code":-32010}, requestBody="{\"method\":\"eth_sendRawTransaction\",\"params\":[\"0xf8c98084773594008301909c943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000757eca3c6f2e5f7e0d3274f906a4a6e8d8b85b77000000000000000000000000810e0726f3372f073af461789f5d085f937897d500000000000000000000000000000000000000000000000000000000000001f42ea00fa3f58fa0117d9102a5757fd7bfe1daa6f2d3b1d9d7d9f69a7acc7ee664bbdda0221161eb42bf476d85229d02d3df1655262bf0f68ef8870355b915bdf8c57436\"],\"id\":590,\"jsonrpc\":\"2.0\"}", requestMethod="POST", url="http://parity.goerli.ethnodes.brainbot.com:8545", code=SERVER_ERROR, version=web/5.0.12)
      at Logger.makeError (/lc/index.js:7523:21)
      at Logger.throwError (/lc/index.js:7532:20)
      at /lc/index.js:19174:32
      at step (/lc/index.js:18942:23)
      at Object.next (/lc/index.js:18923:53)
      at fulfilled (/lc/index.js:18914:58)
      at processTicksAndRejections (internal/process/task_queues.js:93:5) {
    reason: 'processing response error',
    code: 'SERVER_ERROR',
    body: '{"jsonrpc":"2.0","error":{"code":-32010,"message":"Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce."},"id":590}\n',
    error: Error: Transaction gas price supplied is too low. There is another transaction with same nonce in the queue. Try increasing the gas price or incrementing the nonce.
        at getResult (/lc/index.js:12174:21)
        at processJsonFunc (/lc/index.js:19217:22)
        at /lc/index.js:19149:46
        at step (/lc/index.js:18942:23)
        at Object.next (/lc/index.js:18923:53)
        at fulfilled (/lc/index.js:18914:58)
        at processTicksAndRejections (internal/process/task_queues.js:93:5) {
      code: -32010,
      data: undefined
    },
    requestBody: '{"method":"eth_sendRawTransaction","params":["0xf8c98084773594008301909c943ea2a1fed7fdef300da19e97092ce8fdf8bf66a380b8640a798f24000000000000000000000000757eca3c6f2e5f7e0d3274f906a4a6e8d8b85b77000000000000000000000000810e0726f3372f073af461789f5d085f937897d500000000000000000000000000000000000000000000000000000000000001f42ea00fa3f58fa0117d9102a5757fd7bfe1daa6f2d3b1d9d7d9f69a7acc7ee664bbdda0221161eb42bf476d85229d02d3df1655262bf0f68ef8870355b915bdf8c57436"],"id":590,"jsonrpc":"2.0"}',
    requestMethod: 'POST',
    url: 'http://parity.goerli.ethnodes.brainbot.com:8545',
    transaction: {
      nonce: 0,
      gasPrice: BigNumber { _hex: '0x77359400', _isBigNumber: true },
      gasLimit: BigNumber { _hex: '0x01909c', _isBigNumber: true },
      to: '0x3EA2a1fED7FdEf300DA19E97092Ce8FdF8bf66A3',
      value: BigNumber { _hex: '0x00', _isBigNumber: true },
      data: '0x0a798f24000000000000000000000000757eca3c6f2e5f7e0d3274f906a4a6e8d8b85b77000000000000000000000000810e0726f3372f073af461789f5d085f937897d500000000000000000000000000000000000000000000000000000000000001f4',
      chainId: 5,
      v: 46,
      r: '0x0fa3f58fa0117d9102a5757fd7bfe1daa6f2d3b1d9d7d9f69a7acc7ee664bbdd',
      s: '0x221161eb42bf476d85229d02d3df1655262bf0f68ef8870355b915bdf8c57436',
      from: '0x757ECa3c6f2E5f7E0D3274F906a4A6e8d8b85b77',
      hash: '0x0cde3bd1a582c1c906d40241c2643a84fa0d87fe91393edbff05496b8616b578'
    },
    transactionHash: '0x0cde3bd1a582c1c906d40241c2643a84fa0d87fe91393edbff05496b8616b578'
  }
}

scenario-player-run_bf6_stress_hub_node_2021-01-24T04:33:09.log
node_4617_003.tar.gz

Fixed by 1f37f9f

Fixed by 1f37f9f


Correction for raw affiliation Department of Nephrology, Centre Hopitalier Lyon Sud, INSERM 1060, CENS, Université de Lyon, France

Correction needed for raw affiliation Department of Nephrology, Centre Hopitalier Lyon Sud, INSERM 1060, CENS, Université de Lyon, France
raw_affiliation_name: Department of Nephrology, Centre Hopitalier Lyon Sud, INSERM 1060, CENS, Université de Lyon, France
new_rors: 023xgd207;02vjkv261;029brtt94
previous_rors: 023xgd207;04v837171;02vjkv261;029brtt94
works_examples: W4367671817
contact: 901ce7f30d446052752c40d989976dd5:0057aaa40206c6b802763c44023dcc @ univ-nantes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


doc improvements

couple of typos and some spacing

you should've just submitted an issue instead of a pull request for clout

Maybe don't come asking for proofreading on twitter then.
Thumbs down for what ? Spending time reading this and actually contributing something ?

Alright, I'm in the process of updating my write-up. I appreciate the typo fixed and will incorporate them, but what's the deal with spaces before question marks, and with the changed paragraphs that seems to lack actual changes though?

inconsistent line endings and removed byte order mark, you can diff that in the terminal, suppose that's something only windows users are affected by

Bkd


Add documentation to the Actuator API section

Codecov Report

Merging #634 into 2.0.x will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              2.0.x     #634   +/-   ##
=========================================
  Coverage     70.58%   70.58%
  Complexity      917      917
=========================================
  Files           129      129
  Lines          3675     3675
  Branches        259      259
=========================================
  Hits           2594     2594
  Misses          945      945
  Partials        136      136

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 2879296...2ae1eea. Read the comment docs.


Update README.md

According to https://github.com/Nebo15/confex,
Type can be replaced with {module, function, arguments} tuple, in this case Confex will use external function to resolve the type. Function must returns either {:ok, value} or {:error, reason :: String.t} tuple.
I modified the readme to prevent misunderstanding

Coverage remained the same at 88.312% when pulling b67bf898a2fbd7dc1f5fdd4c569b238ace9383c0 on vip30:master into 20af6bbfa3fb621e267e2909e44a7ded7b7eaf99 on edgurgel:master.

Thank you!


[Backport] [2.x] Bump com.squareup.okio:okio from 3.7.0 to 3.8.0 in /test/fixtures/hdfs-fixture (#12290)

Backport of https://github.com/opensearch-project/OpenSearch/pull/12290 to 2.x

❌ Gradle check result for 4d4b247: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/9191
https://github.com/opensearch-project/OpenSearch/issues/12318


New feature: Add ability to check & fix whether signed commits for protected branches

My organization is looking to audit is whether we require signed commits for merging to protected branches on our repos. This feature seemed to be missing from Allstar, so I've added it.
I also added the ability to auto-fix the signed commits setting for protected branches if it is out of sync with the expected setting. I can confirm that running an instance of Allstar with these changes and configured with requireSignedCommits: true in branch_protection.yaml, a repo was properly updated to enable "Require signed commits" on the branch protection settings.
I'm a little unsure about whether some of my error handling logic is correct, but so far everything is looking good in unit tests and manual testing.

@jeffmendoza I think this is ready for re-review. Thanks!

@markdboyd Sorry for the delay, I was traveling last week. Looks great, thanks for the work! I will test this out in staging a bit then deploy to prod.


Task4 completed

Issue: #4

LGTM but merging this commit would mean merging both incorrect PR for task 3 and incomplete PR for task 2 so fix those so can I merge this.


Fixing feature activity check (task #5694)

Adding correct flag check method, so it will properly disable Swagger documentation even if the feature is enabled.

Codecov Report

Merging #513 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #513   +/-   ##
=========================================
  Coverage     29.26%   29.26%
  Complexity      914      914
=========================================
  Files            84       84
  Lines          3116     3116
=========================================
  Hits            912      912
  Misses         2204     2204

Impacted Files
Coverage Δ
Complexity Δ

src/Controller/Api/V1/V0/AppController.php
49.21% <0%> (ø)
34 <0> (ø)
:arrow_down:

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 86335b9...ff4ac36. Read the comment docs.


function to summarize study data types

Per study, take data types from fileview and add to portal study table as stringlist.

Done!

There are some TODOs inline, however.


MetadataContract is showing up as None

Describe the bug
I went through the simple datatokens README.
It went smoothly until the last part, in a Python step. Here's that step, and the traceback
>>> ocean = Ocean(config)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/trentmc/code/play/venv/lib/python3.8/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/play/venv/lib/python3.8/site-packages/ocean_lib/ocean/ocean.py", line 105, in __init__
    addresses.get(MetadataContract.CONTRACT_NAME),
AttributeError: 'NoneType' object has no attribute 'get'

Describe the solution
WIP

I just tried to reproduce this, from everything clean. No issues encountered. Likely what caused the issue was some older artifacts from barge.
Closing issue.


initial commit; n compartments

PULL REQUEST
Purpose and Content
This pull request allows for vegetation to be divided up into an arbitrary amount of stem and leaf layers instead of being hardcoded for one of each, increasing the resolution and accuracy of the plant hydraulics model in the LSM.
Benefits and Risks
(State concisely the benefits to be derived from and the risks associated with this PR)
Linked Issues
(Provide references to any link issues. Use closes #issuenum to automatically close an open issue)

Fixes #
Closes #

PR Checklist

[x] This PR has a corresponding issue OR is linked to an SDI.
[x] I have followed CliMA's codebase contribution and style guidelines OR N/A.
[x] I have followed CliMA's documentation policy.
[x] I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
[x] I linted my code on my local machine prior to submission OR N/A.
[x] Unit tests are included OR N/A.
[x] Code used in an integration test OR N/A.
[x] All tests ran successfully on my local machine OR N/A.
[x] All classes, modules, and function contain docstrings OR N/A.
[x] Documentation has been added/updated OR N/A.

bors r+


Code Review: fix JS API exports (#23888)

fix BohemianCoding/Sketch#23888

Warnings

:warning:
This pull request may need a CHANGELOG entry.

  Generated by :no_entry_sign: dangerJS


SDA-8900 | fix: adjust behavior for creating with -o

Related issue: https://issues.redhat.com/browse/SDA-8900

/lgtm


Only english language available in plasma regional settings

Issue description
Only english language available in plasma regional settings, but all locales generated.

Steps to reproduce

Open system settings.
Open regional settings.

Technical details
- system: `"x86_64-linux"`
 - host os: `Linux 4.9.76, NixOS, 17.09.2754.66b4de79e38 (Hummingbird)`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 1.11.16`

This appears it is also a problem with GNOME, presumable these are using the same locale files.

This appears it is also a problem with GNOME, presumable these are using the same locale files.

I noticed that adding the locales I wanted to i18n.supportedLocales solves the issue. However this is documented as being "all" which should contain all locals. However setting it explicitly to "all" does resolve the issue. So I don't know why the default is being changed.
i18n.supportedLocales = ["all"];

I noticed that adding the locales I wanted to i18n.supportedLocales solves the issue. However this is documented as being "all" which should contain all locals. However setting it explicitly to "all" does resolve the issue. So I don't know why the default is being changed.
i18n.supportedLocales = ["all"];

Oh, I am dumb, I was actually setting it in my config. It was not conflicting because the arrays just got merged as expected.

Oh, I am dumb, I was actually setting it in my config. It was not conflicting because the arrays just got merged as expected.

@florianjacob are you seeing this issue, and are you not setting this value in your config somewhere?

@florianjacob are you seeing this issue, and are you not setting this value in your config somewhere?

This was fixed for me with NixOS 19.09, see https://github.com/NixOS/nixpkgs/issues/37741#issuecomment-544139582 - it probably was a bug in plasma packaging that did not make the locale files available to the applications, and this issue can probably be closed. I always had i18n.defaultLocale = "de_DE.UTF-8";

This was fixed for me with NixOS 19.09, see https://github.com/NixOS/nixpkgs/issues/37741#issuecomment-544139582 - it probably was a bug in plasma packaging that did not make the locale files available to the applications, and this issue can probably be closed. I always had i18n.defaultLocale = "de_DE.UTF-8";


Generate an empty spec/factories.rb file

Our guides say

Use one factories.rb file per project.

and the factory_girl_rails documentation says

If you use factory_girl for fixture replacement and already have a factories.rb file in the directory that contains your tests, factory_girl_rails will insert new factory definitions at the top of factories.rb.

If suspenders generates that file, it'll be easier to conform to our guide.

:+1:

@thorncp just ran into this today. I think we should create an empty file with an empty define block.

Thanks! Closing in favor of PR.


Refactor navigation

Description

Refactor navigation
Make mobile-navigation UX nice

In the next PR going to implement animations and more meaningfull tests
Fixes partly refactor-navigtation
Why was the change made?
What is the motivation for this change?
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Unittest
[ ] Cypress e2e -tests

Caveats?
Does this introduce new warnings or are linter rules suppressed? Why? Is only manual testing applicable for this feature? Is there something special that the reviewer should take into account?
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@kuosmark, I added additional commit now, to add the Language-Dropdown. (For mobile and web-view)
The code was left a bit messy, but maybe its better to implement the localizations using the buttons =)


ScreenOrientation cannot be set to unspecified

If one changes the ScreenOrientation property to something other than unspecified and then change back, the companion will not change its orientation back to unspecified.

Fix on ucr


Fatal error: Unexpected trailing arguments

I tried changing the resource string from RCEDIT
rcedit --set-resource-string 65201 "This is a test"
It always says Unexpected trailing argument, also I tried same thing with .net exe and other machine language exe compiled through C++ or python, still the same thing. Also other functions like setting execution level is also not working. Can't figure out the solution.
Any Help or Suggestion ! THANKS IN ADVANCE

The file path should be passed before passing the flags.
rcdit /path/to/exe --flags


Bump release workflow from python 3.7 to 3.9

Description
Because documentation will no longer build on python 3.7, we bump its release job together with the package build from 3.7 to 3.9.
Closes #191.
Type of change

[x] Bug fix and code cleanup
[ ] New feature
[ ] Documentation update
[ ] Testing

Checklist for the reviewer
This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?
[ ] Is the code properly tested?

This PR has not been tested yet; should be done before merging.

I've bumped the Python version used for the basic CI tests to Python 3.9. This ensures we test building the package distribution and documentation in the same Python version as is being used upon a release. This does not impact the ability to install and use the built package in Python 3.7 or 3.8.

@ajeklund "squash and merge" at will :)


chat commands: change !cmb to check local levels instead of hiscores

Issue: !cmb command shows wrong values for combat skills that are relatively low (e.g. 45 def shows as 1) and computes the wrong combat level.
Cause:  net.runelite.client.plugins.chatcommands.ChatCommandsPlugin.combatLevelLookup queries the hi-scores for combat skill levels, which doesn't work when the combat skills are too low to show up on the hi-scores.
Fix: Get real skill levels from the local client instead of the hi-scores and use them to compute the combat level.
Fixes #18525
Example with low cb (levelled str up to 2 to make sure):

Example with high cb:

This will not work for other players

@raiyni Sorry, you're right. Do you think it makes sense to have the function check locally when the command is issued by the player and query the hiscores when issued by other players?


Cannot set environment variable NO_COLOR

If I set the environment variable NO_COLOR, e.g. to true, and run swift test | ./xcbeautify, I get this error message
ArgumentParser/ParsableArguments.swift:283: Fatal error: Validation failed for `Xcbeautify`:

- One or more Boolean flags is declared with an initial value of `true`.
This results in the flag always being `true`, no matter whether the user
specifies the flag or not.

To resolve this error, change the default to `false`, provide a value
for the `inversion:` parameter, or remove the `@Flag` property wrapper
altogether.

Affected flag(s):
--disable-colored-output

Building for debugging...
[2/2] Write swift-version--58304C5D6DBC2206.txt
Build complete! (2.80s)
fish: Process 40787, './xcbeautify' from job 1, 'swift test | ./xcbeautify' terminated by signal SIGTRAP (Trace or breakpoint trap)

Running the command in zsh, yields a similar output
ArgumentParser/ParsableArguments.swift:283: Fatal error: Validation failed for `Xcbeautify`:

- One or more Boolean flags is declared with an initial value of `true`.
This results in the flag always being `true`, no matter whether the user
specifies the flag or not.

To resolve this error, change the default to `false`, provide a value
for the `inversion:` parameter, or remove the `@Flag` property wrapper
altogether.

Affected flag(s):
--disable-colored-output

Building for debugging...
[2/2] Write swift-version--58304C5D6DBC2206.txt
Build complete! (2.08s)
zsh: broken pipe  swift test |
zsh: trace trap   ./xcbeautify

Hey @ikelax! I'm not able to repro this issue. Can you try with the latest version, 2.8.0? If it's still an issue, can you provide a little more context around your configuration. Are you passing --disable-colored-output too?

Sorry for the late reply. For the late version, 2.11.0, I cannot reproduce this issue.

Great! I'm going to close this issue. Feel free to reopen it if you see it again. Thanks for reporting it.


Update: 10

Ran Plan for dir: test workspace: default
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create

Terraform will perform the following actions:

  # null_resource.example will be created
+ resource "null_resource" "example" {
      + id = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

:arrow_forward: To apply this plan, comment:atlantis apply -d test

:put_litter_in_its_place: To delete this plan and lock, click here
:repeat: To plan this project again, comment:atlantis plan -d test

:fast_forward: To apply all unapplied plans from this Pull Request, comment:atlantis apply

:put_litter_in_its_place: To delete all plans and locks from this Pull Request, comment:atlantis unlock

Locks and plans deleted for the projects and workspaces modified in this pull request:

dir: test workspace: default

atlantis apply

Atlantis commands can't be run on closed pull requests

Error parsing command: EOF found when expecting closing quote


PHP preg_replace() Login

There were many installation problems. Failed to create the directories. I had to create one by one.
When starting for the first time (url / obiz / index.php / user / login) it accused the following error:
Warning: session_start (): Failed to read session data: user (path: / tmp) in path / obiz / openbiz / bin / SessionContext.php on line 79
Warning: preg_replace (): The / e modifier is no longer supported, use preg_replace_callback instead in path / obiz / openbiz / others / Smarty / libs / Smarty_Compiler.class.php on line 270
How to solve?

now only Warning: session_start(): Failed to read session data: user (path: /tmp) in path/obiz/openbiz/bin/SessionContext.php on line 79
Folder /home/user/tmp already exist and permission 0755


CI: add benchmark workflow

Best reviewed commit by commit

This is how the benchmark will look like: https://trym-b.github.io/warchaeology/benchmarks/ubuntu-22.04/

This is how a benchmark failure will look like: https://github.com/trym-b/warchaeology/actions/runs/8201485917

Ready for next round @maeb


No module named captcha.audio

Requirement already satisfied: captcha in /usr/local/lib/python3.6/dist-packages (0.3)
Requirement already satisfied: Pillow in /usr/lib/python3/dist-packages (from captcha) (5.1.0)
limu2048@ubuntu:~$ cd /usr/local/lib/python3.6/dist-packages
limu2048@ubuntu:/usr/local/lib/python3.6/dist-packages$ ls
captcha
captcha-0.3.dist-info
I think I  successfully  install captcha?
But when I import it, it shows the error info below:
No module named captcha.audio
Can you help me? I'm new to python and I've stuck in this problem for a day.

Not sure what’s wrong with the provided description.


Port the three.js webvr examples to three-ecsy

Even so this is not going to be a just-webxr framework, and we will end up porting other examples, as a good starting point I'd like to port the webvr examples first:

[ ] ballshooter
[ ] cubes
[ ] dragging
[ ] lorenzattractor
[ ] panorama
[ ] paint
[ ] rollercoaster
[ ] sandbox
[ ] sculpt
[ ] video
[ ] vive / paint
[ ] vive / sculpt

I'm starting with the panorama example

Moving discussion to https://github.com/ecsyjs/ecsy-three/issues/36


Drop support for ROS distributions Kinetic and Crystal

Kinetic recipes weren't ever generated in the new meta-ros and we don't plan to generate them now.
Crystal distribution reached EOL in Dec 2019:
https://index.ros.org/doc/ros2/Releases/#releases
and we don't plan to update its recipes anymore.

This alligns thud-draft with warrior-draft (as with crystal dropped as well https://github.com/ros/meta-ros/pull/690) and zeus-draft (https://github.com/ros/meta-ros/pull/691), before new changes for Milestone 12 are merged through their corresponding pull requests (thud https://github.com/ros/meta-ros/pull/677, warrior https://github.com/ros/meta-ros/pull/678, zeus https://github.com/ros/meta-ros/pull/679 and dunfell/master https://github.com/ros/meta-ros/pull/680 with build branch changes for all of them in https://github.com/ros/meta-ros/pull/675
Only the README.md, meta-ros-backports-warrior layer itself and dependency on it in LAYERDEPENDS_ros-common-layer + ROS_OE_RELEASE_SERIES is now different between thud and warrior branches.
$ git diff --stat thud-mj8b-crystal-drop warrior-draft-crystal-drop
 README.md                                                                                |    4 +-
 meta-ros-backports-warrior/classes/python3-dir.bbclass                                                                     |    5 -
...
 meta-ros-backports-warrior/recipes-support/spdlog/spdlog_1.3.1.bb                                                          |   19 ---
 meta-ros-common/conf/layer.conf                                                                                |    1 -
 meta-ros-common/conf/ros-distro/ros-distro.conf                                                                            |    2 +-
 40 files changed, 3 insertions(+), 3836 deletions(-)


Review Metadata

As a user of CCBC data
I want to see all the metadata
So that I know what the different tables and columns are
AC:

[x] Schema has descriptions of tables and columns
[x] Metabase shows the description of all the tables and columns
[x] Descriptions are meaningful and understandable by the average user

hey Marcel, at a first glance I can see that there are tables missing descriptions, I haven't looked at the column descriptions yet though:

Would you mind taking another look please?

@MarsRomer I was asked to remove those in code review as they were supposed to be taken care of by Vlad's PR. You can see the comments here:
https://github.com/bcgov/CONN-CCBC-portal/pull/1803

OK I saw that none of the acceptance criteria were checked off but thought that was just an oversight.

@MarsRomer sorry about that. Though we should try and remember to bring this up tomorrow at standup to make sure that this will be taken care of.


[Pytest] Add pytests to verify decoding by l3_dump.py, including LOC-decoding.

This commit adds extensive Python pytests to verify that the output decoded by the Python L3-dump script, l3_dump.py, is correct and as expected based on what was logged in source files.
We also add test-cases that will invoke the unit-test and use-case sample programs with L3_LOC_ENABLED=1, which
will trigger default LOC-encoding.
Python test machinery is provided to either validate expected outputs hard-coded in the test-case. Or, validate against
expected outputs generated by running grep on the source-files to look for L3-logging APIs, and re-constructing the expected decoded LOC-entry (as string).
In order to facilitate verifying L3-dump outputs using pytests, we do slight refactoring of main() in l3_dump.py into a
separate do_main() method that returns actual-output lists used by the pytest cases to verify against expected-lists.
Sample use-case programs are mildly altered to receive a --unit-tests argument, to only run the part of the program that generates a small # of slow and fast logging entries. (The logging done for perf-throughput measurements is skipped under this arg, to keep pytests running fast.)

@gregthelaw - I didn't know about the existence of pexpect() interface till you brought it up.
Took a quick look ... and seems like it's ideally suited for running commands in a child process to which you may want to communicate by sending inputs etc.
I suppose the new test code could be reconstructed to use pexpect, but I'd like to push that off to future. I don't want to spend the time now trying to figure out how to inject this paradigm, while other PRs are waiting behind this one to merge.
I'll track this suggestion as an open item to follow in our Discussions #23 work-items list.


PSP relocation types not supported

It would appear that the custom PSP relocation types aren't supported:
#define R_MIPS_X_HI16   13
#define R_MIPS_X_J26    14
#define R_MIPS_X_JAL26  15

See https://github.com/pspdev/prxtool/blob/08849d87fa8692f2ba29238ec1710f2706e3e021/elftypes.h#L163-L165 and https://github.com/pspdev/prxtool/blob/08849d87fa8692f2ba29238ec1710f2706e3e021/ProcessPrx.C#L1558-L1635

Interesting, they are not supported by PPSSPP either so they can't be used in games. I don't think I'll try to implement them without an executable to test it on.

It would make sense that they're not implemented in PPSSPP as they're used in System PRXs. A small snippet from prxtool output for lowio.prx from FW 6.61:
Debug: Reloc (null):1591 Type:X_HI16 Symbol:256 Offset 0000A848 Info:0000010D
Debug: Reloc (null):1592 Type:R_LO16 Symbol:256 Offset 0000A84C Info:00000106
Debug: Reloc (null):1593 Type:X_JAL26 Symbol:0 Offset 0000A888 Info:0000000F
Debug: Reloc (null):1594 Type:X_J26 Symbol:0 Offset 0000A8D0 Info:0000000E

If anyone wants to look into it, here is where relocations are done:
https://github.com/kotcrab/ghidra-allegrex/blob/e4717ee8bd624cdd1afbab4bf7434d7e9bacfcf1/src/main/kotlin/ghidra/app/util/bin/format/elf/relocation/AllegrexElfRelocationHandler.kt#L46-L73

If anyone wants to look into it, here is where relocations are done:
https://github.com/kotcrab/ghidra-allegrex/blob/e4717ee8bd624cdd1afbab4bf7434d7e9bacfcf1/src/main/kotlin/ghidra/app/util/bin/format/elf/relocation/AllegrexElfRelocationHandler.kt#L46-L73

The way to handle these relocations is specified in PRXTool.
Check https://github.com/pspdev/prxtool/blob/master/ProcessPrx.C#L881 to see how to parse them
and https://github.com/pspdev/prxtool/blob/master/ProcessPrx.C#L1439 to see how to do the actual relocation (I think the missing relocations are TypeB).
Handling these would be a great help for reversing kernel stuff!

The way to handle these relocations is specified in PRXTool.
Check https://github.com/pspdev/prxtool/blob/master/ProcessPrx.C#L881 to see how to parse them
and https://github.com/pspdev/prxtool/blob/master/ProcessPrx.C#L1439 to see how to do the actual relocation (I think the missing relocations are TypeB).
Handling these would be a great help for reversing kernel stuff!

@kotcrab I'm taking a stab at implementing this, but am having trouble logging things.
I'm able to build the plugin and copy / rename everything to get Ghidra to recognize it, but am unable to get any messages to print from the processor code.
I've tried added Msg.warn("blah") and println("blah") from AllegrexElfRelocationHandler.kt, but now output shows up in the "User Log" or otherwise - do you have any advice here?

@kotcrab I'm taking a stab at implementing this, but am having trouble logging things.
I'm able to build the plugin and copy / rename everything to get Ghidra to recognize it, but am unable to get any messages to print from the processor code.
I've tried added Msg.warn("blah") and println("blah") from AllegrexElfRelocationHandler.kt, but now output shows up in the "User Log" or otherwise - do you have any advice here?

@kotcrab I'm taking a stab at implementing this, but am having trouble logging things.
I'm able to build the plugin and copy / rename everything to get Ghidra to recognize it, but am unable to get any messages to print from the processor code.
I've tried added Msg.warn("blah") and println("blah") from AllegrexElfRelocationHandler.kt, but no output shows up in the "User Log" or otherwise - do you have any advice here?

I think you can use the println() function (maybe default JAVA library or Ghidra define) or the Msg library (but more for error messages).
I hope I understood well what you said.

@kotcrab I'm taking a stab at implementing this, but am having trouble logging things.
I'm able to build the plugin and copy / rename everything to get Ghidra to recognize it, but am unable to get any messages to print from the processor code.
I've tried added Msg.warn("blah") and println("blah") from AllegrexElfRelocationHandler.kt, but no output shows up in the "User Log" or otherwise - do you have any advice here?

I think you can use the println() function (maybe default JAVA library or Ghidra define) or the Msg library (but more for error messages).
I hope I understood well what you said.

If you re-read my message, I tried both of those and neither worked for me
On Thu, Jan 21, 2021 at 6:30 AM CelesteBlue-dev<EMAIL_ADDRESS>wrote:

@kotcrab https://github.com/kotcrab I'm taking a stab at implementing
this, but am having trouble logging things.
I'm able to build the plugin and copy / rename everything to get Ghidra to
recognize it, but am unable to get any messages to print from the processor
code.
I've tried added Msg.warn("blah") and println("blah") from
AllegrexElfRelocationHandler.kt, but no output shows up in the "User Log"
or otherwise - do you have any advice here?
I think you can use the println() function (maybe default JAVA library or
Ghidra define) or the Msg library (but more for error messages).
I hope I understood well what you said.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kotcrab/ghidra-allegrex/issues/7#issuecomment-764577887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADCSVXITXUB5E6LNYHZL6TS3AF5ZANCNFSM4SJPNKRA
.

If you re-read my message, I tried both of those and neither worked for me
On Thu, Jan 21, 2021 at 6:30 AM CelesteBlue-dev<EMAIL_ADDRESS>wrote:

@kotcrab https://github.com/kotcrab I'm taking a stab at implementing
this, but am having trouble logging things.
I'm able to build the plugin and copy / rename everything to get Ghidra to
recognize it, but am unable to get any messages to print from the processor
code.
I've tried added Msg.warn("blah") and println("blah") from
AllegrexElfRelocationHandler.kt, but no output shows up in the "User Log"
or otherwise - do you have any advice here?
I think you can use the println() function (maybe default JAVA library or
Ghidra define) or the Msg library (but more for error messages).
I hope I understood well what you said.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kotcrab/ghidra-allegrex/issues/7#issuecomment-764577887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADCSVXITXUB5E6LNYHZL6TS3AF5ZANCNFSM4SJPNKRA
.

Msg.warn etc. should show in user log but I'm not sure if I ever tested this, maybe some magic config file is missing.
println() will show in stdout. Not sure if it's required but I'm always running in debug mode for dev, launch through ./scripts/ghidraDebug
If that fails you should just use the debug mode, it works well. Launch Ghidra in debug mode, open the project in IntelliJ IDEA (community version is perfectly fine), create Remote JVM Debug launch configuration and start it. Now you will be able to place breakpoints on any line and see variables etc. You can also use breakpoints for logging, right click it and you can uncheck suspend and enter log expression.

I'm able to build the plugin and copy / rename everything to get Ghidra to recognize it,

Instead of copying manually you can use ./gradlew ghidraInstall, that will save you some time.

Msg.warn etc. should show in user log but I'm not sure if I ever tested this, maybe some magic config file is missing.
println() will show in stdout. Not sure if it's required but I'm always running in debug mode for dev, launch through ./scripts/ghidraDebug
If that fails you should just use the debug mode, it works well. Launch Ghidra in debug mode, open the project in IntelliJ IDEA (community version is perfectly fine), create Remote JVM Debug launch configuration and start it. Now you will be able to place breakpoints on any line and see variables etc. You can also use breakpoints for logging, right click it and you can uncheck suspend and enter log expression.

I'm able to build the plugin and copy / rename everything to get Ghidra to recognize it,

Instead of copying manually you can use ./gradlew ghidraInstall, that will save you some time.

Ah, thanks for the pointer to debug mode. None of my messages print out still, but I do get a strange warning on stdout whenever I drag a new prx into the project:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by net.sf.cglib.core.ReflectUtils$2 (file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Framework/Generic/lib/cglib-nodep-2.2.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of net.sf.cglib.core.ReflectUtils$2
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

I guess I'll install IDEA and give that approach a shot.

Ah, thanks for the pointer to debug mode. None of my messages print out still, but I do get a strange warning on stdout whenever I drag a new prx into the project:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by net.sf.cglib.core.ReflectUtils$2 (file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Framework/Generic/lib/cglib-nodep-2.2.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of net.sf.cglib.core.ReflectUtils$2
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

I guess I'll install IDEA and give that approach a shot.

I tried both Eclipse (couldn't quite get this to work) and IDEA can connect but none of my breakpoints get hit, despite the console showing that I'm connected and I'm able to pause ghidra.
IDEA Console shows:
Connected to the target VM, address: 'localhost:18001', transport: 'socket'
Ghidra thinks it has the Allegrex CPU loaded, and compiling in IDEA installs the plugin into the Ghidra/Processor/Allegrex folder
I'm not that familiar with Java / Kotlin development and feel like I'm fighting an uphill battle.

I tried both Eclipse (couldn't quite get this to work) and IDEA can connect but none of my breakpoints get hit, despite the console showing that I'm connected and I'm able to pause ghidra.
IDEA Console shows:
Connected to the target VM, address: 'localhost:18001', transport: 'socket'
Ghidra thinks it has the Allegrex CPU loaded, and compiling in IDEA installs the plugin into the Ghidra/Processor/Allegrex folder
I'm not that familiar with Java / Kotlin development and feel like I'm fighting an uphill battle.

Ok they're being loaded at least (ran with -verbose:class)
[5.402s][info][class,load] ghidra.app.util.bin.format.elf.extend.Allegrex_ElfExtension source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.403s][info][class,load] ghidra.app.util.bin.format.MemoryLoadable source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Features/Base/lib/Base.jar
[5.403s][info][class,load] ghidra.app.util.bin.format.elf.ElfDynamicType source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Features/Base/lib/Base.jar
[5.404s][info][class,load] ghidra.app.util.bin.format.elf.ElfDynamicType$ElfDynamicValueType source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Features/Base/lib/Base.jar
[5.405s][info][class,load] ghidra.app.plugin.core.analysis.AllegrexAddressAnalyzer source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.406s][info][class,load] ghidra.app.plugin.core.analysis.AllegrexPreAnalyzer source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.406s][info][class,load] ghidra.app.util.opinion.PspElfLoader source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.407s][info][class,load] ghidra.app.util.opinion.PspElfLoader$Companion source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.408s][info][class,load] kotlin.Metadata source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.409s][info][class,load] kotlin.annotation.Retention source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.409s][info][class,load] kotlin.annotation.Target source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.410s][info][class,load] com.sun.proxy.$Proxy20 source: __dynamic_proxy__
[5.411s][info][class,load] ghidra.app.util.bin.format.elf.relocation.AllegrexElfRelocationHandler source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.411s][info][class,load] allegrex.format.elf.relocation.AllegrexRelocationFixupHandler source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar```

Ok they're being loaded at least (ran with -verbose:class)
[5.402s][info][class,load] ghidra.app.util.bin.format.elf.extend.Allegrex_ElfExtension source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.403s][info][class,load] ghidra.app.util.bin.format.MemoryLoadable source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Features/Base/lib/Base.jar
[5.403s][info][class,load] ghidra.app.util.bin.format.elf.ElfDynamicType source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Features/Base/lib/Base.jar
[5.404s][info][class,load] ghidra.app.util.bin.format.elf.ElfDynamicType$ElfDynamicValueType source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Features/Base/lib/Base.jar
[5.405s][info][class,load] ghidra.app.plugin.core.analysis.AllegrexAddressAnalyzer source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.406s][info][class,load] ghidra.app.plugin.core.analysis.AllegrexPreAnalyzer source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.406s][info][class,load] ghidra.app.util.opinion.PspElfLoader source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.407s][info][class,load] ghidra.app.util.opinion.PspElfLoader$Companion source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.408s][info][class,load] kotlin.Metadata source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.409s][info][class,load] kotlin.annotation.Retention source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.409s][info][class,load] kotlin.annotation.Target source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.410s][info][class,load] com.sun.proxy.$Proxy20 source: __dynamic_proxy__
[5.411s][info][class,load] ghidra.app.util.bin.format.elf.relocation.AllegrexElfRelocationHandler source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar
[5.411s][info][class,load] allegrex.format.elf.relocation.AllegrexRelocationFixupHandler source: file:/C:/Users/john/Desktop/ghidra_9.2.2_PUBLIC/Ghidra/Processors/Allegrex/lib/Allegrex.jar```

Your breakpoints look okay, line breakpoint icon has a tick to confirm it was correctly placed.
Msg.warn works for me so there seems to be some different issue if you can get any of those method to work.

Are you sure the code you're trying to debug is actually executed? The relocations are processed after clicking OK in the import dialog. You can try adding a error("") (not Msg.error) or throw IllegalStateException("") to make it fail.

I'm not that familiar with Java / Kotlin development and feel like I'm fighting an uphill battle.

Yeah. I appreciate you're trying to contribute to the project though.

An illegal reflective access operation has occurred

You can ignore those.

compiling in IDEA

I think you mean running Gradle task ghidraInstall which you can either run from command line or through IDEA. Doing compile in IDEA won't install it into Ghidra.

Is there an easy way to know if Ghidra has loaded the Allegrex jar or if it's just loading the other files?

You can be extra sure if you delete your Allegrex dir and do ghidraInstall to make it reappear. Then dragging file should get detected as PSP Executable in import dialog.

Your breakpoints look okay, line breakpoint icon has a tick to confirm it was correctly placed.
Msg.warn works for me so there seems to be some different issue if you can get any of those method to work.

Are you sure the code you're trying to debug is actually executed? The relocations are processed after clicking OK in the import dialog. You can try adding a error("") (not Msg.error) or throw IllegalStateException("") to make it fail.

I'm not that familiar with Java / Kotlin development and feel like I'm fighting an uphill battle.

Yeah. I appreciate you're trying to contribute to the project though.

An illegal reflective access operation has occurred

You can ignore those.

compiling in IDEA

I think you mean running Gradle task ghidraInstall which you can either run from command line or through IDEA. Doing compile in IDEA won't install it into Ghidra.

Is there an easy way to know if Ghidra has loaded the Allegrex jar or if it's just loading the other files?

You can be extra sure if you delete your Allegrex dir and do ghidraInstall to make it reappear. Then dragging file should get detected as PSP Executable in import dialog.

Ok, so I'm able to add new messages to parsePspRelocationTables() in PspElfHeader.kt and load() in `PspElfLoader.kt and they print out:
2021-01-25 13:40:50 WARN  ghidra.app.util.opinion.PspElfLoader LOADED PSP BINARY
2021-01-25 13:41:11 WARN  ghidra.app.plugin.core.analysis.AllegrexAddressAnalyzer ALLEGREX CAN ANALYZE!

But nothing that I add in AllegrexElfReloationExtension.kt, ie getType gets printed out, nor in canRelocate in AllegrexElfRelocationHandler.kt - it seems like these classes aren't being used?
@kotcrab can you verify that added Msg.warn inside of those functions gets triggered for you from a clean checkout of master?

Ok, so I'm able to add new messages to parsePspRelocationTables() in PspElfHeader.kt and load() in `PspElfLoader.kt and they print out:
2021-01-25 13:40:50 WARN  ghidra.app.util.opinion.PspElfLoader LOADED PSP BINARY
2021-01-25 13:41:11 WARN  ghidra.app.plugin.core.analysis.AllegrexAddressAnalyzer ALLEGREX CAN ANALYZE!

But nothing that I add in AllegrexElfReloationExtension.kt, ie getType gets printed out, nor in canRelocate in AllegrexElfRelocationHandler.kt - it seems like these classes aren't being used?
@kotcrab can you verify that added Msg.warn inside of those functions gets triggered for you from a clean checkout of master?

Ah, I just tried on a game PRX and things get triggered - something else is getting tripped up by Kernel PRXs.
I should have tested a game PRX earlier on... sigh

Ah, I just tried on a game PRX and things get triggered - something else is getting tripped up by Kernel PRXs.
I should have tested a game PRX earlier on... sigh

Oh I see, true, I only tested with games.
Those classes are used but they won't have any references in code.
Will you try to figure this out by yourself? I don't really know where to find kernel PRXs, I guess I'd need to unpack some firmware but it's not something I'm familiar with.
At this point we need to check why Ghidra isn't using relocation handler from the plugin, there's is a bit of hacking going on there (see comment in PspElfConstants.kt). Though your file got detected by PspElfLoader so I can't really tell why without debugging.

Oh I see, true, I only tested with games.
Those classes are used but they won't have any references in code.
Will you try to figure this out by yourself? I don't really know where to find kernel PRXs, I guess I'd need to unpack some firmware but it's not something I'm familiar with.
At this point we need to check why Ghidra isn't using relocation handler from the plugin, there's is a bit of hacking going on there (see comment in PspElfConstants.kt). Though your file got detected by PspElfLoader so I can't really tell why without debugging.

I think I just figured out why:
INFO  In parsePspRelocationTables, sections: 0, relocTables: 0  (PspElfHeader.kt:18)
Which agrees with readelf:
readelf -Sr ~/PSP/Firmware/661/F0/kd/lowio.prx
There are no sections in this file.
There are no relocations in this file.

Will dig in further, I suspect it will require parsing some Sony-specific things from the LOAD header.
Apologies for the spam. I'm just happy to be able to make progress and know that my tools are setup correctly.

I think I just figured out why:
INFO  In parsePspRelocationTables, sections: 0, relocTables: 0  (PspElfHeader.kt:18)
Which agrees with readelf:
readelf -Sr ~/PSP/Firmware/661/F0/kd/lowio.prx
There are no sections in this file.
There are no relocations in this file.

Will dig in further, I suspect it will require parsing some Sony-specific things from the LOAD header.
Apologies for the spam. I'm just happy to be able to make progress and know that my tools are setup correctly.

I'm trying to implement this but I hit a wall. I need to get relocated file from prxtool to compare results of my code against prxtool. I compiled prxtool and I'm trying to relocate lowio.prx from FW 6.61 to 88000000
./prxtool -r<PHONE_NUMBER> -e -o lowio.elf -d lowio.prx
...
Error: Invalid memory address for exports (0x0200B518)
Error: Couldn't load prx file structures

Okay... then I thought to add debug output of relocated instructions, I'd at least have some data to compare.
The first relocation is R_MIPS_X_J26 so I added in FixupRelocs
COutput::Printf(LEVEL_DEBUG, "R_MIPS_X_J26 %08X %08X %08X\n", dwRealOfs, dwInst, dwData);

and got result
Debug: R_MIPS_X_J26 00000004 0800006E 0800886E

So 0800006E got turned into 0800886E, let's look in Ghidra:
First I loaded module without relocation applied:

The jump makes sense.
Next manually relocated using prxtool value:

The jump is outside module bounds.
Looking at sources I don't understand how this can be valid for any jump instruction
https://github.com/pspdev/prxtool/blob/master/ProcessPrx.C#L1600
At this point I don't know if prxtool implementation is correct, maybe I'm doing something stupid. Ideally we would have dump of a relocated module from PSP memory to compare against but I have no idea how to do that on my own.

Blocked until someone provides working implementation of those relocations.
prxtool issue https://github.com/pspdev/prxtool/issues/6 (+I don't think their current relocations impl is correct)
uofw sceKernelApplyPspRelSegment2 fails very early here https://github.com/uofw/uofw/blob/ef05dd20a3f43baa093452b4684346be4354543b/src/loadcore/loadelf.c#L808-L809 when feed lowio.prx data (I copied this to standalone program so maybe I did something wrong)
Fwiw you can take dumped memory and extract relocated modules then feed that into Ghidra.

As you can see when loaded at 8808FD00 the first relocation R_MIPS_X_J26 should result in ae 3f 02 0a

Update: we now have an implementation that's pretty likely correct. Though I'll need to work on it more before it can be merged into master.

Released v1.7 with support for this. Have fun reversing kernel modules, I regret I didn't get to this issue sooner.
Please create new issues if you spot any problems with this (feature ideas/requests are fine too btw).
Unfortunately if there's any bug in relocation processing your existing project will likely be impossible to fix.


Alexnet

加入了alexnet   运行请见alexnet.md

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: zjuter0126:x: jingyangxiangjingyangxiang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@zjuter0126 Some tiny errors mentioned above

@zjuter0126 Some tiny errors mentioned above

I have corrected my errors mentioned above.

@zjuter0126 Got it, I'm testing your PR locally to see if they are working well, please hold a minute : )

LGTM

LGTM
Thank you, the code for densenet is similar to alexnet, you can run it in a similar way.

@zjuter0126 Please remove the *.md and *.png file under the root folder, then this PR would be getting merged.

LGTM


Jekyll 4.x - Map tag included in layout issue

Hello,
I noticed an strange issue occurring only on Jekyll 4+ and if the map tag is included directly in a layout file: the map is showing the location of the current post + the posts before.
Map of first post will be correct, map of second post will display map from first post and second post, etc.

The issue is not occurring when inserting the tag directly in the post content or using jekyll 3.x. Is this something about the new jekyll cache feature?
I created a repository to reproduce the issue easily: https://github.com/Gerfaut/jekyll-maps-issue
Is there anything we can do on the jekyll-maps gem to fix that or is this meant to work like that?
Thanks a lot for your help,
Gerfaut

I believe this is because class GoogleMapTag < Liquid::Tag is always created on every tag rendering.
Hence, LocationFinder instance is kept between pages to render, and @documents is not cleaned.
To fix it, you have to ensure LocationFinder is created from scratch on every GoogleMapTag.render invocation.
The change https://github.com/ayastreb/jekyll-maps/pull/50 should fix the issue.


ReferenceID Absence in Webhook Response Payload

I have implemented the update on the reference ID bit.
But I observed that the Response/Request hitting my webhook after sending messages, does not come back with the custom ReferenceId.
There could be a reason I decided to customize the referenceId instead of using just the message[id]...Encoding of special User Data for instance.
So, it's paramount that it comes back in the webhook Callback.
Could this be looked into please?

Hi hexxondiv ,
reference ID and internal message id  can be received in ACK webbooks .
make sure to enable "Webhook on ACK" .


python310Packages.types-tabulate: 0.8.9 -> 0.8.10

Description of changes
Update to latest upstream release 0.8.10

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 178735 run on x86_64-linux 1

2 packages built:

python310Packages.types-tabulate
python39Packages.types-tabulate


updated axios and axios-ntlm to latest

latest version of axios-ntlm fixes node v18 digital envelope error
Error: error:0308010C:digital envelope routines::unsupported
updated axios to latest: 1.3.2
updated axios-ntlm to latest: 1.4.0 (uses axios 1.3.2)
also 2 tests where failing because of hardcoded response Content-Length
this request needs to be accepted to pass all tests: https://github.com/vpulim/node-soap/pull/1206

Same here, we're upgrading to node 18 and now need to --force to do any npm install...

Looking forward to get this changes

Looking forward to see this update too!

@vision10 plz fix the build

@jsdevel I have included changes from https://github.com/vpulim/node-soap/pull/1206
build should be ok

@jsdevel are we happy to merge now?

is there no one able to merge here?
axios is already at 1.4.0 since this pull request
some of the other pull requests can be closed or just need a bit of attention

idk, this seems like a sleeping law in congress

We might need to sign a competition for this PR to get merged 😆

Agreed it would be great to see this merged

Bump. This is a critical update.

Any update on this one @vpulim

FYI. In order to continue to use this package we all need to now use --force or --legacy-peer-deps until this is merged. Or we will need to find an alternative. Can we maybe delegate somebody to maintain this library since there are many of us who would contribute to resolve things like this?

+1

+1

+1

sdfghujiopm

Hi, why is axios a peer dependency any way? I can't see the reason. If we could move it as a normal dependency we wouldn't be here waiting for using axios 1.0 and this library together.

+1

Same here!
@vpulim Any chance to get this merged?

Any timeline for when this pr will be merged and released ? 🤔

Merge please this is ruining the package

We seriously need someone to accept these pull requests that are piling up, not just mine
@vpulim please give someone access to manage this repository if you are too busy
axios version has known vulnerabilities
https://security.snyk.io/vuln/SNYK-JS-AXIOS-6032459

@RJWerning can you confirm that the override works? I didn't want to risk it because it's a major version upgrade but now there's this new security vulnerability...

If I fork this repo and start maintaining it will someone also help out?
I am planning on forking it to a new repo and start maintaining it. Who is in?

@codersbet Although I may not be able to actively participate in the maintenance, I'm very supportive of the idea if you're willing to take the lead in managing it.

@vpulim , Can you please merge this one?
@codersbet , @rmarfil3 created the fork, but not sure if he is publishing it to npm!

Hi @vpulim thanks a lot for this package. When do you think it will be ready the new release?

Warning.
axios 1.6.1 has an error.
With<EMAIL_ADDRESS>I have an error in my unit test with this message: 'Cannot use import statement outside a module'.
I have reverted back to version 1.0.0 and have no errors.

Hi @savez, could please add some details, like your environment and logs?
I've tested 1.0.1 release in my project and it worked just fine. We run it in node.js 20.

Hi @vpulim thanks a lot for this package. When do you think it will be ready the new release?

New version 1.0.1 has been released.

Hi @savez, could you please add some details, like your environment and logs?
I've tested 1.0.1 release in my project and it worked just fine. We run it in node.js 20.

Yes,
I'm using node20.
But when I running jest it show an error.

If you can post you jest config and tsconfig that would help to identify the problem.
In my tests I just mock soap requests so don't really use it.
Unfortunately, I am very limited in time and can't setup test project to test it.


Make sure component coordinates are integers

If they are not integers compiling the glyf table will fail.

Coverage increased (+0.001%) to 70.535% when pulling 04fdfe42eaf674b3c293e6f8643157662b8dc775 on khaledhosny:ttglyphpen-int-component-coord into bdd0e6768fe97254895734f3bb9802d559318254 on behdad:master.

wait, but doesn't the Glyph.compileCoordinates method rounds float coordinates before packing them?
https://github.com/behdad/fonttools/blob/master/Lib/fontTools/ttLib/tables/_g_l_y_f.py#L587-L590

the auto-rounding of floats was explicitly removed from the ttGlyphPen in https://github.com/behdad/fonttools/commit/f02e771527ed080eb5571be78e7217b6ce999871

Hmm, I’m hitting it in https://github.com/behdad/fonttools/blob/master/Lib/fontTools/ttLib/tables/_g_l_y_f.py#L1077, I don’t see any code rounding component coordinates.

Oh the components!
In that case, I think the rounding should also take place in the _g_l_y_f table module, like with it is with coordinates.

maybe we could add a method called roundOffset() to the GlyphComponent object (which returns int(round(x)), int(round(y))), and use that in GlyphComponent.compile just before packing up the struct?
That's basically what we already do for GlyphCoordinates.

Implemented both of @anthrotype’s suggestions.

Coverage increased (+0.003%) to 70.537% when pulling 09abdc3e7c83fa028e349c3a688ffefeb73a100d on khaledhosny:ttglyphpen-int-component-coord into bdd0e6768fe97254895734f3bb9802d559318254 on behdad:master.

Coverage increased (+0.03%) to 70.562% when pulling b5afd862e132f038ca5752d660e92c5b9a81114c on khaledhosny:ttglyphpen-int-component-coord into bdd0e6768fe97254895734f3bb9802d559318254 on behdad:master.

LGTM


Cleanup NetBeans license header for module project.ant

Review and cleanup the license header in project.ant
Oracle donated NetBeans to the Apache Software Foundation.
Your task now is to replace the old license header that has Oracle (and/or Sun Microsystems
and/or NetBeans) in it with the standard Apache Software Foundation license header.
For an empty file that supports headers you must add the Apache license header.
Note that you have to use the proper license header for Java, .properties or HTML files.
Look at the nearby files for templates and copy-paste the license header.
When done commit the changes with a commit message that starts with first line
"[NETBEANS-54] Module Review " and a message body describing your review.
See a longer explanation about what we are doing here: https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review
Please announce in a comment when you have started work.
Files that need to be looked at in project.ant:
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/build-impl.xsl
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/build.xsl
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/build2.xsl
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/extension1.xml
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/global.properties
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/private.properties
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/private.xml
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/project-modified.xml
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/project.properties
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/project.xml
project.ant/test/unit/src/org/netbeans/spi/project/support/ant/data/test.txt

Reserved by sarvesh in https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review


Support options in Redact

Proposed changes

Support other redact options (e.g. pci, ssn, numbers)
Support multiple instances in query string (e.g., redact=pci&redact=numbers)

Context
Current redact options only supports true and false, while API Docs provides options for pci, ssn and numbers
Other information
Docs for redaction: https://developers.deepgram.com/docs/redaction

See also https://developers.deepgram.com/reference/pre-recorded

This will be considering a major breaking change so we'll consider this in our 1.0 release in the future.

@dvonthenen 👋  I'm curious what your thoughts are on this issue?

Will take a look

Addressed in PR https://github.com/deepgram-devs/deepgram-go-sdk/pull/113. Merged. Closing.


Add shader rendering support for Lattice

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15700/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15700/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15700/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15700/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15700/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15700/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15700/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15700/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15700/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15700/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15700/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


fatal error: EGL/eglplatform.h: No such file or directory

Hello,
When I try to compile the pyflex I get the following:
[  5%] Building CXX object CMakeFiles/pyflex.dir/imgui.cpp.o [ 10%] Building CXX object CMakeFiles/pyflex.dir/shadersDemoContext.cpp.o [ 15%] Building CXX object CMakeFiles/pyflex.dir/opengl/imguiRenderGL.cpp.o [ 21%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/tga.cpp.o [ 31%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/pfm.cpp.o [ 36%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/mesh.cpp.o [ 42%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/extrude.cpp.o [ 31%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/aabbtree.cpp.o [ 47%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/platform.cpp.o [ 57%] Building CXX object CMakeFiles/pyflex.dir/pyflex.cpp.o [ 57%] Building CXX object CMakeFiles/pyflex.dir/opengl/shadersGL.cpp.o [ 63%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/sdf.cpp.o [ 68%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/perlin.cpp.o [ 73%] Building CXX object CMakeFiles/pyflex.dir/opengl/shader.cpp.o [ 78%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/maths.cpp.o [ 84%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/core.cpp.o [ 89%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/png.cpp.o [ 94%] Building CXX object CMakeFiles/pyflex.dir/home/tzortzis/softgym/PyFlex/core/voxelize.cpp.o In file included from /home/tzortzis/softgym/PyFlex/bindings/opengl/../../external/glad/src/glad_egl.c:160:0,                  from /home/tzortzis/softgym/PyFlex/bindings/opengl/shadersGL.cpp:39: /home/tzortzis/softgym/PyFlex/bindings/opengl/../../external/glad/src/../include/glad/glad_egl.h:199:10: fatal error: EGL/eglplatform.h: No such file or directory  #include <EGL/eglplatform.h>           ^~~~~~~~~~~~~~~~~~~ compilation terminated.
There are other warning after than but the error I believe is here since it terminates the compilation. Do you have a solution for this??
Thanks

It seems to be related to EGL installation. Maybe try installing the following packages:
apt-get install libglfw3 libgles2-mesa-dev

It worked. Thanks a lot! :D

fedora34:dnf install libgle-devel.x86_64 libgle.x86_64可以解决


chore(version) Bump to 0.21.9

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

Coverage remained the same at 95.376% when pulling e33e84c8e849fc4485df3fd84430e5e66eb755e5 on jeromesimeon:js-chore-0.21.9 into 0dfaab42a55a253680254b4edc2d4ba43c565137 on accordproject:master.


Remove bootstrap and font-awesome

Since all icons are replaced by lego-ui, no need to include font-awesome icons
All components not apply bootstrap style, so it's better to remove it

Optimize page initial load

solved #237


Fix some typos in comments

Motivation:
I came across a few typos and then went through the entire code base
fixing similar errors when I noticed a pattern.
Modifications:
No changes to the code, just to comments.
Result:
No changes.

@swift-nio-bot test this please

@swift-nio-bot test this please

@swift-nio-bot test this please