SEO: using next-seo and next/head instead of helmet

NextJS has built in Head component to let you define meta tags; and if ever you have dynamic content in your portfolio, you can use getStaticProps/getServerSideProps together with next-seo no need for react-helmet since next is capable of ssr

Yes, I just ran in some issue with importing fonts in globalStyles with Styled Components and it suggested me to use  React-Helmet instead because I don't want to use css files in my project. Gonna remove the Helmet later.


Need Help: p2p  connection issue

Describe the bug
helium_router failed to connect to p2p node.
To Reproduce
Host my own console in Amazon EC2  according to the Guide, But failed to connect to the blockchain.
Expected behavior
A clear and concise description of what you expected to happen.
Device Info (please complete the following information):

Environment (Production, Staging)
Console Device ID
Device dev eui / app eui
How often is device sending data
Region
Hotspots (names) used to transfer data

NOTE: do not post your device app_key
My OUI: 43
My router address: 118t8HZ4dSnwC5C9hxyM4HQQUKMczo2zoe86ten5aYBHW1wnrY5
Additional context
Here is the log of helium_router -> router.log:
[2022-08-27 10:29:53.966] <0.1751.0> [info] [router_console_dc_tracker:init:{156,5}] router_console_dc_tracker init with #{}
[2022-08-27 10:29:53.967] <0.1753.0> [info] [router_v8:init:{57,5}] router_v8 init with #{}
[2022-08-27 10:29:53.971] <0.1757.0> [info] [router_device_devaddr:init:{123,5}] router_device_devaddr init with #{}
[2022-08-27 10:29:53.974] <0.1758.0> [info] [router_xor_filter_worker:init:{175,5}] router_xor_filter_worker init with #{}
[2022-08-27 10:29:53.974] <0.1749.0> [warning] [router_console_ws_worker:handle_info:{283,5}] rcvd unknown info msg: {ws_message,<<"device:all">>,<<"phx_reply">>,#{<<"response">> => #{},<<"status">> =>
<<"ok">>}}, {state,<0.1750.0>,<<"ws://helium_console:4000/socket/router/websocket">>,#Ref<0.140648651.425590795.83649>,#Ref<0.140648651.425590795.83651>}
[2022-08-27 10:29:53.974] <0.1749.0> [warning] [router_console_ws_worker:handle_info:{283,5}] rcvd unknown info msg: {ws_message,<<"organization:all">>,<<"phx_reply">>,#{<<"response">> => #{},<<"status"

=> <<"ok">>}}, {state,<0.1750.0>,<<"ws://helium_console:4000/socket/router/websocket">>,#Ref<0.140648651.425590795.83649>,#Ref<0.140648651.425590795.83651>}
[2022-08-27 10:29:53.974] <0.1749.0> [warning] [router_console_ws_worker:handle_info:{283,5}] rcvd unknown info msg: {ws_message,<<"label:all">>,<<"phx_reply">>,#{<<"response">> => #{},<<"status">> => <
<"ok">>}}, {state,<0.1750.0>,<<"ws://helium_console:4000/socket/router/websocket">>,#Ref<0.140648651.425590795.83649>,#Ref<0.140648651.425590795.83651>}
[2022-08-27 10:29:53.974] <0.1749.0> [warning] [router_console_ws_worker:handle_info:{283,5}] rcvd unknown info msg: {ws_message,<<"organization:all">>,<<"phx_reply">>,#{<<"response">> => #{},<<"status"
=> <<"ok">>}}, {state,<0.1750.0>,<<"ws://helium_console:4000/socket/router/websocket">>,#Ref<0.140648651.425590795.83649>,#Ref<0.140648651.425590795.83651>}
[2022-08-27 10:29:54.435] <0.1714.0> [warning] [blockchain_state_channels_server:load_state_channels:{701,5}] not in db sc ids: []
[2022-08-27 10:29:54.435] <0.1714.0> [warning] [blockchain_state_channels_server:load_state_channels:{709,5}] presumably closed sc ids: []
[2022-08-27 10:29:54.436] <0.1714.0> [warning] [blockchain_state_channels_server:get_new_active:{469,13}] don't have any state channel left unused
[2022-08-27 10:29:54.492] <0.1758.0> [error] [router_xor_filter_worker:get_filters:{666,13}] failed to find routing for OUI: 43 not_found
[2022-08-27 10:29:54.492] <0.1758.0> [error] [router_xor_filter_worker:should_update_filters:{921,13}] failed to find routing for OUI: 43 not_found
[2022-08-27 10:29:54.492] <0.1758.0> [info] [router_xor_filter_worker:handle_info:{398,13}] filters are still up to date
[2022-08-27 10:29:54.721] <0.1650.0> [warning] [libp2p_nat:maybe_apply_nat_map:{137,37}] invalid nat map: {[],[]} {[],[]} {error,{badmatch,{error,einval}}}
[2022-08-27 10:29:54.811] <0.1650.0> [warning] [libp2p_nat:maybe_apply_nat_map:{137,37}] invalid nat map: {[],[]} {[],[]} {error,{badmatch,{error,einval}}}
[2022-08-27 10:29:55.017] <0.1650.0> [warning] [libp2p_nat:maybe_apply_nat_map:{137,37}] invalid nat map: {[],[]} {[],[]} {error,{badmatch,{error,einval}}}
[2022-08-27 10:30:04.432] <0.1715.0> [info] [router_metrics:handle_info:{256,5}] running metrics
[2022-08-27 10:30:08.963] <0.1745.0> [info] [router_sc_worker:maybe_start_state_channel:{354,13}] [active: 0] [overspent: 0] [getting close: 0] [opened: 0] [in flight 0] [max: 25] all active, opening mo
re
[2022-08-27 10:30:08.963] <0.1745.0> [info] [router_sc_worker:open_next_state_channel:{415,17}] failed to get a good expiration no_opened_sc
[2022-08-27 10:30:08.967] <0.1745.0> [info] [router_sc_worker:create_and_send_sc_open_txn:{456,5}] Opening state channel for router: "118t8HZ4dSnwC5C9hxyM4HQQUKMczo2zoe86ten5aYBHW1wnrY5", oui: 43, nonce
: 1, id: <<79,223,228,25,159,12,56,5,47,53,214,251,158,111,225,234,45,113,231,137,0,149,121,100,84,193,65,79,61,177,78,90>>
[2022-08-27 10:30:09.927] <0.1799.0> [warning] [libp2p_transport_proxy:connect_rcv:{116,9}] timeout_proxy_session proxying to "/p2p/11BLPytHFxQYnjE7wEooEUzZrH1KXahHN7Vd3i4Bn6fubzoiS9K"
[2022-08-27 10:30:09.928] <0.1657.0> [warning] [libp2p_group_worker:connecting:{238,5}] Failed to connect to "/p2p/112EGCZWqMbDZNR7jL4LfQ4zrDxPNDN5Vp89Jd3BQG6f1j8x8WvG": [{"/p2p/11BLPytHFxQYnjE7wEooEUzZ
rH1KXahHN7Vd3i4Bn6fubzoiS9K/p2p-circuit/p2p/112EGCZWqMbDZNR7jL4LfQ4zrDxPNDN5Vp89Jd3BQG6f1j8x8WvG",timeout_proxy_session}]
[2022-08-27 10:30:11.699] <0.1676.0> [warning] [libp2p_nat_server:delete_mapping:{192,13}] failed to delete port mapping {2154,2154}: no_nat

Screenshots
If applicable, add screenshots (from Console) to help explain your problem.

I had hosted another router(address:112iF2fACBMpZXx5KXvQ4XW89LHPDmpRyB5rBGBwspBN9vZbcJe1),  but the same thing happened.

I don't think we log when connections are successful, only when they fail.
For the no_nat messsages, you may need to fill out this part of the .env file.
https://github.com/helium/router/blob/master/.env-template#L4-L8
That's mapping your internal AWS address to the external AWS address for gossip purposes, as far as I understand it.
You can also see how many connections your router has by running:
docker exec -it helium_router _build/default/rel/router/bin/router peer book --count


Question - BlockStorage

Hi,
I was wondering if BlockStorage.get(x, y, z) and BlockStorage.set(x, y, z) use relative values (0-15) or absolute values (world coordinates)  as arguments. Due to the lack of Javadoc, these types of ambiguities may sometimes arise.
If you could help me out, I would be really grateful.

As BlockStorage is per-chunk, it uses chunk coordinates (0-15).

Thanks


Does the library support real-time iteration mpc scheme?

Does the libmpc support real-time iteration mpc scheme, like acados library for example? If not, is extending it to support RTI possible?

What does it means real time iteration?

"The scheme allows a fast reaction to disturbances by delivering approximations of the exact optimal feedback control which are iteratively refined during the runtime of the controlled process."
This method is applied in acados mpc library https://docs.acados.org/about_acados/index.html
reference to method:
https://epubs.siam.org/doi/abs/10.1137/S0363012902400713?journalCode=sjcodc

Hello @abdelhakim96, the library uses SLSQP to solve the underlying non-linear optimization problem. However, it is possible to plan to extend the non-linear optimizer class and include such a new scheme.


Custom resolution?

Is there possibility of having custom resolution for training/inference ?

@adhikjoshi Yes you can do both, albeit it gets computationally heavy as you get closer to 512x512.
Typically in my tests, 384x256 for widescreen seems to work well for both training and inference seem to work well. For training, it's a must to keep the frames low (< 4) or else it won't fit into memory.
There may be some duplication problems as the model was trained on videos at a 1:1 ratio to the best of my knowledge.

check width and height in the example yamls in the config folder

I was wondering if it would be successful to train on resolution like 512x256 for stereoscopic videos?


nixos/tests/netdata: fix non-deterministic failure

Motivation for this change
The test sporadically failed on hydra for x86_64-linux  and i686 when a request was made
before the service was actually listening on its port.
Explicitly wait for the port to open.
Things done

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

@GrahamcOfBorg test netdata

Success on x86_64-linux (full log)
Attempted: tests.netdata
Partial log (click to expand)
netdata: exit status 1
syncing
netdata: running command: sync
netdata: exit status 0
test script finished in 25.66s
cleaning up
killing netdata (pid 597)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/3yppr81p7lxajxklpi9d7lg90zz2kxld-vm-test-run-netdata

Success on aarch64-linux (full log)
Attempted: tests.netdata
Partial log (click to expand)
netdata: exit status 1
syncing
netdata: running command: sync
netdata: exit status 0
test script finished in 26.25s
cleaning up
killing netdata (pid 631)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/zw4g1gx7hwrnm96qy6y29xxx2dcdi2j5-vm-test-run-netdata


Update ANCM install section

Fixes #12701

@scottaddie Looks like the build is struggling a bit with the WIP status. I've tried opening/closing and adding/removing [WIP] from the title.


Reinstall option is missing in Windows installer

Description
When making subtle changes to the code and regenerating the installer for Windows, it's not possible to reinstall Mozc because the installer doesn't recognize the update.
Steps to reproduce
Steps to reproduce the behavior:

Install Mozc from the installer for Windows
Update the version of Qt and build the installer again
Launch the regenerated installer.

Expected behavior
Provide an option for reinstall or overwrite installation of Mozc.
Actual behavior

Version or commit-id
Mozc-2.28.5080.100+24.11.oss
Environment

OS: Windows

Right.  The current behavior is what we have inherited from Google Japanese Input's installer, which was written more than 13.5 years ago. Since then the logic hasn't been changed much.
The relevant code is here.
https://github.com/google/mozc/blob/4c85ed6d0a6eb97e21d57adb0aecc87da0e6453c/src/win32/installer/installer_oss_64bit.wxs#L317-L325
I think the easiest (and the safest) way to achieve your request is to change the above lines as follows.
    <Upgrade Id='$(var.UpgradeCode)'>
      <UpgradeVersion
          Minimum='<IP_ADDRESS>' IncludeMinimum='yes'
          Maximum='$(var.MozcVersion)' IncludeMaximum='yes'
          OnlyDetect='no' Property='UPGRADING'/>
      <UpgradeVersion
          Minimum='$(var.MozcVersion)' IncludeMinimum='no'
          OnlyDetect='yes' Property='NEWERVERSIONDETECTED' />
    </Upgrade>

Perhaps we can now switch from <Upgrade> to <MajorUpgrade>, which was added in WiX 3.5, but we really need to be careful about when RemoveExistingProducts action is scheduled when doing such a migration. So probably just flipping IncludeMaximum and IncludeMinimum would be safer for now.

Submitted the above change internally. Should be fixed in the next code push.


Cannot detect .Net Core, I downloaded, but I cannot see .Net Core templates

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/ubuntu18-04/sdk-current
Operating System: Linux Ubuntu 18.04 - x64
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you execute dotnet --version on your terminal and print the result?

Where are you looking for templates? does dotnet new console not work, for instance?

I installed all .Net Core templates but didn't found System libraries, and
I try again install .Net Core templates  this time I didn't see .Net Core tamplates in my Jetbrains Rider!
Now I haven't .Net Core templates, but System libraries found!

I didn't understand this?

Среда,  2 октября 2019, 5:25 +12:00 от Cory Nelson<EMAIL_ADDRESS>Where are you looking for templates? does dotnet new console not work, for instance?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .

--
Кутман Заиров

@ZairovKutman this sounds like a Jetbrains template issue, not related to .NET Core.
Feel free to reopen if you disagree or would like to clarify a bit more what you mean by .NET Core templates.


bug: checkov errors out on dependencies?

[INFO] Initializing environment for https://github.com/antonbabenko/pre-commit-terraform.
[INFO] Initializing environment for https://github.com/bridgecrewio/checkov.git.
[INFO] Installing environment for https://github.com/bridgecrewio/checkov.git.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
Checkov..................................................................Failed
- hook id: checkov
- duration: 0.32s
- exit code: 1

Traceback (most recent call last):
  File "/home/runner/.cache/pre-commit/repor3gr5pl7/py_env-python3/bin/checkov", line 2, in <module>
    from checkov.main import run
  File "/home/runner/.cache/pre-commit/repor3gr5pl7/py_env-python3/lib/python3.10/site-packages/checkov/main.py", line [20](https://github.com/PaloAltoNetworks/vm-series-gh-actions/actions/runs/4184962868/jobs/7251366362#step:8:21), in <module>
    from checkov.argo_workflows.runner import Runner as argo_workflows_runner
  File "/home/runner/.cache/pre-commit/repor3gr5pl7/py_env-python3/lib/python3.10/site-packages/checkov/argo_workflows/runner.py", line 8, in <module>
    from checkov.common.output.report import CheckType
  File "/home/runner/.cache/pre-commit/repor3gr5pl7/py_env-python3/lib/python3.10/site-packages/checkov/common/output/report.py", line [21](https://github.com/PaloAltoNetworks/vm-series-gh-actions/actions/runs/4184962868/jobs/7251366362#step:8:22), in <module>
    from checkov.common.util.json_utils import CustomJSONEncoder
  File "/home/runner/.cache/pre-commit/repor3gr5pl7/py_env-python3/lib/python3.10/site-packages/checkov/common/util/json_utils.py", line 6, in <module>
    from packaging.version import LegacyVersion, Version
ImportError: cannot import name 'LegacyVersion' from 'packaging.version' (/home/runner/.cache/pre-commit/repor3gr5pl7/py_env-python3/lib/python3.10/site-packages/packaging/version.py)

Error: Process completed with exit code 1.

fixed with #14


Options for CAD formats

CAD formats has been introduced by !236
Visual quality highly depends on the meshing parameters.
Currently, these parameters are hard-coded with a trade-off between loading speed and visual quality.
Moreover, it is standard for CAD visualizer to display surfaces boundaries (curves discretized with poly-lines) but sometimes we do not want it.
How to handle these CAD specific options?

https://gitlab.kitware.com/f3d/f3d/-/issues/221

@Meakk you started work on this right ?

dropping priority on this

related to #1193

Discussion needed to identify which option to expose, but first #1193 is needed


Big blank + scrolling on register account page

Problem: in register account page, form far away down the page, there is a big blank and the page scrolls
I am guessing this comes form the "app" class which is used without footer.

What do you think we can do @thimy ?

Is this bug fixed ? I see others problems in mobile mode, but not blank down the register page.

Ok it is not fixed, my screen was too small :]

Closed by https://github.com/pytition/Pytition/pull/149


Ask for endpoint with random port

It would be useful to ask for a connection at a random unused port.  This is handy when making several connections from the same machine where you don't care much about what port is used, as long as it is not already taken.  In TCP-based systems port 0 is often used for this.
I think that @quasiben already brought this up informally, I just wanted to have something here for tracking.

@mrocklin checked to see if passing sockaddr with port 0 to ucp_listener at just UCX C-level works but tests show otherwise. We may need to add this functionality in UCX.
Out of curiosity, once a random port is used for binding where a listener is listening, how does the client know which port to connect to at the listener site? Is this conveyed out of band?

Is this conveyed out of band?

Sort of.  Dask is peer to peer, with a single known peer whose address is conveyed out of band.  Everyone else connects to this central listener (we call this the scheduler) and tells that central listener the address on which they are listening.  When two peers need to communicate to each other the central scheduler tells them the relevant addresses.


prepare cython for PyPy3.10

In the upcoming PyPy support for python3.10 we removed the tp_pypy_flags slot from PyTypeObject. I checked this worked with NumPy + MSVC (that is picky about matching structs when compiling) + PyPy 3.10 nightly build
If accepted, the change should be backported to the 0.29 branch. I expect to start the PyPy release cycle in May, so if this could be incorporated into a new release it would be helpful.

Not a problem with this PR but it always feels to me like we could do better to deduplicate the "fill in a mostly empty type object" code

deduplicate the "fill in a mostly empty type object"

Would a memset(x, 0, sizeof(PyTypeObject)) be better?

deduplicate the "fill in a mostly empty type object"

Would a memset(x, 0, sizeof(PyTypeObject)) be better?

I actually think C does the right thing and zero initializes rest of the structure if you only partially initialize it. But the compile complains about uninitialized fields. It's mainly just to keep the compiler quiet rather for any practical reasons.
Anyway - this looks fine. I'll merge and backport

Thanks!

Backport in https://github.com/cython/cython/commit/bba7d3db9e1404efe3bf53f9010f90e22fe59a5b


Filter Panoramaimages using Pointclouds and Cloudcompare

[x] find out wheter panorama coords match the pointcloud coords
[x] Create a tool to find the right image for given coords
[ ] find matching pans for a larger pointcloud

Pan Coords match with the pointclouds global coords, created a tool to find the closest image to given geo-coords


Running 'az acs kubernetes install-cli' from env giving error

When running from the developer environment, its throwing error:
Connection error while attempting to download client (<urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:749)>)
The same command works fine when executed outside of the dev env.

@brendandburns @colemickens
Any updates?

We are calling the k8s_install_cli(install_location=_get_default_install_location('kubectl')) method from acs module and it is throwing the SSL error. Here is the debug log:
_Command arguments ['project', 'create', '-l', 'westus', '-n', 'demo', '-g', 'tenxkubegroup']
Registered application event handler 'CommandParser.Parsed' at <function _handle_container_ssh_file at 0x10431fd90>
Current active cloud 'AzureCloud'
{'active_directory': 'https://login.microsoftonline.com',
'active_directory_graph_resource_id': 'https://graph.windows.net/',
'active_directory_resource_id': 'https://management.core.windows.net/',
'batch_resource_id': 'https://batch.core.windows.net/',
'gallery': 'https://gallery.azure.com/',
'management': 'https://management.core.windows.net/',
'resource_manager': 'https://management.azure.com/',
'sql_management': 'https://management.core.windows.net:8443/'}
{'azure_datalake_analytics_catalog_and_job_endpoint': 'azuredatalakeanalytics.net',
'azure_datalake_store_file_system_endpoint': 'azuredatalakestore.net',
'keyvault_dns': '.vault.azure.net',
'sql_server_hostname': '.database.windows.net',
'storage_endpoint': 'core.windows.net'}
Registered application event handler 'CommandTableParams.Loaded' at <function add_id_parameters at 0x104769730>
Registered application event handler 'CommandTable.Loaded' at <function add_id_parameters at 0x104769730>
Successfully loaded command table from module 'project'.

Installing kubectl ...
Downloading client to /usr/local/bin/kubectl
Connection error while attempting to download client (<urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (ssl.c:749)>)

The same command works fine when executed outside of the dev env.

What does this mean? Do you mean the acs-engine devenv? It already has kubectl available.
Otherwise, I've not seen this before. Are you running this from a container or on a normal Linux distro or something else? Does SSL work normally elsewhere? Are you running mitmproxy/fiddler etc?

I am running this command on my normal OSX machine. The same command works fine when I run it from outside the az cli devenv but throws me the above error when run from inside the devenv. devenv is the one which we get from . /env/bin/activate
SSL works fine as the command works fine normally. Something is different in the devenv. Might I be missing something?

I assume ./env/bin/activate is just virtualenv. Not really sure I understand why that would affect SSL validation. All we do is make an HTTP request with urlretrieve from urllib/urllib.request.
@derekbekoe do you know why being in the dev virtualenv would affect SSL validation?

Yes I agree. That's what the confusion is. Infact, this can be repro'd on another machine.

@saurabsa What are the repo steps? I can create a VM and test.

@derekbekoe Just get in the devenv using . /env/bin/activate and run the command: az acs kubernetes install-cli

What OS, Python version?
Also, what's the output of pip freeze?

 The issue seems to be because of Python3.6. It can be repro'd if we set the devenv using Python3.6. Python3.5 seems to work fine. Repro steps:
git clone https://github.com/Azure/azure-cli
cd azure-cli
python3.6 -m venv env
. env/bin/activate
python scripts/dev_setup.py
. az.completion.sh
Run command: az acs kubernetes install-cli
Connection error while attempting to download client (<urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:749)>)
This could be repro'd by @derekbekoe as well.

I am getting this error with python 3.5.1 on my Windows 10 machine.

@saurabsa
I believe this is the explanation:
http://stackoverflow.com/a/42334357/4364364
Basically Python 3.6 on OS X doesn't install any certs by default.

@djarvis
pip install certifi
may help with this as well, see:
https://bugs.python.org/issue28150

I'm going to close this, since it is not a cli issue, afaik.

For future reference, the linked python bug contains a command which fixed the issue for me (on Mac OS X 10.12.6):
/Applications/Python\ 3.6/Install\ Certificates.command

TLDR; Quick fix:
Execute the following command to resolve the issue:
/Applications/Python\ 3.6/Install\ Certificates.command
Background:
Python 3.6 has an explicit post-installation step to install certificates since they are not installed at the point of installation. The command above will install the certifi package and resolve any CERTIFICATE_VERIFY_FAILED issue.
Currently on macOS 10.12.16 + Python 3.6.
-Anuj

If I Install certificates  as mentined

TLDR; Quick fix:
Execute the following command to resolve the issue:
/Applications/Python\ 3.6/Install\ Certificates.command
Background:
Python 3.6 has an explicit post-installation step to install certificates since they are not installed at the point of installation. The command above will install the certifi package and resolve any CERTIFICATE_VERIFY_FAILED issue.
Currently on macOS 10.12.16 + Python 3.6.
-Anuj

If I install certificates like above. I get an error like this
azure-mgmt-recoveryservicesbackup 0.1.1 has requirement msrestazure~=0.4.11, but you'll have msrestazure 0.5.0 which is incompatible. azure-mgmt-recoveryservices 0.1.0 has requirement msrestazure~=0.4.11, but you'll have msrestazure 0.5.0 which is incompatible. azure-mgmt-datalake-analytics 0.2.0 has requirement msrestazure~=0.4.7, but you'll have msrestazure 0.5.0 which is incompatible.


updating link to code of conduct in org repository

Signed-off-by: Vanessa Sochat<EMAIL_ADDRESS>This is the 2/3 pull request to update the link to the code of conduct, moving from tob to the .github folder of org. This repo (unlike runtime-spec) has the link also in the contributing.md.

LGTM

LGTM

LGTM

LGTM


Guide to using Prisma with PlanetScale

Created a stub file for PS Guide so that we can collaboratively get it written asap.
https://deploy-preview-2708--prisma2-docs.netlify.app/docs/guides/database/using-prisma-with-planetscale

Placing in the sidebar is a bit weird:

It should probably be at the end of the list and not in the middle of all the Migrations content.

Placing in the sidebar is a bit weird...

Agreed. I've moved to just above 'Advanced database tasks'.

... and also link to PlanetScale's updated docs on migrations with Prisma

Thanks, just added this in


Support custom python-executable

Problem: build pipelines and infrastructure at big companies often targets multiple independent python projects. Currently, in order to use pip-sync, pip-tools have to be installed in every environment. When pip-compile does not have such restriction.
Solution: add the ability to pass custom python executable.  When the option provided, list of installed packages will be collected from the target environment and all install/uninstall commands will be executed inside that environment.
This PR implements a proof concept for collecting early feedback on the idea and implementation. If concept and task will be accepted, sufficient test coverage will be added and PR re-submitted.
Thank you!
Added "--python-executable" option for pip-sync command. It allows targeting environments other than current:
Contributor checklist

[x] Provided the tests for the changes.
[x] Gave a clear one-line description in the PR (that the maintainers can add to CHANGELOG.md on release).
[ ] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

@atugushev could you please comment if this PR makes sense and maintainers agree to add such an option

Hello @MaratFM,
Tganks for pinging! I believe that this functionality would help for such issues as #1087. So i'm :+1: on this.

Generally, it looks good. One thing to consider: could we improve --python-executable to pass binary name somehow? For example: pip-sync --python-executable=python3.

Great idea, I can try using shutil.which for resolving
https://docs.python.org/3/library/shutil.html?highlight=.which#shutil.which
Looks like it works well for aliases and full paths, so we can always use it for resolving:
In [1]: import shutil

In [2]: import sys

In [3]: shutil.which('python')
Out[3]: 'D:\\Envs\\test\\python.EXE'

In [4]: shutil.which(sys.executable)
Out[4]: 'd:\\envs\\test\\python.exe'

@atugushev please take a look when you will have a chance. I added support for binary executable name (like pip-sync --python-executable=python3). In addition, while testing locally, I found that when we use a custom python executable it is no guaranteed that pip version is compatible. So I added an explicit check for it.

@webknjaz @atugushev could you please take a look

@jdufresne could you please help with the review?

Sorry, I will not have the time to review this. Please carry on with the other reviewers.

Same here. I hope @atugushev will be able to get this to the finish line.

I reverted logging message passing, was confused by the recommendation to delegate message formatting to the logging system (#1333 (comment)). Seems like pip-tools uses a custom logging wrapper that has incompatible method contracts.

Thanks @MaratFM! Let's address that in a following up PR.

@atugushev seems like with "# pragma: no branch" it stills shows partial coverage, so I reverted back to "# pragma: no cover"

@atugushev sorry for changing tests back and forth, I could not run tests for all these platforms locally and have to wait when someone will trigger the GitHub actions pipeline


feat: duplicate cleaner

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

Project coverage is 49.12%. Comparing base (23493ad) to head (a62dfc0).

Files with missing lines
Patch %
Lines

pkg/repo/cosmodb.go
0.00%
66 Missing :warning:

pkg/processor/processor.go
57.57%
28 Missing and 14 partials :warning:

pkg/parser/revolut.go
67.03%
17 Missing and 13 partials :warning:

pkg/duplicatecleaner/duplicatecleaner.go
0.00%
29 Missing :warning:

cmd/server/main.go
0.00%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   49.60%   49.12%   -0.49%
==========================================
  Files          13       13
  Lines        2034     2111      +77
==========================================
+ Hits         1009     1037      +28
- Misses        888      927      +39
- Partials      137      147      +10

Flag
Coverage Δ

unittests
49.12% <40.41%> (-0.49%)
:arrow_down:

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

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


DescriptorPool::allocate_set doesn't use max_sets parameter on Metal backend

GFX version: gfx-backend-metal v0.6.4
OS: macOS Catalina v10.15.7
GPU: Radeon Pro 560X

Hi. I'm trying to reuse a single descriptor pool for all uniform buffers, having the following code that creates the pool:
device.create_descriptor_pool(
    MAX_SETS,
    &[DescriptorRangeDesc {
        ty: DescriptorType::Buffer {
            ty: BufferDescriptorType::Uniform,
            format: BufferDescriptorFormat::Structured {
                dynamic_offset: false,
            },
        },
        count: 1,
    }],
    DescriptorPoolCreateFlags::empty()
)

Here MAX_SETS is some number greater than 1 (e.g. 8 for 8 uniform buffers max).
However, it seems that Metal backend doesn't take the max_sets parameter into account: https://github.com/gfx-rs/gfx/blob/8bc6b8c0cf58cafbaad2af74d57c05001a6377d7/src/backend/metal/src/device.rs#L2027-L2035
As far as I understand, descriptors count and descriptor sets count are different things, but the emulated pool uses only descriptor ranges when calculating the available space.
Am I missing something here?  The same code works correctly on Vulkan (though it failed when I used 1 as max_sets, trying to allocate more than 1 sets).

gfx-rs is an unsafe API, just like Vulkan is. If you create a pool with max_sets = 1 but create more than one set from it - this is Invalid Use of the API, and technically UB. So Metal backend is in full right to just assume you are not allocating more sets than necessary, or just ignore max_sets if it doesn't need it.

If you create a pool with max_sets = 1 but create more than one set from it - this is Invalid Use of the API, and technically UB

That's the behaviour I've observed on Vulkan. My situation was like that:

pass max_sets = 1 on Vulkan but create more than one descriptor set: got an error from Vulkan Validation Layers
pass max_sets = N on Vulkan: works fine
pass max_sets = N on Metal: backend panics with FragmentedPool error, because descriptor set pool doesn't have resources available (traces available 0 samplers, 0 textures, and 0 buffers, to be exact); found that Metal doesn't take into account the max_sets parameter

So, the question then is: what is the correct way to specify the maximum number of descriptor sets for the descriptor set pool with respect to both backends?

Let me clarify. If you pass max_sets = N, it's not a guarantee that you'll be able to allocate N descriptors. It's just a hint to the driver to tell the upper bound of what you expect to allocate.
The actual limit will be a subject to what descriptor set layouts (ranges) you are trying to get. Even if that totals up to numbers less than the descriptor ranges you passed at vkCreateDescriptorPool, the driver may still fail to allocate the descriptors if the pool got fragmented.
Going back to your original post and re-reading it, I think you are confused about the ranges in create_descriptor_pool. You think they define a layout of a descriptor set, which you will allocate N times. This is incorrect. These ranges are just what the pool will contain in total, and the max_sets is a separate limit on it. Nothing gets multiplied by max_sets.
Practically speaking, you probably want this instead:
device.create_descriptor_pool(
    MAX_SETS,
    &[DescriptorRangeDesc {
        ty: DescriptorType::Buffer {
            ty: BufferDescriptorType::Uniform,
            format: BufferDescriptorFormat::Structured {
                dynamic_offset: false,
            },
        },
        count: MAX_SETS,
    }],
    DescriptorPoolCreateFlags::empty()
)

On that note, I think the issue is resolved. Happy to continue discussing it (and potentially reopen if needed)!


Cannot log into a different github account after signing into one

Type: Bug

Sign into a github account without access to github copilot.
sign out of github account using left navbar.
click "sign in with github to use github copilot" to try to sign into your account that actually has github copilot.
notice it signs into the same account as before without prompting for the account you actually want to use.

I can't find any way to make it stop autosigning into the wrong account. I've probably tried 10 times and looked through the settings as much as possible. Even the help website online seems to be completely wrong and the instructions don't work or look like my actual UI. (Maybe seems like the person who made this page has a mac? or old version of settings?) https://code.visualstudio.com/docs/editor/settings-sync#_switching-accounts
VS Code version: Code 1.89.1 (dc96b837cf6bb4af9cd736aa3af08cf8279f7685, 2024-05-07T05:13:33.891Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
AMD EPYC 7763 64-Core Processor                 (16 x 2445)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: unavailable_software

Load (avg)
undefined

Memory (System)
63.95GB (48.00GB free)

Process Argv
--crash-reporter-id fbd2bfd0-681c-479d-9aa0-9862b810263f

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

midl3-language-server
Ale
0.0.31

vscode-research
dev
1.2022.1111005

copilot
Git
1.195.0

copilot-chat
Git
0.15.2

tdpcode
Mic
10.2304.707

vscode-azdo-codereview
Mic
1.2023.331002

vscode-nmake-tools
Mic
4.0.230601002

wavework
Mic
1.2024.58003

sarif-viewer
MS-
3.4.4

cpptools
ms-
1.20.5

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
showvideoc:31016891
chatpanelc:31048052
dsvsc021:30996838
0ee40948:31013168
pythoncenvptcf:31049071
a69g1124:31058053
dvdeprecationcf:31061161
pythonprt:31056678
dwnewjupyter:31046869
26j00206:31048877

@kafox2 we use what your browser is signed in to. Are you signed in to the other account in your browser?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


add default deployment mode

deploy.defaultDeploymentMode
Add deploy.defaultDeploymentMode in inferenceservice-config configmap. defaultDeploymentMode now supports two modes: "Serverless" and "RawDeployment", by default, defaultDeploymentMode is "Serverless".
If user wants to change the deployment mode, there are two ways:

modify the inferenceservice-config config map, change the defaultDeploymentMode

deploy: |-
    {
      "defaultDeploymentMode": "Serverless"
    }

add annotation to inference service deploy yaml.

apiVersion: "serving.kubeflow.org/v1beta1"
kind: "InferenceService"
metadata:
  name: "flower-sample"
  annotations:
    "serving.kubeflow.org/deploymentMode" : "RawDeployment"
spec:
  predictor:
    tensorflow:
      storageUri: "gs://kfserving-samples/models/tensorflow/flowers"

moved to https://github.com/kserve/kserve/pull/57


sys/linux: add SECCOMP_IOCTL_NOTIF_ADDFD

Added the ioctl decsription for adding a fd to a seccomp notifier,
currently in linux next.
<EMAIL_ADDRESS>
Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

#533 @ebiggers @morehouse


[otbn,dv] Add flag write coverage points

Note: This is in draft because it depends on the unmerged #6708. Only the last commit is unique to this PR
This means we have to split up the "bna" and "bnaf" encodings, which
previously used the same coverage tracking code ("bna" instructions
don't write the carry flag).
There's also a little bit of cleverness in defining
DEF_MLZ_FLAGS_TOGGLE_COV, where we want to track bits 1, 2 and 3. The
call to DEF_TOGGLE_COV_1(..., 01) tracks bit 2'b01. The call to
DEF_TOGGLE_COV_2(..., 1) tracks bits 2'b10 and 2'b11.

Force-push rebases and also fixes an invalid import (I imported into the otbn_env_cov class, rather than otbn_env_pkg).

Force-push rebases and this PR is now ready for review.


New Sample - React COVID-19 info web part

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
none

What's in this Pull Request?
This PR contains a new sample: react-covid19-info.
It is a web part created to show some COVID-19 stats for a given country.

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

Thanks again @agtenr for the timely web part sample. As per our chat, I've got you on the agenda for the April 9th SPFx call. I really look forward to seeing the demo.
(Just adding this note to the PR itself so others can see this without having to navigate through the commit messages)

Hey @agtenr  thank you for your patience as we had some technical issues today. As per our discussion, you're bumped to the April 23rd call. Let's hope we finally get to see your demo!


I redid the patch in a way the shader name is a bit large in struct so we don't abort when making a depth shader.

I don't like this idea to allow some shaders to have a longer name. It's not worth the risk of problems to improve compatibility with one single map. The original version looked good to me.

I reverted to the simpler check against MAX_QPATH and then disabling too-long depth shader we may be able to support, so we don't have to check other parts of the code for now.

LGTM


Sample Position Tweaks

Currently, the sample data generates, for each map, 25 issues, 5 each of which are in the same spot. This is not a realistic portrayal of issue positions; I'd prefer them to each be positioned individually.
Additionally, all issues currently have a position, while issues' positions are specified to be nullable. It would make for more exhaustive testing if some issues were generated without positions.

the sample data generation is deterministic to be able reproduce issues easily. I have modified the algorithm to generate more random looking output, and nullable positions.


Adding Amlogic platform configuration files.

Adding platform configuration files for Amlogic SoC support.
Usage:

COBALT_ARCH = "armv7ahf"
COBALT_PLATFORM = "amlogic"
COBALT_PLATFORM_NAME = "wpe-${COBALT_PLATFORM}-${COBALT_ARCH}"

@arun-madhavan-013 please make the modifications suggested by @neeraj-deshpande after that we can merge the PR.


pass sources_for_koji_build_id to sources build

OSBS-8041
Signed-off-by: Robert Cerven<EMAIL_ADDRESS>Maintainers will complete the following section:

[ ] Commit messages are descriptive enough
[ ] "Signed-off-by:" line is present in each commit
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request includes link to an osbs-docs PR for user documentation updates

Shall be CLI integration of new option be part of this PR?

Shall be CLI integration of new option part of this PR?

added

rn not updated, fix for unreleased feature


AudioStream issues in SDK7

Currently, the AudioStream component is only being triggered when pointer focus happens.

If I am outside of the scene and enter (with pointer focus, I'm not sure if there is a better name for this?) it does now connect
However, if I then leave the scene and re-enter the AudioStream doesn't reconnect.
On scene exit I am removing the AudioStream by using AudioStream.deleteFrom(this.entity) and re-creating on entry.

If I spawn within the scene I can see the API requests being made to retrieve the token but does not then connect to the stream until focus happens.
In this case, if a player doesn't focus within 5 seconds they would end up with the login dialog appearing and currently there isn't a way for us to hook into this.
This is also the case if I run into the scene without focus.

A simple way to replicate this is with the following code:
import { AudioStream, engine } from "@dcl/sdk/ecs"

export function main() {
  console.log('main')

  const entity = engine.addEntity()

  AudioStream.createOrReplace(entity, {
    url: 'https://icecast.ravepartyradio.org/ravepartyradio-192.mp3',
    volume: 0.1,
    playing: true
  })

  console.log(AudioStream.get(entity))
}

While this seems like a nuisance rather than anything critical, for us, this is very problematic as we require authentication to connect to our streams and the API call to get a token is triggered straight away but because the AudioStream does not connect when intended it results in the token being invalid and a login dialog appears.

Hi @garyrutland
1- the issue would be that leaving the scene while reproducing an AudioStream and after that re-entering the scene won't play the AudioStream?
if I understand correctly you are subscribed to the "scene exit" event and removing the AudioStream component, If you avoid doing that (removing the component), does the issue persist?
2- the browser restrict any media to be played until there is an user input or the pointer is locked by the canvas, that's the reason the scene awaits for any of those input to start playing the AudioStream
could explain a little bit more about how the flow for the scene is? just to understand the issue better
I understood it like: as soon as the scene code is executed you send an auth request to your servers, a token is received and used to start an AudioStream
do I understand it correctly?
Maybe could you wait for the user to perform any input instead of running that flow when the scene is executed?
something like this might help
let isMediaAllowed = false

function detectFirstUserInput(){
  isMediaAllowed = inputSystem.isTriggered(InputAction.IA_ANY, PointerEventType.PET_DOWN)
                  || PointerLock.get(engine.CameraEntity).isPointerLocked
  if (isMediaAllowed)
    engine.removeSystem(detectFirstUserInput)
}

engine.addSystem(detectFirstUserInput)

To clarify, we are upgrading fr SDK6 where all of this works fine.

Doing some more tests with updated versions (@dcl/js-runtime: 7.3.14, @dcl/sdk: 7.3.14) it seems scenario one is now ok.
If what you're saying is true for scenario two (which based on more testing does seem to be true) this has changed since SDK 6 and is also not documented.
However, while doing more tests it feels like there are some inconsistencies in behaviour.
import { getUserData } from "~system/UserIdentity"
import { AudioStream, engine } from "@dcl/sdk/ecs"
import { onEnterSceneObservable, onLeaveSceneObservable } from "@dcl/sdk/observables"

export async function main() {
  console.log("main")

  const myPlayer = await getUserData({})
  const entity = engine.addEntity()

  onEnterSceneObservable.add((player) => {
    console.log("player entered scene: ", player.userId)

    if (player.userId === myPlayer.data?.userId) {
      AudioStream.createOrReplace(entity, {
        url: "http://mediaserv30.live-streams.nl:8086/live",
        volume: 0.1,
        playing: true
      })

      // true
      console.log(AudioStream.has(entity))

      // {url: 'http://mediaserv30.live-streams.nl:8086/live', volume: 0.1, playing: true}
      console.log(AudioStream.get(entity))
    }
  })

  onLeaveSceneObservable.add((player) => {
    console.log("player left scene: ", player.userId)

    if (player.userId === myPlayer.data?.userId) {
      AudioStream.deleteFrom(entity)

      // false
      console.log(AudioStream.has(entity))
    }
  })
}

Scenario one

Spawn in scene (no music)
Left click and the music will start
Walk out of the scene and the music will stop
Walk back into the scene and the music will start again

Scenario two

Spawn in the scene (no music)
Right click and the music will start
Walk out of the scene and the music will stop
Walk back into the scene and the music will NOT start again until a click is done

It seems each time you leave/enter a scene the "active" flag gets reset, and if you're saying this is a browser restriction then this surely only needs to happen once for the entirety of DCL?
On a side note, the example URL used in the docs as of this post does not work and gets a connection refused response.

It is also worth noting, and this was also the case in SDK6 that no matter what the actual state of the player is that playing always returns true.
console.log(AudioStream.get(entity))

@garyrutland issue should be fixed on next release.
please take into consideration that if a user enters your scene through the url or link and does not interact with the page canvas, the browse will most likely to block the audio playing until the user interacts with the canvas


Filter pods through pod list api

Refactored current filter pods flow through the shared pod functions
so filter pod functionalities can be shared between api and cmd.
Signed-off-by: Sujil02<EMAIL_ADDRESS>

@baude PTAL

/approve
LGTM
@baude @vrothberg @TomSweeneyRedHat @rhatdan PTAL

LGTM, nice work @sujil02!

/lgtm


Add Instruction tutorial cards on no dumps

Like when a new user enters the site, have a little tutorial card that will teach them how to use the site, and possibly a how to page, that will allow them to see the page again if they forget.
Should teach them how to hover to embed, enter a new link, etc...
Possibly work on ux as well to make these actions feel natural

Yeee Flyxer style


does it works on oreo?

attach the video or image from gallery works properly in oreo 8.0 or above version..?

Yes, it's working for us. This PR fixed issues introduced in 8.0+
https://github.com/alhazmy13/MediaPicker/pull/73


[BUG] ctx.guild.members doesn't provide the list of all members

Library Version
5.1.0
Describe the Bug
When I use ctx.guild.members within a @slash_command, the list contains only two members: the one whom launched the command, and the bot itself.
I granted the GUILD_MEMBERS privileged intent to my application as required by the Discord API, and setup this intent on my client by giving it an Intents object build with as : intents=Intents.new(default=True, guild_members=True)
Steps to Reproduce

Setup a bot with guild_members privileged intent
Build a slash command using ctx.guild.members within
Call this command for a Discord guild with more than one human member and one one

Expected Results
A list of all members of the guild, and not only the one using the slash command
Minimal Reproducible Code
@slash_command(
        name="get_members",
        description="Get all members of the guild"
    )
    async def get_members(self, ctx: SlashContext):
        members = ctx.guild.members
        print(members)

Traceback
No response
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.
[X] I have attempted to debug this myself, and I believe this issue is with the library

Additional Information
No response

You also need to provide the client flag fetch_members=True to make the bot request members from each guild upon startup.
Without it, you'll only have members that have been added to the local cache over the runtime of the bot.


Creation of PIT over a LINK and LINK-Satellite fails

datavault4coalesce-Version: 1.0.1
Reproduce: Creation of PIT over a LINK and LINK-Satellite
Error: Link-Definition is not been found (empty) in INSERT-Statement

Hi @jabibo and thanks for reaching out!
I looked into this and am able to reproduce the described error. Your suggested fix worked for me, and I implemented it in the linked branch.
But another problem occured when the LEFT JOIN to the Link Satellite should be created, but is not.
After some investigation, I noticed that this is also the case for Hub Satellites. In both cases, the background node template looks for columns with the attribute "is_hk", which should be set by the ColumnDropdownSelector that sets the Hashkey inside a Satellite V0 & V1. But apperently this DropdownSelector does not set this attribute properly right now.
I have reported this problem to Coalesce and will let you know here as soon as I got feedback from them!
In case you don't already use the new Satellite Nodes, where the new ColumnDropdownSelector is used, the linked branch should already work for you!
Best regards
Tim


Exception while executing circuit shown on  SpiceSharpGUI/screenshots/screen02.png

While executing the above circuit, an exception is thrown while parsing the expression mc(0.6,tol) of line 3.
at SpiceSharpGUI.Windows.ViewModels.NetlistResultWindowViewModel.RunSimulations() in C:\Users\a-sch\Downloads\SpiceSharpGUI-master\SpiceSharpGUI-master\src\windows\SpiceSharpGUI.Windows\ViewModels\NetlistResultWindowViewModel.cs:line 217
I have been trying to run  SpiceSharpParser/src/SpiceSharpParser.IntegrationTests/Resources/example01.cir.
It uses an interesting feature like .func, it does run but does not produce any output.
Does anybody have a meaningful .plot statement for this circuit ?
Thanks, Adrian

I will look in a moment. I will reply soon.

@a3an-de  I've checked the circuit from screen02 and it works for me after I've downloaded the SpiceSharpGUI from master branch and compiled it.
Are you put the expression in curly brackets?
Example01.cir comes from some users of SpiceSharp/SharpParser that ageed that I could include it in the tests to validate that the library is able to parse that circuit.
The circuits uses .SAVE statement. It allows to access values in code.
To see it in SpiceSharpGUID you please change .SAVE V(N1) V(N2) V(N3) to .PRINT V(N1) V(N2) V(N3)
If there is any help you need, don't hesitate to ask.

this is the circuit:
V1 IN 0 AC 1
L1 IN MIDDLE {mc (0.6, tol)}
C1 MIDDLE OUT {mc (10e-6, tol)}
R1 OUT 0 {mc (1000, tol)}
.ac oct 100 1 500
.step param X 0 10 1
.param tol = 0.2
.plot ac v (OUT) merge
.end
I have to admit that I am not very fluent in c# nor in the usage of MS Visual Studio. A short script on how to setup SpiceSharpGUI in VS and howto run a proven circuit would be useful.
Adrian

@a3an-de  Do you need any help?
Kind regards,
Marcin

Hi Marcin,
I did reinstall spicesharpgui again,  it did not help.
I noticed that the .csproj file indicates a back level version of the spicesharpparser library 1.4.3, not 1.5.0.
Could that be the reason ? How would I change that ?
Regards, Adrian
Von meinem iPad gesendet

Am 16.05.2020 um 21:33 schrieb Marcin<EMAIL_ADDRESS>

@a3an-de Do you need any help?
Kind regards,
Marcin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I will update the project to use 1.5.0 in few minutes.
Thank you for information

I've updated it to use latest version of SpiceSharpParser.
I've verifed iit and I'm getting the plots as below:

I've updated the file with circuit that you are trying to simulate.
https://github.com/SpiceSharp/SpiceSharpGUI/blob/master/screenshots/band-pass.cir

@marcin-golebiowski  I noticed a subtle difference between my version and your version of the circuit. My version has a space between 'mc' and '(' like 'mc (', yours does not like 'mc('.
After removing the spaces the problem is gone.  We might call this an bug in spicesharpparser.
The more important the problem of the spicesharpparser version is, it might have been fixed with version 1.5.0.
The .csproj file still indicates a back level version of the spicesharpparser library 1.4.3, not 1.5.0.
You can easily see this by using the code browser on the github project website.
Regards, Adrian

@a3an-de I will take a look at the issue with space.
I forgot to push the commit to github. Sorry about that. It should be fine now.
Marcin

@marcin-golebiowski Ok, 1.5.0 is active.
However, I got a weird problem. When running test1.cir every thing looks ok, this is the circuit without an embedded blank in the first mc() call.

When I run test2.cir I get a different curve, this is the circuit with an embedded blank in the first mc() call.

when I insert a blank in all mc() calls, I get an exception.
at SpiceSharpGUI.Windows.ViewModels.NetlistResultWindowViewModel.RunSimulations() in C:\Users\a-sch\source\repos\SpiceSharpGUI\src\windows\SpiceSharpGUI.Windows\ViewModels\NetlistResultWindowViewModel.cs:line 227
---> (Inner Exception #0) SpiceSharp.CircuitException: No model specified
By the way, what is mc() doing ?
Regards, Adrian

@a3an-de  I will take a look tomorrow.
The souce code of mc is available at:
https://github.com/SpiceSharp/SpiceSharpParser/blob/master/src/SpiceSharpParser/ModelReaders/Netlist/Spice/Evaluation/Functions/Random/McFunction.cs

Hi Marcin,
I have been offline for some time and unable to track this issue.
I was trying to find out the status of this problem, I could not find
any information about this issue at all.
Could you tell me how this issue was resolved ?
Regards, Adrian
-------- Forwarded Message --------
Subject: 	Re: [SpiceSharp/SpiceSharpGUI] Exception while executing
circuit shown on SpiceSharpGUI/screenshots/screen02.png (#23)
Date: 	Sun, 17 May 2020 14:15:54 +0200
From: 	Adrian Schuur<EMAIL_ADDRESS>To: 	SpiceSharp/SpiceSharpGUI
<EMAIL_ADDRESS>Hi Marcin,
I did reinstall spicesharpgui again,  it did not help.
I noticed that the .csproj file indicates a back level version of the
spicesharpparser library 1.4.3, not 1.5.0.
Could that be the reason ? How would I change that ?
Regards, Adrian
Von meinem iPad gesendet

Am 16.05.2020 um 21:33 schrieb Marcin<EMAIL_ADDRESS>

@a3an-de https://github.com/a3an-de Do you need any help?
Kind regards,
Marcin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SpiceSharp/SpiceSharpGUI/issues/23#issuecomment-629695665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6ZTCFDDCREACAWQQXRKNDRR3TBBANCNFSM4NC7AMMQ.

Hello,
Sorry for the delay and no information about the status.
I'm very busy at the moment but I will look at this issue as soon as possible.
With kind regards,
Marcin

It's fixed in latest SpiceSharpGUI.
Sorry for not responding.


Feature/mj 01

please review the modified JS to write fetch to added setItem/getItem, modified results HTML/JS as well, attempting to link and display getItem.

LGTM


Add ability to update README file with descriptive metadata

Original README should be not be deleted but rather be renamed as something along the lines of README.yml.timestamp

Fixed with 4422b8e and https://github.com/jic-dtool/dtoolcore/commit/9da17d7ec61e1041315a95fb3ec4ebc3828d991b


nixos/pam_mount: fix cryptmount options

Description of changes
There was a bug in the pam_mount module that crypt mount options were not passed to the mount.crypt command. This is now fixed and additionally, a cryptMountOptions NixOS option is added to define mount options that should apply to all crypt mounts.
Fixes #230920
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

I have tested it in a VM and @jzbor was right with the assumption that the options were not passed. For me it works now, but of course it would be nice if @jzbor can confirm it too.

Hi, I hope I will find the time to test it later today or tomorrow. Do options specified directly inside the cryptmount tag still get applied independently of the new NixOS option?

Yes, the option is just for "general options" that should apply to all crypt mounts.
You can still define specific options in every volume tag.

Okay, thanks a lot for the fix! I am currently running the rebuild to test it...

Yes I can confirm. fstrim now works as expected. Thanks again for the effort. I think the global option is also a great idea.


Feature/input action refactor

Overview
Related to https://github.com/XRTK/XRTK-Core/pull/379

/azp run


x/build/cmd/gopherbot: add Gerrit mock/integration tests

We currently have no real tests for the way gopherbot interacts with Gerrit, nor is there a simple way to implement them. For complex (and, arguably, somewhat dangerous functionality, like #48021, autosubmit) this makes implementation a bit of a crapshoot, since we end up mostly just hoping we got the behavior right and testing in prod.

Since it's not already mentioned, I'll add that one of the few ways to test gopherbot changes now is via its -dry-run mode. This helps, but it's not a complete and ideal solution.
If we mock individual Gerrit responses and create tests based on those, it'll help with some types of changes, but contribute little for types of changes where a sequence of state changes (and behaving exactly like the real Gerrit server does, which changes over time) is critical (as one example, see https://github.com/golang/go/issues/21312#issuecomment-430051456).
There's some similarity and overlap here with #37603, which is about another component maintner that also interacts with Gerrit (and GitHub) (and is one of gopherbot dependencies at this time).

git-codereview implements a fake Gerrit server (type gerritServer) and a git helper (type gitTest) for its tests, which might be relevant here (spotted via #50576).


test cidr handling againts ipv6

In preperation of IPv6 compatibility adding a test to one of the aspects of ip address handling.
/area testing
/area networking
/kind technical-debt
/kind test
/priority normal
diff looks weird here because i introduced subtests for ipv4 and ipv6 as already done for disjointedness, existsting ipv4 tests are untouched.

Thank you @majst01 for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

Thank you @majst01 for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.


Not Supported tempnam(): open_basedir restriction in effect. File(/tmp) is not within the allowed path(s):

Hello, i have problem when sign image. that error

So how can i fix it

I folow guider so it work https://www.digitalocean.com/community/questions/open_basedir-and-safe_mode


swap GuildCreate and Ready, fix auto-mode

This PR swaps the GUILDCREATE and READY discord event handlers; it turns out when I initially wrote it on my end I got the two event types confused. The READY event does NOT contain initial voice state information, so initial setup hasn't been working this whole time. I fixed this in one of my own branches but was still testing some other stuff.
I also re-added the user count for Mumble. This is populated by a MumblePing as opposed to the mumbleUsers map, which only contains data when the bridge is actively connected to mumble. Without that auto-mode didn't work since the bridge never saw anyone connect to Mumble. This also meant I re-closed the Die channel since otherwise the bridge was failing to clean up properly and would also segfault.
If you were in the middle of doing something regarding the mumble/die chan stuff I can drop that commit. Mostly wanted to merge the discord change since that was my bad and actively doing the wrong behaviour.

looks good to me.

looks good to me.


Use forwarding symlinks to reduce toolchain installation size

swift-build, swift-package, swift-test, swift-package-collection, and swift-run are now installed as symlinks to _swift-package-manager, which selects the correct tool based on the path used to invoke it.
Motivation:
Reduce download & installation size of the Swift toolchain
Modifications:

Add a new executable target, _swift-package-manager which runs the appropriate Swift*Tool.main() based on the path used to invoke it.
When installing SwiftPM, only copy _swift-package-manager. Create symlinks named swift-build, swift-package, etc. to _swift-package-manager
In the future, we can probably get rid of the symlinks by teaching the driver how to invoke the subcommands directly.

Result:
-The size of the entire Swift toolchain is reduced by 3-4% in my local tests

@swift-ci please smoke test

@swift-ci please build toolchain

@swift-ci please smoke test

The macOS toolchain build is failing due to an unrelated issue, but I was able to get some concrete numbers by comparing the Ubuntu 16.04 toolchain to the 2/9 nightly snapshot:
Compressed:
2/9 nightly - 598 MB
With this change - 523 MB (12.5% reduction)
Uncompressed
2/9 nightly - 1.98 GB
With this change - 1.71 GB (13.6% reduction)
With this change, we go from 5 to 1 68 MB executables, which lines up with the numbers here - the impact ended up being much larger than I expected

This is awesome!

this is great, thank you @owenv

This looks great  In the long run it might make sense to have swift-package be the "native" one, with swift build, swift run, etc being shorthands swift package run, swift package test, etc.  I think this is what you were referring to in the notes for the PR, and this is something that was discussed a lot in the early days.  But this PR seems like a good and safe step.
It looks as if this doesn't change what gets built at debug time, which I don't think will be a problem but which we should maybe consider in a follow-on PR.

@owenv thanks again for putting together. we would need a bit of time before we can merge this, but will do ASAP

@tomerd no problem! I'm in no rush to land this

This looks great In the long run it might make sense to have swift-package be the "native" one, with swift build, swift run, etc being shorthands swift package run, swift package test, etc. I think this is what you were referring to in the notes for the PR, and this is something that was discussed a lot in the early days. But this PR seems like a good and safe step.

Yeah, I think there's a few different approaches we could take to clean this up. My one concern is that there might be external tools that are looking specifically for swift-package, swift-build, etc. in usr/bin/ instead of invoking SwiftPM through usr/bin/swift.

It looks as if this doesn't change what gets built at debug time, which I don't think will be a problem but which we should maybe consider in a follow-on PR.

I think the other executable targets are worth keeping around for now because they make manual testing much easier and they don't add much time to the build. _swift-package-manager is pretty annoying to use if you don't setup symlinks first.

Could the _swift-package-manager executable be a hidden file? Would it make sense to use the .swiftpm name?

@benrimmington I'm not entirely sure if that would work, but I think it's unnecessary. We already ship other executables like real_swift in the usr/bin/ directory that aren't useable as standalone tools

I just wanted to follow up on this - is there any particular reason it can't be shipped with Swift 5.5/anything I can do to unblock it?

apologies for the delay @owenv. we are waiting for the 5.5 branching to take changes that should land post 5.5

@swift-ci please smoke test

This looks great, @owenv — thank you for your work on this and for your patience!
It looks good to me, but I have a suggestion to simplify things even further:  since swift-package is the main tool of SwiftPM, I think the same thing could be achieved without a separate _swift-package-manager helper.
I suggest the following simplification, which seems to work well in preliminary testing:

use swift-package as the target of the symlinks
replace the main.swift source file in swift-package with the one that you wrote for _swift-package-manager
add the CMake TSCBasic dependency to swift-package instead of _swift-package-manager, etc

Then we still have just the one tool, and one symlink fewer (and SwiftPM target) fewer.
When swift-package was added, it was intended to become the "home" subcommand for package management, and so I think this would fit in well with that.

@abertelrud sure, that makes sense to me. I dropped the underscored targets, but kept the ones for swift-build, swift-test, etc. since they're still useful when not using the install action.

@swift-ci please smoke test

@abertelrud sure, that makes sense to me. I dropped the underscored target, but kept the ones for swift-build, swift-test, etc. since they're still useful when not using the install action.

Absolutely, I did not mean to suggest removing the existing executables, only not adding the new underscored one.  The existing ones are very useful during debug builds (in fact I use them all the time when working on SwiftPM in Xcode, using the scheme that's autocreated for each of them — and I suspect others use them in this was as well).
Perhaps in the future once we can create arbitrary symlinks during a debug build (perhaps with a postbuild action) we'll be able to remove them, but that's a ways off.

@owenv This looks ready to merge at this point.  Thanks again!

@abertelrud thanks for reviewing! There's no rush, but do you mind merging on my behalf, I don't have commit access to this repo


feat(retryableWrites): adding more conditions for retries

Fixes NODE-1456
~WIP b/c I am working on tests to verify it works in native.~ Tests in mongodb/node-mongodb-native#1714

No longer WIP


Change the anonymization display

When I see some anonymized screenshots, I'd like to see if they were anonymized by this plugin or by another way.
Change the display like this:

use a radk gray to mask the content
add a 1px black border around it

The 1px border is a bad idea, as numerous elements are on top of others, and there a re no way of telling if an element is inside another to cancel the border display. The color will be plain grey, without border.


0.1.4 Audio Support - Service is not defined

I get the error
/usr/lib/node_modules/homebridge-camera-ffmpeg/index.js:41
    cameras.forEach(function(cameraConfig) {
            ^
ReferenceError: Service is not defined
    at /usr/lib/node_modules/homebridge-camera-ffmpeg/index.js:55:37
    at Array.forEach (native)
    at ffmpegPlatform.didFinishLaunching (/usr/lib/node_modules/homebridge-camera-ffmpeg/index.js:41:13)

When this code tries to run;
       var microphoneService = new Service.Microphone("Microphone");
What am I missing?

Ah Service is missing. v0.1.5 should fix this.


Initial updates

Make edits t0
_includes/
_layouts/

Apr 2021 --
Arushi revamps

updates to main page
layout consideration
content reshuffling


[FEATURE] Add italian to weblate

Is your feature request related to a problem? Please describe.
I can translate to italian.
Describe the solution you'd like
If you add italian to weblate i can translate the strings for you.
Describe alternatives you've considered
I'm not a programmer therefore i'm unable to add locale support as you described in  the documentation.
Additional context
.

Hi, I added Italian language support on Weblate, you can follow this link.
Thank you very much for your contribution first! You can @mention me in this issue in the future, I will add your name to the contributor page in app.

@FriesI23 Done!


Add aks-engine redirection notice to front page README

What this PR does / why we need it: This commit will mark the official EOL of this codebase for future features and bug fixes!
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Add aks-engine redirection notice to front page README

/lgtm


Support expression in time field for timeseries

Is your feature request related to a problem? Please describe.
Currently if you using the TimeSeries component of the query builder, the user is forced to select a time field. Its not uncommon for users to a time field encoded in a different type e.g. int. ClickHouse supports expressions to allow conversion if required.
Describe the solution you'd like=
Please allow free typing of an expression in the Timefield input or allow switching to an expression mode.
Additional context

This needs alittle care since ther time field is used in the WHERE clause e.g. $__timeFilter(created_at) - this should probably use the alias time or id expect issues on some expressions.

This is now possible in v4
The $__timeInterval macro is still added when in Time Series Aggregate mode, but the query still functions as expected. time alias is also handled in the query. If the user wanted to exclude this macro they could use Table Aggregate mode.


Add files via upload

@AbdAzizFitouri Please provide us with an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

@AbdAzizFitouri You might miss my previous comment. Please react.

@AbdAzizFitouri I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for the understanding!


Health Status of Eureka and Zuul Services

We have Spring Cloud Netflix Eureka , Zuul and Config Server in our environment.
When we access the /health endpoint on them, we see the below information which includes "discoveryComposite",  "discoveryClient" and also "configServer"
discoveryComposite: {
    description: "Spring Cloud Eureka Discovery Client",
    status: "UP",
discoveryClient: {
   description: "Spring Cloud Eureka Discovery Client",
   status: "UP",
  services: [
       "config-server",
      "eureka-service",
      "zuul-service"
]
},
eureka: {
description: "Remote status from Eureka server",
   status: "UP",
applications: {
     ZUUL-SERVICE: 1,
     CONFIG-SERVER: 1,
     EUREKA-SERVICE: 1
}
}
},

What does the discoveryComposite and discoveryClient statuses mean?
Is there any documentation on them.
Also, we see the status information on Config Server too. We see this in all the 3 services, eureka , zuul and config when we access their /health endpoint.
We also , want to know why the config server status is being shown in the /health status of all the 3 services.

We are using Spring Cloud Netflix Dalston.SR2 and Spring Boot 1.5.4 in our env.

At least speaking of the configuration server (if I'm not mistaken), it's known to everyone, in fact it needs to be, because it shares properties and settings to all services.

The discoveryComposite is an aggregate of the health of all the discovery clients.
The discoveryClient is the health of your discovery client (you only have one in this case).
The config client also has a health indicator for the config server https://github.com/spring-cloud/spring-cloud-config/blob/master/spring-cloud-config-client/src/main/java/org/springframework/cloud/config/client/ConfigServerHealthIndicator.java


Issue-4276: update description of wrapDispatch - a nested function within middleware

Thanks for the PR!
To better assist you, please select the type of PR you want to create.
Click the "Preview" tab above, and click on the link for the PR type:

:bug: Bug fix or new feature
:memo: Documentation Fix
:book: New/Updated Documentation Content

@markerikson

Thanks!


feat: Allow excalidraw to calculate width/height based on parent container and only apply that to canvas

Remove width/height from the ".excalidraw" container so it will sized automatically.
updated all ref calculation to ".excalidraw" instead of parent since now ".excalidraw" will get resized
Kept the props width/height which will be only used to determine the canvas width/height but we can remove it as well as we are calculating from parent.
Resize handler is also not needed anymore.

Try here

[ ] Testing
[ ] fix tests (not sure why renders is getting increased due to tis change yet :/ )
[ ] update docs

3. Kept the props width/height which will be only used to determine the canvas width/height but we can remove it as well as we are calculating from parent.

I'd not remove the props since host apps may wanna do something custom and not make the width/height be dependent on parent container — though I guess it could be hacked by setting the width/height on the parent container instead — dunno, we'll see.

Kept the props width/height which will be only used to determine the canvas width/height but we can remove it as well as we are calculating from parent.

I'd not remove the props since host apps may wanna do something custom and not make the width/height be dependent on parent container — though I guess it could be hacked by setting the width/height on the parent container instead — dunno, we'll see.

Now that width/height is actually set by browser I think we shouldn't take it from props any more as it just sets the canvas width/height which can be calculated from ".excalidraw" container and we also need not handle the additional cases of updating offsets when width/height prop updates.

Btw, you're still defaulting the canvas width/height to window.innerWidth/innerHeight
yep since i am updating it in didMount so thats actually not getting used. I will remove that now.

Fixed the tests, not sure if this is the best way to fix it, I had to stub the getBoundingClientRect at few places since now width/height is calculated using this.

There's a problem where when if you put the excalidraw component into a container with no explicit dimensions (which includes rendering it in the root) it'll keep on expanding due to an feedback loop because it seems there's some discrepancy between the <canvas> height and <main> height (in which the canvas is rendered) — 4px discrepancy, not sure where it comes from, but it causes an infinite loop:
https://codesandbox.io/s/excalidraw-v050-forked-9uure?file=/src/App.js
In the example above, I've added a max-height: 500px on the canvas so it doesn't loop ad infinitum and you can debug.
Making <main> into a flex container fixes it.
But, it just fixes this very case. Problem is any other thing can make this blow up again, e.g. adding margin on canvas or whatever (check the styles.scss comments).

There's a problem where if you put the excalidraw component into a container with no explicit dimensions it'll keep on expanding due to a feedback loop because it seems there's some discrepancy between the <canvas> height and <main> height (in which the canvas is rendered) — 4px discrepancy, not sure where it comes from, but it causes an infinite loop:
https://codesandbox.io/s/excalidraw-v050-forked-9uure?file=/src/App.js
In the example above, I've added a max-height: 500px on the canvas so it doesn't loop ad infinitum and you can debug.
Making <main> into a flex container fixes it.
But, it just fixes this very case. Problem is any other thing can cause this feedback loop, e.g. adding margin on canvas or whatever (check the styles.scss comments).
Point is, dropping the <Excalidraw/> component into your DOM shouldn't cause an infinite loop no matter what layout/CSS you happen to have.

Yep this is a know issue when using canvas when we try to resize it according to parent (example), I had faced the same issue in excal app but that go fixed after giving 100% height to #root. There are several stack overflow issues regarding the same as well like https://stackoverflow.com/questions/8624268/html5-canvas-resize-to-parent, https://stackoverflow.com/questions/45949881/canvas-element-not-maintaining-height-of-parent-div
There are two work around for this

Display block in canvas -  this will make sure that 4px difference is not there but still it will have some initial height. Also adding padding, marging will still lead to circular loop since it contributes to increasing height (since we multiply by `window.devicePixelRatio)
Position absoulte the canvas - This is safe fix as now there is no circular dependency, height is also 0 and margin, padding etc wont affect.

I have pushed the fix with absolute, you can try out with version<EMAIL_ADDRESS>tested in your fork as well, works fine.

Good idea about the absolute positioning :+1:


Modify what column source_id is mapped to

The service_offering_source_ref needs to be mapped to the source_id off the Topology request.
This PR makes that change, and modifies the mapping to make use of method names as opposed to instance_variables.

@lindgrenj6 - Please review


Option to exclude certain columns for plotting

Also maybe auto-exclude non-numerical columns by default like groupby agg?
http://stackoverflow.com/questions/13003051/how-do-i-exclude-a-few-columns-from-a-dataframe-plot

It would be great to have a way to slice the df and get only a few columns for plotting, for example.
Something like
df[:,'2010':'2015']

on a data frame with years as columns, ranging from 2008 to 2015.

@ispmarin does df.loc[:,'2010':'2015'].plot() do what you want?


This Action does not work with extracted providers

Description
Providers are being extracted from octodns/octodns to their own repositories in the octodns org: https://github.com/octodns/octodns/issues/622
Currently, this Action does not work with providers that have already been extracted. That means, for example, this Action cannot be used with octodns/octodns at the default branch.
Expected Behavior
This Action should Just Work with octodns: If an extracted provider is used in the config, then install it.
Actual Behavior
If the provider has been installed by other means, this Action continues to function. If not, then an error probably similar to:
2022-01-11T20:48:20  [140712345147200] WARNING Route53 octodns_route53 shimmed. Update your provider class to octodns_route53.Route53Provider. Shim will be removed in 1.0
2022-01-11T20:48:20  [140712345147200] ERROR Route53 Route53Provider has been moved into a seperate module, octodns_route53 is now required. Provider class should be updated to octodns_route53.Route53Provider

Possible Fix
Could detect providers used in config, and install them.
Or install all of the extracted providers.
Or stick with v0.9.14 forever.
Steps to Reproduce

Configure octodns to use an extracted provider
Configure workflow to run this action with octodns_ref set to "master"
Run that workflow

Context
Can't use the latest octodns/octodns with this Action.
This workflow definition failed because I didn't install octodns_route53:
      - name: Run `octodns-sync` with ${{ needs.meta.outputs.config }}
        id: octodns-sync
        uses: solvaholic/octodns-sync@main
        with:
          config_path: ${{ needs.meta.outputs.config }}
          octodns_ref: master

This workflow definition ran successfully:
      - name: Run `octodns-sync` with ${{ needs.meta.outputs.config }}
        id: octodns-sync
        uses: solvaholic/octodns-sync@main
        with:
          config_path: ${{ needs.meta.outputs.config }}
          octodns_ref: v0.9.14

Your Environment

Version used: solvaholic/octonds-sync@main, octodns/octodns at master
Link to your project: https://github.com/solvahol/dns

I took a very superficial look at your repo.  I could not determine offhand what mechanism replaced it. My expectation would be that you (as well as the vast majority of users) have some python package management in place that handles the requirements and installation step, given that you don't change providers very often in workflows like that.
imho this would be the proper place to address your issue (in a "package install" or "dependency solution" step) rather than expecting that some package auto-installs some other packages just due to input in the payload (the configuration file).
For me, requirements.txt would be the perfect place to put the newly created OctoDNS Provider requirements alongside the probably pinned version of octodns itself, rather than to bury it deep down in the workflow scripts.
Personally, I would always refrain from any "auto-install-something-based-on-input-data" pattern for the sake of security, safety and readability. ymmv.

Thank you @istr :bow:

For me, requirements.txt would be the perfect place to put the newly created OctoDNS Provider requirements alongside the probably pinned version of octodns itself, rather than to bury it deep down in the workflow scripts.

I appreciate you taking the time to look into this and share your recommendation. I've been thinking of installing OctoDNS and installing provider dependencies as part of this Action, and now I see they needn't be.
Removing those to a step in the user's workflow would simplify their control of the environment, and their dependency and vulnerability management. And it'd remove complexity from this project. While eliminating this issue. 👍

@barnumbirr Thank you for writing this up and describing workaround you used :bow:
I'll make time this coming weekend (18-19 Jun) to fix this in the octodns-sync Action and its documentation.
Do you think these steps would suffice?:

Have the Action error out when differing octodns versions are indicated, for example in workflow definition, environment variables, and requirements.txt
When the has been moved into a seperate module message occurs, log a link to documented solution
Instruct octodns-sync Action users, in documentation, to specify octodns and provider versions in requirements.txt

👋 I pushed changes to the issue86 branch for this. They're up for review in #89
If you'd like to test the issue86 branch, configure your workflow job step to use it:
        uses: solvaholic/octodns-sync@issue86

To see a test already set up, check out the validate and deploy workflows in solvahol/dns and https://github.com/solvahol/dns/pull/35.
I plan to get these changes, or improved versions, in to main on 19 Jun.

👋 Merging #89 closed this issue. It removed octodns installation to a workflow step.
The user's workflow can install octodns and required providers any way the user likes. Example workflows in documentation suggest installing from requirements.txt, as @barnumbirr described in https://github.com/solvaholic/octodns-sync/issues/86#issuecomment-1153554144.
If you have questions about this, or continue to see problems, please re-open this issue or create a new one :bow:

@solvaholic thanks a bunch. Limitied testing on my end seems to confirm this works as expected 🎉 .
Is a new release of this action planned at some point? I'd prefer to be able to used a defined version in my workflows rather than have to use @main.

Thanks for following up and letting me know @barnumbirr!

Is a new release of this action planned at some point? I'd prefer to be able to used a defined version in my workflows rather than have to use @main.

Yes! I've messed up releases in this repo, and I think there are some feedback to integrate, so I want to properly get my head into it.
I think this'll become 3.0 on 21 Jun. I feel silly going up to 3.x but the changes for this will break things for users who specify octodns_ref for this Action, either in the Action input or through environment variables.


Investigate aligning Collector and Instrument Interfaces

The instrument and collector APIs are currently distinct, investigate the possibility of aligning them to provide a more unified interface.

+1 for a unified interface, there are a few energy meter instruments (equivalent) in LISA that are currently bit-rotting. If the collector API was used for everything, they would probably be best contributed back to devlib and benefit from all the niceness coming with it (context manager etc)


feat(props): display deprecated and default props

Proposed Changes
Inside of the props table,

display the default value when it is in the jsdoc
display the deprecated props with the associated comment

Potential Breaking Changes
None
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

The link for the toast via the home page is broken :eagle: :eye:


Handle lane errors in Swift

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Hello, I am new to fastlane swift, is there any way to handle errors, occurred during Fastfile.swift lane invocation?
I found method func onError(currentLane: String, errorInfo: String), but it seems, that method is not invoked for errors.

@haritowa That is a good question! I have been making some improvements to swift lately but I have not come across this yet so I will look into this for you! 🙃

@joshdholtz Throws lanes will be the best choice, I guess. Maybe I'll investigate it later, unfortunately, there is no chance for me to do any side work for now ¯_(ツ)_/¯

Any updates?


Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError

I'm using a custom dataset and json to train the network.
After a few iterations over the dataset, train.py crashes with the following error:
Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError
However if i set --num_workers=0 or --num_workers=1 the training works properly.
This is how the traceback looks:
Traceback (most recent call last):   File "/home/amuresan/anaconda3/envs/pytorch1.3-gpu/lib/python3.7/multiprocessing/queues.py", line 236, in _feed     obj = _ForkingPickler.dumps(obj)   File "/home/amuresan/anaconda3/envs/pytorch1.3-gpu/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps     cls(buf, protocol).dump(obj) _pickle.PicklingError: Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError
I'm using torch 1.3.1 with CUDA 10.2. I've tried multiple versions of pytorch and the results are the same.

I am experiencing the same issues with CUDA 10.2 and torch 1.4. The machine is definitely not running out of memory. The maximum consumption of memory I have seen before the exceptions occurred was 60Gib / 128Gib.
(pid=62601)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/queues.py", line 236, in _feed
(pid=62601)     obj = _ForkingPickler.dumps(obj)
(pid=62601)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps
(pid=62601)     cls(buf, protocol).dump(obj)
(pid=62601) _pickle.PicklingError: Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError
(pid=62601) Traceback (most recent call last):
(pid=62601)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/queues.py", line 236, in _feed
(pid=62601)     obj = _ForkingPickler.dumps(obj)
(pid=62601)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps
(pid=62601)     cls(buf, protocol).dump(obj)
(pid=62601) _pickle.PicklingError: Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError
(pid=62607) Traceback (most recent call last):
(pid=62607)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/queues.py", line 236, in _feed
(pid=62607)     obj = _ForkingPickler.dumps(obj)
(pid=62607)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps
(pid=62607)     cls(buf, protocol).dump(obj)
(pid=62607) _pickle.PicklingError: Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError
(pid=62601) Train Epoch: 1 [10/120 (8%)] Loss: 0.831871
(pid=62607) Train Epoch: 1 [20/120 (17%)] Loss: 0.883739
(pid=62601) Train Epoch: 1 [20/120 (17%)] Loss: 0.754329
(pid=62607) Train Epoch: 1 [30/120 (25%)] Loss: 0.769951
(pid=62607) ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
(pid=62607)  ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
(pid=62607)  ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
(pid=62607)  ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
(pid=62607)
(pid=62601) ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
(pid=62601)  ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
(pid=62601)
(pid=62601) Traceback (most recent call last):
(pid=62601)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/queues.py", line 236, in _feed
(pid=62601)     obj = _ForkingPickler.dumps(obj)
(pid=62601)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps
(pid=62601)     cls(buf, protocol).dump(obj)
(pid=62601) _pickle.PicklingError: Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError
(pid=62607) Traceback (most recent call last):
(pid=62607)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/queues.py", line 236, in _feed
(pid=62607)     obj = _ForkingPickler.dumps(obj)
(pid=62607)   File "/home/herzc/.conda/envs/torch1.4/lib/python3.7/multiprocessing/reduction.py", line 51, in dumps
(pid=62607)     cls(buf, protocol).dump(obj)
(pid=62607) _pickle.PicklingError: Can't pickle <class 'MemoryError'>: it's not the same object as builtins.MemoryError

@dbolya sorry for the late answer. The machine wasn't running out of RAM.

@che85 seems like you're running out of /dev/shm space. Reduce number of workers in DataLoader or the batch size.

I am encountering this as well. Is there any way to fix this?

@Traderain Check your /dev/shm size, and increase if needed :)


Enable user set storageClass in helm charts

Some components in graphscope-store are using Persistent Volume. We need to enable user to configure storageClass name manually to easier get advantage of dynamic provision.

Closed as completed.


Fix draggable icon

Resolves #106

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8666295) 98.25% compared to head (cd47059) 98.25%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage   98.25%   98.25%
=======================================
  Files          51       51
  Lines         916      916
  Branches      194      194
=======================================
  Hits          900      900
  Misses          3        3
  Partials       13       13

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


pretrained VGG model

hi did you use pretrained VGG model? Then how did you subsample the parameter from fc6 and fc7?

The weights are directly ported from Caffe weights, provided by the authors in their repo. Check preparation part of their README to find the initial pretrained weights. Actually, this weights are taken from this paper.


Feature/mvn project

Doesn't seem to be able to run locally

change from mvn to ant since it's easier with intellij


Only working on canary

Hi
I'm pretty sure I tested this a few weeks ago using edge 86.0.622.51 and I thought it worked just fine. Since then I seem to only be able to get this to run on edge canary.
Currently using the newest edge Version 86.0.622.61 (Official build) (64-bit) and cant seem to get it working. It opens a window but behaves like it failed to instantiate the webview.
Thank your for your help and maintaining c# bindings for this.

Hey! If a non valid Edge Chromium version is present on the current system, webview will fallback to the old Edge. Therefore you should never get a blank window. Only difference should be the used browser engine. May I ask what you mean with "behaves like it failed to instantiate the webview"?

it opens the window, but there doesn't seem to be a webview. On right click there is no context menu, F5(refresh), F12(dev tools) seem to have no effect.

Could you please test, if the minimal example is working for you.

Just tested, the minimal example did work. So something else is going on. I had tried to create my own minimal example and encounter the same issue so it think its something in my project or build.

It seems to specifically break when I use http://localhost:8080

Please execute the following in admin command prompt and try again:
CheckNetIsolation.exe LoopbackExempt -a -n="Microsoft.Win32WebViewHost_cw5n1h2txyewy"

Still seems to be broken after running that command

Do you have something running on http://localhost:8080? Just changing the URL to localhost in the minimal example will not automatically host a webserver. If you want the application to host its content you have to use a HostedContent like in the DesktopApp example.

Yes I am using localhost for development to support hot reloading

Did you check, if the DesktopApp example works? If not, it sounds like there is still a problem with the loopback. If the above command was not working for you, you also could check the loopback settings with this gui manager: https://github.com/tiagonmas/Windows-Loopback-Exemption-Manager

Hi so I'm not sure what happened maybe I gave you false feeback. It randomly started working on my machine I'm not sure what exactly happened or when it started working. Was trying to fix this in between other work issues.
I checked with a coworker that it was still broken on his machine, had him run your command and then it was working on his machine. So I assume running the command is what fixed it on mine but I'm not sure why it didn't work right away.

Is this something that should be documented in the readme? I assume most people will develop on localhost as they build their web ui. Thanks so much for your help investigating this with me, I really appreciate it

Also it would be nice if the window gave more feed back as to what the issue was if thats possible. Right now it just looked like it failed to create the webview.

No problem. The problem with the localhost is already documented in the readme.
But yes, maybe it is possible to check, if the command ran successfully and to give feedback. Good idea! :)

Oh sorry I didn't see that section...and I read the documentation multiple times...haha

Implemented a check. If loopback is not enabled on windows, the library will now show a message.

Have to test it on linux. As soon as everything is running fine, I will release a new version including this feature.

Closed by #13