Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

There should be a way to check attachment status #128

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Marcus-Peterson
Copy link

I had so many freaking issues with the attach(), detach() stuff. I realized, there wasn't a good way to check the actual attachment status of the SIM-card. So I made this, it works.

Marcus-Peterson and others added 2 commits February 26, 2024 14:39
One problem with the MKRNB is that sometimes when the sim card inside of the MKRNB has the status of attached, which essentially means that the sim card will be attached to the MCU, this can cause connections issues, so if you accidentally attached it, you can now easily check the status
@CLAassistant
Copy link

CLAassistant commented Mar 20, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Marcus-Peterson
❌ Marcus Peterson


Marcus Peterson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Comment on lines -1 to +12
// Define the repository information in these attributes
:repository-owner: arduino-libraries
:repository-name: MKRNB
:repository-owner: marcus-peterson
:repository-name: MKRNB_v2

= {repository-name} Library for Arduino =
= MKRNB_v2 Library for Arduino =

image:https://github.com/{repository-owner}/{repository-name}/workflows/Compile%20Examples/badge.svg["Compile Examples Status", link="https://github.com/{repository-owner}/{repository-name}/actions?workflow=Compile+Examples"]
image:https://github.com/{repository-owner}/{repository-name}/workflows/Spell%20Check/badge.svg["Spell Check Status", link="https://github.com/{repository-owner}/{repository-name}/actions?workflow=Spell+Check"]
image:https://github.com/marcus-peterson/MKRNB_v2/workflows/Compile%20Examples/badge.svg["Compile Examples Status", link="https://github.com/marcus-peterson/MKRNB_v2/actions?workflow=Compile+Examples"]
image:https://github.com/marcus-peterson/MKRNB_v2/workflows/Spell%20Check/badge.svg["Spell Check Status", link="https://github.com/marcus-peterson/MKRNB_v2/actions?workflow=Spell+Check"]

This library enables an Arduino MKR NB 1500 board to connect to the internet over a NarrowBand IoT or LTE Cat M1 network.
This library enables an Arduino MKR NB 1500 board to connect to the internet over a NarrowBand IoT or LTE Cat M1 network. It is an extension of the original MKRNB library, designed to provide enhanced control and capabilities for commercial applications.

For more information about this library please visit us at
https://www.arduino.cc/en/Reference/{repository-name}
For more information about the source of this library please visit the official Arduino MKRNB documentation
https://www.arduino.cc/en/Reference/MKRNB
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Although the changes made here would be good to make in your own "hard fork" of the library, they would be harmful to Arduino's repository if introduced by the merge of this pull request. Please revert these changes.

Note that the best workflow for pull requests from a fork is to create a dedicated branch in the fork repository in which to stage the changes made in the pull request. You submitted this pull request from the master branch of your fork and any commit you push to the master branch becomes part of the changeset of this pull request.

@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Mar 24, 2024
Copy link

Memory usage change @ 017c826

Board flash % RAM for global variables %
arduino:samd:mkrnb1500 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table
Board examples/GPRSUdpNtpClient
flash
% examples/GPRSUdpNtpClient
RAM for global variables
% examples/NBSSLWebClient
flash
% examples/NBSSLWebClient
RAM for global variables
% examples/NBWebClient
flash
% examples/NBWebClient
RAM for global variables
% examples/ReceiveSMS
flash
% examples/ReceiveSMS
RAM for global variables
% examples/SendSMS
flash
% examples/SendSMS
RAM for global variables
% examples/Tools/ChooseRadioAccessTechnology
flash
% examples/Tools/ChooseRadioAccessTechnology
RAM for global variables
% examples/Tools/NBScanNetworks
flash
% examples/Tools/NBScanNetworks
RAM for global variables
% examples/Tools/PinManagement
flash
% examples/Tools/PinManagement
RAM for global variables
% examples/Tools/SerialSARAPassthrough
flash
% examples/Tools/SerialSARAPassthrough
RAM for global variables
% examples/Tools/TestGPRS
flash
% examples/Tools/TestGPRS
RAM for global variables
% examples/Tools/TestModem
flash
% examples/Tools/TestModem
RAM for global variables
%
arduino:samd:mkrnb1500 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0
Click for full report CSV
Board,examples/GPRSUdpNtpClient<br>flash,%,examples/GPRSUdpNtpClient<br>RAM for global variables,%,examples/NBSSLWebClient<br>flash,%,examples/NBSSLWebClient<br>RAM for global variables,%,examples/NBWebClient<br>flash,%,examples/NBWebClient<br>RAM for global variables,%,examples/ReceiveSMS<br>flash,%,examples/ReceiveSMS<br>RAM for global variables,%,examples/SendSMS<br>flash,%,examples/SendSMS<br>RAM for global variables,%,examples/Tools/ChooseRadioAccessTechnology<br>flash,%,examples/Tools/ChooseRadioAccessTechnology<br>RAM for global variables,%,examples/Tools/NBScanNetworks<br>flash,%,examples/Tools/NBScanNetworks<br>RAM for global variables,%,examples/Tools/PinManagement<br>flash,%,examples/Tools/PinManagement<br>RAM for global variables,%,examples/Tools/SerialSARAPassthrough<br>flash,%,examples/Tools/SerialSARAPassthrough<br>RAM for global variables,%,examples/Tools/TestGPRS<br>flash,%,examples/Tools/TestGPRS<br>RAM for global variables,%,examples/Tools/TestModem<br>flash,%,examples/Tools/TestModem<br>RAM for global variables,%
arduino:samd:mkrnb1500,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants