Added support for running Mindcode locally

Resolves #27

@schittli, could you kindly checkout this PR and tell me if the Windows script works? Look at bin\run-local.cmd.
It's been a very long time since I wrote Windows command files!
Thanks!

Thanks a lot for writing the windows batch file!, I have a few suggestions.
Unit Tests, ProcessBuilder, diffand Windows
Because we now have run-local.cmd, therefore I guess that the tests should also run.
Because windows does not know diff, I have googled for Java unit tests using ProcessBuilder() with different operating systems. Some developers use this concept to to distinguish between the OS in the code:
File: webapp\src\test\java\info\teksol\mindcode\webapp\SamplesTest.java
    private void compile(String program, File source) throws IOException {
        final boolean isWindows = System.getProperty("os.name").toLowerCase().contains("win");
        …
        final Process diff = new ProcessBuilder()
                .command(isWindows ? "fc.exe" : "diff", "-u", unoptimizedTarget.getAbsolutePath(), optimizedTarget.getAbsolutePath())
                .redirectOutput(diffTarget)
                .start();
    }

At least, on Windows, this code works.

Oh yeah, I had forgotten about that part. I’ll copy your code and make it work here. Then you can try it on Windows.
Thanks!
François

Le 22 avr. 2021 à 08:43, Tom-- @.***> a écrit :

Thanks a lot for writing the windows batch file!, I have a few suggestions.
Unit Tests, ProcessBuilder, diffand Windows
Because we now have run-local.cmd, therefore I guess that the tests should also run.
Because windows does not know diff, I have googled for Java unit tests using ProcessBuilder() with different operating systems. Some developers use this concept to to distinguish between the OS in the code:
File: webapp\src\test\java\info\teksol\mindcode\webapp\SamplesTest.java
private void compile(String program, File source) throws IOException {
    final boolean isWindows = System.getProperty("os.name").toLowerCase().contains("win");
    …
    final Process diff = new ProcessBuilder()
            .command(isWindows ? "fc.exe" : "diff", "-u", unoptimizedTarget.getAbsolutePath(), optimizedTarget.getAbsolutePath())
            .redirectOutput(diffTarget)
            .start();
}

At least, on Windows, this code works.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Feedback to bin\run-local.cmd
Good evening @francois
For not having written a Windows Command File for a long time, your file bin\run-local.cmd looks very good :-)
Nevertheless, I have two hints:
1) PowerShell would be the better solution

This Sub-Command is a PowerShell Command and does not work in cmd: "$((Get-ChildItem .\webapp\ *.jar -Recurse | select -ExpandProperty FullName) -join ';')"
Because by now Linux, even MacOS and every Windows has PowerShell for a long time, I would use PowerShell for this script and not Windows cmd anymore. I will make a suggestion.

2) Nevertheless, as a cmd refresher:

Calling another cmd must use call …, so this is better: call mvnw.cmd install
Echo does not need ". If they are used, cmd will print them, which looks ugly

So that would be the better version, but the java call doesn't work because it just consists of a PowerShell command:
@echo off

rem In order to prevent the important information from being obscured,
rem compile Mindcode, THEN give important information to get it working.
rem This way, if Mindcode can't start, the important information will be
rem available right above the failure.
call mvnw.cmd install

echo *****
echo *****
echo ***** To run Mindcode locally, you need to have a PostgreSQL database accessible.
echo ***** You have a few choices:
echo *****
echo *****     - Install PostgreSQL for Windows: https://www.postgresql.org/download/windows/
echo *****
echo *****     - Use Docker to run PostgreSQL:
echo *****
echo *****           Download Docker from https://www.docker.com/get-started
echo *****           Once done, start a PostgreSQL instance:
echo *****               docker run -p 5432:5432 --name mindcode -v mindcode -e POSTGRES_HOST_AUTH_METHOD=trust postgres:latest
echo *****
echo ***** If your PostgreSQL instance is not available at postgres://<IP_ADDRESS>:5432/mindcode_development, edit
echo ***** both Spring configuration files and set the correct values:
echo *****    webapp\src\main\resources\application.properties
echo *****    webapp\src\test\resources\application.properties
echo *****

java -classpath "$((Get-ChildItem .\webapp\ *.jar -Recurse | select -ExpandProperty FullName) -join ';')" info.teksol.mindcode.webapp.WebappApplication

Proposal to replace bin\run-local.cmd by PowerShell
bin\run-local.cmd has one advantage: it's small, but the Java-Call does not work (as already mentioned)
Therefore, I propose to change to PowerShell. I have written an example that is doing the following steps.
I can send you the PowerShell file as a change request, but I only do this two or three times a year, therefore I'm not a pro doing this. If you want, I can also reduce the script to the minimum, my solution currently performs these steps:

In webapp\src, it searches for the application.properties files
It reads the spring.datasource.url setting
… and it tests if the PostgreSQL Port is open
If the Port is not accessible, then the scripts stops and displays the Message To run Mindcode locally, you need to have a PostgreSQL database accessible. …
If the PostgreSQL Port is accessible, it starts .\mvnw.cmd install
If .\mvnw.cmd has an error, the Script stops
Then, java will be started

The Scrip-Start with the connection-tests looks like this:

In case you want to see it in action, I've captured the whole webapp-startup in a gif: https://imgur.com/jrh84HR
This would be the script which I would send you in a change request:
run-local.ps1.txt

I'm 100% open to using PowerShell. Last time I used Windows for development was 13 years ago! If you can share the PowerShell script, please do so. That will be very helpful, as I don't have a Windows environment to test in.

I'm 100% open to using PowerShell. Last time I used Windows for development was 13 years ago! If you can share the PowerShell script, please do so. That will be very helpful, as I don't have a Windows environment to test in.

Good evening @francois
I don't ask because I'm impatient 🙂, but it could be that we're waiting for each other, which would be unfortunate... hence the question:
Did you see that I created the pull request with the PowerShell script? pull 29
I've been struggling with Git / Github and it was the best I could do on the third try.
I will do my best to correct any change requests asap and once the pull request is integrated, I will test it again.

Hi, sorry about the delay. I received my COVID-19 shot this weekend and wasn't up to working on open-source. I should get around to this PR sometime this week. Thanks again for your effort!

Oh... this is actually good news :-) - so, we're jumped from 1'040'000'000 Vaccinated people to 1'040'000'001
The progress is better than it looks :-)

@schittli, I have updated this PR with your latest changes. If you had time to retest another time, I'd really appreciate! Thanks a bunch :)


App-level Build.gradle file shows error that Cannot resolve Symbol "GradleException"  in every Flutter Project either its new or old , edited or non edited

OS :- Windows 10 home 64bit
IDE:- Android Studio version 3.5
Flutter version :- 1.9. 1 +hotflix .6
Gradle version :- 6.0.1
Error which I am facing.

in App level build.gradle file. I am getting an error ~ cannot resolve symbol "GradleException''.

I have checked the google but I am unable to solve my problem.  I had tried almost everything like  1 Invalidate Chaches/restart

Changes some setting in  File --> Project Structure
a. Project

b.

c.

But nothing worked for me.....
one more thing I have noticed that I am facing similar in error in almost every project either it's new or old. whenever I open Android's app-level Build.gradle file. its automatically showing the error that " cannot resolve symbol GradleException"

Solution for:-
OS:- Windows 10 Home 64bit
IDE :- Android Studio 3.5
Flutter version 1.9.5 hotflix 6
Gradle version 6.0.1

If you are getting property error e.g. Cannot Resolve symbol "Property" go to File => Project Structure => below the "Project Settings " select project and then change the Project SDK from NO SDK to Android API 28 ( whatever the stable one).

If you are getting GradleException error e.g. Cannot Resolve symbol "GradleException"

simple change the name form GradleException ( "  text ") to FileNotFoundException ( "  text ")

Hi @kamalkant56
given your last message I feel safe to close this issue,
if you disagree please write in the comments
and I will reopen it.
Thank you

is that safe to changing name from  GradleException ( " text ") to FileNotFoundException ( " text ") ???

because i did the same thing but i am thinking it is correct way or not
if any body knows correct way tell me

I too have the same problem. It can be fixed as of now by following these steps
https://github.com/flutter/flutter/issues/29608#issuecomment-578383592

Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.

If you are getting GradleException error e.g. Cannot Resolve symbol "GradleException"
simple change the name form GradleException ( "  text ") to FileNotFoundException ( "  text ")

Does not resolve the issue. Neither using only Exception (...
Win 10 H 64b
IDE: AS 3.6
Flutter v:  1.18.0-9.0.pre.122
Gradle: --- I do not know


Integrate RunGPT For LLMs

Description
RunGPT is a wonderful open-source LLM model serving framework belong to JinaAI company. It is designed to simplify the deployment and management of large language models. LlamaIndex can do use this kind of model service to accelerate model deployment and enhance the user experience using llm service.
Integrate RunGPT through using class LLM at llama_index/llms/rungpt.py
Add required Jupter notebooks at docs/examples/llm/rungpt.ipynb
Add required unit test at tests/llms/test_rungpt.py
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Install necessary packages such as run-gpt-torch, and run a simple command to start a service of large language model localhost. The detail tutorial could be found in docs/examples/llm/rungpt.ipynb.

[x] Added new unit/integration tests
[x] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x]  I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@logan-markewich
Thank you for your feedback and suggestions. I appreciate your help in identifying the bug and guiding me towards a solution. The bug that was occurring during the make lint process has been resolved.

Thanks @selmiss !! 💪🏻


Plugin breaks Webpack in CircleCI since v1.5.0-beta.1

Since v1.5.0-beta.1 this plugin somehow breaks Webpack build process in silent since v1.5.0-beta.1. This only happens in my CircleCI environment (node 6.1.0, npm 5.3.0) without any error message.
I have settings like this:
    new ImageminPlugin({
        externalImages: {
            sources: [...]
        }
    }),
    new ExtractTextPlugin({
        filename: '[name].css',
    }),

The problem is the ExtractTextPlugin doesn't execute after I upgraded to v1.5.0-beta.1.
From my test it works after I remove all options in the ImageminPlugin
It only happens in version v1.5.0-beta.1 and after. I switched back to v1.5.0-beta.0 and it works fine.

I just released V1.6.0-beta.1, which works for me on CircleCI.
I don't exactly know what the problem was, but 1.6 was a big refactoring, so I might have inadvertently fixed the issue!


To do list for Milestone 2

Create dashboard (requirements: 3 plots)

[ ] Clean/import dataset (Add raw and processed) (@gauthampughazhendhi @Jacq4nn)
[ ] Create structure of dashboard (@Jacq4nn - send some samples)
[ ] Create world map (@gauthampughazhendhi)
[ ] Add interactions for world map (@Jacq4nn @gauthampughazhendhi)
[ ] Line Plot (@vtaskaev1)
[ ] Dropdown Plot (@Rowansiv)

