Emitter.stream: Is there a way to catch exceptions ?

Consider the following stream with exception:

Stream<int> someStream() async* {
  while (true) {
    await Future.delayed(const Duration(seconds: 1));
    throw Exception("Test exception");
    yield 123;
  }
}

final someStreamEmitter = Emitter.stream((ref) => someStream());

This will produce "[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception"
Is there a way to handle exceptions in stream ? Seems the current implementation  is not passing the onError param to listen()

You are right. We need a better approach on this. I just merged a fix https://github.com/liangxianzhe/creator/commit/e7de5b2be151825c7e7de205e16d2595766c6b73 and published a new version 0.3.2. Tested on the below code, but let me know if you still see issue on have a better approach on the fix.
import 'package:creator_core/creator_core.dart';

Stream<int> someStream() async* {
  while (true) {
    await Future.delayed(const Duration(seconds: 1));
    throw Exception("Test exception");
    yield 123;
  }
}

Future<void> main(List<String> args) async {
  final ref = Ref();
  final someStreamEmitter = Emitter.stream((ref) => someStream());
  try {
    print(await ref.watch(someStreamEmitter));
  } catch (e) {
    print("caught $e");
  }
  print("done");
}

Great. Thanks for the quick fix!


[Feature Request] Can't get Models/Characters through Lua for other themes

I can't get the Models in ScreenGameplay. I did see it's possible to get them with Soundwaves.
Any advices to get them?

The characters in Soundwaves are possible to be obtained as they're being generated on a lua file in ScreenGameplay's background layer. Legacy characters are not possible to be obtained as they are in a separate isolated layer in gameplay and it's not even named, not attached to anything to lua.


added something

added something.....

i am satisfy....

thanks @belal87 ! next time a more descriptive PR would be helpful too, e.g. if your change was formatting just saying that in the title/description is ideal :D


app/ReplicaSet is valid group kind

Signed-off-by: Roke Jung<EMAIL_ADDRESS>https://github.com/open-cluster-management/backlog/issues/12150
It is recommended to use Deployment instead of app/ReplicaSet but ReplicaSet is still a valid kind. We should not exclude it.

/lgtm


Pagination bug: Search with _history  and _count does not include pagination

It seems like a bug, when we do a normal search using _history and _count, we are expecting to have the pagination (link attribute should normally include a relation "next" refering to the next page).
P.S: this is tested on FHIR 4.0.1 and 5.1
For example :
Executed query:
http://hapi.fhir.org/baseR4/MedicationRequest/_history?_count=1
(knowing that if we increased the count, we will have extra resources)
The result is:
{
"resourceType": "Bundle",
"id": "db37220d-b67d-4aeb-acc9-f080ea15200a",
"meta": {
"lastUpdated": "2020-10-02T16:45:54.501+00:00"
},
"type": "history",
"total": 8181,
"link": [ {
"relation": "self",
"url": "http://hapi.fhir.org/baseR4/MedicationRequest/_history?_count=1&_pretty=true"
} ],
"entry": [ {
"fullUrl": "http://hapi.fhir.org/baseR4/MedicationRequest/1495458",
"resource": {
"resourceType": "MedicationRequest",
"id": "1495458",
"meta": {
"versionId": "1",
"lastUpdated": "2020-10-01T13:19:24.698+00:00",
"source": "#yslsxRY7OBkp3r9T"
},
"status": "stopped",
"intent": "order",
"medicationCodeableConcept": {
"coding": [ {
"system": "http://www.nlm.nih.gov/research/umls/rxnorm",
"code": "1043400",
"display": "Acetaminophen 21.7 MG/ML / Dextromethorphan Hydrobromide 1 MG/ML / doxylamine succinate 0.417 MG/ML Oral Solution"
} ],
"text": "Acetaminophen 21.7 MG/ML / Dextromethorphan Hydrobromide 1 MG/ML / doxylamine succinate 0.417 MG/ML Oral Solution"
},
"subject": {
"reference": "Patient/1495412"
},
"encounter": {
"reference": "Encounter/1495443"
},
"authoredOn": "2011-02-17T01:27:51-05:00",
"requester": {
"reference": "Practitioner/1495414",
"display": "Dr. Dagny669 Wyman904"
},
"reasonReference": [ {
"reference": "Condition/1495444"
} ]
},
"request": {
"method": "POST",
"url": "http://hapi.fhir.org/baseR4/MedicationRequest/1495458/_history/1"
},
"response": {
"status": "201 Created",
"etag": "W/"1""
}
} ]
}

This issue was reported by Logica users after we upgraded to HAPI 5.2.0 from HAPI 4.


Update text styles and navigation for CC Search guidelines page.

Fixes

Fixes #399 by @panchovm
Description

This fixes the text style and the navigation of the CC search guidelines page.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I had to update the Table of progress loop as there were three titles in each block it created 30 steps. I feel the code is little flaky but I am up for suggestions.


Adding to a project

Please am new to Maui. Please how do i add this to my project. You only describe how to add on page. Do i need to recreate BottomSheet the project om my project.

You can add the reference to this project. Plz see the sample test app in the repo for example.


feat: Add ToolsAgent for models with tool-calling support

Description
This pull request introduces a new ToolsAgent that simplifies the integration of tools into agents powered by language models such as Ollama and OpenAI. The agent supports tool-driven workflows, enhancing the flexibility of LangChain agents.
Dependencies

Added dependencies for langchain_community, langchain_openai, and langchain_ollama.

Testing

Unit tests for the ToolsAgent have been added, covering various tools and scenarios to ensure robust functionality.
Example tests demonstrate integration with the CalculatorTool and memory.

@davidmigloz & @Ganeshsivakumar

@Heinrich-vanNieuwenhuizen Let me know if you have any comments on my changes, otherwise I'll merge it!

a3.1, as it's a more powerful model than llama3-groq-tool-use for most use ca

Hi @davidmigloz , do your tests pass with llama3.1 mine only works with llama3-groq-tool-use. I
I had mixed results without "Removing the custom tool-calling prompt, as the providers already handle this. "

do your tests pass with llama3.1 mine only works with llama3-groq-tool-use?

I left llama3-groq-tool-use for the tools_test.dart. Although with llama3.1 all the tests pass except the "Test ToolsAgent with messages memory" test, which is a known limitation of llama3.1:

We recommend using Llama 70B-instruct or Llama 405B-instruct for applications that combine conversation and tool calling. Llama 8B-Instruct can not reliably maintain a conversation alongside tool calling definitions. It can be used for zero-shot tool calling, but tool instructions should be removed for regular conversations between the model and the user.

I had mixed results without the custom tool-calling prompt

If we add that custom prompt, the actual prompt that arrives to the model will contain two tools definitions. The one from our custom prompt plus the tool calling prompt that Ollama adds (or the other providers). So that will probably confuse the model (and add many unnecessary input tokens). The custom prompt should only be necessary for models that don't support tool calling. In that case, the user can provide a custom system prompt:
final agent = ToolsAgent.fromLLMAndTools(
  llm: llm,
  systemChatMessage: <their custom prompt>,
);

@Heinrich-vanNieuwenhuizen Let me know if you have any comments on my changes, otherwise I'll merge it!

Okay looks good you can merge

Merged! Thanks again for your contribution @Heinrich-vanNieuwenhuizen 🙌
I'll release it this week.

I've tested it with mistral-nemo, and it's also working pretty well.


Does not report correct bowers version for Chrome 41

Sites using this report Chrome 41.0.2272.118 (Official Build) m as Chrome 27 and out of date.  I was able to verify this issue on a number of sites that use this script.

They were all using something thing that modified the user agent.  Once I disabled these, the issue was resolved.


Debug adapter process has terminated unexpectedly

Issue Type: Bug
I am getting error Debug adapter process has terminated unexpectedly
after updating my node version 6.** to 8.** . kindly help in this
My Launch.js file
{
  "version": "0.2.0",
    "configurations": [
         {
            "type": "node",
            "request": "launch",
            "name": "Launch Program",
            "program": "${workspaceFolder}\\server.js"
        }
    ]
}

VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:46:47.128Z)
OS version: Windows_NT x64 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Core(TM)2 Quad CPU    Q8400  @ 2.66GHz (4 x 2660)

