Failed to resolve dotnet packages on some machines because feed defined in local NuGet.Config is ignored

Failed to resolve dotnet packages on some machines because feed defined in local NuGet.Config is ignored
For example, the below output shows the local config file is used, however the local feed is missing:
Unable to resolve 'dotnet-svcutil (>= 0.5.10306-preview)' for '.NETCoreApp,Version=v1.1'.
NuGet Config files used:
C:\Users\pennerg\AppData\Roaming\NuGet\NuGet.Config
C:\Program Files (x86)\NuGet\Config\Microsoft.VisualStudio.Offline.config
C:\Users\pennerg\AppData\Local\Temp\WCFConnectedService\2017_Mar_21_06_41_30\svcutil_bootstrap\NuGet.Config
Feeds used:
https://api.nuget.org/v3/index.json
C:\Program Files (x86)\Microsoft SDKs\NuGetPackages
I can use dotnet add package -s  to restore the package successfully. The problem is I need to create a process for this operation in the runtime and it's pretty expensive. Is there an equivalent API I could use?
Any recommendation I can always resolve the package successfully? It's a pretty bad experience for our customers.
Thanks,
Hong

I'm not understanding the bug or the issue, can you clarify?

The app is in: C:\Users\pennerg\AppData\Local\Temp\WCFConnectedService\2017_Mar_21_06_41_30\svcutil_bootstrap
Under the root of the app, I have a local NuGet.Config that has the following package source information:

 <packageSources>
    <add key="SvcUtilLocalFeed" value="C:\myfeed" />
  </packageSources>

My project file depends on below package that is located under C:\myfeed
<PackageReference Include="dotnet-svcutil" Version="0.5.10327.0-preview"></PackageReference>
dotnet restore failed on some machines because it ignored the feed in the local NuGet.config, the log below shows c:\myfeed is not used.

Unable to resolve 'dotnet-svcutil (>= 0.5.10306-preview)' for '.NETCoreApp,Version=v1.1'.
NuGet Config files used:
C:\Users\pennerg\AppData\Roaming\NuGet\NuGet.Config
C:\Program Files (x86)\NuGet\Config\Microsoft.VisualStudio.Offline.config
C:\Users\pennerg\AppData\Local\Temp\WCFConnectedService\2017_Mar_21_06_41_30\svcutil_bootstrap\NuGet.Config
Feeds used:
https://api.nuget.org/v3/index.json
C:\Program Files (x86)\Microsoft SDKs\NuGetPackages

This issue was moved to NuGet/Home#4907

@davkean Do you recommend to use dotnet add package -s in this case instead of a local feed? Is there an equivalent API I could use? Thanks!

I've moved this over to NuGet - I don't have the context to answer your question.

But does NuGet owns dotnet commands?

They own some dotnet commands - your problem is with the feed not being used, that is NuGet-related. I don't have enough context to answer questions around this.

I don't need to use the feed if I use dotnet add package -s . Who is the owner for the dotnet add package command? I would like to know if there is an equivalent API.


[Feature Request] Partial course downloads

This would allow downloading specific chapters by specifying a start chapter, an end chapter, or both a start and end chapter

I tried to find that feature and got this request. I think it is really helpful


binwalk error

Hi, i just installed binwalk using these command below
sudo apt install binwalk
then followed with
git clone https://github.com/ReFirmLabs/binwalk.git
cd binwalk
sudo python setup.py install

then i tried to run it but this error appeared.
Traceback (most recent call last):
  File "/usr/bin/binwalk", line 2, in <module>
    from binwalk.__main__ import main
  File "/usr/local/lib/python3.10/dist-packages/binwalk-2.3.3+fa0c0bd-py3.10.egg/binwalk/__main__.py", line 24, in <module>
    import binwalk.modules
  File "/usr/local/lib/python3.10/dist-packages/binwalk-2.3.3+fa0c0bd-py3.10.egg/binwalk/modules/__init__.py", line 17, in <module>
    from binwalk.modules.entropy import Entropy
  File "/usr/local/lib/python3.10/dist-packages/binwalk-2.3.3+fa0c0bd-py3.10.egg/binwalk/modules/entropy.py", line 16, in <module>
    from numba import njit
  File "/usr/lib/python3/dist-packages/numba/__init__.py", line 38, in <module>
    from numba.core.decorators import (cfunc, generated_jit, jit, njit, stencil,
  File "/usr/lib/python3/dist-packages/numba/core/decorators.py", line 12, in <module>
    from numba.stencils.stencil import stencil
  File "/usr/lib/python3/dist-packages/numba/stencils/stencil.py", line 11, in <module>
    from numba.core import types, typing, utils, ir, config, ir_utils, registry
  File "/usr/lib/python3/dist-packages/numba/core/registry.py", line 4, in <module>
    from numba.core import utils, typing, dispatcher, cpu
  File "/usr/lib/python3/dist-packages/numba/core/dispatcher.py", line 13, in <module>
    from numba.core import (
  File "/usr/lib/python3/dist-packages/numba/core/compiler.py", line 6, in <module>
    from numba.core import (utils, errors, typing, interpreter, bytecode, postproc,
  File "/usr/lib/python3/dist-packages/numba/core/callconv.py", line 12, in <module>
    from numba.core.base import PYOBJECT, GENERIC_POINTER
  File "/usr/lib/python3/dist-packages/numba/core/base.py", line 1266, in <module>
    _initialize_llvm_lock_event()
  File "/usr/lib/python3/dist-packages/numba/core/utils.py", line 446, in inner
    res = fn()
  File "/usr/lib/python3/dist-packages/numba/core/base.py", line 1263, in _initialize_llvm_lock_event
    ll.ffi.register_lock_callback(enter_fn, exit_fn)
AttributeError: module 'llvmlite.binding.ffi' has no attribute 'register_lock_callback'

any idea how to solve it?

I'm not affiliated with binwalk aside from being a user, but I recommend the following:

Uninstall the binwalk OS package and never install it again
Create a fresh virtualenv (mkdir venv && virtualenv venv)
Enter the virtualenv (source benv/bin/activate
Install binwalk from github via pip (pip install "git+https://github.com/devTTYS0/binwalk@master")
All done


Add Congestion Handling to all Components

Physical Congestion through Transport and MaxConcurrent limitations
Kanban Congestion through cards limitation

https://github.com/Saldubatech/scala3-sample/pull/15
Congestion handling for:

Transport Capacity
Kanban Cards in Transport
Processor:

Buffer Capacity
WipSlots

Servers

Likely there will need to be more tweaks for complex interactions.


compiling gives segfault

After building zig, compiling crashes. It may be that some of the deps are runtime deps and not compile time. I didn't know which they were so I set no dependencies
Illegal instruction at address 0x2ac45ae
???:?:?: 0x2ac45ae in ??? (???)
???:?:?: 0x2ac472c in ??? (???)
???:?:?: 0x2a32b65 in ??? (???)
???:?:?: 0x2864109 in ??? (???)
???:?:?: 0x2850041 in ??? (???)
???:?:?: 0x285a851 in ??? (???)
???:?:?: 0x27df95f in ??? (???)
???:?:?: 0x27708f3 in ??? (???)
???:?:?: 0x2759937 in ??? (???)
???:?:?: 0x2758965 in ??? (???)
???:?:?: 0x2b525b7 in ??? (???)
???:?:?: 0x275b657 in ??? (???)
???:?:?: 0x275b402 in ??? (???)
???:?:?: 0x7fa7a245c0b2 in ??? (???)
Aborted (core dumped)

fixed as of: d4848e628185c717d976983cd677460e11a39cd7


styhead branch needed

After commit 81e4988 master branch is no longer compatible with oe-core/styhead, the linux-firmware update is only in master.

@sapiippo thanks for the note. I have spawned the styhead branch.


[ feature ] Documentation that matches the YAML spec would be helpful for beginners learning your platform.

User Story

As a new user to your platform
I want to read your documentation
So that I can easily translate your spec into the required YAML format.

I've been fiddling with your platform most of the day. It is pretty cool. One frustrating aspect of learning it is the "guessing" of where the parts in the documentation belong in the yaml spec.
Some kind of complete spec, showing how everything is configured manually would go a long way towards understand the platform. While the opinionated approach is useful to get started, it makes it challenging to advance beyond the basics /demos, and I feel this hurts adoption.

Thanks for writing this up! I hope you're enjoying working with Eidolon. I'm adding a few notes here for when we go through the prioritization review.
I think we have three types of configuration files: machine, agent, and reference. We have a docs section named "Builtin Components" that is generated directly from the code. It includes machine and a flattened list of agent properties.

We should consider at least one level of hierarchy below Agent, e.g.:
Machine

Symbolic Memory
Similarity Memory
File Memory

Agents

APUs
LLM Providers
Tools

@mafudge you might find this helpful as you progress: https://www.eidolonai.com/ide_help . We also have a number of documentation issues pending here: https://github.com/orgs/eidolon-ai/projects/6/views/8 . Thanks again for taking the time to submit this issue!


Add option to call external command on port change.

Add support for the PF_POST_COMMAND environment variable in
port_forwarding.sh. This will cause an external command to be called
with the new forwarding port and the original forwarding port (as
command-line arguments) every time the forwarding port changes.
Use cases:

Update firewall configuration.
Update configuration of anything that needs to bind to the correct
port.

Sorry for not going through this since November. I find the idea very interesting and I personally like the idea a lot, however as I head a full read I realized two things:

there are no comments explaining why this is necesary
adding the comments and accepting the merge means we increase the complexity of the script

At this point I am inclining towards not accepting this merge, as the point of these scripts is to be easy to read/understand/modify. Do you think it would be a good idea if this stayed only as a fork?

Hello!
Yeah, every feature adds complexity. The trick as usual is to find the right feature set for your target audience :)
In my case, I needed to make sure that when the port changes my firewall was updated to allow incoming packets on that port and my daemon configuration was updated to listen on that particular port.
I don't know how to do it without this feature, but like you said, it may also be something that not all people need.
About adding to main branch vs separate branch, I see two possibilities:

Adding it to the main branch may make discovery easier for people who need the feature. There's the valid argument of adding complexity, but my impression is that people  who don't want complexity opt for a more "standard" setup (chrome app, or GUI app).

Adding to a separate branch also works, but I don't know how we can prevent it from becoming stale, i.e. non-mergeable to the main branch if someone wants to use the feature.

WDYT?

the way i solved this was to write out to a file  /var/port_forward    and
use inotifywait   on that file to trigger the other changes necessary on
other daemons.
On Mon, Feb 21, 2022 at 9:05 PM Marco Paganini @.***>
wrote:

Hello!
Yeah, every feature adds complexity. The trick as usual is to find the
right feature set for your target audience :)
In my case, I needed to make sure that when the port changes my firewall
was updated to allow incoming packets on that port and my daemon
configuration was updated to listen on that particular port.
I don't know how to do it without this feature, but like you said, it may
also be something that not all people need.
About adding to main branch vs separate branch, I see two possibilities:

Adding it to the main branch may make discovery easier for people who
need the feature. There's the valid argument of adding complexity, but my
impression is that people who don't want complexity opt for a more
"standard" setup (chrome app, or GUI app).
2.
Adding to a separate branch also works, but I don't know how we can
prevent it from becoming stale, i.e. non-mergeable to the main branch if
someone wants to use the feature.
WDYT?
—
Reply to this email directly, view it on GitHub
https://github.com/pia-foss/manual-connections/pull/139#issuecomment-1047227525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA7ZA6WV4HIIUTVPC5PMZLU4KSJ7ANCNFSM5IOKBLOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

So, I could keep a fork of this somewhere. Is there a preferred method to
do this?
On Tue, Mar 1, 2022 at 3:51 PM Orangepeelbeef @.***>
wrote:

the way i solved this was to write out to a file /var/port_forward and
use inotifywait on that file to trigger the other changes necessary on
other daemons.
On Mon, Feb 21, 2022 at 9:05 PM Marco Paganini @.***>
wrote:

Hello!
Yeah, every feature adds complexity. The trick as usual is to find the
right feature set for your target audience :)
In my case, I needed to make sure that when the port changes my firewall
was updated to allow incoming packets on that port and my daemon
configuration was updated to listen on that particular port.
I don't know how to do it without this feature, but like you said, it may
also be something that not all people need.
About adding to main branch vs separate branch, I see two possibilities:

Adding it to the main branch may make discovery easier for people who
need the feature. There's the valid argument of adding complexity, but my
impression is that people who don't want complexity opt for a more
"standard" setup (chrome app, or GUI app).
2.
Adding to a separate branch also works, but I don't know how we can
prevent it from becoming stale, i.e. non-mergeable to the main branch if
someone wants to use the feature.
WDYT?
—
Reply to this email directly, view it on GitHub
<
https://github.com/pia-foss/manual-connections/pull/139#issuecomment-1047227525
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAA7ZA6WV4HIIUTVPC5PMZLU4KSJ7ANCNFSM5IOKBLOA
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pia-foss/manual-connections/pull/139#issuecomment-1055983957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYUYBHYLJW5ZOV55L3BTSTU52UO3ANCNFSM5IOKBLOA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Polybar does not launch with jsoncpp 1.9.2

Describe the issue
After updating to the latest version of jsoncpp on ArchLinux polybar will not start anymore.
The following output is given:
polybar example
polybar: error while loading shared libraries: libjsoncpp.so.21: cannot open shared object file: No such file or directory

Reverting jsoncpp to 1.9.1 is a workaround.
Expected behavior:
Polybar should start normally
Actual behavior:
Polybar does not start
Was it working before?

Did you also experience this bug in an earlier version of polybar?
Only tested on polybar 3.4.2 from the AUR.

To Reproduce
A minimal but complete config with which the problem occurs:
example config should be fine
List any other steps needed to reproduce the issue besides starting polybar with the config you posted above.
update jsoncpp to 1.9.2
Polybar Log
Post everything polybar outputs to the terminal when you run it and the issue occurs below
polybar: error while loading shared libraries: libjsoncpp.so.21: cannot open shared object file: No such file or directory

Environment:

WM: i3-gaps 4.18
Distro: ArchLinux
Output of polybar -vvv:
polybar: error while loading shared libraries: libjsoncpp.so.21: cannot open shared object file: No such file or directory

Same issue here...

I removed polybar and recompiled and it's back working again for me

Recompiling (in my case the AUR package) also solved this for me.

Duplicate of #885


[FEAT]: Implement Backend API for Squeeze Page - Backend

Description:
Develop the API endpoint for the Squeeze Page.
Purpose:
Provides a backend service that allows us collect the user email, store in the database and send a template to the user.
Requirements:

[ ] Develop a RESTful API endpoint to handle POST requests.
[ ] Validate and verify incoming user data.
[ ] Store user information securely in the database.
[ ] Return appropriate response.
[ ] Handle errors and return appropriate status codes.
[ ] Send template to a specified email address using an email service (e.g., SendGrid, SMTP).

Acceptance Criteria:

The page should be accessible at /api/squeeze.
The endpoint should accept HTTP POST requests.

Squeeze Form Submission:

Endpoint: POST /api/squeeze
Request Body:

{
  "firstName": "string",
  "lastName": "string",
  "email": "string:
}

Validation: Ensure all fields are required and the email is in the correct format.
Response (Success):

{
  "message": "Your request has been received. You will get a template shortly."
}

Response (Error):

{
  "message": "Failed to submit your request",
  "statusCode": 400
}

Tasks:

[ ] Create an endpoint (POST /api/squeeze) to handle contact form submissions.
[ ] Validate user input on the server side.
[ ] Securely store user information in the database.
[ ] Implement error handling and return appropriate status codes.
[ ] Write unit tests for contact form submission and data validation.
[ ] Write integration tests for confirmation of requests.

Testing:

Write unit tests for backend validation and data handling.
Write integration tests for the end-to-end contact form submission process.
Perform security testing to ensure data protection and compliance.
Test various scenarios for form submissions (e.g., successful, invalid data, server errors).

Expected Outcome:
A secure and efficient API endpoint that can receive user email, store it in the database, and confirm the user's request.
The API endpoint allows users to submit a request via the "Squeeze" form, and ensures the template is sent to the appropriate email address.
Labels
feature

These are multiple tickets, split them up as separate tickets

A squeeze page gather far more than the users email

I have updated the endpoint. The figma file only has provision for names and email @markessien

@buka4rill

Not enough

@markessien what else could a squeeze page gather?

@Fifetoyi
Here's an example:
{
  "email"<EMAIL_ADDRESS>  "first_name": "John",
  "last_name": "Doe",
  "phone": "123-456-7890",
  "location": "Lagos, Nigeria",
  "job_title": "Software Engineer",
  "company": "X-Corp",
  "interests": ["Web Development", "Cloud Computing"],
  "referral_source": "LinkedIn"
}

This should be enough

@markessien added more details for the squeeze page

Can we get a second approval on this please? @buka4rill

Add versioning to your endpoints and maintain the snake case

versioning and snake case implemented @Shullyd7

@coderoyalty  and @Fifetoyi   the comment mark made above "These are multiple tickets, split them up as separate tickets" .... did you guys initially add a get or put request .... if yes .. hope i am free to create another ticket  for get ?


Move to github flow

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@974c552). Click here to learn what that means.
The diff coverage is 84.28%.

