Bug: Unknown statusCode, statusCode: 401

Describe the Bug
I can't get any Switchbot devices on homebridge, they don't show on Setting -> DEVICES as below.
(I have confirmed that token and secret are correct using curl command on macOS, I could get "statusCode":100 and full device list.)
To Reproduce
N/A
Expected Behavior
N/A
Relevant Log Output
[5/7/2023, 4:04:33 PM] [Homebridge UI] Homebridge Config UI X v4.50.2 is listening on :: port 8581
[5/7/2023, 4:04:34 PM] [HB Supervisor] Starting Homebridge with extra flags: -I -P /var/lib/homebridge/node_modules -D --strict-plugin-resolution
[5/7/2023, 4:04:34 PM] [HB Supervisor] Started Homebridge v1.6.1 with PID: 4131
[5/7/2023, 4:04:36 PM] Loaded config.json with 0 accessories and 2 platforms.
[5/7/2023, 4:04:36 PM] Loaded 0 cached accessories from cachedAccessories.
[5/7/2023, 4:04:36 PM] ---
[5/7/2023, 4:04:41 PM] Loaded plugin<EMAIL_ADDRESS>[5/7/2023, 4:04:41 PM] Registering platform<EMAIL_ADDRESS>[5/7/2023, 4:04:41 PM] ---
[5/7/2023, 4:04:41 PM] Loading 2 platforms...
[5/7/2023, 4:04:41 PM] [SwitchBot] Initializing SwitchBot platform...
[5/7/2023, 4:04:41 PM] [SwitchBot] [DEBUG] Using debugMode Logging
[5/7/2023, 4:04:41 PM] [SwitchBot] Finished initializing platform:
[5/7/2023, 4:04:41 PM] [SwitchBot] Config OK
[5/7/2023, 4:04:41 PM] [SwitchBot] [DEBUG] Using Default Refresh Rate (2 minutes).
[5/7/2023, 4:04:41 PM] [SwitchBot] [DEBUG] Using Default Push Rate.
[5/7/2023, 4:04:41 PM] [SwitchBot] Executed didFinishLaunching callback
[5/7/2023, 4:04:41 PM] Publishing bridge accessory (name: Homebridge AC3E, publishInfo: {
  username: '0E:3E:14:26:AC:3E',
  port: 51193,
  pincode: '***-**-***',
  category: 2,
  bind: undefined,
  mdns: undefined,
  addIdentifyingMaterial: true,
  advertiser: 'avahi'
}).
Setup Payload:
X-HM://xxxxxxxx
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:

    ┌────────────┐
    │ xxx-xx-xxx │
    └────────────┘

[5/7/2023, 4:04:41 PM] Homebridge v1.6.1 (HAP v0.11.1) (Homebridge AC3E) is running on port 51193.
[5/7/2023, 4:04:42 PM] [SwitchBot] Devices: undefined
[5/7/2023, 4:04:42 PM] [SwitchBot] Unknown statusCode, statusCode: 401, Submit Bugs Here: ' + 'https://tinyurl.com/SwitchBotBug
[5/7/2023, 4:04:42 PM] [SwitchBot] Headers: {"date":"Sun, 07 May 2023 07:10:16 GMT","content-type":"application/json","content-length":"26","connection":"keep-alive","x-amzn-requestid":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","x-amzn-errortype":"UnauthorizedException","x-amz-apigw-id":"xxxxxxxxxxxxxxxx"}
[5/7/2023, 4:04:42 PM] [SwitchBot] [DEBUG] Failed to Discover Devices, Error Message: "Cannot read properties of undefined (reading 'deviceList')", Submit Bugs Here: https://tinyurl.com/SwitchBotBug
[5/7/2023, 4:04:42 PM] [SwitchBot] [DEBUG] Failed to Discover Devices, Error: TypeError: Cannot read properties of undefined (reading 'deviceList')

Config for homebridge-switchbot
{
            "name": "SwitchBot",
            "credentials": {
                "token": "TOKEN",
                "secret": "SECRET",
                "notice": "Keep your Token & Secret a secret!"
            },
            "platform": "SwitchBot"
        }

Screenshots

Device and Model
QNAP TS-230
Node.js Version
v18.16.0
NPM Version
9.5.1
Homebridge Version
v1.6.1
Homebridge Switchbot Plugin Version
v2.6.2
Homebridge Config UI X Plugin Version
v4.50.2
Operating System
Docker(QTS Container Station)

I would try resetting your token and secret

@donavanbecker
Thank you for your suggestion.
I have tried resetting token and secret few times, but I get the same error message as I posted above.

I would think it is a network issue then. Since you are using docker that could be the cause.

I've run curl on the terminal on homebridge below;
curl --request GET 'https://api.switch-bot.com/v1.1/devices' \
  --header 'Authorization: token' \
  --header 't:<PHONE_NUMBER>784' \
  --header 'sign: GyzQdyUScC3LROIVd9DayJDIRMfeBbeDr9EtDRDDrH8=' \
  --header 'nonce: m0x7ril9' \
  --header 'Content-Type: application/json; charset=utf8'

The result  was "statusCode":100, and I could also get an full device list.
Does the problem still belong to docker?
Thanks.

I found that the time inside the QNAP was out of sync.
I set the time in the QTS control panel to synchronise the server and the problem is gone!


How to preserve tabs in code blocks?

For pandoc, the command line option --preserve-tabs makes pandoc preserve tabs in code blocks when converting markdown to HTML. I tried to pass that option to pandoc use by hexo-renderer-pandoc in the following ways:
Adding it to _config.yaml
pandoc:
  extensions:
    - -implicit_figures
    - +gfm_auto_identifiers+angle_brackets_escapable # Not available in pandoc 1.16
    - +pipe_tables+raw_html+fenced_code_blocks
    - -ascii_identifiers+backtick_code_blocks+autolink_bare_uris
    - +intraword_underscores+strikeout+hard_line_breaks+emoji
    - +shortcut_reference_links
  extra:
    - preserve-tabs

Adding it to node_modules/hexo-renderer-pandoc/index.js
  args.push("--preserve-tabs");

  var res = spawnSync('pandoc', args, {
    cwd: process.cwd(),
    env: process.env,
    encoding: "utf8",
    input: src
  });

But neither of them works. After hexo s -g, the generated blog page still replaces each tab with four whitespaces. So I'm wondering how to preserve tabs in code blocks for hexo-renderer-pandoc.

My suspicion is that our code block is hijacked by hexo's syntax highlight plugin and isn't processed my pandoc at all.
Can you try and see if any of the following two approaches works?

if you don't want hexo's syntax highlight, then, in your _config.yml, add

highlight:
  enable: false

to turn off hexo's syntax highlight, and use the following syntax to pass preserve-tabs to pandoc (note the colon!!)
pandoc:
  extra:
    - preserve-tabs:  # note this colon!!

Then your code block will be processed by pandoc and have tabs preserved.

if you want hexo's syntax highlight, then of course pandoc's --preserve-tabs won't be effective. In this case, add to _config.yml the following:

highlight:
  enable: false
  tab_replace: '	' # or put a tab character between the quotes instead

None of them works:
1
highlight:
  enable: false

pandoc:
  extra:
    - preserve-tabs:  # note this colon!!

2
highlight:
  enable: false
  tab_replace: '	'

3
highlight:
  enable: true
  tab_replace: '	'

I suspect that hexo-theme-tree I use hijacks the highlighting of the code blocks. Because I have uninstalled almost all unnecessary npm modules. All modules remained are as follows:
➜  blog git:(master) ✗ npm list
<EMAIL_ADDRESS>/home/searchstar/git/blog
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
I don't think there is any other module that can hijack the highlighting of code blocks.

I removed all code related to highlighting in hexo-theme-tree: https://github.com/seekstar/blog/commit/299bdcea7f26011c69c10e7dcc6c208daafe85d7
And the code blocks becomes plain:

But each tab is still replaced with four whitespaces.

I don't think a theme can hijack the rendering. I feel the issue lies somewhere else. I need your help to pin-point the location of the issue.
Can you locate the following code in node_modules/hexo-renderer-pandoc/index.js
  var res = spawnSync(pandoc_path, args, {
    cwd: process.cwd(),
    env: process.env,
    encoding: "utf8",
    input: src
  });

and change it to
  console.log(args) // ADDED
  console.log(data) // ADDED
  var res = spawnSync(pandoc_path, args, {
    cwd: process.cwd(),
    env: process.env,
    encoding: "utf8",
    input: src
  });
  console.log(res) // ADDED

Then render a minimal working example, such as the one below:
---
title: test codeblock
---

begin

```
function () {
    // <- put a tab here before the slashes
}
```

Then post the content of stdout here. By looking at the output I may be able to see what's causing the issue.

Also, please don't post urls that contain localhost (like your http://localhost:4000/2022/09/30/c-panic/). Those urls are only accessible on your own machine.

I've already tried console.log. When it is added before pandocRenderer, it prints to stdout. But when it is added in pandocRenderer, it does not print anything, even when refreshing the blog page. I suspect that the output is printed elsewhere.
By the way, the URL http://localhost:4000/2022/09/30/c-panic/ was posted to indicate the source blog post of the screenshot. It is useful if you (or someone else) clone my blog repo and hexo g && hexo s to reproduce the problem.

I accidentally found that the re-rendering of a blog post can only be triggered by pressing ctrl+s in the post in vscode. Do you have any suggestions about re-rendering all blog posts without discarding the deployment history?

Do you have any suggestions about re-rendering all blog posts without discarding the deployment history?

I accidentally found that hexo clean && hexo g works fine.
In conclusion, a solution to this issue is disabling highlight of hexo and pass --preserve-tabs to pandoc:
highlight:
  enable: false

pandoc:
  extra:
    - preserve-tabs:  # note this colon!!

And then hexo clean && hexo g and optionally hexo s or hexo d to make the changes take effect. This works for hexo-theme-tree I use out-of-the-box, i.e., there is no need to remove highlight-related code, and the theme can still highlight the tabs-preserved code blocks outputted by pandoc.
Thank you for your attention!


Nft

Description 📜
Please include a summary of the change and which issue is fixed.
Fixes #1357

Type of change 📝
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Add a new project
[] Modified or enhanced the existing projects
[] Improve the readme of any project

Domain of Contribution 📊
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using only HTML & CSS
[] Project using HTML, CSS & JS
[] Project using ReactJs

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

forgot to remove labels 4days ago..so removed now...sorry🙂


Plans for 3.23 Affliction

Hey, let me start off by saying how much I appreciate the fork!
I just wanted to check-in and ask if you are planning to update to 3.23 once the main repo is being updated with changes for 3.23?

Hey @anstaendig!
Any thanks are really appreciated - thus I know that my time is not wasted!
And of course I'm planning to update to 3.23 and moreover I'm planning to make new Sonoma-compatible builds as promised in https://github.com/phen0menon/awakened-poe-trade/issues/5

@shestpa, @alexogar, @anstaendig Hey, pinging you since you've visited this issue before. Looks like original author in collaboration with @hsource got awakened poe trade working on MacOS: https://github.com/SnosMe/awakened-poe-trade/releases
Could you check this out and if everything is okay, ping me here? So I can archive this thread and will provide more details on all possible issues to original author.


Feature request: prefill all to select the apps

If using prefill --all it may download all of the games but when you choose select-apps the games remain unselected. It would be nice to have it automatically mark the games downloaded as selected.

This should be covered in the most recent version, v1.3.2, using the new select-apps-beta.  Using CTRL-A will allow you to select all games in the list.


Edit project

Created the edit project view. You can now edit a project (etc. amount of timelines).
Might not be completely bug free, still needs better testing.

Looks fine. Second member can merge, and then I'll lay out the Edit View before the end of the sprint.

Great work, just remember to test it next week. Can merge after you fix one comment I posted


[Refactor] Remove redundant test.T().Run

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Some tests call test.T().Run only once, which is redundant.
https://github.com/ray-project/kuberay/blob/1ced2b968eabcfee4dcfa61391d307b60e46a2ed/ray-operator/test/e2eautoscaler/raycluster_autoscaler_test.go#L94
Calling another t.Run inside a test function is meaningful only when we need to run multiple test cases, as shown in the following example:
https://github.com/ray-project/kuberay/blob/1ced2b968eabcfee4dcfa61391d307b60e46a2ed/ray-operator/controllers/ray/rayjob_controller_unit_test.go#L225-L226
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Hi @MortalHappiness, I would like to contribute to this issue.

Closed this because I found that only raycluster_autoscaler_test.go has redundancy, and https://github.com/ray-project/kuberay/pull/2588 will remove it.


Add circular dependency detection

Describe the feature
Detect when 2 interfaces depend on each other, to prevent a potential infinite recursion.

Fixed in 8bcf00ea89551559c2f5065d414751aa5f0cf094


refactor: add subtask register for jira

Summary
What does this PR do?
like pr https://github.com/apache/incubator-devlake/pull/4896 and https://github.com/apache/incubator-devlake/pull/5411, add subtask register for jira
Does this close any open issues?
Closes https://github.com/apache/incubator-devlake/issues/4413
Screenshots
Include any relevant screenshots here.

Hi, @chenggui53 Thanks for the good work. I may have to postpone this PR since we are now testing the existing code for v0.19.
This PR will be reviewed, merged, and tested in the v0.20 milestone.

Hi, @chenggui53 Thanks for the good work. I may have to postpone this PR since we are now testing the existing code for v0.19. This PR will be reviewed, merged, and tested in the v0.20 milestone.

@klesh Okay, I'll try to see if the subtaskmeta register part can be further optimized or other things i can do until v0.19 is done.

Hi, @chenggui53 We are planning to code freeze next Tue 9-26, I would like to know if it works for you, do you think the PR can be updated in time?

Hi, @chenggui53 We are planning to code freeze next Tue 9-26, I would like to know if it works for you, do you think the PR can be updated in time?

@klesh Sorry, there have been a lot of chores lately, which has resulted in no updates being submitted. Can you help move it to the next version, I'll be working on an update in the next few days, but there's no guarantee it'll be merged in on 9.26.

@chenggui53 Thanks for the quick response. No problem, take your time.

@chenggui53  There is a conflict, maybe you can fix it.
And @klesh will help to merge it. I think.


State Field Not Appearing on Initial Rendering of the Expense Form

Describe the bug
This was initially reported at, https://opencollective.freshdesk.com/a/tickets/40146. The problem is that the state field within the Recipient's Address section is not displayed and once the user progress in the expense flow, when submitting the expense an error is thrown indicating the state is missing.
There might be more than one way to reproduce this one but here's one way I found.
To Reproduce
Steps to reproduce the behavior:

Go through the expense flow and fill up all the details in the Payee Information section of the expense.
Click next. Do not submit the expense.
Now go back to the collective page and click on Actions -> Submit Expense
Notice that the previous information is loaded but the State under the Recipients Address is missing.

Expected behavior
The initial loading of this form should load all the fields necessary for the expense to complete, including the state field.
Screen-cast

@Betree : I opened a new PR above where we fix this by a react hook that will load the account data after the initial render. The mistake in the previous PR was that since I tried to get the data from the apollo query and whenever the user types something it reevaluates the query. But I think this should solve the problem. 🤔


Improve the docs when running cargo docs and fix the Clippy warnings for mdast_util_to_markdown

This PR is a follow up for #127 to comply with how the docs are written in this project and to fix some of the warnings raised by Clippy when running cargo clippy --all-features --all-targets --workspace

Thanks!
I am currently locally looking through the code and refactoring such things! :)

@wooorm
Sorry I've never worked on a lib before in Rust and I figured I messed up the docs in my initial PR.
Should I close this one then ?

I think I had all of these in https://github.com/wooorm/markdown-rs/commit/e7ed52683a831f5a9fe4c692accf98f7bc9dd82c, thanks! :)
No worries, you do great work!


SkuName for TextAnalytics not working

https://github.com/Azure/bicep-registry-modules/blob/c4d951f3f392dcd1b8cc1c7d1896abeceec133a3/modules/ai/cognitiveservices/main.bicep#L36
I get the error The subscription does not have QuotaId/Feature required by SKU 'S0' from kind 'TextAnalytics' or contains blocked QuotaId/Feature. when deploying TextAnalytics.
I did find the solution to change it to S but this is not in the allowed list. Can this be added to the allowed list.

@MaxBe1985 thanks for submitting an issue. I don't see S listed as a valid option: https://learn.microsoft.com/en-us/rest/api/aiservices/accountmanagement/resource-skus/list?view=rest-aiservices-accountmanagement-2023-05-01&tabs=HTTP.
Here is another doc stating that this value should be a "letter+number code": https://learn.microsoft.com/en-us/azure/templates/microsoft.cognitiveservices/accounts?pivots=deployment-language-bicep#sku
Does setting to S pick any of the S tier options that are available to your subscription?
I will need to look into this. @jceval are you familiar with how the sku selection without a number works?

@MaxBe1985 I see that your issue is pointing to an older commit and is using our legacy module 'bicep-registry-modules/modules/ai/cognitiveservices/main.bicep'
Could you try using the module on the main branch 'avm/res/cognitive-services/account/main.bicep'
@description('Optional. The location to deploy resources to.')
param resourceLocation string = resourceGroup().location

@description('Optional. A short identifier for the kind of deployment. Should be kept short to not run into resource-name length-constraints.')
param serviceShort string = '<serviceshortname>'

@description('Optional. A token to inject into the name of each resource. This value can be automatically injected by the CI.')
param namePrefix string = '<nameprefix>'

module testDeployment './bicep-registry-modules/avm/res/cognitive-services/account/main.bicep' = {
  // scope: resourceGroup
  name: '${uniqueString(deployment().name, resourceLocation)}-test-${serviceShort}'
  params: {
    name: '${namePrefix}${serviceShort}001'
    kind: 'TextAnalytics'
    location: resourceLocation

    sku: 'S'
  }
}

@MaxBe1985 I see that your issue is pointing to an older commit and is using our legacy module 'bicep-registry-modules/modules/ai/cognitiveservices/main.bicep'
Could you try using the module on the main branch 'avm/res/cognitive-services/account/main.bicep'
@description('Optional. The location to deploy resources to.')
param resourceLocation string = resourceGroup().location

@description('Optional. A short identifier for the kind of deployment. Should be kept short to not run into resource-name length-constraints.')
param serviceShort string = '<serviceshortname>'

@description('Optional. A token to inject into the name of each resource. This value can be automatically injected by the CI.')
param namePrefix string = '<nameprefix>'

module testDeployment './bicep-registry-modules/avm/res/cognitive-services/account/main.bicep' = {
  // scope: resourceGroup
  name: '${uniqueString(deployment().name, resourceLocation)}-test-${serviceShort}'
  params: {
    name: '${namePrefix}${serviceShort}001'
    kind: 'TextAnalytics'
    location: resourceLocation

    sku: 'S'
  }
}

Thanks @jceval this solved the problem.


added wavedecn and waverecn to multilevel.py

added an n-dimensional equivalent of wavedec2, waverec2

Coverage remained the same at 0.0% when pulling 336e209b5ed7ec2f32e415eafe64e4d238986533 on grlee77:wavedecn into f7991ffb2283f1cab11e84d946f4d6b9382fee2c on rgommers:master.

@grlee77 this repo has been quiet for a while, but I intend to pick up the pace here after scipy 0.16.0 is out the door.
Will take me a while to review this though. Looks good from a quick browse.

Also note that Github doesn't send notifications if you add commits, so best to leave a comment once you've pushed something that's ready for review/merging.

Hi @rgommers,
I am not in any particular hurry to get it merged.  I just needed a reasonably fast 3D wavedecn/waverecn for personal use and thought I would contribute it back to the project.
From testing on my machine on a 256x256 image, dwtn was taking approximately 70 ms prior to these changes and only about 1ms afterwards.  This is comparable in performance to the Rice Wavelet Toolbox (https://github.com/ricedsp/rwt), but without the limitation to only 1D or 2D signals.
I also tried using a simple parallel/prange in the cython code, but for some reason this made things much slower again, so I commented it back out.  Is there any policy regarding the use of OpenMP in the project?
One thing I will try to clean up a bit is that the previous versions of downcoef/upcoef are still present in _pywt.pyx, because they are still used a couple of places aside from the dtwn/idwtn functions I modified.  I don't think it should be necessary to maintain both the old and new versions of these functions, though.

Coverage remained the same at 0.0% when pulling 4338266b382cc1373f6cd045ecee9df5757e3c3c on grlee77:wavedecn into f7991ffb2283f1cab11e84d946f4d6b9382fee2c on rgommers:master.

Coverage remained the same at 0.0% when pulling 4338266b382cc1373f6cd045ecee9df5757e3c3c on grlee77:wavedecn into f7991ffb2283f1cab11e84d946f4d6b9382fee2c on rgommers:master.

That speedup sounds very good!
There is such a policy - no OpenMP. I'd like to integrate this in Scipy, and there OpenMP isn't accepted (because it's not portable enough - see https://github.com/scipy/scipy/pull/4374#issuecomment-74235386 for example).

I recently rebased this pull request on my local machine to omit some changes made here that were unrelated to this pull request and to clean up the history, but now if I want to push the rebased code back up, it will require a fast foward.  I also updated the commit messages to meet the numpy developer guidelines.
@rgommers / @kwohlfahrt :
I am still relatively new to git.  What is the best way to proceed in this case?  Should I just do the fast-forward since this has not been reviewed yet or is it better to close this pull request and just open a new one using the rebased commits?
Also there are really two separate things contributed here that might be better served as independent pull requests:
1.) adding wavedecn and waverecn python functions
2.) rewriting the dwtn, idwtn cython code for speed.
If I need to open a new pull request anyway, should I split those out to two separate pull requests?