Memory (System)
6.96GB (0.92GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (16)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.0

path-intellisense
chr
1.4.2

jshint
dba
0.10.17

vscode-eslint
dba
1.4.7

xml
Dot
1.9.2

tslint
eg2
1.0.28

vscode-npm-script
eg2
0.3.3

node-version
fiv
0.0.1

beautify
Hoo
1.3.0

sublime-babel-vscode
jos
0.2.9

vscode-JS-CSS-HTML-formatter
lon
0.2.3

csharp
ms-
1.14.0

node-debug2
ms-
1.19.4

team
ms-
1.122.0

debugger-for-chrome
msj
4.2.1

react-beautify
tai
0.3.0

Reproduces only with extensions

Is there any output in the debug console? Can you set "trace": true and see whether it writes a log file path to the debug console?
@isidorn I see a few of these, any tips to get more info from the crash?

@roblourens https://github.com/Microsoft/vscode/issues/41190

i am getting this

No logs, no debug info. WHere I can find it?

but i am able to debug same project in webstrom

:+1:
I am experiencing the same error message,  with VSCode 1.21 and after an update to 1.23 the issue is still here.
In my case the message shows up with chrome Debug extension (launch request on a simple url) on a Windows 7 desktop.

@weinand Sorry for hijacking the issue. I'm also investigating an error like this. In my case everything works when running in hosted mode, i.e. extension host from source. However, when I have Code (or Code Insider) with the extension installed from vsix the debug adapter fails immediately - no output logged anywhere. All my code is within try/catch so I wonder how to debug the issue. I suspect a dependency issue because we identified the commit that broke it.

@riturajbarhgoojer is this still an issue?
Better logs for this error should have come with #41190. If you see any more error info now, please let me know.


Make can't find xcbgen, existing discussions didn't help

Checklist

[x] I have followed every step on the compiling wiki page and
installed all necessary dependencies.
[ ] My problem is not described on the known issues page
[x] I have searched for other open and closed issues that
may have already reported this problem.
[x] I was able to reproduce this build issue in a clean build

Build Process
How did you build polybar?

[ ] From a release archive (Which version?)
[x] By cloning this repository (What is the of git describe --tags?)
[ ] Some other way (How?)

git clone --recursive https://github.com/polybar/polybar
cd polybar
mkdir build
cd build
cmake .. -DPYTHON_XCBGEN=/usr/lib/python2.7/dist-packages/

# i am setting xcbgen manually as suggested here:
# https://github.com/polybar/polybar/issues/2031
# without this cmake cant find xcbgen either even though it exists in python3.9  folder

make -j$(nproc) # build failed

i have also tried with
yay -S polybar

yay can't find xcbgen either
Build Log
[  1%] Generating generated-sources/include/xpp/proto/xkb.hpp
[  2%] Generating generated-sources/include/xpp/proto/randr.hpp
Scanning dependencies of target doc_man
Scanning dependencies of target doc_html
[  3%] Generating generated-sources/include/xpp/proto/composite.hpp
Scanning dependencies of target polybar-msg
[  4%] Generating generated-sources/include/xpp/proto/x.hpp
[  5%] sphinx-build man: see doc/builder-man.log
[  6%] sphinx-build html: see doc/builder-html.log
[  8%] Building CXX object bin/CMakeFiles/polybar-msg.dir/ipc.cpp.o
[  8%] Building CXX object bin/CMakeFiles/polybar-msg.dir/utils/env.cpp.o
Traceback (most recent call last):
  File "/usr/bin/sphinx-build", line 33, in <module>
    sys.exit(load_entry_point('Sphinx==3.3.1', 'console_scripts', 'sphinx-build')())
  File "/usr/bin/sphinx-build", line 22, in importlib_load_entry_point
    for entry_point in distribution(dist_name).entry_points
  File "/usr/lib/python3.8/importlib/metadata.py", line 504, in distribution
    return Distribution.from_name(distribution_name)
  File "/usr/lib/python3.8/importlib/metadata.py", line 177, in from_name
    raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: Sphinx
Traceback (most recent call last):
  File "/usr/bin/sphinx-build", line 33, in <module>
    sys.exit(load_entry_point('Sphinx==3.3.1', 'console_scripts', 'sphinx-build')())
  File "/usr/bin/sphinx-build", line 22, in importlib_load_entry_point
    for entry_point in distribution(dist_name).entry_points
  File "/usr/lib/python3.8/importlib/metadata.py", line 504, in distribution
    return Distribution.from_name(distribution_name)
  File "/usr/lib/python3.8/importlib/metadata.py", line 177, in from_name
    raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: Sphinx
Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
  File "/home/enchilada/polybar/lib/xpp/generators/cpp_client.py", line 3096, in <module>
  File "/home/enchilada/polybar/lib/xpp/generators/cpp_client.py", line 3096, in <module>
Traceback (most recent call last):
  File "/home/enchilada/polybar/lib/xpp/generators/cpp_client.py", line 3096, in <module>
    from xcbgen.state import Module
ModuleNotFoundError: No module named 'xcbgen'
    from xcbgen.state import Module
ModuleNotFoundError: No module named 'xcbgen'
    from xcbgen.state import Module
ModuleNotFoundError: No module named 'xcbgen'
make[2]: *** [doc/CMakeFiles/doc_man.dir/build.make:77: doc/CMakeFiles/doc_man] Error 1
  File "/home/enchilada/polybar/lib/xpp/generators/cpp_client.py", line 3096, in <module>
make[1]: *** [CMakeFiles/Makefile2:466: doc/CMakeFiles/doc_man.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
    from xcbgen.state import Module
ModuleNotFoundError: No module named 'xcbgen'
make[2]: *** [doc/CMakeFiles/doc_html.dir/build.make:77: doc/CMakeFiles/doc_html] Error 1
make[1]: *** [CMakeFiles/Makefile2:493: doc/CMakeFiles/doc_html.dir/all] Error 2
make[2]: *** [lib/xpp/CMakeFiles/xpp.dir/build.make:81: lib/xpp/generated-sources/include/xpp/proto/composite.hpp] Error 1
make[2]: *** Deleting file 'lib/xpp/generated-sources/include/xpp/proto/composite.hpp'
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [lib/xpp/CMakeFiles/xpp.dir/build.make:93: lib/xpp/generated-sources/include/xpp/proto/x.hpp] Error 1
make[2]: *** Deleting file 'lib/xpp/generated-sources/include/xpp/proto/x.hpp'
make[2]: *** [lib/xpp/CMakeFiles/xpp.dir/build.make:89: lib/xpp/generated-sources/include/xpp/proto/xkb.hpp] Error 1
make[2]: *** Deleting file 'lib/xpp/generated-sources/include/xpp/proto/xkb.hpp'
[  9%] Building CXX object bin/CMakeFiles/polybar-msg.dir/utils/file.cpp.o
[ 10%] Building CXX object bin/CMakeFiles/polybar-msg.dir/utils/string.cpp.o
make[2]: *** [lib/xpp/CMakeFiles/xpp.dir/build.make:85: lib/xpp/generated-sources/include/xpp/proto/randr.hpp] Error 1
make[2]: *** Deleting file 'lib/xpp/generated-sources/include/xpp/proto/randr.hpp'
make[1]: *** [CMakeFiles/Makefile2:519: lib/xpp/CMakeFiles/xpp.dir/all] Error 2
[ 11%] Linking CXX executable polybar-msg
[ 11%] Built target polybar-msg
make: *** [Makefile:149: all] Error 2

Environment:
ArchLinux version 5.9.11-arch2-1
Additional context
So i've searched every open issue and discussion about this, since there is a lot., none of them were able to solve my issue.
I'm not using anaconda nor pyenv
I've tried to build manually and with aur helpers
i've tried installing python2 packages
xcbgen exists in usr/lib/python3.9/site-packages
I've even tried installing xcbgen via apt, which i know i shouldn't do on arch

First of, the following should always work on Arch:
# Clone the AUR repository (https://aur.archlinux.org/polybar-git.git for the polybar-git package should work as well):
git clone https://aur.archlinux.org/polybar.git
cd polybar # or polybar-git
extra-x86_64-build # Builds polybar in a clean chroot.

This builds the polybar package on a clean system. If that works but it doesn't directly on your system, the issue likely is with your system setup. Most often those "issues" would be pyenv or conda that shadow the system python executables, but since you don't use those, there must be another issue.

Now, I see a few issues with your machine, any one of them could be responsible for this issue:

Your system has the file /usr/lib/python3.8/importlib/metadata.py, this suggests you are using python 3.8, but since the xcbgen files are in /usr/lib/python3.9/site-packages, your system is also using python 3.9. Since you're on arch, you should really only have python3.9 files (at least in /usr/lib).
"I've even tried installing xcbgen via apt, which i know i shouldn't do on arch":
This is not a good idea at all. Installing any apt package is bound to break your system one way or another.

There is definitely something borked in your system, my best guess is that it is because both the python and python3 executables are actually for 3.8 and not 3.9, this would explain why polybar couldn't find xcbgen during cmake.

i've tried installing python2 packages

You should really only need python2 for building polybar if your distro doesn't have xcbgen for your python3 version.

You were right, thank you!
I uninstalled every python package with
sudo pacman -Rcns python
Then yay was able to build it successfully


Remove fastclick

Not needed anymore.

But it is needed for old iOS devices...

I will probably move it into optional tools wiki page.

would be good to leave a caveat in the wiki that fastclick is still needed for homescreen webapps in ios8


Add new exercise trinary

As described in #98

The last one has been merged (and shown as Merged :wink:)
Thanks, thanks a lot for that impressive work @matthewmorgan. Now, time to enjoy on exercism.io with lots of new exercises to solve and comment.


Limit the bonder’s ability to bond transfers whose amount is greater than the expected hToken amount on the source chain

The bonder should not bond a transfer that sends more hTokens than have been deposited into the source chain. This would protect against the case where a chain has a bug and a malicious user was able to mint ETH/Tokens/etc. out of thin air, among other things.
An example is an attack where an attacker can somehow mint hTokens unexpectedly and try to send them to a chain. For example, imagine there are 1,500 hETH on Optimism at this time. The bonder software should not bond any transfer that is ≥1,500 ETH on any chain when the source is Optimism because, if they did, that would mean that  the attacker was able to mint hTokens unexpectedly.
The implementation can likely use the chain balance and pending amount from the chain.

This old PR could use a review to handle this https://github.com/hop-protocol/hop/pull/413

Handled with new redundant logic.


CI: bump the CI travis's go to 1.10

Bump the CI travis's go to 1.10 to meet
the building kata shim v2 for containerd.
Fixes: 575
Signed-off-by: fupan<EMAIL_ADDRESS>

@lifupan We had a glitch in our CI today. Can you repush so that CI is triggered again.

@amshinde   re-pushed.

lgtm


Adding missed release notes.

The last release was hurried, and we missed a good chunk of the release notes that should have been part of these docs.
Note, once https://github.com/Azure/azure-cli/issues/8592 is finished, the process will be much simplified and we shouldn't have this sort of mistake again.

Docs Build status updates of commit 9f9e532:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit de4ec52:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Uncaught error when disconnecting async Stream

# auth and listener configured normally
stream = Stream(auth, listener)
stream.filter(track=[...], async=True)
stream.disconnect()

This throws an exception:
In [7]: Exception in thread Thread-2:
Traceback (most recent call last):
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 808, in __bootstrap_inner
self.run()
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 761, in run
self.__target(*self.__args, **self.__kwargs)
File "/Users/pikeas/.virtualenvs/play/lib/python2.7/site-packages/tweepy/streaming.py", line 173, in _run
self._read_loop(resp)
File "/Users/pikeas/.virtualenvs/play/lib/python2.7/site-packages/tweepy/streaming.py", line 226, in _read_loop
self._data(next_status_obj)
File "/Users/pikeas/.virtualenvs/play/lib/python2.7/site-packages/tweepy/streaming.py", line 203, in _data
if self.listener.on_data(data) is False:
File "/Users/pikeas/Documents/Code/play/play_django/frankie/twit.py", line 33, in on_data
if 'in_reply_to_status_id' in data:
TypeError: argument of type 'int' is not iterable

Cleaning up this old issue. PR was https://github.com/tweepy/tweepy/pull/407


⚠ Bofrost.de has degraded performance

In 53867c4, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1308 ms

Resolved: Bofrost.de performance has improved in d50a5be after 20 minutes.


[$500] Welcome message received twice when joining the room and again when the role changed

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:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @kadiealexander
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1708647751662189
Action Performed:

Log in to OldDOt as user A
Create a collect workspace
Add user B as an employee
Log in to NewDot as user A
Create a room under the workspace created in step 2 and invite B
Log in as user B and confirm there is a welcome message in the room invited by user A
In OldDot update the user B role to Workspace admin from employee logging in as user A

Expected Result:
User B should not see a 2nd welcome message in the room
Actual Result:
There is a 2nd welcome message received for user B
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0169f2e49ea854c33a
Upwork Job ID:<PHONE_NUMBER>210408448
Last Price Increase: 2024-02-27

Nice - pending proposals!

Apologies, I've been OOO sick, but getting to this today! @srikarparsi can you confirm the proposal and BE need?

Hey @dylanexpensify! Yes, this does look like this needs a BE change. Do you think you could check if there's a wave/vip it fits into. I'm working on two wave issues currently so will probably not be able to pick this up but maybe a volunteer would be willing?

Unassigning myself as I won't be able to work on this this week.

Yes please @hoangzinh!!

Could not reproduce this time, @kadiealexander @hoangzinh can you confirm if you can?

@dylanexpensify Does this go under some specific wave? This is probably the regression of something

@pecanoro yeah this belongs to #wave-collect I'd say! Or VIP-VSB, but confirming

I was discussing this with @srikarparsi and we are not sure if the internal engineer has to fix it or the BZ member has to post in the corresponding wave issue to find a volunteer. Because of that, I am assigning back to @srikarparsi and I will ask in Slack.

Thanks @pecanoro for posting about the BZ process here! @dylanexpensify do you think you could check see if there's any volunteers available to work on this? My plate is currently pretty full right now so I think I'll only be able to get to this late next week or the week after. But I do think it needs to be fixed internally.

@srikarparsi I can do!

Posted here!

bumped again

Waiting for internal

Will bump again today

Not a huge priority yet

Same as above, waiting for someone to take

@m-natarajan can you repro still?

Bump @m-natarajan

Bump @m-natarajan

Bump @m-natarajan

@lanitochka17 can you check for us here, please?

@dylanexpensify I am not able to reproduce this issue on the latest build 1.4.73-7

but if someone has a reproduction, I’ll write again

@dylanexpensify The second tester also could not reproduce the error because No welcome message in the room.
(expensifail and gmail account)
https://github.com/Expensify/App/assets/78819774/d879ef95-a610-4a13-a910-b878c1966a49
https://github.com/Expensify/App/assets/78819774/6a1b5988-8d88-4e50-96fa-daf51f6c929a


JBPM-5233: Data source management

- changes the deployment id separator to avoid WF conflicts in windows systems

Hello @pefernan would you mind review this PR?
Thanks!

+1 (when green)

@pefernan  @manstis
Suggestion added, ready to merge from my side.
Thanks

Hello @csadilek , I don't no why this PR is still in the waiting queue, but it's ready to merge.

@wmedvede probably because those that are able to merge PRs do not constantly scan open PRs across all repositories waiting for either comments to be resolved or builds to go green. IMO it's up to the person who raised a PR(s) to follow the submission through review to merge and closure. If that person needs to request reviews or merge (when reviewed and green) they that is their responsibility.


Implement Protect integration tests

The motivation here is these integration tests will serve as replacements for the Protect unittests as those unittests are filled with implementation details of the old implementation.
I put the new tests in with the cassandra int tests as that is the only protocol that attempts to support protect at the moment.
When new protocols support it they should get their own int tests for it.
I added a new protect feature to allow us to write integration tests for protect while not including it in the public shotover API.
#[cfg(test)] does not allow this as it is only enabled during unit tests not integration tests.
I manually verified that the integration test tests the encryption code path as far as the current Protect implementation will go, but hopefully the message details rewrite will remove a lot of broken stuff and allow this test to actually test that encryption/decryption occurs.

Why are we not including the Protect transform in the public API?


[POC] Implement credentials mapping from CAPI to Rancher for AWS #5

We would like to prove out credentials mapping from CAPI to Rancher cloud credentials.
We can use AWS credentials to prove this. We'd need to watch for AWSClusterStaticIdentity and create am AWS cloud-credentials in the Rancher manager cluster (via API see https://github.com/rancher-sandbox/rancher-turtles/issues/3 or direct see https://github.com/rancher-sandbox/rancher-turtles/issues/4).

Until we decide if we need bi-directional mapping this can be closed.


Call video.play() in dispatched 'loadeddata' event

It works in android XPERIA Z5 Android 7.1.1 Chrome 60.0.3497.100 .

iOS
Android

that's yosasou

works on production!!🎉 🎉


Update swagger_to_sdk_config.json

Update ADLA job Swagger spec location to the latest, which is the right version to be using for job.

Coverage remained the same at 57.959% when pulling 7acd0c7daf87babbae548fddc3af384dc155345a on begoldsm:patch-1 into d9cd81cfb310d3bdecccc1ad29cd291f0acb2216 on Azure:dev.


[WIP] refactor(GUI): remove angular dependency from drive scanner

Remove the Angular dependency from DriveScanner and with it the service,
exposing it through the module directly.
Changelog-Entry: Remove Angular dependency from DriveScanner.

The changes look great. I think we can merge as soon as the tests pass.

Since it's kind of hard to read the diffs here with the indentation changes, the only important changes should be using driveScanner.once instead of driveScanner.on in the tests, because there were some issues with done() being called several times on the first test using driveScanner.on and that killed the subsequent tests as well.


[EOSF-238] Change scielo text to black and only have board container if it exists

Ticket
https://openscience.atlassian.net/browse/EOSF-238
Purpose & Changes
Fix bug on scielo, changing color to black to fix a lot of visibility issues, and only have the advisory container if the current provider actually has an advisory board.

@hmoco Please do screenshots for these things so we can see what has changed.


Implement refactoring for C# scripts files from within Rider

Currently, when a C# (or GDScript) file is renamed outside of the Godot Editor, for example from within Rider, the project breaks.
Godot has no concept of what was renamed and any nodes that had the renamed script file attached are now broken and need to be manually fixed.
In any reasonably large project, this can introduce tons of issues and bugs whenever something is renamed outside of the Godot Editor.
This also prevents taking full advantage of Rider's excellent refactoring possibilities and instead makes those "risky", because accidentally using them might break your project.
Scene files and other resources have recently had UIDs added to them, see this issue for reference: https://github.com/godotengine/godot/pull/50786#issue-951837271
Unfortunately, the same is not yet true for script files, as this metadata would need to be inserted directly into the text of the script file. It has been discussed, but no solution has been realized yet and it is unclear if there will ever be a solution to this.
For that reason, I would really appreciate it if somehow the Godot Rider plugin could automatically fix all affected scene files whenever a C# script file is renamed.
E.g. when ExampleScript.cs is renamed to AnotherExampleScript.cs, the plugin could iterate through all scene files of the project and detect a match of the old name and replace it with the new one.
This is obviously a very naive solution and I'm not 100% sure if this would work in all cases, but it just might.
Any discussion on this would be much appreciated, as I am super looking foward to extensively working with both Rider and Godot/C# in the future and would like that experience to feel just as good as working with Rider and Unity feels today.
I feel, given that Godot 4 Beta is now under active development, now would be a great time to get started on exploring further options to improve this (already very helpful) plugin.

Any update on this? Rename refactoring is a big thing throughout the development of iterations. Switching back to Godot Editor just for renaming is quite unfortunate and frustrating at the moment.


Window version not working

To my delight, today I found that we have now option to install on Windows. Thanks you @eveningkid .
I am facing issue where I am not able to do anything like creating new files. Only black screen coming. I am providing the screenshot how it is looking after installation.
Thanks again for making this and then for window setup. I know it will be resolved soon.

Thank you for your kind words. You should also thank @Pegase745, @plxel, and @albemala who mostly contributed to making Reacto available on all platforms. :)
I sadly have no way to test it on Windows. Two leads to work on this:

open the development tools console (View, Toggle Developer Tools) and share a screenshot of what you see
inside the developer tools, open the Application tab and let me know if the service worker is listed or not (please find a screenshot below):

Hope we can fix this for you!
cc @plxel, have you experienced the same issue? It looks really odd, what do you think? 😕

@indiakamlesh works for me on Windows 10

so..
I got this behavior if I run yarn start, and without waiting for dev server start run yarn run electron
Reacto is started, but nothing is working

I'll generate new releases with the updated version.
@indiakamlesh, please uninstall Reacto and download the latest release (it should be available from one hour after I post this message).
Feel free to reopen the issue if that patch doesn't fix your problem!


Mistral dropout

adds a gaussian dropout layer after the MLP layer and cleans up the model config so it's more easily modifiable from the YML
dropout for mistral can be added via
model_config:
  dropout_p: 0.001

Any news on this? :) I'd love to try the new config out.