@@            Coverage Diff            @@
##             master      #57   +/-   ##
=========================================
  Coverage          ?   79.94%
=========================================
  Files             ?        6
  Lines             ?      399
  Branches          ?        0
=========================================
  Hits              ?      319
  Misses            ?       80
  Partials          ?        0

Impacted Files
Coverage Δ

tests/__init__.py
100% <ø> (ø)

tests/test_data.py
100% <100%> (ø)

tests/test_store.py
100% <100%> (ø)

settings.py
100% <100%> (ø)

queue_publisher.py
38% <50%> (ø)

server.py
76.21% <78.94%> (ø)

Continue to review full report at Codecov.

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


Builtin prelude support for running python in a venv?

The python rules have been really great so far, surprisingly good in fact.  One thing I find missing is a way to load a requirements.txt and automatically create deps for each of the packages.  Is there anything like this?  What's my best workaround if not?

https://github.com/cormacrelf/elk


🐛 Fixed post feature image caption updates saving with every keystroke

closes TryGhost/Product#4093

Made new branch/PR


Add possibility to specify sub-paths for search options

Required files can only be identified using a Directory, EnvrionmentVariable or Registry path. The specified path must contain the required files. It is not possible to locate the required files relative to the specified directory.

Support added in cf8181c6


fix compile error

error message: "Mismatching uSockets/uWebSockets ABI".
cause: struct 'us_socket_context_options_t' has no 'ssl_ciphers' member variable.

I'm installing uWebSockets via vcpkg like so:
vcpkg install uwebsockets:x64-windows
Then to build with Visual Studio 2022, I setup the "Additional Includes Directories" to have the /include directory that vcpkg adds to the installs directory.  I get the ABI mismatch error, but if I put back in the line of code referenced in this commit, then I don't get the error and the build succeeds.
Is there something to do to ensure that the vcpkg installer is getting the latest build?

I don't think vcpkg uses the latest at all, they manually update it with certain commits and releases. No idea what versions work, or don't work. Try and get everything from vcpkg and nothing from GitHub and you should probably be fine.

That seems to do the trick, thanks.  Who manages the repos that vcpkg retrieves the code from?  Is there a way to find out the version that vcpkg is using during install?

Microsoft does that, no idea what versions they use. Should be printable in some way.


[GSoC] Text Generation Notebook Using GPT-Neo

Added Notebook and Readme for Text Generation using GPT-Neo.
Run Tests
Created MyBinder link https://mybinder.org/v2/gh/Warlord-K/openvino_notebooks/gpt-neo

Hi @adrianboguszewski ,All the checks seem to pass in except for one,which is failing for notebooks that I didnt modify,could you please take a look?

If you didn't touch it, don't worry.

@Warlord-K notebook content is very similar to gp2, could you please your notebook to the same folder where gpt2 notebook located and give parent directory more generic name (remove gpt2) like in https://github.com/openvinotoolkit/openvino_notebooks/tree/main/notebooks/107-speech-recognition-quantization (we also have 2 notebooks for wav2vec and data2vec models)

@eaidova Thanks for the feedback, I have updated my PR. I based my notebook on the GPT-2 notebook since the task was very similar ,and the contributing guidelines mentioned a set of rules and coding practices that needed to be followed. I was working on adding optimization code to my notebook but I had a doubt, It has been mentioned that Optimization Notebooks are to be named in 100s while inference notebooks follow the 200s naming scheme. So should I create a new pull request with a new notebook in 100s or should I modify this notebook?

@eaidova Thanks for the feedback, I have updated my PR. I based my notebook on the GPT-2 notebook since the task was very similar ,and the contributing guidelines mentioned a set of rules and coding practices that needed to be followed. I was working on adding optimization code to my notebook but I had a doubt, It has been mentioned that Optimization Notebooks are to be named in 100s while inference notebooks follow the 200s naming scheme. So should I create a new pull request with a new notebook in 100s or should I modify this notebook?