In principle it's OK to just force-push your rebased branch to this PR. However you are right that in this case it would make sense to have separate PRs for (1) and (2). So now closing this and opening two new PRs seems like the way to go.

OK.  closing this one now and will open separate PRs going forward


Implement prepared statements

IBM-Swift/Swift-Kuery#82

Codecov Report

Merging #18 into master will decrease coverage by 1.09%.
The diff coverage is 67.85%.

@@            Coverage Diff            @@
##           master      #18     +/-   ##
=========================================
- Coverage   74.21%   73.11%   -1.1%
=========================================
  Files           3        3
  Lines         380      424     +44
=========================================
+ Hits          282      310     +28
- Misses         98      114     +16

Flag
Coverage Δ

#SwiftKueryPostgreSQL
73.11% <67.85%> (-1.1%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/SwiftKueryPostgreSQL/Utils.swift
89.47% <100%> (+22.8%)
:arrow_up:

...es/SwiftKueryPostgreSQL/PostgreSQLConnection.swift
60.86% <58.13%> (-1.95%)
: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 f009968...e92be08. Read the comment docs.

LGTM


Add curly rule in the ESLint template

Readded the curly rule in eslint. Without it there were patterns like
if (condition) statement
instead of
if (condition) { statement }
which is considered a code smell by most companies.
(tested in aelixir over resolutions)

Closed in favor of https://github.com/datavisyn/visyn_scripts/pull/78


fix(doom): Correct negative offset when resizing terminal

This PR Fix issue #461
For some reason, vim.api.nvim_strwidth returns a larger width after resizing
at certain small window sizes. This PR adds a check for negative values in
offset and sets it to zero in those cases.
Seems like an upstream bug with nvim_strwidth.

Don't know why the ci / pandoc to vimdoc (pull_request) is failing.

Could you also fix a similar error for ‘hyper’ theme

Could you also fix a similar error for ‘hyper’ theme

What issue #? I just need the error message to answer.

yes

Looking forward to the merging of this fix


Route Resource in Route Group with Prefix only loads index method of resource controller

Laravel Version: 5.8.*
PHP Version: 7.1.3
Database Driver & Version:

Description:
Route Resource in Route Group with Prefix only loads index method of Resource Controller.
Index method loads fine , trying to access create & show methods redirects to '/home'.
Steps To Reproduce:
Route::group(
[
'namespace' => 'Admin',
'prefix' => 'admin',
],
function ) {
Route::resource('complaints', 'ComplaintController');

});

Route List
| GET|HEAD  | admin/complaints                  | complaints.index         | App\Http\Controllers\Admin\ComplaintController@index
| GET|HEAD  | admin/complaints/create           | complaints.create        | App\Http\Controllers\Admin\ComplaintController@create
Removing prefix loads all methods as it should.

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


memory leak

There's currently a memory leak somewhere, the memory usage will continue to rise up without any meaningful state size increase
The problem likely lies somewhere in the state implementation, or possibly some caches.
It's somewhat severe, it will double over roughly a 4-5 day preiod in the production environment (60k servers)

After some testing after a long time, i've realised that this is not a memory leak and just go claiming more memory than it needs from the os, and im not sure how to solve that, other than manually freeing that and maybe i also need to lower the amount of allocations as a whole throughout the bot.


core: network: reduce log level of peer drop messages

These messages are too low-level to be logged at info.

This is especially annoying right now because of https://github.com/paritytech/substrate/issues/2127.


Add support for Android Devices

Impossible

can you further explain why it is impossible?

No drivers are available for any other devices, you should do a research before asking.


The web-platform-tests tracking issue

The wpt suite was introduced to this lib in #42. It runs without crashing now and our aim is to improve the test score over time.
This tracking issue lists the high level TODOs. Single test failures are probably best reported and fixed with separate issues (either in the node bindings or the rust library).

[ ] get rid of the wpt fork (how to deal with webaudio/resources/buffer-loader.js, reinstate 8 deleted files)
[ ] how to handle the crashtest tests?
[ ] look into idlharness suite
[ ] collect results of the browser implementations to determine baseline performance
[ ] implement major missing functionality (#31 #28 ...)
[ ] improve reported errors (see #8 #54 )
[ ] check reporter, it seems to count some log lines as errors e.g. × Executing "test"
[ ] focus on foundational test failures (the-audionode-interface, the-audioparam-interface, the-audiocontext-interface, the-offlineaudiocontext-interface, processing-model, the-audiobuffer-interface)
[ ] focus on other test failures
[ ] run WPT in CI?

Thanks for the follow up, two notes:

get rid of the wpt fork (how to deal with webaudio/resources/buffer-loader.js, reinstate 8 deleted files)

I think this can be done by

Implementing the context.decodeAudioData(response, successCallback, errorCallback) signature, which should be rather simple to do on the JS side
Providing a mock to XMLHttpRequest to the runner using raw fs.readFileSync internally

collect results of the browser implementations to determine baseline performance

Maybe a bit old but there is this report: https://webaudio.github.io/web-audio-api/implementation-report.html

I have submitted a PR over at wpt for the buffer-loader.js: https://github.com/web-platform-tests/wpt/pull/43824
Let's see where that gets us

results from #42

RESULTS:

pass: 3413 (+27)

fail: 1353 (-27)

type error issues: 235 (=)

files ignored: 8 (=)

results with #61
  RESULTS:
  - # pass: 3605 (+192)
  - # fail: 1284 (-69)
  - # type error issues: 234 (-1)

results with #66
RESULTS:
  - # pass: 3778 (+173)
  - # fail: 1290 (+6)
  - # type error issues: 59 (-175)

Results with v0.41.1 (no PR needed here due to patch release, just cargo update)
  RESULTS:
  - # pass: 3858 (+80)
  - # fail: 1216 (-74)
  - # type error issues: 62 (+3)

Results from #70
  RESULTS:
  - # pass: 3856 (-2)
  - # fail: 1226 (+10)
  - # type error issues: 63 (+1)
  - # files ignored: 3 (-5)

Results from #76 (webaudio-rs v0.42 mainly):
  RESULTS:
  - # pass: 4304 (+448)
  - # fail: 782 (-444)
  - # type error issues: 59 (-4)
  - # files ignored: 3 (=)

Results from 4b2dc95
RESULTS:
  - # pass: 4394 (+90)
  - # fail: 712 (-80)
  - # type error issues: 57 (-2)

Results from other browsers.
Interesting to see they hit so many more assertions. Probably makes sense if there are worklet-node assertions for example. For more detail I will collect results per subdirectory (the-blabla-interface) and make another comparison.
Chrome 122.0.6261

./wpt run --log-mach=- --log-mach-level=info --headless --binary /Applications/Google\ Chrome.app/Contents/MacOS/Google\ Chrome chrome webaudio/

web-platform-test
~~~~~~~~~~~~~~~~~
Ran 8278 checks (8021 subtests, 257 tests)
Expected results: 8208
Unexpected results: 70
  test: 3 (2 error, 1 timeout)
  subtest: 67 (64 fail, 1 notrun, 1 precondition_failed, 1 timeout)

Firefox 123.0

./wpt run --log-mach=- --log-mach-level=info --headless --binary /Applications/Firefox\ Nightly.app/Contents/MacOS/firefox firefox webaudio/

web-platform-test
~~~~~~~~~~~~~~~~~
Ran 7662 checks (7405 subtests, 257 tests)
Expected results: 7097
Unexpected results: 565
  test: 10 (9 error, 1 timeout)
  subtest: 555 (526 fail, 24 notrun, 5 timeout)

Safari 17.3.1

./wpt run --log-mach=- --log-mach-level=info --headless --channel stable safari webaudio/

web-platform-test
~~~~~~~~~~~~~~~~~
Ran 8268 checks (8011 subtests, 257 tests)
Expected results: 8158
Unexpected results: 110
  test: 14 (5 error, 9 timeout)
  subtest: 96 (66 fail, 24 notrun, 6 timeout)

Results from #82 #85 and https://github.com/orottier/web-audio-api-rs/pull/455 https://github.com/orottier/web-audio-api-rs/pull/454 https://github.com/orottier/web-audio-api-rs/pull/452 https://github.com/orottier/web-audio-api-rs/pull/456
  RESULTS:
  - # pass: 4467 (+73)
  - # fail: 736 (+24)
  - # type error issues: 58 (+1)
  - # files ignored: 2 (-1)

I have made the initial quantitative comparison now per directory (per node)

Methodology:

our score = # pass + # type error
total = the reported number of subtests via the official wpt runner

With #87 and #88 and web-audio-api-rs#1.0.0-rc.1
  RESULTS:
  - # pass: 4772 (+305)
  - # fail: 1052 (+316)
  - # type error issues: 54 (-4)
  - # files ignored: 2 (Ø)

w/ #90
RESULTS:
  - # pass: 5278
  - # fail: 591
  - # type error issues: 46

w/ #104
RESULTS:
  - # pass: 6739
  - # fail: 800
  - # type error issues: 24

w/ #112
RESULTS:
  - # pass: 6927
  - # fail: 677
  - # type error issues: 4
> wpt duration: 1:50.123 (m:ss.mmm)

v0.20.0
  RESULTS:
  - # pass: 7005
  - # fail: 618
  - # type error issues: 4
> wpt duration: 2:15.084 (m:ss.mmm)

The results for the AudioWorkletNode on my machine are
npm run wpt -- --filter the-audioworklet-interface
  RESULTS:
  - # pass: 259
  - # fail: 41
  - # type error issues: 0
> wpt duration: 2:57.474 (m:ss.mmm)

But they are plagued with messages Cannot read properties of undefined (reading 'getElementsByTagName'). Should we port any browser method to NodeJs again?

But they are plagued with messages Cannot read properties of undefined (reading 'getElementsByTagName'). Should we port any browser method to NodeJs again?

Yup I guess something like but I could find where this comes from yet...

/w #132
RESULTS:
  - # pass: 7316
  - # fail: 528
  - # type error issues: 3
> wpt duration: 3:00.313 (m:ss.mmm)


Dynamic linking doesn't work

Hi,
I'm trying to use nana to make an editor interface, but I can't seem to dynamically link nana. Whenever I try to generate dynamic nana files, however, all I get is a lone nana_d.dll file with no .lib to go with it. This means nothing in the library can be used.
I've looked through the project and I can't find any use of, for example, the __declspec macro anywhere. Does nana not support dynamic usage? It seems I have a choice between a large nana_d.lib file or a useless nana_d.dll file, plus there's the fact that with static linking I'd have to swap out other external dependencies.

Sorry, only static linkage is supported.

Are there any plans to support dynamic linking in the future?

@cnjinhao  I am currently working on the dynamic build, I will be testing it in my linux this weekend

@frostbane good job! IMO, the key of the dynamic linking is to share the singleton data between modules(DLL or so).

#265
added modifications on the CMakeLists.txt
checked in windoze and debian

@cnjinhao That may be a complex job to share data between modules across dll/so boundary.

yup sharing singleton and static data will be a pain


feat: change base url, add backup url

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

todo: separate the major version change for azure-pipelines-task-lib in a new PR

Moved to https://github.com/snyk/snyk-azure-pipelines-task/pull/198


multiprocess insert data error

It occurs when I tried to insert data to clickhouse with more than 30 processes. If I set the process pool to 30, the error disappear.
The strange part is I can read data with 200 or even more processes, but it failed if I tried to insert data with only more than 30 processes.
Code I use to create client and insert data
import os
import pandas as pd
import clickhouse_connect
from multiprocessing import Pool

clickhouse_connect.common.set_setting("autogenerate_session_id", False)

def write(file):
    df = pd.read_csv(file)
    client = clickhouse_connect.get_client(host=self.host, port=self.port, user=self.user, password=self.password, connect_timeout=60, send_receive_timeout=600)
    client.insert_df(table="test_table", df=df, database="test_db")

pool = Pool(processes=50)

files = os.listdir("/data/")
for file in files:
    pool.apply_async(write, (file,))
pool.close()
pool.join()

clickhouse-connect and/or ClickHouse server logs
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connectionpool.py", line 715, in urlopen                                                                                httplib_response = self._make_request(                                                                                File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connectionpool.py", line 416, in _make_request                                                                                conn.request(method, url, **httplib_request_kw)                                                                                File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connection.py", line 244, in request                                                                                super(HTTPConnection, self).request(method, url, body=body, headers=headers)                                                                                File "/usr/local/python3/lib/python3.9/http/client.py", line 1285, in request                                                                                self._send_request(method, url, body, headers, encode_chunked)                                                                                File "/usr/local/python3/lib/python3.9/http/client.py", line 1331, in _send_request                                                                                self.endheaders(body, encode_chunked=encode_chunked)                                                                                File "/usr/local/python3/lib/python3.9/http/client.py", line 1280, in endheaders                                                                                self._send_output(message_body, encode_chunked=encode_chunked)                                                                                File "/usr/local/python3/lib/python3.9/http/client.py", line 1079, in _send_output                                                                                self.send(chunk)                                                                                File "/usr/local/python3/lib/python3.9/http/client.py", line 1001, in send                                                                                self.sock.sendall(data)                                                                                ConnectionResetError: [Errno 104] Connection reset by peer

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/clickhouse_connect/driver/httpclient.py", line 414, in _raw_request
    response = self.http.request(method, url, **kwargs)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/request.py", line 81, in request
    return self.request_encode_body(
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/request.py", line 173, in request_encode_body
    return self.urlopen(method, url, **extra_kw)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/poolmanager.py", line 376, in urlopen
    response = conn.urlopen(method, u.request_uri, **kw)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connectionpool.py", line 799, in urlopen
    retries = retries.increment(
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/util/retry.py", line 550, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/packages/six.py", line 769, in reraise
    raise value.with_traceback(tb)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connectionpool.py", line 715, in urlopen
    httplib_response = self._make_request(
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connectionpool.py", line 416, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/urllib3/connection.py", line 244, in request
    super(HTTPConnection, self).request(method, url, body=body, headers=headers)
  File "/usr/local/python3/lib/python3.9/http/client.py", line 1285, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/local/python3/lib/python3.9/http/client.py", line 1331, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/local/python3/lib/python3.9/http/client.py", line 1280, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/local/python3/lib/python3.9/http/client.py", line 1079, in _send_output
    self.send(chunk)
  File "/usr/local/python3/lib/python3.9/http/client.py", line 1001, in send
    self.sock.sendall(data)
urllib3.exceptions.ProtocolError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))

The above exception was the direct cause of the following exception:                                                                                Traceback (most recent call last):
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/clickhouse_connect/driver/client.py", line 640, in insert_df
    return self.insert(table,
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/clickhouse_connect/driver/client.py", line 609, in insert
    return self.data_insert(context)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/clickhouse_connect/driver/httpclient.py", line 255, in data_insert
    response = self._raw_request(block_gen, params, headers, error_handler=error_handler, server_wait=False)
  File "/home/datateam/clickhouse_venv/lib/python3.9/site-packages/clickhouse_connect/driver/httpclient.py", line 425, in _raw_request
    raise OperationalError(f'Error {ex} executing HTTP request attempt {attempts} {self.url}') from ex
clickhouse_connect.driver.exceptions.OperationalError: Error ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer')) executing HTTP request attempt 2 http://<IP_ADDRESS>:8123

Configuration
Environment

clickhouse-connect version: 0.7.8
Python version: 3.9.18
Operating system: centos 7.9

ClickHouse server

ClickHouse Server version: <IP_ADDRESS>

Due to keep alive handling in HTTP, you are likely to get occasional disconnects from the ClickHouse server when Keep Alive sessions time out on the server while the insert is in flight.  These inserts should probably be retried (also, that many concurrent inserts will lead to many other issue -- like "too many parts", and as a best practice you should prefer fewer but larger inserts.)
It may help to expand the urllib 3 connection pool by providing your own pool manager as described here with a larger value, but in general when working any large number of queries, some connection resets are inevitable and your application should be prepared to handle and retry such failures.

You should also update your ClickHouse version. :)


Handle more Biomodels models

Here's links to all the biomodels that currently fail when running PySCeS:
pysces_failures.csv
(Many of these are probably due to #6; putting this here so I don't forget to update it when that gets fixed.)

Thanks Lucian. New PySCeS release is currently waiting on a bug fix in numpy.f2py for Python 3.12 (https://github.com/numpy/numpy/issues/25000), which has been refactored to use meson due to the deprecation of distutils. As soon as this is fixed and I can build PySCeS again, I'll cut a release.

Quoting @luciansmith  from #6 here

The CI is indeed supposed to test everything, but it's always worth checking by hand. Which I did, and it worked! Congratulations!
https://run.biosimulations.org/runs/663eabc116d2b6badabee790
I also tried an updated OMEX file with 297 in it, which successfully ran, but which failed to produce output because of one of the updates: it asks for output from variables with assignment rules (i.e. 'Cdh1in'), and that failed. I'm not sure if that's a problem with the wrapper or PySCeS itself?
https://run.biosimulations.org/runs/663eac7216d2b6badabee7e1#tab=log
The following variable targets cannot not be recorded:
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='Cdh1in']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='IEin']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='Mcmin']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='Mih1']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='SBFin']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='Swe1T']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='kmih']
  - /sbml:sbml/sbml:model/sbml:listOfParameters/sbml:parameter[@id='kswe']

Targets must have one of the following SBML ids:
  - BE
  - BUD
  - Cdc20
  - Cdc20a
  - Cdh1
  - Cdh1tot
[...]

However, this problem pales in comparison to the fact that I filed this issue almost a year ago, and the issue has been solved! The build system now runs fine, everything gets pushed to biosimulations, and the results can be compared to other simulators again! Thank you so much for all the work you put into making this possible; I was very lost at the beginning. I'm going to close this, re-run all the biomodels from my revamped temp-biomodels repository, and update #7 with the new list. Thank you again!

This is most definitely a wrapper issue. There have been major changes and fixes to the way PySCeS deals with assignment rules in version 1.1.1, and also on 1.2.0 and 1.2.1 there have been fixes to this functionality (the latter mainly having to do with substring matching). The wrapper in all eventuality predates this. The data are there:
>>>mod = pysces.model('BIOMD0000000297_url.xml.psc', '.')
>>>mod.__settings__['cvode_return_event_timepoints'] = False
>>>mod.doSim(end=140, points=200)
>>>mod.sim.columns
Index(['Time', 'Trim', 'Sic', 'Clb', 'PTrim', 'PClb', 'SBF', 'IE', 'Cdc20a',
       'Cdc20', 'Cdh1', 'Swe1', 'Swe1M', 'PSwe1', 'PSwe1M', 'Mih1a', 'Mcm',
       'BE', 'Cln', 'mass', 'R1', 'R2', 'R3', 'R4', 'R5', 'R6', 'R7', 'R8',
       'R9', 'R10', 'R11', 'R12', 'R13', 'R14', 'R15', 'R16', 'R18', 'R19',
       'R20', 'R21', 'R22', 'R23', 'R24', 'R25', 'R26', 'R27', 'R28', 'R29',
       'R30', 'R31', 'R32', 'R33', 'R34', 'R35', 'R36', 'R37', 'R38', 'R39',
       'R40', 'R41', 'R42', 'R43', 'R44', 'R45', 'R46', 'R47', 'R48', 'R49',
       'R50', 'R51', 'R52', 'kswe', 'kmih', 'SBFin', 'IEin', 'Cdh1in', 'Mih1',
       'Mcmin', 'Swe1T'],
      dtype='object')

mod.sim contains the simulation results. By default this is a numpy.recarray, but if the user has pandas installed, this can be changed to output a pandas DataFrame by setting mod.enableDataPandas(), which was done here. It is clear that the variables are all tracked by PySCeS.
I played no part in writing the wrapper for PySCeS, so would not really be able to help there unless I get into the code from scratch. But perhaps this info is sufficient to easily update the wrapper. Perhaps this should go into a new issue?

Thanks for the info!  And yes, let's maybe use this issue (#7) to find particular problems and file those as different smaller-chunk issues?
The problem with the wrappers is that the person who wrote all the wrappers (Jonathan Karr) left to go work for industry, so we've been very slowly building back up wrapper expertise as we go.  I can certainly take a look; hopefully the fix will be straightforward.

Continued in #5 as that issue pertains to the present bug.

So!  PySCeS version 1.2.2 is now up and running on biosimulators.org.  I have not yet tackled the issue of exporting variables from assignment rules, but it finally looks like I can do that, and hopefully it'll be a relatively straightforward fix?
Even before that, though, I ran all the biomodels through 1.2.2, and got successful runs for 345 models, where before I only succeeded on 241.  Progress!  Thanks again for all your help.

