[tools] flutter doctor -v returns user's all system paths including non-flutter ones

This change was introduced in https://github.com/flutter/flutter/issues/83727
This prints user's all the system paths, some users might not wanna share this info following the issue template or when the triage team requests flutter doctor -v output.
[✓] Flutter (Channel master, 2.9.0-1.0.pre.297, on macOS 12.1 21C52 darwin-x64, locale en-EE)
    • Flutter version 2.9.0-1.0.pre.297 at /Users/tahatesser/Code/flutter
    • Upstream repository git@github.com:NevercodeHQ/flutter.git
    • Framework revision 7fe3565a9b (3 hours ago), 2022-01-10 02:25:11 -0500
    • Engine revision 8989f3cb25
    • Dart version 2.16.0 (build 2.16.0-144.0.dev)
    • DevTools version 2.9.2
    • PATH =
      /Volumes/Extreme/engine/src/flutter/lib/web_ui/dev:/Volumes/Extreme/depot_tools:/Users/tahatesser/Code/dart/bin:/Users/
      tahatesser/Code/android-sdk/platform-tools:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/Apple/usr/bin:/Applica
      tions/Android
      Studio.app/Contents/jre/Contents/Home/bin:/Users/tahatesser/Code/dart/bin:/Users/tahatesser/.pub-cache/bin

As flutter doctor -v is requested by the triage team on most of the issues, some users even hesitate to share their complete install directory let alone the entire system path (including non-flutter ones)
Proposal
Consider getting path information using another command or add another parameter to flutter doctor -v to print this information so it can be requested when only required

cc: @jmagman

Hi @TahaTesser, Thanks for filing the issue. I am able to reproduce the same behavior.

I wonder could this just use where.exe (Windows) / which (Linux, MacOS) output for flutter and perhaps dart?
It would probably be much more helpful, e.g. here's the output for me on my Windows machine 🙃
[√] Flutter (Channel master, 2.9.0-1.0.pre.299, on Microsoft Windows [Version 10.0.19041.1415], locale ru-RU)
    • Flutter version 2.9.0-1.0.pre.299 at C:\dev\src\flutter
    • Upstream repository git@github.com:nt4f04und/flutter.git
    • FLUTTER_GIT_URL = git@github.com:nt4f04und/flutter.git
    • Framework revision 098721e341 (4 hours ago), 2022-01-10 05:40:18 -0500
    • Engine revision 6a9c04dc47
    • Dart version 2.16.0 (build 2.16.0-144.0.dev)
    • DevTools version 2.9.2
    • PATH = C:\Python38\Scripts\;C:\Python38\;C:\Users\danya\AppData\Local\Programs\Python\Python38\Scripts\;C:\Users\danya\AppData\Local\Programs\Python\Python38\;C:\Python39\Scripts\;C:\Python39\;C:\Program
      Files\ImageMagick-7.0.11-Q16-HDRI;F:\programs\VMware\bin\;C:\Program Files (x86)\Common Files\Intel\Shared
      Libraries\redist\intel64_win\compiler;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\OpenSSH\;C:\Program Files\NVIDIA
      Corporation\NVIDIA NvDLISR;C:\ProgramData\chocolatey\bin;C:\Program Files\Docker\Docker\resources\bin;C:\ProgramData\DockerDesktop\version-bin;C:\Program Files\dotnet\;C:\Program Files (x86)\NVIDIA
      Corporation\PhysX\Common;C:\Program Files\Git\cmd;C:\Program Files (x86)\Windows Kits\10\Windows Performance
      Toolkit\;C:\Strawberry\c\bin;C:\Strawberry\perl\site\bin;C:\Strawberry\perl\bin;;C:\dev\src\gsutil\google-cloud-sdk\bin;C:\Program Files\nodejs\;D:\dev\src\mysql_shell\bin\;D:\dev\src\mysql_server\MySQL
      Server 8.0\bin;C:\Program Files\MongoDB\Server\5.0\bin;C:\Users\danya\AppData\Local\Microsoft\WindowsApps;C:\Users\danya\AppData\Local\Programs\Microsoft VS
      Code\bin;C:\dev\src\flutter\bin;C:\dev\src\choco\dart-sdk\bin;c:\env\commands;C:\Users\danya\AppData\Local\Android\Sdk\platform-tools;C:\Users\danya\AppData\Local\Android\Sdk\emulator;C:\dev\src\NASM;C:\dev\      src\MinGW\bin;C:\dev\src\gsutil\google-cloud-sdk\bin;C:\Users\danya\AppData\Local\GitHubDesktop\bin;C:\Program Files\JetBrains\IntelliJ IDEA Community Edition
      2020.3.3\bin;;C:\Users\danya\AppData\Roaming\Pub\Cache\bin;C:\Program Files\Android\Android Studio\jre\bin;C:\Program Files\Android\Android Studio\jre\bin;C:\Program
      Files\AdoptOpenJDK\jdk-<IP_ADDRESS>-hotspot\bin;C:\Program Files (x86)\Common
      Files\Oracle\Java\javapath;D:\dev\src\github_cli\;D:\dev\src\chromedriver;C:\dev\src\ninja;F:\dev\depot_tools;C:\Users\danya\AppData\Roaming\npm

This prints user's all the system paths, some users might not wanna share this info following the issue template or when the triage team requests flutter doctor -v output.

The existing flutter doctor -v contains paths, device names, and other identifiable information.  It's up to the users pasting it to redact as necessary.
If the triagers aren't going to find PATH output useful in doctor, we can remove it.
@zanderso what do you think?

Let's back out https://github.com/flutter/flutter/pull/95453 and reconsider. Marking this cp: 2.10 based on triager reports that this could discourage issue reporters from providing doctor info.

/cc @crisboarna

Removing cp: 2.10. I misread the commit date. The change hadn't landed in time for the cut.

https://github.com/flutter/flutter/pull/96414 has merged.  Closing this, we can discuss the original feature request in https://github.com/flutter/flutter/issues/83727.


Pod was missing in Pod file

Hi! I had to add the pod by hand. Why is not needed it in the plugin xml?


Posters missing during first run

Describe the bug
Running GAPS for the first time. All movies are imported but all poster images are missing.
Screenshots

Desktop (please complete the following information):

OS: Docker Container
Browser: Chrome
Gaps Version: v0.7.5

I'll need the logs after Gaps has run a search.

Seeing this here as well

I'll need the logs after Gaps has run a search.

GAPS log

I too have the same problem

The mounted folder is unable to be written to. What OS is running docker and now did you configure the folder and mount point?
2020-09-14 04:24:03.552 ERROR 8 --- [taskScheduler-1] c.j.gaps.service.FileIoService           : Can't write to file /usr/data/movieIds.json

Just chiming in here, I am seeing the same error in logs. Going back to the 'alpha' tag on Unraid, it works correctly (all same docker variables, paths, etc)

If you're seeing alpha work, you might want to try switching back to the mainline and going to configuration > Advanced > Delete All Stored Data > Delete. Then rescan plex and rerun gaps. This may be a storage configuration issue.
Follow up question, the title of the issue is "first run", are the images loading after search is complete and refreshing the page?

I've tried deleting stored data and starting completely fresh. The images don't show for me ever, and, the Recommended page becomes empty when refreshed after it finds missing movies (also without poster images)
Heres my log file.
gaps.log
Only thing I see in browser console:
DevTools failed to load SourceMap: Could not load content for http://<IP_ADDRESS>:8484/js/libraries/sockjs.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE
DevTools failed to load SourceMap: Could not load content for http://<IP_ADDRESS>:8484/js/libraries/bootstrap.bundle.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

I saw this happening on my local machine. I'm not sure what happened but I rebuilt and pushed the container and it's fixed. I'm guessing the docker deploy was pushed and didn't build master on the commit I expected causing a bug. Try pulling the new docker and refreshing your libraries, that should fix the issue.

Awesome, thank you! Can confirm it is working as intended.

Awesome!

On Synology NAS with current image loaded, posters are still not working for me.
About screen reports v0.7.5
Interesting that the failed image has the following image address (doesnt seem complete to me to point to a specific image)
http://<IP_ADDRESS>:32400/?X-Plex-Token=XXXXXXXXXXXXXXXXXX

Try v0.8.0. The issue seemed to be a bad build of docker

FWIW, for me on v0.8.0, the posters are missing on the Library page, but (ironically) they are not missing on the Missing page.

Really Dumb question:  I normally pull from docker registry and select "Latest Version".... where else would be best place to get that from?  I thought that was the authoritative place to get most recent builds
Thanks in advance

I need full logs from Docker and JS to be able to fix this.
DockerHub is the only spot to pull gaps from


The first time I click in the code editor, the cursor position is wrong

Hi!
I think this issue it's happening in all challenges. The first time I click in the code editor, the cursor position is wrong. If I try to write something, the text appears in another part. You can test e.g in "Warn your Users of a Dangerous Action" challenge
Thanks a lot and best regards.

@jfromero Thanks for reporting an issue. This is a duplicate of a long-known issue with the editor. See: https://github.com/freeCodeCamp/freeCodeCamp/issues/7847. It will be fixed when the beta site replaces the current site.


DOC: expanding comparison with R section

[ ] closes #9815
[ ] tests added / passed
[ ] passes git diff upstream/master | flake8 --diff
[ ] whatsnew entry

This is the beginning of a quick reference section.  It's incomplete,
just did a rough translation of
http://nbviewer.jupyter.org/urls/gist.githubusercontent.com/TomAugspurger/6e052140eaa5fdb6e8c0/raw/811585624e843f3f80b9b6fe89e18119d7d2d73c/dplyr_pandas.ipynb
into tables.  Should try to get some R experts to comment, and it would
be nice to have the pandas versions link to docs for the functions being
used, but I'm terrible at reStructuredText and gave up for the moment.