it is ok to have optimization part inside 200s notebook series, 100s used for basic API explanation and some typical usecases, so it is fine provide how optimize specific model in model related notebook.
Also will be great if you can share your opinion what can be improved in your tutorial to make it more attractive and interesting for users?

Thanks! I will add the Optimization code to my notebook. I had one idea to make the notebook more attractive to users, I can change the inference to be conversational, like ChatGPT,though it wont be even close to that but it would be more interesting and inviting for users. I could also provide an option to use a larger models for better conversational capabilities.

@eaidova @adrianboguszewski I have added a new notebook that uses personaGPT for conversation. I initially added to GPT-Neo but that didn't give good results,hence i decided to add a different model in a separate notebook. It took me a while since I thought that i was doing something wrong but found out that the model hasn't seen much conversational data and hence can't have conversations. Please provide if there are any improvements that i can make.

[x] each function is described by docstrings and type hints
[x] notebook contains explicit descriptions and explanatory diagrams
[x] the notebook doesn't use any data (image, video, etc.) that is not CC4.0 licensed
[x] there is a README.md file in consistent style (look at other notebooks)
[x] the notebook is added to the main README and notebooks/README files
[x]  there are no grammar, interpunctions or typo issues (use any free tool for that e.g. Grammarly)
[x] there are no committed files besides notebook and readme (please use images or videos from data dir)
[ ] all CI checks passed

@adrianboguszewski 17/19 CI checks are passing and the checks that are failing are in those parts which were not touched by me. So the PR is ready for review from my side.

@AnishaUdayakumar, it's ready :)

@adrianboguszewski I have merged all three notebooks into one, and updated the readme accordingly.

Thanks for the Suggestions, I have incorporated all the changes. Sorry for being a little late. @AnishaUdayakumar Please check if this is alright.

I have incorporated the suggestions, Please review.

@AnishaUdayakumar Thanks for patiently reviewing my PR and providing suggestions. It was one of my first PRs and I got to learn a lot, thanks for providing so much valuable feedback, I have also added Interactive chat feature, so i request you to provide feedback on that as well.

@AnishaUdayakumar Thanks for patiently reviewing my PR and providing suggestions. It was one of my first PRs and I got to learn a lot, thanks for providing so much valuable feedback, I have also added Interactive chat feature, so i request you to provide feedback on that as well.

Glad to know and thanks for considering the suggestions provided. The interactive chat feature for PersonaGPT is not working for me. As soon as I provide the input text and click on the "Chat" button, the model doesn't provide any response. Request to please review it again.

@AnishaUdayakumar Thanks for patiently reviewing my PR and providing suggestions. It was one of my first PRs and I got to learn a lot, thanks for providing so much valuable feedback, I have also added Interactive chat feature, so i request you to provide feedback on that as well.

Glad to know and thanks for considering the suggestions provided. The interactive chat feature for PersonaGPT is not working for me. As soon as I provide the input text and click on the "Chat" button, the model doesn't provide any response. Request to please review it again.

It is working for me here,

Can you please provide a screenshot of the issue?

I think the code is not displaying the conversation within the notebook output, it is only printing the response in the console.
Not sure if it's only me facing this issue.
If you want to display the conversation within the notebook output, you can use the display function to display it.

I think the code is not displaying the conversation within the notebook output, it is only printing the response in the console. Not sure if it's only me facing this issue. If you want to display the conversation within the notebook output, you can use the display function to display it.

I have tried to fix this issue, Please check again.

I think the code is not displaying the conversation within the notebook output, it is only printing the response in the console. Not sure if it's only me facing this issue. If you want to display the conversation within the notebook output, you can use the display function to display it.

I have tried to fix this issue, Please check again.

still the same result, probably it is connected with the button displaying in jupyter lab. Maybe it is better to simplify interface and  demonstrate history like just printed text instead of buttons?
also I think it maybe faster, because in this case you should no need to rewrite whole dialog on each new response

I rewrote the chat function yesterday, could you please check if the newest version works?

I think the code is not displaying the conversation within the notebook output, it is only printing the response in the console. Not sure if it's only me facing this issue. If you want to display the conversation within the notebook output, you can use the display function to display it.

I have tried to fix this issue, Please check again.

still the same result, probably it is connected with the button displaying in jupyter lab. Maybe it is better to simplify interface and demonstrate history like just printed text instead of buttons? also I think it maybe faster, because in this case you should no need to rewrite whole dialog on each new response

If i don't clear the output then the chat button will be on top, I thought that placing it at the bottom would give it chat app like interface. If placing it on top looks better then i can do that as well

I rewrote the chat function yesterday, could you please check if the newest version works?

it does not work, I tested half hour ago with latest changes

I rewrote the chat function yesterday, could you please check if the newest version works?

it does not work, I tested half hour ago with latest changes

Oh okay, I will try to find a different fix.

@Warlord-K I think I have some good idea. maybe we provide some simplified chat interface in notebook and additionally some demo app based on gradio interface?
It can be launched separately after model conversion run inside notebook and in future we also can deploy it in hugging face spaces.
I mean something like that:
https://gradio.app/creating-a-chatbot/

Yeah Sure, I can do that.

@eaidova Could you please take a few minutes review my proposal, I just have one major question that if i should increase or decrease the detail in my proposal and if i should add code into my proposal. Since today is the last day for submission, If you could provide some feedback then it would be amazing.

@eaidova Could you please take a few minutes review my proposal, I just have one major question that if i should increase or decrease the detail in my proposal and if i should add code into my proposal. Since today is the last day for submission, If you could provide some feedback then it would be amazing.

@Warlord-K could you please remember, where I can find link on it?

@Warlord-K I think I have some good idea. maybe we provide some simplified chat interface in notebook and additionally some demo app based on gradio interface? It can be launched separately after model conversion run inside notebook and in future we also can deploy it in hugging face spaces. I mean something like that: https://gradio.app/creating-a-chatbot/

Hi, I am sorry for being silent for a few days. I have been trying to make a gradio interface for the model, but for some reason I get an error while loading the model in another file.

This is my error, Please advise.

@Warlord-K I think I have some good idea. maybe we provide some simplified chat interface in notebook and additionally some demo app based on gradio interface? It can be launched separately after model conversion run inside notebook and in future we also can deploy it in hugging face spaces. I mean something like that: https://gradio.app/creating-a-chatbot/

Hi, I am sorry for being silent for a few days. I have been trying to make a gradio interface for the model, but for some reason I get an error while loading the model in another file.

This is my error, Please advise.

@eaidova @adrianboguszewski @AnishaUdayakumar Could You please advise on fixing this.

@eaidova I have added an option for plain and interactive inference, where plain is console logging and interactive is a gradio demo, Please check if this is suitable.

@eaidova Please check out the new inference style and let me know if I should make any changes.

Hi @eaidova, I have added the interactive style but the treon docker test is failing, I didn't change much but it has started failing, Could you please advice on how i may fix that?

Hi @eaidova, I have added the interactive style but the treon docker test is failing, I didn't change much but it has started failing, Could you please advice on how i may fix that?

do you mean docker or which tests? as I can see everything green except docker (but it is not related to your changes)

Yes, Docker is failing, everything else is green.

Is there any more changes that I need to do?


Add display status

I just started using this with my HA and it is greatly enhancing my macbook status aside from HA official app. If we can get the status of the display this would be perfect.

I have stopped working on this repository in favor of just using the home assistant SSH integration. But if you have any experience coding and go and to create a pull request, add an app feature. I'd be happy to merge it. And if not, since you say you're already using the home assistant companion app that does report the display status. The entity attributes for whether your computer is active or not. Contains information about the display state.


How to use with PetaLinux in a multiconfig setting...?

While this is interesting and we'd like to help grow this, it's kind of hard to use when it's not working right- and it looks like it's all a PetaLinux thing...
No documentation here is a bit of a pain and it's not obvious if it's a bug in the sense of misusing the layer or if PetaLinux is actually breaking things.

And I did make sure it worked with something other than PetaLinux before going there.  ;)
I've my own layers, etc. and all.

Hi @madscientist42 ! , I'm afraid I'm not following what the issue is, this layer has nothing to do with Petalinux (neither the Petalinux Distro nor the Petalinux tool provided by Xilinx). I believe you are referring to the tool, the Petalinux tool might require some additional functionality to allow it to work with this layer, sadly that is something Xilinx would have to do and not something we can modify here.
This is meant to work with a Yocto Project/OE setup, the documentation/instructions are on the README file, following the provided instructions allow you to build either just freertos or both freertos and Linux at the same time using multiconfig in bitbake for qemuarmv5 and qemux86-64 respectively.

Well, then, a bit of education is in order...
Yocto is used by Xilinx IN PetaLinux and PetaLinux is a wrapper around
their gate logic generation code, etc. to help generate Device Tree source
code, etc.  The Yocto parts are just not laid out the way you think they
are and expect them to be.
Here's a little sample of the tree output for one of those project
directories...
zynqmp-iwg35m/
├── build
│   ├── cache
│   ├── conf
│   ├── downloads
│   │   ├── git2
│   │   └── uninative
│   ├── misc
│   │   ├── config
│   │   └── rootfs_config
│   ├── sstate-cache
│   └── tmp
│       ├── buildstats
│       ├── cache
│       ├── deploy
│       ├── hosttools
│       ├── log
│       ├── pkgdata
│       ├── sstate-control
│       ├── stamps
│       ├── sysroots-components
│       ├── sysroots-uninative
│       ├── work
│       └── work-shared
├── components
│   ├── plnx_workspace
│   │   └── device-tree
│   └── yocto
│       ├── cache
│       ├── conf
│       ├── downloads
│       ├── layers
│       ├── sysroots
│       └── workspace
├── images
│   └── linux
│       └── pxelinux.cfg
└── project-spec
├── configs
│   ├── busybox
│   ├── init-ifupdown
│   └── systemd-conf
├── hw-description
└── meta-user
├── conf
├── recipes-apps
├── recipes-bsp
├── recipes-core
├── recipes-graphics
├── recipes-kernel
├── recipes-modules
└── recipes-support
DO note the "Yocto" directory there and a lot of metadata layers.
So...let's start over, shall we?
On Wed, Nov 2, 2022 at 9:11 PM Alejandro Enedino Hernandez Samaniego <
@.***> wrote:

