text:
Fix slow path in reverse_conditional_swap

It seems that this is slow branch, and its logic should be identical to fast branch.
  inline void operator()(_RandomAccessIterator __x,
                         _RandomAccessIterator __y) const {
    typedef typename _VSTD::iterator_traits<_RandomAccessIterator>::value_type
        value_type;
    bool __result = !comp_(*__x, *__y);
    // Expect a compiler would short-circuit the following if-block.
    if (_VSTD::is_trivially_copy_constructible<value_type>::value &&
        _VSTD::is_trivially_copy_assignable<value_type>::value &&
        sizeof(value_type) <= 4 * sizeof(size_t)) {
      value_type __min = __result ? _VSTD::move(*__x) : _VSTD::move(*__y);
      *__y = __result ? _VSTD::move(*__y) : _VSTD::move(*__x);
      *__x = _VSTD::move(__min);
    } else {
      if (!__result) {
        _VSTD::iter_swap(__x, __y);
      }
    }
  }

Logic of fast branch,
if (result)
    min = x;
    y = y;
    x = x;
else
    min = y;
    y = x;
    x = y;

We swap elements only if result is false.
Example to reproduce sort bug:
int main(int argc, char ** argv)
{
    (void)(argc);
    (void)(argv);

    std::vector<std::pair<int64_t, int64_t>> values = {
        {1, 1},
        {3, -1},
        {2, 1},
        {7, -1},
        {3, 1},
        {999, -1},
        {4, 1},
        {7, -1},
        {5, 1},
        {8, -1}
    };

    ::stdext::bitsetsort(values.begin(), values.end());
    bool is_sorted = std::is_sorted(values.begin(), values.end());

    std::cout << "Array " << values.size() << " is sorted " << is_sorted << std::endl;

    for (auto & value : values)
        std::cout << value.first << " " << value.second << std::endl;

    return 0;
}

Output before fix:
Array 10 is sorted 0
1 1
2 1
3 -1
3 1
4 1
7 -1
7 -1
8 -1
5 1
999 -1

Output after fix
Array 10 is sorted 1
1 1
2 1
3 -1
3 1
4 1
5 1
7 -1
7 -1
8 -1
999 -1

Fun fact, in the LLVM patch https://reviews.llvm.org/D93233 everything is set correctly

source: gharchive/pull-request

repo: minjaehwang/bitsetsort

license_type: permissive

license_source: github-api


text:
🛑 [LU] Kylz.nl is down