Heroku

[ ] Set up team Heroku and add collaborators (@Rowansiv)
[ ] Deploy to Heroku (@Rowansiv)

Others

[ ] Reflection (@Jacq4nn)
[ ] Improve README (@vtaskaev1)
[ ] Rearrange Github Folder Structure (@gauthampughazhendhi)

Link to Milestone 2 requirements: https://pages.github.ubc.ca/mds-2021-22/DSCI_532_viz-2_students/materials/release/milestone2/milestone2.html

Useful links for past dashboards:
(Side navigation bars) https://github.com/UBC-MDS/career_deciscion_aid-r
(Nav bars + interactions with world map) https://github.com/UBC-MDS/DSCI_532_Group15_wine


Update sbt to 1.5.5

Updates org.scala-sbt:sbt from 1.5.4 to 1.5.5.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Codecov Report

Merging #119 (486d638) into master (03c038e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           1        1
  Lines          58       58
  Branches        2        2
=======================================
  Hits           57       57
  Misses          1        1

Continue to review full report at Codecov.

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


move to package menu item in method browser produces MNU asSymbol

specifically a MessageNotUnderstood occurred (error 2010), a UndefinedObject does not understand  #'asSymbol' is produced ... stack attached.
commit ids:
JadeiteForPharo 86deecf
PharoGemStoneFFI 3575de4
RemoteServiceReplication 371ae8e
---------------------------------------------
RowanClientServices ef59884

stack.txt

Screen shot of method browser ...

Looks like it was only broken in the method list browser.
When we issue a command, we set the command & commandArgs in the service and pass it to the server. However, the wrong class service was getting sent to the server in this case. Therefore, the command was nil.

See previous comment & commit id for fix details.


Revert "[ConstraintSolver] Prioritize certain type variables while lo…

…oking for bindings"
This reverts commit 001b09747a153763b30ef8a02254c7a0419c6292.

@swift-ci please smoke test

@jrose-apple Is there a way to force merge the changes?


Travis -> GHA

This is another case where we seemingly can't actually test our fork with the versions of Ruby we're acutally using this gem in ( kinda guessed that it's all our "current" versions?).
This will suffice for 'moving off travis' even though they fail, given ultimately whenever we update our fork, the upstream has already migrated to GHA for testing.

https://app.shortcut.com/aptible/story/26221/gha-migration-oauth2


don't support Android ListView renderHeader

with Android ListView renderHeader , it will be a  blank page, It seems that  the  ViewPagerAndroid  don't support  Android ListView renderHeader

try to add removeClippedSubviews to ListView component

set removeClippedSubviews（false） to ListView work for me!!!

thanks


feat: add email subscription/unsubscription

Toward https://github.com/snapshot-labs/workflow/issues/226
Following https://github.com/snapshot-labs/snapshot-hub/pull/951
This PR will add a new subscription/unsubscription features

subscription will allow adding a new subscription when passing an email, and updating an existing subscription when just passing a metadata
delete-subscription will allow deleting an existing subscription

By merging the write function from envelop (https://github.com/snapshot-labs/envelop/?tab=readme-ov-file#subscribe-to-the-mailing-list) into snapshot.js, then sequencer, we can leverage the signature and alias feature of sequencer, and will not require an extra handling of transaction signature.

I am thinking that maybe, we should not use a generic subscription name, and directly use email-subscription, to avoid confusion with the existing subscription type returned by the hub.

I am thinking that maybe, we should not use a generic subscription name, and directly use email-subscription, to avoid confusion with the existing subscription type returned by the hub.

agree


Preparation or improvement of the guide for usage of dynamic systems and reporting  (issue #462 related)

Improve a guide for creating beamer presentations that is stored in the Notebook.

The  updated guide can be found here: Notebook


MunkiAdmin 'hangs' when running makecatalogs

When pressing the 'Save' button, MA shows a dialog that says: 'Status: Running makecatalogs' which never ends. Pressing 'Cancel' does not make the dialog go away and the only left to do is to force quit Munkiadmin.
With Log level set to Verbose, I get the following log entry:
2016/03/26 09:14:02:753  MunkiOperation:makecatalogs
2016/03/26 09:14:02:753  Running /usr/local/munki/makecatalogs with arguments: (
    "--force",
    "/Users/Shared/munki"
)
2016/03/26 09:14:03:017  startOperationTimer

That's a weird one, took me a while to get to the bottom of (or at least get a theory) of what's happening. I think there is a deadlock issue in the MunkiAdmin's makecatalogs operation visible only if there's a lot of stderr output.
Could you try this custom build to see if it helps:
https://www.dropbox.com/s/wlon930g0k3tbi9/MunkiAdmin-1.4.1-11-gf7e0caf.dmg?dl=0
In that build I just changed the order in which MunkiAdmin reads stderr and stdout from makecatalogs.  Needs more investigation...

The new build is completing succesfully. Good Job!

This is now fixed in version 1.4.2.


Create test_test1.py

Added a test_ file to prevent bash error during test run in azure devops pipeline...

Thanks for the fix--apologies for not seeing the PR earlier.


Backup story for TRSS server

The current TRSS server https://trss.adoptopenjdk.net/ is down for almost two days. And we cannot ping the machine. While people are looking into the issue, we need to have a backup story for TRSS server (in case this happens again in the future).
This backup story will include not only quick bring up of a new server, but a schedule for backing up mongodb on the backend (so we have test history available in the new instance).

related: https://docs.mongodb.com/manual/core/backups/#backup-with-mongodump

Related: https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1295

A cron job can be created on the machine to export MongoDB data weekly. Ideally, the exported data should be stored on a separate server (i.e., artifactory, git repo, etc). For now, we can just store the backup data on the machine.

@llxia Can you set up a script to do that? As a first pass I'm happy with that and then I can look at getting the backups exported off the machine

@llxia Can you set up a script to do that? As a first pass I'm happy with that and then I can look at getting the backups exported off the machine

Sorry, missed your above message. Talked with @OscarQQ , he will help with this issue.

Sorry, missed your above message. Talked with @OscarQQ , he will help with this issue.


Fix some BlockDataForm bugs

don't add variations field multiple times
always clone the schema before applying enhancers

I'm a bit torn also in putting components in helpers, even if they are HOCs, we already regret it a couple of times... What do you think?

We have hooks and HOCs in helpers: https://github.com/plone/volto/blob/master/src/helpers/Loadable/Loadable.js and https://github.com/plone/volto/blob/master/src/helpers/AsyncConnect/index.js#L93 to name a few.
From my perspective they don't belong in the components folder, but to a more neutral place, though I can understand the thinking behind keeping helpers "js-only".

"variation" is fine as a name
isDefault into the "variation" choice
rename "variation choices" to "items"
move the title into the extension definition
refactor the hoc to get the extension id
remove / adjust the old doc into extensions.md


Delete MediaReadyActiveCases

Got a request from "National Center for Missing & Exploited Children" to delete this PII file. Please approve and merge.

Or better yet, delete this repository.

Closed by mistake. @edygarcia would request you to please delete the csv file from this repo.


ability to adjust users volume.

Hello!! I have recently discovered your application and it is wonderful! however there doesnt seem to be a function to control individual users volume. Not sure if its possible to do but would be a wonderful feature if it could be implemented in during the upgrading process. thank you so much in advance

Hey logan. I'm currently in the early stages of a full rewrite hoping to improve stability and deliver a far better app. I'll add mark this as a feature to keep in mind along the way.
I'll admit though, I'm not quite sure how it works within the WebRTC API. Perhaps I'm looking in the wrong place and it's part of the voice gateway. Anyway, I'll look into it.

Hello,I am a new user in the Quarrel app and I have a little knowledge with UWP applications,I could help in the project with some contribution?or at least,lend me to test development versions in xbox and pc,thanks!

We're currently working on setting up alpha builds that you can sideload and will auto-update. Currently the rewrite is very incomplete, so there isn't much functionality to test however attempting to make it crash and making reports on how would be very useful.
Also anyone is able to create and submit a pull request

This would be awesome.


Ray steals CTRL+SHIFT+K shortcut from Sublime and Firefox

Describe the bug
I was puzzled today at why the "delete line" shortcut in Sublime stopped working. I realized that I had installed Ray today and it is stealing this shortcut from all applications. CTRL+SHIFT+K is the standard "delete line" shortcut in Sublime, it is also the shortcut to open the browser console in Firefox. This makes it a poor choice for a shortcut.
Versions
Ray version (you can see this in "About Ray"): 1.0.5
Desktop (please complete the following information):

OS: Linux
Version 20.10

We're going to keep it for now, as

there are not too many people complaining (you are the first)
the shortcut can be changed
my guess is that there is no combination this isn't used by all Ray users

We're going to keep it for now, as

there are not too many people complaining (you are the first)
the shortcut can be changed
my guess is that there is no combination this isn't used by all Ray users


How to use AVVideoCompositionCoreAnimationTool to add sticker and text layer on video ?

Checklist

[x] I've read the README
[x] I've searched for existing GitHub issues

MetalPetal does not support AVVideoCompositionCoreAnimationTool. If you'd like to process video files you can use MTIVideoComposition.

But how to add text, gif and sticker on video using MTIVideoComposition,  Any code example ?

Please see #320


multiple_of failing on some negative floats

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Why does it work on x but fails on -x? Is this intended behavior?
Example Code
from typing import Annotated
from pydantic import BaseModel, TypeAdapter, Field

for i in range(-10, 10):
	i = i / 100
	try:
		TypeAdapter(Annotated[float, Field(multiple_of=0.01)]).validate_python(i)
		print(f"  OK: {i}")
	except:
		print(f"FAIL: {i}")

prints:
  OK: -0.1
FAIL: -0.09
  OK: -0.08
  OK: -0.07
FAIL: -0.06
  OK: -0.05
  OK: -0.04
FAIL: -0.03
  OK: -0.02
  OK: -0.01
  OK: 0.0
  OK: 0.01
  OK: 0.02
  OK: 0.03
  OK: 0.04
  OK: 0.05
  OK: 0.06
  OK: 0.07
  OK: 0.08
  OK: 0.09

Python, Pydantic & OS Version
pydantic version: 2.7.2
        pydantic-core version: 2.18.3
          pydantic-core build: profile=release pgo=true
                 install path: /Users/lucas/{redacted}/venv/lib/python3.12/site-packages/pydantic
               python version: 3.12.3 (main, Apr  9 2024, 08:09:14) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
                     platform: macOS-14.5-arm64-arm-64bit
             related packages: typing_extensions-4.12.0 fastapi-0.111.0 pydantic-settings-2.2.1
                       commit: unknown

@plutaniano,
Huh, this is definitely a bug. Thanks for reporting this! Likely an issue due to undesired rounding. A PR with a fix is more than welcome!

Hi, I've been investigating this today, seems that using decimal.Decimal instead of float makes the trick.
from typing import Annotated
from pydantic import TypeAdapter, Field
from decimal import Decimal

for i in range(-10, 10):
    x = i / 100
    try:
        TypeAdapter(Annotated[Decimal, Field(multiple_of=0.01)]).validate_python(x)
        print(f"  OK: {x}")
    except:
        print(f"FAIL: {x}")

@sydney-runkle
Hi!
It looks like the validation in pydantic-core needs some modification. If this issue is still unresolved, could you please assign it to me? I have a good idea of where the changes are needed, so I believe I can submit a PR relatively quickly, including additional test cases.

Hi @K-dash,
I was looking at the same issue, but you made it quicker:).
Just wanted to point out that IMO - to maintain coherence with the existing setting - you might want to add the same test parametrizations of pydantic-core/tests/validators/test_float.py::test_float_multiple_of to pydantic-core/tests/validators/test_decimal.py::test_decimal_multiple_of and make sure tests pass.

Hi @AlessandroMiola
Thank you for your suggestion:).
I agree with your point, so I have added the same test cases to test_decimal_multiple_of as well.


LSTM's .h5 to .tflite

thanks for your reading.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):source
TensorFlow version (use command below):1.12.0
Python version:3.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
1.used the tf.keras to train the LSTM's model, and get the model.h5 file.
2.want convert the .h5 file to .tflite file ,and coded:
converter = tf.lite.TFLiteConverter.from_keras_model_file("model.h5")
tflite_model = converter.convert()
open("_model.tflite", "wb").write(tflite_model)