Hi @madscientist42 https://github.com/madscientist42 ! , I'm afraid I'm
not following what the issue is, this layer has nothing to do with
Petalinux (neither the Petalinux Distro nor the Petalinux tool provided by
Xilinx). I believe you are referring to the tool, the Petalinux tool might
require some additional functionality to allow it to work with this layer,
sadly that is something Xilinx would have to do and not something we can
modify here.
This is meant to work with a Yocto Project/OE setup, the
documentation/instructions are on the README
https://github.com/aehs29/meta-freertos/blob/master/README.md file,
following the provided instructions allow you to build either just freertos
or both freertos and Linux at the same time using multiconfig in bitbake
for qemuarmv5 and qemux86-64 respectively.
—
Reply to this email directly, view it on GitHub
https://github.com/aehs29/meta-freertos/issues/12#issuecomment-1301566925,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKF6CJNOBXB2ADIHUDDAE3WGMNO5ANCNFSM6AAAAAARVVHXIM
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm aware of what the Petalinux tool does and what it is, what I'm trying to say is that its probably incompatible in its current state without some tweaks, the tool would need integration work from the vendor's side to allow it to work automatically.
That being said, I dont see why you couldn't just add the layer to your bblayers.conf manually (the location really shouldn't matter as long as you point to it) along with the multiconfig dependency statement (like the one in the instructions) that would make it build, however, again do note that the current port works on qemuarmv5 and you would have to create your own port for your architecture whether that would be a microblaze or something else, at least you would need to create the freertos-. bbclass just like the one that exists in this layer along with specifying both the BSP_REPO and app repos to allow it to build your own app for a certain architecture.

I'm closing this, since this is work that has to be done by the Petalinux team at Xilinx/AMD.


[feat] Implement OP_DATA_X opcodes,  OP_DATA_7 - OP_DATA_75

Implement OP_DATA_X opcodes,  OP_DATA_7 - OP_DATA_75 reference implementation from btcd
References
Guide on how to add an opcode in shinigami

@b-j-roberts Kindly assign. Thank you.

Thank you, lmk if you have any issues.


Merge null won trigger onError after subscribed

`
class Test{
public ReactiveProperty id = new ReactiveProperty();
public ReactiveProperty age;
}
class Client
{
void Dotest()
{
var test = new Test();
Observable.Merge(test.id, test.age).Subscribe(_ =>{}, error => Debug.Log(error.ToString()));
}
}
`
The Merge wont trigger the OnError when excute Client Dotest function, why? Am i doing it wrong?

I think because you didn't initialize the age like
public ReactiveProperty age;= new ReactiveProperty();


Update backport:prev-major

backport:prev-major currently behaves the same as backport:all-open. Instead we want this to skip backports two major versions back.

buildkite test this

:broken_heart: Build Failed

Buildkite Build
Commit: de0f81eff58831fb28d2d4ffcbca3d6a83b400a4

Failed CI Steps

:pipeline::arrow_up: Upload Pipeline: .buildkite/pipelines/pull_request.yml
:pipeline::arrow_up: Upload Pipeline: .buildkite/pipelines/pull_request.yml
:pipeline::arrow_up: Upload Pipeline: .buildkite/pipelines/pull_request.yml

:broken_heart: Build Failed

Buildkite Build
Commit: 7582a1dde7626028f3bab27c2ea31505bda620a1

Failed CI Steps

Run Tests

History

:broken_heart: Build #12 failed de0f81eff58831fb28d2d4ffcbca3d6a83b400a4

:green_heart: Build Succeeded

Buildkite Build
Commit: e44e1528e62db949c545216c4500bd3577ee003d

History

:broken_heart: Build #13 failed 7582a1dde7626028f3bab27c2ea31505bda620a1
:broken_heart: Build #12 failed de0f81eff58831fb28d2d4ffcbca3d6a83b400a4


Generate PatternMatch

public enum Status
{
    Created,
    InWork,
    Completed,
    Canceled
}

need generate switch operator wrapper for handle all variants, and use it in mappers
such a function will break mappers when i add new enum item
public static class StatusMapper
{
    public static T Match<T>(this Status x, Func<T> onCreated, Func<T> inWork, ....) => x switch
    {
         Created => onCreated(),
         ....
        _ => thow new UnreachException(),
    }
}

I feel your proposal is geared toward Discriminated Union, and out of scope for this library. The author intention is define faster implementation for Enum than the one built-in, hence he even experimented Interceptor to complete overwrite Enum builtin function but failed due to preview feature. Your requested feature can be achieved by DU library like OneOf or source gen one like Dunet

Thanks for the suggestion, but as NotAsea indicated, I think that's out of scope for this library (which aims to be a more-performant version of the built-in functions), so going to close this as out of scope


Correction for raw affiliation Open University Heerlen by Aeres University of Applied Sciences Wageningen

Correction needed for raw affiliation Open University Heerlen by Aeres University of Applied Sciences Wageningen
raw_affiliation_name: Open University Heerlen by Aeres University of Applied Sciences Wageningen
new_rors: 03jnx2c74;018dfmf50
previous_rors: 0349bsm71;015d5s513
works_examples: W3090017053
contact: e795a66806ff17efca45de791b92d32c:f99bae00881c4cec @ aeres.nl

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Vite Compat!

Is there an existing issue for this?

[X] I have searched the existing issues

Do you want this issue prioritized?

[ ] Yes, I have sponsored
[X] Not urgent

Current Behavior
Importing moti via Vite results in some errors, usually in the form of it expecting cjs but getting esm:
/takeout/test/node_modules/@tamagui/animations-moti/node_modules/moti/author/index.js:1 export * from '../build/author'

Expected Behavior
Should work with Vite / esm
Steps To Reproduce
I think you can try the Tamagui starter:
npm create tamagui@latest --template=simple-web

And then add moti and run dev:vite
Versions
- Moti: latest!
- Reanimated:
- React Native:

Screenshots
No response
Reproduction
I promise I'll get a better one soon :)

Does vite have any sort of transpiling the way that next.js has transpilePackages?

Moti uses expo-modules build so presumably any expo module will run into the same issue.


LastPass is not able to auto-login

not necessarily a bug, but LastPass can't autofill the login page for this app
reported by mxterzeroto

Yeah, this is a Flutter limitation. I've filed an upstream bug.

https://github.com/flutter/flutter/issues/27918


Ensure dashboard displays even when there's no remote_url

closes #9
Rather than falling back to a blank dashboard when remote_url cannot be extracted due to:

no remote -> no origin
remotes are named something else but origin

This plugin shouldn't depend on a remote to work when all git history is available locally.
Here are some test cases:
Given the following config:
---@diagnostic disable: missing-fields
return {
    {
        dir = "~/Documents/nvim-plugins/git-dashboard-nvim",
        dev = true,
        dependencies = { "nvim-lua/plenary.nvim" },
    },
    {
        "nvimdev/dashboard-nvim",
        event = "VimEnter",
        opts = function()
            local header = require("git-dashboard-nvim").setup {
                title = "owner_with_repo_name",
            }

            return {
                theme = "doom",
                config = {
                    header = header,
                    center = {
                        { action = "", desc = "", icon = "", key = "n" },
                    },
                    footer = function()
                        return {}
                    end,
                },
            }
        end,
    },
}

fw13 :: Documents/nvim-plugins/git-dashboard-nvim ‹feat-remote› » git remote -v
origin	git@github.com:csessh/git-dashboard-nvim.git (fetch)
origin	git@github.com:csessh/git-dashboard-nvim.git (push)
tdo	git@github.com:csessh/git-dashboard-nvim.git (fetch)
tdo	git@github.com:csessh/git-dashboard-nvim.git (push)

Result: dashboard works, <owner>/<repo name> extracted ok

fw13 :: ~/.dotfiles ‹main*› » git remote -v
origin	git@github.com:csessh/.dotfiles.git (fetch)
origin	git@github.com:csessh/.dotfiles.git (push)

Result: dashboard works , <owner>/<repo name> extracted ok

fw13 :: ~/Documents/Obsidian ‹main› » git remote -v
fw13 :: ~/Documents/Obsidian ‹main› »

Result: without remote url, current directory is used to display instead

fw13 :: Documents/nvim-plugins/stopinsert.nvim ‹main› » git remote -v
origin	git@github.com:csessh/stopinsert.nvim.git (fetch)
origin	git@github.com:csessh/stopinsert.nvim.git (push)

Result: dashboard works, <owner>/<repo name> extracted ok

I also think displaying an empty board like so when there's no commit  is better / less jarring than showing a completely blank screen

These are all great changes, thanks for the contribution @csessh !


Add nullzip import

ref: https://github.com/Neovici/cosmoz-image-viewer/pull/40
Hmm we might should consider separating nullzip from nullxlsx one day ...
@nomego pls review

@jokedst pls merge


Move approx thumb setting up and default false

Unfortunately, it appears that having thumb approximation enabled on full-tracking enabled games leads to the thumb looking jumpy, as the hand tracking in apps (like SteamVR home) seems prioritize finger positions from capacitive touch first before looking at skeletal input. This appears to happen to the index finger due to the trigger gesture as well.
Perhaps there's a way to have it automatically detect what games don't have analog thumb support and enable from there (maybe by querying a predetermined list), but that doesn't sound like a perfect solution either.
Until there's a definite solution to this, the easiest solution is to default thumb approximation to false and have people turn it on when they plan to play a game that requires index emulation. This is a setting that would really benefit from real-time updating.
Currently, my testing platform is very laggy because I'm using Virtual Desktop over a non-local network, so it's possible that what I'm noticing is caused by lag and not actually from the approximation. I'd like to get a confirmation that this is in fact a real issue before merging this in.

After testing over a more robust setup, this does not appear to be an actual issue (and was actually just an artifact from lag originally). Closing this now but if anyone does notice this PLEASE let me know.


Target release_ios_bundle_flutter_assets failed

Here is the error that this PR resolves (shown during "flutter build ios") using Flutter (Channel beta, 1.19.0-4.1.pre, on Mac OS X 10.15.5 19F101, locale en-CA):
This application cannot tree shake icons fonts. It has non-constant instances of IconData at the following
locations:
-
file:///Users/chris/projects/flutter/.pub-cache/hosted/pub.dartlang.org/device_simulator-0.9.6/lib/src/apple_icon.dart:4:24
Target release_ios_bundle_flutter_assets failed: Exception: Avoid non-constant invocations of IconData or try to build again with --no-tree-shake-icons.
build failed.

@vlidholt I'm having the same issue. It would be nice if you could merge this as soon as possible.
[✓] Flutter (Channel stable, 1.20.1, on Mac OS X 10.15.5 19F101, locale en-LV)
    • Flutter version 1.20.1 at /Users/martinsandersons/Development/Flutter
    • Framework revision 2ae34518b8 (6 days ago), 2020-08-05 19:53:19 -0700
    • Engine revision c8e3b94853
    • Dart version 2.9.0

@vlidholt is this package abandoned?