http3: correctly handle closed clients

Fixes https://github.com/quic-go/quic-go/issues/765. Closes https://github.com/quic-go/quic-go/pull/3157.
@zllovesuki, would you mind taking a look at this PR? It now implements the logic described in https://github.com/quic-go/quic-go/pull/3653#issuecomment-1402718763. Specifically,

We immediately remove a client when a request errored. This is an important cleanup, otherwise the client map will fill up with failed clients over time, with no way to garbage collect it.
When using an existing client, and we run into a timeout net.Error, we re-dial the connection before returning to the user.
This re-dial logic only applies to clients that have already completed the handshake. This means that when we get two requests from the application at the same time, and the handshake fails, we won't retry.

Implementation looks fine to me. ginkgo test formats really hurt my eyes

Implementation looks fine to me.

Thank you Rachel!

ginkgo test formats really hurt my eyes

I know. I wish we hadn’t decided to use Ginkgo when we started the project. But now I can’t really justify spending 2 weeks full time to rewrite all the tests. If you have any idea, please let me know :)

Technical debts 🤔
for future features and implementations, let's try using testing, then move things off ginkgo if needed at that time

Yep, that’s what I’m doing. webtransport-go already uses testing.


UX: policy disambiguation

Court Policy
Policy -> Dispute Policy

Hi everyone! I'm new to contributing to Kleros and I'm interested in working on this task. Has anyone already started working on it? If not, can I help with this issue?


chore: connect via pgbouncer

Implements # 888

Need to grant access to pgbouncer role before merging this in.


Add a /reset endpoint that calls exit(0)

To restart the website.

This did weird things on local and didn't behave as expected. We're not doing db migrations yet so this will not be that useful just yet. Closing.


Document the minimal resources needed to run tests for DataFusion

Is your feature request related to a problem or challenge?
As a contributor, I would like to test more locally to reduce CI errors, I tried running cargo test at the project root, and then all my RAM was eaten by tests, and the kernel got killed, forcing me to restart my PC.
I have 32 GB of memory, looks like I need at least 64 GB or 128 GB :D
Describe the solution you'd like
Document the spec (minimal one) needed for running tests of DataFusion in the contributors' guide.
And, we should document which tests are resource-heavy so that contributors can ignore them locally.
Describe alternatives you've considered
No
Additional context
No response