Thanks for this. There's a slightly updated version here, but I can't really remember what changed.
reST can be a bit of a pain. For linking to methods, you can use e.g.
:meth:`~pandas.DataFrame.sample`.

Yeah, I tried :meth: once and didn't like it.
But seriously, I couldn't get it to format well, or deal with the example
arguments. I'll play with it some more eventually.
On Fri, Feb 26, 2016 at 16:40 Tom Augspurger<EMAIL_ADDRESS>wrote:

Thanks for this. There's a slightly updated version here
https://gist.github.com/TomAugspurger/6e052140eaa5fdb6e8c0, but I can't
really remember what changed.
reST can be a bit of a pain. For linking to methods, you can use e.g.
:meth:~pandas.DataFrame.sample.
—
Reply to this email directly or view it on GitHub
https://github.com/pydata/pandas/pull/12472#issuecomment-189492001.

--
Cheers,
Leif

can you update

@jreback thanks for the poke

@leifwalsh thanks.
I merged this in. Pls have a look at the built docs (prob take a few hours). http://pandas-docs.github.io/pandas-docs-travis/comparison_with_r.html
not really sure if there is a way to make the 3 tables (sorting, transforming, grouping) be the same width .....


Code coverage enforcement

https://angular.dev/guide/testing/code-coverage

this was a jasmine command I think.
Besides, the code coverage thing is useless for me, because it would bein accurate to want I want to know.
e2e are with cypress. isolated standalone component, which require dom interaction are done with cypress as well.
Only services, guard, intercetpors, etc. which do not have dom elements to be tested with, or components which expose public methods, are testable, using jest (instead of jasmine).
So it would look like low code coverage, but I dont want to test things, twice..


Style Components do not work even after adding them to peer review and installing.

I used to get warn error Style Components was not in peer. So, I change the location to peer review as below.
"peerDependencies": {
"gatsby": "^2.0.0-beta.19",
"react": "^16.4.0",
"prop-types": "^15.0.0",
"styled-components": "^3.3.3"
},
Then, I install styled-components. The warning went away, but the component's css is absent in deployment and gatsby serve.

Thank you for pointing out :)
It is resolved with b82f187bf28b048d8b5b6f5c50520c21f121a12f


docs: fix incorrect init module path

In Zenith documentation, it need to change directory to my-module before initiate the module to create module to correct location.

Rendered

We can adopt the approach suggested in https://github.com/dagger/dagger/pull/6638#discussion_r1492496828
I've made this change in https://github.com/dagger/dagger/pull/6691


Gateway: Flatten admin experience into SPA

[ ] Flatten gateway info tab into the Gateway admin experience SPA
[ ] Flatten Deposit experience tab ''
[ ] Flatten Withdraw experience tab

Hello @okjodom  I'll like to work on this issue

#119

We can follow up with a bunch of code cleanups :)


Always upload latest prerelease to a dedicated prefix

This way: app.rerun.io/prerelease will always be the latest goodness to land on main.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)

https://app.rerun.io/prerelease is no longer correct - we now use https://app.rerun.io/commit/a935603


Fix filename changing when file uploaded bug

I fixed the bug causing the names of files to change when they are uploaded .
Link to the bug #321

Automated message from Dropbox CLA bot
@varshit97, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here and update the thread so we can consider merging your code.

@smarx Filled the doc

This PR had a small bug. So had to close this. I made another PR for the same bug fix.
Sorry for this confusion.

OK, cool; just FYI next time, you can update a GitHub pull request rather than recreating it when you want to fix bugs in it -- just do another push (or force-push) to the branch you created the PR from.


Update Java native-image smoke test

Summary
Fix https://github.com/paketo-buildpacks/builder-jammy-full/actions/runs/4826133198/jobs/8608409442

hello @ForestEckhardt @dmikusa !
Isn't there a way that instead of maintaining an almost copy of the samples in this repo, you would rather clone it during the smoke tests ?
The samples repo is already referred to by the documentation; is there any reason it could not be used to run smoke tests as well?
Thanks - happy to create an issue if you tell me this proposition makes sense


Feature: configure periodic reports 155965536

What does this PR do?
It adds flexibility of configuring the frequency at which the user is supposed to receive reports from the app.
Description of tasks completed

Adds a dropdown to the UI for the user to select
Adds a frequency field in profile data models
Creates tests for the
How can this be replicated manually
clone and git checkout ft-configure-periodic-reports-155965536
create virtue environment
install dependancies using pip install requirements.txt
run the Django server ./manage.py runserver
signup and go to dashboard

Stories
155965536
screenshots

@mirr254 You need to write tests for your changes.


Migration to GitHubSdk 0.5.0

I thought it might be a good idea to connect the app to the new version of the SDK, which is much more flexible and will allow for more features to be added later, following up from https://github.com/Meisolsson/GitHubSdk/pull/13. There are a couple of breaking changes and I have rewritten a number of classes and tests to take that into account. Because classes in com.github.pockethub.android.ui.item.news have been the most heavily affected, I also decided to migrate that package to Kotlin.

Thanks for the review. I will fix the tests for 0.5.1 and optimize some of the other things you mentioned as soon as I can. Will also keep PRs simple in the future.

I made the changes for the minor comments. I have the migration to 0.5.1 with all the tests passing on a feature branch. I'm assuming you'd prefer that as a separate PR, otherwise I can push the changes here. Let me know.

Well the changes to 0.5.1 would probably be nice to have in this since it's just for the RepoIdentifier. Other than that the only thing i still want changed is the imports which changed all over the place.

Ok, this should do it then.

Looks good.


Search - When pull the scroll thumb search results are empty

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.38-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/35058
Action Performed:

Navigate to staging..new.expensify.com
Click on Search
Pull the scroll thumb through search results

Expected Result:
Search results should be shown
Actual Result:
Search results are empty when pull the scroll thumb
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/3b9f6bab-0234-46d4-ad08-61c541b354fa
View all open jobs on GitHub

I can't seem to reproduce this myself
https://github.com/Expensify/App/assets/30609178/f16d61ea-eb0e-4a26-afc4-3c3817e1ec16

is there a possibility to get this large contacts base account for testing?

@lukemorawski I can add one of your accounts to a high traffic account with 10,000 users in the policy. Could you give me an email to add to it? (Should be a test account since it will really make your App slow)

@thienlnam yes please lukasz.morawski+5 [at] callstack . com

Added!

other bugs are being squashed now

Asked to be re-tested https://expensify.slack.com/archives/C9YU7BX5M/p1709681685389939

Looks like this is still reproducible - @lukemorawski do you have the capacity right now to handle this or should we pass it on to another contributor?

Yup can agree with that, it seems to only impact very high traffic accounts - so not a real priority right now


Not working with 1.20.5: FileNotFoundError

Running shulkr 1.20.4 1.20.5 yields the error when processing 1.20.5. It is working with other versions (e.g 1.20.3 1.20.4).
Moving generated sources
Traceback (most recent call last):
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 825, in move
    os.rename(src, real_dst)