add google analytics and link checking workflows

described in these docs: https://jupyterbook.org/advanced/html.html#advanced-html-outputs

Turns out Google Analytics isn't working. Seems like we could either generate an 'old-style' tracking ID or update JupyterBook to > 0.11.2 https://github.com/executablebooks/jupyter-book/issues/1300


Configure Checks : In Active ranges when Choose Books option is selected only the last books results are shown

Describe the bug
On the zzz6 project, Config Checks select Repeated words when the option Choose Books is selected with Psalms Genesis. The Results are displayed only for the book of Genesis. Instead of both books.
To Reproduce
Steps to reproduce the behavior:

Set BCV to Psalms 1:1
Open zzz6 project using the Open scripture editor from the main menu
Open Configure check and select Repeated words
Open Show Check Results ..
Notice that there are 5 words in the results list for the Psalms 1:150 active ranges
Opt Choose Books in active ranges and include Genesis to the list
Notice that the Results are updated with only Genesis. Instead of displaying both Psalms Genesis

Expected behavior
When more than 1 is selected in the Active ranges Choose Books.Results if available in all books should be displayed.
Results from Psalms

Results from Psalms and Genesis

Could be linked to #1171

depends on

https://github.com/paranext/ux/issues/96 and https://github.com/paranext/ux/issues/120

This was fixed as a part of https://github.com/paranext/paranext-core/pull/1319

This particular bug was fixed, but there is a related bug that still exists. #1171 was reopened to cover this.


CC83926: Fixing parameter type of OnGetMinMaxInfo()

Hello, @Saisang, @ghogen, @Mikejo5000
This proposed file change comes from https://github.com/MicrosoftDocs/cpp-docs.de-de/pull/13 .
Could you review this contribution and help to merge if agreed?
Many thanks in advance.

@v-makoud : Thanks for your contribution! The author, @, has been notified to review your proposed change.

Hi @corob-msft, @Mikejo5000, @Saisang, or @ghogen:
Please confirm that this PR is ready for merge.
Thank you.


UF-305 - Upgrade GwtBootstrap to version 0.9.3

Depends on https://github.com/droolsjbpm/guvnor/pull/344/files

+1 to merge (with others) but think @wmedvede should comment on removal of the workaround.


Proposal to combine efforts into a new applet.

Hi!
I'm the owner of the Arch Update Checker.
I was wondering if you would be open to combining our efforts into something like a Plasmoid Updater, and similar things while making documentation for others to follow!
Maybe we could even create a superior Arch Update Applet!

a superior Arch Update Applet

I'm not sure. The current functionality that already exists probably suffices for the Plasmoid format. Anything more would lean towards a full-fledged application.
I've done almost everything I wanted, just a few things left.

something like a Plasmoid Updater

Do you mean upgrading other plasmoids?


Project needs pull request and issue templates

I propose we use something similar to https://github.com/IBM/nodejs-itoolkit/pull/92

Closed by 62669c9b421d4378c87b3ff1de686ecb247fafa2


Need help in vrp constraint that is not all points need to be explored

Hello,
Could anyone pls help me on these constraints:

Not all points need to be explored.
Then the objective function is maximizing the number of points explored
I had known how to limit the distance of a route, but still stuck with the above constraints.
Thanks so much! I am working with Python.

Just use a Disjunction. i.e. Solver try to visit the node (and pay the traveling cost etc...) or pay the penalty cost and don't visit the node, knowing that the solver will try to minimize the objective (sum of all cost).
https://github.com/google/or-tools/blob/6a177338f944e09edd007cfddf814bb930454095/ortools/constraint_solver/routing.h#L514-L533

Should work, if it limit to 7 nodes it just means solver didn't find a route with more than 7 nodes which have a distance (i.e. CumulVar at route end node) lower than maximum_distance.

@Mizux Thank you, I think I have found the solution, seem like there is problem with float type less than 1, I have converted all data into int, and it works

or-tools which is a C++ library use int64 ...


How can i call "CipherText.setScale"

I need to change scale of CipherText
And CipherText.scale of node-seal is read-only property.
But MS-SEAL has setScale Function.
I wonder if you could help me by support CipherText.setScale in this great library?
Thank you!

@Jongsic Thank you for your suggestion. This feature will be added in the next release over the next day or two.

@Jongsic, a new release has been published.

Thank you for your fast and perfect supports!


Performance of where method

  @where: (selector = {}, options = {}) ->
    result = @modelize @find(selector, options)
    result.setQuery selector
    result

I noticed that where() is invalidated on every change in cursor returned by find. Is it possible to do this by using transform option on find? That way we would only do fetch and init when we need to.
Something like this:
  @where: (selector = {}, options = {}) ->
    options = _.extend({transform (doc) -> @init(doc)}, options)
    @find(selector, options)

@czeslaaw you bring up a good point; however in practice (for me at least) the where() method is intended to be a "fetched" representation of the docs in question. Meaning, when I call where() I'm expecting to get an array of documents back that can be manipulated/treated as such (using underscore methods for sorting, etc.). That was somewhat the purpose of the difference between the where() and find() methods; find() returns a cursor rather than an Array.
But you pointed out a flaw in find() which is that it was just giving you a straightforward Collection.find(), which meant that you would just be getting plain documents back, and not models. e.g. MyModel.find({property: 'xyz'}) wasn't going to return instances of MyModel unless you explicitly called MyModel.init() on the results.
Check out dec87ab060ee1736f87770a819e9445dc182f800 (and update your package to v0.9.4), and you'll see what I did was move the "modelizing" logic to instead be a "transform" inside of find(). So now if you use the find() method you will automatically get transformed models upon fetching the results. And the where() method differs by automatically calling fetch() which gives you back an Array (technically, a Relation) of results.
In summary -- use Model.find() when you're just wanting a cursor. A good example of this is when setting up your publish methods in your server you can return Model.find(selector). Another example where I'd only need the cursor is in Iron Router, when passing data to the template. But when I'm purposefully wanting to fetch the results in an array, I use Model.where(selector).
Thinking about it further, I suppose the where() method might be more properly named (or aliased) as fetch().

That's what i needed. Thanks!


Storyhighlights time bar issue while moving (forward | backward) between the stories

The width of the time bar is fixed and not going to start from the base position that it should be while moving to the previous story.

@emadbaqeri Story highlight is changed in the previous refactor, please have a look at that on the develop branch

@emadbaqeri Story highlight is changed in the previous refactor, please have a look at that on the develop branch

There is another issue with that which Ehsan mentioned yesterday. I have opened a new issue about #279.


[SPARK-6375][SQL] Fix formatting of error messages.

Test build #29056 has started for   PR 5155 at commit b898188.

This patch merges cleanly.

Test build #29056 has finished for   PR 5155 at commit b898188.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29056/
Test PASSed.