I think the doctests are the main culprit, see #5347
Agree on documenting this as part of contributor/developer guide 👍

Thank you for bringing this up @SteveLauC. I had the same experience when I first starting working on DataFusion. You can work around memory limitations by running cargo test -- --test-threads=1 which will only run a single test at a time. It will be slower but consume substantially less memory. I think it would be a good idea to document this workaround since many new contributors  won't have enough RAM to run all tests at max parallelism on their system.
The precise memory requirements to run cargo test will vary over time and depend on your exact development set up. Running in wsl on Windows for example is a bit more intensive as you need to reserve some memory for Windows. I personally upgraded from 32->64GB and that was plenty running natively on linux.

cargo test -- --test-threads=1

Thanks for showing me this! This indeed makes the memory usage controllable:)

Glad that worked for you! I opened #9402 to document this in the contributors guide.
There does not seem to be a way to configure Cargo to default to --test-threads 1 other than setting environment variables, so documentation may be our best bet for now... see https://github.com/rust-lang/cargo/issues/8430

I had my computer upgraded to 64GB of ram - the workaround mentioned above I used as well but the time to run the tests was using that was horrendous. Using linux under WSL2 in windows here.


How to clear the SelectedItem

Describe the bug
I have a use case where pressing a cell navigate to the detail page. However, the current situation is that when returning from the detail page to the list page, it is not possible to select the same cell again because the SelectedItem remains the same and does not trigger the OnSelectedItemChanged event.
When I set SelectedItem to null after navigated to the detail page, it causes a crash. Please give a help, thanks.
To Reproduce
View
<dg:DataGrid ItemsSource="{Binding Items}"
             SelectionEnabled="True"
             SelectedItem="{Binding SelectedItem}">
    <dg:DataGrid.Columns>
        <dg:DataGridColumn Title="Name" PropertyName="Name" />
    </dg:DataGrid.Columns>
</dg:DataGrid>

ViewModel
[ObservableProperty]
Item? selectedItem;

partial void OnSelectedItemChanged(Item? value)
{
    try {
        if (value is null)
            return;

        Shell.Current.GoToAsync(nameof(DetailPage));
    }
    finally {
        SelectedItem = null;
    }
}

Screenshots
StackTrace
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.Maui.WeakEventManager.HandleEvent(Object sender, Object args, String eventName)
   at Maui.DataGrid.DataGrid.OnSelectionChanged(Object sender, SelectionChangedEventArgs e)
   at Microsoft.Maui.Controls.SelectableItemsView.SelectionPropertyChanged(SelectableItemsView selectableItemsView, SelectionChangedEventArgs args)
   at Microsoft.Maui.Controls.SelectableItemsView.SelectedItemPropertyChanged(BindableObject bindable, Object oldValue, Object newValue)
   at Microsoft.Maui.Controls.BindableObject.SetValueCore(BindableProperty property, Object value, SetValueFlags attributes, SetValuePrivateFlags privateAttributes)
   at Microsoft.Maui.Controls.BindableObject.SetValue(BindableProperty property, Object value, Boolean fromStyle, Boolean checkAccess)
   at Microsoft.Maui.Controls.BindableObject.SetValue(BindableProperty property, Object value)
   at Microsoft.Maui.Controls.SelectableItemsView.set_SelectedItem(Object value)
   at Microsoft.Maui.Controls.Handlers.Items.SelectableItemsViewHandler`1.UpdateVirtualSingleSelection()
   at Microsoft.Maui.Controls.Handlers.Items.SelectableItemsViewHandler`1.UpdateVirtualSelection()
   at Microsoft.Maui.Controls.Handlers.Items.SelectableItemsViewHandler`1.PlatformSelectionChanged(Object sender, SelectionChangedEventArgs args)
   at WinRT._EventSource_global__Microsoft_UI_Xaml_Controls_SelectionChangedEventHandler.EventState.<GetEventInvoke>b__1_0(Object sender, SelectionChangedEventArgs e)
   at ABI.Microsoft.UI.Xaml.Controls.SelectionChangedEventHandler.Do_Abi_Invoke(IntPtr thisPtr, IntPtr sender, IntPtr e)

Looks like you are using CommunityToolkit.Mvvm
This is using a source generator to produce the public property from the backing field. You could try setting the backing field to be null instead of the public property.
Another possibility is that it has to do with the Shell navigation, and if you are really doing a master-detail approach then maybe what you need is something more like the Popup control from CommunityToolkit.Maui.
That control would be awaitable and return a value, if you like. That opens up a lot of options, and has a more predictable control flow.
Just some thoughts. If you can reproduce this problem with a minimal test case, and upload it as a new public repository on here, then I could try some more things.

Normal shell navigation isn't really supposed to involve executing more lines of code after the GoToAsync method. If you want to clear the selected item afterwards with normal shell navigation, the best place would be in the Appearing event of the page.


reactivity lost for props in nested solid-directories

Describe the bug
In the following project-directory-setup:
`

root

package.json
src

Lib.tsx

dev

package.json
src