and get the error:
WARNING:tensorflow:From D:\python\python3\lib\site-packages\tensorflow\python\framework\graph_util_impl.py:245: extract_sub_graph (from tensorflow.python.framework.graph_util_impl) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.compat.v1.graph_util.extract_sub_graph
Traceback (most recent call last):
  File "D:/code/load_forecast/test/test1.py", line 5, in <module>
    tflite_model = converter.convert()
  File "D:\python\python3\lib\site-packages\tensorflow\lite\python\lite.py", line 456, in convert
    **converter_kwargs)
  File "D:\python\python3\lib\site-packages\tensorflow\lite\python\convert.py", line 397, in toco_convert_impl
    input_data.SerializeToString())
  File "D:\python\python3\lib\site-packages\tensorflow\lite\python\convert.py", line 172, in toco_convert_protos
    "TOCO failed. See console for info.\n%s\n%s\n" % (stdout, stderr))
tensorflow.lite.python.convert.ConverterError: TOCO failed. See console for info.
2018-12-06 13:41:26.456495: I tensorflow/lite/toco/import_tensorflow.cc:1280] Converting unsupported operation: TensorArrayV3
.........
2018-12-06 13:41:26.495220: I tensorflow/lite/toco/import_tensorflow.cc:1329] Unable to determine output type for op: TensorArrayGatherV3
2018-12-06 13:41:26.506452: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 120 operators, 212 arrays (0 quantized)
2018-12-06 13:41:26.508258: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 120 operators, 212 arrays (0 quantized)
2018-12-06 13:41:26.513681: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 106 operators, 194 arrays (0 quantized)
2018-12-06 13:41:26.515613: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before dequantization graph transformations: 106 operators, 194 arrays (0 quantized)
2018-12-06 13:41:26.517624: F tensorflow/lite/toco/tooling_util.cc:625] Check failed: dim >= 1 (0 vs. 1)

3.but success to convert the linear's and cnn's .h5 file to .tflite
want to ask the question: Did someone successfully transform the .h5 model to .tflite?
look forward to your answer.

How about adding the line:
converter.allow_custom_ops=True

How about adding the line:
converter.allow_custom_ops=True

get the same error...

Same problem here @t-kuha . Trying to convert a CRNN using LSTM layers .h5 model (saved with keras) to .tflite and also getting
F tensorflow/lite/toco/tooling_util.cc:627] Check failed: dim >= 1 (0 vs. 1)
Any suggestion yet ?

Hi everyone,
A quick search and I guess I found the reason of this error. The LSTM layer is not yet supported for custom models
https://www.tensorflow.org/lite/tf_ops_compatibility
bottom of the page.

@AntoineWeber tk, I have asked some people to give me the same reply before, but I hope to support LSTM as soon as possible.

@AntoineWeber , to be precise, it is tf.tanh not supported or LSTM, you can use it but need to fall back on Linear? Is my understanding correct?

@AntoineWeber , to be precise, it is tf.tanh not supported or LSTM, you can use it but need to fall back on Linear? Is my understanding correct?

any updates on which part isn't supported?

@nparikhsfn
I don't understand what you mean by linear. But I guess you could reimplement yourself an lstm layer trying to dodge the use of tanh layers. GRU is not listed on this list (?) and it also contains tanh.
Haven't retried since then to convert models to tflite. If you really need to use layers which are not supported you can always try to compile Tensorflow for your embedded platform if your memory permits but this can be a pain.
@pigubaoza I guess you should stick to the list on this link : https://www.tensorflow.org/lite/tf_ops_compatibility

@JamesZhangjp This is a stale issue. If anyone looking for LSTM example, you can check this TF resource where you can find couple of examples.
I am providing one complete example with recent tf-nightly. Please take a look at the gist here. Thanks!
I am closing this issue. Feel free to reopen if there is any issue. Thanks!


Many links broken

Gitea version (or commit ref): 15f6ec9
Git version: N/A
Operating system: N/A
Database (use [x]): N/A

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)

https://try.gitea.io/yasuokav/link-test/issues/2
https://try.gitea.io/yasuokav/link-test/src/branch/master/README.md

[ ] No
[ ] Not relevant

Log gist: N/A

Description
The link should be
https://try.gitea.io/yasuokav/link-test/issues/1
instead of
https://try.gitea.io/yasuokav/link-test/issues/yasuokav/link-test/issues/1
Please check the links above for more details.
I think this issue because of #4058 @JonasFranzDEV
Screenshots
N/A

It should be related to the remove of leading '/' when base isn't absolute (with a scheme). This test should be change to add a check if base start with a '/' to keep it.
As this error is related to a suggested solution of mine ^^.

And I'm afraid release/v1.4 is also affected.

@lunny fix is ready. I will backport it.


Refactored combined use of await and .then to use await only

In the file /src/extensions/functions/updateVersions.ts, a combination of async/await and .then notation was used. Refactored that code to use async/await only. Closes #223

Works great, sorry about CI


Implementing #845, extended wp context menu

Please check whether you like the feature, then merge :)

Looks fine for me. Working with multiple context menus and even more with sub-menus is really ugly. Should we merge this? Or do you want to add the missing entries first? (I can remove the waypoint-activity then.)

I am in the process of implementing the missing entries, wanted have it ready already yesterday, but some small things still missing. So latest this weekend.

Looks good. I'll remove the waypoint-activity later today. Should we invoke the context menu with onClick?


Add achievements

Progress

[x] store achievements in json and make it easy to add new ones
[x] check for new achievements
[x] store earned achievements in database
[x] display toasts when new achievements are earned
[x] implement achievement screen

[x] Add core achievement functionality
[x] show more info when you click on each achievement tile (including assets such as app logo)