FileNotFoundError: [WinError 2] The system cannot find the file specified: 'C:\\mcdiff\\.\\yarn\\namedSrc' -> 'C:\\mcdiff\\.\\src'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Scripts\shulkr.exe\__main__.py", line 7, in <module>
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\shulkr\__main__.py", line 5, in main
    cli()
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\click\core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\click\core.py", line 1055, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\click\core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\click\core.py", line 760, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\shulkr\cli.py", line 67, in cli
    run(versions, mappings, repo, message, tags, undo_renamed_vars)
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\shulkr\app.py", line 87, in run
    create_version(version)
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\shulkr\version.py", line 68, in create_version
    raise e
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\shulkr\version.py", line 58, in create_version
    generate_sources(version, mappings, repo_path)
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\minecraft\source.py", line 135, in generate_sources
    _generate_sources_with_yarn(version, path)
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\site-packages\minecraft\source.py", line 67, in _generate_sources_with_yarn
    shutil.move(
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 845, in move
    copy_function(src, real_dst)
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 436, in copy2
    copyfile(src, dst, follow_symlinks=follow_symlinks)
  File "C:\Users\Samuel\AppData\Local\Programs\Python\Python311\Lib\shutil.py", line 256, in copyfile
    with open(src, 'rb') as fsrc:
         ^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\mcdiff\\.\\yarn\\namedSrc'

Looks like this is due to Yarn for 1.20.5 (and possibly up) building into a build folder.

A temporary workaround is to use the Mojang mappings:
shulkr --mappings mojang 1.20.4 1.20.5

Upgrading to shulkr 0.7.2 should fix the issue when using Yarn mappings!


Incorporate set_time_limit method into Python wrapper for KnapsackSolver C++ class

This extremely simple PR incorporates the set_time_limit method of C++ KnapsackSolver class into its corresponding Python wrapper by modifying the SWIG interface.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

Googlers: Go here for more info.

I've already signed the CLA

Can you please send this PR against master insted (our current dev branch).
stable branch is just a Release branch...

I've modified it, please tell me if everything is correct now.


v1.0 drops embeddings_util.py breaking semantic text search

Describe the bug
The previous version of the OpenAI Python library contained embeddings_utils.py which provided functions like cosine_similarity which are used for semantic text search with embeddings. Without this functionality existing code including OpenAI's cookbook example: https://cookbook.openai.com/examples/semantic_text_search_using_embeddings will fail due to this dependency.
Are there plans to add this support back-in or should we just create our own cosine_similarity function based on the one that was present in embeddings_utils:
def cosine_similarity(a, b):
    return np.dot(a, b) / (np.linalg.norm(a) * np.linalg.norm(b))

To Reproduce
Cookbook example cannot be converted to use v1.0 without removing the dependency on embeddings_utils.py https://cookbook.openai.com/examples/semantic_text_search_using_embeddings
Code snippets
from openai.embeddings_utils import get_embedding, cosine_similarity

# search through the reviews for a specific product
def search_reviews(df, product_description, n=3, pprint=True):
    product_embedding = get_embedding(
        product_description,
        engine="text-embedding-ada-002"
    )
    df["similarity"] = df.embedding.apply(lambda x: cosine_similarity(x, product_embedding))

    results = (
        df.sort_values("similarity", ascending=False)
        .head(n)
        .combined.str.replace("Title: ", "")
        .str.replace("; Content:", ": ")
    )
    if pprint:
        for r in results:
            print(r[:200])
            print()
    return results

results = search_reviews(df, "delicious beans", n=3)

OS
Windows
Python version
Python v3.10.11
Library version
openai-python==1.0.0rc2

@RobertCraigie, makes sense. Thank you for the super fast response!

@mrbullwinkle
In the meantime, I hope this piece of code can help you.
source: https://learn.microsoft.com/en-us/azure/ai-services/openai/tutorials/embeddings?tabs=python-new%2Ccommand-line
def cosine_similarity(a, b):
    return np.dot(a, b) / (np.linalg.norm(a) * np.linalg.norm(b))

def get_embedding(text, model="text-embedding-ada-002"): # model = "deployment_name"
    return client.embeddings.create(input = [text], model=model).data[0].embedding

def search_docs(df, user_query, top_n=4, to_print=True):
    embedding = get_embedding(
        user_query,
        model="text-embedding-ada-002" # model should be set to the deployment name you chose when you deployed the text-embedding-ada-002 (Version 2) model
    )
    df["similarities"] = df.ada_v2.apply(lambda x: cosine_similarity(x, embedding))

    res = (
        df.sort_values("similarities", ascending=False)
        .head(top_n)
    )
    if to_print:
        display(res)
    return res

res = search_docs(df_bills, "Can I get information on cable company tax revenue?", top_n=4)

@CristianPQ you beat me to adding a ref to that code sample to this issue after I added it earlier today. I am the author of that code sample. :relaxed:

Very unprofessional that such functions just get removed. It is a moving API one cannot rely on, very annoying.

@RobertCraigie , is the support added in cookbook repo  for the functions provided in embeddings_utils.py ?

@logankilpatrick , can you help here?

Hi folks - any updates on the cookbook yet?

Still can't see the updates on the cookbook

Why the frick would you guys remove this. This API is god awful.

FWIW, if you're using embeddings from the OpenAI API, you can get cosine similarity with just a @ b, since they're normalized to 1 anyway (assuming a and b are NumPy arrays or tensors or similar, not plain lists)

Also BTW there is still a reference to embeddings_utils in the actual user guide in the docs here: https://platform.openai.com/docs/guides/embeddings/use-cases


Function calls are not supported in decorators - Angular 7

Hi,
When executing ng build --prod the following error appears.
ERROR in Error during template compile of 'AppComponent'
  Function calls are not supported in decorators but 'beforeMethod' was called in 'log'
    'log' calls 'beforeMethod' at src\app\kaop-test.ts(4,10).

This only happens with AOT enabled.
Example repo: kaop-ts-issue

Hi @Nawsen,
After reading angular issues everywhere seems like aot compiler made lots of people happier with that change.
I find a fix by explicitly using join point decorator on the method and importing the advice rather than creating a new decorator expression.
I don't understand very well what compiler is trying to achieve by forcing developers to do it that way.
workaround: https://github.com/Nawsen/kaop-ts-issue/pull/1
please @mgechev, since you were on AOT stuff, AOP as well, can you give us some advice to properly solve this problem?


Support for Vue 3

Since Vue 3 is now in RC, would it be possible to support Vue 3?

@cnguyen-de, thanks for your patience on this!
We're currently working towards releasing v1.0 of this plugin. Feel free to follow along in https://github.com/vue-gapi/vue-gapi/issues/105 and/or leave feedback.
I'm leaning towards supporting Vue 3 in a v2.0 – which would be a fast-follow. Hopefully all of that dust will settle over the next ~month.

@cnguyen-de, a Vue 3-compatible version of this plugin is in beta and available via vue-gapi@next.
$gapi is available as a global property and accessible via this.$gapi inside the application, analogous to the Vue v2.x usage. We might add some additional features around the new Composition API at some point and are open to feedback!

@cnguyen-de, a Vue 3-compatible version of this plugin is in beta and available via vue-gapi@next.
$gapi is available as a global property and accessible via this.$gapi inside the application, analogous to the Vue v2.x usage. We might add some additional features around the new Composition API at some point and are open to feedback!

@namoscato thanks for the update, will check it out asap. Will report back if I encounter any issues.

@namoscato thanks for the update, will check it out asap. Will report back if I encounter any issues.

Would love to see an example in Vue 3

@StavrosFerkadis, the plugin is pretty similar at the moment in Vue 3. You can view updated examples here.


VoiceOver (iOS) not reading ARIA state changes when what-input is loaded

I've created a test case here:
https://codesandbox.io/s/4wlrq70km0
Test 1
Steps to reproduce:
Open https://4wlrq70km0.codesandbox.io/ in Safari on your iOS device. Tested with version 11.4.1.

double tap the "Click me" button
VoiceOver should read button as "Click me, expanded"
double tap the "Click me" button again

Expected results:
VoiceOver should read button as "Click me, collapsed"
Actual results:
VoiceOver does not announce the aria-expanded change
Test 2

comment out Line no. 3 import "what-input"; and click on Save button so what-input is not loaded
repeat steps above, you will find you get the expected result this time.

Hi @davepenfold, thanks for reporting this. I checked out your example last night on my phone that's been upgraded to iOS 12 and found that double tapping the screen or directly on the button to initiate a click on the button produced what I would expect as a correct result of VoiceOver announcing "click me, expanded" and "click me, collapsed". A single click directly on the button did not activate it, but simply announced the button and its state.
Is it possible that the behavior that you're seeing is an issue with VoiceOver that was changed in iOS 12? Are you planning on upgrading in the near future so you can test on your end?


Update SplitButton.js

Fix warnings when building in IntelliJ IDEA

How come I'm not getting any warnings? Are you using different lint settings?

Then why do you have this warning?

The lint check here is successful, so I think it's something on your end:
https://github.com/traccar/traccar-web/actions

Actually I tried to run the lint directly and I see it as a warning. Not sure why I'm still able to build everything without any issues.


Fix pipeline json types

Pull request acceptance prerequisites:

[x] Signed the CLA (if not already signed)
[x] Rebased/up-to-date with base branch
[ ] Tests pass
[x] Updated CHANGELOG.md with patch notes (if necessary)
[x] Updated CONTRIBUTORS (if attribution is requested)

The various json parsing changes broke how booleans behave in pipelines, this adds some explicit pipeline testing acceptance tests and should resolve the issue.
Related to #973

Jenkins, test this please.


UI changes of Receive Help screen.

[XD link]
https://xd.adobe.com/view/f11ac530-31c1-477d-adf4-22524b8b7ed3-a449/screen/8043e25c-3bfb-4f1b-a2bd-e2ab26bfc1dd/
[Issues to be handled -
1,3,4,5,6,8].

@cakesoft-swati Could we get a more higher-resolution image attached to this issue? I'm having trouble reading many of the items and seeing what exactly they refer to.

-Verified this issue on Dev app v1.4.1(192)

-Verified this issue on Dev app v1.4.1(192)


Implement bus locking on SAM0 I2C

After experiencing a few deadlocks, it was discovered that this bus does not implement any form of mutual exclusion... this patch addresses this and resolves potential deadlocks.
In my situation, we have an I/O expander with LEDs attached, and a charger controller... polling the charge controller, and updating LED state "occasionally stopped working" (exacerbated by more LED activity), along with all other items in the system work queue stopping.
After some digging, I discovered that there appears to be some clock stretching (I suspect from the I/O expander, though I'm not sure why that would be necessary...), and this often coincides with the fault.
Other drivers implement locking, which seems sensible and appears to fully resolve the issue (e.g: STM32).

As a bit of additional context, the charge controller is polled from the system queue, while the LEDs are updated from their own thread. I believe this is key to the issue presenting.

Rebase, for clean merge

@teburd I'm not sure about this one. In general the caller should be ensuring that access to a device is done in a thread safe way. Doing this on a case by case basis will lead to inconsistent results across platforms. Are you sure? I would double-check with @tbursztyka

There's no clarity on if its the device or the callers responsibility for synchronization. I've seen both. However for i2c most drivers have internal semaphores for ensuring thread safety as it is today, sam0 here is one of the unexpected ones.
See for example i2c_nrfx_twim.c which has two synchronization semaphores in fact to avoid issues.

I also linked to the STM32 driver in the original message as a precursor... I feel it would be "odd" to place this responsibility on the user / not on the driver...
Additionally, in this instance, it's not so much about locking the bus, as maintaining sync between the transfer start / end... in my case, I believe a stretched clock gave another thread the opportunity to come in and attempt to make another transfer. See here.

@mbolivar-nordic please look again

I've marked the PR as dev-review so we can discuss next week

Thanks @teburd - just to confirm, is that this meeting?
Depending on how things go earlier in my day, I might be able to attend on the 17th.

I'm gonna leave this unmerged for now; it would be good to settle on if we need a general policy if we can do that in next week's dev-review

I'm gonna leave this unmerged for now; it would be good to settle on if we need a general policy if we can do that in next week's dev-review

oops sorry @mbolivar-nordic, was on a merge binge and took this one in.

@attie-argentum I realize this PR is merged but shouldn't the lock be placed ahead of the var assignments here?
https://github.com/zephyrproject-rtos/zephyr/blob/be1380cc51bd872a5b0e8114cd60bddc37b863c5/drivers/i2c/i2c_sam0.c#L392-L395
It appears to me if another transaction is initiated, it could corrupt the driver's copy of the msg.

@sslupsky - that looks like a very fair comment to me... fancy making a PR?

@attie-argentum I could do so but might take a while.  I am in the middle of migrating a bunch of drivers from 2.3 to 3.1.  One of the surprises is the existence of this PR which addresses an issue I raised af few years ago.  I submitted a PR back then to fix the problem but the PR didn't get any attention so I closed it.  As I read these comments about thread safety I think it was the same issue then.  But, you have a friend in @carlescufi, as he merged it.  So, thank you for finally getting this issue resolved in the tree.
There are a couple another issues related to this driver that need to be resolved.  One is power management.  When the transfer gets underway, if there is nothing else going on the kernel will idle and, if you sleep during idle, it create long delays.  I got around the issue by using a busy wait during the transfer.  However, it looks like there is a way to tell the kernel that the driver is busy now and prevent kernel idle from putting the mcu to sleep.  I need to do a little work on this an test it out.
Finally, the clock configuration for sam0 needs some attention.  There is no gclk configuration in the device tree.  This is necessary for the I2C and UART drivers because if you want to use the I2C or UART to wake from sleep, you need to configure a gclk to connect to GCLK_SLOW.
The last thing in my existing driver that has not made its way into the tree is support for I2C slave mode.

See #48766

@attie-argentum Thank you for the update.


Uber sensor

Description:
This adds an Uber sensor for tracking price and pickup time estimates with the Uber API.
Related issue (if applicable): balloob/home-assistant.io#324
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: uber
    start_latitude: 37.8116380
    start_longitude: -122.2648050
    end_latitude: 37.7768520
    end_longitude: -122.4155500
    server_token: 'SERVER_TOKEN_FROM_DEVELOPER.UBER.COM'

Checklist:
If code communicates with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

@balloob Fixed all the issues you mentioned, thanks so much for the pointers. Everything is so much cleaner now! :)

Bueno :+1: :dolphin:


ImportError: cannot import name 'OrderedDict' from 'typing'

from deepface import DeepFace
DeepFace.stream(db_path = "path/to/images/db")
results in the below error:
ImportError: cannot import name 'OrderedDict' from 'typing' (C:\Users..\AppData\Local\Programs\Python\Python37\lib\typing.py)
Please how can this be resolved, i want to get facial recognition directly from webcam stream.

This was resolved by removing the import from typing.py and including it in functiontype.py
from typing_extensions import OrderedDict


Internet Reachability is not getting Detected.

Ask your Question

I'm using the isInternetReachable from net NetinfoState, but when I'm on Someone's WIFI Hotspot and turn off their mobile data, my app still doesn't detect that it's offline. How can I work around this.?
PS: This is my module right now
https://hatebin.com/nanoqwwlkn

We have the same issue in our app. isInternetReachable is basically mimicking isConnected

@premdasvm Tried the same scenario listener is not getting called.

We have the same issue in our app. isInternetReachable is basically mimicking isConnected

@geekabhinav I replaced the NetInfo package and added my own logic to check the internet connectivity.
 async function checkInternet() {
    let count = 0;
    let seconds = 0;
    setInterval(async () => {
      count++;
      try {
        const response = await fetchWithTimeout('https://clients1.google.com/generate_204', {
          timeout: 2900,
        });
        store.dispatch(connection(true));
      } catch (error) {
        // Timeouts if the request takes
        // longer than 6 seconds

        console.log(error.name === 'AbortError');
        store.dispatch(connection(false));
        // ActivityRoom.room.customErrorInvoke(151, 'Internet Loss');
      }
    }, 3000);
  }

  async function fetchWithTimeout(resource, options) {
    const {timeout = 8000} = options;

    const controller = new AbortController();
    const id = setTimeout(() => controller.abort(), timeout);

    const response = await fetch(resource, {
      ...options,
      signal: controller.signal,
    });
    clearTimeout(id);

    return response;
  }

On Android we basically just pass on the information that the OS gives to us. We rely on the system detecting it and if it doesn't correctly detect it, then we can't.
On iOS we use a "ping" to  test the connection manually. When there's a successful connection, we wait a little while before we try again. That means that there might be a delay before the connection drop is detected.

To make it more consistent, isn't better to ping in both platforms? I'm having the same issue. I'd like to pass my custom health check url.
Thank you.


Installer add same apt source multiple time

During the installation /etc/apt/sources.list is modified and jumpscale append deb http://mirror.unix-solutions.be/ubuntu/ bionic main universe multiverse restricted to it.
Problem is that it appends to the files even if the line is already there, which make apt not happy to have duplicated

➤ Dina-Abd-Elrahman commented:
verification steps:

check /etc/apt/source.list
reinstall JumpscaleX
check  /etc/apt/source.list  again to make sure that the installer doesn't append another line

➤ Dina-Abd-Elrahman commented:

the error appears with me today

W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Translations (main/i18n/Translation-en_US) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11 (main/dep11/Components-amd64.yml) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11 (main/dep11/Components-all.yml) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11-icons-small (main/dep11/icons-48x48.tar) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11-icons (main/dep11/icons-64x64.tar) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target CNF (main/cnf/Commands-amd64) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target CNF (main/cnf/Commands-all) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Translations (main/i18n/Translation-en_US) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11 (main/dep11/Components-amd64.yml) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11 (main/dep11/Components-all.yml) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11-icons-small (main/dep11/icons-48x48.tar) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target DEP-11-icons (main/dep11/icons-64x64.tar) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target CNF (main/cnf/Commands-amd64) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3
W: Target CNF (main/cnf/Commands-all) is configured multiple times in /etc/apt/sources.list:57 and /etc/apt/sources.list.d/vscode.list:3the problem is the line exists in /etc/apt/sources.list ` file and there is a file in the directory ` /etc/apt/sources.list.d/
/etc/apt/sources.list.d/vscode.list

➤ kristof de spiegeleer commented:
is this still issue?


Running On Linux PC

Hi can you tell me how to get this working on Linux i've tried various methods but they don't work at all.

please be more specific / give more information


Remove the concept of WorkerPool

Issue #323 - WorkerPool and WorkProcessor have been removed, no more Disruptor::handleEventsWithWorkerPool

Farewell, WorkerPool, king of work queues, whose long and faithful friendship those who knew you won't forget.

Why remove WorkerPool And WorkProcessor ?

你发的邮件我已经收到了。
                                    以上为自动回复邮件

Why remove WorkerPool And WorkProcessor ?

See the linked issue https://github.com/LMAX-Exchange/disruptor/issues/323


EntryPointNotFoundException while in Unity Editor

Trying to call Vibration.VibrateIOS(ImpactFeedbackStyle.Medium) while inside the Unity Editor returns an error. It works fine once built to the device though. I'm running this on a Windows 10 computer, so perhaps I'm missing some native IOS stuff. Might just deny it if I'm in the Unity Editor, but wanted to post this just in case.
The line that's causing the error:
`
#if UNITY_IOS
public static void VibrateIOS(ImpactFeedbackStyle style)
{
    _impactOccurred(style.ToString());
}

public static void VibrateIOS(NotificationFeedbackStyle style)
{
    _notificationOccurred(style.ToString());
}

public static void VibrateIOS_SelectionChanged()
{
    _selectionChanged();
}

#endif
`
Error:
_EntryPointNotFoundException: impactOccurred assembly: type: member:(null)
Vibration.VibrateIOS (ImpactFeedbackStyle style) (at Assets/Vibration/Vibration.cs:84)
SingletonMaster.VibrateInput () (at Assets/Scripts/Singletons/SingletonMaster.cs:133)
SingletonMaster.Vibrate (VibrationType t) (at Assets/Scripts/Singletons/SingletonMaster.cs:109)
MainMenu.Vibrate (VibrationType t) (at Assets/Scripts/Management/MainMenu.cs:162)
MainMenu.ToggleMenu (System.Int32 index) (at Assets/Scripts/Management/MainMenu.cs:118)
UnityEngine.Events.InvokableCall1[T1].Invoke (T1 args0) (at <7b2a272e51214e2f91bbc4fb4f28eff8>:0) UnityEngine.Events.CachedInvokableCall1[T].Invoke (System.Object[] args) (at <7b2a272e51214e2f91bbc4fb4f28eff8>:0)
UnityEngine.Events.UnityEvent.Invoke () (at <7b2a272e51214e2f91bbc4fb4f28eff8>:0)
UnityEngine.UI.Button.Press () (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/UI/Core/Button.cs:70)
UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/UI/Core/Button.cs:114)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:57)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/ExecuteEvents.cs:272)
UnityEngine.EventSystems.EventSystem:Update() (at Library/PackageCache/com.unity.ugui@1.0.0/Runtime/EventSystem/EventSystem.cs:501)