LGTM (we can add the TODO later or we can just do that when we fix https://issues.apache.org/jira/browse/SPARK-2775)


Release for 2.13.0-M3

This probably needs to wait for https://github.com/scalamacros/paradise/issues/112.

Macro paradise is now available for 2.13.0-M3.

Done.

Many thanks Miles!


Deleting 4 bar gas infrastructure

When you delete the gas infrastructure at 4 bar, which is a realistic situation, the load still displays a gas load at 4 bar. I tried this in the following LES: https://moses.energytransitionmodel.com/testing_grounds/506/gas_load

If there is absolutely no infrastructure installed at a level in the gas network, it is assumed that capacity at that level is infinite, and efficiency is 100% in both directions (no loss). We only start limiting capacity and efficiency once something has been installed.
It would not be difficult to have capacity become zero in these situations; in which case no gas would be able to flow across a pressure level when there is no infrastructure there. @ChaelKruip What do you think?

@ChaelKruip What do you think?

I think the current approach is logical. If nothing is installed it cannot lead to a constraint either (the absence of a pressure level implies that the surrounding levels are connected directly).
Closing this. If you disagree @Sanne512 please re-open this issue.

(the absence of a pressure level implies that the surrounding levels are connected directly).

I agree with this part. And consequently for me it would make sense if you look at the yearly gas flow (see below) that the two columns of 0.125 bar - 4 bar and 4 bar- 8 bar would be merged to one column of 0.125 bar - 8 bar. I don't know if this is possible?

I don't know if this is possible?

Yes this is certainly possible but I'm not sure if it worth the effort. I'd have to talk to @antw and @grdw about this but my early estimate is that merging the bars based on the gas-assets list would take at least 5 man-days of work.
Changing the label from 'bug' to 'feature'.

@ChaelKruip have you checked with @antw and @grdw how much effort this takes?
Including @MarjoleinSchwachofer for further reference from Alliander

@ChaelKruip have you checked with @antw and @grdw how much effort this takes?

About two man-days work. The chart has hard-coded data series and names. To make this dynamic is not incredibly difficult but needs quite a bit of code to be rewritten.


Remove deprecated code

It's always pleasure to do that.

Coverage increased (+0.38%) to 91.98% when pulling 2a42bd3fda19e58b606a164d2e3a74dd210b6937 on remove-deprecated-code into a4cad604e7efcf650b9d497b940a8636d46d3ac4 on master.

Coverage increased (+0.38%) to 91.98% when pulling 2a42bd3fda19e58b606a164d2e3a74dd210b6937 on remove-deprecated-code into a4cad604e7efcf650b9d497b940a8636d46d3ac4 on master.


Not able to send all coins from mobile wallet

Main net
Steps:

Create account with balance 100.9824 XYM
Go to send form and try send all of it (minus fee)

Result: App allow me send only 99.78 but fee for this transfer is 0.0176. I'm not able send all of it from wallet.

I tested this issue in testnet ,Android mobile and is fixed , this PR can be merged

Hi, I tried testing this issue, added 100 to the wallet , the recommended fee is 0.0176 or no fees, i selected 99 to send, but it says not enough funds ... so i think still not able to send all the coins . so retest failed , please have a look

Tested. I was able to send all coins.
Tested, merged and released.


SkeletonTheme can't be changed at runtime

I have two themes in my site that can be changed in any moment. I'd like to change SkeletonTheme with it.
I've tried using ternary operators in the colors, and I even made a wrapper component:
import React from 'react';
import {SkeletonTheme} from "react-loading-skeleton";

const ThemeAwareSkeletonTheme = ({theme, children}) => {
    if (theme === "white") {
        return (<SkeletonTheme color={"#eee"} highlightColor={"#f5f5f5"}>{children}</SkeletonTheme>)
    } else {
        return (<SkeletonTheme color={"#245b89"} highlightColor={"#152e4d"}>{children}</SkeletonTheme>)
    }
};

export default ThemeAwareSkeletonTheme;

But it doesn't work, if the first loaded theme is white, it can't be changed to the other one, it just keeps the same colors.
I can't find in the code what is causing it, can you help me to find it?
Thanks!
Regards

Fixed in v1.3.0


In central configuration, "use window detection" seems broken.

It's pretty simple:
in central configuration, when selecting "use window detection" and hitting submit - it ignored it, and instead it selects "use motion detection"

Will try to reproduce.

For me it's a bit different - I can't get rid of the checked "use motion detection". Whenever I get back to central configuration, motion detection is again activated. Checking and unchecking "window detection" seems to work.

Duplicate of #514  which will bee fixed in 6.3.0


[WIP] Add v2.1.0-beta.20181015 release notes

See https://github.com/cockroachdb/cockroach/issues/31085.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/441819cc4a58a9cbbba6765d71e1330af12fcfa2/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f353f8667cf4b3e28dc328d3103883a98e78836e/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fc9aeee6bd0ca4573ea8d754bda9ffc94568cd45/


Update Visualforce pages to use slds and Lightning stylesheets

Critical Changes
Changes
We switched to Lightning Stylesheets on our Visualforce pages in order to take advantage of accessibility improvements as they are released for Lightning pages.
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

Tracking W-7780322

@force2b @RadSFDeveloper This is ready for your review.  Thanks!

@npsp-reedestockton Looks like there is a build failure due to a markup problem on the BDI_BatchOverride page
https://sfdo-metaci.herokuapp.com/builds/298664

@RadSFDeveloper On that last screen capture with the date picker differences, the two fields in the two screen caps are different. I would be surprised if that makes a difference, but can you confirm that the pickers are still different if you use the "Date of First Payment" field in both branches?

@RadSFDeveloper On that last screen capture with the date picker differences, the two fields in the two screen caps are different. I would be surprised if that makes a difference, but can you confirm that the pickers are still different if you use the "Date of First Payment" field in both branches?

This is the branch org date picker on the same field and it still differs, which is fine but wanted to double check.

@sam-knox Would you please review the Changes section of the PR?

@rjanjanam There is one failing robot test here, but it appears to be BGE related rather than something about the work on this PR.

@rjanjanam There is one failing robot test here, but it appears to be BGE related rather than something about the work on this PR.

@npsp-reedestockton I checked the failure. It is failing for some other reason for the batch process taking too long. Robot tests look good .


Rebase async branch on new master

The branch is rebased. We will test the code later.
https://github.com/metricq/metricq-cpp/compare/experimental-async-cpp-db


nixos/bluetooth: disable restart on unit changes

Motivation for this change
Fix issue #93461
Things done

[ ] Tested via one or more NixOS test
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @wmertens

LGTM! I think we can just merge.


ADD support for updating TILLER_HISTORY_MAX when upgrading

When doing a helm init --max-history=N --upgrade --debug, the returned deployment manifest shows TILLER_HISTORY_MAX env var set on the deployment with the correct value.
But when doing a kubectl log <tiller_pod>, it shows Max history per release is 0. Doing a describe on the deployment also shows that the TILLER_HISTORY_MAX is missing.

a new comment

The code here looks very frail and I'm afraid of introducing new bugs. Would you mind adding some unit tests around updateEnv?

+1

👍

Hi guys,
I've taken the liberty to try working on the tests but I'm not sure how to send a PR to a PR and I'm not sure how to move this forward. Any comments/help will be appreciated.
https://github.com/davidchua/helm/commit/af316d34ff0fed9f59301766e0f44fc8d4b3babb
cc @mattclegg

@bacongobbler can we merge this as there is some unit tests now ?

LGTM!

those unit tests have not been merged into this PR. If someone wants to incorporate those changes @davidchua suggested and re-open a new PR, we can take a look into it.

@bacongobbler I'll incorporate those changes and re-open a new PR as recommended. Thank you for all of your help!

I've opened a new PR #5867 as suggested here. I checked in #helm-dev that this made sense. Thx! @bacongobbler


Tooltip on disabled elements and btn-group

Hi, I saw so many issues with that, but the only way to do it is to wrap the disabled element with a
 (or a , it can work).
BUT!
I would like to put it on a btn wrapped by a btn-group and wrap my disabled element by a span breaks the CSS...
Within a

Within a

Without a wrapper, but tooltip doesn't show up

http://jsfiddle.net/5jexwyj2/
Working on the PR next

Hi @kkirsche!
You appear to have posted a live example (http://fiddle.jshell.net/5jexwyj2/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

W005: Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work
line 53, column 5: W008: Tooltips and popovers within button groups should have their container set to 'body'. Found tooltips/popovers that might lack this setting.
line 56, column 5: W008: Tooltips and popovers within button groups should have their container set to 'body'. Found tooltips/popovers that might lack this setting.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

We won't be providing any overrides in Bootstrap for this kind of thing, at least for v3.x. It's a bit too much to account for—
s, s, etc would all need it. For now, you'll need to provide the CSS overrides for this.


Periodic CI job in OpenStack foundation infrastructure

Can we test external cloud provider with kubespray in OpenStack infra?
cc @pabelanger

I think we can, right now the kubespray job is just running on openstack-dev/sandbox and to wrap it up and move it into openstack-zuul-jobs for projects to use.
What we'd really need is some instructions on how the cloud provider would be installed with kubespray, I don't really know how that would work.

https://github.com/kubernetes-incubator/kubespray/blob/c2a46e4aa30c51bea96cdf3fc780245903aaa350/docs/openstack.md is all is needed to install kubespray with openstack provider AFAIK

@bogdando @pabelanger i've added a request in kubespray as well - https://github.com/kubernetes-incubator/kubespray/issues/2642


@CaseInsensitiveXPath doesn't work for some locators

@CaseInsensitiveXPath doesn't work for some locators such as :
//android.widget.LinearLayout[./android.widget.TextView[contains(@text, 'someText')]]//*[contains(@resource-id, 'id/someId')]
Expected result: the text is converted to case insensitive mode
Actual: some parts of the locator converted with text. It looks like this :
*** Element info: {Using=xpath, value=//android.widget.LinearLayout[./android.widget.TextView[contains(translate(@text, 'SOME_TEXT')]]//[CONTAINS(@RESOURCE-ID, 'ID/SOME_ID', 'some_text')]]//[contains(@resource-id, 'id/some_id'),translate('Some_text')]]//[contains(@resource-id, 'id/some_id', 'SOME_TEXT')]]//[CONTAINS(@RESOURCE-ID, 'ID/SOME_ID', 'some_text')]]//*[contains(@resource-id, 'id/id/some_id'))]}

Device: Samsung, Lg, HTC, OnePlusOne
OS: 6, 7, 8, 9 Android
Appium version [e.g. 1.15.1]

https://github.com/qaprosoft/carina/pull/854


Longhorn migration

Longhorn finally has SPDK in preview, and will soon have ARM and backup/restore. Should have similar perf to mayastor with improved support and features

Migrated to v1 but SPDK likely won't have Talos support until Q4

Need to move ephemeral off eMMC to avoid flash degredation, but upcoming SPDK uses block disks and might not support partitioning...


[Proposal] Validation rule for hexadecimal values

This will be useful for checking hexadecimal values, for instance the color hex like #000000 (black) and #ffffff (white)

@prady00  Are you thinking about submitting a pull request? If not I can give it a go.

Couldn't this easily be accomplished using the regex validation rule?

Yeah that's a good point @lukasgeiter

@bmitch : I've already coded it for one of my current project, just need to move it to Validator.php and do a simple PR.

@lukasgeiter using regex all types of validations can be performed. IP, alpha, numbers, URLs etc.
We include easy subsystem for something fast and non-time consuming in terms of development.

@prady00 Maybe not all types, but in most cases that's true. I just wanted to point out the possibility of using the regex validation rule, in case someone didn't know about it. I'm not particularly against it.
Just send the PR and we'll see if Taylor is up for it.


net: websocket: Use system provided base64 function

Instead of using mbedtls_base64, use system provided base64 encoding
function.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #6767 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #6767   +/-   ##
======================================
  Coverage    54.4%   54.4%
======================================
  Files         446     446
  Lines       42349   42349
  Branches     8024    8024
======================================
  Hits        23039   23039
  Misses      16079   16079
  Partials     3231    3231

Continue to review full report at Codecov.

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


Fix buddybuild

Fix the .podspec on our fork of react-native-image-crop-picker so that it is in sync with the version specified in our package.json. This will allow us to fix buddybuild.
Also note that the referenced tag does not exist yet -- after I merge this, I will set the tag to the merge commit.

Merging this.


Add M13ik's file

i have created a new file, created a new file and made a commit and opened a pull request :)

:)


Rearranged providers.

Great analysis and improvements. I agree with all of them.
One potential drawback will be that we can not translate the compatibility screen, but we would use it very rarely and it is a reasonable compromise.


DBZ-6225 DBZ-6226 DBZ-6231 Fix test failure

https://issues.redhat.com/browse/DBZ-6225
https://issues.redhat.com/browse/DBZ-6226
https://issues.redhat.com/browse/DBZ-6231

@Naros Thanks for the fix!


Can't see my Stripe products on Supabase & on my site

I followed the ReadMe perfectly to set up the project. However, even though everything is configured on Vercel, Supabase & Stripe (with registered products), I can't see my subscriptions and I'm left with the message: "No subscription pricing plans found. Create them in your Stripe Dashboard. "
I noticed that the products are not updating from Stripe and the Webhook is not working. I have the following error message on Stripe:
Webhook Error: No signatures found matching the expected signature for payload. Are you passing the raw request body you received from Stripe? https://github.com/stripe/stripe-node#webhook-signing
Thanks for any help :)

I'm having the same issue. When I deploy the project the webhook either isn't created in Stripe, or fails authentication and I have to retry the webhook from the stripe dashboard.

Having a similar problem I don't see any webhook created in stripe. I guess there is a way to manually add this step?

@joaogarin I found that if you click the "Skip this form" during setup of the stripe integration, the webhook is not created.
I also discovered if you don't allow some time between supabase setup and then configuring products in Stripe - the integration will fail. I can't reproduce an exact use case.

Thank you, will try it again see if I can get a different result.

Sorry for this. We've added some instructions on how to set up the webhooks manually: https://github.com/vercel/nextjs-subscription-payments#configure-webhook


dorssel.usbipd-win version 0.3.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This is an MSI installer. I tested winget locally with silent, normal (progress), and interactive. All behave as they should. Could it be that the validation server is just overloaded and simply timed out?

@dorssel are there any dependencies required? Sometimes we see UI pop when there are dependencies.