Here's a new list of successes and failures with PySCeS.  711 now successfully run!
pysces_compare.csv

Great, thanks for your efforts! :smile:


[Android] Debugging in Android

Hello,
is there any tutorial on how to debug RN code in android - especially core code?
@ide @brentvatne If you can gimme a hint as to how to actually do it, I can write a PR for the docs :)
Thanks
Philipp

The gist is: build from source, use Android Studio, click the button with the Bug button, use breakpoints. I I think it's a pretty normal workflow but wish that building from source were a lot faster so people would do it a lot more -- it takes about 10-15 seconds to get a basic app on a device/simulator.

Debugging core RN code (Java I assume?) is the same as debugging any other Android application. We'd definitely appreciate some docs on it for developers working on native modules!

@astreet I was a little bit confused concerning the setup, but this thread helped me:
https://github.com/facebook/react-native/issues/3337
Especially the first answer from @chirag04 was extremely helpful and it would be nice to have it documented for other users as well I think.


Mobile Safari reader view doesn't show article content

e.g. https://wellcomecollection.org/articles/Y4h5jBEAAB0AHNbs only displays the title and contributor info


Problems with ladder games

I'm seeing some strange behaviour in the ladder games.

When I play a game using the Play as Humans button, it's never added to my matches. (Against a human opponent and I click submit of course.)

When I click on the 'Defeat the ogres' link to see what went wrong in a match that I lost, I always win. I've must have tried this over 20 times today and yesterday, and won ALL of them.

That can't be right..

Returning 404s or bogus json objects in response to level.session or thang.type causes the simulation software to lock up.  So whatever is going on here is more subtle than simple network failures.

More data.  Rerunning the bad match ups in the simulator (tested in chrome and IE) gives correct/reasonable results.

So it's possible that the bad matchups are being caused by a bad simulation client being unable to load certain code properly and incorrectly erroring out those players, and we only see intermittent bad matches due to this because many correct simulation clients are also simulating?

I've noticed that when i lose a match and click it to see it. I typically always win the match.

I've hooked it up so that we are tracking who simulates which matches, so if you see a bad match, hover over it and look for patterns in who is simulating it using which browser, etc.

I found and fixed two sources of possible error so far:

The random seeds were not taking into account the number of times each session had been submitted for real-time playback in the Simulator, whereas when playing, the random seeds would take that into account–but differently depending on which side you were playing, so even two players watching the same match between themselves would see a different random seed. I have taken the submission count out of the random seeds for now; it was only really interesting for sync PVP, anyway, so I'll fix it when we get that going again.

chess was right–the match playbacks were remembering each players' flag history, but the Simulator wasn't taking those into account. Most players didn't use flags, but some of them did, and this would cause discrepancies. Now the Simulator also remembers the last flags placed by each player. This isn't perfect, because the flags can be updated independently of a code resubmission, but that should be an even smaller problem similar to when players submit again and you can't see the code they had at the time your match was played with them, which we can fix by a better architecture (like in #651).

Let's keep our eyes out for more sources of discrepancy, but I'm guessing things will be a lot more accurate now.

Seen a couple more, so I guess something is still up. I'll add more analytics.

We now show the random seed that was used to simulate the match when you hover over the row, so let's see whether the random seed is the same if you ever see two matches against the same player in quick succession and they disagree. Or, if the match disagrees, see if the random seed it spits out in the JS console when replaying the match is the same as the one in the tooltip on the match result row.


Add wasmtime-specific C APIs for tables

This commit adds a suite of wasmtime_funcref_table_* APIs which mirror
the standard APIs but have a few differences:

More errors are returned. For example error messages are communicated
through wasmtime_error_t and out-of-bounds vs load of null can be
differentiated in the get API.

APIs take wasm_func_t instead of wasm_ref_t. Given the recent
decision to remove subtyping from the anyref proposal it's not clear
how the C API for tables will be affected, so for now these APIs are
all specialized to only funcref tables.

Growth now allows access to the previous size of the table, if
desired, which mirrors the table.grow instruction.

This was originally motivated by bytecodealliance/wasmtime-go#5 where
the current APIs we have for working with tables don't quite work. We
don't have a great way to take an anyref constructed from a Func and
get the Func back out, so for now this sidesteps those concerns while
we sort out the anyref story.
It's intended that once the anyref story has settled and the official C
API has updated we'll likely delete these wasmtime-specific APIs or
implement them as trivial wrappers around the official ones.

Also FWIW I've tested these through the Python/Go extensions (bindings written), so it should hopefully be working!

I agree; this feels like functionality that we won't want in wasmtime-specific APIs, but for now, this works.


Version bump

Signed-off-by: David Sergio Matusevich<EMAIL_ADDRESS>Description:
Version bump to beta

signature is still invalid


Use full schema object instead of an array for jsx-curly-spacing

See: https://github.com/yannickcr/eslint-plugin-react/issues/1290

@yannickcr it'd be great to release this as a semver-patch ASAP (assuming nothing minor or major is already in master)

Is there something which is blocking publishing a release with this PR? Not having it is blocking upgrades from eslint 4.1.1 to 4.2.0.

@ZebraFlesh in fact this PR is not necessary for upgrading to eslint 4.2.0; the v4.2.0 upgrade is necessary to avoid an error. This PR enforces the schema - without this PR, invalid configs pass.

@ljharb is it that 4.2 doesn't bail with an error but still logs an error message? That's what I experience and why I've decided to wait for this to be published before upgrading.

@lukeapage yes, exactly - it logs an error message (which is easily ignored) but no longer fails. You're welcome to wait, but it's not really "blocking", it's just slightly inconvenient :-)

That's exactly what I'm talking about: I don't want to try and explain to everyone why there are new errors in the logs and why they're okay.

error messages shouldn't be considered "ok" or "can be ignored", that's what warning messages are for
it would be good to get a resolution for this, rather than have these warnings show up

This is a warning message (from eslint 4), not an error message.

can we please get a ping on this thread, when the next release goes out with this fix?

If, like me, you're annoyed by the error message, my recommendation would just be to replace your old file with the patched one manually for now.
Quick way to do it (from your project directory):
curl -o ./node_modules/eslint-plugin-react/lib/rules/jsx-curly-spacing.js "https://raw.githubusercontent.com/jseminck/eslint-plugin-react/75fb917c1ce27540f48cbe3fa7436a6822d4b953/lib/rules/jsx-curly-spacing.js"

The new release will come around soon enough, assuming you don't clear out your modules too often this will clear up your terminal a bit until the next version rolls around. 🙂

Please publish this on NPM asap.

Released in v7.2.0, really sorry for the delay.

@yannickcr Thank you!

Thank you for the release!!


⚠ Jenkins Plugins Artifactory has degraded performance

In c04173d, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1210 ms

Resolved: Jenkins Plugins Artifactory performance has improved in d3cc659 after 12 minutes.


Restore grub artifacts to leap

tumbleweed seems to have some issues

change lgtm, but wasn't there an issue in leap about arm and amd ... that it used to ship all of them on the amd package or something like that?

mostly that tumbleweed didnt ship both arches while leap did. But doesnt matter becuase now the build is done with templates depending on the arch


[!] Unexpected network error

The short example code snippets work fine but on longer code I get [!] Unexpected network error. Perhaps there is a maximum length you support, or maybe the code is too complex.
Out of curiosity for whether it would work I used the Opal Ruby to JavaScript transpiler to transpile a short snippet of Ruby code which ends up being much longer JS code, but still a single ES5 file. Then I tried running that through Nectar.
nectar --target mac-osx hello.js --single
hello.js.zip

We are working on python and ruby transpilation + compilation with NectarJS
I'll look  soon on code size and erros


feat: update batchmap and mapstream to use Map proto

Kindly explain what this PR does.

    e2e_suite.go:157: Redis resources are deleted
--- PASS: TestMapSuite (200.10s)
    --- PASS: TestMapSuite/TestMapStreamUDFunctionAndSink (165.09s)
PASS
ok      github.com/numaproj/numaflow/test/map-e2e       201.036s

    e2e_suite.go:157: Redis resources are deleted
--- PASS: TestMapSuite (133.92s)
    --- PASS: TestMapSuite/TestBatchMapUDFunctionAndSink (108.23s)
PASS
ok      github.com/numaproj/numaflow/test/map-e2e       134.844s


Added auto label for ChemMaster

About the PR
Added auto label for ChemMaster output depending from solutions. Thus, the label chooses the most significant solution name. If we have solution A = 50u and solution B = 51u, output will be labeled according to solution B. Label can also be manually changed. Also fixed bug when couldn't transfer solution back into container in ChemMaster.
Screenshots

Changelog
:cl: artak10t

add: Added auto labeler for ChemMaster.
add: Added popup messages for ChemMaster buttons.

label lineedit should be below the 'buffer' buttons imo

Below you mean above pill types or just below buffer buttons or below buffer buttons but inside buffer itself?

above pill types

Ready for review. Unfortunately I don't have enough UI knowledge to make it look better


Module arguments with keyword-list will be reformatted

