fix: useTradeFees infinite render

Description
⚠  I need to do more testing to confirm this fixes it, I'm not yet convinced.
Fixes an infinite loop in the useTradeFees hook, specifically, inside the useEffect that dispatches findPriceHistoryByAssetId.
cryptoPriceHistoryData is a dependency of the useEffect, and is set by an async dispatch, findPriceHistoryByAssetId (this also tells us that this method as incorrectly named, as it causes a side-effect, but that's not obvious from the "find" nomenclature.
This can put us in a state where calling findPriceHistoryByAssetId is causing itself to be called again by the useEffect, ad infinitum.
If we call the dispatch method, just return without setting trade fees.
Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A
Risk
TODO
Testing
TODO
Engineering
☝
Operations
☝
Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4592

PR #4593   👈

This comment was auto-generated by Graphite.

Closing, as I'm pretty sure this was fixed by https://github.com/shapeshift/web/pull/4592.


Possible causes of NaN values

Hi! I would like to ask a question about this part of the code where it is managed the presence of NaN values
https://github.com/aimagelab/mcmr/blob/main/main.py#L817-L824
In particular, what can cause the presence of NaN values in the mask?
Once detected and managed a NaN mask, how this can impact a proper training?

Hi Alberto, as far as I remember NaN values occur with some renderers when the camera lies inside the object that it is trying to render.
In our case, that happens if the object increases so much in size (through the scale and the location of the vertices) that some vertices exceed the distance of the camera, falling "behind" it.
In those cases, without the check you linked, the loss becomes NaN too causing all the network weights to collapse to NaN values.
It happens rarely, but it could happen, in particular in the first training steps. Those lines of code prevent the training to break.

@stefanopini many thanks for the explanation, this makes totally sense


Add another resources entry

Pull Request
Proposed Changes
Added a new entry to the Issue_Resource_Template.md
Types of changes made

[ ] Fixed Bug
[ ] Added new Features
[x ] Edited Documentations
[ ] Enhanced the working of code
[ ] Additional Changes
Mention any Additional Change here

Checklist:

[x ] My code follows the Mentioned Guidelines for the project
[x ] I have self-reviewed the changes made
[ ] I have made the changes easy to understand by adding comments
[x ] My changes does not generate new warnings/errors
[ ] I have attached relevant screenshots to display the changes made
[ ] I have made the required changes to the documentations

@all-contributors add @chuckyroh for docs


DataObject $allowed_actions property is public not private in version 3+

I believe there might be a bug in the DataObject class:
public static $allowed_actions = null;

Shouldn't this be:
private static $allowed_actions = null;

As per the documentation:
http://doc.silverstripe.org/framework/en/changelogs/3.1.0#statics-in-custom-page-classes-need-to-be-private
Even if the documentation was referring to the subclass, the subclass can't have a private static $allowed_access because the DataObject class defines the property as public.

Closed via 18683079bcd1312492238f2252933e231d14af16


Fixed a offset-by-one bug in leaderboard record deleting

For example, the old ranks are A:1, B:2, C:3, therefore totalRanks is 3.
When A is deleted, the remained are B:2, C:3 and the shifting should be from index 0 to index 1 which is (totalRanks-1) -1 instead of totalRanks-1.
When B is deleted, the remained are A:1, C:3 and the shifting should be from index 1 to index 1 which is (totalRanks-1) -1 instead of totalRanks-1.

Duplicate of https://github.com/heroiclabs/nakama/pull/255, closing without merge.


"TypeError: Super expression must either be null or a function" when built with webpack on version 2.3.9

Software version:

Browser: Google Chrome Version 85.0.4183.121 (Official Build) (64-bit)
Vue Version: 2.6.12
vue-grid-layout Version: 2.3.9

Describe the bug
When vue-grid-layout is updated from 2.3.8 to 2.3.9, my project starts to throw an error when trying to open a page where the vue-grid-layout is used:
vendor.e2d7ffc643d1eaf0ca78.js:22 TypeError: Super expression must either be null or a function
    at 0.7047a8ba9ffdfea99f38.js:8
    at 0.7047a8ba9ffdfea99f38.js:8
    at 0.7047a8ba9ffdfea99f38.js:8
    at Object.5014 (0.7047a8ba9ffdfea99f38.js:8)
    at n (0.7047a8ba9ffdfea99f38.js:8)
    at Object.bc21 (0.7047a8ba9ffdfea99f38.js:8)
    at n (0.7047a8ba9ffdfea99f38.js:8)
    at Object.<anonymous> (0.7047a8ba9ffdfea99f38.js:8)
    at Object.2af9 (0.7047a8ba9ffdfea99f38.js:8)
    at n (0.7047a8ba9ffdfea99f38.js:8)

l | @ | vendor.e2d7ffc….js:22
-- | -- | --
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | Promise.then (async) |   |
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | Lt | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | g | @ | vendor.e2d7ffc….js:22
  | i | @ | vendor.e2d7ffc….js:22
  | i | @ | vendor.e2d7ffc….js:22
  | i | @ | vendor.e2d7ffc….js:22
  | i | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | vendor.e2d7ffc….js:22
  | (anonymous) | @ | app.60df97a….js:1
  | g | @ | vendor.e2d7ffc….js:22
  | i | @ | vendor.e2d7ffc….js:22
  | i | @ | vendor.e2d7ffc….js:22
  | _t | @ | vendor.e2d7ffc….js:22
  | Ft.confirmTransition | @ | vendor.e2d7ffc….js:22
  | Ft.transitionTo | @ | vendor.e2d7ffc….js:22
  | e.push | @ | vendor.e2d7ffc….js:22
  | Wt.push | @ | vendor.e2d7ffc….js:22
  | w | @ | vendor.e2d7ffc….js:22
  | Wt | @ | vendor.e2d7ffc….js:1416
  | n | @ | vendor.e2d7ffc….js:1416
  | s._wrapper | @ | vendor.e2d7ffc….js:1416

Everything works fine on 2.3.8.
To Reproduce
Steps to reproduce the behavior:

Build a project with webpack v3
Try open a page where vue-grid-layout is used
See error

Expected behavior
No error.
Screenshots

Please provide a sample where we can reproduce the error

I had the same problem.

I had the same problem.

Same problem. "vue-grid-layout": "^2.3.8", I had to remove the caret in my package json to keep it to 2.3.8 and have it work again on deploy. In my local environment the library worked well either way, so I've no idea where the problem might be.

Can anyone try with version 2.3.10? I think it might be a problem with the version of interact.js used in version 2.3.9.

When trying to build with 2.3.10, I get the following error:
ERROR in ./node_modules/vue-grid-layout/dist/vue-grid-layout.common.js
Module parse failed: Unexpected token (10332:31)
You may need an appropriate loader to handle this file type.
|     subModification.prepareStates(modifiers);
|     state.subModification = subModification;
|     subModification.startAll({ ...arg
|     });
|   },

When downgrading back to 2.3.8, everything works fine.

@savandriy I've been trying to reproduce that error, with no luck. Can you setup a quick sample for me to test?

When trying to build with 2.3.10, I get the following error:
ERROR in ./node_modules/vue-grid-layout/dist/vue-grid-layout.common.js
Module parse failed: Unexpected token (10332:31)
You may need an appropriate loader to handle this file type.
|     subModification.prepareStates(modifiers);
|     state.subModification = subModification;
|     subModification.startAll({ ...arg
|     });
|   },

When downgrading back to 2.3.8, everything works fine.

This is still the case at 2.3.7

@savandriy I've been trying to reproduce that error, with no luck. Can you setup a quick sample for me to test?

You can try this , config & lib are from the project where I have problems
https://github.com/baboon-king/vue-grid-layout_build_error_demo

Working sample: https://github.com/jbaysolutions/vue-grid-layout-sample-v2

Version 2.3.12-legacy shouldn't have these issues

Version 2.3.12-legacy shouldn't have these issues


AutoRom download fails to contact s2roms.cc

Issue
AutoRom install appears to no longer work. Looks like the process fails to connect to s2roms.cc.
Full Stacktrace
Running AutoRom -v gives the following:
AutoROM will download the Atari 2600 ROMs in link_map.txt from atarimania.com and s2roms.cc.
They will be installed to
        /home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/multi_agent_ale_py/ROM/