I found the problem. It's a driver popup for Oracle that I locally have in my trusted publishers list...
Will have to rethink this. In any case, this installer won't work. Will abort this PR.

I fixed the problem by improving the installer. It now detects silent install and will deal with the driver install accordingly. Updated to 0.3.1. It now verifies.


Splash screen icon size

When opening an android app built with bubblewrap, the splash screen icon is huge, specially for small/medium devices.
Looking at the code, the splash screen icon size goes from 300 to 1200, these shouldn't be the sizes of a splash screen icon.
const SPLASH_IMAGES: IconDefinition[] = [
  {dest: 'app/src/main/res/drawable-mdpi/splash.png', size: 300},
  {dest: 'app/src/main/res/drawable-hdpi/splash.png', size: 450},
  {dest: 'app/src/main/res/drawable-xhdpi/splash.png', size: 600},
  {dest: 'app/src/main/res/drawable-xxhdpi/splash.png', size: 900},
  {dest: 'app/src/main/res/drawable-xxxhdpi/splash.png', size: 1200},
];

Is it possible to generate the splash screen icons in a lower dimension?
it could be these dimensions
MDPI icon-48x48, splash-470x320

HDPI icon-72x72, splash-640x480

XHDPI icon-96x96, splash-960x720

XXHDPI icon-144x144

XXXHDPI icon-192x192

UP
for me too, it's an important point in my application
for most of my clients the logo is huge on the splash screen

Duplicate of https://github.com/GoogleChromeLabs/bubblewrap/issues/226.


Typescript annotations in JS?

Hi,
I checked out your source code and found it quite odd to see typescript annotations within JS files.
So my question: what part exactly transpiles it? Babel? Is that feature documented anywhere?
And why not go with Typescript then?
Regards

Ok just realized its actually flow that is doing that.


Support tow display modes

Support tow display modes. Native windows can be displayed independently and  embedded native window to flutter

I don't get exactly. You can already launch a separate window through FFI or platform channels.


Package lists should not be arch-dependent

We need to be able to filter out stuff like intel-media-driver because that will break the aarch64 builds. Marco has a package list incoming ...

What’s the alternative?


Django 1.11.3 ?

Hi after upgrading to Django 1.11.3 i get this error :
Exception Value: context must be a dict rather than Context.
when using in my template :
{% social_widget_render "facebook/share_button.html" href=meta.url %}

Getting the same issue here on 1.11

the same issue on 1.11.3

the same issue on 1.11.4

I am still getting the same error in 1.11.5


Feat pip 2 and 3

Description
implementing pip2 and pip3
Related issue(s)
If this Pull Request is related to an issue, mention it here.

Fixes #695

@b00f
still some unit tests fail. I didn't fix them because I've wanted you see them, maybe I make mistake in business logic.

@amirvalhalla Thanks


[DEV-2959] improve expiry time selection

DEV-2959 improve expiry time selection

[ ] Move new DateAndTimePicker component to packages (and refactor) once approved


Improvement in the docker run workflow

Add a docker-compose file, instead of docker run call
Move all related env. vars in a .env file and document them.
Improve the exposure of docker container run in launch_fluent.

Re point 2, should this all be just env var configurable? Should we not allow it to be configured also via launch_fluent arguments?
Are we proposing to use the docker Python package?
N.b. during hackathon, I created the branch feat/hackathon as a basic way to get around hardcoded container repository (which we could not use) and mount path. Shall we address these in the same issue?
@mkundu1 @raph-luc


Collections and Communities –– URIs?

Hi, Is it correct that each community, sub-community, and collection has its own URI?
If so, will we need 2 more keys in the i18n folder for the labels?
I see we have already "item.page.uri": "URI" but not for collections/communities

That i18n key is used as a label on item pages for the dc.identifier.uri field. As far as I can tell we're not rendering that field for communities or collections anywhere, so I don't think we need an i18n key for it.
If the question is: shouldn't we render the handle for collections and communities on their homepages, then that's a different matter.
Since we'll no longer have the handle in the UI URLs in dspace 7, the handle URL can no longer be easily derived from it. So yes in my opinion it makes sense to add it to the community/collection homepages.

Sounds good, thanks.
So seems to me it's consistent to add the keys
"collection.page.handle" and "community.page.handle".
?

Closing, as this was already done. Communities and Collections display URIs on their homepages


New version: BookxSoftware.BookxNotePro version <IP_ADDRESS>1

This pull request is automatically generated by 🥟 Dumplings in #3324
Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 695.)

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-02-07 02:44:10.768 [FAIL] Installer failed security check. Url: http://www.bookxnote.com/setup/bookxnotepro-windows-x86-<IP_ADDRESS>1-20240206.zip Result: 0x80004005

(Automated response - build 695.)

/AzurePipelines run


bs-alert sends an action when it is dismissed

This allows consumers to react to the dismissal, and helps in cases
where we are reusing the same bs-alert with different messages, to
ensure that the component is torn down and re-rendered, rather than
staying around stuck in its hidden state.
fixes #223
@mixonic for your review, in case there's a better way to "reset" this component.
If it is possible for it to observe its template contents then it could flip
from hidden -> visible when the contents change. I'm not sure if that's easy to do, though.

My only thought is that the component could just hide. The "dismissed" state is directly related to the content being displayed- so once I hide a given message the UI can stay hidden until the success message changed.
I'm not sure if clearing the success message on the controller is really necessary. It probably should be cleared on willTransition, but actually clearing it when the user clicks dismissed doesn't feel necessary.

The component itself isn't torn down, which is part of the problem. There's a little more context in #223.
A better solution (to come) will be to use a flash-style service, like ember-cli-flashes.
Discussed w/ @mixonic in person, merging