In b5bc41e, [LU] Kylz.nl (https://kylz.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: [LU] Kylz.nl is back up in e583cce.

source: gharchive/issue

repo: minoplhy/upptime-status

license: MIT


text:
Remove / from .git suffix for binderUrl

This PR removes the / from the .git suffix for the binderUrl. When including the / the repository is not recognized, for example by a privately hosted GitLab server.
Now the following config DOES work, while without this PR it does not:
binderOptions: {
      repoProvider: "git",
      repo: "https://gitlab.example.com/user/repo",
      ref: "e6c970126f4e819e4a3eb717a86ba9fb5237fde3",
    }

Thanks!

authors: hugokerstens minrk

repo: minrk/thebelab


text:
🛑 Matrix is down

In 207d62b, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in 8b94a15 after 17 minutes.

repo: mint-lgbt/status


text:
V2.1 get_transaction_data() not isolating a single account when passing an id

Version 1.x allowed for getting transactions from a single account.  It appears that get_transaction_data() accepts an id field, but regardless of which id I pass, it seems to return all transactions for all accounts (up to the limit).

@dsteiner10 can you provide a snippet of the code you are using to show the behavior?
It does look like we are passing the id to the endpoint if supplied, so I'll be curious to know if that's simply not what Mint wants.

I'm having the same issue.  I have verified that you are passing the id to mint using print to show the url you are using:
account id is: 11277025_11182526
**** url before id appended: https://mint.intuit.com/pfm/v1/transactions?limit=5000&
**** url after id appended: https://mint.intuit.com/pfm/v1/transactions?limit=5000&id=11277025_11182526&
I think the issue is that mint doesn't recognize it as a valid filter.  What I get back is data for all my accounts.  This is the first part of what is returned to me.  The first transaction (for example) is from another account and you can see the id does not match:
[{'type': 'CashAndCreditTransaction', 'id': '11277025_2497679463_0', 'accountId': '11277025_12036151', 'accountRef': {...}, 'date': '2022-06-21', 'description': 'TARGET DEBIT CRD : ACH TRAN', 'category': {...}, 'amount': -17.77, 'status': 'POSTED', ...}, {'type': 'CashAndCreditTransaction', 'id': '11277025_2497679462_0', 'accountId': '11277025_12036151', 'accountRef': {...}, 'date': '2022-06-21', 'description': 'TARGET DEBIT CRD : ACH TRAN', 'category': {...}, 'amount': -6.44, 'status': 'POSTED', ...}, {'type': 'CashAndCreditTransaction', 'id': '11277025_2497303724_0', 'accountId': '11277025_10967816', 'accountRef': {...}, 'date': '2022-06-21', 'description': 'WITHDRAWAL Debit Car... 30 #5421 ', 'category': {...}, 'amount': -8.53, 'status': 'POSTED', ...}, {'type':

Here is the code that I was previously running on 1.x:
I found the same thing that chdavis8 found.  mintapi is passing the id in the url, but it doesn't seem like mint does anything with the id.
`# Get basic account information
trans_dict=[]
roi_accts = ['CK','SV','NT','CC','RE','RL','RT','SL','IV','VE','VL','ED']
account_info = mint.get_account_data()
for num,account in enumerate(account_info[:5]):
if account["name"].startswith(tuple(roi_accts)):
print(num,"-",account["name"], account['id'])
trans_dict.append(mint.get_transaction_data(limit=10000, start_date=start_date, include_investment=False,id=account["id"]))
#print(trans_dict[-1])
else:
print("###################ACCOUNT NOT TRACKED:",account["name"],"###########################")`

Yes I found the same thing. I may just throw in a filter to the data returned from the API for the time being.

Actually, I'm going to hold off on the idea mentioned as #509 will most definitely address this need.

I'm going to close this one out in favor of the linked issue.

repo: mintapi/mintapi


text:
Loading times in gabo.es are around to 38-55secs the first time you load a document

Describe the bug A clear and concise description of what the bug is.
Loading times in gabo.es are around to 38-55secs the first time you load a document
To Reproduce Steps to reproduce the behavior:

Go to gabo.es
See that the loading times are very long.
When I reload a second time the page, it loads instantly. But if I wait 5mins it has the same slow behaviour again.

Expected behavior A clear and concise description of what you expected to
happen.
The Site should already have the document, therefore loading should be instant.
Screenshots If applicable, add screenshots to help explain your problem.

what's the second transclusion actually doing? what's the underlying API call? @ericvicenti

can you export the capture in a HAL file?

Is there a way to see this issue locally?
The server side is taking a very long time to load:

Load siteInfo
getPath with the pretty path
getAccount of the author

https://github.com/mintterteam/mintter/blob/master/frontend/gateway/pages/[pageSlug].tsx#L26-L52

related to https://github.com/mintterteam/mintter/issues/1325 ?

@GaboHBeaumont is this still a problem?

Nope! Issue solved!
On Sun, 16 Apr 2023 at 10:59, Horacio Herrera @.***>
wrote:

@GaboHBeaumont https://github.com/GaboHBeaumont is this still a problem?
—
Reply to this email directly, view it on GitHub
https://github.com/mintterteam/mintter/issues/1332#issuecomment-1510201672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7TCNFSQ5GR7I3OOMOQJRDXBOYFNANCNFSM6AAAAAAWBIJ5GQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Gabriel H Beaumont
mintter.com
@gabohbeaumont https://twitter.com/GaboHBeaumont
+34655943504
Madrid

authors: GaboHBeaumont ericvicenti horacioh juligasa

repo: mintterteam/mintter


text:
GDPR compliance

Implement compliance with GDPR. @patrys is already working on a detailed list of changes to add in Saleor. Once we have it, we'll create specific issues with details.

Let's start a project for this so I can file notes and you'll convert them to issues as you see fit.

https://github.com/mirumee/saleor/projects/8

"Cookies" are not the right term here. There are different kinds of cookies. For example the session cookie is used to keep track of the cart and we don't need a separate agreement to do that since this is the main reason of your visit. Tracking cookies on the other hand are used to gather information that is not critical to fulfill our obligations so they do need an explicit consent and cannot be enabled by default. I think it's more useful to think in terms of processing goals rather than focusing on the means.

I think we can close this issue, since we have this project.

authors: elwoodxblues maarcingebala patrys

repo: mirumee/saleor


text:
VAT issue

What I'm trying to achieve
Hi,
Thanks for all your efforts for Saleor and support here.
I am new using Saleor, and currently I am trying to develop an e-commerce web site with it. As I do not have so much experience with front-end frameworks, I decided to use the branch/version 2.9 with Django templates. Is it a bad idea? What  do you think?
So, my problem is that I can not get tax setting working. I tried different things but it does not work at all. So I changed settings:
DEFAULT_COUNTRY = "GB"
DEFAULT_CURRENCY = "GBP"
VATLAYER_ACCESS_KEY = "dfd5.................."
In http://127.0.0.1:8000/dashboard/taxes/ page I get the list of countries. When I fetch the tax rates, I get "Tax rates updated successfully".
I get confused in this step. I will only sell products in the UK. So will the tax calculated in the checkout according to the country?
I tried to test the payment but the tax is not calculated in the checkout at all.
In the http://127.0.0.1:8000/dashboard/taxes/configure-taxes/ page, I tried with different options, but the tax is not calculated.
What I expected to happen
What I am trying to is to add tax for certain products and let the customer choose for some other product types.
What would be the best approach? At the end I am planning to add the tax as a variant. So the customer could be able to choose, do you think it is a good idea?

Hi,
Can you provide those details:

Do see the tax rate for UK in the dashboard?
All products prices are entered with tax included is checked or not checked?
Did you try to pay using UK as shipping country?
Charge taxes for this item is checked on products?
Product Types have a tax type (e.g. Standard)?

I will only sell products in the UK. So will the tax calculated in the checkout according to the country?

EU law requires to charge the VAT in accordance to shipping country.

What I am trying to is to add tax for certain products and let the customer choose for some other product types.

For that, you would use a tax type/rate for the product type, for example "Standard", "Medicines", "Food", etc.

I decided to use the branch/version 2.9 with Django templates. Is it a bad idea? What do you think?

That's fine but no longer maintained.


text:
i18n for category names

We should use translated category names e.g. "Hra" instead of "Game".

The database model is prepared to contain category names in different languages, but there is currently no data source for that. I wait with the decision to hardcode any data. First we should look around and try to find some source we do not need to maintain.

authors: jmlich misli

repo: misli/fedora-software

license_source: bigquery


text:
chore: add zh-cn localization config to vuepress config file

The site don't have entry for zh-CN document, this PR adds it.

🙏🏻

repo: misskey-dev/misskey-hub


text:
Cannot drag markers on ai2-test

Describe the bug

Markers can no longer be dragged around in the designer on ai2-test.
Affects

[x] Designer
[ ] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Other... (please describe)

Expected behavior

Markers should be draggable.
Steps to reproduce

Create a new project with a Map
Place a Marker
Attempt to drag the marker. It won't move.

I was able to repro this on Chrome and Firefox.

Fixed by #1937

repo: mit-cml/appinventor-sources


text:
Make PhoneCall request READ_PHONE_STATE dangerous permission also

Fixes #1719
Change-Id: Ibacdc5f81f57ab6e49dc906daec47ae593fc94bc

@ewpatton Moved this PR to the ucr branch...

authors: ewpatton jisqyv


text:
Update AIProject.java

Improvements Implemented:
Improved Readability:
Separated the initialization logic into a dedicated initializeProject() method.
Added detailed comments for methods and fields to clarify their purpose.
Replaced repetitive error handling with a helper method showErrorDialog() to streamline error messaging.
Enhanced Robustness:
Used a try-with-resources block for managing the ZipFile to ensure proper resource handling and avoid potential memory leaks.
Improved validation logic in validateProject() to ensure that essential components are present before marking the project as valid.
Better Exception Handling:
Consolidated exception handling for ZipException and IOException with clearer, user-friendly error messages.
Avoided nested exception-prone logic by structuring the code more systematically.
Modern Java Practices:
Used final for immutable fields to enforce immutability.
Leveraged generics for collections (LinkedList and LinkedList).
Simplified file name extraction with the File class methods, reducing manual string manipulation.
Increased Maintainability:
Encapsulated reusable logic (e.g., error dialog display) in helper methods.
Ensured that the code adheres to single responsibility principles by breaking complex logic into smaller, focused methods.
Consistency in Naming and Formatting:
Ensured consistent naming conventions for methods and variables.
Reformatted and structured code for better readability.

Can one of the admins verify this patch?

Thank you for pointing this out. I have adjusted the indentation throughout the file to follow the Google Java Style Guide, ensuring that block indentation now uses 2 spaces consistently. Please let me know if there are any additional style issues or further improvements you’d like to see.


text:
after updating to latest version of HA getting login error

ogger: homeassistant.config_entries
Source: custom_components/volkswagen_we_connect_id/init.py:47
Integration: Volkswagen We Connect ID
First occurred: 16:30:49 (1 occurrences)
Last logged: 16:30:49
Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 382, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/volkswagen_we_connect_id/init.py", line 47, in async_setup_entry
await hass.async_add_executor_job(_we_connect.login)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
self.__session.login()
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 76, in login
response = self.doWebAuth(authorizationUrl)
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 242, in doWebAuth
raise AuthentificationError('You have to login at myvolkswagen.de and accept the terms and conditions')
weconnect.errors.AuthentificationError: You have to login at myvolkswagen.de and accept the terms and conditions

I get same error i logged in to myvolkswagen.de as mentioned in message it dosen't solve the problem

App was updated today so wonder if they did something to the login again.
Sebastian King | @.*** | +44 7711 702100

From: sm4upt @.>
Sent: Tuesday, February 14, 2023 5:11:55 PM
To: mitch-dc/volkswagen_we_connect_id @.>
Cc: Sebastian King @.>; Author @.>
Subject: Re: [mitch-dc/volkswagen_we_connect_id] after updating to latest version of HA getting login error (Issue #143)
I get same error i logged in to myvolkswagen.de as mentioned in message it dosen't solve the problem
—
Reply to this email directly, view it on GitHubhttps://github.com/mitch-dc/volkswagen_we_connect_id/issues/143#issuecomment-1430094271, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWUI3QGO4RCLPSWWBB4DCNTWXO4FXANCNFSM6AAAAAAU3ZKJBU.
You are receiving this because you authored the thread.Message ID: @.***>

I restored HA to 2023.2.3 and still had the issue. Going to myvolkswagen.de and accepting all the terms on my account, seems to have fixed it for me.

logged in to myvolkswagen.de again changed the country accepted all the terms and now it works again.

Same here!
Sebastian King | @.*** | +44 7711 702100

From: sm4upt @.>
Sent: Tuesday, February 14, 2023 6:29:54 PM
To: mitch-dc/volkswagen_we_connect_id @.>
Cc: Sebastian King @.>; Author @.>
Subject: Re: [mitch-dc/volkswagen_we_connect_id] after updating to latest version of HA getting login error (Issue #143)
logged in to myvolkswagen.de again changed the country accepted all the terms and now it works again.
—
Reply to this email directly, view it on GitHubhttps://github.com/mitch-dc/volkswagen_we_connect_id/issues/143#issuecomment-1430191309, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWUI3QEP2BQFA27S5I5ZGYDWXPFKFANCNFSM6AAAAAAU3ZKJBU.
You are receiving this because you authored the thread.Message ID: @.***>

stopped again with another error so guess VW are "doing stuff"

logged in to myvolkswagen.de again changed the country accepted all the terms and now it works again.

Did the trick here too

stopped again with another error so guess VW are "doing stuff"

Same here, with a different error message:
Logger: homeassistant.config_entries
Source: custom_components/volkswagen_we_connect_id/__init__.py:47
Integration: Volkswagen We Connect ID
First occurred: 00.12.00 (1 occurrences)
Last logged: 00.12.00

Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 382, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/volkswagen_we_connect_id/__init__.py", line 47, in async_setup_entry
    await hass.async_add_executor_job(_we_connect.login)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
    self.__session.login()
  File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 76, in login
    response = self.doWebAuth(authorizationUrl)
  File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 243, in doWebAuth
    raise APICompatibilityError('No user id provided')
weconnect.errors.APICompatibilityError: No user id provided

logged in to myvolkswagen.de again changed the country accepted all the terms and now it works again.

This worked for me on .co.uk and setting country to UK in the policy agreement, thanks

All good here now. One to remember for the future.
Sebastian King | @.*** | +44 7711 702100

From: SwannyBFC @.>
Sent: Wednesday, February 15, 2023 11:30:50 AM
To: mitch-dc/volkswagen_we_connect_id @.>
Cc: Sebastian King @.>; Author @.>
Subject: Re: [mitch-dc/volkswagen_we_connect_id] after updating to latest version of HA getting login error (Issue #143)
logged in to myvolkswagen.de again changed the country accepted all the terms and now it works again.
This worked for me on .co.uk and setting country to UK in the policy agreement, thanks
—
Reply to this email directly, view it on GitHubhttps://github.com/mitch-dc/volkswagen_we_connect_id/issues/143#issuecomment-1431221685, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWUI3QFJUV273I6KPLLKNCLWXS46VANCNFSM6AAAAAAU3ZKJBU.
You are receiving this because you authored the thread.Message ID: @.***>

Same error here.
I finally solved. Ive tried the .de and .co.uk policy agreement and finally I went back to .es and works.
I don't know what really happened

I seem to have gotten over the "terms and conditions error but am now stuck with...
weconnect.errors.APICompatibilityError: No user id provided
Not sure where / how I can provide a user id?

Thanks @angelitoo776 - how did you repeat the login in HA.
Did you remove the integration and re-add it?
I can't see any option to do it any other way?

Thanks @angelitoo776 - how did you repeat the login in HA.
Did you remove the integration and re-add it?
I can't see any option to do it any other way?

I removed the integration an re-add it. Unfortunately its a bit messy because you remove the entity id and you lose history data.
I think that you can try to reconnect the integration (it must appear a button to reconnect or something similar in the integration) if it doesn't work... re-add it

I've removed the integration, but now get an "unknown error" when I try to re-add it!
Similar error in the logs.
Logger: custom_components.volkswagen_we_connect_id.config_flow
Source: custom_components/volkswagen_we_connect_id/config_flow.py:39
Integration: Volkswagen We Connect ID
First occurred: 19:34:34 (1 occurrences)
Last logged: 19:34:34

Unexpected exception
Traceback (most recent call last):
  File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 64, in async_step_user
    info = await validate_input(self.hass, user_input)
  File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 39, in validate_input
    await hass.async_add_executor_job(we_connect.login)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
    self.__session.login()
  File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 76, in login
    response = self.doWebAuth(authorizationUrl)
  File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 243, in doWebAuth
    raise APICompatibilityError('No user id provided')
weconnect.errors.APICompatibilityError: No user id provided

I had both and resolved it by:

initial error: You have to login at myvolkswagen.de and accept the terms and conditions
--> go to myvolkswagen.de, login, accept the terms and conditions, done
new error: No user id provided
--> go to We Connect Shop, there i was again presented after login with new terms and conditions, accept it, done
i my case, for what ever reason, all "online" free services like "We Connect Plus", "We Connect" were not selected anymore. Bought all of them AGAIN (did it months ago ...) for 0€ and done.

Reload Integration and SoC et. al is back in my dashboard!

Thanks for the help but still no joy.
When I go to We Connect Shop, everything seems OK (www.volkswagen.co.uk/en/myvolkswagen.html)

I am in the UK.
Did you specifically have to go to myvolkswagen.de?   I went to myvolkswagen.co.uk, as that is where my profile is registered.
Maybe I'll wait a few days (Sadly)

VW may fix something
At the weekend I can go to the upstream PyPi library and see if I can get that to logon there first.

Cheers

Fixed it.  In my case, the EV itself said I needed to re-logon.
So re did my login in the car, and now I can re-add the integration.
What a palaver!

I just logged into the danish Volkswagen site and got two terms to agree to, that didn't show up yesterday, and now it's working again. This is all very confusing.

It's a mess right now. My car had 11 alerts that i need to relogin. After i just did that 😂

Tried to log in to myvokswagen.de and accept terms. Still not working for me.
2023-02-17 19:39:02.834 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id
File "/config/custom_components/volkswagen_we_connect_id/init.py", line 47, in async_setup_entry

Hi for me it's the same problem.
I tried to remove from Hassio integration and also the file from hacs (also file from custom_components)
After reboot of HA I have only...

I wento to my car, into VW customers website: I din't undertsand were I need to accept...
Thank you very much

I got same error and never get it working not matter what I do.
it is either plugging version 1.3
or the new HA version which is broken this integration
update
i have tested 1.2 also same error

I also tried to log in to myvokswagen.de and accept terms. Tried also the UK and Swedish sites. Still not working for me, cannot add the integration as I was quick to remove it.
Have not seen the mentioned prompt in the car yet.
2023-02-19 11:31:23.995 ERROR (MainThread) [custom_components.volkswagen_we_connect_id.config_flow] Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 64, in async_step_user
info = await validate_input(self.hass, user_input)
File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 39, in validate_input
await hass.async_add_executor_job(we_connect.login)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
self.__session.login()
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 76, in login
response = self.doWebAuth(authorizationUrl)
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 242, in doWebAuth
raise AuthentificationError('You have to login at myvolkswagen.de and accept the terms and conditions')
weconnect.errors.AuthentificationError: You have to login at myvolkswagen.de and accept the terms and conditions

Hello from Spain.
After several tries, for me is working from myvolkswagen.de only. i have tried from .es .uk and no success.
Try from .de then accept all. Logout and then login again and accept all.
Keep the country Germany, restar HA and everything works fine.

If you log out of the app, and then login again it should fix it as well.

After some more tries on https://vwid.vwgroup.io/account, switching between countries, logging out and in again and accepted the cookies on the site I managed to re-add the integration to Home Assistant. I also logged out and in in the Android App once. I did not have to login on the Infotainment screen but exactly what step that helped is unclear.
God luck to the rest of You that have problems!

I'm getting a bit of a different error message. I did accept conditions, added country and restarted both HA, relogged in both we connect and HA without any luck:
File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login self.__session.login() File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 75, in login authorizationUrl = self.authorizationUrl(url='https://identity.vwgroup.io/oidc/v1/authorize') File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 96, in authorizationUrl redirect = tryLoginResponse.headers['Location'] File "/usr/local/lib/python3.10/site-packages/requests/structures.py", line 52, in getitem return self._store[key.lower()][1] KeyError: 'location'

Hi,
Same Error since today. Login at my Volkswagen and check if there was again something changed, but there is no change to see. Logout from the App at the Phone and restart HA did not help.

This seems to be an issue with the backend python / VW API:
https://github.com/tillsteinbach/VWsFriend/issues/435

Same here. Was working fine until I restarted HA around 15:25 CET.

@gapthorpe : thx. That explain the issue, so we need wait that Till Steinbach check it.

Let's wait. There is a big possibility that everything will magically work again. If not we need to wait for a fix in the WeConnect library :)

I also have problems. Removed the integration and can't add it again. Error code below. Hope the integration will start to work soon again.
Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 64, in async_step_user
info = await validate_input(self.hass, user_input)
File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 39, in validate_input
await hass.async_add_executor_job(we_connect.login)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
self.__session.login()
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 75, in login
authorizationUrl = self.authorizationUrl(url='https://identity.vwgroup.io/oidc/v1/authorize')
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 96, in authorizationUrl
redirect = tryLoginResponse.headers['Location']
File "/usr/local/lib/python3.10/site-packages/requests/structures.py", line 52, in getitem
return self._store[key.lower()][1]
KeyError: 'location'

From another integration someone has pointed out this
VW modified these URLs:
LOGIN_BASE = "https://emea.bff.cariad.digital/user-login/v1"
API_BASE = "https://emea.bff.cariad.digital/vehicle/v1"
And some other stuff maybe.
Broke TRONITY too but they have resolved.
Sebastian King | @.*** | +44 7711 702100

From: Mitch @.>
Sent: Monday, February 27, 2023 6:10:31 PM
To: mitch-dc/volkswagen_we_connect_id @.>
Cc: Sebastian King @.>; Author @.>
Subject: Re: [mitch-dc/volkswagen_we_connect_id] after updating to latest version of HA getting login error (Issue #143)
Let's wait. There is a big possibility that everything will magically work again. If not we need to wait for a fix in the WeConnect library :)
—
Reply to this email directly, view it on GitHubhttps://github.com/mitch-dc/volkswagen_we_connect_id/issues/143#issuecomment-1446812369, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWUI3QH7LJPREKPB3KMWLRDWZTUZPANCNFSM6AAAAAAU3ZKJBU.
You are receiving this because you authored the thread.Message ID: @.***>

I have the same "KeyError: 'location')" error report in HA. Haven't removed the integration though. Ohme charger stopped seeing the ID's SoC yesterday afternoon too. I've contacted Ohme with the details.

The weconnect library needs to be bumped to v0.54.1. But this causes dependancy conflicts with home assistant. I installed it manually and it seems to work (but I don't recommend this to anyone).
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. T his behaviour is the source of the following dependency conflicts.                                           nuheat 1.0.0 requires requests==2.28.1, but you have requests 2.28.2 which is incompatible.                  ibm-watson 5.2.2 requires websocket-client==1.1.0, but you have websocket-client 1.5.1 which is incompatible .                                                                                homeassistant 2023.2.5 requires requests==2.28.1, but you have requests 2.28.2 which is incompatible.        Successfully installed requests-2.28.2 weconnect-0.54.1

Well, for what it's worth: I did remove the custom integration, restarted home-assistant and then added the integration again. No luck so far, it keeps on failing on "KeyError: 'location' ".

fix is up
https://github.com/tillsteinbach/WeConnect-python/releases/tag/v0.54.1

Steinbach has released a fix now @ChristianH21220 @gapthorpe
https://github.com/tillsteinbach/VWsFriend/releases/tag/v0.23.7

Had the same issue. Tried installing 1.1.4 that was just send out as pre-release. Now I get the following in the log:
`Logger: homeassistant.util.package
Source: util/package.py:100
First occurred: 13.51.10 (3 occurrences)
Last logged: 13.51.13
Unable to install package weconnect==0.54.1: ERROR: Cannot install weconnect==0.54.1 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts [notice] A new release of pip available: 22.3 -> 23.0.1 [notice] To update, run: pip install --upgrade pip `

just installed version V0.1.4 via hacs now i get this error ''Config-flow kon niet geladen worden: 500 Internal Server Error Server got itself in trouble'' anyone have an idea? version V0.1.3 i get one Unexpected error when I enter the username and password

just installed version V0.1.4 via hacs now i get this error ''Config-flow kon niet geladen worden: 500 Internal Server Error Server got itself in trouble'' anyone have an idea? version V0.1.3 i get one Unexpected error when I enter the username and password

I get the same response with the error mentioned above in the log.

Just like I wrote above... this integration needs the newer version (0.54.1) of weconnect python library. But weconnect v0.54.1 depends on requests v2.28.2. Home assistant core depends on requests v2.28.1. Due to these dependancy conflicts it's impossible to resolve this properly right now. Home assistant core will need to bump the requests package dependancy.
You can manually install weconnect 0.54.1 (this will break some dependancies... but home assistant still works).

pip install weconnect==0.54.1

You can manually install weconnect 0.54.1 (this will break some dependancies... but home assistant still works).

pip install weconnect==0.54.1
Did try a manual update of the weconnect package which allowed me to sign in again, and I did get some sensors. But Soc was still not available. Seems we need to wait for an update to HA Core.

Just like I wrote above... this integration needs the newer version (0.54.1) of weconnect python library. But weconnect v0.54.1 depends on requests v2.28.2. Home assistant core depends on requests v2.28.1. Due to these dependancy conflicts it's impossible to resolve this properly right now. Home assistant core will need to bump the requests package dependancy.
You can manually install weconnect 0.54.1 (this will break some dependancies... but home assistant still works).

pip install weconnect==0.54.1

This is really helpfull. Thanks!
Maby we can PR the the version bump in HA.

Opened an PR:
https://github.com/home-assistant/core/pull/88956

After installing the weconnect integration version 0.1.4, the integration didn´t work yet. Then I connected to the homeassistant container, did a pip install weconnect==0.54.1 and did another restart of homeassistant. The integration now partly works with a great number of sensors not working (see attached file).
has_logs_vw_weconnnectid.log

Fixed in https://github.com/mitch-dc/volkswagen_we_connect_id/releases/tag/v0.1.4 but you also need to
update Homeassistant to at least version 2023.3.0.

I installed 2023.3.0, but it still has requests==2.28.1. Seems like the PR wasn't included in the 2023.3.0 release.

@jonasbkarlsson It will probably be included in the april release of HA core.
https://github.com/home-assistant/core/pull/88956#issuecomment-1451347982

My hope it is included in the .1 release

https://github.com/home-assistant/core/pull/88956#issuecomment-1451662705
Let's wait for .1

My hope it is included in the .1 release

It's here 😀

Yes! Thank you very much @mitch-dc !!!!

Updated to 2023.3.1. Itegration still not working. Removed and added the integration.
Some sensors have appeared but not all. There are still errors.
Unexpected exception importing platform custom_components.volkswagen_we_connect_id.sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 779, in get_platform
cache[full_name] = self._import_platform(platform_name)
File "/usr/src/homeassistant/homeassistant/loader.py", line 796, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
File "/usr/local/lib/python3.10/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/config/custom_components/volkswagen_we_connect_id/sensor.py", line 100, in
device_class=SensorDeviceClass.energy,
File "/usr/local/lib/python3.10/enum.py", line 437, in getattr
raise AttributeError(name) from None
AttributeError: energy

authors: ChristianH21220 ColinRobbins DarkFox PalmSwe SwannyBFC angelitoo776 danros3 davidwalton0 gapthorpe geukster gmwubs jonas-wilnerzon jonasbkarlsson jthun koburg melstig mikrohard mitch-dc mtjoernelund mvanninhuys northalpha pansinmanteca petosak sebking1971 sm4upt torge70 viddi87


text:
add associate elastic ip support

associate elastic ip support
after is not release elastic ip
before
aws.elastic_ip = true

after
aws.elastic_ip = '54.199.192.70'

I'm interested in this as well. It is important to me to keep my Elastic IP upon termination and I'm glad I looked through the code before terminating an instance the first time I used this plugin.
I had Amazon associate a PTR record with my EIP and scrub it from mail blacklists, so I'm perfectly happy paying a couple dollars a month to maintain it when I'm not using it.
The code, as currently written, will release the EIP from my account, so I'm glad someone has submitted a PR with the option to allow it to be retained, though it looks like this PR code may give the flexibility to release it, it doesn't presently provide the ability to ask for that.

authors: LTangaF suthio

repo: mitchellh/vagrant-aws

license: mit


text:
The game can disconnect if run in background isn't enabled

Maybe force run in background when connected to a server?

Confirmed this will happen 100% of the time. Network connection times out.


text:
Use emoji and characters as markers. Add ~marker filter syntax.

Description
Allows a per-flow custom marker to be set. Can either be a character or emoji (:bomb:, for example). Adds a filter syntax for searching flows with a specific marker (~marker). The flow.mark command has an optional third argument, marker, which can be used to set the marker on a flow.
None of this changes the standard mark behavior. Only the presentation of the marker in the view is different. The filter ~marked will match all marked flows regardless of what marker is being used. The filter ~marked :grapes: will only match the flows that are marked and have the marker attribute set to 🍇.
I had to bump the file format and add an importer for that.
Checklist

[x] I have updated tests where applicable.
[x] I have added an entry to the CHANGELOG.

Thanks for the really interesting PR! I was very skeptical initially, but I think overall is a smart addition. 😃 I'm not so sure about 🍇, but being able to distinguish between 🟠,🟣,⚫,🔵,🟤,🟡,🟢, and ⚪ seems rather useful!
1. I'm hitting a traceback when opening mitmproxy ->  `n` -> `enter` -> `m`.

Traceback (most recent call last):
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/master.py", line 54, in run_loop
    loop()
  File "/home/user/venv/lib/python3.9/site-packages/urwid/main_loop.py", line 287, in run
    self._run()
  File "/home/user/venv/lib/python3.9/site-packages/urwid/main_loop.py", line 385, in _run
    self.event_loop.run()
  File "/home/user/venv/lib/python3.9/site-packages/urwid/main_loop.py", line 1494, in run
    reraise(*exc_info)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/compat.py", line 58, in reraise
    raise value
  File "/usr/lib/python3.9/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/main_loop.py", line 1454, in faux_idle_callback
    callback()
  File "/home/user/venv/lib/python3.9/site-packages/urwid/main_loop.py", line 574, in entering_idle
    self.draw_screen()
  File "/home/user/venv/lib/python3.9/site-packages/urwid/main_loop.py", line 588, in draw_screen
    canvas = self._topmost_widget.render(self.screen_size, focus=True)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/widget.py", line 145, in cached_render
    canv = fn(self, size, focus=focus)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/container.py", line 1089, in render
    body = self.body.render((maxcol, maxrow-ftrim-htrim),
  File "/home/user/venv/lib/python3.9/site-packages/urwid/widget.py", line 145, in cached_render
    canv = fn(self, size, focus=focus)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/decoration.py", line 226, in render
    canv = self._original_widget.render(size, focus=focus)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/widget.py", line 145, in cached_render
    canv = fn(self, size, focus=focus)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/container.py", line 1089, in render
    body = self.body.render((maxcol, maxrow-ftrim-htrim),
  File "/home/user/venv/lib/python3.9/site-packages/urwid/widget.py", line 145, in cached_render
    canv = fn(self, size, focus=focus)
  File "/home/user/venv/lib/python3.9/site-packages/urwid/listbox.py", line 439, in render
    middle, top, bottom = self.calculate_visible(
  File "/home/user/venv/lib/python3.9/site-packages/urwid/listbox.py", line 325, in calculate_visible
    focus_widget, focus_pos = self._body.get_focus()
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/flowlist.py", line 61, in get_focus
    f = FlowItem(self.master, self.master.view.focus.flow)
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/flowlist.py", line 12, in __init__
    w = self.get_text()
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/flowlist.py", line 23, in get_text
    return common.format_flow(
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/common.py", line 735, in format_flow
    return render_func(
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/common.py", line 596, in format_http_flow_table
    items.append(format_right_indicators(
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/common.py", line 389, in format_right_indicators
    indicators.append(("mark", render_marker(marker)[0]))
  File "/mnt/c/Users/user/git/mitmproxy/mitmproxy/tools/console/common.py", line 168, in render_marker
    if len(marker) == 1:
TypeError: object of type 'NoneType' has no len()

I would be very inclined to not introduce additional dependencies here. Can you maybe add mitmproxy/utils/emoji.py with something like this?

#!/usr/bin/env python3

# auto-generated. run this file to refresh.

emoji = {
# execute once to fill
}

if __name__ == "__main__":
    import requests
    import io
    import re
    from pathlib import Path

    out = io.StringIO()

    r = requests.get("https://api.github.com/emojis")
    for name, url in r.json().items():
        codepoints = url.rpartition("/")[2].partition(".png")[0].split("-")
        try:
            emoji = "\u200d".join(chr(int(c, 16)) for c in codepoints)
        except ValueError:
            continue  # some GitHub-specific emojis, e.g. "atom".
        print(f'    "{name}": "{emoji}",', file=out)

    Path(__file__).write_text(
        re.sub(r"(?<={\n)[\s\S]*(?=}\n)", lambda x: out.getvalue(), Path(__file__).read_text("utf8"), 1),
        "utf8"
    )

Is there a specific reason why you kept marker and marked separate? I'd say we just make marked a str, where a truthy value denotes that it is marked and an empty string denotes that it is not marked. That way they can never go out of sync. :)

The new update:

Removes the dependency on emoji in favor of your proposal.
Keeps the :emoji_name: syntax to avoid confusion between the letter X and the emoji named X.
Drops the Flow().marker attribute as suggested. Flow().marked may either be a bool or str.
The marked attribute will render as either an emoji, single char (A-Za-z0-9) or the default "red ball" otherwise.

Excellent! This looks good to merge except for two minor issues:

We have a pretty cruel CI that mandates test files for all modules: https://github.com/mitmproxy/mitmproxy/pull/4587/checks?check_run_id=2657505739#step:5:10
Can you please just add an empty test_emoji.py? We only define a constant table, nothing to test here.
Is looks like dumpfile-13.mitm is a leftover that can be removed?

Both fixed.

Exellent, thanks!
It looks like our individual_coverage script fails on modules that have no test code. I know I said the opposite thing before, but could you add a simple dict lookup test maybe? 😅

🚀

authors: mhils rbdixon

repo: mitmproxy/mitmproxy


text:
Fix tests after 2037

Background:
As part of my work on reproducible builds for openSUSE, I check that software still gives identical build results in the future.
The usual offset is +16 years, because that is how long I expect some software will be used in some places.
This showed up failing tests in our package build.
See https://reproducible-builds.org/ for why this matters.
Description
Make tests pass after 2037
Checklist

[X] I have updated tests where applicable.
[ ] I have added an entry to the CHANGELOG.

Awesome, thanks! 🍰

Wanted to mention this in the commit message, but forgot to push it:
Year 2133 was chosen because it has the same weekdays as 2037
and it is far enough in the future.

and I tested that it also passes on i586 atm with python3.10

authors: bmwiedemann mhils


text:
Remove RES from departments.json

What are the relevant tickets?
Part of https://github.com/mitodl/hq/issues/4855
Description (What does it do?)
This PR removes RES from departments.json
How can this be tested?

Additional Context

This PR will be on hold until data migration is done for existing courses to remove RES department first.

This PR is still on hold until the relevant changes are made in open and the courses are updated there after the changes.

repo: mitodl/ocw-hugo-themes


text:
🛑 Main OS is down

In 3c5129d, Main OS (https://os.mitramanikhanal.com.np) was down:

HTTP code: 530
Response time: 111 ms

Resolved: Main OS is back up in 52a7468 after 19 hours, 34 minutes.

repo: mitraNBIT/Uptime


text:
fix for missing Language header (with tests)

I only adjusted the tests based on @dmbasso's branch. Let me know if anything else is needed to get this merged.

hey, the tests and the feature belong together, right? So could you squash the commits? (You can use git rebase --interactive)
Also this needs a rebase which will probably fix the tests.
Can you adjust your commit message so it explains the change and not the fix? Is there any issue? Please place a reference to the issue describing the problem. If there is no issue, could you create one?
More information about a good commit message style is available on http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits/#how-to-write-good-commit-messages .

Hi, thanks for the hint :)
I have squashed the commits and rebased to current master. Please let me know if anything is still missing, thanks!

Current coverage is 82.93%

Merging #110 into master will increase coverage by +0.01% as of 7ca606b

@@            master    #110   diff @@
======================================
  Files           21      21
  Stmts         3770    3772     +2
  Branches         0       0
  Methods          0       0
======================================
+ Hit           3126    3128     +2
  Partial          0       0
  Missed         644     644

Review entire Coverage Diff

Powered by Codecov

lgtm, @erickwilder can you take a short look and merge?

Looks good

authors: codecov-io erickwilder felixonmars sils1297

repo: mitsuhiko/babel


text:
Inexact comparison of floating-point numbers in insta tests

I often have a need for comparing snapshots with some tolerance. I write a lot of numerical code, and although for some code I want to really be exact, for many things I need to be able to provide some amount of "tolerance", typically in the form of a relative or absolute error.
There are several reasons for this:

I don't want tests to break just because I reordered a floating-point summation.
The result is non-deterministic due to e.g. parallel computation.
In some cases I simply want to give quite a bit of leeway for the result and I'm more interested in the "trend" of the result (e.g. by plotting graphs and verifying that they satisfy expectations). Here I might for example only want insta tests to fail if some numbers deviate by more than 1% compared to the current reference value (which has been manually confirmed).

I'm not sure exactly how such functionality could look like in insta, as I'm not familiar with the inner workings, but I assume it would not work with Debug output, and could only work with serialized snapshots comparisons, such as JSON. This also raises a question of granularity: What if you have different "tolerances" for different numbers?
My motivation for writing this is to see if there is more interest in having this kind of functionality in insta. If so, we can flesh it out more together.

Insta fundamentally compares strings. There is not much that exists today to make the actual comparison "fuzzy". This is something that probably would need to be addressed differently than today but I'm not sure what a good approach would be yet. In some ways if the snapshot is parsable then it could be loaded in structural form and specific fuzziness could be applied. Eg: instead of comparing Debug to Debug it would compare a parsed version of Debug with another parsed version of Debug etc.

Huh, using redactions for this is a clever idea, @mitsuhiko! Looking forward to trying it out :-)

authors: Andlon mitsuhiko

repo: mitsuhiko/insta


text:
relax strict test for bytes type

This is a possible solution to #683. I'm not sure if you prefer to keep an assert here or if it conforms with your general coding style.

at first glance the errors on Travis are unrelated

I also learn towards keeping it an assertion.

OK, changed and rebased.

Alright, this is now the minimal change. I've removed the comment and rather extended the commit message.

Thanks!

authors: Midnighter RonnyPfannschmidt untitaker

repo: mitsuhiko/werkzeug


text:
Custom methods for clicks on context menu items

Is it somehow possible to pass custom functions to the context menu that should be executed when a certain item was clicked?
e.g. i need to execute a custom function when the user clicks on a spell suggestion. Can i somehow make your context menu execute my custom function on click?
Thanks a lot :)

The library doesn't yet let spelling suggestions' handler be customized, would accept a PR for that!
For other items, you can override the main template.

Hi,
thanks a lot for your reply.
Unfortunately i noticed i need to change some other things as well. So i guess it's easier to create a menu myself (i will orientate myself on your work if thats okay) for my specific use case.
Thansk a lot anyway.

authors: ehhc wearhere

repo: mixmaxhq/electron-editor-context-menu


text:
install without jbuilder

Assuming I have no access to internet.
How to install yojson ?
It desperately is looking for easy_format and biniou despite the fact I installed them manually.
Best regards

There must be something wrong with your ocamlfind config. Check ocamlfind query easy-format and ocamlfind query biniou. Note that it's easy-format, not easy_format.

Ok, so I tried, and it returns that it does not find the libraries.
But I have manually copy them into the ocaml path which allow me to use the installed library.
That may show that ocamlfind is maintaining an "external DB" (through a file) instead of scanning the ocaml path :/

So actually, I used all latest versions of easy-format, biniou and yojson that don't use jbuilder.
For easy-format, compilation and installation were OK.
For biniou I faced a type error on bi_util.ml(i) and other files depending on it.
So at least I took all source files of latest to compile them into v1.0.13.
Compilation and installation seems to be fine but tests failed.
ocamlfind is complaining that nix.cmxa is not a compilation unit description :/

You need to use versions of packages that are compatible with each other,
and this is managed by opam with the official package definitions in
opam-repository. This is the only install method that I can afford to
support.
On Dec 19, 2017 2:25 AM, "Yoann Couillec" notifications@github.com wrote:

So actually, I used all latest versions of easy-format, biniou and yojson
that don't use jbuilder.
For easy-format, compilation and installation were OK.
For biniou I faced a type error on bi_util.ml(i) and other files
depending on it.
So at least I took all source files of latest to compile them into v1.0.13.
Compilation and installation seems to be fine but tests failed.
ocamlfind is complaining that nix.cmxa is not a compilation unit
description :/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mjambon/yojson/issues/49#issuecomment-352701684, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAU84TSdGCFrTZRje4E4n2fh_LlFiOzdks5tB473gaJpZM4RGoFZ
.

I understand :)
That was such a challenge to compile in the way I tried.
At least, I managed to move my project on a machine with internet
(halleluyah)
We can close the issue.
Thanks for everything Martin !
2017-12-19 19:21 GMT+01:00 Martin Jambon notifications@github.com:

You need to use versions of packages that are compatible with each other,
and this is managed by opam with the official package definitions in
opam-repository. This is the only install method that I can afford to
support.
On Dec 19, 2017 2:25 AM, "Yoann Couillec" notifications@github.com
wrote:

So actually, I used all latest versions of easy-format, biniou and yojson
that don't use jbuilder.
For easy-format, compilation and installation were OK.
For biniou I faced a type error on bi_util.ml(i) and other files
depending on it.
So at least I took all source files of latest to compile them into
v1.0.13.
Compilation and installation seems to be fine but tests failed.
ocamlfind is complaining that nix.cmxa is not a compilation unit
description :/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mjambon/yojson/issues/49#issuecomment-352701684,
or mute
the thread
<https://github.com/notifications/unsubscribe-
auth/AAU84TSdGCFrTZRje4E4n2fh_LlFiOzdks5tB473gaJpZM4RGoFZ>

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mjambon/yojson/issues/49#issuecomment-352843465, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABKxlC9IU3r--CO4VdPAKj2voW4pNQbKks5tB_6rgaJpZM4RGoFZ
.

--
Yoann Couillec

authors: mjambon yoanncouillec

repo: mjambon/yojson


text:
Error when downloading from servers that doesn't support Head requests

I'm getting errors when trying to download some CSV files, even though there are no problems in the URL (tested via browser).
Is this something I'm doing wrong? Any tips on how to debug this?
Code:
from pypdl import Downloader
dl = Downloader()
dl.start('https://www.gov.br/anp/pt-br/centrais-de-conteudo/dados-abertos/arquivos/shpc/dsas/ca/ca-2004-01.csv')

Result:
ERROR:root:(ConnectionError) [Server Returned: Forbidden(403), Invalid URL]

this is caused because the head request failed. pypdl first sends a head request to get metadata and to ensure the file exist or not apparently the link you provided doesn't implement the head request support or they don't allow it this cause the head request that pypdl send to fail giving you the error.
its quite an easy fix we just need to add code to send a get request if head request fails this will fix the issue also improving the error message to display something like Header Returned: 404 would be better rather than the current message. thanks for reporting this was a bug that i didn't anticipate.

I can confirm that it's working smoothly now with v1.3.2. Thanks a lot!

@mjishnu - this seems to have regressed with the move to AIOHttp. The first HEAD request raises an exception because the default raise_for_status=True kwarg, so the second request is never hit.

authors: deepdelirious mjishnu schoulten

repo: mjishnu/pypdl


text:
Is "full-width" never "full"?

Trying to create a base layout with MJML with full-width background, but I don't seem to completely understand it, or I am missing as I always get a padding what ever I do.
I would understand this code should produce a full width sections with no padding.
<mjml>
  <mj-body width="3000px" padding-left="0">
    <mj-section full-width="full-width" background-color="green" padding-left="0">
      <!-- Your columns go here -->
    </mj-section>

    <mj-section full-width="full-width" background-color="red" padding-left="0">
      <!-- Your columns go here -->
    </mj-section>
  </mj-body>
</mjml>

I've tried with a mj-wrapper but it all gets messed up when nesting sections.
Have not found any working samples of a proper "stripped" full width.
MJML environment (please complete the following information):

MJML Version 4.2.9
MJML App and jetBrains plugin

Try putting padding="0px". I believe the px is required.

As your body is 3000px wide you won’t see the difference in the preview between a full width and non full width section. Try open the HTML in the browser instead as the preview iframe has some limitations.

On 12 Sep 2022, at 21:32, luison @.***> wrote:

Trying to create a base layout with MJML with full-width background, but I don't seem to completely understand it, or I am missing as I always get a padding what ever I do.
I would understand this code should produce a full width sections with no padding.

<mj-section full-width="full-width" background-color="red" padding-left="0">
  <!-- Your columns go here -->
</mj-section>

I've tried with a mj-wrapper but it all gets messed up when nesting sections.
Have not found any working samples of a proper "stripped" full width.
MJML environment (please complete the following information):
MJML Version 4.2.9
MJML App and jetBrains plugin
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Thanks for prompt reply.
This was a preview of more complex layout. I am testing it as the block content gets previewed/edited before it is sent, in case that is what is causing the additional padding.
The code is based on this other issue that I've created separately in order not to mix subjects as that issue is image size related. https://github.com/mjmlio/mjml/issues/2545

Part of the padding problem was truly caused by my editor prior to email delivery and viewed on the browser seems ok, sorry, but I am still stuck in how does one suppose to correctly produce a full-width email (including text body)  considering that  "full-width" is only compatible with mj-section and mj-wrapper and mj-body can't be set to 100%.
How am I supposed to set a fluid full-width email, including the body?

Full width only affect the background not the content. Mjml only support for centered mail with a max width on the body

On 13 Sep 2022, at 20:01, luison @.***> wrote:

Part of the padding problem was truly caused by my editor prior to email delivery and viewed on the browser seems ok, sorry, but I am still stuck in how does one suppose to correctly produce a full-width email (including text body) considering that "full-width" is only compatible with mj-section and mj-wrapper and mj-body can't be set to 100%.
How am I supposed to set a fluid full-width email, including the body?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Full width only affect the background not the content. Mjml only support for centered mail with a max width on the body

I want to think that it is due to some complicated limitation of making it responsive on that situation and not only for not being the "usual".
In my case, I was hoping for a fluid layout with a full-width bottom section on a different colour and a responsive signature/footer with two columns.  Have not found any hacks so far.

authors: DanaIntouch iRyusa luison

repo: mjmlio/mjml


text:
EmptyMJMLError: No root "" or "" found in the file

Hello,
I'm getting this error in transpiling a file thus done:
<mjml>
    <mj-text>Hello</mj-text>
</mjml>

Same replacing <mjml> with <mj-body>.
This is the command I'm using: mjml -r index.mjml. The package version is 3.0.0.

Hi Manaus,
You don't have enough elements.
You need: mjml->mj-body->mj-container->mj-section->mj-column->mj-text
rather than just
mjml->mj-text
Kind Regards,
:----------------------------:
Dale McConnell
On 20 October 2016 at 12:43, Manaus notifications@github.com wrote:

Hello,
I'm getting this error in transpiling a file thus done:

Hello

Same replacing  with .
This is the command I'm using: mjml -r index.mjml. The package version is
3.0.0.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mjmlio/mjml/issues/417, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKxLOM4R--eSVL1FZQwnPgvY705fS-KDks5q11PzgaJpZM4KcB0g
.

Indeed @Manaus, as @dalefish said your markup is missing elements. I'm closing as his answer should fix your issue, feel free to reopen if not the case!

authors: Manaus dalefish ngarnier


text:
Connection aborted, error(10054)

Hello,
I am running version 15. 01. 10 installed again in a clean way.
While I do not consider this issue as a program error, I put it here if you could provide any solution from the PYTHON-ONEDRIVE program and in case it were interesting to other users who may experience this problem due to growing their data volume.
Onedrive stage is as follows: Three folders in OneDrive
A folder named DAT with more than 30. 000 files
A folder named INF with 487 files in 10 subfolders
A folder named CONTROL with 31 files in 8 subfolders
Any access to get files from INF or CONTROL folders works correctly (Only get operations)
On the other hand, any attempt to access and download files from DAT folder just end abnormally with message
onedrive.api_v5.ProtocolError: (None, "('Connection aborted.', error(10054, 'Se ha forzado la interrupci\xf3n de una conexi\xf3n existente por el host remoto'))")
I created for testing a new folder named PRU and copied in it two files from DAT. Access to these files under PRU is correct.
I can deduce that the problem may be due to an excessive amount of files in folder DAT.
Despite my adsolutely no knowledge of programming in Python language, it can be seen in the source that  tries a LISTDIR function that may not be completed.
Is there any solution possible from python-onedrive or on the contrary it is neccesary to modify the DAT folder structure?
Thank you and best regards
Below is the complete sequence of trace
C:\Users\ANGEL>onedrive-cli -c "B:/OD.lcrc" get "ALQS2D/DAT/filetest.txt"  "%USERPROFILE%/AppData/Temp/filetest.txt"
Traceback (most recent call last):
File "C:\Python27\Scripts\onedrive-cli-script.py", line 9, in
load_entry_point('python-onedrive==15.01.10', 'console_scripts', 'onedrive-cli')()
File "C:\Python27\lib\site-packages\onedrive\cli_tool.py", line 332, in main
contents = api.get(resolve_path(optz.file), byte_range=optz.byte_range)
File "C:\Python27\lib\site-packages\onedrive\cli_tool.py", line 255, in
resolve_path = ( (lambda s: id_match(s) or api.resolve_path(s))
File "C:\Python27\lib\site-packages\onedrive\api_v5.py", line 592, in resolve_path
root_id = dict(it.imap(op.itemgetter('name', 'id'), self.listdir(root_id)))[name]
File "C:\Python27\lib\site-packages\onedrive\api_v5.py", line 608, in listdir
.listdir(folder_id=folder_id, limit=limit, offset=offset)['data']
File "C:\Python27\lib\site-packages\onedrive\api_v5.py", line 358, in listdir
return self(ujoin(folder_id, 'files'), dict(limit=limit, offset=offset))
File "C:\Python27\lib\site-packages\onedrive\api_v5.py", line 333, in call
try: return self.request(api_url(), **kwz)
File "C:\Python27\lib\site-packages\onedrive\api_v5.py", line 164, in request
raise raise_for.get(code, ProtocolError)(code, message)
onedrive.api_v5.ProtocolError: (None, "('Connection aborted.', error(10054, 'Se ha forzado la interrupci\xf3n de una conexi\xf3n existente por el host remoto'))")

If you're doing "ls" on that dir (with a cli tool), have you checked the --help on that command?
% ./onedrive-cli ls -h
usage: onedrive-cli ls [-h] [-r {[offset]-[limit] | limit}] [-o] [folder]

positional arguments:
  folder                Folder to list contents of (default: me/skydrive).

optional arguments:
  -h, --help            show this help message and exit
  -r {[offset]-[limit] | limit}, --range {[offset]-[limit] | limit}
                        List only specified range of objects inside. Can be
                        either dash-separated "offset-limit" tuple (any of
                        these can be omitted) or a single "limit" number.
  -o, --objects         Dump full objects, not just name and id.

I suspect that you might want to pass some limits via "-r" option, so that onedrive connection won't time-out while it's generating multi-megabyte json of folder contents.
Don't think I'm aware of any other issues with huge number of files in dirs, except for that listing thing.
One usual solution to avoid worrying about such limits is to use some hash function to put files into "bucket dirs", but it's usually only applicable to files/dirs created and used in some automatic fashion (e.g. caches, databases, backups, fs backends, etc), i.e. not really human-usable.

Just noticed that you're actually doing the "get" of the file by its path, not "ls", so my suggestion above is probably not applicable, sorry.
OneDrive didn't really support "get dir/a/b/somefile.txt" last time I checked, only supporting getting the file by its "id" string (regardless of where it's actually located), so in order to do human-friendly "get dir/a/b/somefile.txt" operation, command-line tool does "resolve_path" first, which involves getting a list of OneDrive root files/dirs by it's well-known id ("me/skydrive"), finding id for "dir" there, getting list for that id, etc.
That "resolve_path" thing doesn't account for dirs like the one you have there, always trying to grab the full list.
Guess some checks and workaround for this particular case can be implemented there, but it'd still be quite slow and inefficient (making tons of extra requests), so if you have that DAT dir filled by some script or tool, maybe keeping a local index of file id's and requesting files by that (if necessary) would be still a better idea.
Workaround for current version might be requesting dir listing in chunks (as proposed above) with the "-o" option, finding necessary file there and requesting it by id - "get" should support it.
Guess best way to do that with this module would be from a python script, also caching "name-id" pairs locally, or failing that - process the cli tool output in some automated fashion (which is probably a bad idea, as it's not really designed for this).

Added per-500-items pagination to "resolve_path" method mentioned above in 64d35ba and 15.02.01, maybe try it and see if it will work for you?
500-item requests take a few (4-8) seconds, which shouldn't be enough to trigger any kind of timeout, even with OneDrive being under stress, I think.
Largest listing I've tried was 5k items, and it completed in ~70s, which seem to be pushing API limits a bit too far.
As for efficiency, as you can probably guess/calculate, to retreive file by its name among ~30k items, about 30 http requests will be required on average, 2-4 in the best case, ~60 - worst.
Requests for more items stop when the right name is found, hence the best/worst disparity.
Again, if you plan to routinely deal with such large directories, it might be a good idea to keep a local cache of file name-id pairs, to avoid this lenghty process entirely and be able to request correct file right away (in a single get request).

Hello
Thank you very much for the suggestion. I will do the test con nuew versión as soon as possible.
I did a test yesterday with 15.01.10 version in a 500 elements folder and works O.K.
We are considering two options: ONE is perform a division of a such a huge directory in smaller folders ( the more suitable for us will be folders with about 1,5 to 2k elementes), and TWO work with file identifications. We have obtained the identification of a file but appears in two formats, such as <<id: 6ea482f9ed79f808>> and the two lines after <<id: file.6ea482f9ed79f808.6EA482F9ED79F808!74594>>
A problem we encounter is that we don't have clear which is onedrive-cli parameter format in the GET statement to facilitate a ID rather than a FILE NAME to get the file. We tried with help and we dikd some test by with NO success.
C:\Users\ANGEL>onedrive-cli get -h
usage: onedrive-cli-script.py get [-h] [-b BYTE_RANGE] file [file_dst]
positional arguments:
file                  File (object) to read.
file_dst              Name/path to save file (object) as.
optional arguments:
-h, --help            show this help message and exit
-b BYTE_RANGE, --byte-range BYTE_RANGE
Specific range of bytes to read from a file (default:
read all). Should be specified in rfc2616 Range HTTP
header format. Examples: 0-499 (start - 499), -500
(end-500 to end).
I will inform ypou about the test with new version as soon as possible. Could yo tell me te format of GET with identificactions?
Thanks you a lot for your time and your suggestions. Regards

Github seem to have snipped things with <<...>> around them from your last message, but I've got them from the notification mail:

We have obtained the identification of a file but appears in two formats, such as <<id: 6ea482f9ed79f808>> and the two lines after <<id: file.6ea482f9ed79f808.6EA482F9ED79F808!74594>>

"File id" that I've been talking about is the latter one - file.6ea482f9ed79f808.6EA482F9ED79F808!74594, as presented by e.g. "info" command, or "ls -o".
The other id (6ea482f9ed79f808) is a user id, and you probably saw that one under "from:", e.g.:
...
created_time: 2014-11-16T12:49:01+0000
description:
from:
  id: 6ea482f9ed79f808
  name: ...
id: file.6ea482f9ed79f808.6EA482F9ED79F808!74594
is_embeddable: True
link: ...
name: 中文.txt
parent_id: folder.6ea482f9ed79f808.6EA482F9ED79F808!123
shared_with:
  access: Just me
size: 53
...

It's a actually a tree structure, similar to YAML format or python syntax, so all the indented stuff under "from:" is an info about the user who uploaded the file - user id and name.
You can also see this user id included in the id for every file that you've ever uploaded from that account, it certainly isn't enough to identify any particular file.
Getting the file by id like file.6ea482f9ed79f808.6EA482F9ED79F808!74594 with commandline tool's "get" should work - just use e.g. onedrive-cli get 'file.6ea482f9ed79f808.6EA482F9ED79F808!74594' (no idea if quotes are needed on windows, or if they maybe should be double-quotes), and seem to work fine here.
If it throws some error for you, looking at the traceback might help, maybe cmd.exe is mangling passed id somehow (might happen if "!" is something special there, and/or no quoting was used).

Hello and thank you very much again. I have installed and tested the new version 15. 02. 1
Tests conducted with the new version and their results are as follows:

download file using its identification is fast and it is independent of the folder structure and number of files inside it, such as you said. I didn´know before te existence of fileid's.
(2) Using the detailed full path for a file who is inside a folder with 0.5K files has a SUCCESFUL DOWNLOAD in about 15 seconds.
(3) Using the detailed full path for a file who is inside a folder with 2K files has a SUCCESFUL DOWNLOAD in about 25  seconds.

This download times are acceptable for us, but we will considered using of fileid's.
We have seen that obtaining fileid's  is possible with from ONEDIVE-CLI a folder level with
            Onedrive-cli ls –o [foldername]

Is it possible to obtain fileids individually using onedrive-cli?
We have tested    ls -o    using a filename instead a foldername and, logically, it said «not found»
The next step for us is to generate an executable bundled with onedrive-cli and all its accessories modules so that it can be excutable in a Windows environment that does not have Python, PyYAM, Py32win and so on, so that it can be called from user routines. Do you know if Is it possible?
Thank you very much for the information, the new release and your time and effort. Sincerely grateful.

download file using its identification is fast and it is independent of the folder structure and number of files inside it, such as you said. I didn´know before te existence of fileid's.

There are also --path (-p) and --id (-i) top-level options, which can be used with any command which takes file/folder argument to force treating it as a path or id (depending on option).
You might want to use -i, if you're always using ids for some ops, so that argument won't ever be mistaken for a filename.

Is it possible to obtain fileids individually using onedrive-cli?

It is now.
Just added (0a14444 or 15.02.2) a global --object-key option, which can be used with any command that outputs objects (e.g. "ls -o"), for example onedrive-cli -k id ls -o should print a bunch of id's only, or onedrive-cli -k id info somefile.txt would only print id of a specified file.
More complex usage of this option can involve passing it a python format string, e.g. {0[name]!r} {0[count]:04d} (name repr + 0-prefixed 4-digit display of count), but you probably don't need that.
See onedrive-cli -h output for more details.
As also mentioned above, onedrive-cli is intended to be used by humans, and it might be way easier, efficient and more reliable to script whatever automated behavior you want in python, rather than parsing cli tool's output.

We have tested ls -o using a filename instead a foldername and, logically, it said «not found»

Yeah, naturally, "ls" only works with folder names or id's, if you want to see metadata of one specified file/folder, use "info" command, e.g. onedrive-cli info somedir/somefile.txt.

The next step for us is to generate an executable bundled with onedrive-cli and all its accessories modules so that it can be excutable in a Windows environment that does not have ... Do you know if Is it possible?

Yeah, definitely possible, and there should be a few tools to do that.
py2exe is the most well-known one, I think, followed by PyInstaller and cx_Freeze, and there are probably others.
I've never used python on windows directly, and have no experience with any of the above tools, so can't really make any recommendations here.

Hello,
Thanks, again, a lot for all.
I have tested the new version 15.01.2 and using the new features to obtain fileids individually as sohwn next in detail
C:\Users\ANGEL>onedrive-cli -V
python-onedrive 15.02.2
C:\Users\ANGEL>onedrive-cli -c "B:\OD.lcrc" -k id info "ALQS2D/xCONTROL/TAZAS/PRUPY.txt"
file.6ea482f9ed79f808.6EA482F9ED79F808!74586
And thanks again for your indications about programs to generate an executable: py2exe, Pyinstaller y cx_Freeze. That is my next step.
I am sincerely grateful to you

You're welcome, thanks for parsing all those ridiculous walls of rambling text I tend to throw at people.
Someone was bound to stumble upon these limitations or missing features sooner or later, and fixes were fairly trivial anyway, certainly shorter than I might make them sound in the description, so it's no problem.
With regards to the question, I guess you mean the output that you see in the "cmd" window (assuming it is, from a screenshot and windows environment), obviously downloaded files (which might also be considered to be "results") will be put in the current dir or wherever is specified.
onedrive-cli uses standard streams (stdout, stderr) for all output, and application that runs it can redirect/substitute them for onedrive-cli to wherever it wants (pipes, usually) and then read output from there.
Which is a complicated way of saying "that output goes wherever application wants it to".
For instance, in python you can do (using standard subprocess module):
>>> import subprocess
>>> output = subprocess.check_output(
...   ['onedrive-cli', '-c', 'B:\OD.lcrc',
...    '-k', 'id', 'info', 'ALQS2D/xCONTROL/TAZAS/PRUPY.txt' ])
>>> output
'file.6ea482f9ed79f808.6EA482F9ED79F808!74586\n'
>>> file_id = output.strip()
>>> file_id
'file.6ea482f9ed79f808.6EA482F9ED79F808!74586'

(though from python, of course you can do it easier by just importing "onedrive" module)
There, output of a subprocess was captured as a python string, to use later in the script somewhere.
In most unix shells, you'd do same thing via file_id=$(onedrive-cli ...), I think windows batch scripting language allows something like that too (though I wouldn't recommend writing any but the most trivial applications in it).

repo: mk-fg/python-onedrive

license: WTFPL


text:
Ability to silence warnings (e.g. "Param does not appear in signature")

Is your feature request related to a problem? Please describe.
When building mkdocstrings-python with many classes (especially Pydantic classes) I get many warnings for:

Parameter 'param_name' does not appear in the function signature

Describe the solution you'd like
I'd very much like a way to silence this warning in particular. Probably a way to silence other warnings would be nice too.
That way I could run mkdocs in strict mode and check if it fails in CI.
Describe alternatives you've considered
Currently just dealing with it, and getting the console spammed. 😄
One possible other thing is to not create this warning when there are **kwargs, but I think most people would want it there by default anyways.
Additional context
I'd be fine with being able to silence just this warning. I assume Pydantic support isn't something that will be worked on (likely living in an extension to Griffe or something?), which is fine. On a longer-term scale, it would probably be something that would be configured through mkdocstrings.

It's now fixed, you should be able to use warn_unknown_params: false to disable these warnings :slightly_smiling_face:

authors: NowanIlfideme pawamoy

repo: mkdocstrings/griffe

license: ISC


text:
Update monitor.py

fix cpu percentage sign

Hi kjames2001,
The figure shown is the Load Average on the CPU, it is not the CPU usage as a percentage. You'll notice that this number never goes much above 4 (meaning all four cores are fully utilised).

Sorry for not reading the command properly before commiting. Thank you for the amazing scripts!


text:
Is this project dead?

Looking to use it in shiny bling-bling new LitElement application, but last updates are in 2017 and there is no major version yet.
In what status is this project?
UPDATE:
Some article: An Introduction to the Redux-First Routing Model

@mksarge if you'd like, I'm willing to maintain

@mksarge bump on this if you'd like :)

I don't think so. I've been using this for a relatively production-grade project for a while and things seem to work just fine (at least for my use). There don't seem to be many updates that are needed (low issues count) so I'd personally attribute the old modification dates on the code to this repo being "done" moreso than it being out of date

what a coincidence that as soon as I say that I end up running into issues. Since it seems like there arent many great substitures for the simplicity of this repo that aren't also unmaintained, I have started a fork of this repo to hopefully provide a place where fixes can be centralized and published. I cant guarantee i'll maintain it forever or be able to keep publishing in every format that this repo made available, but I'll do at least what is needed for my project and will try and be active for at least the near future in case anyone wants to make contributions

authors: MoralCode oswee tannerlyons

repo: mksarge/redux-first-routing


text:
Example of response generation with optional arguments

The example is meant to extend the README, whereby it:

uses a checkpoint from the mlx-community
loads the model and tokenizer with load
transforms a prompt into a chat template with the apply_chat_template method of the tokenizer
passes optional arguments into generate, which show how to control the generation process

The PR is currently pending due to a CircleCI job on hold. Could someone with the necessary permissions take a look at this?

Sorry for the delay! Tests are running and we can merge this once they are clear. Thanks for the contribution, very nice!

authors: a-wozniakowski awni

repo: ml-explore/mlx-examples


text:
Huggingface mlx-community/Phi-3-mini-128k-instruct-4bit model cannot be loaded

I am interested in running the mlx-community/Phi-3-mini-128k-instruct-4bit model with swift, but it cannot be loaded. Here is the output I am seeing:
➜  mlx-swift-examples git:(main) ./mlx-run llm-tool eval --model mlx-community/Phi-3-mini-128k-instruct-4bit -m 100000 -p "<|system|>
You are a helpful assistant.<|end|>
<|user|>
How to explain Internet for a medieval knight?<|end|>
<|assistant|> "
Error: typeMismatch(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "rope_scaling", intValue: nil), _JSONKey(stringValue: "long_factor", intValue: nil)], debugDescription: "Expected to decode String but found an array instead.", underlyingError: nil)

Note that the mlx-community/Phi-3-mini-4k-instruct-4bit model can be loaded. The 128k model includes a non-null rope_scaling field in the config.json, and the 4k model doesn't have a rope_scaling factor. The model 128k load is failing because it can't process the rope scaling

Interestingly we have support for that model in Python but it looks like it is just ignores the rope scaling.
Somehow it still gives pretty good results for short generations 🤔 but presumably it would not work too well beyond 4k

It looks like the ropeScaling is a dictionary of heterogenous types.  We already have:
    var ropeScaling: [String: StringOrNumber]? = nil

I could change this to StringOrNumberOrArray (something like that anyway!)
The code uses the ropeScaling["type"] to figure out what to do and if it isn't a type it recognizes it will just use a ropeScaling of 1.
Specifically it needs to handle:
        "long_factor": [
            1.0700000524520874,
            1.1200000047683716,

I am not sure what is intended with the array -- I guess they are supposed to correspond to the hidden layer counts?  There are also two sets of values here.
Anyway, we could at least enable it to load if we changed this.

I am not sure what is intended with the array -- I guess they are supposed to correspond to the hidden layer counts? There are also two sets of values here.

Those are values per dimension. So you scale each dimension with a unique value. Python implementation is here.
There are two of them because one gets used for short sequences and the other gets used for positions after a certain step in longer sequences.

I think I've fixed this in https://github.com/ml-explore/mlx-swift-examples/pull/155.

This should be working as of #107

authors: DePasqualeOrg awni davidkoski neilmehta24

repo: ml-explore/mlx-swift-examples


text:
[Question] Chat is not active after Android build.

❓ General Questions

I built the mlcchat app according to the Android app build provided by < https://llm.mlc.ai/docs/deploy/android.html >.
However, when I downloaded the default 'Llama-2-7b-chat-hf-q4f16_1' and entered the chat, the following error occurred and the chat did not run.
My device name is "Galaxy tabs8", and it's a device that uses the Qualcomm Snapdragon 8 Gen 1 SM8450.

MLCChat failed

Stack trace:
org.apache.tvm.Base$TVMError: InternalError: Check failed: static_cast<size_t>(gfunc.num_args) == args.size() (4 vs. 0) : ValueError: Invoking function create_tir_paged_kv_cache requires 4 inputs but only 0 inputs are provided.
Stack trace:
  File "/mnt/f/mlc-llm/3rdparty/tvm/src/runtime/relax_vm/vm.cc", line 652

at org.apache.tvm.Base.checkCall(Base.java:173)
at org.apache.tvm.Function.invoke(Function.java:130)
at ai.mlc.mlcllm.ChatModule.reload(ChatModule.java:43)
at ai.mlc.mlcchat.AppViewModel$ChatState$mainReloadChat$1$2.invoke(AppViewModel.kt:642)
at ai.mlc.mlcchat.AppViewModel$ChatState$mainReloadChat$1$2.invoke(AppViewModel.kt:640)
at ai.mlc.mlcchat.AppViewModel$ChatState.callBackend(AppViewModel.kt:543)
at ai.mlc.mlcchat.AppViewModel$ChatState.mainReloadChat$lambda$3(AppViewModel.kt:640)
at ai.mlc.mlcchat.AppViewModel$ChatState.$r8$lambda$JJKpoRMMpp77FzXKA0o00i8lgRA(Unknown Source:0)
at ai.mlc.mlcchat.AppViewModel$ChatState$$ExternalSyntheticLambda3.run(Unknown Source:8)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:487)
at java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:644)
at java.lang.Thread.run(Thread.java:1012)

Error message:
InternalError: Check failed: static_cast<size_t>(gfunc.num_args) == args.size() (4 vs. 0) : ValueError: Invoking function create_tir_paged_kv_cache requires 4 inputs but only 0 inputs are provided.
Stack trace:
  File "/mnt/f/mlc-llm/3rdparty/tvm/src/runtime/relax_vm/vm.cc", line 652

Same error！！！

Oh! I checked and found that the Llama model experienced this error. I noticed that the same symptom occurred when I built the apk file you provided.
No error occurred when configured with Mistral-7B.
Could you please let me know why this problem is currently happening only in Llama?

Oh! I checked and found that the Llama model experienced this error. I noticed that the same symptom occurred when I built the apk file you provided.
No error occurred when configured with Mistral-7B. Could you please let me know why this problem is currently happening only in Llama?

Same issue. Only happening in Llama. Not in RedPajama3b q4f16, phi2 q4f16 and Mistral7b q4f16

Maybe caused by this? But I have absolutely no idea how this part work right now
https://github.com/mlc-ai/mlc-llm/blob/c30348a10eabff5b27a5f5a40370af9796890a2d/cpp/llm_chat.cc#L625C29-L625C50

Related to #1651

#1798 Should fix this.

@Kartik14
Sorry, but a new error is occurring...
Please check the error below.

MLCChat failed

Stack trace:
org.apache.tvm.Base$TVMError: TVMError: Function vm.builtin.paged_attention_kv_cache_attention_with_fused_qkv(0: relax.vm.PagedAttentionKVCache, 1: int64_t, 2: double, 3: runtime.NDArray, 4: runtime.NDArray) -> void expects 5 arguments, but 4 were provided.
Stack trace:
  File "/mnt/d/mlc-llm/3rdparty/tvm/include/tvm/runtime/packed_func.h", line 1908

at org.apache.tvm.Base.checkCall(Base.java:173)
at org.apache.tvm.Function.invoke(Function.java:130)
at ai.mlc.mlcllm.ChatModule.prefill(ChatModule.java:63)
at ai.mlc.mlcchat.AppViewModel$ChatState$requestGenerate$1$1.invoke(AppViewModel.kt:667)
at ai.mlc.mlcchat.AppViewModel$ChatState$requestGenerate$1$1.invoke(AppViewModel.kt:667)
at ai.mlc.mlcchat.AppViewModel$ChatState.callBackend(AppViewModel.kt:548)
at ai.mlc.mlcchat.AppViewModel$ChatState.requestGenerate$lambda$4(AppViewModel.kt:667)
at ai.mlc.mlcchat.AppViewModel$ChatState.$r8$lambda$lluIrcsPALEW5nCb2tohZYadhTY(Unknown Source:0)
at ai.mlc.mlcchat.AppViewModel$ChatState$$ExternalSyntheticLambda3.run(Unknown Source:6)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:487)
at java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:644)
at java.lang.Thread.run(Thread.java:1012)

Error message:
TVMError: Function vm.builtin.paged_attention_kv_cache_attention_with_fused_qkv(0: relax.vm.PagedAttentionKVCache, 1: int64_t, 2: double, 3: runtime.NDArray, 4: runtime.NDArray) -> void expects 5 arguments, but 4 were provided.
Stack trace:
  File "/mnt/d/mlc-llm/3rdparty/tvm/include/tvm/runtime/packed_func.h", line 1908

Hi @Moon-Ahn, I feel the latest Android app should have addressed this issue. Could you please try out the latest one?
Also cc @Kartik14 for confirmation.

Hello @MasterJH5574 ,
I downloaded the new version of apk and tried it, but an error occurred again.
This was also a problem that only occurred in llama, but I'm just watching the situation because other people keep posting issues on my behalf.
I'm currently facing another problem, so I'll check if the error has been corrected later.
However, I would like to inform you that there was still an error when I checked it three days ago.

@Moon-Ahn Hi, It might be because of an old version of Android model lib. The function signature was recently changed to have 5 arguments. Can you please try downloading the latest model libs (or compile the library again if you are building yourself)

@Kartik14
I understood it as saying 'Llama-2-7b-chat-hf-q4f16_1-MLC' after changing the 3rdParties to the latest version, is that correct?
I installed apk and proceeded with it, so I installed quantized files on the homepage of huggingface provided through the app.
The route where I installed the apk is as follows.
" https://github.com/mlc-ai/binary-mlc-llm-libs/releases/ "

ohh, if you are installing via the apk, you shouldn't need anything else. Are you getting error in the released apk?

@Kartik14
I'm sorry, the apk file I downloaded seems to be old. Currently, llama works fine in the apk file in the link above. Sorry for the confusion.
I have now re-run and downloaded with " git clone --recursive https://github.com/mlc-ai/mlc-llm/ " through the guidance of **https://llm.mlc.ai/docs/deploy/android.htm**l , and then re-quantized the Llama model and ran the app.
However, the error ' "TVMError: Function vm.builtin.paged_attention_kv_cache_attention_with_fused_qkv(0: relax.vm.PagedAttentionKVCache, 1: int64_t, 2: double, 3: runtime.NDArray, 4: runtime.NDArray) -> void expects 5 arguments, but 4 were provided." ' is still output.

As you mentioned, I think 'try downloading the latest model libs (or compile the library again if you are building yourself)' could be the right way, but I don't think I understood exactly what it meant.
Can you tell me more accurately?
Does that mean running git clone --recursive https://github.com/mlc-ai/mlc-llm/ is not enough?

The problem has been resolved.
The environment variable of tvm was set to the previous version.

The problem has been resolved.
The environment variable of tvm was set to the previous version.

could you please share steps how you resolved this issue

repo: mlc-ai/mlc-llm


text:
Not working correctly on 0.9.9.24

I have 2 inverters. One is D-NS (DT) and other is D-ES (ES).
Since few last updates the D-ES (ES) is not updating on its own so I have to reload integration continously to get new values.
D-NS (DT) inverter is working without a problem.

Enable debug logging and provide the logs please.

Same problem with GW10K-ET

I do have the same problem with the GW5048-EM

Probably the same problem as in #283, fix will be there soon.

Fixed in v0.9.9.25

authors: AlienVi22 beilm midadami mletenay

repo: mletenay/home-assistant-goodwe-inverter


text:
"pl": "one", "few", "many", "other"

CLDR 30:
<pluralRule count="one">i = 1 and v = 0 @integer 1</pluralRule>
<pluralRule count="few">v = 0 and i % 10 = 2..4 and i % 100 != 12..14 @integer 2~4, 22~24, 32~34, 42~44, 52~54, 62, 102, 1002, …</pluralRule>
<pluralRule count="many">v = 0 and i != 1 and i % 10 = 0..1 or v = 0 and i % 10 = 5..9 or v = 0 and i % 100 = 12..14 @integer 0, 5~19, 100, 1000, 10000, 100000, 1000000, …</pluralRule>
<pluralRule count="other">   @decimal 0.0~1.5, 10.0, 100.0, 1000.0, 10000.0, 100000.0, 1000000.0, …</pluralRule>

On this page "pl" has only "one", "few" and "other":
http://mlocati.github.io/cldr-to-gettext-plural-rules/

This is the intended behavior: as you can see from the rules, the CLDR case for "other" never occurs for integer numbers.
So, since gettext only works with integers, that case is meaningless in .po files.
For that reason, the "other" CLDR case is removed and in gettext the "many" is renamed to "other".
See also this discussion I've had with Daiki Ueno, the official maintainer of gettext: http://lists.gnu.org/archive/html/bug-gettext/2015-05/msg00007.html

You are right - I have miss-understood the formular.
If "v = 0" you have only 3 rules.
Sorry for disturbing.

No problems at all!
Ciao!

Thanks a lot for this great code!

authors: doublex mlocati

repo: mlocati/cldr-to-gettext-plural-rules


text:
Definitive fix to 0 velocity notes, fix note velocity scaling and better .gitignore

Changes:

fixed the 0 velocity notes issue
fixed the velocity scaling/mapping
made the midi event handler easier to read
added a better .gitignore file

I noticed that the code isn't following a proper code format? Would be ok if i open another PR to fix that? ;)

hi! sorry I should have tested your code myself but a bit busy atm :) thankyou for sticking at it. feel free to clean up formatting (and thanks so much) - if you don't  get around to it, I'll have a stab. FWIW, I also plan to test it all a bit, and then replace the checked in binaries with new ones done as proper github releases. I've let this project rot a bit, sorry.


text:
Tutorial ambigious

Hello.
Pretend that you don't know how to play the game at all.  (I did not a minute ago.)
In the tutorial, on the second screen, drag the three all the way to the right (a line four blocks long).  Now, drag the 4 all the way to the left (a line four blocks long).
Ok, I followed the instructions in the tutorial.  Now it just sits there.. I tap some things..  Nothing happens.
The tutorial does move to the next screen if I make the 3-line 3 blocks long and the 4-line 4 blocks long.  So, I think the length of lines matters!  (Right?)
The tutorial should have a step that shows the user how to make lines the correct length.

Thanks for the feedback @daveloyall !
I worked on a few different iterations of the tutorial, made a few internal testing and the current one was the one that seemed to be the more intuitive... But creating tutorial is way more complex than I expected, I received different feedbacks everytime someone played it.
In this specific case, it seemed that highlighting the lines that weren't valid would have been enough but I guess I was wrong.

I'm thinking about adding a step (before the one above ☝ ) with a 4x2 grid, and a 3 in the corner, explaining that the line should have the same length of the number.
What do you think?

Well, you can just tell the user with text, yes.
But, it may be more fun to just say "only some lines are valid!" and let them figure it out.  The small board you described would be useful for that, yes.
I didn't notice the highlighting until I started playing the game itself.

The eye don't see what the mind don't know
--Somebody

When there is one line, maybe I just think it changes colors sometimes.  When there are two lines and one of them is invalid, maybe I just think different lines have different colors.
Parting thought: now that I know how to play, I won't be useful for tutorial feedback anymore.  I could try and pretend that I don't know the game mechanics, but, that's not as useful as feedback from a true neophyte!

@daveloyall the problem with the tutorials, is that there's not bullet-proof solution.
For example, see this user on Hacker News who had a totally different experience and idea from yours (https://news.ycombinator.com/item?id=21834409).

Parting thought: now that I know how to play, I won't be useful for tutorial feedback anymore. I could try and pretend that I don't know the game mechanics, but, that's not as useful as feedback from a true neophyte!

😬 That's why I had to find new people for testing each tutorial improvements.

authors: daveloyall mmazzarolo

repo: mmazzarolo/ordinary-puzzles-app