Try this
    public static void VibrateIOS(ImpactFeedbackStyle style)
    {
#if UNITY_IOS
        _impactOccurred(style.ToString());
#endif
    }

    public static void VibrateIOS(NotificationFeedbackStyle style)
    {
#if UNITY_IOS
        _notificationOccurred(style.ToString());
#endif
    }

    public static void VibrateIOS_SelectionChanged()

    {
#if UNITY_IOS
        _selectionChanged();
#endif
    }    ```

Same error for me on Mac machine.
EntryPointNotFoundException: _impactOccurred
Vibration.VibrateIOS (ImpactFeedbackStyle style) (at Assets/Scripts/Vibration/Vibration.cs:85)

As I see Vibration.cs already have those conditional ifs although it did not help:
    public static void VibrateIOS(ImpactFeedbackStyle style)
    {
#if UNITY_IOS
        _impactOccurred(style.ToString());
#endif
    }

I believe the issue is that unity editor is also considered UNITY_IOS. The solution is then to put #if !UNITY_EDITOR before the code invocation in the library. I went a bit further and added a Log when the vibration should happen in editor (just for testing) but it might be an overkill
#if UNITY_IOS
#if UNITY_EDITOR
        Debug.Log($"Impact with style {style} occured");
#else
        _impactOccurred(style.ToString());
#endif
#endif

Maybe it makes sense to create a wrapper around the Library that would encapsulate the logging part
static class MyVibration
{
    public static void Vibrate(ImpactFeedbackStyle style) // choose the name that fits you or just reuse the interface of the library
    {
#if UNITY_EDITOR
        Debug.Log($"Impact with style {style} occured");
#else
        Vibration.VibrateIOS(ImpactFeedbackStyle.Heavy);
        // Please note, Android is not handled, it's just an example
#endif
    }
}


No instructions how to run tests; no wdio.conf.js

This all looks great, thank you for putting it together. But instructions are lacking. How do I run tests? I expected to see a wdio.conf.js file.

Maybe it's npm run test:chromedriver, and other commands in package.json. It would be good to document that.

thanks, i will definitively do that!
wdio configuration is located in the config folder.
you can use the integrated debug scripts for visual studio code, or you can run the tests with npm run test:chromedriver.
i will add a description in the near future

npm run test:chromedriver immediately leads to errors about chromedriver. It would be good if you documented the dependency on chromedriver and referred to installation instructions.

if you've done a npm install before, there shouldn't be any other dependency. (except of the current stable chrome version).
if there are still errors, please open another issue with these errors!

Also, it's unclear what process transpiled the .ts config files into .js. I assume you ran tsc at some point, but I think it should be documented as part of a boilerplate project like this.

i will document this, too.
it's being transpiled on the fly with ts-node.
you can look at following file for example:
https://github.com/pako88/wdio-mocha-typescript/blob/master/config/chromedriver.conf.js

I understand, but then your repo should come without the .js files.

Also, no errors during npm install.
$ npm install                                                                                <<<

><EMAIL_ADDRESS>install /Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/fibers
> node build.js || nodejs build.js

  CXX(target) Release/obj.target/fibers/src/fibers.o
  CXX(target) Release/obj.target/fibers/src/coroutine.o
  CC(target) Release/obj.target/fibers/src/libcoro/coro.o
  SOLINK_MODULE(target) Release/fibers.node
Installed in `/Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/fibers/bin/darwin-x64-83/fibers.node`

><EMAIL_ADDRESS>install /Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/chromedriver
> node install.js

ChromeDriver binary exists. Validating...
ChromeDriver is already available at '/var/folders/q0/f4tncvcs2k7_v_skcsrsq78m0000gn/T/83.0.4103.39/chromedriver/chromedriver'.
Copying to target path /Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/chromedriver/lib/chromedriver
Fixing file permissions.
Done. ChromeDriver binary available at /Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/chromedriver/lib/chromedriver/chromedriver

><EMAIL_ADDRESS>postinstall /Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /Users/thomer/tmp/nobackup/wdio-mocha-typescript/node_modules/ejs
> node --harmony ./postinstall.js

Thank you for installing EJS: built with the Jake JavaScript build tool (https://jakejs.com/)

added 892 packages from 573 contributors and audited 892 packages in 15.891s

54 packages are looking for funding
  run `npm fund` for details

found 14 low severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details

Wait, wait, wait. Now that I do it again, it works. Weird! I apologize, I must have done something wrong before. Closing this issue, though my comment about additional documentation stands.

I realize now that the .js files you included are wrappers around the .ts files.

i added the documentation to the readme.
feel free to reopen this issue if somethings missing from your point of view.


Console settings: Python executable must not be 'pythonw.exe'

From pierre.raybaut on 2011-10-08T09:13:44Z
When running Spyder with pythonw.exe on Windows, the console plugin should convert pythonw.exe to python.exe when attributing the default value of Python executable option.
Now that it's done in distributed beta versions, we must checked existing option value to fix it too.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=794

From pierre.raybaut on 2011-10-16T06:06:17Z
This issue was closed by revision f3892e5f2559 .
Status: Fixed


Publish latest version to npm (2.3.4)

Currently the latest version available on npm is 2.3.2, please publish 2.3.4 to npm
https://www.npmjs.com/package/cordova-plugin-telerik-imagepicker

Please publish the latest version of plugin. Thank you.

@luke-rogers @misharysh - I just published v2.3.3 to npm.
Which features do you want in 2.3.4. Have you any reference to any PR if there any ?

Thanks for doing that.
On the releases page it says 2.3.4 is the latest release, but from looking at the commits it seems like 2.3.4 doesn't actually contain anything different to 2.3.3.


Figure 4 does not correspond to its caption

The caption writes '(c) The impact of the green list size parameter $\gamma$ on the z-score'. However, the figure looks like $\delta$.

Hi, thanks for the pointer! In a prior version of the figure we may have only shown variations on $\delta$ but in the camera copy the Fig. 4 shows 8 total settings, the cross of 2 values of $\gamma$ and 4 values of $\delta$.


avatar url without s=40 param

Hi @huchenme
Is it possible to get user avatars without s=40 size parameter?
Thanks.

@khoren93 Already removed in API

:tada: This issue has been resolved in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ISPN-10135 Performance Regression for HotRod-TX

try to prepare the transaction in the primary owner

https://issues.redhat.com/browse/ISPN-10135

run performance tests please
cs-hotrod

run performance tests please
cs-hotrod

run performance tests please
cs-hotrod-tx

run performance tests please
cs-hotrod-tx

@gustavolira the results page is empty

@danberindei we are looking into it

@pruivo funny, this PR made me look at the library regression test, and there's a huge drop in library repl read performance as well

run performance tests please
cs-hotrod-tx

@pruivo funny, this PR made me look at the library regression test, and there's a huge drop in library repl read performance as well

@danberindei it's because https://issues.jboss.org/browse/ISPN-11550

I deleted outdated/inconsistent report request/results.
I am going to rerun the tests because HotRod-TX should be stable.
Stable in terms of having a closer throughput and service time.

run performance tests please
cs-hotrod-tx

run performance tests please
cs-hotrod-tx

Looks good to me!

Merged, thanks Pedro!


Initial selected index

Hi
I have 3 tabs with XLPagerTabStrip: Closed - Opened - Upcoming
When user open this screen, by default it's present VC with "Closed"
But when this screen opened first, I need to open middle tab bar and user shouldn't see this. Can I make this?
P.S. I know about func self.moveToViewController(at: <Int>) but it visible for user

Found this problem with solving in issues :D


E: Unable to locate package libc6-dev-amd64

Window ver:
Microsoft Windows [Version 10.0.16299.19]
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 16.04.2 LTS
Release: 16.04
Codename: xenial
When I try "apt-get install vim git libc6-dev-amd64", the system cannot install libc6-dev.
The bash shows me:
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libc6-dev-amd64

on Xenial it's called libc6-dev
https://packages.ubuntu.com/xenial-updates/amd64/libc6-dev/filelist

https://packages.ubuntu.com/xenial/libc6-dev-amd64
It seems libc6-dev-amd64 is needed only in a 32bit system. WSL is 64bit only.

It seems libc6-dev-amd64 is needed only in a 32bit system.

...needed available...
Like this. I would cite #228 and the rest, but here you're actually asking for libc6-dev targeting arch amd64. Which is just apt install libc6-dev.


benchmark update

Provides more detailed information on the benchmarks. specifically, maximum and minimum times, and std-deviation of the timings.

Thanks for the review


[DET-1881] Enable build and publish of CLI and SDK via GitHub Actions.

This needs to notify a slack channel when it does work. It also seems like all the version and git tagging would still be done by hand, is that true? Why not get that automated as well?

This needs to notify a slack channel when it does work.

ohh yea good point, will add that.

This needs to notify a slack channel when it does work. It also seems like all the version and git tagging would still be done by hand, is that true? Why not get that automated as well?

we don't have automation for tags for anything else, that's the only reason. we can add it but should probably do it for everything. can look into it as a follow up

I think we want to split these out into separate workflows (or one workflow with a single sdk OR cli option). The sdk needs to finish before the cli if they both rev their versions. I think it'd be tempting to rev both, then kick off one job and then it's a race which may sometimes fail.
I don't know if I prefer one over the other. I guess maybe the or version would be easier to maintain.
Other than that, yeah, the job looks right, so either split them into two files or make it run in sdk or cli context.


Colors are leaked into the terminal

When I Ctrl-Z to suspend terminal vim, it seems the last segment's background color gets leaked into the terminal.
I have this set in the vimrc: set t_ti= t_te= so that vim doesn't clear the scrollback buffer when I quit or suspend.

Cannot reproduce. What terminal is this? Does it help to call reset or tput reset ?

If at all, I think this is a vim bug. So closing.


setup: Jest for client and server

command: yarn test
Test will be run for both client and server.
Closes #14

Please provide review as the conflits are resolved. @ViralRuparel  @rishabh1403


Update update_cumulative_es_rules.py

Added a function to check if the FR tracking data for the most recent month is complete; if not, do not update the month.

Thanks @mfebrizio for the suggestion. I'll move those functions to a separate file in the next revision.


Bindings - simplify

Working on inventory bindings - Removed tab, equip
Replaced by 1, 2, 3, 4 - pressing it again will equip targeted item
Bug: When pressing both buttons it selects both
Need: Possibility to bind two buttons at once (shift + x, ctrl + x, alt + x)

Bug fixed, removed a few useless binds (loot/drop is on the same button for example)
Still need the two bidding possibilities

Closing for now


Reconsider identifiers

Is it really a good idea to have that freedom on identifiers (ID)?
Some identifiers that I believe are good to have:

abc, name1 and such (C-style) should be ID (Latin, Arabic digits)
sin² should be valid (mix Latin with unicode Arabic digits)
αθηνα should be valid (Greek)
ΩS¹, sinβ should be a valid (mix Greek with Latin and unicode digits)
+, /, -, * should be valid (operators) but we may reserve a few as keywords like @ (like Java), # (?) or % (like Idris)
⁄, ⌀, ⁒, ·, ÷ should be valid (unicode operators)
::, :<, >>= are valid (several operators)
+₁, +₂, ÷₂, *‘ should be valid (operators followed by unicode digits)

Something that are currently ID, I think are confusing:

+1 -- looks like λx. x + 1. Similarly 1+
call/cc looks like call / cc to people w/o scheme experience. We may call similar function callCC, shiftReset
suc-suc -- because we know suc is not a "number" so we know it's not suc - suc, but if it's o-o, it's less clear. It can be o - o.
a+b -- even worse.
a+b=c -- also valid in Agda. EVEN worse.
+-comm -- commonly used in Agda. In Idris it's called plusCommutative iirc. I think the Idris name is acceptable. I'm not sure though.

I am thinking about a criterion that is similar to Haskell:

Let \a denote Latin/Greek/CJK/Russian alphabets
Let \o denote operators (unicode inclusive)
Let \u denote unicode superscript and subscript (numeric and alphabetic)
Let \d denote digits (unicode inclusive, exclude super/subscript)

Then \a[\a\d]*\u* | \o+\u* basically describes what I wanted. This will cause some renaming in the current stdlib.

By that, we may have shorter expressions like a+b like in Haskell

Any idea?

I think as long as we disallow white-spaces in identifiers, I'm totally fine :D
If in addition we enforce white-space in between identifiers, then having the current naming freedom also induce no ambiguity (and arguably still easy to parse by human).
While probably it is better to use addComm over a+b=b+a, naming proof terms of more complicated relation with the white-space collapsing, symbolic style can be easier and clearer in some situations.
Naming things is already hard enough for us programmers, and I think having a wider range of naming style to choose can help alleviate the burden a bit.

The grammar for identifiers you just mentioned looks cool to me. I always love laying limitations to the surface syntax, as long as it won't bring so much burden and extra mental models to the end users. My principle is making "conventions/specifications" work without any external tools, like linters.
My misc preferences:

call/cc: cooltt endorses this too, but I don't
+-comm: I don't like plusComm because it doesn't distinguish "plus" and + (unless like Python we know + is __add__), maybe commOf+? Not sure about this either

The grammar for identifiers you just mentioned looks cool to me. I always love laying limitations to the surface syntax, as long as it won't bring so much burden and extra mental models to the end users. My principle is making "conventions/specifications" work without any external tools, like linters.
My misc preferences:

call/cc: cooltt endorses this too, but I don't
+-comm: I don't like plusComm because it doesn't distinguish "plus" and + (unless like Python we know + is __add__), maybe commOf+? Not sure about this either

Thank you!
Eventually you'll end up putting all those properties into classes like CommutativeSemiring or TotalOrder or something like that. I think there will be one name that rules them all, and it won't matter much because you won't be proving +-comm by using its name -- instead you'll use solveRing or lia or something

In general, I am fine with current identifiers, that is, composing IDs from a sequence of visually visible characters.
I don't like putting restrictions on identifiers (because remembering these rules is also a burden to users) except those that significantly improve readability:

no whitespaces inside (like mixfix operators)
no leading numbers (looks like literal suffixes in other languages: 114514ULL)

And there are other reasons:

+-comm looks better than plusComm because for humans, splitting by the hyphen - is easier than splitting by any upper-case alphabet. I know you can choose plus_comm but the underlines are less beautiful -- they are not centered.
IDs like a+b=c, +1 and a+b=b+a do not actually make code harder to understand. Because the formatter always produce a + b = b + a if it is really an expression.

Fair. Closing.


Adding in metadata pydantic model and verification checks during runtime

Please refer to CONTRIBUTING.md for detailed instructions.
Summary

PR Checklist

[ ] My PR is less than 500 lines of code
[ ] I have added sufficient comment as docstrings in my code
[ ] I have made corresponding changes to the documentation

[ ] I have written unit-tests to test all of my code

@snova-jayr please add to PR description to justify the size of the changes in detail. Thank you!


add share functionality in minimal mode

hi @dileep-gadiraju  i have tested above pr . kindly accept this


Create a class for MailboxId

This PR changes several files, but the change is very simple. We had the concept of mailboxId but it was represented as a string. This PR adds a class for that concept and change the usage in all the places it was used.
The only place where mailboxId is being used as a string is GRPC message MailboxContent. We need to do that to keep GPRC protocol simple and backward compatible.
I think this will make the code easier to read but also improve error messages. For example, in case of timeout, before this PR the message show was:
Received error query execution result block: {1000=Timed out while offering data to mailbox:<PHONE_NUMBER>000154169|2|4|1|4

And now it will be
Received error query execution result block: {1000=Timed out while offering data to mailbox: {"requestId":<PHONE_NUMBER>000154169, "senderStageId": 2, "senderWorkerId": 4, "receiverStageId": 1, "receiverWorkerId": 4}

This message is still far from being great, but at least it doesn't look like magic numbers like the ones used in the TV show Lost.

Codecov Report
Attention: Patch coverage is 0% with 37 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (364c46a).
Report is 317 commits behind head on master.

Files
Patch %
Lines

...apache/pinot/query/planner/physical/MailboxId.java
0.00%
24 Missing :warning:

.../query/mailbox/channel/MailboxContentObserver.java
0.00%
4 Missing :warning:

...e/pinot/query/planner/physical/MailboxIdUtils.java
0.00%
2 Missing :warning:

...apache/pinot/query/mailbox/GrpcSendingMailbox.java
0.00%
2 Missing :warning:

...org/apache/pinot/query/mailbox/MailboxService.java
0.00%
2 Missing :warning:

...va/org/apache/pinot/query/routing/RoutingInfo.java
0.00%
1 Missing :warning:

...he/pinot/query/mailbox/InMemorySendingMailbox.java
0.00%
1 Missing :warning:

...g/apache/pinot/query/mailbox/ReceivingMailbox.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12955       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2428        -8
  Lines        133233   132842      -391
  Branches      20636    20571       -65
=============================================
- Hits          82274        0    -82274
- Misses        44911   132842    +87931
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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

I've created this PR mainly to have a nicer message in some cases. For example, when there is a timeout sending to some mailbox the current name is quite difficult to understand even for Pinot devs.

How about we introducing a util which takes the pipe format mailbox id and convert it to the pretty format? Then we can use it when we want a nicer message.

How about we introducing a util which takes the pipe format mailbox id and convert it to the pretty format? Then we can use it when we want a nicer message.

I think that goes against the idea you explained in the first comment and in fact that is similar to what this PR does. IICU your first comment, you indicated that you think in the future we may change the format of the mailbox id to something that does not carry the same information. If that is the case, we cannot assume that in the future we are going to be able to extract, from the mailbox id, the stage and other information that can be useful in the messages.
Maybe what you wanted to say is that we may want to change the format (but not the content) of the mailbox id in the future. In that case this current proposed implementation is fine because this PR does not modify the format used to send the mailbox id in the binary protocol. Instead it keeps in memory a MailboxId object but when it is serialized/deserialized it is converted to the current string form.
So instead of having to use a util to extract a human description from the current mailbox id, this PR extracts the information beforehand when the message is deserialized and keeps it in memory. Given the information carried by the mailbox id is mostly integers, this object representation is probably more memory efficient than the String and it is for sure easier to read (for example, having a List<MailboxId> is easier to understand than List<String>).


Using SwipeLayout without CoordinatorLayout

Hi!
I used SwipeLayout to make swipe panel and I wanted to animate translateTo() method, but there is no good way to do it. I'm missing point of extension in SwipeLayout so I can access orientationStrategy field.

Could you provide extra info about what you want to do.

I extended SwipeLayout, added method translateToWithAnimation(int), which will create animation that calls public super::translateTo(int) method. But I can't determine current translationX/Y value, because SwipeLayout::orientationStrategy field is inaccessible.

You don't need to extend swipelayout. You can pass through a listener and be notify about translation ( you can get x or y as percentage ) and apply your transformations. See Swing example.

I'll try to explain what I'm doing in code. Here is very rough example:
public class SwipeMakerAnimated extends SwipeLayout {
    public SwipeMakerAnimated(Context context) {
        super(context);
    }

    ...

    public void animatedTranslateTo(int position){
        // here orientationStrategy and getDelta are not accessable
        int currentPosition = orientationStrategy.getDelta();

        ValueAnimator animator = ValueAnimator.ofInt(currentPosition, position);
        animator.addUpdateListener(new ValueAnimator.AnimatorUpdateListener() {
            @Override
            public void onAnimationUpdate(ValueAnimator animation) {
                int position = (int)animation.getAnimatedValue();
                SwipeMakerAnimated.this.translateTo(position);
            }
        });
    }
}

Better override onTranslateChange(...) There you can get the position X (percentage ) call super and apply animation


Sharable configuration via URL

Could be done by continously updating the URL hash to a base64 value:
window.location.hash = btoa(JSON.stringify({template, input}))

Then, on page load check if window.location.hash decodes and fill the values. Won't work for excessively long URLs, but browsers should be able to handle URLs up to 32k characters long.

That is a great idea, I am now adding this kind of feature! 🥳

@silverwind, I implemented could you check now?

Works great, thanks!


Swipe down to close bottom sheet modals

Hi,
I was browsing through the material design docs, and when I was looking at bottom sheet modals. Usually in most apps, swiping down on a bottom sheet modal closes it. Could you please add this feature
https://material.io/components/sheets-bottom

Is this issue still exist?


Some Clarifications regarding Exercise 2/3

Hi, I would like to ask for clarification regarding the following:

in exercise 2 we have to differentiate "early" and "late" stages of the optimization. I couldn't find any indication as to how these are actually defined so I set the early stages to the first 25% of the iterations. This passes the test but seems a bit arbitrary to me. Are we supposed to decide this number on our own or did I miss a variable that indicates a number for this in the code?
the exercise 3 description defines the temporal smoothness loss for joint positions as the sum of all the absolute distances. Is this supposed to be the mean instead of the sum in the implementation? If I just sum up all the values I get gigantic loss values leading to unstable optimization. Using the mean works much better but technically doesn't match the exercise description.

Yes, you are supposed to decide this number by yourselves. There is no absolute definitions on the "early" and "late" stage.
Using "mean" or "sum" is what you need to explore. The exercise description is just an example. It can be formulated in that way, but that is not the only way.


Improve subquery handling

#90 fixes subquery handling in Serialize Result but Compute Struct also have subqueries with Scalar link type because they are similar operators.
https://cloud.google.com/spanner/docs/query-execution-operators?hl=en#serialize_result

A serialize result operator is a special case of the compute struct operator

This PR fixes the bug and simplify IsVisible related logics:

Relational operators is always rendered.
Scalar operators isn't rendered except subquery operators.
Subquery operators are appeared with Scalar link type.

(Subqueries are usually optimized, so subquery operators don't appear in other cases AFAIK.)

Before
SELECT si.*,
  ARRAY(SELECT AS STRUCT a.*,
        ARRAY(SELECT AS STRUCT so.*
              FROM Songs so
              WHERE a.SingerId = so.SingerId AND a.AlbumId = so.AlbumId)
        FROM Albums a
        WHERE a.SingerId = si.SingerId)
FROM Singers si;

+-----+---------------------------------------------------------+
| ID  | Query_Execution_Plan (EXPERIMENTAL)                     |
+-----+---------------------------------------------------------+
|   0 | Distributed Union                                       |
|   1 | +- Local Distributed Union                              |
|   2 |    +- Serialize Result                                  |
|   3 |       +- Table Scan (Full scan: true, Table: Singers)   |
|  14 |       +- [Scalar] Array Subquery                        |
|  15 |          +- Local Distributed Union                     |
|  16 |             +- Compute Struct                           |
| *17 |                +- FilterScan                            |
|  18 |                |  +- Table Scan (Table: Albums)         |
|  31 |                +- Array Subquery                        |
|  32 |                |  +- Local Distributed Union            |
|  33 |                |     +- Compute Struct                  |
| *34 |                |        +- FilterScan                   |
|  35 |                |           +- Table Scan (Table: Songs) |
|  31 |                +- [Scalar] Array Subquery               |
|  32 |                   +- Local Distributed Union            |
|  33 |                      +- Compute Struct                  |
| *34 |                         +- FilterScan                   |
|  35 |                            +- Table Scan (Table: Songs) |
+-----+---------------------------------------------------------+
Predicates(identified by ID):
 17: Seek Condition: ($SingerId_1 = $SingerId)
 34: Seek Condition: (($SingerId_2 = $SingerId_1) AND ($AlbumId_1 = $AlbumId))
 34: Seek Condition: (($SingerId_2 = $SingerId_1) AND ($AlbumId_1 = $AlbumId))

After
+-----+---------------------------------------------------------+
| ID  | Query_Execution_Plan (EXPERIMENTAL)                     |
+-----+---------------------------------------------------------+
|   0 | Distributed Union                                       |
|   1 | +- Local Distributed Union                              |
|   2 |    +- Serialize Result                                  |
|   3 |       +- Table Scan (Full scan: true, Table: Singers)   |
|  14 |       +- [Scalar] Array Subquery                        |
|  15 |          +- Local Distributed Union                     |
|  16 |             +- Compute Struct                           |
| *17 |                +- FilterScan                            |
|  18 |                |  +- Table Scan (Table: Albums)         |
|  31 |                +- [Scalar] Array Subquery               |
|  32 |                   +- Local Distributed Union            |
|  33 |                      +- Compute Struct                  |
| *34 |                         +- FilterScan                   |
|  35 |                            +- Table Scan (Table: Songs) |
+-----+---------------------------------------------------------+
Predicates(identified by ID):
 17: Seek Condition: ($SingerId_1 = $SingerId)
 34: Seek Condition: (($SingerId_2 = $SingerId_1) AND ($AlbumId_1 = $AlbumId))

I have found more edge case of subquery.
Generally, many operators can have Scalar typed subquery so this PR seems works well.
SELECT so.* FROM Songs so
WHERE IF(so.SongGenre = "ROCKS", TRUE, EXISTS(SELECT * FROM Concerts c WHERE c.SingerId = so.SingerId))

+-----+----------------------------------------------------------------------+
| ID  | Query_Execution_Plan (EXPERIMENTAL)                                  |
+-----+----------------------------------------------------------------------+
|   0 | Distributed Union                                                    |
|   1 | +- Local Distributed Union                                           |
|   2 |    +- Serialize Result                                               |
|  *3 |       +- FilterScan                                                  |
|   4 |          +- Table Scan (Full scan: true, Table: Songs)               |
|  16 |          +- [Scalar] Scalar Subquery                                 |
|  17 |             +- Global Stream Aggregate (scalar_aggregate: true)      |
| *18 |                +- Distributed Union                                  |
|  19 |                   +- Local Stream Aggregate (scalar_aggregate: true) |
|  20 |                      +- Local Distributed Union                      |
| *21 |                         +- FilterScan                                |
|  22 |                            +- Index Scan (Index: ConcertsBySingerId) |
+-----+----------------------------------------------------------------------+
Predicates(identified by ID):
  3: Residual Condition: IF(($SongGenre = 'ROCKS'), true, $sv_1)
 18: Split Range: ($SingerId_1 = $SingerId)
 21: Seek Condition: ($SingerId_1 = $SingerId)


Can't download: 'Progress' has no attribute 'get_default_columns'

(geosynth) ➜  geosynth python -m geosynth download non-hdr --dst=./ --variant=demo
Traceback (most recent call last):
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/geosynth/__main__.py", line 3, in <module>
    run_app(prog_name="geosynth")
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/geosynth/cli/main.py", line 36, in run_app
    app(*args, **kwargs)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/typer/main.py", line 328, in __call__
    raise e
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/korny/micromamba/envs/geosynth/lib/python3.9/site-packages/geosynth/cli/download.py", line 32, in download
    *Progress.get_default_columns(),  # type: ignore[reportGeneralTypeIssues]
AttributeError: type object 'Progress' has no attribute 'get_default_columns'

Sorry for the late response, turns out an insufficient lower bound was put on the dependency rich. We require at least version v11.2.0. This has been fixed in geosynth v1.0.1.


Forward exceptions on creating execute observable to ThrownExceptions

If the executeAsync func throws an exception while creating the observable (i.e. synchronously), then this exception is lost by frameworks relying on ICommand.Execute - this swallows any exceptions returned from ReactiveCommand.ExecuteAsync.
This change ensures these exceptions are forwarded to ThrownExceptions, which would seem to be the expected behaviour.
Fixes #973

:+1: LGTM

Thanks for the contribution @csmager


Documentation about internal process.

We probably should have documentation for how this bundle works for future development.

Agreed.


FAB menu toggle listener being called twice when .open(true) is called on FAB menu object

Works ok on v1.5.4. The toggle listener is only being called twice on v1.5.5.

Could you provide the code? I can't reproduce this.


Using Coq for proofs does not work

After following the instructions to install Archetype (on Ubuntu 20.04) and installing why3-coq and coqide, the generated Coq file gives errors such as:
The reference BuiltIn.string was not found
in the current environment.

To reproduce:
Try to verify a goal of the following (once converted to whyml and having chosen to prove a goal using Coq):

coq_fail_example.arl

archetype trivial_test

entry exec () {
  effect {
    ()
  }
}

function test(x : int) : bool {
  specification {
    postcondition testSpec {
      result = true <-> x < 10
    }
  }

  effect {
    return (x < 10)
  }
}

Specifically,
Install archetype, why3-ide, why3-coq, and coqide. And VSCode with the Archetype extension
Run >Archetype: Verify with Why3 in VSCode on coq_fail_example.arl (shown above, note the extension of this upload is .txt rather than .arl due to GitHub).
This relates to https://github.com/edukera/vscode-archetype/issues/8 but here is raised as a general problem with using Archetype with Coq and I have reproduced the issue on Ubuntu with a standard installation rather than just NixOS. I also intend to raise this issue on the why3-club mailing list as it may just be a why3 issue.
I am really keen to try out what seems to be an amazing project using Coq as I feel that at some critical times, interactive proofs are necessary and useful. :)

Hi,
Thanks for the report. What are your versions of Why3 & Coq?

Hi @strub,
Coq 8.12.0 and Why3 1.3.3
(Though I have also tried with other versions of Coq and Why3 with the same result).

I have asked for help relatedly in the why3-club mailing list with a simplified example: https://lists.gforge.inria.fr/pipermail/why3-club/2020-September/002264.html

Apparently it's a bug in Why3: https://lists.gforge.inria.fr/pipermail/why3-club/2020-September/002265.html

See my fix here: https://github.com/Coda-Coda/why3-fork/commit/f1b5c3550b4b47ef40a9205e41fca1da6661c0a1 (requires building why3 from source with this lib/coq/BuiltIn.v altered as per the commit).


Bump dependencies and parser refactor

Bump php to 8+ versio
Bump dependencies
Refactor the parser for compatibility with the new nikic/php-parser version. Now the parser infers the types firstly from the code and then from the docblock

By now, tests are not green and there could be some code style errors.

@gossi I'm back from vacation and I start again to work on our libraries.
I'ld like to know your opinion about libraries namespace:

continue with gossi namespace (gossi\docblock, gossi\php-code-generato etc.)
change to phpowermove (phpowermove\docblock, phpowermove\php-code-generator etc.)

If we change the namespace, we should update packagist, too.

I think makes more sense to use the new namespace. Usually as an end User you use the package name and copy it to github.com/ to reach for the repo. That's a nice "ui" to it, which I think is valuable to have.
Feel free to change them all.
Do they restart as v1? Or do they continue with their current version number? 🤔

Imho we should continue to the current version number. Namespace change introduce a bc break so we should bump to the next major version.
On Packagist, it could be nice to have gossi/docblock for current version (so that a user can continue with it since he'll decide to update both version and namespace) and phpowermove/docblock starting from next version. I don't know if it's possible and how. Anyway, we'll write this change in the documentation.

Yeah, makes sense. Go for it.
On packagist, you can mark a package "abandonned" and set the successor to it. That's what packagist is offering. Dunno if they "announce" it on the cli, when running the install command.

Yeah, makes sense. Go for it.
[cut]
Dunno if they "announce" it on the cli, when running the install command.
Yes, they do.

I'm gonna close this PR: I'll re-submit it later, after changing the namespace. I started the update of the first library https://github.com/phpowermove/docblock/pull/18
:crossed_fingers:


Fix: GitHub Workflow trigger

GitHub Workflow was triggered only when push event was fired.
Now it is also triggered when pull_request event is fired.
This enables external contributors to run tests when they send pull requests.
(push event is fired when push is done to forked repository, but push event is not fired when push is done to kitsuyui's repository)

Codecov Report
Patch and project coverage have no change.

Comparison is base (710b767) 100.00% compared to head (ece024f) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #53   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          178       178
=========================================
  Hits           178       178

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

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

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


Results from [for item in items(object): { }] are ordered differently to the original order

Bicep version
Bicep 0.4.1008
Description
When using the item(parentObject) syntax in a FOR loop the resulting order the objects are processed does not match the order of the objects in the parentObject
To Reproduce
Given a JSON object in a parameter file:
"VM": {
  "NICs": {
    "first": {
       "key": "value"
    },
    "alternate": {
       "key": "value"
    }
  }
}

and Bicep variable declaration referencing the object in a for statement using the items(object) syntax:
var NICs = [for nic in items(VM.NICs): {
  ...do stuff...
}

results in NICs being an array with the objects in alphabetical order (alternate then first) instead of the order they were originally.
Doing a little more digging I see the Bicep documentation states the items() function creates a dictionary of objects which from experience explains the ordering. If this is the final format for the items() function then it would be great if the documentation mentioned the ordering. Also it doesn't necessarily need to be ordered, an unordered dictionary would also work and the results could be as expected.
I also see the ARM template generated by Bicep uses an ARM items() function which doesn't seem to exist in the ARM Functions documentation.

If this is the final format for the items() function then it would be great if the documentation mentioned the ordering.

Agreed this would be good to doc -- FYI @stephaniezyen / @mumian / @tfitzmac
I assume this matches some equivalent behavior in .net. Out of curiosity, does the order changing affect the behavior of your deployment? If so, it would be a justification for changing/adding additional functionality i.e. items(VM.nics, { perserveOrdering: true })

I will work on the documentation.

A PR has been created - https://github.com/MicrosoftDocs/azure-docs-pr/pull/182252

I'd like to leave it open for now to understand if this is the intended behavior and, if so, is it something we should change.

This behavior is by design. If it is causing any issues, please let us know and we can revisit the decision.

I'm happy with it ordering alphabetically as a dictionary does.
The main issue I have is that my VMs have a requirement that NICs are built and referenced in the order they are in the parameter file.
In the end I changed to an array to fix it, before I realised it was alphabetical. Had I known sooner I could've renamed the objects to force ordering.
So as long as it's documented no one else should fall into the same trap.
Thanks


Add  new methods to Polyline

I propose this PR in order to add new method to Polyline:

addUniquePoint(): Add a point if not already existing
getMiddlePoint(): Return the middle point of Polyline

Tests are added too.

Hi,
thanks for your pull request!
I'll take a look in the next few days and keep you updated!

Hi,
I've merged your PR and did some refactorings.
Thanks again!

Great!
Thank you to take it :)
All my best!


Question: setup after add-on installation?

Hi,
I just installed this add-on as described in the README. I have a cc2531 USB stick flashed with the firmware to work with zigbee2mqtt.
After I install the add-on, so I have to edit any config file to specify my mosquito server and credentials? Or does this add-on figure all this out on HassOS? I figure I need to sort this out before attempting to add devices. Appreciate any guidance, thank you.

@vinhdizzo if you use https://github.com/zigbee2mqtt/hassio-zigbee2mqtt (different repo) you will get mqtt auto detection.
If you want to use this add-on, you will need to create a user in your home assistant (if you use the integrated mqtt addon) and use those credentials in the config file for the add-on (no file editing on disk, just the regular config in the UI).


Error during installation (Ubuntu 14) - git@github.com:firebase/php-jwt.git, git was not found

When i run the Build command line i´m receiving this issue:

git@github.com:firebase/php-jwt.git, git was not found

root@abbegroup201:/var/www/TaskBoard-master# ./build/build-all
Building...
Installing dependencies
Warning: This development build of composer is over 60 days old. It is recommended to update it by running "./build/composer.phar self-update" to get the latest version.
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file

Installing firebase/php-jwt (1.0.0)
Downloading: 100%
Downloading: 100%
Downloading: 100%
Failed to download firebase/php-jwt from dist: The "https://api.github.com/repos/firebase/php-jwt/zipball/58c1e61d0dff797f2989c54415229f4016d57a6d" file could not be downloaded: php_network_getaddresses: getaddrinfo failed: Name or service not known
failed to open stream: php_network_getaddresses: getaddrinfo failed: Name or service not known
Now trying to download from source
Installing firebase/php-jwt (1.0.0)
Cloning 58c1e61d0dff797f2989c54415229f4016d57a6d

[RuntimeException]
Failed to clone git@github.com:firebase/php-jwt.git, git was not found, check that it is installed and in your PATH env.
sh: 1: git: not found
install [--prefer-source] [--prefer-dist] [--dry-run] [--dev] [--no-dev] [--no-plugins] [--no-custom-installers] [--no-autoloader] [--no-scripts] [--no-progress] [-v|vv|vvv|--verbose] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--ignore-platform-reqs] [packages1] ... [packagesN]

The answer is in the error:

git was not found, check that it is installed and in your PATH env.

You need to install git to clone git repositories (I'm guessing you downloaded TaskBoard rather than cloning it).
If that doesn't work, you can always just try running the build again. It's possible there was a temporary connectivity issue as the download link in the error message just downloaded for me when I clicked it to test.


cli: Allow selecting port for TCP routes

Allows users to specify the port they wish to allocate.
If the port is take it will return a duplicate route error.
Closes #1524

LGTM

Whoot! I was just looking at my backlog this morning, and wondering whether I needed to ask y'all about this . . .


Remove pyyaml 5.4.1 from the docker image

What does this PR do?
Remove pyyaml 5.4.1 from the docker image
Which scenarios this will impact?
Motivation
We don't use pyyaml 5.4.1 anymore and there's no need to build it with this hack anymore. We now use 6.0.2 which pins cython https://github.com/yaml/pyyaml/blob/main/CHANGES#L16
Additional Notes

/merge


jwt implementation

It would be good to keep an eye on this issue:
https://github.com/meanjs/mean/pull/1163
Having a jwt implementation in pean would be great.

Agreed!


Question: Conversion from Modulo to UBig?

I want to perform some arithmetic operations in a ModuloRing then use to_be_bytes on the result.
This is only available on UBig but I couldn't find a way to convert a Modulo to a UBig.

Use Modulo::residue.

Totally missed that, thank you!


KeyError: 'CenterHead is not in the head registry'

(centerpoint) exinova@exinova-B560M-AORUS-PRO-AX:/media/exinova/ssd/CenterPoint-master_py$ python tools/train.py configs/nusc/pp/nusc_centerpoint_pp_02voxel_two_pfn_10sweep.py
2022-12-15 16:35:21,463 - INFO - Distributed training: False
2022-12-15 16:35:21,463 - INFO - torch.backends.cudnn.benchmark: False
2022-12-15 16:35:21,484 - INFO - Finish RPN Initialization
Traceback (most recent call last):
File "tools/train.py", line 168, in
main()
File "tools/train.py", line 141, in main
model = build_detector(cfg.model, train_cfg=cfg.train_cfg, test_cfg=cfg.test_cfg)
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/builder.py", line 53, in build_detector
return build(cfg, DETECTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/builder.py", line 21, in build
return build_from_cfg(cfg, registry, default_args)
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/utils/registry.py", line 82, in build_from_cfg
return obj_cls(**args)
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/detectors/point_pillars.py", line 18, in init
reader, backbone, neck, bbox_head, train_cfg, test_cfg, pretrained
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/detectors/single_stage.py", line 25, in init
self.bbox_head = builder.build_head(bbox_head)
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/builder.py", line 45, in build_head
return build(cfg, HEADS)
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/builder.py", line 21, in build
return build_from_cfg(cfg, registry, default_args)
File "/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/utils/registry.py", line 70, in build_from_cfg
"{} is not in the {} registry".format(obj_type, registry.name)
KeyError: 'CenterHead is not in the head registry'
what is wrong about this ? do anyone knowns?

it seems you are using the Det3D repo instead of the current one
"/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/detectors/point_pillars.py""

it seems you are using the Det3D repo instead of the current one
"/media/exinova/ssd/CenterPoint-master_py/Det3D/det3d/models/detectors/point_pillars.py""

but when i uninstall det3d, error also happend:
(centerpoint) exinova@exinova-B560M-AORUS-PRO-AX:/media/exinova/ssd/CenterPoint-master_py$ python tools/train.py configs/nusc/pp/nusc_centerpoint_pp_02voxel_two_pfn_10sweep.py
Traceback (most recent call last):
File "tools/train.py", line 14, in
from det3d.datasets import build_dataset
ModuleNotFoundError: No module named 'det3d'

add centerpoint's path to pythonpath