[x] implement all achievements into app (as in create them and put them in the .json)
[x] implement a way to share the activity on social media
[x] Make sure sharable image is within an aspect ratio that social media platforms allow (so it doesn't get cropped)

Add a way to display and earn achievements. There are a couple of things to keep in mind:
Achievement data
Specifically, I am talking about how we store the requirements, names, descriptions, etc. of the achievements. I think it would be best to do this in a json file, since dart can parse things to a map from json.
What we need to store (? is nullable)

category
name
description
variable (what thing this achievement is dependent on)
upgradable? (enables leveling up if true)
level cap? (int, if undef/null there isn't one)
level up amount (list, the last element will keep going until the level cap is hit)

Earning/losing achievements
Generally, the user is going to be earning achievements, but, lets say they entered in a class wrong (they put a 92 instead of an 82). Lets also say this would have let them be able to level up one of their achievements, but now that it's different, that achievement doesn't apply anymore. We have to remove the achievement.
Since there aren't going to be that many achievements, we can most likely just loop through all of the possible achievements each time, and just recalculate everything every time classes, activities, or something else happens. This would handle both adding and removing achievements.
The user should get notified when they earn an achievement. As we calculate each achievement, if the achievement does apply, we can also check to see if the user already has this achievement (or if they have a higher level now than they did before). If this is the case, then we should display a toast at the bottom of the screen, telling them they've earned an achievement, and to check the achievements tab for more info.

Last item remaining was completed by a360e6fbc77b64cab66a22af6e12af2da307160a. There may be more achievements added later, and more properties (such as rarity) may be added later, but as it is, this is fully functional.


Fix missing cjs usage in examples

Related to #422

Thank you. I triggered another nightly release too just now

@Apollon77 Oops, I deleted packages/matter-node-ble.js/require/package.json as I couldn't recall why it was there. Just realized that it was there for compatibility of older Node.js (older than Node.js 12).

next nightly on it's way


[AUTOMATED] Update terraform-null-label versions to support Terraform 0.13

What

Updates terraform-null-label versions 0.14.1, 0.15.0, and 0.16.0 to version 0.17.0
This is an automated commit and pull request created by Microplane. More information here.

Why

This module and the mentioned previous versions are used extensively across the large majority of Cloud Posse's Terraform Modules. They're currently holding back support for Terraform v0.13. Doing this upgrade in mass will alleviate the pain in our efforts to support Terraform v0.13

/test all

hds-staging-lb-us-east-2-960392145.us-east-2.elb.amazonaws.com

/rebuild-readme

/test all


Revert "Fix serialization of transaction_paymentInfo"

Reverts paritytech/substrate#4146

I have also faced some issues with arbitrary_precision  of serde and json_rpc crate in other repos... shan't we for now make the fee RPC return a (potentially lossy) SaturatedInto<u64> of the fee? that's the only problem here. I can make a quick patch for that


[BUGFIX]: Fix/sanity check deps and npm audit vulns

Reason:
I am currently using this library and noticed (during a docker build) that some of the dependencies point to master branches and not releases, this is sketchy if any of these repositories get hijacked and is quite easy to fix.
Changes:
Point web3bch/bch-wallet-bridge.js to release v0.1.1, not master.
Point christroutner/bitcoincashjs-lib to release v4.0.1, not master.
Also updated deps and fixed npm audit critical fixes.
Test Results:
  2599 passing (58s)
  1 failing

  1) #Script
       #encode
         Encode SLP SEND OP_RETURN properly
           should correctly compile OP_RETURN SLP SEND transaction:
     TypeError: Bitcoin.script.compile2 is not a function
      at Script.encode2 (src/script.js:129:245)
      at Context.<anonymous> (test/unit/scripts.js:104:35)
      at processImmediate (internal/timers.js:456:21)

-------------------------|----------|----------|----------|----------|-------------------|
File                     |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
-------------------------|----------|----------|----------|----------|-------------------|
All files                |    77.69 |    52.79 |    80.46 |    81.21 |                   |
 src                     |    73.97 |    48.64 |    76.92 |     78.1 |                   |
  address.js             |    95.45 |    87.76 |      100 |       95 | 55,67,222,620,727 |
  bch-js.js              |    93.16 |    44.44 |      100 |    93.16 |... 19,121,127,129 |
  bitcoincash.js         |     93.1 |    72.22 |    94.12 |    98.11 |               109 |
  blockbook.js           |    66.67 |       50 |       75 |    72.22 |... 46,349,351,352 |
  blockchain.js          |    47.06 |    16.52 |       90 |    53.85 |... 00,803,805,806 |
  control.js             |    66.67 |        0 |      100 |       75 |       78,79,91,92 |
  crypto.js              |      100 |        0 |      100 |      100 |               133 |
  ecpair.js              |      100 |    88.89 |      100 |      100 |                27 |
  electrumx.js           |    91.04 |    75.56 |      100 |    98.36 |               445 |
  encryption.js          |    93.33 |       75 |      100 |      100 |                38 |
  generating.js          |       75 |       20 |      100 |    81.82 |             28,29 |
  hdnode.js              |    88.89 |    78.57 |     87.5 |     87.8 |39,146,147,606,636 |
  ipfs.js                |    97.87 |    93.18 |      100 |     97.8 |           214,436 |
  mining.js              |    60.61 |       15 |      100 |    71.43 |... 42,54,55,67,68 |
  mnemonic.js            |     56.8 |       44 |    71.43 |    61.39 |... 04,506,511,514 |
  ninsight.js            |    88.24 |    58.33 |      100 |    93.75 |                17 |
  openbazaar.js          |    83.33 |     62.5 |      100 |    90.91 |        64,118,208 |
  price.js               |     62.5 |        0 |      100 |    71.43 |             28,29 |
  raw-transactions.js    |    48.21 |       25 |      100 |    51.92 |... 23,349,351,352 |
  schnorr.js             |    23.81 |      100 |     5.88 |    23.81 |... 86,490,494,498 |
  script.js              |      100 |      100 |      100 |      100 |                   |
  socket.js              |    11.76 |        0 |        0 |    15.38 |... 16,21,23,24,25 |
  transaction-builder.js |    76.47 |    77.78 |       70 |    78.79 |... 58,62,63,64,66 |
  util.js                |    55.56 |     12.5 |      100 |    58.82 |... 87,89,92,94,95 |
  wallet.js              |     5.88 |      100 |        0 |     5.88 |... 57,61,65,69,73 |
 src/slp                 |    87.15 |    66.47 |      100 |    88.97 |                   |
  address.js             |    96.15 |    83.33 |      100 |    95.83 |            59,105 |
  ecpair.js              |      100 |      100 |      100 |      100 |                   |
  nft1.js                |    83.75 |    65.22 |      100 |    83.33 |... 15,316,405,406 |
  slp.js                 |    94.44 |       75 |      100 |    94.44 |                23 |
  tokentype1.js          |    86.54 |    78.95 |      100 |    86.73 |... 80,281,429,430 |
  utils.js               |     85.5 |    62.39 |      100 |    89.73 |... 37,848,849,851 |
-------------------------|----------|----------|----------|----------|-------------------|

Seem's like something has changed in one of my updates, not sure what compile2 is used for since I am not using those features from this library.

Thanks!

Hi @SeedyROM, I appreciate the PR, but I can't prioritize reviewing the code at the moment. We're preparing for the BCH hard fork on November 15th and the next round of grants for the PSF. If you'd like to join our Telegram channel and tell us more about your use case, we'd love to hear about it.

The encode test that failed is absolutely critical for generating SLP transactions and other OP_RETURN based transactions. So this PR can not be accepted unless that test is fixed.
Overall though, I agree with your amassment that the dependencies should be npm libraries and not GitHub repos.
I need to look at those dependencies closer. I'm not sure what the web3 dependency is doing. That had something to do with Badger Wallet, and it should probably be taken out.
The christroutner repo is mine, so I'm not worried about it getting hijacked. There are occasionally issues around Schnorr signatures, Script contracts, and OP_RETURN data that causes me to go in and edit that library.


fix: StructArray.from_arrays takes Iterable

I used an Iterable instead of a Sequence (as per the docs) because the code only uses __iter__
 and Iterable is what is being used elsewhere in the stubs.
resolves #42

resolved by #35


Missing a javaDoc Comment

Summary
when I ran the checkstyle i get this error "Missing a JavaDoc Comment. Anyone know how to resolve this problem?
Description
Describe the problem and code snippets if necessary
Screenshots (if any):
Insert Images here if necessary

I ignored it since we are not required to include Java Documentation for the project

yeap, i think we can ignored it, thats what i did too

I asked my TA before, he said can just ignore cos the previous projects required them to write JavaDocs but it has been removed for this sem

Hello! I think this checkstyle error will pop up when some of your classes are declared as Public classes, but should be nested inside the relevant packages instead. Maybe you can check to see and change the access modifiers of your classes accordingly.
Hope this helps!

I just ignored it as i received the same error, hopefully i do not get penalised

Hi, I ignored it when I got the error on terminal as well. I think it should be okay since Codecrunch did not say anything about it!

Hi, I got the same error but I think we can ignore it since codecrunch didnt raise any issues on this & java docs isn't required for the project

Hi, I experienced the same issue. But it is okay to ignore it because codecrunch didn't raise any issues

i asked my TA, she said can just ignore the warning :)

there is no need for javadocs comments for this proj :)

just submit without javadocs

you can ignore it! follow CC's error msg


fixes maybstd imports for no_std on thumbv6m-none-eabi

I'm having a tiny bit of trouble getting my Cargo workspace to completely work with patches, paths, a 0.0.0 version for this library and a few other build dep things, but I think this fixes the inability to use borsh in no_std b/c a failure to use alloc::sync on line 32.
The remainder of the changes are just in keeping with the rest of the codebase, which only depends on rc and sync when cfg(feature = "rc")

alloc::sync does exist, so not sure what could be the issue there, but I did notice that rc was broken when used without std feature. I fixed that specific problem with #88. Were you using this with std feature not set and rc set? If not, can you produce a repro of this error or more details?

Hmmm, this could be unique to my env. I'm currently blocked just adding borsh as a dependency to the sample Ledger Nano S rust app.
Cargo.toml (simplified):
[dependencies]
borsh = { version = "0.9", default-features = false }
nanos_sdk = { git = "https://github.com/LedgerHQ/ledger-nanos-sdk" }
nanos_ui = { git = "https://github.com/LedgerHQ/ledger-nanos-ui" }

and this is the error produced:
> cargo build
    Updating crates.io index
   Compiling borsh-derive-internal v0.9.3
   Compiling borsh-schema-derive-internal v0.9.3
   Compiling borsh-derive v0.9.3
   Compiling borsh v0.9.3
error[E0432]: unresolved import `alloc::sync`
  --> /home/sunnyg/.cargo/registry/src/github.com-1ecc6299db9ec823/borsh-0.9.3/src/lib.rs:32:56
   |
32 |     pub use alloc::{borrow, boxed, format, rc, string, sync, vec};
   |                                                        ^^^^ no `sync` in the root

For more information about this error, try `rustc --explain E0432`.
error: could not compile `borsh` due to previous error

But to answer your questions, as you can see above I've disabled all features, under the assumption that you couldn't use rc without std. If that's the fix, then my bad!

So that's not the fix - I'm getting substantially more errors, including the aforementioned one. The target is thumbv6m-none-eabi - is that the culprit? I'm not too familiar with embedded rust.

So that's not the fix - I'm getting substantially more errors, including the aforementioned one. The target is thumbv6m-none-eabi with a recent nightly toolchain - could that be the culprit? I'm not too familiar with embedded rust.

Hmm, yeah so this is just a target-specific issue. I think support for this can be added, but I don't personally know the cleanest way to exclude Arc things just for this family of architectures. Also not sure if we are open to very specific configurations in this repo, we can open an issue but you might want to just fork where you remove the incompatible parts as a stopgap solution for now.

where they're exported in maybestd, they aren't behind any flag, which (at least for me) was unexpected. Put another way, feature = "std" seems "broken" if rc's feature-gates are ignored by the std/no_std.

So with respect to the users of borsh::maybestd, I do understand this as a breaking change, but with regards to the actual usage of the library, this change doesn't seem to break anything (though correct me if I'm wrong, I might be missing something).

Ack that this is probably correct, but I'm just being cautious since I'm not sure of all cases the lib is used. cc @matklad @frol any objections to putting rc and sync behind the rc feature flag?

any objections to putting rc and sync behind the rc feature flag?

No objections


fix: use None check instead of falsy

Description

Previously I raised a fix to the editor implementation in the OH repo, but forgot to fix it here: https://github.com/All-Hands-AI/OpenHands/pull/4705. This seems to degrade the performance on swe-bench a bit and causes the agent to stuck in the loop.

@ryanhoangt Can you add new testcases to make sure this doesn't appear again in the future? I've been trying to run some eval and was puzzled by the perf degradation 😅 good catch!


Auto configuration of memcached, hazelcast and ehcache registries

This pull attempts to auto configure ticket registries and provides property placeholders to make sure overlays are unnecessary, and module configuration is enabled automatically. Changes are:

Additional props for hazelcast. It is auto reloaded when the module is used.
Additional props for ehcache. It is now configured auto reloaded when the module is used.
Additional props for memcached. It is now configured auto reloaded when the module is used.

Jboss shall be removed.

A big +1. So much easier for configuration.
I see a broken test on SAML in the build.

Thanks. Pull still in progress. I am working on JPA ticket registry, to also refactor tests. That one is particularly tricky, because one could use the JDBC module without using the registry. So I'll review and update the full when it's done.

I am going to defer the work on JPA ticket registry for now. We do have to move it to another module to make auto-config work because right that that feature is bundled with a bunch of authN handlers and choose one without the other is difficult. Another module also has the advantage that it allows us to clean up the ticket core API to remove JPA annotations.