Test.tsx
`

If I access component Lib from Test.tsx like import Lib from "../src/Test", the props of Lib will not react on state-updates from Test, but only when I vite build. In dev-mode this set-up works.
I wanted a directory set-up like this, so that I could easily build an example-site for a solid-library.
Your Example Website or App
https://github.com/bigmistqke/bug-report
Steps to Reproduce the Bug or Issue

git clone https://github.com/bigmistqke/bug-report.git
root-directory and dev-directory: npm install
dev-directory: npm run dev
open browser: observe how there are two numbers counting up
dev-directory: npm run build & npm run serve
open browser: observe how there is only 1 number counting up

Expected behavior
I expect there to be no difference between build and dev
Screenshots or Videos
https://user-images.githubusercontent.com/10504064/211232854-aa4767be-b184-4847-ae95-b9c903a21ad3.mov
Platform

macOS
firefox
108.0.2

Additional context
No response

This sounds like bundling issue. My guess is that in dev mode it walks up to the nearest node module, but on build we are getting duplication somewhere. Like a version of Solid imported for each project. Be sure you are externalizing solid-js and solid-js/web for any build processes that involve libraries.

I updated the code to include external: ["solid", "solid/web"] in the rollup-config, but the problem still remains.
On further inspection, the problem also seems to occur when npm linking...
say if i have a directory

lib

dist
src

project

src

and I would do

lib: npm link .
project: npm link lib
project: npm run dev :arrow_right: works
project: npm run serve :arrow_right: does not work

The external should be [ "solid-js", "solid-js/web" ]. You missed the -js.

you are correct, but problem is still remaining with [ "solid-js", "solid-js/web" ]

Well as suspected there are 2 copies of Solid in the build which is causing this. Let me see if I can track down why.

Ok so best I got is this. Solid is installed in both root and nested. Unfortunately modules are resolved relative to their location. When you import ../../src into your project it is resolving a different solid-js (the one in the root node_modules). Even if it is the same version. So each library ends up pulling in their own version. I confirmed this by deleting solid-js from the application's node modules and it worked. Since in this case the application would fail to find solid-js in the dev node_modules and fallback to the root. Why does this work in dev? Well Vite aggressively de-dupes in its optimization step..

so strange that it is bundled in the root-directory, since solid-js in the root-directory should be a peerDependency.
how do people nowadays do linked packages and stuff then in solid, since the problem is also occuring with npm link?

The npm link issue was caused by npm link itself.
From relative-deps readme:

These work only if there are no peer / shared dependencies involved. If there are shared dependencies, the linked library will resolve those in their own node_modules, instead of the node_modules of the hosting project, where it would normally be looked up. This results in peer dependencies ending up "twice" in the dependency tree, which often causes confusing behavior.

Tried out relative linking with this package instead and the double-library-issue vanished.

Ok awesome. Thanks for following up. I'm going to close this now. But this was useful information to know.

An alternative solution could be using resolve.dedupe. It seems cleaner to me. resolve.preserveSymlinks broke the @tiptap/* dependencies in my case.


extend metrics and rebuild prometheus exporting logic

I know that this PR is large but it grew a long time in our fork now and the changes depend on each other which makes it very hard to split those up into several commits.
I am happy to assist in any way it helps you to make a review, also a e.g. a peer-review via video chat is completely fine for me.
Just let me know how I can help getting this merged.
To make it a bit better I added an extensive commit description that I quoted below to explain the different measures.
To wet the appetite why this is useful here a short list of which metrics Traefik supports now for Prometheus and it is very easy for other providers to move along!
To get the other providers on the same feature level (not required to work right now, see commit description) I shoutout to @aantono and @adityacs who integrated those to extend the implementation of those providers.
Which metrics exist now with small samples (numbers are totally random):

traefik_config_reloads_total: useful to see how often config changes happen
traefik_config_reloads_failure_total: useful to see failing config reloads and alert on those
traefik_config_last_reload_success: useful to see when the last reload has happened
traefik_config_last_reload_failure: useful to see when the last config reload has happened
traefik_entrypoint_requests_total: same as before but on entrypoint level now
traefik_entrypoint_request_duration_seconds: same as before but on entrypoint level now
traefik_entrypoint_open_connections: useful to see how many open connections exist currently. It's split up even by protocol, so you could understand e.g. there 1k open websocket connections, 300 open SSE connections and currently 1.5k http requests in flight
traefik_backend_requests_total: same as before but on backend level
traefik_backend_request_duration_seconds: same as before but on backend level
traefik_backend_retries_total: same as before but on backend level
traefik_backend_open_connections: same as in entrypoint description but split up for each backend
traefik_backend_server_up: you can see how many backend servers exist and are healthy of a specific backend. useful to alert when e.g. 80% of your backend servers become unhealthy.

Additionally, this PR tackles https://github.com/containous/traefik/issues/2349 and https://github.com/containous/traefik/issues/1759.
Commit message with detailed explanations
This commit extends the metrics Registry interface and extends it's usage in order to get more insightful metrics. It also renames Prometheus metrics and the service label (to either backend or entrypoint) in order to fix #1759.
In order to extend the Registry interface the MultiRegistry is used so that single metric providers do not necessarily have to implement all of the metrics, but only a subset.
At the moment only the Prometheus implementation supports all new metrics, but extension of the other providers should be straightforward and can be done afterwards.
Additionally the Prometheus exporting logic was adapted.
The problem with it before was that the Prometheus client library is stateful and samples all tracked metrics on the client.
This is a problem given the things you are tracking are dynamic.
We have a metric traefik_backend_server_up which can be used to alert given too many servers of a backend become unhealthy.
Now when Traefik learns about a new backend it will create the concrete metric with the labels for this backend.
The problem is now that given the backend is undeployed, Traefik will still export the metric of the old backend because it holds the Prometheus metric state and it will only be gone after a restart of Traefik.
This leads to wrong results when you calculate e.g. how many of your backends are healthy / all backends and thus breaks alerting.
To fix this one has to implement a custom logic for the metric sampling.
I got a confirmation for this approach on the Prometheus users google group from Prometheus maintainers.
To fix the problem I implemented a configuration generation logic so that metrics that belong to a dynamic configuration (e.g. backends, entrypoints) get removed  again eventually given they belong to an old configuration and are not tracked recently.
https://groups.google.com/forum/#!topic/prometheus-users/EbumSJYsJyc
This PR also splits up the metric collection of requests, request  durations and retries into separate metrics for entrypoints and backends.
This will fix #1759. Before there was only one metric traefik_requests_total which was tracked at the entrypoint level with the label service="$entrypointname" and at the backend level with service="$backendname".
So at each request two traefik_requests_total metrics were trackend and so a calculation of e.g. sum(traefik_requests_total) would yield twice the requests that actually happened.
This is fixed by splitting up the metric into two.
For other metric providers this problem was the same and as I did not adapt those yet, they will only count the requests now against the backends.
An extension of them should happen afterwards.

Great addition... I'll add the DataDog and StatsD counters as soon as this PR gets merged!

Will update influx metrics as well once this PR is merged.


MINOR: Trigger release only if tag is added

@harpgl-bot retest this please

This change may work not like you expect. Documentation says:
"Setting tags to true causes the branch condition to be ignored, otherwise $TRAVIS_TAG is ignored, and the branch condition is considered". (https://docs.travis-ci.com/user/deployment-v2/conditional#tag)
As far as I understood, after this change job will also start if you'll put a tag to some feature branch, not only release one.

Yeah, that's indeed strange. Since nobody will/should add tags somewhere else I will remove the branch condition


pkg-config: Build error

ChromiumOS 52.
This one for /homebrew/dupes/nano
==> Tapping homebrew/dupes
Cloning into '/usr/local/.linuxbrew/Library/Taps/homebrew/homebrew-dupes'...
remote: Counting objects: 43, done.
remote: Compressing objects: 100% (43/43), done.
remote: Total 43 (delta 0), reused 2 (delta 0), pack-reused 0
Unpacking objects: 100% (43/43), done.
Checking connectivity... done
Tapped 38 formulae (105 files, 335.7K)
==> Installing nano from homebrew/dupes
==> Installing dependencies for homebrew/dupes/nano: pkg-config, libxml2, homebrew/dupes/ncurses, gettext, libmagic
==> Installing homebrew/dupes/nano dependency: pkg-config
==> Downloading https://pkgconfig.freedesktop.org/releases/pkg-config-0.29.1.tar.gz
Already downloaded: /home/chronos/user/.cache/Homebrew/pkg-config-0.29.1.tar.gz
==> ./configure --prefix=/usr/local/.linuxbrew/Cellar/pkg-config/0.29.1_2 --disable-host-tool --with-internal-glib --with-pc-path=/usr/local/.linuxbrew/lib/pkgconfig:/u
Last 15 lines from /home/chronos/user/.cache/Homebrew/Logs/pkg-config/01.configure:
2016-10-27 23:53:22 +0200

./configure
--disable-debug
--prefix=/usr/local/.linuxbrew/Cellar/pkg-config/0.29.1_2
--disable-host-tool
--with-internal-glib
--with-pc-path=/usr/local/.linuxbrew/lib/pkgconfig:/usr/local/.linuxbrew/share/pkgconfig:/usr/local/lib/pkgconfig:/usr/lib/pkgconfig:/usr/local/.linuxbrew/Library/Homebrew/os/mac/pkgconfig/0

READ THIS: https://github.com/Linuxbrew/brew/blob/master/docs/Troubleshooting.md#troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/Linuxbrew/homebrew-core/issues

Please report the output of brew install pkg-config && brew gist-logs pkg-config

https://gist.github.com/9fe9091ab1dc1bdb2666195a0446d6d1
https://gist.github.com/a1e23e689e81d1d7039a84d82bbfaa1d

I can't find an error message in either of those log files. ./configure just exits—I presume unsuccessfully—with no error message.
As a possible workaround (untested), try
mkdir /home/linuxbrew
ln -s /usr/local/.linuxbrew /home/linuxbrew/.linuxbrew
brew install --force-bottle pkg-config

Just using the --force-bottle switch solved it. Thanks


[TP] Infer local n_heads instead of ad-hoc model changes

Stack from ghstack (oldest at bottom):

-> #498

Today we modify model attributes n_heads and n_kv_heads after applying TP, which is also after the model's creation.
https://github.com/pytorch/torchtitan/blob/e457deb080e7c5eaa461ff03cfab7b1b8b55e583/torchtitan/parallelisms/parallelize_llama.py#L401-L404
The ad-hoc modification is for the following code to work in TP case:
https://github.com/pytorch/torchtitan/blob/e457deb080e7c5eaa461ff03cfab7b1b8b55e583/torchtitan/models/llama/model.py#L193-L195
because the n_heads and n_kv_heads per rank are different from the non-TP case.
This has a couple side-effects:

requires an application runner (not necessarily the model author) to be familiar with the model's details as well as the consequence of TP;
renders the TP APIs less standalone;
hard to compose with pipeline parallel, in particular if the user uses the tracing frontend -- the attribute modification may be non-effective or the attributes not available.

Fortunately, view ops has a nice feature that a size of -1 can be inferred from other dimensions. This PR thus utilizes that to make the code generally workable for both TP and non-TP cases.

Is it because changing the attribute makes tracing not working? Instead of using -1, since DTensor now supports pretty much all attention ops. Shall we go full DTensor mode inside attention so that no module attribute is needed? @tianyu-l @wanchaol

Tracing (PP) today happens before TP hence before the attribute change. And it would "burn" known attribute values into the traced model. That's why "after-trace" attribute change would not be effective.
I think "-1" would work for DTensor's view op too :)

There are indeed pros and cons of keeping the result in DTensor form vs local form. I'd like to learn from any discussion you may have.

(some of) 4 GPU integration tests are failing. @kwen2501 can you check if it's caused by this
https://github.com/pytorch/torchtitan/actions/runs/10220000239

Is it because changing the attribute makes tracing not working? Instead of using -1, since DTensor now supports pretty much all attention ops. Shall we go full DTensor mode inside attention so that no module attribute is needed? @tianyu-l @wanchaol

@fduwjj
Currently there's an issue #267 blocking full DTensor mode. After fixing the issue, we should be good even without this PR's change.

(some of) 4 GPU integration tests are failing. @kwen2501 can you check if it's caused by this https://github.com/pytorch/torchtitan/actions/runs/10220000239

Hmm, there is a tracer error in that CI run above.
Interestingly, when I merge this PR (yesterday), the CI is all green. And it seems unlikely that the "-1" change would make export uncomfortable. (Export likes determinism too.)
I wonder if there is some other change that may cause the failure -- like a change in pytorch nightly?
Btw, it seems the trunk of titan is still healthy.


Support hidden options in utils.field()

Allow to hide options when defining them using tmt.utils.field which can come handy for deprecated backward-compatible options that should not be listed on the --help page.

/packit test

Hm, so it seems there is no hidden option in click.Option on el8 as there's only old python3-click-6.7-8 available. So, I guess we drop this? Or, hide the option if hidden is available? @happz, what do you think?

Hm, so it seems there is no hidden option in click.Option on el8 as there's only old python3-click-6.7-8 available. So, I guess we drop this? Or, hide the option if hidden is available? @happz, what do you think?

I think there's already some old-click-only magic somewhere already...
Anyway, how about not calling it "hidden" but rather "deprecated"? Then we'd express the goal in a declarative manner, marking the option as deprecated, leaving the hands of follow-up tools free to act differently. E.g. we could pass hidden=True to click.option() when running with recent-enough Click and add e.g. 'deprecatedto the option'shelpbefore handing it toclick.option(), and tmt could generate a nice warning when being given such an option, and so on. The important bit the field is deprecated/obsolete, hidden=True` is just one of the many possible reactions.
Maybe it could be a non-boolean option, maybe a string to carry tmt version, as in "deprecated since x.y"?
A bit too verbose, but I included the answer to your question: with old Click, I'd add "deprecated" to the field help, with new Click I'd do that & add hidden=True as well. But it depends on the train of thoughts ^^

Using deprecated sounds even better! Let's handle it in this way. Regarding the type: Not sure how much it is interesting for the user to know since when the option is deprecated... On the other hand, it could be quite useful to know when the option will be removed (if it will be). Not sure. Both bool and str are fine with me. @lukaszachy, @adiosnb, thoughts?

I for one would prefer the string, deprecated='1.21', for example. It can always be ignored, it's bool-ish as any other string, and the info can be used by tmt itself, for example: when such a field is initialized from fmf or CLI option, tmt could emit a warning. I didn't think about such a possibility before, but I like it more and more, especially with some fields waiting to be changed (e.g. tmt * export options being moved to their respective plugins - main command could retaint them, for backward compatibility, and hint user automagically, without plugins taking care of that). It would also go with current schema validation warnings, and tmt lint could print out "you're using a key deprecated in 1.21, it's 1.956 now, maybe it's time to update" messages.
Is it enough? :)

"you're using a key deprecated in 1.21, it's 1.956 now, maybe it's time to update"

You got me! :-)
+1 for the string.


feat: unify scaffold command with prompt

As discussed in a separate channel, this unifies the scaffold command into a single prompt.  This prevents us from polluting the CLI commands with too many different commands, and gives us the option to choose better names later.
It seems that this command will be renamed to init but that's out of scope for this quick PR.

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

Thanks! We'll review this as part of https://snyksec.atlassian.net/browse/CTX-584 ❤


Remove excess logs unless debug mode is toggled on

All the build logs were helpful for debugging, but seems excessive to show users.

It's really just a lot on startup bc it prerenders all pages/assets. Maybe it could be toggled off just for startup, and then report a summary instead?


Added names for the apps from the letter "b"

Added names to 36 casks
Added appcast's sha256 for 2 casks


Where do I find standardized submodel templates?

When an overview page of submodel templates is available, revise the following topic:
Which names are already defined for important submodels?

