Ludwig serve doesn't work

I tried this in Google Collab and on my Mac.
Environment

Google Collab : Python 3.6.9
Mac : Python 3.7.6
pip install 'ludwig[serve]'  done, no errors

sudo -H pip install 'ludwig[serve]'                Requirement already satisfied: ludwig[serve] in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (<IP_ADDRESS>) Requirement already satisfied: pandas>=0.19 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (1.0.3) Requirement already satisfied: tqdm in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (4.46.0) Requirement already satisfied: Cython>=0.25 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.29.17) Requirement already satisfied: numpy>=1.15 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (1.18.1) Requirement already satisfied: scikit-learn in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.22.1) Requirement already satisfied: PyYAML>=3.12 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (5.3.1) Requirement already satisfied: absl-py in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.10.0) Requirement already satisfied: tensorflow==1.15.3 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (1.15.3) Requirement already satisfied: h5py>=2.6 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (2.10.0) Requirement already satisfied: tabulate>=0.7 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.8.7) Requirement already satisfied: scipy>=0.18 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (1.4.1) Requirement already satisfied: uvicorn; extra == "serve" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.11.8) Requirement already satisfied: fastapi; extra == "serve" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.61.1) Requirement already satisfied: pydantic==0.32.2; extra == "serve" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.32.2) Requirement already satisfied: python-multipart; extra == "serve" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from ludwig[serve]) (0.0.5) Requirement already satisfied: python-dateutil>=2.6.1 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from pandas>=0.19->ludwig[serve]) (2.8.1) Requirement already satisfied: pytz>=2017.2 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from pandas>=0.19->ludwig[serve]) (2020.1) Requirement already satisfied: joblib>=0.11 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from scikit-learn->ludwig[serve]) (0.15.1) Requirement already satisfied: six in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from absl-py->ludwig[serve]) (1.15.0) Requirement already satisfied: keras-applications>=1.0.8 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.0.8) Requirement already satisfied: tensorflow-estimator==1.15.1 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.15.1) Requirement already satisfied: tensorboard<1.16.0,>=1.15.0 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.15.0) Requirement already satisfied: termcolor>=1.1.0 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.1.0) Requirement already satisfied: keras-preprocessing>=1.0.5 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.1.2) Requirement already satisfied: wheel>=0.26; python_version >= "3" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (0.34.2) Requirement already satisfied: astor>=0.6.0 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (0.8.1) Requirement already satisfied: protobuf>=3.6.1 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (3.13.0) Requirement already satisfied: opt-einsum>=2.3.2 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (3.3.0) Requirement already satisfied: grpcio>=1.8.6 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.31.0) Requirement already satisfied: wrapt>=1.11.1 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (1.11.2) Requirement already satisfied: gast==0.2.2 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (0.2.2) Requirement already satisfied: google-pasta>=0.1.6 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorflow==1.15.3->ludwig[serve]) (0.2.0) Requirement already satisfied: uvloop>=0.14.0; sys_platform != "win32" and sys_platform != "cygwin" and platform_python_implementation != "PyPy" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from uvicorn; extra == "serve"->ludwig[serve]) (0.14.0) Requirement already satisfied: click==7.* in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from uvicorn; extra == "serve"->ludwig[serve]) (7.1.2) Requirement already satisfied: h11<0.10,>=0.8 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from uvicorn; extra == "serve"->ludwig[serve]) (0.9.0) Requirement already satisfied: websockets==8.* in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from uvicorn; extra == "serve"->ludwig[serve]) (8.1) Requirement already satisfied: httptools==0.1.*; sys_platform != "win32" and sys_platform != "cygwin" and platform_python_implementation != "PyPy" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from uvicorn; extra == "serve"->ludwig[serve]) (0.1.1) Requirement already satisfied: starlette==0.13.6 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from fastapi; extra == "serve"->ludwig[serve]) (0.13.6) Requirement already satisfied: setuptools>=41.0.0 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorboard<1.16.0,>=1.15.0->tensorflow==1.15.3->ludwig[serve]) (47.1.1.post20200604) Requirement already satisfied: werkzeug>=0.11.15 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorboard<1.16.0,>=1.15.0->tensorflow==1.15.3->ludwig[serve]) (1.0.1) Requirement already satisfied: markdown>=2.6.8 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from tensorboard<1.16.0,>=1.15.0->tensorflow==1.15.3->ludwig[serve]) (3.2.2) Requirement already satisfied: importlib-metadata; python_version < "3.8" in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from markdown>=2.6.8->tensorboard<1.16.0,>=1.15.0->tensorflow==1.15.3->ludwig[serve]) (1.6.0) Requirement already satisfied: zipp>=0.5 in /Users/macmini/opt/anaconda3/lib/python3.7/site-packages (from importlib-metadata; python_version < "3.8"->markdown>=2.6.8->tensorboard<1.16.0,>=1.15.0->tensorflow==1.15.3->ludwig[serve]) (3.1.0)
Screenshots
Image

you have t ospecify the path to the model, not to the experiment. In your specific case, add /model to the path you are providing.

I have the same problem : https://p479.p0.n0.cdn.getcloudapp.com/items/X6uNKepR/Image%202020-09-01%20at%2010.23.31%20AM.png?v=bcb4256b9eaddfcd61b2457b6ffe1a46

I still have the same problem screenshot

Are you sure you are running from the virtual environment where you installed ludwig[serve]? Does the model directory contain the model files (weights, hyperparameters and train set metadata)?

Yes,  I used the same virtual environment. I installed FastAI and launched a server with the Ludwig API and it worked but when I use
ludwig serve I always have the same issue.
Since I need to put into production 2 models at the same time I will keep the FastAI option, but it's a pity that the error thrown  by ludwig was not more explicit about the origin.

This is the code that generates your error:
try:
    import uvicorn
    from fastapi import FastAPI
    from starlette.datastructures import UploadFile
    from starlette.requests import Request
    from starlette.responses import JSONResponse
except ImportError:
    logger.error(
        ' fastapi and other serving dependencies are not installed. '
        'In order to install all serving dependencies run '
        'pip install ludwig[serve]'
    )
    sys.exit(-1)

It means that you problem has to do with your environment and your dependencies.
Could you please try to open a python prompt and load those dependencies one by one? i expect one of them will fail, and we can figure out from there.

Also I just pushed to master a commit that prints the full error: https://github.com/uber/ludwig/commit/42ed14f9183bf698c7a2ec382db5e235c0ab4557
Can you please install ludwig from master and try again?
pip uninstall ludwig & pip install git+http://github.com/uber/ludwig.git

Hello,
Since I was using FastAPI for the server I was not using ludwig serve anymore since this afternoon (Paris time). I tried what you suggested (the import of all the dependencies) and they passed so I tried ludwig serve again and it worked this time. However what I noticed when I was installing FastAPI manually is that Pydantic causes some problems during the installation and you have to try many times. So my best guess is that when you install FastAPI with pip install ludwig[serve] there's a dependency that doesn't install correctly hence causing the other problems.
Thanks for your prompt answer and help with this matter.

I see, thank you for reporting on that.
Glad things worked out anyway :)
If you happen to be able to post here what error were you encountering without ht sudo -H I can try to dig deeper.
Thank you so much anyway!


Deploy available version

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

Merging into a branch.


Breaking API change

It appears between 1.0.0 and 1.1.0 the options.strict, which would throw if there was an error and strict was truthy (false by default), was removed:
https://github.com/shinnn/broccoli-clean-css/blob/v1.0.0/index.js#L42-L44
This results in builds failing since they started pulling in newer versions of the lib, see: https://github.com/ember-cli/ember-cli-preprocessor-registry/issues/8
Would you be willing to accept a PR to bring it back for 1.x and in 2.x revisit removing it to respect semver?
Thanks, and great plugin :+1:

@jasonmit Thanks for this report, and sorry for my late, really late response (over 1 year!).
I had thought of the best way to fix https://github.com/ember-cli/ember-cli-preprocessor-registry/issues/8  for a long time.

As you said, broccoli-clean-css v1.1.0 includes a breaking change and I should add strict option again and publish v1.1.1.

Basically, I should have not add strict option to this plugin. It does not solve any problems, but in fact it dangerously hides errors that the CSS authors would rather fix. Even if I revert strict option removal, https://github.com/ember-cli/ember-cli-preprocessor-registry/issues/8 is not actually solved. What they really should do is not pretending that they didn't see the error, but editing their CSS file and fix broken @import.

I've struggled with those two my own conflicting opinion, and here is my final conclusion:

Publish broccoli-clean-css v2.0.0 without strict option.
Hazardous strict option will never been restored in v1.x. If you want to swallow the error at any cost, use clean-css inline option instead. Otherwise, though I think this is the worst solution, pin the version of broccoli-clean-css in package.json to 1.0.0.

So, https://github.com/ember-cli/ember-cli-preprocessor-registry/issues/8 won't be fixed by my side.
Thanks.


Outgoing connection to Azure SQL Server are closed on Consumption work profile

This issue is a: (mark with an x)

[ ] bug report -> please search issues before submitting
[ ] documentation issue or request
[x] regression (a behavior that used to work and stopped in a new release)

Issue description
We are running an azure container app on the consumption work profile. (1vcpu/2GB Ram) and is running with 2 instances. The deployed docker container is a Keycloak v26 release. Keycloak connects to our Azure SQL database. The container keeps open a connection pool of 20 connections to the sql database per replica.
Since around 2-3 weeks, we suddenly see "Connection reset by peer" errors when these connections are tried to beused. If i understand correctly, something inbetween (or the Azure sql server itself) closed the connections.
Since we initially thought, that this was due to killing of idling connections, we enabled sql connection validation and configured it that every 10 seconds a "select 1" statement is sent for each connection to keep it alive and validate it. We see on the sql server side that it works and each connection is used every 10 seconds.
However, the problem continues and we see the same amount of connection reset by peer errors.
Our current investigations are:

On Azure SQL side we are far away from any connection limits etc.
All zure applications connection to this Azure sql server behave normal.
This behaviour is not happending all the time. Sometimes, we do not see any dropped connections for a couple of hours, then it starts again.
We deploy the exact same docker container/configuration from an azure VM. The problem does not occur. Everything is stable.
We deploy the exact same configuration but with a dedicated work profile. The problem does not occur. Everything is stable

Steps to reproduce

Create Container app with consupmtion work profile
Deploy Keycloak container

Expected behavior [What you expected to happen.]

Connections to sql server are not closed.

Actual behavior [What actually happened.]
The connection to the sql server remains open (if it is active)
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Ex. Did this issue occur in the CLI or the Portal?

Hi @claria , can you send us the ARM resource URI to acasupport(at)microsoft(dot)com? We will investigate.

I sent the resource uri via mail.

[like]  Jenny Lawrance reacted to your message:

From: Georg Sieber @.>
Sent: Tuesday, November 5, 2024 10:09:44 AM
To: microsoft/azure-container-apps @.>
Cc: Comment @.***>
Subject: Re: [microsoft/azure-container-apps] Outgoing connection to Azure SQL Server are closed on Consumption work profile (Issue #1338)
I sent the resource uri via mail.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/azure-container-apps/issues/1338#issuecomment-2456757002 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFLXSMPIMPUA7TCN3CGTBOLZ7CKORBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI2DAMRYHA3DMMRRGOSG4YLNMWUWQYLTL5WGCYTFNSWHG5LCNJSWG5C7OR4XAZNMJFZXG5LFINXW23LFNZ2KM5DPOBUWG44TQKSHI6LQMWVHEZLQN5ZWS5DPOJ42K5TBNR2WLKJUGE4TGOJYHE4TJAVEOR4XAZNFNFZXG5LFUV3GC3DVMWVDENRTGIZDSMRZGAZIFJDUPFYGLJLMMFRGK3FFOZQWY5LFVI2DAMRYHA3DMMRRGOTXI4TJM5TWK4VGMNZGKYLUMU.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This is a count of all the "connection reset by peer" errors we see. We have two replica of the container app running. As you can see sometimes the error do not occure for over an hour for one replica while still happening for the other:
Logs are written to a log analytics workspace. Please have a look by yourself if you want.

Any update @JennyLawrance ?

Yes, We applied a fix on your environment last week. Let us know if you see improvement in the connection status.

Hi @JennyLawrance,
no, unfortunately the situation did not improve. These are the experienced connection error counts over the last week which have not really changed:

Hi @claria , I was able to confirm that we had an internal crossing of wires, and the fix wasn't applied to this environment as I previously stated. I apologize for the miscommunication here. We applied it today again (and I confirmed that at my side.)
Can you check the status on your end and let us know what you see in your logs?
Thanks, Jenny

Hi @JennyLawrance,
i can confirm that the errors are completely gone after your fix. What did you change and how can we prevent the errors to happen in the future?
We plan to create multiple similar environments. Do we have to expect the same error to happen there?

@JennyLawrance Could you please confirm if this fix was applied globally or only on this one affected instance?

@claria Thanks for your patience. We have applied the fix to West Europe, all newly created Container App replicas will get the fix.
@jsheetzmt , can you send across your Azure resource ID for the impacted instance? We will investigate and find out if you are running into the same root cause as the one discussed for West Europe.


tenant name with ‘-’ seems not be properly handled.

Hello, I created tenants "app-elk" and "app_elk" and set "Read and Write" to user "app-elk". On the "Read and Write" dialog of user "app-elk"，I could only see "app_elk"。

Open Distro for Elasticsearch version 1.12.0
Please help! Thanks a lot.

Thanks for the report, @ZeLAxyz, and apologies for the late response. I am not able to reproduce in more current versions of OpenSearch, closing as resolved. Please let me know if that's not the case and we can definitely look into it!


gdrive spreadsheet in 3bot

The idea here is to be able to grab info from google drive docs.
Expose openresty endpoint:

[ ] http://$IPADDR/gdrive/spreadsheet/$gsheet_guid.png

The gedis actor should:

get the doc if older than 5 minutes.
Will cache the pdf on filesystem for 5 min.
save the slide to /sandbox/var/gdrive/spreadsheet/$gsheet_guid.pdf

resuming investigations to find a way to convert sheets to pdf

Gdrive api used in the gdrive client doesn't provide export methods for spread sheet but I found an endpoint which serves the spreadsheet in pdf format but it has 2 problems:
1- it requires the spread sheet to be public
2- we still need to find a way to convert the response which is a html file containing the pdf to pdf directly, I'm still investigating if there are headers to set the response type or not
example url:
https://docs.google.com/spreadsheets/d/1qRiT7Ui98QCZcYm43Kd7ZMERbbVicY-TKBe5soSmOcs//export?format=pdf&size=0&fzr=true&portrait=false&fitw=true&gridlines=false&printtitle=true&sheetnames=true&pagenum=CENTER&attachment=true

finally reached the problem, The problem was that the sheets api doesn't provide export method but we can still use the drive api with sheets token to do that which was very strange that you use a service with another service key, it's working now and added to the endpoints

Can you update the docs with the endpoint.

updated

Verified on
digitalmeX: 5f5bdd33e1f771e3b62119984d49d7ee4b148639
jumpscaleX: https://github.com/threefoldtech/jumpscaleX/commit/060ed8f93c3d28ea2a9bc256d03b9cc359640bf1


List the PackageIdentifier of the package to be added to automation
ByteDance.Feishu
Details about the application
https://www.feishu.cn/api/downloads
PackageVersion: $result.versions.Windows.version_number
InstallerUrl: $result.versions.Windows.download_link

@SpecterShell I'm working on AutomationV2, hopefully, all the errors will be fixed soon since I'll be re-adding the packages to the automation.

Fixed in Automation v2.0.0.


Build simulator on Melodic and Gazebo 9

The simulator does not build on melodic instead of kinetic. The package that does not buil is uuv_sensor_ros_plugins.

Error log:
Errors     << uuv_sensor_ros_plugins:make /home/michael/manta_ws/logs/uuv_sensor_ros_plugins/build.make.004.log
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
make[2]: *** [CMakeFiles/uuv_gazebo_ros_camera_plugin.dir/src/UnderwaterCameraROSPlugin.cc.o] Error 4
make[1]: *** [CMakeFiles/uuv_gazebo_ros_camera_plugin.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
make[2]: *** [CMakeFiles/uuv_gazebo_ros_rpt_plugin.dir/src/RPTROSPlugin.cc.o] Error 4
make[2]: *** Waiting for unfinished jobs....
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
make[2]: *** [CMakeFiles/uuv_gazebo_ros_dvl_plugin.dir/src/DVLROSPlugin.cc.o] Error 4
make[2]: *** Waiting for unfinished jobs....
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
make[2]: *** [CMakeFiles/uuv_gazebo_ros_rpt_plugin.dir/src/ROSBasePlugin.cc.o] Error 4
make[1]: *** [CMakeFiles/uuv_gazebo_ros_dvl_plugin.dir/all] Error 2
make[1]: *** [CMakeFiles/uuv_gazebo_ros_rpt_plugin.dir/all] Error 2
make: *** [all] Error 2
cd /home/michael/manta_ws/build/uuv_sensor_ros_plugins; catkin build --get-env uuv_sensor_ros_plugins | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
...............................................................................
Failed     << uuv_sensor_ros_plugins:make                [ Exited with code 2 ]
Failed    <<< uuv_sensor_ros_plugins                     [ 55.6 seconds ]
Abandoned <<< manta_description                          [ Unrelated job failed ]
Abandoned <<< uuv_descriptions                           [ Unrelated job failed ]

Replacing our uuv folders with those from uuv_simulator did not affect the problem. This makes me think the problem is rooted in the installation or setup

The file /uuv_sensor_plugins/uuv_sensor_ros_plugins looks very interessting

There might be a case of "but it's working on my computer" here... I am not able to reproduce the build error you attached.
build.make.000.log

That is very interesting! The bug is probably caused by the installation or the dependencies then. What path variables did you use?

Path variables do not affect the problem

Full log of failed build:
build.make.009.log

The simulator is up and running on my computer. I mostly followed the readme in manta-auv.

The simulator is up and running on my computer. I mostly followed the readme in manta-auv.

works for me too

The simulator is up and running on my computer. I mostly followed the readme in manta-auv.

works for me too

It seems like the simulator works on melodic if you have not previously installed kinetic?

It seems like the simulator works on melodic if you have not previously installed kinetic?

It definitely seems to be working on clean installs. I'll try it out on my computer today, and if it works we can close the issue

Had some issues using
git clone .
The simulator managed to  build when I used
git clone -b melodic

Had some issues using
git clone "URL for vortex_msgs".
The simulator managed to build when I used
git clone -b melodic "URL for vortex_msgs"

Known issue, README needs to be updated, or better yet, merge the melodic branch into master.

Update: removing and reinstalling all of ros melodic and the workspace did not solve the issue for me.
Clean ubuntu install is up next..

Had some issues using
git clone "URL for vortex_msgs".
The simulator managed to build when I used
git clone -b melodic "URL for vortex_msgs"

The simulator is up and running on my PC as well, after I git cloned the Melodic branch of vortex_msgs

Since the simulator works for nearly everyone - poor Michael - this issue is hereby closed.


Issue with using discriminator

Hello,
I'm trying to use discriminator but when the validation occurs, it gets the discriminator name, and that's it, I does not get the relevant object.
For example
  payment_method:
    type: object
    required:
    - payment_method_type
    discriminator: payment_method_type
    properties:
      payment_method_type:
        type: string
        enum:
        - Crypto
        - CreditCard

The schema gets payment_method_type as String - for example it equals Crypto but it does not get the definition of Crypto and compares the request to this object.
Please advise.

discriminator support was never implemented.  #241 is for tracking this.

@whitlockjc , thank you for the reply, do you believe adding support for discriminator is possible in a relatively short amount of time? I saw that the project is using schema from 'http://json-schema.org/draft-04/schema' and that discriminator is configured as a String, do you know of any json-schema that has the discriminator configured properly?

The discriminator in OAS is very complex to implement.  It seems simple but since any model can have a discriminator, you have to walk a schema and its referenced/sub schemas and basically turn them into an anyOf.  It's quite complex.  sway is where the current work is happening since swagger-tools is deprecated (See #335).


Find out how to test generators

Generator projects are something we definitely should test as they are key parts of the libraries.

I use following approach. https://github.com/kant2002/WinFormsComInterop/blob/0b2edf8bb8c6f1a1586f63f1d4b8ac18ab21bf45/WinFormsComInterop.SourceGenerator.Tests/CodeGenerationTestBase.cs#L13
Maybe your specific way how to test would be different. For example you can consider creation of small application which declare parsers with specific features, then these application would run and validated to be producing desired outputs.

@kant2002 Generating the output as a string and comparing it is definitely one way, thanks for showing it to me! I'll have to consider if this is the best approach.
I'm planning to move away my generation logic to use Scriban templates, so maybe involving the compiler won't even be necessary.


KeyError and CreateRuleException while using modifying use statement

Snakemake version 6.1.1 and git version 6.1.1+7.g5349f2a4
Describe the bug
Errors are thrown if we have modified rules made using the use rule ... as ... with: syntax in a Snakemake module while snakemake --list.
Minimal example
Assume there are two rules defined in a module other_workflow.smk. There is one scenario in which everything works as expected and 2 in which errors are thrown.
rule some_task:
    output: "output.txt"
    shell: "cat hello >> {output}"

use rule some_task as another_task with:
    output: "another.txt"

Working Snakefile

module other_workflow:
    snakefile: "other_workflow.smk"

use rule * from other_workflow

❯ snakemake -l
some_task
another_task

Buggy Snakefile

module other_workflow:
    snakefile: "other_workflow.smk"

use rule * from other_workflow as other_*

❯ snakemake -l
KeyError in line 7 of /home/avmo/src/sandbox/snakemake-module-issue/other_workflow.smk:
'some_task'
  File "/home/avmo/src/sandbox/snakemake-module-issue/Snakefile", line 5, in <module>
  File "/home/avmo/src/sandbox/snakemake-module-issue/other_workflow.smk", line 7, in <module>

It seems like an issue with the scope of the modules.

Buggy Snakefile

module other_workflow:
    snakefile: "other_workflow.smk"

use rule * from other_workflow

use rule some_task from other_workflow as other_some_task with:
    output:  "some-result.txt"

❯ snakemake -l
CreateRuleException in line 7 of /home/avmo/src/sandbox/snakemake-module-issue/other_workflow.smk:
The name another_task is already used by another rule
  File "/home/avmo/src/sandbox/snakemake-module-issue/Snakefile", line 9, in <module>
  File "/home/avmo/src/sandbox/snakemake-module-issue/other_workflow.smk", line 7, in <module>

It seems like an issue with importing rules multiple times

Really bad workaround for case 2. Buggy Snakefile
If other_workflow.smk is re-written as
rule some_task:
    output: "output.txt"
    shell: "cat hello >> {output}"

use rule other_some_task as another_task with:
    output: "another.txt"

the 2. Buggy snakefile case works where it shouldn't have!
❯ snakemake -l
other_some_task
another_task

KeyError in case 2 fixed in https://github.com/snakemake/snakemake/pull/1292
CreateRuleException still persists, and cannot modify rules after inheriting.


Recursive descent

A draft implementation of a recursive descent parser.
Currently missing:

[x] not feature complete (many non-terminals are unimplemented)
[x] positions are missing
[ ] error reporting is really bad

Syntax change: Unicode literals now have the syntax \u<HEXADECIMAL>+ instead of \u{<HEXADECIMAL>+}

Tests are failing, because escaping unescaped characters in strings is still missing in the various different backends.

The current error is that we produce:
val NEWLINE_CHAR = #"
";

from:
extern ml """
val NEWLINE_CHAR = #"\n";
"""

which seems like a good test case :)

By the way, there's a leftover debug print somewhere which outputs:
ModuleDecl(trailing_comment,List(),List(FunDef(IdDef(main),List(),List(),List(),None,Return(Literal((),ValueTypeApp(Unit,List()))))))

(can't really find where it's from, but it's in the logs: 59)

I am merging this PR now. We should not forget about the TODOs that are left. It would also be great to decide whether we want to share the escaping in the various backends (and maybe move it closer to the unescaping?).

Good job @dvdvgt !


Drone Helm chart left abandoned.

there has been a few issues with drone for the past year when deploying with a helm chart. if you have an existing helm deployment and want to update it/redeploy it for some reason, either a configuration change or some other reason, the helm chart will never redeploy successfully. the pod will hang forever as it dosent know how to handle the service existing before the pod. this causes errors and the pod will forever loop, never deploying. this has been an issue for over a year, and due to the supposed razor focus on developing gitness, the drone chart has been left to rot.

the error in the drone pod is as follows:
time="2024-09-19T18:53:15Z" level=fatal msg="main: invalid configuration" error="Invalid port configuration. See https://discourse.drone.io/t/drone-server-changing-ports-protocol/4144"
the only current solution to this is to DELETE the helm created service, then recreate it after the pod has launched, but before it realises it isnt exposed. rediculous


Problem With passive="true": string not assignable to boolean

@perfectmemory/ngx-contextmenu: ^15.1.1
typescript: ~4.9.4
I'm using Angular 15 and was just starting to make a menu following the examples:
<context-menu #nodeContextMenu>
    <ng-template contextMenuItem passive="true">
        Node Actions
    </ng-template>
    <ng-template contextMenuItem (execute)="false;">nothing</ng-template>
</context-menu>

This breaks the Angular build with the error:
error TS2322: Type 'string' is not assignable to type 'boolean'.
<ng-template contextMenuItem passive="true">
                             ~~~~~~~

I tried passive=true (same error), passive="1" (same error), [passive]="true" (no error but doesn't seem to work, either).

Hi @bcwhite-code,
As you can see here, it is a really standard input, nothing fancy, so maybe there is a problem in the setup of your project. Could you provide a stackblitz with a minimum viable reproduction of your problem please ?

Hmmm...  Okay.  Thanks for the info.  In the end, it didn't provide the interface I wanted (which was do a select while holding right-button down) so I ended up writing my own.


adding model files for platforms notification events

Summary

code after discussion here: https://github.com/Adyen/adyen-go-api-library/issues/90
working in collaboration with @KadoBOT to allow support for platforms notification events
ran command node index.js generate -l "go" -s "platforms notifications" -v6 in adyen-openapi-generator to generate the model files

Tested scenarios

Fixed issue:

Thanks @namanhkapur
I've converted it to a draft, while we still don't have the helpers to serialize the JSON.

I'm thinking of adding the README so we know where the models were pulled from. Should I add anything else (i.e. docs folder or api folder)?

I'm thinking of adding the README so we know where the models were pulled from. Should I add anything else (i.e. docs folder or api folder)?

We normally have a Readme inside the src folder https://github.com/Adyen/adyen-go-api-library/blob/develop/src/binlookup/README.md

LGTM. I'll leave it up to @KadoBOT

any idea why tests are failing? and any way to rebuild and verify manually?

@namanhkapur they'll fail on external PR's because we use secret variables, to test it manually you'll need a .env file in the root with test credentials. I'll test it locally and merge.


Support request body overrides in C#

Current coverage is 84.92% (diff: 100%)

Merging #954 into master will increase coverage by 0.01%

@@             master       #954   diff @@
==========================================
  Files           337        337
  Lines         13091      13104    +13
  Methods           0          0
  Messages          0          0
  Branches       1713       1715     +2
==========================================
+ Hits          11116      11129    +13
  Misses         1541       1541
  Partials        434        434

Powered by Codecov. Last update 66d2df4...9acced2

Ready now, not too much change from your last review, just shifting some bits out of TypeNameGenerator into the CSharpSampleTypeNameConverter and getting compilecheck working.

Any further thoughts?


🐛  maxImages (3) has already been acquired, call #close before acquiring more

What's happening?
application is crashing randomly in production with below error
Fatal Exception: java.lang.IllegalStateException: maxImages (3) has already been acquired, call #close before acquiring more.
       at android.media.ImageReader.acquireNextImage(ImageReader.java:527)
       at android.media.ImageReader.acquireLatestImage(ImageReader.java:411)
       at com.mrousavy.camera.core.CameraSession.configureOutputs$lambda$1(CameraSession.kt:299)
       at android.media.ImageReader$ListenerHandler.handleMessage(ImageReader.java:819)
       at android.os.Handler.dispatchMessage(Handler.java:106)
       at android.os.Looper.loop(Looper.java:264)
       at android.os.HandlerThread.run(HandlerThread.java:67)

Reproduceable Code
const device = useCameraDevice('back');

  const isFocused = useIsFocused();

<Camera
          style={StyleSheet.absoluteFill}
          device={device}
          isActive={isFocused}
          ref={scanRef}
          photo={true}
          orientation="portrait"
          onError={onCameraError}
        />

Relevant log output
not available

Camera Device
not available

Device
OPPO CPH2239
VisionCamera Version
3.6.15
Can you reproduce this issue in the VisionCamera Example app?
I didn't try (⚠ your issue might get ignored & closed if you don't try this)
Additional information

[ ] I am using Expo
[ ] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

No output logs available, no camera device available, and no reproduceable sample available - sorry but I cannot help you.

@mrousavy https://github.com/mrousavy/react-native-vision-camera/issues/1387 this seems to be same issue. And here we also have logcat output. Does this help?


asciidoctor-kroki  silently skip some plantuml diagrams rendering

asciidoctor-kroki  v 0.17, asciidoctor.js 3.0.2.
Building single HTML file using node.js.
Some PlantUML diagrams does not rendered without any visible error from asciidoctor-kroki or Node.JS.
A the result, I've got a package of HTML documents from build procedure, where some documents has broken diagrams.
The asciidoctor-kroki options are:
const html = asciidoctor.convertFile(source, {
  to_file: false,
  standalone: true,
  safe: 'safe',
  extension_registry: registry,
  attributes: {
    'kroki-server-url': krokiServer,
    'kroki-fetch-diagram': true,
    'kroki-default-options': 'inline',
    'imagesoutdir': dstDir + 'images',
    'imagesdir': 'build/images',
    'allow-uri-read': 'true',
    'data-uri': 'true',
    'icons': 'font',
    'webfonts!': '',
    'stylesheet': 'build/images/' + cssDestinationFilename,
    'kroki-plantuml-include': path.resolve(__dirname, 'plantuml-styles.iuml'),
    'uri-project-root': projectRootUri,
    'uri-repository-root': repositoryRootUri
  }
})

The fragment of resulting HTML document source is:

plantuml::sdk-hierarchy-v2-yooid.puml[svg]

See here, the asciidoctor left original plantuml block source instead of HTML block.
The document contains about 15 diagrams, most of them build successfully, but some has been skipped like this.
I've thorougly inspected all the diagrams in the document and I found PlantUML internal error (java.lang.IllegalArgument wzception) in some of them after PlantUML upgrade to version 1.2023.9.
I know how fix these diagrams, but there is a problem with asciidoctor-kroki:  the asciidoctor-kroki implementation suppress errors from PlantUML, shows successful build with broken documents.
There is a problem for large documentation project with hundreds of documents.
I'm not JavaScript developer, but I can try to help you with debug and fix that if you tell me some suggestions where to find the cause in asciidoctor-kroki source.

Could you please provide a source file that reproduces this issue?

Here is a fragment of asciidoctor document with problem diagram:

Предназначен для сервисов, которым требуется:

*  Возможность получить авторизацию YooId для последующей работы через один или несколько сервисов API

plantuml::sdk-hierarchy-v2-yooid.puml[svg]

=== Привязка Кошелька

Here is a source plantuml diagram file:
@startuml
title Базовый модуль SDK: YooId
!include <archimate/Archimate>
!include lib/plantuml-styles.iuml

Grouping(SDK, "SDK") {
    !include lib/sdk-component-yooid.iuml
    !include lib/sdk-component-tmx.iuml
}

Application_Component(App, "Приложение ЛК YooKassa")
Rel_Aggregation(App, SdkTmx, "содержит")
Rel_Aggregation(App, SdkYooId, "содержит")

Application_Function(FuncApp, "Личный кабинет YooKassa")
Rel_Assignment_Up(App, FuncApp, "реализует")

Application_DataObject(UserAuthorization, "Авторизация\nпользователя YooId\n(авторизация СЦА)")
Rel_Association_Left(App, UserAuthorization, "хранит\nна устройстве")

!include lib/sdk-component-clientid.iuml
@enduml

The problem root is
Grouping(SDK, "SDK") {
}

The plantuml 2023.9.1 crashes with java.lang.IllegalArgumentError and asciidoctor-kroki ignores this showing successful build with

plantuml::sdk-hierarchy-v2-yooid.puml[svg]

in resulting HTML document.

Thanks!
Since you are using a relative paths with the !include directive, could you please provide the directory structure? Do you use !include inside lib/plantuml-styles.iuml, lib/sdk-component-yooid.iuml, lib/sdk-component-tmx.iuml or lib/sdk-component-clientid.iuml?

<Project_Root>/
build.js
sdk/
sdk-hierarhy-usecases.adoc with plantuml::sdk-hierarchy-v2-yooid.puml[svg]
sdk-hierarchy-v2-yooid.puml
...and other adoc and puml files
lib/
sdk-component-yooid.iuml and other iuml files
But dir structure doesn't matter, I think. The error triggering instruction is Grouping(SDK, "SDK") { } and you can comment out all includes — the error still reproducible.

OK, so the following should be enough:
@startuml
Grouping(SDK, "SDK") { }
@enduml

I get the following error:

GET https://kroki.io/plantuml/svg/eNpzL8ovLcjMS9cIdvHWUVACkkqaCtUKtVwAaXkHMw== - server returns an empty response or a 404 status code - plantuml::/path/to/diag.puml[]

Thanks. Could you suggest how I can enable verbose (debug) logging/tracing the kroki requests/responses within asciidoctor-kroki.js? I'am not JavaScript developer, but I think, I can make some code navigation and updates in JS code.

Might be related: https://github.com/asciidoctor/asciidoctor/commit/28c69077665c655b37281029feb1876d98527093

That's correct. This should have been addressed in 2.0.18. However, there's no guarantee it covers every use case. A registry is not really designed to be reused. Ideally, it should be created per conversion.


Firefly reports error during funds migration (Ledger)

Issue description
Firefly reports error during funds migration with Ledger wallet. This is a new migration with no previously created account in Firefly.
Wallet version
1.2.0
System specification
MacOS 11.4
Is it a repeatable issue?
Y
Steps To reproduce the issue
All steps followed until being asked to migrate tokens
Clicked on Migrate tokens
Expected behaviour
Would have expected all transactions  to have been migrated
Actual behaviour
Error being displayed (see below)
A single transaction shows as been migrated when error pop-up displays
Errors
App Version: 1.2.0
platform: macOS
platformVersion: 11
platformArchitecture: x64
cpuCount: 8
totalMem: 16384.0 MB
freeMem: 2211.2 MB
userPath: /Users//Library/Application Support/Firefly
Error Type: Preload Context Error
Error: Uncaught TypeError: Cannot read property 'forEach' of null
at /Applications/Firefly.app/Contents/Resources/app.asar/public/build/preload.js:2:548089
at /Applications/Firefly.app/Contents/Resources/app.asar/public/build/preload.js:2:590377
at Array.forEach ()
at destroy (/Applications/Firefly.app/Contents/Resources/app.asar/public/build/preload.js:2:590365)
at /Applications/Firefly.app/Contents/Resources/app.asar/public/build/preload.js:2:582435

After repeating the same steps several times, it appears that a single transaction gets transferred every single time. The transferred address in the Chrysalis mainnet displays the correct balance (checked in mainnet explorer). Manually creating the account in firefly allows the address (and correct balance) to be displayed (in Firefly). Looking at the Firefly original error, it simply looks like Firefly has an issue iterating an array of transactions (Error: Uncaught TypeError: Cannot read property 'forEach' of null). Or the array is not correctly constructed. Hope that helps a little....

Another finding that may help....
After manually creating the account in Firefly (previous comment), I repeated the original migration using the option to migrate a different index (even if I had only a single index). The original error (Error: Uncaught TypeError: Cannot read property 'forEach' of null) did not occur and transactions are being migrated. So it looks like the array object is dependent on the account creation...

Hello, I have the same issue with firefly version 1.3.0, did you find a solution?

@dsx5000 Do you still have this issue?

Hello,
it was possible to solve by installing firefly version 1.2.0.
Thank you!

Am 06.01.2022 um 00:48 schrieb MiKeF05 @.***>:
@dsx5000 https://github.com/dsx5000 Do you still have this issue?
—
Reply to this email directly, view it on GitHub https://github.com/iotaledger/firefly/issues/1408#issuecomment-1006165739, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXBQGRLNEJ62NLUK6VFXO63UUTKDLANCNFSM5CGIN4PQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.


Kill/detach issue

re #71063

run cluster debugging as described here https://github.com/Microsoft/vscode/issues/71063#issuecomment-476669886
child process 3 is selected, detach action shows
select cluster, kill action shows
select child process 3 again
:bug: kill action still shows

Other weird behaviors to note:

Before you hit the breakpoint, selecting any child processes selects the parent (like #71205)
After you hit the breakpoint, you can select other child processes and detach from them as expected
After detaching from a child process, this reverts back to this original issue

this is a duplicate of #71183 (which is already targeted for the release).


new api to get cli version

Description
Gets the latest release of the Dockstore CLI from the CLI repo (dockstore/dockstore-cli). The UI will then use that as the location of the dockstore script rather than the Dockstore webservice repo (dockstore/dockstore).
This is the UI PR, the webservice PR which adds the API to get the latest CLI release is
https://github.com/dockstore/dockstore-ui2/pull/1556
Issue
Addresses SEAB-3952
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future

If we anticipate ever returning additional information about the CLI, we might create an object CliInfo (or whatever) and serialize that instead.  The version (and other information we decide to include) can be its properties.  If we did this, we'd want to tweak the endpoint naming, of course...


[CELEBORN-1356] Split rpc module into rpc_app and rpc_service

What changes were proposed in this pull request?
Split the rpc transport module into rpc_app and rpc_service to allow for them to be independently configured.
Why are the changes needed?
We need the ability to independently configure communication between application components (driver/executors in spark applications) and those to/from Celeborn service (master/workers) components.
This is particularly relevant for TLS support where applications might be running with TLS disabled for their rpc services or using self-signed certificates (see CELEBORN-1354 for an example), while services would have signed certs.
Does this PR introduce any user-facing change?
Yes, it allows users to independently configure rpc env within the application and those to/from services.
Backward compatibility is maintained - and so existing rpc is the fallback in case rpc_app or rpc_service config is not found.
How was this patch tested?
Unit tests were enhanced, existing tests pass.

If it makes it easier to review, I can split it into two - updates to CelebornConf, which is mostly config updates; and wiring up the new transport module to RpcEnv and usages.
Please let me know if that helps.

The test failure seems to be some timeout issue ? this is ready for review.
+CC @otterc , @waitinfuture, @FMX , @pan3793

code change lgtm, could you pls add an item to the migration guide?

code change lgtm, could you pls add an item to the migration guide?

I was planning to add that as part of https://issues.apache.org/jira/browse/CELEBORN-1353 .. should I be adding it as part of this PR instead ?

@mridulm thanks for opening a dedicated ticket for docs, merged to main for 0.5

Thanks for the reviews @FMX and @pan3793 !


Hacked PS2 running as a controller over the network

Is your feature request related to a problem? Please describe.
I have a hacked ps2 with freemcboot.  I'm interested in writing some homebrew to use the ps2 network and ps2 controller to use it as a game pad.
Describe the solution you'd like
Before I build the developer tools I want to know if this is even possible or the write tool to build such a project.
Describe alternatives you've considered
I couldn't find anything that supported a socket connection as a game pad.
I could look at more source code for android wifi controllers.  Those are the only thing I know that work.

Please don't e-mail me and create an issue, I get notifications about GH issues and will respond when there's time 🙄
As per my mail response; this is not really a feature request for ViGEm but an entire project. Check the Contributing section in the README, there's links to likeminded folks, cheers.


Possible bug - createWatchProgram optionsToExtend not applied correctly

Bug Report
This is a possible bug in the usage of optionsToExtend in ts.createWatchProgram. It's not clear to me exactly how this is supposed to work, so this could be a caller error or it could be an implementation error.
The module typescript-estree calls into createWatchProgram with a host object that contains some defaults for the options:
optionsToExtend: {
    allowJs:true,
    allowNonTsExtensions:true,
    checkJs:true,
    noEmit:true,
    noUnusedLocals:true,
    noUnusedParameters:true
}

my project config file contains some options
{
    noUnusedParameters:false
}

The config that the program actually uses is the same as the original
{
    allowJs:true,
    allowNonTsExtensions:true,
    checkJs:true,
    noEmit:true,
    noUnusedLocals:true,
    noUnusedParameters:true
}

I think the intention from the caller is that these values would be the default values if the project config file does not contain alternatives, however when the config is parsed, these options actually overwrite the parsed config.
So what is the behaviour of optionsToExtend actually supposed to be? Are these override options or are these default options intended to be overwritten?
If these are to be overwritten by the project config file, the call into extend (in multiple places) is in the wrong order. ts.extend(a, b) applies a onto b, as opposed to b onto a like some other common extend apis, so this could be the reason.

The lines of code in question: L39954
lib/typescript.js-39951-        var errors = [];
lib/typescript.js-39952-        var parsedConfig = parseConfig(json, sourceFile, host, basePath, configFileName, resolutionStack, errors, extendedConfigCache);
lib/typescript.js-39953-        var raw = parsedConfig.raw;
lib/typescript.js:39954:        var options = ts.extend(existingOptions, parsedConfig.options || {});
lib/typescript.js-39955-        var watchOptions = existingWatchOptions && parsedConfig.watchOptions ?
lib/typescript.js-39956-            ts.extend(existingWatchOptions, parsedConfig.watchOptions) :
lib/typescript.js-39957-            parsedConfig.watchOptions || existingWatchOptions;

The caller creates a host by calling with what it thinks should be default options:
https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-estree/src/create-program/createWatchProgram.ts#L264

Turns out optionsToExtend does indeed mean overrides as far as I can understand it. Closing this out.


[FR] - Misleading unsafeHashableScriptData

External
Area
Other  HashableScriptData
Describe the feature you'd like
We have been successfully using unsafeHashableScriptData however its description recommends NOT to use it:
Warning: Creating HashableScriptData from a ScriptData value pretty much guarantees the original bytes used to create the value will be different if we serialize again. Do not use this.

However it is unclear how to construct a ScriptRedeemer from ScriptData if we shouldn't use  unsafeHashableScriptData - in general, if usage of unsafeHashableScriptData is discouraged, it is unclear how to go from ScriptData to HashableScriptData.
Please either change the description of unsafeHashableScriptData or provide other (safe?) means going from ScriptData to HashableScriptData.

You can use fromAlonzoData in this way:
import qualified Cardano.Ledger.Plutus.Data as LPD
import qualified PlutusLedgerApi.V1                as PV1
import qualified Cardano.Api.Shelley               as C

toHashableScriptData = C.fromAlonzoData . Data @ledgerEra . PV1.toData

But I don't think it matters if you're creating the data from scratch (programatically) rather than by deserialisation from a bytestring.


No repeat_kv in llama_flash_attn_monkey_patch.py ?

In llama 2's code, the code use repeat_kv before attention to repeat k/v heads(GQA).

But I didn't find any repeat_kv in the llama_flash_attn_monkey_patch.py.
I think this should be a mistake🤔If llama 2 is evaluated without repeat_kv, I think the result would also be incorrect.
Is my understanding correct?

Thanks for your attention! We don't think this would influence the result. Feel free to compare the generation results of Llama 2 w/ and w/o the llama_flash_attn_monkey_patch.py and check for yourself~

Thank you. 😃After checking the original configuration of LLaMA-2, I found
self.num_key_value_groups = self.num_heads // self.num_key_value_heads

it mean by default self.num_key_value_groups=1, so it doesn't matter whether adding it or not.
But just to be safe, I think it will be more flexible if we add this, because maybe some people will adjust the num_key_value_heads when using GQA

Thanks for pointing it out! The current llama_flash_attn_monkey_patch does not support GQA, feel free to submit a merge request to add this feature.


makedpx - add ffmpeg logs

This relies on https://github.com/kieranjol/IFIscripts/issues/76 being fixed first

fixed https://github.com/kieranjol/IFIscripts/commit/48a8cc40b430b3d5e5576a83e94947bf57d2b0ad and https://github.com/kieranjol/IFIscripts/commit/00df1750ecb5dfc1647f4c288ffb6ab117ec8633


Remove dependency on Nuget

This project has a dependency on Nuget which should be possible to be removed by adding the required code to this repository or creating it's own repo and adding that as a submodule. Note I am speaking here as a C & C++ developer, I am not sure what the consensus in other communities is.
As it stands, the dependencies are not clear, and turning this into a working lightweight solution is much harder.  Take a look at the first response here by David Moore for the sort of thing developers are looking to use instead of the code here.
I would expect all required dll's for querying the install to be installed along with the install of VS2017. I also would expect any code needed to query this would be extremely simple - all complexity of actual implementation should be handled in the library code installed alongside VS. Sadly it looks like this is now too late, but at the very least an example with minimal dependencies should be produced.

I have whittled down this example to something that uses only built-in windows tools.
At worse you can copy the COM interface (which is what the nuget package provides).
Also check out @heaths 's https://github.com/Microsoft/vswhere

Also be sure to upvote https://developercommunity.visualstudio.com/content/problem/35325/missing-vs150comntools.html

Thanks @refack that's helpful. In the end I am able to use the key in:
SOFTWARE\Microsoft\VisualStudio\SxS\VS7\15.0\
as this has the root folder which I can then find VC\Auxiliary\Build\vcvarsall.bat from which is what I need. However a simple official example would still be useful.

@dougbinks MS don't garantee that key will be there (even within the VS2017 lifecycle). Follow the conversation at https://codereview.chromium.org/2754263002/
especially the caveat mentioned in http://code.qt.io/cgit/qt-labs/qbs.git/commit/?id=72463bbd87044148fcafcdc2c1189aa7a5593cf9

I'm going to stick with the registry setting until such a time as they disappear it. Hopefully by then we'll have a decent environment variable, code sample, or registry key to use.

Make sure to look for the Buildtools SKU, using it is the strongest indicator to MS that that's the kind of product we want
From the QT installer
// Detect Visual C++ Build Tools
   QSettings vcbtRegistry(
               QStringLiteral("HKEY_LOCAL_MACHINE\\SOFTWARE") + wow6432Key()
               + QStringLiteral("\\Microsoft\\VisualCppBuildTools"),
               QSettings::NativeFormat);
   const QStringList &vcbtRegistryChildGroups = vcbtRegistry.childGroups();
   for (const QString &childGroup : vcbtRegistryChildGroups) {
       vcbtRegistry.beginGroup(childGroup);
       bool ok;
       int installed = vcbtRegistry.value(QStringLiteral("Installed")).toInt(&ok);
       if (ok && installed) {
           MSVCRegistryEntry entry;
           entry.version = childGroup;
           const QSettings vsRegistry(
                       QStringLiteral("HKEY_LOCAL_MACHINE\\SOFTWARE") + wow6432Key()
                       + QStringLiteral("\\Microsoft\\VisualStudio\\") + childGroup
                       + QStringLiteral("\\Setup\\VC"),
                       QSettings::NativeFormat);
           entry.installDir = vsRegistry.value(QStringLiteral("ProductDir")).toString();
           result.append(entry);
       }
       vcbtRegistry.endGroup();
   }

2017 Build tools https://www.visualstudio.com/thank-you-downloading-visual-studio/?sku=BuildTools&rel=15#
2015 Build tools http://landinghub.visualstudio.com/visual-cpp-build-tools
HOWTO command line install https://blogs.msdn.microsoft.com/vcblog/2016/11/16/introducing-the-visual-studio-build-tools/

With multi-instancing and much more granular feature selections, an environment variable does not provide you with any guarantees of what is installed which is why tools like vswhere allow you to specify which features you actually require. We will be including vswhere in the install in a well-known and relatively fixed location (using a common environment variable defined by Windows would be required to find the root, but that would be all).
Indeed, the registry key is not supported and probably will be removed at some point (same problems as with an environment variable) at which point vswhere can be used.
As for the original issue, package managers such as nuget (which work for VC++, C#, VB.NET projects and more), NPM, bower, pip, and more are common these days. The VC++ team has also been building vcpkg as a more capable alternative to nuget for including heads, libs, and compiler settings into native projects - even for cross-platform code. We will not be duplicating the headers and will use the nuget package as a way to componentize and better support development of this and other modules. That's what package managers are for: componentization of discrete portions of software. Package managers such as NPM even support version comparisons of semantic versions to say you will support any major or minor releases or to fix on a particular version (something nuget lacks, unfortunately).
The DLLs to query VS are included in the install. This is sample code how to call into it. You can use vswhere - either by pulling it down as a package or checking it in - or wait until we include it in the install but you'll still need to call into it.

With multi-instancing and much more granular feature selections, an environment variable does not provide you with any guarantees of what is installed which is why tools like vswhere allow you to specify which features you actually require. We will be including vswhere in the install in a well-known and relatively fixed location (using a common environment variable defined by Windows would be required to find the root, but that would be all).

Thank you from a huge community! (although adding an option in the vs_installer to select "default" instance would be just as great)

Including vswhere as a DLL in a well-kown fixed location using a common environment variable defined by windows would be a great step forwards.
The programming community I am from (C and C++ games programmers) does not look look kindly on package managers, hence the original request.

According to this blog: https://blogs.msdn.microsoft.com/vcblog/2017/03/06/finding-the-visual-c-compiler-tools-in-visual-studio-2017/
a simpler solution exists without dependency on nuget here:
https://github.com/mluparu/vs-setup-samples/tree/master/Setup.Configuration.VC.NoDeps

@dougbinks if you want to duplicate and maintain duplicate code feel free. Package managers exist to avoid such a coding practice. It's not like using nuget means you always get the latest. Nuget, unlike NPM, does not support a syntax to grab a newer version than what you specify in a packages.config file. Your call, but the sample will continue to use the recommendation of a nuget package to avoid duplication.
Starting with 15.2 Preview 2 vswhere is also now in the box: https://blogs.msdn.microsoft.com/heaths/2017/04/21/vswhere-is-now-installed-with-visual-studio-2017/


How I can change the number of homes ?

Hi ! I just want to know how I can have more sethomes to have many homes

Hi ! I just want to know how I can have more sethomes to have many


Eliminate truth-matching hacks

As of PicoRun4, vertexIDs are no longer rewritten to be consecutive, and the 2x2-MLReco pipeline is fully propagating vertexIDs through it, so we no longer need the workarounds that we previously needed.  Remove them here.
This comes with some new code to match run and vertexID numbers and updates to the storage classes that hold onto the GENIE tree(s) once loaded.
I also snuck in a few small updates (bypassing the GENIE splash screen, fixing a 'deprecated method' warning from FHICL).
This PR depends on some others that are used in the input file preparation:
https://github.com/DeepLearnPhysics/larcv2/pull/43
https://github.com/DeepLearnPhysics/SuperaAtomic/pull/17
https://github.com/DeepLearnPhysics/larnd2supera/pull/3

Verified that this actually works on PicoRun4.1 files.  The larnd2supera PR listed above hasn't yet been merged, but we're going to merge to flow2supera very soon anyways, and the same commits will need to be ported there instead...


Sourcelink and include PDBs in nuget packages

This commit reinstates sourcelinking of PDB files and including them in nuget packages.
Closes #4095

ported to master, 7.4, 6.x


fix: accordion unique key prop issue fix

Fixes
Fixes #100

unique key prop issue in Accordion list has been fixed, by adding a accordionId in accordionListData and wrapping the Accordion component in src/components/AccrdionList.jsx/AccordionList in React.Fragment and assiging the accordionId to it
Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the Contribution Guidelines and my PR follows them
[x] I have titled the PR appropriately
[x] I have updated my branch with the main branch
[x] I have added tests and/or screenshots that prove my fix is effective or that my feature works
[ ] I have added the necessary documentation about the functionality in the appropriate .md file
[ ] I have added inline documentation to the code I modified

If you have questions, please create a GitHub Issue in this repository.

:ok_hand: Nice work!

👌 Nice work!

Thank You @aaronshivers 😊


Optimization in audio graph traversal

Some improvements could be made in graph.rs when rendering an audio quantum:

[x] Store edges more efficiently (currently a single HashMap that needs to be iterated many times)
[ ] Use a specialized container type for the nodes. e.g. https://crates.io/crates/intmap
[x] Avoid the remove and insert calls of the currently processing node. That was necessary for borrow reasons. Wrap nodes in Cell or equivalent
[x] Clear the input buffers when processing of a Node is done (we don't need them anymore and this way they can be reused)
When a Node has only one outgoing connection in the graph, its outputs can be moved instead of copied to that Node's inputs this is not useful because the inputs are immutable anyway and need to be copied/mutated to outputs
[ ] Decouple the graph-topology code from the audio-specific code

I have a branch ready for item 1 at feature/intmap-for-graph
But I will have a look at the other items first before deciding if this is worth the hassle of an exotic dependency

I took a stab at Avoid the remove and insert calls of the currently processing node. @b-ma but it's very tricky. We will need a lot of unsafe code to work around it. Not sure if that would be beneficial for the project. I tried a safe, intermediate solution but it had no benefits: 084d8187f7307ab4ca
I will have another look at Use a specialized container type for the nodes now

I tried a safe, intermediate solution but it had no benefits: https://github.com/orottier/web-audio-api-rs/commit/084d8187f7307ab4ca96d5ca95749454aa380eb2

Just tested, no sign of improvement neither sorry...
I just wonder if we could not also try to bypass the HashMap altogether to use some kind of Vec<Option<Node>> for nodes and delegating to something like https://docs.rs/index-pool/latest/index_pool/ to manage the indexes. Then in the graph parsing we could just add/remove the nodes like that let node = self.nodes.swap(index, None), or is it silly?

I just re-tested using RefCell<Node> in the HashMap and managed to have it working this time (slowly understanding some stuff :). It's there https://github.com/orottier/web-audio-api-rs/compare/main...b-ma:web-audio-api-rs:test/graph-render and the perf improvements are quite good (better than chrome on several test cases :)
before
+ id      | name                                                                     | duration (ms) | Speedup vs. realtime  | buffer.duration (s)
- 1       | Baseline (silence)                                                       | 26            | 4615.4x               | 120
- 2       | Simple source test without resampling (Mono)                             | 41            | 2926.8x               | 120
- 3       | Simple source test without resampling (Stereo)                           | 55            | 2181.8x               | 120
- 4       | Simple source test without resampling (Stereo and positionnal)           | 173           | 693.6x                | 120
- 5       | Simple source test with resampling (Mono)                                | 82            | 1463.4x               | 120
- 6       | Simple source test with resampling (Stereo)                              | 116           | 1034.5x               | 120
- 7       | Simple source test with resampling (Stereo and positionnal)              | 232           | 517.2x                | 120
- 8       | Upmix without resampling (Mono -> Stereo)                                | 46            | 2608.7x               | 120
- 9       | Downmix without resampling (Stereo -> Mono)                              | 44            | 2727.3x               | 120
- 10      | Simple mixing (100x same buffer) - be careful w/ volume here!            | 1755          | 17.1x                 | 30
- 11      | Simple mixing (100 different buffers) - be careful w/ volume here!       | 1733          | 17.3x                 | 30
- 12      | Simple mixing with gains                                                 | 340           | 352.9x                | 120
- 13      | Granular synthesis                                                       | 2662          | 2.8x                  | 7.5
- 14      | Synth (Sawtooth with Envelope)                                           | 3442          | 34.9x                 | 120
- 15      | Synth (Sawtooth with gain - no automation)                               | 2778          | 43.2x                 | 120
- 16      | Synth (Sawtooth without gain)                                            | 1681          | 71.4x                 | 120
- 17      | Substractive Synth                                                       | 423           | 283.7x                | 120
- 18      | Stereo panning                                                           | 82            | 1463.4x               | 120
- 19      | Stereo panning with automation                                           | 82            | 1463.4x               | 120
- 20      | Sawtooth with automation                                                 | 75            | 1600.0x               | 120
- 21      | Stereo source with delay                                                 | 210           | 571.4x                | 120

after
+ id      | name                                                                     | duration (ms) | Speedup vs. realtime  | buffer.duration (s)
- 1       | Baseline (silence)                                                       | 21            | 5714.3x               | 120
- 2       | Simple source test without resampling (Mono)                             | 30            | 4000.0x               | 120
- 3       | Simple source test without resampling (Stereo)                           | 44            | 2727.3x               | 120
- 4       | Simple source test without resampling (Stereo and positionnal)           | 158           | 759.5x                | 120
- 5       | Simple source test with resampling (Mono)                                | 75            | 1600.0x               | 120
- 6       | Simple source test with resampling (Stereo)                              | 106           | 1132.1x               | 120
- 7       | Simple source test with resampling (Stereo and positionnal)              | 209           | 574.2x                | 120
- 8       | Upmix without resampling (Mono -> Stereo)                                | 39            | 3076.9x               | 120
- 9       | Downmix without resampling (Stereo -> Mono)                              | 35            | 3428.6x               | 120
- 10      | Simple mixing (100x same buffer) - be careful w/ volume here!            | 1599          | 18.8x                 | 30
- 11      | Simple mixing (100 different buffers) - be careful w/ volume here!       | 1604          | 18.7x                 | 30
- 12      | Simple mixing with gains                                                 | 300           | 400.0x                | 120
- 13      | Granular synthesis                                                       | 2347          | 3.2x                  | 7.5
- 14      | Synth (Sawtooth with Envelope)                                           | 2899          | 41.4x                 | 120
- 15      | Synth (Sawtooth with gain - no automation)                               | 2212          | 54.2x                 | 120
- 16      | Synth (Sawtooth without gain)                                            | 1332          | 90.1x                 | 120
- 17      | Substractive Synth                                                       | 414           | 289.9x                | 120
- 18      | Stereo panning                                                           | 71            | 1690.1x               | 120
- 19      | Stereo panning with automation                                           | 73            | 1643.8x               | 120
- 20      | Sawtooth with automation                                                 | 62            | 1935.5x               | 120
- 21      | Stereo source with delay                                                 | 201           | 597.0x                | 120

The downside is that I didn't manage to get rid of unsafe code in 2 places. It very localized and seems to be the same problem each time (i.e. returning a reference to the buffer in Graph::render() and AudioParamValues::get()) so maybe you would have an idea to handle that?

Amazing, I did not realize there was this much to gain still from the Graph::insert/remove stuff.
Let's continue discussing at https://github.com/orottier/web-audio-api-rs/pull/199

I'm closing this issue because I think the leftover point are no longer really interesting, given the current implementation


Added Configuration Provider

I had to define the same currency symbol for each input field. This is why I created a settings provider to provide the ability to define the currency symbol (and every other property) in one place.

Hi @ChristianKohler ! Great job! The code seems fine to me, but I would like @nick-woodward check it too.

I'll try to look at it when I get off work tonight.

@nick-woodward Great review. Thanks. Should I update the reviewed changes?

@ChristianKohler Yeah, update with the changes I've listed and I'll look it over again.


ADD: An extra Vault share without pubkeys

Please implement a "Family-Share" where the public keys are removed in the vault share, so a user can share and backup his vault with "family and friends" without them seeing the balances of the user.
After a user has successfully backed up or shared their vault, let them automatically return to the main home view.

Maybe we can just https://github.com/vultisig/vultisig-ios/issues/443 , when user backup a vault share with a password, then the content will be encrypted , then user can share it with their family and friends
If they don't want family and friends to see the balance , then don't share the password

@johnnyluo but technically not possbile, correct?

Will be included with the QR vault share on #849

I think we can close this one now @realpaaao agree?

close as completed


Fix network connection handling

We were not overriding the wrapBootstrapDisconnect function with that of appium-android-driver, so it would fail with no bootstrap.
Fixes https://github.com/appium/appium/issues/6658

👍


issues with PHP preloader

We're trying to preload our laravel application using Brents example but are hitting an issue with amphp
Fatal error: Cannot redeclare Amp\Process\escapeArguments() (previously declared in /app/vendor/amphp/process/lib/functions.php:30) in /app/vendor/amphp/process/lib/functions.php on line 30
Weird thing is that amphp isn't in the preload, but as a solution would it make sense to surround helper functions provided by amphp with ( ! function_exists('...')) like many other packages do?

I think fixing all libraries to include such a check is the wrong way of fixing it. If the preloading requires a file that's then again required by Composer, then that's where things go wrong. IMO support should be added in Composer to check whether "autoload": { "files": [...] } files are already preloaded and skip them in this case.

I'm not sure who is requesting it when tbh. The preloading works, as it just loads the framework classes. We use rubix/ml in our app which requires amphp. And hitting any route in our app, once everything is loaded, throws the fatal error, so the way down the rabbit hole is quite interesting.
Will give it a dive next week to see what methods are actually going into amphp in a preload manner, as it might be that many many more packages have the same outcome, or if it's only amphp which I doubt.

After some testing I can conclude that surrounding with ( ! function_exists('...')) doesn't have any effect, and not preloading anything still yields this error. Amphp is the only package in our +50 composer.json that gives error sadly. I've asked for Brent's input to see if he has any pointers as I believe that he's one of few with the experience that could point in the right direction.

@danijelk Try adding debug_print_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS) to the top of the conflicting file. That way you will see where it is loaded from and should be able to judge which of the two cases should not be happening. That should put you on the right track to fix the issue.

@enumag thanks for hint. I ended up figuring some details out, this project explains the issue quite well.

Using require_once will execute all files, resolving all the links (parent classes, traits, interfaces, etc.) before compiling it, and may output heavy errors on files that shouldn't be executed. Depending on your application, you may want to use one over the other.

Since the preload.php example uses require_once __DIR__ . '/vendor/autoload.php'; logically it will preload all helper functions, traits etc etc before it even scans our ->paths().
With this it makes sense that a second autoloader call will give us the error. Just for the record, even if we use require_once() it still fails, I guess it doesn't know it's preloaded and tries to require it again.
So with debug_backtrace() I could see what was calling the amphp ... functions.php
Preload
Array
(
    [0] => Array
        (
            [file] => /app/vendor/composer/autoload_real.php
            [line] => 69
            [function] => require
        )

    [1] => Array
        (
            [file] => /app/vendor/composer/autoload_real.php
            [line] => 59
            [function] => composerRequirebebea9ec969fefbfb154e44968b8fac9
            [args] => Array
                (
                    [0] => 8dc56fe697ca93c4b40d876df1c94584
                    [1] => /app/vendor/composer/../amphp/process/lib/functions.php
                )

        )

    [2] => Array
        (
            [file] => /app/vendor/autoload.php
            [line] => 7
            [function] => getLoader
            [class] => ComposerAutoloaderInitbebea9ec969fefbfb154e44968b8fac9
            [type] => ::
            [args] => Array
                (
                )

        )

    [3] => Array
        (
            [file] => /app/preload.php
            [line] => 3
            [args] => Array
                (
                    [0] => /app/vendor/autoload.php
                )

            [function] => require_once
        )

)

Regular request

Array
(
    [0] => Array
        (
            [file] => /app/vendor/composer/autoload_real.php
            [line] => 69
            [function] => require
        )

    [1] => Array
        (
            [file] => /app/vendor/composer/autoload_real.php
            [line] => 59
            [function] => composerRequirebebea9ec969fefbfb154e44968b8fac9
            [args] => Array
                (
                    [0] => 8dc56fe697ca93c4b40d876df1c94584
                    [1] => /app/vendor/composer/../amphp/process/lib/functions.php
                )

        )

    [2] => Array
        (
            [file] => /app/vendor/autoload.php
            [line] => 7
            [function] => getLoader
            [class] => ComposerAutoloaderInitbebea9ec969fefbfb154e44968b8fac9
            [type] => ::
            [args] => Array
                (
                )

        )

    [3] => Array
        (
            [file] => /app/public/index.php
            [line] => 24
            [args] => Array
                (
                    [0] => /app/vendor/autoload.php
                )

            [function] => require
        )

    [4] => Array
        (
            [file] => /Users/r/.composer/vendor/laravel/valet/server.php
            [line] => 191
            [args] => Array
                (
                    [0] => /app/public/index.php
                )

            [function] => require
        )

)

So with the help of Laraload's idea I simply removed the require_once of the autoloader, and switch from require_once to opcache_compile_file, and with come notices everything works and I see opcache stats have preloaded the things I needed.
Having that said, this is still a mystery to me, I can't figure out why amphp is the only package that has this issue. There are +50 helper functions that are preloaded the same way, but none of them crash like this.
Just as I was about to hit submit I tried some things, and found that if I remove the if(IS_WINDOWS) statement, everything works fine... I'll check with @nikic to see if this is a known issue or behaviour.

@danijelk It's indeed conditionally defined. Do you know whether there's a way to do if (is_preload()) { return; } there, so it's simply not preloaded?

@danijelk It's indeed conditionally defined. Do you know whether there's a way to do if (is_preload()) { return; } there, so it's simply not preloaded?

I see your idea, but I believe forcing this logic to every package is bad practise, just like you mentioned in your first comment. Either opcache preloader sees the condition and therefor doesn't preload it, which means amphp should move logic out of autoloaded helper, or this is a bug in the preload logic and can be fixed there.
Would be great with a composer->autoload->without_helpers :)

@danijelk Did you report this upstream somewhere?

@kelunik Didn't get a reply at that point when trying though github and twitter, will try to add to php issue list once back from vacation to push for an answer.

I can't really replicate it in an empty project so I'll close it for now, must be something in the laravel/composer combo that loads it one too many times.

just to add to the conversation, since I've been struggling with this for hours and this was the only thing issue was the only thing I could find on it. I'm also having this issue with symfony 5.1.
All autoloaded files that have functions declared conditionally will produce this issue.

Please open an upstream issue in symfony then.

it's not a symfony, laravel or composer issue, the problem is with this file: https://github.com/amphp/process/blob/master/lib/functions.php
When included it causes a duplicate definition in combination with opcache preload. Even though the docs state:

include will execute code in the file, while opcache_compile_file() will not. That means only the former supports conditional declaration (functions declared inside an if-block).

I'll have a look at reporting an issue with php itself and see where that leads.


Add storage mechanism

In the current implementation, the todos are stored in-memory. Add a storage solution like BadgerDB BoltDB or LevelDB, although the last one could be overkill.
Also look at Skate

#2 resolves the issue using BoltDB


UI appearance improvement

Color of output file link in Timeline Step view does not have sufficient contrast (dark mode -> see image).

Also "New Experiment" button on experiments-page is not properly aligned (but only when switching back to experiments-page from e.g. experiment/experiment-workspace...)


Add "create new" page

Page path is /
Form fileds:

Title
Text

Sign block with buttons for:

Mobile-ID
Smart-ID
ID-Card

If method is mobile-id, smart-id show form for phone and idc fields

Use Vue Composition API with

ei saa aru "Use primary button for primary action"

type="primary"

tel ja isikukood lahtrid keksivad


type Int128

for ipv6 and etc.

You could use FixedString(16) for ipv6 addresses (if there are no need for arithmetic operations).
Currently we don't have a plans to introduce Int128 type.

BTW, We have plans to introduce bitwise operations on FixedString data type to enable operations with IPv6 network masks.


Vertical scroll bar in preview component of UI is buggy

In the upcoming Hamilton 2.3 release the UI will have a preview componeent that let you see the generated code in real-time to the right of the UI window. A problem is that the vertical scrollbar gets stuck sometimes as if the WebView doesn't repaint the area where the scrollbar is located. This is no big issue, but it looks bad and makes it difficult to scroll vertically down long code files.

The preview feature has been postponed so this issue is not really nescessary right now.


QR code not scanning

I have set this up in a little app and while most QR codes seem to work no problems, this one in the image and a few others don't seem to work.
I get no errors, it's like it just doesn't recognise it as a QR code.
However, using an iPhone camera I get a URL to google drive from it for a restaurant menu, so it seems to be a valid QR code.
Do you have any ideas why it wouldn't work?

Not all grades of customization on the QR Codes are supported.


How to handle custom Response?

How to handle custom response? Such as when the data is wrapped like this example from Yii2 documentation site :
https://www.yiiframework.com/doc/guide/2.0/en/rest-response-formatting
{
    "items": [
        {
            "id": 1,
            ...
        },
        {
            "id": 2,
            ...
        },
        ...
    ],
    "_links": {
        "self": {
            "href": "http://localhost/users?page=1"
        },
        "next": {
            "href": "http://localhost/users?page=2"
        },
        "last": {
            "href": "http://localhost/users?page=50"
        }
    },
    "_meta": {
        "totalCount": 1000,
        "pageCount": 50,
        "currentPage": 1,
        "perPage": 20
    }
}

Hi @juliardi
How do you use the extension? Could you post some code snippet? What are you trying to achieve?

I use the extension like the snippet code in the README. The only thing different is my API server, which return a wrapped up data, just like from Yii2 documentation site I mentioned above. I assume that this library, by default, can only populate response data to a model if it is not wrapped. What I am asking is, how to populate a response data to my model, if it is wrapped ?
I am sorry if I am not clear enough.

I released beta 5 with $itemsProperty in Connection class. You can use like this
    'components' => [
        'rest' => [
            'class' => 'simialbi\yii2\rest\Connection',
            'baseUrl' => 'https://api.site.com/',
            'itemsProperty' => 'items'
        ]
    ]

Please test if it works for you. I would appreciate a feedback :blush:

I've tested it in my code, and it works in my case. Thanks for your fast response.
Just adding my opinion here, maybe it would be better if we can get the response object from the model. Something like this :
$item = MyItem::find()->where(['id' => 1])->one();

$response = $item->getHttpResponse(); // returns 'yii\httpclient\Response' instance here

Thx for your feedback and proposal. I tried to be as close to the default DB ActiveRecord handling as possible. But I move your proposal to Discussions...


macOS: Cmd+C does not copy

Description of the problem or steps to reproduce
Selecting text and using Cmd+C to copy does not do anything. Ctrl+C works correctly.
clipboard is set to "external"
Specifications
Version: 2.0.11
Commit hash: 225927b9
Compiled on July 24, 2022
OS: macOS 12.6
Terminal: Built-in Terminal.app as well as Iterm 2

Just a user, but I believe its because Cmd is not supported in many terminals. If your use iTerm, you can remap one or both of your Cmd keys to Ctrl under Preferences > Keys > Remap Modifiers.


Add whitelist for crate packaging.

Locks are included with crates? But looks good!

Don't know about lock files. cargo package includes everything by default. Changed to an explicit whitelist because we need so many exclusions for dot files and docs and IDE stuff.


Incorrect tag name in compare links at bottom

Hi and thanks for your Fastlane plugin. But I encountered some issue using it.
My command is:
stamp_changelog(
  section_identifier: '1.7.4',
  git_tag: '1.7.4'
)

Before execution:
## [Unreleased]
### Added
- Use Fastlane

## [1.7.3] - 2016-11-04
### Changed
- Set background as white

## [1.7.2] - 2016-11-04
### Fixed
- Fix chart layout

### Changed
- Update dependencies

[Unreleased]: https://github.com/GlobalSport/mycoach-player-android/compare/1.7.3...HEAD

After execution:
## [Unreleased]

## [1.7.4] - 2016-11-09
### Added
- Use Fastlane

## [1.7.3] - 2016-11-04
### Changed
- Set background as white

## [1.7.2] - 2016-11-04
### Fixed
- Fix chart layout

### Changed
- Update dependencies

[Unreleased]: https://github.com/GlobalSport/mycoach-player-android/compare/1.7.3...HEAD
[1.7.4]: https://github.com/GlobalSport/mycoach-player-android/compare/HEAD...1.7.4

Hey @mathroule, I am not sure what do you see as a problem?
stamp_changelog correctly stamped your unreleased section with the provided section identifier parameter (section_identifier: '1.7.4'). Moreover, since you provided git tag parameter (git_tag: '1.7.4') it also creates a link to your GitHub commit diff.
Next time you run for example:
stamp_changelog(
  section_identifier: '1.7.5',
  git_tag: '1.7.5'
)

your file should look as following:
## [Unreleased]

## [1.7.5] - 2016-11-10
### Added
- More awesome features

## [1.7.4] - 2016-11-09
### Added
- Use Fastlane

## [1.7.3] - 2016-11-04
### Changed
- Set background as white

## [1.7.2] - 2016-11-04
### Fixed
- Fix chart layout

### Changed
- Update dependencies

[Unreleased]: https://github.com/GlobalSport/mycoach-player-android/compare/1.7.3...HEAD
[1.7.4]: https://github.com/GlobalSport/mycoach-player-android/compare/HEAD...1.7.4
[1.7.5]: https://github.com/GlobalSport/mycoach-player-android/compare/1.7.4...1.7.5

@mathroule has this answer help you☝ ?


[Parity w/ .NET] Kept tentantId & channelData in CreateConversation

Parity with: https://github.com/microsoft/botbuilder-dotnet/pull/2481
Before implementation

After implementation

Pull Request Test Coverage Report for Build 79119

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 88.596%

Totals

Change from base Build 77828:
0.002%

Covered Lines:
4429

Relevant Lines:
4805

💛  - Coveralls


Add Cartfile

Right now if one wants to use just RMDateSelectionViewController via Carthage, the RMActionController dependency is not resolved automatically so it needs to be added to Cartfile manually.
This PR adds missing Cartfile so it can be resolved automatically

Thanks for the PR 🙂!
Will have to give it a quick check before I merge, to make sure the current develop commit of RMDateSelectionViewController is compatible with version 1.3.1 of RMActionController. Otherwise, I'm fine 🙁.

The dependency version is copied from the Cocoapods podspec so it should be fine 🙂

Hello?

@CooperRS please

Hey! Sry, for the long delay 🙈. Merging now!


xonsh: add misssing which dependency

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ x ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ x ] Tested execution of all binary files (usually in ./result/bin/)
[ x ] Fits CONTRIBUTING.md.

xonsh uses which to check for a git binary, the current package does not express that dependency, and thus xonsh will crash if run on a system where which is not installed.

you might want to add it as a build input

It doesn't need to be there to build, only at runtime, should I still use a build input for that?
Do I just add it to the propagatedBuildInputs line, or do I add a seperate buildInputs line?

That would be propagatedBuildInputs. But how does it act when it cannot find git? Since git is not an input either. Often having a program use which is undesirable, because you do not know which it will find and use. Instead you would want it to use the version of the application you specify. I suggest you wait for comments of the maintainer @spwhitt.

If it doesn't find git it just skips testing the repository status of a directory.
It might make sense patch xonsh to use a specified git, though though that would involve changing git from an optional dependency to a hard dependency. Upstream wants to be able to install where git isn't present, but since we can guarantee git it would probably be an acceptable change.
I'll wait and see what @spwhitt has to say.

@tdimiduk to elaborate on @FRidh's comment: simply adding which to the expression parameters does nothing. You either need to ensure that which ends up in the xonsh wrapper PATH (adding to propagatedBuildInputs may do this, I've not looked it up) or substitute calls to which in the source with ${which}/bin/which.

Weird. When I was testing it seemed to work with that and not without it,
but I must have accidentally had some state persisting into the test that I
thought worked.
I can make that change, but I guess it is still up for discussion whether
we should switch away from using which entirely here?
On Fri, May 13, 2016 at 10:01 AM, Joachim Fasting<EMAIL_ADDRESS>wrote:

@tdimiduk https://github.com/tdimiduk to elaborate on @FRidh
https://github.com/FRidh's comment: simply adding which to the
expression parameters does nothing. You either need to ensure that which
ends up in the xonsh wrapper PATH (adding to propagatedBuildInputs may do
this, I've not looked it up) or substitute calls to which in the source
with ${which}/bin/which.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/15432#issuecomment-219051368

@tdimiduk how did you determine that xonsh needs which? As far as I can tell, it implements its own locate_binary method --- this is what it uses to find e.g., git.

Ah, apparently I wasn't fully awake this morning and didn't notice that for some I am getting an old version (0.1.3 instead of the current 0.2.7) which does not use which after all. Sorry for the unnecessary PR. Now to figure out why nix things I want the old version of xonsh ...

@tdimiduk did you use nix-env to install the application? If so, you perhaps hadn't updated your channel.

I tried both nix-env and nixos-rebuild and thought I was up to date, but judging by the giant stream of updates currently coming down I obviously was out of date somehow.


Publish settings-cycler to OpenVSX?

It would be great if settings cycler was published to Open VSX, for use in alternate installations (like code-server).
Documentation for the process is here:
https://github.com/eclipse/openvsx/wiki/Publishing-Extensions

+1.
Some of our internal tooling use open VSX and it would be excellent to publish cycler there.

FYI - https://open-vsx.org/extension/jellydn/vscode-settings-cycler


fixed some semantical issues

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

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

Since we no longer have only one way of spacing in this file, it is highly outdated. Hence, I'm closing this PR


Use パドックコメント

There aren't as many nulls as previously thought:

Example:

This is a historical feature.

Check if this can be scraped from the website before the race, and if so, whether it's the same as SKB, and if so, consider using in training.


Add Fractions

Now that we have nailed down denominators for demographics and some economics we can implement fractions. Approach will be to assign 1 to responses that fall within category and 0 to responses that don't and then take mean grouped by geographic area (PUMA or borough)

Have fractions of demographic indicators, next need to crosstab by race


Is it EEG a tool for Performance tracing open-sourced ?

As describe in paper 9.2 section TensorFlow:
Large-Scale Machine Learning on Heterogeneous Distributed Systems:

We also have an internal tool called EEG (not included in the initial open source release in November, 2015) that we use to collect and visualize very fine-grained information about the exact ordering and performance...

I want to know that Does EEG open source?
EEG includes 3 tools:

Linux ftrace
Internal Google tracing tools
The CUDA Profiling Tools Interface

And what's the method of "internal Google tracing tools"?
Have I written custom code:  N/A
OS Platform and Distribution: Ubuntu 16.04
TensorFlow installed from: docker
TensorFlow version: 1.8.0
Bazel version: N/A
CUDA/cuDNN version 9.1, 7.0
GPU model and memory P100, 16GB
Exact command to reproduce N/A
Mobile device N/A

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

@tensorflowbutler Yes, I have update these field.

Nagging Assignee @drpngx: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Note: This isn't a bug or issue, so should not have been filed on GitHub issues.
No, EEG is not open source.  It makes extensive use of some google-specific libraries for thread tracing, as well as an internal wrapper around Linux kernel tracing that would be difficult to open-source.
The trace-visualization server wouldn't be too hard to port, but somebody would have to provide all of the equivalent kernel-level context-switch traces and application-level threadpool tracing.