Module build failed {filename} is not part of the compilation at Angular 5.0.0

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

angular: "5.0.0"
angular/cli: "1.5.0-rc.8"
ngtools/webpack: "1.8.0-rc.8"
Repro steps.

webpack building with AngularCompilerPlugin
The log given by the failure.
Module build failed: Error: {filename} is not part of the compilation.

Desired functionality.

Mention any other details that might be useful.

Hello.
I am trying to update angular v5.0.0 released yesterday.
But the error is occurred when i try to compile with AngularCompilerPlugin plugin.
The webpack throws an error as
Module build failed {filename} is not part of the compilation
// ex) filename => app.main.ts

my tsconfig.json is as follows
// tsconfig.json
{
  "version": "1.0.0",
  "compileOnSave": true,
  "compilerOptions": {
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "sourceMap": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "noImplicitAny": false,
    "removeComments": false,
    "suppressImplicitAnyIndexErrors": false,
    "strictNullChecks": false,
    "typeRoots": [
      "node_modules/@types"
    ]
  },
  "lib": [
    "dom",
    "dom.iterable",
    "es2015",
    "scripthost"
  ],
  "exclude": [
    "node_modules",
    "Content/scripts/app.aot",
    "Content/scripts/app/typings",
    "Content/scripts/app/app.main.aot.ts",
    "Content/homeworks"
  ]
}

And this is my AngularCompilerPlugin option
new AngularCompilerPlugin ({
    tsConfigPath: rootDir + '/tsconfig.json',
    entryModule: contentDir + '/scripts/app/app.module#AppModule'
}),

Other webpack configuration is not changed.
And everything worked well until I replaced the AotPlugin.
So I tried to fix the plugin's code directly to fix the problem.
And there's one question in this process,
If there is no key in object, it will exist as undefined.
In compiler_host.ts, Why does stats compare to null ?
const stats = this._files[fileName];
if (stats == null) {
    ....
}

I changed above code as follows for quickfix the issue
const stats = this._files[fileName];
if (!stats) {
    ....
}

It's worked, However i think it is not the best practice.

I will put a comment with my personal account to check the progress of the issue.

Hello, this issue is a duplicate of https://github.com/angular/angular-cli/issues/8284#issuecomment-341417325. You'll find more information there about what this error means.


feat: architecture refactor

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Resolves: #598
What is the new behavior?

Refactored to the new architecture

Does this introduce a breaking change?

[ ] Yes
[ ] No

scarb fmt check


Uncover Dark iOS 12.1.1 iPhone 8 PLUS 64gb

Sent with GitHawk

Was jailbroken with uncover 3.1.2 until I started getting random reboots and ran into a crash
So I decided to try out uncover dark to see if it will fix the problem but no 😕
I still had tweaks installed I dont know if that’s causing the problem. Did the root recover and nothing still
Sent with GitHawk

Yeah this even happened in my iPhone 5s :(

You guys have some missing dependencies. Switching jailbreaks could cause issues, however as unc0ver dark is branched off of un0ver, it should largely use the same files. Try turning on airplane mode on the OP. I’m not the developer, but if the code has inline documentation at the least, I may be able to give you more information. I would restore rootfs. Do a clean install with the latest version of unc0ver. Run the app after a fresh reboot. Turn on airplane mode, wait 30-60 seconds. Make sure Power is plugged in, low power mode is not on, and dnd probably doesn’t need to be on. There’s a variety of things to try.

Also note, unc0ver dark uses a different apt version, and attempts to make unc0ver compatible with Sileo. It’s not quite the same. BigBoss repo isn’t supported by Sileo. Without knowing more, I could not say that they are interchangeable, but it doesn’t seem so on it’s face. I may be wrong. Don’t use them both w/o the technical knowledge to know the apt are compatible.


Uninstalling does not stop application

When you uninstall the program it does not stop the running application and says "some files could not be removed". This results in the application entry getting removed, but the application still remains (and actually started on boot).
Happens on Windows 8.1
Solution: Stop application before uninstalling.

Thanks for your report. Please reopen the issue in https://github.com/stevschmid/track-o-bot if it still persists.


Topic/sagrawal/temp pa 1671 temp

What this PR does / why we need it:
Which issue(s) this PR fixes (optional)
Closes #
Special notes for your reviewer:

close


Added feedback form and sitemap to the footer

I have added the feedback form and the sitemap to the footer as requested in issue.
This is the sample image from the issue, showing how the footer should look:

BEFORE MAKING CHANGES:

AFTER MAKING CHANGES:

MOBILE VIEW:

NOTE: In order to link the form in footer to your formspree form

go to formspree
2)register and create a form
3)copy the form code of your form
4)create a .env file in frontend folder
5)inside it write as follows: REACT_APP_FORMSPREE_FORM_FORM_CODE =  your-form-code-here

After doing the above the form should work.
I request the maintainer to please mark this as 'hacktoberfest-accepted' while merging this. I want this contribution to count for hacktober fest.
Thanks!

Summary by CodeRabbit

New Features

Introduced a responsive Footer component with a feedback form for user submissions.
Added a ResponseSuccess component to display confirmation messages upon successful feedback submission.
Integrated the Footer component into the Home page layout.

Configuration Updates

Added new environment variables for forms, GitHub repository URL, and contact email.

Dependencies

Added new dependencies: @formspree/react and dotenv.
Introduced dotenv-webpack as a development dependency.

Please let me know if there are any changes that need to be made.

Please let me know if there are any changes that need to be made.

Can you address the PR review comments by the coderabbitai?

made changes suggested by coderabbitai
👨💻 Changes made

Used Material-UI's styling solution instead of inline styles.
Used function declaration and added PropTypes.
Improved sitemap maintainability by using constants or environment variables for URLs
Improved the feedback form by removing redundant success message and disabling the submit button while the form is being submitted

Changes not made

Consider using a translation function for the success message.
I did not used translation functions as i feel its beyond the scope of this PR. The project does not use tranlation functions as of now. Hence i haven't used translation functions despite coderabbitai having mentioned it.
Footer component is missing the sitemap element.
This was a wrong analysis by coderabbitai. The footer component does have the sitemap. It can clearly be seen in the screenshots attached above.

Note to reviewers
If there are any changes that need to be made please let me know.
If merging please add the  label. I would be grateful.
📷 Screenshots
ADDED MUI'S STYLING SOLUTION INSTEAD OF INLINE STYLES:

ADDED PROP TYPES:

USED CONSTANTS AND ENVIRONMENT VARIABLES FOR SITEMAP MAINTAINABILITY


How to download shapenet watertight meshes dataset?

Hi, how can I download shapenet watertight meshes dataset? I failed to find the download link in code or doc.

 In fact, only vanilla shapenet data is needed to generate watertight meshes.
But maybe the doc can be improved.

Hi @jianglongye I had a same question. could I ask from where you know "only vanilla shapenet data is needed to generate watertight meshes." That means no need to preprocess shapenetv2 to be watertight? Thank you!

It's too long ago. As far as I can remember, at that time, kaolin can take the vanilla shapenet data as input and output watertight meshes. There is a build-in preprocess for shapenet in kaolin (at that version).

I see. Thanks! But it seems current kaolin does not have this kind of preprocessing? So maybe no need to convert shapenet v2 to watertight?

It depends on which task your are working on.