for use with Multi-Agent-ALE-py. Existing ROMS will be overwritten.
Traceback (most recent call last):
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connection.py", line 169, in _new_conn
    conn = connection.create_connection(
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/util/connection.py", line 96, in create_connection
    raise err
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/util/connection.py", line 86, in create_connection
    sock.connect(sa)
OSError: [Errno 113] No route to host

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connectionpool.py", line 382, in _make_request
    self._validate_conn(conn)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connectionpool.py", line 1010, in _validate_conn
    conn.connect()
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connection.py", line 353, in connect
    conn = self._new_conn()
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connection.py", line 181, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f0f25e49b50>: Failed to establish a new connection: [Errno 113] No route to host

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/requests/adapters.py", line 439, in send
    resp = conn.urlopen(
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/connectionpool.py", line 755, in urlopen
    retries = retries.increment(
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/urllib3/util/retry.py", line 574, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='s2roms.cc', port=443): Max retries exceeded with url: /s3roms/Atari%202600/P-T/Tetris%202600%20%28Colin%20Hughes%29.zip (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f0f25e49b50>: Failed to establish a new connection: [Errno 113] No route to host'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kaleab/anaconda3/envs/mava-exp/bin/AutoROM", line 15, in <module>
    AutoROM.main(args.accept, specific_dir=args.dir)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/AutoROM/AutoROM.py", line 237, in main
    manual_downloads(installation_dirs, manual_map, checksum_map)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/AutoROM/AutoROM.py", line 98, in manual_downloads
    download = requests.get(link)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/requests/api.py", line 76, in get
    return request('get', url, params=params, **kwargs)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/home/kaleab/anaconda3/envs/mava-exp/lib/python3.8/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='s2roms.cc', port=443): Max retries exceeded with url: /s3roms/Atari%202600/P-T/Tetris%202600%20%28Colin%20Hughes%29.zip (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f0f25e49b50>: Failed to establish a new connection: [Errno 113] No route to host'))

You can see more of the stack trace here - https://github.com/instadeepai/Mava/pull/299/checks?check_run_id=3349654636 .

Hey, so s2roms.cc is just completely down. We're working on a release of AutoROM that doesn't depend on these somehow, but it looks like it's going to take awhile because no good solution exists. All the ROMs you need are available in this .rar file until things are sorted out though: http://www.atarimania.com/rom_collection_archive_atari_2600_roms.html

Extract the ROMS folder from the link by @jkterry1 , and run python -m atari_py.import_roms <path to ROMS> solves the problem from my side.

My temp fix for this:
sudo apt-get install unrar
sudo apt-get install unzip -y
wget http://www.atarimania.com/roms/Roms.rar
unrar x Roms.rar
unzip ROMS.zip
python -m atari_py.import_roms ROMS
rm Roms.rar ROMS.zip "HC ROMS.zip"

@quantumiracle Doing python -m atari_py.import_roms Roms.rar worked in py3.7 & py3.8, but failed in py3.6.

Fixed in the newest release.


http://localhost:8080/pvws  404NotFound

After I deployed the pvws.war package in /webapps/ , it creates a /pvws/ dirctory
But  I still cannot access http://localhost:8080/pvws, it suggests 404NotFound.
I tried to look at the log and found this, so what happened and what should I do?
19-Apr-2023 15:27:34.524 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployWAR Deploying web application archive [/home/zhu/apache-tomcat/webapps/pvws.war]
19-Apr-2023 15:27:35.186 SEVERE [localhost-startStop-1] org.apache.catalina.core.StandardContext.startInternal One or more listeners failed to start. Full details will be found in the appropriate container log file
19-Apr-2023 15:27:35.189 SEVERE [localhost-startStop-1] org.apache.catalina.core.StandardContext.startInternal Context [/pvws] startup failed due to previous errors
19-Apr-2023 15:27:35.200 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployWAR Deployment of web application archive [/home/zhu/apache-tomcat/webapps/pvws.war] has finished in [675] ms

What's your tomcat version? If it's 10 or higher, see #13

it is 8.5.87

It works, Thanks very much
I think it is just the matter of  jdk version. I can not successfully bulid the project or run the  binary file in tomcat under jdk11 but in jdk17 both of them works.


Revert "Configure cloud provider"

Reverts ViniciusMeyerCTT/github-actions-continuous-delivery-azure#5

ads


Created CONTRIBUTING.md file

Please check if the PR fulfills these requirements:

[ ] The PR title and message describe the solution briefly.
[ ] Documentation has been updated/added if relevant

What is the current behavior?

What is the new behavior?
Other information?

Thanks for your contribution @Poobalan1210 🥳🥳🥳


Fix identifier of Zoom installer

Adjusts the package identifier used for Zoom to match the latest download:

Thank you!


Adding ciapkg API sub-package, improved directory management, etc.

New
ciapkg.io.loadDependencies - Download and load CIAtah dependencies. Branch from class function.
ciapkg.getDirExternalPrograms - Returns the directory where external programs are stored. All functions should call this to find external program directory.
ciapkg.getDirPkg - Standardized location to obtain relevant CIAtah directories, e.g. location of default data folder.
ciapkg.api - This package will eventually contain pass-through functions for all CIAtah functions, e.g. this will allow CIAtah to keep organization clear by putting functions in related subpackages but also allow users to quickly access all functions using ciapkg.api.
Updated
ciapkg.nwb.setupNwb - Update _external_programs to call ciapkg.getDirExternalPrograms() to standardize call across all functions.
ciapkg.versionOnline - Updated to handle new VERSION file that includes datestamp on 2nd line.
example_downloadTestData - Function now calls data directory via standardized ciapkg.getDirPkg('data') to avoid placing data in incorrect folder.
createHdf5File - Close space_id, dset_id, and fid with low-level HDF5 functions before appending data with hdf5write to avoid read/write issues.
readHDF5Subset - Updated support for files with datasets that contain 2D matrices.
loadMovieList - Fixed loading HDF5 datasetname that has only a single frame, loadMovieList would ask for 3rd dimension information that did not exist.
loadNeurodataWithoutBorders - Parse the algorithm associated with the NWB signal extraction data.
saveNeurodataWithoutBorders - Function checks that yaml, matnwb, and nwb_schnitzer_lab loaded, else tries to load to make sure all dependencies are present and active. Update _external_programs to call ciapkg.getDirExternalPrograms() to standardize call across all functions. Added a check for inputs with a single signal and function returns as it is not supported.
computeCnmfeSignalExtraction_batch - Added trace origin type to output structure.
playMovie - Added feature to sub-sample movie to make display run faster for larger movies.
plotSignalsGraph - Function outputs the modified traces for parent functions to use for additional plotting behavior.
CIAtah class
A number of functions change calling to external programs from hardcoded value to calling ciapkg.getDirExternalPrograms().
loadDependencies - Calls non-class function for use in more functions without needing to load CIAtah class.
modelAddNewFolders - Added support for direct input of method type, useful for command-line or unit testing. 'Add CIAtah example folders.' now adds the absolute path to avoid path errors if user changes Matlab current directory.
modelExtractSignalsFromMovie - adding CELLMax and EXTRACT to public repo in anticipation of their repos being made public. Update _external_programs to call ciapkg.getDirExternalPrograms() to standardize call across all functions.
modelPreprocessMovieFunction - _inputMovieF0 now saved to processing subfolder.

Tested Matlab 2019b in Windows 10. Merging.


Fix typo

Thanks :octocat:
It took a while :blush:
But your commit did it to the master branch and the change should be deployed in a matter of minutes.


Disable Java WCOW tests

After discussing issues with the Java container image for Windows Server (WCOW), we were recommended to disable this test. The JDK on Windows Server has poor support for containerization, unless it's being run on a Hyper-V container (which is not the case on AKS).

Passing tests: https://github.com/ItalyPaleAle/dapr-ci-test/actions/runs/2476668129


making buttons on the reviews page more interactive

Assign me this issue under HACKTOBERFEST2023

@sagarb2003 Thanks, you're assigned.


Integration won't work after Update.

**Suddenly lost connection to all devices, worked for a very long time before. Updated addon, and HA and now it don't want to connect anymore **
Restarting the Add-On and then going back to the integration to put in the configure just to move back to the Add-On and seeing this error in the log file
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
2023-09-06 20:41:03.442  INFO  Eufy Security server listening on host <IP_ADDRESS>, port 3002
2023-09-06 20:41:05.313  INFO  Connected to station T8410P3122094E5E on host <IP_ADDRESS> and port 26407
2023-09-06 20:41:05.499  INFO  Connected to station T8010T2322090471 on host <IP_ADDRESS> and port 19961
2023-09-06 20:41:07.395  INFO  Push notification connection successfully established
2023-09-06 20:41:54.511  INFO  Client disconnected with ip: <IP_ADDRESS> port: 58132 code: 1000 reason: Normal Closure
I searched for the error here and found some relations to trusted device, but trying to follow those steps and nothing changes.
What should I do?

Same here. It seems to be happening every few weeks

Nothing is working still.. several weeks gone by

Still not working after several weeks

It’s been around 8 weeks now I’ve been unable to get this integration working. I’ve exhausted all avenues.

Still no-one is bothering to follow the bug template but everyone is complaining!


Prev/Next jump to anchor

Hi, great plugin.
What about prev/next buttons on mouse/browser or mobile phone(prev)? It get back to section but withouth animation, (on mobile when we scroll many time by sections it takes and remember every anchors so if u want go back from entire page by one click of prev button (as it looks like one page) it's hard). Is there any way to animate by sections on prev/next buttons or just turn it off? Or script only works with anchors and there is nothing to do with it?
Regards

Hi,
I'm going to be introducing new methods for previous and next shortly.


Lastlogin attribute doesn't update until login has completely succeeded without errors

It should probably update somewhere before the OAuth calls and the access checks. it's intended to show active users, and a user who logs in and encounters auth errors (eg oauth) probably shouldn't be marked as inactive.

Easyish to fix, if you know where to look in the login code. Last Login is touched whenever the internal header is displayed (BootstrapSkin::displayInternalHeader()), which is correct. I think this is just missing a relevant touch on the login action on acc.php.

#232


Vehicle unable to move normally

Hello, I have now integrated the plugin into Movebase and lowered the frequency to enable global full coverage path planning. However, after 3 to 4 rounds, the car will have two goals, which means it has a goal. When the car advances towards this goal, the car will have the next goal, and then on the way to the next goal, it will go back to the previous goal, This leads to two targets within a certain area, and the car will move back and forth, ultimately causing it to be unable to travel. The relevant video has been sent to your email, please take a look

Pro tip: you can add GIFs in your comments to show what's happening.
I think what is happening here is that your local planner sees two poses in the path, and does not "know" which one to pick first. You might want to interpolate the global plan first, before you follow it with a local planner, or configure your local planner such that it prioritizes points further along the path (if possible).

Pro tip: you can add GIFs in your comments to show what's happening.
I think what is happening here is that your local planner sees two poses in the path, and does not "know" which one to pick first. You might want to interpolate the global plan first, before you follow it with a local planner, or configure your local planner such that it prioritizes points further along the path (if possible).

So should I give up the TEB planner and choose another planner

I think the TEB planner is overkill in this context, as it is designed to a lot of the global planning for you too. I would use something like a DWA planner, or something even simpler to follow a path like this.
If you do want to use the TEB planner, I think it allows you to sample the global plan with "via points": global_plan_viapoint_sep, and add a weight to the optimization such that its plan is pulled towards the via points. That should help with getting the TEB planner to work.
Disclaimer: I have never tried this combination myself. I'd love to hear if and how this works for you.

I think the TEB planner is overkill in this context, as it is designed to a lot of the global planning for you too. I would use something like a DWA planner, or something even simpler to follow a path like this.
If you do want to use the TEB planner, I think it allows you to sample the global plan with "via points": global_plan_viapoint_sep, and add a weight to the optimization such that its plan is pulled towards the via points. That should help with getting the TEB planner to work.
Disclaimer: I have never tried this combination myself. I'd love to hear if and how this works for you.

I tried to increase or decrease the proportion coefficient, via points, etc., but it still doesn't work, and I have a question. I will give it a target point, will this code eventually stop near the target point


3 camera Crashes

Hello there,
when i try to use 3 cameras i have a freezing cam, and i can do nothing ?

And the last log :
[2017-04-26 22:44:43.825]: main - Starting PSMoveService v0.9-alpha 8.2.0 [2017-04-26 22:44:43.832]: USBAsyncRequestManager::startup - Requested LibUSBApi [2017-04-26 22:44:43.832]: USBAsyncRequestManager::startup - Creating LibUSBApi [2017-04-26 22:44:43.835]: USBAsyncRequestManager::startup - Initialized USB API [2017-04-26 22:44:43.838]: DeviceManager::startup - Platform Hotplug API is ENABLED [2017-04-26 22:44:43.841]: bluetooth_get_host_address - Found a bluetooth radio [2017-04-26 22:44:43.842]: bluetooth_get_host_address - Retrieved radio info [2017-04-26 22:44:43.910]: PSMoveController::open - Opening PSMoveController(\\?\hid#{00001124-0000-1000-8000-00805f9b34fb}_vid&0002054c_pid&03d5&col01#8&db1520c&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}) [2017-04-26 22:44:43.913]: PSMoveController::open -   with serial_number: 0007048c4a09 [2017-04-26 22:44:43.993]: DeviceTypeManager::update_connected_devices - Device device_id 0 (PSMove) opened [2017-04-26 22:44:43.994]: PSMoveController::open - Opening PSMoveController(\\?\hid#{00001124-0000-1000-8000-00805f9b34fb}_vid&0002054c_pid&03d5&col01#8&25ebb5a1&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}) [2017-04-26 22:44:43.997]: PSMoveController::open -   with serial_number: 000704a26513 [2017-04-26 22:44:44.74]: DeviceTypeManager::update_connected_devices - Device device_id 1 (PSMove) opened [2017-04-26 22:44:44.143]: TrackerDeviceEnumerator - Skipping device (USB\VID_1415&PID_2000\b1_p1.5) - Operation not supported or unimplemented on this platform [2017-04-26 22:44:44.145]: TrackerDeviceEnumerator - Skipping device (USB\VID_1415&PID_2000\b1_p1.6) - Operation not supported or unimplemented on this platform [2017-04-26 22:44:44.148]: TrackerDeviceEnumerator - Skipping device (USB\VID_1415&PID_2000\b1_p1.1) - Operation not supported or unimplemented on this platform [2017-04-26 22:44:44.151]: PS3EyeTracker::open - Opening PS3EyeTracker(USB\VID_1415&PID_2000\b1_p1.1, camera_index=0) [2017-04-26 22:44:44.451]: SharedMemory::initialize() - Allocating shared memory: tracker_view_0 [2017-04-26 22:44:44.667]: DeviceTypeManager::update_connected_devices - Device device_id 0 (PSEYE) opened [2017-04-26 22:44:44.669]: PS3EyeTracker::open - Opening PS3EyeTracker(USB\VID_1415&PID_2000\b1_p1.5, camera_index=1) [2017-04-26 22:44:44.895]: SharedMemory::initialize() - Allocating shared memory: tracker_view_1 [2017-04-26 22:44:44.904]: DeviceTypeManager::update_connected_devices - Device device_id 1 (PSEYE) opened [2017-04-26 22:44:44.907]: PS3EyeTracker::open - Opening PS3EyeTracker(USB\VID_1415&PID_2000\b1_p1.6, camera_index=2) [2017-04-26 22:44:45.160]: SharedMemory::initialize() - Allocating shared memory: tracker_view_2 [2017-04-26 22:44:45.170]: DeviceTypeManager::update_connected_devices - Device device_id 2 (PSEYE) opened [2017-04-26 22:44:56.352]: ClientConnection::start - Starting client connection id 0 [2017-04-26 22:44:56.353]: ClientConnection::send_connection_info - Sending connection id to client 0
what is wrong please ?

@ychoucha read the instructions at the beginning of that program - you're more than likely demanding too much of your USB bandwidth.
This again is something that should have been asked on the PSMoveService google group.


support bcrypt-ing passwords

The concourse 4.0.0+ basic_auth users require that their passwords are provided in bcrypt.
 add_local_users:
    description: |
      List of local concourse users to add with their bcrypted passwords.
      bcrypted password must have a strength of 10 or higher or the user will not be able to login
    default: []
    example:
    - some-user:$2a$10$sKZelZprWWcBAWbp28rB1uFef0Ybxsiqh05uo.H8EIm0sWc6IZGJu
    - some-other-user:$2a$10$.YIYH.5EWQcCvfE49xH/.OhIhGFiNtn.tQq.4pznpcrqZvoLxuKeC

Adding this to safe would make adding this into the genesis kit a lot smoother.

Well, safe fmt already exists. Who knew? I didn't.


The level of randomess

If you loop a pattern a 1000 times, you'll get no more than 16 unique values. I have been using a RandomHelper class and I made a minor modification in you project and achieved 100% unique records.
Here is the change I made:
In GenerationConfig: Look at the commented code.

The class I mentioned:

Are you aware of the default behaviour of the Random class? Did you allowed the repeat-possibility on purpose? I'm asking because there is a chance I might not have fully understood the mechanics of you library.
Thanks.

Does fail these tests though:
Can_Configure_And_Produce_Output_With_Seed
Can_Configure_And_Produce_Output_With_Seed2
GeneratePattern_Overloads2

Hi,
The library does allow you to introduce a Seed value which is used to initialize the Random object.  This allows you to produce the same output which can be useful in certain circumstances.  Typically you do that by inserting a config section at the top of the input template:
<# { 'Seed':100 } #><<\.\w\W\L\l\V\v\d\D\S\s>>

or by explicitly supplying it in the GenerationConfig object:
string text =<EMAIL_ADDRESS>new GenerationConfig() { Seed = 100 });

So deterministic output is a feature, However when you say that you are only getting 16 values back when looping over 1000 values then that doesn't sound right.
I ran a quick test using the following:
        [TestMethod]
        [TestCategory("Randomness")]
        public void Level_Of_Randomness()
        {
            var pattern = @"(\L\L\L\L\L\L-\L\L-\L\L\L\L\L\n){1000}";
            var text = AlphaNumericGenerator.GenerateFromPattern(pattern, new GenerationConfig(){Seed = 100});
            File.WriteAllText(@"c:\temp\random.txt", text);
            Console.WriteLine(@"'{0}' produced \n{1}", pattern, text);
        }

and the produced file had no duplicate values although I doubt this is a valid test for the scenario you were using.
Have you got a unit test that reproduces what you were seeing?
Also if you want to create a PR with your RandomHelper.cs changes then I can take a look at including them.
Thanks,
Gary

Could it be that I was doing a loop instead? Like:
#region Randomness
[TestMethod]
[TestCategory("Randomness")]
public void Level_Of_Randomness()
{
var pattern = @"(\L\L\L\L\L\L-\L\L-\L\L\L\L\L\n){1000}";
var text = AlphaNumericGenerator.GenerateFromPattern(pattern, new GenerationConfig() { Seed = 100 });
var segments = new List(text.Split(new[] { "\n" }, StringSplitOptions.RemoveEmptyEntries));
		Console.WriteLine(@"'{0}' produced {1} values, out of which, {2} are unique and {3} are duplicates.", pattern, segments.Count, segments.Distinct().Count(), segments.Count - segments.Distinct().Count());
	}

	[TestMethod]
	[TestCategory("Randomness")]
	public void Level_Of_Loop()
	{
		var pattern = @"(\L\L\L\L\L\L-\L\L-\L\L\L\L\L)";
		var segments = new List<string>();

		for (var i = 0; i < 1000; i++)
		{
			segments.Add(AlphaNumericGenerator.GenerateFromPattern(pattern, new GenerationConfig() { Seed = 100 }));
		}

		Console.WriteLine(@"'{0}' produced {1} values, out of which, {2} are unique and {3} are duplicates.", pattern, segments.Count, segments.Distinct().Count(), segments.Count - segments.Distinct().Count());
	}
	#endregion

You'll see that the for loop produces only one unique value and 999 are duplicates. I didn't know that  the pattern can have a segment describing loops.

Ah ok that explains it a bit more, what is happening is that when you create a new GenerationConfig object it initializes its internal Random property using DateTime.Now.Millisecond as the initial seed value.  So when you your code runs in a loop a large number of iterations get initiated with the same Millisecond value which then produces exactly the same output.
If you create the GenerationConfig object outside the loop then you don't get any duplicates:
[TestMethod]
        [TestCategory("Randomness")]
        public void Level_Of_Loop()
        {
            var pattern = @"(\L\L\L\L\L\L-\L\L-\L\L\L\L\L)";
            var segments = new List<string>();

            var config = new GenerationConfig();

            for (var i = 0; i < 1000; i++)
            {
                segments.Add(AlphaNumericGenerator.GenerateFromPattern(pattern, config));
            }

            Console.WriteLine(@"'{0}' produced {1} values, out of which, {2} are unique and {3} are duplicates.", pattern, segments.Count, segments.Distinct().Count(), segments.Count - segments.Distinct().Count());
        }

// Output ->
// '(\L\L\L\L\L\L-\L\L-\L\L\L\L\L)' produced 1000 values, out of which, 1000 are unique and 0 are duplicates.

I can see how you would expect to be able to repeatedly call GenerateFromPattern in a loop and get back correct (random) results. I might need to rework that method to not behave counter intuitively.

That was it. I can work with the workaround provided. Thanks.


Place latest contribution at top of table

Recent contributions should appear on the top. The latest contributions should be the first line in the table

The idea here would be to sort the ratings, by date in the backend


Add deploy data to heartbeat payload

In order for ChainAPI to distinguish the Airnode deployment we need to add the following data

cloud provider
region
stage
Airnode address

I tested this in conjunction with #1337 and the data in question was present. Screenshots are available in the linked issue.


feat(db-*): use custom DB adapters for globals and collections

Docs

Converting to Draft.
I need to resolve conflicts before merging.


Here is a regex that works for React and Preact JSX/TSX "class" and "className" with + without brackets =)

Had a friend who writes React ask about this and helped him figure out a working regex, here it is:
"headwind.classRegex": {
  "javascriptreact": "(?:\\b(?:class|className)?\\s*=\\s*{?[\\\"\\']([_a-zA-Z0-9\\s\\-\\:/]+)[\\\"\\']}?)",
  "typescriptreact": "(?:\\b(?:class|className)?\\s*=\\s*{?[\\\"\\']([_a-zA-Z0-9\\s\\-\\:/]+)[\\\"\\']}?)"
}

@GavinRay97 this solved my problem thank you

And for twin.macro while we’re at it:
{
  "headwind.classRegex": {
    "typescriptreact": "(?:\\b(?:class|className|tw)?\\.?\\w?\\s*=*\\s*{?[\\\"\\'\\`]([_a-zA-Z0-9\\s\\-\\:/]+)[\\\"\\'\\`]}?)",
    "javascriptreact": "(?:\\b(?:class|className|tw)?\\.?\\w?\\s*=*\\s*{?[\\\"\\'\\`]([_a-zA-Z0-9\\s\\-\\:/]+)[\\\"\\'\\`]}?)"
  }
}

The debugging example in the README is super helpful and the only thing after that is escaping the regex.

Here's a slightly more full-featured regex for twin.macro:
"headwind.classRegex": {
	"javascript": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]([\\w\\s\\-\\:\\[\\]]+)[\\\"\\'\\`]}?)",
	"javascriptreact": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]([\\w\\s\\-\\:\\[\\]]+)[\\\"\\'\\`]}?)",
	"typescript": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]([\\w\\s\\-\\:\\[\\]]+)[\\\"\\'\\`]}?)",
	"typescriptreact": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]([\\w\\s\\-\\:\\[\\]]+)[\\\"\\'\\`]}?)"
},

I'm new to regex, so I just want to break it down (more for my sanity that anything else):

(?:) is a non-capturing group, which means it should attempt to match the contents, but not return them.
(?:class|className|tw) matches class, className, or tw.
(?:=(?:{\s*)?)? optionally matches an = followed by an optional { followed by optional whitespace (i.e., tw={ or - tw={ ).
(?:\.\w*)? optionally matches a . followed by any number of letters (i.e., tw.div).
(?:\(\s*\w*\s*\))? optionally matches a ( followed by optional whitespace then any number of letters then more optional whitespace and a ) (i.e., tw(div) or tw( div )).
[\"\'\`] matches a ", ', or `.
([\w\s\-\:\[\]]+) matches any number of letters, whitespace, hyphens, and square brackets (i.e., all of the tailwind classes) and returns them.
}? optionally matches the closing } when that's needed.

That regex is wonderful, thank you @j0hnm4r5! But there's a few problems with it:

it doesn't work for interpolated strings
it doesn't support slashes like h-1/2
it doesn't support dash prefixes like -top-0.5
it doesn't support periods like right-1.5

Here's one that fixes these issues:
  "headwind.classRegex": {
    "javascript": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]((?:[\\w\\s\\-\\/\\:\\.\\[\\]]|\\$\\{(.*?)\\})+)[\\\"\\'\\`]}?)",
    "javascriptreact": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]((?:[\\w\\s\\-\\/\\:\\.\\[\\]]|\\$\\{(.*?)\\})+)[\\\"\\'\\`]}?)",
    "typescript": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]((?:[\\w\\s\\-\\/\\:\\.\\[\\]]|\\$\\{(.*?)\\})+)[\\\"\\'\\`]}?)",
    "typescriptreact": "(?:\\b(?:class|className|tw)(?:=(?:{\\s*)?)?(?:\\.\\w*)?(?:\\(\\s*\\w*\\s*\\))?[\\\"\\'\\`]((?:[\\w\\s\\-\\/\\:\\.\\[\\]]|\\$\\{(.*?)\\})+)[\\\"\\'\\`]}?)"
  },

Here is the regexr for the new regex with all passing tests: https://regexr.com/60e31

@KnifeFed no, headwind itself doesn't support JIT.

Thank you so much for your regex @bbugh.
I'm not a regexpert (aha) but something is missing in your regex:
When some tailwind classes are set as important (eg: !{class1} !{class2} ...), the regex doesn't work...
I hope you could fix it !
Thank you again :)


Code not working

The code in the coding window is right. Multiple students have had the same code. The preview though for the program does not work though? Confirmed Bug?

What is the code, for what level, and how does it not work?

It was some desert levels. But the code in the editor did not match what the hero placeholder was doing.

Still not working? If I can get some more details, I can try to debug this.


Memory leak

I'm seeing some serious CPU cranking and it looks like your plugin is the cause.

It's an issue with Sierra & Alfred-Workflow.
Run this workflow.

Thanks a lot!


ci: add workflow to release on GitHub

Stack from ghstack (oldest at bottom):

#93
-> #92

This workflow can be called either directly via the web UI or indirectly via another workflow.
This worflow always requires a Git tag as an input to create a release of this exact tag.

Please just review the PR and DO NOT merge it!


Rename repository

As we go through the hard process of renaming the packaging, we should consider renaming repositories to be more understandable as well. Thanks @dcolligan
https://help.github.com/articles/renaming-a-repository/

Closing in favor https://github.com/ga4gh/server/issues/1585


Wrong password

Since today i can not log in anymore. It always says that my password is wrong although it is correct.
I used this integration for about 2 weeks without any problems.
Today i got informed, that i have to reconfigure the integration

Please fill out the template.

I had this, had to restart HA to fix it

Can confirm I had exactly this issue today. After an update it kept telling me my password was wrong, despite me validating that it still worked.
Found this thread and the previous answer from @stueng - restarted HA and the issue went away.


duplicity: add paramiko,pycrypto,ecdsa to deps

paramiko enables sftp:// for duplicity

Pushed to master and release-15.09.

that was quick, thanks!


Research (and fix) state not found warning

We are recently seeing a warning in zebrad. We want to check it in more details and decide what to in the context of this ticket.
https://github.com/ZcashFoundation/zebra/pull/8865#issuecomment-2352971509
2024-09-16T13:37:54.535314Z  WARN zebra_state::service::finalized_state::disk_db: could not canonicalize "/var/cache/zebrad-cache/state/v25/mainnet": No such file or directory (os error 2)

In regtest mode:
2024-09-19T11:59:37.538954Z  WARN zebra_state::service::finalized_state::disk_db: could not canonicalize "/tmp/testp3siw8dm/515/node0/state/v25/regtest": No such file or directory (os error 2)

Is this only in CI?

Is this only in CI?

No, I've seen it locally as well.


handle error where react-on-rails package entry missing

This change is

Nice work @benjiwheeler! Could you add a simple rspec test? I'll merge right after that.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

@benjiwheeler you have lint errors: https://travis-ci.org/shakacode/react_on_rails/builds/174902848
try
cd client
npm run lint -- --fix
Offenses:
lib/react_on_rails/version_checker.rb:62:36: C: Style/PreferredHashMethods: Use Hash#key? instead of Hash#has_key?.
if parsed_package_contents.has_key?("dependencies") &&
^^^^^^^^
lib/react_on_rails/version_checker.rb:63:10: C: Style/MultilineOperationIndentation: Align the operands of a condition in an if statement spanning multiple lines.
parsed_package_contents["dependencies"].has_key?("react-on-rails")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/react_on_rails/version_checker.rb:63:50: C: Style/PreferredHashMethods: Use Hash#key? instead of Hash#has_key?.
parsed_package_contents["dependencies"].has_key?("react-on-rails")
^^^^^^^^
93 files inspected, 3 offenses detected
rake aborted!
Command failed with status (1): [cd /home/travis/build/shakacode/react_on_r...]
/home/travis/build/shakacode/react_on_rails/rakelib/task_helpers.rb:20:in block in sh_in_dir' /home/travis/build/shakacode/react_on_rails/rakelib/task_helpers.rb:20:in each'
/home/travis/build/shakacode/react_on_rails/rakelib/task_helpers.rb:20:in sh_in_dir' /home/travis/build/shakacode/react_on_rails/rakelib/lint.rake:7:in block (2 levels) in <top (required)>'
/home/travis/.rvm/gems/ruby-2.3.1/gems/rake-11.2.2/exe/rake:27:in <top (required)>' /home/travis/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in eval'
/home/travis/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `'
Tasks: TOP => default => lint => lint:lint => lint:rubocop
(See full trace by running task with --trace)

Coverage decreased (-0.06%) to 82.927% when pulling 8f770159d80ed735e11cabe313b779bae34f5c45 on benjiwheeler:master into 447c4ebaf818672acce19370511cbbf1321f4a30 on shakacode:master.

Having trouble running rspec with this Gem bundle... getting:
Your bundle is locked to activejob (4.2.3), but that version could not be found in any of the sources listed in your Gemfile. If you haven't changed sources, that means the author of activejob (4.2.3) has removed it.

@benjiwheeler Did you try running bundle install? you might need to cd spec/dummy and run bundle install there too!
See https://github.com/shakacode/react_on_rails/blob/master/CONTRIBUTING.md

yes, ran bundle install on app dir and spec/dummy both... tried changing ruby version and rerunning... now getting mini_racer error... reinstalling xcode commandline utils etc...

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

.delete_by, line 1 at r2 (raw file):

20170101

what is this file?

Comments from Reviewable

@benjiwheeler Let's get this in the next release coming up soon!

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

removed .delete_by

Coverage decreased (-0.06%) to 83.051% when pulling ed354942a3e81505453eb8573d769c5e9aa7f9af on benjiwheeler:master into 7f859e131937116de3d3d0dedbc1a871ea745c24 on shakacode:master.

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

Comments from Reviewable


Add nchunks argument to Array constructor

To me it seems that asking the user to specify the chunk sizes is somewhat cumbersome in practice. In my usage at least I'm always calculating what the appropriate chunk-size is given my desired nchunks and the size of the array.
As a user I often have a good idea of how many chunks I want as it relates to the amount of parallelism my desktop can handle - either a 1:1 mapping nchunks:ncores or some amount of over-subscription to allow for dynamic scheduling.
Just wondering if anyone else is of the same opinion or if I'm the only one?!

To work well (automatically) it might also need a minsize argument to specify the minimum chunk size in each dimension... in which case maybe it's adding too much complexity for too little gain? :thinking:
Anyway, I just thought I'd put the idea out there. Feel free to close if it's not deemed worthwhile...

I agree that this could be useful, but agree that the complexity could add
more maintenance burden.  I could go either way on this.
For now I think that the right thing to do is to consider how the API would
behave in a number of different situations.
On Thu, Oct 11, 2018 at 7:40 PM Dave Hirschfeld<EMAIL_ADDRESS>wrote:

To work well (automatically) it might also need a minsize argument to
specify the minimum chunk size in each dimension... in which case maybe
it's adding too much complexity for too little gain? 🤔
Anyway, I just thought I'd put the idea out there. Feel free to close if
it's not deemed worthwhile...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/4085#issuecomment-429157192, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszEHyPN9wtZo5IjdnKL5J_FRb1ky0ks5uj9bqgaJpZM4XYi1k
.

@dhirschfeld did you have any follow up here?  It seems like things here have stalled.  Shall we close?

I still think (done well) it could be a nice improvement but there are some design issues which make it not entirely trivial to implement so I'm unlikely to have time to get stuck in here.
Given that I'll go ahead and close to reduce noise in the issues. It doesn't prevent anyone picking it up in future...


jsrt: Help US to update documentation and samples

Recent changes, we have differences between ChakraCore JSRT docs and the actual implementation. (including the new API I have recently added)
[optional, bonus]
++ add more samples to https://github.com/Microsoft/Chakra-Samples/tree/master/ChakraCore Samples (move sample.cpp from this repo's test/native-tests/pick-a-sub-folder and add code comments)
EDIT: more detail and typos

https://github.com/Microsoft/ChakraCore/blob/master/test/native-tests/test-static-property/sample.cpp ?

If I have time wednesday I'm pretty good at writing documentation and samples if no one beats me to it by then.

To give some extra pointers, we miss docs for modules APIs in ChakraCore.h and debugging APIs in ChakraDebug.h. Would appreciate PRs to https://github.com/Microsoft/ChakraCore-wiki for docs.
Samples go to https://github.com/Microsoft/Chakra-samples.

@mattzeunert did a nice job by documenting xcode output -> https://github.com/Microsoft/ChakraCore-wiki/pull/45


chore: upgrade flutter to 3.19.1

resolves #1924
Note:

The overall design changed slightly, I think whenever working on the UI we need to adjust minor things... or live with it that the color tone changed a bit.
this does not upgrade flutter rust bridge which is a beast in itself

@holzeis : fixed the firefox issue in e43730f (#2081)

Two more things.

The payout chart doesn't fit anymore, throwing a layout error.

══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
The following assertion was thrown during layout:
A RenderFlex overflowed by 40 pixels on the bottom.

The relevant error-causing widget was:
  Column
  Column:file:///Users/holzeis/workspace/coblox/10101/mobile/lib/features/welcome/payout_chart.dart:79:12

To inspect this widget in Flutter DevTools, visit:
http://<IP_ADDRESS>:9100/#/inspector?uri=http%3A%2F%2F<IP_ADDRESS>%3A56088%2FtVP8iz4aw_0%3D%2F&inspectorRef=inspector-0

The overflowing RenderFlex has an orientation of Axis.vertical.
The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and
black striped pattern. This is usually caused by the contents being too big for the RenderFlex.
Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the
RenderFlex to fit within the available space instead of being sized to their natural size.
This is considered an error condition because it indicates that there is content that cannot be
seen. If the content is legitimately bigger than the available space, consider clipping it with a
ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex,
like a ListView.
The specific RenderFlex in question is: RenderFlex#bded4 relayoutBoundary=up6 OVERFLOWING:
  creator: Column ← PnlLineChart ← FractionallySizedBox ← Expanded ← Column ← Expanded ← Column ←
    ConstrainedBox ← Container ← Transform ← Center ← AnimatedBuilder ← ⋯
  parentData: offset=Offset(0.0, 23.5) (can use size)
  constraints: BoxConstraints(0.0<=w<=390.0, h=188.2)
  size: Size(390.0, 188.2)
  direction: vertical
  mainAxisAlignment: spaceEvenly
  mainAxisSize: max
  crossAxisAlignment: center
  verticalDirection: down
◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤◢◤
════════════════════════════════════════════════════════════════════════════════════════════════════

The channel config modal is not moved when the keyboard is toggled.

@holzeis :

Two more things.
* The payout chart doesn't fit anymore, throwing a layout error.

I'm giving up on this :/ Might be better to spend some time replacing the carousel but I don't have the nerves for this right now.

    * The channel config modal is not moved when the keyboard is toggled.

This is fixed:

I'm giving up on this :/ Might be better to spend some time replacing the carousel but I don't have the nerves for this right now.

Maybe this gets "fixed" if we switch back to the previous font and size.

Maybe this gets "fixed" if we switch back to the previous font and size.

Be my guest figuring out what it was before and setting it to this value now 🙈

The small dots indicating that there is a carousel are also gone.


Save user settings

This just adds the dependency electron-settings and uses it to load whatever server url you last used.
I did this because as I'm packaging the app for Windows users, I realized a common user error will be to launch and click run right away, but non developers likely won't have a local server running. The best way to handle it for both dev's and non dev's is to simply load whatever you last used, and I imagine we might need to save similar settings for convenience in the future.

I would rather just change the default to one of the CS servers than add ~3000 lines of code in the dependencies, TBH.

My opinion is being able to save configuration variables is a good thing.

But we don't have to have any configuration variables.
We only have one, right now, and AFAIK none of our future plans require any additional options in the UI.

If we add even a second configuration option, we'll use this. (And you'll get to say "I told you so!".)

Yeah that's fair. Thanks for the quick review!

@OmarShehata I've changed my mind on this, but I can't reopen b/c the branch is deleted. Do you mind re-pushing?

I don't have it, but feel free to install it.


Chains Add improvements

Original issue: https://github.com/cosmos/relayer/issues/51
As a follow-on from https://github.com/cosmos/relayer/pull/49, there are a couple of features I would like to see added to relayer chains add.

relayer chains add --url [url] - This would fetch the json and add the chain after verifying it. Think usage from a github repo (https://raw.githubusercontent.com/cosmos/relayer/tree/master/demo/ibc0.json)
relayer chains add --cosmos [url] - This would query the /genesis endpoint to fetch most of the data and then would rely on sane defaults for gas and gasPrices based on that. This is basically figuring out how to unmarshal gaia based config.

Closing this as we have have the chains add url already and the other one seems less useful and more gaia specific


New Feature: add support for Monad2C and Monad3C to OptionT, closes #379

@pbadenski could you please review this PR?

Looks good to me! 👍


Add ESLint rule to prevent Vitest VS Code extension from blowing up

Is your feature request related to a problem? Please describe.
See https://github.com/Shopify/theme-tools/pull/17 for context on what can go wrong.
TL;DR the Vitest VS Code extension does not support it.each nor tests that have the same name. We should write an ESLint/TSLint rule that prevents folks from committing tests that break the Vitest VS Code extension.
Describe the solution you'd like

I would want a rule that warns users against it statements that are taking variables as argument.
e.g.
// bad
const testCases = [...]
for (const testCase of testCases) {
  it(`does something for ${testCase.foo}`, () => {
    //...
  });
}

// also bad
it.each(['a', 'b', 'c'])(async (letter) => {
  // ...
});

I would want a rule that warns users against tests that have the same name (OK if in different describe blocks, not ok if it's in the same)
e.g.
describe('Parent1:', () => {
  // ok
  it('unique', () => {});

  // bad, same name at same level
  it('not-unique', () => {});
  it('not-unique', () => {});
});

describe('Parent2:', () => {
  // ok (since in different describe block)
  it('unique', () => {});
})

No longer an issue. Fixed upstream by the vitest extension :)


[WIP] Add the CK machine and related docs

I added some docs. Will implement the CK machine and likely add more docs, so do not merge yet.

@effectfully Just as a useful convention, if you want a PR not to be merged, just add [WIP] to the beginning of the title.

Seems fine, feel free to merge.

Oops, sorry, didn't know I shouldn't close a pull request.


[BUG]: OnApproveActions doesn't cater to subscriptions

Is there an existing issue for this?

[X] I have searched the existing issues.

🐞 Describe the Bug
type onApproveActions seems to only be applicable to standard orders and not subscriptions:
export type OnApproveActions = {
    order: {
        capture: () => Promise<OrderResponseBody>;
        authorize: () => Promise<OrderResponseBody>;
    };
    redirect: (redirectURL: string) => void;
    restart: () => void;
};

where the paypal response for a subscription onApprove looks like this:

😕 Current Behavior
No response
🤔 Expected Behavior
No response
🔬 Minimal Reproduction
No response
🌍 Environment
| Software         | Version(s) |
| ---------------- | ---------- |
| react-paypal-js  |   7.4.2    |
| Browser          |   Brave    |
| Operating System |   Windows  |

Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

➕ Anything else?
No response

Hey @AJDowds! I'm a member of PayPal's devrel team. Thanks for your patience on this; I know it's been a while. I've talked to the team that maintains this project; they've created tickets for it, and they're hoping to get them completed during the first sprint in January. I'll follow up with them on Jan. 10 and let you know what the status is.

Hi @mkirlin and @AJDowds here is work progress related to this issue: https://github.com/paypal/paypal-js/pull/140.
Still waiting for feedback and any concern about this change.

Hey @AJDowds! Looks like @borodovisin was got the fix for this merged a few days back, so I'm going to go ahead and close this issue. Please reopen if you continue to run into this problem. We appreciate your patience on this!


[Feature] Press Right button to create midpoints for an exact movement turns

Something available from Drag Ruler was that with either ENTER or right click you can create midpoints for a complex movement route where exact location is important to be shown to players.

Elevation Ruler currently defined keybinds to set waypoints when using the ruler or dragging tokens. Default settings are "=" and "-" for add/remove, respectively. But I hear you about wanting to right-click.

Thank you for considering the right click option.

Added in v0.9.5. Also check out the additional keybindings; I separated Token Ruler from regular ruler keybindings for adding waypoints.


Incorrect answer for question “ Which of the following statements about enabling CodeQL scanning default setup are true?”

In practice exam GitHub Advanced Security, question 74:

Which of the following statements about enabling CodeQL scanning default setup are true?

the following answer is also correct:

You can enable default setup on any repository, regardless of the contents of the repository

See:

A repository must meet all the following criteria to be eligible for default setup, otherwise you need to use advanced setup.

Code scanning is not already enabled.
GitHub Actions are enabled.
Publicly visible.

We recommend enabling default setup for eligible repositories if there is any chance the repositories will include at least one CodeQL-supported language in the future. If you enable default setup on a repository that does not include any CodeQL-supported languages, default setup will not run any scans or use any GitHub Actions minutes. If CodeQL-supported languages are added to the repository, default setup will automatically begin scanning CodeQL-supported languages and using GitHub Actions minutes.

Reference:
https://docs.github.com/en/code-security/code-scanning/enabling-code-scanning/configuring-default-setup-for-code-scanning-at-scale#eligible-repositories-for-codeql-default-setup-at-scale

Hey @carlos-quintero
Taking a quick look and seems you are right.
When I'm trying to remember the past - I'm almost sure it wasn't like that few months ago, meaning you couldn't turn on CodeQL unless the repository contained one of the supported languages. Maybe that was changes by GitHub
Would you like to create a PR to fix this? It's fairly easy [docs]
Thanks for raising the issue! :rocket:

Done in #269
Thanks for raising the issue :rocket:


cmd/go: TestBuildIDContainsArchModeEnv/386 fails on linux/386 in Go 1.14 and 1.13, not 1.15

This test failure is uncovered as of linux-386-longtest builder now testing linux/386, but currently masked by #39309. It became exposed when I was testing a fix for #39309 in CL 237617 and CL 237619.
TestBuildIDContainsArchModeEnv was added in CL 43855 to test that GOARM and GO386 environment variables contribute to the build ID and staleness checks. It's always skipped in short mode.
I haven't confirmed it, but there's a chance that the TestBuildIDContainsArchModeEnv/386 subtest may not have taken into account that it might be run on linux/386, where GOARCH=386 is already the starting condition.
Either because of that, or for another reason, TestBuildIDContainsArchModeEnv/386 has a different stale reason when run on linux/386:
go_test.go:4901: wrong reason for Stale=true: "build ID mismatch", want "stale dependency"

The check for stale reason changed from "build ID mismatch" to "stale dependency: runtime/internal/sys" as part of CL 73212, later changed to just "stale dependency" in CL 112975 (/cc @ysmolsky), and finally was completely removed in the test refactor in CL 214387 (/cc @matloob) (it still checks for staleness, just not the exact reason). This is why this issue is happening on 1.14 and 1.13, but not 1.15.
Next step in this issue is to investigate if this failure suggests there is a problem in cmd/go, or whether the test on 1.14 and 1.13 release branches is bad and should be fixed for linux/386.
/cc @bcmills @matloob @jayconrod

I was able to reproduce this on a linux-386 gomote after setting GOOS=linux GOARCH=386 GOHOSTOS=linux GOHOSTARCH=386.
I confirmed GOARM, GO386 and other environment variables are incorporated into the action ID (cache key) at least as far back as go1.13.12 (link). So this seems like an overly brittle test.
Perhaps we should just backport CL 214387, the new version of that test? @matloob WDYT?

@jayconrod What we need here for the 1.15 milestone is to confirm the current 1.15 test behavior (which is different from 1.14 and 1.13 test behavior) is intended and correct. If it isn't, it should be fixed before the 1.15 release.
Based on your comment and @bcmills's reaction, it sounds like that's the case, and are now just thinking about how to address the issue in 1.14 and 1.13. Is that the case? Or is there more investigation to do for 1.15 here? Thanks.

I don't think there's anything left to do for 1.15.
(Users don't typically check for staleness explicitly anyway, so they don't care what specific text we use to report it as long as we correctly deduce whether targets actually need to be rebuilt. Since they don't care about the specific text, we shouldn't either.)

I think Jay's suggestion to backport CL 214387 is probably the right approach for 1.14 and 1.13.

Thanks for confirming.
@bcmills FWIW, if backporting the test script test proves to be tricky due to merge conflicts or missing code in previous versions, another way to fix the test in 1.14 and 1.13 release branches would be with by removing the stale reason from the wantStale helper:
-tg.wantStale("mycmd", "stale dependency", "should be stale after environment variable change")
+tg.wantStale("mycmd", "", "should be stale after environment variable change")

When the reason parameter to wantStale helper is the empty string, it disables the reason check (the staleness is still checked independently of the reason), see here.

@gopherbot Please backport to the last 2 releases. This is a test fix. It makes a cmd/go test less strict and enables it to pass in all environments, which matches the behavior of the same test in the latest version, which has been confirmed to be most optimal.

I've opened 2 backport issues. Closing this, since this is resolved for 1.15.

When the reason parameter to wantStale helper is the empty string, it disables the reason check (the staleness is still checked independently of the reason), see here.

Hmm, I wrote that based on memory, but now that I look at the code, it looks like it would fail even if reason parameter is the empty string, if isStale gives a non-empty why.
In that case, the smallest direct test fix I can think of is to add a new wantStaleIgnoreReason helper. That's probably better than a hack like a single space as the reason.


Could not restore the channel.backup file from iCloud Drive

I tried to recover the LN wallet with my seed - the history of on-chain funds is not visible but I suppose that is another issue. Anyway, I did not have any on-chain funds, just funded LN channels which are not visible at the moment.
Thought that the channel.backup file is actually backed up both on my phone / iCloud Drive because when I went to Channel Backup before both backups had green dot. By mistake, I have not backed up the channel.backup file anywhere else.
So back to restoring - I input the seed and click the button to select the channel.backup file but it was not anywhere (folder Zap was not either on phone drive nor on iCloud Drive). I suppose the app is not able to access the hidden folder because when I go to Settings -> iCloud -> Manage Storage -> mainnet (it's how the app appears on this list) -> Documents -> there are many channel.backup files (I suppose the original one is actually there but it is not possible to move it to another folder).
Is there a way to recover these LN funds?
Thanks a lot!

Device: iPhone XS
OS: iOS 12.4

Update: I have generated a completely new wallet nad it seems the channel.backup file is not created in a visible folder on iCloud Drive. It is created on the device though.
(unfortunately, the previous wallet did not create channel.backup on the device either so these LN funds are probably lost for me - unless it is possible to access the data in Settings -> iCloud -> Manage Storage -> mainnet -> Documents)

Hey @xAdamPos.
Zap should write the channel backup file on the device and to your iCloud Drive. If you navigate to your Settings -> Channel Backup you can see the status of your SCB file.
There seems to be a bug in the restore process for our recent mainnet release (#230). We're working to fix it.

can you see the backup in the phones Files app?

Unfortunately not.
Only in Settings - iCloud - Manage Storage - mainnet  - Documents (and this is not accessible directly from the app).

On 2 Sep 2019, at 15:12, Otto Suess<EMAIL_ADDRESS>wrote:
can you see the backup in the phones Files app?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

There seems to be a bug in the restore process for our recent mainnet release

I'm pretty sure these are separate bugs. Mainnet funds are recovered again since the latest release, but there is something wrong with how the channel backup files are stored.
I also see a "mainnet" folder in my iCloud Store settings on my phone. It contains a folder with the same digits as the Zap/... folder on the device. It contains a 0.1 KB channel.backup file, which I can't open or otherwise inspect.

Hey, @xAdamPos @Sjors.
I'm a little confused about the issue (or don't totally understand). If the app is deleted, the local backup will also be deleted. Not much we can do there. Users can periodically email themselves (or text, etc etc) the backup if they'd like. Maybe we don't put a green checkmark next to it in the settings? Was that misleading from a UX point of view?
In regards to the backup stored in iCloud, what is the issue here? The idea is that if your device does get damaged/stolen or you do delete the app, you have your seed and the iCloud backup to restore. Is there something wrong with this idea?

The idea is right, of course, I don’t need the local backup, cloud backup is fine. But the iCloud backup has not been created (or it is not in the on iCloud Drive at least). I guess it is still available in the Application Data in Settings but this is not accessible by the user. That is why the seed is the only thing I have got at the moment and LN funds are probably lost :-(

2019 v 16:14, JimmyMow<EMAIL_ADDRESS>

Hey, @xAdamPos https://github.com/xAdamPos @Sjors https://github.com/Sjors.
I'm a little confused about the issue (or don't totally understand). If the app is deleted, the local backup will also be deleted. Not much we can do there. Users can periodically email themselves (or text, etc etc) the backup if they'd like. Maybe we don't put a green checkmark next to it in the settings? Was that misleading from a UX point of view?
In regards to the backup stored in iCloud, what is the issue here? The idea is that if your device does get damaged/stolen or you do delete the app, you have your seed and the iCloud backup to restore. Is there something wrong with this idea?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/LN-Zap/zap-iOS/issues/240?email_source=notifications&email_token=ALJBCUPIWHTFTU5NKGJWEIDQHZWNFA5CNFSM4IRWJUB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5YKW6Y#issuecomment-527477627, or mute the thread https://github.com/notifications/unsubscribe-auth/ALJBCUJBAVPYLFS5EIXBFE3QHZWNFANCNFSM4IRWJUBQ.

where exactly in the settings do you see the file?

Maybe we don't put a green checkmark next to it in the settings?

Yes, maybe don't even show it. It's not really a backup if it's tied so strongly to the app. When I delete Instagram it doesn't delete my camera roll, so it's not obvious to users. Instead, encourage users to copy the backup somewhere save. Maybe have a green dot if you know they've saved the backup elsewhere and make it red again when there's a new channel.

the iCloud backup has not been created (or it is not in the on iCloud Drive at least

That indeed appears to be the main issue.

As described above:
Settings -> iCloud -> Manage Storage -> mainnet (it's how the app appears on this list) -> Documents -> there are many channel.backup files

2019 v 16:25, Otto Suess<EMAIL_ADDRESS>

where exactly in the settings do you see the file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/LN-Zap/zap-iOS/issues/240?email_source=notifications&email_token=ALJBCUIDCDA67NCTFCCERDLQHZXU3A5CNFSM4IRWJUB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5YL3QY#issuecomment-527482307, or mute the thread https://github.com/notifications/unsubscribe-auth/ALJBCUPEQPWLOZOLL7NQ7ZTQHZXU3ANCNFSM4IRWJUBQ.

Hey, @xAdamPos @Sjors I've had a difficult time reproducing this. What iPhone and iOS version are you running? Attempts at recovering my wallets have been successful. Are you still facing this issue?

I haven't tried reproducing, but those backup files are definitely not on my iCloud. My best guess is that it didn't make an (iCloud) backup because I closed the app right after broadcasting the opening transaction, and deleted it before the app could find out the channel was confirmed.
I still need to try if I can recover the funds without a backup using desktop Lnd: https://bitcoin.stackexchange.com/questions/90196/how-to-recover-funds-from-a-force-closed-lnd-lightning-channel

The same situation on my side. I have not tried reproducing but the iCloud backup file has not been created on my iCloud therefore I am not able to restore the channel funds. I didn’t close the app right after broadcasting the opening tx - the channels were successfully opened for a while.
I am still curious if the iCloud backup file couldn’t be hidden in Settings -> iCloud -> Manage Storage -> mainnet -> Documents because I see many backup files there (screenshot). I believe that the first channel.backup (size 3.4 kb) is actually the one I have been looking for and was not created in the iCloud Drive. I still believe the file exists and this is the one :) Unfortunately, these files are not accessible.

I can show the situation to you at LN conf in Berlin :)

So i just ran into this issue today.  I think there is some confusion between "icloud backup" and "icloud drive".  I think the confusion comes from that they use the same storage, but are separated.
Zap uses the icloud Backup which is something you would use if you want to restore files for an iphone that has been corrupted. Or that were on a old iphone (lost/stolen/bricked) and you have a new iphone.  You CAN NOT manually download these files, they are not accessible in the icloud Drive app. The issue is that you will have to RESET the device and then select "Restore from iCloud Backup".
https://support.apple.com/kb/ph12521?locale=en_US
Zap does NOT backup the files to the icloud Drive, which afaict you would have to manually add files to the icloud drive, and then they would be accessible across devices using the icloud drive app or website.
If it is possible to automatically backup the files to the icloud Drive that would be ideal.
Short story is that icloud is a confusing hot mess.

Is there any progress regarding the channel backups in iOS? Is it possible for the channel.backup file to be automatically saved to iCloud Drive (not iCloud Backup as mentioned above) / Google Drive so the user can restore the app without restoring iPhone? Right now I have to manually save the file after each channel update. Thanks for any follow up!

i have this problem too, i have a channel Backup on my google drive, and i'm trying to restore on ZAP mobile (ios), but unsuccessful. i have now put on icloud drive but still not recovered

There hasn't been any commits in nearly 6 months.

I'd like an update on this as well.  Still no sign of the backup file on the latest version of zap.
I would not recommend this app until this issue is resolved, user funds are at risk.


Support parsing and status publishing for the new cellular APIs

This is a first part in a series of commits implementing the new cellular APIs in EVE. All parts together will provide many new features for cellular connectivity, such as support for user credentials, multiple modems, multiple SIM slots, switching between 5G/4G/3G, and much more.
Part I changelog:

added support for parsing and status publishing for the new cellular APIs into zedagent
extended JSON structs used to move config and status between NIM and wwan microservice to accommodate newly introduced attributes
NIM and wwan microservice for now work only in the backward-compatible mode. Support for the new APIs will be added into these microservices in the next parts of this commit/PR series.
added support for device models where cellular modems are referenced by USB or PCI address, not by interface name. Multiple modems can be initialized in different order on each boot, meaning that interface naming is not stable and therefore not suitable for modem identification.
added publishing of port MTU. This is for every port, not just wwan. However, especially for wwan interfaces this is useful because MTU config comes from the networt and it can vary quite a bit.
in DeviceNetworkStatus (DNS), errors collect from verification take precedence over whatever failures are detected when DNS is composed. For cellular connectivity, this ensures that we will not overwrite errors reported by the wwan microservice, which are much more useful than whatever NIM would report from its viewpoint.
moved ip utils and generics to sub-packages of pillar/utils so that they can be used from pillar/types without circular dependencies (used them for methods of wwan structures)
moved wwan types from zedroutertypes.go to a separate file wwan.go (these types are not used by zedrouter)

added support for device models where cellular modems are referenced by USB or PCI address, not by interface name. Multiple modems can be initialized in different order on each boot, meaning that interface naming is not stable and therefore not suitable for modem identification.

Those changes look OK if the usbaddr is provided for a cellular modem.
But I wonder whether it makes sense to also look at the Unique string we save to handle multiple cellular modems even when they are configured using wwan0-type ifnames.

Those changes look OK if the usbaddr is provided for a cellular modem.
But I wonder whether it makes sense to also look at the Unique string we save to handle multiple cellular modems even when they are configured using wwan0-type ifnames.

I'm not sure if I understand the question. What do you mean by "Unique string we save"?

I'm not sure if I understand the question. What do you mean by "Unique string we save"

I was referring to https://github.com/lf-edge/eve/blob/fec97665f8c9d89bb1fa201b6b75f1e2ca05bc85/pkg/pillar/types/assignableadapters.go#L76C1-L76C1
That is only used for logs, but the MacAddr on the following line is used to check (and rename) that what was previously called "eth4" remains as "eth4" even after it has been assigned to a guest VM and handed back to the host.
It might make sense to explore whether we want to put some other unique string in Unique (or in some new field) and use that to check that the USB device is what we expect it to be after assigning it away and getting it back from the guest VM. (Doesn't need to be part of this PR but might make sense to explore.)

It might make sense to explore whether we want to put some other unique string in Unique (or in some new field) and use that to check that the USB device is what we expect it to be after assigning it away and getting it back from the guest VM. (Doesn't need to be part of this PR but might make sense to explore.)

But isn't USB address stable in that it would not change even after assigning a modem to a VM and later bringing it back to the host? For interfaces the order may change with such action and hence the generated names could differ and need renaming, but I would think that USB (or PCI) address should remain the same (?)

But isn't USB address stable in that it would not change even after assigning a modem to a VM and later bringing it back to the host? For interfaces the order may change with such action and hence the generated names could differ and need renaming, but I would think that USB (or PCI) address should remain the same (?)

Might need a device with two or more USB controllers on the PCI bus to explore whether the numbering issue we fixed for ethernet ports applies there as well.
The test would be something like assign both USB controllers to vfio-pci, and then assigned them back to the host in different order, and check whether the bus part of the USB address changes as a result of this or is stable independely of when the OS re-enumerates the USB resources when the USB host controller driver attaches to the PCI device.
But as you said, that wasn't my original comment.

Edit: now I re-read you previous comment and realized that you are referring to models without USB/PCI addresses. In that case I don't think we can do much to ensure that mapping between config and HW is as expected - already the first device boot could produce unexpected order, so trying to preserve it (across assignments or reboots) would not help unless user updates config. With multiple modems I would say that using USB/PCI addreses is almost a requirement (they are fairly likely to change order between reboots, modem boot time can vary quite a bit). After this and one more PR I will submit spec.sh change where USB address is detected and put into the device model for every cellular modem. All already generated device models have at most one cellular modem in them AFAICT.

I hope that a normal reboot would always result in the same lsusb output, but it might still make sense to add a check by saving some string extracted from the USB device (could be the manufacturer,deviceId if the address is based on bus,port) so that one can detect when things "moves".
But as I said, doesn't need to be part of this PR. Just something to consider.


Missing Lively

Free and open-source software that allows users to set animated desktop wallpapers and screensavers.
https://github.com/rocksdanister/lively

😊🙏


Adds a application as example

Given @jhrdt great contribution on #20, I could add an example application with Django's native admin running in parallel with this Django Public Admin:

Here are what I suggested to the docs
And the files are in the example/ directory

Hej,
I am just curious, but adding manage.py at the top level of the repository and including a complete django project, interferes with the app character/layout of the repository/project.
I wouldn't expect to see a demo django project in a repository for a django app.
Looking at the layout of other app projects like DRF or nested_admin, makes believe that this change makes things more complicated.
There are other possibilities, like a second repository (that includes the example project) with a git submodule that contains django-public-admin. Or a dedicated branch for development.
There might also be the problem, that the example django project need to be up to date with djangos latests releases. This might also include to adapt to new django project layout/conventions that might come over time (e.g. changes in settings.py and other stuff).

I hear you, @jhrdt.
In my opinion, we need to differentiate what is a Python package and what is a repository. I am adding things to a respository, not to the Python package — neither the example/ directory nor the manage.py is copied to site-packages.

(tempenv-25aa264095cf2) $ pip install git+https://github.com/cuducos/django-public-admin.git#v4
Collecting git+https://github.com/cuducos/django-public-admin.git#v4
  Cloning https://github.com/cuducos/django-public-admin.git to /tmp/pip-req-build-3f7j6jht
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Collecting django>=2
  Downloading Django-3.1.2-py3-none-any.whl (7.8 MB)
     |████████████████████████████████| 7.8 MB 2.4 MB/s
Collecting pytz
  Using cached pytz-2020.1-py2.py3-none-any.whl (510 kB)
Collecting sqlparse>=0.2.2
  Downloading sqlparse-0.4.1-py3-none-any.whl (42 kB)
     |████████████████████████████████| 42 kB 1.4 MB/s
Collecting asgiref~=3.2.10
  Using cached asgiref-3.2.10-py3-none-any.whl (19 kB)
Building wheels for collected packages: django-public-admin
  Building wheel for django-public-admin (PEP 517) ... done
  Created wheel for django-public-admin: filename=django_public_admin-0.0.3-py3-none-any.whl size=5350 sha256=107c3d309fa4d1447b17aad8d09cae3d8c880a41d2a7e382a45441f89b41b89c
  Stored in directory: /tmp/pip-ephem-wheel-cache-4jp9yx6r/wheels/12/df/ff/e2dd49080d61059ec977b22a684b464bb5db0a93211d47be5c
Successfully built django-public-admin
Installing collected packages: pytz, sqlparse, asgiref, django, django-public-admin
Successfully installed asgiref-3.2.10 django-3.1.2 django-public-admin-0.0.3 pytz-2020.1 sqlparse-0.4.1
(tempenv-25aa264095cf2) $ pwd
/home/cuducos/.virtualenvs/tempenv-25aa264095cf2/lib/python3.8/site-packages/public_admin
(tempenv-25aa264095cf2) $ ls -laGh
total 24K
drwxr-xr-x  4 cuducos 4.0K Oct  8 14:49 ./
drwxr-xr-x 21 cuducos 4.0K Oct  8 14:49 ../
-rw-r--r--  1 cuducos 1.1K Oct  8 14:49 admin.py
-rw-r--r--  1 cuducos    0 Oct  8 14:49 __init__.py
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 __pycache__/
-rw-r--r--  1 cuducos 3.4K Oct  8 14:49 sites.py
drwxr-xr-x  3 cuducos 4.0K Oct  8 14:49 templates/
(tempenv-25aa264095cf2) $ cd ..
(tempenv-25aa264095cf2) $ ls -laGh
drwxr-xr-x 21 cuducos 4.0K Oct  8 14:49 ./
drwxr-xr-x  3 cuducos 4.0K Oct  8 14:48 ../
drwxr-xr-x  3 cuducos 4.0K Oct  8 14:49 asgiref/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 asgiref-3.2.10.dist-info/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:48 _distutils_hack/
-rw-r--r--  1 cuducos  152 Oct  8 14:48 distutils-precedence.pth
drwxr-xr-x 19 cuducos 4.0K Oct  8 14:49 django/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 Django-3.1.2.dist-info/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 django_public_admin-0.0.3.dist-info/
-rw-r--r--  1 cuducos  126 Oct  8 14:48 easy_install.py
drwxr-xr-x  5 cuducos 4.0K Oct  8 14:49 pip/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:48 pip-20.2.3.dist-info/
-rw-r--r--  1 cuducos    0 Oct  8 14:48 pip-20.2.3.virtualenv
drwxr-xr-x  4 cuducos 4.0K Oct  8 14:48 pkg_resources/
drwxr-xr-x  4 cuducos 4.0K Oct  8 14:49 public_admin/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 __pycache__/
drwxr-xr-x  4 cuducos 4.0K Oct  8 14:49 pytz/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 pytz-2020.1.dist-info/
drwxr-xr-x  6 cuducos 4.0K Oct  8 14:48 setuptools/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:48 setuptools-50.3.0.dist-info/
-rw-r--r--  1 cuducos    0 Oct  8 14:48 setuptools-50.3.0.virtualenv
drwxr-xr-x  5 cuducos 4.0K Oct  8 14:49 sqlparse/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:49 sqlparse-0.4.1.dist-info/
-rw-r--r--  1 cuducos   18 Oct  8 14:48 _virtualenv.pth
-rw-r--r--  1 cuducos 5.6K Oct  8 14:48 _virtualenv.py
drwxr-xr-x  4 cuducos 4.0K Oct  8 14:48 wheel/
drwxr-xr-x  2 cuducos 4.0K Oct  8 14:48 wheel-0.35.1.dist-info/
-rw-r--r--  1 cuducos    0 Oct  8 14:48 wheel-0.35.1.virtualenv

So I would agree with basically all your sentences if you were addressing a Python package, but I cannot agree on the same statements for a single repository.

I am just curious, but adding manage.py at the top level of the repository and including a complete django project, interferes with the app character/layout of the repository/project.

It was zero influence over the project as a Python package, for sure. And it has an arguably positive influence on the repository since now future users have an working example of the package available here.

I wouldn't expect to see a demo django project in a repository for a django app.

I think this is pretty nice. I like it that, for example, django-multiselectfield and django-tables2 have working examples —  although they moved the manage.py and the application interely to a example/ folder. Othe packages form the Flask world also have working examples, such as Flasgger, Dynaconf, both with +1k starts on GitHub… so I just can't see why not.

Looking at the layout of other app projects like DRF or nested_admin, makes me believe that this change makes things more complicated.

And surely we can be selective to find examples to whatever we are claiming : )

There are other possibilities, like a second repository (that includes the example project) with a git submodule that contains django-public-admin. Or a dedicated branch for development. A docker container that contains a developtment environment might also be an option.

I think all of that would make sense if the work on the main branch of this repo was intense  — which is not the case. We have 5 contributors and 33 starts at the moment. And only 29 commitis on the main branch. We're in alpha. Git submodules, adding dependency on Docker, or changing branches just to have an working example seam all good solutions… but an overkill at this time. I prefer to keep it simple.

There might also be the problem, that the example django project need to be up to date with django's latests releases. There might also be the problem, that the example django project need to be up to date with django's latests releases.

This is not a problem, actually. At least, not a problem added by this PR. If new Django versions breaks anything, carrying about the versioning at pyproject.toml and all the consequences is a problem that we already have.
Firthermore, I haven't mentioned the possibility of runing integration tests with this working example, which would be a plus, right?
Anyway… I do agree with the confusion caused by manage.py at the root level, so I might move everything related to the example to the example/ directory.
I udnerstand we might disagree on many points here, but I would love to still have a follow up from you. In sum, I am trying to keep things as simple as possible (at least, while this is still a small project).

I hear you, @jhrdt.
In my opinion, we need to differentiate what is a Python package and what is a repository. I am adding things to a respository, not to the Python package — neither the example/ directory nor the manage.py is copied to site-packages.
So I would agree with basically all your sentences if you were addressing a Python package, but I cannot agree on the same statements for a single repository.

I am aware, that the final artifact will only contain application code.

I am just curious, but adding manage.py at the top level of the repository and including a complete django project, interferes with the app character/layout of the repository/project.

It was zero influence over the project as a Python package, for sure. And it has an arguably positive influence on the repository since now future users have an working example of the package available here.

I wouldn't expect to see a demo django project in a repository for a django app.

I think this is pretty nice. I like it that, for example, django-multiselectfield and django-tables2 have working examples — although they moved the manage.py and the application interely to a example/ folder. Othe packages form the Flask world also have working examples, such as Flasgger, Dynaconf, both with +1k starts on GitHub… so I just can't see why not.

Looking at the layout of other app projects like DRF or nested_admin, makes me believe that this change makes things more complicated.

And surely we can be selective to find examples to whatever we are claiming : )

Yes, thats a matter of taste.

There are other possibilities, like a second repository (that includes the example project) with a git submodule that contains django-public-admin. Or a dedicated branch for development. A docker container that contains a developtment environment might also be an option.

I think all of that would make sense if the work on the main branch of this repo was intense — which is not the case. We have 5 contributors and 33 starts at the moment. And only 29 commitis on the main branch. We're in alpha. Git submodules, adding dependency on Docker, or changing branches just to have an working example seam all good solutions… but an overkill at this time. I prefer to keep it simple.

I would agree on that.

There might also be the problem, that the example django project need to be up to date with django's latests releases. There might also be the problem, that the example django project need to be up to date with django's latests releases.

This is not a problem, actually. At least, not a problem added by this PR. If new Django versions breaks anything, carrying about the versioning at pyproject.toml and all the consequences is a problem that we already have.

Right.

Firthermore, I haven't mentioned the possibility of runing integration tests with this working example, which would be a plus, right?

Yes, that is a benefit.

Anyway… I do agree with the confusion caused by manage.py at the root level, so I might move everything related to the example to the example/ directory.
I udnerstand we might disagree on many points here, but I would love to still have a follow up from you. In sum, I am trying to keep things as simple as possible (at least, while this is still a small project).

So at the end, its a matter of taste : ) .

So at the end, its a matter of taste : )

Maybe it is. But either way I do like to understand your side of the story — and, by doing that, even calling my side into questioning. Many many thanks for contributing to the debate. Probably if this package gets more popular over time, Many of these ideas will be needed!


Support target=_blank on pages via MD

Is your feature request related to a problem? Please describe.
For the resources page I would like links to open on external tab, without adding html to the MD file. This will allow the user to open several links without always having to navigate back.

Fixed using attributes plugin e.g.
{: target=_blank}
* ["Splunk is Lambda Ready: Announcing a New Partnership with AWS"](https://www.splunk.com/en_us/blog/it/splunk-is-lambda-ready.html){: target=_blank}


disable other repositories / get packages from epel

Cookbook version
2.1.1
Chef-client version
latest
Platform Details
Amazon Linux AMI release 2016.09 x86_64
Scenario:
I want to install some packages from the epel repository: nrpe, nagios-plugins-uptime
The problem is that the packages are also in the Amazon repository, but very outdated
How to disable amzn-main repository?
Steps to Reproduce:
yum_repository 'epel' do
description 'Extra Packages for Enterprise Linux'
mirrorlist 'http://mirrors.fedoraproject.org/mirrorlist?repo=epel-6&arch=$basearch'
gpgkey 'http://dl.fedoraproject.org/pub/epel/RPM-GPG-KEY-EPEL-6'
action :create
only_if { platform_family?('rhel') }
end
package ['nrpe', 'nagios-plugins', 'nagios-plugins-disk', 'nagios-plugins-load', 'nagios-plugins-uptime']
Expected Result:
nagios-plugins.x86_64                                                                         2.1.4-3.el6                                                                 epel
nagios-plugins-disk.x86_64                                                                    2.1.4-3.el6                                                                 epel
nagios-plugins-load.x86_64                                                                    2.1.4-3.el6                                                                 epel
nrpe.x86_64                                                                                2.15-7.el6                                                                  epel
Actual Result:
nagios-common.x86_64                                                                       3.5.1-1.8.amzn1                                                          @amzn-main
nagios-plugins.x86_64                                                                      1.4.16-5.8.amzn1                                                         @amzn-main
nagios-plugins-disk.x86_64                                                                 1.4.16-5.8.amzn1                                                         @amzn-main
nagios-plugins-load.x86_64                                                                 1.4.16-5.8.amzn1                                                         @amzn-main

You'll want to look into setting the priority level on the yum_respository resource such that it's higher than the Amazon one. Closing this as it's not an issue with the cookbook but please do come ask us in http://community-slack.chef.io/ if you have questions.


#58: 'digits' option added to passwordPolicy map in UpdateRealmAction

Hi,
I had apparently overlooked the digits option for password policy in the UpdateRealmAction. I have added it here now.

Hey @aberg09 thanks for your contribution :)

Released with version 0.2.54 could you please check it? :)

Released with version 0.2.54 could you please check it? :)

I have problems downloading the fat jar for this version (I can download it from github but when I try to use it I get 'Invalid or corrupt jarfile'). I have tried older versions (53-49) and there the same happens. The most recent version that runs is 48.
Indeed I also see that 48 is the version offered on https://github.com/mayope/keycloakmigration/releases.
The download URL I am using is
https://github.com/mayope/keycloakmigration/releases/download/[version]/keycloakmigration-[version]-fat.jar

And thank you, as always, for your quick reply :)

Sry I forgot to draft the release. It should be available here now: https://github.com/mayope/keycloakmigration/releases/tag/0.2.54

It works now. Thanks one more time :) !


Does this code base support ray distributed training

thank you for your amazing work!
just curious about training in the multiple devices with this codebase
does this support ray
https://github.com/ray-project/ray

Sorry, we don't support ray by now.
To train models in multiple devices, we support using slurm, which refer to the https://github.com/open-mmlab/mmclassification/blob/master/tools/slurm_train.sh

is it possible to integrate ray distributed training in this codebase

This issue will be closed as it is inactive, feel free to re-open it if necessary.


BUG: x-axis labels of xyaxes diagram are incorrectly positioned on iOS

When viewing a plot which contains xyaxes on mobile the x-axis labels are positioned too far from the x-axis.
See below:

Tested on
iOS 17.x Safari
The bug does not occur on desktop browsers - even when viewed at the same screen width.

I don't have easy access to an iOS device, but I'll try to figure out how to handle this bug

I'm curious, could you show me what this example diagram looks like in iOS?

This is what it looks like on Safari iOS, and the same issue occurs on Chrome for iPhone.

fixed in<EMAIL_ADDRESS>


Support Python >=3.7

Any reason for requiring Python >=3.10?

Personal preference, sorry

maybe, because this


Support async/await by default

Main use case is to make Action.perform cleaner by using syntax like
async perform(params) {
  let User = this.modelFor('user');
  let mailer = this.service('mailer');
  let user = await User.find(params.id);

  mailer.send('welcome', user);

  return user;
}

Initial Stub Results
I tried the async-generator transform and am getting the following error:
Error building your app:
Error: /Users/iradchenko/battleground/test/app/actions/index.js: You gave us a visitor for the node type "ForAwaitStatement" but it's not a valid type
    at verify (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/visitors.js:196:13)
    at Object.explode (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/visitors.js:72:3)
    at traverse (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/index.js:77:12)
    at NodePath.traverse (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/path/index.js:144:25)
    at exports.default (/Users/iradchenko/battleground/test/node_modules/babel-helper-remap-async-to-generator/lib/index.js:10:8)
    at PluginPass.Function (/Users/iradchenko/battleground/test/node_modules/babel-plugin-transform-async-to-generator/lib/index.js:13:56)
    at newFn (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/visitors.js:276:21)
    at NodePath._call (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/path/context.js:76:18)
    at NodePath.call (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/path/context.js:48:17)
    at NodePath.visit (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/path/context.js:105:12)
    at TraversalContext.visitQueue (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitSingle (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/context.js:108:19)
    at TraversalContext.visit (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/context.js:192:19)
    at Function.traverse.node (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/iradchenko/sandbox/denali/node_modules/babel-traverse/lib/context.js:150:16)

So i don't have a ton of hands-on experience with async/await, but it's certainly intriguing. As I understand it, an async function interops with non-async functions by simply returning a promise, right?
If that's the case, it sounds like there should be nothing stopping a Action's responder method from being async, since Denali handles returned promises (obviously theres some kind of transpilation issue happening, but I'm thinking more on a conceptual level).

This should be resolved on master now. Feel free to reopen/refile if we are still missing something though.