Hi, here an example tested on master (https://github.com/doorgan/sourceror/commit/e8eaf2c378bac6993e581a7b83d6a79f808c9611)
iex(2)> """
...(2)> @foo [
...(2)>   a: 1,
...(2)>   b: 2
...(2)> ]
...(2)> """ |> Sourceror.parse_string!() |> Sourceror.to_string() |> IO.puts()
@foo a: 1,
     b: 2
:ok

Hey there! This one was a bug in core Elixir, a fix was already merged and Sourceror's vendored module updated accordingly :)


TFS check-in fails with unclear error

It's a bug report originally posted to JetBrains Rider issue tracker (RIDER-15059). I'm duplicating it here, because the stacktrace is all plugin's.

I am attaching what I found in idea.log.
I am using VSTS plugin.
Please let me know if I can provide more info.
P.S. I have the 5 binaries running in my task list (I think they are related to VSTS plugin). You can see it in image attached. Is that normal? Given that I am not performing any active actions with TFS?

scratch (1).log

I will try to provide more info, if you need and I'm able to do that.


text work planning & discussion

Druid text planning
This is intended as a medium-level sketch for how we will implement multi-line text, text editing, and (eventually) rich text in druid.
This covers a number of things: the storage of text itself, the storage of style and color information, the storage of selection state, and the storage of line breaks and width measurement, as well as the relationships between these, as is  required for things like interactive editing.
Representation of text
We are going to adapt the xi-rope crate as our main way of representing text buffers. This is a not-insignificant depedency, but ultimately it feels worthwhile; the design of the rope (using smart pointers under the hood) makes it very well suited to druid's Data model, and xi-rope includes a large API for things like representing and applying edits, diffing, text search, and rich
text spans.
We will use xi-rope's Rope to store our text, and the Spans type to (later) represent rich text information.
These two types will be used together, and will be wrapped up in some druid type, with a name like TextBuffer.
Line breaking and width measurement
In order to display text to the user in a meaningful way, we need to be able to do line breaking and width measurement. A version of this is provided by piet; we will wrap this API in druid, providing our own TextLayout type.
This type will own a copy of the TextBuffer, and will also have a width; it will be responsible for both painting text as well as for converting from points on the screen (e.g. from a mouse click) to offsets in the buffer.
You will create a TextLayout from a TextBuffer and a PietText object (discussion: can we make PietText not be bound to a lifetime?); you will then be able to paint it, using a convenience method looking something like, TextLayout::draw(&self, ctx: &mut PaintCtx, point: Point, env: &Env). You will also be able to query the layout for information like the offset in the buffer that corresponds to a given point in the layout.
When the buffer changes elsewhere, you will update the TextLayout, by calling a method like (e.g.) TextLayout::update_buffer(&mut self, buffer: &TextBuffer) and then the TextLayout will update its internal state. An important part of this design is that the layout will be able to update itself incrementally, by comparing the old and the new buffers; however for our first pass we may just recompute the entire layout from scratch.
This TextLayout type will be the basis for all text that is displayed in druid; we will discourage the use of piet directly.
Selection state and editing
The last major component to manage will be selection state. This is slightly tricky; in xi, for instance, we treat selection state as a property of the 'view' (which corresponds to our TextLayout, above) but this might not work in druid. The main question is whether or not selection state should be part of the data model, or whether it is just part of the view state. I'm of two minds. The main question seems to be whether or not it is reasonable during normal use for the selection state to be modified elsewhere in the application.
I see a number of options: the selection state could be part of the TextBuffer, it could be part of the TextLayout, or it could be independent?
In any case, selection exists at the intersection of the buffer and the layout; it is stored in terms of offsets into the buffer, but it can only be modified with access to the layout (for instance: if we press the 'down' arrow, how do we know what offset in the buffer corresponds to our current horizontal position on the subsequent line?)
Regardless of where the selection is stored, edit operations can be considered as functions of the form, Fn(TextBuffer, Selection, &TextLayout, &EditOp) -> (TextBuffer, Selection). That is: it takes a buffer and a selection, and referencing the layout produces a new buffer and a new selection.
Rich text
TextBuffer will be able to store style (font, color, size) information for regions of its content. This is not going to be fully exposed initially; for the time being a given buffer + layout must use a single style.
As follow-up work, however, we will want to allow proper rich text. This is going to require some additional work in piet, and also a major design decision: should piet handle the layout of rich text (requiring us to expose our TextBuffer and related styling types there) or should we implmenet a lower-level piet api that operates on glyph runs, but which requires more work in druid in order to handle actual line breaking?
This is a complicated question; it makes sense for piet to be as low level an api as is reasonable, but moving to using runs is potentially a reasonable large project, and it may not offer any real efficiency improvements over an approach that just relies on directwrite/coretext/etc to do more of the heavy lifting.
Next steps:
I'm going to play around with this general design, and see if it feels like a reasonable direction.
Open questions:
where does selection state live?: I think it should be part of either TextBuffer or of TextLayout (which owns a copy of the TextBuffer), largely because it makes it easy for TextLayout to also be responsible for drawing the selection. Between these two it comes down to whether or not we want selection state to be part of the data model, or not.
How do we handle layout for rich text? we'll cross this bridge when we get to it.
What do we need in a font api? We didn't touch on this, but an additional aspect of this work is going to be a fleshing out of the piet api for resolving and using fonts.
What other minor changes might we want in the piet api? For instance, can we make PietText not be bound by a lifetime? This may not be necessary, but might be a quality of life improvement?

There are a few things I would love to have exposed by piet which are related to your rich text subtopic. The first is the ability to modify font size, bold, italics, ect of text being passed to new_text_layout on a [index, index] level so that it is still a single layout solution with correct character spacing and such. This one is probably the most work especially with different text shaping backends that may be supported. Secondly draw_text should allow coloring different portions of a text layout in similar fashion. Finally I would like a way to control font fallback with one such usage case being the ability to provide a specific emoji set instead of relying on the system emoji font.

styling for ranges is definitely one thing we would like to do, although it's going to come after some initial work.
Explicit font fallback is something else that would be nice to have, but is also very platform dependent, and I'm going to punt on that for the time being, but it might be worth opening an issue in piet?

Other cases to think about for text editing are password and credit-card or date inputs, where the text display is separate from the input.

Would RichText also allow for clickable regions with Hover/Active states? This way one could have 'links' in their text to trigger commands etc... Or is that out of scope here?

From what I have found it is currently not possible to load custom fonts into druid. Any upcoming API should allow for loading fonts on the fly, or at the very least at startup.

yea, the ability to at least bundle fonts with your application will be part of #397.

For what it's worth, I've recently been working on kas-text for roughly this area. Current status is that multi-line text layout works, also with editing but only over a String buffer (so not appropriate for large documents), HarfBuzz shaping is integrated (optional). BIDI is next on my to-do list, then some fixes for text navigation. Rich text is also intended, but I haven't put much thought into that or how to handle large documents yet.
In my case, I have a single prepared::Text type corresponding roughly to your TextLayout; it contains three things: environment data (default font size, alignment, bounding box), the text (currently just a String), and "prepared" data (contiguous runs divided by break-points, positioned glyphs without wrapping, and finally the lines as slices over the glyph runs with offsets). This allows re-wrapping without re-shaping and is set-up to allow easy BIDI integration, but rich text may be harder.
For font-handling I currently use ab-glyph (from glyph-brush). Supporting other font libraries and glyph-renderers should be possible.
Hopefully we can collaborate on this, since text processing is a complex topic and one that should be addressable in a flexible enough way that works with most GUI toolkits. For my part I've been focussing on hacking things together to meet my design requirements, leaving a lot of clean up and testing work for later.

Tristan says on Zulip, and I agree: Godot has implemented bidi, shaping, variable fonts, OpenType features, fallback and more, and all their code is confined to a few branches which might be handy to reference: https://godotengine.org/article/complex-text-layouts-progress-report-2


The height of Editor does not decrease when I remove newlines

Firstly thank you for this API
I integrated the editor in a uitableviewcell and I increase the height of the cell from the delegate function heightDidChange.
But when I remove the newlines the height always stays the same, the heightDidChange is not called
is there a way to know the new editor height if i delete some text (newlines) to decrease cell height ?

@Abdelilahskali I can see you closed this issue.
Did you solve this issue? If so, can you share the solution please?

@Abdelilahskali  can you please share solution.

This solution work for me
Update code in "RichEditorView"
private func updateHeight() {

    runJS("RE.getHtml()") { content in

        let attributedString = NSMutableAttributedString(string: content,
                                                         attributes: [
                                                             .font: UIFont.systemFont(ofSize: 16.0)
                                                         ])

        let stringToMeasure = attributedString.string.replacingOccurrences(of: "<br>", with: "\n")
        let modifiedAttributedString = NSAttributedString(string: stringToMeasure, attributes: [
            .font: UIFont.systemFont(ofSize: 16.0)
        ])

        // Calculate the bounding rect
        let maxWidth: CGFloat = UIScreen.main.bounds.width - 30
        let options: NSStringDrawingOptions = [.usesLineFragmentOrigin, .usesFontLeading]
        let boundingRect = modifiedAttributedString.boundingRect(with: CGSize(width: maxWidth, height: .greatestFiniteMagnitude), options: options, context: nil)

        // Extract the calculated height
        let height = ceil(boundingRect.height)
        self.editorHeight = Int(height)
        print(height)

    }
}


build: move build configuration out into plugin

What this PR changes/adds
Applies the "EDC Build Plugin", which in turn takes care of configuring other Gradle plugins, configuration etc.
Why it does that
To centralise the build and it slimmer and more readable. Also, the time spent in the configuration phase significantly drops from ~30sec to ~3sec.
Further notes

Optimizations in the build plugin will come soon, which will make the build even faster and more streamlined.
I re-ran the openapi generator and merger, some new files emerged there. I'm assuming that's caused by the recent package/module renaming.
removed generating the Swagger UI, because it causes more troubles than it's worth. We may publish our yaml file to SwaggerHub in the future, if a need arises.

Should be merged after the Milestone 7 release!
Linked Issue(s)
Closes #2129
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report
Base: 64.74% // Head: 64.16% // Decreases project coverage by -0.58% :warning:

Coverage data is based on head (f3148d1) compared to base (6623720).
Patch coverage: 87.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2130      +/-   ##
==========================================
- Coverage   64.74%   64.16%   -0.59%
==========================================
  Files         792      805      +13
  Lines       16881    17043     +162
  Branches     1090     1096       +6
==========================================
+ Hits        10929    10935       +6
- Misses       5497     5652     +155
- Partials      455      456       +1

Impacted Files
Coverage Δ

...ceconnector/core/CoreDefaultServicesExtension.java
68.42% <ø> (ø)

...dataspaceconnector/core/CoreServicesExtension.java
93.93% <ø> (ø)

...onnector/core/base/CommandHandlerRegistryImpl.java
50.00% <ø> (ø)

...aspaceconnector/core/base/OkHttpClientFactory.java
92.85% <ø> (ø)

...core/base/RemoteMessageDispatcherRegistryImpl.java
14.28% <ø> (ø)

...r/core/base/agent/ParticipantAgentServiceImpl.java
100.00% <ø> (ø)

...ctor/core/event/EventExecutorServiceContainer.java
100.00% <ø> (ø)

...dataspaceconnector/core/event/EventRouterImpl.java
85.00% <ø> (ø)

...r/core/health/HealthCheckServiceConfiguration.java
100.00% <ø> (ø)

...econnector/core/health/HealthCheckServiceImpl.java
97.67% <ø> (ø)

... and 416 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Hide auto mode feature is not working

I’m using ataDisableAutoMode = true but my expectation was for this option to remove the auto option from the HK heatercooler drop-down menu.

You can try 2.3.0 beta.
Changes

removed from config.json ataDisableAutoMode, ataDisableHeatMode, ataAutoHeatMode
added to config.json ataHeatDryFanMode, ataCoolDryFanMode, ataAutoDryFanMode
cleanup
config schema updated

In this upcoming version I have added possibility to configure individual mode for HEAT, COOL and AUTO inclusive disable it. We have only one problem if disable some function because Homa app doesn't allow to remove the AUTO, COOL  or HEAT option from the HK heater/cooler drop-down menu.

2.3.0 released

unfortunatelly disabled operating mode cannot be hiden in HomeKit app due to HomeKit app limitations


New version: CometNetwork.BitComet version 1.93

Result: Installation Successful

Manifest
Add/Remove Programs

Name
BitComet
BitComet 1.93

Version
1.93
1.93

Publisher
CometNetwork
CometNetwork

ProductCode

BitComet_x64

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2808
Microsoft Reviewers: Open in CodeFlow

@denelon https://github.com/microsoft/winget-pkgs/labels/Blocking-Issue


Rework NetworkUpdate workaround

The libvirt version check which is currently to decide whether to swap
libvirt.NetworkUpdate arguments is not enough. The fix was backported to
older libvirt in RHEL/CentOS (for example in
libvirt-6.0.0-37.1.module+el8.5.0+13858+39fdc467 ), and the current version
checking code would be swapping the arguments when it's no longer needed.
This commit uses the same code as libvirt client libraries instead, it
checks if the libvirt connection supports the new NetworkUpdate call or
not. I've tested this code with both libvirt-6.0.0-37.1 and
libvirt-6.0.0-37 and in each case, the correct call was done to
libvirt.NetworkUpdate
Please make sure you read the contributor documentation before opening a Pull Request.

I've updated this to make use of the new NetworkUpdateCompat which takes care of this at the digitalocean/go-libvirt level.

Thank you @cfergeau ! I hope it gets merged soonish

Thanks. Can you cherry-pick 0b2f7083058b680210af2abf06944f0513c083f3 and 7a733d242f45a91c50d25c3f97fbf11bcbde5b31 from dmacvicar-cfergeau-network-update to get the pipeline passing?

Thanks. Can you cherry-pick 0b2f708 and 7a733d2 from dmacvicar-cfergeau-network-update to get the pipeline passing?

Done! I squashed the commit fixing linting in Remove NetworkUpdate workaround, if you prefer to keep it separate to keep authorship info, let me know!
NB: since 'Allow edits by maintainers' is checked, you could add a cfergeau remote set to git@github.com:cfergeau/terraform-provider-libvirt.git and push directly to my network-update instead of creating an intermediate origin/dmacvicar-cfergeau-network-update branch


test: Setup jenkins ci

Automate prometheus-operator e2e tests via Jenkins.

test this please

test this please

test this please

test this please

@brancz Could you re-review this PR?

lgtm :+1: nice job!


How to produce unsigned long numbers?

Hello sir. I want to compare the output with clickhouse's sipHash64, which produces UInt64 data type. But this library sometimes outputs "negative values"?
For instance, in clickhouse:
select sipHash64('hello')
gives:<PHONE_NUMBER>2830962361
And your libary gives: -8304253580878589255
Can you add a method for this?

I got it. Turns out the difference is that the first one is unsigned, the second is signed, in binary format, they're the same.


Error when user tries to join server

17.09 22:30:33 [Server] SEVERE Sep 17, 2019 10:30:33 PM me.lucko.luckperms.lib.hikari.pool.PoolBase isConnectionAlive
17.09 22:30:33 [Server] INFO WARNING: luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mysql.jdbc.JDBC4Connection@bbed24a (No operations allowed after connection closed.). Possibly consider using a shorter maxLifetime value.

https://github.com/lucko/LuckPerms/wiki/Storage-system-errors#mysql-no-operations-allowed-after-connection-closed-error

Thank you. I was unable to change my wait_timeout through godaddy but changing the maximum-lifetime value worked. Don't see an error!


Remove 5th Release Notes button card from the Data Overview page

[ ] Remove 5th Release Notes button card from Data Overview page. All release notes live on the Training & Outreach pages now.

Fixed this in PR #366


Rancher UI env_file vs volume path

Rancher Version:
v1.2.0-pre3
Docker Version:
1.12.3
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04.1 LTS - Cloud Server
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher , internal DB
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Starting a docker-compose project from the rancher UI
env_file variable seems to look in the rancher-server containers
while volumes are mount from host.
Let say :
On host I have  : /home/username/data/{several files}
and /home/username/data/.env
rancher-server is started with a volume : /home/username/data/:/data
then on the ui if I want to start a container using environment value from .env and mounting volume /home/username/data/ to /data the only working way seems to put in docker-compose.yml textbox :
env_file: /data/.env
volumes:  /home/username/data/:/data
Expected:
Is this a normal behavior ? ie :  mounting volume from host but using env_file from rancher-server containers ?
Thanks a lot,
Regards,

The Rancher compose executor service runs in the server container and interprets the environment file.  For a HA setup there is no actual guarantee that the compose file will be interpreted on any one host in particular, so using env_file is not very practical.
The actual container being run is scheduled to a host and volumes come from that host.


Doc: envar file

Why do you prompt me into a y/n and then give me no indication, why I could choose one or the other?
I strongly suggest to make clear, what is a good default.

Moved to #4273


Allow to use different Config Adapters

@snowair How about https://github.com/whoops-php/phalcon as debugbar dependency?

To enable whoops ,put it into index.php :
    new \Snowair\Debugbar\Whoops\WhoopsServiceProvider();
    $application = new \Phalcon\Mvc\Application($di);


Applications cards overflow badly

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

Expected behavior 🤔
Proper overflow behavior. Ellipses maybe? Or should this UI be a table rather than cards?
Steps to reproduce 🕹
Steps:

Context 🔦
No response
Your environment 🌎
No response

i think ellipses would solve it, but the title text should be small enough so that titles that aren't too long are always fully readable


DIP-15 specify HMAC hashing algorithm to be SHA256

closes #91

@hashengineering Can you you please review this also?


Add challenges with start position and finish zone

Multiple challenges can be set on the same arena. For now, challenges have the following properties: robot start position and finish zone position. When the robot enters the finish zone, a message is displayed.
Note that arenas and challenges are like DB tables, where challenges have a foreign key into an arena table entry.
Addresses https://github.com/FRUK-Simulator/Simulator/issues/86

Thx for reviewing! I'm planning to change the size of the arenas to be a bit more realistic and in line with the size of the robot. But I'll do that after this is merged.


[core] Update slot components to use overridesResolver part 2

Updates component starting with B to use the overridesResolver on the slots components
Part 1: #25853

I'm not sure I need to review the changes. It feels like if we break something, all the developers that are migrating from v4 will spot it 😁

Fair enough 😄 Then, I will be merging if the build is green, if there is something that needs attention I will mention you.


Task/api auth

Codecov Report

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

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage        ?   96.66%
=======================================
  Files           ?        2
  Lines           ?       30
  Branches        ?        5
=======================================
  Hits            ?       29
  Misses          ?        1
  Partials        ?        0

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 0ec7751...6271a1f. Read the comment docs.


Multi GPU training docs should show batch-size behaviour

I am fairly new to multi-GPU training, and it took me quite a while to understand that when using DDP the effective batch size scales with num_gpus * batch_size.
In fact, I thought that the batch-size that I provide to the lightning data module would be divided by the number of GPUs so that I often ran into memory errors.
Maybe this could be more clarified in the docs. https://pytorch-lightning.readthedocs.io/en/stable/advanced/multi_gpu.html

it's mentioned here: https://pytorch-lightning.readthedocs.io/en/stable/advanced/multi_gpu.html#batch-size

I did see that, but since I don't know what horovod is, I did not know if the same thing applies to DDP only.

both are different actually. accelerator="ddp|horovod" there actually means either ddp training or horovod training.

Thanks a lot for clarifying that. I still think this could be made a bit easier to understand in the docs.

yeah sure!! do send a PR to update those if it's confusing :)


Do you have plans to upgrade to swift5?

Do you have plans to upgrade to swift5?

#913 updates the project to Swift 5.

Updated. Will push 0.12 version soon


Reverse geolocation support for the frontent

Would it be possible to add ability to reverse geocode the current tracked location when updating it on the map and display it as a text and geo URI that can be easily copied or shared into another app e.g. osmand?
To give a bit of context to my use case we are group of visually disabled friends where some of us are blind others have different degree of low vision problems. We were using Google maps location sharing in the past as it also prints out the location address where available. Now people tend to stop sharing their location with google and I'd like to support that initiative. However I'd like to find out a replacement solution.
Looking at leaflet there appear to be some geocoder plugins. Most promising appears to be Leaflet Control Geocoder.
Some ideas to think about.

To avoid rate limiting by services such as nominatim we can't query too much. Perhaps limit it to one reverse geocode call per minute or 30 seconds or something similar.
How to integrate thiswith the map? Ideally it should be displayed all the time not as a popup and should adress both requirements: provide text description of location, be able to copy both the adress and the data and don't take too much space. I'm thinking about a html link.

I will spend a few days trying to add this on my own however I am not sure I have enough skills and can learn them as I need them. I'll be glad to receive hints or encouraging comments if you like the idea or if you can support it in some way.

You can get the location by clicking the little human icon and press "Navigate to". But given your requirements in might not be easy enough.

I've now started looking at the code and I have discovered there are two controls added to the leaflet. They both are titled View my location. First one requests location from the browser, the second one shows the users popup. So this is now starting to look more usefull to me.
I have not explored group shares so far, so I'll comment about what I have understood about single location sharing.
The user marker on a map consists of:

An icon (I assume that's the human icon you are reffering to me).
Nick (if this is a group share)
Speed
Relative time of recent activity
Each time update data is received from the server the marker gets updated. I can perfectly read speed and last activity time. Would it be acceptable to add an address label here and populate it with reverse geocoder result?
When I click on the speed or last time text, I can see View on map and navigate to controls being added. Navigate to is what I was looking for. The issue with these two is that these are not focusable with the keyboard and don't react to keypresses such as enter or space to activate. I think I would able to tweak these at least.

Still I'm happy to receive hints about reverse geocoding. Does it make sense what I'm trying to describe?


@Deprecated *Codec

I think we could mark all *Codec classes as @deprecated and remove them in netty 5 as you can do the same by just write a decoder and encoder and combine them via CombinedChannelDuplexHandler.
This will remove a few classes and so simplify the class-hierarchy and also remove the need for maintain them.
@trustin wdyt ?

OK, except for HttpClient/ServerCodec.

@trustin yep... I'm only talking about the "base classes".

At the end we did not do this... so let me close it


New command: m365 pp dataflow list

Usage
m365 pp dataflow list
Description
List all dataflows in the specified Power Platform environment.
Options

Option
Description

-e, --environmentName <environmentName>
The name of the environment where the dataflows are located.

--asAdmin
Run the command as admin and retrieve dataflows for environments you do not have explicitly assigned permissions to.

Examples
List all dataflows in the given Power Platform environment.
m365 pp dataflow list --environmentName Default-d87a7535-dd31-4437-bfe1-95340acd55c5

List Power Pages websites models in the given environment as admin.
m365 pp dataflow list --environmentName Default-d87a7535-dd31-4437-bfe1-95340acd55c5 --asAdmin

Default properties

ID (msdyn_dataflowid)
Name (msdyn_name)
ismanaged
statecode

Checkout: https://learn.microsoft.com/en-us/power-apps/developer/data-platform/webapi/reference/msdyn_dataflow?view=dataverse-latest
Additional Info
See our m365 pp card list command on how to implement the as admin, and how to retrieve all items from a given endpoint: https://github.com/pnp/cli-microsoft365/blob/main/src/m365/pp/commands/card/card-list.ts
In this case the API call would be ${dynamicsApiUrl}/api/data/v9.2/msdyn_dataflows

LGTM 👍
Lets open it up 🚀🚢


DEV-555 Make approve project more robust

This change is

SonarQube analysis reported 3 issues

 1 major
 2 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 4 issues

 1 critical
 3 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AdapterDeterLab.java#L424: Either log or rethrow this exception.

SonarQube analysis reported 1 issue

 1 critical

Watch the comments in this conversation to review them.


TypeScript 3.5 compatibility issue

Repro:
https://github.com/laat/scroll-behavior-polyfill-ts3.5
tsc output:
node_modules/scroll-behavior-polyfill/dist/index.d.ts:5:9 - error TS2669: Augmentations for the global scope can only be directly nested in external modules or ambient module declarations.

5 declare global {
          ~~~~~~

Found 1 error.

local patch I currently use to fix the issue

diff --git a/node_modules/scroll-behavior-polyfill/dist/index.d.ts b/node_modules/scroll-behavior-polyfill/dist/index.d.ts
index 0ef8139..5ef52be 100644
--- a/node_modules/scroll-behavior-polyfill/dist/index.d.ts
+++ b/node_modules/scroll-behavior-polyfill/dist/index.d.ts
@@ -10,4 +10,5 @@ declare global {
         };
     }
 }
+export {};

Thanks for reporting this issue. It has been fixed in the latest release and has been published to NPM.


"nice to meet you" on LG G3 D852

With latest master, I still can't run any commands in recovery mode on this LG G3 D852 phone (after adding usb rules as per #30). i was hoping that #27 would help - and it does get further than before, but it still fails with a friendly error message.
here's a debug trace:
$ sudo python3 lglaf.py --cr --debug
LGLAF.py: DEBUG: Using endpoints 83 (IN), 02 (OUT)
LGLAF.py: DEBUG: Doing KILO challenge response
LGLAF.py: DEBUG: Challenge: b'f29ae130'
LGLAF.py: DEBUG: Response: b'2f47ca81ebeee6f414263c0542c8d132'
LGLAF.py: DEBUG: KILO METR Response -> Header: b'4b494c4f4d455452000000000200000000000000000000002f5d0000b4b6b3b0', Body: b''
LGLAF.py: DEBUG: Hello done, proceeding with commands
LGLAF.py by Peter Wu (https://lekensteyn.nl/lglaf)
Type a shell command to execute or "exit" to leave.
# uname -a
Hello, I am LAF. Nice to meet you.#

not sure what that message means... anything i can do to help diagnostics here?

"Hello, I am LAF. Nice to meet you." is the standard response when the command isn't in the list of supported commands. The EXEC shell is severely limited on these newer laf binaries, likely due to it being a path to root previously. The obvious supported commands in the binary I'm looking at are: dmesg, umount, fota, gota, ls, mkdir, getenforce, ps, grep. There may be others, but these are the ones that are checked directly by stricmp (or similar.)

yeah, so that's exactly what I figured here: they locked down the commands so you can't actually run anything. i'm not sure what I can actually do with this here - i was also hoping to root this device to make it useful again. as it is now it's full of spyware and thoroughly contaminated by a number of apps that can't be uninstalled because provided by the previous carrier.
the neighborhood shady cell phone repair shop told me they could root it for 20$ so maybe i'll just go and do this instead. ;)
thanks for the support, and yes, clearly, LG is not going to get any love from me in the future, considering the amount of time I wasted on this crap. the gory details are here.
so probably not this program's fault, closing... :(

@anarcat do commands like ls work for you? If you have grep, you could read text files with grep ^ /proc/cpuinfo for example (assuming that SELinux is not locking down things too much).

On 2017-11-28 23:43:48, Peter Wu wrote:

@anarcat do commands like ls work for you? If you have grep, you could read text files with grep ^ /proc/cpuinfo for example (assuming that SELinux is not locking down things too much).

grep, ls, nothing works at all.

Sounds like pull #28 is relevant?

definitely - i confirm that --rawshell fixes the issue and i can run commands again - woot!

well, i can run some commands - but most commands are denied by the LAF. not sure i can go much further, unfortunately - as long as i can't run shell commands... i'm a still stuck and they win. :/
more precisely, i can't seem to install anything that would give me root out there. i can't write files, i can't use the mount command, ... the sh command is out there (e.g. in /system/bin) and i can see it with ls, but i can't run it.
of the commands you mentioned, the following works: ps, ls, mkdir (but does nothing), gota (same), fota (same), getenforce (says "Enforcing"). any other commands i can think of give the "Hello" message. i have tried using absolute paths to commands as well, and the S prefix (e.g. Ssh also fails).
btw, i can trigger a traceback when i mess around with the -c arguments:
$ python3 lglaf.py --debug --rawshell --cr -c '!S sh'
LGLAF.py: DEBUG: Using endpoints 83 (IN), 02 (OUT)
LGLAF.py: DEBUG: Doing KILO challenge response
LGLAF.py: DEBUG: Challenge: b'447d547a'
LGLAF.py: DEBUG: Response: b'9520395ef5261583d7a27fcb1f620fa8'
LGLAF.py: DEBUG: KILO METR Response -> Header: b'4b494c4f4d455452000000000200000000000000000000002f5d0000b4b6b3b0', Body: b''
LGLAF.py: DEBUG: Hello done, proceeding with commands
LGLAF.py: WARNING: Header field requires a DWORD, got bytes b'S'
Traceback (most recent call last):
  File "lglaf.py", line 491, in main
    payload = command_to_payload(command, args.rawshell)
  File "lglaf.py", line 451, in command_to_payload
    return make_request(command, args, body)
  File "lglaf.py", line 166, in make_request
    set_header(0, cmd)
  File "lglaf.py", line 163, in set_header
    (type(val).__name__, val)
AssertionError: Header field requires a DWORD, got bytes b'S'

is that normal?

i can run -c '!CTRL RSET' though. :)

https://github.com/Lekensteyn/lglaf#advanced-usage

Arguments are comma-separated and must either be four-byte sequences (such as \0\1\2\3) or numbers (such as 0x03020100). If no arguments are given, but a body is needed, keep two spaces between the command and argument.

Are you able to use grep to read any file? (what about !EXEC  grep --help\0'?) Note that the command must exit properly, otherwise the output is never returned. (e.g. waiting for stdin does not work. That is another caveat of --rawshell).

grep fails:
$ python3 lglaf.py --debug  --cr -c '!EXEC grep --help\0'
LGLAF.py: DEBUG: Using endpoints 83 (IN), 02 (OUT)
LGLAF.py: DEBUG: Doing KILO challenge response
LGLAF.py: DEBUG: Challenge: b'7eb4580a'
LGLAF.py: DEBUG: Response: b'7d3a3cd9a0e2f840a35c73a30fe5bdee'
LGLAF.py: DEBUG: KILO METR Response -> Header: b'4b494c4f4d455452000000000200000000000000000000002f5d0000b4b6b3b0', Body: b''
LGLAF.py: DEBUG: Hello done, proceeding with commands
LGLAF.py: DEBUG: Header: b'EXEC' b'\0\0\0\0' b'\0\0\0\0' b'\0\0\0\0' b'\0\0\0\0' b'#\0\0\0' b',\xf5\0\0' b'\xba\xa7\xba\xbc'
Hello, I am LAF. Nice to meet you.

id also fails similarly, even with two spaces (completely missed that part ;).


data get_bars 2 minutes

Hello,
I need to run a program using your library, but I need the timespan for 2 minutes candles.
I can download bars for 1 minute or 5 minutes, but I don't find the way to download for 2 minutes.I s there any option to download them??
Thank you very much,

@ingelrii Sorry, we will consider it later. But temporarily unable to support the timespan for 2 minutes

Thank you very much for your quick response. I hope it will be available soon.


WIP: fixing some issues

Codecov Report

Merging #2892 into dev will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##              dev    #2892   +/-   ##
=======================================
  Coverage   95.47%   95.47%
=======================================
  Files          73       73
  Lines       13951    13951
=======================================
  Hits        13320    13320
  Misses        631      631

Impacted Files
Coverage Δ

qiita_db/processing_job.py
72.3% <0%> (ø)
:arrow_up:

qiita_db/util.py
95.11% <100%> (ø)
:arrow_up:

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 0a2b9dd...a0a7d1f. Read the comment docs.


langchain_cohere: add cohere as a partner package

Description: adds support for langchain_cohere

overall lgtm! should update imports in the docs as well

as in in docs/docs/integrations/chat_models/cohere.ipynb etc


fix(executor): Don't hold onto intermediate execution cache across block boundaries

Overview
Drops the intermediate in-memory revm cache between blocks when executing multiple blocks in a row with the StatelessL2BlockExecutor.

Codecov Report
Attention: Patch coverage is 71.87500% with 9 lines in your changes missing coverage. Please review.

Project coverage is 77.7%. Comparing base (5c30c4c) to head (54e599d).

Files
Patch %
Lines

crates/executor/src/lib.rs
68.9%
9 Missing :warning:

Additional details and impacted files

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


[Static analysis] Integer overflow / underflow

This is a feature request for new static analysis module.
Issue: An overflow happens when the limit of the type variable is exceeded. What happens is that the value resets to zero instead of incrementing more. On the other hand, an underflow happens when you try to subtract a bigger number from the smaller one. The solution would be to use the SafeMath library or perform pre-condition checks on any math operation.
Recognition: It should not be hard to find the usage of standard arithmetic operations ('+', '-', '*', '/', '++', '--', '**', etc, but the two main concerns that I see are:

How to determine that there is a possibility that this variable could exceed his limits? Maybe check if it depends on user input?
How to check that this variable is changing his value inside the SafeMath library (it could have any name) or a function that ensures the safety and prevents overflows/underflows?

This should be solved by the SMT checker. The integration of SMT support in Remix is kind of tracked in https://github.com/ethereum/solidity/issues/3857