See also question 12 https://github.com/admin-shell-io/questions-and-answers#id12

Are there machine-readable files from official sources that implement these submodel templates? Also considerations about formats would be interesting:

AAS, serialized as AASX, JSON, XML etc.
JSON-Schema, XSD to perform validation against.

@alw-iwu Currently, a publicly available repository of sub-model templates as AASX files is being prepared by IDTA. We are not aware of any automatic validation implementations.


Walking to pokemons but not catching them?

Expected Behavior
Catch the pokemons -.-'
Actual Behavior
So sometimes i see its just walking to some random places but not catching the pokemon and then rewalk from there to some place..
like this:
[#] Walking from (30.319318300000003, 78.0343003) to (30.3218183, 78.0343003) fo
r approx. 10.0 seconds
walking hook  8 [#] Finished walking to (30.3218183, 78.0343003)
(30.3218183, 78.0343003, 0)
[#] Scanning area for objects (46 / 100)
map_cells are 21
[#] Walking from (30.3218183, 78.0343003) to (30.3243183, 78.0343003) for approx
. 10.0 seconds
walking hook  8 [#] Finished walking to (30.3243183, 78.0343003)
(30.3243183, 78.0343003, 0)
[#] Scanning area for objects (47 / 100)
map_cells are 21
[#] Walking from (30.3243183, 78.0343003) to (30.326818300000003, 78.0343003) fo
r approx. 10.0 seconds
walking hook  8 [#] Finished walking to (30.326818300000003, 78.0343003)
(30.326818300000003, 78.0343003, 0)
[#] Scanning area for objects (48 / 100)
map_cells are 21
[#] Something rustles nearby!
[x] Found pokemon at distance<PHONE_NUMBER>48m
[#] Walking from (30.326818300000003, 78.0343003) to (30.326137919779427, 78.033
94158134181) for approx. 3.0 seconds
[#] Finished walking to (30.326137919779427, 78.03394158134181)
But some times it catches a pokemon
Why is this so?

Just by looking at this - you are using an outedated version. Please pull the latest version of dev and let me know how you go

Latest version of dev crashes with
Traceback (most recent call last):
  File "pokecli.py", line 107, in <module>
    main()
  File "pokecli.py", line 99, in main
    bot.start()
  File "/PokemonGo-Bot/pokemongo_bot/__init__.py", line 26, in start
    self._setup_api()
  File "/PokemonGo-Bot/pokemongo_bot/__init__.py", line 160, in _setup_api
    player = response_dict['responses']['GET_PLAYER']['player_data']
KeyError: 'player_data'

my observation is this is that
bot scans the area -> found pokemon > walk to location
then upon arrival on pokemon location starts to walk again to another location then the loop goes on. then sometimes actually meets the pokemon at the end of the walk and initiates a capture. but detecting pokemon and walking to location and then scan and move again.
. Im not sure if this is because of despawn but it happens fairly even with the latest dev. (including master)

@eggins i am getting same error as @ckrmml when runnning Dev

@harsh1412 please, destroy your entire environment and reinstall everything from scratch. I also recommend using virtualenv. @eggins can you link the installation tutorial, please?

Please follow the tutorial on the Wiki: https://github.com/PokemonGoF/PokemonGo-Bot/wiki/Installation

@eggins using windows here :p no wiki for windows

@douglascamata why do i need to do all that stuffs? The master is running fine except for the walking error other then that Dev should also run..
I may be missing some files.??
Could you check if Dev have complete project?

@harsh1412 we are still developing features in the dev part -and once there is a stable release we will release it to the public

In my experience, the player_data key is not present if there are login issues.
Just run the script again in a few minutes.
And if you're not planning on developing, use the master release. :)

by experience yeah player_data is normally associated with login issues as said by Calcyfer

These are both incorrect.
Delete the pgoapi folder, and re run the pip install -r requirements.txt

@eggins @Calcyfer @hero16 When i try to run Dev project just by downloading the zip it shows this:
Traceback (most recent call last):
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokecli.py", line 42, in
from pokemongo_bot import PokemonGoBot
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot_init_.py", l
ine 10, in
from pgoapi import PGoApi
ImportError: No module named pgoapi
Then i copied the pgoapi from master...
Now this error pops up:
Traceback (most recent call last):
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokecli.py", line 42, in
from pokemongo_bot import PokemonGoBot
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot_init_.py", l
ine 11, in
from cell_workers import PokemonCatchWorker, SeenFortWorker
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\cell_workers__
init__.py", line 3, in
from pokemon_catch_worker import PokemonCatchWorker
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\cell_workers\po
kemon_catch_worker.py", line 5, in
from utils import distance, print_green, print_yellow, print_red
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\cell_workers\ut
ils.py", line 5, in
from colorama import init
ImportError: No module named colorama

Check the previous comment from @eggins , you need to install the required modules using
pip install -r requirements.txt

I just redownloaded/cloned the project, followed the wikilink by @eggins https://github.com/PokemonGoF/PokemonGo-Bot/wiki/Installation and it worked

@eggins @ckrmml @Calcyfer got the dev bot working but the walking walking walking is still there..
Is this normal or its a bug?
[#] Something rustles nearby!
[#] A Wild Doduo appeared! [CP 372]
[x] Using Pokeball...
[x] Captured Doduo! [CP 372]
[#] A Wild Zubat appeared! [CP 78]
[x] Using Pokeball...
[x] Captured Zubat! [CP 78] - exchanging for candy
[#] Zubat has been exchanged for candy!
[#] Scanning area for objects (4 / 100)
[#] Walking from (30.326818300000003, 78.04430029999999) to (30.3243183, 78.0443
0029999999) for approx. 10.0 seconds
[#] Finished walking
[#] (30.3243183, 78.04430029999999, 0)
[#] Scanning area for objects (5 / 100)
[#] Walking from (30.3243183, 78.04430029999999) to (30.3218183, 78.044300299999
99) for approx. 10.0 seconds
[#] Finished walking
[#] (30.3218183, 78.04430029999999, 0)
[#] Scanning area for objects (6 / 100)
[#] Walking from (30.3218183, 78.04430029999999) to (30.3218183, 78.041800299999
99) for approx. 8.0 seconds
as we can see bot walked 3 times for something right??

As you can see, the bot scans for objects in a cell but doesnt find anything. You need to set a location that has a couple of pokestops.

@eggins Please add a note that we need GIT to run the Dev project..

@Calcyfer I am not using -s in the command so pokestops are irrelevant in this matter, right?

@harsh1412 There has just been a commit that removed the -s parameter and replaced with -m or --mode.
The default parameter is --mode all , meaning search for stops and pokemon.

@Calcyfer Oww.. but i am near so many pokestops and its not farming any of them..

What command line are you using? (dont forget to remove your credentials)

@Calcyfer pokecli.py -a google -u xxx -p xxx -l "xxx,xxx"  -w 30 --maxstep 10

Noob question: how do i use the web? :p

Your walking speed is pretty high, that's why objects won't load.
Try reducing -w to 4 or so.
For web, use python -m SimpleHTTPServer 8000 (or another port) in the web folder and open <IP_ADDRESS>:8000 in your browser

Okay thanks @calcyfer :D

@calcyfer I tried the map but it shows directory in the <IP_ADDRESS>:8000

Make sure you run the command in the \PokemonGo-Bot-dev\web folder, where the index.html file is.

@calcyfer just navigated to web folder from browser and it worked. lol

@jtdroste This issue can be closed.

Looks like another issue pops up with the latest dev:
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokecli.py", line 107, in
main()
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokecli.py", line 104, in mai
n
bot.take_step()
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot_init_.py", l
ine 31, in take_step
self.stepper.take_step()
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\stepper.py", li
ne 58, in take_step
self._work_at_position(position[0], position[1], position[2], True)
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\stepper.py", li
ne 100, in work_at_position
self.bot.work_on_cell(cell, position, pokemon_only)
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot_init.py", l
ine 52, in work_on_cell
hack_chain = worker.work()
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\cell_workers\se
en_fort_worker.py", line 35, in work
self.stepper._walk_to(self.config.walk, *position)
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\stepper.py", li
ne 79, in _walk_to
self._work_at_position(i2f(self.api._position_lat), i2f(self.api._position_l
ng), alt, False)
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\stepper.py", li
ne 100, in work_at_position
self.bot.work_on_cell(cell, position, pokemon_only)
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot_init.py", l
ine 38, in work_on_cell
worker.work()
File "C:\Users\DrDeath\Desktop\PokemonGo-Bot-dev\pokemongo_bot\cell_workers\po
kemon_catch_worker.py", line 86, in work
self.transfer_pokemon(list(Set(id_list2) - Set(id_list1)))
NameError: global name 'id_list1' is not defined

Best to create a new issue for this, as it's unrelated to this one.


Fix WebGL / texImage2D base definition

Added 'ArrayBufferView' variant of texImage2D function.
Issue: https://github.com/Microsoft/TypeScript/issues/3344

Hi @atebits, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks @atebits for the change. The library is auto-generated from IE's DOM webidl definitions and we do not make manual changes to the DOM definitions. This allows us to get an updated version of the library as the DOM evolves without having to manual changes.  as for the issue, I believe this is a bug in our auto-generation script, @zhengbli can fix that for now. We are planning on opening the generation script sources as well to allow contributions to lib issues.

Makes sense, thanks!


Compile issue - missing java files

Has anyone successfully imported this project into IntelliJ ?
When I tried to do that, it failed complaining about gradle issue.
Second, after I manually downloaded dependent jar libraries, and compiled this project, I found a lot of java files reference this java file, com.sixt.service.framework.protobuf.RpcEnvelope.java, but it does not exist.
Same issue happens to com.sixt.service.framework.protobuf.FrameworkTest.java.
Or these two java files will be generated automatically ? Did I miss anything ?
Thanks.

What was the issue you had with gradle when importing it?
RpcEnvelope is generated code.  I suggest to run gradlew build from the command line.  You can also open the gradle tool window, and run Tasks | other | generateProto and generateTestProto
Sorry for the confusion.  We will add some more docs on how to properly get started soon.

Here is the error message I got.
Error:(57, 0) Cause: repository not found: C:\xxx\IntelliJProjects\java-micro-master
Thanks.

It sounds like your intelli-j is setup wrong, or you are using it wrong.  I suggest going to File | Open | select the build.gradle in the directory of your repo, hit Ok, select 'Use gradle wrapper task configuration', and hit Ok.  That should work fine.

I was using import project last time. But this time, just did what you mentioned above, open.
I got the same error.
By the way, Line 57 is listed below (maybe this will give you some clue ?),
Error:(57, 0) Cause: repository not found: C:\xxx\IntelliJProjects\java-micro-master
git = org.ajoberstar.grgit.Grgit.open(file('..'))

It's not clear to me what is causing that.  I literally just checked out the repo and built on an out-of-date intelli-j (2016.2.5) on a different computer, and had no problems (i did have to install protoc - https://github.com/google/protobuf/releases/tag/v3.2.0).  By the end of next week, we should be publishing jars so that you won't even need to check out and build the repository anyway...

It worked fine for me. This is what I did:

Checkout repository
Open IntelliJ and do File -> Open on the folder java-micro
Wait
IntelliJ will ask you which projects to import as Gradle project: Select both
A Gradle tool window on the right appears
Run Gradle task install to publish a jar to the local maven repository

I did not experience any compile errors.
git = org.ajoberstar.grgit.Grgit.open(file('..')) indicates that it could not open the Git repository. Maybe something with the checkout went wrong?

Thank bkrahmer and tomasulo for the help.
Found out the reason, my gradle is too old, 2.12.
I just found this project is upgraded to use gradle 3.3, so I went ahead to use gradle 3.3, and java-micro is imported into IntelliJ successfully, and jar files are published to the local gradle repository.
But I found still no RpcEnvelope.java file got generated.
I do NOT have protoc installed on my machine, maybe that is the reason. I will try it later.
But anyway, compilation issue is solved. Thank you all for the help.


Add code

What

Add code

Why

Need module for named subnets

[ ] drop the concept of public/private
[ ] add a flag to enable nat gateways
[ ] make the subnets named (e.g. user inputs a list of names, and it creates a list of subnets with those names)

Interface should be
module "subnets" {
  namespace = "${var.namespace}"
  stage = "${var.stage}"
  names = ["charlie", "echo", "bravo"]
  vpc_id = "vpc-1234"
  base_cidr = "<IP_ADDRESS>./24"
  nat_enabled = "true"
}

[ ] add CI See this for example: https://github.com/cloudposse/terraform-null-label/pull/9/files

Closed in favor of https://github.com/cloudposse/terraform-aws-named-subnets/pull/3


jpegrescan: 2016-06-01 -> 2019-03-27

Description of changes

Updated.
Hard $PATH using libjpeg_original: When I originally packaged this script I wanted to leave it up to the user to decide between different jpegtran implementations so I didn't hard code the library $PATH. However, after all these years,

I've seen mozjpeg's jpegtran output corrupted jpegs enough times to rule it out completely.
That, while libjpeg_turbo is faster in many other use cases, with this particular script there's no meaningful difference (used time libjpeg in.jpeg out.jpeg on multiple occasions with the different jpegtran). e.g. even for a 130MB jpeg, the difference in time was negligible.
That libjpeg_original produces slightly smaller file sizes (0.1-0.5%) even without the controversial and generally unsupported arithmetic coding flag.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Builds & works for me, macos 10.15 & nixos x86_64.

thanks :)


Random eoe indicator

There seems to be some issues with the current master branch, like the eoe indicator detecting is reversed, and not all org-babel-elixir-filter-regexps are applied.
And the eoe indicator also have a fixed value, so I made a random number, so that it is almost impossible to conflict with the normal output.

Hey, thanks :)


New item in Menu shows first character lower case

If I add an item to the menu in _config.yaml, visually the item shows up all in lowercase. Archive, Tags, Categories, etc. all show up with the first letter Uppercase but anything else shows up lowercase. No idea why and I've been trying to find out where in the code it's causing this issue but I cannot find it.

Hi, @MayankTahil
Thanks for your interesting, Custom menu item showing up lowercase caused by this code, in file layout/_partial/header.ejs:
222

223

224        <% for (var i in theme.menu){ %>
225          <a id="beautifont" class="main-nav-link" href="<%- url_for(theme.menu[i]) %>"><%= __(i.toLowerCase()) %>
226        <% } %>
227

228

In the line 225, in order to support multiple languages, we have to use every menu item's lowercase to match various languages' relevant string in those .yml files in the languages/ directory.
So you could change these line of code according to your needs. :smile: And I will be grateful if you create a Pull Request after your optimization.


Java: dbscheme and library changes related to Java 14 extractor upgrade

Depends on corresponding internal PR for the Java 14 extractor upgrade.
Standard language features in Java 14 will be supported. Full CodeQL-level support for preview features will follow in a subsequent PR.
Tests are expected to fail on this PR, but should succeed on the internal PR.

Rebased to resolve submodule conflict on internal PR.


Issues with fresh installed ArduinoIDE, boards, libraries

I reinstalled my Windows and had to reinstall everything.
After installing Arduino IDE, boards I couldn't compile IR-Blaster.
I collected and installed all the libraries and compiled it successfully.
But it doesn't work:

When I try to send json link it says JSON parsing failed (but the same link works on my previous device)
IR-diod doesn't send any signals. No signal on PIN? I tried GPIO4, GPIO5, different ir-diodes, transistors - nope.

The new version of Arduino JSON is more strict so you’ll need to make sure your json is perfect and has quotes everywhere there’s a string or it won’t work
As for number 2 I have no idea what IR-diod is

I tried this one from IR-blaster page and it looks OK (tried to use double quotes and HTTP codes %27, %22):
http://<IP_ADDRESS>:80/json?plain=[{'data':[3396,1822,444,1248,446,298,462,284,468,286,468,1240,448,292,462,290,464,344,448,1246,446,294,460,292,462,290,464,1268,422,294,462,286,468,344,452,1242,448,292,462,288,466,286,466,1266,424,290,464,286,466,344,452,1240,450,290,464,288,466,286,466,1242,450,288,466,286,466,344,452,1240,452,284,466,288,464,288,466,1240,456,280,470,286,466,344,452,288,466,286,466,1240,454,282,468,286,466,1242,452,284,468,354,452,1240,454,1238,454,284,466,288,466,288,464,288,464,290,464,354,452,1240,454,1240,452,1238,454,1238,454,286,466,288,464,288,464,346,450,290,464,288,464,1244,450,286,466,288,464,288,464,288,464,346,450,290,464,288,464,288,464,288,464,288,464,288,464,290,464,346,452,290,464,288,464,288,464,288,466,288,464,288,464,290,464,346,452,288,464,288,464,288,464,288,464,288,464,288,464,290,464,346,450,290,464,288,464,288,464,288,464,290,464,290,462,290,464,344,452,1244,450,1240,452,1242,450,1240,452,284,466,288,464,290,464,1252,450],'type':'raw','khz':38}]

And does that give you the JSON error or just doesn't send a signal?


Buttons are customized

Related Issue

Issue No. #107

Changes you made :

Customized the  buttons

Describe the changes :

Check list :

[ x] I have followed code style of this project.
[x ] I have read the CONTRIBUTING document.
[ x] All the aspects mentioned in the issue are covered.
[x ] The website is properly working after my changes.
[ x] 🌟 ed the repo

Type of change :

[ x] New feature (non-breaking change which adds functionality)

Output Screenshots :

Did not modify the Book Now buttons as it is with cards and adding border radius to it will not look good

Good to go @itzwow
LGTM 🤩


Error when trying to run Tinker to create tenant

Bug description
When following the quick start documentation of stancl/tenancy from https://tenancyforlaravel.com/docs/v3/quickstart, When I reach the last step which is running tinker I get the following error :
Stancl\Tenancy\Exceptions\TenantDatabaseAlreadyExistsException  Tenant cannot be created. Reason: Database tenantfoo already exists.

Despite this error the tenant is created in the database, is it maybe related to version of Laravel ? I'm using Laravel v9.51.0
Steps to reproduce
-> php artisan tinker
-> tenant1 = App\Models\Tenant::create(['id' => 'foo']);
Expected behavior
don't know
Laravel version
9.51.0
stancl/tenancy version
3.6

Did you manage to fix the problem?

There's nothing to fix, the exception says what's wrong. There already is a tenantfoo database. To be able to create a tenant with the ID of foo you should delete that database.


[5.4] Attribute set to Collection is not converted to array

Laravel Version: 5.4.19
PHP Version: 5.6.4
Database Driver & Version: Mysql 5.6.17

Description:
Attributes that are collections are not converted to arrays when toArray() method is called on the model.
I discovered this when using collection cast, but it actually applies to all collection attributes.
Steps To Reproduce:
use Illuminate\Database\Eloquent\Model;

class Dummy extends Model {}

$dummy = new Dummy;

$dummy->foo = collect([1, 2, 3]);

dump($dummy->toArray());

Expected results:
array:1 [
  "foo" => array:3 []
]

Actual results:
array:1 [
  "foo" => Collection {#602}
]

Fix:
Adding code below to attributesToArray() method will give expected results.
foreach ($attributes as $key => $value) {
    $attributes[$key] = $value instanceof Arrayable ? $value->toArray() : $value;
}

Conclusion:
I think attributesToArray() method of the model should check if attribute implements Arrayable interface and then convert it to array with toArray() method.
Please correct me if I am wrong and there is actually a reason for current behavior.

Have you tried casting it to array?

toArray() is recursive for all attributes and relations, so, one way would be to add 'foo' as an attribute.
class Dummy extends Model
{
    protected $appends = ['foo'];

    public function setFooAttribute($value)
    {
        $this->foo = $value;
    }

    public function getFooAttribute()
    {
        return $this->foo;
    }
}

then toArray() will convert 'foo' to an array as well.
Otherwise, you could:
var_dump(json_decode(json_encode($dummy), true));

Thanks for replies!
@Dylan-DPC I want to cast this attribute to collection so casting it to array is kinda counter-productive.
@devcircus I think your code will recurse infinitely - it should be $this->attributes['foo'] instead of $this->foo in both places. Said that it probably solves the issue, however it is almost 10 lines of extra code, with use of functionality that wasn't made for the purpose.
Don't get me wrong, but I didn't add the issue to solve my particular problem, it couldn't be easier after you recognize the cause. I wanted to point out to a behaviour that is probably unexpected by 90% of people especially considering how collections in Laravel do it.

Feel free to open a PR. Closing since it's not actually a bug


Restarting insight creates a duplicate block

I'm using upstart on ubuntu. Every time I restart insight a duplicate block is created.
sudo restart insight

This is outdated, as things have significantly changed with storing information about the blockchain, and should no longer be an issue.


Upgrade to Postgres 17

This updates the Postgres stub to use the newest version 17: https://www.postgresql.org/docs/17/release-17.html
The official Docker image is here: https://github.com/docker-library/postgres/blob/172544062d1031004b241e917f5f3f9dfebc0df5/17/bookworm/Dockerfile

Hey @ziadoz, don't we also need to adjust the following in all runtimes?:
https://github.com/laravel/sail/blob/93a2136b71f5471e7683e98422cc6c4d65ee2eef/runtimes/8.3/Dockerfile#L8

@Jubeki Ah, I hadn't noticed these. Yes, they do need updating.


Location event time is imprecise

Your Environment

Plugin version: 3.0.0-alpha.24
Platform: Android
OS version: 8
Device manufacturer and model: Galaxy  #S8
Cordova version (cordova -v): 7.1.0
Cordova platform version (cordova platform ls): Android 6.3.0
Plugin configuration options:
Link to your project:

Context
I've read all the docs and Issues in this repository hoping to find an answer to my predicament.
In my context it's ok that it sometimes takes a while before sending and syncing the data to my backend, but the correct timestamp of the location is a must for me.
I've tried to send a new Date in the custom post template, but it doesn't refresh the Date, just gets the date of when the template was set.
This problem occured in all of devices i used to test, not just the galaxy s8, even tho they were all made by samsung but other models.
Could you please recommend a way to solve this issue or should i try to send the location manually when the event location is fired?
I'm trying to solve this issue for a couple of days now and already lost some hair hahahah
Thanks.

There is time in location event, it's time recorded by gps sensor (or any other provider). That is the only value you want to use.

I'm sorry if i didnt't specify why how or when the time is imprecise.
My problem seems to be when the plugin tries to sync all the stored locations when it reaches the syncThreshold number.
I tested several times letting the device without any connection and then when i turn it on again, the recorded time in my db is just couple of minutes before the actual time my server commited the transaction. Usually it would be fine, but i left the device recording the locations for at least 30 minutes.
Hope i made myself clear this time.
Thanks for the reply.

There is time in location event, it's time recorded by gps sensor (or net provider...). That is the only value you want to use.

Reading carefully what you said.. Does this could mean that if the device is in another country/timezone that is not the same as mine, the provided time could be different?
If this is true, is it possible to get the timezone along with the time?

After using RAW_LOCATION_PROVIDER I got better results.
Thanks.


Revoked access to the app, from Gmail

I just got this from Gmail:
"Hello,
Although you do not need to take any action, we'd like to let you know that the third-party apps below will no longer have access to some of your Google Account data, including your Gmail content. This change will take effect from March 31, 2019.
Got Your Back: Gmail Backup
We are making this change as part of our ongoing efforts to ensure that your data is protected and private. These applications have not yet complied with the updated data privacy requirements announced on October 8, 2018"

Can confirm - I also received the same email this morning.

Does anyone know what changes / have a summary of how to make GYB compliant?

See https://developers.googleblog.com/2018/10/elevating-user-trust-in-our-api.html specifically the Application Review section I think its just some paperwork on google's end

GYB 1.20 is now available. Basic instructions for installing and fixing the issue are in the release notes:
https://github.com/jay0lee/got-your-back/releases
can a few people try upgrading and fixing and give me feedback on the process?

here is what I get at the last step gyb --action quota --email
Traceback (most recent call last):
  File "gyb.py", line 2026, in <module>
  File "gyb.py", line 1365, in main
  File "gyb.py", line 321, in requestOAuthAccess
  File "oauth2client/_helpers.py", line 133, in positional_wrapper
  File "oauth2client/client.py", line 2135, in flow_from_clientsecrets
  File "oauth2client/clientsecrets.py", line 165, in loadfile
  File "oauth2client/clientsecrets.py", line 122, in _loadfile
  File "json/__init__.py", line 296, in load
  File "json/__init__.py", line 348, in loads
  File "json/decoder.py", line 337, in decode
  File "json/decoder.py", line 353, in raw_decode
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 11 column 1 (char 467)
[2471] Failed to execute script gyb

my bad, and extra "`" somehow ended up in the client_secrets.json file.
It works now.

Yes, that's fixed in 1.21.
On Fri, Mar 22, 2019, 7:32 PM Marc Neiger<EMAIL_ADDRESS>wrote:

my bad, and extra "`" somehow ended up in the client_secrets.json file.
It works now.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jay0lee/got-your-back/issues/195#issuecomment-475815375,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEaMsFi4cOXB9qGy1T33Ex39rjgAYud8ks5vZWgjgaJpZM4cDPkW
.

GYB 1.20

Appears fine here, and awesome job with the instructions!
I use GYB to backup a folder from my primary account, and restore it to a secondary GMail account. (I do not have G Suite.)
I appears I'll need to do more than simply running gyb-setup.bat for the second email address.
Should I just setup GYB in seperate folders for that, and set up two projects?
And, maybe when that's done, is there a way to merge the json files to use both in the same directory?
Thanks,
Appears you Do have our backs!


Wrong source links handling in issue view

I'm running iMDone-atom 1.1.10, Atom 1.05 on Windows 10.
In issue views, links to the source code in tasks point to wrong path. For example:

When my project is located in C:\Users\marek\foo and TODO item is in C:\Users\marek\foo\path\to\file.rs:143

iMDone generates following HTML task node:

        path\to\file.rs:143

And after click it tries to open C:\Users\marek\foo\Users\marek\foo\path\to\file.rs.

@jajakobyly I think we've got this handled.  Let me know if it's working ok for you and I'll close out the ticket

iMDone 1.1.12, behaviour didn't change. I'll try to prepare simple demo


[WIP] KillSwitch terminates pending instance

KillSwitch Terminates Pending Instances :no_good_woman: :x:
Overview
This updates the lucet_runtime_internals::instance::execution module so that KillSwitch::terminate will also terminate pending instances.
Before if an instance was remotely terminated before running, the result would be a KillError::NotTerminable error. Moreover, the instance would still run a guest context if run was called afterwards.
Now if an instance is remotely terminated before running, the result will now be Ok(KillSuccess::Cancelled) rather than a KillError::NotTerminable error. The instance will then enter a Domain::Cancelled state, and will not enter a guest region when run is called.
Example
This is best summarized in a small snippet, assuming an initialized instance inst:
let inst: Instance = // ...
let kill_switch = inst.kill_switch();
assert_eq!(kill_switch.terminate(), Ok(KillSuccess::Cancelled));

// if terminated before running, the guest should not start at all
match inst.run("onetwothree", &[]) {
    Err(Error::RuntimeTerminated(TerminationDetails::Remote)) => {}
    res => panic!("unexpected result: {:?}", res),
}

Implementation
In order for this to happen, it means that an instance is now considered terminable when it is not running. In the original implementation, terminable was really a flag that meant that the instance could be safely signalled. (Recall: Depending on the current execution domain, it may or may not be safe to signal the instance to terminate)
If the instance is now terminable when the guest context is not active (i.e. it has not yet started), we now need to indicate whether or not the instance can be safely signalled using the execution domain. This means touching some of the :ghost: fun :ghost: parts of lucet.
The TLDR of the changes here is:

Domain now has two additional variants: Pending, and Cancelled. A new instance will be in the Pending state. If a kill switch calls its terminate method before the instance runs, that instance will be left in the Cancelled domain.
Signal behavior now depends on KillState::execution_domain.

The terminability member is no longer semantically useful. (This part, is the :hot_pepper: spicy part.)
lucet_context_activate should call a Rust function to lock the domain Mutex, and update the execution domain before jumping to guest code. This means we now have a enter_guest_region that corresponds to the exit_guest_region that we use as a backstop callback.

This is ready for another round of review! After addressing your notes, this feels much more concise.  The documentation for the lucet_runtime_internals::instance::execution module is still stale, but I can take a pass updating that once the changes here look good :slightly_smiling_face:

This PR has been a real journey. sunglasses

Yep!! Thanks for reviewing along the way :heart:


Can you please also add ASP MVC views

Hello I am trying to use this skeleton for asp core view for a dashboard, can you also add ASP MVC views

Hi!
As a matter of fact, the dashboard used to use MVC views earlier but I migrated it to Razor Pages because I found that architecture a better fit for this kind of an application. So I don't plan to re-add this, however, if you still want this, you can create a branch at the last commit before the migration and cherry-pick the changes made after that.


Add support for scylladb/cassnadra

@uriyyo I had a small mistake on the tests, dropping un-existed keyspace

@uriyyo seems that I've forgotten to update the readme on the Scylla/Cassandra support...

@fruch Yup, we forgot to do it.
Would you like to submit PR to fix it?


New version: michidk.vscli version 0.1.8

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Executable-Error


Highlight potentially insecure contract functions

As pointed out in both the WHG and CD audits, we should make sure to document and highlight the contract functions that are insecure or incredibly powerful.
Examples:

Kernel.setApp()
EVMScriptRunner.runScript()
ACL permission functions

My senses is that this can be closed now due to your recent edits @sohkai.  Feel free to reopen if you disagree.

I think we may want to add even more warnings in the future, and perhaps even provide a "safety" guide, but that can be a separate concern in the future.


Where are floating point literals for C# explained?

I was looking for the documentation of "real literals" (e.g. 3.14e-2m) in the C# guide and couldn't find anything. I think this syntax is likely going to be confusing to newcomers and so it should be explained somewhere.
I tried various search terms on Google and also clicked through the TOC of the C# guide and didn't find anything. Did I just miss the article? (In which case, maybe its SEO could be improved.) Or does it actually not exist?

I tried various search terms on Google

Did you mean that you used Google to search the entire Internet or only the docs site? That "real literal" (BTW where did you get the term, is that what the docs say?) thing is known as scientific notation and it's pretty common/well known.

Did you mean that you used Google to search the entire Internet or only the docs site?

I searched the whole internet. Ideally, I think docs.MS should be in the first page of results. But I understand that's not going to happen every time.

where did you get the term, is that what the docs say?

It's what the spec calls them.

thing is known as scientific notation and it's pretty common/well known

Yeah, "c# scientific notation" is one of the search terms I used. Lots of results from Stack Overflow, none I could see from docs.MS.

It's what the spec calls them.

Right, I forgot that they use "real" rather than "floating point" for literals.

Yeah, "c# scientific notation" is one of the search terms I used. Lots of results from Stack Overflow, none I could see from docs.MS.

I see. "scientific notation" alone produces quite different search result. I'm not sure if MS docs is the right place to explain common notions that can be found in pretty much every programming language and even in old pocket calculators. But it could at least mention the term so that interested people know what to search for.

@svick there is one mention here (with the link to the spec):
https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/value-types-table#remarks
Though, it says "real numerical literal", not "real literal".