Room - Inconsistent error message when entering [[<<> in the room name field

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

Version Number: 1.4.7-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team_
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to + > Start chat > Room.
Click Create room without entering room name.
Type [[<<> (don't copy paste) in the name field.

Expected Result:
The error message for invalid room name should be consistent.
Actual Result:
The error message for invalid room name is not consistent.
The error message changes from "Room names can only include lowercase letters, numbers and hyphens" to "Invalid characters" when typing [[<<>.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/842c4f1f-e183-438a-8bb4-167e6c0d41c8
View all open jobs on GitHub

Not reproduced on Production
https://github.com/Expensify/App/assets/93399543/f30b571d-5518-45bb-b074-e917950b9df1

Offending PR https://github.com/Expensify/App/pull/30996
But I think the implementation of PR is correct
This is a problem with the validation of Form and FormProvider

There were some changes in form validation in old Form component during the migration and not all changes were included in the new FormProvider. It is fixed in this commit https://github.com/Expensify/App/pull/32432/commits/b9fc2f4b464f81c286f2e8ab9905625a8b8797fd

Revert deployed so closing this out!


⚠ Bofrost.lu has degraded performance

In 6773f8c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1433 ms

Resolved: Bofrost.lu performance has improved in ed8d0f5 after 6 minutes.


fix ci

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

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

Project coverage is 82.73%. Comparing base (b5769e4) to head (c5c2377).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #858   +/-   ##
=======================================
  Coverage   82.73%   82.73%
=======================================
  Files         120      120
  Lines       34937    34937
=======================================
  Hits        28905    28905
  Misses       6032     6032

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


feat(config): configure which screenshots to take

Action:
I decided to add the takeWhen config for screenshot plugin in package.json for those who explicitly want to configure which automatic screenshots to take, and which not:
  "detox": {
    "artifacts": {
      "plugins": {
        "screenshot": {
          "takeWhen": {
            "testDone": true
          }
        }
      }
    }
  }

Motivation:

appNotReady.png screenshot (https://github.com/wix/Detox/pull/1971) is hopefully going to be one of the first we will be adding to the plugin.
a testStart screenshot of the current test is usually identical to a testDone of the previous test.

@noomorph kudos. Can you also put this on Detox docs wherever seems apt? Among other things -- what's the default value for each field?

@d4vidi , please check #1998.


[Feature] Change default registration behaviour

Currently, registration, region summaries and figures are not run by default. I think it makes sense to have everything as an opt out (to make usage simpler for new users).

Updated in 0.4.0 (brainreg branch).


Correctly handle assert_* intrinsics

Description of changes:
Adds support for the assert_uninit and assert_zero intrinsics. Inserts the code for them and assert_inhabited into a single function.
Resolved issues:
Resolves #6
Resolves #7
Call-outs:

All backends check the condition for assert_inhabited first, then proceed to do the other ones. This proposal follows the same pattern.
The actual error for assert_inhabited (i.e., in src/test/rmc/Intrinsics/Assert/inhabited_panic.rs) is

[main.assertion.1] line 13 a panicking function std::mem::MaybeUninit::<T>::assume_init is invoked: FAILURE

This is due to the Nevers hook being applied instead of the Intrinsics one. I have tried doing some changes to the application condition (for example, return false if the instance is an intrinsic) to no success (ended up in some code that expected the Nevers hook to have applied).

Some reorganization in tests. The top level folder Assert now contains the multiple assertions test only, and another Assert folder has been created within Intrinsics to keep the tests related to the intrinsics in this PR.

Testing:

How is this change tested? Existing regression + 3 new tests + 1 removed test.

Is this a refactor change? No.

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
[x] Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

@zhassan-aws brought #648 to my attention, thanks!!
Removing/Changing output_of_instance_is_never seems the way to go, but it appears that our backend relies on it in a few places so I do not think it will be trivial. I could create a tracking issue for the inhabited.rs test and come back to check once the change has taken place.


Fixes GLIBC 2.29 issues, electron issues, and more

This fixes the following issues:

Broken prebuild linux packages for anything < Ubuntu 20.04, Debian 11, and more
Running Electron apps on any OS < Ubuntu 20.04, and Debian 11
Unable to install better-sqlite3 on those OSs

Compatibility
This causes no compatibility issues with later versions of GLIBC, so all new OSes work just fine. I have confirmed this on Ubuntu 20.04 using a package built on 18.04. It should be a set and forget fix.
This took some real sleuthing to figure out for my app. Hope this helps others!
Workaround until merge
Workaround until this is merged: Downgrade to 7.1.2, as this was built on 18.04 before Github changed ubuntu-latest build images to 20.04 instead of 18.04
Fix #586
Fix #403

@JoshuaWise hoping you have a minute to look at this! Simple fix that helps a lot of folks :-)

Thanks!

This has been published in v7.4.1

This is amazing. Thank you so much for figuring this out, I was running into the same issues. Great work!

@rathboma Hmm, we have report that our user on Ubuntu 18.04 has the same issue with better-sqlite 8.0.0 + electron
Failed to process command Error: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /tmp/.mount_RemNotvHCOdw/resources/app.asar.unpacked/node_modules/better-sqlite3/build/Release/better_sqlite3.node)
    at process.func [as dlopen] (node:electron/js2c/asar_bundle:5:1810)
    at Module._extensions..node (node:internal/modules/cjs/loader:1205:18)
    at Object.func [as .node] (node:electron/js2c/asar_bundle:5:2037)
    at Module.load (node:internal/modules/cjs/loader:988:32)
    at Module._load (node:internal/modules/cjs/loader:829:12)
    at c._load (node:electron/js2c/asar_bundle:5:13339)
    at Module.require (node:internal/modules/cjs/loader:1012:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at bindings (/tmp/.mount_RemNotvHCOdw/resources/app.asar/node_modules/bindings/bindings.js:112:48)
    at new Database (/tmp/.mount_RemNotvHCOdw/resources/app.asar/node_modules/better-sqlite3/lib/database.js:48:64) {
  code: 'ERR_DLOPEN_FAILED'
}

OS
Kernel Version - 5.4.0-84-generic
Name - Ubuntu Linux
Version - 18.04


Add styles to the project

Add CSS style to the project

create the project style using CSS import
create is mobile first display using min-width of 768px for desktop view
use react-progressive-bar to display the progress bar of the project
use flexbox to display various segment of the page

Thank you very much 🙏🏾


Build error iOS

System:
OS: macOS 11.5.2
CPU: (8) arm64 Apple M1
Memory: 735.16 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.6.1 - /opt/homebrew/bin/node
Yarn: 1.22.11 - /opt/homebrew/bin/yarn
npm: 7.20.5 - /opt/homebrew/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.10.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
Android SDK:
API Levels: 29, 30, 31
Build Tools: 28.0.3, 29.0.2, 30.0.3, 31.0.0
System Images: android-29 | Intel x86 Atom, android-31 | Google APIs ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7583922
Xcode: 12.5.1/12E507 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.2 => 0.64.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
I am getting this error

How did you fix this?

If you get the error Incompatible pointer types sending 'UIView *' to parameter of type 'AnimationView * _Nonnull' or Incompatible pointer types initializing 'UIView *' with an expression of type 'AnimationView *'
please ensure that the AppDelegate is a Objective C file and not an Objective C++ file.  To change this, rename it from AppDelegate.mm to AppDelegate.m


stubs mappings directory should be the same for convert and run mojo.

this allows to easy convert and run stub server from project

Current coverage is 56.54% (diff: 100%)

Merging #89 into master will not change coverage

@@             master        #89   diff @@
==========================================
  Files           166        166
  Lines          4550       4550
  Methods           0          0
  Messages          0          0
  Branches        763        763
==========================================
  Hits           2573       2573
  Misses         1667       1667
  Partials        310        310

Powered by Codecov. Last update e8031c0...5042840


Include scopeValidation race condition fix into a patch release

https://github.com/aspnet/DependencyInjection/pull/470#issuecomment-281744594
/cc @Eilon @davidfowl @DamianEdwards

Would this be for 1.0.x, 1.1.x, or both?

@Eilon this feature was added in 1.1 so no need for a patch in 1.0, I guess.
Personally I would like a patch release for this. We used this feature and hit the error (as mentioned here) so we had to disable it.

Thanks, we'll evaluate this for the next patch release.

This patch fix is approved. Please follow the normal code review / pull request process and make sure you make the change in the correct branch.

Fixed in https://github.com/aspnet/DependencyInjection/pull/513

Preview builds of this patch fix should be available on the following feeds:

https://dotnet.myget.org/gallery/aspnet-1-0-5-may2017-patch-public (for fixes in the 1.0.x release)

Feed URL to add to NuGet.config: https://dotnet.myget.org/F/aspnet-1-0-5-may2017-patch-public/api/v3/index.json

https://dotnet.myget.org/gallery/aspnet-1-1-2-may2017-patch-public (for fixes in the 1.1.x release)

Feed URL to add to NuGet.config: https://dotnet.myget.org/F/aspnet-1-1-2-may2017-patch-public/api/v3/index.json

If you have a chance, please try out these preview builds and let us know if you have any feedback!

Awesome @Eilon, thanks! I'll try them out tomorrow.

@Eilon is it correct that there are no patch packages for Microsoft.AspNetCore.Mvc.Razor.ViewCompilation?

@henkmollema there will be an update to that package, but I think only due to its dependencies (e.g. MVC) being updated. I don't think there are any actual functional fixes in that package.

@Eilon @pakrym  I still hit some kind of race condition when enabling scope validation. I've created a repro here: https://github.com/henkmollema/DiConcurrencyBug.
/cc @davidfowl

@henkmollema can you file a separate bug please, I'll investigate it today.

@pakrym done: #518.

@henkmollema @Eilon This was caused by my mistake, fix was backported to a wrong branch and is not included in the build on the patch feed.

I merged it to the right branch and verified that package built from it fixes the issue.

I see, thanks!

When can I expect these packages to show up at https://dotnet.myget.org/gallery/aspnet-1-1-2-may2017-patch-public?

@henkmollema the builds happen automatically, and I see the last build was pushed about 3 hours ago. Can you try updating to see if you get the new packages?

@Eilon a bit late, but it works. Thanks!

@henkmollema good to know, thanks!


Items from OS status bar are displayed in side panel

Open side panel
Switch between "Client" and "Social" tabs several times

Actual Result: Items from OS status bar are displayed in side panel (see the attached image for details)

Expected Result: No changes in side panel

The reason for this bug is an event from 3rd party component pan gesture recognizer.
"switch panGesture.state {
case UIGestureRecognizerState.began:"


Nexus Web file upload does not handle File Too large case (HTTP 413).

When uploading a file that is too large, the request will fail with a 413 error code, however Nexus Web does not display any helpful error message or any message at all, it will act as if the upload succeeded.

@kenjinp somewhere between the SDK and the reducer, the call to createFile() is not awaited on, and the returned in-progress Promise is considered as a positive proof of success of the operation.

This has now been fixed as part of the refactor and new SDK implementation.


Correctly propagate error message when sending an RPC

Description
In the oracle repo, we're trying to account for specific reasons why a transaction may fail. We found that when a transaction fails and the RPC returns a specific error, it was being logged with DEBUG=* set, but the error itself was not being propagated to the call site. Instead, just a generic error Error: Invalid JSON RPC response: {"id":10,"jsonrpc":"2.0"} is given.
Here's an example without this PR, where I've set the transaction to have a gas price that is lower than the gas price minimum. You can see the error gasprice is less than gas price minimum is given in the RPC response, but the useful error information is stripped.
rpc:response { jsonrpc: '2.0',
  rpc:response   id:<PHONE_NUMBER>344892,
  rpc:response   error:
  rpc:response    { code: -32000,
  rpc:response      message: 'gasprice is less than gas price minimum' } } +1s
  kit:provider:response { id: 10, jsonrpc: '2.0', result: undefined } +0ms
  kit:tx:result send-error: Error: Invalid JSON RPC response: {"id":10,"jsonrpc":"2.0"} at Object.InvalidResponse (/Users/trevor/celo-oracle/node_modules/web3-core-helpers/src/errors.js:42:16) at /Users/trevor/celo-oracle/node_modules/web3-core-requestmanager/src/index.js:144:36 at decoratedCallback (/Users/trevor/celo-oracle/node_modules/@celo/contractkit/lib/providers/celo-provider.js:102:13) at /Users/trevor/celo-oracle/node_modules/@celo/contractkit/lib/utils/rpc-caller.js:50:13 at process._tickCallback (internal/process/next_tick.js:68:7) +0ms
[2020-06-11T20:26:14.408Z] ERROR: oracle/64779 on Trevors-MacBook-Pro.local: Error reporting (context=reporter, errorContext=report)
    Error: Invalid JSON RPC response: {"id":10,"jsonrpc":"2.0"}

With these changes, we now have more informative errors:
  rpc:response { jsonrpc: '2.0',
  rpc:response   id:<PHONE_NUMBER>447892,
  rpc:response   error:
  rpc:response    { code: -32000,
  rpc:response      message: 'gasprice is less than gas price minimum' } } +2s
  kit:provider:response { id: 10,
  kit:provider:response   jsonrpc: '2.0',
  kit:provider:response   result: null,
  kit:provider:response   error:
  kit:provider:response    { message: 'gasprice is less than gas price minimum',
  kit:provider:response      code: -32000 } } +0ms
  kit:tx:result send-error: Error: gasprice is less than gas price minimum at decoratedCallback (/Users/trevor/celo-oracle/node_modules/@celo/contractkit/lib/utils/rpc-caller.js:130:21) at XMLHttpRequest.request.onreadystatechange (/Users/trevor/celo-oracle/node_modules/web3-providers-http/src/index.js:102:13) at XMLHttpRequestEventTarget.dispatchEvent (/Users/trevor/celo-oracle/node_modules/xhr2-cookies/dist/xml-http-request-event-target.js:34:22) at XMLHttpRequest._setReadyState (/Users/trevor/celo-oracle/node_modules/xhr2-cookies/dist/xml-http-request.js:208:14) at XMLHttpRequest._onHttpResponseEnd (/Users/trevor/celo-oracle/node_modules/xhr2-cookies/dist/xml-http-request.js:318:14) at IncomingMessage.<anonymous> (/Users/trevor/celo-oracle/node_modules/xhr2-cookies/dist/xml-http-request.js:289:61) at IncomingMessage.emit (events.js:203:15) at endReadableNT (_stream_readable.js:1145:12) at process._tickCallback (internal/process/next_tick.js:63:19) +0ms
[2020-06-11T20:31:14.499Z] ERROR: oracle/65005 on Trevors-MacBook-Pro.local: Error reporting (context=reporter, errorContext=report)
    Error: gasprice is less than gas price minimum

The reason why this doesn't work out of the box is because of web3's HttpProvider send function: https://github.com/ethereum/web3.js/blob/1.x/packages/web3-providers-http/src/index.js#L95. The callback is not called with an error if the content of the RPC response includes an error.
Other changes
I gave higher precedence to the error message rather than the stack in order to get a clean error (eg gasprice is less than gas price minimum) instead of the error message along with the stack in one big ugly string.
I originally wanted to override the type in the types directory of contractkit, but apparently this doesn't allow you to change the existing type definition and only allows you to extend it. My solution was to patch the type def
Tested
Tested the fix with the oracle client
Related issues
n/a
Backwards compatibility
I believe it should still be backward compatible

Thanks! I did try the web3-celo.d.ts approach, but I was having trouble. I read a bit online that it can only help with adding properties to an object's type definition but not necessarily changing the existing types of properties in an object