I see. What i am confused is that in some paper it said it used kaolin to get a watertight mesh but current kaolin does not have that function (if I am not wrong?) (of course there are other GitHub repo like manifold... that can use to get watertight

Yeah, from the latest doc, I can only find this check_sign function which is related to watertight.
Maybe you still need other ways to convert the shapenet mesh to watertight, or maybe the authors can give different solutions.


[Help]: Velocity Map from sbas_parralell

Hi,
I'm running a long time-series using the previous version of pygmtsar, because all my data progress is saved in the last version.
Last I ran sbas_paralell because sbas.sbas has an issue, and now I have disp grid for each date.
I'm trying to find the code to calculate the velocity displacement from these files, and cannot find any.
Could you please help explain how to produce the velocity grd from disp grd in pygmtsar?
OS and software version

OS: Ubuntu 20.04.5 LTS
PyGMTSAR version: pygmtsar-v2023.3.11

Depending on your PyGMTSAR version, you can try the functions sbas.trend() and sbas.stl(). If these are not available, the linear regression trend can easily be computed as in the trend() function:
    def trend(self, data, deg=1):
        import xarray as xr

        trend = xr.polyval(data.date, data.polyfit('date', deg).polyfit_coefficients)
        return trend

Divide the LOS, vertical, or east-west displacement trend difference (last minus first value per pixel) by your time interval in years to get the velocity in mm/year.


Added usage to the readme, tidied things up

Made a few simple updates to the readme

LGTM. @thegodone : you ok with merging this?

good to me

Thanks @bmiles !


Using Votenet for Object Detection

I was wondering why
wget https://raw.githubusercontent.com/RoboticVisionOrg/benchbot_examples/master/semantic_slam_attempt/votenet_benchbot.py
404: Not Found

That's an issue in the tutorial; the link there is old.
Thanks for catching that for us @ernestoMM94! I've fixed the link in the tutorial, which should now point to the correct location


Backend process should be able to control the environment variables

This is so the instance can be configured and modified from a UI based admin interface.

API for this merged in #6473.


Firefox display stare very big

the star viewed in chrome very well :

but in firefox 57.0.1  displayed very big :

I'm forget to set unit for starWidthAndHeight and starSpacing and after sees your codepen and set the unit fixed in firefox too :

                    <StarRatings
                      rating={this.rateCalc('quality')}
                      starWidthAndHeight='20'
                      starSpacing='4'
                      isAggregateRating={false}
                      numOfStars={ 5 }
                    />

chaned to :

                    <StarRatings
                      rating={this.rateCalc('quality')}
                      starWidthAndHeight={'20px'}
                      starSpacing={'4px'}
                      isAggregateRating={false}
                      numOfStars={ 5 }
                    />

@hemedani Did that change fix it? I am not seeing any issues in firefox...

yes when setting unit everything working fine.
please set a px for a default unit and when we forget to set that automatically fixing  unit for us
tank you


Update README to match the new structure

Tried to reflect and explain the new structure in the README file.
I fear it might not be "simply explained" and/or compelling enough for people coming on github.com/fsprojects/Fabulous

Changed the README based on the feedback I received from some C# people.
The README has been split in different files :

Primary README is only a brief explanation of what we can find on this repo
Each part has its own README (Fabulous has a FABULOUS.md instead)
Roadmap has been split and integrated inside its corresponding README


Cannot access OpenCamera subfolder

Describe the bug
As I use both phone Camera app and OpenCamera, in DCIM folder, I have a Camera and OpenCamera folder. The OpenCamera pictures aren't visible in aves.
To Reproduce
Steps to reproduce the behavior:

Install OpenCamera
Set image saving to default DCIM/OpenCamera
Take some photos with OpenCamera
Try to see them in aves

Expected behavior
Another album OpenCamera should be displayed, like most other photo gallery app.

Most likely that app creates media items but doesn't register them with the Android Media Store. Feel free to file an issue on their repo.


Bug: Only one #Label can be added via subject field

Create a new info item
Insert in subject for example "subject1 #Label1 #Label2"
Save item

Only one label is recognized and the other stays in subject

The same applies to adding labels for topics via the add topic dialog.

1d


No matching versions for v2 when fetching package with go get

This is the error I get when I try to run the command specified in the README.

@ayoisaiah thanks for catching, updated readme.
Additionally you can use add version as following:
go get<EMAIL_ADDRESS>


Failing to extract span context for Traefik Jaeger Tracing

Do you want to request a feature or report a bug?
Bug
What did you do?
I set up Traefik, attached Jaeger as Tracer and some other services proxied behind Traefik.
Stack: Docker Compose, File + Docker provider.
I can not see if this is a misconfiguration. I lean to believe this might be a bug. So far everything is reachable from the other end. I checked connections via wget -qO- tracer:<port>/endpoints and vice versa to the controller. If I can provide further details, please let me know. Both GitHub example repos as well as docs are rare, so it's not that easy to compare my stack with others. Thanks in advance!
What did you expect to see?
Traces that combine spans which show the full request cycle (Traefik > App > Traefik).
What did you see instead?
Separated traces of Traefik and Applications. And errors in the Traefik error.log that explain that the (…) SpanContext was not found in the Extract carrier.
Output of traefik version:
/etc/traefik # traefik version
Version:      2.3.6
Codename:     picodon
Go version:   go1.15.6
Built:        2020-12-17T16:34:27Z
OS/Arch:      linux/amd64

Edit: I updated to latest and the problem persists:
 # traefik version
Version:      2.4.2
Codename:     livarot
Go version:   go1.15.7
Built:        2021-02-02T17:20:41Z
OS/Arch:      linux/amd64

What is your environment & configuration?

macOS 11.1
* Docker Compose docker-compose version 1.27.4, build 40524192
Jaeger as Container jaegertracing/all-in-one:1.21.0
Traefik as Container traefik:2.4.2

Traefik Compose
# Network
ROOT_DOMAIN=example.test
DEFAULT_NETWORK=traefik

---
version: '3'
services:
    image: "traefik:2.4.2"
    hostname: "controller"
    restart: on-failure
    security_opt:
      - no-new-privileges:true
    ports:
      - "443:443"
      - "80:80"
      # The Web UI (enabled by --api.insecure=true)
      - "8080:8080"
      - "8082:8082"
      - "8083:8083"
    networks:
      - default
    working_dir: /etc/traefik
    volumes:
      - /private/etc/localtime:/etc/localtime:ro
      - ${PWD}/controller/static.yml:/etc/traefik/traefik.yml:ro
      - ${PWD}/controller/dynamic.yml:/etc/traefik/dynamic.yml:ro
      - /var/run/docker.sock:/var/run/docker.sock:ro
      - cert-storage:/usr/local/share/ca-certificates:ro
      - ${PWD}/logs/traefik:/var/log/traefik

volumes:
  cert-storage:
    driver_opts:
      type: none
      o: bind
      device: ${PWD}/certs/certs

networks:
  default:
    external: true
    name: ${DEFAULT_NETWORK}

Traefik Conf
# static.yaml (Traefik conf)
debug: true

log:
  level: DEBUG
  filePath: /var/log/traefik/error.log
  format: json

serversTransport:
  insecureSkipVerify: true

api:
  dashboard: true
  insecure: true
  debug: true

providers:
  docker:
    exposedByDefault: false
    swarmMode: false
    watch: true
    defaultRule: "Host(`{{ normalize .Name }}.example.test`)"
    endpoint: "unix:///var/run/docker.sock"
    network: traefik
  file:
    filename: /etc/traefik/dynamic.yml
    watch: true

tracing:
  serviceName: "controller"
  spanNameLimit: 250
  jaeger:
    samplingType: const
    samplingParam: 1.0
    samplingServerURL: http://tracer:5778/sampling
    localAgentHostPort: <IP_ADDRESS>:6831
    gen128Bit: true
    propagation: jaeger
    traceContextHeaderName: "traefik-trace-id"
    collector:
      endpoint: http://tracer:14268/api/traces?format=jaeger.thrift

Jaeger
---
version: '3'
services:
  tracer:
    image: "jaegertracing/all-in-one:1.21.0"
    hostname: "tracer"
    command:
      - "--log-level=info"
      - "--admin.http.host-port=:14269"
      - "--query.ui-config=/usr/local/share/jaeger/ui/conf.json"
    environment:
      SPAN_STORAGE_TYPE: memory
    restart: on-failure
    security_opt:
      - no-new-privileges:true
    expose:
      # (default) host:port for the UDP server
      #   @see --processor.zipkin-compact.server-host-port
      - 5775/udp
      # (default) Agent: accept jaeger.thrift in compact Thrift protocol used by most current Jaeger clients
      #   @see --processor.jaeger-compact.server-host-port
      - 6831/udp
      # (default) Agent: accept jaeger.thrift in binary Thrift protocol used by Node.js Jaeger client
      #   @see --processor.jaeger-binary.server-host-port
      - 6832/udp
      # (default) Agent: serve configs, sampling strategies, @see --http-server.host-port
      - 5778
      # (default) The host:port (e.g. <IP_ADDRESS>:14250 or :14250) of the query's gRPC server
      #   @see --query.grpc-server.host-port
      - 14250
      # (default) Collector: can accept spans directly from clients in jaeger.thrift format over binary thrift protocol
      - 14268
      # Collector: admin port: health check at `/` and metrics at /metrics
      - 14269
      # Agent: admin port: health check at `/` and metrics at /metrics
      - 14271
      # (default) Query Service & UI: The host:port (e.g. <IP_ADDRESS>:14268 or :14268) of the query's HTTP server
      #   @see --query.http-server.host-port
      #   + (http) `/api/*` endpoints and Jaeger UI at `/`
      #   + (grpc) Protobuf/gRPC QueryService
      - 16686
      # Query Service & UI: admin port: health check at `/` and metrics at /metrics
      - 16687
    volumes:
      - /private/etc/localtime:/etc/localtime:ro
      - ${PWD}/tracer/conf:/usr/local/share/jaeger
      - ${PWD}/logs/jaeger:/var/log/@TODO
      - cert-storage:/usr/local/share/ca-certificates
    networks:
      - default
    labels:
      - "traefik.enable=true"
      - "traefik.docker.network=${DEFAULT_NETWORK}"
      # Admin UI router
      - "traefik.http.routers.tracer-router.rule=Host(`tracer.$ROOT_DOMAIN`)"
      - "traefik.http.routers.tracer-router.entrypoints=https"
      - "traefik.http.routers.tracer-router.tls=true"
      - "traefik.http.routers.tracer-router.tls.options=default"
      - "traefik.http.routers.tracer-router.service=tracer"
      # Service/ Load Balancer
      - "traefik.http.services.tracer.loadbalancer.passhostheader=true"
      - "traefik.http.services.tracer.loadbalancer.server.port=16686"
      - "traefik.http.services.tracer.loadbalancer.server.scheme=http"

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
Single entry
{
  "level":"debug",
  "middlewareName":"tracing",
  "middlewareType":"TracingEntryPoint",
  "msg":"Failed to extract the context: opentracing: SpanContext not found in Extract carrier",
  "time":"2021-02-02T23:16:51+01:00"
}

Logs
{"level":"debug","msg":"Reporting span 79286f384d3b9b912c4b92ad02dc706a:2c4b92ad02dc706a:0000000000000000:1","time":"2021-02-03T12:02:23+01:00","tracingProviderName":"jaeger"}
{"level":"debug","middlewareName":"tracing","middlewareType":"TracingEntryPoint","msg":"Failed to extract the context: opentracing: SpanContext not found in Extract carrier","time":"2021-02-03T12:02:29+01:00"}
{"level":"debug","middlewareName":"auth@file","middlewareType":"BasicAuth","msg":"Authentication succeeded","time":"2021-02-03T12:02:29+01:00"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:461c666cd634ffd1:41ddd3c10c662912:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:41ddd3c10c662912:5364c358cbd6db57:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:5364c358cbd6db57:765ff1b2dfd4dd01:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:765ff1b2dfd4dd01:6e3cc63d3dc5a43a:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:6e3cc63d3dc5a43a:4c4ecc40ce9b68e1:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:4c4ecc40ce9b68e1:5a8e7785199b7274:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:5a8e7785199b7274:223b0ede9038d936:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 388aa1f4282e0fb2223b0ede9038d936:223b0ede9038d936:0000000000000000:1","time":"2021-02-03T12:02:29+01:00","tracingProviderName":"jaeger"}
{"level":"debug","middlewareName":"tracing","middlewareType":"TracingEntryPoint","msg":"Failed to extract the context: opentracing: SpanContext not found in Extract carrier","time":"2021-02-03T12:02:33+01:00"}
{"level":"debug","middlewareName":"auth@file","middlewareType":"BasicAuth","msg":"Authentication succeeded","time":"2021-02-03T12:02:33+01:00"}
{"level":"debug","msg":"Reporting span 02a9543f2fc252557b4b2dcaf71bac7d:44695a42b9926466:578074b65aa32461:1","time":"2021-02-03T12:02:33+01:00","tracingProviderName":"jaeger"}
{"level":"debug","msg":"Reporting span 02a9543f2fc252557b4b2dcaf71bac7d:578074b65aa32461:2ca148487ec0313b:1","time":"2021-02-03T12:02:33+01:00","tracingProviderName":"jaeger"}

Hello @franz-josef-kaiser,
The issue is quite old and the Jaeger client version has been (updated in v2.6.7)[https://github.com/traefik/traefik/pull/9000], so I am wondering if the issue is still occurring, did you try with a version of Traefik >=V2.6.7?

Hi @rtribotte!
You are right in that this issue is quite old. I am sorry, but I did not test this for two years.

Hello @franz-josef-kaiser,
I wanted to try out the reproduction case you have shared with the latest v2.9, but I noticed that the Jaeger service is referencing a config file that you didn’t share:
- "--query.ui-config=/usr/local/share/jaeger/ui/conf.json"

Can you please share it with us or make the reproduction case more minimal?
Also, does the client request contains already tracing data? If so, can you please also share how to do the client request?


GitHub actions for deployment

Task linked: CU-85zrtkha7 CI/CD for automatic change log, PYPI release and GitHub Release


Add skills PPLTool execution cluster setting

Description
Add skills PPLTool execution cluster setting
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
Version
List the OpenSearch version to which this PR applies, e.g. 2.14, 2.12--2.14, or all.
Frontend features
If you're submitting documentation for an OpenSearch Dashboards feature, add a video that shows how a user will interact with the UI step by step. A voiceover is optional.
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@zane-neo - Is this new for 2.16 or something that was missed before?

@zane-neo - Is this new for 2.16 or something that was missed before?

This is for 2.16.

Please hold on the merge, confirming if this PR is necessary.

This is not necessary since the cluster setting code has been reverted: https://github.com/opensearch-project/skills/pull/381, closing it now.


SW-1252 Remove viability test selection

Previously, we'd planned to let users select a viability test whose result would
be used as the accession's viability percentage; once a test was selected, the
accession's viability% would no longer be directly editable. The server thus
needed to keep track of which test, if any, was selected.
We've decided to change the behavior of the app such that the user can edit the
accession's viability% at any time, and finishing a test offers to copy the test's
percentage to the accession.
Since there's no more concept of selecting a test, we can get rid of the support
for test selection that was added in commit 5c239e5 and the support for searching
whether or not a test is selected, which was added in commit 939f3db.

Current dependencies on/for this PR:

main

PR #583   👈

This comment was auto-generated by Graphite.


Missing configuration texts in getter node in version 0.8.0

can confirm ... it works well with 0.14.6. sorry my mistake.


Added vue-responsive-video-background-player

Idea based on Tesla's Website. Demo Page: https://avidofood.github.io/vue-responsive-video-background-player/

@Scrum I'm still new with Github, so I just resolved the conflict. Anything else I should do?


Button bounce up is lagging.

Not sure if it's just me. But after you press and release, the button has a very noticeable lag when it bounces up.

Never mind. I've a double tap gesture on the parent view :)


Implement subscriptions to data pipes

It looks like Alan was using https://github.com/ReactiveX/RxPY for this functionality.

Consider two options:
Creating an Rx observable at either the api, webapi, or point objects.
Implement something like blinker and have signals emitted when conditions are met; for instance, when a value is updated, or when a new value is inserting into the Point object. To monitor new Point additions, the same thing can be done to monitor a Points mutable sequence's list object, which contains Points.

Signaling makes more sense to me... it would be easier to create a function that receives a list of Points, and subscribe to a named signal called  something like 'point.webid+stream', i.e. "E0e5R4zUACHUOxUKQtNYteqg5pBaQDDa5BGAxAAd2Lck3QRU1FXFBPV0VSXEFUTEFOVEEgREFUQSBDRU5URVJcU0VSVkVSIFJBQ0sxXElPTiA2MjAwIFBPV0VSIE1FVEVSMQ_value"
we can then have the setattr call check if a value changes, and emit a signal with the correct webid and value, with the option of passing the point object as a data kwarg as well.


Comiling error with Portenta H7

Hello,
Although the documentation stated that it is compatible with Portneta H7, I received a compiling error.
Is there anything I can make, or the STM32H747X is not supported?
Thanks

This library is for SAMD boards, such as SAMD21/SAMD51, while Portenta H7 is STM32H7 using MBED OS.
Therefore, the Portenta H7 will never be supported by this library.
Even the STM32_TimerInterrupt library doesn't not support that Portenta H7 boards.

I just have a look at the samd_timerinterrupt documentation you mentioned. It is written by someone else and certainly is mistaken and must be corrected.

You can now use my new Portenta_H7_TimerInterrupt Library

Initial Releases v1.2.1

Initial coding to support Portenta_H7 boards such as Portenta_H7 Rev2 ABX00042, etc., using ArduinoCore-mbed mbed_portenta core
Permit up to 16 super-long-time, super-accurate ISR-based timers to avoid being blocked
Using cpp code besides Impl.h code to use if Multiple-Definition linker error.
Bump version to v1.2.1 to sync with STM32_TimerInterrupt library


Cannot run drive.py

Hi, thank you for the code. I have the following issue:
Command used:
python drive.py model.json
Error:
Using Theano backend.
Can not use cuDNN on context None: Disabled by dnn.enabled flag
Mapped name None to device cuda: GeForce GTX TITAN X (0000:01:00.0)
(13106) wsgi starting up on http://<IP_ADDRESS>:4567
(13106) accepted ('<IP_ADDRESS>', 53040)
<IP_ADDRESS> - - [06/Sep/2018 15:54:56] "GET / HTTP/1.1" 404 366 0.006205
Only change I made is in my theanorc file to disable the dnn flag.
Please let me know how exactly to run this code.

Seems like the issue has been resolved now; @LeenaShekhar feel free to provide your solution in comments.


Unable to use Stubber for S3 generate_resigned_url

Describe the bug
This works:
>>> import boto3
>>> client = boto3.client("s3")
>>> client.meta.method_to_api_mapping['put_object']
'PutObject'
>>> client.meta.service_model.operation_model('PutObject')
OperationModel(name=PutObject)

whereas this does not:
>>> client.meta.method_to_api_mapping['generate_presigned_url']
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'generate_presigned_url'
>>> client.meta.method_to_api_mapping.get('generate_presigned_url')
>>> # Returns None.

As a consequence, using
with Stubber(client) as stubber:
    stubber.add_response("generate_presigned_url", response, expected_params)

fails, because this call in _add_response()
https://github.com/boto/botocore/blob/2baa022100f8fdb4d65fe3b902ac40b415bbeebe/botocore/stub.py#L249-L250
makes operation_name = None and in turn
https://github.com/boto/botocore/blob/2baa022100f8fdb4d65fe3b902ac40b415bbeebe/botocore/stub.py#L406-L407
raises an error:
  File "/.../.venv/lib/python3.10/site-packages/botocore/stub.py", line 410, in _validate_operation_response
    operation_model = service_model.operation_model(operation_name)
  File "/.../.venv/lib/python3.10/site-packages/botocore/utils.py", line 1421, in _cache_guard
    result = func(self, *args, **kwargs)
  File "/.../.venv/lib/python3.10/site-packages/botocore/model.py", line 364, in operation_model
    raise OperationNotFoundError(operation_name)
botocore.model.OperationNotFoundError: None

Expected Behavior
Stubber adding the expected response to its queue.
Current Behavior
See above.
Reproduction Steps
See above.
Possible Solution
I suspect that the list of service methods in data/ may not contain all actual service methods.
Additional Information/Context
No response
SDK version used
1.26.104
Environment details (OS name and version, etc.)
macOS Ventura 13.2.1 (22D68), Python 3.10.10

Hi @jenstroeger, thanks for reaching out.  The reason this is happening is because generating a presigned url is not actually an API call, but instead done locally.  If you enable debug logs and generate a presigned url, you can see there isn't actually a response since a request isn't being made.  You can also check the S3 documentation to see that generate_presigned_url is not listed.  Since this is the case, you shouldn't need to use a stubber for this method.  Hope that helps, and please let me know if you have any follow-up questions.

@RyanFitzSimmonsAK oh so for testing I can just call that Boto3 function then, good to know 👍🏼
Let me try that out first, and then close this issue.

Looking at the documentation, I believe that is correct, since a request isn't being made.

Hi @jenstroeger; is this issue still a problem for you?

@RyanFitzSimmonsAK I think it’s ok. If we have problems I’ll reopen, but for now I think we’re good. Thank you for your support 👍🏼


ghci.conf script for :hlint fails on file path containing a space

Fix that works for me is to just show the string passed to hlint in order to insert quotes:
 ...hlint \" ++ (show . unwords) ...
Tested on mac (not sure if this is system dependent).

Which ghci.conf script are you referring to?

Isn't the show/unwords the wrong way round? I'd have expected the right version to be:
        ,":cmd return $ \":! hlint \" ++ unwords (map (show . takeWhile (/=',') . drop 2 . dropWhile (/= '(')) $ lines hlintvar1) ++ \" \" ++ " ++ show extra

Does that work for you? Important to show each individual file separately, rather than putting them all together in a single quote (which only works if you have one file loaded). If you confirm that works I'll make the change.


ref: exp-related updates in other to cmds, etc.

Matches certain changes in #2200 and #2243 (mainly for consistency in terminology, structure, etc.).

[x] Change base to master before merging (after #2243).

Need a review pls 🙂


Expose detailed usage information for external command invocations (including system time, user time)

The time command can only measure the total elapsed time since it takes an arbitrary closure.
However, when running external commands, the OS kernel can provide more detailed usage information. This is exposed in os.ProcessState.SysUsage, which in turn is syscall.Rusage. On UNIX, this struct provides much more information than timing. On Windows, this is mostly just timing information.
This can be implemented as a command measure-external, which (like time) takes a callback that gets the usage information:
measure-external &on-end=[u]{ ... } external-name args...

@xiaq, What you said about timing external versus builtin commands is not entirely true. It is only approximately true because the os.ProcessState.SysUsage() Go function only reports information for external commands. I expect to find a lower level implementation that does provide stats for the current process as well as our child processes.
When I use time on a posix shell function (elvish lambda) I both expect to see the time spent by the function executing the scripting language of the shell as well as that of any external commands it runs. I wouldn't expect to have to time each external command individually.
Given how resource usage data is tracked on UNIX kernels the idea here doesn't actually work in practice unless you time a single external command. If I have an elvish function with more than one external command and I preface each with the proposed command the behavior is subject to races. Specifically, it means that if you have a pipeline, measure-external ... | measure-external ..., one of those will produce incorrect data. That's due to the kernel updating the child rusage data when the external command terminates. But measure-external has to query the rusage data twice, before and after running the external command, to produce a delta. One of those "before" snapshots will now be incorrect as it won't include the data from the other external command.

@krader1961 I am not sure about the problem you are point out, but it's worthwhile reviewing why measuring external commands is different from measuring Go code. This is because a Unix kernel only provides resource measurement on a process granularity:

To measure external commands, the wait4 syscall [1] has a *rusage argument, which contains usage information for the process only.

To measure Go code, the only approximation calling getrusage on either the current process or current thread, none of which matches to an Elvish function call. Measuring the current process obviously over-measures since it includes everything else running in the Elvish process. Measuring the current thread also over-measures since multiple goroutines can be scheduled on the same thread.

So while accurate usage measurement on a process level is provided by the Unix kernel, accurate usage measurement of Elvish function call is not supported by the kernel; such support can only come from the Go runtime, which I don't think exist, and will likely never exist: goroutines are explicitly very lightweight, so I don't see the Go team adding any kind of accounting functionality to it (which adds overhead), and this is not something Elvish can easily add on its own.
[1] This is the Go binding, but it mirrors the C interface closely enough


Not all required components are registered in the ServiceCollection

.Net 8.0 (new project, not upgraded)
Functions Version 4 (Isolated)
Using the example you have in DarkLoop.Azure.Functions.Authorization.Isolated I get the following errors:

System.AggregateException : Some services are not able to be constructed (Error while validating the service descriptor '
ServiceType: DarkLoop.Azure.Functions.Authorization.FunctionsAuthorizationMiddleware
Lifetime: Singleton
ImplementationType: DarkLoop.Azure.Functions.Authorization.FunctionsAuthorizationMiddleware': Unable to resolve service for type 'DarkLoop.Azure.Functions.Authorization.IFunctionsAuthorizationProvider' while attempting to activate 'DarkLoop.Azure.Functions.Authorization.FunctionsAuthorizationMiddleware'.)

I had to fix it with a workaround by changing my startup to look like this:
.ConfigureFunctionsWebApplication(builder =>
{
    new FunctionsAuthorizationExtensionStartup()
        .Configure(builder);

    // Explicitly adding the extension middleware because
    // registering middleware when extension is loaded does not
    // place the middleware in the pipeline where required request
    // information is available.
    builder.UseFunctionsAuthorization();
})

The problem is that out of the box, the DI calls are missing for registering the internal concrete FunctionsAuthorizationProvider to the interface IFunctionsAuthorizationProvider.  The only place you have this registration is in the startup, which is not needed or used in the .net 8 functions where it uses top-level statements instead of a startup.
You also have a typo in the readme for isolated where AddFunctionsAuthenticationation should be AddFunctionsAuthentication
Thanks!

@teknofobia Thanks for the great feedback. I will work on these items.

@teknofobia I tried creating a new .NET 8 functions app and it works, which looks the same as the Isolated sample included in the repo (No startup for the app) as extensions are discovered by the framework by using the startup classes for extensions (WorkerExtensionStartup), not the app. The sample only uses top level statements in Program.cs.
Does your Program.cs differs from the one in SampleIsolatedFunctions.V4?

Thanks for looking into it.
Here is my program.cs
var host = ApplicationConfiguration.InitializeHostBuilder()
    .Build();

host.Run();

And this is the method it calls for the configuration
public static IHostBuilder InitializeHostBuilder()
 {
     return new HostBuilder()
         .ConfigureFunctionsWebApplication(builder =>
         {
             new FunctionsAuthorizationExtensionStartup()
                 .Configure(builder);
             // Explicitly adding the extension middleware because
             // registering middleware when extension is loaded does not
             // place the middleware in the pipeline where required request
             // information is available.
             builder.UseFunctionsAuthorization();
         })
         .ConfigureFunctionsWorkerDefaults()
         .ConfigureAppConfiguration((context, builder) =>
         {
             builder.SetBasePath(context.HostingEnvironment.ContentRootPath)
                 .AddJsonFile("appsettings.json", optional: true, reloadOnChange: false)
                 .AddEnvironmentVariables();

             builder.ConfigureApplication(context.HostingEnvironment);
         })
         .ConfigureServices((ctx, services) =>
         {
             services.AddApplicationInsightsTelemetryWorkerService();
             services.ConfigureFunctionsApplicationInsights();

             services.RegisterFunctions(ctx.Configuration, ctx.HostingEnvironment)
                 .RegisterInfrastructure(ctx.Configuration)
                 .RegisterApplication();
         });
 }

If I remove the new FunctionsAuthorizationExtensionStartup().Configure(builder); I get the error from my original post because it can't find the registration.  It seems to be completely ignoring your startup class unless I make the call explicitly.
The only thing different from a brand new .net 8 v4 function app that might be interfering is I am using Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore for ASP.NET Core integration.

@teknofobia, thanks for sharing this.

The only thing different from a brand new .net 8 v4 function app that might be interfering is I am using Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore for ASP.NET Core integration.

This is actually a requirement for this framework to work. You see in my example, I'm calling ConfigureFunctionsWebApplication.

When you setup a new Functions project, the call to ConfigureFunctionsWorkerDefaults does not exist and you shouldn't call it as it adds the default worker middleware after having added the asp.net middleware. Actually, ConfigureFunctionsWebApplication calls ConfigureFunctionsWorkerDefaults with specific instructions to take the HTTP calls all the way to the ASP.NET core handlers. You can see this here.

I don't see any authentication or authorization setup in your snippet. Are you doing so within RegisterFunctions?

I would advise starting a new app from scratch, make sure you configure auth and the functions authorization middleware, create a simple test function in your app and once you see it working, start moving the existing pieces one by one to try to find what is affecting middleware initialization.

I will go ahead and close this issue as I cannot reproduce this behavior from a ASP.NET Core integrated Isolated project


Update for Fedora Asahi Remix 40

The Fedora Copr builds of krunvm for Fedora Asahi Remix currently only has builds for Fedora 38 and 39. Would it be possible to enable the build for Fedora 40 now that it has been released?
https://fedoramagazine.org/fedora-asahi-remix-40-is-now-available/
$ sudo dnf copr enable slp/asahi-krun

Do you really want to enable copr.fedorainfracloud.org/slp/asahi-krun? [y/N]: Y
Error: It wasn't possible to enable this project.
Repository 'fedora-40-aarch64' does not exist in project 'slp/asahi-krun'.
Available repositories: 'fedora-38-aarch64', 'fedora-38-i386', 'fedora-39-x86_64', 'fedora-39-i386', 'fedora-38-x86_64', 'fedora-39-aarch64'

The idea is to use krun instead of krunvm on Asahi. You might find this guide useful until the packages land in the distro: https://gist.github.com/teohhanhui/042a395010d9946ceee14768736e3780

Thanks for the info! I see, they are both separate yet related projects.
From the guide you linked, I found this Fedora Copr to to have krun for Fedora 39 and 40.
$ sudo dnf copr enable teohhanhui/asahi-krun

What do you think about removing your old Copr to avoid confusion? It has packages for Fedora 38 and 39 but Fedora 38 is end-of-life and the other Copr has updated packages for Fedora 39 as well. Otherwise, they provide the same packages.
https://copr.fedorainfracloud.org/coprs/slp/asahi-krun/packages/
https://copr.fedorainfracloud.org/coprs/teohhanhui/asahi-krun/packages/


Update rubocop to use project style, fix a ton of stylistic things

I went through the whole of the rubocop configuration and tried to set it up to be in line with the project style, and then did commits (that were mostly bulk things) to fix the project style to match it (this gets it in line with the standard Ruby style guide and also fixes some small performance things, including ones that showed up in Codeclimate).
It is humongous. I apologize. It does not break any tests, and it only touches the Ruby code, and most of the things I changed would be glaringly horribly wrong if I did them incorrectly (such as mistyping a method name) or would not be able to cause an issue (such as whitespace changes). Almost everything should act exactly the same, and the few places I changed (such as removing unused variables, or converting num.times.collect { thing } to Array.new(num) { thing }) are recommended to be changed and should not impact functionality at all (just improve performance / readability).
Looking back at this I don't actually know if you'll accept it, because it's huge. I don't think it does anything bad, though.

The travis build literally got one failure on that test that we keep having issues with.

When reviewing, you'll probably want to view "Use safe assignment in conditions (parenthesize assignment)" ignoring whitespace.

I've dropped some more unnecessary changes and tried to reduce the size of this thing – if you skip all the changes to .rubocop.yml, these are the changes:
https://github.com/Marri/glowfic/pull/159/files/89e4779bc325f9b263e108a33b79624a3bdb982a..7a02beef4047b9d55714eefe8f6cf557fb6be8aa
It looks to be 37 lines, which is fortunately not all that huge! When I run this, I get a total of 39 offences. If I try running on master, Rubocop complains that Style/MethodCallParentheses should be Style/MethodCallWithoutArgsParentheses, and once I fix that, it gives me a total of 165 offences. (Lots of the defaults seem weird.)

And now with the commits hopefully more properly organized:
https://github.com/Marri/glowfic/pull/159/files/c6e01f50b66aa7bb99d0d2dacd69c4ddd71d04e8..4a748a8757d3511fe7effcdac448f42241d42d4a
https://github.com/Marri/glowfic/pull/159/files/c6e01f50b66aa7bb99d0d2dacd69c4ddd71d04e8..4a748a8757d3511fe7effcdac448f42241d42d4a?w=1
(+182 -186 with whitespace; +138 -142 without)

Rebased: https://github.com/Marri/glowfic/pull/159/files/eaa5f270036f2e490e82743641fc25828fc77a01..eb3102619661c23d49da01b77bf04e0c5f27b5ba?w=1


Route53: Implement dummy GetChange endpoint

Description
This PR implements a dummy GetChange endpoint for the Route53 mock within moto. This change has been discussed in #3449.
The Route53 mock in moto does not store Change IDs in it's backend at the moment, this is why I created a dummy endpoint that always returns INSYNC status together with the same SubmittedAt date for all requests.
Ideally, the Route53 mock backend would store Change IDs the same way it stores hostedzones, healthchecks, etc. I've discussed the possibility of doing this with @bblommers and he's keen on that, but for now I'll stick with this approach as the ideal option would require significant changes in the route53 mock to support and store those change IDs.
Why
GetChange is used by existing tooling around Route53, which makes supporting this endpoint in moto very advantageous when using moto in local development environments as well as testing for Route53.
Testing
I've added one test to ensure get_change always return the Id it was asked for as part if it's response and Status is INSYNC.

Coverage decreased (-0.2%) to 94.015% when pulling 52903d0cbac2cf1f3490e444d2b3e1f2b44b3c26 on relvira:relvira_r53_get_change into c902de8a098d2537792ec5533feb01085febd5ea on spulec:master.


Fancy changes!

A month of work put into one PR.
The shortest update summary: I got the application to behave like typical Python package, extended algorithms, begin to work towards tabu search implementation, enhanced tests.

Tests for front end are broken, though…


How to get tool_call_id?

How do you get the tool call id, or the id of the tool?
Tool output has this field
tool_call_id: Option<String>

The ID of the tool call in the ‘required_action’ object within the run object the output is being submitted for.

But, the run object doesn't have a required_action object
https://platform.openai.com/docs/api-reference/runs/object

Oops misread api sorry!


Clear Canvases on New File Upload

If you upload one image, do your thing and then upload another, the new image will be placed over top the old one. This makes you have to manually remove some of the funky background. It would be nice if everything cleared itself after a new upload.

Hi, can I work on this issue?

The one you submitted did the trick @malviya-abhishek. Thanks for the help all!